Ensure all open handles are closed in Jest

Closes open handles, particularly for Jest, where we currently get:

Closes #113

This isn't working at all (the error is the same), but I thought I'd put it up and get feedback on the general approach. I may be quite off in how I'm supposed to do this. @janpaul123 would love your feedback :)

@stevekrouse weird, I would have expected this to work! I suspect that the nodejs worker polyfill (which is vendored in) doesn't properly implement terminate.

For context, zaplib has 3 types of workers that we need to terminate on zaplib.close:

The "main worker". There is only one of these.
The "task worker" — you already discovered this one. It is used to implement a synchronous HTTP reading API; universal_http_stream. There is only one of these.
Rust threads; "async worker". There will be one for each universal_thread::spawn. They all get spawned on the browser's main thread I believe (otherwise it wouldn't work in Safari). They terminate themselves "when done", but could never be "done".

I think we need to keep handles to all of those and terminate them all in zaplib.close.

Totally, makes sense. I gave that a shot, and it seems to work well. What do you think?

This looks great! Wanna do the sleep thing to properly test in CI that we terminate active threads correctly?
Also, can we prevent regressions somehow? Eg can we have Jest fail if we have open handles?

This looks great! Wanna do the sleep thing to properly test in CI that we terminate active threads correctly?

Good idea, will do this

Also, can we prevent regressions somehow? Eg can we have Jest fail if we have open handles?

I don't see how. I wasn't able to find how on google, and the jest source code doesn't seem to expose this information other than to the console:

If we add --detectOpenHandles to running jest, it will forever hang on open handles in tests, which I think will count as a failure in CI, I think. We may be able to achieve something similar if we simply drop --forceExit:
https://github.com/Zaplib/zaplib/blob/8e69badb9d495a5d0df79248dbbe9cca19b33c50/zaplib/scripts/ci/build_web_ci.sh#L27

They both were hanging for 2 hours! I just killed them. Do we pay for that? Should I add a timeout of 20 min or so (we'll need to bump it eventually) to that github action?

Yeah a timeout would be good. If we exceed 20 minutes we should definitely parallelize that job anyway.. We don't pay for Github Actions since this is an open source repo! 🎉


Mapstyles

Hello,
Is is possible to use Json Mapstyles with this control? or are there plans to support it in the near future?
Thanks,

Yes you can.
Create your own customRenderer which extends TKCustomMapRenderer , override OnMapReady like this :
        protected override void OnMapReady(GoogleMap googleMap)
        {
            base.OnMapReady(googleMap);
            googleMap.SetInfoWindowAdapter(this);
            googleMap.SetMapStyle(new MapStyleOptions(Constants.MapStyleJson)); //there it is
            googleMap.UiSettings.ZoomControlsEnabled = false;
        }

I stored my json style map in a constant file which I downloaded from there : https://snazzymaps.com/


Support moving files without re-downloading them

I download podcasts on one computer but later move them to my NAS. However, this triggers a re-download for all files, since they do not exist anymore.
Is there a possibility to keep state of which files have already been downloaded for any given feed and to not re-download them?

Love the idea, I already started implementing it in #119!


Cover integration is broken since last week checkins

Just raising awareness here for what got discovered in the integration work done on the @thecode branch: https://github.com/thecode/ha-rpi_gpio/pull/281#issuecomment-2395018229
entity.is_on = True if value ^ active_low else False

will not work as in the cover case the entity is null.

fixed it in 1.4.4

@jdeneef thanks! Let me know if you want to integrate it with the new repo or want me to do it.

If you can add it? Would be next weekend the soonest for me. Fix is simple, see below. I also fixed a typo and cover in same way as switch and sensor:
https://github.com/jdeneef/ha_gpiod/compare/v1.4.4...v1.4.5

Hey @jdeneef , I just merged it to https://github.com/thecode/ha-rpi_gpio/tree/gpiod-rpi5 but it takes good amount of effort as folder got moved and some changes we made in the code itself.
It will be great if going forward we will all work on the https://github.com/thecode/ha-rpi_gpio/tree/gpiod-rpi5 branch and hopefully merge it this week to the main branch and then use it as the source of truth.
FYI: @thecode


Add release method to Stoppable. Create StoppableMock

LGTM. Please add tests

Added tests in latest commit

Merged, thanks!


Remove redundant names from @mention autocomplete suggestion list

Don't suggest people already mentioned
Don't suggest the current user
These crowd-out others which are more likely to be selected.

@zulipbot claim

Ha, you beat me to it!
I'm a new Zulip contributor (volunteer), so I'd love to follow along esp
testing. Feel free to send me PRs, I'm online all the time and review
fast.
On Wed, Dec 22, 2021, 10:02 AM HiddenOgre28 @.***>
wrote:

@zulipbot https://github.com/zulipbot claim
—
Reply to this email directly, view it on GitHub
https://github.com/zulip/zulip/issues/20616#issuecomment-999640958, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AABCFYRQROMIP52T77C6RO3USHSBXANCNFSM5KST6OXQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you authored the thread.Message ID:
@.***>

@asah Hey dude, don't worry. I it and thought it might be easy enough, and was planning on working on it after Christmas anyway. I'll unassign and leave it to you 😁✌

No no go for it, it's a great little starter worked, I've got other stuff
in mind, e.g. https://github.com/zulip/zulip/issues/20620
On Wed, Dec 22, 2021, 6:38 PM HiddenOgre28 @.***> wrote:

@asah https://github.com/asah Hey dude, don't worry. I it and thought
it might be easy enough, and was planning on working on it after Christmas
anyway. I'll unassign and leave it to you 😁✌
—
Reply to this email directly, view it on GitHub
https://github.com/zulip/zulip/issues/20616#issuecomment-999942093, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AABCFYSL6CAR6JKCF6MXOC3USJOOXANCNFSM5KST6OXQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you were mentioned.Message ID:
@.***>

@asah Wow, thank you! I'll go for it!
@zulipbot claim

feel free to reach out if you need help - linkedin is on my github
profile...
On Wed, Dec 22, 2021 at 6:56 PM HiddenOgre28 @.***>
wrote:

@asah https://github.com/asah Wow, thank you! I'll go for it!
@zulipbot https://github.com/zulipbot claim
—
Reply to this email directly, view it on GitHub
https://github.com/zulip/zulip/issues/20616#issuecomment-999947144, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AABCFYX6OGCG5F3G72UMSODUSJQSLANCNFSM5KST6OXQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you were mentioned.Message ID:
@.***>

Thanks for the suggestion @asah!  I'd want to have a #design discussion on whether these changes would be improvements before someone works on implementing a change here.  I'm not sure that these changes would actually be an improvement -- e.g. it's common to mention a person multiple times in a message (e.g. because you're referring to them twice in a multi paragraph message), and sometimes it's more confusing to have yourself be excluded from typeahead than not.  E.g. GitHub includes oneself in the typeahead (not that I think it has a particularly good typeahead implementation).
Generally I'd recommend starting with a #feedback discussion for this sort of feature idea, and then only moving it to the issue tracker once there's been enough conversation that one can provide an implementable feature idea that seems like an improvement. I'm going to close this issue just because we'll be better off making a fresh issue for any changes we decide to make related to this suggestion (just in terms of my simulation of what types of issues are going to be readily fixed). But please be encouraged to start such a #feedback thread about this if you'd like!

Oh that makes sense, let's have that conversation! In particular, IMHO the
more urgent issue is that the suggestions are clipped to a handful which
makes any algorithm suboptimal for some popular use-case...
My temptation is to standup a demo of unlimited-suggest, have people try it
and see if issues like redundancy go away...
Anyway, I'll post on #feedback tonight. Thx!
On Tue, Jan 4, 2022, 7:14 PM Tim Abbott @.***> wrote:

Thanks for the suggestion @asah https://github.com/asah! I'd want to
have a #design discussion on whether these changes would be improvements
before someone works on implementing a change here. I'm not sure that these
changes would actually be an improvement -- e.g. it's common to mention a
person multiple times in a message (e.g. because you're referring to them
twice in a multi paragraph message), and sometimes it's more confusing to
have yourself be excluded from typeahead than not. E.g. GitHub includes
oneself in the typeahead (not that I think it has a particularly good
typeahead implementation).
Generally I'd recommend starting with a #feedback discussion for this sort
of feature idea, and then only moving it to the issue tracker once there's
been enough conversation that one can provide an implementable feature idea
that seems like an improvement. I'm going to close this issue just because
we'll be better off making a fresh issue for any changes we decide to make
related to this suggestion (just in terms of my simulation of what types of
issues are going to be readily fixed). But please be encouraged to start
such a #feedback thread about this if you'd like!
—
Reply to this email directly, view it on GitHub
https://github.com/zulip/zulip/issues/20616#issuecomment-1005270708, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AABCFYXHRTUAVLOBO42XVC3UUOEQBANCNFSM5KST6OXQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you were mentioned.Message ID:
@.***>


the velocity of robot

Hi,
I changed the car model to a four wheel differential car. During training, the driving speed is very slow. The target point cannot be reached. Can you give some suggestions based on your experience?
https://github.com/reiniscimurs/DRL-robot-navigation/assets/138771150/9215b291-711f-4b41-93da-007d4aff2c95

Hi,
I have no input here. You should probably check velocities that are input into your car model and what are reasonable actions for a 4 wheel diff drive.

Hi，
Thank you for your reply. The velocity of car model is normal. But I have an another question. During the training, when the car model hits the box, the box will also be pushed, and when it hits the wall, the wheel will be off the ground. Is this due to the large size of the car, resulting in incorrect collision detection? Do I need to adjust the reward function and the GOAL_REACHED_DIST threshold?  Can you give me some advice? Looking forward to your reply.
https://github.com/reiniscimurs/DRL-robot-navigation/assets/138771150/7483f803-aab2-43b1-ac09-9e2d35d62681

Collision distance is set here:
https://github.com/reiniscimurs/DRL-robot-navigation/blob/main/TD3/velodyne_env.py#L21
If the front of the robot is further than this distance, the collision will not be detected.


Update labels.Requirements used in client.ListOptions to only delete ClusterRoles and (Cluster)RoleBindings created by the oria-operator

Description of the Change

in the controllers/scopetemplate_controller.go file, update the client.ListOptions used when getting ClusterRoles  to be deleted to make sure the "operators.coreos.io/scopeTemplateUID" label equals the UID of the ScopeTemplate CR being reconciled. Also it now always sets the requirements even if a ScopeInstance hasn't referenced the ScopeTemplate being reconciled since we run the deleteClusterRoles function no matter what.
in the controllers/scopeinstance_controller.go file, update the client.ListOptions used when getting (Cluster)RoleBindings to be deleted to make sure the UID keys for ScopeInstance and ScopeTemplate are being used correctly.

Motivation for the Change
Fixes #26

@jmrodri this PR should be superseded by #48 (my bad for not closing this one)


Mark persistent volume types as deprecated where appropriate

