Unable to deploy to Heroku

This is the build log (off of clicking on the "Deploy to Heroku" button, and then inputting the key/secret keys from twitter and pressing "Deploy App")

-----> Node.js app detected
-----> Creating runtime environment
   NPM_CONFIG_LOGLEVEL=error
   NPM_CONFIG_PRODUCTION=true
   NODE_VERBOSE=false
   NODE_ENV=production
   NODE_MODULES_CACHE=true

-----> Installing binaries
engines.node (package.json):  8.1.2
engines.npm (package.json):   unspecified (use default)
   Resolving node version 8.1.2...
   Downloading and installing node 8.1.2...
   Detected package-lock.json: defaulting npm to version 5.x.x
   Bootstrapping npm 5.x.x (replacing 5.0.3)...
   npm 5.4.1 installed

-----> Restoring cache
Skipping cache restore (not-found)
-----> Building dependencies
Installing node modules (package.json + package-lock)
   ><EMAIL_ADDRESS>install /tmp/build_ce6603f36ffa53245bc022daa2183b3f/ypinskiy-GBF-Raiders-4ffb934/node_modules/semantic-ui
   > gulp install

   /tmp/build_ce6603f36ffa53245bc022daa2183b3f/ypinskiy-GBF-Raiders-4ffb934/node_modules/semver/semver.js:295
   throw new TypeError('Invalid Version: ' + version);
   ^

   TypeError: Invalid Version: https://registry.npmjs.org/gulp/-/gulp-3.9.1.tgz
   at new SemVer (/tmp/build_ce6603f36ffa53245bc022daa2183b3f/ypinskiy-GBF-Raiders-4ffb934/node_modules/semver/semver.js:295:11)
   at compare (/tmp/build_ce6603f36ffa53245bc022daa2183b3f/ypinskiy-GBF-Raiders-4ffb934/node_modules/semver/semver.js:571:10)
   at Function.gt (/tmp/build_ce6603f36ffa53245bc022daa2183b3f/ypinskiy-GBF-Raiders-4ffb934/node_modules/semver/semver.js:600:10)
   at Liftoff.handleArguments (/tmp/build_ce6603f36ffa53245bc022daa2183b3f/ypinskiy-GBF-Raiders-4ffb934/node_modules/gulp/bin/gulp.js:99:14)
   at Liftoff.<anonymous> (/tmp/build_ce6603f36ffa53245bc022daa2183b3f/ypinskiy-GBF-Raiders-4ffb934/node_modules/liftoff/index.js:198:16)
   at module.exports (/tmp/build_ce6603f36ffa53245bc022daa2183b3f/ypinskiy-GBF-Raiders-4ffb934/node_modules/flagged-respawn/index.js:17:3)
   at Liftoff.<anonymous> (/tmp/build_ce6603f36ffa53245bc022daa2183b3f/ypinskiy-GBF-Raiders-4ffb934/node_modules/liftoff/index.js:190:9)
   at /tmp/build_ce6603f36ffa53245bc022daa2183b3f/ypinskiy-GBF-Raiders-4ffb934/node_modules/liftoff/index.js:164:9
   at /tmp/build_ce6603f36ffa53245bc022daa2183b3f/ypinskiy-GBF-Raiders-4ffb934/node_modules/v8flags/index.js:93:14
   at FSReqWrap.oncomplete (fs.js:135:15)
   npm ERR! code ELIFECYCLE
   npm ERR! errno 1
   npm ERR<EMAIL_ADDRESS>install: `gulp install`
   npm ERR! Exit status 1
   npm ERR!
   npm ERR! Failed at the<EMAIL_ADDRESS>install script.
   npm ERR! This is probably not a problem with npm. There is likely additional logging output above.

   npm ERR! A complete log of this run can be found in:
   npm ERR!     /app/.npm/_logs/2017-09-10T10_24_43_409Z-debug.log

-----> Build failed
   We're sorry this build is failing! You can troubleshoot common issues here:
   https://devcenter.heroku.com/articles/troubleshooting-node-deploys

   If you're stuck, please submit a ticket so we can help:
   https://help.heroku.com/

   Love,
   Heroku

!     Push rejected, failed to compile Node.js app.
!     Push failed

In Heroku, there is a menu option to run console commands directly. Try using that to install gulp manually first to make sure it installs and then try to deploy the app. Let me know if that doesn't work, I'll try something else.

i'm having the same problem. i've updated gulp and tried installing it through the cli, got these errors:
peppy@peppy-PC MINGW64 /d/Projects/heroku/GBF-Raiders (master)
$ gulp -v
[06:36:07] CLI version 3.9.1
[06:36:07] Local version 3.9.1

peppy@peppy-PC MINGW64 /d/Projects/heroku/GBF-Raiders (master)
$ npm install gulp
npm WARN deprecated<EMAIL_ADDRESS>Please update to minimatch 3.0.2 or higher to avoid a RegExp DoS issue
npm WARN deprecated<EMAIL_ADDRESS>Please update to minimatch 3.0.2 or higher to avoid a RegExp DoS issue
npm WARN deprecated<EMAIL_ADDRESS>graceful-fs v3.0.0 and before will fail on node releases >= v7.0. Please update to<EMAIL_ADDRESS>as soon as possible. Use 'npm ls graceful-fs' to find it in the tree.

><EMAIL_ADDRESS>install D:\Projects\heroku\GBF-Raiders\node_modules\semantic-ui
> gulp install

module.js:491
    throw err;
    ^

Error: Cannot find module 'ini'
    at Function.Module._resolveFilename (module.js:489:15)
    at Function.Module._load (module.js:439:25)
    at Module.require (module.js:517:17)
    at require (internal/module.js:11:18)
    at Object.<anonymous> (D:\Projects\heroku\GBF-Raiders\node_modules\global-prefix\index.js:12:11)
    at Module._compile (module.js:573:30)
    at Object.Module._extensions..js (module.js:584:10)
    at Module.load (module.js:507:32)
    at tryModuleLoad (module.js:470:12)
    at Function.Module._load (module.js:462:3)
npm WARN<EMAIL_ADDRESS>No repository field.
npm WARN optional SKIPPING OPTIONAL DEPENDENCY<EMAIL_ADDRESS>(node_modules\fsevents):
npm WARN notsup SKIPPING OPTIONAL DEPENDENCY: Unsupported platform for<EMAIL_ADDRESS>wanted {"os":"darwin","arch":"any"} (current: {"os":"win32","arch":"x64"})

npm ERR! code ELIFECYCLE
npm ERR! errno 1
npm ERR<EMAIL_ADDRESS>install: `gulp install`
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>install script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.

npm ERR!      C:\Users\peppy\AppData\Roaming\npm-cache\_logs\2017-09-10T22_37_27_212Z-debug.log

@he4rtofcourage what node and npm versions do you have?

@zhurai mentioned in a Reddit post that this worked for them "I tried forking and connecting it to the fork on my own repo and it worked (i didn't do anything else). Previously i was attempting to use the purple "Deploy to heroku" which faces that error. I'm not sure why this happens though since I did not change the npm version after forking."

yeah, i forked and deployed from my repo and now it works perfectly. deploying from the purple button still gives out errors.

I might just delete that button after all this.


Download keeps calling onError

Hello.
I'm trying to use PRDownloader in my app and each time I try to download a video file it just calls onError().
Here is my code:
                            Uri uri = Uri.parse(videoURL);
                            String subPath = URLUtil.guessFileName(uri.toString(), null, null);
                            subPath = subPath.replaceAll("-","");
                            int downloadID =PRDownloader.download(videoURL,"some/directory/"
                                    ,subPath.substring(0,subPath.length()-4) +".dat")
                                    .build()
                                    .setOnStartOrResumeListener(new OnStartOrResumeListener() {
                                        @Override
                                        public void onStartOrResume() {
                                            Log.d("Download","onStartOrResume" );
                                        }
                                    })
                                    .setOnPauseListener(new OnPauseListener() {
                                        @Override
                                        public void onPause() {
                                            Log.d("Download","onPause" );
                                        }
                                    })
                                    .setOnCancelListener(new OnCancelListener() {
                                        @Override
                                        public void onCancel() {
                                            Log.d("Download","onCancel" );
                                        }
                                    })
                                    .setOnProgressListener(new OnProgressListener() {
                                        @Override
                                        public void onProgress(Progress progress) {
                                            Log.d("Download","Progress: " + progress);
                                        }
                                    })
                                    .start(new OnDownloadListener() {
                                        @Override
                                        public void onDownloadComplete() {
                                            Log.d("Download","Complete");
                                        }

                                        @Override
                                        public void onError(Error error) {
                                            Log.d("Download","Error "+error);
                                        }
                                    });

videoURL is a valid url  from amazon CDN.
I've tried to download the video with my browser and it downloads  successfully but not in my app.

It happens to me too.
I do not know if you have put in the Application:
PRDownloader.initialize(getApplicationContext());
It keeps happening to me, too.

Sorry for the delay. I have fixed the release mode issue. Please try the updated library and confirm me.
compile 'com.mindorks.android:prdownloader:0.3.0'


*.MLNET change issues with old mbconfig files

[ ] Evaluate page doesn't work until retrained or until zip is renamed to mlnet

[ ] Old zip doesn't get cleaned up on retraining

Suggested fix: detect if there is a zip file and rename.

renaming file from .zip to .mlnet is a break change. It will prevent old model builder from loading existing .mbconfig once we update model name.
I feel like a safer way to fix this issue is by introducing a new field in ITrainResult calls ModelPath and pointing it to the existing model.  The old .mbconfig can be safely migrated to the new schema by filling ModelPath with trained model. And for new .mbconfig the model can be saved with .mlnet extension.

Validated by Jake


deadlock: create clients with the same grpc env

Signed-off-by: youjiali1995<EMAIL_ADDRESS>
What have you changed?
Use the same grpc env to create deadlock clients
What is the type of the changes?

Bugfix (a change which fixes an issue)

How is the PR tested?
Manual tests.
Does this PR affect documentation (docs) or should it be mentioned in the release notes?
No
Does this PR affect tidb-ansible?
No

@MyonKeminta @AndreMouche PTAL

What else will happen besides creating more threads?

What else will happen besides creating more threads?

No other influence.

/test


Delay command registration to unblock AltGr keybindings

Fix #1375.

I don't know how to test it, so I'm just merging it.

@smoothdeveloper Could you try this with French keyboard layout (I heard you had a problem with AltGr key)? Thanks.


add a helper for Helsinki workshop

Manana kindly offered to participate as helper. @rkdarst @blankdots

Thanks for including me!

Cool @Mko13 welcome! Please take a look at our: https://github.com/coderefinery/manuals/blob/master/helping-and-teaching.md and in zulip for announcements https://coderefinery.zulipchat.com/#narrow/stream/135843-workshops/topic/Helsinki.20May.202019


Request for Updates to SC4S Build for RKE Cluster Deployment

Description: I am currently deploying Splunk Connect for Syslog (SC4S) on an RKE cluster and have made some progress by updating the Dockerfile and entrypoint.sh to change the user to syslog. Below are the details of my setup and the steps I followed:
Configuration Details:
Provider: RKE1
Kubernetes Version: v1.28.7
Architecture: Amd64
SC4S Version: 4.8.1
Deployment Method: AzureDevOps pipelines with Helm chart
Private Registry: JFrog
Steps to Reproduce:
image-sc4s.zip
pipeline-sc4s.zip

