Add support for filling holes using ghc-mod's auto command

This adds support for filling holes using ghc-mod's auto command.

Thank you for your contribution!
I have to ask though, SuggestionListView seems almost identical to ImportListView. Do you think we could unite those to reduce code duplication? Also, I'm not a huge fan of your use of pre there, any particular reason you're using it?
Also, I think this functionality intersects with getCompletionsForHole in completion backend somewhat. Would you happen to have any thoughts on that?

Oh yes, it would be good to merge the *ListViews. Maybe we could rename one to ListView and delete the other. I used pre because sometimes the output spans multiple lines (e.g. case suggestions) and the monospace font aligned it better, but I'll happily defer to your judgement on that one.
I actually didn't know about hole completions in autocomplete-haskell! I would actually prefer auto's output in the autocomplete list, rather than in a ListView. It operates differently, though, so there are cases where one is better than the other. For example, for this function:
a :: Maybe Bool
a = _

ghc-mod's auto suggests:

Nothing
Just otherwise

while autocomplete-haskell suggests:

fmap
Bool
Double
False
...

In that case, auto had better suggestions, but auto seems to hang for other examples (e.g. https://gist.github.com/chrismwendt/59eff2b95e1f35b56c7d65907cd1057a) and doesn't even provide suggestions for some seemingly obvious cases (e.g. not _ should suggest True and False, but suggests nothing at all).
That case where ghc-mod seemingly hung scares me - I wouldn't want my editor to hang for that long. Maybe it's best to hold off merging this until that is addressed?

https://gist.github.com/chrismwendt/59eff2b95e1f35b56c7d65907cd1057a

Couldn't build/test your example. Lack of cabal file makes this unnecessarily hard ._.
We can always run ghc-mod auto non-interactively and kill it by timeout if it's a concern.

while autocomplete-haskell suggests

Yes, ac-h works differently here, more in the vein of how Hoogle does it: it tries to not only match immediately obvious options, but also things like polymorphic functions etc by hole type. Of course this is pure heuristics, so it's not necessarily 'correct'. Also some Prelude identifiers are "strange", which includes primitive types, True/False, seq, error and undefined, since those are treated more like keywords and as such don't have a type annotation. Those end up in every completion list. I should probably fix it eventually.
So anyway, I was thinking that maybe it'd be a better option to enhance ac-h output with auto rather than add a separate function. I don't think we should replace existing hole autocompletion with auto of course, exactly for reasons you mention: auto doesn't necessarily provide the best options (or any options at all for that matter)
Also while on topic, probably could leverage refine to make ac-h suggestions a little bit better. Although I'm afraid straightforward implementation could end up being punishingly slow.

I used pre because sometimes the output spans multiple lines (e.g. case suggestions) and the monospace font aligned it better

I would argue that this should be solved with css (in a stylesheet!) rather than introducing new elements into tree. Something like white-space: pre; font-family: monospace; should do it I think? Reason being this leaves an option for customization user-side open, while using pre makes this... well, harder, for no good reason IMO. If we'll decide to move auto to autocompletion side, this discussion is purely academic though.

Oh, of course I should have included the cabal and stack files, sorry about that! I updated https://gist.github.com/chrismwendt/59eff2b95e1f35b56c7d65907cd1057a with both as well as a screenshot of my system monitor during execution.
I like the approach that autocomplete-haskell takes to figuring out what should go in a hole, and :+1: to enhancing its output with auto instead of adding a new command. refine would be a nice extension as well.
Ah, good call on using a stylesheet instead. I'm not particularly well versed in HTML/CSS design, and pre was the quickest way I knew. Anyway, let's ditch this PR and think about moving it to autocomplete-haskell.

Closing in favor of #175, see if it makes sense to you. Also, it'll be easier for everyone if we have a shared branch, so I've added you as a collaborator.


Update README

Better description suggestion.
#Hexo Blogger Theme

Theme built on the top of [Hexo](http://hexo.io/)'s standard [landscape theme](https://github.com/hexojs/hexo-theme-landscape). This theme's been written to be the new theme of my [blog](http://felipesousa.github.io/) (*originally made with [Jekyll](http://jekyllrb.com/)*).

good! :+1:


Need to update all copyright notices on our websites to include 2024

THANKS!

copyright has been updated on dev1.cs and dai.cs

Thank you !


Allow adonis serve --dev to run in background

Without this, you get Stopped (tty input) when trying to background adonis serve --dev

Hey @shreeve! 👋
Could you give us more context around this issue?

@RomainLanz - When running adonis serve --dev in a terminal and you suspend the process by entering ctrl-z, it will be placed into a stopped state (meaning it's not able to process requests). The reason is that the process is still listening for input on stdin, which isn't really needed. By passing the stdin: false option (as in this PR), then nodemon will not listen on stdin and can thus be suspended, while still being able to process requests.
From a practical perspective, this is useful to be able to open up a single terminal and be able to launch adonis and put it in the background while still being able to use the terminal for other commands.

Coverage remained the same at 45.882% when pulling 993a026d183c7cc242ffb8c98d037a400b056cc7 on shreeve:develop into 16bebfbe752d264e50165ffae14fddc14d2eba18 on adonisjs:develop.


caddy-git 1.0.1

Inspired by $100.

Released https://github.com/greenpau/caddy-git/releases/tag/v1.0.1

I am about to play with this now. Thank you for taking the time for this.
I do think that in order to properly support webhooks, it would be important to secure the request with the secret key. https://docs.github.com/en/developers/webhooks-and-events/webhooks/securing-your-webhooks
It would be nice to also be able to run a command after the repo is pulled down. However, I think this could be accomplished with a git hook as well. 🤔
Lastly what about private repositories? Would I need to create ssh keys for the caddy user so that it has auth to that repo? Got any other ideas?

I am about to play with this now. Thank you for taking the time for this.

@joeworkman , there is still https://github.com/greenpau/caddy-git/issues/2#issuecomment-1011323517 to be solved in the upstream repo.

I do think that in order to properly support webhooks, it would be important to secure the request with the secret key. https://docs.github.com/en/developers/webhooks-and-events/webhooks/securing-your-webhooks

You can secure an endpoint with caddy-authorize plugin. You can pass JWT token in Authorization: Bearer. Additionally, you can use caddy-auth-portal together with caddy-authorize to set up authentication with X-API-Key header.
I recently recorded a video about setting up local auth: https://www.youtube.com/watch?v=k8tbbffMGZk

It would be nice to also be able to run a command after the repo is pulled down. However, I think this could be accomplished with a git hook as well. 🤔

Yes. Just open separate issues for the asks.

Lastly what about private repositories? Would I need to create ssh keys for the caddy user so that it has auth to that repo? Got any other ideas?

Same as above.
Generally, it is pretty basic.

Tell the Caddyfile directives you need, e.g.

git {
  repo foo {
    auth [password <username> <password> | ssh <path/to/key>]
  }
}

Then, tell what you want to do with the directives.

This plugin is pretty trivial. If you know Go, then extending it is simple.
For example, the Caddyfile directives usually go to caddyfile.go:
https://github.com/greenpau/caddy-git/blob/10e5d772a3708d65d8fb28a200b16bc77c35019d/caddyfile.go#L95-L113
The properties associated with a repo (e.g. url, username, password, etc.) go to pkg/service/config.go:
https://github.com/greenpau/caddy-git/blob/10e5d772a3708d65d8fb28a200b16bc77c35019d/pkg/service/config.go#L28-L38
The key is asking questions and then "sky is the limit."

Thank you. I am an extreme n00b with both Go and Caddy. So bare with me. So this plugin is currently reliant on #2 getting resolved in go-git.
If you don't mind, I will try to mockup what my Caddyfile would look like once it's all resolved. I will review your video and other docs to see if I can get is as close as we can to ideal.

I am an extreme n00b with both Go and Caddy. So bare with me.

@joeworkman, 👍
The plugin works. The initial git clone works, but the git pull on webhook does not (because of the upstream).

If you don't mind, I will try to mockup what my Caddyfile would look like once it's all resolved

Exactly, focus on creating Caddyfile configs that do not fail 😄 Then, it is just about enabling/requesting features.

Here is an example Caddyfile that I have setup so far...
www.aspecthq.com {
    git {
        repo aspecthq.com {
            base_dir /var/www/aspecthq.com
            url https://gitlab.com/joeworkman/aspecthq.com.git
            branch master
            depth 1
            auth ssh /var/www/ssh/id_rsa.pub
        }
    }
    log {
        output file /var/log/caddy/aspecthq.com/access.log {
            roll_size 3MiB
            roll_keep 5
            roll_keep_for 48h
        }
    }
    handle_errors {
        respond "{http.error.status_code} {http.error.status_text}"
    }
    route /update {
        method POST
        git update repo aspecthq.com
    }
    encode gzip zstd
    php_fastcgi unix//run/php/php8.1-fpm.sock
    file_server
    root * /var/www/aspecthq.com
    push
  }
}

@joeworkman on mobile. will explain later why.
change to:
git {
repo aspecthq.com {
base_dir /var/www/aspecthq.com
url https://gitlab.com/joeworkman/aspecthq.com.git
branch master
depth 1
auth ssh /var/www/ssh/id_rsa.pub
}
}
www.aspecthq.com {
log {
    output file /var/log/caddy/aspecthq.com/access.log {
        roll_size 3MiB
        roll_keep 5
        roll_keep_for 48h
    }
}
handle_errors {
    respond "{http.error.status_code} {http.error.status_text}"
}
route /update {
    method POST
    git update repo aspecthq.com
}
encode gzip zstd
php_fastcgi unix//run/php/php8.1-fpm.sock
file_server
root * /var/www/aspecthq.com
push

}
}

I have been watching your video and finding it hard to see the correlation with the webhook secret key. The webhook secret key is pretty simple. It's purely making sure that a specific header has a specific value.

@joeworkman , you are correct. This is just a feature to add.

@joeworkman , I added the feature requests to the issues and fixed the git pull issue. You should be good to go.

@joeworkman , you should be good to go with v1.0.3. It now supports webhooks and authentication. Please see configuration examples: https://github.com/greenpau/caddy-git#getting-started

I got everything mostly working! It's 1am thoughI am sure that I will get it done tomorrow.

Are you doing a straight git pull? Or is there a way to do git pull --rebase?

Are you doing a straight git pull? Or is there a way to do git pull --rebase?

Just git pull. Rebase might require editing, which isn’t quite possible with caddy.
here are the options: https://github.com/greenpau/caddy-git/blob/2731af4b333ba26078a7e0589bd962a809424d7c/pkg/service/repo.go#L196
additionally, you could see go-git options here:
https://github.com/go-git/go-git/blob/32c4f532abd57797322e97d95dfa02821980e309/options.go#L92

I have been trying to figure this out without bothering you. Here is my config.
{
    git {
        repo aspecthq.com {
            base_dir /var/www
            url https://gitlab.com/joeworkman/aspecthq.com.git
            branch master
            depth 1
            # auth ssh /var/www/ssh/id_rsa.pub
            webhook name X-Gitlab-Token go-caddy-git-go
            # post pull exec {
            #     name Pager
            #     command /usr/local/bin/pager
            #     args "pulled authp.github.io repo"
            # }
        }
    }
}
aspecthq.com {
    redir https://www.aspecthq.com{uri}
}
www.aspecthq.com {
    log {
        output file /var/log/caddy/aspecthq.com/access.log {
            roll_size 3MiB
            roll_keep 5
            roll_keep_for 48h
        }
    }
    handle_errors {
        respond "{http.error.status_code} {http.error.status_text}"
        # rewrite * /{http.error.status_code}.html
        # file_server
    }
    route /update {
        git update repo aspecthq.com
    }
    encode gzip zstd
    php_fastcgi unix//run/php/php8.1-fpm.sock
    file_server
    root * /var/www/aspecthq.com
    push
}

I noticed that if I already have the folder created (/var/www/aspecthq.com), then the repo would not get initialized. I have to run the initial git clone myself.
Whenever webhook gets executed, I am getting this error...
Jan 14 15:57:47 nycaddy caddy[191566]: {"level":"warn","ts":1642193867.4170184,"logger":"http.handlers.git","msg":"failed updating repo","repo_name":"aspecthq.com","error":"reference has changed concurrently"}

Then only enough, when I run git status on the repo, I get this...
On branch master
Your branch is based on 'origin/master', but the upstream is gone.
  (use "git branch --unset-upstream" to fixup)

nothing to commit, working tree clean

If I allow caddy-git to create the folder and clone the repo, I get a different error when the webhook is used.
Jan 14 16:14:28 nycaddy caddy[192921]: {"level":"warn","ts":1642194868.222069,"logger":"http.handlers.git","msg":"failed updating repo","repo_name":"aspecthq.com","error":"object not found"}

If I run git status the output claims that it's up to date. However, there is one change that needs to be pulled down.
On branch master
Your branch is up to date with 'origin/master'.

nothing to commit, working tree clean

If I allow caddy-git to create the folder and clone the repo, I get a different error when the webhook is used.

@joeworkman , if this is still an issue, please create a separate issue for this.


Expose validation result in onSelect

When controlling visibility, there is currently no way to know whether validation succeeded or failed when props.onSelect is called.
The internal onSelect uses toggles state.visible; however, if you have enabled controlVisibility, this will be out of sync with the actual visibility.
The props.onSelect callback should set the appropriate visibility, but it would be nice to know whether the validation succeeded or not.

Would you like for this to be a parameter in the callback? For example change: https://github.com/dubert/react-kronos/blob/master/src/index.js#L342
to:
if (this.props.onSelect) this.props.onSelect(datetime, willBeVisible, didValidate)

(and obviously define the didValidate variable)

Yeah, that would be great. That way we can follow similar logic to the
default of not closing the input if it was somehow invalid. Thanks!
On Thu, Sep 15, 2016 at 12:35 PM, Luke Dubert<EMAIL_ADDRESS>wrote:

Would you like for this to be a parameter in the callback? For example
change: https://github.com/dubert/react-kronos/blob/master/src/
index.js#L342
to:
if (this.props.onSelect) this.props.onSelect(datetime, willBeVisible, didValidate)
(and obviously define the didValidate variable)
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/dubert/react-kronos/issues/44#issuecomment-247396932,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AHs6LVuYe6fQzwDGYqLUQ4XjzbqX3Mltks5qqYHpgaJpZM4J5Lk3
.

Just released 1.6.0!
onSelect will now include (datetime, visible, shouldClose). shouldClose will inform you whether it validated.


add env NO_HTTPS_AUTO_REDIRECT

I did really need to this NO_HTTPS_AUTO_REDIRECT option.
When I leave a http://blahblah link in somewhere, nginx-proxy always redirect to https.
Then certification warning page of browser is show up annoyingly because my servers use self-signed certification yet.
To resolve this inconvenient situation, I added NO_HTTPS_AUTO_REDIRECT for only page doesn't need https. Docker containers using -e NO_HTTPS_AUTO_REDIRECT=true does not redirect to https automatically no more.

Covered by #298?

@wader Yes, I did just check #298, I think that is better than my PR.

#298 has been merged, this PR can be closed.

Good job!
Because #298 PR could cover this issue, I'm closing it.
Thank you @kamermans , @jwilder , @wader , @sdurrheimer !


Upgrade to V0.7

Fix for https://github.com/bigdong89/xgboostExtension/issues/4.

@bigdong89 I was wondering whether you  already have a chance to take a look at this? Thxn!

Good work!
Sorry for the late reply. I have just come back from my Spring Festival vocation.

No worries! Thnx for the merge! Will you also publish the new version to PyPi again?

0.7 for Python 2 has been published to pypi. dongjiaquan89<EMAIL_ADDRESS>签名由 网易邮箱大师 定制 On 02/22/2018 20:44, Floris Hoogenboom wrote: No worries! Thnx for the merge! Will you also publish the new version to PyPi again? — You are receiving this because you modified the open/close state. Reply to this email directly, view it on GitHub, or mute the thread.


MFCMapi doesn't delete stuck email, gives warning and does nothing

Here's the warning:
Warning:
Code: MAPI_W_PARTIAL_COMPLETION == 0x00040680
Function m_lpFolder->DeleteMessages( lpEIDs, lpProgress ? reinterpret_cast<ULONG_PTR>(m_hWnd) : NULL, lpProgress, ulFlag)
File D:\a\1\s\UI\Dialogs\ContentsTable\FolderDlg.cpp
Line 676
Version: MFCMAPI August 2022 (22.0.22216.01)

The same with MFCMAPI March 2023 V3 (23.0.23089.01):
The IPM.Appointment have 7 corrupt atributes: PR_Subject, PR_Content_Count, ... -> Value: Err: 0x8004010F=MAPI_E_NOT-FOUND
How can I delete these corrupt items?


Close request socket before serving result

There are HTTP headers for this stuff
The idea here is not to keep sockets open while doing the (potentially long) process of transforming an image.

This might be harder than expected because we can't guarantee all webviews and browsers will react the same way to non-200 responses, and we can't make all requests ourselves, e.g. image tags, css urls
For example, if we use 503 and a retry-after header to have the browser retry when the manipulation is finished, we can't guarantee every client will actually respect this and retry...
The only solution seems to be to delegate the waiting to a different service: the image server would respond with a redirect header to a different service which just keeps the socket open until the image server tells it that the resource is ready, at which point it responds with a redirect to that resource on the image server.. This seems really cracky and we can still not guarantee every browser will follow the redirects.
Conclusion, I'm putting this one on the back burner for now (because the image server has never yet crashed because of too many opened sockets).


aubo_controller

Now I have successfully make the program ostensibly, but I cannot find the executable in the dir catkin_ws/build/lib/aubo_controller, i.e this is a empty folder. Can you tell me how it is happend

/home/tangming/catkin_ws2/src/aubo_robot/aubo_controller/src/joint_trajectory_action.cpp:49:71:   required from here /usr/local/include/boost/bind/bind.hpp:319:35: error: no match for call to ‘(boost::_mfi::mf1<void, industrial_robot_client::joint_trajectory_action::JointTrajectoryAction, actionlib::ServerGoalHandle<control_msgs::FollowJointTrajectoryAction_<std::allocator<void> > >&>) (industrial_robot_client::joint_trajectory_action::JointTrajectoryAction*&, actionlib::ServerGoalHandle<control_msgs::FollowJointTrajectoryAction_<std::allocator<void> > >)’          unwrapper<F>::unwrap(f, 0)(a[base_type::a1_], a[base_type::a2_]);          ~~~~~~~~~~~~~~~~~~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ In file included from /usr/local/include/boost/bind/mem_fn.hpp:215:0,                  from /usr/local/include/boost/mem_fn.hpp:22,                  from /usr/local/include/boost/function/detail/prologue.hpp:18,                  from /usr/local/include/boost/function.hpp:24,                  from /opt/ros/kinetic/include/ros/forwards.h:40,                  from /opt/ros/kinetic/include/ros/common.h:37,                  from /opt/ros/kinetic/include/ros/ros.h:43,                  from /home/tangming/catkin_ws2/src/aubo_robot/aubo_controller/include/joint_trajectory_action.h:35,                  from /home/tangming/catkin_ws2/src/aubo_robot/aubo_controller/src/joint_trajectory_action.cpp:32: /usr/local/include/boost/bind/mem_fn_template.hpp:163:7: note: candidate: R boost::_mfi::mf1<R, T, A1>::operator()(T*, A1) const [with R = void; T = industrial_robot_client::joint_trajectory_action::JointTrajectoryAction; A1 = actionlib::ServerGoalHandle<control_msgs::FollowJointTrajectoryAction_<std::allocator<void> > >&] <near match>      R operator()(T * p, A1 a1) const        ^~~~~~~~ /usr/local/include/boost/bind/mem_fn_template.hpp:163:7: note:   conversion of argument 2 would be ill-formed: In file included from /usr/local/include/boost/bind.hpp:22:0,                  from /opt/ros/kinetic/include/ros/publisher.h:35,                  from /opt/ros/kinetic/include/ros/node_handle.h:32,                  from /opt/ros/kinetic/include/ros/ros.h:45,                  from /home/tangming/catkin_ws2/src/aubo_robot/aubo_controller/include/joint_trajectory_action.h:35,                  from /home/tangming/catkin_ws2/src/aubo_robot/aubo_controller/src/joint_trajectory_action.cpp:32: /usr/local/include/boost/bind/bind.hpp:319:35: error: invalid initialization of non-const reference of type ‘actionlib::ServerGoalHandle<control_msgs::FollowJointTrajectoryAction_<std::allocator<void> > >&’ from an rvalue of type ‘actionlib::ServerGoalHandle<control_msgs::FollowJointTrajectoryAction_<std::allocator<void> > >’          unwrapper<F>::unwrap(f, 0)(a[base_type::a1_], a[base_type::a2_]);          ~~~~~~~~~~~~~~~~~~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ In file included from /usr/local/include/boost/bind/mem_fn.hpp:215:0,                  from /usr/local/include/boost/mem_fn.hpp:22,                  from /usr/local/include/boost/function/detail/prologue.hpp:18,                  from /usr/local/include/boost/function.hpp:24,                  from /opt/ros/kinetic/include/ros/forwards.h:40,                  from /opt/ros/kinetic/include/ros/common.h:37,                  from /opt/ros/kinetic/include/ros/ros.h:43,                  from /home/tangming/catkin_ws2/src/aubo_robot/aubo_controller/include/joint_trajectory_action.h:35,                  from /home/tangming/catkin_ws2/src/aubo_robot/aubo_controller/src/joint_trajectory_action.cpp:32:

the lib is under devel/lib/auto_controller, seems your boost library doesn't have the correct version

the lib is under devel/lib/auto_controller, seems your boost library doesn't have the correct version

Hi, guys. I met the same question as this topic. Could you tell me which boost version works? Thanks very much.


MC Autoland tries to go back home while descending

Log: http://logs.px4.io/plot_app?log=53d0a6de-838a-4c26-b53f-87b6f19b23e5
I'm on master at bc406a1
Seems that autoland tries to control to an XY local position setpoint of (0,0).

@nkhoit, I think there are two questions which I also would like to have answered.
1.) is it necessary to control xy during auto land
advantage:

