Unable to pass parameters to setupBubble() method from onStartCommand()

Hello Dear,
I was using version 0.5.0 and everything was working fine, but I wanted to upgrade to the latest version (v0.5.1) to have the new updates, unfortunlly, my class which implements the FloatingBubbleService() has an override for onStartCommand() method, once I upgraded the dependency, this overridden method stopped working, it is not called at all.
NOTE: I have made sure that I am registering the service in the <application> tag in the Manifest file and starting the service with an intent.
Also I would like to mention that I am using this method to initialize a lateinit variable that holds a value passed from the intent that starts the service to be used inside setupBubble() method, so if you have a better way to do that please suggest.
I would be glad if you could help me with this issue 🙏🏻
Thanks in Advance 🙏🏻❤

Hi @moazelsawaf,
In v0.5.1, I removed showBubbles() from onStartCommand() (which I should have done a long time ago) and moved it to the onCreate() method. You can easily choose the initial view by overriding initialRoute()
The reason for this is that if you pass the intent to the service multiple times throughout the service lifecycle, you must override the onStartCommand() method first. Otherwise, the bubble will appear overlapped each time you send the intent.
Therefore, in this version, onStartCommand() is not being touched, and it should behave normally. The reason why the overridden method is not working may be something else.
If you are still experiencing the issue, please provide more information so that we can solve it step by step?
Thank you!

Looks like a bug!.
After a short investigation, I found out that it's hard to pass bubble properties via onStartCommand() to  setupBubble(). This is a silly bug that can be fixed in a blink of an eye. I'll fix it in the next version.
Thank you for pointing it out 💖.

Great ❤
Thank you so much @TorryDo for caring 🙏🏻❤
Waiting for the next release Insha'Allah 💪🏻❤

The issue is now gone in v0.5.2
If you have any further concerns, don't hesitate to reopen the issue.
Cheers 🍺🍻

Unfortunately, the issue still exists, setupBubble() is being called before onStartCommand() so I am still unable to initialize the variables.
📃 Here are the logs of printing a statement in the first line of each method:
2023-04-17 04:41:49.114 12555-12555 BubbleService           dev.moaz.dash_bubble_example         D  setupBubble
2023-04-17 04:41:49.135 12555-12555 BubbleService           dev.moaz.dash_bubble_example         D  onStartCommand

That's weird🤔. Did you override initialRoute() and return Route.Empty?

No, I am not overriding initialRoute() ...
I tried now and overridden it to return Route.Empty, but setupBubble() method stopped being called at all ... 😔
📃 Here are the new logs:
2023-04-17 05:04:33.646 18188-18188 BubbleService           dev.moaz.dash_bubble_example         D  initialRoute
2023-04-17 05:04:33.648 18188-18188 BubbleService           dev.moaz.dash_bubble_example         D  setupNotificationBuilder
2023-04-17 05:04:33.652 18188-18188 BubbleService           dev.moaz.dash_bubble_example         D  onStartCommand

I'm confused now 😅. Please take a quick look at Sample (line 41 -> 60)
Is your code somthing similar to this?

    override fun initialRoute(): Route {
        return Route.Empty
    }

    private var size = 0

    override fun onStartCommand(intent: Intent?, flags: Int, startId: Int): Int {

        val _size = intent?.getIntExtra("size", 0)

        size = _size ?: 0

        showBubbles()

        return START_STICKY
    }

Hmm, sorry I was not calling showBubbles() method, now after calling it the issue should be solved for setupBubble() because I have the control to decide when to call it, but the problem persists for setupNotificationBuilder() because it is still being called before onStartCommand() method.
📃 Here are the new logs:
2023-04-17 05:15:29.895 21779-21779 BubbleService           dev.moaz.dash_bubble_example         D  initialRoute
2023-04-17 05:15:29.897 21779-21779 BubbleService           dev.moaz.dash_bubble_example         D  setupNotificationBuilder
2023-04-17 05:15:29.900 21779-21779 BubbleService           dev.moaz.dash_bubble_example         D  onStartCommand
2023-04-17 05:15:29.900 21779-21779 BubbleService           dev.moaz.dash_bubble_example         D  setupBubble

So you want to pass parameters to notification too?
You can do that by using this code, which will update the notification. the setupNotificationBuilder() is basically just the method to init the notification.
        val notification = // your notification
        NotificationManagerCompat.from(this).notify(notificationId(), notification)

Do you mean I should not override setupNotificationBuilder() method for now and use your idea of updating the notification in onStartCommand() after initializing my variables?

Nope, you can simply using the code above in onStartCommand(). It'll update the current notification.

That is what I meant 👌🏻
It worked, but unfortunately, I have to request a new unnecessary permission from the user which is POST_NOTIFICATIONS, that would complicate the process.

Maybe I have to revamp setupNotificationBuilder() soon.
Thank you!

hi @moazelsawaf , 👋
You can pass parameters to the notification now. Simply by following these steps:

update version to v0.5.3
override "initialNotification" and return null
pass the notification to notify(Notification).

In Android 13+, the notifications are only visible if "POST_NOTIFICATIONS" permission is granted. However, the service will still run fine without the notification.
Thank you!

Thanks for your concerns ❤
Should I handle the notification my self or it is already handled inside the library?

If you want the notification visible, fist include "POST_NOTIFICATIONS" in your manifest file. Then ask for permission.
If permission is not granted, the notification won't visible, 'initialNotification' and 'notify' will still able to call but without notification.

Thank you so much ❤
Now everything is clear, I will add the permission request for Android 13, but I wanted to ask a question, in the old logic in the earlier versions, would it be also a must to request the POST_NOTIFICATIONS permission in Android 13?

No, it's not required.
Foreground notification working fine in ealier versions.
You can find more info here: Android Service.
Thank you.

Thank you so much for the clarification 🙏🏻❤


FactoryGuy.make() doesn't handle ID relationships, Ember Data does

If I do this (with parent.children being a hasMany relationship to child):
FactoryGuy.make('child', {id: 1});
FactoryGuy.make('parent', children: [1]);

JSONAPIFixtureConverter.extractRelationships() returns undefined for these relationships.
However, Ember Data itself will look up these IDs and replace them with the child records, if FactoryGuy wasn't replacing the IDs with undefined. Based on the Ember Data behavior - and this worked for our project with Factory Guy 0.9.1 - it seems simple enough to fix.
I'll prepare a pull request, let me know if you have further insight into this.

Thanks for PR, and this is good point, since ED will return that record.  The only thing that I would like is a test or two ( to show failure without your change ) to make sure it stays working in the future.   If you can slap those tests in the PR I'd be much obliged.

A test or two has revealed this is slightly more complicated than I initially thought. Will re-do the pull request by next week (with appropriate tests). Happy holidays!

I added several tests, only handle strings and numbers as IDs, and assert the relationship isn't polymorphic.


⚠ Bofrost.lu has degraded performance

In 17dc747, Bofrost.lu (https://www.bofrost.lu) experienced degraded performance:

HTTP code: 200
Response time: 1619 ms

Resolved: Bofrost.lu performance has improved in 97d37f2 after 22 minutes.


Cant deploy Identity and Authentication with Facebook Login Mechanism enabled.

Hi i am attempting to deploy with facebook activated as a login mechanism however i get the below error on the AWS cloud formation stack.
Resource handler returned message: "'MemorySize' value failed to satisfy constraint: Member must have value less than or equal to 3008 (Service: Lambda, Status Code: 400, Request ID: ebcbe62b-f833-4cd5-af54-4f691fe777fe)" (RequestToken: f9c9cdf2-214f-e173-161b-4de53965f051, HandlerErrorCode: InvalidRequest)
It deploys fine without the facebook login mechanism enabled.
I am using Unreal Engine 4.27.2.

Hello amar0k,
Thank you for pointing out this issue. To help us investigate would you be able to share the following:

IDE and Version [e.g. Visual Studio 2019 16.11.7]
OS and Version that you are developing on: [e.g. Windows 10]
Region you are trying to deploy in.

@AMZN-johnlaug Hi John,
Thanks for the reply this was resolved by AWS increasing the limit to 10 GB for Lambda on my account.
Thanks.

Hi amar0k,
That's excellent news.
I have found the problem in our code and we will work on a fix in a future release.
Thank you for providing the references, that'll be helpful for anyone else running into this.

In the meantime, if you want to fix this yourself (without relying on the limit increase)
You can remove the MemorySize line here:
https://github.com/aws/aws-gamekit-unreal/blob/main/AwsGameKit/Resources/cloudResources/cloudformation/identity/cloudFormation.yml#L312
This lambda shouldn't need that much memory and should work fine with the default.

In the meantime, if you want to fix this yourself (without relying on the limit increase) You can remove the MemorySize line here: https://github.com/aws/aws-gamekit-unreal/blob/main/AwsGameKit/Resources/cloudResources/cloudformation/identity/cloudFormation.yml#L312
This lambda shouldn't need that much memory and should work fine with the default.

Thanks @AMZN-johnlaug ! Appreciate the help.

cant deploy fb login Login Mechanism getting these errors and then it starts to rollback pls help (thanks in advance)
~ FbCallbackHandlerApiResource | CREATE_FAILED: Resource creation cancelled
~ gamekit-dev-gkone-identity: GetUserLambdaHandler | UPDATE_COMPLETE:
~ gamekit-dev-gkone-identity: PollFacebookLoginCompletionLambda | DELETE_COMPLETE:
~ gamekit-dev-gkone-identity: FbLoginUrlHandlerApiResource | DELETE_COMPLETE

Logical ID          :   FacebookUserPoolIdentityProvider
Status                :   CREATE_FAILED
Status Reason   :     v6.0 is not a valid value for api_version (Service: AWSCognitoIdentityProviderService; Status Code: 400; Error
Code: InvalidParameterException; Request ID: be5cd116-9240-450b-82d4-c02c960c39e3; Proxy: null)
Unreal error : Error : FeatureResourceManager::CreateOrUpdateResources() for Identity And Authentication feature: Could not
create resources.
Thank you @buchandl  For the support

Yes , Changing v6.0 to v12.0 solved the problem.
I did not find the directory and don't know how to update it (instance template)?
[your_unreal_project]/[game_alias]/[aws_region]/cloudformation/identity/cloudformation.yml
did you mean?
[Unreal_project][Unreal_project]\dev\cloudformation\identity\cloudformation.yml
in my case
gkone/gkone/dev/cloudformation/identity/cloudformation.yml
I was just testing and was not afraid of losing any data
so I deleted all resources from Unreal and  from AWS console also.
and Deployed it again from Unreal. that worked with same project
Thank you @buchandl

Hi,
We've released a new version of the plugin that fixes both MemorySize and api_version issues. You can find the plugin here.


docker-compose: adjust the path of the Python directory

It will actually be easier to just land https://github.com/ansible/ansible-ai-connect-service/pull/1138 which set the final location.


Updating .lib pointers and tracing fix

@hasnainvirk  You're missing the link to the newlib-nano article.
Also it's called newlib-nano, not new-lib-nano.
Also you're mixing tabs and spaces in README, please fix that. Now the indentation looks weird on GH where it uses 8 spaces per tab.


Release package

Package should be released on PyPi

PyPi package has been successfully released v0.0.1, whooooo!!! https://pypi.org/project/napari-console/
So easy thanks to the great instructions on the napari cookiecutter, cc @tlambert03 :-)