Set up an RKE cluster with the above configuration.
Update the Dockerfile and entrypoint.sh to change the user to syslog.
Push the custom SC4S image to the JFrog registry using Azure pipelines.
Modify the image in the values.yaml file to use a custom SC4S image.
Use AzureDevOps pipelines to deploy SC4S using the Helm chart.

Expected Behavior:  I should be able to change any configuration in the values.yaml file using the syslog user without encountering issues related to creating folders and files inside the pods.
Actual Behavior: For example, when configuring custom index using values.yaml file, the configuration is applied but the pods show error creating folder (see screenshot attached)
Request: I would like to request updates to the SC4S build to better support this deployment strategy. Specifically:
Improvements or changes to streamline the process of changing the user to syslog within the SC4S deployment.
Guidance on using a non-root user, such as the syslog user (UID 1024), as our internal policy does not permit the use of root on cluster pods.
Additional Information:
Attached are the updated Dockerfile, entrypoint.sh file, pipeline, and values.yaml file.
This request is related to the Splunk ODS case opened Number 3623626.
Thank you

We will check and get back, please expect a reply on it in a week.


Add Default Value Option

You can add a default value by using 'default:' with single value.
For example: default:ITEM_ID
What kind of change does this PR introduce? (check at least one)

[ ] Bugfix
[x] Feature
[ ] Test
[ ] Docs
[ ] Refactor
[ ] Build-related changes
[x] Other, please describe:

Description: default value example \examples\parameters\default.js and add run command 'npm run parameters:default'

Hi @mhkhoraidah sorry for the delayed response but this PR does not pass the tests so we could not merge it. I really like the change but we cannot merge it 😅


Update TPU setup

What does this PR do?
Update TPU setup
PR review
Anyone in the community is free to review the PR once the tests have passed.
If we didn't discuss your PR in Github issues there's a high chance it will not be merged.
Did you have fun?
Make sure you had fun coding 🙃

Hello @kaushikb11! Thanks for opening this PR. We checked the lines you've touched for PEP 8 issues, and found:

In the file starters/cifar10-baseline/baseline.py:

Line 26:121: E501 line too long (131 > 120 characters)


Import feature should be improved to include variables in other containers (ex. EtherCAT)

The import feature currently only supports importing IO variables from a single target in one container. The import feature should support a prompt or other method that allows the users to import other variables.

This is already possible with the pull request #43 from LabVIEW-DCAF/Fixex42  …
Fixex42
I added that capability while fixing the bug.


Systematic documentation of  BrAPI endpoint  versioning

It may be helpful that the endpoints themselves be systematically tagged in their Markdown  documentation  with  respect to the specific version of the BrAPI specification within  which  they were introduced. Maybe deprecation decisions  could  be  similarly indicated (e.g. when deprecated endpoints  will be removed)?
Alternately, the  root  README markdown  file one each major subfolder could  have a single summary table / listing of endpoints with their origin version of introduction and deprecation (if  applicable) at the  top of each  major BrAPI component set.   One  additional  step would be to have the table entries point  to  the specific Markdown and/or YAML spec (section) which  describes the endpoint (this  latter web page could be the Apiary or  Swagger doc version).

I would love to see this. The information has also already been compiled in the excel files floating around.


Update Scalameta. Fixes #429

Contributing to Twilio

All third-party contributors acknowledge that any contributions they provide will be made under the same open-source license that the open-source project is provided under.

[x] I acknowledge that all my contributions will be made under the project's license.

To avoid regression, would you mind adding a small specfile that tests this functionality?

I actually have a test ready but it passes even with the old version for some reason when comparing the structures (generating files shows the difference though). I'll add the test nevertheless. Perhaps you have an idea what needs to be compared to show the behavior.

my thought would be that Lit.Int(10) should be used anywhere, whereas Term.Name("10") should always fail. I presume the scala.meta upgrade would fix this by escaping the Term.Name("10")

@ import $ivy.`org.scalameta::scalameta:4.1.0`, scala.meta._
@ Term.Name("10").syntax
res1: String = "10"

@ import $ivy.`org.scalameta::scalameta:4.2.4`, scala.meta._
@ Term.Name("10").syntax
res1: String = "`10`"
@ Defn.Object(List(), Term.Name("10"), Template(List(), List(), Self(Name(""), None), List())).syntax
res2: String = "object `10`"

So perhaps we could simply add
Term.Name("10").syntax shouldBe "`10`"

as additional test condition.

@sbrunk I'm onboard with that, though still interested in why https://github.com/twilio/guardrail/pull/430#issuecomment-548574779 is the case

my thought would be that Lit.Int(10) should be used anywhere, whereas Term.Name("10") should always fail. I presume the scala.meta upgrade would fix this by escaping the Term.Name("10")

I'm not sure I understand. Why should Lit.Int(10) be used anywhere?

@sbrunk This is excellent, thank you for your investigation! I'll try to get a bugfix release out with this at some point this weekend.
cc @nb-ceffa, @tomasherman

Thank you!

If only every bugfix was that simple 😀


Support external routers

Currently, react-modal uses window.history.pushState to update the window location query.
This is not picked up on by React Router or NextJS' Router, so those tools are unable to pick up on the change and don't track the activity of the modal doing its thing.
I have some ideas for how this should be implemented in practice.

@jmikrut There's already a resetInternalState method on the ModalProvider that -- if exposed to the context -- will do exactly what you're describing. Aliasing this method for easier consumption may be a good idea, too.


Todo Annotations should not be able to be marked as 'Correct'

Is your feature request related to a problem?
Currently you can mark Todo Annotations as 'Correct' which signals, that this is something that can be left the way it is. But the way Todos are used, that doesn't make much sense.
Desired solution
Either disable or better hide the 'Correct' Button for Todo Annotations.
Possible alternatives (optional)
Maybe there needs to be a different annotation, that marks things as "hey, this needs to be changed and doesn't currently work"
Screenshots (optional)
No response
Additional Context (optional)
No response

Will be solved by #771.

I'd consider this a separate issue than #771.

Thinking about this a little more I'd close this for a different reason: The todo annotation actually has an effect on the generated wrappers, in that TODO comments are being generated. It is meant to denote further changes that have to happen afterwards. Therefore, it makes sense to be able to mark this as correct.
Just taking notes about annotation can be solved by a mix of #771 (as you suggested) and particularly #854.


Cow: Watch 1<PHONE_NUMBER>

Cow requests the watch of the watch_number 1<PHONE_NUMBER>. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

Approved by Jeff Schaller in Charcoal HQ


django-axes==5.40.1 , Running Django tests now cause  "AttributeError: 'WSGIRequest' object has no attribute 'axes_ip_address'"

Hi,
We have recently updated this package as well as Django to 4.1.7  (Python 3.9)  but we seem to be seeing these errors when running our Django tests (which worked with previous versions). We disable axes as recommended (see below). We were wondering if there are extra steps we're now missing when trying to disable axes when running the tests.
Packages involved
channels==4.0.0
Django==4.1.7
django-axes==5.40.1
django-ipware==5.0.0
djangorestframework==3.14.0
requests==2.28.2

Django settings for tests:
AXES_HANDLER = 'axes.handlers.dummy.AxesDummyHandler'
ATOMIC_REQUESTS = False
AXES_ENABLED = False
AXES_LOCK_OUT_AT_FAILURE = False
AXES_ONLY_USER_FAILURES = False

