What todo with the terrain estimator

@LorenzMeier @tumbili
We have a separate terrain estimator and the position estimators which sometimes also provide a bottom estimate. I don't know your plans for the position estimators. Should we remove the terrain estimator where the position estimator provides a bottom estimate? Should both estimates co-exist?

@AndreasAntener I can tell you that we are working on an integrated terrain estimator for ekf2, so we will not use the existing terrain estimator in the library. I guess for now the terrain estimator is still important for fixed wings which use the old ekf.

@tumbili ok, any suggestions what we should do with INAV? Its currently mixing both and the bottom estimate published in local position is an unfused lidar reading.

@tumbili who's working on INAV/Flow the most now? I was wondering if it would be ok if we just publish the bottom distance in local position also from the terrain estimator instead of the plain lidar value.

If needed i could make the terrain estimator publish on lpos and change inav to use that

For LPE, I am getting ready to add a separate terrain altitude state (vertical distance above terrain). The issue is really that you lose your global altitude accuracy from lidar if you give it a separate terrain altitude state. That means that if we want precise terrain following, we need two modes in mc_pos_control. One that tracks terrains altitude, and one that tracks global altitude. The easiest approach would just be to make this dependent on a parameter that the use can set, similar to yaw mode.

@jgoppert I agree, for that the position controller would need two modes, which also would allow him to react on the loss of the terrain estimation.
Since your explanation on the dev call I think I'm one step further here. But I still don't fully understand your comment I think ;). If one flies outside the lidar range and back in again, you might end up with an offset global altitude compared to terrain estimation. Is that what you meant?
Here I was only talking about letting INAV publish the separate terrain estimation (for local and global), but actually let it internally do what ever it wants (not messing with it).

@AndreasAntener I just added terrain estimation to LPE and test flew it. For now I'm outputting AGL as local z. I'm also sending terrain altitude so if we fix mc_pos_control, it should work well.
tz is the distance of the altitude relative to the home altitude.
The dynamics I'm using are just:
d/dt(tz) = 0
with sonar/lidar measuring:
y = -(z - tz);
So when it goes out of range the the tz state freezes, and the AGL is reasonable, difference just estimated from change in ASL (from baro/ gps).

Closing this, as the old terrain estimator is/can be used with INAV but the new estimators bring their own. Position reset logic with the old one is not implemented though.


Extending Be My AI functionality to support follow-up questions

@alekssamos Greetings. Now that CloudVision supports Be My AI, would it be possible to enhance the add-on to support the asking of follow-up questions, like what JAWS also does in its May 2024 release, after image recognition is finished?
Thanks.

@alekssamos Any chance of implementing this? With Be My AI, the ability to ask follow-up questions is very important.

@amirsol81  I'm trying right now, I'm 70% done, so far the error, I don't know how to fix it. https://github.com/alekssamos/cloudvision/commit/eb0528d029c4a08702941f1fee2b56672d041e7a


Replace createEAttribute, ERef... with Factory create... (ID...)

Use factory method instead of self defined model operations

