Credentials are not returned when trying to login, app gets stuck

Checklist

[X] The issue can be reproduced in the auth0_flutter sample app (or N/A).
[X] I have looked into the Readme, Examples, and FAQ and have not found a suitable solution or answer.
[X] I have looked into the API documentation and have not found a suitable solution or answer.
[X] I have searched the issues and have not found a suitable solution or answer.
[X] I have searched the Auth0 Community forums and have not found a suitable solution or answer.
[X] I agree to the terms within the Auth0 Code of Conduct.

Description
After logging in with auth0, the credentials cannot be retrieved on Android or the Web (Works for iOS). The warning message, No stored state: unable to handle response appears on the console when the webpage gets stuck and you have to kill the app and relaunch the app to try logging in again.
Initially, I was using the flutter_appauth package for logging in using Auth0 and everything was working fine. Now, I am trying to migrate it to the official Auth0 package i.e. auth0_flutter and it is now working for Android and Web.
Reproduction

Login function below:

Future<Credentials?> login() async {
    try {
      if (kIsWeb) {
        /// Attempt to login the user with the redirect method
          await _auth0Web.loginWithRedirect(
              audience: auth0Audience,
              scopes: authScopes['auth'],
              redirectUrl: auth0RedirectUri);

         return await _auth0Web.credentials();
      } else {
        /// Handle the needed credentials for mobile
        Credentials? credentials = await _auth0
            .webAuthentication(scheme: 'com.project.portal')
            .login(
              audience: auth0Audience,
              scopes: const {'openid', 'profile', 'email', 'offline_access'},
              redirectUrl: auth0RedirectUri,
              idTokenValidationConfig: IdTokenValidationConfig(issuer: 'https://$auth0Domain'),
              safariViewController: const SafariViewController(),
              parameters: {'prompt': 'login'},
            );

        return credentials;
      }
    } on Exception catch (e) {
      print('Error logging in: $e');
      return null;
    }
  }

Call above login function on a button click:

ElevatedButton(
  onPressed: () async {
    final credentials = await login();

    setState(() {
      _credentials = credentials;
    });
  },
  child: const Text('Login'),
),

Additional context
On Auth0, I have first created a new Native Application.
I have created a flutter application that supports iOS, Android, and the Web which uses the same configuration from Auth0.
The auth0 domain is a custom domain

Flutter Doctor Output
Doctor summary (to see all details, run flutter doctor -v):
[✓] Flutter (Channel stable, 3.13.9, on macOS 14.1.1 23B81 darwin-x64, locale en-IN)
[✓] Android toolchain - develop for Android devices (Android SDK version 34.0.0-rc2)
[✓] Xcode - develop for iOS and macOS (Xcode 15.0.1)
[✓] Chrome - develop for the web
[✓] Android Studio (version 2022.3)
[✓] VS Code (version 1.85.0)
[✓] Connected device (3 available)
[✓] Network resources

• No issues found!

auth0_flutter version
1.3.1
Flutter version
3.13.9
Platform
Android, Web
Platform version(s)
No response

Hi @aakashawalkar, thanks for raising this.
Can you confirm that you can indeed reproduce this on the auth0_flutter sample app?

Yes, I see the same behavior on the sample app.

Thanks. @poovamraj could you please take a look?

@Widcket @poovamraj -
When I tried tweaking the Auth0 configuration most of the time I saw the below error message
No stored state - unable to handle the response

Also, sometimes, I see below error messages while logging in -
E/flutter (26530): [ERROR:flutter/runtime/dart_vm_initializer.cc(41)] Unhandled Exception: a0.sdk.internal_error.unknown: Received error with code a0.sdk.internal_error.unknown
E/flutter (26530): #0      MethodChannelAuth0FlutterWebAuth.invokeRequest (package:auth0_flutter_platform_interface/src/method_channel_auth0_flutter_web_auth.dart:45:7)
E/flutter (26530): <asynchronous suspension>
E/flutter (26530): #1      MethodChannelAuth0FlutterWebAuth.login (package:auth0_flutter_platform_interface/src/method_channel_auth0_flutter_web_auth.dart:21:9)
E/flutter (26530): <asynchronous suspension>
E/flutter (26530): #2      WebAuthentication.login (package:auth0_flutter/src/mobile/web_authentication.dart:82:25)
E/flutter (26530): <asynchronous suspension>

Has any solution been found for this issue? I am in the same situation. Trying to migrate away from flutter_appauth, and it works in iOS but not in Android.

Any workarounds to fix this?


[MINOR][DOCS] Document default fractional numeric literals in SQL

What changes were proposed in this pull request?
Explain what type literals like 123.456 and 123.456E0 have in SQL.
Why are the changes needed?
In Python (and I think Java too) fractional numeric literals are typically floats. To get decimals, you need to provide an explicit postfix or use an explicit class. In Spark, it's the other way around. I found this surprising and couldn't find documentation about it.
I discovered this after reading SPARK-45786. I did a little searching and came across https://github.com/apache/spark/pull/10796, which shows that we used to default to floats as the fractional numeric literal, but then switched to decimals.
There is an additional wrinkle I discovered in #45003. If the fractional literal has an exponent, then it's a double, not a decimal.
None of this is obvious from the docs, at least not to me.
Does this PR introduce any user-facing change?
Yes, it clarifies the user-facing documentation about fractional numeric literals.
How was this patch tested?
No testing.
Was this patch authored or co-authored using generative AI tooling?
No.

cc @yaooqinn @HyukjinKwon

Thanks, merged to master


Proposal for Set-Based Delete Operation

One of the drawbacks of EF has always been the ability to perform set-based deletes on a foreign key or other criteria. E.g. to perform the following SQL query operation via EF, a fetch operation needs to be run to return EF entities, which are then marked for deletion in the DbContext and then submitted to the repository as a batch. Alternatively, SQL can be executed directly against the database, or a stored procedure can be called. Each of these options is less than ideal though when working with EF.
delete
	f
from
	dbo.FaqCategory c
inner join
	dbo.Faq f
		on c.FaqCategoryId = f.FaqCategoryId
where
	c.Category like '%test%'

// current EF batch delete process
var query = from c in DbContext.FaqCategories
            join f in DbContext.Faqs
                on c.FaqCategoryId equals f.FaqCategoryId
            select f;

IEnumerable<Faq> faqsToDelete = query.AsEnumerable();

DbContext.RemoveRange(faqsToDelete);
DbContext.SaveChanges();

I'd like to propose proper support for set-based deletes to generate the SQL listed above (or equivalent) and send it directly to the database.
Possibly by overloading the RemoveRange method to allow for an IQueryable<TEntity> parameter to be passed in, where instead of generating a select statement, a delete statement is generated.
// possible RemoveRange overload example
public void RemoveRange<TEntity>(IQueryable<TEntity>) {...}

var query = from c in DbContext.FaqCategories
            join f in DbContext.Faqs
                on c.FaqCategoryId equals f.FaqCategoryId
            select f;

DbContext.RemoveRange(query); // or DbContext.Faqs.RemoveRange(query);
DbContext.SaveChanges();

I know there are some other issues that would need to be worked out, like the unit of work scope and tracking for any existing objects that could get out of sync. Perhaps set-based operations would be single unit of work operations only, or stop tracking all entities in the context, or invalidate the entire context requiring a new instance for further operations, or return the keys for all deleted entities to locally mark them as deleted. I think having a real set-based feature would outweigh many of the potential limitations it might cause for the state of a DbContext.
If set-based delete is feasible at all, it would also be great to have a set-based update feature, although there would obviously be additional issues to work out regarding setting values.

@brent-russell thanks for submitting your proposal. We have discussed similar ideas in the past, but the idea of RemoveRange() overload that takes a query is very interesting (Incidentally I have talked to customers in the past that simply expected things to work this if the argument passed to RemoveRange() was an EF query :smile:).
Just nitpicking on terminology, we usually refer to this feature as "set-based" (deletes or updates) as opposed to "batch" (or batching). The latter is more about the ability to pack multiple CUD statements in one round trip to the database, and we actually do have support for that in EF7.
BTW, I also agree that deletes are possibly the easiest thing to implement. We don't have plans to add this feature in the short term but I think when we decide to tackle it, we should try to deliver set-based deletes first.
Re the relationship between this and our unit of work, personally I think it would be ok if the were kept separate, e.g. the same way no-tracking queries can be executed from a DbContext but don't affect the state of entities in the context.

@divega yes, sorry, my language-brain-part did not translate my conceptual-brain-part information correctly. Definitely looking for a single set-based query as opposed to a SQL batch. I'll update the issue to avoid confusion.
Running it as a no-tracking operation seems great to me. In most cases it would likely be the first, only, or last operation run on those particular entities in a unit of work, so tracking wouldn't be required. It might be good to give it a different method name though to avoid confusion with the standard "remove" behavior.

BTW this is already tracked by https://github.com/aspnet/EntityFramework/issues/795

Thanks. I was sure we had I but couldn't find it. Closing as dupe.
@brent-russell do you intend to contribute this?

@divega I would like to, and had a look through the code this weekend, but realistically won't have time for it until probably Feb-Mar 2016.


Different predictions between versions 0.4 and .6

First of all thank you for the awesome package!!!
A multi-class classification model trained in R package version 0.4;
predictions using the model differ between versions 0.4 and 0.6
only if one of the features equals to zero (0.0) and the feature is zero
for at least one class.
Install version 0.4-4
library(devtools)
install_version("xgboost", "0.4-4")

Example is a modification of a test in the R package
library(xgboost)

## Make one feature == 0 for all samples in a class
iris[1:50, 1] <- 0.0

lb <- as.numeric(iris$Species) - 1
bst <- xgboost(data = as.matrix(iris[, -5]), label = lb,
               max_depth = 3, eta = 0.5, nthread = 2, nrounds = 5,
               objective = "multi:softprob", num_class = 3)

pred_v04 <- predict(bst, as.matrix(iris[, -5]))

## Save model and predictions result for version 0.4
save(bst, pred_v04, file = "IRIS_pred_and_model_v04.RData")

In new session install version 0.6
library(devtools)
install_version("xgboost", "0.6-4")

Get model and predictions for version 0.4
library(xgboost)
load("IRIS_pred_and_model_v04.RData")
iris[1:50, 1] <- 0.0

Re-predict using latest version and compare results
pred_v06 <- predict(bst, as.matrix(iris[, -5]))
all.equal(pred_v04, pred_v06)
[1] "Mean relative difference: 1.809"

Setting only some of the feature values for the class to zero does not cause
difference. (i.e.  iris[1:5, 1] <- 0.0 does nothing).

Setting a feature to a non-zero value for a class does not cause difference either
(i.e. iris[1:50, 1] <- 0.1 does nothing)

SessionInfo()

