[perftests] Add some sanity asserts to the low level perf test suite …

What's in this pull request?

This just consists of ensuring that:

min <= max.
min <= mean.
min <= median.
max >= mean.
max >= median.

Before merging this pull request to apple/swift repository:

[ ] Test pull request on Swift continuous integration.

Triggering Swift CI
The swift-ci is triggered by writing a comment on this PR addressed to the GitHub user @swift-ci. Different tests will run depending on the specific comment that you use. The currently available comments are:
Smoke Testing
    Platform     | Comment
    ------------ | -------------
    All supported platforms     | @swift-ci Please smoke test
    OS X platform               | @swift-ci Please smoke test OS X platform
    Linux platform              | @swift-ci Please smoke test Linux platform

Validation Testing
    Platform     | Comment
    ------------ | -------------
    All supported platforms     | @swift-ci Please test
    OS X platform               | @swift-ci Please test OS X platform
    Linux platform              | @swift-ci Please test Linux platform

Note: Only members of the Apple organization can trigger swift-ci.

@swift-ci Please smoke test OS X


Properly remove json paths

current implementation WILL NOT actually remove these paths - it just changes the obj variable reference (which is not same as changing actual reference of original window object)

Good stuff


New implementation of tf.clip_by_value changes behavior of the op.

FYI, @jart and @nfelt
System information

Have I written custom code (as opposed to using a stock example script provided in TensorFlow): yes
OS Platform and Distribution (e.g., Linux Ubuntu 16.04): Debian GNU/Linux (upgraded from: Ubuntu 14.04.5 LTS)
TensorFlow installed from (source or binary): binary (pip package)
TensorFlow version (use command below): Nightly 1.8.0 build 254 (tf-nightly)
Python version: Python 2.7.13
Exact command to reproduce within TensorBoard's repository:
bazel test //tensorboard/plugins/pr_curve:pr_curves_plugin_test

Describe the problem
Every night, TensorBoard's python tests on travis installs the latest nightly version of TensorFlow. One day, I find that TensorBoard's :pr_curves_plugin_test is failing for python 2.
https://github.com/tensorflow/tensorboard/blob/master/tensorboard/plugins/pr_curve/pr_curves_plugin_test.py
Interestingly, the test still passes for python 3.
I confirmed that the test succeeds if I install tf-nightly build 253 and then bazel test the target within TensorBoard's repo.
pip install -I https://ci.tensorflow.org/view/tf-nightly/job/tf-nightly-linux/TF_BUILD_IS_OPT=OPT,TF_BUILD_IS_PIP=PIP,TF_BUILD_PYTHON_VERSION=PYTHON2,label=cpu-slave/253/artifact/pip_test/whl/tf_nightly-1.8.0.dev20180409-cp27-cp27mu-linux_x86_64.whl

The :pr_curves_plugin_test fails if I install tf-nightly build 254.
pip install -I https://ci.tensorflow.org/view/tf-nightly/job/tf-nightly-linux/TF_BUILD_IS_OPT=OPT,TF_BUILD_IS_PIP=PIP,TF_BUILD_PYTHON_VERSION=PYTHON2,label=cpu-slave/254/artifact/pip_test/whl/tf_nightly-1.8.0.dev20180410-cp27-cp27mu-linux_x86_64.whl

Therefore, a TensorFlow change introduced to build 254 seems to be a likely cause. During setUp, the :pr_curves_plugin_test runs a demo script that seeds randomness and then generates test data from a tf.distributions.Normal distribution.
Could any changes to TensorFlow's seeding or tf.distributions.Normal logic (introduced to build 254) cause this breakage? Regardless of how we fix :pr_curves_plugin_test (Either TensorBoard's test logic could change, or we could partially roll back a TensorFlow change.), I'm curious about the root cause of the difference in behavior. Thanks!
Source code / logs
Here are the logs for the failed test. They just describe how the floating point data derived from the PR curves plugin demo have changed.
Testing //.../plugins/pr_curve:pr_curves_plugin_test; 40s linux-sandbox
...F....
======================================================================
FAIL: testPrCurvesDataCorrect (__main__.PrCurvesPluginTest)
Tests that responses for PR curves for run-tag combos are correct.
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/home/travis/.bazel-output-base/bazel-sandbox/2643229171274820909/execroot/org_tensorflow_tensorboard/bazel-out/k8-fastbuild/bin/tensorboard/plugins/pr_curve/pr_curves_plugin_test.runfiles/org_tensorflow_tensorboard/tensorboard/plugins/pr_curve/pr_curves_plugin_test.py", line 217, in testPrCurvesDataCorrect
    pr_curve_entry=entries[0])
  File "/home/travis/.bazel-output-base/bazel-sandbox/2643229171274820909/execroot/org_tensorflow_tensorboard/bazel-out/k8-fastbuild/bin/tensorboard/plugins/pr_curve/pr_curves_plugin_test.runfiles/org_tensorflow_tensorboard/tensorboard/plugins/pr_curve/pr_curves_plugin_test.py", line 87, in validatePrCurveEntry
    assert_allclose(expected_precision, pr_curve_entry['precision'])
  File "/home/travis/virtualenv/python2.7.14/lib/python2.7/site-packages/numpy/testing/nose_tools/utils.py", line 1396, in assert_allclose
    verbose=verbose, header=header, equal_nan=equal_nan)
  File "/home/travis/virtualenv/python2.7.14/lib/python2.7/site-packages/numpy/testing/nose_tools/utils.py", line 779, in assert_array_compare
    raise AssertionError(msg)
AssertionError:
Not equal to tolerance rtol=0, atol=1e-07
(mismatch 75.0%)
 x: array([0.333333, 0.385321, 0.542169, 0.75    ])
 y: array([0.333333, 0.391026, 0.630137, 0.666667])
----------------------------------------------------------------------
Ran 8 tests in 38.439s
FAILED (failures=1)
================================================================================

Thanks for reporting it. There was a bug clip_by_value.
Please try tf-1.8rc1 to see if the issue still exists.

Thank you, @bignamehyp. The demo works again. See https://github.com/tensorflow/tensorboard/pull/1150. Could you please point us to what PR fixed the problem?

It's an internal change and and get merged in a big PR.


Error when plotting

Hello,
I have a strange error when trying to use plotnet. Let's assume I have two models, called m1 and m2, which are trained using the exact same formula on two different datasets.
If I do
> plotnet(m1,alpha=0.6)

Everything works fine.
But if run
> plotnet(m2,alpha=0.6)

What i get is
Error in as.character(forms) :
  cannot coerce type 'closure' to vector of type 'character'

Do you have any idea of what could cause this problem?

Can you explain the models and input data a bit more?  I can't diagnose anything from the example.

Yeah, I'm sorry, you're right. I created a gist with the example code and data, I hope this helps.

You need to rename the formula object to something else.  formula is already a function in R and that's causing problems in the NeuralNetTools functions.  This should work:
library(nnet)
library(NeuralNetTools)

formulaA = AKEC ~
  tf +
  Freq_Sentence +
  Height +
  LifeSpan +
  Depth +
  tf.idf +
  Maximality

formulaS = SEMEVAL2010 ~
  tf +
  Freq_Sentence +
  Height +
  LifeSpan +
  Depth +
  tf.idf +
  Maximality

AKEC_train <- read.csv("https://gist.githubusercontent.com/basaldella/ebb672efbb12c565dec585a6ed0916b4/raw/b9dc49dffa6e8caa758cacd0a5acf8148b73c89e/akec.top.csv",stringsAsFactors = TRUE)
SEMEVAL_train <- read.csv("https://gist.githubusercontent.com/basaldella/ebb672efbb12c565dec585a6ed0916b4/raw/b9dc49dffa6e8caa758cacd0a5acf8148b73c89e/semeval.top.csv",stringsAsFactors = TRUE)

set.seed(1203)
akecModel = nnet(formulaA,
                 size = length(attr(terms(formulaA),"variables")) * 2,
                 entropy=T,data = AKEC_train)

set.seed(1203)
semevalModel = nnet(formulaS,
                    size = length(attr(terms(formulaS),"variables")) * 2,
                    entropy=T,data = SEMEVAL_train)

plotnet(akecModel)
plotnet(semevalModel)

Now it works like a charm. Sorry for my mistake, but I'm far from being an R expert... Thanks a lot for your help!

No problem, I'm happy to help.


FastParseExample hash collision

System information

TensorFlow installed from (source or binary): source
TensorFlow version (use command below):  1.12.0
Python version: 2.7
Bazel version (if compiling from source): 0.19.2
GCC/Compiler version (if compiling from source): GCC4.8.5

Describe the current behavior
build config index retry always failed.
code in tensorflow/core/util/example_proto_fast_parsing.cc function FastParseExample
https://github.com/tensorflow/tensorflow/blob/a93f06d160955c99bc279a419902ac40824a2cab/tensorflow/core/util/example_proto_fast_parsing.cc#L988
ok flag is not reset to true. And this will fail for 1000 times, if hash collision is found at first time.
Describe the expected behavior
reset ok flag and rehash with new seed, so that retry logic will work

@yann-yy were you able to build and install TF? Please provide OS? If the build is failing, please take a look at the tested build configurations. Thanks!

@yann-yy were you able to build and install TF? Please provide OS? If the build is failing, please take a look at the tested build configurations. Thanks!

hi, @jvishnuvardhan ,
Thanks for your reply. This is not a build issue. And I have submit a pull request to fix this. This issue could be closed.

@yann-yy Thank you! I am closing this issue. Thanks again!

I think this should be kept open until the PR with the fix lands

Thank you for #29121, fixing this


[WIP] - ECS Integration

This PR adds support for Amazon ECS as the backend container service.
The functionality will allow users to

[x] Use Fargate/Fargate Spot to run containers.
[ ] Define Task Definition files for different shiny apps.
[ ] Integration and Unit tests.

Hi @kochie
Thanks for working on this, looks very interesting! I was wondering whether you need any help? As you may have noticed in the last release, some major changes were made in the backend API. This should make it a lot easier to implement new backends, but this also means that the current code may need some changes.

Hey @LEDfan thanks for the offer! Yes it would be great to have some input. I've actually completed this for a client already and we we're able to get it running perfectly in their account. There were some specific issues related to their environment that can be fixed up, happy to discuss in code review. I'll update this branch with the latest version.

Hi, we released ShinyProxy 3.1.0 today, which contains full support for ECS. It's based on this contribution, but was extended and fully integration in ShinyProxy. Thank you again for your contribution!


feat: Added codecarbon integration

This PR introduces the following modifications:

adds decorator integration of codecarbon
updates requirements and gitignore

Any feedback is welcome!
cc @benoit-cty

Hi @frgfm thanks for this ! I do it work ? do we have an associated login ? or anyone cloning this repo will impact our score ?

I think it's better not to track public users and to offer access to anyone who is close to the project and making them aware of gpu abuse.

Hello,
Your current code will work with no config file : the user emissions will be tracked but not send to the CodeCarbon API. Your user will have a local file emissions.csv. It's the default behavior of CodeCarbon.
So I think you are fine: When you are in an environment with proper config file, your emissions will go into the CodeCarbon API.