Ecore4CPP modified, self defined EFactory::createE{...} removed from model ecore.ecore (part of issue #4 )


UI: Apply discount to basket/order

Designs
https://projects.invisionapp.com/share/KH11D3ZN53GC#/screens/462885073
https://projects.invisionapp.com/share/KH11D3ZN53GC#/screens/462972188
https://projects.invisionapp.com/share/KH11D3ZN53GC#/screens/462972599
Requirements

[ ] Allow the user to apply at most one discount (we should make sure the backend enforces this if it doesn't yet)
[ ] Discount computation should occur only on the backend, this might require some API updates.
[ ] User can remove the discount (I don't think the designs have this, but it really should be allowed)
[ ] Ensure the discounted price is applied to checkout

Question: for the one-discount-per-order rule: should the customer be able to apply a discount over an already applied discount (i.e. I have discount code "halfoff" and try to apply "differentcode"; should "differentcode" overwrite "halfoff")? Or should it present an error and request that the user hit the clear button before applying another discount?

@jkachel I'd say just let the new code override the previous one


Differentiate between logging in and signing up

Hi,
I would like to put in a feature request to differentiate between signing in and signing up.  Right now the buttons always say "Sign In". It would be really nice to pass in a parameter that allowed us to set the language to "Log in" or "Sign up".
This request is based off of user feedback, people have come to our sign up screen and asked where do I Sign Up?
Thank you!
Laura

This is something we are looking into with our UX team to determine the best practice moving forward.

@ldetmer As a temporary workaround, you may be able to use custom strings.  The documentation doesn't specifically mention buttons, but I don't see why they wouldn't be included.

I posted a question on stackoverflow on using custom strings:
http://stackoverflow.com/questions/41829364/use-different-bundles-to-load-strings-from-on-runtime-in-ios
Please have a look and let me know if you have any ideas on this.

Can I please have an estimation as to when this task will be implemented? I see that on Sep 12 a priority:p3 tag was added to the issue, but I don't understand how priorities work, since there are almost 4 months past since then and I see that some p:2 tasks are not done also. I think this is an important issue, because otherwise users will be asked to "sign in" to an app that they haven't yet signed up. I assume it is also easy to fix, because all it needs is changing some labels.
Please let me know, thanks.

@yorgosg it's unlikely either of them will have time to work on this soon.
Until then, PRs to expose more customizability would be good--FirebaseUI Auth is intended to expose a good auth flow by default, but our framework shouldn't be hostile toward developers who want to do extra work to add onto that.

What about adding the boolean flat autoDetectSignup to FUAuth defaulted to YES;
if parameter is set to NO FUI FUIEmailEntryViewController will not be displayed. Instead it will directly show FUIPasswordSignUpViewController
In such case if email field is blank right navigation button will be "Sign Up" once the user add any character to the email field it will switch to "Sign-In"
Additionally we can implement boolean flag disableSignUp which will completely disable signup
If mentioned functionality is good to go I can implement the feature snd submit a PR

@morganchen12  any thoughts on my suggestion ?

This mostly sounds good to me but the changing text seems potentially confusing. I don't have a better UX approach though.
@samtstern WDYT?

@morganchen12 how does the iOS flow differ from the Android flow?  On android we have:

A first screen that only has an email box and a "next" button
Three cases:

Never seen the email --> sign up screen
Email has a password already --> sign in screen
Email is linked to some social account --> move to that social flow

It's the exact same thing, but the issue we're trying to solve is before any of the three cases when entering the email we show "Sign In". In cases where the user doesn't have an account yet this led to confusion because there was no "Sign Up" text input.

Oh I see.  On Android we just say Next and it seems to satisfy both cases.

@najdanovicivan let's do this in two parts. We can implement the flag for disabling sign-up and only allowing sign-in since that's a separate issue.

@morganchen12 FWIW we have the 'allow new email accounts' boolean flag on android.

@samtstern can you provide complete explanation how does  'allow new email accounts' work on Android so that we can have similar functionality on iOS

allowNewEmailAccounts set to NO would only allow login from accounts with emails that already exist in the auth database. This is useful for invite-only services that don't allow direct sign-up, but still want to allow for people to use third-party providers.
So instead of going to the "register new user" flow, the Auth flow would always go to login.

@morganchen12 in such case if only email and password method is enabled should FUIEmailEntryViewController be displayed or should we immediately display FUIPasswordSignInViewController ?

Ok. I'll start working on it.

I've submitted pull request for allowNewEmailAccounts. However I've noticed some additional things that have to be taken care off

Left bar button for closing the signup view 'cancelBarButton' is added in viewWillAppear method.  It will be good to add hideCancelButton parameter for FUIAuth. Does something similar exist on android ?

There is an issue in FUIAccountSettingsOperation line 183 completion block. It breaks the compilation.  It should be changed to  ^(FIRAuthDataResult * _Nullable authResult,  NSError *_Nullable reauthError) { ...  I can submit separate PR for this

I don't know how this works on android but allowNewEmailAccounts does not impact social logins. So if any additional providers are enabled user will still be able to create the account. We can check newUser property from FIRAdditionalUserInfo and delete account in such case. but it won't make much sense. The only case It may have use is if signups are enabled only on specific dates or incorporate the use of Firebase Remote Config for enabling/disabling signup. Otherwise user will have no way to create account at all. More info how android implementation takes care of this will be useful

You can go ahead and implement this, and we can just hold off its release if it's not in Android (cc @samtstern)

I'll push a fix for this. Tests on Auth are still broken due to a duplicate symbols issue.

@samtstern can you advise here?

@morganchen12

I'd be OK if iOS was ahead of Android on this feature.
Social logins are much harder to handle here since there's no separate "sign in" versus "sign up" API call.  I guess you could use the "get providers for email" method after successful social sign in and then check if the account is pre-existing but we didn't do that as it's pretty complex.

@najdanovicivan, go ahead and make that as a separate PR.

@morganchen12  should we open a new issue for this or should I reference this one ?

@morganchen12 I've created the issue #451 and submitted PR #452

Let's open a new issue.

@morganchen12 now the discussed features are implemented, but we haven't touch the subject hare. The issue is about differentiating between signup and login @samtstern said that Android use "Next" as the button title why don't we do the same on iOS in FUIEmailEntryViewController ?

That sounds good. The distinction still isn't immediately clear, but it's an improvement over what we currently have.

I'see that button ins FUIEmailEntryViewController has had title "Next" from the beginning. The issue I see is in the title which is "Sign in with email". Maybe good idea will be to change this title to "Enter your email" or something similar to exclude the reference to sing in action

Sounds good to me.

@samtstern how is this done on Android. I can make a PR to change the title but I want to make sure that we have same things on both platforms

@najdanovicivan on Android the title is always "Sign in" on that screen, which I don't think is ideal.  So iOS can take the lead on improving that.

The most basic improvement would be to just change it from "Sign in" to "Sign in or Register".
As Sam said, we can make the changes in this repo before porting to Android.

@morganchen12 i was just looking at localization files and found this
/* Title for email entry screen, email text field placeholder. Use short/abbreviated translation for 'email' which is less than 15 chars. */
"EnterYourEmail" = "Enter your email";
Why don't we put kStr_EnterYourEmail as title since there is already localization for it. If we decide to use "Sign in or Register". we'll need translations for all supported languages.

Let's do this for the time being.


Add mechanism to generate file with all release images info

Is your feature request related to a problem? Please describe.
Parent Issue: https://github.com/SUSE/kubecf/issues/220
At the time of release, we would require a YAML file to be generated with information about all the release images being used.
Describe the solution you'd like
To get a file containing the list of images published that are being used by kubecf.
Describe alternatives you've considered
Create a Bazel rule that takes the required files to calculate the images being used by kubecf in the same way that the cf-operator does.
Additional context
A use case for this is to copy the images to a production registry.

closing as https://github.com/cloudfoundry-incubator/kubecf/pull/557 is DONE.


Enhancement: Metrics for website scraping

Thanks for the heads up, but actually no, just plain timing metrics for identifying if the scraping pipeline gets slower/faster/changes.
I'm already using trafilatura for extraction, so I feel safe in that regard, though it would probably be best to add an extraction test as well.


Expose processing duration metrics over Prometheus

This PR introduces new metrics which will be exposed to Prometheus.
The metrics are regarding the duration of the operations, more information in the linked Issue (link)
I already tested it on a local setup and it worked as expected, for more questions please reach out to me.

/LGTM


Test changes on kcc-UID2-2674-implement-shared-publish-to-docker-vers…

…ioned

This PR is for testing purpose


Allow facts.j2 override

Your tomcat role create an ansible fact for tomcat to expose tomcat version https://github.com/silpion/ansible-tomcat/blob/master/templates/facts.j2
But it could be cool to allow user to customize the facts, indeed I wanted to put catalina_base on it to easily manipulate it later on post_process task

Hi,
this is something I want to do/use way more in future versions. Feel free to add any information configurable from defaults/main.yml and go for PR :)

1.1.0 integrates #15, #16, #17. Again thank you very much for your suggestions and work.


[ENH] The part entity

This PR adds a description for the part entity that identifies whether reconstructed anatomical image contains magnitude or phase information.

Should we consider real and imag here as well, or is it too much?

Not sure.
Would be kind of nice and easy to integrate.
Then again, from an information perspective, if you have magnitude and phase, you have real and imag (right?!). It will be less work for future workflow developers if they know which of these two formats (real/imag vs phase/magnitude) will be used in any BIDS-compatible input data.
So I think I would vote against it, but I'm open to be convinced.

Then again, from an information perspective, if you have magnitude and phase, you have real and imag (right?!)

To properly reconstruct phase images, one usually need images from multiple channels. I remember the disappointment when sqrt(Re 2 + Im2) gave me the same magnitude image scanner gave, but arctan-1(Im/Re) did not give the same phase image :) Phase measured in each channel is subjected to that channel's sensitivity profile, so it is required to combine them to get proper phase images. Bottomline, without that information real/imag --> mag/phase transformation is not that ideal and vice versa.
I know that methods like fslcomplex has more through implementations for mag/phase conversion as a prior step for B0 estimation, for which the above non-ideal situation would not be a concern. But allowing part to use real, imag would be good for allowing some other qMRI applications, such as QSM.

Ah, good to have a proper MR physicist on board, @Agah 😆.
So I'm all for real/imag as well then. Can you draft something up?
I also slightly edited this text because I think it still needed to be converted from the old repo. See https://github.com/bids-bep001/bids-specification/pull/18

Yes the real/imag seems a good idea and also a simple back-compatible extension


Reveal Pull Request 06/09/2023

Slightly modified "Reveal_Type" function to print a more clear output;
Made a comment on Reveal_Size about the usage of -s option (unclear usage or not working)

Glad I contributed to it.
Thanks for clarifying the -s option, I would have updated the code to specify output format for memory, but the argument parsing is done in a way that prevents me on doing it.
However if you intend to expand further the project I would like to contribute to it and also I would suggest some refactoring to multiple files, that can also lead to more usability in future projects.

Contributions are welcome!
About this pull request, I can still merge it if you do the naming changes for the labels in the Reveal_Type function that I have suggested and remove the comments. I can think later about an improvement for the help page. Once everything is done, I can create a new release for them.
About your other suggestions, it would be better if you suggest them by creating new issues on the project's issue page so they can be discussed before creating any pull requests.

Reveal_Type corrections done, also removed the comments.

Thank you for your pull request.


PostData become only PostData after upgrading to ElasticSearch.Net 7

ElasticSearch.Net 7:
Visual studio Intellisense found error when passing a dynamic parameter to Search(PostData)
After upgrading the library from 5 to 7 this error is encountered
Should able to pass dynamic parameter to the method

Hi @argelj289. Can you please provide some more information and a small working example of the code you're migrating so that I can provide some guidance? What is the IntelliSense error? There are breaking changes between 5.x and 6.x and again to 7.x which you're running into here.
The main change between 5.x and 7.x is that the generic argument on the Search method is constrained to implementing IElasticsearchResponse. Possibly this is what you're seeing?
Here's a small example of sending a request using the low-level client:
private static void Main(string[] args)
{
	var client = new ElasticLowLevelClient();

	var a = new
	{
		query = new
		{
			match_all = new { }
		}
	};

	SendSearch(client, a);
}

public static void SendSearch(IElasticLowLevelClient client, dynamic search)
{
	var response = client.Search<BytesResponse>(PostData.Serializable(search));

	// TODO - Deserialize bytes from response
}

Again, it would be helpful to understand your requirement and existing code to provide a more specific answer.
Have you reviewed the 7.x documentation about PostData and searching with the low-level client? This may be useful if you haven't seen it already.

@argelj289 Is this still an issue for you?

Closing this stale issue.


feat(view-engine-hooks-resource): Add default ViewEngineHooksResource and decorator

@davismj This should actually go in the templating repo also. Do you mind updating your existing PR with this?

i don't understand? there is a templating pr as well
https://github.com/aurelia/templating/pull/371

I mean the actual ViewEngineHooksResource type should go in the templating repo, not in the binding repo.

The binding engine is a lower dependency than templating. The actual view engine and all it's details are defined in templating, so technically binding shouldn't know anything about that. @jdanyow may have some thoughts on the camel case helper.

Turns out the camelCase class was not required after all. Pull request updated here: https://github.com/aurelia/templating/pull/381


use go-bindata to autogenerate the chart

Right now we have the chart that is installed by draft init embedded directly in the code as well as in chart/. This means that maintainers need to make changes in two different places, and it has gone out of sync many times. We should use go-bindata to autogenerate the go code based on the charts as we do with the draft packs in #108.
blocked by #236 as that will change the directory structure of the chart.

@bacongobbler, what is the user's experience of this? Is this an internal thing, or will they see some change as a result?

No user experience change, just less code duplication.


set to pylint options

Fixes Pylint errors.
Changes made in this Pull Request:
PR Checklist

[ ] Tests?
[ ] Docs?
[ ] CHANGELOG.md updated?
[ ] AUTHORS.md updated?
[ ] Issue raised/referenced?

I'm just going to self-merge to fix CI.


Interface segregation syntaxes support

If my object supports few interfaces I need to write wrapper for each one.
Can you support some syntaxes for single object which support several interfaces?
For example:
@provide(scope=Scope.REQUEST)
def get_db(self) -> I1 | I2 | I3:
    return object()

Python Union has different semantics so we have different typehint for that case: AnyOf, which actually creates 1 factory and several aliases.
https://dishka.readthedocs.io/en/stable/provider/provide.html


[DashboardEditor] There should be a dynamic way of changing layouts / breakpoints

What package is this for?

[x] React
[ ] Angular

Summary
Currently, the only way of changing layouts or breakpoints is to resize your window, which limits users to their current screen size. This causes issues for users trying to set up their dashboards for screens smaller or larger than their current window because they either can't in the case of larger screen sizes, or they need to manually resize their window and hope to get the right layout. The other issue is that the user doesn't know what breakpoint they are at if they are manually resizing their window
Specific timeline issues / requests
Do you want this work within a specific time period? Is it related to an
upcoming release?
November release

:tada: This issue has been resolved in version 2.141.0-next.2 :tada:
The release is available on:

npm package (@next dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This issue has been resolved in version 2.141.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


git rebase problem

Hi, since some time I'm using bash-it - it's cool
but today I wanted to rebase on branches and see following error
/usr/local/Cellar/git/2.4.5/libexec/git-core/git-rebase--interactive: line 245: -w: command not found
Could not execute editor```

can you help?
I've tried without success `git config --global core.editor "vim"`

Where do you see Bash-it involved in this? Does the same work in a terminal when you're not using Bash-it?

Question: Do you have the textmate plugin enabled? It tries to use $(which mate) -w as the Git editor.
Try disabling the textmate plugin:
bash-it disable plugin textmate

Then please try the rebase again.

wow that was fast ;)
So problem doesn't occur when I use my previous bash settings.
And thank you for tip for disabling textmate - it helped ;)

Great, good to hear!


sveltekit: AuthApiError is not defined & Property 'status' does not exist on type 'AuthError'

Bug report
Describe the bug
Discussed in https://github.com/supabase/supabase/discussions/10109
Originally posted by cananda November 5, 2022
I'm using all latest versions of sveltekit (535) and supabase-auth-helpers (0.8.6). Following the example here: https://github.com/supabase/auth-helpers/blob/main/packages/sveltekit/README.md
In this piece of code -
    if (error) {
      if (error instanceof AuthApiError && error.status === 400) {
        return invalid(400, {
          error: 'Invalid credentials.',
          values: {
            email
          }
        });
      }

.. I get AuthApiError is not defined and I also see 'Property 'status' does not exist on type 'AuthError'.ts(2339)' for error.status
Am I missing some imports?
To Reproduce
Follow sveltekit example with latest sveltekit and auth-helper-sveltekit

OS: Linux Fedora 37
Browser (if applies): Chrome/Firefox
Version of supabase-js: 2.0.6
Version of Node.js: 18.7.0

The example is missing the part where it imports AuthApiError, you will need to import that into the file first. I'll get the README updated and the docs updated too.

Fixed in commit f1456ca


Change image tag on containers

The Puppet code that used the ::docker::image and ::docker::run classes did not fit into our current deployment model as it meant Puppet managed which images were running, whereas we need a job which deploys a newer image and then restarts the service.
the ::docker::run defined type is useful as it creates an init script, so we're going to have a generic "release_env" tag, and then use docker image tag to point this specific tag to a specific image.

Looks good


New package: MakiePDF v1.0.2

Registering package: MakiePDF
Repository: https://github.com/daizutabi/MakiePDF.jl
Created by: @daizutabi
Version: v1.0.2
Commit: 2a09118f5ea95bf5d67562ed071a161f2f5b3c45
Reviewed by: @daizutabi
Reference: https://github.com/daizutabi/MakiePDF.jl/issues/1#issuecomment-1859626778

Close


Save a local copy of gist for situation that cannot connect network

Is it possible to save a copy of all gist in local? If the network is connected, using the one from gist. Otherwise we still have a copy in local and still can work.

hello guys, this is a really useful features that can be included in the plugin.
seems the repo isn't active development anymore,
guys, how about we pick this up and work it out together?


“Sushi swap” dapp name is not displayed in connection prompt in Valora app when user tries to connect the sushi swap to valora app

Frequency: 100%
Repro on build version: Android Internal Release Build V1.25.1, iOS Test flight Release build V1.25.1
Repro on devices: iPhone 13 mini (15.1.1), Google Pixel 2XL(11.0), One plus 7T(11.0), iPhone XR(14.6)
Pre-condition:

User must have installed the app & completed on boarding flow.
User must have opened https://app.sushi.com/swap in web browser.

Repro Steps:
1.Tap on Connect to wallet button from web page.
2. Select Wallet Connect button.
3. Select “Valora” option from “Choose your Preferred Wallet” popup & observe when user redirects to valora app.
Bug:  “Sushi swap” dapp name is not displayed in connection prompt in Valora app when user tries to connect the sushi swap to valora app
Expected Behavior:  “Sushi swap” dapp name must be displayed in connection prompt in Valora app when user tries to connect the sushi swap to valora app
Investigation:

Dapp name option is not displayed in list of connected apps screen when sushi dapp is connected.
"Success! Please go back to  to continue" is seen on the homepage after connecting the DApp

Impact: Bad user impact.
Attachment:
Sushi Connection Prompt.jpg
Connected App screen.jpg

@ValoraQA this appears to be a bug with Sushi swap and not Valora.

Hey @MuckT, We have verified the above issue on Latest Android Internal Release Build and Test flight Release Build V1.28.1 & Observed that the issue is fixed now. “Sushi swap” dapp name is able to see in connection prompt in Valora app when user tries to connect the sushi swap to valora app.
Devices: Samsung Galaxy A9 pro (8.0), iPhone 13 mini(15.1.1), iPhone 6+(12.5.5), Google Pixel 2XL(11.0)
Attachments: Sushi.mp4
Thanks..!!


Chinese client- Gw2.dat error

why i can't star....

help....THX....

only star one game...can't open 2....

I am sadly not familiar with the chinese client of gw2 and do not have the client to test it on my end.
My best guess would be that your Gw2.dat was not run with the -shareArchive argument. Have you been able to use older versions of LB?

I am sadly not familiar with the chinese client of gw2 and do not have the client to test it on my end.
My best guess would be that your Gw2.dat was not run with the -shareArchive argument. Have you been able to use older versions of LB?

1.6   1.7  1.9 ALL don't star 2.......BUT,i copy 2 GW2 folder in  D and E ....Step1, LB GW2,ok,star,Install Path is D:\GuildWars2\GW2-64.exe...Step2,Set Directory,E:\GuildWars2\GW2-64.exe,and LB GW2.ok,star...Although it can be started in this way, I still hope not to copy multiple clients

This is likely a mutex issue. Someone would need to run the Chinese version to the point where the handle is set, and verify the string.

This is likely a mutex issue. Someone would need to run the Chinese version to the point where the handle is set, and verify the string.

...i need Someone,thank~.......and 2.20,Start crash...Exchange2.15

run the Chinese version to the point where the handle is set, and verify the string.

and how to run the Chinese version to the point where the handle is set, and verify the string....if i can,i do it...thank you ~

I am sadly not familiar with the chinese client of gw2 and do not have the client to test it on my end.
My best guess would be that your Gw2.dat was not run with the -shareArchive argument. Have you been able to use older versions of LB?

how can run  Gw2.dat with the -shareArchive ....i use older LB also can't star 2 in ONE client...thank you~

.....What happened after I started LB now.....if i want STAR 3,4,5,must copy 3,4,5client in Different partitions...GOD~

The Chinese version of the account is not only email, but also can use mobile phone and custom personalized account registration, now automatically login account can only add email, I hope you can open the function to add various accounts, thank you
---Original---
From<EMAIL_ADDRESS>Date: Thu, May 28, 2020 00:55 AM
To<EMAIL_ADDRESS>Cc: "State change"<state_change<EMAIL_ADDRESS>Subject: Re: [TheCheatsrichter/Gw2_Launchbuddy] how (#181)
I am sadly not familiar with the chinese client of gw2 and do not have the client to test it on my end.
My best guess would be that your Gw2.dat was not run with the -shareArchive argument. Have you been able to use older versions of LB?
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub, or unsubscribe.

Unfortunately, unless one of us gets their hands on the client we are likely not going to be able to do much. The problem I have found while trying to accomplish this in the past, is that accounts seem to be directly tied to "you the person" meaning that it is exceedingly hard for someone outside of China to get and use an account.


How do you use the included CSS in a component?

nevermind, figured it out.

@synic I will update the README.md file to clarify that. Thanks for bringing this to my attention!


Difference not detected when an implementation returns nullopt and another returns some value.

In differential.cpp, there is a logic error where all implementations returning std::nullopt are ignored:
https://github.com/sigp/beacon-fuzz/blob/32b6db205143b1dd574b1df5e39c3d2d0f474ed2/files/lib/differential.cpp#L21-L31
This means a difference is not identified when 1 impl returns a std::nullopt and another returns a value.

Resolved


New Theme - Port Hugo

Thanks; a tip: if you put the URL to your demo in demosite in theme.toml, that will be used for the demo button. But you can do that at any time.

@bep thanks for the tip. I'll get the demos up soon.

@bep I just created a small npm package and a short guide to help with deploying from the exampleSite folder to Netlify. The npm package could be used to help any build process for the exampleSite folder, but the guide specifically walks through the Netlify setup process.
Figured I'd share this with you in case it can be of use for helping others deploy theme demos to the site.


Create check.yml

update

What does that do / fix / improve?

What does that do / fix / improve?

It is make PR pkg for install

What is "PR pkg"

What is "PR pkg"

https://github.com/stackblitz-labs/pkg.pr.new

@appujet You wanted to add smt to this pr

This pr looking good but it is just create package of PR

Is it required to do with pnpm?
Can we not use smt like bun or normal npm for packages?
Since it's a container there isn't a cache and thus you cannot profit from pnpm technically
I'm currently considering to approve this pr otherwise

Run actions/setup-node@v4
Found in cache @ /opt/hostedtoolcache/node/20.17.0/x64
Environment details
/usr/local/bin/pnpm store path --silent
/usr/local/lib/node_modules/corepack/dist/lib/corepack.cjs:23509
throw new UsageError(This project is configured to use ${result.spec.name} because ${result.target} has a "packageManager" field);
^
UsageError: This project is configured to use yarn because /home/runner/work/lavalink-client/lavalink-client/package.json has a "packageManager" field
at Engine.findProjectSpec (/usr/local/lib/node_modules/corepack/dist/lib/corepack.cjs:23509:21)
at async Engine.executePackageManagerRequest (/usr/local/lib/node_modules/corepack/dist/lib/corepack.cjs:23539:24)
at async Object.runMain (/usr/local/lib/node_modules/corepack/dist/lib/corepack.cjs:24232:5) {
clipanion: { type: 'usage' }
}
Node.js v20.17.0
Error: /usr/local/lib/node_modules/corepack/dist/lib/corepack.cjs:23509
throw new UsageError(This project is configured to use ${result.spec.name} because ${result.target} has a "packageManager" field);
^
UsageError: This project is configured to use yarn because /home/runner/work/lavalink-client/lavalink-client/package.json has a "packageManager" field
at Engine.findProjectSpec (/usr/local/lib/node_modules/corepack/dist/lib/corepack.cjs:23509:21)
at async Engine.executePackageManagerRequest (/usr/local/lib/node_modules/corepack/dist/lib/corepack.cjs:23539:24)
at async Object.runMain (/usr/local/lib/node_modules/corepack/dist/lib/corepack.cjs:24232:5) {
clipanion: { type: 'usage' }
}
Node.js v20.17.0

@dawgcodes


Android: Improve Icon Cache Cleanup

The newest Android / Google Play Store framework has a feature that suggests to users that apps should be uninstalled if their use of device storage is too high.  This is probably a good time to get our ducks in a row w.r.t. how we cache icons.
Though they are very small and the dataset changes very slowly, we essentially never clean up the favicon/profilepic caches.  We should refit those caches with the nice, new cleanup framework that we implemented for the offline image cache last year.

fixed in #938


Add more requirements.txt

Hi! Thanks for the amazing work :smile:
I wanted just to report that more requirements need to be satisfied in order to run the repository smoothly.  I will list them here and open a pull request later on :smile:

numpy <2.0.0  I installed the 1.26
scipy< 1.13     I installed the 1.12

Thanks Matteo!


Discrepancy related unknown dependencies

All the dependencies show up as 'unknown' and still see recommendations for companion packages.

is someone working on this?

@samuzzal-choudhury is someone looking into this?

@SriKrishnaPaparaju - please provide the manifest file and also the stack-id if you have for this stack request. TBH @SamuelStuchly , this looks to be handled at backbone layer.

Unassigning myself. This should be handled at Backbone. @samuzzal-choudhury - please assign this to yourself

@samuzzal-choudhury I think its not a bug. We show recommendations based on the user's stack. For unknown deps, its a case of deps not present in the graph, which eventually will get populated via the unknown flow.

@yzainee In my opinion, this still requires handling at the API side as this makes the user experience inconsistent. Possibly (I am yet to check this), we can add a filter at the API layer for recommendations when analyzed dependencies count is zero.

The changes have been moved to production. Hence, closing the issue.


added qOSMO as an asset

qOSMO IBCDenom on osmosis  ibc/42D24879D4569CE6477B7E88206ADBFE47C222C6CAD51A54083E4A72594269FC

Do you have a pool id for qOSMO? If not, is a staging link required to to create one?

You may use this staging link to create a pool.
https://osmosis-frontier-git-jp69-add-qosmo-osmo-labs.vercel.app/

@JeremyParish69 - pool id is 956

Thanks for confirming. Could you please add to the osmosis.zone.json file?

@JeremyParish69 have added to osmosis.zone.json file


ETL commodities not in some sheets.

there are some sheets where only the commodity type is entered, not the commodity. We need commodity entered in all of these.
this is at least the case in:

https://docs.google.com/spreadsheets/d/1KDhQpIXLcy4uI8FJ17W9r7NPQIiQBdzjSzSCX7K9L7w/edit#gid=141760938
https://docs.google.com/spreadsheets/d/1nArJqi0RnyZ_WsKO9zxCSZGkG5ltonPVnoSvIZVWpww/pub#
https://docs.google.com/spreadsheets/d/17FDwVbDCReeWxufE2Gab1yw1PqJpQIt6pASM92d7sl4/pub#

Done


curl on system-wide python on Mac never install completely to Python3

[x] I am on the latest Poetry version.

[x] I have searched the issues of this repo and believe that this is not a duplicate.

OS version and name: MacOSX 10.13.4, MacBook Air 11" (Early 2015)

Poetry version: 0.11.2

Issue
Running curl -sSL https://raw.githubusercontent.com/sdispater/poetry/master/get-poetry.py | python and curl -sSL https://raw.githubusercontent.com/sdispater/poetry/master/get-poetry.py | python3 outside or inside virtual environment gives a poetry that doesn't work. When pip3 freeze, too few packages are shown...
However, using pip3 install poetry in system-wide, or pip install poetry inside the virtual environment in PyCharm works perfectly.
I also noticed that PyCharm detected the wrong pip version.

When you say that poetry doesn’t work, what do you mean? Does poetry show an error when you run a particular poetry command?

BTW, how do uninstall poetry (completely)? I want to replicate the error message (and clean my system Python2 and Python3).

@patarapolw pip uninstall poetry, if you installed through pip or the official get-poetry.py script.

Cannot replicate the error for now, but I believe:-

pip uninstall poetry, if you installed through pip or the official get-poetry.py script.

I might have to pip freeze | xargs pip uninstall -y, but I don't know of the side-effects, if I do it on system-wide Python.

The new installer is out which fixes a lot of installation issues.


lint: Only fail command if error is returned.

If errors only contain warnings, treat this as a success.

cc @rawlingsj


chore: zeebe-knative-worker to 0.0.8

chore: Promote zeebe-knative-worker to version 0.0.8

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approvers:
If they are not already assigned, you can assign the PR to them by writing /assign  in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


Fixed inconsistent Messages

Changed that instead of telling the player the word "gamemode", "GameMode" because all other instances of this word are written in pascal case, and this in just lowercase.

Changed "gamemode" to "GameMode"


feat: Add javascript definitions to .sqlx files

Description
This draft PR introduces a new feature that allows users to navigate directly to the row of a variable or function when clicked. The functionality is demonstrated below:

The implementation is still a work in progress and may be somewhat inefficient or messy at this stage. I’d appreciate any feedback, change requests, or suggestions for improvement. Feel free to use this as a foundation to restructure or refine as needed.
Outstanding Tasks

Improve Variable Validation: Refine logic to better identify actual JavaScript variables (currently, it captures anything starting or ending with a period).

Enhance Search Logic: Update the search logic to accurately retrieve the correct row number, especially when the same name appears multiple times in the .js file.

Consolidate Classes: Consider merging the current three classes into a single DataformDefinition class to streamline the logic for retrieving the appropriate definition. This consolidation isn’t critical for now, but it may improve maintainability and scalability in future iterations.

Questions / Next Steps
I'm considering how best to structure functions for reusability, especially if we decide to add onHover functionality. Any guidance or suggestions on this would be greatly appreciated!
TODOs

[ ] Update documentation before merge

Hi @HampB ,
thanks for the PR. The basic functionality is definitely there albeit needs refining. Would be doing some tests and suggesting changes as as we continue to work on it.

One of the edge case I found was the following
config {
  type: 'table'
}

js {
  const SIX_MONTHS_AGO_FROM_NOW = parameters.getDateMonthsBefore(new Date(), -6)
}

select ${SIX_MONTHS_AGO_FROM_NOW} as some_time

would have been nice if the Dataform graph had some metadata of where the javascript variables are coming from. I feel our best bet for the above is use the output of getMetadataForSqlxFileBlocks  function and check if there is a js block and then find for the declaration there first ?

Done some fixes to adress issues in https://github.com/ashish10alex/vscode-dataform-tools/pull/52
Still unfixed issues:

Cant click the module part of a statement like this ${module.varOrFunction} (should take you to the top of that file?)
The search in .js doc matches the first occurrence of text (pass the searchTerm with a prexif of const, var, function, and so on or do a regex based search?)

The search quite often takes you to the wrong place in our production repo.

The search in .js doc matches the first occurrence of text (pass the searchTerm with a prexif of const, var, function, and so on or do a regex based search?)

yes, agree, ill write some code to make this change

putting the file in here which I am using to test the features
config {
  type: 'table'
}

/*
create a parameters.js file in the includes directory and add
getDateMonthsBefore function

function getDateMonthsBefore(date, months) {
  date.setMonth(date.getMonth() + months);
  let date_string = date.toISOString().split('T')[0];
  date_string = `'${date_string}'`;
  return date_string

}
*/

js {
  let some_mod = require("includes/parameters.js")
  const six_months_ago_from_now = some_mod.getDateMonthsBefore(new Date(), -6)
  const four_months_ago_from_now = some_mod.getDateMonthsBefore(new Date(), -6)
  const some_month = "january"
  const some_number = 11;
  let some_coniditon = true;
  let another_coniditon = true;
  var some_floating_points = 9.4;
}

select
  *
, ${six_months_ago_from_now}  some_time
, ${four_months_ago_from_now}  some_time
, ${some_number}  some_number
, ${some_coniditon}  some_condition
, ${another_coniditon}  another_coniditon
, ${some_floating_points}  some_floating_points
, "${some_month}"  some_month
, ${some_mod.getDateMonthsBefore(new Date(), -4)} date_wt_func
, ${parameters.OXRC_DATA_STARTING_POINT}  as date_from_file

FROM ${ref("0100_CALENDAR")}

Thanks for the additions @ashish10alex!
I think this should be production ready soon.
But first another issue that has come up in one of our iterations is that, in some cases, all words on the line receive the function or variable definition instead of just the intended variable or function name.
For example, this code assigns the function definition to "Var1", "Var2", AS, and unique_key:
js {
  let utils = require("includes/functions/utils.js");
}

SELECT
${utils.createUniqueKey(["Var1", "Var2"])} AS unique_key

The same thing seems to happen when the identifier name is the same as the variable inside of the ${}. Like this:
${some_coniditon} AS some_condition
One quick solution could involve adding a check like this inside the DataformJsDefinitionProvider:
const startIndex = line.indexOf(variableOrFunctionName);
const endIndex = startIndex + variableOrFunctionName.length;

if (position.character < startIndex || position.character > endIndex) {
    return undefined;
}

I can see if i can find a better solution and give it a try later, if you do not have any clear way of solving it? :-)

Hi @HampB , thanks, I was unable to reproduce the issue you were having. I used the following code to test. Please let me know what I am doing differently
config {
  type: 'table'
}

/*
let someVariable = "createUniqueKey";

function createUniqueKey(someList){
  return someList.map(item => item.toString()).join("")
}

module.exports = {
    createUniqueKey,
    someVariable,
};
*/

js {
  let some_mod = require("includes/functions/utils.js")
}

select
  "${some_mod.someVariable}" as someVariable
, "${some_mod.createUniqueKey(['Var1', 'Var2'])}" AS unique_key

https://github.com/user-attachments/assets/8a2377ae-0315-423b-a4c8-91d0e3e0ff28

thanks @HampB for the clarification. Please go ahead with the fix that you were suggesting. Will review that

This PR should be ready for final review and testing.
I've been running this branch during the day and have not ran into any issues so far.

Thanks @HampB , I will update the docs sometime this weekend and merge this PR


Delay

It would be useful to have a delay optional parameter that would add a sleep between each retry.
Also a msg or message would be useful just as a way to have also in the output why it may be flaky (in cases it flakes or fails especially).

Flaky can be configured to call a function after a flaky failure. It was designed as a way to skip reruns, but it could also be used to delay them. See #55 for more details. I'd imagine you'd do something like this:
import time

def delay_rerun(*args):
    time.sleep(1)
    return True

@flaky(rerun_filter=delay_rerun)
def test_flaky_thing():
    pass

Sorry, I didn't address your second point. I'm not sure what kind of message you mean. The flaky report includes info about flaky tests failing.


add how to add new workload to benchmark runner

This PR contain how to add new workload readme file

@RobertKrawitz, lets discuss it in the next meeting, I think that the best practice is to enrich the document while adding a new workload and see if there is a missing stuff or need to correct something. I have added the highlights and verified that add all the important things is there.

@RobertKrawitz, lets discuss it in the next meeting, I think that the best practice is to enrich the document while adding a new workload and see if there is a missing stuff or need to correct something. I have added the highlights and verified that add all the important things is there.

I concur.  So let's keep this one on hold for the time being (and please fix those two other nits and say what you can to help guide someone to the correct grafana instance).


popovers: Change style of emoji and giphy search bar.

This pull request  is aimed at improving the style of emoji and Giphy popovers by aligning their styling with that of the existing dropdown-widget both in dark and light mode as noticed by @amanagr in #26929
Fixes: #26929

Screenshots and screen captures:
Light Mode

Dark Mode

Self-review checklist

[x] Self-reviewed the changes for clarity and maintainability
(variable names, code reuse, readability, etc.).

Communicate decisions, questions, and potential concerns.

[x] Explains differences from previous plans (e.g., issue description).
[x] Highlights technical choices and bugs encountered.
[x] Calls out remaining decisions and concerns.
[x] Automated tests verify logic where appropriate.

Individual commits are ready for review (see commit discipline).

[x] Each commit is a coherent idea.
[x] Commit message(s) explain reasoning and motivation for changes.

Completed manual review and testing of the following:

[x] Visual appearance of the changes.
[x] Responsiveness and internationalization.
[x] Strings and tooltips.
[x] End-to-end functionality of buttons, interactions and flows.
[x] Corner cases, error conditions, and easily imagined bugs.

Thanks! Can you add screenshots with text for both popovers, in addition to the ones you have without?

We should not have an x in the GIPHY search box when there is no text there.

Thanks! Can you add screenshots with text for both popovers, in addition to the ones you have without?

Sure, Here they are:

We should not have an x in the GIPHY search box when there is no text there.
We can remove the x altogether as it is not there in emoji nor dropdown-widget popover.

I think there should be an x in both of them when there's text there, and not when there isn't. That's our standard approach across the Zulip app.

Okay, will fix it. Also I noticed there are several other search bars where x is there whether there's any text or not like search usernames, Search all messages, filter streams etc. So I am thinking either I can create another pull request fixing all these or I can make changes in this pr only, what do you think will be better?

@nimishmedatwal it would be really helpful to have before and after screenshots here, too, so it's more obvious what's being changed. Table syntax is really great for showing side-by-side before and after versions.

I think there should be an x in both of them when there's text there, and not when there isn't. That's our standard approach across the Zulip app.

Actually I was thinking to add that functionality in another pull request, as I noticed that many of the search bars are having x even when there's no text, like search messages, search usernames and filter streams.

Those search bars are closed by x, which is not a use for x that we have here.

We should not have an x in the GIPHY search box when there is no text there.

@alya I've added that functionality, can you please give it another review?

Please update the screenshots in the PR description, showing boxes with and without text. Videos are quite difficult for me to review.

Please update the screenshots in the PR description, showing boxes with and without text. Videos are quite difficult for me to review.

Updated !

This PR breaks tab keyboard navigation from the search box input to the x.

Works for me other than that.

This PR breaks tab keyboard navigation from the search box input to the x.

@alya I've fixed tab keyboard navigation in the giphy search box input.
Screenshots :

I am hesitant to add this tab navigation in emoji search box input as in emoji-picker.js keydown event on tab is used to select the selected emoji by this code. Although if needed I'll remove this and change it to clear-button. So let me know what you think.
if (event_name === "tab") {
    $selected_emoji.trigger("focus");
    update_emoji_showcase($selected_emoji);
    return true;
}

What if we make the first tab go to the x, and tabbing when x is selected would go to the first emoji? That would probably feel pretty reasonable.

What if we make the first tab go to the x, and tabbing when x is selected would go to the first emoji? That would probably feel pretty reasonable.
We want the same behavior in the GIPHY popover as well. The current behavior there seems weird -- I'm needing to press Tab more than once to get to each next location...

@alya actually that is because tabindex of simplebar-content-wrapper is set to 0 so it's focusing to that wrapper when pressing tab.

@alya I've corrected that and now the emoji and giphy popovers tab focus are working perfectly fine.

@amanagr I'd like to pass this on to you for further manual testing and code review. Not a release priority.

@amanagr made the requested changes, kindly give it another review.

@amanagr made the requested changes!

@nimishmedatwal sorry for the late followup, can you rebase this?

The search icon is not aligned in the center in both these popovoers, can you please fix it too?

@amanagr I am having my mid semester exams right now so if this issue is not high priority would it alright if I work on it after 22nd March ?

@amanagr I've fixed the search icon in emoji popover, also since these changes were made quite some time ago, I can't recall why exactly I did them. For now, I've reverted them back as they had no impact on the functionality of my pull request.

Why is E: Package 'libmagic1' has no installation candidate this error coming in Installing dependencies. I've not made any changes related to that.

@nimishmedatwal thanks for the update! Left a comment, otherwise LGTM.

@amanagr I've changed the code so that it uses a common variable wherever possible. Thank you for your thorough review.

@nimishmedatwal left a minor comment, otherwise LGTM.

@amanagr now using hide and show  jQuery methods.

LGTM! Thanks!

Hey @nimishmedatwal, how's the progress on this? Please let me know if you're stuck anywhere and would need some help. Otherwise, we could have one more look at the code and move this forward to be merged.

@sayamsamal I've resolved the conflicts and this pr is ready to be reviewed again!

Thanks @nimishmedatwal. I'll review this very soon.

Okay, I have left you an initial review to keep you unblocked. It would also be nice to see https://github.com/Giphy/giphy-js/issues/174 mentioned in the commit message since that version bump actually fixed an issue for us, and also since as a bonus that issue was started by one of our maintainers.
I still haven't performed a manual review BTW. (mostly a reminder to myself)

Closing in favour of #30624


IGVF-224 ontology term

Coverage test is failing but it seems to be flaky. We could try rerunning it.


ARROW-17319: [Python] Update VCPKG to include updated aws-c-io

Should solve https://issues.apache.org/jira/browse/ARROW-17319

@github-actions crossbow submit -g wheel

@github-actions crossbow submit -g wheel

Closing because it has been untouched for a while, in case it's still relevant feel free to reopen and move it forward 👍


GK-200MP2-B Camera Not Working

GK-200MP2-B Camera is not working. No camera entity.
Calling
- service: sonoff.send_command
data:
device: '012345'  # use quotes, this is important
cmd: left
gives green tick but nothing comes up on the camera.
No way to view the camera stream also.

I have the same problem. I have identified that PTZ has stopped working for the cameras I recently added to eWeLink on the smartphone. For example, I had a camera that PTZ commands were working, but when I deleted it from eWeLink and added it again, PTZ commands in HA stopped working. But the cameras that are longer on eWeLink, PTZ commands work on HA. I believe it is a compatibility issue with some eWeLink update.
One could remove a GK-200MP2-B camera from eWeLink on smartphone, and remove from HA. And then add the same camera to see if the PTZ will stop working?
eWeLink - Version: 4.23.0
core-2022.6.7
Home Assistant OS 8.2
Sonoff - Version: 3.0.5 (200f243)

@iotinkognito Older versions of eWeLink are not running RTSP. With me RTSP is only working on version 4.23.0 (the latest one so far).


fix heroku deploy link

Following the instructions here: https://devcenter.heroku.com/articles/heroku-button
Probably fixes: https://github.com/RSS-Bridge/rss-bridge/issues/1501
Basically just adding a link to the repo so the deploy can happen automatically.

I didn't notice any difference between those two urls. Maybe because im not logged in.


Refactored backend path

Together with https://github.com/preview-code/backend/pull/7

LGTM when the nit of @txsmith has been fixed.


rubber-underpants chord.identify

Calling chord.identifyArray with an empty array currently blows up. This change will cause it to return undefined instead. Thoughts?

Lgtm.  Thanks for the fix!


Not convert False filter.

Boolean False is possible to be a filter of some resource.
Need to exclude it in this hack.

:+1:

What if the hack checked directly for empty list and empty dict? It is still two conditions and should not match any false positives.

Not only empty list and empty dict, but None is another one will be lost. Not sure if there are others. That's why this hack not check directly for empty list and empty dict.

Ok then. No further comments from me.


ci: add GH workflow to install crossplane in kind

Adds an initial GH Workflow
CI Testing steps intended:

install kind
install crossplane from helm
equinix-metal provider

lint
test
build

create equinix-metal project
use crossplane cli to install equinix-metal provider
create providerconfig resource (and secret)
check for ready provider
create device, vlan, etc
check for created resources
delete resources
check for deleted resources
delete equinix-metal project

Part of #6

Description of your changes

Checklist

I have:

[ ] Run make reviewable to ensure this PR is ready for review.
[ ] Ensured this PR contains a neat, self documenting set of commits.
[ ] Updated any relevant documentation, examples, or release notes.

@displague I think failure here is because it is trying to build an arm image on x86 host.. this works in Jenkins because we build in cross container. Do you know if GH action would let you use that here?

@displague if you want to merge this I can make the adjustments in my PR to get it running :)


[Suggestion] Configurable recipes for basin fermenting

Would be nice for modpack integration if the lidded basin recipes were configurable via datapack, kubejs, or crafttweaker

just realized this already works for datapacks, whoops


Bug: Model update() only set first array value

Describe the bug
When updating a Model with more than 1 array values, it only updates the first value (for first column) in the database.
CodeIgniter 4 version
CodeIgniter 4.0.4
Affected module(s)
CodeIgniter\Model
Expected behavior, and steps to reproduce if appropriate
Code used from the Controller:
$db = \Config\Database::connect();
$data = [
    'is_allowed' => 1,
    'is_dismissed' => 0
];
$questionsModel = new \App\Models\QuestionsModel();
$questionsModel->update(intval($id), $data);

d($data);
d($db->getLastQuery()->getQuery());

And:
$db = \Config\Database::connect();
$questionsModel = new \App\Models\QuestionsModel();
$questionsModel->where('id', $id)
        ->set($data)
        ->update();

d($data);
d($db->getLastQuery()->getQuery());

Both methods for updating mutiple columns are printing this query;
UPDATE questions SET is_allowed = 1, updated_at = '2020-10-13 12:17:50' WHERE questions.id IN (1)
And:
UPDATE questions SET is_allowed = 1, updated_at = '2020-10-13 12:17:50' WHERE id = '1'
Expected query should be:
UPDATE questions SET is_allowed = 1, is_dismissed = 0, updated_at = '2020-10-13 12:17:50' WHERE id = '1'
Context

OS: Linux
Web server Apache 2
PHP version: PHP 7.4.6

Are you sure you added is_dismissed to the Model's $allowedFields array?

Well, you're right, I didn't add it to $allowedFields.
Thank you!


(Suggestion) Disable Update on start

Seems un-needed to steamcmd update each time you start it, especially if you use steamguard. You'l end up getting rate limited, if you fail the guard code because it keeps restarting due to failing steam guard code.

Hey! Thanks for your feedback :)
Steam credentials are now cached, which removes the need to re-obtain your Steam Guard code on container rebuild/restart.


sup_all_atm.pdb is not created

Hi,
while training the modeling, I always get this error (each time with a different pdb file).
As far as I understand, the TMP directory is created and the original pdb
file is moved there, but then the *sup_all_atm.pdb is not created.
Can you help me?
Ivan
Epoch 0:   0%|                                                                                | 0/203 [00:00<?, ?it/s]
Traceback (most recent call last):
File "/opt/tools/deg/PreMut/src/train.py", line 48, in
training.fit(model_name='egnn')
File "/opt/tools/deg/PreMut/src/train.py", line 43, in fit
self.trainer.fit(model=lit_module,datamodule=datamodule)
File "/opt/tools/deg/miniforge3/envs/PreMut/lib/python3.10/site-packages/pytorch_lightning/trainer/trainer.py", line 582, in fit
call._call_and_handle_interrupt(
File "/opt/tools/deg/miniforge3/envs/PreMut/lib/python3.10/site-packages/pytorch_lightning/trainer/call.py", line 38, in _call_and_handle_interrupt
return trainer_fn(*args, **kwargs)
File "/opt/tools/deg/miniforge3/envs/PreMut/lib/python3.10/site-packages/pytorch_lightning/trainer/trainer.py", line 624, in _fit_impl
self._run(model, ckpt_path=self.ckpt_path)
File "/opt/tools/deg/miniforge3/envs/PreMut/lib/python3.10/site-packages/pytorch_lightning/trainer/trainer.py", line 1061, in _run
results = self._run_stage()
File "/opt/tools/deg/miniforge3/envs/PreMut/lib/python3.10/site-packages/pytorch_lightning/trainer/trainer.py", line 1140, in _run_stage
self._run_train()
File "/opt/tools/deg/miniforge3/envs/PreMut/lib/python3.10/site-packages/pytorch_lightning/trainer/trainer.py", line 1163, in _run_train
self.fit_loop.run()
File "/opt/tools/deg/miniforge3/envs/PreMut/lib/python3.10/site-packages/pytorch_lightning/loops/loop.py", line 199, in run
self.advance(*args, **kwargs)
File "/opt/tools/deg/miniforge3/envs/PreMut/lib/python3.10/site-packages/pytorch_lightning/loops/fit_loop.py", line 267, in advance
self._outputs = self.epoch_loop.run(self._data_fetcher)
File "/opt/tools/deg/miniforge3/envs/PreMut/lib/python3.10/site-packages/pytorch_lightning/loops/loop.py", line 199, in run
self.advance(*args, **kwargs)
File "/opt/tools/deg/miniforge3/envs/PreMut/lib/python3.10/site-packages/pytorch_lightning/loops/epoch/training_epoch_loop.py", line 214, in advance
batch_output = self.batch_loop.run(kwargs)
File "/opt/tools/deg/miniforge3/envs/PreMut/lib/python3.10/site-packages/pytorch_lightning/loops/loop.py", line 199, in run
self.advance(*args, **kwargs)
File "/opt/tools/deg/miniforge3/envs/PreMut/lib/python3.10/site-packages/pytorch_lightning/loops/batch/training_batch_loop.py", line 88, in advance
outputs = self.optimizer_loop.run(optimizers, kwargs)
File "/opt/tools/deg/miniforge3/envs/PreMut/lib/python3.10/site-packages/pytorch_lightning/loops/loop.py", line 199, in run
self.advance(*args, **kwargs)
File "/opt/tools/deg/miniforge3/envs/PreMut/lib/python3.10/site-packages/pytorch_lightning/loops/optimization/optimizer_loop.py", line 200, in advance
result = self._run_optimization(kwargs, self._optimizers[self.optim_progress.optimizer_position])
File "/opt/tools/deg/miniforge3/envs/PreMut/lib/python3.10/site-packages/pytorch_lightning/loops/optimization/optimizer_loop.py", line 247, in _run_optimization
self._optimizer_step(optimizer, opt_idx, kwargs.get("batch_idx", 0), closure)
File "/opt/tools/deg/miniforge3/envs/PreMut/lib/python3.10/site-packages/pytorch_lightning/loops/optimization/optimizer_loop.py", line 357, in _optimizer_step
self.trainer._call_lightning_module_hook(
File "/opt/tools/deg/miniforge3/envs/PreMut/lib/python3.10/site-packages/pytorch_lightning/trainer/trainer.py", line 1305, in _call_lightning_module_hook
output = fn(*args, **kwargs)
File "/opt/tools/deg/miniforge3/envs/PreMut/lib/python3.10/site-packages/pytorch_lightning/core/module.py", line 1661, in optimizer_step
optimizer.step(closure=optimizer_closure)
File "/opt/tools/deg/miniforge3/envs/PreMut/lib/python3.10/site-packages/pytorch_lightning/core/optimizer.py", line 169, in step
step_output = self._strategy.optimizer_step(self._optimizer, self._optimizer_idx, closure, **kwargs)
File "/opt/tools/deg/miniforge3/envs/PreMut/lib/python3.10/site-packages/pytorch_lightning/strategies/strategy.py", line 234, in optimizer_step
return self.precision_plugin.optimizer_step(
File "/opt/tools/deg/miniforge3/envs/PreMut/lib/python3.10/site-packages/pytorch_lightning/plugins/precision/precision_plugin.py", line 121, in optimizer_step
return optimizer.step(closure=closure, **kwargs)
File "/opt/tools/deg/miniforge3/envs/PreMut/lib/python3.10/site-packages/torch/optim/optimizer.py", line 113, in wrapper
return func(*args, **kwargs)
File "/opt/tools/deg/miniforge3/envs/PreMut/lib/python3.10/site-packages/torch/autograd/grad_mode.py", line 27, in decorate_context
return func(*args, **kwargs)
File "/opt/tools/deg/miniforge3/envs/PreMut/lib/python3.10/site-packages/torch/optim/adam.py", line 118, in step
loss = closure()
File "/opt/tools/deg/miniforge3/envs/PreMut/lib/python3.10/site-packages/pytorch_lightning/plugins/precision/precision_plugin.py", line 107, in _wrap_closure
closure_result = closure()
File "/opt/tools/deg/miniforge3/envs/PreMut/lib/python3.10/site-packages/pytorch_lightning/loops/optimization/optimizer_loop.py", line 147, in call
self._result = self.closure(*args, **kwargs)
File "/opt/tools/deg/miniforge3/envs/PreMut/lib/python3.10/site-packages/pytorch_lightning/loops/optimization/optimizer_loop.py", line 133, in closure
step_output = self._step_fn()
File "/opt/tools/deg/miniforge3/envs/PreMut/lib/python3.10/site-packages/pytorch_lightning/loops/optimization/optimizer_loop.py", line 406, in _training_step
training_step_output = self.trainer._call_strategy_hook("training_step", *kwargs.values())
File "/opt/tools/deg/miniforge3/envs/PreMut/lib/python3.10/site-packages/pytorch_lightning/trainer/trainer.py", line 1443, in call_strategy_hook
output = fn(*args, **kwargs)
File "/opt/tools/deg/miniforge3/envs/PreMut/lib/python3.10/site-packages/pytorch_lightning/strategies/strategy.py", line 378, in training_step
return self.model.training_step(*args, **kwargs)
File "/opt/tools/deg/PreMut/src/LitModule.py", line 259, in training_step
return self.step(batch,batch_idx,state='train')
File "/opt/tools/deg/PreMut/src/LitModule.py", line 199, in step
graph, label, info = self.prepare_graph(batch=batch,state=state)
File "/opt/tools/deg/PreMut/src/LitModule.py", line 134, in prepare_graph
graph, label_coords = pdb_reader.pdb_to_graph()
File "/opt/tools/deg/PreMut/src/PDB_all_atom.py", line 529, in pdb_to_graph
node_coords, label_coords = self.get_xyz()
File "/opt/tools/deg/PreMut/src/PDB_all_atom.py", line 257, in get_xyz
wild_df, mutant_df = self.do_TMalign(wild_ppdb=wild_ppdb,mutant_ppdb=mutant_ppdb)
File "/opt/tools/deg/PreMut/src/PDB_all_atom.py", line 242, in do_TMalign
pdb = PandasPdb().read_pdb('TMP/{0}{1}.sup_all_atm.pdb'.format(self.wild_pdb,self.mutant_pdb))
File "/opt/tools/deg/miniforge3/envs/PreMut/lib/python3.10/site-packages/biopandas/pdb/pandas_pdb.py", line 94, in read_pdb
self.pdb_path, self.pdb_text = self._read_pdb(path=path)
File "/opt/tools/deg/miniforge3/envs/PreMut/lib/python3.10/site-packages/biopandas/pdb/pandas_pdb.py", line 316, in _read_pdb
with openf(path, r_mode) as f:
FileNotFoundError: [Errno 2] No such file or directory: 'TMP/1ire_7w8m.sup_all_atm.pdb'

Hello,
Could you check if you have installed TMalign? It is required for the training portion.


Support for windows OS

Would it be possible to add support for Windows OS as well?
I don't really have the coding skills necessary but it looks like the vm is created using the "Other install media" Xenserver template and I don't think windows can boot using this template.
From the file create_vm.rb:
oim = env[:xc].call("VM.get_by_name_label",env[:session],"Other install media")['Value'][0]
Perhaps it would be possible to add a provisioner variable like template_name so that another template may be used instead?
Best regards,
Christopher

Hello Christopher! I'm just a user of this application, and I am working with Windows OS guests that I built from the virtual machines provided by modern.ie But I'm pretty sure that I'm not experiencing any issues related to the using Other Install Media. I can't say for sure yet, since I'm experiencing other issues that seem to unrelated, but I can create a patch for this and submit it as a PR if this is something that would be useful.

Hi,
Many thanks for your response! :)
What I have seen so far is that the vm will not boot at all when I use the "Other install media" but if I detach the disk and attach it to a vm that was create using a Xenserver windows template it works fine.
My images is a Windows server 2012 R2 image built with Packer.
The error occurs just as I turn on the vm so it might perhaps be a boot loader thing or something like that but I realize that I owe it to you to do some proper research here since it seems to be working for you when you use the modern.ie machines.
I will do some more research over the weekend and get back to you.
Would it be possible for you to share your vagrantfile for a working windows box with me?
Meanwhile, many thanks for opening up the dialog. :)

Hi,
I have done some more research now and it turns out that I did too much copy paste and
I missed to change the pv property to false so it all works fine now. :)
It may be a good idea to provide the possibility to set the template anyway but it's not a high priority task.
Many thanks for your help and feedback! :)


Clustering a large number of points

I'm having a problem clustering a large number of points (my current set is 40K+ points, but I've even gone down to about 15K points). It takes very long to cluster, and then when zooming in/out it takes similar time for the layer to render. Most of the time the browser (in my case Chrome) asks to kill the tab.
I should add that to overcome this, I had decided to show the data as a heat map until a zoom level is reached, after which I thought I could show the cluster with the flag removeOutsideVisibleBounds set to true, but that didn't seem to help either and resulted in the same delay.
Is anyone else experiencing this issue?

@danzel I have the same issue, using the angular directive for Leaflet, just like @vadvv and it takes a lot of time to in zoom/in-out.
Example: http://dev.yachtingsbest.com/
Code: http://dev.yachtingsbest.com/src/app/controllers/spotsCtrl.js
@vadvv did you find any solution to this one ?

Sorry dud3. No luck yet. I ended up violating DI and using the original library since the number of points I was working with in the worst case scenario was in the 50K to 200K range.


(Object branch) missing Jsonable files

As I found out when I tried to work in the office today and found that I'd left all of the files uncommitted on my laptop at home.  ARGH

Fixed now.


all: git precommit hook to check for go-fmt issues

Raised and fixed by @ferhatelmas in https://github.com/census-instrumentation/opencensus-go/pull/417, that was an issue in which we hadn't run go-fmt on a file. Let's perhaps explore the possibility of a git precommit hook to handle checking if all go files are go-fmt'd. The Go project does this.

We can do this at CI, need to fix https://github.com/census-instrumentation/opencensus-go/blob/master/.travis.yml#L19 to do the right thing to check the formatting errors rather than rewriting.

Awesome, thanks for tackling this. I had seen that CI line but with the new PR I see we actually now use the result to enforce go-fmt-ing


Split CI GitHub workflows into 3 files

The ci.yaml file could be split into:

ci-docs.yaml
ci-lint.yaml
ci-test.yaml

Then, ci-test.yaml could ignore paths such that PRs only touching docs/** such as https://github.com/ordinals/ord/pull/3681 would not need to be tested.

I'm fine with that, although, in practice, running the tests takes a lot longer than building the docs or linting, and they happen in parallel, so it doesn't slow down the overall run time.
If we do split them, I'd vote to call them docs.yaml, lint.yaml, and ci.yaml or test.yaml.


Fixed a bug for operator SDIV.

According to Solidity Docs: "In Solidity, division rounds towards zero. This mean that int256(-5) / int256(2) == int256(-2)."
In big.Int, Div implements Euclidean division in which the direction of rounding depends on the sign of denominator.

Example
Solidity/EVM
big.Int.Div()

7/3
2
2

7/(-3)
-2
-2

(-7)/3
-2
-3

(-7)/(-3)
2
3

case SDIV in contract.go needs to be modified if we want to keep the same behavior as Solidity/EVM's.

@dshe-certik thanks so much for this. I'd like to also add a unit test for this at the same time as fixing. If you are willing/able to do this then great and I can give you some pointers (it should be in evm_test.go), otherwise I am happy to do it.
One thing we do need in order to accept contributions is DCO signoff. Please just follow instructions here: https://github.com/hyperledger/burrow/pull/1360/checks?check_run_id=519925126 and I will make sure your original commit is merged as part of this change.

@dshe-certik thanks so much for this. I'd like to also add a unit test for this at the same time as fixing. If you are willing/able to do this then great and I can give you some pointers (it should be in evm_test.go), otherwise I am happy to do it.
One thing we do need in order to accept contributions is DCO signoff. Please just follow instructions here: https://github.com/hyperledger/burrow/pull/1360/checks?check_run_id=519925126 and I will make sure your original commit is merged as part of this change.

Thanks for the suggestion! I will do it.
Actually I just found a similar problem with SMOD and another problem with JUMP and JUMPI. I'll fix them and added unit tests.

Actually I just found a similar problem with SMOD and another problem with JUMP and JUMPI. I'll fix them and added unit tests.

Fabulous! :pray:


Update pt-br.yml - [merged]

In GitLab by @brunocalado on Dec 15, 2023, 15:25
Merges brunocalado-master-patch-81158 -> master
Update pt-br.yml

In GitLab by @Asacolips on Jan 23, 2024, 08:43
mentioned in commit c2e68e650f676a3514ae94d49fddbbb2b6aa7836

In GitLab by @Asacolips on Jan 23, 2024, 08:43
approved this merge request


[IT-2158] Setup SSM explorer and ops center

A pre-requisite[1] for setting up SSM explorer and opscenter is to setup a service role for automation[2]
[1] https://docs.aws.amazon.com/systems-manager/latest/userguide/Explorer-setup-related-services.html
[2] https://docs.aws.amazon.com/systems-manager/latest/userguide/automation-cf.html

This is not needed because this role already exists.  It was created by AWS when using the AWS console to manually setup Explorer and OpsCenter.


Migration to Bevy 0.12

Currently blocked by an issue with the embedded_asset! macro, see https://github.com/bevyengine/bevy/issues/10377

I apologize I didn't realize this would show up in that PR.. I thought issue mentions were typically within repository bounds...

Last thing to note, hot reloading will probably not work with the above fix, not that it is needed but if somehow someone happens upon this and tries to make changes and wonders why when they enable the hotreloading feature for embedded assets and it does not work for the shader, this is why.


EvaluatePrequential Plot does not show up if batch_size != 1

Expected behaviour
EvaluatePrequential plot can be generated independent of the batch size
Actual behaviour
For batch_size != 1, the plot only shows up after the last evaluation and no metrics are plotted
Steps to reproduce the behaviour
%matplotlib notebook
from skmultiflow.data.generators.waveform_generator import WaveformGenerator
from skmultiflow.classification.trees.hoeffding_tree import HoeffdingTree
from skmultiflow.data.file_stream import FileStream
from skmultiflow.evaluation.evaluate_prequential import EvaluatePrequential

stream = WaveformGenerator()
stream.prepare_for_use()
ht = HoeffdingTree()
evaluator = EvaluatePrequential(show_plot=True,
                                pretrain_size=200,
                                max_samples=20000, batch_size=2)
evaluator.evaluate(stream=stream, model=ht)

(Jupyter notebook server 5.5.0, matplotlib 2.2.2)

https://github.com/scikit-multiflow/scikit-multiflow/blob/105a1412be9b7df2cb82f4bfdaa046e38139f8c3/src/skmultiflow/evaluation/evaluate_prequential.py#L254-L259
Using "bitwise or" evaluates to False for e.g. batch_size=2 and self._update_metrics() will not be called.
I changed it to the "logical or" and the plotting works for arbitrary batch sizes.


test PR with Github API

Please pull this in!

Docs Build status updates of commit ac38d50:
:warning: Validation status: warnings

File
Status
Preview URL
Details
Dependent Files

sdp-test/page-model-documents/Tutorial.yml
:warning:Warning

Details

sdp-test/page-model-documents/Tutorial.yml

[Warning] Unable to find uid "challenge-uid".

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Racist People

I have a problem every now and then there are racist people on your server. It's illegal to treat people differently because of their race or culture. I think you need an anti-bypass filter to not let anyone say mean things.
Footage: https://www.youtube.com/watch?v=igUk84Uez2Y

These people think it's funny, and some even legitimately think these things

I added a filter to usernames and to chat messages also added the ability to block people.  Let me know if you have more ideas on how to handle the issue. I am all ears.


Map git repo and owners to Oracle/espresso/dali datasets

In Mulitproduct ETL job:

Map git repo to dali dataset groups or oracle/espresso databases
Set owners for all datasets under such dataset groups or databases

Also update etl_config_tables DDL, add default timestamps.

Good to go!


Automated readability index

As we grow up, we can understand more and more complex texts. But how to assess the level of complexity of a given text, and how to teach the computer to do so? In this exercise, you will write a program determining how difficult a text is and for what age it is best suited.
For a formula for calculating the complexity, we will use Automated readability index (ARI) developed in 1967 year. The ARI assesses the U.S. grade level required to read a piece of text. In some ways, it is similar to other formulas. Its difference is rather than counting syllables, it counts characters. The more characters, the harder the word. It also counts sentences. This sets it apart from some other formulas.

What concepts does this exercise teach?
What value does it bring to awk?
Is there a reason it's not being added to the problem specs?

* Is there a reason it's not being added to the problem specs?

Hi Isaac,
Thank you so much for your quick feedback.
It is my first pull request to AWK track, and I assumed I could omit some details in the PR description. For example, I don't know how to generate the file tests.toml and miss it in my PR. I would appreciate it if you could tell me how to format a Pull request properly and how to generate such a file. Did I understand correctly that your last question was related to this file (tests.toml)?

What concepts does this exercise teach?
What value does it bring to awk?

How should I answer these questions? Could you please give me examples of other problems with the answer to these questions?
My goal was to develop an exciting problem to solve that would be suitable for solving using the AWK language. The task I created may interest the user of the site Exercism. I understand that the task may have some flaws in the current form that I am ready to eliminate with your help.
Best regards,
Jegors

I don't know how to generate the file tests.toml and miss it in my PR.

A quick overview of the exercism landscape:

typically, practice exercises have "canonical data" that is shared between all language tracks
this canonical data forms the basis of the unit tests
look in the exercises directory of the problem-specifcations repo
the tests.toml file maps the test cases in problem-specifications canonical-data to the unit tests.
the configlet tool creates tests.toml
if there is no "upstream" canonical data, then we don't need tests.toml.

The purpose of all the files in a practice exercise are all documented.

What concepts does this exercise teach?
What value does it bring to awk?

How should I answer these questions? Could you please give me examples of other problems with the answer to these questions?
My goal was to develop an exciting problem to solve that would be suitable for solving using the AWK language. The task I created may interest the user of the site Exercism. I understand that the task may have some flaws in the current form that I am ready to eliminate with your help.

It is an interesting exercise. It does really cover 2 distinct concepts:

formatting output
calculating a value and looking it up in a map.

It would be better to focus just on the second concept.
The "contributing guide", HOWTO-port-an-exercise.md, doesn't cover the case of inventing a new exercise.

@IsaacG I would be in favour of adding this exercise here before adding it to problem-specs.

@rabestro I assume, on your local repo, this succeeds:
./bin/validate_one_exercise ./exercises/practice/ari

@rabestro I assume, on your local repo, this succeeds:

➜  awk git:(ari) ✗ ./bin/validate_one_exercise ./exercises/practice/ari
grep: invalid option -- P
usage: grep [-abcdDEFGHhIiJLlMmnOopqRSsUVvwXxZz] [-A num] [-B num] [-C[num]]
	[-e pattern] [-f file] [--binary-files=value] [--color=when]
	[--context[=num]] [--directories=action] [--label] [--line-buffered]
	[--null] [pattern] [file ...]
➜  awk git:(ari) ✗

@rabestro I assume, on your local repo, this succeeds:

Would you like me to install some specific version of grep? I am using MacOS.

I use Homebrew on a Mac. Install it then brew install grep to get GNU grep (with the -P option for perl-compatible regular expressions). brew will let you know when you need to adjust your PATH.

➜  awk git:(ari) ✗ ./bin/validate_one_exercise ./exercises/practice/ari
There should be exactly one commented skip directive in test-ari.bats
➜  awk git:(ari) ✗ ./bin/validate_one_exercise ./exercises/practice/ari
Processing ari
test-ari.bats
 ✓ Exercism is an online, open-source, free coding platform
 ✓ ARI was designed for military use in 1967
 ✓ Billy always listens to his mother

3 tests, 0 failures

Updated 1 path from the index

Does Exercism properly process math formulas?
Score = 4.71 ( { characters \over words }) + 0.5 ({words \over sentences}) - 21.43

How should I answer these questions? Could you please give me examples of other problems with the answer to these questions?
My goal was to develop an exciting problem to solve that would be suitable for solving using the AWK language. The task I created may interest the user of the site Exercism. I understand that the task may have some flaws in the current form that I am ready to eliminate with your help.

Generally a problem should center around some sort of goal or "area" that it teaches/requires, eg parsing inputs, formatting outputs, performing graph traversal, etc. It is helpful to highlight what this exercise "teaches".
Glenn called out that this covers "formatting output" and "calculating a value and looking it up in a map".
It might be worth figuring out if we can tag those in the config.json

@IsaacG I would be in favour of adding this exercise here before adding it to problem-specs.

Sounds good.

In line with all the other exercises, I think this exercise's name should not be abbreviated. ari isn't descriptive and all the other exercise names are.

@rabestro Can you fix the exercise name to not be abbreviated? It might be easier to do the rename separate from going over the language.

Readability
Readability Index
Text Readability
Automated Readability Index
Readability Score

@rabestro Can you fix the exercise name to not be abbreviated? It might be easier to do the rename separate from going over the language.

➜  awk git:(ari) ✗ ./bin/validate_one_exercise ./exercises/practice/automated-readability-index
Processing automated-readability-index
test-automated-readability-index.bats
 ✓ Exercism is an online, open-source, free coding platform
 ✓ ARI was designed for military use in 1967
 ✓ Billy always listens to his mother

3 tests, 0 failures

Updated 1 path from the index
➜  awk git:(ari) ✗

Sorry for lengthy delay in reviewing. I just have a few suggestions for the instructions.

I see. More than 1 million students registered on the site. Of course, I really want to see my problem published as soon as possible. But at the same time, I want there to be no flaws.

Awesome work @rabestro! Thanks for sticking with all those requests for change.

https://exercism.org/tracks/awk/exercises/automated-readability-index

https://exercism.org/tracks/awk/exercises/automated-readability-index

The very first community solution is there.


Problem in running "Dapr Actor Sample"

When I follow the instruction to run dotnet-sdk/samples/Actor on local dev machine at the first time, everything looks fine.  it looks like dapr should be running based on the docker container.
Recently, the code can not work, I try to figure it out. the strange thing is that dapr can work even without docker!! can you let me know how can i check if the dapr works properly or not?

@Shawn-Fan Dapr runtime can run without docker, however for local Actor development you need docker as it runs a placement service(used by Actors) in a docker container. It also runs a redis docker container which is used by default by dapr cli to provide state & pubsub capabilities.
Could you pleases explain specific issues you are running into?

The problem is for the example code under "dotnet-sdk/samples/Actor/"folder. At the first time, it can be run on the local dev machine when docker is running(The command "daps run --port 3500 --app-id demo_actor --app-port 5000 dotnet run " can be excuted successful), Actor service works fine, Actorclient running well. Without running docker, actor service and actor client can not work properly.
But recently, I try the same  code above, same steps as before, everything is fine but actor client can not connect the actor service. I turn off the docker, run the actor service ((The command "daps run --port 3500 --app-id demo_actor --app-port 5000 dotnet run ") can be excuted successful so the question arised.

@Shawn-Fan  for using Actors, you would need placement service. And by placement service is setup to run in a docker container when you do dapr init. Without the placement service, clients will not be able to discover service which hosts an actor type.

@amanbha
Fallowng is the exciting result:
docker ps
CONTAINER ID        IMAGE               COMMAND                  CREATED             STATUS              PORTS                      NAMES
77d38ee4b04b        daprio/dapr         "./placement"            4 hours ago         Up 9 seconds        <IP_ADDRESS>:50005->50005/tcp   dapr_placement
41de22934a5d        redis               "docker-entrypoint.s…"   4 days ago          Up 9 seconds        <IP_ADDRESS>:6379->6379/tcp     friendly_northcutt
MacBook:DemoActor ia$ dapr run --port 3500 --app-id demo_actor --app-port 5000 dotnet run
ℹ  Starting Dapr with id demo_actor. HTTP Port: 3500. gRPC Port: 63497
✅  You're up and running!   Press Ctrl+C to shut down.
After running service client:
MacBook:ActorClient ia$ dotnet run
Making call using actor proxy to save data.
ERROR: DaprHttpInteractor:  System.Net.Http.HttpRequestException: Connection refused
---> System.Net.Sockets.SocketException (61): Connection refused
at System.Net.Http.ConnectHelper.ConnectAsync(String host, Int32 port, CancellationToken cancellationToken)
--- End of inner exception stack trace ---
at System.Net.Http.ConnectHelper.ConnectAsync(String host, Int32 port, CancellationToken cancellationToken)
at System.Net.Http.HttpConnectionPool.ConnectAsync(HttpRequestMessage request, Boolean allowHttp2, CancellationToken cancellationToken)
at System.Net.Http.HttpConnectionPool.CreateHttp11ConnectionAsync(HttpRequestMessage request, CancellationToken cancellationToken)
at System.Net.Http.HttpConnectionPool.GetHttpConnectionAsync(HttpRequestMessage request, CancellationToken cancellationToken)
at System.Net.Http.HttpConnectionPool.SendWithRetryAsync(HttpRequestMessage request, Boolean doRequestAuth, CancellationToken cancellationToken)
at System.Net.Http.RedirectHandler.SendAsync(HttpRequestMessage request, CancellationToken cancellationToken)
at System.Net.Http.HttpClient.FinishSendAsyncBuffered(Task1 sendTask, HttpRequestMessage request, CancellationTokenSource cts, Boolean disposeCts)    at Dapr.Actors.DaprHttpInteractor.SendAsyncHandleSecurityExceptions(Func1 requestFunc, CancellationToken cancellationToken) in /Users/ian/Desktop/dotnet-sdk-master/src/Dapr.Actors/DaprHttpInteractor.cs:line 479
Unhandled exception. System.Net.Http.HttpRequestException: Connection refused
---> System.Net.Sockets.SocketException (61): Connection refused
at System.Net.Http.ConnectHelper.ConnectAsync(String host, Int32 port, CancellationToken cancellationToken)
--- End of inner exception stack trace ---
at System.Net.Http.ConnectHelper.ConnectAsync(String host, Int32 port, CancellationToken cancellationToken)
at System.Net.Http.HttpConnectionPool.ConnectAsync(HttpRequestMessage request, Boolean allowHttp2, CancellationToken cancellationToken)
at System.Net.Http.HttpConnectionPool.CreateHttp11ConnectionAsync(HttpRequestMessage request, CancellationToken cancellationToken)
at System.Net.Http.HttpConnectionPool.GetHttpConnectionAsync(HttpRequestMessage request, CancellationToken cancellationToken)
at System.Net.Http.HttpConnectionPool.SendWithRetryAsync(HttpRequestMessage request, Boolean doRequestAuth, CancellationToken cancellationToken)
at System.Net.Http.RedirectHandler.SendAsync(HttpRequestMessage request, CancellationToken cancellationToken)
at System.Net.Http.HttpClient.FinishSendAsyncBuffered(Task1 sendTask, HttpRequestMessage request, CancellationTokenSource cts, Boolean disposeCts)    at Dapr.Actors.DaprHttpInteractor.SendAsyncHandleSecurityExceptions(Func1 requestFunc, CancellationToken cancellationToken) in /Users/ian/Desktop/dotnet-sdk-master/src/Dapr.Actors/DaprHttpInteractor.cs:line 479
at Dapr.Actors.DaprHttpInteractor.SendAsyncHandleUnsuccessfulResponse(Func1 requestFunc, String relativeUri, String requestId, CancellationToken cancellationToken) in /Users/ian/Desktop/dotnet-sdk-master/src/Dapr.Actors/DaprHttpInteractor.cs:line 415    at Dapr.Actors.DaprHttpInteractor.SendAsync(Func1 requestFunc, String relativeUri, String requestId, CancellationToken cancellationToken) in /Users/ian/Desktop/dotnet-sdk-master/src/Dapr.Actors/DaprHttpInteractor.cs:line 336
at Dapr.Actors.DaprHttpInteractor.InvokeActorMethodWithRemotingAsync(ActorMessageSerializersManager serializersManager, IActorRequestMessage remotingRequestRequestMessage, CancellationToken cancellationToken) in /Users/ian/Desktop/dotnet-sdk-master/src/Dapr.Actors/DaprHttpInteractor.cs:line 164
at Dapr.Actors.Communication.Client.ActorRemotingClient.InvokeAsync(IActorRequestMessage remotingRequestMessage, String methodName, CancellationToken cancellationToken) in /Users/ian/Desktop/dotnet-sdk-master/src/Dapr.Actors/Communication/Client/ActorRemotingClient.cs:line 40
at Dapr.Actors.Client.ActorProxy.InvokeAsync(Int32 interfaceId, Int32 methodId, String methodName, IActorRequestMessageBody requestMsgBodyValue, CancellationToken cancellationToken) in /Users/ian/Desktop/dotnet-sdk-master/src/Dapr.Actors/Client/ActorProxy.cs:line 186
at ActorClient.Program.Main(String[] args) in /Users/ian/Desktop/dotnet-sdk-master/samples/Actor/ActorClient/Program.cs:line 39
at ActorClient.Program.(String[] args)

@Shawn-Fan  Thanks for reporting the issue, I believe the sample doc is slightly incorrect as the app is running on port 5001. I will update it. Could you please try and run the actor service as dapr run --port 3500 --app-id demo_actor --app-port 5001 dotnet run

Thanks! It works.

Why would the application run on 5001 when the app-port was specified as 5000?
Just a question as I was playing with this the other day, and noticed that I only got a response on 5001...
In the documentation I always try and document the why, not the how.  For example,
Because the -app_port number is set to 5000, interaction will occur.
So
dapr run --port 3500 --app-id demo_actor --app-port 5001 dotnet run
Is much better in IMO, rather than just the line which we all cut and past.
As I'm new to Docker, I'm wondering why the  --port is specified, as from the application side, we never do anything with it - why is it there?  My assumption is that it's the port that is used to communicate inside Docker, and the app-port is the port we used to communicate from outside docker.
Regards,
Richard
From: Shawn Fan<EMAIL_ADDRESS>Sent: November 18, 2019 2:42 PM
To: dapr/dotnet-sdk<EMAIL_ADDRESS>Cc: Subscribed<EMAIL_ADDRESS>Subject: Re: [dapr/dotnet-sdk] Problem in running "Dapr Actor Sample" (#159)
Thanks! It works.

You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHubhttps://t.sidekickopen09.com/s1t/c/5/f18dQhb0S7lM8dDMPbW2n0x6l2B9nMJN7t5XWPfhMynW2B86_W5vMG5HW56dBWl7BRWd8102?te=W3R5hFj4cm2zwW3P28X24hCPvhW43T4NT3zfQ0QW3H91fn3K9xWcW3H7fhG4cPVPfW4cgtx924Zxw3W3zbV9S4cNcV-W3F6d9843X5bgW3SYLpP3T1MdZW1pM-H93T0tv-W43SlzP2dTvhGW38fX_G1Xj0FQW2xH7SS1VhNtWW1Z5pK42-tMWRW1S4k012sQ9XXW2RkR-h3d339zW2MH8RQ2YsjVfW3bdRWH36j8xPW2FJ-dn2zP9h7W3d9KBt38h9qlW3b44Yj2TvQpgW2FFf_22vNJYBW3b2vsH2Wv9vCW3bjd6m2Wp_CDW2nSH011hV0BXW4hF6zG3_X4wHW4fdbr31YZSRg4rQsX1&si=8000000002124969&pi=36ca5fd2-db12-454e-9203-401e7912bfca, or unsubscribehttps://t.sidekickopen09.com/s1t/c/5/f18dQhb0S7lM8dDMPbW2n0x6l2B9nMJN7t5XWPfhMynW2B86_W5vMG5HW56dBWl7BRWd8102?te=W3R5hFj4cm2zwW3P28X24hCPvhW43T4P243X5bgW3SYLpP3T1MdZW1LFyV_4hCWypW49M7TJ1Gy80dW3Qzvpc2xQD4lW2Whl-M1ShxNsW2Wsxj136mWWcW2vQ08_2vz2R1W2TqDYK32mtXMw2WyVnP5LP2&si=8000000002124969&pi=36ca5fd2-db12-454e-9203-401e7912bfca.

@codeputer I believe you wanted to say Dapr instead of Docker.
The app-port is the parameter to tell Dapr on what port the application will be listening.
Dapr and app both communicate with each other over http/grpc. --port is the port used by dapr for its http server. --app-port is the port at which app will be listening.

Thanks and your right - I meant Dapr...
So why would the application respond on  5001 when the app-port was specified to be 5000
Did I miss something?
R
From: Aman Bhardwaj<EMAIL_ADDRESS>Sent: November 18, 2019 3:04 PM
To: dapr/dotnet-sdk<EMAIL_ADDRESS>Cc: Richard Reukema<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [dapr/dotnet-sdk] Problem in running "Dapr Actor Sample" (#159)
Closed #159https://t.sidekickopen09.com/s1t/c/5/f18dQhb0S7lM8dDMPbW2n0x6l2B9nMJN7t5XWPfhMynW2B86_W5vMG5HW56dBWl7BRWd8102?te=W3R5hFj4cm2zwW3P28X24hCPvhW43T4NT3zfQ0QW3H91fn3K9xWcW3H7fhG4cPVPfn4cgtx9133&si=8000000002124969&pi=f0930631-ce56-4ae1-807d-251daaa7c8e7.

You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://t.sidekickopen09.com/s1t/c/5/f18dQhb0S7lM8dDMPbW2n0x6l2B9nMJN7t5XWPfhMynW2B86_W5vMG5HW56dBWl7BRWd8102?te=W3R5hFj4cm2zwW3P28X24hCPvhW43T4NT3zfQ0QW3H91fn3K9xWcW3H7fhG4cPVPfW4cgtx924Zxw3W3zbV9S4cNcV-W3F6d9843X5bgW3SYLpP3T1MdZW1pM-H93T0tv-W43SlzP2dTvhGW38fX_G32nlmQW2zJCBG212XgvW2vSWlW2-tMWVW2Twb0J2sQ9XXW2RkR-h3d339zW2MH8RQ2YsjVfW3bdRWH36j8xPW2FJ-dn2zPCsJW3d9KBt38h9qmW3ddgNT2Wv9vsW2Cwm2H2KCDLrW3bcdl836k0_lW3b3KzC2vNJYBW3b2vsH2Wv9vCW3bjd6m2WxDQ3W212T651hSlZmW41YspH22TG_Mn1-YpN6123&si=8000000002124969&pi=f0930631-ce56-4ae1-807d-251daaa7c8e7, or unsubscribehttps://t.sidekickopen09.com/s1t/c/5/f18dQhb0S7lM8dDMPbW2n0x6l2B9nMJN7t5XWPfhMynW2B86_W5vMG5HW56dBWl7BRWd8102?te=W3R5hFj4cm2zwW3P28X24hCPvhW43T4P243X5bgW3SYLpP3T1MdZW1LFyV_4hCWypW49M7TJ1Gy80dW3Qzvpc2xQD4lW2Tz5rb2qSw6FW2TqD8g20W-f1W2vQ08_2CJnFfW2TqDYK32mtXMw2WyVnP5LP2&si=8000000002124969&pi=f0930631-ce56-4ae1-807d-251daaa7c8e7.

@codeputer --app-port is just telling dapr that application will be listening on 5001, its not configuring app to run on 5001. The port logic is for the application to decide. For more info, please see: https://github.com/dapr/cli#launch-dapr-and-your-app


Change to Pydantic Version to v2

change ArangodbClient to python-arango
change python to >py3.10
dump pre-commit version
rename Graph to GrapModel (Name Collision)
dump Shylock Version
add Shylock AsyncDB Class
WIP EdgeModel Funcs like
UPSERT, get_many
change tests to python-arango
fixme
ENV Variable in pre-commit tests did not run

I would be happy to receive comments.
This is my first project with Asyncer, so I'm not sure if it will make an improvement.
But aioarangodb is not maintained for a long time and python-arango regularly. Hence my decision to do it this way.
Looking forward to the exchange of thoughts!


PDO_OCI for PHP8

Describe the feature
Could not install pdo_oci on PHP 8.0.10
Version

[x] I have checked releases, and the feature is missing in the latest patch version of v2.

https://github.com/yiisoft/db/pull/238/checks?check_run_id=3641368345

@samdark
I could not reproduce the issue.
Test workflow: https://github.com/shivammathur/test-setup-php/actions/runs/1249207383/workflow
Extensions pdo_oci and oci8 should install correctly for all PHP versions. Might be a temporary issue.
Can you switch to shivammathur/setup-php@verbose, and if it fails again, provide me the logs.

@samdark
I could not reproduce the issue.
Test workflow: https://github.com/shivammathur/test-setup-php/actions/runs/1249207383/workflow
Extensions pdo_oci and oci8 should install correctly for all PHP versions. Might be a temporary issue.
Can you switch to shivammathur/setup-php@verbose, and if it fails again, provide me the logs.

When I put the verbose version it works perfectly, when I put the normal version it doesn't work, here you can see it.
Verbose work:
https://github.com/terabytesoftw/db/runs/3641761681?check_suite_focus=true
No verbose:
https://github.com/terabytesoftw/db/runs/3641793839?check_suite_focus=true

@samdark @terabytesoftw Recent release broke cache support. Fixing it now.

Fixed in 48aac87d46a6be92606792479960dc0bbb325532
Please test and let me know.

Worked. Thank you very much for fixing it so fast 👍
https://github.com/yiisoft/db/runs/3644169663


Support JAX-RS Async API

Related to https://discuss.elastic.co/t/jaxrs-async-distrubuted-tracing-not-works/189531.
No trace context propagation when using the async JAX-RS API, as follows:
client.target(url).request().async().get();

We currently don't have dedicated support to JAX-RS client through the API, as decided and documented in #200 .
Instead, we decided to rely on our HttpUrlConnection instrumentation, which works well for sync requests and async requests that make use of the Java Executors. The older (1.x) com.sun Jersey implementation seems to rely on them, but not the newer (2.x) org.glassfish implementation. For the latter, it should be easy to add support by applying the Executor instrumentation to org.glassfish.jersey.client.ClientExecutor.
Other than Jersey, we may include RESTEasy as well in this issue.

Hello @eyalkoren , I will take this one.

@kananindzya awesome! ❤ Thanks!

org.glassfish.jersey.client.ClientExecutor uses ExecutorServices under the hood, so I'm not sure what's missing here.
Without reproducing the problem mentioned in https://discuss.elastic.co/t/jaxrs-async-distrubuted-tracing-not-works/189531 and understanding the root cause, I think it's a bit premature to attempt a fix.

Lowering priority as we can't reproduce that.
Asked the user again to try and find concrete details. If we get them, we can revisit.

closing, can't reproduce.