R version 3.3.1 (2016-06-21)
Platform: x86_64-pc-linux-gnu (64-bit)
Running under: Ubuntu 15.10

locale:
 [1] LC_CTYPE=en_AU.UTF-8       LC_NUMERIC=C
 [3] LC_TIME=en_AU.UTF-8        LC_COLLATE=en_AU.UTF-8
 [5] LC_MONETARY=en_AU.UTF-8    LC_MESSAGES=en_AU.UTF-8
 [7] LC_PAPER=en_AU.UTF-8       LC_NAME=C
 [9] LC_ADDRESS=C               LC_TELEPHONE=C
[11] LC_MEASUREMENT=en_AU.UTF-8 LC_IDENTIFICATION=C

attached base packages:
[1] stats     graphics  grDevices utils     datasets  methods   base

other attached packages:
[1] xgboost_0.6-4

loaded via a namespace (and not attached):
[1] magrittr_1.5      Matrix_1.2-7.1    tools_3.3.1       stringi_1.1.2
[5] grid_3.3.1        data.table_1.10.0 lattice_0.20-34

There is a specification change that changes the missing value spec to NA in 0.47 https://github.com/dmlc/xgboost/blob/master/NEWS.md#v047-20160114
The different behavior was due to this, to recover old behavior, manually set missing=0 during DMatrix construction

Thank you!
Yay, you are right missing = 0 fixes it.
I've came across this while using caret wrapper with a model trained with xgboost v0.4-4, not sure if the current CRAN version got it but just in case here is the commit which switches to using NA as missing values
https://github.com/topepo/caret/commit/c060b14076ee7db5b70e05ca32edaeb96d74e7c8
K.


Use solc --base-path to project root by default

currently, source files in the compilerinput are canonic, this can cause some issues with metadata/hashes.
in #1307 we already save the artifacts with relative files, this should be obsolete if we set base-path == root and strip the root from all sources

there's actually no support for setting the basePath in the compilerinput, instead we should be able to get the same behaviour by setting current dir when invoking solc to root and stripping the root from the compilerinput sources


laser_geometry: 2.1.0-2 in 'eloquent/distribution.yaml' [bloom]

Increasing version of package(s) in repository laser_geometry to 2.1.0-2:

upstream repository: https://github.com/ros-perception/laser_geometry.git
release repository: https://github.com/ros2-gbp/laser_geometry-release.git
distro file: eloquent/distribution.yaml
bloom version: 0.9.1
previous version for package: 2.1.0-1

laser_geometry
* Merge pull request #46 <https://github.com/ros-perception/laser_geometry/issues/46> from sloretz/eigen3_cmake_module
* Contributors: Jonathan Binney, Shane Loretz

@jonbinney From the linked issue I see 2.1.1 as the latest release in the eloquent branch but this is 2.0.1-2.

Interesting - I'll look into this tomorrow morning.

Huh... I'm not sure what happened. I used catkin_prepare release - it bumped the version number in the package.xml to 2.1.1 and pushed the tag 2.1.1 in the eloquent branch. Then I ran bloom-release --rosdistro eloquent --track eloquent laser_geometry and it created the release and this PR. @wjwwood is there something else required?

It usually means that your devel branch is setup wrong in the ros2-gbp repository.  Indeed, if you look at https://github.com/ros2-gbp/laser_geometry-release/blob/master/tracks.yaml#L87, it points to the ros2 branch, not the eloquent branch.  If you do a PR to the release repository to change that to the eloquent branch, merge that, then do bloom-release again, it should do what you expect.

Thanks @clalancette! I'll try that.

Alternatively you can run bloom with --edit-track to be reprompted for the contents of the track. I'm going to close this and expect a new PR from the rerun of bloom.


Install JDK7 and 8 in one playbook

- { role: williamyeh.oracle-java,
    java_version: 7,
    java_subversion: 80,
    java_set_javahome: true,
    java_install_jce: true }

- { role: williamyeh.oracle-java,
    java_version: 8,
    java_subversion: 92,
    java_install_jce: true }

For the second role, the error part is:
TASK [williamyeh.oracle-java : compose filename, if necessary] *****************
task path: /etc/ansible/roles/williamyeh.oracle-java/tasks/set-role-variables.yml:111
skipping: [<IP_ADDRESS>] => {"changed": false, "skip_reason": "Conditional check failed", "skipped": true}
TASK [williamyeh.oracle-java : compose url for downloading file, if necessary] *
task path: /etc/ansible/roles/williamyeh.oracle-java/tasks/set-role-variables.yml:116
ok: [<IP_ADDRESS>] => {"ansible_facts": {"jdk_tarball_url": "http://download.oracle.com/otn-pub/java/jdk/8u92-b14/jdk-7u80-linux-x64"}, "changed": false}
It looks jdk_tarball_file is not cleared after first role execution. The second role finds jdk_tarball_file still exists.

This role doesn't consider the situation that two or more JDK/JRE versions co-exist within the same managed node. In fact, I'm also against such use case, even for non-Java runtime such as rvm (for Ruby), virtualenv (for Python), and nvm (for Node.js).


chore: automated PR to update ReleaseNotes for 202307.04.0 VHD

This is an automated PR to bump ReleaseNotes for the VHD release with image version 202307.04.0

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall First build on releaseNotes/202307.04.0 at 61.489%

Totals

Change from base Build<PHONE_NUMBER>:
61.5%

Covered Lines:
2561

Relevant Lines:
4165

💛  - Coveralls


Website updates proposal

In order to refocus our messaging around source{d} Engine, I propose the following changes on the website

[ ] New header navigation menu

Why source{d}? [New]

source{d} Engine

Resources [new]

Blog
Forum
Docs (Move under Resources)
Playground
Codebase Analysis (Kubernetes, Tensorflow, etc)

Community

Open Source (Moved under community)

Company

[ ] Take source{d} Lookout page down for now and move it under prrojects on the Open Source page

On  https://sourced.tech/engine/

[ ] Make it clear that the card with the questions are clickable to access the actual queries

[ ] Add logo of languages that are supported

@src-d/applications just to avoid confusion I'm closing this now and moving it to https://github.com/src-d/feature-idea/issues/141. Since some of these are major things we're drafting this first and will go through the regular feature-idea -> backlog -> tasks workflow. We'll be still working with Zuri on this.


Authenticating with contents of service account key rather than filepath?

Use Case
We're running some tooling in a CI pipeline that passes in the JSON contents of a service account key as an environment variable. We then need a separate step to write the JSON contents into a file before newing up the client with WithCredentialsFile. Is it currently possible to pass the JSON contents directly to the go client? If not, would y'all accept a PR to that adds a WithCredentialsJSON or similar to take the JSON contents directly?

We do something similar here, except we read a file.


Crash on Windows with version 23 and later

Since version 23, even a minimal electron application cannot be run. Tested on Windows x64 on electron-quick-start with modifications in your documentation. Everything works up to the latest version 22. Since version 23 the application quits immediately. Everything works fine on macOS x64.

This is fixed in v24.0.0-alpha.6+wvcus.2 and v23.1.1+wvcus.2 respectively. Let me know if there are any further issues.

Thank you for the quick response. I can confirm that the binaries are working again on Windows, so we can close this issue now. Thanks again :)


As an operator I want to set my promotion type to Shipping Discount

[ ] User can select Shipping Discount in the Promotion Type dropdown
[ ] When User has selected Shipping Discount as the Promotion Type, the Action Menu area of the promo builder updates to show:
- [ ] Eligible Shipping Methods header
- [ ] Label above Shipping Method field is "Include the following shipping methods"
- [ ] Shipping Method field is empty

Mockup: Promotions / Detail Page
Promotion Type Dropdown https://www.figma.com/file/vWot2cXXgjcsLpMq4SAX90/Kinetic?node-id=2082%3A47363&t=sFagdUnSMbq5ZIJn-0

Action Menu

I think for matching Strings for Shipping Methods we might want add the ability to provide wildcards or regex? So that for dynamic shipping methods (e.g. UPS, FedEx, Shippo) if the provider makes a small change it won't break the promotion?

@zenweasel I think you're comment is more relevant to https://github.com/reactioncommerce/kinetic/issues/179 Given those requirements do you think we're covered on this topic? If a user can see a list of Shipping Methods for the store when they click into the Shipping Methods field then I think we should be ol

Closed via #195


ZH-trans: add notes.md

^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
Please delete this note before submitting the pull request.
For 1.14 Features: set Milestone to 1.14 and Base Branch to dev-1.14
For Chinese localization, base branch to release-1.12
For Korean Localization: set Base Branch to dev-1.13-ko.
Help editing and submitting pull requests:
https://kubernetes.io/docs/contribute/start/#improve-existing-content.
Help choosing which branch to use:
https://kubernetes.io/docs/contribute/start#choose-which-git-branch-to-use.
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^

Ref: https://github.com/kubernetes/website/blob/release-1.12/content/en/docs/setup/release/notes.md
因为这个文件比较大，所以我暂时只翻译一部分同时也方便 review。另外会在两天内更新完毕剩下的部分，建议更新这个 pr 时，同时进行 review，不然积累太多 review 会造成困扰。

/cc @chenrui333  @tengqm
/hold

/cc @chenrui333 @tengqm

[trans-bot:N/A]
Thank you @xichengliudui, the related task issue has no assignee. will not be reported to the task issues

@tengqm @AdamDang ping. :)

/approve cancel
I'd rather do a complete review after the translation is ready. Or else I cannot recall where I reviewed last time.

sorry, my mistake, I thought the translation was complete. Did not realize it just reached to line ~900 😄

/approve cancel
/lgtm cancel

/cc @SataQiu

@xichengliudui 预览格式有些问题..
https://deploy-preview-13278--k8s-v1-12.netlify.com/zh/docs/setup/release/notes/

@xichengliudui ping
this PR's base branch is release-1.12, do we want to switch to release-1.14?

Close this pr and create a new pr based on 1.14.


fix: block input send event while ime composition

Before the fix, when users activated an input method (such as Chinese, Japanese, Korean, etc.) and entered composition mode, pressing the Enter key would trigger the onKeyPress method in the Input component and ultimately send a message, which was unintended.

After the fix, when users are in composition mode (indicating that they have not finished typing and the input method is still in use), pressing Enter should select the current candidate word (or execute other features of the input method) without triggering the message sending logic of the input component.

Additionally, some optimization work was done, such as:

Extracting the input component into a separate file to make the code more focused.
Handling the logic where the input should be disabled when the agent has not yet been initialized.
Fixing the issue where hooks were used in the branch logic of the TypingChat component.
Optimize Eslint config to check react & react-hooks rules

Looks great, thanks! I fixed a couple typos ("setTypingAcitve" -> "setTypingActive", "addAssistanctMessageToChat" -> "addAssistantMessageToChat").