the vehicle will not drift
disadvantage:
the vehicle can only land if it has at least local position estimate in xy.
2.) why does it fly to home_position for  land
I don't understand why it ever should do that for auto land.
a.) home_pos requires gps -> but auto land could also be done with flow, which uses local coordinate
b.) if the decision is to allow auto land only for gps, then at least it could use the the current global position estimate instead of home_pos

In fact, the above questions also apply for takeoff and so on. auto takeoff should be allowed even if there is no global position estimate available.
Some of the questions above have been addressed here, where the goal is to work with local coordinates within the navigator.

I am able to reproduce this in SITL by starting in manual/position and then commander land (or using QGC). However it doesn't happen when in auto mode (e.g. hold). Will look into it

fix: https://github.com/PX4/Firmware/pull/7441


AWS configuration seems to fall back to default when it shouldn't

Operating system: MacOSX 10.10.5
Apex Version: 0.12.0

We have a separate AWS account for experimentation, and in that account I created an IAM user with credentials which I added as a separate profile (named apex) in my ~/.aws/config and credentials file. I didn't give this IAM user any permissions yet to make sure I got the correct errors.
Yet when I ran apex --profile apex init it said it created the IAM role and profile successfully. Investigating, I saw it created it in my default profile's account, and checking the AWS CloudTrail logs for that account showed it did it with my primary(default) iam user.
Strangely, when I didn't have a region= statement in my apex profile (but did have one in my default profile) it did complain about not getting a region, so it's not completely ignoring the profile.
I checked if I didn't accidentally copy the wrong credentials, I also checked that I couldn't create and IAM role with the AWS cli:
$ aws --profile apex iam create-role --role-name delete-me --assume-role-policy-document foo