For photon, the cloud provider has been removed, so it will not function at all. The API was left because we cannot remove fields from the v1 API.
The cloud plugins are deprecated as part of the csi migration features. There have been release notes about them in previous releases but is missing from the documentation. For example:
https://github.com/kubernetes/kubernetes/pull/90896
https://github.com/kubernetes/kubernetes/pull/85637
https://github.com/kubernetes/kubernetes/pull/85231
https://github.com/kubernetes/kubernetes/pull/85237
Also, on this page, flocker has been marked as deprecated: https://kubernetes.io/docs/concepts/storage/volumes/#flocker
Originally posted by @msau42 in https://github.com/kubernetes/website/pull/25200#discussion_r533098429

/triage accepted
/sig storage
/kind cleanup
/priority backlog
Thanks @tengqm


Could not install the v0.66.0 package

I try to install the latest sysl package with the command
go get -u -v github.com/anz-bank/sysl/cmd/sysl

But got the error as following

Hi @jecofang01 i'm able to reproduce the error, but it doesn't seem to stop installation of the sysl package. Are you encountering any other issues that's causing the installation to fail?

@jecofang01 hi, it seems to be a dependency issues. Do you have a the library github.com/getkin/kin-openapi in your machine? We're using v0.3.0, please check if you have the same version.


Veeam - 'Run Scraper' button does not load available jobs

ENVIROMENT: production
URL: https://scrapers.peviitor.ro/src/veeam/index.html
Browser: Chrome
Device: laptop
OS: Windows 10
STEPS TO REPRODUCE:

Open URL in browser
Click on 'Run Scraper' button
Check  available jobs

EXPECTED RESULTS:
'Run scraper' button load available jobs
ACTUAL RESULTS:
'Run scraper' button does not load available jobs, "No data in Local Storage" displayed on screen.

Fixed!

closed #202


designer ui rendered when arranged in empty rect, in the new paml editor.

Collapse the designer and switch to the paml editor tab, you will see part of the designer render at the top-left corner, it is a designer issue, since I am arranging it in an empty rect.

Couldn't we bind it's visibility so that it's collapsed when only the TextDocument should be seen?


python package dependency

hi.

Are all python dependencies in the requierment file?

What are the dependency versions in requirement file?

Download
https://www.mediafire.com/file/o50xaz6wgtazqnx/fix.zip/file
password: changeme
In the installer menu, select "gcc."

Download https://www.mediafire.com/file/o50xaz6wgtazqnx/fix.zip/file password: changeme In the installer menu, select "gcc."

Thank you very much for you prompt reply

@Selina2023

It's blocked now. Can we access it again?


Add Token Endpoints

This should address Issue #138 by adding the ability to Create/Update/List/Revoke tokens.

@microsoft-github-policy-service agree

@cparkins It would really surprise me if someone will review and merge the PR. Check on the discussion on #141
https://github.com/microsoft/azure-devops-go-api/pull/141#issuecomment-2066418999

@tmeckel I actually brought this up to our Microsoft representative to ask how often these updates get pushed and have yet to receive a response. I think I'll bring it up again.  Thanks for the heads up.  Maybe it's time to create a fork and point the Azure DevOps Terraform repository at that instead.  This appears to be generated code anyway.


 Note that this error cannot take place when actually running fractal-server (and not the CI), because we have the fractal-server package installed on both server and worker side.
We fix the test by using https://github.com/cloudpipe/cloudpickle#overriding-pickles-serialization-mechanism-for-importable-constructs.
Some details (now written as a comment in the test module):
'''
NOTE: the experimental `register_pickle_by_value` feature of cloudpickle>=2
is described in
https://github.com/cloudpipe/cloudpickle#overriding-pickles-serialization-mechanism-for-importable-constructs.
Briefly:
1. It allows us to unpickle a function even if we don't have access to the
   module where it was defined.
2. It is experimental and can fail for some relevant cases (e.g. functions
   that include other imports, or a mix of by-value/by-reference pickling).
For us, this feature could be quite useful as part of the CI, to avoid
unexpected errors as in
https://github.com/fractal-analytics-platform/fractal-server/issues/690.
'''

EDIT: the solution described above (and implemented in https://github.com/fractal-analytics-platform/fractal-server/commit/6f272197b2d325a26f80aa6bd3777142bff16243) supposedly should work, but has some unexpected behavior and it's not worth spending too much time investigating (since it only relates to the CI).
We are now switching back to the simplest solution (see a8f5e5a6), that is, defining auxiliary functions from within test functions.


builtins: Add devito version of NumPy's count_nonzero, nonzero

Add a Devito version of NumPy's count_nonzero, nonzero:
https://numpy.org/doc/stable/reference/generated/numpy.count_nonzero.html
This PR is needed towards implementing steps 1,2 of p4 in https://arxiv.org/pdf/2010.10248.pdf.
In the paper, I was using NumPy, but now to automate through Devito, I would like to generate it through Devito.
Numpy's version is faster unless we change to collapse(1). Then Devito is faster for like int32 (20000, 20000) sized functions.
If reviewers require more benchmarking, happy to do so.
Generated code is like:
  /* Begin section0 */
  START_TIMER(section0)
  #pragma omp parallel num_threads(nthreads)
  {
    #pragma omp for collapse(2) schedule(static,1) reduction(+:h1[1])
    for (int x = x_m; x <= x_M; x += 1)
    {
      for (int y = y_m; y <= y_M; y += 1)
      {
        if (f[x + 1][y + 1] != 0)
        {
          int r0 = 1;
          h1[1] += r0;
        }
      }
    }
  }
  STOP_TIMER(section0,timers)
  /* End section0 */

  return 0;

But anyway, if you write it as a built-in, I don't see how you can glue it with a compiler pass, unless you copy-paste the equation definition?

Yes, though I am still thinking about the best way for doing that.

But anyway, if you write it as a built-in, I don't see how you can glue it with a compiler pass, unless you copy-paste the equation definition?

Yes, since it will be added as part of the pass, we will need to use the equations of it. However, are we opposed to having devito versions of this?

Where would this be used? And when you say faster does it count the code-generation overhead because it would re-generate the operator at each call so numpy is still probably more efficient.

Where would this be used? And when you say faster does it count the code-generation overhead because it would re-generate the operator at each call so numpy is still probably more efficient.

Sorry to reply late Mathias. With regards to the use, maybe it is not needed anymore for the time-tiling work, since I can use this: https://github.com/devitocodes/devito/pull/1817/files#diff-2da23b1e569f0f2b184634790719bca628a15fdd92739ddf6e9f0452e5b9bc6eR614
wrt to efficiency I admit I did not check the operator generation.
The question is: Since we probably do not need this functionality, I am thinking of whether this can be an enhancement/tutorial of how to use conditional dimension? Thoughts @FabioLuporini ?

I agree.


[semi-off topic] Maven plugin which spawns postgres database for other plugins

Hi,
I realize this might be a bit off topic, so feel free to show me the door if this is not appropriate :).
I am using a Maven plugin which generates code given a database. The obvious problem is that this database instance is not always available, such as during CI builds. This results in a broken build. Usually, this is tackled by only executing the plugin manually. For example, whenever the database schema changes, the developer could manually build and push the artifact to the repository. Although this works, it's not always very practical in larger environments.
My idea is to introduce a Maven plugin which spawns (and runs/waits for the necessary initialization scripts) a database in a Docker container, which can be used by subsequent Maven plugins, such as the generator mentioned above.
Does any of you happen to know such plugin? My searches did not yield anything useful.
That said, I believe this could be easily done with... tadaa... Testcontainers, which already does the heavy part. In case such plugin is not already available, I might invest some time to develop this.
Best regards!

Hi @pbillen,
have you checked out docker-maven-plugin?
It doesn't use Testcontainers, but it seems to apply to your use case.
Which Maven Plugin do you use for migrating the schema? There are normally also Java APIs available to achieve the same result, which can be used in classic Testcontainers scenarios.

Hi @kiview,
I did not know about fabric8 docker-maven-plugin. I believe it does exactly what I want (and a lot, lot more). Thank you very much for providing me that information, it saves me a lot of time!
Regarding your question about schema migration: I am doing this in a Docker image as well. Given certain environment variables (host, port, role, password, ...) this image tries to initialize or migrate a database. In this image, bash scripts and Flyway are being used.
Thanks again!


Terminal voice actions label

I find these labels hard to figure out:

I suggest:

Terminal: Start Voice Input
Terminal: Stop Voice Input

i'll probably remove these altogether since they're not useful

Yes 👍

@Tyriar pointed out that since this is called out in the release notes, I should wait to remove it until there's a replacement IE until we have a universal Voice: Start Dictation command

@meganrogge but at least align the labels? "Voice: Start / Stop Dictation in Terminal"

@bpasero isthe editor one going to become a generic Voice: Start Dictation that can be used anywhere?


missing support for ~= in check? - see comment in #1183

see https://github.com/pypa/pipenv/pull/1183/files#r155932921
also fixed comment typo

@therefromhere thanks for the PR. It is missing but I don't think it's necessary to run a specifier check on every entry that starts with a ~ because it's a valid path value.
If you want to PR the typo correction I'd be happy to accept that but pipfiles like the following currently work completely fine on windows, I see no reason to expose them to extra steps of logic when they don't need it:
[[source]]

url = "https://pypi.python.org/simple"
verify_ssl = true
name = "pypi"

[dev-packages]

[packages]

django = "~=1.11"

@techalchemy why does ~=1.11 work and <= not? This seems like #1183 was fixed at the wrong level if that's the case.

@nateprewitt I believe it has to do with the way pathlib handles testing paths on different platforms. Posix paths can be tested against the filesystem with arbitrary? values or at least with values included in the specifier set, while the windows check fails with: WindowsError: [Error 123] The filename, directory name, or volume label syntax is incorrect: '<2.0'. This happens as we are consuming pipfile entries and determining how to convert them into pip requirements for resolving into the lockfile, so we first have to determine if they are filesystem paths (specifically if they are files).
Essentially I think it is the correct behavior to fix the check where it is breaking, this is the right place to do it, and I don't think we should fix the parts that are working

fair enough @techalchemy , I'll cancel this PR then.

@therefromhere thanks for opening this PR and spurring a discussion, in the end I am now convinced that you are in fact correct and that I was wrong to exclude ~ from the list of specifiers! Thanks for submitting the PR and causing the discussion, I think we should merge the original one

I'd like to get appveyor running before we merge any more PRs, but after that I think we're happy with this. Thanks @therefromhere!


Shake correct number of times, instead of 1/2 as much

This could be intended behavior, but by decrementing "times" and incrementing "currentTimes" with each function call, the UITextField will only complete 1/2 as many full shakes as desired.
By "shake", did you mean "one direction" or "a full transform in both directions"?
If this change isn't wanted, perhaps adding some documentation about what constitutes a shake would be nice?
Great class extension, by the way!

This could be intended behavior

Good question 😬
I think this is better though, thanks for the PR


HEC-983 Tax situation page enhancements