Fix LOG4J2-2388 / Blocking thread when logging message in an interrup…

…ted thread

Fix unfinite loop in org.apache.logging.log4j.flume.appender.FlumePersistentManager.send when the thread is interrupted (Note : The loop has been completely removed since it does not seems to serve any purpose)
Preserve Thread interruption status on InterruptedException
Add unit test for this case

PR has been merged


[otbn] Spurious WE strobe check on both regfiles

We have reason to believe that WE strobes on regfiles are prone to glitch attacks, hence we decided to augment the Ibex register file and the peripheral CSR nodes with additional checks that trigger alerts in case a spurious WE is seen:

https://github.com/lowRISC/ibex/pull/1605
https://github.com/lowRISC/opentitan/issues/11036#issuecomment-1076939473.

Should we do the same on the OTBN register files?
This check would leverage prim_onehot_check which has been optimized to have minimal area and delay overhead - see https://github.com/lowRISC/opentitan/pull/11994.

i think for otbn it is not quite as critical due to the fact that there is always a host processor controlled redundancy.
However, if it is cheap to add, I think it would be worth it.

Adding these checks should be straightforward and they are pretty lightweight.
Let me take a look.

I think it's worth doing, still a reasonable chunk of D2S to close out so shouldn't prove disruptive to that, especially if @msfschaffner can do the RTL work.

should we close this issue now? as it relates to otbn, i think all the work is done?


use empty(port) instead of !port to check for empty string

I was getting this error message Error! No serial port found even when a port exists, seems the issue is with how it's checking for empty string.

LGTM, thanks!


added DNS category and a resource record type subcategory w/ a stub f…

…or A records

[ x] I have read freeCodeCamp's contribution guidelines.
[x ] My pull request has a descriptive title (not a vague title like Update index.md)
[x ] My pull request targets the master branch of freeCodeCamp.
[x ] None of my changes are plagiarized from another source without proper attribution.
[x ] My article does not contain shortened URLs or affiliate links.

If your pull request closes a GitHub issue, replace the XXXXX below with the issue number.
Closes #XXXXX
I will add additional record types once this category is added.

I corrected your folder and file name to guide/english/dns/resource-record-types/a/index.md.  In the future, please make sure the file name of the content is index.md.

LGTM


Fix recent searches keyboard navigation

Fixes

Fixes #3195 by @fcoveram
Fixes #480 by @obulat
Description

This PR refactors the way the keyboard navigation in a search layout on a narrow screen works.
I extracted non-related changes to other PRs, and will un-draft this PR when those PRs are merged.
Some other related improvements in this PR are:

handleClear and handleClearSingle functions were combined into one function with an optional entry argument. If  entry is passed to the function, only that entry is removed, otherwise, all recent search entries are removed.
the mobile modal now displays "No recent searches" text just like the desktop when recent searches list is blank.

Testing Instructions

Run the app using just frontend/run dev:only, and open localhost:8443/search?q=cat with a narrow screen.
Try navigating on the page using the keyboard. If you get to the search input, and press Tab again, you will navigate to the content settings button, and then - to the main contain. In production, you will not be able to navigate to the content settings button because when you get inside the search input, the recent searches modal opens, and then you can only close it using the Back button at the right side of the search input.
If you navigate using the keyboard to the search input, and then press a letter or an arrow key, the recent searches modal will open. Here, you can select one of the options using arrow keys. You can also clear recent searches by clicking "Clear all" or "Clear" cross button next to one of the recent searches.
When you click on the search input, the recent searches modal will open, and the cursur will be where you clicked (i.e, if you click in the middle of the search word, it will stay there and not move to the end of the search term as in prod"
Checklist

[x] My pull request has a descriptive title (not a vague title likeUpdate index.md).
[x] My pull request targets the default branch of the repository (main) or a parent feature branch.
[x] My commit messages follow best practices.
[x] My code follows the established code style of the repository.
[x] I added or updated tests for the changes I made (if applicable).
[ ] I added or updated documentation (if applicable).
[x] I tried running the project locally and verified that there are no visible errors.
[ ] I ran the DAG documentation generator (just catalog/generate-docs for catalog
PRs) or the media properties generator (just catalog/generate-docs media-props
for the catalog or just api/generate-docs for the API) where applicable.

Developer Certificate of Origin

Developer Certificate of Origin
Developer Certificate of Origin
Version 1.1

Copyright (C) 2004, 2006 The Linux Foundation and its contributors.
1 Letterman Drive
Suite D4700
San Francisco, CA, 94129

Everyone is permitted to copy and distribute verbatim copies of this
license document, but changing it is not allowed.

Developer's Certificate of Origin 1.1

By making a contribution to this project, I certify that:

(a) The contribution was created in whole or in part by me and I
    have the right to submit it under the open source license
    indicated in the file; or

(b) The contribution is based upon previous work that, to the best
    of my knowledge, is covered under an appropriate open source
    license and I have the right under that license to submit that
    work with modifications, whether created in whole or in part
    by me, under the same open source license (unless I am
    permitted to submit under a different license), as indicated
    in the file; or

(c) The contribution was provided directly to me by some other
    person who certified (a), (b) or (c) and I have not modified
    it.

(d) I understand and agree that this project and the contribution
    are public and that a record of the contribution (including all
    personal information I submit with it, including my sign-off) is
    maintained indefinitely and may be redistributed consistent with
    this project or the open source license(s) involved.

I'm not able to navigate to the "Clear" button from the search bar with the recent searches modal open using the Tab key. Is that intentional?

It wasn't intentional, and is fixed now. Use arrows for navigating between the search entries, and tabs to move to "Clear" button and "Clear " buttons.

@obulat please ping me when you think this is stable enough for final review! I don't want to review prematurely and miss any changes

@obulat please ping me when you think this is stable enough for final review! I don't want to review prematurely and miss any changes

I think it's ready for review now

I tried the testing steps locally and it doesn't seem like the current search shows up in the recent searches. If you also navigate to any of the recent searches, then tab out of the modal, the search text still remains as the recent search (rather than reverting to the current search). Are both of those intentional?

I agree with @zackkrida's notes. A mockup/prototype showing the ideal flow could help align our expectations. And once we land on a solution, implement it throughout different PRs.
What do you think of this idea?

@fcoveram I think that's a great idea!
@obulat do you mind sharing what's changed functionality wise since my last review? I don't see that captured in a comment or the PR description but I might be missing something.

I wrote why I asked for a new review yesterday, but the comment didn't get posted.
I refactored the way the recent searches are saved in the store, and most of the points from @zackkrida's message are fixed.

LGTM as it fixes the linked issues well. to @AetherUnbound's points, I believe those are existing bugs we need to rectify. To clarify what those bugs are, and list some more:

Homepage searches are not added to the recent search list 😢

The way we used local storage in the search store was persisting the state intermittently. I tried different options, but the only one that worked consistently was to create a function that persists the recent searches by both setting the recentSearches state property and the value of useStorage composable.

Hovering over recent searches updates the search input text (great!) but removing focus from the final choice doesn't clear the input state back to the original search term

Hovering does not update the input text, but selecting the options using arrow keys does (Google search box has the same behavior). Then, if you close the recent searches popover/modal using the "Back" button (in the narrow view), the "Escape" or "Enter" keys, the search is executed with the selected recent search.
We could make it so that pressing "Escape" key or clicking "Back" button reset to the original search term, but I think keeping the term that is visible in the search bar is more intuitive.

The recent search popup can not be closed with the keyboard when there are no recent searches. It relies on a focus loss of the "clear" button to hide itself and this is impossible when the clear button isn't displayed

The handler for hiding the recent searches when there are no recent searches wasn't working because the event handler wasn't falling through to the VButton component. Fixed in the most recent commit.

Recent searches can not be navigated into via the keyboard to focus their individual removal buttons

You need to use the arrow keys to navigate to the recent search entries. But from there, you cannot navigate into the individual clear buttons. You can navigate into the individual removal buttons using "Tab" key on a narrow screen (they come after the main "Clear" button in the tabbing order), but not on desktop.
I tried searching for what the behavior should be in such cases when the combobox options have inner actions (other than searching when you press "Enter" on the option), but couldn't find any. I like @fcoveram's idea of creating a separate issue for finding the ideal flow for these individual actions.
@zackkrida, @AetherUnbound, could you re-review checking that the previous fixes are still working, and the 3 points here were also addressed.

Clear all recent searches and search from the homepage. The recent searches should contain the latest search entry.
Clear all recent searches and open an SSR-ed search page (http://localhost:8443/search?q=cat). The recent searches should have "cat" option.
On desktop, navigate into the recent searches popover using "Tab" key, clear all the recent searches, and tab out of the popover. It should close.
Select the recent searches using vertical arrow keys. The search bar text should be updated. If you press "Enter" or close the popover using "Back" button on narrow screen or "Escape" key, the search results for the updated search term should appear.

We could make it so that pressing "Escape" key or clicking "Back" button reset to the original search term, but I think keeping the term that is visible in the search bar is more intuitive.

I think this is the behavior that I'd expect, but if the current implementation is what makes the most sense than I won't block it!

Edit: That said, having the current search in the recent searches makes the existing behavior much easier to deal with!

Right, I forgot to update the long text in the description to say that I re-added the current search to the recent searches. I looked at other examples of recent searches, and realized that if someone selected a different option, but wants to go back to the original search term, we need to have it in the options list.


[DNM] Health check for stores

Filter out unknown and down stores from balance.

LGTM

LGTM

LGTM


DRILL-5208: Finding path to java executable should be deterministic

See DRILL-5208 for background. Instead of using “find” to locate the
java command, we use the any information available, resorting to find
only if the “usual suspects” fails. The result is that we use the JDK
java when available, instead of randomly choosing JDK or JRE java.

Adressed code review comments and rebased onto latest master.

Please squash the commits as well.

+1, looks good.


Original /etc/hosts not restored after an error

Hi there,
I guess that after an error, the original /etc/hosts should be restored, for cleaning things up. It's the behavior I can observe if hitting Ctrl-C when Kubefwd is up and running in a healthy state.
However I found two cases where this didn't happen:
1
There was an unexpected error due to some timeout (can't provide more details sorry - no idea what exactly happened). Kubefwd didn't gracefully handle the issue and was seemingly waiting forever.
I hit Ctrl-C, and /etc/hosts wasn't restored.
2
In this other case, Kubectl did catch the issue and gracefully turned itself off. But /etc/hosts wasn't restored.
~ $ sudo kubefwd services -n default -n kafka

 _          _           __             _
| | ___   _| |__   ___ / _|_      ____| |
| |/ / | | | '_ \ / _ \ |_\ \ /\ / / _  |
|   <| |_| | |_) |  __/  _|\ V  V / (_| |
|_|\_\\__,_|_.__/ \___|_|   \_/\_/ \__,_|

Press [Ctrl-C] to stop forwarding.
Loading hosts file /etc/hosts
Backing up your original hosts file /etc/hosts to /etc/hosts.original
Fwd <IP_ADDRESS>:5432 as db-postgresql:5432 to pod db-postgresql-0:postgresql
Fwd <IP_ADDRESS>:5432 as db-postgresql-headless:5432 to pod db-postgresql-0:postgresql
Fwd <IP_ADDRESS>:443 as kubernetes:443 to pod db-postgresql-0:8443
Fwd <IP_ADDRESS>:9092 as wd-kafka.kafka.svc.cluster.local:9092 to pod wd-kafka-0:kafka
Fwd <IP_ADDRESS>:9092 as wd-kafka-headless.kafka.svc.cluster.local:9092 to pod wd-kafka-0:9092
Fwd <IP_ADDRESS>:2181 as wd-kafka-zookeeper.kafka.svc.cluster.local:2181 to pod wd-kafka-zookeeper-0:client
Fwd <IP_ADDRESS>:2181 as wd-kafka-zookeeper-headless.kafka.svc.cluster.local:2181 to pod wd-kafka-zookeeper-0:2181
Fwd <IP_ADDRESS>:3888 as wd-kafka-zookeeper-headless.kafka.svc.cluster.local:3888 to pod wd-kafka-zookeeper-0:3888
Fwd <IP_ADDRESS>:2888 as wd-kafka-zookeeper-headless.kafka.svc.cluster.local:2888 to pod wd-kafka-zookeeper-0:2888
E1105 17:41:08.849920   34632 portforward.go:190] lost connection to pod
E1105 17:41:08.849954   34632 portforward.go:190] lost connection to pod
E1105 17:41:08.849971   34632 portforward.go:190] lost connection to pod
E1105 17:41:08.849985   34632 portforward.go:190] lost connection to pod
E1105 17:41:08.850454   34632 portforward.go:190] lost connection to pod
E1105 17:41:08.850513   34632 portforward.go:190] lost connection to pod
E1105 17:41:08.850541   34632 portforward.go:190] lost connection to pod
E1105 17:41:08.850703   34632 portforward.go:190] lost connection to pod
E1105 17:41:08.850739   34632 portforward.go:190] lost connection to pod