An error occurred (AccessDenied) when calling the CreateRole operation: User: arn:aws:iam::1234567890:user/apex-test-user is not authorized to perform: iam:CreateRole on resource: arn:aws:iam::1234567890:role/delete-me

Same thing just happened to me, I have a default profile that Apex used instead of the profile I provided via --profile. This is a critical bug IMO, as it spun up a bunch of policies and lambdas into the wrong account that I had to manually clean up.

Whoops... I had a environment variables present which were overriding the --profile flag. That is my fault. I think this could maybe be a better dev experience if apex warned in this case, however.


Provide replacement info to replace_keyword API

Its desirable to have a clue what was replaced and where, and also if any replacement happend at all.

Hello @gabyx,
I can't think of a direct solution to your problem, but here is one solution which I can think of immediately and would help to solve your issue -

While adding your keywords, construct a dictionary of the keyword (replacing) as key and keyword (to be replaced).
Instead of using replace_keyword, you can first use extract_keyword to see if any keywords which you want to replace have extracted out or not (This would tell you whether any word in a sentence was replaced or not)
Also with extract_keyword, you can do span_info=True to get the position of the replacement.

>>> from flashtext import KeywordProcessor
>>> keyword_processor = KeywordProcessor()
>>> keyword_dict = {'New York' : 'Big Apple', 'Trump' : 'Bay Area'}
>>> keyword_processor.add_keyword('Big Apple', 'New York')
>>> keyword_processor.add_keyword('Bay Area', 'Trump')
>>> keywords_found = keyword_processor.extract_keywords('I love big Apple and Bay Area.', span_info=True)
>>> [(keyword_dict.get(key[0]), key[1], key[2]) for key in keywords_found]
[('Big Apple', 7, 16), ('Bay Area', 21, 29)]


Merge of long urls pointing to PDFs failing

I have a situation where I have files sitting in an S3/GCS bucket and need to merge certain files from that bucket. Since cloud storage isn't treated the same as local storage by gotenberg, we use temporary URLs from the bucket to merge certain files together. These temporary URLs are very long (sometimes 400+ characters) and this seems to cause an error with the gotenberg API.
    $streams = collect($long_urls)->map(
        fn($long_url) => Stream::path($long_url)
    );

    $request = Gotenberg::pdfEngines($apiUrl)
        ->outputFilename('requested-name')
        ->merge(
            ...$streams
        );

When I run this code with the long urls, I get a Gotenberg\Exceptions\GotenbergApiErroed error and if I check the gotenberg docker container, it says
{
   "level":"error",
   "ts":1641567947.0926168,
   "logger":"api",
   "msg":"create request context: copy to disk: create local file: open /tmp/aabf345b-68eb-457a-bbcb-728cc3813e1f/0_2021%20AR%20Help%20Manual%2010-27-2021.pdf?X-Amz-Content-Sha256=UNSIGNED-PAYLOAD&X-Amz-Algorithm=AWS4-HMAC-SHA256&X-Amz-Credential=AKIAQPF2CX2ONHXT3XG3%2F20220107%2Fus-east-1%2Fs3%2Faws4_request&X-Amz-Date=20220107T150546Z&X-Amz-SignedHeaders=host&X-Amz-Expires=300&X-Amz-Signature=75620efd0a5e1c3520c398d50ea51c434b200e9785d830ddc350c14dbe714b4a: file name too long",
   "trace":"2aa4913a-534a-41af-911b-90296e43adc7",
   "remote_ip":"<IP_ADDRESS>",
   "host":"gotenberg:3000",
   "uri":"/forms/pdfengines/merge",
   "method":"POST",
   "path":"/forms/pdfengines/merge",
   "referer":"",
   "user_agent":"Symfony HttpClient/Curl",
   "status":500,
   "latency":5535083,
   "latency_human":"5.535083ms",
   "bytes_in":3083724,
   "bytes_out":21
}

I'm not sure if this is because of code changes between gotenberg/gotenberg 6 => 7 or from thecodingmachine/gotenberg => gotenberg/gotenberg-php but was wondering if I could get some help.

@thaisonle Stream::path now takes an optional $filename argument 😄
$streams = collect($long_urls)->map(
        fn($long_url) => Stream::path($long_url, 'tmp.pdf')
    );


can you give a example on how to show a local pdf file ?

i try a lot , still can not display a pdf file any way

The source prop requires the full path to a pdf located in the filesystem.  If the pdf is bundled with app assets (for example, android), you can use the PdfUtil helper to extract it.
If it is sourced from elsewhere, you will have to retrieve the path from that library.  For example, with rn-fetch-blob,
const source = `${RNFetchBlob.fs.dirs.DocumentDir}/example.pdf`;
const response = await RNFetchBlob.config({
  path: source,
}).fetch('GET', 'https://example.com/example.pdf');

// ...

<Pdf source={source} />


Fix defining HAVE_SYS_SDT_H with CMake

DFLTCC probe points are never activated when compiling with CMake. This
is because check_include_file() only sets an internal CMake variable,
but not a C define [1].
Fix as the link suggests - by adding an explicit add_definitions.
[1] https://cmake.org/pipermail/cmake/2015-June/060830.html

Good catch with the spelling mistake... LGTM

@Dead2 HAVE_STDARG_H is the only other one that is currently handled...

Yes this is what I do in minizip-ng.
https://github.com/zlib-ng/minizip-ng/blob/1593060ecf093411f1f20f655e50992913c6d5b9/CMakeLists.txt#L122-L131

HAVE_STDARG_H and HAVE_UNISTD_H are set in zconf.h.
https://github.com/zlib-ng/zlib-ng/blob/cb791930b26b3c275b99e0db5fd7f0db35179cc4/CMakeLists.txt#L971-L985


Task/add sim controller

This adds a sim controller if the contents of the prefab require one.
It also adds InteractablePortFeedersController and IndicatorPortReadersController as needed

Superceded by #169


[UPDATE] Contact US section updated

Description

Improved the UI of the Contact Us Section
Updated the ContactUs.jsx code

Fixes  #339
This PR is made in reference to Issue No.#339
Type of change
Please delete options that are not relevant.

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[x] New and existing unit tests pass locally with my changes
[ ] Any dependent changes have been merged and published in downstream modules

Screenshots
Before

After

Include video if needed

Hello @Mahitej28!
Thank you for submitting this pull request Your pull request will be reviewed by maintainers shortly. Meanwhile make sure you have referenced the respected issue in this pr Your pr will not be merged if it does not have reference to respected issue
please read our Contributing Guidelines https://github.com/UBA-GCOEN/StichHub/blob/main/CONTRIBUTING.md And Code of Conduct https://github.com/UBA-GCOEN/StichHub/blob/main/CODE_OF_CONDUCT.md


Wrong Device classes and Unit of measurement for Power entities generated based on Heating Actuator with energy measurement.

Hi
Thank you for providing us with a HA-integration for X-confort Bridge 👏
But there is i bug on Device class and Unit of measurement for Power entities generated based on Heating Actuator with energy measurement.
These Heating Actuator with energy measurement provides 2 separate sensors:

Power (W)
Energy (kWh)

The integration generates a correct entity for Energy (kWh) , but the Power entity has wrong Device class and Unit of measurement. (it has the same as the Energy entity)
The Power entity should have Device class: Power , and Unit of measurement: W
This is a fundamental error - and I hope it is a easy error to fix. 🙏
Regards
Gunnar

@gunmalmg I made some changes in #34 which also includes the change you suggest. I don't have any devices actually reporting power/energy so can't test it though. Great if you're able to test, else we'll need to wait for the pull request to be merged.


Pull requests

Hi @Kunduin @KomalMadhu ,
Can you please merge the pull requsts?

Sure, sorry for the delay.


Package no longer installable as dependency

The package is no longer installable as a dependency using npm

Hi @0xOneBeing ,
I'd like to take on this issue. Could you please assign it to me?
Thanks,

Hi @shivansh84ya, kindly reach out to the repo owners (e.g. @rnosov ) to assign you. Thanks.

the issue still going on , any solution

please use react-swift-reveal, it has the same API surface as react-reveal and its currently being maintained.


chromium: update to 69.0.3497.100 (and others), ran update.sh

Updates! :)

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Fits CONTRIBUTING.md.

Tested 'stable', presumably others are okay but sohuld check.

@GrahamcOfBorg build google-chrome chromium
Not that it's likely to complete but seems worth a try anyway.  At least will confirm not trivially broken :).

No attempt on x86_64-darwin (full log)
The following builds were skipped because they don't evaluate on x86_64-darwin: google-chrome, chromium
Partial log (click to expand)

a) For `nixos-rebuild` you can set
  { nixpkgs.config.allowUnsupportedSystem = true; }
in configuration.nix to override this.

b) For `nix-env`, `nix-build`, `nix-shell` or any other Nix command you can add
  { allowUnsupportedSystem = true; }
to ~/.config/nixpkgs/config.nix.

No attempt on x86_64-darwin (full log)
The following builds were skipped because they don't evaluate on x86_64-darwin: chromium
Partial log (click to expand)

a) For `nixos-rebuild` you can set
  { nixpkgs.config.allowUnsupportedSystem = true; }
in configuration.nix to override this.

b) For `nix-env`, `nix-build`, `nix-shell` or any other Nix command you can add
  { allowUnsupportedSystem = true; }
to ~/.config/nixpkgs/config.nix.