The error traceback:
Traceback (most recent call last):
  File "/Users/robertm/dev/platform/d3_platform/src/platform_v2/site_setup/tests/test_rest_api.py", line 515, in test_update_settings
    self.login(self.client, self.primary_project_users[self.user_constants.ADMIN], data={'project': self.primary_project.id})
  File "/Users/robertm/dev/platform/d3_platform/src/platform_v2/core/tests/tests.py", line 123, in login
    return client.post(reverse('rest_login'), valid_login_data, **header, format='json')
  File "/Users/robertm/dev/platform/d3_platform/lib/python3.9/site-packages/rest_framework/test.py", line 296, in post
    response = super().post(
  File "/Users/robertm/dev/platform/d3_platform/lib/python3.9/site-packages/rest_framework/test.py", line 210, in post
    return self.generic('POST', path, data, content_type, **extra)
  File "/Users/robertm/dev/platform/d3_platform/lib/python3.9/site-packages/rest_framework/test.py", line 234, in generic
    return super().generic(
  File "/Users/robertm/dev/platform/d3_platform/lib/python3.9/site-packages/django/test/client.py", line 541, in generic
    return self.request(**r)
  File "/Users/robertm/dev/platform/d3_platform/lib/python3.9/site-packages/rest_framework/test.py", line 286, in request
    return super().request(**kwargs)
  File "/Users/robertm/dev/platform/d3_platform/lib/python3.9/site-packages/rest_framework/test.py", line 238, in request
    request = super().request(**kwargs)
  File "/Users/robertm/dev/platform/d3_platform/lib/python3.9/site-packages/django/test/client.py", line 810, in request
    self.check_exception(response)
  File "/Users/robertm/dev/platform/d3_platform/lib/python3.9/site-packages/django/test/client.py", line 663, in check_exception
    raise exc_value
  File "/Users/robertm/dev/platform/d3_platform/lib/python3.9/site-packages/django/core/handlers/exception.py", line 56, in inner
    response = get_response(request)
  File "/Users/robertm/dev/platform/d3_platform/lib/python3.9/site-packages/django/core/handlers/base.py", line 197, in _get_response
    response = wrapped_callback(request, *callback_args, **callback_kwargs)
  File "/Users/robertm/dev/platform/d3_platform/lib/python3.9/site-packages/sentry_sdk/integrations/django/views.py", line 85, in sentry_wrapped_callback
    return callback(request, *args, **kwargs)
  File "/Users/robertm/dev/platform/d3_platform/lib/python3.9/site-packages/django/views/decorators/csrf.py", line 55, in wrapped_view
    return view_func(*args, **kwargs)
  File "/Users/robertm/dev/platform/d3_platform/lib/python3.9/site-packages/django/views/generic/base.py", line 103, in view
    return self.dispatch(request, *args, **kwargs)
  File "/Users/robertm/dev/platform/d3_platform/src/platform_v2/core/views.py", line 23, in dispatch
    return super().dispatch(request, *args, **kwargs)
  File "/Users/robertm/dev/platform/d3_platform/lib/python3.9/site-packages/django/utils/decorators.py", line 46, in _wrapper
    return bound_method(*args, **kwargs)
  File "/Users/robertm/dev/platform/d3_platform/lib/python3.9/site-packages/django/views/decorators/debug.py", line 92, in sensitive_post_parameters_wrapper
    return view(request, *args, **kwargs)
  File "/Users/robertm/dev/platform/d3_platform/lib/python3.9/site-packages/dj_rest_auth/views.py", line 48, in dispatch
    return super().dispatch(*args, **kwargs)
  File "/Users/robertm/dev/platform/d3_platform/lib/python3.9/site-packages/rest_framework/views.py", line 509, in dispatch
    response = self.handle_exception(exc)
  File "/Users/robertm/dev/platform/d3_platform/lib/python3.9/site-packages/rest_framework/views.py", line 469, in handle_exception
    self.raise_uncaught_exception(exc)
  File "/Users/robertm/dev/platform/d3_platform/lib/python3.9/site-packages/rest_framework/views.py", line 480, in raise_uncaught_exception
    raise exc
  File "/Users/robertm/dev/platform/d3_platform/lib/python3.9/site-packages/rest_framework/views.py", line 506, in dispatch
    response = handler(request, *args, **kwargs)
  File "/Users/robertm/dev/platform/d3_platform/src/platform_v2/accounts/rest_api/views.py", line 1143, in post
    self.login()
  File "/Users/robertm/dev/platform/d3_platform/src/platform_v2/accounts/rest_api/views.py", line 1059, in login
    reset_user_attempts(self.request, credentials)
  File "/Users/robertm/dev/platform/d3_platform/lib/python3.9/site-packages/axes/attempts.py", line 98, in reset_user_attempts
    attempts_list = filter_user_attempts(request, credentials)
  File "/Users/robertm/dev/platform/d3_platform/lib/python3.9/site-packages/axes/attempts.py", line 41, in filter_user_attempts
    username, request.axes_ip_address, request.axes_user_agent
  File "/Users/robertm/dev/platform/d3_platform/lib/python3.9/site-packages/rest_framework/request.py", line 418, in __getattr__
    return self.__getattribute__(attr)
AttributeError: 'Request' object has no attribute 'axes_ip_address'

Any advice would be much appreciated

Hey @rmincling!
One option you'd have is to just remove axes from INSTALLED_APPS for your test settings, which would probably be the easiest option for running the tests, if your web site works correctly with the settings.
Another is to take a closer look at the error.
The IP address is calculated in the axes.proxy.AxesProxyHandler.update_request method:
https://github.com/jazzband/django-axes/blob/e56f4c92716f79250ab526c800cd598cd51dcfff/axes/handlers/proxy.py#L92
The calculation is done, by default, with the django-ipware package ipware.ip.get_client_ip method:
https://github.com/jazzband/django-axes/blob/e56f4c92716f79250ab526c800cd598cd51dcfff/axes/helpers.py#L151
But it can also be overridden with the AXES_CLIENT_IP_CALLABLE setting, so you could implement a function, lambda, or a callable for resolving the IP from the Django REST Framework HTTP request object as well.
Btw did you try Axes with the newest version of Django (i.e. 4.1.8 or 4.2.1) and DRF yet? Probably won't affect the result but could be good to check since upgrading the version should be a rather minimal thing to test.


404 error on View File for Commits with Spaces in File/Folder

Gitea version (or commit ref): 1.12.4
Git version: 2.20.1
Operating system: Debian 10.5
Database (use [x]):

[ ] PostgreSQL
[ ] MySQL
[ ] MSSQL
[x] SQLite

Can you reproduce the bug at https://try.gitea.io:

[ ] Yes (provide example URL)
[x] No
[ ] Not relevant

Log gist:

gitea[10132]: [Macaron] 2020-09-16 22:45:50: Started GET /TestUser1/tmpGit/src/commit/23bfc188a8d3fed0156640551136192fbe2588d7/th for <IP_ADDRESS>
gitea[10132]: [Macaron] 2020-09-16 22:45:50: Completed GET /TestUser1/tmpGit/src/commit/23bfc188a8d3fed0156640551136192fbe2588d7/th 404 Not Found in 4.921573ms
gitea[10132]: [Macaron] 2020-09-16 22:48:26: Started GET /TestUser1/tmpGit/src/commit/23bfc188a8d3fed0156640551136192fbe2588d7/file%20with%20spaces.txt for <IP_ADDRESS>
gitea[10132]: [Macaron] 2020-09-16 22:48:26: Completed GET /TestUser1/tmpGit/src/commit/23bfc188a8d3fed0156640551136192fbe2588d7/file%20with%20spaces.txt 200 OK in 7.585426ms
Description
Gitea appears to incorrectly handle the "View File" option when viewing commits to files with spaces either in the file name, or in the containing directory structure.
Have tested in both FF 80.0.1 & Chromium 85.0.4183.102
When accessing a commit view, & selecting the "View File" option, it redirects to an incorrect link.
Example:

Filename: "file with spaces.txt"
"View File" link: http://<IP_ADDRESS>:3000/TestUser1/tmpGit/src/commit/23bfc188a8d3fed0156640551136192fbe2588d7/th
-- Result: 404
Correct Link: http://<IP_ADDRESS>:3000/TestUser1/tmpGit/src/commit/23bfc188a8d3fed0156640551136192fbe2588d7/file with spaces.txt
-- Result: 200 (page loads fine when manually browsing to this link)

Tested directly against gitea localhost:3000 bind, taking reverse proxy out of the chain.
Same git repo uploaded to try.gitea.io doesnt exhibit the same issue.
Screenshots

Duplicate of #12768 and should be fixed in 1.12.5 by #12776


Solr Rank Eval API

Instead of having Maven as runtime container, RRE could be wrapped in a Solr RequestHandler or (single query evaluation) in a SearchComponent.

Big +1 to this idea.  Maven implies a very narrow definition of how search apps are built, and ups the integration points into your existing app...!


"Generic type 'AggregateQuerySnapshot' requires 1 type argument(s)."

My Angular app won't compile with rxfire 6.0.5. The error is
Error: node_modules/rxfire/firestore/lite/interfaces.d.ts:8:29 - error TS2314: Generic type 'AggregateQuerySnapshot<T>' requires 1 type argument(s).

 8 export type CountSnapshot = lite.AggregateQuerySnapshot<{
                               ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
 9     count: lite.AggregateField<number>;
   ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
10 }, any, DocumentData>;
   ~~~~~~~~~~~~~~~~~~~~~

This is with Firebase 9.23.0. I can't get Firebase 10 to work, see my StackOverflow question.

Hello, I encountered with the same issue.
Temporarily, I've changed my package.json to force it to use 6.0.4.

Why is rxfire trying to use Firestore Lite? I never imported anything from firebase/firestore/lite.

Same issue, I temporarily fixed it by forcing it to 6.0.4.

Same issue here while using @angular/fire - fixed by installing rxfire 6.0.4 too.

Same issue here with @angular/fire but in my case installing rxfire 6.0.4 didn't help. I tried to install 6.0.3 because this was my previous version in package-lock but it didn't help too so I had to restore my whole previous package-lock where I use rxfire 6.0.3, so probably some other package makes issue too

I think the issue here is that<EMAIL_ADDRESS>introduced a breaking change and now requires firebase@10 to function. So sticking to<EMAIL_ADDRESS>with firebase@9 is the solution.

I think the issue here is that<EMAIL_ADDRESS>introduced a breaking change and now requires firebase@10 to function. So sticking to<EMAIL_ADDRESS>with firebase@9 is the solution.
Adding this override to package.json resolves the issue for me:
{
  "dependencies": {
    "@angular/fire": "~7.6.1"
  }
  "overrides": {
    "<EMAIL_ADDRESS>{
<EMAIL_ADDRESS>"6.0.3"
    }
  }
}

I had commented about my temporary fix by forcing my app to use 6.0.4 but this is more reliable solution.
I was getting unexpected firebase integration errors such as "FirebaseError: Firebase: No Firebase App '[DEFAULT]' has been created - call initializeApp() first (app/no-app)". Then I tried to force 6.0.3 and these errors are also solved.
Thanks rgant.

This seems to be happening if you're using Firebase SDK Version < 10.
I was having this problem when I was using it with  Firebase SDK version 9.23.0 and then I updated the Firebase SDK to version 10.5.2 and the problem went away.
This means that the issue is with rxfire module that failed to be backward compatible to its complimentary Firebase SDK Version, specifically speaking in respect to the AggregateQuerySnapshot class that expects 2 different types of data for the 2 versions, hence the error.
export type CountSnapshot = lite.AggregateQuerySnapshot<{     count: lite.AggregateField<number>; }, any, DocumentData>; This is only compatible with Firebase SDK versions >10 as it expects 3 generic type datas and failed to be compatible with versions less than 10 that only expect 1 generic type data for AggregateQuerySnapshot.

@eyoeldefare See the overrides necessary: https://github.com/FirebaseExtended/rxfire/issues/88#issuecomment-1715851191

  "overrides": {
    "<EMAIL_ADDRESS>{
<EMAIL_ADDRESS>"6.0.3"
    }
  }

I tried this, I get Override for<EMAIL_ADDRESS>conflicts with direct dependency error when I perform npm install

@Steve-from-Memphis

I tried this, I get Override for<EMAIL_ADDRESS>conflicts with direct dependency error when I perform npm install

Your "dependency": { "@angular/fire": "version` must match the override version.


:sparkles: add new category

what is the difference between potential and optional?

@shawn-hurley so this is a category for violations that found a match for a certain condition but couldn't generate valid output (this is one example I found in windup report, there could be more) . For instance, an XML condition matched in an OR caused as a result of builtin.xml but the file was malformed and a valid violation couldn't be created for condition that followed builtin.xml

I wonder if this is a second set of things that are errors? we already need to add a "ran but didn't find any issues" rules as well

@shawn-hurley makes sense to me. lets close this PR for now, I'll follow up with an issue instead


Suggestion in registering sections using Modules for 1.x

Working with section from theme-scripts is a bit confusing at first, but I think this has to do with the fact that sections are instantiated, but are a wrapper of sorts.  Take for instance https://github.com/Shopify/starter-theme/blob/9ea85b5ac9e04cb37b892d2a1c0302164e9bc863/src/assets/scripts/sections/product.js#L40, it registers this section, at the same time it runs it. I get this is almost like a class constructor, but it leads to little portability of what OOP can do for us.
Not sure if this goes against the opinionated nature of slate, but In my themes I am now using OOP in the following way:
in templates/product, which "inits" product-template.js
$(document).ready(() => {
  sections.load('product');
});

in sections/product-template.js i am now doing something like:
import { Product } from "../classes/Product";

const ProductTemplate = {
  onLoad() {
    this.Product = new Product(this.$container);
  },

  onUnload() {
    this.$container.off(this.namespace);
  }
}
// Similar to how module exports works
sections.register('product', ProductTemplate);

Then in ../classes/Product I am able to be a bit more free, and not tied to the sections.register function.
class Product {
  constructor($container) {
    ...
  }
}

This pattern gave me a bit more freedom. It might not be for everyone.

@el-rotny Could you reopen this in https://github.com/Shopify/theme-scripts as it's more particular to that repo? I have also been considering of using Classes with theme-sections.js... Happy to hear someone else thinking of it!

No problem. Thanks @t-kelly


3.10: Added additional cursor stats properties.

Added the following properties:
cursorsCreated
cursorsRearmed
cacheHits
cacheMisses
https://github.com/arangodb/docs/blob/main/3.10/release-notes-api-changes310.md#cursor-api

Codecov Report

Merging #217 (d2f41f3) into main (6da57e0) will decrease coverage by 0.10%.
The diff coverage is 50.00%.

@@            Coverage Diff             @@
##             main     #217      +/-   ##
==========================================
- Coverage   99.86%   99.75%   -0.11%
==========================================
  Files          26       26
  Lines        3731     3739       +8
==========================================
+ Hits         3726     3730       +4
- Misses          5        9       +4

Impacted Files
Coverage Δ

arango/cursor.py
96.92% <50.00%> (-3.08%)
:arrow_down:

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


Questions: Function composition and type hints

Can the map function be overloaded with a kernel from a language like numba? It seems that function composition is a more general form of broadcasting and map.
Also, can kernels be selected based on other host type systems besides ndtypes?  @sklam talked about translation, but it seems like 484, protocols etc have more granularity than ndypes, so you'd be losing data.
Otherwise we have yet another array type system in python, when there is some talk on standardizing on mypy stuff: https://github.com/python/typing/issues/516

Indeed, new kernels can be dynamically inserted into the lookup table.  One of the goals is to add kernel jit compilation + insertion to Numba.
Function composition is a bit of an orthogonal issue: For jit compiled functions it seems less useful (just write a new function), for precompiled C kernels it could be added, but making it fast without temporary xnd containers is not trivial.
A couple of points about PEP 484:

Mypy is more of a powerful linter than a real type checker. We need 100% accuracy, which mypy does not provide.

Datashape is far more suitable for low-level C types than PEP 484, which does not mention them at all.

Datashape types include array sizes, which makes them dependent types.  Static array bounds checking in general is undecidable, so mypy cannot help, even if PEP 484 did include C types.

More points about PEP 484:
4: Datashape types ndt_t contain full memory access information, including alignment. They are used to traverse memory much in the way a compiler generates code based on the types of data structures. Thus, the types need to be a) in C to be fast and b) not bloated so the code is readable.