Done..

Hope the report is useful!
Victor

Yeah, noticed the same issue when I hit Ctrl+C to shutdown kubefwd. Had to manually copy the /etc/hosts.original to /etc/hosts to restore the original /etc/hosts.
I am running version 1.4.3 on MacOS Sierra.

@indrayam I don't ever explicitly restore the original, essentially it there if for some reason kubefwd were to really mess up the /etc/hosts file. However, entries should be removed properly after a `Ctrl-C'. I believe I fixed some of the issues that were preventing them from being completely removed.
If you have time please let me know if you can reproduce this issue in 1.4.5. For now, I am going to close th issue but will re-open if you still have concerns as it should be much more stable in the current release.

Hi,
Using version 1.4.7, I just faced the same issue when no service actually exist on my namespace. No explicit error is raised in this situation (which would be nice) and the program just stop by itself and the /etc/hosts isn't restored.
Here's the output (I didn't CTRL-C)
 _          _           __             _
| | ___   _| |__   ___ / _|_      ____| |
| |/ / | | | '_ \ / _ \ |_\ \ /\ / / _  |
|   <| |_| | |_) |  __/  _|\ V  V / (_| |
|_|\_\\__,_|_.__/ \___|_|   \_/\_/ \__,_|

Press [Ctrl-C] to stop forwarding.
'cat /etc/hosts' to see all host entries.
Loading hosts file /etc/hosts
Original hosts backup already exists at /etc/hosts.original

Done...

@Charrette kubefwd does not restore the "hosts.original", it is just there in case you would like to restore it. There was no error thrown since not finding services to forward is technically not an error.
Please let me know if it left your /etc/hosts is a poor state, that should not happen and would be a bug.


docs(spec): Custom Security Resources Specification

Signed-off-by: Daniel Pacak<EMAIL_ADDRESS>

Let's do that when the API is stable.


Section of what data is collected by MDE

I'm sitting on the phone with a bank that wants to use this product and we've had four or five calls talking about data handling.... they want to have this data in very clear terms.
"file data (such as file names, sizes, and hashes)"?
does that mean the files will be collected?
"process data (running processes, hashes)"
does this mean the process memory?
This kind of stuff has to be very specific.

Hello @YulelogPagoda! Can you please tell us to which article(s) your feedback applies?

https://docs.microsoft.com/en-us/windows/security/threat-protection/microsoft-defender-atp/data-storage-privacy

@YulelogPagoda thank you!

Doing repo cleanup, I'm closing this issue as the article was moved to a new repo, you can find the current article at
https://docs.microsoft.com/en-us/microsoft-365/security/defender-endpoint/data-storage-privacy?view=o365-worldwide
I'm sending this issue (and others for this area) to the current authors for their review. If there's still an update needed, please file new feedback on the current article in the other repo.


Feat/backend alerting

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


Add the Iterator Symbol to functions which return nodes (ex. cts.search)

Here is a snippet of JS that works in QConsole:
let results = [];

var items = cts.search(
    cts.andQuery([
      cts.directoryQuery("/searchable/"),
      cts.elementWordQuery(
      xs.QName("maintenance-committee"),
        "AV-003", "case-insensitive")]));

for (item of items){
   var value = item.xpath("/searchable/db-meta/maintenance-committee/string()");

   results.push(value);
}

Here is the same code written in TS:
let results = [];

let items = cts.search(
    cts.andQuery([
      cts.directoryQuery("/searchable/"),
      cts.elementWordQuery(
      xs.QName("maintenance-committee"),
        “AV-003", "case-insensitive")])).toArray();

for (let item of items){
    let value: string = item.xpath("/searchable/db-meta/maintenance-committee/string()”);

results.push(value);

}

Without the .toArray(), the resulting JS output from the TS transpiler doesn’t work.

@christyharagan, what do you make of this? Is the transpiler just doing a bad job, or is there something more serious going on?

You should never use toArray() when you don’t have to. The whole point of the iterator (i.e. what you get by calling the Symbol.iterator method) is to not have to eagerly evaluate all of the results. cts.search() returns a ValueIterator (in MarkLogic 8) which is (confusingly) bother an iterator and and iterable. I don't know the TypeScript-specific issue here, but here’s a polyfill for ES2016 Map type that exposes its interface.

No, it was just poor typing effort on my behalf. It was a good catch by Salim. I'll put a fix in...

Christy probably fixed this in one of the 0.3.0 to 0.4.2 releases, but it has now moved over to Sequence. Hopefully that works equally well..

@sjordan1975 Would you be able to run a test with the current code in this repo? It is tagged with 0.5.0, but not yet released on NPM..


k8s indexer-service auto-discovery

Adds the option to automatically detect the indexer-service pod IPs behind a kubernetes service, instead of statically specifying them with --indexer-service-metrics-endpoint.
The k8s service will be continously watched for any changes, so that autoagora still functions in the case of the indexer-service being a k8s deployment (ie. the indexer-service pods are ephemeral).
The new option is --indexer-service-metrics-k8s-service.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

47 of 101 (46.53%)  changed or added relevant lines in 7 files are covered.
2 unchanged lines in 1 file lost coverage.
Overall coverage increased (+2.8%) to 36.451%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

autoagora/price_multiplier.py
0
3
0.0%

autoagora/main.py
0
4
0.0%

autoagora/misc.py
8
13
61.54%

autoagora/subgraph_wrapper.py
0
7
0.0%

autoagora/query_metrics.py
24
33
72.73%

autoagora/k8s_service_watcher.py
11
37
29.73%

Files with Coverage Reduction
New Missed Lines
%

autoagora/price_multiplier.py
2
0%

Totals

Change from base Build<PHONE_NUMBER>:
2.8%

Covered Lines:
152

Relevant Lines:
417

💛  - Coveralls


Adding EmailAddress Scalar

Adding EmailAddress Scalar type to HotChocolate.Types.Scalars
Part of #2927

beginning wip: https://github.com/wonbyte/hotchocolate/commit/8eb34e569793b39e15c072c41a932c6693bf1653

beginning wip: https://github.com/wonbyte/hotchocolate/commit/8eb34e569793b39e15c072c41a932c6693bf1653

Merged


User account menu is displayed as a subcategory - wrong level

Current behavior

Expected behavior

This menu should be displayed at Sidebar's root level only
Can you handle fixing this bug by yourself?

[ ] YES
[x] NO

Which Release Cycle state this refers to? Info for developer.
Pick one option.

[ ] This is a bug report for test version on https://test.storefrontcloud.io - In this case Developer should create branch from develop branch and create Pull Request 2. Feature / Improvement back to develop.
[x] This is a bug report for current Release Candidate version on https://next.storefrontcloud.io - In this case Developer should create branch from release branch and create Pull Request 3. Stabilisation fix back to release.
[x] This is a bug report for current Stable version on https://demo.storefrontcloud.io and should be placed in next stable version hotfix - In this case Developer should create branch from hotfix or master branch and create Pull Request 4. Hotfix back to hotfix.

Environment details

Browser:
OS:
Node:
Code Version:

Additional information

@pkarw issue was already reported, fixed and merged in 1.10.0-rc.1  :)
#2612
#2625


WIP - Simplify function signature for GetSidecarServiceInstances

Tests in pilot/pkg/proxy/envoy/discovery_test.go fails with this change and I don't know why.
Any tips would be appreciated.

@rosenhouse: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-presubmit.sh
aa4deacfacf634bafeb7cb608742eb947a8ca3f8
link
/test istio-presubmit

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@rosenhouse: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-presubmit.sh
bab1ab19e07eb9963b044ec219cb36f0d0eb126e
link
/test istio-presubmit

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@rosenhouse: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-pilot-e2e.sh
e6435af39ad51431457713df36d359d900afab09
link
/test istio-pilot-e2e

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/test istio-pilot-e2e

@rosenhouse: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-pilot-e2e.sh
e6435af39ad51431457713df36d359d900afab09
link
/test istio-pilot-e2e

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@rosenhouse: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-pilot-e2e.sh
649ee90a24117fec45460cd54d33199ad7e0fe96
link
/test istio-pilot-e2e

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@rosenhouse: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-pilot-e2e.sh
f99de5f0ad2e9715310f7ab8ab52f3cbb0a3cd9f
link
/test istio-pilot-e2e

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/test istio-pilot-e2e

/assign @rshriram


cc cmac alt

Description
Add the 'MBEDTLS_CMAC_ALT' #Define in order to support the alternate CMAC.
Status
READY/IN DEVELOPMENT/HOLD
Todos

[ ] Tests
[ ] Documentation
[ ] Changelog updated
[ ] Backported

Automatic CI verification build not done, please verify manually.

This is a duplicate of #866


Proxy timeouts after 60 seconds

Is this a bug report?
POST requests with form data seems to be timedout at 60 seconds. Happens in node version 13.x and 14.x but not 12.x. This might be a bug in node-http-proxy.
Steps to reproduce

Create script testUpload.sh (or use scripts/testUpload.sh in supplied repo)

ENDPOINT=$1
FILE=large.file
RATELIMIT=10M

echo "Testing upload to $ENDPOINT"

if ! [ -f "$FILE" ]; then
  echo "Generating test file..."
  dd if=/dev/zero of=$FILE bs=1048576 count=2048
fi

echo "Testing upload at $RATELIMIT"

curl -X POST $ENDPOINT -H "accept: application/json" -H "Content-Type: multipart/form-data" -F "file=@${FILE}" --limit-rate $RATELIMIT --progress-bar --verbose -o ${RATELIMIT}.out

Run script as testUpload.sh URL
Wait for it

After 60 seconds I recieve
...
* Recv failure: Connection reset by peer
* stopped the pause stream!
* Closing connection 0

Simiar behavior is seen in browser.The request gets net::ERR_CONNECTION_RESET with error message TypeError: Failed to fetch in Chrome and TypeError: NetworkError when attempting to fetch resource. in Firefox.
Expected behavior
Request goes on until finished.
Actual behavior
Request vanishes. Our server got yielded client disconnected. Client never receives a response.
Setup
Frontend is created with Create-React-App with a separate API on port 5000.  setupProxy.js is configured as
const { createProxyMiddleware } = require("http-proxy-middleware");

const TIMEOUT = 30*60*1000;
module.exports = (app) => {
  app.use(
    "/api",
    createProxyMiddleware({
      target: "http://<IP_ADDRESS>:5000",
      changeOrigin: true,
      pathRewrite: {
        "^/api": "",
      },
      proxyTimeout: TIMEOUT,
      timeout: TIMEOUT,
      onError: (err, req, res) => console.log(err)
    })
  );
};

client info
The app is run inside Docker container based on node:14.5.0-alpine. Same error is encountered when run an Ubuntu system. This is the case for node versions 13.x and 14.x but not 12.x.
target server info
A Python FastAPI server that allows file uploads on endpoint POST /v1/storage.
Reproducible Demo
Here is a minimum working example of the front end1l; https://github.com/johanbook/http-proxy-middleware-upload-bug. Bash script included.

Are these options even supported, it always passes an empty object here https://github.com/chimurai/http-proxy-middleware/blob/master/src/http-proxy-middleware.ts#L25

Even we are also facing the same issue, but request is getting timed out for 30 seconds only.
Some one looking against this??
please help @chimurai

@nanepallysaikumar I downgraded my node version to 12.10 which fixed it

@johanbook But isn't the issue due to the http proxy middleware?.
How come node version is related to the time out issues?
Currently we are using the node version 10.15.0 but we are NOT using in our react application, it is just confined to our local machine.
´´´
const options = {
target: "/urltohit",
changeOrigin: true,
logLevel: 'debug',
pathRewrite: {
'^/api': '/'
},
timeout: 60000,
proxyTimeout: 60000,
onError: (err, req, res) => {
console.log('Error due to server down', err);
res.status(500).send({ error: 'Not Reachable!' });
}
};
const proxyData = proxy(options);
const app = express();
const port = process.env.NODE_PORT || 5000;
app.set('port', port);
app.use(function(req, res, next) {
res.setHeader('x-frame-options', 'DENY');
res.setHeader('X-XSS-Protection', '1; mode=block');
res.setHeader('X-Content-Type-Options', 'no-sniff');
next();
});
app.use(favicon(__dirname + '/build/favicon.ico'));
app.use(express.static(path.join(__dirname, 'build')));
app.use(log);
app.use('/api', proxyData);
app.get('*', function(req, res) {
return res.sendFile(path.join(__dirname, 'build', 'index.html'));
});
// establish http server connection
app.listen(port, () => {
console.log(App running on port ${port});
});
´´´
Can you please give some insights how was it used by you? Currently its getting time out for 30 seconds

@nanepallysaikumar my guess is that it was a breaking change between node 12 and 13 that broke http-proxy-middleware.
Well, you are using it in your express server and the error might arise from server-side. However,  if you already are using node 10 to run your express application it might be something else.  Maybe try upgrading node to 12 and see if there is any difference

I seem to have a similar problem using node 14.0.20. Did anyone discover anything except downgrading to 12?

Found this line in my code base :D
router.use(express.json()); // http-proxy-middleware stops working if this middleware is added before it.

Removing/Moving that middleware fixed my issue. Hopefully, this PR solves problems like this in the future

This might be due to https://github.com/nodejs/node/issues/35661, as pointed out in https://stackoverflow.com/questions/62911472/60-second-timeout-in-http-proxy-middleware/65106664#65106664

After upgrading my Node version this is no longer a issue

After upgrading my Node version this is no longer a issue

Removing/Moving that middleware fixed my issue.

Thank you so much @Mousaka . This solved my issue! What a headache.

Found this line in my code base :D
router.use(express.json()); // http-proxy-middleware stops working if this middleware is added before it.

Removing/Moving that middleware fixed my issue. Hopefully, this PR solves problems like this in the future

@Mousaka Thanks for your advice!
I had a problem that this proxy-middlware works properly in GET and DELETE, but always PENDING-CANCELED in POST and PUT.
i think this is due to the body(payload) parsing. and when i moved middleware before the router.use(express.json()) It works.

I found this GitHub issue after all my POST requests to the proxied URL were timing out with errors from the upstream host. The timeout could be 30, 60 or 120 seconds depending on the host server technology and configuration. I assumed it was my http-proxy-middleware config, but it's actually because of a proxy middleware before mine that I can't control (in this case, a Google Cloud Function). To fix, I followed #492.
Simply:
const { createProxyMiddleware, fixRequestBody } = require('http-proxy-middleware');

const proxy = createProxyMiddleware({
  //...usual config here
  onProxyReq: fixRequestBody,
});

// or
const proxy2 = createProxyMiddleware({
  //...usual config here
  onProxyReq: (proxyReq, req) => {
    // do what you want first, then
    fixRequestBody(proxyReq, req);
  },
});


Use gender-neutral pronoun in Readme

Just reading through the Readme and this tripped me up a little. Small change to make this more inclusive.

Thank you!


About NNFDIV

Thanks for your great work about single image generation.
Could you release the code of NNFDIV? Thanks.

Hi @zhangXJ147,
Thanks for asking. we will try to add it soon.
Niv

Dear @zhangXJ147,
Please check https://github.com/nivha/nnf_diversity
Niv

@nivha
I got it. Thank you!


CMake logic for CONDUIT also requires MFEM

I was building VisIt with Conduit and without MFEM and I got some missing include files in my build.
The file mdserver/main/CMakeLists.txt only adds Conduit includes if MFEM is also found. There may be other instances of this. I didn't check any further. I just added MFEM to my build.
This is on the 3.3RC.

Thanks Eric, I think we need some extra logic to make this a configure error before it turns into compile error.


Pass editor view as parameter to markerDOM function

I'd like to configure fold indicators based on another StateField I have active. I've messed around with CSS solutions and other ways of getting this info to the markers but I think just passing the view to the markerDOM function may be the best solution, so we can do something like
foldGutter({
  markerDOM: (open: boolean, view: EditorView) => {
    const fieldValue = view.state.field(myField)
    if (fieldValue) {
      // return something different
    }
    return !open ? rightCaret() : downCaret();
  },

In my specific case I want to hide the markers so that they only show when that range is folded or the user is hovered over/has a selection in a foldable block. I could do that by returning a different DOM element or adding a CSS class or something.
I hope that makes sense!

Thanks for the quick reply, first of all!

Isn't is more robust to put the information you need into the marker object and make sure it is replaced with a marker with different data when appropriate?

That sounds fine to me, I guess my question would be where would I do that? The only information I have when the marker is created is whether it is open or not, right?
Admittedly now that I write it out I'm not even sure this PR would do the job either - having access to the view still wouldn't give me the necessary information to be able to associate a marker with a specific foldable block.

Oh indeed, I was thinking about gutter markers in general—in this case, the fold gutter is regenerating its marker set on every change, so you really don't want to do anything expensive in them, or anything that makes them unique enough to require re-renders all the time.
I can imagine using the view in such a method to attach some kind of event handlers to markers, but since pointer interaction on these is already handled by the library, I don't think that is very useful here.
Given how the fold gutter works, I don't think markers that depend on the folded (or foldable) content are realistic. You can try to build something like that, but you'd have to set up an entirely custom gutter.


Add Clone() method to mysql.GTIDSet

With the changes introduced in c6f8498, ParseMariadbGTIDSet now returns a *MariadbGTID (rather than MariadbGTID) wrapped in a GTIDSet interface. This fixed a bug around never incrementing the GTID; however, broke the way we store the GTID in various parts of our application as we expected the concrete type.
AFAIK Go does not expose an easy way to access the underlying value of an interface without making it less generic (i.e. casting the GTIDSet to either *MysqlGTIDSet or *MariadbGTID), so I propose adding a Clone() method to the GTIDSet interface to account for this change in behavior.
Currently, the only other option we have is to track the flavor of the GTID we're passing around and calling ParseGTIDSet to create a new one which is not ideal:
clone, _ := mysql.ParseGTIDSet(flavor, gtid.String())

LGTM


Integrate code from Release Notes Generator action

Migrate planned library code from the Release Notes Generator
Migrate related unit tests.
Consider code improvements and non-persistent usage of GithubManager methods.
Confirm How to for library development with target action development.
Integrate library to origin project.

Optional:

introduce pylint tool into local run and CI

Development of the library has been ended.


AppleFormatReport binary Images arch is wrong?

System Library.
I got arch arm64,but acturally is arm64e

I think I have the same problem. I wonder if there is a way to fix this?

@kstenerud Hello, I wonder if you know a way to fix this? If you give me some pointers or a place to start to look at I can make a fix and submit a PR.
Any  help is highly appreciated

I have the same problem too.
When I symbolized the AppleFormat crash log, I got the error atos cannot load symbols for the file ~/Library/Developer/Xcode/iOS DeviceSupport/14.6 (18F72) arm64e/Symbols/usr/lib/system/libsystem_c.dylib for architecture arm64.
So I changed the Binary Images architecture from arm64 to arm64e and it worked.
I debug the code to see why the architecture is wrong and found this:

I wonder if there is a way to fix this?
@happy201993 @nacho4d  Have you worked it out yet? Any help is highly appreciated

@AndyXB I am still investigating this issue.
I also noted the cpusubtype is that weird number: -2147483646. KSCrash expects to be 2 so it is recognized as arm64e I wonder if we should interpret that cpusubtype in a different way (-2147483646 is Int32 min (-2147483648) plus 2 ... this is suspicious)...
In the mean time I have fixed other minor arm64e things in https://github.com/kstenerud/KSCrash/pull/415 but this main issue (the main issue) is still remaining. I am still investigating...

Update: I found the reason. KSCrash is using its own logic to calculate architecture name from cputype and cpusubtuype. It should use NXGetArchInfoFromCpuType like in this stackoverflow answer. I have tried and works great. I will update above pull request now :)

Great! LGTM! Thx.

Thanks!

Just for the record. Above pull request solves the problem but it is not merged yet. People can use my fork if needed of course.
(Unfortunately I am not supposed to use my own fork for client projects. So it would be great it @kstenerud approves it.)
@kstenerud Could you take a look at it please?

您好，我已收到您的邮件，会尽快给您回复。


build.py raises exception

2016-10-11 14:30:35,332 - INFO: Applying patches via quilt... Traceback (most recent call last):   File "build.py", line 57, in <module>     exit(main())   File "build.py", line 39, in main     builder.apply_patches()   File "/home/william/bin/ungoogled-chromium/buildlib/linux.py", line 63, in apply_patches     cwd=str(self._sandbox_dir))   File "/home/william/bin/ungoogled-chromium/buildlib/common.py", line 91, in _run_subprocess     return _util.subprocess_run(*args, env=new_env, **kwargs)   File "/home/william/bin/ungoogled-chromium/buildlib/_util.py", line 232, in subprocess_run     with subprocess.Popen(*popenargs, **kwargs) as process:   File "/usr/lib/python3.4/subprocess.py", line 859, in __init__     restore_signals, start_new_session)   File "/usr/lib/python3.4/subprocess.py", line 1457, in _execute_child     raise child_exception_type(errno_num, err_msg)
FileNotFoundError: [Errno 2] No such file or directory: 'quilt'
Here is the output when build.py runs. It says it cannot find the directory quilt. I'm trying to build on Debian Jessie.

I had the same error. You need the dev-packages to get packaging work in Debian.
Just run this:
sudo apt-get install packaging-dev

Huh, I didn't know this package existed. Though I'm pretty certain you just need to manually install quilt and build-essential in order to build Chromium. I'll amend the build instructions.


feat(PLT-4441): initialize library

:tada: This PR is included in version 1.0.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Tooltip bigger than expected

Version
5.1.1
Steps to reproduce
.setOption({
    grid:
    {
        left: "3%",
        right: "4%",
        bottom: "10%",
        containLabel: true
    },
    tooltip:
    {
        triggerOn: "click",
    },
    xAxis:
    [
        {
            type: "category",
            data: props.chart_data_arr.map((chart_data:BarChartData<number>) => chart_data.name),
            axisTick: {
                alignWithLabel: true
            }
        }
    ],
    yAxis:
    [
        {
            type: "value",
        }
    ],
    series:
    [
        {
            name: "Resume",
            type: "bar",
            barWidth: "60%",
            data: props.chart_data_arr.map((chart_data:BarChartData<number>) => chart_data.value)
        }
    ],
    dataZoom:
    [
        {
            type: "slider",
            bottom: "4%",
        },
        {
            type: "inside",
        }
    ],
});

What is expected?
A size for the tootlip enought for displaying the data but not more.r
What is actually happening?
A long (in the vertical) tooltip with the good information in it but a lot of unnecessary blank.

I use in the app a Popover material ui component for making custom tooltip for my app maybe it present a problem

Missing executable option.


Add python dependencies to website

Not sure if this is the right place for issues on the website, but the dependencies for running an ipython kernel are missing: http://beakernotebook.com/getting-started

Hi Diogo, this is a fine place for that.
Which platform do you mean?  For Mac, for IPython (this is down in the "optional components" section, below the "Download base install" button) we recommend installing Python with Anaconda, which should be complete.

Sorry, misread anaconda to conda (the package manager)! It may be useful to explicitly provide a list of the pip dependencies somewhere if an individual wanted to use an existing virtualenv though (I've only seem them listed in the Dockerfile and an install script).

You are right, we should make that easier to find.  I'll add a link to the Python section for people who want to use a preexisting python.

how's this? https://github.com/twosigma/beaker-notebook-www/commit/ef2ac3510ecfb98527acea6490f5d5bcea5cef2d

Looks good to me! (:


Eslint is not being run... but I think something else is wrong too!

I was trying to use eslint to fix my code style once and for all -- so I don't need to manually change them. But it first turns out that we don't really have a test set up to run it, not even in package.json. So I manually ran node_modules/.bin/eslint src, and sure enough the entire repo has been a bit sloppy too:
.....\color.js\src\deltaE\deltaEITP.js
  24:2  warning  Missing semicolon  semi

.....\GH\color.js\src\deltaE\deltaEJz.js
  32:2  warning  Closing curly brace appears on the same line as the subsequent
block  brace-style
  36:2  warning  Closing curly brace appears on the same line as the subsequent
block  brace-style

I figured re-running with --fix would solve the issue, but check out what I got instead:
diff --git a/src/deltaE/deltaEJz.js b/src/deltaE/deltaEJz.js
index 80f116e..5ad170b 100644
--- a/src/deltaE/deltaEJz.js
+++ b/src/deltaE/deltaEJz.js
@@ -29,11 +29,13 @@ Color.prototype.deltaEJz = function (sample) {
                // both undefined hues
                Hz1 = 0;
                Hz2 = 0;
-       } else
+       }
+ else
        if (Number.isNaN(Hz1)) {
                // one undefined, set to the defined hue
                Hz1 = Hz2;
-       } else
+       }
+ else
        if (Number.isNaN(Hz2)) {
                Hz2 = Hz1;
        }

Yep, it broke the indentation. I think.
(I don't exactly think the original putting else and if on different lines is normal either, but a more conventional else if got put in an unindented line too.)

Yep, adding an indent setting seems to fix the fix. I will do some more tinkering to check out some other rules before I do the PR.

The recommended rules does pick out some fishy things, and the false-positive rate does get reduced with appropriate env settings... But I still don't think it's my call to make. Heck, the indent thing alone will create a big blame-disturbing commit if fixed.

Yeah, I think we both tend to use it as an editor plugin, rather than a terminal command, that's why there was nothing there.
Do avoid auto fix for block-level things, as it will screw up the indentation as you noticed. We use smart tabs, that unfortunately many tools do poorly with.
@svgeesus these are files you edited, could you take a look please?


Document OneDriveSetup.exe /allusers

We spent literally weeks with Microsoft support trying to find out how to pre-install OneDrive for all users on a system so that end-users would not need how to do this, only to discover that this is actually a setting in the installation. ((Why this is not the default baffles me/us/ and baffled Microsoft support).  Please document the existence of this extremely useful option needed to deploying OneDrive for you business users:

OneDriveSetup.exe /allusers *

They call it Per Machine installation https://docs.microsoft.com/en-us/onedrive/per-machine-installation

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

ID: 9733a774-90dc-8726-3300-4660a092e427
Version Independent ID: 3ab12184-9b19-1686-e9d9-a96a93e3fea4
Content: OneDrive guide for enterprises
Content Source: OneDrive/plan-onedrive-enterprise.md
Service: one-drive
GitHub Login: @kaarins
Microsoft Alias: kaarins

@blouwagie Thank you for submitting feedback and contributing to the docs. We are currently investigating this.

https://github.com/MicrosoftDocs/OfficeDocs-SharePoint/issues/1538

@blouwagie Thank you for submitting feedback. We understand that this issue has been resolved.
Please feel free to re-open this issue if there is a specific area of the docs that we can improve or make better. Thank you.


remove fshandler and related tests

https://github.com/cloudwego/hertz/issues/32
remove fshandler and related tests

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

yes, there are two  invalid tests in the context_test.go. I fix them


iconoir-sound-off not rendering

Version<EMAIL_ADDRESS>Are you running from source/main: main

The sound-off icon does not render.
Inspecting the mask data in chome returns:
This page contains the following errors:
error on line 1 at column 133: AttValue: ' expected
Below is a rendering of the page up to the first error.

data:image/svg+xml;charset=utf-8,<svg width="24" height="24" viewBox="0 0 24 24" stroke-width="1.5" fill="none" xmlns="http://www.w3.org/2000/svg"><g clip-path="url(#clip0_3173_16686)%22%3E%3Cpath%20d=%22M18%2014L20.0005%2012M22%2010L20.0005%2012M20.0005%2012L18%2010M20.0005%2012L22%2014%22%20stroke=%22currentColor%22%20stroke-linecap=%22round%22%20stroke-linejoin=%22round%22/%3E%3Cpath%20d=%22M2%2013.8571V10.1429C2%209.03829%202.89543%208.14286%204%208.14286H6.9C7.09569%208.14286%207.28708%208.08544%207.45046%207.97772L13.4495%204.02228C14.1144%203.5839%2015%204.06075%2015%204.85714V19.1429C15%2019.9392%2014.1144%2020.4161%2013.4495%2019.9777L7.45046%2016.0223C7.28708%2015.9146%207.09569%2015.8571%206.9%2015.8571H4C2.89543%2015.8571%202%2014.9617%202%2013.8571Z%22%20stroke=%22currentColor%22/%3E%3C/g%3E%3Cdefs%3E%3CclipPath%20id=%22clip0_3173_16686%22%3E%3Crect%20width=%2224%22%20height=%2224%22%20fill=%22white%22/%3E%3C/clipPath%3E%3C/defs%3E%3C/svg%3E

Upvote & Fund

We're using Polar.sh so you can upvote and help fund this issue.
We receive the funding once the issue is completed & confirmed by you.
Thank you in advance for helping prioritize & fund our backlog.

Same here with version 6.9.0 in Firefox browser.

Same with version 6.11.0.
Do you see this bug?
Planing to fix this?

From what I can see this Pull Request takes care of the problem: https://github.com/iconoir-icons/iconoir/pull/366


Destroying wavesurfer region plugin causus events to emit that appear the same as user events

I've setup a react component with wavesurfer and it works great! One thing I observed that I thought was a bit odd is if I provide functionality where a user interaction causes wavesurfer to be destroyed and recreated there are events emitted during the wavesurfer destroy() teardown that seem like internal events mixed with user events.
Specifically for the region plugin when destroy is called clear calls remove from region.js which emits region-removed. This is the same event that gets emitted when a user interaction causes a region to be removed, but instead it is being emitted because of an internal mechanism in the plugin teardown. It's also a public event in the docs so I think it is meant to be used for tracking user space interactions, not internal wavesurfer events if that makes sense?
This is the stack trace from devtools.
remove (region.js:133)
(anonymous) (index.js:220)
clear (index.js:219)
destroy (index.js:175)
destroyPlugin (wavesurfer.js:660)
(anonymous) (wavesurfer.js:674)
destroyAllPlugins (wavesurfer.js:673)
destroy (wavesurfer.js:1740)

I wanted to get the maintainers thoughts to see if it makes sense to change the plugin destroy and clear functions work so that clear doesn't emit a region-removed event (because it calls remove). Instead it would do the same operations, but emit a regions-destroyed or other event. I know that clear is removing the regions, but this feels like an internal event being mixed with events that can be tied to user interactions. If breaking this up/changing this makes sense I'd be happy to help out with that, just let me know.
Wavesurfer.js version(s):

5.2.0

Browser and operating system version(s):

OS: macOS Monterey 12.0.1
Browser:

edge 95.0.1020.44
chrome 95.0.4638.69
safari 15.1

Code needed to reproduce the issue:
Tried to setup an example in codepen, but running into some audio loading errors, hopefully this provides an idea, and it should be able to run locally with an audio file.
Use behaviour needed to reproduce the issue:
Trigger an event where wavesurfer is destroyed with the region plugin registered and observe the region plugin emit a 'region-removed' event.

I wanted to get the maintainers thoughts to see if it makes sense to change the plugin destroy and clear functions work so that clear doesn't emit a region-removed event (because it calls remove). Instead it would do the same operations, but emit a regions-destroyed or other event.

Makes sense to me. Please open a pull request with your suggestions so it can be reviewed.


GEODE-5674: Stop using random values for test ports

It seems our test framework will fairly frequently have test failing due to
multiple users trying to bind the same port number.  This happens when the
different users happen to randomly generate the same port.  To keep this from
happening, this change will simply hand out the available ports consecutively
as the test runs.  For dunit tests, each child vm will be given a portion of
the overall range for their use.
Thank you for submitting a contribution to Apache Geode.
In order to streamline the review of the contribution we ask you
to ensure the following steps have been taken:
For all changes:

[x] Is there a JIRA ticket associated with this PR? Is it referenced in the commit message?

[ ] Has your PR been rebased against the latest commit within the target branch (typically develop)?

[x] Is your initial contribution a single, squashed commit?

[x] Does gradlew build run cleanly?

[ ] Have you written or updated unit tests to verify your changes?

[ ] If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion under ASF 2.0?

Note:
Please ensure that once the PR is submitted, you check travis-ci for build issues and
submit an update to your PR as soon as possible. If you need help, please send an
email to<EMAIL_ADDRESS>

This change will fail any UDP connection as isPortKeepable will always fail on those.  That ends up breaking the single point of contact between the AvailablePortHelper and AvailablePort code which was fundamental to this change.  This is probably salvageable, but will require a rather fundamental rework.  Closing this PR for now.


Vehicle offset from lane center Calculation

i would like to work on this issue for swoc 2021
Can you please put more details about the issue and also the files we need to change?

i would like to work on this issue for swoc 2021
Can you please put more details about the issue and also the files we need to change?

you have to calculate the road curvature and lane vehicle offset.
code : https://github.com/akshitagupta15june/lane_detection_opencv/blob/master/detector_opencv.py

you have to calculate the road curvature and lane vehicle offset.
code : https://github.com/akshitagupta15june/lane_detection_opencv/blob/master/detector_opencv.py

assigning you

assigning you


Does OnChartClick and OnChartOver not fired?

Hello :)
Is your feature request related to a problem? Please describe.
I need to know when a bar on the Bar Chart is clicked. I tried with OnChartClick and OnChartOver events but it does not write nothing to the console. Can you help me?
<Chart Config="_config1" @ref="_chart1" Class="mt-2 ml-2" Width="100px"
    OnChartClick="(i) => OnChartClick(i)"
    OnChartOver="() => OnChartOver()"></Chart>

@code {
...

    private void OnChartOver()
    {
        Console.WriteLine("ChartOver");
    }

    private void OnChartClick(int i)
    {
        Console.WriteLine("ChartClick " + i);
    }
}

Describe the solution you'd like
I'd like the OnChartClick and OnChartOver work (with their parameters) .
https://www.chartjs.org/docs/2.8.0/general/interactions/events.html -->
onHover: Called when any of the events fire. Called in the context of the chart and passed the event and an array of active elements (bars, points, etc).
OnClick: Called if the event is of type 'mouseup' or 'click'. Called in the context of the chart and passed the event and an array of active elements.
Thank you.

Hey! I have just release a new version of the component and I added a better OnClickChart. More info on PureSorceCode.com
Please let me know if it is working.

Hello, It woks very well :)
Thank you!


Log file viewer exhibits NPE in pure Sling if the "Apache Sling Logging Configuration" is not persitent

The log file viewer attempts to determine the configured log file directory using the apache sling logger configuration. However, the configuration is not persistent in pure Sling but falls back to the implementation's default (${sling.home}/logs). This leads to an NPE:
org.eclipse.jetty.servlet.ServletHandler /system/console/logviewer
java.lang.NullPointerException: null
        at io.neba.core.logviewer.LogfileViewerConsolePlugin.getConfiguredLogfile(LogfileViewerConsolePlugin.java:337)
        at io.neba.core.logviewer.LogfileViewerConsolePlugin.getLogfileDirectory(LogfileViewerConsolePlugin.java:328)
        at io.neba.core.logviewer.LogfileViewerConsolePlugin.resolveLogFiles(LogfileViewerConsolePlugin.java:286)

It must thus be expected that the properties of the configuration is null.

Fixed in 3.5.0


Radar holes

Plugin and CLI to fill in small holes within the radar composite, prior to extrapolation nowcasting.  (Met Office reviewer will find notebooks help with understanding / justifying some details.)
Testing:

[x] Ran tests and they passed OK
[x] Added new tests for the new feature(s)

 Here is an overview of what got changed by this pull request:

Issues
======
- Added 2

Complexity increasing per file
==============================
- improver/cli/fill_radar_holes.py  1
- improver_tests/acceptance/test_fill_radar_holes.py  1
- improver_tests/nowcasting/utilities/test_FillRadarHoles.py  3
- improver/nowcasting/utilities.py  1

See the complete overview on Codacy

 Here is an overview of what got changed by this pull request:

Complexity increasing per file
==============================
- improver/cli/fill_radar_holes.py  1
- improver_tests/acceptance/test_fill_radar_holes.py  1
- improver_tests/nowcasting/utilities/test_FillRadarHoles.py  3
- improver/nowcasting/utilities.py  1

See the complete overview on Codacy

 Here is an overview of what got changed by this pull request:

Issues
======
- Added 1

Complexity increasing per file
==============================
- improver/cli/fill_radar_holes.py  1
- improver_tests/acceptance/test_fill_radar_holes.py  1
- improver_tests/nowcasting/utilities/test_FillRadarHoles.py  3
- improver/nowcasting/utilities.py  1

See the complete overview on Codacy

Wasn't sure which label to go with but @tjtg is going to rework the pytests unit tests using fixtures, so that we get a good "first example" of this unit test style for others to build on.  So this shouldn't be merged until I've pulled that in.

 Here is an overview of what got changed by this pull request:

Issues
======
- Added 7

Complexity increasing per file
==============================
- improver/cli/fill_radar_holes.py  1
- improver_tests/acceptance/test_fill_radar_holes.py  1
- improver_tests/nowcasting/utilities/test_FillRadarHoles.py  3
- improver/nowcasting/utilities.py  1

See the complete overview on Codacy

 Here is an overview of what got changed by this pull request:

Complexity increasing per file
==============================
- improver/cli/fill_radar_holes.py  1
- improver_tests/acceptance/test_fill_radar_holes.py  1
- improver_tests/nowcasting/utilities/test_FillRadarHoles.py  3
- improver/nowcasting/utilities.py  1

See the complete overview on Codacy

 Here is an overview of what got changed by this pull request:

Complexity increasing per file
==============================
- improver/cli/fill_radar_holes.py  1
- improver_tests/acceptance/test_fill_radar_holes.py  1
- improver_tests/nowcasting/utilities/test_FillRadarHoles.py  3
- improver/nowcasting/utilities.py  1

See the complete overview on Codacy

 Here is an overview of what got changed by this pull request:

Complexity increasing per file
==============================
- improver/cli/fill_radar_holes.py  1
- improver_tests/acceptance/test_fill_radar_holes.py  1
- improver_tests/nowcasting/utilities/test_FillRadarHoles.py  3
- improver/nowcasting/utilities.py  1

See the complete overview on Codacy

Replaced by #1250


Data Science with Python :  Self-Organizing Map

Welcome to 'DSWP' Team, good to see you here
This issue will helps readers in gaining all the guidance that one needs to know about Self-Organizing Map. Tutorial to Self-Organizing Map and how it's applied using sample code.
To get assigned to this issue, add your serial numbers mentioned in the spreadsheet of "Data Science with Python", the approach one would follow and choice you prefer (Documentation, Audio, Video). You can go with all three or any number of options you're interested to work on.
If you had referred any resources, add them up in "DS Resources". Similarly if you had used datasets, include them in "DS Datasets".
Domain : Machine Learning
Mentors Assigned : Arya Patel / Kareem Negm
Points to Note :
The issues will be assigned on a first come first serve basis, 1 Issue == 1 PR.
"Issue Title" and "PR Title should be the same. Include issue number along with it.
Changes should be made inside the Datascience_With_Python/ directory & Datascience_With_Python branch.
Follow Contributing Guidelines & Code of Conduct before start Contributing.
This issue is only for 'GWOC' contributors of 'DSWP' domain.
All the best. Enjoy your open source journey ahead. 😎

Hi, I would like to contribute to this issue.
Tanya Malhotra
Data Science with Python, Batch - 12
Sr No. - 429
https://github.com/Tanya205
Documentation

Issue assigned to @Tanya205 for documentation

Hello @prathimacode-hub , please assigned me the issue


Cannot run or debug python files: An Invalid Python interpreter is selected

Does this issue occur when all extensions are disabled?: Yes (Impossible to run with Python extension disabled; Extension Bisect didn't find anything)

VS Code Version: 1.91.1
OS Version: Windows_NT x64 10.0.22631
Python extension: v2024.10.0
Python Debugger extension: v2024.8.0

I develop on a remote server:
SUSE Linux Enterprise Server (SLES) , version 15-SP3
Using the VSCode extension:
Remote Development v0.25.0
I cannot run or debug python files. Keep getting the error:
"An Invalid Python interpreter is selected, please try changing it to enable features such as IntelliSense, linting, and debugging. See output for more details regarding why the interpreter is invalid."
Steps to Reproduce:

export PATH=/python/Python-3.11.9.build/bin:$PATH
create and activate a venv
my settings.json in the .vscode dir in the repo: {"python.defaultInterpreterPath": "/bin/python",}
try to run a python file in the repo using F5 or the 'Run Python File' button
Error: "An Invalid Python interpreter is selected, please try changing it to enable features such as IntelliSense, linting, and debugging. See output for more details regarding why the interpreter is invalid."

After selecting the path to my venv in the UI, the same outcome repeats over and over

My venv is fully functional: I can run python files in terminal but not in the UI
My VSCode installation and the Remote Development extension work without any problems on other remote machines
I have tried reinstalling Python and Python Debugger extensions, which didn't change anything (on other machines they work without problems, same extension versions but different OS versions)
I have tried re-creating my venv, with the same outcome

/extPython

It looks like this is caused by the Python extension. Please file the issue to the Python extension repository. Make sure to check their issue reporting template and provide them relevant information such as the extension version you're using. See also our issue reporting guidelines for more information.
Happy Coding!


Feature/get images digests and repositories

This Pull Request adds a new label on the podman_image_info metric : the sha256 Digest of the image
This is a more secure way to distinguish image when the tag might be overwritten.
With the digest, it is possible to identify the image in a registry even if the tag has been reused on a new image.

Hi @eesprit
Thanks for the PR.
DCO check failed, I would appreciate it if you can sign your commits and also update  README.md metrics output examples.

Hi @navidys
Thanks for four feedback.
I updated the README with some examples, and added the missing signoff on commits.

LGTM


examples/api fails to build with instructions provided

I couldn't get examples/api project to build following just the instructions provided in the readme.
That is:
pnpm install
pnpm build

I got the following error running pnpm build:

Environment:
os: macOS 14.5 (Sonoma)
plugins-workspace commit: 788989aa60be8801313a3c7c99c1a27238805587
vite --version: vite/5.3.5 darwin-arm64 node-v22.5.1
pnpm --version: 9.6.0

Got it working. I had to pnpm i && pnpm build at the project's root first.
Missed that part of the instruction.
Closing issue.


[YUNIKORN-1142] Document new admission controller registration behavior

JIRA: https://issues.apache.org/jira/browse/YUNIKORN-1142

Rebased on latest master.


Hospitalisation data doesn't match data from covid19.admin.ch

Firstly, thanks for the great work on this data source!
We currently source it as one of our supported datasets in the covidregionaldata R package (see here). A contributor/colleague who helps run the ECDC forecasting hub and is trying to integrate hospitalisation forecasts recently flagged a significant mismatch (https://github.com/epiforecasts/covidregionaldata/issues/412) between your data and the hospitalisation data sourced from covid19.admin.ch. Do you have any idea what might be driving this as we are not sure which data source to now use for Switzerland and/or if we need to support both.
Thanks,
Sam

Hi @seabbs
Hospitalisation data are a tricky thing. I am sharing with you what we - to the bet of our knowledge - know about it regarding Switzerland (as a whole) and the Canton of Zurich (in particular).
The data which is available via covid19.admin.ch originates from a system run by the KSD (https://www.vtg.admin.ch/de/organisation/astab/san/ksd/spitalkapazitaeten.html). The system was built and is designed for short-term events, like e.g. a plane crash, when first responders/emergency actors need to be able to assess quickly where they can transport patients to according to their status resp. medical needs. Medical institutions are required to update their capacities in that system once a day (as far as I know).
During the pandemic the system has proven to not be fully suitable for a long term situation like the present one, and flexible enough to cover e.g. additional capacities. Those are two reasons why certain Cantons, like e.g. Zurich, began to collect data independently form the KSD-System. In the case of the Canton of Zurich that data is more accurate in terms of time and completeness, that the one for Zurich being reported from the KSD. What's the status for other Cantons in this regard, I cannot assess. I can only state, that in our repository we make openly accessible the data which each Canton is making public on its website. In some cases that's the KSD data about their Canton, in others it's data from their own collections.
Hope this helps, best
Andreas


chore(pingcap/tiflow): tiflow upgrade to go1.20

tiflow ci on master upgrade to go1.20.

add go1.20 on old pipelines
split pipelines for latest branch because goversion diverse from v7.0.0

releated to https://github.com/pingcap/tiflow/pull/8328

/hold


refactor: Update owner schema and remove username

No need of username for owner login and registration

Hello @Haider8! Thanks for opening this PR.

In the file alembic/versions/54f029830204_update_owner_table.py:

Line 33:91: E501 line too long (108 > 90 characters)
Line 34:91: E501 line too long (109 > 90 characters)
Line 35:91: E501 line too long (110 > 90 characters)

In the file app/crud/owner.py:

Line 39:91: E501 line too long (92 > 90 characters)

Do see the Hitchhiker's guide to code style


add load hook

I want loaded hook to configure in my gem after default rspec-rails configuration.
For example, devise.gem can write in devise/railtie.rb like this
  ActiveSupport.on_load :rspec_rails do
    RSpec.configuration.include Devise::TestHelpers, type: :controller
  end

Sorry, can you explain the purpose of this?

a lot of gem provides helpers for rspec,  for example devise and factory_girl,
but we are requested to to write config.include Devise::TestHelper in spec/rails_helper by myself and I feel it lazy.
Now I'm creating a new gem that has helpers using rspec-rails's methods, I want to include the helpers automatically.
At first, I included like
module MyNewGem
  class Railtie < ::Rails::Railtie
    config.after_initialize do
       RSpec.configuration.include MyNewGem::TestHelper
    end
  end
end

but it doesn't work because rspec-rails's methods are included by require 'rspec/rails'.
In this case, @items_and_filters are
  @items_and_filters=
    [[MyNewGem::TestHelper, {}],
     [RSpec::Rails::FixtureSupport, {:use_fixtures=>true}],
     [RSpec::Rails::FixtureSupport, {}],
     ...

I want to include my helpers after require 'rspec/rails', but require 'rspec/rails' is in spec/rails_helper.rb and I have no time to do.
so I want hook after rspec_rails was ready.

This seems useful but with the split of spec_helper and rails_helper in RSpec 3 we're encouraging users to write rails-less specs where it makes sense to do so, as you can get really fast pass/fail feedback from RSpec when you don't need to boot rails.  I worry that this would couple 3rd party gems to only be usable when rails is booted, which could get in the way of being able to also use the gem w/o rails loaded.

I see the appeal of this since it would remove a step for users to gain functionality from a gem they're using.
I also feel configuration transparency has been a goal in RSpec, and this change hides what modules get included where, and doesn't give users a chance to opt out of that behavior.
I don't worry that it couples 3rd party gems to only be usable with rails, since modules could still be included manually (as they are now) wherever a person wants, but I do think this could be confusing. For example, I have used factory_girl in a rails project, with non-activerecord objects - it would seem strange to need to include FactoryGirl methods in one of those specs, but not in another because the other one has loaded rails.
Also the cost of change here is that gems would have to update to use the new hook (not a big deal), but then they also must either arbitrarily require a recent rspec-rails that runs the hook or give instructions on how to use it with an older rspec, which would be a documentation burden that projects would have to carry for a long time.
These are some of the things that came to mind looking at this. Interesting proposal.

3rd party gems are able to configure themselves by configuring RSpec directly, it's just the require has to be in our spec_helper. I think thats enough really.

The consensus is that the current behavior has advantages over this automatic hook, and it's not too onerous to have a line in the configuration.
Thanks for this contribution, however I am closing this without merging. Keep the ideas coming!


OSX: cant build

Hello,
under BigSur I can't build the plotter, any hints?
best regards Dirk
mkdir: object_blstmp: File exists
mkdir: object_relic_s: File exists
mkdir: object_gmp: File exists
mkdir: object_sodium: File exists
/Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/ranlib: file: /Users/nerle/mad/chia-plotter/build/libbls.a(mp_clz_tab.o) has no symbols
/Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/ranlib: file: /Users/nerle/mad/chia-plotter/build/libbls.a(obprintf.o) has no symbols
/Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/ranlib: file: /Users/nerle/mad/chia-plotter/build/libbls.a(obprntffuns.o) has no symbols
/Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/ranlib: file: /Users/nerle/mad/chia-plotter/build/libbls.a(obvprintf.o) has no symbols
/Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/ranlib: file: /Users/nerle/mad/chia-plotter/build/libbls.a(repl-vsnprintf.o) has no symbols
/Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/ranlib: file: /Users/nerle/mad/chia-plotter/build/libbls.a(relic_dv_mem.c.o) has no symbols
[ 99%] Built target combined_custom
[100%] Built target bls
[100%] Linking CXX executable chia_plot
clang: warning: argument unused during compilation: '-fmax-errors=1' [-Wunused-command-line-argument]
ld: library not found for -lsodium
clang: error: linker command failed with exit code 1 (use -v to see invocation)
make[2]: *** [chia_plot] Error 1
make[1]: *** [CMakeFiles/chia_plot.dir/all] Error 2
make: *** [all] Error 2

You're missing a lib. install it with brew.

For me, brew ls libsodium output the following, which I assume I have sodium installed.
/usr/local/Cellar/libsodium/1.0.18_1/include/sodium/ (62 files)
/usr/local/Cellar/libsodium/1.0.18_1/include/sodium.h
/usr/local/Cellar/libsodium/1.0.18_1/lib/libsodium.23.dylib
/usr/local/Cellar/libsodium/1.0.18_1/lib/pkgconfig/libsodium.pc
/usr/local/Cellar/libsodium/1.0.18_1/lib/ (2 other files)
But I still got ld: library not found for -lsodium error. Any suggestions?

I had to add /usr/local/include to the include paths in the CMakefile but my issue was with the headers, and it looks like this isn't finding the lib (unless you left out some important part of the output)

Seems like one of the new commits broke it. I git checkout 47dc5cbc932f3a1a21e435cc9f2185322e5fae70 to an older commit and it works now.

@madMAx43v3r probably need some integration testing in place :)

Doesn't work for me - tried both hints.
Any other suggestions?


add in --sourcemap-output  ?

Any chance at adding in sourcemap into the build?
$NODE_BINARY "$REACT_NATIVE_DIR/local-cli/cli.js" bundle \
  --entry-file "$ENTRY_FILE" \
  --platform ios \
  --dev $DEV \
  --reset-cache \
  --bundle-output "$BUNDLE_FILE" \
  --assets-dest "$DEST"
  --sourcemap-output "$DEST/main.jsbundle.map"

I'm not sure if there are any implications for having it on all the time. Any thoughts?

I think the main thing I'd want to do is mirror whatever the Debug scheme does with the copied schemes. Does the Debug scheme generate sourcemaps? I haven't tested thoroughly, but from memory I tried to just get the schemes that were Debug that weren't named Debug to be treated as if they were for the rest of the RN build process.
That's a lot of words and context for: Sure I'd be happy to if that's what vanilla React Native does with the Debug scheme.

Sure that makes sense. I don't think it does by default. No worries thanks for the thoughts.