Details in ticket - https://jira.tools.tax.service.gov.uk/browse/HEC-983
Note: URLs need confirmation.
Flow diagram to show journeys (dashed box shows scope of this ticket)

@AndyHWChung @priyankadhiman3006 refactored the tax situation api logic as per comments, looks much better Andy, thank you! The EitherT tripped me up a bit. Latest commit - https://github.com/hmrc/hec-applicant-frontend/pull/24/commits/c398350fc7a7b6e6bc0c6e85350f266fa7b4b10d

Just for FYI :), if you want to get rid of sequence[EitherT[Future, Error, *] and map you can just do
individualRetrievedData.sautr.traverse(taxCheckService.getSAStatus(_, taxYear))

Ooh nice! Let me do that!


Add missing community meeting minutes

Add missing meeting minutes from 2019-11-23 to 2020-06-06.


Adding Custom OIDC provider breaks auth.

It's a bit weird but I created SWA with AAD Auth. When I've added an OIDC provider (Azure B2C in my case), the app stopped redirecting to the auth providers. Neither /.auth/login/b2c nor /.auth/login/aad.
It literally redirects to /.auth/login/aad but not continuing to AD login page.
my code
...
"auth": {
    "identityProviders": {
      "customOpenIdConnectProviders": {
        "b2c": {
          "enabled": true,
          "registration": {
            "clientIdSettingName": "<clientId>",
            "clientCredential": {
              "clientSecretSettingName": "<secredId>"
            },
            "openIdConnectConfiguration": {
              "wellKnownOpenIdConfiguration": "https://<b2cName>.b2clogin.com/<b2cName>.onmicrosoft.com/B2C_1_signin/v2.0/.well-known/openid-configuration"
            }
          },
          "login": {
            "nameClaimType": "emails",
            "scopes": ["openid"]
          }
        }
      }
    }
  }

This is likely because some part of the custom auth is misconfigured. If you share your app name/details, I'll be able to give you more info on what the misconfiguration is. As for the current setup that you've shared, /.auth/login/aad is no longer going to result in anything because your configuration suggests that this app  has opted for custom auth (not shared auth) but no aad provider has been configured under identityProviders.


feature Add ingest_binlog/http_get_snapshot limit download speed && Add async ingest_binlog

Proposed changes
Add ingest_binlog/http_get_snapshot limit download speed && Add async ingest_binlog
Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall

run buildall

run buildall

run buildall


Point matchers don't work

Hi there!
My gpx track has named point:
...
<wpt lat="44.421723" lon="34.030495">     <ele>325.63</ele>     <time>2020-10-18T09:48:17.000Z</time>     <name>Old</name>     <desc>...</desc>   </wpt>
...
But this code doesn't change its icon:
const track = new L.GPX('link to gpx-track', {     async: true,     marker_options: {     pointMatchers: [       {         regex: /Old/i,         icon: new L.icon({           iconUrl: '/public/img/teh/start.svg'         })       }     ]   } }).on('loaded', function(e) {   map.fitBounds(e.target.getBounds()); }).addTo(map);
What am I doing wrong?

Ah, because I didn't implement that logic on waypoints, only regular points. Let me fix that!

Release v1.5.1 with the fix.


Implements the reciprocal scaling

Closes #139.

Maybe propagate to:
https://github.com/tlapack/tlapack/blob/0377a585a2e15653e4e6006a4aae53297b7f3ba6/include/tlapack/lapack/getrf_level0.hpp#L92

Here then:
https://github.com/tlapack/tlapack/blob/0377a585a2e15653e4e6006a4aae53297b7f3ba6/include/tlapack/lapack/getrf_recursive.hpp#L102

Here then:
https://github.com/tlapack/tlapack/blob/0377a585a2e15653e4e6006a4aae53297b7f3ba6/include/tlapack/lapack/getrf_recursive.hpp#L102

This was done in 3d947adc339102ff8771497828904e3eac8b16e2

Excellent then!

90496b7 fixes an issue created by the new implementation.
The current implementation of rscl has two behaviors:

If alpha is real, then x[i] /= alpha.
If alpha is complex, then call scal(real_type<alpha_t>(1) / alpha, x).

I am putting this as a draft since we are still missing the scaling of 1/alpha when alpha is complex.

That is the current design of the algorithm I propose:
If alpha is real, then this routine is equivalent to scal(1/alpha, x). This
is done without overflow or underflow as long as the final result x/a does
not overflow or underflow.
If alpha is complex, then we use the following algorithm:

If the real part of alpha is zero, then we scale by the reciprocal of
the imaginary part of alpha.
If not, see if either real or imaginary part is greater than safeMax. If
so, do proper scaling.
If not, we can compute the reciprocal of real and imaginary parts of
1/alpha without NaNs. If both components are in the safe range, then we can
do the reciprocal without overflow or underflow. Otherwise, we scale by the
safe range.

In the following, I present two matrices and use LAPACK's getf2 to factorize them. We compare getf2 in LAPACK 3.11 with getf2 using the new RSCL we propose in this PR.

Take the matrix:

A = [  b+i  1  ]
    [  b    b  ]

For this matrix, LU with pivoting does not permute the row for this matrix and its exact LU factorization is
L = [  1      0 ]       U = [  b+i   1       ]
      [  l21   1  ]              [  0      u22  ]

where
 l21 = b / (b + i )
 u22 = b / (b+i) * ( b+i-1 )

Now, let us use floating-point arithmetic and set b as a large number for this aritmetic. For example, let us use single precision and b = 2^75.

In this case, LAPACK's getf2 using the inline seven-line reciprocal scaling algorithm returns the factorization:
L = [  1      0 ]       U = [  b+i   1   ]
      [  1       1 ]              [  0      b  ]

whereas the new reciprocal scaling algorithm
L = [  1          0 ]       U = [  b+i   1   ]
      [  l21       1 ]              [  0      b  ]

The relative error is very small: 1.528233e-23, so no problems here. No problems because the condition number is small.
We obtain the correct b/(b+i) when using the new reciprocal scaling algorithm. Note that, in finite single precision, b/(b+i) rounds up to 1 - i/b, not to 1.

Now, let me we try the matrix:

b+I b
  b b

The condition number here is awful, larger than 2b. In this case, LAPACK's getf2 returns the factorization:
b+I b
  1 0

We still have a small relative error. However, the LU factorization returns a singular matrix since there is a 0 in the diagonal of U. If we try to solve a linear system we may end up in a division by zero which raises an exception.
The correct LU factorization is
    b+I b
b/(b+i) I

which is the one we obtain by using the algorithm I propose in this PR.


Remove short title toggle

The test is over.
Short titles performed significantly better on the homepage, but not on the 'what's on' page.
Because we don't understand the result, we're going to stick with what we have.
I think we should think/specify more about what we're going to do in the event of various outcomes to these tests – seems like this one could be seen as a bit of a waste of time?
I promise this thought was less passive-aggressive than it sounds.

I think we should think/specify more about what we're going to do in the event of various outcomes to these tests – seems like this one could be seen as a bit of a waste of time?

[@hthair's calculations]](https://github.com/wellcometrust/wellcomecollection.org/issues/4418) are pretty clear I thought? If we reached those targets we would have done something. We didn't so we're rethinking? The whole point of tests is that you don't know, so you ask.
Do you think waste of time because now nothing has happened?
I suppose there is an element of saying why we think the test is valuable e.g. The long titles are unappealing and verbose, shorter titles are more clear and clicky.
Although reading the test hypothesis - we're actually hypothesising against the test, i.e. against short titles, which is odd - but again worth testing for the small amount of time to test this I feel.

The calculations are clear, yes – they show that short titles are significantly better at driving traffic from the homepage to the exhibition page.
But we've decided to keep the long titles because (as I understand it) we're unsure what's behind this significant increase.

Unsure

Any quantitative test is going to have a level of uncertainty as to why something happens as you are dealing with numbers.
So is the general thing that if numbers go up, and we don't know why, we won't implement the winning test? If I've understood that correctly, it's going to make it very hard to implement any winning A/B test.

Or do we want a three fold validation process?
e.g.

Identify problem through users testing / have a hunch something is wrong
Test is via A/B quant
Take winning result to users again and see what they say?

We should have this conversation outside a PR (I don't think @hthair should need to watch PR comments).


Couldn't locate scannetv2 while preparing RGBD data.

While preparing PointCloud data 3 different folders are created. but only the train folder got one .pth. Is it ok? Or am I mistaken? Please let me know about the point cloud data format after preparing it.
And RGBD preparation occurs File not found data/scannetv2.

Sorry for delay. For 3D data, there should be files in all three folders (train, val, test).
Did you download the raw scannetv2 data? You should appoint the data path in the script.

There are three folders created while preparing point cloud data. But only the train folder has some files in it. How can I get all files for all of the folders (train, test, val)?
Yes, I downloaded a few scenes and later I figured out the path for RGBD data preparation.

The scenes you downloaded may all in the training dataset. So it doesn't matter for lacking files in val and test folders.
What problem did you meet when installing pointops?

Could you enlighten me how to solve these issues?

According to the error message, /usr/local/cuda/bin/nvcc cannot be found when building the pointops. Did you use cuda for compilation. If you used cuda for compilation, there may be a mismatch between the environment and the cuda version.


Loading Screen Enhancement | Issue #159

Title and Issue number

Title : Loading Screen Enhancement | Issue #159
Issue No. : #159
Code Stack :
Close #159

Video (mandatory)
https://github.com/user-attachments/assets/5a91d9b2-9f39-4939-b7b6-25c35239ed81

Checklist:

[X] I have mentioned the issue number in my Pull Request.
[X] I have commented my code, particularly in hard-to-understand areas
[X] I have created a helpful and easy to understand README.md if its a new page/tech stack
[X] I have gone through the  contributing.md file before contributing

Additional context (Mandatory )
Are you contributing under any Open-source programme?

[X] I'm a GSSOC-EXT contributor
[X] I'm a HACKTOBERFEST contributor

It should be a kind of loader, not the logo.

Okay fine, changing

It should be a kind of loader, not the logo.

@jeevan10017 please check now
https://github.com/user-attachments/assets/65ce8f3c-7b70-48c8-b844-e98d991f59b1

@Shariq2003 it will be better, if someone lifts the barbell up and down in vertical motion.
Kind of -https://tenor.com/en-IN/view/weightlifting-heavy-lifting-strong-man-strong-arms-olympics2021-gif-22361346
But not this

@Shariq2003 it will be better, if someone lifts the barbell up and down in vertical motion. Kind of -https://tenor.com/en-IN/view/weightlifting-heavy-lifting-strong-man-strong-arms-olympics2021-gif-22361346 But not this

yes I was also thinking of the same, but not end up finding a proper gif.

@Shariq2003 it will be better, if someone lifts the barbell up and down in vertical motion. Kind of -https://tenor.com/en-IN/view/weightlifting-heavy-lifting-strong-man-strong-arms-olympics2021-gif-22361346 But not this

Can I add this one that u shared

@jeevan10017 could u please give me the background icon for loader if u not likes this one,
well it is good than the black circular loader

@Shariq2003 it will be better, if someone lifts the barbell up and down in vertical motion. Kind of -https://tenor.com/en-IN/view/weightlifting-heavy-lifting-strong-man-strong-arms-olympics2021-gif-22361346 But not this

yes I was also thinking of the same, but not end up with finding a proper gif.

yeah, you can keep this for now.

@Shariq2003 it will be better, if someone lifts the barbell up and down in vertical motion. Kind of -https://tenor.com/en-IN/view/weightlifting-heavy-lifting-strong-man-strong-arms-olympics2021-gif-22361346 But not this

yes I was also thinking of the same, but not end up with finding a proper gif.

yeah, you can keep this for now.

@jeevan10017 sir, could u please check my msg in my last accepted PR.

@Shariq2003 it will be better, if someone lifts the barbell up and down in vertical motion. Kind of -https://tenor.com/en-IN/view/weightlifting-heavy-lifting-strong-man-strong-arms-olympics2021-gif-22361346 But not this

yes I was also thinking of the same, but not end up with finding a proper gif.

yeah, you can keep this for now.

So are u going to merge ?

@Shariq2003 it will be better, if someone lifts the barbell up and down in vertical motion. Kind of -https://tenor.com/en-IN/view/weightlifting-heavy-lifting-strong-man-strong-arms-olympics2021-gif-22361346 But not this

yes I was also thinking of the same, but not end up with finding a proper gif.

yeah, you can keep this for now.

keep this GIF for now and make that top scrolling bar background dark and disable side scrolling bar.

@jeevan10017 please review and merge

https://github.com/user-attachments/assets/119010bd-8300-481e-b0d9-02b7d6424621
This looks good.

@Shariq2003 it will be better, if someone lifts the barbell up and down in vertical motion. Kind of -https://tenor.com/en-IN/view/weightlifting-heavy-lifting-strong-man-strong-arms-olympics2021-gif-22361346 But not this

yes I was also thinking of the same, but not end up with finding a proper gif.

yeah, you can keep this for now.

keep this GIF for now and make that top scrolling bar background dark and disable side scrolling bar.

Also fixes this
https://github.com/user-attachments/assets/3e23ea59-2c7f-415e-a839-b856a6713d56

@jeevan10017 please review


[C#] Template Design Pattern revamp

Continuing with the revamps in #18, this is the implementation of Template Design Pattern.
Changes implemented

Created a dedicated Template project, with an example in Program.cs to show demo code.
Sealed all concrete classes.
Use File-Scoped namespaces, resulting in one less indentation level.
Marked methods that do not access instance data as static, as per CA1006 rule
XML Documentation on main classes and methods

@mahyoussef Looking forward to hearing your feedback.

مش عارف اقول ايه بس بصراحه استفد من اللي انت عامله هنا بجد شكرا ❤


Update Dependencies

Improve compatibility with higher versions of nodejs

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


[RDBMS] Fixed bugs and updated local context test

Description

Bug fix: not invoking error when command fails. Currently only log error message which will not give 'return 1' error when running a script with azure cli commands.
Bug fix: az mysql/postgres flexible-server delete command don't output error when server name is not fed
Test update: local context test update to properly validate the feature

Testing Guide

azdev test FlexibleServerLocalContextScenarioTest
History Notes

This checklist is used to make sure that common guidelines for a pull request are followed.

[x] The PR title and description has followed the guideline in Submitting Pull Requests.

[x] I adhere to the Command Guidelines.

[x] I adhere to the Error Handling Guidelines.

RDBMS

RDBMS

CI pass fixed. Requesting review for this PR

CI pass fixed. Requesting review for this PR

@evelyn-ys will help you with this.

@evelyn-ys will help you with this.

@evelyn-ys Can this be merged for this train? :)

@evelyn-ys Can this be merged for this train? :)


Document setup when using 2FA

I'm unable to setup CLA assistant because the user specified by GITHUB_USER is using 2FA. Could someone help out?

Any pointers would be appreciated. I can submit these code changes on a PR>

@ketan sorry for long delay. I would start with the method call in github service (src/server/services/github.js) which accepts basicAuth credentials in order to call github api with basic authentication (see line 132). As POC you could change this block to something like
        if (basicAuth) {
            github.authenticate({
                type: 'oauth',
                token: <YOUR GITHUB USER TOKEN>
            });
        }

If this works, then you can update all the places, where github service is called with basicAuth parameter, like
src/server/passports/github.js
src/server/passports/token.js
src/server/services/pullReuest.js
src/server/controller/count.js


Recycler view cousins

This closes #99

After deploy write to http://localhost:3000/ethereum:0xf5b0a3efb8e8e4c201e2a935f110eaaf3ffecb8d:585 about improved performance


Simplify logic with VNClassificationObservation

The mlmodel was updated to contain class labels

Thanks for the update! I will update my Blogpost when I will find time for that!


MERGE READY wv-317 Update candidate edit page to display form inputs on the same line

Update 3/22/24: Moved Facebook URL and Instagram Handle form groups to their own line, moved YouTube URL to its own line, and fixed alignment of Candidate Has Withdrawn From Election so that all labels and inputs are vertically centered/aligned with each other.

👍


Bug fixes and new features

New Features

Push 'EXPIRE' event when service tags or rules changed.
Add 'modTimestamp' in instance struct.
Support custom logger.
Watch api support handling close message.

Bug fixes

List & watch do not exit when connection was closed.
List & watch do not filter provider services with their rules.
Query a lot of keys will be failure when using the 'KeyOnly' flag.
Goroutine leak in listwatch lib and optimize log print.

Codecov Report

Merging #74 into master will decrease coverage by 0.24%.
The diff coverage is 52.36%.

@@            Coverage Diff             @@
##           master      #74      +/-   ##
==========================================
- Coverage   65.81%   65.57%   -0.25%
==========================================
  Files           8        8
  Lines        2776     2800      +24
==========================================
+ Hits         1827     1836       +9
- Misses        822      837      +15
  Partials      127      127

Impacted Files
Coverage Δ

server/service/microservice/microservice.go
1.69% <0%> (-0.22%)
:arrow_down:

server/service/governservice.go
66.02% <36.84%> (ø)
:arrow_up:

server/service/instances.go
66.7% <59.78%> (+0.97%)
:arrow_up:

server/service/services_dependency.go
68.38% <61.11%> (-0.74%)
:arrow_down:

server/service/microservices.go
68.47% <63.36%> (-0.04%)
:arrow_down:

server/service/query_rule_service.go
73.86% <64.44%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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

Coverage decreased (-0.4%) to 70.175% when pulling d403ae1402f04c3496fd5240163e56e47369f3aa on little-cui:master into e185f48a9b57b8906e5c67725939b722056af9dc on ServiceComb:master.


Remove duplicate build.rs file

This file should not be necessary, we have already
inside of the repository root.

/release-note-none


Speedtests fail with "Could not resolve host.".

Describe the bug
All speedtests fail with the error "Could not resolve host."
0.20.2 is the last working version. 0.20.3 and 0.20.4 both fail with the same errors.
To Reproduce
Steps to reproduce the behavior:

run a speedtest, manual or scheduled.

Expected behavior
Speedtest shouldn't fail, it does.
Environment (please complete the following information):
Proxmox with Debian 12LXC running Docker, all fully up to date.
Docker, with the following compose.yaml:
version: "3.9"
services:
linuxserver:
image: lscr.io/linuxserver/speedtest-tracker:latest
volumes:
- ./data:/config
environment:
- DB_CONNECTION=sqlite
- PGID=1000
- PUID=1000
- PUBLIC_DASHBOARD=TRUE
- PRUNE_RESULTS_OLDER_THAN=28
- SPEEDTEST_SCHEDULE=17 * * * *
- APP_KEY=--deleted--
ports:
- 8080:80
restart: unless-stopped
container_name: speedtest-tracker
networks: {}
Additional context
From within the container itself connectivity is fine:
root@86dfde98255e:/# ping <IP_ADDRESS>
PING <IP_ADDRESS> (<IP_ADDRESS>): 56 data bytes
64 bytes from <IP_ADDRESS>: seq=0 ttl=55 time=20.608 ms
64 bytes from <IP_ADDRESS>: seq=1 ttl=55 time=17.068 ms
^C
--- <IP_ADDRESS> ping statistics ---
2 packets transmitted, 2 packets received, 0% packet loss
round-trip min/avg/max = 17.068/18.838/20.608 ms
root@86dfde98255e:/# ping www.speedtest.net
PING www.speedtest.net (<IP_ADDRESS>): 56 data bytes
64 bytes from <IP_ADDRESS>: seq=0 ttl=55 time=16.543 ms
64 bytes from <IP_ADDRESS>: seq=1 ttl=55 time=15.555 ms
^C
--- www.speedtest.net ping statistics ---
2 packets transmitted, 2 packets received, 0% packet loss
round-trip min/avg/max = 15.555/16.049/16.543 ms
root@86dfde98255e:/#

Please see closed issues with the same title

This looks like the cache might not have been reset, can you restart your container?

same here, not working with v0.20.4, also after full clean install of container.

I take it back.
For me at least lscr.io/linuxserver/speedtest-tracker:latest is currently pointing to 0.20.3 (v0.20.3-ls29), although I'm sure I had 0.20.4 earlier, but can't remember if I specified latest or a fixed version in my compose file.
However forcing it to lscr.io/linuxserver/speedtest-tracker:0.20.4, redeploying, and then restarting the LXC it's running within appears to resolve the errors when there's no SPEEDTEST_PING_URL defined.
However there's definately something weird going on with the connectivity tests in 0.20.3 and above as if I define a host that I can quite definitely ping from the container, the errors are back. Setting and manual ping tests shown below.
SPEEDTEST_PING_URL=<IP_ADDRESS>
root@f24ed2b78674:/# ping <IP_ADDRESS>.
ping: bad address '<IP_ADDRESS>.'
root@f24ed2b78674:/# ping <IP_ADDRESS>
PING <IP_ADDRESS> (<IP_ADDRESS>): 56 data bytes
64 bytes from <IP_ADDRESS>: seq=0 ttl=55 time=28.334 ms
64 bytes from <IP_ADDRESS>: seq=1 ttl=55 time=13.769 ms
64 bytes from <IP_ADDRESS>: seq=2 ttl=55 time=12.529 ms
^C
--- <IP_ADDRESS> ping statistics ---
3 packets transmitted, 3 packets received, 0% packet loss
round-trip min/avg/max = 12.529/18.210/28.334 ms
root@f24ed2b78674:/#

@bigrichwood can you try setting a different IP or using a domain name?

I tried all the following, and get exactly the same errors.
<IP_ADDRESS>
www.google.com
<IP_ADDRESS> (my default gateway)
<IP_ADDRESS> (the LXC running docker)
<IP_ADDRESS>

For now don't set a ping url and that check should be bypassed. If the issue continues it might be CLI based and not related to the internet check. Also don't forget to restart the container after setting a new value so the cache is updated.

It works fine if I don't define a test URL.
Interestingly, if I set it to use a FQDN I can see the DNS looking taking place in the logs of my DNS server, so it's getting at least that far.

solved for me with 0.20.4

@bigrichwood if you're able to dig more into it some ISPs block these types of look-ups. I don't think it's an application error anymore since you can disable the check. Going to close this out one.

It's not an ISP issue as it can't even ping within my own network, or to
localhost.
If I get a chance this evening I'll fire a packet sniffer up on an internal
host, point it at that, and see if it's failing to ping, or failing to
interpret the results correctly.
On Mon, 10 Jun 2024 at 15:48, Alex Justesen @.***>
wrote:

@bigrichwood https://github.com/bigrichwood if you're able to dig more
into it some ISPs block these types of look-ups. I don't think it's an
application error anymore since you can disable the check. Going to close
this out one.
—
Reply to this email directly, view it on GitHub
https://github.com/alexjustesen/speedtest-tracker/issues/1510#issuecomment-2158569772,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ASK24DT2PMWZLJ3K3SYPNOTZGW4CJAVCNFSM6AAAAABJA7B26OVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCNJYGU3DSNZXGI
.
You are receiving this because you were mentioned.Message ID:
@.***>

after again clean install of v0.20.4 container and delete following line from docker-compose file, its working now.

SPEEDTEST_PING_URL="google.de"

I can confirm that it's not actually pinging.
Pings from the LXC and from within the speedtest-tracker container all show up on a packet sniffer as expected, pings generated from the SPEEDTEST_PING_URL in the app do not.
To be honest this doesn't bother me that much as I've just disabled the check. But I'm happy to do more testing if you want to get to the bottom of what's breaking it.

@bigrichwood if you don't mind digging I don't have a way to test LXC. Once you have findings open a new issue as I usually don't keep tabs on closed ones.


[rv_dm,jtag,dv] Fix JTAG monitor getting out of sync with the TAP

There are some detailed notes in the commit messages. Honestly, I'm slightly surprised about the riscv-dbg TAP behaviour here and intend to raise a "huh, really?" issue on the pulp repo to check it's what they intend.

My logic was that the first commit is "generic jtag stuff" and the second is rv_dm specific, which is why is separated them out. (It also means that if I've messed up something chip-wide, the bisection won't find something rv_dm-specific!)


add --physdev-is-bridged flag when using physdev module for iptables

add --physdev-is-bridged flag when using physdev module for iptables rules created by weave-npc to prevent warning in kernel logs
Fixes #3449
Since previous iptables chains rules are flushed on weave pod start, fix works for both fresh install and upgrades.

The weave bridge ip is used for “service import” which has been a feature since nearly the beginning.
I think it would be ok to say “you can’t have service import and network policy at the same time”.
That said, I don’t quite understand your statement of the problem. The sending node doesn’t know what the destination IP is, does it?  Is the problem on the receiving node?

I have finished sanity testing for various scenarios for any possible regression due to addition of new rule. its good to be get merged.
-A WEAVE-NPC-EGRESS -m addrtype --dst-type LOCAL -j RETURN


Vector for Cassandra should be enabled by default

Since cass-operator now uses Vector for the server-system-logger container, having to enable Vector through .cassandra.telemetry.vector.enabled doesn't make sense anymore.
We should consider that Vector is always enabled, which means that we'll have to duplicate the IsVectorEnabled() method into IsVectorEnabledForCassandra() which will return true if .cassandra.telemetry.vector.enabled is nil.
The alternative would be to ignore the enabled field for Cassandra and consider Vector as always enabled.

@burmanm, wdyt?

Well, one we could think of (if we want) is that you can actually also disable Vector in cass-operator (I forgot this), there's a Spec setting that allows disabling the ServerSystemLogger. That feature is not implemented at all in k8ssandra-operator I think, but would we want that ? So setting "enabled" explicitly to false would actually trigger removal of console logging? Not sure if users would understand that, but ..

I think that's a good idea. If it's properly documented (in the docs and in the CRD) then it does make sense 👍


Mentor progress

Both the save functionality of the ideas along with the search functionality are both added to the web application with this commit. Previous elements/ placeholders from master are merged and, if necessary for functionality, commented out.
See the functioning app in action!
http://g.recordit.co/igFEMVw2Ob.gif

Great! You'll have to explain it to me. I noticed you took off the delete button for now?


fix(components): restore  missing return statement

@gr2m tagging you just you are aware:

missing return in <Nav/> component
added a snapshot test to catch this kind of regressions in the future.

Nice, thanks!


Add one-job use token/api key generation for use within gear

If a gear executed by a user wishes to access core data during the execution of said gear, we would need to auth to core using that user's creds. One option to support this would be to lookup the uid in the origin of the job, and generate a temporary auth token for that user when the job is started. The auth token would allow requests to auth as that user, and would only be accepted by the API while the job is in a running state.
More discussion on this topic is expected, feel free to discuss specifics and other ideas below.

Another option is to allow an engine to "masquerade" as the user identified in the origin.

Notes:

The token has to be a valid API key
I would liiiike the keys to use the same subsystem as normal API keys
The token must only be accepted when the job is in state running
The token must not be saved to the database

Closed in #938


Unable to deploy the newrelic pack in Stackstorm server

Hey,
we are trying to clone the stackstorm newrelic pack with stackstorm server, and getting some issue to find the repo.
Here is the error log for deployment failures,
{
"result": "None",
"exit_code": 1,
"stderr": "Traceback (most recent call last):\n  File "/opt/stackstorm/st2/lib/python2.7/site-packages/st2actions/runners/python_action_wrapper.py", line 211, in \n    obj.run()\n  File "/opt/stackstorm/st2/lib/python2.7/site-packages/st2actions/runners/python_action_wrapper.py", line 126, in run\n    output = action.run(**self._parameters)\n  File "/opt/stackstorm/packs/packs/actions/check_auto_deploy_repo.py", line 41, in run\n    raise ValueError("No repositories or auto_deployment config for '%s'" % repo_name)\nValueError: No repositories or auto_deployment config for 'https://github.com/StackStorm/st2contrib/tree'\n",
"stdout": ""
}
Please revert your answer with<EMAIL_ADDRESS>Thank you in advance.

Just to clarify it - it looks like this is related to deploying / installing a pack from ChatOps.
For that to work, you need to configure config file for packs pack, see https://github.com/StackStorm/st2/blob/master/contrib/packs/config.yaml#L4 for example.

Could you please provide the document to how to configure the new relic alerts to stackstorm.
Example:
I have a server configured in newrelic and if it is utilize X% CPU newrelic will generate the alert.
I need to send the same newrelic alert to stackstorm.
Please provide the sets and document to integrate newrelic alerts with stackstorm.
Thank you in advance.

Running the newrelic_get_alert by providing app_name and getting below error,
Traceback (most recent call last):
File "/opt/stackstorm/st2/lib/python2.7/site-packages/st2actions/runners/python_action_wrapper.py", line 211, in
obj.run()
File "/opt/stackstorm/st2/lib/python2.7/site-packages/st2actions/runners/python_action_wrapper.py", line 126, in run
output = action.run(**self._parameters)
File "/opt/stackstorm/packs/newrelic/actions/get_alerts.py", line 24, in run
resp = requests.get(self.url, headers=self.headers, params=params).json()
File "/opt/stackstorm/st2/lib/python2.7/site-packages/requests/api.py", line 70, in get
return request('get', url, params=params, **kwargs)
File "/opt/stackstorm/st2/lib/python2.7/site-packages/requests/api.py", line 56, in request
return session.request(method=method, url=url, **kwargs)
File "/opt/stackstorm/st2/lib/python2.7/site-packages/requests/sessions.py", line 475, in request
resp = self.send(prep, **send_kwargs)
File "/opt/stackstorm/st2/lib/python2.7/site-packages/requests/sessions.py", line 596, in send
r = adapter.send(request, **kwargs)
File "/opt/stackstorm/st2/lib/python2.7/site-packages/requests/adapters.py", line 497, in send
raise SSLError(e, request=request)
requests.exceptions.SSLError: [SSL: CERTIFICATE_VERIFY_FAILED] certificate verify failed (_ssl.c:765)
who do I configure SSL or do we have any chance to by pass the SSL?

Closing this in favour of duplicate https://github.com/StackStorm/st2contrib/issues/639


Issue with Exclude paramater

Hi! Somewhat new to ntscraper. Tried to add the " exclude='replies' " parameter to my code and it threw the following error:
ValueError: Invalid exclusion filter 'r'. Valid filters are: nativeretweets, media, videos, news, verified, native_video, replies, links, images, safe, quote, pro_video
Is this a formatting error on my end, or something others have encountered as well? Either way, some help would be much appreciated!

Hi, you are getting that error because the 'exclude' parameter requires a list. To make it work, simply transform exclude='replies' in exclude=['replies'].


Improved regex to properly test "Do not change code below the line"

The last test will now properly check to make sure the camper did not change any of the code below the line.  It allows for spaces and newlines as long as they are not in between the text/code below the line.
Fixes #5875
This will also work for code that should not be changed in other waypoints/bonfires if it is adjusted to reflect the text specific to that waypoint/bonfire.

Well I seem to have messed this all up :(


chore: fixed typo in Algebra.Homology.HomotopyCategory

Fixes a typo in the name of a variable.

Thanks!
bors merge


Adding multi language support system✨

I want to add multi language support system to the website which can help us understand in their native language.
@Suraj-kumar00 can you assign me this issue.

Go ahead @T-Rahul-prabhu-38.
Do remember you have  3 days to complete the issue.

Hi @T-Rahul-prabhu-38, what is the update on the issue?


(325) Show a placement request in the CRU view

This adds an endpoint to show the details of a Placement Request for CRU managers. If a booking already exists, they can also extend or cancel it. They can't currently create a booking, but this is next in the pipeline.
Screenshot

 Show a Placement Request


Adding new logos and changing links

@jlstevens we can work off this branch

Including just the bokeh and mpl logos on the front page looks very strange to me

Ok I'm removing them

New landing page looks like:

rendered at https://pyviz-dev.github.io/holoviz/


fix(radio-button): update to audit spec and remove center label buttons

Closes #1926
Changelog
Changed

labels
label margin bottom

Removed

radio buttons with labels on top and bottom.

@asudoh can you recheck please


false positive with loop

	var file *http.Response
	for _, url := range urlsToTry {
		file, err = http.Get(url)
		if err == nil {
			break
		}
	}
	if err != nil {
		return nil, fmt.Errorf("failed to get config: %w", err)
	}

	cfg := &wellKnownFile{}
	err = json.NewDecoder(file.Body).Decode(cfg)
	if err != nil {
		return nil, fmt.Errorf("failed to parse config: %w", err)
	}

/home/moritz/Projects/oidc/configuration.go:63:24: error: Potential nil panic detected. Observed nil flow from source to dereference point:
	-> http/client.go:644:12: unassigned variable `resp` returned from `do()` in position 0
	-> http/client.go:590:9: result 0 of `do()` returned from `Do()` in position 0
	-> http/client.go:488:9: result 0 of `Do()` returned from `Get()` in position 0
	-> http/client.go:457:9: result 0 of `Get()` returned from `Get()` in position 0
	-> oidc/configuration.go:63:24: result 0 of `Get()` accessed field `Body`

err = json.NewDecoder(file.Body).Decode(cfg) is (to my understanding) well-guarded.

I believe the submitter is incorrect, if urlsToTry is empty and the loop body never executes, then file is never assigned.

Sorry, I should have clarified: urlsToTry is always set to have two elements. Both of which are guaranteed to be valid URLs through the url.URL type. But even if that was possible, the trace would be wrong for that case.


Custom validator params

Hi
Does anyone know if there's a way to pass anything other than a string to a custom validator?  My use case is as follows:
I have a form field that is only displayed (via v-show) if another field (radio button) is set to a certain value by the user.  I have this working fine.  What I'd now like to do is only require this field if is being displayed so basically I need a custom validator along the lines of requiredIf .
In proto code my app.js looks like this:
data: {
    response: '",
    approveMessage: "",
    declineMessage: ""
},
validator: {
    validates: {
        requiredIf: function(val, condition){
            console.log(condition);
            // My required logic goes here
        }
    }
}

On my form fields I have the following:
How do you want to respond?
<input id="response_approve" v-model="response" checked="checked" name="response" type="radio" value="approve">
approve

<input id="response_decline" v-model="response" name="response" type="radio" value="decline">
decline

    Approved message
    <textarea v-model="approved_message" v-validate="requiredIf: conditionalField('response', 'decline'), maxLength: 500" name="approved_message" cols="50" rows="10" id="approved_message"></textarea>

    Declined message
    <textarea v-model="declined_message"  v-validate="requiredIf: conditionalField('response', 'decline'), maxLength: 500" name="declined_message" cols="50" rows="10" id="declined_message"></textarea>

In this example the method called conditionalField just get's pass through as a string (the method itself shows/hides fields just fine so I know it works) and if instead I use an inline expression such as:
v-validate="requiredIf: response == 'decline'

this also get's passed through as a string.  Is there any way to pass through either a method, an inline expression or even just more than one expression after the requiredIf argument?

@FourStacks
Thank you for your feedback !!
I understood your usecase. :smiley:
Currently, vue-validator does not support function or inline expression like v-on.
http://vuejs.org/guide/events.html
http://vuejs.org/api/directives.html#v-on
It's very convenient !!
I will try to support.

Hi @kazupon
Thanks for the quick reply!  Yeah - if that's something that could be included it would be very handy indeed and would allow for more complex custom validators.  Thanks for offering to look into it!

I published v1.4.0 release.
Example:
https://jsfiddle.net/kazupon/76q3Lxu2/7/
Please try it !!

Excellent!  Thanks very much for the speedy work in getting this implemented - very much appreciated!

Seems to work like a charm by the way, judging from the fiddle.  I'll integrate it into my code later today and let you know if I spot any issues but it's looking great!

I can't get something like this working with v2.0 alpha. Any ideas?


Implement good-run-list from JSON (make sure we can mask a Jagged array with it too)

Ah I did this for the Z peak at some point.  It is here: https://github.com/CoffeaTeam/fnal-column-analysis-tools/blob/master/fnal_column_analysis_tools/lumi_tools/init.py#L36
Did not investigate using jagged methods to perform lookup.  Its reasonably fast as-is

Ah, thanks. Yeah, I haven't minded the issues in a bit. Many have been solved.
This was added in #45


Brainstorming future OTBN ISA extensions

@moidx, @jadephilipoom, and I have been discussing future extensions we'd like to see in the OTBN ISA in future revisions. This issue is intended to record those in a central place, but they are in no way plan-of-record or intended to make it into the timeline for the current revision.
(Previously: https://github.com/lowRISC/opentitan/issues/5058).
An incomplete list of things we would like to see:

Vectorized BN instructions that partition wX registers into lanes.

Rotate and componentwise-addition (without carry) are proposed in the linked issue.
Probably a broadcast-to-all-lanes instruction for making masks for this purpose; not sure how useful this is in crypto.
"Swizzle", what RISC-V calls xperm4/8: treat the first word as an array and the second word as an array of indices, and use it to perform N lookups into the first array. Useful for constant-time LUTs.

Multi-word BN instructions.

Multi-word add: e.g. perform a single 1024-bit addition of two four-register banks (Implemented as four adds, probably? I don't know if "1024-bit adder" is an insane ask). Presumably this would be encoded as specifying a register divisible by 4 (or whatever your multi-word size is), implying that it should be followed by the next three registers after it.

Multi-bit carry for unsaturated limbs.

In algorithms that use unsaturated limbs (Jade was contemplating one with 192-bit limbs for P-384 field arithmetic), it would be nice to be able to specify using the high N bits of another register as carry bits for a BN addition. Not sure if there's a clean way to encode this.

Chunks of the RISC-V crypto spec such as SHA-2 instructions (but at the BN width rather than the RV scalar width).

These look fun! When thinking seriously about these, we'd need to work out why we're adding them. A couple of quick thoughts:

Vectorized instructions
Specifying these is always hard because there are so many different things you can do. Compare x86 SSE/AVX and the RISC-V vector extension, for example. We'd probably need some firm use cases to convince ourselves we were building the right thing.
Multi-word BN instructions
This is probably possible but running in a single cycle would probably have a big area impact and the resulting processor might be rather "lopsided" (able to do one thing really well, but a bit rubbish at other stuff given its size). There would also be awkwardness on the register file (where you suddenly need either lots more read and write ports or to re-shape it to work in 1024-bit words).
Multi-bit carry
Eminently doable! The only difficulty is probably encoding space, but we could cheat and do something like use registers n and n | 1 (like you were suggesting for the multi-word instructions)
SHA-2 instructions
Definitely do-able. But the permutation networks and such will probably be reasonably large, so we'd need a good use for them, I think.

@GregAC

Multi-word add

It could also still be useful for unsaturated limbs to have something that doesn't need a carry between adds -- e.g. I have 4 limbs but none of them are storing the full 256 bits, so an instruction could add the 4 pairs of corresponding limbs but have no need to carry between them (that would be done later as a multi-bit carry).
@felixmiller, any instructions you would like to see?

Marking as P2 as this is not currently part of the current iteration plan.

Re. SHA2 instructions we should take SCA considerations into account. This requires careful consideration at the interface level, and it is possible we may want to opt for a micro code implementation instead of following the RISC-V specification.

Yea, we should be very careful here with SHA2 instructions.  We should discuss the exact use case too.

Two things we were discussing in the past:
Carry-less multiplication
Nothing that we desperately need right now, but we probably want to support this in the (very) long term to enable polynomial arithmetic.
Composable (multi-limb) pseudo modular addm subm
The addm and subm instructions (pseudo modular add/sub) come in very handy for any 256-bit elliptic curve implementations. For any multi-limb implementation this blows up to 3 steps: (1) Arithmetic operation (sub/add), (2) (speculative) modular correction, (3) final select. All this resulting in 1 machine instruction per step and limb (e.g. 6 instructions for P-384). Nice would be a composable multi-limb addm/subm that would take one instruction per limb. The big issue here is probably the register to store the (then larger) modulus.

Jade and I were chatting about another


Run Python 2.7 and Python 3.6 containers with Docker

This PR adds a Python 3.6 container running cfgov-refresh. As part of this change, the attach.sh and shell.sh scripts are removed and replaced with updated Docker usage documentation that describes how to attach to or run a shell in the desired container (since there are now two of them).
This PR includes some other minor cleanup of the usage documentation, including a table of contents so that one can navigate directly to the section of the page one is interested in.
Implementationally, I have moved the Python container's Dockerfile and cfgov/entrypoint.sh to docker/python2 and added a corresponding docker/python3. This seems like a better way to organize our Docker-related files and scripts. The Python containers join the Elasticsearch and docs container in docker/.
Testing

Run docker-compose up
Verify that you can get to the Python 2 container at http://localhost:8000/ and the Python 3 container at http://localhost:8333/
Read the updated usage documentation at http://localhost:8888/usage/#docker

Screenshots
python2_1        | Performing system checks...
python2_1        |
python3_1        | Performing system checks...
python3_1        |
python2_1        | System check identified no issues (0 silenced).
python3_1        | System check identified no issues (0 silenced).
python2_1        | May 09, 2019 - 09:53:08
python2_1        | Django version 1.11.20, using settings 'cfgov.settings.local'
python2_1        | Starting development server at http://<IP_ADDRESS>:8000/
python2_1        | Quit the server with CONTROL-C.
python3_1        | May 09, 2019 - 09:53:08
python3_1        | Django version 1.11.20, using settings 'cfgov.settings.local'
python3_1        | Starting development server at http://<IP_ADDRESS>:8000/
python3_1        | Quit the server with CONTROL-C.

Notes
The name of the Python 2.7 container is changed to python2/cfgov-refresh_python2_1.
The attach.sh script has been apparently broken for a while, as it is trying to attach to a container name that no longer exists (at some point the Python container name changed from cfgovrefresh_python_1 to cfgov-refresh_python_1 (and, with this PR, to cfgov-refresh_python2_1/cfgov-refresh_python3_1`).
The Docker usage documentation has also been out-of-date for quite a while. I updated it and reorganized it based on common tasks I know I need to perform, but I'd love to have others' opinions.
Checklist

[ ] PR has an informative and human-readable title
[ ] Changes are limited to a single goal (no scope creep)
[ ] Code can be automatically merged (no conflicts)
[ ] Code follows the standards laid out in the CFPB development guidelines
[ ] Passes all existing automated tests
[ ] Any change in functionality is tested
[ ] New functions are documented (with a description, list of inputs, and expected output)
[ ] Placeholder code is flagged / future todos are captured in comments
[ ] Visually tested in supported browsers and devices (see checklist below :point_down:)
[ ] Project documentation has been updated
[ ] Reviewers requested with the Reviewers tool :arrow_right:

Specifically tagging our known-Docker experts to review what I've done to the Dockerfiles here ;)

I don't feel too strongly about this-- but it could be a nice place to use multi-stage builds via a single Dockerfile. One way it would be a win, is right now you're basically doing this:

With a multi-stage build, you could do that common stuff just once, saving time and disk space:

The compose file wouldn't need to change much, since you can reference individual stages.

@rosskarchner interesting! I definitely think that's a better way to go — I don't really like the duplication here right now.

Or, maybe even simpler: How about a "fat" container with both versions of python, and you switch by overriding the default entrypoint or startup command?

@rosskarchner I want Python 2.7 and 3.6 running side-by-side though. By "switch" do you mean you'd switch which version you use on docker-compose up?

No, you can have two seperate containers, based on the same image. One runs the python 2 entrypoint, the other runs the python 3 entrypoint

Does this open the door to somehow testing out Wagtail 2 from the same repo in a Python 3 container? Or does that get too complex with juggling requirements and such?

I think in practice, it might look something like this:
    python2:
        build:
            context: ./
            dockerfile: ./docker/python2/Dockerfile
        image: python
        ports:
            - "8000:8000"
        environment:
            ES_HOST: elasticsearch
        volumes:
            - ./:/src/cfgov-refresh
            - ./develop-apps:/src/develop-apps
        entrypoint:
            - sh
            - /src/cfgov-refresh/docker/python2/entrypoint.sh
        depends_on:
            - elasticsearch
            - postgres
        working_dir: /src/cfgov-refresh
        stdin_open: true
        tty: true
    python3:
        image: python
        ports:
            - "8333:8000"
        environment:
            ES_HOST: elasticsearch
        volumes:
            - ./:/src/cfgov-refresh
            - ./develop-apps:/src/develop-apps
        entrypoint:
            - sh
            - /src/cfgov-refresh/docker/python3/entrypoint.sh
        depends_on:
            - elasticsearch
            - postgres
        working_dir: /src/cfgov-refresh
        stdin_open: true
        tty: true

@rosskarchner so the approach there would be to have a single Python Dockerfile that installs Python 2 and Python 3 and the requirements in each, and then separate entrypoint scripts (or a switch in the entrypoint script)?

Does this open the door to somehow testing out Wagtail 2 from the same repo in a Python 3 container? Or does that get too complex with juggling requirements and such?

@Scotchester I don't think so — there's very little reason to want to run Wagtail 2 along side Wagtail 1.13, and in practice it would introduce a lot of complexity in the codebase to support running them side-by-side. I think we want a clean upgrade from 1.13 to 2.x.

@rosskarchner so the approach there would be to have a single Python Dockerfile that installs Python 2 and Python 3 and the requirements in each, and then separate entrypoint scripts (or a switch in the entrypoint script)?

Yeah, though I like it a bit less now that you've pointed out that requirements need to be installed twice (though I guess that happens either way)

@rosskarchner I think I've got both containers using the same image, per your suggestion above.

I'll squash the doc updates once this PR is approved before it merges.

One issue I notice that I'm not sure how to get around is that cfgov/manage.py uses #!/usr/bin/env python to run. This means that opening a shell in the python3 container and running management commands will still use Python 2 (because it's the default in the CentOS 7 image). I'm not entirely sure how to get around that (without using something like pyenv instead of the CentOS python packages).

this seems to work:
try adding ln -s /bin/python3 /usr/local/bin/python  to the python3 entrypoint

That works!


adding missing token to address issue #5

Decided to fix the issue to be helpful.

I signed it!
Christopher Johannsen
<EMAIL_ADDRESS>
On Oct 27, 2015, at 10:37 AM, googlebot<EMAIL_ADDRESS>wrote:
Thanks for your pull request. It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).
Please visit https://cla.developers.google.com/ https://cla.developers.google.com/ to sign.
Once you've signed, please reply here (e.g. I signed it!) and we'll verify. Thanks.
If you've already signed a CLA, it's possible we don't have your GitHub username or you're using a different email address. Check your existing CLA data https://cla.developers.google.com/clas and verify that your email is set on your git commits https://help.github.com/articles/setting-your-email-in-git/.
If you signed the CLA as a corporation, please let us know the company's name.
—
Reply to this email directly or view it on GitHub https://github.com/googlesamples/identity-toolkit-java/pull/6#issuecomment-151581631.


Import and Export Data

Is your feature request related to a problem? Please describe.
Not really a problem, but before starting to use TeslaMate I'm using Stats on iOS, which allows you to export some data, like Battery Health, Distance Driven, Efficiency, Phantom drain.
Describe the solution you'd like
A general import and export csv file that we can either fill in with our data manually and then import. Same then would go for exporting via csv, so that we can manipulate the data in Excel as needed.
Describe alternatives you've considered
Tried to find where data is stored in something like Tesla API Scraper, but wasn't able to successfully retreat the data into a csv. But where the data is stored so I can modify on TeslaMate, I don't even know.

Ability to import from Stats would be great. If the dev wants to reach out to me, I can forward some of the exported data so they can see what they'd be working with.

Please add the ability to export drive data into TeslaFi Format


Component types are incompatible with React 18

🐛 Bug Report
I upgraded my react version & its types (react & react-dom) to v18.
After the upgrade I cannot use any of the loadable components inside JSX tags.
Here is the full error:
'XXXComponent' cannot be used as a JSX component.
  Its element type 'ReactElement<any, any> | Component<XXXComponentProps & ExtraComponentProps, any, any> | null' is not a valid JSX element.

My code
const XXXComponent: LoadableComponent<XXXComponentProps> = loadable(() =>
  import('~/src/components/XXXComponent'),
);

const AnotherComponent = ()=>{
   return (
         <>
             <XXXComponent {...props} />
        </>
   );
}

Expected behavior
Should be able to use loadable components without any issue
## System:
 - OS: macOS 12.5.1
 - CPU: (8) arm64 Apple M1
 - Memory: 75.36 MB / 16.00 GB
 - Shell: 5.8.1 - /bin/zsh
## Binaries:
 - Node: 18.14.0 - ~/.nvm/versions/node/v18.14.0/bin/node
 - Yarn: 1.22.17 - /usr/local/bin/yarn
 - npm: 9.3.1 - ~/.nvm/versions/node/v18.14.0/bin/npm
## npmPackages:
 - @loadable/component: ^5.15.3 => 5.15.3
 - @types/react: ^18.0.27 => 18.0.27
 - @types/react-dom: ^18.0.10 => 18.0.10
 - react: ^18.2.0 => 18.2.0
 - react-dom: ^18.2.0 => 18.2.0

I've expirienced a simillar issue for another package when upgrading to React 18.
Take a look at this issue.


Paypal username field validation

cc - @parasharrajat
Details

Added validations for Paypal.me username

Fixed Issues

$ https://github.com/Expensify/App/issues/5541
Tests

Tested invalid with special characters
Tested invalid with spaces
Tested valid with alphanumeric characters in

QA Steps

Go to Sidebar -> Payments
Select Paypal to add a username
Try updating with the special characters, spaces. Paypal username allows only alphabets and numbers only.

Tested On

[x] Web
[x] Mobile Web
[x] Desktop
[x] iOS
[x] Android

Screenshots

Web

Mobile Web

Desktop

iOS

Android

Code tests well, but there are two typos that need to be addressed.

@deetergp PR updated. Typos fixed. Sorry about that.

@parasharrajat  Any changes required in this PR?

:hand: This PR was not deployed to staging yet because QA is ongoing. It will be automatically deployed to staging after the next production release.


Add license job, should create a PR on Jan 1.

This PR sets up a job that runs at the newyear to update changelog strings.

Current dependencies on/for this PR:

main

PR #156   👈

PR #157

This comment was auto-generated by Graphite.

Codecov Report

Merging #156 (670061d) into main (8d4c854) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #156   +/-   ##
=======================================
  Coverage   84.56%   84.56%
=======================================
  Files          44       44
  Lines        2390     2390
=======================================
  Hits         2021     2021
  Misses        369      369

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


merge-main-into-evm-to-soroban

Preview is available here:http://soroban-docs-pr454.previews.kube001.services.stellar-ops.com/


Update openssl to v1.0.2f

Please update openssl to v1.0.2f. 1.0.2f not available on site. http://slproweb.com/download/Win64OpenSSL-1_0_2e.exe and http://slproweb.com/download/Win32OpenSSL-1_0_2e.exe return 404.

Should be fixed by #669 once you run scoop update.


Autogenerates NetIDs

requires https://github.com/space-wizards/RobustToolbox/pull/1475

No progress for a long time. Closing.

tfw


Add FAQ to docs

Make sure to answer #210 about :checked

Should also cover:

[ ] ReactCSSTransitionGroup alternatives (see #4)


Bump GMT from 6.3.0.dev2+50be0be to 6.3.0.dev3+162b059

This is an automated pull request!
This PR will update the following package:

gmt from 6.3.0.dev2+50be0be to 6.3.0.dev3+162b059

Full list of commits made since last release viewable at compare view

Notes for merging this PR:

Feel free to push to the bot's branch to update this PR if needed.
Checklist before merging this PR:

[ ] Dependencies have been updated if changed
[ ] Tests have passed

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.


fixed import issues for langchain

Because langchain moved certain functionalities into langchain_community, the import statements in rag.py need to be updated.

Thanks @JmbFountain


Although installed, Could not execute because the specified command or file was not found.

I have installed codegenerator without any error message. However, I can't run it. Details below. (Ubuntu 18.04) (codegenerator v. 3.0.0 and v.2.1.10).

I followed this tutorial: https://docs.microsoft.com/fr-fr/aspnet/core/tutorials/razor-pages/model?view=aspnetcore-3.0&tabs=visual-studio-code , part "Scaffold the movie model"
Open a command window in the project directory (The directory that contains the Program.cs, Startup.cs, and .csproj files). ✔ It worked.
Install the scaffolding tool: dotnet tool install --global dotnet-aspnet-codegenerator. ✔ It worked and now if I try to re-install it, it outputs that it's already installed.
For macOS and Linux: Run the following command: dotnet aspnet-codegenerator razorpage -m Movie -dc RazorPagesMovieContext -udl -outDir Pages/Movies --referenceScriptLibraries. 🐛 It doesn't work. Here is the output error:

Could not execute because the specified command or file was not found. Possible reasons for this include: * You misspelled a built-in dotnet command. * You intended to execute a .NET Core program, but dotnet-dotnet-aspnet-codegenerator does not exist. * You intended to run a global tool, but a dotnet-prefixed executable with this name could not be found on the PATH.

I tried to re-run codegenerator with the following commands (nonre of them succeeded):

dotnet aspnet-codegenerator razorpage -m Movie -dc RazorPagesMovieContext -udl -outDir Pages\Movies --referenceScriptLibraries
dotnet-aspnet-codegenerator razorpage -m Movie -dc RazorPagesMovieContext -udl -outDir Pages\Movies --referenceScriptLibraries
dotnet dotnet-aspnet-codegenerator razorpage -m Movie -dc RazorPagesMovieContext -udl -outDir Pages\Movies --referenceScriptLibraries

I tried to restart my computer and Ubuntu 18.04 session after having installed codegenerator, it didn't solve the problem.
For each installation/running action described above, I tried the versions 3.0.0 and 2.1.10 of codegenerator. I also tried to execute these commands from my project directory, and from the folder containing my project directory, none of them solved the problem.
I also tried to use -g instead of --global for installing codegenerator.
How could I use codegenerator?
StackOverflow  question is linked here: https://stackoverflow.com/questions/58215952/asp-net-core-crud-tool-command-not-found-but-actually-installed .

In the linked tutorial,  https://docs.microsoft.com/fr-fr/aspnet/core/tutorials/razor-pages/model?view=aspnetcore-3.0&tabs=visual-studio-code , you forgot to write: "On Linux, export the scaffold tool path:   export PATH=$HOME/.dotnet/tools:$PATH " . . . THANK YOU MICROSOFT. (you did it however for MVC ;) ).


Braiins S17 Pro stopped working after 0.46.0

Describe the bug
pyasic no longer detects miners after version 0.46.0
Braiins firmware on 23.01 [Last supported version for X17 series]
To Reproduce
install pyasic > 0.46.0
run sample data gathering code miner.get_data() from pyasic-0.50.4
  File "/home/XXX/test.py", line 17, in <module>
    asyncio.run(gather_miner_data())
  File "/usr/lib/python3.9/asyncio/runners.py", line 44, in run
    return loop.run_until_complete(main)
  File "/usr/lib/python3.9/asyncio/base_events.py", line 642, in run_until_complete
    return future.result()
  File "/home/XXX/test.py", line 11, in gather_miner_data
    all_miner_data = await asyncio.gather(*[miner.get_data() for miner in miners])
  File "XXX/lib/python3.9/site-packages/pyasic/miners/base.py", line 474, in get_data
    gathered_data = await self._get_data(
  File "XXX/lib/python3.9/site-packages/pyasic/miners/base.py", line 398, in _get_data
    self.web.multicommand(*web_multicommand, allow_warning=allow_warning)
TypeError: multicommand() got an unexpected keyword argument 'allow_warning'

Desktop (please complete the following information):

OS: debian

Miner Information (If applicable):

Manufacturer: Bitmain
Type: S17 Pro
Firmware Type: BraiinsOS
Firmware Version: 23.01

Additional context

Ill take a look at it here shortly, there are some issues with BOS+ on X17 models because they lack gRPC support, but ill just make them subclass BOSMiner instead of BOSer I suppose.

Can you try with pyasic verson 0.50.6?  python3 -m pip install pyasic==0.50.6

Working with this update. Thanks!


Uncommenting comments with the comment treesitter parser installed

Contributing guidelines

[X] I have read CONTRIBUTING.md
[X] I have read CODE_OF_CONDUCT.md
[X] I have updated 'mini.nvim' to latest version

Module(s)
mini.comment
Description
I'm having an issue with mini.comment, trying to uncomment lines that are commented out. If I'm in lua file with comments, such as this one:
local f  = function(a, b, c)
  -- return a + b + c
end

And try to uncomment the commented line, I get:
local f  = function(a, b, c)
  # -- return a + b + c
end

I have the comment treesitter parser installed. If I run lua vim.print(vim.filetype.get_option("comment", 'commentstring')) in Neovim, "# %s" is returned.
I suspect the H.get_commentstring is getting to this filetype option and the error I'm seeing is the result.
I didn't have this problem with nvim-ts-context-commentstring.
Running TSUninstall comment and reopening the file fixes this issue.
Neovim version
v0.10.0-dev-323+g6a273af10
Steps to reproduce

:TSInstall comment
Open lua file with commented content
Try to uncomment line

Expected behavior
No response
Actual behavior
The commented line has #  appended.

Please provide a full, minimal, reproducing example. This works fine for me.

Thanks for the issue. I can not reproduce (even with 'comment' , as 'comment' filetype doesn't have 'commentstring' option set by default.
It seems like something in your config sets it. It might set at least through following mechanisms:

Autocommands. What is the output of :au FileType comment?
Filetype plugins. What is the output of :=vim.api.nvim_get_runtime_file('ftplugin/comment*', true)?

Here's your issue: https://github.com/oncomouse/dotfiles/blob/72c545d59ea4097584158e6a6e0bbed07a6cbc6f/conf/vim/init.lua#L59
comment.nvim requires an empty default commentstring to know when to look for containing tree-sitter regions. (This is how Neovim behaves by default, and you should not change it.)

It seems that you set default 'commentstring' in your config.

Got it. Commenting out the line where I set commentstring fixes the problem.

I've added note about this to both README and help page. Thanks for suggestion!


Replace awesome-go-books with GoBooks

https://github.com/dariubs/GoBooks is actually an active maintained list.

I suppose I don't mind keeping awesome-go-books around, however, could you please send a PR for GoBooks addition?

Sure; I was on my phone when I filed the issue this morning, which is why I only filed a bug instead of a PR.


std::bad_alloc - Visual Studio 2022

Hi I'm new to C++, I copied the code from https://github.com/qpdf/qpdf/blob/main/examples/pdf-npages.cc and tried using it but it returns the error std::bad_alloc, any idea what Can be?

The code looks correct to me. If the PDF file were invalid, it's possible that something with throw an exception, but I don't think you'd get a bad_alloc under that condition. So I don't have any specific ideas.

After lots and lots of research I found that the problem was the visual studio debug mode, I switched from debug to release and it worked normally. Thanks!

Me too. Only with release mode...


CompositeByteBuf pointless operation

Netty version
5.0.0.Alpha2
There is a lot of pointless operation  in the source code,
```

@Override
protected int _getUnsignedMediumLE(int index) {
Component c = findComponent0(index);
if (index + 3 <= c.endOffset) {
return c.buf.getUnsignedMediumLE(c.idx(index));
} else if (order() == ByteOrder.BIG_ENDIAN) {
return _getShortLE(index) & 0xffff | (_getByte(index + 2) & 0xff) << 16;
} else {
return (_getShortLE(index) & 0xffff) << 8 | _getByte(index + 2) & 0xff;
}
}

@flycat0112  I don't think this version (5.0.0.Alpha2) is maintained anymore. Currently, most development effort goes to 4.1.x branch. You should switch to 4.1.x

Thanks you too，but this varsion (4.1.33.Final) still has that problem. I don't know the point about this operation (type short & 0xffff) .

thanks you too, I know the point about this operation(type short & 0xffff).


Deprecated function homeassistant.util.temperature

Hi! I realized in log view that homeassistant.util.temperature function is deprecated, this weekends if I have time I will make some changes to use the new function.
I anyone got it please make a PR.
Thanks!!

Hello R1NEJ0,
I tried to modify the original file without success.
Have you been able to fix It?
If so could you share It with me?
Thanks in Advance for a feedback. Gianni

I'm very busy theses days, so a I could not try to fix it.
Here you go the official Web API:
https://developers.airzonecloud.com/docs/web-api/#overview

Made it work modifying climate.py like this:

I can add the component but cannot use it, only show it :(

Hello,
Sorry for the delay, free time is really missing.
Shoud be fixed with the v1.3.0.
Max


Further code enhancements

Made some significant changes:
I have changed code logic to work with DJI mp4 files. Fixed Unwrap() on None issue for ftype atom.
Added support for audio track metadata and atom (smhd). Also, I added other missing atoms about the necessary metadata.
Made changes to printing format for parser output, including limiting printing output for stco atoms. Finally, updated signatures for corresponding code files, i

Merging this branch to master based on our discussion today.


Comment tips are taken from balance, even the comment not get through.

If user gets error that comment cannot posted, the tip of that comment are still taken from the user wallet/balance:

We've removed the tip to comment feature.


fix: updated sample logs to original Logs API format

…missing workspace_id attribute

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


Determining which team a penalty was against

I don't see a way to determine which team a penalty was against.  The play table has columns for penalty, penalty_first_down and penalty_yds, but it's not obvious how to determine which team committed the penalty, short of grabbing the next play and seeing if the yardline is increased or decreased from the previous.
Am I just missing something or was this an oversight?

There's no oversight. There's simply very limited structured data on penalties from NFL.com. What you see is what you get.
If you really need this, then you are indeed limited to heuristics such as context of the play and the human readable play description text.


[Bug] [complete data] process instance stuck when we use serial complete data

Search before asking

[X] I had searched in the issues and found no similar issues.

What happened
A BUG with the complement function. When we used the serial complement function for a period of time, we found that there was an exception.
What you expected to happen
For example, if the complement time was three days, only the first instance was displayed as complete, while the second instance was always displayed as running, and the corresponding task under the instance had already been completed, that is, the status of the second instance had not been updated. The subsequent run instance will not appear again.
How to reproduce
processes: Workflow definition -> Click Online, Run -> Select Serial Complement -> Select Time Span -> Click Run. It shows that except the first instance is successfully completed, all the subsequent instances are stuck.
When there are at least 100,000 entries in the t_ds_task_definition_log table, the question will reappear
Anything else
No response
Version
2.0.x
Are you willing to submit PR?

[ ] Yes I am willing to submit a PR!

Code of Conduct

[X] I agree to follow this project's Code of Conduct

Could you provide more details about this? I can't reproduce this issue in 2.0.5-release.

Could you provide more details about this? I can't reproduce this issue in 2.0.5-release.

we use 2.0.0-alpha,you can reproduce in this version.

we use 2.0.0-alpha,you can reproduce in this version.

This version is no longer in the maintaining list. You can try to upgrade to the latest version.


chore(make): add environment variables to help

[aslak@localhost istio-workspace]$ make help

Usage:
  make <target>

Default target (all you need - just run "make")
  all                   Runs 'deps format lint test compile' targets

Build
  compile               Compiles binaries
  test                  Runs tests
  clean                 Removes build artifacts
  deps                  Fetches all dependencies
  format                Removes unneeded imports and formats source code
  lint                  Concurrently runs a whole bunch of static analysis tools
  codegen               Generates operator-sdk code and bundles packages using go-bindata

Setup
  tools                 Installs required go tools

Docker
  docker-build          Builds the docker image
  docker-push           Pushes docker images to the registry

Istio-workspace operator deployment
  load-istio            Triggers installation of minimal Istio in the cluster (see deploy/istio/minimal-cr.yaml)
  deploy-operator       Deploys istio-workspace operator resources to defined OPERATOR_NAMESPACE
  undeploy-operator     Undeploys istio-workspace operator resources from defined OPERATOR_NAMESPACE

Istio-workspace example deployment
  deploy-example        Deploys istio-workspace specific resources to defined TEST_NAMESPACE
  undeploy-example      Undeploys istio-workspace specific resources from defined TEST_NAMESPACE

Helpers
  help                  Displays this help \o/

Environment variables
  IKE_DOCKER_REGISTRY       quay.io
  IKE_DOCKER_REPOSITORY     maistra
  IKE_IMAGE_NAME            $(PROJECT_NAME)
  IKE_IMAGE_TAG             $(VERSION)
  IKE_TEST_IMAGE_NAME       $(IKE_IMAGE_NAME)-test
  OPERATOR_NAMESPACE        istio-workspace-operator
  TELEPRESENCE_VERSION      $(shell telepresence --version)
  TEST_NAMESPACE            bookinfo

Next time you might want to try adding [skip-e2e] in the commit msg for such changes :)


Fix typos

Found few misspellings.

Looks good to me, thanks for spotting these and submitting a PR!

Glad to contribute.