Sure. The other problem, aside from standards proliferation, is that  many real programs have abstraction and type hierarchy, rather than just a collection of functions.
For example, the distribution hierarchy in Pymc3.
I don't see how one can do  fast type hierarchy programming (use dispatch to  organize code on type lattice) with ndtypes and gumath.  It seems like somehow interfacing with pep 484 (not mypy) is one way to do this.
Regarding the obstacles you mentioned:

the type hints don't have to use mypy, the annotations are available for any program to use.
pep 484 protocols can map to type kinds and lower level types can be written represent concrete ndt types. (ie write an int64 class). Then the int64 implements the int abstract protocol, the right kernel is called  and numba can generate code at function call time for the concrete argument.  Then someone else can call it with int32.

Same with a bayesian package with functions over "abstractparametricdistribution" which I will call with "myconcretedistribution".

there is talk of array sizes are being added to typing (static checking with mypy is different).
alignment etc could be a keyword

Gufuncs are multimethods.  If f() should take {int16, int32, ...}, one has to add kernels for all signatures.
Datashape also had the Signed kind for all signed integers, so if a function has that signature, Numba could possibly generate kernels on the fly.
I'm not sure what you mean by "fast type hierarchy programming".  Selecting the multimethod is done purely by switch statements on the C level, using __annotations__ on the Python level should be much slower than that.
If people insist on shoehorning datashape into the (IMO bulky) PEP 484 syntax, this is a start:
from ndtypes import *

class NdtInt(object):
    def __init__(self):
        self.t = ndt("Signed")

class NdtInt64(object):
    def __init__(self):
        self.t = ndt("int64")

class NdtTuple(object):
    def __init__(self, *args):
        s = '(' + ', '.join(str(x.t) for x in args) + ')'
        self.t = ndt(s)

But I don't think that such a syntax is appropriate for huge nested types like the Lahman database:
http://matthewrocklin.com/blog/work/2014/11/19/Blaze-Datasets

It's not, but I'm not talking about a date use case. How would the user write a custom type? How would you deal with the distribution example I provided or defining gufuncs on something like the pymc type hierarchy : https://github.com/pymc-devs/pymc3/blob/master/pymc3/distributions/continuous.py

@datnamer Here's a concrete example for defining new functions and types:
https://github.com/plures/gumath/issues/4

This is great.
We could also use examples of how to define a new primitive type -- like a
bfloat8 or fixed width float.
Travis
On Fri, Mar 30, 2018, 3:33 PM Stefan Krah<EMAIL_ADDRESS>wrote:

@datnamer https://github.com/datnamer Here's a concrete example for
defining new functions and types:
#4 https://github.com/plures/gumath/issues/4
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/plures/gumath/issues/1#issuecomment-377616627, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAPjoGgAZPSUaTzGTbxRyZX2uVem_48qks5tjpavgaJpZM4SQM6X
.


Win32Exception: Permission denied on Azure

Good morning,
I'm trying to launch library on Azure WebApp. Unfortunately all the time I have an error:
System.ComponentModel.Win32Exception (13): Permission denied    at Wkhtmltopdf.NetCore.GeneratePdf.GetByteArray[T](String View, T model)
For test purpose I injected also IRazorViewToStringRenderer class from the library to check is HTML rendered properly
await this.razorViewToStringRenderer.RenderViewToStringAsync("Views/Home/Index.cshtml", results);
At this moment everything working good, method is returning rendered HTML.
Problem is starting when I try to call
await generatePdf.GetByteArray("Views/Home/Index.cshtml", results);
About environment:
Everything is deployed by Azure Pipelines for sure I'm giving there 755 chmod for Linux/wkhtmltopdf and Windows/wkhtmltopdf.exe files.
WebApp is launched on App Service Plan S1 with Linux environment

Hi!
Locally in your pc is running fine? what version of the library are you running?

Yes, locally everything works perfect, latest version

Hi @fpanaccia,
Any updates in this area ?

Same happens for AWS

@fpanaccia I have the same problem

@fpanaccia Any news about this?

I had the same problem, but making wkhtmltopdf executable helped me. I wrote in console:
chmod +x /path_to_app/Rotative/Linux/wkhtmltopdf
Maybe it will help someone. @adriantheking are you really sure Azure Pipelines gives 755 chmod for wkhtmltopdf?

@lookinsidethebox Yes I double checked it. How you running your application ?

@adriantheking I publish the application with MS Visual Studio in a local folder, manually copy files from the folder to my server (Alt Linux 8.2) and restart the application with supervisor.
Have you tried asking Azure Support? Maybe you need special permissions to run scripts on your server.

This errors is expected on azure since azure doesn't let you execute .exe files on the app services.
There is no way to fix this unless you run your own app inside a own VPS or this library changes to use .dll directly (which lead in a total source code refactor) instead of executing the .exe.


chore: adding bug issue template

What does this PR do?
Screenshot / video of UI

What issues does this PR fix or reference?
Fixes https://github.com/containers/podman-desktop-extension-ai-lab/issues/959
How to test this PR?

cc @jeffmaury


DefaultServeMux

In the standard library, passing nil to ListenAndServe makes use of DefaultServeMux():
http.ListenAndServe(addr, nil)

Negroni UseHandler doesn't work this way:
n.UseHandler(nil)

[negroni] PANIC: runtime error: invalid memory address or nil pointer dereference

I didn't notice this was the problem right away because I had my own little convenience function much like Run that happened to be receiving a nil mux.
Anyway, not sure if this is something that needs to be fixed or documented. Just thought I'd point out the issue I ran into when first adopting Negroni. Thanks Jeremy.

Maybe even just a better error from UseHandler instead of panicking down the line?

@nathany this is a good point, I will leave this issue to remind me to update the documentation

I think maybe just panic()ing right away with a message indicating a nil handler is the best approach here. The alternative is to just exit the method early (skipping the pushing of a handler onto the stack), but I think that behavior is less expected by the user (e.g. they might think that they are passing in a non-nil handler that is being ignored).

To add a bit more context, I think negroni.Use is fundamentally different than http.ListenAndServe. The latter is a convenience function that is called to start a webserver (with the DefaultServerMux if no handler is specified), whereas negori.Uses sole purpose is to push a handler onto the stack -- i.e. you wouldn't call it if you weren't trying to push a handler onto the stack -- so pushing a nil should be unexpected.

Did not mean to close :smile:

@jszwedko I agree, I think it just needs a better error message.

Thanks


Support for numpy

It is rather difficult to get numpy support since it requires it to be compiled inside the docker image. Adding numpy to the requirements.txt for these alpine images causes an error like:
ERROR: Command errored out with exit status 1: /usr/local/bin/python /usr/local/lib/python3.7/site-packages/pip install --ignore-installed --no-user --prefix /tmp/pip-build-env-xjbzgbxn/overlay --no-warn-script-location --no-binary :none: --only-binary :none: -i https://pypi.org/simple -- setuptools wheel 'Cython>=0.29.13' 'numpy==1.13.3; python_version=='"'"'3.6'"'"' and platform_system!='"'"'AIX'"'"'' 'numpy==1.14.5; python_version>='"'"'3.7'"'"' and platform_system!='"'"'AIX'"'"'' 'numpy==1.16.0; python_version=='"'"'3.6'"'"' and platform_system=='"'"'AIX'"'"'' 'numpy==1.16.0; python_version>='"'"'3.7'"'"' and platform_system=='"'"'AIX'"'"'' Check the logs for full command output.

Could you provide a build option like (not sure if all packages are necessary):
  - name: numpy
    packages:
      - gcc
      - gfortran
      - python
      - python-dev
      - py-pip
      - build-base
      - py3-numpy
      - wget
      - freetype-dev
      - libpng-dev
      - openblas-dev

Hi @Anderssorby I don't believe you have the right mix of packages there.
However this is a moot point because for numpy we do not recommend Alpine Linux.
You can use the python3-debian template to get numpy, or wait for #26 to be merged and then use that if you need the http-style template.
Alex


Helm Chart for Deployments

What would you like to be added: Helm chart for standardised deployment
Why is this needed: Helm is widely used in the k8s community for deploying resources across clusters. This would allow for teams to bake existing workflows in things like terraform, ansible, etc. more easily than the current Makefile, as well as allow more customisation where needed for teams that may have different access control needs or configuration needs wrt some of the resources created for this.

Looking at the Makefile, I don't think this is an insurmountable task at all :D Looks like the only iffy part might be the cert generation?
If I can find the spare cycles, I might just pick this one up myself.

Hey thanks, I think a Helm chart would be great, and I'd be happy to be a co-maintainer on one in https://github.com/helm/charts/. I think we'd want to keep it in the helm/charts repository so we can get all the CI/release goodness that is automatically setup there

@micahhausler agreed on putting it in the helm repo, figured I'd file the FR here for tracking for the EKS side of the house. (:

I think the name is great

🎉

you should probably poke some org admin who can invite the listed OWNERS. I'll have @jqmichael and @nckturner comment on the chart PR with their owner/org request status

I'll try but I don't see much movement in that area these days. It seems it's always the same people (and bottleneck) for reviewing chart PRs.

NB, for anyone arriving here: @max-rocket-internet has closed the initial PR opened in the main helm repo in favour of one in the aws repo: https://github.com/aws/eks-charts/pull/28 (also opened by him). Upvote it there (bearing in mind it is stalled on an a docker image build).

For continuation, there's a new PR in the aws repo now that a docker image has been published: https://github.com/aws/eks-charts/pull/286