PyPi package has been successfully released v0.0.1, whooooo!!! https://pypi.org/project/napari-console/
So easy thanks to the great instructions on the napari cookiecutter, cc @tlambert03 :-)


Modal Added and changes under the dev flag

Date:

Developer Name:

Issue Ticket Number

Description

Documentation Updated?

[ ] Yes
[ ] No

Under Feature Flag

[ ] Yes
[ ] No

Database Changes

[ ] Yes
[ ] No

Breaking Changes

[ ] Yes
[ ] No

Development Tested?

[ ] Yes
[ ] No

Screenshots

Screenshot 1

Test Coverage

Screenshot 1

Additional Notes

closing this PR as this feature has be moved to a different PR


r_data is undefined after a simultaneous write to an empty AsyncFIFO

 Issue by whitequark
Friday Sep 20, 2019 at 20:02 GMT
Originally opened as https://github.com/m-labs/nmigen/issues/217

The behavior of a dual port block RAM with read and write ports in different clock domains is not defined during a simultaneous read and write to the same address. (In fact it is not possible to make it defined.) However, this is exactly what is happening when an element is written into an empty AsyncFIFO[Buffered], since the produce and consume pointers are equal. If the phase of read and write clocks is just wrong, the output register of the BRAM could end in a metastable state.
It is not clear how this can be fixed. A workaround is to use AsyncFIFOBuffered instead, which resynchronizes the BRAM output.

To fix this problem, we can add a small amount of logic that prevents this kind of condition from arising. We can convince ourselves that it's sound using formal verification.


Why does ajax-adapter have static setHttpClientFactory function?

The adapter definition has this static method (setHttpClientFactor) for defining the method that will determine the runtime instance of the httpClient used by the adapter.
I'm personally leery of statics, especially when there is no default implementation. I don't see the need in this case. IMO, it should be an adapter instance method such as;
setHttpClient(client) {
    this.client = client;
}

Configuration now looks like this:
var adapter = breeze.config.initializeAdapterInstance('ajax', 'aurelia', true);
adapter.setHttpClient(aurelia.container.get(HttpClient));

That's the way it's done with the Angular ajax adapter. It seems clearer to me and slightly easier to test. And it's one fewer method to export.
Thoughts?

I like that better too.  So there's no need for me to worry about a second/third/... instance of the adapter coming into play right?  I wasn't sure about that.

Breeze treats it as a service in general. You can create your own instances for special purposes but you'd go out of your way to do that.


Form explainer test

Here's some form explainer code that I want to test on the dev server.

:+1: Looks good.

Exciting, @himedlooff !
Does this mean that there is a prototype live on build that we can play around with?  :rocket:

@mthibos, once this is merged it should fire off a chain of events that will put this on build. If not I'll have to do some asking around.

@himedlooff Couldn't find anything, what's the URL?

@ascott1, @contolini, not seeing this on build. Do I need to add something to the Gruntfile to get it to show up there?

Hmm, I thought build ran the default grunt tasks. @fna would know more.

Got some emails about failing browser tests, that might be the issue.

Ahh, it probably is.

@OrlandoSoto, is there anything that a new page should watch out for regarding browser tests? Not sure why adding a new page would cause them to fail.

@himedlooff hmmm browser tests should not fail when a new page is added,, are you testing from Jenkins? forward me the test logs so I can see why it's failing

@OrlandoSoto, emailed you the failing test notifications I received.

on further investigation, it looks like our Sauce Tunnel in Jenkins stopped running for a few hours on Saturday which caused the Jenkins jobs to fail.
This is a known issue where the Sauce Labs tunnel becomes stale, we have a daily job in Jenkins that refreshes the tunnel everyday to work around this problem.
Subsequent browser tests executed successfully in Jenkins on Sunday


api.qbit.ninja is down

500 (Internal Server Error)
[FileNotFoundException: Could not load file or assembly 'System.Net.Http, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=b03f5f7f11d50a3a' or one of its dependencies. The system cannot find the file specified.]
   System.Web.Http.GlobalConfiguration..cctor() +0