In another PR, we can throw a warning when there is no config file (so that core team doesn't forget to set it when running a training)


i cannot run the script, i it still working

Can you please help

I got the almost the same problem :/
`root@kali:~/Desktop/scripts/IPTV/iptv# python iptv.py
################
IPTV
v1.2.3
################
Menu
0 - Exit
1 - Search for some Servers
2 - Look at the servers list
3 - Select language, default is Italian
4 - Brute force all server from the list
5 - Brute force random server from the list
6 - Brute force specific server from the list
7 - Provide a random server to attack
Please select an option: 1
Fetching URLs plase wait...
Traceback (most recent call last):
File "iptv.py", line 38, in
cr.search_links()
File "/root/Desktop/scripts/IPTV/iptv/Crawler.py", line 62, in search_links
for url in google.search(self.searchString,stop=1):
AttributeError: 'module' object has no attribute 'search'
`


[Feature Request] Django requests not compatible with interia request payloads?

I've run into another snag. When attempting to post interia js forms the django request only includes the payload in a raw bytestring via request.body.
const form = useForm({
    key: 'value'
});
form.post(usePage().url);

request.POST is empty as that's only populated from traditional form submissions. I'm unable to access key param in the view without processing the request.body bytestring first, which seems totally out of scope for individual projects to handle. DRF has solutions for this, but DRF shouldn't necessarily be required for this adapter to work I wouldn't think? Is there a way for the adapter to handle this?

Per discord conversation, this is what I have so far (maybe a naive implementation) which could hopefully give people access to inertia request data out of the box.
# settings.py - Setting for parsing application/json requests and placing data in request.JSON. If you are using something like Django Rest Framework, this is done already and available through request.data, so it can be disabled here.
INERTIA_PARSE_JSON_REQUEST = True

# middleware.py
if settings.INERTIA_PARSE_JSON_REQUST and request.body and 'application/json' in request.content_type:
    try:
        request.JSON = json.loads(request.body)
    except json.JSONDecodeError:
        request.JSON = {}

@ChrisxHart You can pass the following argument to form.post to make the request compatible with Django.
form.post(usePage().url, { forceFormData: true });

I like the idea of baking some json support into Inertia Django because it's such a tightly coupled concept.
@svengt's solution is perfect if you want to maintain Django standards, but for those who don't, I think I'm going to add something very similar to what @ChrisxHart wrote there into core. It'll probably be a very similar syntax, just lazily evaluated.
Thanks both of you, I'll leave this open until I have a PR up.

closing per #30


Joining issue

The Joining issue is still up

I will be taking a look at this a little later on today. I think discord has had an update recently to their join endpoint so I will sort it out and then push it to the repo and close this comment once I have done that.

Stop buying those walmart tokens, and actually use correct terminology when creating an issue. Also please be more specific.

After doing some investigation it seems that your tokens had gotten flagged and discord was serving them a captcha... as of right now the tool does not support captcha for joining, I am extremely busy with other things, but I will get onto adding this feature soon. Anyone who sees this and would like to write the code for the captcha on join I am open to prs otherwise this is closed.


Some Questions About the Fault Tolerance Mechanism of VirtualService

After configuring the virtualservice, if 100% of the traffic points to subset1 and the endpoints of subset1 are unhealthy, what configuration allows the traffic to automatically request the endpoints of subset2。
istio verison 1.9.2
envoy version 1.17
k8s verison 1.20

Istio makes use of  weighted clusters to represent different subsets, currently it is not viable with VS setting to do failover between subsets.

Is there any solution or configuration to solve this problem?

Istio makes use of weighted clusters to represent different subsets, currently it is not viable with VS setting to do failover between subsets.


v20.2.5 improved AIStrategyProcessor::processPath

[Current coverage][cc-pull] is 100%

Merging [#513][cc-pull] into [master][cc-base-branch] will not change coverage

@@           master   #513   diff @@
====================================
  Files          27     27
  Lines         422    423     +1
  Methods       106    106
  Messages        0      0
  Branches        0      0
====================================
+ Hits          422    423     +1
  Misses          0      0
  Partials        0      0

Powered by Codecov. Last updated by [779bbd1...93b896e][cc-compare]
[cc-base-branch]: https://codecov.io/gh/eugene-matvejev/battleship-game-api/branch/master?src=pr
[cc-compare]: https://codecov.io/gh/eugene-matvejev/battleship-game-api/compare/779bbd137ad7f2ea5b05e1a1dcaddef0af4a5027...93b896eb492702f5cc4ec010a231680217503074
[cc-pull]: https://codecov.io/gh/eugene-matvejev/battleship-game-api/pull/513?src=pr


BuiltContent.razor example confusion

Are these sequence numbers duplicates in the cycle?
https://github.com/dotnet/blazor-samples/blob/6c84ed3a8bf2905add913b1c4fbd2f6fe7224dec/8.0/BlazorSample_BlazorWebApp/Components/Pages/BuiltContent.razor#L18-L26

Thank! I think I know the reason, this kind of repetition in the static loop body should be no problem.


Defer code loading when active record is properly loaded

This is well known issue. We should prefer to load code that reference active record only when it's AR is already loaded.
It can break other gems initializing process like https://github.com/thoughtbot/factory_bot_rails/pull/432
Related:

https://github.com/rails/rails/issues/46567
https://github.com/paper-trail-gem/paper_trail/commit/fc6c5f69410f1b79e617610d522dbedfa4f04f7f (inspiration)
https://github.com/thoughtbot/factory_bot_rails/pull/432

Fix: https://github.com/DatabaseCleaner/database_cleaner-active_record/issues/89

Closing in favor of https://github.com/DatabaseCleaner/database_cleaner-active_record/pull/91


fix: ethers-contract circular dep on ethers-signers

As title, ethers-contract depended on signers which messes up the auto-release process, fixes by removing the dep and pulling the EIP712 tests which require both signers + contract + eip712 derive to the top level crate tests

In a bit of a rush to fix the release flow and start automating it, so merging.


Feature/sidebar icon item

relates to https://github.com/IPG-Automotive-UK/cloud-licensing-web/issues/274

The sidebar items in cloud licensing have an Icon with text underneath it (stacked) rather than the default inline icon and text that is currently available.

Changes
This PR enables to user to pass in an optional display property which can be either "inline" or "stacked" which will allow the user to specifically set the orientation of the icon and text in the sidebar item. The default behavior of this component is maintained as the default value of the display property is "inline".

In order to reduce code duplication I have created separate components inside sidebar item for the Count badge and the expand icon so that I can use them for both the SidebarInLine and SidebarStacked components.
The SidebarInLine and SidebarStacked components implement their desired layouts respectively which the main SidebarItem component switches between depending on the display value.
I have updated the Sidebar Item types to include the new display property
I have added SidebarItem.stories to show what a stacked SidebarItem will look like
I have also updated the Sidebar.stories to include a view of what a sidebar with stacked items will look like.

Dependencies
Do Not Merge: This PR is built on the work done here :  https://github.com/IPG-Automotive-UK/react-ui/pull/583 this will need to be approved and merged before this can be merged.
UI/UX
Cloud Licensing Web Figma: https://www.figma.com/proto/O3IAobLTPcjIO1WaxaPod6/IPG-Cloud-Licensing?node-id=68-14265&starting-point-node-id=68%3A14265
Cloud Licensing Sidebar in Figma:

Author checklist before assigning a reviewer

[x] Reviewed my own code-diff.
[x] Assigned to me.
[x] Relevant tags added.
[x] Appropriate tests have been added.
[x] Lint and test workflows pass.

Can I see the implemented screenshot here @jegasriskantha

@jegasriskantha I missed a comment from @Sowbhagya-ipg, Is it mandatory to integrate it somewhere and share the screenshot before approving?

@Sowbhagya-ipg this is the actual component in this PR:
this is it implemented within the sidebar
During implementation in the cloud licensing app the sidebar width, sidebar logo and version number can be modified / removed.

This looks good


[Event Hubs Bindings] Bindings Configuration Precedence Fix

Summary
The focus of these changes is to fix a bug in the client factory where behavior differed when creating clients when a connection string and an explicit Event Hub name were both configured.  Creation for the producer and consumer gave precedence to the connection string, as documented, allowing both to be specified without error.  The event processor gave precedence to the event hub value, causing inconsistent behavior and potential errors to surface when it disagreed with the connection string.
Also included is a tweak for test assertions to reduce flakiness observed during nightly runs.
References and Resources

Azure Event Hubs trigger binding attributes
IcM incident #361375349 (Microsoft internal)

API change check
API changes are not detected in this pull request.


DKG - Switch to V1

Description
Describe the changes or additions included in this PR.
Test plan
How did you test the new or updated feature?

Release notes
Check each box that your changes affect. If none of the boxes relate to your changes, release notes aren't required.
For each box you select, include information after the relevant heading that describes the impact of your changes that a user might notice and any actions they must take to implement updates.

[ ] Protocol:
[ ] Nodes (Validators and Full nodes):
[ ] Indexer:
[ ] JSON-RPC:
[ ] GraphQL:
[ ] CLI:
[ ] Rust SDK:

@mystenmark adding you as backup in case I go OOO before this is ready to merge


depth file

hello,can you provide depth map file?

@zjw102, hi, it's already in the provided link for downloading the shapenet55 dataset, for example, all the png files with "_depth0001.png" are the depth mapts.
sfr-ulip-code-release-research/shapenet-55/rendered_images/02691156-10155655850468db78d106ce0a280f87/02691156-10155655850468db78d106ce0a280f87_r_000_depth0001.png


Breaking Android changes for libc 1.0

https://github.com/rust-lang/libc/issues/641 listed some things that we may want to break when libc gets a 1.0. https://github.com/rust-lang/libc/pull/634 addressed some of it, but I don't believe that is everything since (iiuc) it went into a 0.2.x release.
We are ramping up toward a 1.0 release. Is there anything else incorrect that we may want to change? Cc @ndusart since you authored the issue and PR, @maurer since I think you're the main Android maintainer at this point.

That's a looong time ago for me :sweat_smile: , I need to get back into this crate internals to provide meaningful information.
From a quick glimpse, I would say that it should be best to look into the symbols that are skipped from the libc-test and see if they fails because they are just unaligned with latest headers or if the skip is there for a valid reason.
Most of the skips I introduced are gone (and comments about these skips are now weirdly in the solaris part of the test ^^) so I suppose most breaking changes have already been done (e.g: https://github.com/rust-lang/libc/commit/7d5e632d36fea980a7fadaafd279c4604978cad7)
I cannot provide the necessary effort to be exhaustive in this matter though and I didn't need to do any Android native development for a while so I cannot say easily if libc crate is off for the latest versions of NDK.


Background call issue

I built iOS and android video call app using quickblox webRTC.
I used Q-municate.
It has all functions.
Background call runs well in iphone to iphone, and android to android.
But iphone->android, android->iphone background call don't run at all.
Push notification runs well in all platforms.
I hope to get correct answer as soon as.
Thanks

?

@starlucas629, what version of Q-mun do you use?

I used last version in github

You state you use latest version 2.7 from Goole Play market, and there are no background calls between iOS - Android?

Yes.

It is not true. Background calls work fine, our testers have just tested this and it's all ok.

I only changed quickblox account keys, and firebase sender id.
buildTypes {
release {
minifyEnabled false
debuggable false
ext.enableCrashlytics = true
signingConfig signingConfigs.release
buildConfigField "String", "APP_ID", ""70164""
buildConfigField "String", "AUTH_KEY", ""65Re2y7gEG2jus6""
buildConfigField "String", "AUTH_SECRET", ""FzGhdxZma9cPwE6""
buildConfigField "String", "API_ENDPOINT", ""https://api.quickblox.com""
buildConfigField "String", "CHAT_ENDPOINT", ""chat.quickblox.com""
buildConfigField "String", "ACCOUNT_KEY", ""SX9yCWuaQsCJ7RUDkiru""
buildConfigField "String", "FIREBASE_AUTH_PROJECT_ID", devFirebaseAuthProjectId
resValue "string", "facebookAppId", prodFacebookAppID
resValue "string", "sender_id", ""998226485013""
manifestPlaceholders = [fabricApiKey:prodFabricApiKey]
}
    debug {
        debuggable true
        versionNameSuffix ' (debug)'
    }
}

Background call runs well in same platforms.
But it doesn't run between iphone and android.

Do push messages work between platforms?

No, It doesn't work too

After I call from iphone  to android (android is in background mode), if i open android app, I can see push alert.
Dose the push notification run?

I already checked all doc.
I think my Quickblox account has issue.
It is free account.
May be can I use paid account?
Thanks.

If I use paid account, can I solve this issue?

I think you can resolve issue on your current account, but sure you can use paid account and get more privileges.

means?
Now can I solve this issue in free account?

Please let me know what should I change in the github code.

All you need to change is stated in the doc. So, I want to clarify, is cross-platform messaging working? Does offline notification come on Android (on android side app is not running), when you send chat message from iOS? And from Android? Also, attach your project build.gradle file.


FUIPasswordSignInViewController Not Dismissing

Step 2: Describe your environment

Objective C or Swift: Swift
iOS version: 11
Firebase SDK version: 4.10.0
FirebaseUI version: 4.5.3
CocoaPods Version: 1.4.0

Step 3: Describe the problem:
Steps to reproduce:

Set FUIAuthPickerViewController as rootViewController of applications UIWindow
Select email Sign in method
Enter login information and press sign in button

Observed Results:
View shows activity indicator, indicator goes away and app stay on sign in screen
Expected Results:
View Should return control to the FUIAuthPickerViewController and call the method
authUI:(FUIAuth *)authUI
    didSignInWithUser:(nullable FIRUser *)user
                error:(nullable NSError *)error

Relevant Code:
window.rootViewController = FUIAuth.defaultAuthUI()?.authViewController()

The issue seems to be in FUIPasswordSignInViewController line 136 where the view calls dismissViewControllerAnimated which only works if the original FUIAuthPickerViewController was presented modally

duplicate of #254


Add ESLint and Fix Linting in whole project

@aashutoshrathi has submitted a pull request. See it on IssueHunt

@aashutoshrathi has rewarded $14.00 to @aashutoshrathi. See it on IssueHunt

:moneybag: Total deposit: $20.00
:tada: Repository reward(20%): $4.00
:wrench: Service fee(10%): $2.00


feat & bugfix: Added metadata features and fixed naming bug

Allowing users to store metadata for each prompt
First create meta subdir inside the root template dir and then create prompt metadata.

@masci should we disable more lint errors? I am not sure if these are valid errors, wdyt?

Thanks a lot @masci  :pray:
What do you think of the PR?


Add history in readme support for /makerelease

Also allow issue comments to trigger /makerelease
Improve test mocking

/makerelease


Add Garden Linux image flavour for hypervisor generation V2 [Azure]

What would you like to be added:
As a user I want to create Garden Linux images for hypervisor generation V2.
Why is this needed:
See https://docs.microsoft.com/en-us/azure/virtual-machines/generation-2

This is getting more urgent as gen2 has instance types that are not supported with gen1.

Docs: https://docs.microsoft.com/en-us/azure/virtual-machines/generation-2

I tested the azure garden linux image with HyperV generation 2, it works out of the box. Generation 2 images use uefi boot and SCSI, both is available in the garden linux azure image. I also tried to enable secure boot, but that fails with a "Your virtual machine is prevented from booting due to secure boot validation failure. This could be due to unsigned boot components." message.

Note for secure boot: As mentioned in the other comment, secure boot is currently not working with garden linux and azure. The keys used for signing the garden linux boot image are unknown to azure.


what devices are supported?

hello, i have tried to use megablocks in V100 + pytorch2.4.0+cu121, but get error with "cannot support bf16". If i use megablocks in fp32, i get error "group gemm must use bf16". So i change my enviroment to A6000, and then get error:
[rank1]:   File "/usr/local/lib/python3.11/dist-packages/megablocks/ops/sort.py", line 34, in forward
[rank1]:     ops.sort(x, end_bit, x_out, iota_out)
[rank1]: RuntimeError: no kernel image is available for execution on the device

So i am wondering what devices are supported? Only A100/A800 and H100/H800 ?

We have only tested on A100/H100 at this time. It is possible with some tweaks to likely get it working on other hardware

ok, thanks！I will check it.

After i change my env to A100，the issue still exists. I have no idea how to do.
[rank0]:   File "/usr/local/lib/python3.11/dist-packages/megablocks/layers/moe.py", line 468, in forward
[rank0]:     out = self.experts(x, scores, expert_weights, top_experts)
[rank0]:           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
[rank0]:   File "/usr/local/lib/python3.11/dist-packages/torch/nn/modules/module.py", line 1553, in _wrapped_call_impl
[rank0]:     return self._call_impl(*args, **kwargs)
[rank0]:            ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
[rank0]:   File "/usr/local/lib/python3.11/dist-packages/torch/nn/modules/module.py", line 1562, in _call_impl
[rank0]:     return forward_call(*args, **kwargs)
[rank0]:            ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
[rank0]:   File "/usr/local/lib/python3.11/dist-packages/megablocks/layers/moe.py", line 429, in forward
[rank0]:     x, tokens_per_expert = self.forward_fn(x, expert_weights, top_experts)
[rank0]:                            ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
[rank0]:   File "/usr/local/lib/python3.11/dist-packages/megablocks/layers/moe.py", line 262, in parallel_forward_once
[rank0]:     indices, bin_ids, bins, tokens_per_expert = (self.indices_and_bins(top_experts))
[rank0]:                                                  ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
[rank0]:   File "/usr/local/lib/python3.11/dist-packages/megablocks/layers/moe.py", line 161, in indices_and_bins
[rank0]:     output = ops.sort(top_expert, self.sort_end_bit)
[rank0]:              ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
[rank0]:   File "/usr/local/lib/python3.11/dist-packages/torch/autograd/function.py", line 574, in apply
[rank0]:     return super().apply(*args, **kwargs)  # type: ignore[misc]
[rank0]:            ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
[rank0]:   File "/usr/local/lib/python3.11/dist-packages/megablocks/ops/sort.py", line 34, in forward
[rank0]:     ops.sort(x, end_bit, x_out, iota_out)
[rank0]: RuntimeError: no kernel image is available for execution on the device

this is my env:
nv
root@I1cedf606ff00701a02:~/workspace/DynamicMoE# nvidia-smi
Wed Oct  9 11:16:58 2024
+---------------------------------------------------------------------------------------+
| NVIDIA-SMI 535.86.10              Driver Version: 535.86.10    CUDA Version: 12.2     |
|-----------------------------------------+----------------------+----------------------+
| GPU  Name                 Persistence-M | Bus-Id        Disp.A | Volatile Uncorr. ECC |
| Fan  Temp   Perf          Pwr:Usage/Cap |         Memory-Usage | GPU-Util  Compute M. |
|                                         |                      |               MIG M. |
|=========================================+======================+======================|
|   0  NVIDIA A100-PCIE-40GB          On  | 00000000:40:00.0 Off |                    0 |
| N/A   34C    P0              36W / 250W |      4MiB / 40960MiB |      0%      Default |
|                                         |                      |             Disabled |
+-----------------------------------------+----------------------+----------------------+

+---------------------------------------------------------------------------------------+
| Processes:                                                                            |
|  GPU   GI   CI        PID   Type   Process name                            GPU Memory |
|        ID   ID                                                             Usage      |
|=======================================================================================|
|  No running processes found                                                           |
+---------------------------------------------------------------------------------------+

pytorch:
Name: torch
Version: 2.4.0+cu121
Summary: Tensors and Dynamic neural networks in Python with strong GPU acceleration
Home-page: https://pytorch.org/
Author: PyTorch Team
Author-email<EMAIL_ADDRESS>License: BSD-3
Location: /usr/local/lib/python3.11/dist-packages
Requires: filelock, fsspec, jinja2, networkx, nvidia-cublas-cu12, nvidia-cuda-cupti-cu12, nvidia-cuda-nvrtc-cu12, nvidia-cuda-runtime-cu12, nvidia-cudnn-cu12, nvidia-cufft-cu12, nvidia-curand-cu12, nvidia-cusolver-cu12, nvidia-cusparse-cu12, nvidia-nccl-cu12, nvidia-nvtx-cu12, sympy, triton, typing-extensions
Required-by: megablocks, stanford-stk, torchaudio, torchvision

and i have downloaded the group-gemm utils:
pip install git+https://github.com/tgale96/grouped_gemm@main

After I reinstall megablock from
pip install megablocks[all]

instead of installing form source(what i have done), the issue disappears.

Hm... maybe something is wrong with grouped gemm wheels...? @tgale96 any ideas here

ok!
And what is the differences of "grouped" impl and "sparse" impl? Which one gains better throughput?

They are different implementations for a groupedmatmul. Grouped is a series of iterated kernel calls with standard matmuls, whereas sparse is the strategy described in MegaBlocks https://arxiv.org/abs/2211.15841

But why recommend using grouped one instead of sparse one for H-GPUs?
Installing megablocks[gg] enables dMoE computation with grouped GEMM. This feature is enabled by setting the mlp_impl argument to grouped. This is currently our recommended path for Hopper-generation GPUs.

We have observed better performance on H100s with grouped-gemm. This may not be true with the latest versions of triton

Thanks!

There is another issue.
i haved tried dMoE fwd, which pass in my 1xA100 device but stop(not break) in my 2xA6000 device. Both are launch by
torchrun --standalone --nnodes=1 --nproc-per-node=[1 or 2] [my python script]

And this is my python script:
from megablocks import dMoE
from megablocks import Arguments

import os

import torch
import torch.distributed as dist

rank = int(os.environ["RANK"])
world_size = int(os.environ["WORLD_SIZE"])
print(f"Running example on {rank=} in a world with {world_size=}")
dist.init_process_group("nccl")
torch.cuda.set_device(rank)
num_node_devices = torch.cuda.device_count()
rank_lists = list(range(0, num_node_devices))
ep_groups = dist.new_group(rank_lists)
device = f'cuda:{rank}'

args = Arguments()
args.moe_expert_model_parallelism = True
args.expert_parallel_group  = ep_groups
args.device = device
args.bf16 = True
args.fp16 = False
args.mlp_impl = 'grouped'

model = dMoE(args)
model.to(device)

print("build up model success!")

x = torch.rand((2, 4, 1024), dtype=torch.float32).to(torch.bfloat16)
x = x.to(device)

torch.manual_seed(42)
torch.cuda.manual_seed(42)
logits = model(x)

if rank == 0:
    print("didn't crash yay!")

I try to debug, and find the the code process stop there:
# megablocks/layers/moe.py --> class ParallelMLP --> func parallel_forward_once --> lines 288

        # This view updates the shape of the tensor from [sl, bs, hs] to
        # [sl * bs, hs] prior to the permutation.
        x = x.view(-1, x.shape[-1])
        print("pass!")
        output = ops.gather(x, indices, bin_ids, bins, self.top_k)
        print("break!")
        assert output is not None
        x = output

the output is
Running example on rank=1 in a world with world_size=2
Running example on rank=0 in a world with world_size=2
build up model success!
build up model success!
pass!
pass!

Is there anything wrong? Or more information needed?

OK, i have tried it in 2xA100 device and the issue disappears. It seems that some tweaks are needed to adapt it to devices other than A100/H100.

Hm... I'm not super sure what happens if you are on one GPU -- it might be some error with torch dist initialization that is buried. I unfortunately do not have other GPUs to test on, but if you manage to narrow it down happy to help fix.

ok, i will try it.


Update to version 3.28.0

Why the update?
It has recently been merged an important fix that allow to use swagger-ui post multiple files (previously was broken):
https://github.com/swagger-api/swagger-ui/issues/4600
Why this version?
Is the last tagged version in swagger-ui, and it has the fix (tested locally).

Hey @gvergine - thanks for the PR. I ended up pulling in the latest release (3.30.0) in #14 .
Sorry for the delay.


Support jetty-client 9.3.11

We found a http proxying bug that manifested with squid 3.1 and jetty client 9.2:
https://bugs.eclipse.org/bugs/show_bug.cgi?id=475546
Also, this will Includes support for HTTPS forward proxies, which means that the useSSL flag will now actually be respected:
https://github.com/eclipse/jetty.project/issues/416
Unfortunately jetty-client 9.3.x targets JDK 8. td-client-java still targets JDK 7.
How can embulk fix this issue?

Sorry, this issue should be filed in embulk-output-td.


Clean up monitor package

Coverage increased (+0.1%) to 74.858% when pulling fc416b146f57c7973b8a74ccad8a5eea23bde22c on issue/649-cleanup-monitor-api into f3610c5d31b255e18dffa3654363262478ba9a14 on develop.

@rvolosatovs

Coverage increased (+0.1%) to 74.848% when pulling a1ada035ba3e295e102cff4c61c9308efbd45969 on issue/649-cleanup-monitor-api into f3610c5d31b255e18dffa3654363262478ba9a14 on develop.

Coverage increased (+0.1%) to 74.848% when pulling 7a1332a22cef8a4f951b2fd6a0c4054f2549df6b on issue/649-cleanup-monitor-api into f3610c5d31b255e18dffa3654363262478ba9a14 on develop.

Coverage increased (+0.02%) to 74.848% when pulling 72449f9461725f6cd6c8b0616ee6bbb62c4e265b on issue/649-cleanup-monitor-api into 645889932a2217bf394bd5c30fd979ef50128721 on develop.

Coverage increased (+0.02%) to 74.848% when pulling 3fb0dccfbf568866d0ef5f34f0a7a510b0b636ff on issue/649-cleanup-monitor-api into 645889932a2217bf394bd5c30fd979ef50128721 on develop.

Coverage increased (+0.04%) to 74.864% when pulling 3b0952ac307868a428d417f035aa526055c476db on issue/649-cleanup-monitor-api into 645889932a2217bf394bd5c30fd979ef50128721 on develop.

Coverage increased (+0.04%) to 74.864% when pulling 6f7575838b53ca3a858dcf4083f14d849097e72d on issue/649-cleanup-monitor-api into 645889932a2217bf394bd5c30fd979ef50128721 on develop.

Coverage increased (+0.04%) to 74.864% when pulling f0f50024db81db2c0dbe858e8be406e42cb82768 on issue/649-cleanup-monitor-api into 645889932a2217bf394bd5c30fd979ef50128721 on develop.

Coverage increased (+0.04%) to 74.883% when pulling e75750591e68e987fa9e51ffed0c5942a6d3ad5a on issue/649-cleanup-monitor-api into aab6115a3fba8efd99451f9b51fb891716690ec9 on develop.


🚀 Feature: Add latest_report_status to puppetdb backend provider

🔖 Feature description
Hello, It would be handy if I could quickly filter my resources by their last puppet run state. This way I can quickly read failed nodes etc.
I would like to add latest_report_status which is a parameter in the nodes endpoint as a tag.
🎤 Context
as said I want to filter quickly on the status of my nodes
✌ Possible Implementation
You would have to alter type.ts
......
latest_report_status: string;
  /**
   *  The status of the latest report. Possible values come from Puppet's report status failed, changed, or unchanged.
   */
....

and transform.ts
.....
 const latest_report_status = node.latest_report_status;

  return {
    apiVersion: 'backstage.io/v1beta1',
    kind: 'Resource',
    metadata: {
      name: certName,
      annotations: {
        [ANNOTATION_PUPPET_CERTNAME]: certName,
      },
      namespace: DEFAULT_NAMESPACE,
      description: node.facts?.data
        ?.find(e => e.name === 'ipaddress')
        ?.value?.toString(),
      tags: kernel ? [kernel.toString().toLocaleLowerCase('en-US'), latest_report_status.toString().toLocaleLowerCase('en-US')] : [],
    },
    spec: {
      type: type,
      owner: DEFAULT_ENTITY_OWNER,
      dependsOn: [],
      dependencyOf: [],
    },
  };

👀 Have you spent some time to check if this feature request has been raised before?

[X] I checked and didn't find similar issue

🏢 Have you read the Code of Conduct?

[X] I have read the Code of Conduct

Are you willing to submit PR?
Yes I am willing to submit a PR!

I made a fork here with the changes https://github.com/negast/backstage

Awesome @negast! Please create a PR to the master branch & link it to the issue. Implementation details can than be discussed in the PR with the reviewers.


Add docs for highlight selection

Docs for React Spectrum so far, React Aria examples to come.

Merge after release


Unable to install jschon via pip

Hi,
If I try to install jschon I get this error:
$ pip3 install jschon
ERROR: Could not find a version that satisfies the requirement jschon (from versions: none)
ERROR: No matching distribution found for jschon

And if I specify a version too:
$ pip3 install jschon==0.2.0
ERROR: Could not find a version that satisfies the requirement jschon==0.2.0 (from versions: none)
ERROR: No matching distribution found for jschon==0.2.0

I have tried both pip and pip3 but I get the same result.

What version of Python are you using? Only Python 3.8 and 3.9 are supported.

What version of Python are you using? Only Python 3.8 and 3.9 are supported.

Oh, I'm using 3.6.9. I guess I need to update my distribution then.


Update packit config

Update this to the latest syntax and build for all supported releases.
Closes: #238

Note: do not merge until the CI is green

/packit rebuild

/packit rebuild-failed

Codecov Report
Base: 58.71% // Head: 58.68% // Decreases project coverage by -0.03% :warning:

Coverage data is based on head (9bfd457) compared to base (b81adaf).
Patch has no changes to coverable lines.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #239      +/-   ##
==========================================
- Coverage   58.71%   58.68%   -0.04%
==========================================
  Files         102      102
  Lines        9492     9492
==========================================
- Hits         5573     5570       -3
- Misses       3441     3443       +2
- Partials      478      479       +1

Impacted Files
Coverage Δ

timestamp/timestamp_linux.go
58.75% <0.00%> (-1.88%)
:arrow_down:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


ctrl-c with no selection - Bad state for CUT callback in SelectionManager

Here's a little bug I encountered early on.
To reproduce, just open the desktop build, and hit ctrl-c without doing anything else. You will go straight to the node-webkit crash screen, with the message, "Uncaught node.js Error. Bad state for CUT callback in SelectionManager". If you draw on the canvas and then hit ctrl-c without a selection, you get the "Your piskel seems to have unsaved changes..." message. If you select ok, it will crash in the same way.
Ctrl-c works fine if you have a selection, but crashes without one. Maybe just don't throw an error in SelectionManager?
By the way, I noticed that the node-webkit version defined in package.json is v~0.10.1, which is a bit behind. It might be safe enough to say "latest"?

Thanks for the report !
Looks like every uncaught exception leads to this in node-webkit. I will review the rest of the project and will swap exceptions for error logs.

I'm not completely sure about this, but can you use process.on('uncaughtException') to create a generic exception handler? Then you could just log it from there.
https://nodejs.org/api/process.html

Moved all non critical errors to console.error for now. Should reduce the amount of crashes for desktop
WIll be released for v0.5.0


format code

Summary of changes
The project is formatted with mix format,  for a better future developing experience.
related issue

[x] All the codes (*.ex, *.exs) are formatted with mix format;
[x] Travis CI scripts are updated, with mix format --check-formatted added in scripts when running on Elixir 1.10.x, in order to check future integrations;
[x] Travis CI elixir and otp_release configurations are updated to the latest stable version (22.2) in order to stick with the latest Elixir version and formatting rules.

Specification fixes will follow in another pull request.
Checklist

[x] New functions have typespecs, changed functions were updated
[x] Same for documentation, including moduledocs
[x] Tests were added or updated to cover changes
[x] Commits were squashed into a single coherent commit
[x] Notes added to CHANGELOG file which describe changes at a high-level

Coverage decreased (-0.2%) to 71.937% when pulling fb81ef34d4be2af4e69cc22122c81f81efba71f9 on qhwa:chore/format into 741ed40e30bd1af248cc10005dcfe6cd75058a53 on bitwalker:master.

CI is broken, fixing it.

@bitwalker Is there anything else I need to update?

Thanks!


chore(deploy): use runlevel label on namespace

All namespaces created before the openshift apiserver are up need a run-level
With the move to a namespace prefix, OLM now blocks later controllers (including
machine API) because some caches aren't filled on admission. Causes installer
to hang trying to apply the service account:
E1009 01:37:54.959270       1 sync.go:52] error running apply for (/v1, Kind=ServiceAccount) openshift-operator-lifecycle-manager/olm-operator-serviceaccount: serviceaccounts "olm-operator-serviceaccount" is forbidden: caches not synchronized

/approve
/lgtm

/test images

/lgtm


Added 'layoutIfNeeded' when changing the title of the cancel button.

After allowCancel is set to false,
When the cancel button is displayed after entering the passcode, the title of the cancel button is changed from 'cancel' to 'delete'.
So I added a layoutIfNeeded function when the title of the cancel button changes.

Oh awesome! Thanks so much for that @jangsy7883! :D


Not thread safe?

As there are so many static functions and members, I think zxing is not thread safe.
Any body know how to make it thread safe?

Well, static functions aren't an issue, but static functions with side effects certainly are.
We (Glass Echidna) didn't write zxing-cpp, we're just maintaining a centralised location for people to submit patches. So please don't blame us for the terrible code :wink:
This library is stable enough to use in most cases (I've been using it in production systems for over a year), however from a code quality stand-point I think the project is beyond redemption. If you want to drastically modify it (say make things thread safe), I think you'd be better off performing an entire rewrite and simply zxing-cpp for reference purposes.
If someone were to do so I'm sure they'd be very popular indeed!

It's not thread safe. I would say it's 'quite' safe though if you are running a thing or two parallel. Making it thread safe wouldn't be that big of a hassle. Static functions by themselves has little to do with thread safety.


Per-rule scheduling

Would be nice to have per-rule customization of scheduling. For example, apply BackOffScheduler only to certain rules.

Closed by #49


rebar3 support working with cwd rebar3 escript

I had to make some changes to relx that I haven't got merged into it and rebar3 yet, so only works wtih the bundled rebar3 escript right now.
I don't like some of the changes and will have to investigate fixes.
First, it seems relx isn't replacing variables in the template strings within the variable, which is why I changed runner_base_dir to {runner_base_dir,       "$(cd dirname $0 ; /bin/pwd)/.."}. Second, I had to change it to ; because && was being replaced with &&.

Nevermind, the relx patch isn't needed. But rebar3 script here is still "special", it has a patch for building escripts like cuttlefish I need to get merged to rebar3 repo.

I guess the rebar3 patch will eventually go into rebar3 master?

on a fresh clone of vernemq (rebar3-support branch) a ./rebar3 release builds the release! :) However the cuttlefish config file vernemq.conf is not generated.. do I have to invoke the rebar3_cuttlefish plugin somehow in advance?

Yea, the rebar3 patch was merged last night. So now would work with any rebar3 we release.
The cuttlefish plugin should be invoked due to {provider_hooks, [{post, [{release, cuttlefish}]}]}. I'll take a look in a minute.


stream/ws_client.py doesn't close the connection if _preload_content is True

Using kubernetes-4.0.0 python client and exec call as below:-
stream( api.connect_get_namespaced_pod_exec,
name=pod_name, namespace=namespace, command=cmd,
stderr=True, stdin=False, stdout=True, tty=False)
After this call it returns output as a string and not closing connection so If I use it in scale env then I see below error:-
filedescriptor out of range in select
So IMO ws_client.py should be fixed from below code snippet to close connection before return:-
client.run_forever(timeout=_request_timeout)
return WSResponse('%s' % ''.join(client.read_all()))
Please fix this if my observation is correct here.

/kind bug

@mbohlool

Here's a simple example that demonstrates the problem generically with select:
#!/usr/bin/env python3
import select
import sys

def main():
    file_objects = []
    # Show that 1023 is okay.
    while not file_objects or file_objects[-1].fileno() < 1024:
        file_objects.append(open('/dev/null'))

    # FD 1023 works
    print(select.select((file_objects[-2],), (), ()))
    # FD 1024 does not
    print(select.select((file_objects[-1],), (), ()))

if __name__ == "__main__":
    sys.exit(main())

I verified that on both OS X and Linux the following exception occurs:
Traceback (most recent call last):
  File "./select_demo.py", line 19, in <module>
    sys.exit(main())
  File "./select_demo.py", line 15, in main
    print(select.select((file_objects[-1],), (), ()))
ValueError: filedescriptor out of range in select()

I'm going to work on a hopefully simple PR to change from select to poll as I just started running into this issue when using this library within a flask application that supports thousands of threads.

Just to add to the above, the root issue is with whatever the file descriptor number is that is being selected on. In my case, this is due to external open files, but for you it could be due to the reason you've described. Either way, using select seems like a non-ideal way to go.

@bboe it's not actually the file descriptor number being selected on, but the ulimit on the total count. The default ulimit for open files is 1024. File descriptor numbers are per-process, but ulimits account for all processes running as a given user.
It doesn't matter which socket polling method you use, you'll still have just as many file descriptors total. To scale to thousands of file descriptors, you'll need to just adjust the ulimits on the process.
That said, a single select statement also cannot poll more than 1024 file descriptors either, however in the example above you only poll one descriptor at a time so that doesn't matter. But if you do need one statement to watch more than 1024 file descriptors, you'd want to switch to epoll or run select on multiple threads.

@PaulFurtado it is actually the file descriptor being selected on. Here's an example showing a single file descriptor selection failing:
#!/usr/bin/env python3
import select
import sys

def do_select():
    file_objects = []
    while not file_objects or file_objects[-1].fileno() < 1024:
        file_objects.append(open('/dev/null'))

    # FD 1023 works
    print(select.select((file_objects[-2],), (), ()))
    # FD 1024 does not
    print(select.select((file_objects[-1],), (), ()))

def do_poll():
    file_objects = []
    while not file_objects or file_objects[-1].fileno() < 1024:
        file_objects.append(open('/dev/null'))

    poll = select.poll()
    poll.register(file_objects[-1], select.POLLIN | select.POLLOUT)
    print(poll.poll())

def main():
    do_poll()
    print('poll works')

    do_select()

if __name__ == "__main__":
    sys.exit(main())


InputEventScreenDrag.relative does not work properly under "Emulate Touchscreen"

Godot version:
3.0
OS/device including version:
All devices.
Issue description:
According to the docs:
InputEventScreenDrag.relative is relative to the position the touch began.
InputEventMouseMotion.relative is relative to it's position in the previous frame.
 in /main/input_default.cpp Line: 298
You can see the relative prop of the mouse event being directly copied into the relative prop of the touch event.

@JPTeasdale Can you (or anyone else) still reproduce this bug in Godot 3.2.1 or 3.2.2beta4?

Closing due to lack of response. Please comment if you can still reproduce this bug.
PS: If no minimal reproduction project was included in the original bug report, please upload one as well to ease troubleshooting.


Make sure we can catch search.envri.eu/

Currently, the search engine can only be served from a sub-path (setting base_path: "" defaults to "search"; setting it to "/" is not reachable).
We need to add a rule to either serve the search engine from /, or redirect / to /{{ base_path }}

Needs https://github.com/QCDIS/KMS-generic-helm-charts/issues/34

Currently works with https://search.envri.eu:32443. We now want to catch traffic on port 443.

Resolved by deploying HAProxy on the old VM, that forwards traffic to the k8s cluster nodes, on the nodePort controlled by ingress.


Add context 'user_id' extraction for 'message_changed' events

I noticed that context.user_id was not being set for message_changed message subtype events. user is located in payload → event → message for these events. The first commit in this PR addresses this issue.
The second commit addresses a typing issue I noticed by ensuring that in is not being compared to None (the original code would generate "in" not supported for type "None" warnings. I also removed the # type: ignore comments on the associated return statements as they don't seem to be required.
Category (place an x in each of the [ ])

[X] slack_bolt.App and/or its core components
[X] slack_bolt.async_app.AsyncApp and/or its core components
[ ] Adapters in slack_bolt.adapter
[ ] Document pages under /docs
[ ] Others

Requirements (place an x in each [ ])

[X] I've read and understood the Contributing Guidelines and have done my best effort to follow them.
[X] I've read and agree to the Code of Conduct.
[X] I've run ./scripts/install_all_and_run_tests.sh after making the changes.

Hi @eddyg, thanks for sending this PR! Could you add the unit tests covering the pattern? Adding new files like the following would be easier to maintain for us. Please note that we need tests for both App and AsyncApp. Thanks!

https://github.com/slackapi/bolt-python/blob/v1.15.1/tests/scenario_tests/test_message_thread_broadcast.py
https://github.com/slackapi/bolt-python/blob/v1.15.1/tests/scenario_tests_async/test_message_thread_broadcast.py

Hi @eddyg, thanks for sending this PR! Could you add the unit tests covering the pattern? Adding new files like the following would be easier to maintain for us. Please note that we need tests for both App and AsyncApp. Thanks!

Alright, I took a shot at a simple test case (including an async version) to verify context.user_id in message_changed events...

@eddyg Thanks for adding tests! I have a few things to verify before merging this. I will work on it early next week.


Grammar is ambigous

For example, this code produces 16 possible parsings.
my_var => 9001;
my_add_ten => fn(x){
    return(+(x, 10));
};
if(true, fn(){
    print(my_add_ten(my_var));
});

Which is a problem because of two reasons:

Not knowing what the code will be interpreted as.
Takes exponentially more time to parse as the source code grows in size.

Comparing the trees with this amazing tool reveals that numbers can be parsed as variables - this should be a fairly simple fix.
Thanks for reporting! I'll be sure into looking for more ambiguities. I've set up a bit of code to warn about ambiguous syntax.

Alright, I fixed this in commits https://github.com/liam4/tlnccuwagnf/commit/2df352713feb6544ff54bec8c813905796149b7b and https://github.com/liam4/tlnccuwagnf/commit/702c7a831195177bcac55855f7e661894672dfeb! If you find any more ambiguities (the interpreter will warn you about any ambiguity issues) be sure to file another issue.


Handle argparse for Python 3.6

Checklist before submitting

[x] Did you read the contributor guide?
[x] Did you update any affected documentation?
[x] Did you write any tests to validate this change?

Description
Fixes #197

Codecov Report

Merging #205 (ee04a93) into master (e6acc8a) will increase coverage by 0.04%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master     #205      +/-   ##
==========================================
+ Coverage   94.24%   94.28%   +0.04%
==========================================
  Files          15       15
  Lines        1093     1101       +8
==========================================
+ Hits         1030     1038       +8
  Misses         63       63

Flag
Coverage Δ

unittests
94.28% <100.00%> (+0.04%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

open-ce/open-ce
85.19% <100.00%> (+2.58%)
: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 e6acc8a...ee04a93. Read the comment docs.

Codecov Report

Merging #205 (ee04a93) into master (e6acc8a) will increase coverage by 0.04%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master     #205      +/-   ##
==========================================
+ Coverage   94.24%   94.28%   +0.04%
==========================================
  Files          15       15
  Lines        1093     1101       +8
==========================================
+ Hits         1030     1038       +8
  Misses         63       63

Flag
Coverage Δ

unittests
94.28% <100.00%> (+0.04%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

open-ce/open-ce
85.19% <100.00%> (+2.58%)
: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 e6acc8a...ee04a93. Read the comment docs.

Codecov Report

Merging #205 (ee04a93) into master (e6acc8a) will increase coverage by 0.04%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master     #205      +/-   ##
==========================================
+ Coverage   94.24%   94.28%   +0.04%
==========================================
  Files          15       15
  Lines        1093     1101       +8
==========================================
+ Hits         1030     1038       +8
  Misses         63       63

Flag
Coverage Δ

unittests
94.28% <100.00%> (+0.04%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

open-ce/open-ce
85.19% <100.00%> (+2.58%)
: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 e6acc8a...ee04a93. Read the comment docs.

Codecov Report

Merging #205 (1e2f37c) into master (e6acc8a) will increase coverage by 0.04%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master     #205      +/-   ##
==========================================
+ Coverage   94.24%   94.28%   +0.04%
==========================================
  Files          15       15
  Lines        1093     1101       +8
==========================================
+ Hits         1030     1038       +8
  Misses         63       63

Flag
Coverage Δ

unittests
94.28% <100.00%> (+0.04%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

open-ce/open-ce
85.19% <100.00%> (+2.58%)
: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 e6acc8a...1e2f37c. Read the comment docs.

Codecov Report

Merging #205 (1e2f37c) into master (e6acc8a) will increase coverage by 0.04%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master     #205      +/-   ##
==========================================
+ Coverage   94.24%   94.28%   +0.04%
==========================================
  Files          15       15
  Lines        1093     1101       +8
==========================================
+ Hits         1030     1038       +8
  Misses         63       63

Flag
Coverage Δ

unittests
94.28% <100.00%> (+0.04%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

open-ce/open-ce
85.19% <100.00%> (+2.58%)
: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 e6acc8a...1e2f37c. Read the comment docs.


Issue with queryString.stringify(data)

I am getting the following error when calling Oauth.getAccessToken (data)
Type error:  Cannot read property 'stringify' of undefined.
Please note that I downloaded and included the dependency re query-string in my index.html

i am getting the same error

@vcmcinc my problem was because i didn't put the query_string on the gulpfile. After that, it works.

hi! i had the same error, is there some solution for that ??
Thanks !

What @gordinh said.

Sorry to bother you but I am running into that exact same issue.
Problem is that I don't get how you "put the query_string" on the gulpfile ???
@gordinh could you share a code snippet with us please ?

Ok after few trials I got what you meant : I had to include the query-string js file before the angular-oauth2 js file !!!! Hope this helps the next guy ;)

I found out my issue was that I was using import instead of const so I change this:
import {queryString} from 'query-string';
to this:
const queryString = require('query-string');

In my case, I did import { qs } from "qs;   The issue is 'named import' versus 'default export'.  In the qs library, Im guessing it is a default export so the answer is to remove the curly braces from the import statement.
import qs from "qs";


Italy Country Code and Length

Congratulations for the really good work you made.
italy code is +39 and not +41, also the maxLength is 10 and not 13. are you updating it in the next update?

I have the same issue, there are some news about the fixing?

I have the same issue, there are some news about the fixing?

⚠ The package is deprecated and this repository is no longer maintained ⚠
I found this on the readme, so i think nobody is going to change it.
I searched a new one and i found this on pub dev, which is intl_phone_number_input


Create musdb handler and tests

Create dataset handler and tests for new dataset. Manually tested and unit tested to test validity. Pre-commit ran for style checks.

accidental approve. please ignore.

@skim0514 great to have this in torchaudio! @nateanl @mthrok if you want me to review this, let me know!

@skim0514 great to have this in torchaudio! @nateanl @mthrok if you want me to review this, let me know!

Sure, please do. We appreciate your perspective as a practitioner.


Thrusters and gyros can flicker between on/off rapidly

Description

flickers between ticks looks weird as hell
Reproduction

take eshuttle
ghost
(?) profit idk if theres more steps but ive seen this a few times

Screenshots

Additional context

Yeah making it dynamically take power was a terrible idea, need to revert https://github.com/space-wizards/space-station-14/pull/26690


ci: Add workflow to build disk image

And upload to ghcr.io because for some reason this isn't working with quay.io.

The image failure is unrelated, see https://github.com/CentOS/centos-bootc-layered/pull/22

See an example successful run of this here: https://github.com/cgwalters/centos-boot-layered/actions/runs/7280872408/job/19840208917

To fetch the content, today this is technically possible with skopeo but has absolutely atrocious ergonomics:
$ skopeo copy docker://ghcr.io/cgwalters/fedora-bootc-cloud-disk:eln oci:disk
$ find disk/blobs/ -type f -size -5M | xargs -r rm
$ mv disk/blobs/* fedora-bootc-cloud-eln.qcow2.zst

After you'll want to decompress: zstd --rm -d fedora-bootc-cloud.qcow2.zst
Or...with oras:
$ oras pull ghcr.io/cgwalters/fedora-bootc-cloud-disk:eln

(Though you'll still want to decompress)
Also in all the above we'll s/cgwalters/centos-bootc/ - though I need to figure out what's blocking shipping to quay.io here, I thought they supported general artifacts.

Ah OK this is failing because the GH workflow in this repository I think doesn't have permission to create packages - cc @lmilbaum
I need to dig into why quay.io is still allow-listing only certain media types which is blocking this use case of managing disk images there.

@cgwalters GW workflow should be working now


Fixup content item discrepancies

Addresses two issues:

The @type for LtiLink was inconsistent with the spec.  Solution was changing the string in LtiConstants.  I can see an argument that this is a breaking change, and could just add a new property with the correct value, but it seems like this was the intended use of the current property.

JsonLdObjectConverter was not being properly invoked when the ToJsonLdString extension was used because of loop-preventing code in JsonLdObjectContractResolver.  Tweaked it so the creator of the contract resolver can choose whether or not to use that behavior.

Added a test for the extension method.
Refactored existing tests for JsonLdObject to use the extension method for validation.

@tscbp Thank  you so much for this pull request. You are right that is the intended use of @type.

@andyfmiller Thanks for maintaining the library!


What's main for " ERROR: NoneType" in trace print?

Ubuntu 16.04
python 3.5.2
sanic 0.4.1
vritualenv
# change work dir
os.chdir(os.path.dirname(os.path.realpath(__file__)))

app = Sanic(__name__)
app.static('/static', './static')

class DemoApi(HTTPMethodView):
    def get(self, request: Request):
        return json({'demo': 'success'})

app.add_route(DemoApi.as_view(), '/demo')

if __name__ == "__main__":
    app.run(host="<IP_ADDRESS>", port=88, debug=True)

When I request '/demo', it is work ok.
But after a time, console trace this:
2017-03-03 11:43:28,657: INFO: Goin' Fast @ http://<IP_ADDRESS>:88
2017-03-03 11:43:28,684: INFO: Starting worker [14417]
2017-03-03 11:44:32,536: ERROR: NoneType

The  "ERROR: NoneType" message occurs frequently, in debug and release mode.
What it's main and how to fix it?

I have trying to catch the error like this:
class SentryErrorHandler(ErrorHandler):
    def default(self, request, exception):
        if exception is None:
            return text('unknown error occurred')
        print(str(exception))
        return super().default(request, exception)

app = Sanic(__name__, error_handler=SentryErrorHandler())

I got the output print like this:
2017-03-03 11:51:32,811: INFO: Goin' Fast @ http://<IP_ADDRESS>:88
2017-03-03 11:51:32,838: INFO: Starting worker [14515]
Request Timeout
2017-03-03 11:53:54,071: ERROR: NoneType

Executing <TimerHandle cancelled None created at /env/sanic/lib/python3.5/site-packages/sanic/server.py:105> took 24.346 seconds

It seems connection timeout?

I reproduced this behavior.
If you run the code below and access by browser, the behavior will occur.
If keep-alive is enabled, connection_lost() will not be executed.
self._ timeout_handler.cancel() will not be executed.
If there is no new request, Request Timeout will be raised.
from sanic import Sanic
from sanic.response import json
from sanic.config import Config

Config.REQUEST_TIMEOUT = 3

app = Sanic()

@app.route("/")
async def test(request):
    return json({"hello": "world"})

if __name__ == "__main__":
    app.run(host="<IP_ADDRESS>", port=8000)

when query from one server to sanic, sanic print "ERROR: NoneType" and the server will print follow
Unsolicited response received on idle HTTP channel starting with "HTTP/1.1 408 Request Timeout\r\nConnection: close\r\nContent-Length: 22\r\nContent-Type: text/plain; charset=utf-8\r\n\r\nError: Request Timeout"; err=<nil>

how i can disable this message?

This message is output by this code.
If the Request Timeout Error Handler is set, the message does not occur.

thanks for reply.
I have tried
@app.exception(SanicException)
to catch all exception, but it not work.
Is it need to write like this?
@app.exception(NotFound, ServerError, RequestTimeout...)
Can I do it simple?

Hello sanic and thak you very much
2017-03-09 14:06 GMT+02:00 UltraGIS<EMAIL_ADDRESS>
thanks for reply.
I have tried
@app.exception(SanicException)
to catch all exception, but it not work.
Is it need to write like this?
@app.exception(NotFound, ServerError, RequestTimeout...)
Can I do it simple?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/channelcat/sanic/issues/511#issuecomment-285334438,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AXWG59yCQHEqWjOwHDNeWiQKaOU_-Ej6ks5rj-tggaJpZM4MR1WZ
.

I think custom error handler will solve.

Thanks very much.


CSS file that begins "@charset "UTF-8"; cause Uncaught TypeError

TypeError: buffer.string.length.substr is not a function     at Object.characterSet [as character set] (/Users/blanu/.atom/packages/css-clean/src/css-clean-src/capture/modules/characterSet.js:4:40)     at capture (/Users/blanu/.atom/packages/css-clean/src/css-clean-src/capture/capture.js:153:25)     at CleanCss.valueOf (/Users/blanu/.atom/packages/css-clean/src/css-clean-src/cleanCss.js:29:19)     at atom-text-editor.module.exports.convert (/Users/blanu/.atom/packages/css-clean/main.js:35:12)     at CommandRegistry.module.exports.CommandRegistry.handleCommandEvent (/Applications/Atom.app/Contents/Resources/app.asar/src/command-registry.js:260:29)     at /Applications/Atom.app/Contents/Resources/app.asar/src/command-registry.js:3:61     at KeymapManager.module.exports.KeymapManager.dispatchCommandEvent (/Applications/Atom.app/Contents/Resources/app.asar/node_modules/atom-keymap/lib/keymap-manager.js:580:16)     at KeymapManager.module.exports.KeymapManager.handleKeyboardEvent (/Applications/Atom.app/Contents/Resources/app.asar/node_modules/atom-keymap/lib/keymap-manager.js:388:22)     at WindowEventHandler.module.exports.WindowEventHandler.handleDocumentKeyEvent (/Applications/Atom.app/Contents/Resources/app.asar/src/window-event-handler.js:98:36)     at HTMLDocument.<anonymous> (/Applications/Atom.app/Contents/Resources/app.asar/src/window-event-handler.js:3:61)
[Enter steps to reproduce below:]

...
...


Design diagrams

Design specific diagrams namely: application flow diagram, database design, sequence diagram for visual representation of the programme

will he do it?


[relay][convert layout]: Infer correct layout for nn.batch_flatten

Currently during the ConvertLayout pass, whenever there is a nn.batch_flatten op, additional layout transforms are inserted before the nn.batch_flatten op. This may not be efficient.
In addition, for BYOC targets (which expects ops to have a certain layout), insertion of these transforms can create additional overhead - especially if they cannot be handled by target which then creates additional compiler regions.
This can be avoided by propagating the updated layouts to the nn.batch_flatten op.

Dropping as this can lead to issues with downstream ops such as dense layer whose weights assume original layout from the flatten layer.


Drafting and completing the WP3.5 (UKD-UKD-05) work plan

Expecting IPC team to contact in w/b 22/NOV with workplan form

Workplan form should now be completed in cooperation with recipient groups. The Work Plan form is available at: https://forms.gle/8QTHbZisz8DxTutm7

04/FEB/22

according to AprajitaV, this requires different workplan forms
AprajitaV will confirm when forms are ready and reporting can start

25/MAR/22

awaiting instructions from AprajitaV

06/MAY/22

Manda met Steve Ridgeway, AprajitaV, and KnutO to  discuss contribution.
Manda will now complete the workplan form and update the profile  explorer. If necessary, Manada to contact AprajitaV for write privileges to
the explorer.

27/MAY/22

workplan has been submitted
profile explorer not modified, waiting for feedback on the workplan

26/JUL/22

at meeting with @tms-epcc, @aprajita  said will remind Steve Ridgeway and Steve Marghein to do the IPC
check on S5

is done will send approval shortly
On Fri, 29 Jul 2022 at 11:19, Terry Sloan @.***> wrote:

26/JUL/22

at meeting with @tms-epcc https://github.com/tms-epcc, @aprajita
https://github.com/aprajita said will remind Steve Ridgeway and
Steve Marghein to do the IPC
check on S5

—
Reply to this email directly, view it on GitHub
https://github.com/lsst-uk/lsst-ir-fusion/issues/45#issuecomment-1199111582,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAOUIEV6DRLBXAFPL6CAGKTVWOV2RANCNFSM5GQGJENA
.
You are receiving this because you were mentioned.Message ID:
@.***>

07/SEP/22 : Contribution IPC (Steve Marghein) and Receipient reviewer (Knut Olsen) approved workplan with minor modifications i.e.  they  requested links to rep and notebooks.
12/SEP/22 : @raphaelshirley provided these.


ml-model-mapping is not in beta anymore

Please use the form below, leaving the prefilled data to help us. Thank you.
Page link: ml-model-mapping
Document link: ml-model-mapping.md
My Issue/Suggestion
Remove 'beta' warning as this has become GA with Mendix 10.0.

Thanks for flagging this, @dlhartveld. I have removed the alert from the doc.


"windows" call my function when click close btn

i know have "data-close-action" can call my function,
but i need not to empty the window,
because i used vue js
i gived v-if to show or hide.
i try to edit metro.min.js  :: "t.remove():e.hide()" -> "1:1"
but i think it's not good solutions.

i find i can used "Drag element" to do this.
i try it.

You can use the attribute onCloseClick to define the event for the close button.

thank you, i will try this.


IA-2411 Allow multiple reference Forms and reference Instances

Enable the possibility to have:

multiple reference Forms per OrgUnitType
multiple reference Instances per OrgUnit

Related JIRA tickets :

IA-2411
IA-2229 (Epic)

Self proofreading checklist

[x] Did I use eslint and black formatters
[x] Is my code clear enough and well documented
[x] Are my typescript files well typed
[x] New translations have been added or updated if new strings have been introduced in the frontend
[x] My migrations file are included
[x] Are there enough tests
[x] Documentation has been included (for new feature)

Changes

back-end

models

OrgUnitType

replace reference_form (FK) by reference_forms (M2M)
migrate DB data from reference_form to reference_forms

OrgUnit

replace reference_instance (FK) by reference_instances (M2M)
migrate DB data from reference_form to reference_forms

OrgUnitReferenceInstance

used as through table

Instance

add the logic to flag and unflag an instance as reference

APIs

mobile

adapt FormSerializer (per request)

new reference_form_of_org_unit_types field

adapt OrgUnitTypeSerializerV1 (nothing exposed as requested by the mobile app development team)

remove all reference* fields (unused by the mobile)

adapt MobileOrgUnitSerializer

new reference_instances field

web

adapt InstancesViewSet

list()and retrieve()

adapt OrgUnitTypeSerializerV2

add reference_forms and reference_forms_ids fields
new validate_reference_forms() method

adapt OrgUnitViewSet

this is what the front-end uses to flag an instance as reference
new logic in partial_update() based on an explicit code to trigger a flag or unflag action

adapt tests
adapt fixtures

front-end

allow multiple reference forms per OrgUnitType
allow multiple reference instances per OrgUnit
new logic to flag or unflag a reference instance (instance is called submission in the front-end)
new component to display and switch between multiple reference instancess
deleted OrgUnitActions.tsx and the SpeedDialsActionfrom the Org unit details page

replaced by a link to the Instance details page

adapt fixtures
adapt tests

How to test
Front-end (watch the video below for help):

Go to Org Units > Organisation unit type

edit a Org unit type
set some Reference forms

Go to Forms > Submissions

filter by Forms (choose those flagged above)
link some Forms to one Org unit to create Reference submissions

Go to Org unit details

you can see one or more Reference submissions and switch between them
each Reference submission has a link to its details

Video
https://github.com/BLSQ/iaso/assets/281139/a2ab7dad-6c53-4b46-a324-ca33d55ed204
Notes

Almost no test, is it on purpose?

@cypress

@cypress

As discussed with @kemar, I believe the goal was to be able to see the history of the reference instances. Therefore, the unicity on org_unit_id and form_id in the join table would be problematic.
@madewulf Should the code allow to see the history of the reference instances, or only seeing the current one is enough?

@cypress

I have a warning while editing reference form on an org unit type just:

@beygorghor thanx for the review! But I can't reproduce this. Can you give me the exact URL please?


traefik pending forever.

I'm using Virtualbox. It has a host-only adp in vboxnet0. It has eth1 = <IP_ADDRESS>
How do I config traefik right? also how do I access traefik UI?

Thanks!
Cheers.

kubectl patch svc traefik -n kube-system -p '{"spec": {"type": "LoadBalancer", "externalIPs":["<IP_ADDRESS>"]}}'


feat: add action v2 execution on requests and responses

Adds execution of target calls in GRPC and REST calls to specific endpoints.
Closes #7245
Definition of Ready

[x] I am happy with the code
[x] Short description of the feature/issue is added in the pr description
[x] PR is linked to the corresponding user story
[x] Acceptance criteria are met
[ ] All open todos and follow ups are defined in a new ticket and justified
[x] Deviations from the acceptance criteria and design are agreed with the PO and documented.
[x] No debug or dead code
[x] My code has no repetitions
[x] Critical parts are tested automatically
[x] Where possible E2E tests are implemented
[ ] Documentation/examples are up-to-date
[x] All non-functional requirements are met
[ ] Functionality of the acceptance criteria is checked manually on the dev system.

translation missing for Errors.action.NotEnabled

@adlerhurst Totally missed that you did more comments on the code which changed with the merged PR, only saw the last one, so will have a look at Monday.


Bad gateway when trying to get factors of Virahanka-Fibonacci numbers

Currently attempting to fetch https://numberscope.colorado.edu/api/get_oeis_factors/A000045/128 returns an error 502 "Bad Gateway". Note that the same requests for A000001 (number of groups) and A000040 (primes) return fine.
This is a blocker for https://github.com/orgs/numberscope/discussions/7

Hmm, nothing at all about A000045 in the api.log...

However, in /var/log/nginx/error.log there were numerous entries of the form
2024/07/20 14:01:12 [error] 1739#1739: *408 upstream prematurely closed connection while reading response
header from upstream, client: <IP_ADDRESS>, server: numberscope.colorado.edu, request: "GET /api/get_oeis
_factors/A000045/128 HTTP/1.1", upstream: "http://unix:/home/scope/repos/backscope/backscope.sock:/api/get
_oeis_factors/A000045/128", host: "numberscope.colorado.edu"

so that's probably some kind of clue.

And indeed, each attempt to load the offending URL from numberscope.colorado.edu results in another copy of such an error.log entry. And judging from https://stackoverflow.com/questions/36488688/nginx-upstream-prematurely-closed-connection-while-reading-response-header-from, the message seems to mean that the python process to produce the result is either crashing or returning nothing. So why isn't there anything in the backscope logs?

Oh, the journalctl command in the administration guide was wrong. It should be sudo journalctl -u numberscope (I will submit a PR to fix). And that produces lots of messages like:
[2024-07-20 14:15:43 -0600] [6776] [INFO] Worker exiting (pid: 67>
 Traceback (most recent call last):
   File "/home/scope/repos/backscope/.venv/bin/gunicorn", line 8, >
     sys.exit(run())
              ^^^^^
   File "/home/scope/repos/backscope/.venv/lib/python3.11/site-pac>
     WSGIApplication("%(prog)s [OPTIONS] [APP_MODULE]").run()
   File "/home/scope/repos/backscope/.venv/lib/python3.11/site-pac>
     super().run()
   File "/home/scope/repos/backscope/.venv/lib/python3.11/site-pac>
     Arbiter(self).run()
   File "/home/scope/repos/backscope/.venv/lib/python3.11/site-pac>
     self.manage_workers()
   File "/home/scope/repos/backscope/.venv/lib/python3.11/site-pac>
     self.spawn_workers()
   File "/home/scope/repos/backscope/.venv/lib/python3.11/site-pac>
     self.spawn_worker()
   File "/home/scope/repos/backscope/.venv/lib/python3.11/site-pac>
     worker.init_process()
   File "/home/scope/repos/backscope/.venv/lib/python3.11/site-pac>
     self.run()
   File "/home/scope/repos/backscope/.venv/lib/python3.11/site-pac>
     self.run_for_one(timeout)
   File "/home/scope/repos/backscope/.venv/lib/python3.11/site-pac>
    self.accept(listener)
   File "/home/scope/repos/backscope/.venv/lib/python3.11/site-pac>
     self.handle(listener, client, addr)
   File "/home/scope/repos/backscope/.venv/lib/python3.11/site-pac>
     self.handle_request(listener, req, client, addr)
   File "/home/scope/repos/backscope/.venv/lib/python3.11/site-pac>
     respiter = self.wsgi(environ, resp.start_response)
                ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
   File "/home/scope/repos/backscope/.venv/lib/python3.11/site-pac>
     return self.wsgi_app(environ, start_response)
            ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
   File "/home/scope/repos/backscope/.venv/lib/python3.11/site-pac>
     response = self.full_dispatch_request()
                ^^^^^^^^^^^^^^^^^^^^^^^^^^^^
   File "/home/scope/repos/backscope/.venv/lib/python3.11/site-pac>
     rv = self.dispatch_request()
          ^^^^^^^^^^^^^^^^^^^^^^^
   File "/home/scope/repos/backscope/.venv/lib/python3.11/site-pac>
     return self.ensure_sync(self.view_functions[rule.endpoint])(*>
            ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^>
   File "/home/scope/repos/backscope/flaskr/nscope/views.py", line>
     seq = fetch_factors(valid_oeis_id, wants)
   File "/home/scope/repos/backscope/flaskr/nscope/views.py", line>
   fac = gen_to_python(pari(val).factor())
                         ^^^^^^^^^^^^^^^^^^
   File "cypari2/gen.pyx", line 4363, in cypari2.gen.Gen.factor
cysignals.signals.SignalError: Illegal instruction

so this seems to be cypari acting up again... I will see if I can reproduce running locally.

Hmm, running on my machine, get_factors for A000045 had no problems, although starting at VF(290), it started reporting "no_fac" rather than a factorization, presumably because the numbers got so large.


[FEATURE] Item Breaking Mechanics

Items will not break, but rather be broken, which will be indicated via tooltip and also the item will be grayed out, making it unusable until it is repaired via the anvil.

If you have mending, this can also work by slowly repairing the item, thus making it usable over time. Might add repair kits to use on armor/tools as well.

Broken:
Armour still supplies armour points
TODO:
Tooltip
Textures
Compat


Multi Server?

Does this node libary support multiple server at the same time without hard coding these servers? (means reading server ip's from an config file)

You can pass in the options

Could I also say that it should only join a channel on an specific server?

options.channels


Godot 4.0rc1 Editor gets stuck when importing files

Godot version
4.0rc1
System information
Windows 11
Issue description
When importing a file by dragging it from the windows file explorer and dropping it on the editor filesystem, the editor gets stuck and has to be forcibly closed.

After restarting the resources are correctly imported.
Steps to reproduce
Drag and drop a png from the windows file explorer to the godot editor filesystem.
Minimal reproduction project
Happens on empty projects

According to #73125 this can crash (not surprising, a freeze/deadlock typically ends with a crash eventually due to stack overflow), and is a regression since beta 17.

Duplicate of https://github.com/godotengine/godot/issues/53871.


.Net: Access to Chat history in semantic kernel function

Discussed in https://github.com/microsoft/semantic-kernel/discussions/7186

Originally posted by JP-droidic July 10, 2024
Hi all
is it possible to access/retrieve chat history in autocalled SK function?

This works in python but not .Net

More information about this task:

The functionality is not supported in Python yet.
Could be implemented in different ways (e.g. by adding ChatHistory to KernelArguments, by extending InvokeAsync methods with new ChatHistory parameter etc.)
ADR is required to investigate and find the best possible approach.
The solution could potentially solve the problem with lack of ChatHistory in Kernel object, which can remove a need to use IChatCompletionService directly for most cases.

Is there any update on this work @markwallace-microsoft? This would be a great feature to have.

@gdodd1977 We will try to get this done during our current sprint which starts today


Update to swift 403

Update to swift403

Codecov Report

Merging #236 into master will decrease coverage by 0.58%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #236      +/-   ##
==========================================
- Coverage   77.72%   77.13%   -0.59%
==========================================
  Files          47       46       -1
  Lines        6939     6831     -108
  Branches      568      560       -8
==========================================
- Hits         5393     5269     -124
+ Misses       1432     1429       -3
- Partials      114      133      +19

Flag
Coverage Δ

#CHTTPParser
77.13% <ø> (-0.38%)
:arrow_down:

#KituraNet
77.13% <ø> (-0.59%)
:arrow_down:

Impacted Files
Coverage Δ

Sources/KituraNet/HTTP/HTTPServerRequest.swift
72.41% <0%> (-4.32%)
:arrow_down:

Sources/KituraNet/IncomingSocketHandler.swift
80.65% <0%> (-3.65%)
:arrow_down:

Sources/KituraNet/HTTP/HTTPServer.swift
78.57% <0%> (-3.05%)
:arrow_down:

Sources/KituraNet/HTTPParser/URLParser.swift
97.33% <0%> (-2.67%)
:arrow_down:

Sources/CHTTPParser/http_parser.c
55.08% <0%> (-2.48%)
:arrow_down:

Sources/KituraNet/IncomingSocketManager.swift
85.5% <0%> (-1.45%)
:arrow_down:

Sources/KituraNet/HTTPParser/HTTPParser.swift
99.02% <0%> (-0.98%)
:arrow_down:

Sources/KituraNet/FastCGI/FastCGIServer.swift
63.58% <0%> (-0.62%)
:arrow_down:

...s/KituraNet/HTTP/IncomingHTTPSocketProcessor.swift
84.75% <0%> (-0.61%)
:arrow_down:

...urces/KituraNet/FastCGI/FastCGIServerRequest.swift
80.66% <0%> (-0.56%)
:arrow_down:

... and 1 more

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 fc6e217...417ce5d. Read the comment docs.


chore(sequencer_node): add node compilation

This change is

crates/mempool_node/src/test_utils/compilation.rs line 7 at r4 (raw file):

#[cfg(test)]
#[path = "compilation_test.rs"]

no longer needed.
Code quote:
#[cfg(test)]
#[path = "compilation_test.rs"]


Upgraded from 1.6.6 to 1.7.0 caused ECS providers to be retrieved

Bug
What did you do?
Upgraded from 1.6.6 to 1.7.0, and it is causing an exception when polling the ECS API.
to be thorough i tried every single minor version 1.7.0 -> 1.7.20. No solution yet
What did you expect to see?
I expected to see my ecs providers display
What did you see instead?

Output of traefik version: 1.7.0-alpine
What is your environment & configuration (arguments, toml, provider, platform, ...)?
[entryPoints]
  [entryPoints.http]
  address = ":80"

  [entryPoints.traefik]
  address = ":8080"
  [entryPoints.traefik.auth.basic]
  users = ["credentials"]

[ping]
[api]
  dashboard = true
  [api.statistics]
    recentErrors = 10

logLevel = "INFO"

[traefikLog]
  format   = "json"

[ecs]
clusters = ["development", "development2"]
watch = true
domain = "example.com"
autoDiscoverClusters = false
refreshSeconds = 15
exposedByDefault = false
region = "us-east-2"

If applicable, please paste the log output in DEBUG level (--log.level=DEBUG switch)

{"level":"error","msg":"Error in Go routine: runtime error: invalid memory address or nil pointer dereference","time":"2019-12-18T20:26:19Z"}
{"level":"error","msg":"Stack: goroutine 70 [running]:\nruntime/debug.Stack(0x26cb820, 0x17, 0xc0012ee0f0)\n\t/usr/local/go/src/runtime/debug/stack.go:24 +0xa7\ngithub.com/containous/traefik/safe.defaultRecoverGoroutine(0x2246600, 0x4603250)\n\t/go/src/github.com/containous/traefik/safe/routine.go:148 +0x89\ngithub.com/containous/traefik/safe.OperationWithRecover.func1.1(0xc00101be18)\n\t/go/src/github.com/containous/traefik/safe/routine.go:156 +0x56\npanic(0x2246600, 0x4603250)\n\t/usr/local/go/src/runtime/panic.go:513 +0x1b9\ngithub.com/containous/traefik/provider/ecs.(*Provider).listInstances(0xc0004432b0, 0x29eaee0, 0xc000087400, 0xc000423330, 0x0, 0x0, 0x0, 0x0, 0x0)\n\t/go/src/github.com/containous/traefik/provider/ecs/ecs.go:322 +0xe62\ngithub.com/containous/traefik/provider/ecs.(*Provider).loadECSConfig(0xc0004432b0, 0x29eaee0, 0xc000087400, 0xc000423330, 0x2030e80, 0x203000, 0x203000)\n\t/go/src/github.com/containous/traefik/provider/ecs/ecs.go:426 +0x4d\ngithub.com/containous/traefik/provider/ecs.(*Provider).Provide.func2.2(0x0, 0x0)\n\t/go/src/github.com/containous/traefik/provider/ecs/ecs.go:144 +0xb7\ngithub.com/containous/traefik/safe.OperationWithRecover.func1(0x0, 0x0)\n\t/go/src/github.com/containous/traefik/safe/routine.go:160 +0x62\ngithub.com/containous/traefik/vendor/github.com/cenk/backoff.RetryNotify(0xc0004231d0, 0x29c9760, 0xc0004231f0, 0x2783850, 0xc0004231c0, 0xc0008b9980)\n\t/go/src/github.com/containous/traefik/vendor/github.com/cenk/backoff/retry.go:37 +0xa2\ngithub.com/containous/traefik/provider/ecs.(*Provider).Provide.func2(0xc0000884d0)\n\t/go/src/github.com/containous/traefik/provider/ecs/ecs.go:181 +0x1c0\ngithub.com/containous/traefik/safe.(*Pool).Go.func1()\n\t/go/src/github.com/containous/traefik/safe/routine.go:78 +0x37\ngithub.com/containous/traefik/safe.GoWithRecover.func1(0x2783a68, 0xc000036660)\n\t/go/src/github.com/containous/traefik/safe/routine.go:142 +0x4d\ncreated by github.com/containous/traefik/safe.GoWithRecover\n\t/go/src/github.com/containous/traefik/safe/routine.go:136 +0x49\n","time":"2019-12-18T20:26:19Z"}
{"level":"error","msg":"Provider connection error panic in operation: %!s(\u003cnil\u003e), retrying in 543.930941ms","time":"2019-12-18T20:26:19Z"}

Stack: goroutine 70 [running]:
runtime/debug.Stack(0x26cb820, 0x17, 0xc0012ee0f0)
	/usr/local/go/src/runtime/debug/stack.go:24 +0xa7
github.com/containous/traefik/safe.defaultRecoverGoroutine(0x2246600, 0x4603250)
	/go/src/github.com/containous/traefik/safe/routine.go:148 +0x89
github.com/containous/traefik/safe.OperationWithRecover.func1.1(0xc00101be18)
	/go/src/github.com/containous/traefik/safe/routine.go:156 +0x56
panic(0x2246600, 0x4603250)
	/usr/local/go/src/runtime/panic.go:513 +0x1b9
github.com/containous/traefik/provider/ecs.(*Provider).listInstances(0xc0004432b0, 0x29eaee0, 0xc000087400, 0xc000423330, 0x0, 0x0, 0x0, 0x0, 0x0)
	/go/src/github.com/containous/traefik/provider/ecs/ecs.go:322 +0xe62
github.com/containous/traefik/provider/ecs.(*Provider).loadECSConfig(0xc0004432b0, 0x29eaee0, 0xc000087400, 0xc000423330, 0x2030e80, 0x203000, 0x203000)
	/go/src/github.com/containous/traefik/provider/ecs/ecs.go:426 +0x4d
github.com/containous/traefik/provider/ecs.(*Provider).Provide.func2.2(0x0, 0x0)
	/go/src/github.com/containous/traefik/provider/ecs/ecs.go:144 +0xb7
github.com/containous/traefik/safe.OperationWithRecover.func1(0x0, 0x0)
	/go/src/github.com/containous/traefik/safe/routine.go:160 +0x62
github.com/containous/traefik/vendor/github.com/cenk/backoff.RetryNotify(0xc0004231d0, 0x29c9760, 0xc0004231f0, 0x2783850, 0xc0004231c0, 0xc0008b9980)
	/go/src/github.com/containous/traefik/vendor/github.com/cenk/backoff/retry.go:37 +0xa2
github.com/containous/traefik/provider/ecs.(*Provider).Provide.func2(0xc0000884d0)
	/go/src/github.com/containous/traefik/provider/ecs/ecs.go:181 +0x1c0
github.com/containous/traefik/safe.(*Pool).Go.func1()
	/go/src/github.com/containous/traefik/safe/routine.go:78 +0x37
github.com/containous/traefik/safe.GoWithRecover.func1(0x2783a68, 0xc000036660)
	/go/src/github.com/containous/traefik/safe/routine.go:142 +0x4d
created by github.com/containous/traefik/safe.GoWithRecover
	/go/src/github.com/containous/traefik/safe/routine.go:136 +0x49

@kave, the error in your log is referencing this line:
https://github.com/containous/traefik/blob/v1.7/provider/ecs/ecs.go#L322
privateIP: aws.StringValue(containerInstance.PrivateIpAddress),
Does one of your containerInstances not have a privateIP address?

@dtomcej you are absolutely right. i forgot to mention in the original issue post that we were using traefik to discover frontend/backends from 2 clusters. It was the 2nd cluster that was misconfigured and had some containers in a restart cycles. I removed the bad cluster and everything worked. i'll focus on the aws configurations now instead of traefik.
Thanks!


Bad credentials

Visual Studio Code Version : 1.10.2
Code Settings Sync Version : 2.6.2
Operating System : Win10
Occurs On: Upload / Download
Proxy Enabled: No
GIst Id: 6ee4c45164d90ea69858e4d54b31358c
One of my VS Code instance is giving me a "Bad credentials" error. I wasn't able to find any documentation regards this. It just says that I need to reset my token but, I'm not sure why would I need to do that when the rest of my instances are working as expected.
Thank you,

You faced this error after update ?
For now, try this and post comment https://github.com/shanalikhan/code-settings-sync/issues/177

Yes, I do.
John Merced
*General Disclaimer: *The information contained in this electronic
communication is to be considered confidential and intended only for the
use of the recipient named above. The information is or may be legally
privileged and expresses the opinion of the writer only. If the reader of
this message is not the intended recipient, you are hereby notified that
any dissemination, distribution, or copying of this communication, or any
of its contents, is strictly prohibited. If you have received this
communication in error, please re-send this communication to the sender
listed above, delete the original message and any copy of it from your
computer system.
On Wed, Apr 5, 2017 at 11:14 AM, Shan Khan<EMAIL_ADDRESS>wrote:

You faced this error after update ?
For now, try this and post comment #177
https://github.com/shanalikhan/code-settings-sync/issues/177
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/shanalikhan/code-settings-sync/issues/280#issuecomment-291893902,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAMRZq8pFtu92j1lS3yxekVB58CDOeFFks5rs6-_gaJpZM4M0VZy
.

@shanalikhan I updated my gist but I'm still facing the same issue.

People have solved this error using the link i have given above. see this
https://github.com/shanalikhan/code-settings-sync/issues/293


pagination doesn't work

Hi, I try to use the pagination directive and on the official plnkr demo we can see that directive seems not to be working at all. Why ?

The official demo works fine. It doesn't have the current page listed there - @Foxandxss have any ideas how we should spruce this up so it's more obvious what is happening? Should we add something like The current page is: {{currentPage}}?

That is the type of thing I never noticed. Yeah, that plunker and demo is horrible. Let me thing about that.

hmm ok, but why the page number buttons are not generated ?

There is a pagination control and a pager control. You are looking at the pager and I guess you want the pagination.

you're right, I didn't notice that. Now it's fine, thanks !

Keeping this issue open since we do need to update the pager docs

here's a quick crack at it:
http://plnkr.co/edit/mZbFVPI6DFDy4NdWlG1m?p=preview
I can create a PR if it looks good to you guys