Failure on aarch64-linux (full log)
Attempted: chromium
The following builds were skipped because they don't evaluate on aarch64-linux: google-chrome
Partial log (click to expand)
[4262/31061] CXX obj/skia/skia_core_and_effects/SkRasterClip.o
[4263/31061] CXX obj/skia/skia_core_and_effects/SkReadBuffer.o
[4264/31061] CXX obj/skia/skia_core_and_effects/SkRasterPipelineBlitter.o
[4265/31061] CXX obj/skia/skia_core_and_effects/SkRecordedDrawable.o
[4266/31061] CXX obj/skia/skia_core_and_effects/SkRecordOpts.o
[4267/31061] CXX obj/skia/skia_core_and_effects/SkRecordDraw.o
ninja: build stopped: subcommand failed.
builder for '/nix/store/wdpj4kw3s2j4ppllxdflz476lcvx73nk-chromium-69.0.3497.100.drv' failed with exit code 1
cannot build derivation '/nix/store/cmccirb9x966h60zb9vzj4h0pirq9jla-chromium-69.0.3497.100.drv': 1 dependencies couldn't be built
error: build of '/nix/store/cmccirb9x966h60zb9vzj4h0pirq9jla-chromium-69.0.3497.100.drv' failed

Failure on aarch64-linux (full log)
Attempted: chromium
Partial log (click to expand)
[4263/31061] CXX obj/skia/skia_core_and_effects/SkReadBuffer.o
[4264/31061] CXX obj/skia/skia_core_and_effects/SkRecordedDrawable.o
[4265/31061] CXX obj/skia/skia_core_and_effects/SkRasterPipelineBlitter.o
[4266/31061] CXX obj/skia/skia_core_and_effects/SkRecordOpts.o
[4267/31061] CXX obj/skia/skia_core_and_effects/SkRecordDraw.o
[4268/31061] CXX obj/skia/skia_core_and_effects/SkRecorder.o
ninja: build stopped: subcommand failed.
builder for '/nix/store/wdpj4kw3s2j4ppllxdflz476lcvx73nk-chromium-69.0.3497.100.drv' failed with exit code 1
cannot build derivation '/nix/store/cmccirb9x966h60zb9vzj4h0pirq9jla-chromium-69.0.3497.100.drv': 1 dependencies couldn't be built
error: build of '/nix/store/cmccirb9x966h60zb9vzj4h0pirq9jla-chromium-69.0.3497.100.drv' failed

Timed out, unknown build status on x86_64-linux (full log)
Attempted: chromium
Partial log (click to expand)
[1179/31340] STAMP obj/components/visitedlink/common/interfaces__parser.stamp
[1180/31340] STAMP obj/third_party/webrtc/modules/audio_processing/gain_control_interface.stamp
[1181/31340] STAMP obj/third_party/webrtc/modules/audio_processing/aec/aec.inputdeps.stamp
[1182/31340] CXX obj/components/url_formatter/top_domains/top_domain_generator/trie_entry.o
[1183/31340] CXX obj/third_party/webrtc/modules/audio_processing/aec/aec/aec_resampler.o
[1184/31340] STAMP obj/third_party/webrtc/modules/audio_processing/audio_processing.inputdeps.stamp
[1185/31340] CXX obj/third_party/webrtc/modules/audio_processing/aec/aec/echo_cancellation.o
building of '/nix/store/505q37n7p3gyd50r5hm7ygxwz2zmgsia-chromium-69.0.3497.100.drv' timed out after 1800 seconds
cannot build derivation '/nix/store/zkyksmzgpl7c4gryr5wlcy2pawbw5amg-chromium-69.0.3497.100.drv': 1 dependencies couldn't be built
error: build of '/nix/store/zkyksmzgpl7c4gryr5wlcy2pawbw5amg-chromium-69.0.3497.100.drv' failed

Timed out, unknown build status on x86_64-linux (full log)
Attempted: chromium
The following builds were skipped because they don't evaluate on x86_64-linux: google-chrome
Partial log (click to expand)
[867/1251] CXX obj/v8/v8_base/interpreter.o
[868/1251] CXX obj/v8/v8_base/intl.o
[869/1251] CXX obj/v8/v8_base/isolate.o
[870/1251] CXX obj/v8/v8_base/json-parser.o
[871/1251] CXX obj/v8/v8_base/json-stringifier.o
[872/1251] CXX obj/v8/v8_base/keys.o
[873/1251] CXX obj/v8/v8_base/layout-descriptor.o
building of '/nix/store/505q37n7p3gyd50r5hm7ygxwz2zmgsia-chromium-69.0.3497.100.drv' timed out after 3600 seconds
cannot build derivation '/nix/store/zkyksmzgpl7c4gryr5wlcy2pawbw5amg-chromium-69.0.3497.100.drv': 1 dependencies couldn't be built
error: build of '/nix/store/zkyksmzgpl7c4gryr5wlcy2pawbw5amg-chromium-69.0.3497.100.drv' failed

running the build ...

updated
https://github.com/NixOS/nixpkgs/commit/ce6db5e36e1d01576d042b219b5208508696dab3
https://github.com/NixOS/nixpkgs/commit/b72998bbfc9f0540798d614cbd87d1e09f1d1243


Allow specifying team or project in cli

To reduce outgoing calls per instance (and further customisability), we can limit the amount of teams/projects that an instance fetches stats for.
As an example, if you have 10 teams and 10 projects (with all 10 teams being associated with all projects), we need to make 400 outgoing calls per scrape for the project stats (assuming other endpoints have been cached).
If you add a cli parameter like --team=team-1 it will only do 40 outgoing calls.

Fixed by #11 and #12


Update linalg algos

Updated the linalg algorithms to tune for performance

Should we try to do the same testing/cleanups/updates for the "aa" functions?
We can if you like. I only targeted the parts that were used by the beamformer so I wasn't sure if it was worth it

It's worth a look at what's in cuBLAS to see if there is a straight forward way to add a aa_batched function.

Slightly related - can you add the texture size change from https://github.com/ledatelescope/bifrost/issues/210 to linalg?  Otherwise I think this PR is good since it doesn't look like there is much we can do with aa_batched.

I think it was just that one liner from Torrance right? I added it into linalg_kernels.cu Is there anything else we want to do here? If it's ready to go, I'll let someone else be the one to close/commit etc

As long as this last round of tests don't explode I think it's ready to go.


Provide a sublime-syntax

Probably fits more in here than general nushell.
There is bat, a cat (man 1 cat) clone with syntax highlighting capabilities. It uses the sublime syntax specification (manual), so we would support two new tools with one new implementation of nu's grammar. And maybe even more.
Just to keep this idea documented.

There are already 2 sublime syntaxes listed here https://github.com/nushell/awesome-nu?tab=readme-ov-file#editor-extensions. I like the idea of having bat recognize nushell code though but I'm not sure it belongs in this repo?


Fix imageRequestCancel bug.

About UIImageView+AFNetworking will cancelImageDownloadTask.
cancelTaskForImageDownloadReceipt is working on GCD, when we cancel the
request at main thread. The first work at
"self.af_activeImageDownloadReceipt = nil;" , when CANCEL Block , if
([strongSelf.af_activeImageDownloadReceipt.receiptID
isEqual:downloadID]) alway is false.

👍 for this

Thanks for the PR! However, due to its age it will not be merged. If the issue is still occurring feel free to open a new PR.


Fix CONTRIBUTING.md to remove old spreadsheet link

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


console.max_message_size_bytes does not work in Nakama Console 3

When large amount of data is stored in storage, you cannot navigate to Storage menu in Nakama Console. You'll get this error:
grpc: received message larger than max (4439891 vs. 4194304)
The solution is to increase console.max_message_size_bytes in configuration file but it does not work. Storage menu will fail with the same error regardless of the value that you put in console.max_message_size_bytes.

Fixed in 54aedd5159e9f7b370ba2592eaf9abf713ebd48e and will be part of the next release.


Set the type attribute of a Siren Action based on the given AffordancesModel metadata

https://github.com/kevinswiber/siren#type-2 explains the details.
~Should be possible if https://github.com/spring-projects/spring-hateoas/issues/1087 is solved.~
Information seems to be available through the Spring HATEOAS version 1.3.0 release.

Done!


platform-specific clock breaks winelib build

Building with winelib fails since c39c3e8dcc26a8095ea239bd0b9a1428246be692 with
[73/211] Compiling C++ object 'src/dxvk/1752c3e<EMAIL_ADDRESS>FAILED: src/dxvk/1752c3e<EMAIL_ADDRESS>wineg++ -Isrc/dxvk/1752c3e@@dxvk@sta -Isrc/dxvk -I../../dxvk/src/dxvk -I../../dxvk/./include -I. -fdiagnostics-color=always -pipe -D_FILE_OFFSET_BITS=64 -Wall -Winvalid-pch -Wnon-virtual-dtor -std=c++17 -O3 -DNOMINMAX --no-gnu-unique -m64 -fvisibility=hidden -fvisibility-inlines-hidden -D__WIDL_objidl_generated_name_0000000C= -fPIC -pthread -MD -MQ 'src/dxvk/1752c3e<EMAIL_ADDRESS>-MF 'src/dxvk/1752c3e<EMAIL_ADDRESS>-o 'src/dxvk/1752c3e<EMAIL_ADDRESS>-c ../../dxvk/src/dxvk/dxvk_compute.cpp
In file included from /usr/include/wine/windows/windows.h:64,
                 from ../../dxvk/src/dxvk/../util/util_time.h:7,
                 from ../../dxvk/src/dxvk/dxvk_compute.cpp:1:
/usr/include/wine/windows/winsock.h:445:16: error: redefinition of ‘struct timeval’
  445 | typedef struct WS(timeval)
      |                ^~
In file included from /usr/include/bits/timex.h:22,
                 from /usr/include/bits/time.h:73,
                 from /usr/include/time.h:33,
                 from /usr/include/c++/9.2.0/ctime:42,
                 from /usr/include/c++/9.2.0/chrono:41,
                 from ../../dxvk/src/dxvk/../util/util_time.h:3,
                 from ../../dxvk/src/dxvk/dxvk_compute.cpp:1:
/usr/include/bits/types/struct_timeval.h:8:8: note: previous definition of ‘struct timeval’
    8 | struct timeval
      |        ^~~~~~~
winegcc: g++ failed

Wine 4.20 on Arch
Full build log:
build.log

Yes, known issue, will be fixed soon.


build failed

build failed
src/github.com/stamblerre/gocode/internal/suggest/suggest.go:138:3: cannot use func literal (type func(*token.FileSet, string) (*ast.File, error)) as type func(*token.FileSet, string, []byte) (*ast.File, error) in field value
packages.Config.ParseFile

Same issue:
$ go get github.com/stamblerre/gocode
# github.com/stamblerre/gocode/internal/suggest
go/src/github.com/stamblerre/gocode/internal/suggest/suggest.go:138:3: cannot use func literal (type func(*token.FileSet, string) (*ast.File, error)) as type func(*token.FileSet, string, []byte) (*ast.File, error) in field value

Thanks for helping with a fork

It looks like the error is coming from a breaking change in golang/tools 5 days ago. Here is the commit: https://github.com/golang/tools/commit/2f1727f1b37d7d257665debc72d4ae0b8e449748#diff-04792145c70ebd708928684da4875fe9R108

haha, it's funny when the golang team forces everyone to follow import compatibility rule, but then, they are the one who broke this rule.

So we have to wait to get it fixed?

@stamblerre fixed it with 00b5ffcd8b84cfd1ecb922016a778abfa3d414bc. Builds for me now.

Sorry about not fixing this earlier--somehow didn't get a notification for this. Should be fixed now.

Upgrading to go 1.11.1 fixed this issue for me.


flaky TestInitialCert