[TypeInitializationException: The type initializer for 'System.Web.Http.GlobalConfiguration' threw an exception.]
   System.Web.Http.GlobalConfiguration.get_Configuration() +14
   System.Web.Http.GlobalConfiguration.Configure(Action`1 configurationCallback) +15
   QBitNinja.WebApiApplication.Application_Start() in C:\Users\NicolasDorier\Desktop\Mainnet-QBitNinja\QBitNinja\QBitNinja\Global.asax.cs:14

[HttpException (0x80004005): The type initializer for 'System.Web.Http.GlobalConfiguration' threw an exception.]
   System.Web.HttpApplicationFactory.EnsureAppStartCalledForIntegratedMode(HttpContext context, HttpApplication app) +475
   System.Web.HttpApplication.RegisterEventSubscriptionsWithIIS(IntPtr appContext, HttpContext context, MethodInfo[] handlers) +118
   System.Web.HttpApplication.InitSpecial(HttpApplicationState state, MethodInfo[] handlers, IntPtr appContext, HttpContext context) +176
   System.Web.HttpApplicationFactory.GetSpecialApplicationInstance(IntPtr appContext, HttpContext context) +347
   System.Web.Hosting.PipelineRuntime.InitializeApplication(IntPtr appContext) +297

[HttpException (0x80004005): The type initializer for 'System.Web.Http.GlobalConfiguration' threw an exception.]
   System.Web.HttpRuntime.FirstRequestInit(HttpContext context) +652
   System.Web.HttpRuntime.EnsureFirstRequestInit(HttpContext context) +89
   System.Web.HttpRuntime.ProcessRequestNotificationPrivate(IIS7WorkerRequest wr, HttpContext context) +188

Yep I know :/
I bandaged it for now.

the server is very slow :/

Looking to it... indeed very slow. Trying to find out what happens.

I scaled up the machine. Unsure if it is the root cause though. Let me know how it goes.

the server is very slow again :/

just restarted the web app now. I need to find out the underlying issue.
That said, please all host your own qbit ninja, I documented better, my server are not meant for production use.

Okay, thank you, I'll start setting up my server tomorrow.

it is not

https://api.qbit.ninja/ is very very slow :/, I'm setting up a qbit, it's in sync. thnks a lot.

restarted it, might be a bit better. I will eventually find out what the slowness, it seems quite recent.

I'm still waiting to sync my qbit, did you find out the reason for the slowness?

I think there is a memory leak. I have not yet identified the root cause.
My server is processing around 55 request per second, which is quite significant. I probably did not had the problem in the past because people were using it way less.

I understand, the qbit is very good, congratulations. tries to restart the application

transactions are not appearing in the blockchain https://api.qbit.ninja/transactions/7da18136a14598a4b6ca459e5f2a5927f642cfc92296d0bf2362c2ff8ec0cc47

Yep, the indexer froze, restarted it.


Doesn't do anything.

In a relative layout. Text size stays the same.
    <me.grantland.widget.AutofitTextView
        android:id="@+id/tvMessage"
        android:layout_width="match_parent"
        android:layout_height="wrap_content"
        android:layout_above="@+id/lButtons"
        android:layout_below="@+id/barcode_scanner"
        android:layout_margin="10dp"
        android:gravity="center"
        android:textSize="24sp"
        ads:minTextSize="10sp"
        />

Looks like you need to set the maxLines to get it to work. Now I see that using a margin the text does not fit properly.

Yeah, that's required. I'll be closing this since it's resolved.


Can not connect to docker daemon after docker upgrade

This issue was mentioned in this post
Looks like the issue will be gone if the docker.service is edited. However, as my last reply to the above post, in my case, I have a CentOS 7 and the config needs to be ExecStart=/usr/bin/dockerd -H unix:///var/run/docker.sock -D -H tcp://<IP_ADDRESS>:2375  rather than what the document says ExecStart=/usr/bin/dockerd -H fd:// -D -H tcp://<IP_ADDRESS>:2375. Unfortunately this setting would not work with Swarm node, saying time out to start swarm container.

Hi
This is quite an old issue and in the meantime we have updated our documentation. Therefore I'm closing this issue. Feel free to re-open it if you need more help.


Added clarifying information to README

Stuff that as a new user would have been helpful to have.
Signed-off-by: Simeon H.K. fitch<EMAIL_ADDRESS>

looks good i'm going to merge. thanks @metasim!

833a4d59c0791090cf8a4abddee359fc59bc491f


Make rand optional

After #270, making rand optional means that it's possible to use a very stripped-down version of the crate:
$ cargo tree -e normal --no-default-features
statrs v0.17.1
├── approx v0.5.1
│   └── num-traits v0.2.19
└── num-traits v0.2.19

Like the nalgebra feature, rand is enabled by default. This added modularity might make #165 easier, too.
This requires a breaking change: Removing the rand::distribution::Distribution<T> supertrait from statrs::distribution::Distribution<T> and ::DiscreteDistribution<T>. Semantically, it means that the statrs distributions are not required to be rand distributions and are not logical extensions of them. Practically, it just means that new distributions don't necessarily have to implement the rand Distribution trait (all current distributions still do implement it with the feature enabled).
I would argue that the distribution traits still make sense without rand/RNG. This also allows using most of the statistics module without needing to compile rand and all that entails.

Thanks again for what you've put in with the last 24 hours. It looks good, just gotta handle the merge.

This should be good to go. I resolved the merge conflicts manually, and the CI failure is just codecov

Needed to rebase again after the other PRs were merged.

coverage CI looks stuck. I'm messing around with it on poorly named pr #279

Now, I will merge the correct thing. Whoops.


Sponsors' Profiles

Profiles like Participant Profile - not just a link to the website

Sounds fine to me. In this case, can we create a Sponsor's Profile - much like the Exhibitor's Profile?
So that by clicking on their logotype the user is not directed to the sponsor's website but to the Profile on our website?

Sounds fine to me. In this case, can we create a Sponsor's Profile - much like the Exhibitor's Profile?
So that by clicking on their logotype the user is not directed to the sponsor's website but to the Profile on our website?

@jprzybylski4 yes....  there is a ticket called "fake company reg" with cost 1... which allows to register via https://admin.ecommercegermanyawards.com/admin/booking and assign a new company.... if company exists (for instance in EBE context - please let me know - we will re-use it)


fix fix bug of limit without order by

Proposed changes
Issue Number: close #xxx
Problem summary
Limit without order by  did not skip rows of offset.
Checklist(Required)

[ ] Does it affect the original behavior
[ ] Has unit tests been added
[ ] Has document been added or modified
[ ] Does it need to update dependencies
[ ] Is this PR support rollback (If NO, please explain WHY)

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

it seems useless to modify it because the result is not stable without order by clause.


How do I implement the client?

Hey there,
You have an example app for pheonix, and a lot of code for elm, but I don't understand how to implement the elm code client side.
Maybe I am not familiar enough with js but if you are importing js aren't you using node or something else? What is my index.html supposed to look like to get the elm app working?
Thanks.
Sean

Here is the Elm code for my client side:
module Slock exposing (..)
import Browser
import Html exposing (Html)
import Html exposing (h1, text)
import Json.Decode exposing (..)
import Phoenix
import Phoenix.Socket as Socket
import Phoenix.Channel as Channel
import Phoenix.Config
import PhoenixPorts

main : Program () Model Msg
main =
    Browser.element
        { init = init
        , view = view
        , update = update
        , subscriptions = subscriptions
        }

type Msg
  = NewMsg Value
  | PhoenixMsg (Phoenix.Msg Msg)

type alias Model =
  { myModelField : String
  , phoenixModel : Phoenix.Model Msg ()
  }

phoenixConfig =
  Phoenix.Config.new PhoenixMsg PhoenixPorts.ports

-- N.B. Do not suffix the endpoint url with `/websocket` as this is automatically added by `phoenix.js`.
endpoint =
    "ws://localhost:4000/socket"

socket =
    Socket.init endpoint

-- channelModel here can be used to dynamically connect to channels.  See advanced usage.
channels channelModel =
  [ Channel.init "lobbies:lobbies"
        -- register an handler for messages with a "new_msg" event
        |> Channel.on "new_msg" NewMsg
  ]

init : () -> ( Model, Cmd Msg )
init flags =
    ( { myModelField = "Hello"
      , phoenixModel = Phoenix.new
      }, Cmd.none )

update : Msg -> Model -> ( Model, Cmd Msg )
update msg model =
    case msg of
        PhoenixMsg phoenixMsg ->
            let
                ( phoenixModel, cmd ) =
                    Phoenix.update phoenixConfig socket channels () phoenixMsg model.phoenixModel
            in
            ( { model | phoenixModel = phoenixModel }, cmd )
        NewMsg value -> ( model, Cmd.none)

subscriptions model =
    Phoenix.connect phoenixConfig

view : Model -> Html Msg
view model =
    h1 [] [text model.myModelField]```

Nvm I see your examples in the Assets folder now

@Sean-Green I hope you managed to get it working.  If you need any more help or you think the docs could be clearer please let me know.


Connection string

The connection to include the server name, and other parameters needed to establish the initial connection.

Does the connector support Single Sign On - (SSO) options?

NW RFC Library 7.5 supports SSO by defining snc_qop, snc_myname, snc_partnername and snc_lib  connection values. These properties aren’t exposed in NwRfcNet since is difficult to test SSO scenarios.
Are you using Kerberos authentication in an Active Directory environment ?

Yes, by default were using Kerberos authentication

I have opened a new issue to follow up SSO #29. I have made a small test and it is possible to use SSO. I will upload tomorrow a change.
#29

Thank you!

SSO connection exposed and documented in Readme

Thanks, tested it out and it works!


[ubuntu] update version info

Update to 18.04.6
Remove lts: false flags

The release notes link is broken, they don't seem to be updated yet: https://wiki.ubuntu.com/BionicBeaver/ReleaseNotes/

I see the same problem as @captn3m0.
On https://wiki.ubuntu.com/Releases they link to 18.04.5 for the 18.04.6 changes. I think this is an error.
In https://fridge.ubuntu.com/2021/09/17/ubuntu-18-04-6-lts-released/ the writer links to https://wiki.ubuntu.com/BionicBeaver/ReleaseNotes. I think we should update to this.

@captn3m0 @BiNZGi i have changed link with correct one , could you review again ?


Package request: grayjay

Project description

Metadata

homepage URL:
source URL:
license: mit, bsd, gpl2+ , ...
platforms: unix, linux, darwin, ...

Note for maintainers: Please tag this issue in your PR.

Add a :+1: reaction to issues you find important.

Duplicate of https://github.com/NixOS/nixpkgs/issues/366543


fix(server): mutex calls to sqlitex

Fixes #13154
Fixes #13140
Motivation
zombiezen/go-sqlite is not thread safe when used through a single connection. The current code is provably racing (run the server with -race and a few workflows being run) and it will tell you this if you argo list via the server a few times. Unless it crashes first.
Modifications
This change doesn't attempt to move to a multiple connection model, it's a minimal change to stop the server crashing all the time, by mutexing the use of the sql connection.
Verification
Running v3.5.7
Inject 200 example/dag-diamond.yamls into your cluster. Run argo list -s <server> to list the workflows. It will crash within a few attempts, usually
With this change it didn't crash in 200 attempts at argo list.
-race build of the server doesn't complain any more either.

cc @jiachengxu for review

@agilgur5, @tczhao, @isubasinghe, @terrytangyuan - could someone take a look at this one so we can prep a 3.5.8 with it in?

good catch @Joibel

Nice work figuring out the root cause @Joibel!
Since we hadn't found any nil pointers in our own code, I was suspecting it might be within the SQLite library, although I definitely did not expect that it wasn't thread-safe. Those GoDoc comments really belong in the README of the library IMO 😕
Will you or @jiachengxu be taking up the full fix with thread pooling?

so we can prep a 3.5.8 with it in?

Would like to get a fix for #13177 in as well and then release it ASAP

Will you or @jiachengxu be taking up the full fix with thread pooling?

Wrote up a follow-up issue to track this in #13638


Can you set the default variable Name is the same as the name of the ID

like this:

+1

I add this option ，and create new commit pull request，you can use [TangentLu666/android-butterknife-zelezny]:https://github.com/TangentLu666/android-butterknife-zelezny  to download


Best Practices for Mutating Collections Multiple Times

I am working with the fast_immutable_collections package in Dart and have a question regarding the optimal way to handle multiple mutations on a collection. My goal is to understand the best practices for performance, especially in relation to the copy-on-write optimization that this package offers.
Here's the core of my query: When needing to mutate a collection multiple times, is it more efficient to unlock the collection, perform all mutations, and then lock it again, or should I simply use the immutable collection methods for each addition?
To give a concrete example, I am comparing these two approaches:
Approach 1: Unlocking and Locking
var nums = IList();
final newNums = nums.unlock;
for (var i = 0; i < 10; i++) {
  newNums.add(i);
}
nums = newNums.lock;

Approach 2: Using Immutable Collections
var nums = IList();
for (var i = 0; i < 10; i++) {
  nums = nums.add(i);
}

My concern is about the performance implications of these two methods, particularly regarding the copy-on-write feature. Does the first approach of unlocking and locking impact the optimization adversely? Or is it more efficient compared to adding items individually using immutable collection methods?
Any insights or recommendations on this would be greatly appreciated.
Thank you for your time and for maintaining this valuable package.

You should simply use the immutable collection methods for each addition.
It actually doesn't copy-on-write. Instead, it uses a special data structure that records only the change. And it will also detect internally that there were to many additions, and it will "flush" the contents from time to time, to optimize reading and memory.
In other words, I suggest you do what's easier for you, and trust FIC to do the necessary optimizations internally to be fast and save memory.
Of course, if you have a very large number of operations, you can always measure things yourself too see if it's fast. But FIC will have a very good performance most of the time.


Enable cuDNN executor by default

🚀 Feature
Now that cudnn-fe is publicly installable, we should add it into the default executors.
Motivation
cudnn often gives great performance :-)
Pitch
https://github.com/Lightning-AI/lightning-thunder/blob/73ca5dc79179773c542d71b3f81d02ae0a220329/thunder/__init__.py#L162
this logic to be updated to include cudnn
Additional context
we should be sure we are not regressing anything, and that may mean inserting new guards in the cudnn executor. Probably best to ask @parthmannan @mpatel31415 to try it

So far I have seen cuDNN SDPA give performance improvement and haven't seen any failures across a few models. Will test a few more models/configs and can confirm but I think enabling cuDNN executor by default would be beneficial.

Currently cudnn executor is overly optimistic when claiming sdpa. Link
This change was made in #57. Before including cudnn executor in default list, the checker should be made stricter.

To address the pain point of debugging with cudnn executor, there is a custom CudnnGraphNotSupportedException starting v1.3. The sdpa checker function can throw errors with a pseudo code like:
def _cudnn_operation_checker(
    a: TensorLike
):

   try:
        _make_cudnn_operation_graph(a)
    except cudnn.cudnnGraphNotSupportedError as ex:
        return False
    except Exception as e:
        raise

Does the above approach look reasonable?
Any unhandled exceptions throw will be either bugs to solve or user errors.

cudnn executor is overly optimistic when claiming sdpa.

ahh; yes, we should fix that first. Thanks for remembering this!

sdpa checker function can throw errors with a pseudo code like:

Yeah, this seems like a great approach to me!


__moduleName is an undeclared identifier