In meantime, I provide a helm chart which optionally support the cert-manager or static generated tls certificates.
https://github.com/jkroepke/helm-charts/tree/main/charts/amazon-eks-pod-identity-webhook


[awx] chart deleted

Hi @luhahn ,
what happened to the helm chart ? just realized that it was totally removed and looked around for the reason or the new home but could not find any.
Best.
-Raji

Hi there,
since there are a lot of updates for AWX I’ve decided to end support, since the Chart had it’s flaws anyways.
However the true reason why i ended the Chart is, that there is now an awx Operator, which works much better than my old helm Chart. Even the newest AWX versions are supported here. Plus the Operator is maintained from ansible directly ??
https://github.com/ansible/awx-operator
However, if you want to i can reupload the latest Version of the Chart into the repo again.
Best regards,
Lucas
Von: @.>
Gesendet: Donnerstag, 11. März 2021 11:39
An: @.>
Cc: Lucas @.>; @.>
Betreff: [novumrgi/helm] [awx] chart deleted (#13)
Hi @luhahnhttps://github.com/luhahn ,
what happened to the helm chart ? just realized that it was totally removed and looked around for the reason or the new home but could not find any.
Best.
-Raji
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/novumrgi/helm/issues/13, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AOXDEVLR7X6QI3U5XR5SQMDTDCFUJANCNFSM4ZAASOIA.

no worries and thanks for the instantaneous response , much appreciated .
looking to stay with helm for now but will keep an eye on the redhat/ansible operator .
Thanks again .
Best.
-Raji

Hello, can you please share the awx helm chart with me?
This is my mailbox<EMAIL_ADDRESS>If the tone is offensive, please forgive me

Hi all, I've readded the helm chart for AWX since a few users are still using it. Still I recommend using the operator :)

tks a lot !!!!!! And thank you also for your suggestions!  <3
Lucas Hahn @.***> 於 2021年4月14日 週三 下午3:45寫道：

Closed #13 https://github.com/novumrgi/helm/issues/13.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/novumrgi/helm/issues/13#event-4591975311, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AJSQI74NXOEIBCWP6P56ZV3TIVB3NANCNFSM4ZAASOIA
.


"libdriver-event-schema.so" should be renamed to make it clear what it's for

Hello. I'm the Debian maintainer. I just pushed a new package, and I have a suggestion. All the other libscap libraries are called libscap-..... A user seeing this file in /usr/lib/ARCHITECTURE/ is more or less clear on what this file is (it'd be better if these files were called libfalco-scap-... or something, but this is close-enough. The libdriver-event-schema.so file is different from all the others, and have a very generic name. Can we please rename it to libscap-whatever?
Thanks.

Hi!
Thanks for opening this request!

(it'd be better if these files were called libfalco-scap-...

Well, Falco is not the only consumer of these libraries, therefore we actually used the library real name.

The libdriver-event-schema.so file is different from all the others, and have a very generic name.

I'll ask @gnosek that is doing a big refactor around that code.
Do we even need this lib nowadays?
Thank you very much!

Well, Falco is not the only consumer of these libraries, therefore we
actually used the library real name.

OK, that's reasonable. But "driver-event-schema" is too generic. Driver
for what?

IMO event-schema just should be an unconditionally static library (along with all the tiny libraries I created while refactoring) and simply link into libscap.so (there's zero value in having scap_error or scap_platform_util etc. as separate .sos IMO)


PrometheusPort in poller args is a string - should be an int

https://github.com/NetApp/harvest/blob/e63ae7fd1d78f157e3310e8460c5903734cd4b26/cmd/poller/poller.go#L758

@Hardikl  pls add testing steps if any. I am moving this to testme for now. If no testing needed then pls move to untested.

Verified the Prometheus Port updated to integer value from string. After the update verified all the port details are displayed in status command.
[root@scspo2403645002 harvest]# ./bin/harvest status
Datacenter            Poller                PID        PromPort        Status
+++++++++++++++++++++ +++++++++++++++++++++ ++++++++++ +++++++++++++++ ++++++++++++++++++++
umeng_aff300          umeng_aff300          26878      12990           running
F2240-127-26          F2240-127-26          26888      12991           running
+++++++++++++++++++++ +++++++++++++++++++++ ++++++++++ +++++++++++++++ ++++++++++++++++++++
Hence marking the issue fixed.


Added "BC_" prefix for Blueprintable Actor and Scene Components

Proposing to use the BC_ prefix for blueprintable actor and scene components.  I noticed that these were not mentioned anywhere at all, so I figured it made sense to add it to the Miscellaneous list.

In order for this to fit in with the others, I think it would be BPC_, however I and many others use BP_*Component, that is, having Component be a suffix. The reason for this is it matches the naming convention of every other component, and BPC_Component would be redundant.
But maybe BPC_ would be worthwhile. Leaving this issue open for discussion.

BPC_ is good.

After experimenting with BC_ and BPC_, I've found that they both affect sorting in the content browser in a way that inhibits my, my team's, and other's workflow (although minutely). This, however, could be a side effect that BP_*Component is already ingrained in our heads.
I'm not opposed to BC_ or BPC_, but I've decided to stick to (and add) BP_*Component and defer this to user forks.


lineToLabel throws an error in funnel plot

G2 Version: 3.0.1
Platform: all
Mini Showcase(like screenshots): https://antv.alipay.com/zh-cn/g2/3.x/demo/funnel/basic.html

This is caused by a bug in<EMAIL_ADDRESS>workaround is downgrade the dependency of it.


Key of regions cache is the name, not start key

Current coverage is 20.68% (diff: 0.00%)

Merging #19 into master will decrease coverage by 0.11%

@@             master        #19   diff @@
==========================================
  Files            22         22
  Lines          1784       1784
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
- Hits            371        369     -2
- Misses         1401       1403     +2
  Partials         12         12

Powered by Codecov. Last update eb761e3...393e4c4

fixed in https://github.com/tsuna/gohbase/commit/29d1d78bc0eb58da9eb390cac2b898a0642661e5. Closing the Pull request.


Current minimum stored after fit and current do not correspond with just calling jl.fit() with minuit

I got
13:59:20 ERROR     Current minimum stored after fit      joint_likelihood.py:366
                  (98625.011053318) and current
                  (98624.88334719518) do not correspond!

when trying to fit the HAWC data with hal. I had encountered this problem in other sources too.
I checked the minuit minimizer and the best-fit parameters. It seems like the best-fit parameters does correspond to the current minimum  98624.88334719518, but jl._minimizer.minuit.fval gives a different likelihood (98625.011053318).
The strange thing is I clearly saw that the _minimizer reach a value lower than 98625.011053318 during the fitting process.

Can you post more detail on the code you are using? Is it a generic problem?

Hi Michael
I can't reproduce the error with only publicly available HAWC data. However, it seems more like a problem with the minimizer, when I switch to ROOT minimizer, the error disappears which is strange since I thought they were both using minuit. Also, ROOT sometimes produces the same error, but the discrepancy in likelihood is much smaller(<10^-5).
Here is the number when I try to access the minimum of the joint likelihood object.
![image](https://user-images.githubusercontent.com/20786734/193081128-ab00560f-2fbe-4843-ab07-9548b8e80867.png)

The underlying code is still very different. We have never really understood why. Does the problem persist with different data? For example, you could generate some fake x-y data (examples in the docs) and try that. If the problem is still there I can have a look.

Hello, I was gonna open an issue about this problem but I see there's one already. I'm also using some private HAWC data. I noticed that my problem happen when I was trying to fit the position of a source. When fitting for the spectrum, I don't see issues. @jasonfan1997 what were you trying to fit when you see this error?

Hello, I also met the same problem when I fitted the position of a source with private HAWC data. I tried to track the errors.  The fit program uses  _compute_covariance_matrix (https://github.com/threeML/threeML/blob/7836d98943c5e7c6a8d8aa5fecde54cb15950700/threeML/minimizer/ROOT_minimizer.py#L179 ) to calculate covariance matrix after  _minimizer(https://github.com/threeML/threeML/blob/7836d98943c5e7c6a8d8aa5fecde54cb15950700/threeML/minimizer/ROOT_minimizer.py#L134)  gives the best fit result.  I find that after calculating the covariance matrix using Hesse in ROOT_minimizer.py , Hesse will give a new best fit result which does not correspond with the best fit result fitted by _minimizer. It seems that Hesse doesn't calculate the matrix based on the prior best fit result and has fitted again.  Also, minuit_minimizer.py has the same issue.  If I adopt the the function of  _compute_covariance_matrix  in minimization.py, the parent class of  ROOT_minimizer.py, the fit will success.


upgrade jasmine to 2.99, ts-node to 5.0 and protractor to 5.3.0

Upgrade the following devDependencies packages:
    "jasmine-core": "~2.99.1",
    "protractor": "~5.3.0",
    "ts-node": "~5.0.0",
    "codelyzer": "~4.1.0",

[35 / 47] Compiling TypeScript (devmode) @build_bazel_rules_typescript//internal/tsc_wrapped:tsc_wrapped [for host]; 1s processwrapper-sandbox

ERROR: /home/circleci/ng/packages/angular_devkit/build_optimizer/BUILD:12:1: Compiling TypeScript (devmode) //packages/angular_devkit/build_optimizer:lib failed (Exit 1)

packages/angular_devkit/build_optimizer/src/helpers/transform-javascript.ts(8,30): error TS7016: Could not find a declaration file for module 'source-map'. '/home/circleci/.cache/bazel/_bazel_circleci/9ce5c2144ecf75d11717c0aa41e45a8d/execroot/angular_devkit/external/build_bazel_rules_typescript_tsc_wrapped_deps/node_modules/source-map/source-map.js' implicitly has an 'any' type.
packages/angular_devkit/build_optimizer/src/build-optimizer/rollup-plugin.ts(16,30): error TS7016: Could not find a declaration file for module 'source-map'. '/home/circleci/.cache/bazel/_bazel_circleci/9ce5c2144ecf75d11717c0aa41e45a8d/execroot/angular_devkit/external/build_bazel_rules_typescript_tsc_wrapped_deps/node_modules/source-map/source-map.js' implicitly has an 'any' type.
  Try `npm install @types/source-map` if it exists or add a new declaration (.d.ts) file containing `declare module 'source-map';`

Seems to be unrelated to this change and all recent PR-s / respective builds seem to fail with a similar reason.
Independently - though,  types/source-map seems to be deprecated and the type definitions seem to be available directly in source-map as well.
npm WARN deprecated<EMAIL_ADDRESS>This is a stub types definition for source-map (https://github.com/mozilla/source-map). source-map provides its own type definitions, so you don't need @types/source-map installed!

The build seemed to fail initially with my PR although the failure seemed unrelated. Reverting #445 seemed to make the build green.
@alexeagle @hansl  - I am no expert on bazel. But FYI - so you can review what could be wrong.

Yes, master is red, I'm working on it


utils_cython not found error

After compiling the cython scripts, and running python sim.py gives the following error-
ModuleNotFoundError: No module named 'prediction.M2I.marginal_prediction.src.utils_cython'

After compiling the cython scripts, and running python sim.py gives the following error-
ModuleNotFoundError: No module named 'prediction.M2I.marginal_prediction.src.utils_cython'

Hello, I also encountered a similar problem. Have you solved it?

Try to also compile the utils_cython in marginal_prediction as follows?
cd prediction/M2I/marginal_prediction/src/ cython -a utils_cython.pyx && python setup.py build_ext --inplace

Try to also compile the utils_cython in marginal_prediction as follows? cd prediction/M2I/marginal_prediction/src/ cython -a utils_cython.pyx && python setup.py build_ext --inplace

Hello, I tried the above commands but found some errors as follows, have you encountered similar errors?

I think these are just some warnings from Cython that do not block the compiling process.

Thank you very much, but I still get No module named 'prediction.M2I.relation_predictor.src.utils_cython' error when I try to run python sim.py after compiling the utils_cython.

Could you run the same script in the relation_predictor/src folder and try again?
cd prediction/M2I/relation_predictor/src/ cython -a utils_cython.pyx && python setup.py build_ext --inplace


dex configuration and login support

dex+oidc should be configurable for CAPV clusters so that we can login with our tooling.

[x] Verify opsctl login works
[x] Verify SSO via browser works.

Check https://github.com/giantswarm/mc-bootstrap/pull/477. I guess it will fix dex-operator and login will be fine.

I enabled dex by getting support from Antonia. See
After providing the necessary oidc flags, I can log in and I can run kubectl commands after running opsctl login.
I updated the default values in LastPass. Everything is OK in CAPV side and I am closing this issue.


WIP: Telegram Bot: Add new commands like help, stop and add a view button after retweet; Tweepy auth function called to telegram class

To many components in 1 PR. split it up.


Stopwatch on Login page doesn't work

From VS Mobile Center:

OnNavigatedTo() doesn't get called 🤔


Streamline vcs calls

Previously, git was being called for each path comparison. This PR makes it so that git is only called once. It also removes the async calls in favor of a single lodash each statement.

In our project, with roughly 465 files being processed, we were getting Warning: spawn EMFILE Use --force to continue. errors. This PR is largely a result of fixing that error.


Image checker

Description of the change:
Plugin to check the image directory against all the linked images in markdown files to see which images are not currently in use.

Closes #3052
Note:
I have done this so far...please let me know if i have made any mistakes or suggest changes.

I think all my previous commits are still here...how do i remove them?

@tulikavijay - any luck with this?

@ksigler7 I am working out the old failed merges and previous commits right now.


Anyway i can ignore logs when find certain value in my customer property. Does this class support filtering

I have added following in my appender settings
   <parameter>
      <parameterName value="_context" />
      <dbType value="String" />
      <size value="25" />
      <layout type="log4net.Layout.RawPropertyLayout">
        <key value="Context" />
      </layout>
    </parameter>

    <filter type="log4net.Filter.PropertyFilter">
      <key value="Context" />
      <stringToMatch value="Transaction" />
      <acceptOnMatch value="false" />
    </filter>

What i am doing is logging lot's of scenario. But i wanted to skip logs going to db command when Context Property have value ="Transaction"
But it seems to be not working.
Please let me know what i need to correct
you can check my related question on stackoverflow
https://stackoverflow.com/questions/59751628/how-to-ignore-logs-when-some-property-have-certain-value-in-log4net

Hi Frank,
Pardon to Reopening the issue.
I were able to workout the configuration and it is like

One more question is if i want to put a case like
TransactionId is null or empty or txnId is null or 0
If that can type of OR statement possible here in filter configuration


MIP-588 cross validation prototype

Implementation of CV for Linear Regression. This is inefficient due to two reasons:

The implementation of KFold does a lot of calls to run_udf_on_local_nodes due to limitations in the current UDF generator.
Cross-validation is completely parallelizable and should be done asynchronously but it is currently done synchronously.

UDF generator new feature: when a UDF takes a relational table as input, the row_id is passed as well and it is used as the index of the corresponding dataframe.
UDF generator new feature: a new constant DEFERRED can now be passed instead of an output relation's schema. This allows the user to defer the declaration of the schema to runtime. She is then required to pass the desired schema as an extra argument to run_udf_on_local_nodes or run_udf_on_global_node. The implementation is temporary and should be redesigned once the UDF generator is refactored.
UDF generator enhancement: remove the variable name prefix in column names of relations/dataframes.
Test case generator enhancement: when a generated input makes no sense it can be skipped by the implemented of compute_expected_output by simply returning None.
Various small fixes

Closed in favor of https://github.com/madgik/MIP-Engine/pull/244


remove trailing whitespaces in missing spans

Close #3423

Build failures are unrelated to the PR.
Clippy fails to compile:
error[E0433]: failed to resolve: could not find `item_path` in `ty`
   --> clippy_lints/src/utils/mod.rs:105:10
    |
105 | impl ty::item_path::ItemPathBuffer for AbsolutePathBuffer {
    |          ^^^^^^^^^ could not find `item_path` in `ty`
error[E0433]: failed to resolve: could not find `item_path` in `ty`
   --> clippy_lints/src/utils/mod.rs:106:33
    |
106 |     fn root_mode(&self) -> &ty::item_path::RootMode {
    |                                 ^^^^^^^^^ could not find `item_path` in `ty`
error[E0433]: failed to resolve: could not find `item_path` in `ty`
   --> clippy_lints/src/utils/mod.rs:107:30
    |
107 |         const ABSOLUTE: &ty::item_path::RootMode = &ty::item_path::RootMode::Absolute;
    |                              ^^^^^^^^^ could not find `item_path` in `ty`
error[E0433]: failed to resolve: could not find `item_path` in `ty`
   --> clippy_lints/src/utils/mod.rs:107:57
    |
107 |         const ABSOLUTE: &ty::item_path::RootMode = &ty::item_path::RootMode::Absolute;
    |                                                         ^^^^^^^^^ could not find `item_path` in `ty`
error: aborting due to 4 previous errors

and Cargo fails on a test which seems to have been occurring for a while https://github.com/rust-lang/cargo/issues/6258.

Thanks!


New PR to test SignCLA at 2019-07-09 11:49:39.967379288 -0700 PDT m=+4.356508467

Body of PR

unsigned-fail-fork-staging: test succeeded


add maximum size for desktop

Preview environment created!
The preview environment has been deployed to the cluster.
Client URL: https://pr-1120--epp.elifesciences.org
Storybook URL: https://pr-1120--epp-storybook.elifesciences.org
Note: If this is a new environment, it can take a few minutes for DNS to update.

Preview environment images deployed!
The preview environment has been updated:
client image tag: preview-1120-2299ae2319847704f23a99b799362280f57d4715-20230905.1218
storybook image tag: preview-1120-2299ae2319847704f23a99b799362280f57d4715-20230905.1218
Client URL: https://pr-1120--epp.elifesciences.org
Storybook URL: https://pr-1120--epp-storybook.elifesciences.org
Note: Updated images can take a few minutes to roll out.

@will-byrne Can you just look over this. tests and linting have past.


Cleanup hasValue

Refactor slightly the flow

Warnings

:warning:

The README has not been updated. Please update the README.

Messages

:book:

We were able to automatically fix some formatting issues in this PR for you!

  Generated by :no_entry_sign: dangerJS

Nice. Thanks!


Date discrepancy: pg module query result differs from database value

When using the pg module in Node.js to query a resource with a date column, the retrieved date value is different from the expected value. For example, if the date column in the database has '2023-05-21' stored, the pg module query result is '2023-05-20T23:00:00.000Z'.

The issue has been resolved in the commit e374ceb


NEW [broker_test] add broker test in cmd way.

Codecov Report
Patch coverage: 5.46% and project coverage change: +14.43 :tada:

Comparison is base (f3163b4) 4.14% compared to head (fc6b7b3) 18.58%.

Additional details and impacted files
@@             Coverage Diff             @@
##           master    #1223       +/-   ##
===========================================
+ Coverage    4.14%   18.58%   +14.43%
===========================================
  Files          24       17        -7
  Lines        7140     5327     -1813
===========================================
+ Hits          296      990      +694
+ Misses       6844     4337     -2507

Impacted Files
Coverage Δ

nanomq/bridge.c
0.00% <0.00%> (ø)

nanomq/cmd_proc.c
0.00% <0.00%> (-12.50%)
:arrow_down:

nanomq/conf_api.c
0.00% <0.00%> (ø)

nanomq/rest_api.c
0.00% <0.00%> (ø)

nanomq/web_server.c
0.00% <0.00%> (ø)

nanomq/webhook_inproc.c
0.00% <0.00%> (ø)

nanomq/apps/broker.c
30.71% <19.69%> (+12.73%)
:arrow_up:

nanomq/pub_handler.c
55.08% <32.50%> (+54.18%)
:arrow_up:

nanomq/sub_handler.c
66.13% <50.00%> (+21.30%)
:arrow_up:

nanomq_cli/dds2mqtt/mqtt_client.c
63.17% <50.00%> (+63.17%)
:arrow_up:

... and 1 more

... and 10 files with indirect coverage changes

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


WIP feat(plugins): add new plugin tasks for EventHubs (#38)

This commit adds the type: io.kestra.plugin.azure.messaging.eventhubs.Produce

What changes are being made and why?

How the changes have been QAed?

Setup Instructions

Nice! I see you implemented functionality to consume messages from the EventHubs. Could that be implemented as a trigger in a similar fashion as we support SQS, PubSub or Kafka triggers?


Cannot read properties of undefined (reading 'init')

Cannot read properties of undefined (reading 'init')

Getting the same! 😢

Looks like this is getting deprecated: https://developers.googleblog.com/2021/08/gsi-jsweb-deprecation.html

Then how to solve this problem? who has a good code example?

i am getting the above error

Getting the same issue @naumansigma added.

Guys, please take a look of the google oauth doc
https://developers.google.com/identity/sign-in/web/sign-in?hl=es-419

I also faced this issue as this method is deprecated. I have re wrote my code with new flow

Add google identity script in your index.html

Add an element, keep in mind the id use this id in step 4

Create an callback function to take response from google, pass this callback in step 4

const handleCredentialResponse = (response) => {
   console.log(response)
    // you will get credential, response.credential
    // you can use jwt-decoder to decode the credential and get user details
    // pass this credentials to firebase to get firebase credentials and use it to signin
  const firebaseCredential = firebase.auth.GoogleAuthProvider.credential(response.credential);
  firebase
         .auth()
         .signInWithCredential(firebaseCredential)
         .then((user) => {console.log(user)}
  };

more about google credential https://developers.google.com/identity/gsi/web/reference/js-reference#CredentialResponse

On your login page add this code in your react component

useEffect(() => {
    const init = () => {
      window.google.accounts.id.initialize({
        client_id: process.env.REACT_APP_GOOGLE_CLIENT_ID_LOGIN,
        callback: handleCredentialResponse,
      });
    };
    init();

    // initiate a pop up to login
    window.google.accounts.id.prompt();

    // render login button
    window.google.accounts.id.renderButton(
      document.getElementById("googleSignInButton"),
      { theme: "outline", size: "large", width: "310px" }
    );
  }, []);

more render button options https://developers.google.com/identity/gsi/web/reference/js-reference#GsiButtonConfiguration

I also faced this issue as this method is deprecated. I have re wrote my code with new flow

Add google identity script in your index.html

Add an element,this is where a google sigin button will appear.
The identity script will insert a sigin button no need to add any button manually
Keep in mind the id use this id in step 4

Create an callback function to take response from google, pass this callback in step 4

const handleCredentialResponse = (response) => {
   console.log(response)
    // you will get credential, response.credential
    // you can use jwt-decoder to decode the credential and get user details
    // pass this credentials to firebase to get firebase credentials and use it to signin
  const firebaseCredential = firebase.auth.GoogleAuthProvider.credential(response.credential);
  firebase
         .auth()
         .signInWithCredential(firebaseCredential)
         .then((user) => {console.log(user)}
  };

more about google credential https://developers.google.com/identity/gsi/web/reference/js-reference#CredentialResponse

On your login page add this code in your react component

useEffect(() => {
    const init = () => {
     // pass the callback function created in step 3
      window.google.accounts.id.initialize({
        client_id: process.env.REACT_APP_GOOGLE_CLIENT_ID_LOGIN,
        callback: handleCredentialResponse,
      });
    };
    init();

    // initiate a pop up to login
    window.google.accounts.id.prompt();

    // render login button
   // use the element ID created in step 2
    window.google.accounts.id.renderButton(
      document.getElementById("googleSignInButton"),
      { theme: "outline", size: "large", width: "310px" }
    );
  }, []);

more render button options https://developers.google.com/identity/gsi/web/reference/js-reference#GsiButtonConfiguration

I also faced this issue as this method is deprecated. I have re wrote my code with new flow

Add google identity script in your index.html

Add an element,this is where a google sigin button will appear.
The identity script will insert a sigin button no need to add any button manually
Keep in mind the id use this id in step 4

Create an callback function to take response from google, pass this callback in step 4

const handleCredentialResponse = (response) => {
   console.log(response)
    // you will get credential, response.credential
    // you can use jwt-decoder to decode the credential and get user details
    // pass this credentials to firebase to get firebase credentials and use it to signin
  const firebaseCredential = firebase.auth.GoogleAuthProvider.credential(response.credential);
  firebase
         .auth()
         .signInWithCredential(firebaseCredential)
         .then((user) => {console.log(user)}
  };

more about google credential https://developers.google.com/identity/gsi/web/reference/js-reference#CredentialResponse

On your login page add this code in your react component

useEffect(() => {
    const init = () => {
     // pass the callback function created in step 3
      window.google.accounts.id.initialize({
        client_id: process.env.REACT_APP_GOOGLE_CLIENT_ID_LOGIN,
        callback: handleCredentialResponse,
      });
    };
    init();

    // initiate a pop up to login
    window.google.accounts.id.prompt();

    // render login button
   // use the element ID created in step 2
    window.google.accounts.id.renderButton(
      document.getElementById("googleSignInButton"),
      { theme: "outline", size: "large", width: "310px" }
    );
  }, []);

more render button options https://developers.google.com/identity/gsi/web/reference/js-reference#GsiButtonConfiguration

M getting error - > Property 'accounts' does not exist on type 'typeof google' Any idea

@suresh-webonise
did you load the  in index.html
can you console.log(window.google) and check what does it shows

any help here

@uzairsaddique it is a TypeScript error. You can use @ts-ignore to fix it for now.

Not fixed
Its a defer error
Removed in the script file
On Thu, 2 Mar 2023, 4:18 pm Myke Miroshnikov, @.***>
wrote:

@uzairsaddique https://github.com/uzairsaddique it is a TypeScript
error. You can use @ts-ignore to fix it for now.
—
Reply to this email directly, view it on GitHub
https://github.com/partnerhero/gapi-script/issues/29#issuecomment-1451707540,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AGLZQHYSEDYIO2VGJ3AYPJDW2B6Z7ANCNFSM6AAAAAAVFUUFYM
.
You are receiving this because you were mentioned.Message ID:
@.***>

Hey this is an issue for us too, did anyone find a package version which works here or found a pin that would fix the above error?

@uzairsaddique @krubot-sky
try remove defer and async
make sure script is loaded
or you can check if the script is loaded
useEffect(() => {
    const init = () => {
     // pass the callback function created in step 3
      window.google.accounts.id.initialize({
        client_id: process.env.REACT_APP_GOOGLE_CLIENT_ID_LOGIN,
        callback: handleCredentialResponse,
      });
    };

    // add a check here
    if(window.google){   <<<<--------- add this condition
      init();

      // initiate a pop up to login
      window.google.accounts.id.prompt();

      // render login button
     // use the element ID created in step 2
      window.google.accounts.id.renderButton(
        document.getElementById("googleSignInButton"),
        { theme: "outline", size: "large", width: "310px" }
      );
    }
  }, [window.google]);                    <<<<--------- add this dependency

Got it, thanks!
On Thu, 2 Mar 2023, 6:37 pm Tushar Kolhe, @.***> wrote:

@uzairsaddique https://github.com/uzairsaddique @krubot-sky
https://github.com/krubot-sky
try remove defer and async
make sure script is loaded
or you can check if the script is loaded
useEffect(() => {
const init = () => {
// pass the callback function created in step 3
window.google.accounts.id.initialize({
client_id: process.env.REACT_APP_GOOGLE_CLIENT_ID_LOGIN,
callback: handleCredentialResponse,
});
};
// add a check here
if(window.google){   <<<<--------- add this condition
  init();

  // initiate a pop up to login
  window.google.accounts.id.prompt();

  // render login button
 // use the element ID created in step 2
  window.google.accounts.id.renderButton(
    document.getElementById("googleSignInButton"),
    { theme: "outline", size: "large", width: "310px" }
  );
}

}, [window.google]);                    <<<<--------- add this dependency
—
Reply to this email directly, view it on GitHub
https://github.com/partnerhero/gapi-script/issues/29#issuecomment-1451877961,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AGLZQH4VB2GBL4EIWML2DDDW2CPBPANCNFSM6AAAAAAVFUUFYM
.
You are receiving this because you were mentioned.Message ID:
@.***>

I used this package instead:
https://github.com/i7N3/google-oauth-gsi
and it works perfect for my needs

@hyperparameters useEffect will not work for everybody because window.google shouldn't trigger the useEffect. On his case, probably it has worked because there's something else causing the component rerender that gets window.google ready to use.
Instead of a if(window.google) use window.onload:
useEffect(() => {
    const init = () => {
     // pass the callback function created in step 3
      window.google.accounts.id.initialize({
        client_id: process.env.REACT_APP_GOOGLE_CLIENT_ID_LOGIN,
        callback: handleCredentialResponse,
      });

      // initiate a pop up to login
      window.google.accounts.id.prompt();

      // render login button
     // use the element ID created in step 2
      window.google.accounts.id.renderButton(
        document.getElementById("googleSignInButton"),
        { theme: "outline", size: "large", width: "310px" }
      );
    };

    window.onload = function () {
      init();
    };
  }, []);

Taken from Google on this part of their docs: https://developers.google.com/identity/gsi/web/guides/display-button?authuser=3&hl=pt#button_rendering


in model.summary(), connected_to -> connected_from

I think it more makes sense since the connected_to shows from which layer(s) the current layer is connected from -- so that it specifies they are the inputs.

"Connected to" does not imply a direction, just a link. "Connected" is used with "to", not "from".


Warning Ergo config

Problem Statement
Hey team
Been running the watcher all well, performed the updates as required from v1 to v2. Un-stake, update and restake with the appropriate updates to local.yaml. When i restaked and run the watcher keep getting this in the logs:

Not sure how to resolve it.
What i have tried so far

Unstake again and re-stake on v2
Updated the starting block height way before
Updating the starting block height after the block indicated

Not sure if there is something else i should be doing

user advised to update the blockheight to 1221300 which seems to work


Fixes unit test for newer versions of psycopg2.

Version 2.8.0 of Psycopg2 introduces granular errors for queries. This commit fixes a unit test to assert on the specific error instead of the now-updated base class. Functionally, the patcher still works fine with the latest versions of Psycopg2.
The previous matching class was ProgrammingError. For the specific test, it is now UndefinedColumn, which is a subclass of ProgrammingError.
Issue #, if available:
#159
Description of changes:
Modified test to check for UndefinedColumn instead of ProgrammingError.
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Sanity check : Do we need to do modification in AWS SDK handler w.r.t to this change?

Nope this was a pure test error. Basically, Psycopg2 was updated so that they have more exceptions, so the only change that this'll do is that there are more types of exceptions that can be caught and stored in the segment.


KAFKA-7223: Add late-record metrics

Add late record metrics, as specified in KIP-328
Committer Checklist (excluded from commit message)

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation (including upgrade notes)

Something crazy happened with the core tests.
Retest this, please.

Test failures unrelated:
kafka.api.ClientIdQuotaTest.testThrottledProducerConsumer
kafka.api.ConsumerBounceTest.testClose
kafka.api.UserQuotaTest.testQuotaOverrideDelete

Retest this, please.

Made another pass over the PR. @bbejeck could you do a second review as well?

Thanks for the review, @bbejeck . Just one clarifying question...

Thanks for confirming @bbejeck .
I'll merge the PR once jenkins passed.

retest this please

jdk8 failed this time (unrelated). I've submitted #5792 to fix it.

Cherry-picked to 2.1.


Update README

Checklist

Please ensure the following tasks are completed before submitting this pull request.

[x] Read and understand the Code of Conduct.
[x] Read and understood the licensing terms.
[x] Searched for existing issues and pull requests before submitting this pull request.
[x] Filed an issue (or an issue already existed) prior to submitting this pull request.
[x] Submitted against master branch.

Description

What is the purpose of this pull request?

This pull request:

adds a second Binder badge to allow opening a demo notebook which uses the latest master.
updates the current Binder badge to point to the demo notebook, so that users can be immediately dropped into an environment to begin experimentation.
updates README copy.
adds prerequisites.
moves roadmap to separate file.

Related Issues

Does this pull request have any related issues?

No.
Questions

Any questions for reviewers of this pull request?

No.
Other

Any other information relevant to this pull request? This may include screenshots, references, and/or implementation notes.

No.

Wonderful, looks good. Thanks for moving the roadmap.


quill mentioned people list

Hey @MadSpindel @cgross @cwhite-connectfirst @eltimn
how do i get the list of mentioned people
suppose i have list of people
const list = ["john", "jack", "anna", "himanshu"]
and i mentioned @john and @jack so how do i get the list contains these two names
eg => mentioned = ["john", "jack"];
if i remove @jack from mentioned so my mentioned list should like this
eg => mentioned = ["john"]
How can i achieve this kind of functionality
Thanks

did you get the solution ?? i am really


Bad format for Python/starlark style doctring

Description
Now we have a docstring of format arg (type): desp, as there is a space after arg, it couldn'be recognized by LSP as starlark-lsp use regex ^(\S*)\s*:\s*(.*)$ to match line start.

Example:
def base(os: str, language: str, image: Optional[str]):
    """Set base image

    Args:
        os (str): The operating system (i.e. `ubuntu20.04`)
        language (str): The programing language dependency (i.e. `python3.8`)
        image (Optional[str]): Custom image (i.e. `python:3.9-slim`)
    """

According to https://stackoverflow.com/questions/3898572/what-are-the-most-common-python-docstring-formats , our docstring style likes Google, though it don't allow space ahead of chars at left of colon.
If we use format at image line 3, things could be better, and we could use LSP hover to display arguments.
Example:
def base(os: str, language: str, image: Optional[str]):
    """Set base image

    Args:
        os(str) : The operating system (i.e. `ubuntu20.04`)
        language(str) : The programing language dependency (i.e. `python3.8`)
        image(Optional[str]) : Custom image (i.e. `python:3.9-slim`)
    """

Related issue:
https://github.com/tensorchord/envd-lsp/issues/35
Reproduction
Additional Info

Message from the maintainers:
Impacted by this bug? Give it a 👍. We prioritise the issues with the most 👍.

It still could not handle docstring like:
commands (Dict[str, str]): map name to command, similar to Makefile as space inside type.
Maybe revised regex to upstream is better?

It still could not handle docstring like: commands (Dict[str, str]): map name to command, similar to Makefile as space inside type.
Maybe revised regex to upstream is better?

I think so. But you cannot use . here. It should start with a non-empty character.

It still could not handle docstring like: commands (Dict[str, str]): map name to command, similar to Makefile as space inside type.
Maybe revised regex to upstream is better?

I think so. But you cannot use . here. It should start with a non-empty character.

Thanks. I think ^(\S.*?)\s*:\s*(.*)$ might be enough.

closed by https://github.com/tensorchord/envd-lsp/pull/37


Support for Open API 3.x

Would like to know if support for Open API spec 3.x is in the cards. The current version of the spec is 3.0.2
https://github.com/OAI/OpenAPI-Specification/blob/master/versions/3.0.2.md

just pinging to get notifications

Much needed

Very much needed!

Agreed.

Very much needed!

any plans for this to be addressed?
Can we help out?

As mentioned in https://github.com/kongchen/swagger-maven-plugin/issues/881#issuecomment-1308395438, @kbuntrock created a plugin just for that: io.github.kbuntrock:openapi-maven-plugin


k3s 0.10.2 -> 1.0.0 upgrade: "failed to find any PEM data in key input"

Maybe I am not supposed to do this way, but I was testing the upgrade to version 1.0.0 in my cluster.
I run the update with only one of the worker nodes, and end up with:
Nov 19 11:01:45 0019d182f673.agrotis.io k3s[8952]: time="2019-11-19T11:01:45.544051881-03:00" level=error msg="tls: failed to find any PEM data in key input"
Nov 19 11:01:50 0019d182f673.agrotis.io k3s[8952]: time="2019-11-19T11:01:50.575954470-03:00" level=error msg="tls: failed to find any PEM data in key input"

repeating in loop.
(node end up not been listing by the master)
Wasn't I supposed to be able to upgrade each node at a time?

Server nodes should be upgraded first.

yeah, thats it. wrong upgrade order. Thanx


The "Shop by Filter" need to include taxes if the storefront has the "Tax Inclusive" option enabled

Describe the bug
The "Shop by Filter" need to include taxes if the storefront has the "Tax Inclusive" option enabled in admin.
To Reproduce
Steps to reproduce the behavior:

Add one product to a new Category
Make the price filterable in the new Category
Enable "Tax Inclusive" in admin
Go to the new Category store front
Filter the products by price
The product wont appear because the price including taxes in more than the maximum price generated in the function getCategoryProductMaximumPrice() in \packages\Webkul\Shop\src\Http\Controllers

Expected behavior
Maximum price should include taxes when the "Tax Inclusive" is enabled in store admin

Hello @emilthidell, Thank you for the information.

Fixed on #9711


docker: add some tweaks  to nvidia docker

From @kenhester suggestions
Should fix issues with #2023 as cupy-cuda12x seems to install and work fine inside the image
To be discussed, do we keep nvidia-clang in CI

To be discussed, do we keep nvidia-clang in CI

Probably we can drop it
NVC for openmp offloading on NVidia cards
Amdclang for openmp offloading on AMD cards
icx for openmp offloading on Intel cards
That's completely fine by me
IIRC, we're still stuck with our own fork of clang, so one more reason to drop it.
Can we try again NVC + openmp offloading, now that we finally have NVC 23.5 ?

Can we try again NVC + openmp offloading

Yes
Ok removing clang build and replacing by nvc


stacked.config warning should not be printed when creating an app

Currently when you run stacked create app my_app it prints out the warning that stacked.json is not found. We should remove this.

I think this warning is useful and helpful - please consider leaving it in. Not everybody reads documentation, and, as a matter of fact, that is exactly how I discovered that project structure can be adjusted to one's liking. The more the system is self-documenting, the happier its user is :)
I left a comment to this effect in our discussion in Slack as well.

I think this warning is useful and helpful - please consider leaving it in. Not everybody reads documentation, and, as a matter of fact, that is exactly how I discovered that project structure can be adjusted to one's liking. The more the system is self-documenting, the happier its user is :)
I left a comment to this effect in our discussion in Slack as well.

Regarding to point 1, we were talking yesterday with Dane and we will do some changes when time is available.
Thanks to your comments we realized that the use case you were talking about is a case which only happens as a CLI developer and is probably not needed as it is.
As a note, we are probably going to have 3 use cases in the config location.

Stacked CLI uses custom config, user conventions per project
Stacked CLI uses XDG_CONFIG_HOME config, global user conventions
Stacked CLI uses default config, Stacked conventions

Given that my issue was created with the wrong idea of the solution I'm closing this.


Validation within a transaction?

Prisma (especially Prisma2) shows a lot of potential to be a huge game changer and our team is excited about transitioning onto it. I'm posting here as requested from the Prisma Slack channel.
Is it possible to apply some pending changes in a transaction, run custom validation, then either commit those changes or rollback?
For example, given parent model Foo, which has a has-many relationship to Bar, and that Bar has an isPrimary boolean field, I need to validate that one Foo can have many Bar but only one of those Bar assigned to that Foo may have isPrimary set to true. What would be the recommended solution here?
Specifically, we need to run this validation before a mutation is committed to the database using nexus-prisma. As one possible solution, we were wondering if it would be possible to start a transaction, apply the changes of the mutation, validate the model, then either commit the pending changes or rollback.
Any advice on if this is or will be possible with Prisma2 would be greatly appreciated. Thank you!

@jmadson - Prisma will be implementing full support for long-running transactions, which should fully satisfy your use case. See the comment here: https://github.com/prisma/prisma/issues/1844#issuecomment-846477520
I will close this issue as a duplicate, but please reopen if you think there is some nuance that I am missing!


Add MessagePack codec

This adds MessagePack codec support.
To quote MessagePack, "it's like JSON but fast and small". I was wanting to make a param payload that included a decently-sized byte array inline, and JSON is pretty inefficient at that, so I added MessagePack support.
The CI hasn't finished running on my machine but I decided to go ahead and submit it anyways because the change is so isolated.

I think there is a deeper question here, of how many codecs we want to have available in leptos itself. I don't think this is an issue for MessagePack or this commit, but it's worth thinking about moving forward

Overall this looks good, thanks for contributing!

I think there is a deeper question here, of how many codecs we want to have available in leptos itself. I don't think this is an issue for MessagePack or this commit, but it's worth thinking about moving forward

I haven't worked with this package before, so someone please feel free to correct me, but from skimming the package...honestly...should something like this be a flat replacement in most cases (1:1)?
These little improvements towards an optimal Leptos sound wonderful, especially for a nosql or graphql DB when possible.

I think there is a deeper question here, of how many codecs we want to have available in leptos itself. I don't think this is an issue for MessagePack or this commit, but it's worth thinking about moving forward

I haven't worked with this package before, so someone please feel free to correct me, but from skimming the package...honestly...should something like this be a flat replacement in most cases (1:1)?
These little improvements towards an optimal Leptos sound wonderful, especially for a nosql or graphql DB when possible.

A flat replacement? This isn't really for db access, this is for the server fn encoding between server and client.


Parrallelize array_wrap

Describe the problem

We need to figure out what the __array_wrap__ method is used for.
Now that we have an implementation for __array__, we should add support for __array_wrap__. It can be found here
Source code / logs

I can't quite understand from the documentation what this function does. I haven't heard of anyone asking for it, so I'll assume it's low priority.

Deprecated function.


fix: peacefully ignore query files without a valid language

This fixes the error where out-of-tree queries are annotated with errors due to different directory structures
see https://github.com/NixOS/nixpkgs/issues/189838#issuecomment-1380814689
the screenshot in the original comment is from https://github.com/tree-sitter/tree-sitter-scala, and the query file is under a directory named queries, which is not a valid language name, and this pull request silently ignores it

@figsoda Does it mean that the directory structure should be changed to make the query linter work in the mentioned repository?

yes, it just silently ignores the error and playground doesn't have any other way of knowing the language AFAIK

I have checked couple of tree-sitter grammars and each one uses queries directory name :confused: . What should be the proper name then?

The language of the grammar (scala) would be the name that makes playground work, whether that's proper is up to your definition

Thanks all clear then. Would you like me to add that constraint to the readme under query-linter section?

feel free to do so, I'm not in charge of repo though so it would be up to the maintainers to merge it


TOF Matter Cluster (CON-1320)

Describe the question/query that you have
Is there a Matter cluster related to TOF (Time-Of-Flight) sensors?
Additional context
We have a hardware with TOF(Time-Of-Flight) sensors and we need a appropriate clusters for guide our implementation in a project
We research about it in many sites and repositories but we had no success..

@gustavof04 No, there is no such cluster.

You can download and check the latest released specifications from here for the supported device types and clusters.


amam/fix translation staging

Description
Fix all text translation crowdin
Type of change

[x] Bug fix
[ ] New feature
[ ] Refactor code
[ ] Update translation text
[ ] Breaking change
[ ] Dependency update
[ ] This change requires a documentation update
[ ] Release

📝Changed routes:

/about/_leaders
/about/_our-history

(view full size)
(view full size)

/about/_our-numbers
/about/_story_constant

(view full size)
(view full size)

/dbot
/dmt5

(view full size)
(view full size)

And 25 other routes:

/dtrader
/help-centre/_search-results
/help-centre/account
/help-centre/deposits-and-withdrawals
/help-centre
/help-centre/security
/help-centre/trading
/home/_what-our-clients-say
/markets/_available-options
/markets/_available-trades
/markets/_other-markets
/markets/_what-trade
/markets/_why-trade
/markets/forex/_forex
/partners/_about-deriv
/partners/_deriv-numbers
/partners/_hero
/partners/_why-choose-us
/partners/affiliate-ib/_deriv-ib-programme
/partners/affiliate-ib/_faq-data
/partners/affiliate-ib/_faq
/partners/affiliate-ib/_why-us
/partners/payment-agent/_faq-data
/partners/payment-agent/_faq
/payment-methods/_expanded-list

Commit 2eef7b5f0fd5ec217f9f173adfa186e759fb18c5 (https://deriv-com-git-fork-mustofa-binary-amam-fixtranslationstaging.binary.sx).