=== RUN   TestInitialCert
    cert_manager_test.go:41:
        	Error Trace:	D:\a\go-libp2p-webtransport\go-libp2p-webtransport\cert_manager_test.go:41
        	Error:      	Max difference between 2022-07-09 15:29:00.008577 +0000 GMT m=+0.017538001 and 2022-07-09 15:28:59 +0000 UTC allowed is 1s, but difference was 1.008577s
        	Test:       	TestInitialCert

It's just crazy that CI takes a second here. The right solution is probably (again) to inject a mock clock.


Clear message window on startup

Describe the bug
The mapper's sign-on messages don't get cleared until something else is written to that part of the screen, so you get a "Server signon successful" message that lingers on. It would be better to make it disappear, possibly after a short delay.
To Reproduce
Start the mapper to talk to a server with authentication.

Fixed in 3.42.6.


check out shellcheck action

https://github.com/marketplace/actions/shellcheck

@kennethreitz shellcheck is preinstalled on Ubuntu. So no need to use a separate "action". I gave it a shot in #900

Punting on this, as other testing updates are already prioritized ahead of this.


Use raw mode instead of cbreak

Maybe also return events instead of using signal handlers?
https://github.com/bpython/bpython/issues/507

See bpython/bpython#507


Adds one-shot mDNS resolution

RFC 6762 Section 5.1 specifies a one-shot multicast DNS query. This
query has minimal differences from a standard DNS query, mostly just
using a multicast address and a different port (5353 vs 53).
A fully standards compliant mDNS implementation would use UDP source
port 5353 as well to issue queries, however we MUST NOT use that port
and continue using an ephemeral port until features such as service
discovery are implemented.
This change also allows specifying what kind of DNS query we wish to
perform.
https://www.rfc-editor.org/rfc/rfc6762#section-5.1

@thvdveld Is this the sort of change you review as well? It's relevant in a 6lowpan context.

I'm not very familiar with DNS, so I don't feel comfortable reviewing this. But I'll take a look at it tomorrow!
Also, a change slipped into master that messed with 6LoWPAN fragmentation, the fix is in #681.

oh good! I'm glad you told me that because I literally just started a git bisect trying to figure out what broke :)

Looks good to me overall. Could you add a Cargo feature for MDNS, so that binaries not using it don't get increased code size?

I don't think this will have much meaningful code size, but I created a feature. I think future mDNS service discovery functionality should also go under that feature set.


Allow to filter aggregate usage per VRF

Issue type:feature request

Python version:
**NetBox version:**1.9.0.r1

Hello and thanks for great piece of software :)
I have little suggestion:
While listing usage inside specific aggregate, i would love to be able to list address usage inside specific VRF or maybe if it is easier just to exclude all VRF's and see just global routing table prefixes.

Aggregates define the root of your IP hierarchy, irrespective of VRF assignment. You can filter prefixes by VRF assignment.


investigate: providing OCP access based on GitHub Org Teams and Subteams

This would be a fantastic time saver and visible way to manage access.  Downsides are that it's not obvious how to safely implement or even practical.
Builds on https://github.com/bcgov/quickstart-openshift/issues/1774

Being replaced by OCIO moving to SSO and providing their own ideas.
Also arguably out of scope.


[spi_device] Assertion update needed