Hi, what is the purpose of __moduleName that is in all the components ?
It's not declared anywhere and also not imported so this causes problems with typescript
Cannot find name '__moduleName'.
I can get around this by defining this identifier in webpack, but i think this should be addressed. Using these measures is something usually reserved for integrating with pure JS libraries like jquery, and not native typescript projects.
By using this "unknown identifier", all folks who use bundlers will have to add extra configuration, and this library won't work out-of-the-box.

@jhuntoo __moduleName is for SystemJS. It works for me fine but I don't use Webpack or JSPM. Your solution with require solves that issue. I will remove __moduleName right now.


[REQUEST] Use new API for OutputChannel

Thanks for your work on this extension, it has been very useful :)
Using this feature in this part of the code we can easily improve the result view.
I'd recommend setting either json or maybe shell to let jq -C do the coloring part.

Thank you!


light bulb icon is too hollow

At first sight this looked like screen cheese, not like help

Filled in the top of these so they look better when on top of text:

cc @mjbvz


Update bfe_modules.go

add // mod_doh

Codecov Report

Merging #410 into develop will increase coverage by 0.01%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##           develop     #410      +/-   ##
===========================================
+ Coverage    53.07%   53.09%   +0.01%
===========================================
  Files          256      256
  Lines        23625    23625
===========================================
+ Hits         12539    12543       +4
- Misses        9599     9600       +1
+ Partials      1487     1482       -5

Impacted Files
Coverage Δ

bfe_websocket/server_conn.go
74.24% <0.00%> (-2.28%)
:arrow_down:

bfe_spdy/server_conn.go
78.09% <0.00%> (-0.45%)
:arrow_down:

bfe_http2/server.go
71.67% <0.00%> (+0.32%)
:arrow_up:

...e_modules/mod_trust_clientip/mod_trust_clientip.go
46.15% <0.00%> (+3.29%)
:arrow_up:

bfe_util/fixed_pool.go
100.00% <0.00%> (+16.66%)
:arrow_up:

Continue to review full report at Codecov.

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


Аdd the ability to disable backup rotation

Hello!
I use nxs-backup to collect backups in s3 storage. There was a need to change the user role to storage.uploader. I configured backup rotation at the storage level, but now I get errors about the impossibility of deleting old backups. As a temporary solution, I changed the configuration to the following:
 retention:
 days: 365
 weeks: 52
 months: 12

Hi @Dzalgz.
This feature is included in the 3.9.0 release.
New ideas and suggestions are welcome.


Job Dependencies

mProv jobs may need some method for dependencies.  ie. Repo sync finish before image generation starts.
Can probably use job params for this.  Will need a modification to the job sever.

Added a default for 'image-update' jobs, moving to mprov-ng/mprov_jobserver repo for jobserver side of the implementation


bug: defining a component using TimeProps will throw a TypeScript compiler error in regards to the ref prop

Hi Justin Mahar, I'm a big fan of this library and I really enjoy using it :-) I've recently come across what seems to like be a bug when I was trying to use the datetime attribute with it, so I reproduced some sample test cases below. There's also a CodeSandbox link to try it out: https://codesandbox.io/s/react-html-props-and-timeprops-zyv6pn?file=/src/App.tsx
Test cases
With reference (TimeProps)
Compiler error here:
import { TimeProps } from "react-html-props";

const date = new Date(Date.UTC(2020, 11, 20, 3, 23, 16, 738));
export const Test1 = ({...props}: TimeProps) => {
	return (<time dateTime={date.toISOString()} {...props}>January 1st, 2000</time>)
}

Compiler error log

Type '{ children: string; ref?: LegacyRef<HTMLElement> | undefined; key?: Key | null | undefined; defaultChecked?: boolean | undefined; defaultValue?: string | number | readonly string[] | undefined; ... 263 more ...; onTransitionEndCapture?: TransitionEventHandler<...> | undefined; }' is not assignable to type 'DetailedHTMLProps<TimeHTMLAttributes<HTMLTimeElement>, HTMLTimeElement>'.
  Type '{ children: string; ref?: LegacyRef<HTMLElement> | undefined; key?: Key | null | undefined; defaultChecked?: boolean | undefined; defaultValue?: string | number | readonly string[] | undefined; ... 263 more ...; onTransitionEndCapture?: TransitionEventHandler<...> | undefined; }' is not assignable to type 'ClassAttributes<HTMLTimeElement>'.
    Types of property 'ref' are incompatible.
      Type 'LegacyRef<HTMLElement> | undefined' is not assignable to type 'LegacyRef<HTMLTimeElement> | undefined'.
        Type 'RefObject<HTMLElement>' is not assignable to type 'LegacyRef<HTMLTimeElement> | undefined'.
          Type 'RefObject<HTMLElement>' is not assignable to type 'RefObject<HTMLTimeElement>'.
            Property 'dateTime' is missing in type 'HTMLElement' but required in type 'HTMLTimeElement'.ts(2322)

lib.dom.d.ts(13167, 5): 'dateTime' is declared here.

Without reference (TimePropsWithoutRef)
No compiler errors here!
import { TimePropsWithoutRef } from "react-html-props";

export const Test2 = ({...props}: TimePropsWithoutRef) => {
	return (<time dateTime={date.toISOString()} {...props}>January 1st, 2000</time>)
}

export type TimeProps = React.DetailedHTMLProps<React.TimeHTMLAttributes<HTMLTimeElement>, HTMLTimeElement>
export const Test3 = ({...props}: TimeProps) => {
	return (<time dateTime={date.toISOString()} {...props}>Hello World</time>)
}

Suggested fix
After looking into the code, the time element is categorized under the section of general elements, although it has its own unique types. If TimeProps is redefined as below using the TimeHTMLAttributes type from our friendly neighborhood @types/react package, then the error will go away.
export type TimeProps = React.DetailedHTMLProps<React.TimeHTMLAttributes<HTMLTimeElement>, HTMLTimeElement>;

Likewise, the TimePropsWithoutRef should also be updated to follow suite :-)
export type TimePropsWithoutRef = React.TimeHTMLAttributes<HTMLTimeElement>;

https://github.com/DefinitelyTyped/DefinitelyTyped/blob/d30ce2911b69c8c3ee099e981bd4f5120107354e/types/react/v15/index.d.ts#L3122

Shipped to npm. 🚀 Install 2.0.3 with:
npm i react-html-props@latest

Thanks for contributing!


Upgrade to a 10.x version of Mermaid.js

In trying to demo the Mermaid.js features of Polyglot Notebooks, I'm running into discrepancies between Polyglot Notebooks and the official Mermaid.js documentation, which seems to only reflect the current version (10.1.0 at the moment) instead of the configured version in MermaidMarkdownFormatter.cs of 9.1.7.
Because any invalid mermaid syntax causes a generic error display, this can lead to a frustrating user experience with mermaid code that matches the documentation and should work, but does not run on the version of mermaid that Polyglot uses. I've seen this particularly with title elements in various supported charts.
Additionally, using a more recent version of Mermaid gives us support for two new chart types:

Mindmaps
Timelines

I think both of these charts would demo well and help "wow" people at what's possible in Polyglot Notebooks and can personally see myself getting a lot of benefits out of Mindmaps in particular.
I'd like for us to upgrade from 9.1.7 to 10.1.0 or the latest stable version so we more accurately match the official documentation. This will reduce developer confusion and frustration and will enable a few more features. We also might want to consider a regular upgrade / update cadence for Mermaid.js in the future as this problem will likely recur.

Closing this issue as it is now available


Display more meaningful names for tolerances

Add modifiers and upper and lower limits to the name of a tolerance

Fixed with #414


Create GUI for creating aliases

Feature Request
Desired behaviour
Follows https://github.com/dockstore/dockstore-ui2/pull/552
Create user interface for generating aliases (ex: uuids) and sharing them as in the organizations wireframe (sharing button)
┆Issue is synchronized with this Jira Story
┆Issue Type: Story
┆Fix Versions: Dockstore 1.7
┆Issue Number: DOCK-546

Webservice does not currently return aliases for org/collection (lazy loaded)


Object Pascal is not a scripting language or "memory safe" at all, FYI

Not that it matters, but as someone who uses the language on a daily basis I found it rather odd to see it included here. Nothing safe about this for example.

Oh my bad, I assumed it was! Are there version of Pascal which are memory safe?

I mean, in what sense? It's hard to explain, but basically both Delphi and Free Pascal (as I was using in that link) are like:

Classes are pure "reference" heap-based types sort of similar to say D or Swift classes (minus any kind of implicit reference counting or garbage collection)
Objects are roughly equivalent to C++ classes in their support for inheritance (though it's just single inheritance) and ability to be either stack or allocated
Records are basically just like C++ structs or classes, except with no support for inheritance

There's various things about the overall language that make it more safe than C or C++ in most cases in practice, contextually, but also nothing stopping you from doing dangerous things with raw pointers / inline assembly / etc whenever you want, if that makes sense.

Wikipedia says

Pointers in Pascal are type safe; i.e. a pointer to one data type can only be assigned to a pointer of the same data type. Also pointers can never be assigned to non-pointer variables. Pointer arithmetic (a common source of programming errors in C, especially when combined with endianness issues and platform-independent type sizes) is not permitted in Pascal. All of these restrictions reduce the possibility of pointer-related errors in Pascal compared to C, but do not prevent invalid pointer references in Pascal altogether. For example, a runtime error will occur if a pointer is referenced before it has been initialized or after it has been disposed.

If we really can't define Pascal as a memory safe language, I'm fine dropping it and replacing it with one that is truly memory safe..

I mean, again, it depends on what you mean. Pascal as described in that article lacks many features added later in various different implementations, though generally yes, is type-safer than C. Note that there's no "Pascal standard" that anyone cares about currently, different parties just extend it however they see fit.
At the broadest level though (in current "Object" Pascal implementations, specifically) there is no garbage collection or automatic memory management of any kind other than in a couple of specific cases for specific types, such as the "magic" compiler-implemented strings and dynamic arrays that it has (which are atomically reference counted in a way that requires nothing on the programmer's side).
That's basically what I was mostly getting at here, originally.

Hmm. Looks like we need to drop Pascal in favor of a different language.. since we agreed to include Kotlin, maybe Kotlin replaces it? Google gave Kotlin a big boost:
https://developer.android.com/kotlin

Looks like we need to drop Pascal in favor of a different language.

Might I suggest D? (Tongue in cheek) Seriously though, I am professionally involved in porting half a million lines of Pascal code to D. D is a fitting successor to Pascal (as well as to C++, Python; Basic...)
I just pushed changes to #365 that should guarantee memory-safety.

There’s been a bigger push for Kotlin IMO and D is not really memory safe
either …
On Mon, Jan 3, 2022 at 8:14 AM Bastiaan Veelo @.***>
wrote:

Looks like we need to drop Pascal in favor of a different language.
Might I suggest D? (Tongue in cheek) Seriously though, I am professionally
involved in porting half a million lines of Pascal code to D. D is a
fitting successor to Pascal (as well as to C++, Python; Basic...)
I just pushed changes to #365
https://github.com/coding-horror/basic-computer-games/pull/365 that
should guarantee memory-safety.
—
Reply to this email directly, view it on GitHub
https://github.com/coding-horror/basic-computer-games/issues/343#issuecomment-1004198549,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AALTWVIPYXDHALUAYXCDGLTUUHDO7ANCNFSM5LDNT7WA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you commented.Message ID:
@.***>

D is not really memory safe

Neither is Kotlin. D already has the features that are proposed in that post, nota bene.

@Akira13641

At the broadest level though (in current "Object" Pascal implementations, specifically) there is no garbage collection or automatic memory management of any kind other than in a couple of specific cases for specific types, such as the "magic" compiler-implemented strings and dynamic arrays that it has (which are atomically reference counted in a way that requires nothing on the programmer's side).

I don't know if you count Interfaces as "couple of specific cases", but they are reference counted.

We'll have to go back to the TIOBE top languages list and decide which one we want to replace with. Them's the rules..

That's fine if that is the rule you want to have. The value of that index is frequently questioned though (see Wikipedia and plenty other places) and especially once you get into the promille range, noise gets more prominent in the ranking.
But since it's you who is setting the rules, you can also just say Pascal stays in because I like the language (and because all the work has been done). Is there any downside to include other languages, and expand the opportunities for learning?

Well, I do believe very strongly in the languages being FULLY memory safe, so I want to stick with that..

@baka0815

I don't know if you count Interfaces as "couple of specific cases", but they are reference counted.

That's true I guess yeah, although Free Pascal at least has a compiler directive that makes them not (which is quite useful in some cases, though generally I don't use interfaces that much, or even classes that don't implement any interfaces these days for that matter, preferring "extended records" most of the time).
Can't remember if a similar directive is available in Delphi.

I'm still torn on what to do here and could use advice. I feel VERY STRONGLY we want only absolutely safe languages, no exceptions.

I'm still torn on what to do here and could use advice. I feel VERY STRONGLY we want only absolutely safe languages (in the official list of languages, I mean), no exceptions.
For people who want to contribute C or C++ versions (or any other memory unsafe language) that is also fine, but those must be linked from the readme.md.

Frankly, I think a larger oddity with your original list is moreso that you included languages that are fully memory safe, but still are absolutely not "scripting languages" by any metric I'm aware of (notably C#, VB.NET, and Java, for example).

I am afraid there is a need for nuance when talking about safety.

Java has an unsafe escape hatch.
C# has an unsafe escape hatch.
Python is not "absolutely memory-safe".
Ruby and Perl have had vulnerabilities.

It appears to me that the linked-to unsafe Delphi code requires the {$PointerMath On} pragma, from which one might conclude that code that does not include that pragma cannot do unsafe sings that require pointer arithmetic. Given that Pascal can do array bounds checking, I think the way in which Pascal is used to port these BASIC examples is very safe.

Also, from your Pascal Wikipedia quote:

... but do not prevent invalid pointer references in Pascal altogether. For example, a runtime error will occur if a pointer is referenced before it has been initialized or after it has been disposed.

A runtime error does not mean the program is memory-unsafe. An error due to memory-unsafety causes undefined behaviour, including the program continuing with wrong results, or affecting other parts of the system. That's not the case here. If the program is terminated due to a memory error then that is considered a safe outcome!

Also, from your Pascal Wikipedia quote

TBH, you really can't define "Pascal" by what Wikipedia has to say in reference to what sounds like an implementation from around 35 - 40 years ago. The closest you can get to accurate nowadays is "literally whatever the developers of Delphi and / or Free Pascal happen to decide to implement at any given time".

"literally whatever the developers of Delphi and / or Free Pascal happen to decide to implement at any given time"

True.
I get your point and I agree.
@coding-horror: Drawing a clear line at languages that are absolutely safe is problematic because safety is not a clear cut concept. Even Ada is not completely safe, otherwise SPARK would not need to exist. What do you actually mean with "memory-safe scripting languages"? I think you actually want languages that are suitable for beginners, that are quick to get going with and that help you from making programming mistakes to a reasonable degree. Because even though SPARK may be the safest language in existence, nobody would ever recommend SPARK to a beginning programmer because [s]he would not get things done (let alone afford the license fees).

Well SPARK isn't in the top 20 TIOBE languages, is it? That's another inclusion criteria we care about.. this isn't an exercise in speaking forgotten languages, or languages nobody (statistically speaking) uses..
https://www.tiobe.com/tiobe-index/

I'm not suggesting SPARK or Ada to be included.
I think, because BASIC was easy to learn and hard to make dangerous mistakes in, and the book was influential to beginning programmers, that you want those same traits in the selected modern languages. What we are trying to say is that "memory safe scripting languages" is not a very good description then. So why not make the requirement "a language that is easy to learn and hard to make dangerous mistakes in"? Then all currently selected languages can stay including Pascal, and C/C++ stay out.
OTAH, I might be misunderstanding the objective of the project completely...

a language that is easy to learn and hard to make dangerous mistakes in

That seems fine!

I've updated the official languages to 10:

C#
Java
JavaScript
Kotlin
Lua
Perl
Python
Ruby
Rust
VB.NET

Rationale is here https://discourse.codinghorror.com/t/updating-101-basic-computer-games-for-2022-and-beyond/7927/116


Integrate Jupyter model to superset

Hi,
First of all thanks to everyone who is contributing on Apache Superset, a really beautiful data visualization tool with the rich features.
Currently, I am working on a project where I have a situation where I need some guidance.
I have three systems, one has the database, another has jupyter notebook and third has superset.
First, we create the data model in R/Python using jupyter nb. Now we would like to run these data model on the database via superset so that we can visualize the data. And each time when we run the data model on data it should visualize the latest data from the database.
We know that;

We can directly connect superset with database and play with data.
or can run data model manually at the backend and generate the output data as csv and can visualize the result on the superset.

We can not go with the 1st approach because we have the separate users for both Jupyter and superset. Jupiter user cannot access superset to create the data model, slices, and dashboard. Also, we do not want to expose raw data with superset user and superset user should able to only see the graphs/charts generated from the output of the data models.
We can also not go with 2nd approach because every time when we want to see the latest data, we have to first run data model manually.
So is there any solution to my problem where I can run data models in the background whenever I reload the dashboard and I can see refreshed data.
Any lead/suggestion will be highly appreciated.
Thanks!

What if at the end of your model running you INSERT back into your DB the info you'd want visualized?
If I'm not understanding exactly, could you give some idea about what you've visualizing in the notebook.

Using pandas to_sql() function you can directly sync your df to a database which is connected to superset.

reopen

this should not be closed. its important feature to have

Have an idea to use jupyter lite as one of superset module like sqllab. But it's hard to integrate them.


Update python-publish.yml us release/v1 instead of master.

This should fix #90
see: https://github.com/pypa/gh-action-pypi-publish?tab=readme-ov-file#-master-branch-sunset-

Thanks!


Pull request for v0.4

v0.4

Coverage increased (+0.09%) to 67.146% when pulling f9c446b145cc3c1d557e34f4f8375459179fe46b on devel into ce17b0fc3c069a9a20beecd6ed2f540355b5e666 on master.


Architecture Updates: Theming & Fix up some components dark theming

Description

Separates "theme" out of "bg". Clashing occurred as whilst we had dark theming in /flowforge, we didn't always utilise the ff-bg-dark theme, because it was opinionated on the background colour too, which in some cases 9e.g. sign up box) differed by a level of grey.
This PR adds a new ff-theme-<theme> class set which will make it easier to inherit the dark themed classes on a component-by-component basis, without affecting the background-color.
In re-architecting this, also noticed a fair few components (data-table, dropdown, notification and the initially flagged checkbox/radio-group pair) that needed theming defined in dark mode. Whilst we hadn't utilised these components in a dark mode yet, they should still be well defined. These can sanity checked from within the components documentation where it's possible to switch to dark mode in the top-right.
Includes a version bump to v0.7.0

Related Issue(s)
Issue first raised here: https://github.com/flowforge/flowforge/issues/2419 but scale of the issue realised to be much bigger, and so this PR addresses the wider "Theming" architecture of forge-ui-components.
Checklist

[x] I have read the contribution guidelines
[x] Documentation has been updated

just spotted a slight problem on a border collor with checkbox in dark mode - fixing now


CI_DB_utility::csv_from_result is vulnerable to CSV formula injection attacks

The CI_DB_utility::csv_from_result is currently vulnerable to CSV formula injection attacks which could lead to data breach, Malicious formulae pose a risk even when the embedding spreadsheet doesn’t contain any sensitive information, as they can be used to compromise the viewer’s computer.
See here for background.
https://www.contextis.com/blog/comma-separated-vulnerabilities
http://georgemauer.net/2017/10/07/csv-injection.html
See here for partial solution, it's a bit of a sledgehammer, as I've not built in any way to turn it off where a developer is prepared to accept that risk and explicitly turn it off when formula's are required in a CSV spreadsheet, but more for proof of concept, or what's required to negate:
https://github.com/munkiepus/CodeIgniter/pull/1/commits/c60df53f399bb7652ebb238b455174c6f0d4be12

The esoteric nature of this aside, I'd understand it if we were talking about XLS, but the concept of formulae doesn't exist in CSV.
If some particular software treats CSV as XLS, that's not our problem.

The problem is that it's not just some software that treats CSV as formula, it happens in the 3 most likely places where spreadsheets would be opened and more, Excel, Open office and google sheets. It's a well-documented vulnerability and easily reproducible. You're right in saying that it's somewhat esoteric and a bit convoluted, but it is a real attack vector to which the framework is vulnerable and which could be easily mitigated against.

if anyone ends up here looking to mitigate this, i have a solution which overrides the offending method here:
https://gist.github.com/munkiepus/537d502c97aaeaef63ddfab907811854

Well, the main thing that we disagree on is that you say the framework is vulnerable to it and I say it's not; the framework doesn't suffer any effects from this.
Forget the esoterics, the fact that somehow an XLS vulnerability is translated into CSV, the ignored user warnings from the said software ... Spreadsheet software is simply not our problem. Sorry.


Managers don't handle :retry_ifadded message.

When running dialyzer, it notifies about never handling the :retry_ifadded message.
I've never seen this in production, and its safe to ignore at the moment. (a warning will be logged)
  defp consume(:removed, :ifadded, state) do
    case Nerves.NetworkInterface.ifup(state.ifname) do
      :ok ->
        {:ok, status} = Nerves.NetworkInterface.status(state.ifname)
        notify(Nerves.NetworkInterface, state.ifname, :ifchanged, status)

        goto_context(state, :down)
      {:error, _} -> # this never gets called from what ive seen but i suppose it _is_ possible.
        # The interface isn't quite up yet. Retry
        Process.send_after self(), :retry_ifadded, 250
        goto_context(state, :retry_add)
    end
  end

we will need to do something with that info, or remove it entirely.
def handle_info(:retry_ifadded, %{ifname: ifname} = s) do
    # GOTO Different context maybe?
   {:noreply, s}
end

This may have been a premature optimization at the nerves_network_interface level.
I had a read thru the c code and i don't see any references to returning an {:error, term()} tuple.


Demos Export to xlsx - style formatting are not exported

I don't know what is going wrong but the demos are not working correctly. When I click to export to xlsx the table is plain without any format or style
https://tableexport.v3.travismclarke.com/examples/bootstrap.html

I'm also facing this issue.


fix: latitude and longitude mapping issue for version 5.0

According to neo4j PointValue document, x represent longitude and y represent latitude
Change the mapping of latitude and longitude

Hello, thanks for your contribution!
Before we proceed, can you please sign Neo4j's CLA?

@fbiville I have sent the email with Neo4j CLA, if there is anything else that is required, please let me know, thanks.

@yhhongyang 2 questions:

you signed the CLA document meant for IBM workers: do you work for IBM? If not, you need to pick the other one.
your commit's email address does not match the email address you mention in the email you sent for CLA approval. Can you please either reply by mentioning the commit's email or amend your commit with the email address mentioned in CLA email?

Thanks a lot!

@fbiville For your 2 questions

Yes, I work for IBM.
I resend the email, and change the email address with the one that I used to commit code, could you please check again, thanks.

@yhhongyang thanks but the emails still don't match

@yhhongyang could you maybe send the CLA and amend the commit with your IBM email address? thanks!

@fbiville I checked the email I sent earlier and found the problem and fixed it. Could you please check it again? Thanks

@yhhongyang thanks! can you also please send the signed CLA with that email address please?

I think I sent that  in 07/16

I know you sent it but as I tried to say multiple times: the commit email and the email you use and mention for the CLA need to match.
Now that you amended your commit with your IBM email address, can you please also use and mention the very same IBM email address with the signed CLA?

Did I send it right this time?

Did I send it right this time?

Yes, thanks!


Other ORM like mongoid

Is it possible that this become easily extensible with other ORM like mongoid. I saw there is the active_record.rb in Railties. Can we do some configuration and use other file like mongoid.rb to change the data store and still use this paginator.

This has been implemented a while ago, maybe close this?


The new look and behavior of scrollbar is not that good

I was using a previous version of perfect-scrollbar and today I downloaded the latest version, however it seems that the scrollbar get thiner and when I hover the mouse over it it becomes larger, I am posting this to request this to be a secondary effect that can be added as option (I think it's better), and the second thing I would like to know how I can get the old look and behavior while using the current version  ?  Thanks

@medBouzid it does seem like having this as an option would be good, but for me, I just override the css so it always does what I want (which is staying a static size and always visible);
.ps-container{
    div{
        border-radius: 0 !important;
    }
    .ps-scrollbar-y-rail, .ps-scrollbar-x-rail{
        opacity: 1 !important;
    }
    .ps-scrollbar-y-rail, .ps-scrollbar-y-rail .ps-scrollbar-y{
        right: 0 !important;
        width: 0.5rem !important;
    }
    .ps-scrollbar-x-rail, .ps-scrollbar-x-rail .ps-scrollbar-x{
        bottom: 0 !important;
        height: 0.5rem !important;
    }
}

@itsOgden Thank you for the answer and the snippet of code :) I was using the previous version.. so I didn't changed the old css file, I just changed the JavaScript file to "upgrade", now I run the latest version of perfect-scrollbar with the old css, your solution is good as well, thanks

I'm really sorry that the new design is not good for you, I just wanted it to look like ones in iOS. However, now the style is in a plain CSS and it's much easier to play around it!
Sorry again, and thanks for your opinion.


Revert "test pr"

Reverts myakove/github-gitlab-webhook-server#59

The following are automatically added:

Add reviewers from OWNER file (in the root of the repository) under reviewers section.
Set PR size label.
New issue is created for the PR.

Available user actions:

To mark PR as verified add !verified to a PR comment, to un-verify add !-verified to a PR comment.
Verified label removed on each new commit push.
To cherry pick a merged PR add !cherry-pick <target branch to cherry-pick to> to a PR comment.
To add a label by comment use !<label name>, to remove, use !-<label name>
Supported labels:
do-not-merge
verified
wip
lgtm
target-version-

!target-version-dev1

!target-version-dev1

!target-version-dev1

!target-version-dev2

Manual cherry-pick is needed
Cherry pick failed for 3850c6e2918cd4dad348177bc45256eda66db022 to dev1:
fatal: bad object 3850c6e2918cd4dad348177bc45256eda66db022
To cherry-pick run:
git checkout dev1
git checkout -b 3850c6e2918cd4dad348177bc45256eda66db02
git cherry-pick 3850c6e2918cd4dad348177bc45256eda66db022
git push origin 3850c6e2918cd4dad348177bc45256eda66db02

Manual cherry-pick is needed
Cherry pick failed for 3850c6e2918cd4dad348177bc45256eda66db022 to dev2:
fatal: bad object 3850c6e2918cd4dad348177bc45256eda66db022
To cherry-pick run:
git checkout dev2
git checkout -b 3850c6e2918cd4dad348177bc45256eda66db02
git cherry-pick 3850c6e2918cd4dad348177bc45256eda66db022
git push origin 3850c6e2918cd4dad348177bc45256eda66db02


array is implemented in menu driven format

Assigned Issue Number ?
#1591
Language(s) Opted ?
c++
Number of file(s) added or updated ?
1
Name and location of the file(s) ?
/algo_ds_101/Data-Structures/Array
Checklist

[x] I've followed the contributing guidelines
[x] The content added isn't directly copied from any sources like gfg, tutorialpoint, etc.
[x] Input has been taken dynamically.
[x] I've referred the correct issue number.
[x] Comments have been added
[x] I've  synced-up my forked repo.
[x] Ain't pushing from the main branch.

all the function operation which can be performed on an array is implemented by me.
like :- push , binary search, linear search, merging , intersection, sum of elements etc.

please merge #1713
i have done all the changes which you told me in prev PR
thank you


Java resources for backend and android development

Description
We can add resources for java because it is also used in android development and backend development. So, java can help in these 2 domains and in DSA. I would like to contribute to this issue.
Screenshots
No response
Additional information
No response

Hey @suryanshsoni120, I don't think that would be a good idea. As the roadmaps are aimed at beginners, it would be better to keep just one core technology. If we include Java as well, folks would language hop a lot, and in the end not get anything done. Same applies to Backend.
@kaiwalyakoparkar would like to hear your thoughts on this


Failed to install

I tried to install the academic package as described in tutorial but get this error information:
ERROR: Cannot uninstall 'ruamel-yaml'. It is a distutils installed project and thus we cannot accurately determine which files belong to it which would lead to only a partial uninstall.
How to resolve this issue please?

This error appears to be related to your system setup, rather than Academic itself.
See https://stackoverflow.com/questions/49932759/pip-10-and-apt-how-to-avoid-cannot-uninstall-x-errors-for-distutils-packages
You could also consider installing in virtual Python environment: https://realpython.com/pipenv-guide/


Fixed new contact when creating a new contact in the account

Q
A

Bug fix?
yes

New feature?
no

BC breaks?
no

Deprecations?
no

Fixed tickets
fixes #1960

Related issues/PRs
none

License
MIT

Documentation PR
none

What's in this PR?
When adding a new person in the organization -> people list the newly created person wont be displayed after saving.

StyleCI fails.

@danrot was fixed by a rebase


Model to estimate the CPU usage for a Pod to be scheduled

Considering the historical data about container images, resource requests, and their corresponding average resource utilizations, it may be possible to train a ML model to predict the resource utilization for a pod to be scheduled. E.g., if five pods are already running and Horizontal Pod Autoscaler decides to create a sixth pod, then the average CPU utilization across the existing five pods can be considered as the average CPU utilization for the sixth pod.

To begin with, we predict utilization for a container based on its requests/limits similar to the way it is done in the k8s scheduler-plugin Trimaran https://github.com/kubernetes-sigs/scheduler-plugins/blob/master/pkg/trimaran/targetloadpacking/targetloadpacking.go#L197


update setuptools_scm dependency

Is there a reson to restrict setuptools_scm version to <6.1?
See https://bugs.gentoo.org/835890

Not that I am aware of. Thanks for pointing this out.


Document Federation CoreDNS support.

It is not entirely clear how to use CoreDNS as a DNS provider for federation. We need a doc walking through the setup.
cc @kubernetes/sig-federation-misc

Do you have further ideas of how this would fit into existing content? Need any guidance from the docs team?

I would give it a try and come up with a draft in couple of days. any pointers that would help me in this task would be great and thanks for help offer.

Sure,
I'd suggest starting with the page template for tasks: https://kubernetes.io/docs/contribute/page-templates/
Do you know where you're going to put the page on the site?

the federation deployment using CoreDNS would mostly be associated with this page https://kubernetes.io/docs/admin/federation/kubefed/

@jaredbhatti this doc will probably be one of the user guides.

https://github.com/kubernetes/kubernetes/issues/40536 is the umbrella issue for tracking this and others targeting improvements in non-cloud deployment of the federation control plane.

I am currently facing the issue mentioned in http://stackoverflow.com/questions/43759332/coredns-in-k8s-federation-in-on-prem-clusters-doesnt-work. I would appreciate if someone could tell me the reason of this behavior or possible fixes of this problem. I have followed all the steps mentioned in the documentation.

Have you asked the kubernetes-dev group (https://groups.google.com/forum/#!forum/kubernetes-dev)?


Add azure blob storage backend

Q
A

🐛 Bug fix?
no

🚀 New feature?
yes

⚠ Deprecations?
no

❌ BC Break
no

🔗 Related issues
#352

❓ Documentation
https://github.com/snyk/driftctl-docs/pull/222

Description
ACC test pass here

Codecov Report

Merging #1357 (ef76776) into main (2116470) will increase coverage by 6.56%.
The diff coverage is 22.29%.

@@            Coverage Diff             @@
##             main    #1357      +/-   ##
==========================================
+ Coverage   75.12%   81.69%   +6.56%
==========================================
  Files         482      414      -68
  Lines       18122    15118    -3004
==========================================
- Hits        13615    12350    -1265
+ Misses       4216     2461    -1755
- Partials      291      307      +16

Impacted Files
Coverage Δ

pkg/helpers/azure/storage.go
0.00% <0.00%> (ø)

pkg/iac/terraform/state/backend/backend.go
45.16% <0.00%> (-3.12%)
:arrow_down:

pkg/iac/terraform/state/enumerator/azurerm.go
0.00% <0.00%> (ø)

pkg/analyser/analysis.go
90.28% <21.42%> (-6.06%)
:arrow_down:

...kg/iac/terraform/state/backend/azureblob_reader.go
28.57% <28.57%> (ø)

pkg/iac/terraform/state/terraform_state_reader.go
63.01% <40.00%> (-9.08%)
:arrow_down:

...iac/terraform/state/enumerator/state_enumerator.go
50.00% <50.00%> (-8.34%)
:arrow_down:

pkg/helpers/azure/blob.go
92.85% <92.85%> (ø)

pkg/terraform/plugin_client.go
0.00% <0.00%> (-100.00%)
:arrow_down:

pkg/remote/aws/client/s3_client_factory.go
0.00% <0.00%> (-100.00%)
:arrow_down:

... and 173 more


AMI release to get Cluster DNS IP from SCIDR for Self Managed Nodegroups

Issue:
Getting Cluster DNS IP from SCIDR for Self Managed Nodegroups, when DNS IP is not provided by user. This is being done as a part of SCIDR Feature Launch
Description of changes:
Added a code to find SCIDR from Describe Cluster and assign DNS IP using that. This is done as a part of new SCIDR feature, where we allow user to give SCIDR value as well. For Mnaged Nodegroups this is handles in ISO. For unmanaged nodegroups we are handling it by creating a new AMI version. If the SCIDR is given then the Cluster DNS IP is set using that. To make it backward compatible, if SCIDR is null DNS IP will be set 10... or 172... depending on the vpc-ipv4-cidr-blocks as it was happening earlier.
Testing:
Since the feature is not launched yet. I added the service model in the nodegroup created during testing by : aws configure add-model
Created a file in the nodegroup that holds the value of SCIDR fetched from Describe Cluster API and DNS Cluster IP.
Created a cluster with SCIDR, then created a Self managed nodegroup with new Service Model loaded in it and one with current API.
Created a cluster and Self managed nodegroup in external-client-account as well to test backward compatibility
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

This change has broken any users which are not using managed node groups and are running in a private vpc i.e. we pass the ca bundle and the endpoint but not the DNS IP as we want the bootstrap.sh to calculate this.


feat(material-experimental/mdc-chips): Allow custom role for chip set

The default mat-chip-set role is 'presentation' if chips are
present, and null if there are no chips. Some users want to
specify custom roles, in particular 'list' (they would use
'listitem' for the chips, which have no default role).
Currently this isn't possible, because mat-chip-set overwrites
their custom role with 'presentation'. Add an Input to allow a
custom role.

What's the reason this shouldn't be hardcoded as list/listitem?

What's the reason this shouldn't be hardcoded as list/listitem?

I considered that, and I don't think it's a bad idea! This PR doesn't preclude us from changing the default role in the future, and even if we decide to do so, I still think it would be nice to provide a way to override it. There are a few internal examples of people using mat-chip-set in a non-list/listitem way, and the original design intended for this component to be pretty generic/non-prescriptive.

I don't think the failing ci/circleci: tests_saucelabs tests are related to this change. It looks like they're timing out.


Different performance between model saved as fine-tuned PLM and state_dict

Version
PyABSA = 2.3.4rc0
Torch = 2.1.1
Transformers = 4.35.2
Describe the bug
I've fine-tuned a model with config FAST_LSA_S_V2 using the same dataset using the APCTrainer. In one of the runs I saved it as a state_dict file and in the other a saved as PLM. I've then used the model on sample data using the APC.SentimentClassifier and the HF text-classification pipelines, but I get different results despite the model being trained the same way with the same data.
Code To Reproduce
Loading and testing the state_dict version:
sentiment_model = APC.SentimentClassifier('checkpoints/fast_lsa_s_v2_uber_acc_98.2_f1_98.14/')
examples = [
    "ty images city officials are standing their ground in defense of the law delivery workers like all workers deserve fair pay for their labor and we are disappointed that [B-ASP]Uber[E-ASP] doordash grubhub and relay disagree vilda vera mayuga head of the city's department of consumer and worker protection said in",
    "as prepared just a mile away for it to be delivered to me the food arrived stone cold i complained to [B-ASP]Uber[E-ASP] and [B-ASP]Uber[E-ASP] told me to go pound sand readers purporting to work for [B-ASP]Uber[E-ASP] left dozens of comments castigating me and others for not preemptively tipping arguing that such deliveries are not worth"
]

sentiment_model.predict(
    text=examples,
    eval_batch_size=32,
)

output:
[{'text': "ty images city officials are standing their ground in defense of the law delivery workers like all workers deserve fair pay for their labor and we are disappointed that Uber doordash grubhub and relay disagree vilda vera mayuga head of the city's department of consumer and worker protection said in",
  'aspect': ['Uber'],
  'sentiment': ['Negative'],
  'confidence': [0.9339152574539185],
  'probs': [array([0.93391526, 0.05876274, 0.007322  ], dtype=float32)],
  'ref_sentiment': ['-100'],
  'ref_check': [''],
  'perplexity': 'N.A.'},
 {'text': 'as prepared just a mile away for it to be delivered to me the food arrived stone cold i complained to Uber and Uber told me to go pound sand readers purporting to work for Uber left dozens of comments castigating me and others for not preemptively tipping arguing that such deliveries are not worthwh',
  'aspect': ['Uber', 'Uber', 'Uber'],
  'sentiment': ['Negative', 'Negative', 'Negative'],
  'confidence': [0.9557020664215088, 0.9557020664215088, 0.9557020664215088],
  'probs': [array([0.95570207, 0.03284235, 0.01145565], dtype=float32),
   array([0.95570207, 0.03284235, 0.01145565], dtype=float32),
   array([0.95570207, 0.03284235, 0.01145565], dtype=float32)],
  'ref_sentiment': ['-100', '-100', '-100'],
  'ref_check': ['', '', ''],
  'perplexity': 'N.A.'}]

With the HF text-classification pipeline
model_tokenizer = AutoTokenizer.from_pretrained('checkpoints/fast_lsa_s_v2_uber_acc_98.2_f1_98.14/fine-tuned-pretrained-model/')
model = AutoModelForSequenceClassification.from_pretrained('checkpoints/fast_lsa_s_v2_uber_acc_98.2_f1_98.14/fine-tuned-pretrained-model/')
sentiment_pipeline = pipeline('text-classification', model=model, tokenizer=model_tokenizer, device=1)
examples_no_tag = [{'text':re.sub(r"\[B-ASP\](.+?)\[E-ASP\]", r"\1", ex), 'text_pair': 'Uber'} for ex in examples]
sentiment_pipeline(examples_no_tag, top_k = 3)

Output:
[[{'label': 'Neutral', 'score': 0.38777175545692444},
  {'label': 'Positive', 'score': 0.3418353199958801},
  {'label': 'Negative', 'score': 0.27039292454719543}],
 [{'label': 'Neutral', 'score': 0.3863997459411621},
  {'label': 'Positive', 'score': 0.3450266420841217},
  {'label': 'Negative', 'score': 0.2685735821723938}]]

Expected behavior
I would expect there would be some correspondence between the output probability in both versions of the model.
Thanks!

The model saved as huggingface format is not intended as instant inference but further finetuing and the state_dict is the recommended save mode. If you want to run the model on pipeline, there is a model have been released at: https://huggingface.co/yangheng/deberta-v3-base-absa-v1.1

I see. What is required to make that model available to be run with huggingface pipeline?
Also, is there a checkpoint for the huggingface model? I would like to replicate the results I get with the pipeline with PyABSA.

I am sorry for that, it is tricky to train models compatible with huggingface pipeline, and I have cleaned the original materials such as codes so I am afraid that I cannot provide detailed help for that.


Add checkbox for swift test --generate-linuxmain

Adds a checkbox to the PR template so the linuxmain is regenerated every time new tests are added, to prevent us from missing tests on Linux on accident.

Hey @MrLotU, you just merged a pull request, have a coin!
You now have 84 coins.


Joint creation UI

The way that I currently have things set up for body orientation uses a Slotmap<PeripheralNode>, meaning that we can encode context into our body orientation by showing what motors are connected to what, or what sensors are connected to specific motors. This also allows disjoint sensors to exist, and if need be we can flatten the Slotmap entirely for ease of use early on.
That being said, I think it could be cool to have a small web GUI written up that allows users to create Nodes and connect them with each other. They could then serialize this to Json and load it as the model's body in the jetson software
Something like a UI that would allow developers to make this:

I'll take it.

We no longer need this as the joints will be inferred from the URDF file generated by solidworks


xsuaa migration api Principal.getName response change

Hi,
We're trying to upgrade our project using the xsuaa migration guide.
https://github.com/SAP/cloud-security-xsuaa-integration/blob/master/spring-xsuaa/Migration_JavaContainerSecurityProjects.md
After upgrading, we found that the Principal.getName api now returns<EMAIL_ADDRESS>in Cloud Foundry.
The prefix "user/ldap" or "client/" is added and since we are using this api for user authorization, it creates a problem.
Is there a way to prevent this from happening ?
Regards,
Kamal

Hi @kamakumar ,
You may like to get the name like it it is done here (from USER_NAME claim):
https://github.com/SAP/cloud-security-xsuaa-integration/blob/master/java-security/src/main/java/com/sap/cloud/security/adapter/xs/XSUserInfoAdapter.java#L89
Does this help?
I will update the documentation accordingly...
Best regards,
Nena

Your use-case isn't documented anywhere and might not be supported in the near future as it uses classes from the deprecated Spring OAuth2 lib.
    @GetMapping("/v2/sayHello")
    public String sayHello(@AuthenticationPrincipal Principal principal) {

        principal.getName(); // returns token.getUserName() @see AuthenticationToken
    }

References:

https://github.com/spring-projects/spring-security/wiki/OAuth-2.0-Migration-Guide
https://stackoverflow.com/questions/59273338/what-is-the-replacement-for-the-deprecated-authorizationserver-in-spring-securit
https://github.com/SAP/cloud-security-xsuaa-integration/blob/master/spring-xsuaa/src/main/java/com/sap/cloud/security/xsuaa/token/OAuth2AuthenticationConverter.java
https://github.com/SAP/cloud-security-xsuaa-integration/blob/master/spring-xsuaa/src/main/java/com/sap/cloud/security/xsuaa/token/AuthenticationToken.java


WIP:fix: bringing spectral package back to passing KFM

DO NOT MERGE.

@wtrocki I took a chainsaw to this. With these changes spectral lint will pass KFM with no errors or warnings.
note that to do so I had to delete 4 specific RHOAS rules as outlined in the comments in the ruleset.
I also had to turn off 6 more oas rules. 4 for warnings and 2 for errors.
obviously this is not the solution we are looking for. do you want to move ahead with these or should I keep the oas rules?


how to run in MVC

HI @clue    This project is very good
I want to integrate him into MVC，but  I don't know how to do it
I've been trying for a long time,
I hope the great God for advice
Thanks

Thanks for your interest, but unfortunately this question is somewhat beyond the scope of this project and I don't see how this could be answered without an in-depth understanding of your particular code.
As a first step, I would suggest making yourself familiar with the examples, the documentation and/or ReactPHP in general.
If this is for a commercial project and you want a quote, please check out my profile and send me an email. Otherwise, you may want to check out the common community channels (such as IRC, StackOverflow etc.).
I hope this helps and you understand I'm closing this ticket for now. If your problem persists and you feel this is an issue that targets a broader audience, please comment back and we can re-open this. :+1:


OpenMMIMDRunner fails to import in visualisations.ipynb

When running the second cell of this tutorial, an error is thrown when running from nanover.ase.openmm import OpenMMIMDRunner, stating ImportError: cannot import name 'OpenMMIMDRunner' from 'nanover.ase.openmm'. Is this because this wrapper is depreciated?
Speaking to @rhoslynroebuck about this, it appears this is a general problem (occurred on both of our machines, Mac and Windows), and she suggested that this line could be replaced with one of the following:
from nanover.ase.openmm import ASEOpenMMRunner
from nanover.openmm import OpenMMRunner
What do you think @jbarnoud @Ragzouken ? I am happy to fix this in a new PR if either of these solutions are suitable!

Yup definitely an oversight of the deprecation: Rho's first suggestion should be the right one, please go ahead


Bratseth/distribute each constant once

@lesters please review
I recommend looking at each commit separately in order.

Thanks, fixed.


effect is a keyword and therefore it shouldn't be used as an identifier

effect is a keyword that changes a module into an effects module. Therefore a definition like that:
effect x =
    x ++ x

should result with a parser's error.
It's similar to a code like that:
import x =
    x ++ x

which raises a parser's error correctly.

The term alias is also a keyword that can be used as a variable. There are certain keywords that can never be ambiguous, so it does not seem terrible to allow their use.
That said, it's better to make suggestions like this on elm-discuss and refine them into well-supported suggestions once you have some community feedback.


Latest

Thank you for this. However, it's preferable to have a list of all garden quest rather than a variable for each
Feature added in 2bbc62cc91a21acca90756c90a4f0eb74dc0feb5


[WIP - DO NOT MERGE] [Constraint solver] Enable constraint propagation pass.

Enable the constraint propagation pass in an effort to do a pre-pass to
reduce the amount of work we need to do during constraint solving.
There are still improvements that can be made here, but this is
currently passing tests and showing benefit in cases where we have
overloads taking concrete types, so it's worth enabling and then
incrementally improving.
Despite dramatically speeding up some test cases, because this is a
prepass it is going to result in a marginal slow-down in type checking
of some code. The standard library build gets approximately 15% slower
on my machine with this enabled.

Replace this paragraph with a description of your changes and rationale. Provide links to external references/discussions if appropriate.

Resolves SR-NNNN.

@swift-ci Please smoke test

@swift-ci Please test source compatibility

@swift-ci Please smoke test

@swift-ci Please smoke test

@swift-ci Please test source compatibility

@swift-ci Please test source compatibility

@swift-ci Please smoke test

@swift-ci Please test source compatibility

@swift-ci Please test source compatibility

There's little value in leaving this open. It's easy to enable this at some point if we can improve it to where it shows more value.


Awaiting 1/1 prerequisites of: get-history-archive-state

After I running a quick start version , it shown Awaiting 1/1 prerequisites of: get-history-archive-state issue(this msg shown in status after I use stellar-core -c info command ) , and can't sync the nodes . Is there some one can help me to fix that issue ?

I can confirm the issue. stellar-core is stuck in this cycle:

Awaiting checkpoint
Awaiting 1/1 prerequisites of: get-history-archive-state
Downloading ledger files
Connected
Catching up
Go to 1

This persists not only with docker-stellar-core-horizon but also with my custom configuration. It looks like there are some problems with http://history.stellar.org.

Seems is my configuration file issue , it supposed to create a history file , but it not . Here is my config file , I am wondering why it can't create the file as I wished .
[HISTORY.cache]
get="cp /opt/stellar/history-cache/{0} {1}"
mkdir="mkdir -p /opt/stellar/history-cache/{0}"
put="cp {0} /opt/stellar/history-cache/{1}"

#The history store of the Stellar testnet
[HISTORY.h1]
get="curl -sf http://history.stellar.org/prd/core-testnet/core_testnet_001/{0} -o {1}"

[HISTORY.h2]
get="curl -sf http://history.stellar.org/prd/core-testnet/core_testnet_002/{0} -o {1}"

[HISTORY.h3]
get="curl -sf http://history.stellar.org/prd/core-testnet/core_testnet_003/{0} -o

Finally, how to solve this problem?


parent is not reliable?

Who can tell the different between an element generated from selector/xpath and element generated from parent function? Here is an issue,
let parentEl = parent(element "#start-of-content")   //this works fine
let subEl = unreliableElementsWithin "//*[@id='noExist']"     //but when this returns 0 element, the parentEl will be massed up.
Please give this a try and let me know whether it happens.

Looking at the code, parent is reliable.

You can write an xpath selector that is equivalent to your first line.  id('start-of-content')/..
Then you could write a selector for the second like this id('start-of-content')/..//id('noExist').
You can test these selectors in chrome dev console by using the $x function like this $x("id('start-of-content')/..") and $x("id('start-of-content')/..//id('noExist')")
Hope this helps!

Hi Chris, the issue happens regardless which web address. The example I give to you it is this web page. Just get a parent element and save it in variable, and then unreliableElementsWithin find some element, when you get 0 element, the parent element variable will be changed. But when it returns some elements, then no issue.  And it probably is the selenium issue. I guess parent function using some selenium function to get parent. And it is the root issue. I will close this ticket for now.

It may not be an issue with Selenium.  I will try to reproduce. It will be a few days though, I am traveling multiple times over the next few days.

No worry, I already change to other way dealing my use case.
And even you get parent by FindElement(By.XPath(".."), issue is same.
The parent element looks like this,

After running


jsdom should return real charset

Now jsdom returned just string "UTF-8" without detect real charset. It`s a bug or you have reasons for this? Can i make PR to fix it?

We don't support encodings, so chances are if you have a document in jsdom, it's encoded in UTF8.

Not always. If page have tag <meta http-equiv="Content-Type" content="text/html; charset=windows-1251">, then encoding is windows-1251, not UTF-8.

Right, but we don't support parsing it using that encoding, so it is utf-8 regardless.

So, maybe it makes sense to add a non-standard api (or another temporary package) to get real encoding? In any case, it is very important begavior, that should be written in readme.

What do you mean "the real encoding"? The real encoding is UTF8 for any JavaScript strings (basically) so any jsdom you construct will always be utf8.

Technically UTF-16, but yeah.

Well, technically UCS-2. But for the purposes of text-manipulation APIs on the web, UTF-8.
In reality all the meta charset does is change how the browser interprets byte streams into strings (whether they be JavaScript strings in jsdom, or C++ strings in Chrome, or Rust strings in Servo). In jsdom there's no way to feed it a byte stream so meta charset is not really relevant anyway.

Can i make PR to fix it?

Unfortunately, it's much more complcated than that. Document encoding calculated from different sources: content-type HTTP-header, BOM, parent document encoding (for iframes), <meta> tags.

Here is the complete algorithm if you mind to elaborate: https://html.spec.whatwg.org/multipage/syntax.html#determining-the-character-encoding

We'd also need to introduce some way of feeding jsdom a byte string instead of a string, so that decoding even is meaningful.


How to add more tags?

I tried editing config.def.h and adding tags to the variable *tags, but each time i open a window in a different tag, dwm crashes. Help please.

https://github.com/siduck/chadwm/issues/78

basically u just need to define more colors in tagschemes

ohh thx sorry


Is there a Enable Hours, Minutes function ?

I want to enable only a range of hours(9 - 17) and Minutes(00, 10, 25, 45, etc). Is there a function to do so?
setHoursDisabled and setMinutesDisabled seem too much of a hassle in this case.

How exactly would you like the signature of the methods to look like?
So far I think you can only go with

https://github.com/smalot/bootstrap-datetimepicker#setminutesdisabled
https://github.com/smalot/bootstrap-datetimepicker#sethoursdisabled