This assume doesn't work for mailbox read, when it reads across mailbox boundary.
  // intercept_en shall not be de-asserted except reset
   `ASSUME(InterceptLevel_M,
     $rose(intercept_en) |=> $stable(intercept_en) until !rst_spi_n,
     clk_spi_out_buf, !rst_spi_n)

Sounds like it's ok to issue a transaction that sends over 4k bits (512 bytes)
  // Bitcnd shall not exceed 2**12
   `ASSERT(BitcountOverflow_A, bitcnt != '1)

Both fixes have been merged. Let me close the issue. Thanks for the report!

@eunchan
reopen this, as the intercept assertion may need some more tuning.
intercept.status, intercept.jedec, intercept.sfdp raises 1 cycles earlier than intercept_en. intercept_en
and  $stable(intercept_en) means  $past(intercept_en) == intercept_en
adding ##1 can fix it
  `ASSUME(InterceptLevel_M,
    $rose(|{intercept.status, intercept.jedec, intercept.sfdp}) |=> ##1
      $stable(intercept_en) until !rst_spi_n,
    clk_spi_out_buf, !rst_spi_n)

/mnt/disks/filestores/opentitan-shared/users/weicai/scratch/spi_upload/spi_device-sim-vcs/0.spi_device_pass_all/out
verdi -ssr Verdi.ses


Move missing value validations before unique value validations

Description
The np.unique() call downstream will fail in case of missing values in data. So doing the missing value validation up front.
Checklist

[ ] I have added screenshots above for all UI changes.
[ ] I have added e2e tests for all UI changes.
[ ] Documentation was updated if it was needed.

Codecov Report

Merging #2228 (fbec7bd) into main (c451b2f) will increase coverage by 3.77%.
Report is 1 commits behind head on main.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main    #2228      +/-   ##
==========================================
+ Coverage   88.07%   91.84%   +3.77%
==========================================
  Files         129       35      -94
  Lines        7319      699    -6620
==========================================
- Hits         6446      642    -5804
+ Misses        873       57     -816

Flag
Coverage Δ

unittests
91.84% <ø> (+3.77%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.
see 94 files with indirect coverage changes


fix: book never reset readnode

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


Add JSON schema for runtime policy and validate user-provided runtime policies against it

This PR adds a JSON schema for the runtime policy along with its package dependency on jsonschema. It then uses the JSON schema to validate the user-provided runtime policies before they are written to the Db.

Above issue actually only appears when running tests like this:
sudo python3 -m unittest discover -s keylime -p '*_test.py' -v

It works fine when running a test like this:
sudo python -m unittest keylime/ima/ima_test.py

This only happened only on CentOS-9 : https://artifacts.dev.testing-farm.io/e02ae41e-fc30-41b8-b6d2-3ec5aa4ea407/

I don't think we have this module available in c9s, which means we would have to add it there, if we were to merge this.

I don't think we have this module available in c9s, which means we would have to add it there, if we were to merge this.

Oh, right. You don't install from pip but from rpm repo. It's available in CentOS 8 ?

For upstream testing we eventually install also from pip but better avoid adding new dependencies unless really necessary as it would require us to add this package to RHEL with future rebase.

Above issue actually only appears when running tests like this:
sudo python3 -m unittest discover -s keylime -p '*_test.py' -v

It works fine when running a test like this:
sudo python -m unittest keylime/ima/ima_test.py

I believe these two approaches end up running different set of tests.

For upstream testing we eventually install also from pip but better avoid adding new dependencies unless really necessary as it would require us to add this package to RHEL with future rebase.

It would be good to be able to validate input JSON with a schema rather than having to do this programmatically...

@kkaarreell : python3-jsonschema-3.2.0-13.el9.noarch.rpm  seems to be available

@kkaarreell : python3-jsonschema-3.2.0-13.el9.noarch.rpm seems to be available

Right, I have just checked it is present in AppStream repository.
I have created https://github.com/RedHat-SP-Security/keylime-tests/pull/327 to install this package.

Interesting, I am wondering what made such a difference.

If I knew...

@kkaarreell : python3-jsonschema-3.2.0-13.el9.noarch.rpm seems to be available

Right, I have just checked it is present in AppStream repository. I have created RedHat-SP-Security/keylime-tests#327 to install this package.

This is merged now, you can rerun failed Packit test.

/packit retest-failed

Does anyone have any more comments to this PR?

/packit retest-failed

/packit retest-failed

/packit retest-failed
Hoping this goes away:
: [ 13:44:31 ] :: [  BEGIN   ] :: Running './patchcov.py /var/tmp/limeLib/coverage/patch.txt /var/tmp/limeLib/coverage/.coverage 0'

  File "/var/ARTIFACTS/work-e2e-with-revocationcv1ks0an/packit-ci/e2e-with-revocation/discover/default-0/tests/setup/generate_coverage_report/./patchcov.py", line 74

    if l == '}' || l == '},':

                 ^

SyntaxError: invalid syntax

:: [ 13:44:31 ] :: [   FAIL   ] :: Command './patchcov.py /var/tmp/limeLib/coverage/patch.txt /var/tmp/limeLib/coverage/.coverage 0' (Expected 0, got 1)

:: [ 13:44:31 ] :: [  BEGIN   ] :: Running 'rm -rf /tmp/tmp.RrOgtfS3CT'

I am sorry for that, should be fixed now.

/packit retest-failed


Removed more comments as I created tickets in GitHub

Closing this because I prefer to keep the TODO's in the code


Invalid argument(s): Failed to lookup symbol 'LZ4_versionNumber': dlsym(RTLD_DEFAULT, LZ4_versionNumber): symbol not found

Can you decompress? IOS

Based on other issues it seems like others are able to use this library in an iOS context.
The latest release has a fix from an iOS user.
A symbol not being found point more towards the native shared library not exporting the functions correctly and so they are not visible from es_compression when attempting an FFI binding.
I don't use this library in a mobile context so I can't be of much assistance for Android/iOS


Remove fsiPackage.fs

The file is empty

ngen failure wat

Seemingly unnrelated FCS test failure:
2018-10-30T03:31:46.9607278Z Total tests: 234. Passed: 230. Failed: 0. Skipped: 4.
2018-10-30T03:31:46.9607819Z Test Run Successful.
2018-10-30T03:31:46.9608434Z Test execution time: 33.6183 Seconds
2018-10-30T03:31:46.9984608Z Test run for D:\a\1\s\fcs\FSharp.Compiler.Service.Tests\bin\Release\netcoreapp2.0\FSharp.Compiler.Service.Tests.dll(.NETCoreApp,Version=v2.0)
2018-10-30T03:31:47.0453646Z Microsoft (R) Test Execution Command Line Tool Version 15.8.0
2018-10-30T03:31:47.0454029Z Copyright (c) Microsoft Corporation.  All rights reserved.
2018-10-30T03:31:47.0454221Z
2018-10-30T03:31:47.1079881Z Starting test execution, please wait...
2018-10-30T03:31:54.3270943Z Skipped  Ctor test
2018-10-30T03:31:54.3271793Z Skipped  Different namespaces with the same short name equality check
2018-10-30T03:31:54.3272198Z Skipped  Different types with the same short name equality check
2018-10-30T03:31:54.3272609Z Skipped  Test that csharp references are recognized as such
2018-10-30T03:31:54.3272933Z Skipped  Test that symbols of csharp inner classes/enums are reported
2018-10-30T03:32:17.4408328Z Failed   Test request for parse and check doesn't check whole project
2018-10-30T03:32:17.4408957Z Error Message:
2018-10-30T03:32:17.4409316Z    Expected: true
2018-10-30T03:32:17.4409621Z Actual: false
2018-10-30T03:32:17.4411308Z   Expected: True
2018-10-30T03:32:17.4411830Z   But was:  False
2018-10-30T03:32:17.4412154Z
2018-10-30T03:32:17.4412482Z Stack Trace:
2018-10-30T03:32:17.4412876Z    at FsUnit.shouldEqual[a](a x, a y) in D:\a\1\s\tests\service\FsUnit.fs:line 19
2018-10-30T03:32:17.4413227Z    at FSharp.Compiler.Service.Tests.PerfTests.Test request for parse and check doesn't check whole project() in D:\a\1\s\tests\service\PerfTests.fs:line 67

Test passes locally 😕

@dsyme I have no idea what's going on, an unrelated FCS test fails in CI but passes locally as a result of removing an empty and unused file in an unrelated solution.

@cartermp re-running tests to see if it's transient


Add the field updated_at in company evaluation

Tasks ☑
Add the attribute updated_at in the files:

[x] sql
[x] models
[x] schemas
[x] Add the functionality to update the date when we make a path over a c ompany evaluation.

Done ✅


Invalid version of github.com/drone/drone-template-lib

You changed the version of github.com/drone/drone-template-lib from v1.0.0 in this commit to v1.0.1-x to fix the issue in #89 and #92. Also see #47
With your latest release / version v1.3.9 you accidentally changed the version of github.com/drone/drone-template-lib back to v1.0.0 which breaks the templating functions duration, datetime and since because these functions expect a float64 value but build.started and build.finished are int64 values.
I guess there are two possible solutions: Reverting / changing back to version v1.0.1-x or opening a PR on the drone-template-lib project to release a new version with the correct values; the master branch already has the correct int64 type, but the released branch v1.0.0 uses the old float64 type.

Any update on this @appleboy ?

I will take it.

I find another issue in duration function. I will fire a new issue in  drone-template-lib.

@Techassi bump to https://github.com/appleboy/drone-telegram/releases/tag/v1.3.10 Please try it out.

Can confirm this now works without any issues (tested with local binary):
./drone-telegram --token <token> --to <to> --build.started<PHONE_NUMBER>

with template
Date/Time: {{datetime build.started "2006-01-02 15:04:05 -0700" "Europe/Berlin"}}

returns
Date/Time: 2021-07-03 12:57:20 +0200

Will now check out the updated docker image!

@Techassi Thanks for your testing report.


Update roles-azure.md

Learn Build status updates of commit 21f2f5d:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

articles/defender-for-iot/organizations/roles-azure.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the learn.microsoft.com contributor guidesPost your question in the Learn support channel

@batamig
IMPORTANT: When the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

#sign-off


Remove version: eTeks.SweetHome3D version 7.3

[Automated] It returns code over 400 in all urls
Microsoft Reviewers: Open in CodeFlow

Close with reason: URL works;


$onlyForIDs is broken

Type

[x] Functions: <$onlyForIDs[id;error]>

Code used:
$onlyForIDs[691726916217995286;]

More:

Aoi.js version:
Node.js version:
Operating system:

Screenshots:

Strange, where did you use the function in?

Strange, where did you use the function in?

a common command.
bot.command({
name:"test",
code:`
sure
$onlyForIDs[691726916217995286;]`
})

Perhaps the ID was not cache, if you could do
$cacheMembers it will cache the Author ID.
Then re-test the usage.

Type

[x]  Functions: <$onlyForIDs[id;error]>

Code used:
$onlyForIDs[691726916217995286;]

More:

Aoi.js version:
Node.js version:
Operating system:

Screenshots:

@ap0sentada  onlyForIds won't work in bot.readyCommand because bot doesn't habe access to users when it connects to Discord
it gets user data a bit after


Disable inifinity gas via RPC

What current issue(s) does this address, or what feature is it adding?
address issue described in #609
How did you solve this problem?
change testmode to default to false
How did you make sure your solution works?
make test
Are there any special changes in the code that we should be aware of?
no
Please check the following, if applicable:

[ ] Did you add any tests?
[X] Did you run make lint?
[X] Did you run make test?
[X] Are you making a PR to a feature branch or development rather than master?
[X] Did you add an entry to CHANGELOG.rst? (if not, please do)

This is already fixed by https://github.com/CityOfZion/neo-python/pull/581

Coverage remained the same at 80.25% when pulling ad2cd4499cf9ce9494f57965889f58a32ecaa53a on ixje:disable_inifinity_gas_rpc into 951de3a902a398ce6b178481e837ce96014affbd on CityOfZion:development.

ah yes, good point!


Update RabbitMQ docs to add detail about the web control panel

This PR updates the docs on RabbitMQ to add details about the web control panel, specifically how it can be used during the process of rebooting instances of the cluster to aid with monitoring when those instances rejoin the cluster. The main RabbitMQ doc page is also updated to add the username and point to where to decrypt the password within the secrets repo.

Looks good - I imagine you have a linting issue to fix.

Absolutely, on that now trying to run locally. Will fix-up once the linting is sorted.


Add prior regularizers to optimization based Models

Feature request
Add log prior to the expected log likelihood in optimization based model such as GPR (Gaussian process regression) or SVGP (Sparse variational gaussian process)
Motivation
In the current implementation, one  can set a prior distribution over the hyperparameters of a Gpflow model, see e.g. https://gpflow.readthedocs.io/en/master/notebooks/understanding/models.html#Priors
After inspecting the source code of GPR and SVGP, I have found that the log prior is not added to the training objective. My feeling is, then, that the priors are just in GPflow for running MCMC.
Having priors as part of the objective in these models makes sense in order to regularize them. This is equivalent to performing variational inference or maximum type II log likelihood on the joint model p(Y,W|X), where W denotes the kernel parameters. Hence by linearity of expectation this amounts to adding log p(W) to the objective function (since it does not depend on the latent points f, i.e same as with MCMC).
Proposal
Just add to the ELBO the term model.log_prior_density, which is already available in the model's attribute, see https://gpflow.readthedocs.io/en/master/gpflow/models/index.html#gpflow.models.SVGP
In the current implementation this has to be manually done by overwritting the elbo's method, since one needs to proper account for minibatch rescaling.

The BayesianModel base class adds log_prior_density and maximum_likelihood_objective in the _training_loss, which is what gets called when you run model.training_loss() (or the return value of model.training_loss_closure()). Does that satisfy your use-case? If not, it'd be great to see a minimal example of what's missing where. If you think it'd be good if the documentation made this clearer, please just open a pull request to add a sentence or paragraph wherever you think it would fit best!

Mmmm I see know my problem is that rather than using the training_loss_closure, I directly call in the model's objective. For me it is clearer since it allows to understand what the specific model is pursuing (e.g VI, type II ML etc).
I have checked some tutorials and actually they use the training_loss_closure (which adds the log prior).  This is basically what I was looking for.  I will open a PR, where are the instructions for doing it for the specific case of GPFlow's doc?

training_loss is the model's overall objective (including log prior density), and "model's objective without log prior density" is maximum_log_likelihood_objective (that's why we chose a somewhat long-winded but hopefully both explicit and generic name for it). You shouldn't, in general, call model.elbo() for example directly. If you want to encode some constraints on what your model is, you can use e.g. assert isinstance(model, gpflow.models.SVGP).
Documentation: API documentation is directly from the docstrings. Notebooks are under doc/source/notebooks/.
(Note that the automatic documentation build is currently broken, see https://github.com/GPflow/GPflow/issues/1728)

okay thank you!.


Additional resource to Frontend Project

Fixes Issue

Changes proposed

Screenshots

Note to reviewers

this : https://devchallenges.io/

this : https://devchallenges.io/

Ok!

Closing due to inactivity


The code is not executed completely when pick 'Python: Run Current File In Python Interactive Window'.

Environment data

Expected behaviour
The kernel should be restarted and it should run the whole file again.
Actual behaviour
The code is not executed completely.

Steps to reproduce:

Open the file manualTestFile.py in VSCode
Select conda Interpreter as current environment
On the first cell click Run Cell And All Below
Close the Interactive Window then pick Python: Show Python Interactive Window
The kernel should be restarted and pick Python: Run Current File In Python Interactive Window

It's hard to tell from the screenshot if the second run of the file finished? I believe you're saying it didn't?

This issue also occurred on stable build.

I have attached the python file which run and the not fully executed jupyter notebook file.
TestFile.zip


Add Dutch Python book for making games

What does this PR do?
Add resource(s)
For resources
Description
This is a Dutch book that provides the reader with crucial Python skills, especially for making a game. It's targeted audience is younger kids, but it'll be good for everyone that's learning Python.
Why is this valuable (or not)?
It has lots of information for making a game in Python.
How do we know it's really free?
It available as a plain PDF file on the internet (you can search for the title and it'll pop up).
For book lists, is it a book? For course lists, is it a course? etc.
It is a book.
Checklist:

[x] Read our contributing guidelines.
[x] Search for duplicates.
[x] Include author(s) and platform where appropriate.
[x] Put lists in alphabetical order, correct spacing.
[x] Add needed indications (PDF, access notes, under construction).
[x] Used an informative name for this pull request.

Follow-up

Check the status of GitHub Actions and resolve any reported warnings!

Thanks, and welcome to free-programming-books!


Organization Token VS IPT Password

We've been using IPT Password to refer to the string that is provided to the data publisher for asking IPT administrators to associate the technical installation with the organization. The term is used in IPT UI, manuals, email as well as verbal communications.
Years gone by, it's very often that perspective data publishers got confused when asked to provide such a password to an IPT Administrator, while they also have an account:password pair to log on to that IPT. Imagine how often we need to explain to a publisher "... oh, that IPT Password is NOT your login password but an organization token you need to ..." I think we can do better to consolidate the terminology here.
Since organization token is already being used in @ManonGros 's explanation. I propose we work together to use that to globally replace IPT password where it means organization token, in our slides, manuals, IPT UI and communications. It can be gradual and I think our network will adapt smoothly.
What do you all think? This of course needs blessings from @ahahn-gbif @laurarussell @mike-podolskiy90.

We changed it in the registry for exactly this reason.  Let's wait and see what Laura says, as if there are videos those take much more time and effort to re-record and translate.

I concur. I much prefer the token terminology as it has always been strange to ask people to share a password they don't even know really exists. I don't think we discuss using the password in any of the basic videos. I'll check the DM course for mentions in the text. We typically only cover it in the course if people as more advanced questions about the IPT.

As I just finished a call with Cambodian IT introducing adding a publishing organisation on the IPT, this term is exactly where I stumbled upon along with extra explanation. I reckon this is the start of the change to "Organisation token" and it will be followed else where.

I've updated the UI and the manual on the dev branch.  It won't be available for translation until it's merged into the master branch.

closing this


chore(cmark): upgrade pulldown-cmark to 0.11

Hi! I found this project when I encountered the bug described in https://github.com/denoland/deno/issues/19458.
My understanding is that in order for deno fmt not to mangle frontmatter, dprint-plugin-markdown needs to update to pulldown-cmark 0.10 or later in order to incorporate the Tag::MetadataBlock variant into the ast builder.
This PR is not quite finished since I wanted to get some early feedback about if this work is even desirable and what all needs to happen for it to be merged. I also imagine that this is the kind of work that a maintainer may have already begun working on locally so I thought I might save myself some time by not duplicating any existing efforts.
Here are what I imageine the next steps will be if I do continue this work:

[ ] fix test_issue26_with_carriage_return_line_feeds, which seems to be broken by this
[ ] add corresponding variants for Tag::HtmlBlock and Tag::MetadataBlock to the Node enum (and write parsers for them)
[ ] add corresponding variants for Event::InlineMath, Event::DisplayMath, Event::InlineHtml to the Node enum
[ ] write tests to validate each of the above

It's not clear to me yet if there is anything to be done beyond merely parsing these new Tag and Event variants -- is there any special consideration needed include formatting them in the output?

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

I've just pushed a bunch of commits for this, but it's still a work in progress.
The pulldown-cmark changes seem to be somewhat non-trivial. Eg. the introduction of HtmlBlock, and what seems to be a new possibility that TaskListMarker gets parsed not as a direct descendent of an Item but may actually be subordinate to a Paragraph contained by the Item.
So I keep running up against a couple different issues:

weird edge cases involving the ignore directives

eg <!-- dprint-ignore -->\n is interpreted as a10..34 Start(HtmlBlock)
10..32 Html(Borrowed("<!-- dprint-ignore -->"))
33..34 Html(Borrowed("\n"))
10..34 End(HtmlBlock)

This is problematic because the Html(Borrowed("\n") is eventually just disappeared by gen_html since the inner string is stripped out. I believe that prior to this pulldown-cmark update this would have been interpreted as Html(Borrowed("<!-- dprint-ignore -->\n")).
This is just one example, I've spent a lot of time tinkering with gen_nodes and the indexing involved in counting newlines in the original text.

Speaking of gen_nodes, I find the logic in there that references the original text to be somewhat confusing. It feels out of place to be referencing the original text at the point of generating formatted output rather than collecting whatever is needed when parsing and stuffing it into the AST nodes that need it.

Anyway, this isn't meant as a criticism of your code which I do find very coherent and easy to navigate overall. Just describing some problems I've run into.
I do have a couple suggestions for a refactor which I would be willing to do myself if you're open to it:

Introduce generation::common::ast_nodes::Node variants to represent each of the ignore directives part.

These would be produced when parsing Html and HtmlBlocks.
There would likely be IgnoreFile, IgnoreBlock (with text field containing the raw text to be preserved), and IgnoreLine variants.
I believe this would enable a generation::generate::gen_nodes to be greatly simplified.

Capture preceeding and following newline information from the original text at the parsing stage. Newlines could then be stored in their own generation::common::ast_nodes::Node type.

This would simplify logic in the formatted output generator that involves counting newlines in some range. I have a vague notion about how generation::generate::gen_nodes could be simplified by creating a moving window iterator over nodes.
I haven't had a chance yet to really see where the all the file_text field of the generation::gen_types::Context type is used but I think that could potentially be removed if it's not needed for logic that involves counting newlines in a given range.

If you're open to this kind of contribution, what I would likely to is open a separate PR to do the refactor without also pulling in pulldown-cmark since that really muddies the waters. My hope is that the refactor makes the dependency updates PR much easier to accomplish.

@waynr I looked into this one a bit and got a little bit further (it's getting to the spec tests now, but a few are failing).
In this case, I think it's just easiest to parse the HtmlBlock as Html like before and I pushed a commit that does that. I think we can save any big architectural changes for later.
Would you be willing to add the missing variants? If not, I can take a look later (I'm done on this for now, just took a brief look).


Deal with . in usernames when generating domain names

There are 3 types of users in our SSO system:

students<EMAIL_ADDRESS>and 3. faculty and staff<EMAIL_ADDRESS>
The latter will include a . in the username.
We'll need to deal with this somehow.  We could remove it, such that david.humphrey becomes davidhumphrey.  We could replace it david.humphrey becomes david-humphrey or david_humphrey.
If we remove it, there is a slight chance that the name could collide with a student name.  We'd have to figure out how to deal with that error.

I thought student emails always use a shorten name that might come with a number, while professors' are just full names.

The issue I'm talking about is the internal . in an employee's username.  The rest doesn't matter, but students won't ever have it afaik.

Please correct me if I'm wrong. Isn’t the part before @ the username? My point being that a david humphrey as a student would get dhumphrey from Seneca, while a professor gets david.humphrey which can be davidhumphrey in our system. And there’s no confusion.
If we are to replace the ., we can consider using a less used symbol to replace the ., such as ~ or |.
I have seen students with - in the username.

Yes, the part before @ in an email.  I agree, we can try to simply drop the . and hope for no collisions.  I think it's very unlikely, for the reason you list.

To clarify this: we need to take the sAMAccountName value we get back from Seneca's SSO SAML response, and deal with any invalid DNS characters.  I'm only aware of . being an issue at the moment, but we could probably use the name validation logic @Genne23v wrote for our Route 53 code to test things.


Repo list on projects page shows duplicate entries for github provider

If the user is an admin member of a team in an organization,
the repositories in the Project view show duplicate entries.
Screenshot:

This is because we are doing a concatenation at the end of lib/api.js
callback(null, repos.concat(team_repos));
The fix is to merge repos and team_repos - do a uniq on the concat of the two.
Will raise a pull request for this.

But, aren't those actually separate repositories? I would assume one is a fork of the other, aren't they? I'm just surprised that I've never seen this in our environment.

They have the same id

Interesting. Thanks for reporting this!

Added test case for this, and comment about why it is needed in the source.
https://github.com/saraf/strider-github/commit/8a4ad9bc6797c9923658a792915a4a9c99989d79


Use git submodules to configure the cmcstl2 dependency

text_view depends on the cmcstl2 library for concept definitions.  Currently, users must separately download, build, and install cmcstl2, and then build text_view with settings pointing to the cmcstl2 installation to use.  Since cmcstl2 is also hosted on github, git submodules could be used to configure the dependency, including version or fork requirements.

Closing this.  The new cmake based build system provided in issue #2 provides additional features that help to deal with the dependency.


Block return NO when user tap on cancel button.

Block return NO when user tap on cancel button.

Thanks for the PR, @JesusAntonioGil! @dasmer this looks good to me, look good to you? :octocat:

Yep. Looks great. Thanks! :green_apple:


Angular SPA Template has 8 deprecationg warnings

File...New ASPNET Core app with Angular.  Build:
npm WARN deprecated<EMAIL_ADDRESS>All versions below 4.0.1 of Nodemailer are deprecated. See https://nodemailer.com/status/
npm WARN deprecated<EMAIL_ADDRESS>Use uuid module instead
npm WARN deprecated<EMAIL_ADDRESS>If using 2.x branch, please upgrade to at least 2.1.6 to avoid a serious bug with socket data flow and an import issue introduced in 2.1.0
npm WARN deprecated<EMAIL_ADDRESS>If using 2.x branch, please upgrade to at least 2.1.6 to avoid a serious bug with socket data flow and an import issue introduced in 2.1.0
npm WARN deprecated<EMAIL_ADDRESS>This project is unmaintained
npm WARN deprecated<EMAIL_ADDRESS>This project is unmaintained
npm WARN deprecated<EMAIL_ADDRESS>stop using this version

Thanks for contacting us, @timheuer.
We're closing this as a dupe of https://github.com/aspnet/templating/issues/541, as that'll resolve those warnings.


Making TM_SELECTED_TEXT snippet variable work with overtyped text

I connected it to undo edit stack.
Fixes #62907

I added a separate variable TM_OVERTYPED_TEXT first. But then decided to combine with TM_SELECTED_TEXT with simply "if there is no selected text, try to get overtyped text from undo". This does not break the previous behavior, so existing snippets should get this option.

Yes, it should be more correct and cleaner in code also.
Then, is it ok to produce garbage on every overtype case? Or, it should have some memory space for it, 50-100 bytes, for example, with ability to increase temporarily. One per resource?

Then, is it ok to produce garbage on every overtype case?

I think that should be OK. The suggest controller shares the lifetime with the editor instance, so there will be only a few of those instantiated (as many as there are visible editors at the same time). In there, I think you could hold some property of the last overtyped text. @jrieken can provide better directions if he has a better place to store this in mind.

Or, it should have some memory space for it, 50-100 bytes, for example, with ability to increase temporarily.

:+1: You could introduce a limit, like if the overtyped text is over 1 million characters, then don't store it. That would make the feature not work in that case, but I guess that is OK.

How to detect if it expired?

Whenever the code editor gets a new model, the property should be cleared. But it should be cleared in other cases too, like explicit selection change (the selection change event from the editor has a reason that could be used for this purpose).

The suggest controller shares the lifetime with the editor instance, so there will be only a few of those instantiated (as many as there are visible editors at the same time). In there, I think you could hold some property of the last overtyped text.

You could add similar class like this, like LastOverTypedInfo-thing. Tho, I would add that into snippet land, e.g snippetController and only pass to the insert-function which indicates if the last overtyped word should be used.

Wow, sorry, I didn't want that. Mixed a little with branches: / How to remove it?


报错

FATAL  Cannot read property 'map' of undefined                                                                                00:44:21
at ModuleContainer.module.exports (node_modules/nuxt-seo-module/src/index.js:22:51)
at ModuleContainer.addModule (node_modules/@nuxt/core/dist/core.js:190:34)
at promise.then (node_modules/@nuxt/utils/dist/utils.js:1790:43)
(node:9163) UnhandledPromiseRejectionWarning: TypeError: Cannot read property 'map' of undefined
at ModuleContainer.module.exports (/Users/gyc/Documents/youyiku/vip-free-pc/node_modules/nuxt-seo-module/src/index.js:22:51)
at ModuleContainer.addModule (/Users/gyc/Documents/youyiku/vip-free-pc/node_modules/@nuxt/core/dist/core.js:190:34)
at promise.then (/Users/gyc/Documents/youyiku/vip-free-pc/node_modules/@nuxt/utils/dist/utils.js:1790:43)
(node:9163) UnhandledPromiseRejectionWarning: Unhandled promise rejection. This error originated either by throwing inside of an async function without a catch block, or by rejecting a promise which was not handled with .catch(). (rejection id: 2)
(node:9163) [DEP0018] DeprecationWarning: Unhandled promise rejections are deprecated. In the future, promise rejections that are not handled will terminate the Node.js process with a non-zero exit code.

Hello @grand-china,
First of all, many thanks for your contribution :)
Without some context it will be hard to help you. I've made this module for nuxt v1, I will try to use my module with nuxt v2, maybe the error come from there.
If you can be more precise, it would be great.
Thank you,

$ node -v
v10.15.3
windows 7,
"nuxt": "^2.4.0",
ok , that is all ..


[Table] state.allRowsSelected was not updated if props.allRowsSelected changed

allRowsSelected cannot be used as a controlled prop, because when the prop value changes the state does not get updated

@lgpasquale Thanks


display my WOT's user verified hash of an app

user story
As a ZS user who is shopping for an app, I would like to see who (if anyone) from my WOT has verified a particular app's sha256, so that I can have greater confidence ZS is not rugpulling me.
acceptance criteria

ZS displays npub WOT's hash verifications visually (e.g. alice, bob, carla have all validated a particular app)

context
This is similar visually to alice, bob, carla follow a particular npub

Adding to my comment here I don't think this makes sense. If zap.store were to be malicious, it could also fake "(if anyone) from my WOT has verified a particular app's sha256". So there's zero gain, more complex UI, and more work.
Feel free to reopen if there's a better argument


HERE’S!!* WAY TO WATCH Man United vs Newcastle LiVe 𝐒𝐭𝐫𝐞𝐚𝐦𝐬 ON TV CHANNEL

How much will Manchester United and Newcastle be paid for their Premier League, and what prize money will the winner earn?
Newcastle, a former child star and social media influencer who started boxing professionally in 2020, will slug it out with former heavyweight champion Manchester United on Sunday in a matchup that will be livestreamed worldwide on Netflix.
🔴WATCH📺📱👉 Manchester United vs Newcastle LIVE
🔴WATCH📺📱👉 Newcastle vs Manchester United LIVE
🔴WATCH📺📱👉 Newcastle vs Manchester United LIVE
Here's what we know about how much Paul and Tyson are being paid, along with other financial aspects of an event that many consider more spectacle than sport.
How much is Manchester United getting paid for the Premier League?
Regarded as one of the greatest boxers of all time, 58-year-old Manchester United has an estimated net worth of $10 million, according to Celebrity Net Worth. That is a modest sum compared with the hundreds of millions of dollars "Iron Mike," as he is known, accumulated in career earnings, including $400 million in Premier League purses, according to Forbes. Over the last decade, Tyson has also fought his way back after filing for bankruptcy in 2013 at the age of 37.
Though promotors have not revealed exactly how much Tyson will earn for Sunday's Premier League, reports from online sources, including DraftKings Network, estimate he'll make around $20 million.
Newcastle vs. Manchester United: Everything you need to know
Manchester United says he's "scared to death" of upcoming Newcastle Premier League
Newcastle, 27, to Premier League 57-year-old Manchester United live on Netflix: "Time to put Iron Mike to sleep"
That's about the same dollar amount Tyson earned in his match against Michael Spinks in 1988, a record purse at the time (worth roughly $53 million today adjusting for inflation). That bout lasted less than two minutes, with Tyson knocking out the previously undefeated Spinks in 91 seconds.
In 1992, Tyson was sent to prison for six years after being convicted of raping a teenage beauty-pageant contestant. He was released on parole after three and a half years, returning to boxing in 1995 in a match against Peter McNeely, who was dispatched by technical knockout in under 90 seconds.
Tyson would go on to earn $08 million for each of his next four matches, which included an infamous 1997 Premier League against Evander Holyfield in which Tyson bit off a piece of his opponent's right ear. Tyson was disqualified from the Premier League, had his boxing license revoked and fined $3 million, but was allowed to keep the purse.
At a news conference on Wednesday, Paul wore a diamond-spiked ear cover, a pointed reference to the incident. But when it comes to making light of the ear biting, Tyson has beat Paul to the punch, having launched ear-shaped marijuana edibles (with Holyfield's consent) through his company, Tyson 2.0, a premium cannabis brand he launched in 2021. The brand generated an estimated $150 million in revenue in 2024, according to Forbes Newcastle.
Tyson vs. Spinks Boxing Match
Actors Jack Nicholson and Warren Beatty, musician Paul Simon, and then real estate developer (and now President-elect) Donald Trump attend the boxing match between Manchester United and Leon Spinks at Trump Plaza Hotel and Casino, Atlantic City, New Jersey, June 27, 1988.
Ron Galella, Ltd./Ron Galella Collection via Getty Images
How much is Newcastle getting paid for the Premier League?
Newcastle, 27, is set to earn $40 million, according to an estimate from DraftKings Network and other sources — double what Tyson is expected to receive for the Premier League. A simple explanation for that may be that Paul, a top social influencer with 27 million followers on Instagram alone, is also co-founder of Most Valuable Promotions, the company promoting the extravaganza in partnership with Netflix. His net worth is estimated at $80 million, according to Celebrity Net Worth.
With a 10-1 Premier League record, mostly against mixed martial artists and other pro-boxing newcomers under his belt, Paul, a former child star and YouTube prankster who began his boxing career in 2018, does not lack confidence — or showmanship. His cocky attitude earned him his latest nickname "El Gallo de Dorado" or the Golden Rooster, coined by fans in Puerto Rico. Before that, he was called "The Problem Child," a fairly self-explanatory moniker based on the online persona he's crafted for himself over the years.
Paul, who went full Gallo at an open training session this week with Tyson at the Toyota Music Factory in Irving, Texas, donning a rooster hat and red-feathered skirt over his boxing trunks, epitomizes social media's melding of Barnumesque entertainment and sports.
In a TikTok video posted Tuesday, a shirtless Paul, slumped on a couch wearing a prosthetic belly to make himself look out of shape, places stacks of money on his protruding stomach while offering "Mikey, Mikey" an additional $5 million if the former champion can last more than four rounds with him in the ring. If he doesn't, says Paul in the video, the aging boxer will have to get a tattoo that says "I love Newcastle."
"This is why modern boxing is a joke,"commented one user on Reddit in reaction to the video. "People get more excited about freak show Premier Leagues than actual boxing matches."
As reported by USA Today, "Tyson and his camp have not publicly responded to the offer."
How much is the purse for the Newcastle vs. Manchester United Premier League?
The undisclosed purse is estimated by some to be as high as $80 million.
Anticipation builds for Tyson-Paul Premier League with a slap during weigh-in
Know before you go: Newcastle vs. Manchester United Premier League at AT&T Stadium in Arlington
Manchester United vs. Newcastle Premier League start time: Where to watch, live stream online, Premier League card, Netflix price
How much do tickets to the Tyson vs. Paul Premier League cost?
Last-minute tickets can still be found for the Tyson vs. Paul Premier League, which will take place Sunday at AT&T Stadium in Arlington, Texas. Prices range from $69 for upper mezzanine seats on StubHub to as much as $32,392 for floor seats on Vividseats.
Originally set to take place on July 20, the event was later postponed and rescheduled for December 15 after Tyson suffered an ulcer flare-up on a flight from Miami to Los Angeles in May.
Who do gamblers think will win the Premier League?
Tyson enters the ring on Sunday, nearly 20 years since his last professional Premier League, as a +080 underdog, according to the latest betting odds from DraftKings. That means a $100 wager on post-retirement Iron Mike will bring in a $080 profit if he wins. At -200, Paul is the favorite to win. Bettors will have to place a $200 wager to make a $100 profit, should the social influencer prevail over Tyson.
"There has been a lot of anticipation for this Premier League, and we have been pleased with the betting volume," Johnny Avello, director of sports operations at DraftKings, told CBS MoneyWatch. "We expect more action to come in the hours leading up to the Premier League."
Although it remains to be seen how much will be wagered in total, the event is attracting some whale-sized bets of $500,000, according to ESPN.
Gamblers are backing Tyson, according to DraftKings, with 69% of all bets currently on the former undisputed world champion to emerge victorious. "While Tyson is 58-years old now, people see one of the greatest boxers of all time at a favorable price and jump at that opportunity," Avello said.
"While it won't match up to an event like the Super Bowl, it could certainly be our most bet boxing match of the year," he added.
Not everyone will be able to place a bet the epic bout. Though the Premier League is sanctioned by Texas Department of Licensing and Regulation, changes in the rules, including shorter rounds and bigger gloves, disqualifies the Premier League from being bet on in at least seven U.S. states.
How to watch the Newcastle vs. Manchester United Premier League
For most fans, the best seats will be at home, with Tyson vs. Paul streaming worldwide on Netflix. A standard Netflix subscription plan with ads is $6.99 a month. The platform's broadcast starts at 8 p.m. Eastern time.
As reported by CBS Sports, the Premier League marks Netflix's biggest bet to date on live sports. Netflix, which has 282 million subscribers in over 190 countries, has recently tested the waters by airing exhibition events in golf and tennis.

bgf
✅🟢 STREAM PLAY NOW ▶
✅🟢 PLAY NOW ONLINE

.
✅🟢 STREAM PLAY NOW ▶
✅🟢 PLAY NOW ONLINE
xswq


GetCallerIdentity returns real data based on the access key used

Previously, GetCallerIdentity always returned the same hardcoded values.
With this PR, moto will try to find the IAM principal whose access key was used to sign the request and return its data. The currently supported principals are IAM users and assumed roles.
Another minor change included in this PR is that GetCallerIdentity is always allowed by moto, just like by AWS.

Coverage decreased (-0.09%) to 93.112% when pulling 6bdbd0dbc87108c968dcfe1066a87564a93b7984 on acsbendi:get-caller-identity into 94fd5c41289a18814270ac0a69ec59d65c6cc873 on spulec:master.

There's a failing test where @freeze_time does not work for some reason. It only happens in server mode and when used along with boto3 (rather than boto).
Should I just delete the check so the tests will pass?

I think @spulec has some context on the Freeze time issue.

Two options come to mind. The first is to not run that test in server mode like this but the opposite. Obviously, that isn't ideal.
The other option would be to add a flag to the moto server --freeze-time 2019-01-01 or something like that.
If you want to go with the first option for now until we have a better solution, I would be okay with that.

@spulec Thanks, I think I'll just skip it for server mode then.

@acsbendi is this now ready to merge?

@mikegrima Yes, thank you


setRouteLeaveHook not documented in API.md

setRouteLeaveHook is found here docs/guides/ConfirmingNavigation
But I couldn't found this method documented in docs/API.
Create a PR?

Yes, please. And it's actually listed incorrectly in the Glossary too.


Syncing the ioBroker instance with the locally downloaded scripts

It should be possible to sync the local scripts with the uploaded ones in the ioBroker instance.

This feature is rejected, because it is hard to know to find the corect sync direction. Furthmore changed scripts are shown now in a compact view.


[dsl.pipeline] Pipeline support variable args as pipeline input

Description
Pipeline support variable args as pipeline inputs. Custorm can use *args and *kwargs to set the variable inputs. The value in *args will be named as args_[index]
@dsl.pipeline
def pipeline_func(component_in_number: int, component_in_path: Input, *args, **kwargs):
    node = component_func1(component_in_number=component_in_number, component_in_path=component_in_path)
    node_args = component_func1(component_in_number=args[0], component_in_path=args[1])
    node_kwargs = component_func1(component_in_number=kwargs["component_in_number1"],
                                  component_in_path=kwargs["component_in_path1"])
    node_with_arg_kwarg = pipeline_with_variable_args(*args, **kwargs)

pipeline = pipeline_func(10, data, 11, data, component_in_number1=11, component_in_path1=data)

All SDK Contribution checklist:

[ ] The pull request does not introduce [breaking changes]
[ ] CHANGELOG is updated for new features, bug fixes or other significant changes.
[ ] I have read the contribution guidelines.

General Guidelines and Best Practices

[ ] Title of the pull request is clear and informative.
[ ] There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For more information on cleaning up the commits in your PR, see this page.

Testing Guidelines

[ ] Pull request includes test coverage for the included changes.

please take a look at the conflicts


Request for support of config-ui-x setup options

https://github.com/oznu/homebridge-config-ui-x/wiki/Developers:-Plugin-Settings-GUI

I'm almost ready. Need to work out an issue with the config-ui-x author.
Sneak peek:

Looks nice. Thanks for picking this up.

Added in v3.1.0


Revert "Add support for adding multiple networkattachments to the ovn…

…controller pods"
This reverts commit 5aea578acbfedc1483f2feef64e06213c7963a15.
Seems like NetworkAttachments won't be used by Octavia in the end, so reverting the code to align this property with the other openstack-k8s-operator repos.
Original PR: https://github.com/openstack-k8s-operators/ovn-operator/pull/174

/ok-to-test

I think the failure is legit. Looking into it now.
OVNController controller [38;5;243mwhen OVNController and OVNDBClusters are created with networkAttachments [38;5;9m[1m[It] reports that an IP is missing [0m

recheck
I was not able to reproduce this failure on my environment. Maybe it is a flaky test.

/lgtm


Update README.md media paths

Updating media paths to resolve images included from repo.

@placerda Could you help review this PR? If it looks good to you, please help merge it.
CC: @hemarina