FR: VBA version

I'm using this in an Excel workbook. I've converted the JavaScript version to a
Windows Scripting Component (WSC), which I call from Excel macros and use the
results to format cells.

This works, but our IT group is moving us to Windows 7, and now registering
components requires admin access. Major PITA. Any other solution (XLL, etc.)
would have the same problem.

So, I'd like to suggest a VBA version as a feature request. This would allow
use within Word, Excel, and Access without the need for an external library.
And of course anyone still using legacy ASP.

Converting from C# to VB.NET is easy (the one built into SharpDevelop IDE works
well), but moving it back to VBA from there is the hurdle to get over.

Original issue reported on code.google.com by<EMAIL_ADDRESS>on 10 Nov 2011 at 8:36

Hey - I recently had a similar problem - needed a string diff algorithm for VBA. Just spent ages and have finally found a solution.
The relatively simple workaround uses the javascript version of the code available in this package.
Basically I read in the javascript source code and append a couple of lines to calculate the string differencing result, then call a new javascript function (also appended to the end of the code) from VBA which returns this result back to VBA.
Note: you'll need to add a reference to Microsoft Script Control v1.0 in VBA
Public Sub x()
'Thanks to http://ramblings.mcpher.com/Home/excelquirks/snippets/scriptcontrol and Neil Fraser (Google)
    Dim z As Variant
    Dim o As New ScriptControl
    Dim str1 As String
    Dim str2 As String
    str1 = "This is one string"
    str2 = "This is a different string"
    o.Language = "JScript"
    With o
    'Performs diff of strings str1 and str2 - you'll need to change the filepath below
    'Remove line ["dmp.diff_cleanupSemantic(d);" & _] to not clean up the semantics
    'Got idea for this from source code of https://neil.fraser.name/software/diff_match_patch/svn/trunk/demos/demo_diff.html
        .AddCode ReadTxtFileToString("C:\diff_match_patch_20121119\javascript\diff_match_patch.js") & _
        "var dmp = new diff_match_patch();" & _
        "var d = dmp.diff_main('" & str1 & "', '" & str2 & "');" & _
        "dmp.diff_cleanupSemantic(d);" & _
        "function returnToVBA() {return d;}"
        z = .Run("returnToVBA")
        Debug.Print ""
    End With
End Sub

Function ReadTxtFileToString(path As String) As String
    Dim oFSO As FileSystemObject
    Dim tmpStr As String
    Set oFSO = New FileSystemObject
    Dim oFS As TextStream
    Set oFS = oFSO.OpenTextFile(path)
    tmpStr = oFS.ReadAll
    oFS.Close
    ReadTxtFileToString = tmpStr
End Function

Hope that helps someone!


grafana_server not being installed.

Bug Report
What happened: Running cockpit ceph installer and it said everything was good, but the grafana server was not installed
What you expected to happen: The grafana server installed with dashboards etc.
How to reproduce it (minimal and precise):
Rerun ansible-playbook
Share your group_vars files, inventory and full ceph-ansibe log
all.yml.txt
dashboards.yml.txt
mgrs.yml.txt
mons.yml.txt
osds.yml.txt
Environment:

OS (e.g. from /etc/os-release): Centos 7
hosts.txt

Kernel (e.g. uname -a): Linux star1 3.10.0-1160.15.2.el7.x86_64 #1 SMP Wed Feb 3 15:06:38 UTC 2021 x86_64 x86_64 x86_64 GNU/Linux

Docker version if applicable (e.g. docker version):

Ansible version (e.g. ansible-playbook --version): ansible-playbook 2.9.16

ceph-ansible version (e.g. git head or tag or stable branch): ceph-ansible-4.0.41-1.el7.noarch

Ceph version (e.g. ceph -v): ceph version 14.

2.16 (762032d6f509d5e7ee7dc008d80fe9c87086603c) nautilus (stable)
logit.txt

think you're missing the dashboard users:
#############
# DASHBOARD #
#############
dashboard_enabled: True
# Choose http or https
# For https, you should set dashboard.crt/key and grafana.crt/key
# If you define the dashboard_crt and dashboard_key variables, but leave them as '',
# then we will autogenerate a cert and keyfile
dashboard_protocol: http
dashboard_port: 8443
dashboard_admin_user: admin
dashboard_admin_user_ro: false```

grafana_admin_user: admin
This variable must be set with a strong custom password when dashboard_enabled is True
grafana_admin_password: admin

so does this mean I need to add:
dashboard_protocol: http
dashboard_port: 8443
dashboard_admin_user: admin
dashboard_admin_user_ro: false
to my all.yml file?
Does the admin user need to have an account on the dash server? Dash is the server I'm trying to deploy grafana too.
There is no .ansible directory on the dash server in the /root directory while there is on all of the osd's amd mons'.
Is it even talking to it?

When looking at the output I see:
PLAY [grafana-server] **********************************************************
TASK [set ceph grafana install 'In Progress'] **********************************
Tuesday 23 February 2021  06:27:58 -0500 (0:00:00.075)       0:04:37.250 ******
skipping: [dash]
TASK [ceph-facts : include facts.yml] ******************************************
Tuesday 23 February 2021  06:27:58 -0500 (0:00:00.062)       0:04:37.313 ******
skipping: [dash]
TASK [ceph-facts : set grafana_server_addr fact - ipv4] ************************
Tuesday 23 February 2021  06:27:58 -0500 (0:00:00.053)       0:04:37.367 ******
skipping: [dash]
TASK [ceph-facts : set grafana_server_addr fact - ipv6] ************************
Tuesday 23 February 2021  06:27:58 -0500 (0:00:00.076)       0:04:37.443 ******
skipping: [dash]
TASK [ceph-facts : set grafana_server_addrs fact - ipv4] ***********************
Tuesday 23 February 2021  06:27:58 -0500 (0:00:00.052)       0:04:37.496 ******
skipping: [dash] => (item=dash)
TASK [ceph-facts : set grafana_server_addrs fact - ipv6] ***********************
Tuesday 23 February 2021  06:27:58 -0500 (0:00:00.061)       0:04:37.558 ******
skipping: [dash] => (item=dash)
TASK [ceph-prometheus : create prometheus directories] *************************
Tuesday 23 February 2021  06:27:58 -0500 (0:00:00.067)       0:04:37.626 ******
skipping: [dash] => (item=/etc/prometheus)
skipping: [dash] => (item=/var/lib/prometheus)
TASK [ceph-prometheus : write prometheus config file] **************************
Tuesday 23 February 2021  06:27:58 -0500 (0:00:00.069)       0:04:37.696 ******
skipping: [dash]
TASK [ceph-prometheus : make sure the alerting rules directory exists] *********
Tuesday 23 February 2021  06:27:58 -0500 (0:00:00.053)       0:04:37.749 ******
skipping: [dash]
TASK [ceph-prometheus : copy alerting rules] ***********************************
Tuesday 23 February 2021  06:27:58 -0500 (0:00:00.054)       0:04:37.804 ******
skipping: [dash]
TASK [ceph-prometheus : create alertmanager directories] ***********************
Tuesday 23 February 2021  06:27:59 -0500 (0:00:00.053)       0:04:37.857 ******
skipping: [dash] => (item=/etc/alertmanager)
skipping: [dash] => (item=/var/lib/alertmanager)
TASK [ceph-prometheus : write alertmanager config file] ************************
Tuesday 23 February 2021  06:27:59 -0500 (0:00:00.071)       0:04:37.929 ******
skipping: [dash]
TASK [ceph-prometheus : include setup_container.yml] ***************************
Tuesday 23 February 2021  06:27:59 -0500 (0:00:00.057)       0:04:37.987 ******
skipping: [dash]
TASK [ceph-grafana : include setup_container.yml] ******************************
Tuesday 23 February 2021  06:27:59 -0500 (0:00:00.046)       0:04:38.033 ******
skipping: [dash]
TASK [ceph-grafana : include configure_grafana.yml] ****************************
Tuesday 23 February 2021  06:27:59 -0500 (0:00:00.055)       0:04:38.089 ******
skipping: [dash]
TASK [set ceph grafana install 'Complete'] *************************************
Tuesday 23 February 2021  06:27:59 -0500 (0:00:00.057)       0:04:38.147 ******
skipping: [dash]

https://github.com/ceph/ceph-ansible/blob/master/group_vars/all.yml.sample will show you the example under the dashboard section. I however have had no luck getting the dashboard installed.
Regards

@lphabc if your dashboard installation is entirely skipped, it's because either:

you have overridden dashboard_enabled and set it to false somewhere in your group_vars/host_vars/inventory
or
you don't have at least 1 node in your grafana-server group

see from the current code in stable-4.0:
- import_playbook: dashboard.yml
  when:
    - dashboard_enabled | bool
    - groups.get(grafana_server_group_name, []) | length > 0


How to increment major and minor version

Hey folk, I found your nice plugin and wanted to try it. After execution mvn ci-friendly-flatten:version using version 1.0.0 I noticed that it increments patch version to 1.0.1. I wonder how to increment major and minor versions. Checking the code I don't see it's possible to do (e.q. using a config)
https://github.com/outbrain/ci-friendly-flatten-maven-plugin/blob/e7d7b07ad9b059f834e3affa2e847dcf67bb247c/src/main/java/com/outbrain/ci/friendly/flatten/maven/plugin/VersionMojo.java#L120C18-L120C35
Thanks

Hi @maksim-wezaam
Thanks for your question .
Currently its not supported we just increment the minor version .
The workaround for this is - create new commit with new tag of the required version .
There are two options:

Support increment major and minor - which will require new flags to be provided
Provide the new version you would like to move for example from 1.0.0 to new version 5.0.0 - in this case it would be much more flexible .

Looking forward to hear back...

Hey @ayoukhananov, yeah I was thinking to have a param. I've tested it and created the MR https://github.com/outbrain/ci-friendly-flatten-maven-plugin/pull/57, please check.

@maksim-wezaam the most flexible solution is option 2 that @ayoukhananov suggested - provide the new version as a parameter (-DnewVersion=). Would that work for you?
If it would then I would prefer implementing that option.

That possible but in this case we have to manually detect the new version. It's ok for major versions because it's rare operation. Following to semantic versioning we should update MINOR version for a new feature and it's frequent operation. Having the param in the pom that I suggested we can define what we want to update and the plugin will do the job to automatically bump a desired version.
Of course we are humans and will forget to set it properly ))) To avoid that we were thinking to detect it from commit messages. I mean if we put a special keyword in a commit message then the plugin could parse it and define what to update. But again we are humans... Another way is to use branch names. If we use prefixes for branches like feature/ bugfix/ then it's easy to detect. I didn't check it yet if it's possible to implement.

I've integrated it into our pipeline, thanks!
Later, I will try to research how to detect a version based on the message or branch I mentioned above.


Adding description of unknown property name handling

Adding text as discussed in the Issue https://github.com/Materials-Consortia/OPTiMaDe/issues/92.

Edit: Also, @CasperWA is trying to push us all towards the formatting of one sentence per line in the raw

This will tend to give rather long lines which are not so comfortable...
What about folding the sentences to some reasonable width (80 symbols) and starting each sentence on a new line? I can probably even get an Emacs mode for this...

Edit: Also, @CasperWA is trying to push us all towards the formatting of one sentence per line in the raw

This will tend to give rather long lines which are not so comfortable...

It's much worse detecting minor changes in a long sentence (git-wise) if the sentence is split on several lines.

What about folding the sentences to some reasonable width (80 symbols) and starting each sentence on a new line? I can probably even get an Emacs mode for this...

Most editors should be capable of wrapping long single-lines already, I believe?
I feel like I must be adamant about this particular thing. It will significantly reduce minor errors and/or decrease time spent on PR reviews, since the diff for a change in every sentence will be clearly stated. So in my opinion the benefits far outweigh the drawbacks :)

It's much worse detecting minor changes in a long sentence (git-wise) if the sentence is split over several lines.

No really; actually line-wise diff (the usual vcs system default) will work better with folded lines. With 1 sentence/line we will have to resort to 'wdiff' (word diff) to see that actual changes.

What about folding the sentences to some reasonable width (80 symbols) and starting each sentence on a new line? I can probably even get an Emacs mode for this...

Most editors should be capable of wrapping long single-lines already, I believe?

They should :)

I feel like I must be adamant about this particular thing. It will significantly reduce minor errors and/or decrease time spent on PR reviews, since the diff for a change in every sentence will be clearly stated. So in my opinion the benefits far outweigh the drawbacks :)

OK, lets give it a try and see. Actually I was myself advocating 1 sentence/line policy, however not for manual editing but for automatic processing :). We can try and see if it works for editing as well.
But currently the text is in a mixed mode – some sentences are folded, and some long sentences start on a line after another long sentence. Will you fix that?

Edit: Also, @CasperWA is trying to push us all towards the formatting of one sentence per line in the raw

This will tend to give rather long lines which are not so comfortable...

It's much worse detecting minor changes in a long sentence (git-wise) if the sentence is split over several lines.

Actually, does Github fold long lines in diffs? As I currently see, long lines introduce a horizontal scroll-bar; If this is always the case, then I would say this is totally unusable and a complete show stopper for long lines.
Can you confirm that Github interface handles long lines decently?

Edit: Also, @CasperWA is trying to push us all towards the formatting of one sentence per line in the raw markdown. This has advantages when looking at text changes and for suggesting changes, etc. I think this is a good idea. (Which I think we should update the whole spec to when/if converting to RST)

BTW, should we open a separate issue on that and move the discussion there?

It's much worse detecting minor changes in a long sentence (git-wise) if the sentence is split over several lines.

No really; actually line-wise diff (the usual vcs system default) will work better with folded lines. With 1 sentence/line we will have to resort to 'wdiff' (word diff) to see that actual changes.

One more thought: it is indeed awkward to detect changes in a sentence when the whole paragraph gets refolded; changes, which are only cosmetic, can then propagate arbitrarily far.
But what I would suggest is a combination of a current approach: we fold lines in each sentence independently, and start each sentence on a separate line. In this way, when you change a sentence, you will only see the corresponding lines of this sentence changed. Easy to read, easy diff, easy to merge, no unnecessary merge conflicts. What you think?
E.g.:
folded sentences, each sentence starts on its own line (no scroll-bars on wide-enough displays):
Lorem ipsum dolor sit amet, consectetur adipiscing elit, sed do eiusmod tempor
incididunt ut labore et dolore magna aliqua.
Ut enim ad minim veniam, quis nostrud exercitation ullamco laboris nisi ut aliquip
ex ea commodo consequat.
Duis aute irure dolor in reprehenderit in voluptate velit esse cillum dolore eu
fugiat nulla pariatur.
Excepteur sint occaecat cupidatat non proident, sunt in culpa qui officia deserunt
mollit anim id est laborum.

 * Lorem ipsum dolor sit amet, consectetur adipiscing elit, sed do eiusmod tempor
   incididunt ut labore et dolore magna aliqua.

 * Ut enim ad minim veniam, quis nostrud exercitation ullamco laboris nisi ut aliquip
   ex ea commodo consequat.
   Duis aute irure dolor in reprehenderit in voluptate velit esse cillum dolore eu fugiat
   nulla pariatur.
   Excepteur sint occaecat cupidatat non proident, sunt in culpa qui officia deserunt
   mollit anim id est laborum.

long line sentences suggestion (inevitable scroll-bars in Github and some other tools):
Lorem ipsum dolor sit amet, consectetur adipiscing elit, sed do eiusmod tempor incididunt ut labore et dolore magna aliqua.
Ut enim ad minim veniam, quis nostrud exercitation ullamco laboris nisi ut aliquip ex ea commodo consequat.
Duis aute irure dolor in reprehenderit in voluptate velit esse cillum dolore eu fugiat nulla pariatur.
Excepteur sint occaecat cupidatat non proident, sunt in culpa qui officia deserunt mollit anim id est laborum.

 * Lorem ipsum dolor sit amet, consectetur adipiscing elit, sed do eiusmod tempor incididunt ut labore et dolore magna aliqua.

 * Ut enim ad minim veniam, quis nostrud exercitation ullamco laboris nisi ut aliquip ex ea commodo consequat.
Duis aute irure dolor in reprehenderit in voluptate velit esse cillum dolore eu fugiat nulla pariatur.
Excepteur sint occaecat cupidatat non proident, sunt in culpa qui officia deserunt mollit anim id est laborum.

Hi, folks, for some reason I can not find a text of any new reviews on the GitHub, so I found a button "re-request review" and clicked on it. If this does not appear on the Github Web, please e-mail me the links/text.


Teams specific examples would be good

Would be a good idea to add (or link to) some Teams specific examples here.

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 1170b28d-13ae-2e47-e14b-7cd9af0c61d3
Version Independent ID: 506a2fe7-420f-9b76-53db-c89f4a977849
Content: Implement channel-specific functionality - Bot Service
Content Source: articles/v4sdk/bot-builder-channeldata.md
Service: bot-service
Product: bot-framework
GitHub Login: @RobStand
Microsoft Alias: kamrani

@drub0y Can you provide a sample message (or other activity) from MS Teams that contains a sample entity?

Work in progress.
See PR: https://github.com/MicrosoftDocs/bot-docs-pr/pull/1697

Added the following Teams section: https://docs.microsoft.com/en-us/azure/bot-service/bot-builder-channeldata?view=azure-bot-service-4.0#adding-a-bot-to-teams.


Enlarge place bounding boxes to align with obscuration box edges

Currently, community-curated places are bounded by the smallest box that contains the entire place boundary (aka "place bounding box").
Obscured observations are publicly visible as located within a .2 degree by .2 degree box (aka "obscuration box").
If an observation location is obscured, it will only be recorded as within a community-curated place if the entire obscuration box is contained within the place bounding box.
This means that obscured observations located in obscuration boxes that cross place bounding boxes will not be recorded within those places. As a result, people may be confused as to why some of their observations are not included in the City Nature Challenge, for example.
Increasing the size of place bounding boxes by rounding out to the next .2 degree will ensure that the edges of place bounding boxes coincide with the edges of obscuration boxes. This will ensure that even obscured observations are included in places as expected.
This makes sense for large places (e.g. the CNC project boundaries like this one), but probably not for places below a certain size.
From talking with Ken-ichi, it doesn't sound like this will result in an false inclusion of obscured observations with true coordinates outside a given place boundary.
Open questions:
-How do we determine the threshold for which places should use boundary box rounding?
-In the short term, should we selectively enlarge the place bounding boxes of places associated with CNC projects? (is that even possible?)
-What possible unintended consequences could this have? e.g. will it cause a massive re-indexing of places?

I know you probably are already all discussing this but wasn't this done
specifically so that there wasn't the geoprivacy loophole where the
obscuring box and the place only share a small area? (or when the place is
tiny)? It seems mostly relevant for things that auto-obscure for collection
risk, but could come up in other cases too.
Don't need to reignite any obscuring discussion, mostly i am following this
because i have mixed feelings too. For instance, the Montpelier, VT place
is big and populated enough that most privacy issues that would come from
landowners aren't an issue. I obscure stuff in our yard but i don't care if
people know I live in Montpelier especially since that is public
information anyway as I am on the conservation commission. What might
matter is if a rare orchid is observed that only occurs in hemlock swamps,
for instance, because we only have a couple of small ones. But that aside
our bioblitz this summer is going to be a bit of a bummer if we can't have
obscured stuff from private land because none of them will show up as
entirely in the place.
On Fri, Apr 27, 2018 at 4:33 PM, carrieseltzer<EMAIL_ADDRESS>wrote:

Currently, community-curated places are bounded by the smallest box that
contains the entire place boundary (aka "place bounding box").
Obscured observations are publicly visible as located within a .2 degree
by .2 degree box (aka "obscuration box").
If an observation location is obscured, it will only be recorded as within
a community-curated place if the entire obscuration box is contained
within the place bounding box.
This means that obscured observations located in obscuration boxes that
cross place bounding boxes will not be recorded within those places. As
a result, people may be confused as to why some of their observations are
not included in the City Nature Challenge, for example.
Increasing the size of place bounding boxes by rounding out to the next .2
degree will ensure that the edges of place bounding boxes coincide with the
edges of obscuration boxes. This will ensure that even obscured
observations are included in places as expected.
This makes sense for large places (e.g. the CNC project boundaries like this
one https://www.inaturalist.org/observations?place_id=118683), but
probably not for places below a certain size.
From talking with Ken-ichi, it doesn't sound like this will result in an
false inclusion of obscured observations with true coordinates outside a
given place boundary.
Open questions:
-How do we determine the threshold for which places should use boundary
box rounding?
-In the short term, should we selectively enlarge the place bounding boxes
of places associated with CNC projects? (is that even possible?)
-What possible unintended consequences could this have? e.g. will it cause
a massive re-indexing of places?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/inaturalist/inaturalist/issues/1745, or mute the
thread
https://github.com/notifications/unsubscribe-auth/AG1tk4yiP5IcvMp68dYBmHgeSDG1ShkBks5ts4CNgaJpZM4Tq-bb
.

--
Charlie Hohn
Montpelier, Vermont

Your assessment of the current situation is correct, I think. In the proposal (if I'm understanding it correctly), Obs A would be returned in a search for observations in Place, but Obs B would not, b/c its private coordinates are not within the place boundary.

Ok - if it was how I understood it (where the proposed changes would return both Obs A and Obs B) then it makes sense to me if the decision was that we are more concerned about errors of omission (Obs A not being returned when it should) then errors of commission (Obs B being returned when it shouldn't).
But if its not how I understood it and the the change is to use the true private coords (ie return Obs A but not B) in the search then all this bounding box stuff just seems like a more complex way of just indicating which places should use private coords (alla admin level 0,1,2 places) and which should (alla all the rest) - where the bounding box rounding treatment would be the indicator of which places would get treated as admin level places and which wouldn't.
Am I correct that there's no difference in bounding box rounding vs just 'whitelisting' places to use private coords (alla admin level 0,1,2)? Or am I missing something?

I think you're right that my proposal is a more complex way to achieve the
same effect of 'whitelisting' places to use true locations for determining
observation inclusion.
On Tue, May 1, 2018 at 3:29 PM, Scott Loarie<EMAIL_ADDRESS>wrote:

Ok - if it was how I understood it (where the proposed changes would
return both Obs A and Obs B) then it makes sense to me if the decision was
that we are more concerned about errors of omission (Obs A not being
returned when it should) then errors of commission (Obs B being returned
when it shouldn't).
But if its not how I understood it and the the change is to use the true
private coords (ie return Obs A but not B) in the search then all this
bounding box stuff just seems like a more complex way of just indicating
which places should use private coords (alla admin level 0,1,2 places) and
which should (alla all the rest) - where the bounding box rounding
treatment would be the indicator of which places would get treated as admin
level places and which wouldn't.
Am I correct that there's no difference in bounding box rounding vs just
'whitelisting' places to use private coords (alla admin level 0,1,2)? Or am
I missing something?
[image: screen shot 2018-05-01 at 12 22 54 pm]
https://user-images.githubusercontent.com/598026/39489435-64b8319a-4d3a-11e8-8ad5-f931e5c7d3ac.png
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/inaturalist/inaturalist/issues/1745#issuecomment-385765340,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AHJ1f1XcDxBqlQZK8uTKgDJp4haL41VWks5tuLewgaJpZM4Tq-bb
.

--
Carrie E. Seltzer, Ph.D.

IMO, the difference is that whitelisting places requires manual intervention, while Carrie's proposal applies to all places. Carrie's is actually technically easier (no UI), applies universally, and has a complicated but consistent explanation. Explaining why an obs does or doesn't show up in a search under whitelisting would be, "Because a curator/admin said so."
I also feel obligated to point out that both approaches fail to solve the problem that kicked this all off, namely that you can find the locations of obscured observations by searching very small places.

If Carrie's proposal applies to all places, how would it be effectively different then just making all places search on private coordinates? ie do you have an example of an obs that would be returned differently under a search made on Carries proposal than a search made on private coords?

I do not have such an example, but maybe Carrie does.

In her proposal Carrie was asking if there should be a place size (or other) threshold where this would apply, so it wouldn't apply to all places (assuming we do set a threshold, which I would be for.)

Ok - if there is a difference in the bounding box rounding vs using private coords behavior for excepted places please explain so I can better understand.
But if there is no difference, might it be easier to just use admin_level or some other col directly on places to indicate which places should be excepted and just search on private coords for those places?
But regardless, seems like the crux of this issue are Carries first two questions above, ie which places should be excepted and/or how to make that decision

After discussion, we are NOT doing what I originally proposed, and are instead implementing a custom admin designation for CNC places (as Scott suggested).


justify-content-between adds horizontal scrollbar to Chrome

Operating system and version: Windows 10 Pro
Browser and version Chrome
Reduced test case https://codepen.io/anon/pen/PvpvMy

Missed the container wrapper...


Using system font instead of Microsoft Sans Serif

Do you mind taking this? Microsoft Sans Serif is an eyesore I always fix in Windows Forms projects. It makes a big difference in feel when it matches the OS UI font. The tray context menu is already doing this.
Thanks!
This project is cool. I started using the Java version and started thinking about doing a C# port. :D

+1 to this. I like Tomight for its minimalist design so any minor tweak in appearance is most welcome.

@ccidral I haven't seen any activity in the last 18 months. Need any help with the project?

Closing to clean up my PRs list due to inactivity. Please feel free to reopen!


Add indicators of authors quality based on forward citations

It can be done by looking for REF(eid) and counting citations in a window of time (for both focal and matches author).
It would be very nice to have, but also very demanding as many queries would be used and it would take time.

It could be done even faster with the CitationsOverview() class of scopus, but this requires a special API key.

This can be done now much more easily and efficiently using the query_size() function. It should be enough to build a function that queries the list of papers that cite the papers of an author and are published before the year provided (REF(eid OR eid OR ...) AND PUBYEAR BEF X)


Play TS. Is this possible?

Good day.
Does it possible to play TS stream without any conversion to HLS or DASH?
Thanks,
Mikhail.

It's up to your player. Browsers aren't usually capable of that.

No. I meant not to play HLS or DASH but rather play stream as is. For example I can use ffmpeg to push TS stream to module. Can I play this stream  with another ffmpeg using same URL as used to push the stream similar to rtmp module. For example to feed multiple clients with raw TS.

Oh, I get it. No, currently it's not supported and probably will never be. However I will probably support (push) exec which will receive mpeg-ts on stdin. This will probably solve most issues.


Parsing: hgld: entries contain multiple lexemes, separated by ,

From @xrotwang on July 15, 2016 19:37
These entries should be split. Also, often a specific meaning is indicated enclosed in ", e.g.
kohI, kohhi7 “small intestine, waist, abdomen”

For some languages, [] seem to be used, e.g.:
ĩík [swell], pux [pot boil over]

Copied from original issue: glottobank/lexibank-data#2

formspec should handle this.


44

Codecov Report

Merging #65 into master will decrease coverage by 1.64%.
The diff coverage is 72.34%.

@@            Coverage Diff             @@
##           master      #65      +/-   ##
==========================================
- Coverage   86.78%   85.14%   -1.65%
==========================================
  Files         113      110       -3
  Lines        1037     1003      -34
  Branches       44       37       -7
==========================================
- Hits          900      854      -46
- Misses        130      141      +11
- Partials        7        8       +1


Now that controller is resolved at action scope, add an OnArg/OnInit where we can pass the arg to the controller directly

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Dictionary like scalar kernels

Which issue does this PR close?
Partially implements #1975.
Rationale for this change
Enhancement to add like kernels for dictionary.
What changes are included in this PR?
Like kernels for dictionary and string
Are there any user-facing changes?
No

A side effect of this PR resulted in some nice performance improvements as well. Some othese range form 2-3% to about 30/50% for some use cases.
On my OCI 4 core arm machine these are the improvements I am getting

Click me
like_utf8 scalar equals time:   [332.65 µs 332.75 µs 332.92 µs]
                        change: [-11.762% -11.542% -11.363%] (p = 0.00 < 0.05)
                        Performance has improved.
Found 7 outliers among 100 measurements (7.00%)
  3 (3.00%) high mild
  4 (4.00%) high severe

Benchmarking like_utf8 scalar contains: Warming up for 3.0000 s
Warning: Unable to complete 100 samples in 5.0s. You may wish to increase target time to 10.0s, enable flat sampling, or reduce sample count to 40.
like_utf8 scalar contains
                        time:   [1.9687 ms 1.9700 ms 1.9712 ms]
                        change: [-0.8125% -0.7296% -0.6506%] (p = 0.00 < 0.05)
                        Change within noise threshold.

like_utf8 scalar ends with
                        time:   [333.54 µs 333.57 µs 333.60 µs]
                        change: [-6.2180% -6.1563% -6.0802%] (p = 0.00 < 0.05)
                        Performance has improved.
Found 8 outliers among 100 measurements (8.00%)
  1 (1.00%) low severe
  2 (2.00%) high mild
  5 (5.00%) high severe

like_utf8 scalar starts with
                        time:   [354.51 µs 354.59 µs 354.70 µs]
                        change: [-5.8751% -5.8337% -5.7908%] (p = 0.00 < 0.05)
                        Performance has improved.
Found 3 outliers among 100 measurements (3.00%)
  1 (1.00%) low mild
  2 (2.00%) high severe

like_utf8 scalar complex
                        time:   [8.2678 ms 8.2691 ms 8.2704 ms]
                        change: [+0.3933% +0.4141% +0.4367%] (p = 0.00 < 0.05)
                        Change within noise threshold.
Found 2 outliers among 100 measurements (2.00%)
  2 (2.00%) high mild

nlike_utf8 scalar equals
                        time:   [359.67 µs 359.72 µs 359.76 µs]
                        change: [-33.604% -33.579% -33.553%] (p = 0.00 < 0.05)
                        Performance has improved.
Found 5 outliers among 100 measurements (5.00%)
  1 (1.00%) low severe
  1 (1.00%) low mild
  2 (2.00%) high mild
  1 (1.00%) high severe

nlike_utf8 scalar contains
                        time:   [2.0056 ms 2.0071 ms 2.0086 ms]
                        change: [-9.1066% -9.0256% -8.9406%] (p = 0.00 < 0.05)
                        Performance has improved.
Found 2 outliers among 100 measurements (2.00%)
  2 (2.00%) high mild

nlike_utf8 scalar ends with
                        time:   [357.22 µs 357.28 µs 357.34 µs]
                        change: [-35.930% -35.904% -35.861%] (p = 0.00 < 0.05)
                        Performance has improved.
Found 7 outliers among 100 measurements (7.00%)
  1 (1.00%) low severe
  5 (5.00%) high mild
  1 (1.00%) high severe

nlike_utf8 scalar starts with
                        time:   [377.53 µs 377.68 µs 377.84 µs]
                        change: [-33.147% -33.106% -33.066%] (p = 0.00 < 0.05)
                        Performance has improved.
Found 3 outliers among 100 measurements (3.00%)
  2 (2.00%) high mild
  1 (1.00%) high severe

nlike_utf8 scalar complex
                        time:   [8.3052 ms 8.3066 ms 8.3081 ms]
                        change: [-2.9553% -2.9313% -2.9080%] (p = 0.00 < 0.05)
                        Performance has improved.

ilike_utf8 scalar equals
                        time:   [2.8657 ms 2.8660 ms 2.8663 ms]
                        change: [-4.2960% -4.1779% -4.0962%] (p = 0.00 < 0.05)
                        Performance has improved.
Found 1 outliers among 100 measurements (1.00%)
  1 (1.00%) high mild

ilike_utf8 scalar contains
                        time:   [4.4981 ms 4.4989 ms 4.4997 ms]
                        change: [-56.457% -56.444% -56.432%] (p = 0.00 < 0.05)
                        Performance has improved.
Found 2 outliers among 100 measurements (2.00%)
  2 (2.00%) high severe

ilike_utf8 scalar ends with
                        time:   [2.9145 ms 2.9158 ms 2.9172 ms]
                        change: [-3.4589% -3.3895% -3.3206%] (p = 0.00 < 0.05)
                        Performance has improved.
Found 1 outliers among 100 measurements (1.00%)
  1 (1.00%) high severe

ilike_utf8 scalar starts with
                        time:   [2.9147 ms 2.9154 ms 2.9162 ms]
                        change: [-4.4854% -4.4584% -4.4289%] (p = 0.00 < 0.05)
                        Performance has improved.
Found 3 outliers among 100 measurements (3.00%)
  1 (1.00%) low mild
  1 (1.00%) high mild
  1 (1.00%) high severe

ilike_utf8 scalar complex
                        time:   [10.155 ms 10.157 ms 10.158 ms]
                        change: [-1.9673% -1.9384% -1.9101%] (p = 0.00 < 0.05)
                        Performance has improved.
Found 3 outliers among 100 measurements (3.00%)
  3 (3.00%) high mild

nilike_utf8 scalar equals
                        time:   [2.9256 ms 2.9261 ms 2.9267 ms]
                        change: [-2.7385% -2.7064% -2.6726%] (p = 0.00 < 0.05)
                        Performance has improved.
Found 6 outliers among 100 measurements (6.00%)
  4 (4.00%) high mild
  2 (2.00%) high severe

nilike_utf8 scalar contains
                        time:   [4.5042 ms 4.5065 ms 4.5089 ms]
                        change: [-56.448% -56.424% -56.402%] (p = 0.00 < 0.05)
                        Performance has improved.

nilike_utf8 scalar ends with
                        time:   [2.9386 ms 2.9390 ms 2.9394 ms]
                        change: [-1.3554% -1.3115% -1.2672%] (p = 0.00 < 0.05)
                        Performance has improved.
Found 3 outliers among 100 measurements (3.00%)
  2 (2.00%) high mild
  1 (1.00%) high severe

nilike_utf8 scalar starts with
                        time:   [2.8966 ms 2.8975 ms 2.8983 ms]
                        change: [-3.2629% -3.2297% -3.1958%] (p = 0.00 < 0.05)
                        Performance has improved.

nilike_utf8 scalar complex
                        time:   [10.201 ms 10.202 ms 10.205 ms]
                        change: [-2.0189% -1.9831% -1.9477%] (p = 0.00 < 0.05)
                        Performance has improved.
Found 3 outliers among 100 measurements (3.00%)
  2 (2.00%) high mild
  1 (1.00%) high severe

I actually tried that.
There were a lot of if else paths in the kernels. It would mean I would have to pick one of the below options

write a function which would create a closure based on the pattern. This lead to some very ugly code with Fn's being wrapped in boxes. So I didn't prefer that.
Next option is to wrap the if else options inside the closure , but this would mean a comparison would happen for each element which wasn't optimal.
The next option was to use ArrayAccessor and compare_op inside each if else. But that is as good as in lineing the function our self. Hence I went with this approach.

Let me know your thoughts here.

Can you not just take the contents of the macro and make it into a generic function on a concrete ArrayAccessor<Item=&str>?

Okay , let me try that.


Create pipeline to create and publish a lua rock

Right now, Lazy.nvim fails to pull the luarock and then has to clone the repo and build from source. There needs to be a github action that creates the rock and then publishes it to LuaRocks.

Implemented in #8.


update grpc proto to lightos v2.3.x

Codecov Report

Merging #14 (01d8816) into master (75e0c22) will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff           @@
##           master     #14   +/-   ##
======================================
  Coverage    7.17%   7.17%
======================================
  Files           4       4
  Lines         237     237
======================================
  Hits           17      17
  Misses        219     219
  Partials        1       1

Continue to review full report at Codecov.

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


Clarify documentation for quick start

Issue Type

Documentation Report

Your Environment
Vagrant 1.9.5
VirtualBox 5.1.22r115126
ansible <IP_ADDRESS>
  config file =
  configured module search path = Default w/o overrides
  python version = 2.7.10 (default, Feb  7 2017, 00:08:15) [GCC 4.2.1 Compatible Apple LLVM 8.0.0 (clang-800.0.34)]

Your OS

macOS (version)

Summary
Just want to ask a few questions. I'm pretty new to this kind of setup and want to learn more.
1. Download this project and put it wherever you want.

Can I really put this anywhere? I've got it in a folder of VMs under /User/jasonglisson/VMs

2. (Optional) Copy default.config.yml to config.yml and modify it to your liking.

I leave this in the Drupal VM folder, right?
If the path is given to your project in the next step, why is this optional?

3. Create a local directory where Drupal will be installed and configure the path to that directory in config.yml(local_path, inside vagrant_synced_folders).

I did this, but it's telling me it can't find anything at that path.
If I have multiple local sites that I want to setup, do I change this each time to the new location?

4. Open Terminal, cd to this directory (containing the Vagrantfile and this README file).

I get the Vagrantfile from the VM folder that I just cloned, correct?

5. Type in vagrant up, and let Vagrant do its magic.

I get the terminal error below when I do this. It seems like it's looking for the VM inside the directory where I want the local site to be.

There was an error loading a Vagrantfile. The file being loaded
and the error message are shown below. This is usually caused by
a syntax error.

Path: /Users/jasonglisson/Sites/drupal8/Vagrantfile
Line number: 0
Message: LoadError: cannot load such file -- /Users/jasonglisson/Sites/drupal8/lib/drupalvm/vagrant

I watched the video here (http://docs.drupalvm.com/en/latest/getting-started/installation-macos/) and it clarified some things.
So you run vagrant up from the VM folder it seems.
The path you set obviously is where you want the project to be setup.
I guess if someone could help clarify the docs, that would be excellent.

(Optional) Copy default.config.yml to config.yml and modify it to your liking.

Yes, put that file inside the Drupal VM directory.

If the path is given to your project in the next step, why is this optional?

You don't technically need to change the path from the default. By default, Drupal VM will share a 'drupal' directory in Drupal VM's directory into the VM. That's often useful for when you want to quickly build a Drupal 8 site from scratch—Drupal VM does all the work for you, and you don't need to point to a specific directory.
Note that config.yml only needs to have the variables you want to override—your config.yml could just be a few lines telling Drupal VM where your codebase is.

If I have multiple local sites that I want to setup, do I change this each time to the new location?

If you want to have a bunch of sites running in one Drupal VM instance, you can add multiple shared folders, one per codebase. See: http://docs.drupalvm.com/en/latest/getting-started/syncing-folders/
If you have one codebase and a multisite setup, see: http://docs.drupalvm.com/en/latest/deployment/multisite/

I get the terminal error below when I do this. It seems like it's looking for the VM inside the directory where I want the local site to be.

I'm not quite sure how that error message appeared—did you download Drupal VM, then place Drupal VM inside the folder lib/drupalvm/vagrant? Or were you in the directory /Users/jasonglisson/Sites/drupal8/ and ran vagrant up?
For help with debugging, please write up the steps it took to get to the error, e.g.:

I downloaded Drupal VM to x directory.
I created a config.yml file in x directory with the contents linked in this gist: [link].
I ran the command vagrant up.
I got the following error: ...

Please let us know if you have any other specific questions.

Thank you for the great explanation and all the hard work. This definitely helped me out.
I was downloading the Drupal VM and then moving it or pointing it to another directory, but I see nothing wrong with just pulling it to the directory where I want my site and then installing the VM there. Then I just work out of the web directory. Anything wrong with that in particular?

@jasonglisson - No problem with doing that, though I typically recommend nowadays that you use a project layout where the first directory contains Drupal VM (or a 'vendor' directory if using Composer, along with composer.json/lock), a README and documentation, config for the site (if using Drupal 8 configuration management), and then last but not least, a docroot or web directory, inside which the actual Drupal site codebase is stored (this would be your Apache docroot).
That way you aren't mixing things like Drupal VM configuration or configuration dumps with your actual site code (if you do that, there's an increased risk that someone could find a way to get at information or discover things you don't want them to, in all those ancillary directories).
Closing out this issue, but feel free to open a new issue if you find some new problem or question!


How to resize a frame that is pulled from a video?

$video
    ->frame(FFMpeg\Coordinate\TimeCode::fromSeconds(10))
	->resize(new FFMpeg\Coordinate\Dimension(162, 78))
	->save($Save_Location."/".$FileName[0].".jpg");

I have everything working fine but I had trouble finding how to resize thumbnails that are pulled from video.  Above is my attempt.  Can anyone point me in the right direction?
Thanks,
Travis

That's not possible with the current code. I would recommend saving the file, loading it with the image adapter of your choice(gd or imagick) and resize it using this.
But I'll do a pr after my previous have been merged(so I can still focus on them, before making 100 of pull requests...)

any plan to merge this into release?

@foobarhe Feel free to make a pull request. :)

This doesn't works for me, as the resizing is done after the frame video filter:
  0 => "-y"
  1 => "-i"
  2 => "/var/www/html/storage/app/upload-tmp/vod-channel-PWGubwymi2wkCkEg5hZu8j/XEXYEAyAMwm5HHp1CgeiRj"
  3 => "-vf"
  4 => "fps=2.9411764705882/1"
  5 => "/tmp/vod-49418d94da11d227/frame-%06d.jpg"
  6 => "-s"
  7 => "170x100"
]```

but if I manage to put the `-s 170x100` switch before the `-vf` it works.

How can I achieve this with with this lib ? as the filters are always concatenated AFTER the `frame` I don't know how to do that..

Unfortunately, this is not yet possible, as it is not possible to place filters whereever you like.

It very easy like this:
$ffmpeg = FFMpeg::create();
$ffmpeg->open('video path')
       ->frame(TimeCode::fromSeconds(1))
       ->addFilter(new CustomFrameFilter('scale=320x160')) //resize output frame image
       ->save('save path');

It very easy like this:
$ffmpeg = FFMpeg::create();
$ffmpeg->open('video path')
       ->frame(TimeCode::fromSeconds(1))
       ->addFilter(new CustomFrameFilter('scale=320x160')) //resize output frame image
       ->save('save path');

Really thank you !

It very easy like this:
$ffmpeg = FFMpeg::create();
$ffmpeg->open('video path')
       ->frame(TimeCode::fromSeconds(1))
       ->addFilter(new CustomFrameFilter('scale=320x160')) //resize output frame image
       ->save('save path');

You saved my day. Thanks a lot.

It very easy like this:
$ffmpeg = FFMpeg::create();
$ffmpeg->open('video path')
       ->frame(TimeCode::fromSeconds(1))
       ->addFilter(new CustomFrameFilter('scale=320x160')) //resize output frame image
       ->save('save path');

How does this code work for you? It doesn't work for me. Or do I need to add something else?


Chore/test coverage/participation resource

Checklist

[x] Server: I added multiple integration tests (Spring) related to the features

Motivation and Context

This PR increases the coverage of Participation resource from 32% to 90% . All REST endpoints are covered now.

see inline comments and please merge develop

I resolved the inline comments, removed not used tests and added a couple of new test cases. Also merged developed.


feat: add tx-pause and safe-mode pallets

Closes #279

Looks good, thanks @Moliholy !
Let's wait to merge it until #308 has been merged.


Resize factor

Introduced the option to also use a crop factor between 0 and 1 to crop the shown glacier(and surrounding) area without having to check what the current grid resolution and size is.
As this functionality is extended the old functionality of resizing the plot, i renamed the former function to a more abstract name. resize_ds_by_nr_of_grid_points => resize_ds
Also the variables x_nr_of_grid_points and y_nr_of_grid_points got renamed to x_crop and y_crop as they are not necessarily representing number of grid points anymore.
Also added an example of this new option to the general_styling tutorial notebook.

Thanks @afisc for this new option!


.hears() function weird problem with a generated array (facebook-related)

Hi, I am using an api controller to get keywords from an certain API and trying to place the result array in the ".hears" function. For some reason it doesn't work. "keywords" is getting the right array for sure (checked that). Does someone has any idea? Thanks in advance.
api.getKeywords(function(keywords) {
controller.hears(keywords,'message_received', function (bot, message) {
...
}
});

Hi, I have made a separate file for keywords.
module.exports = {    name: [     'identify yourself',     'who are you',     'name',     'bot'   ], greet: [     'hi',     'hello',     'hey',     'yo'   ] }
This is my bot.js where I have my controller.
`controller.hears(keyword.name, 'message_received', function(bot, message) {
bot.reply(message, ':|] I am a bot for Zyloon.');
})
controller.hears(keyword.greet, 'message_received', function(bot, message) {
bot.reply(message, quickReplyTemplate.template('gettingStartedTemplate'))
})`

Did this help? Closing for now, feel free to reopen!


mypy: Clean out or report issues in mypy/typeshed for type: ignores

Now that we've finished #127, it's worth doing a bit of work on the type: ignores in our mypy annotations, trying to fix upstream in typeshed anything where that's possible.
I don't want this issue to remain open forever, so we should happily close this as soon as we've done a sweep through our type: ignore entries.
@neiljp I feel like you'd probably be pretty effective at doing this.

@zulipbot claim

#157 Should cover most of this, I think?
I think I had a copy/paste issue, as the work on generate_manifest.py I intended to link to
python/mypy#1747, which was my reported issue; the corresponding PR I just worked on is python/mypy#1748.

Yeah, this seems good enough for this issue (though of course you're welcome to contribute fixes upstream to clean this out further!).  Thanks @neiljp!

For the record, the correct links I reported and which are now fixed were in python/typeshed, not python/mypy. I'm just putting together a PR which removes these ignores and upgrades to mypy 0.560.


Add deprecate package information in this repo

Due to the limitation of go package manager, there is not a very good way to mark a package is deprecated, therefore, in our readme file, add a new link and a new file, that includes the deprecated package information.
Service Package Request

A file to list all the deprecated packages in Azure Golang SDK

Can you clarify what you are requesting to happen here.
cc: @kimprice @mikekistler

Technically though we wouldn't list any packages as deprecated until we have a stable package to replace it, right?

Technically though we wouldn't list any packages as deprecated until we have a stable package to replace it, right?

Yes, I think we can deliver this message after we have GO T2 GA, let me close this issue for now.


[table-extenstion] Add trait bounds (copy + drop) to Table key

Motivation
According to https://github.com/move-language/move/issues/95, Move lacks of lifetime support so we have to find a way to work around this. Otherwise, there is no way to implement clean APIs for many struct built on top of Table such as IterableTable.
This PR add trait bounds (copy + drop) to the key
Have you read the Contributing Guidelines on pull requests?
Yes
Test Plan
cargo test

I suggest to conduct an out-of-band meeting to discuss this early next week for interested parties. Perhaps hold on merging or working on the alternative, making keys everywhere copyable, until we discussed. Anybody wanting to join, say so on Discord and I setup an invite.

Updated, changing key to be copyable everywhere

@wrwg @davidiw plz have a look, thanks

/land


Weird bug with vterm

I am not entirely sure whether this is an upstream issue or not, but I will post it here anyway until we understand this issue.
EDIT1: Just tried vterm in a vanilla emacs and doom-emacs without personal config, I can confirm this is not related to vterm or my personal config, at least not directly. Are you aware of any popup related change that may cause these 2 issues?
EDIT2: Looks like this is not an issue specific to vterm, but popup buffer in general, doing grep in emacs also has the same symptom. (The buffer is not updating itself unless I gave some input)
What did you expect to happen?
Before update, I was able to use vterm normally in terminal Emacs without evil mode.
The default keybinding C-c o t toggles popup vterm. C-c o T to open a new vterm.
What actually happened?
After I did an update today, 2 things no longer work for me.

If I open a file with emacs -nw , then open vterm by C-c o t and run a long job. (e.g. compile a project), vterm will stop updating buffer after about 1 second. And it will no longer update its content unless some input was pressed.
I don't have this issue if: I start emacs without a filename in the argument, or if I use emacs in GUI mode.

Also, C-c o T does not do what it supposed to anymore - "Open vterm here", it now also pops vterm into the popup buffer like C-c o t does. Not sure if this is a bug or not but I don't like the new behavior.

I may have found the issue myself..
I have enabled centaur-tabs / tabs module for my doom-emacs.
The package itself does not support no-window mode and its occupied with updating warning messages.
It would be nice if doom-emacs can by default disable centaur-tabs in no-window mode until maybe it is supported one day in the upstream.
I will disable this module in my config for now.

Regarding second point - it seems it's caused by newest version of the emacs-vterm package.
Bringing the previous versions fix the +vterm/here issue.
You may bring the previous version by:
git checkout 5b242c73bbc2236bf1176b561b1d9d53cb2a2440 -- modules/term/vterm/packages.el
then:
bin/doom update

Thanks @AdamMarkowski
Since work around has been provided, closing this issue.

A workaround isn't a substitute for a solution.

The package itself does not support no-window mode and its occupied with updating warning messages.

@aidermoi This isn't easy to do when we have to accommodate daemon users. Daemon users can open terminal frames next to GUI frames, if they wanted, so there is no trivial way to disable centaur-tabs for no-window mode without polling every time the user enters the window, so I'll leave it to users to disable the features that don't work for their use case.

Regarding the second point - it seems it's caused by the newest version of the emacs-vterm package.
Bringing the previous versions fixing the +vterm/here issue.

@AdamMarkowski Is this still the case on the latest commit of Doom?

I've fixed the issue to C-c o T, but cannot reproduce the redrawing issue. I cannot reproduce the redrawing issue, however and have no other leads to go on. Please include your M-x doom/info (always include it please).

Can confirm this is a working solution for me!

Ok. Will do.
About the redraw issue, maybe I should be more clear. The redraw issue was coming from centaur-tabs itself. (It was doing some polling action with tons of errors, which may hault buffer refresh of other buffers) Turning it off resolved the redraw issue.


Build failing with error - MSB4175 : The task factory "CodeTaskFactory" could not be loaded from the assembly

Win UI 3 build started failing with error - MSB4175 The task factory "CodeTaskFactory" could not be loaded from the assembly "Microsoft.Build.Tasks.Core.dll". Operation did not complete successfully because the file contains a virus or potentially unwanted software at (Package)\build\Microsoft.WinUI.AppX.targets
Version Info
dot net - 5.0.100-preview.5.20279.10
Win UI  - 3.0.0-preview2.200713.0
Win 10 - May 2020 Update (19041)
Desktop - Yes

I had the same error. Checking my code a bit I saw that I had the package "Microsoft.CodeDom.Providers.DotNetCompilerPlatform" from the NuGet console, I removed it and it already lets me compile my project.

I have only these two..

Check on menu Project > Manage NuGet Packages and check if is allready installed:

Earlier screenshot was also from Installed tab :)

Another thing that can interfere is the windows defender. You can add the dll in the path that causes you problems as an Exclusion.

We're getting this same thing using the net35 sdk docker image in our CI.

we are experiencing the same problem starting today

it's definitely windows defender

Windows defender log show it detecting files in the temp directory that look like dynamically compiled assemblies. Probably from tasks.core.dll

we are also seeing similar issues but another item we noted is bunch of random .dll files being written to C:\temp, is anyone setting this issue along with this error as well? Defender flagging this as VirTool:MSIL/EMesbild.A and not much info on Microsoft references.

The Learn more link takes me to https://go.microsoft.com/fwlink/?linkid=142185&name=VirTool:MSIL/Emesbild.A&threatid=2147741673.

FWIW, updating Microsoft Windows Defender seems to fix the problem. 😎

Updating the Windows Defender virus definitions definitely seems to solve the problem.

Seeing this too, coinciding with the randomly named DLL's being created in a temp directory. Is that (DLL creation) the signature of the virus, or is there a virus signature in these newly created DLL's. The latter would be a lot more suspicious. I'm hesitant to whitelist this without finding a definitive answer

Updating the Windows Defender virus definitions definitely seems to solve the problem.

What version of definitions are you using?  And by chance what version was it broken in?

Upgrading from the definitions installed 2 days ago:
Security Intelligence Update for Microsoft Defender Antivirus - KB2267602 (Version 1.319.1877.0)
to:
Security Intelligence Update for Microsoft Defender Antivirus - KB2267602 (Version 1.319.2051.0)
did fix it for me

Updating the Windows Defender virus definitions definitely seems to solve the problem.

What version of definitions are you using? And by chance what version was it broken in?

Upgrading from the definitions installed 2 days ago:
Security Intelligence Update for Microsoft Defender Antivirus - KB2267602 (Version 1.319.1877.0)
to:
Security Intelligence Update for Microsoft Defender Antivirus - KB2267602 (Version 1.319.2051.0)
did fix it for me

Thanks @perkerk !
Yes, Updated Microsoft Defender Antivirus fixed the issue.
Validated with V 1.319.2068.0 👍

I've no nuget packages installed.  I get this error using msbuild v14.  But not with the one in the framework directory, or using msbuild from vs2019.


[MINOR][DOC] Fix the building document to describe Java 8 only insteaof Java 8+

What changes were proposed in this pull request?
Since we didn't test Java 9 ~ 11 up to now in the community, fix the document to describe Java 8 only.
How was this patch tested?
N/A (This is a document only change.)

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/4146/
Test PASSed.

Test build #97637 has started for PR 22781 at commit 82069cd.

Test build #97637 has finished for PR 22781 at commit 82069cd.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/97637/
Test PASSed.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/4148/
Test PASSed.

Test build #97639 has started for PR 22781 at commit 0af884c.

Test build #97639 has finished for PR 22781 at commit 0af884c.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/97639/
Test PASSed.

Thank you for review, @HyukjinKwon , @cloud-fan , @felixcheung .

Merged to master/branch-2.4.


try for preact dev tools

won't work in chrome extensions dev tools

try for DL page

https://github.com/amitsingh-007/bypass-links/pull/133


Use timezone-aware datetimes

Naive timestamps are used in multiple places which renders a lot of warnings,
these should be replaced with django.utils.timezone.* functions and modules instead.

Thank you for reporting, there's a new release coming soon i'll make sure to include this as well.


missing baseline files

@doutriaux1 These files were also added in #71. The issue is the filenames are wrong. To avoid having duplicate files, please rename the existing ones.

@sankhesh in the interest of time let's merge this and then remove the duplicate ones later. I can't spend too much time fixing these issues.

@doutriaux1 I fixed it.

Thx!


[tr069] fix nginx config for tr069 endpoints

I had forgot the midleware ;)

modified the nginx config since i only need one config for tr069


Long calendar event name displays the entire name instead of fading.

Yesterday I had a calendar event that somebody else invited me to. Instead of fading after a few words it displayed the entire name (~70 characters), shoved the day/time column to the end of the name, and meshed with the weather on the right.

I'll make sure this will be solved in v2.


chore: uno samples bump + adjust tests

bump samples to latest stable 5.5.x Uno
Adjust tests now failing with latest Uno due to TemplatedParent changes

⚠⚠ The build 148805 has failed on Uno Toolkit - CI.

/azp run

⚠⚠ The build 149154 has failed on Uno Toolkit - CI.


Plonky3 EVM verifier?

Is there any plan to implement Plonk3 EVM verifier?
or converting Plonky3 proof to the SNARK proof?

We have already implemented this. See: https://docs.succinct.xyz/verifying-proofs/solidity-and-evm.html.


WorldPay: Remove country overlap with WorldPay US

@ntalbott As we discussed.

Um, what's the reasoning for this? Does this product/endpoint not actually support US?

@j-mutter so the deal is, Worldpay has a bunch of different products, and they don't provision the product that ActiveMerchant calls "Worldpay" for US merchants. The country list is supposed to indicate where a gateway account is acquirable, not where the gateway can be used for processing, so this change (better) reflects reality.
@markabe I'd also like us to change the display name of the Worldpay gateway to Worldpay Global to help reflect this distinction.
Overall, Worldpay is a mess, and after lots of merchant confusion, this is one piece of what we're doing to try to help future merchants be less confused.

@ntalbott Updated display name.

@j-mutter any concerns remain before I merge? We could do this just on the Spreedly side, but our thinking was that it wasn't just an issue for our customers.

As long as it's just the display name that's changed, I think we're ok. We'll need to update some stuff on our end, but otherwise I think this is ok.  @girasquid thoughts?

@j-mutter so this is going to change the display name, and the supported countries for the worldpay adapter. No change to the class name and such, though - this is all metadata tweaks.

No change to the class name and such, though - this is all metadata tweaks.

In light of that, I don't think we have any issues here. Thanks for doing this!


is jacoco code coverage possible with zerocode?

Below is the build.gradle file of my test project. I want to check code coverage with jacoco. The jacoco is generating a empty report with this message "No class files specified." Please help with this.
apply plugin: 'java'
apply plugin: 'jacoco'

sourceCompatibility = 1.8

jacoco {
    toolVersion = "0.8.6"
}
dependencies {
    testCompile group: 'junit', name: 'junit', version: '4.12'
    testCompile("com.orgname:zeta:0.0.3-SNAPSHOT")
    testCompile 'org.junit.platform:junit-platform-commons:1.5.2'
    implementation group: 'org.jsmart', name: 'zerocode-tdd', version: '1.3.24'
}
test {
    jacoco {
        destinationFile = file("$buildDir/reports/jacoco/jacocoTest.exec")
    }
    testLogging {
        events "passed", "skipped", "failed", "standardOut", "standardError"
    }
}
jacocoTestReport {
    reports {
        xml.enabled true
        html.enabled true
        csv.enabled false
    }
}

build.finalizedBy(jacocoTestReport)

@pavithra0530 , Jacoco works with Maven. For Gradle the configs might be slightly different, but it should work. @BeTheCodeWithYou , have you used code-coverage with Gradle, do you have an example repo you can point to please?


[PROPOSAL] - Access android (MTP) devices

Might help fix #779
This PR contains a proposal of how MTP devices like android phones could be accessed by Files UWP.
What works:

Navigation inside the MTP device
Pasting/Deleting/Renaming inside the MTP device
Creating new ~files~ folders inside the MTP device.
Files can only be copied to, explorer does the same
Opening files inside the MTP device

@duke7553 what do you think?

@gave92 I'm impressed by the amount of work you put into this, and will look forward to learning more about this change tomorrow

Description of code changes
The main issue when accessing MTP devices is that windows does not map them to normal drives. Files inside such devices will not have a filesystem "path", which results in the impossibility of using functions such as StorageFolder.GetFolderFromPathAsync() and FindFirstFileExFromApp.
The only way to get the contents of a folder is querying the StorageFolder with the folder.GetFilesAsync()/folder.GetFoldersAsync() methods.
Note: getting the parent of a StorageItem also doesn't work (storageItem.GetParentAsync() returns null)
In order to navigate inside MTP devices it's necessary to:

Keep a reference to the root storage folder of the device -> returned by StorageDevice.FromId()
Keep a reference to the current storage folder
Still have a way to retrieve a specific storage folder from a path (necessary e.g when navigating up, when using the NavigationToolbar, etc)

For point (1) I added the property Root to the INavigationControlItem interface. The property will be populated for removable drives when creating a DriveItem.
For point (2) I added the CurrentStorageFolder property to ItemViewModel. The WorkingDirectory property now simply returns the CurrentStorageFolder path. As operations with storage folders are async, changing the CurrentStorageFolder is done in a method and not in a property setter.
Point (3) is the real "fun". To get a storage folder from a path:

I split the given path into components (the same way it's done for NavigationToolbar)
For each path component, starting from the Root folder (or the CurrentStorageFolder), I drill down calling folder.GetFolderAsync(component) until I get a reference to the storage item I'm interested in.

This is done in GetFolderFromRelativePathAsync. This behaves the same as StorageFolder.GetFolderFromPathAsync unless we are in a removable drive (where the Root property is not null).
This method returns an object that contains the retrieved StorageFolder and a Path that in case of removable drives will be a "virtual" path (= "root_name\folder1_name\folder_2_name...")
Most of the other code changes are simply replacing calls to StorageFolder.GetFolderFromPathAsync with StorageFolder.GetFolderFromRelativePathAsync
Pros:

no changes for normal drives
done in UWP (I discarded the idea of doing this in the fulltrust process like the recycle bin)

@gave92 Get an error when trying to access to my OnePlus 5T file system.
Translated exception: Access denied.

Thanks for the feedback I only tested this on my phone.
Where inside the RefreshItemsAsync method is that exception thrown?

In EnumerateItemsFromStandardFolder (ItemViewModel.cs) in foreach:

As i understand, problem is in await _rootFolder.GetFilesAsync()

@tsvietOK please let me know if latest commit fixes it thanks!

@gave92 Ok, now it is working, but much slower then windows explorer. 17sec to load a folder contains 416 elements.
Enumerating items in \\?\ONEPLUS A5010\Внутренний общий накопитель\Music (device) completed in 17396 milliseconds.

Now i am getting an exception when trying to create bitmap

Exception thrown: 'System.UnauthorizedAccessException' in System.Private.CoreLib.dll
An exception of type 'System.UnauthorizedAccessException' occurred in System.Private.CoreLib.dll but was not handled in user code
Access denied. (Exception from HRESULT: 0x80070005 (E_ACCESSDENIED))

@gave92 Another bug, files are not displayed at all, only folders exist.

Ah that's also why you are getting the exception for the bitmap

Ok, in this case Create file function should be disabled.
File paste causing app crash.
If start app when device already connected, app freezes at splash screen.

I believe in your case the cause of file paste causing app crash is the same as crash when creating bitmap which is also caused by the UnauthorizedAccessException when enumerating files.
I'm not sure how to debug (and solve) this.
Ok, in this case Create file function should be disabled. -> yup I'll do that as soon as I know this thing doesn't work just on my phone 😂

Menu "Create file" in windows explorer is not exists for my phone too.
Version from store starting fine with device connected.

If you are willing to debug the app and see why it freezes at splash screen it would be a huge help as I can't reproduce it.

Get another bug. App crash on startup when at least one element pinned to sidebar.

Update: App starts fine when device connected, app crash caused by issue above.

@duke7553 Turns out the issue causing very slow folder contents enumeration was getting the item thumbnain. It should be now much faster while still using GetItemsAsync.

Getting an exception when navigating to my phone storage.

@tsvietOK thanks for the feedback. The exception should not be thrown on that line. It would be great if you could step through the code and get the line.
On a guess it may be here. If that's the case, is _workingRoot null at that point?

@gave92 The problem is in line return new StorageFolderWithPath(await StorageFolder.GetFolderFromPathAsync(value)); in StorageFileExensions.cs

Exception appeared when trying to get storage folder
Value = "\\?\ONEPLUS A5010\"

Is the rootFolder parameter null at that point?

Is the rootFolder parameter of that function null when the exception happens?

Yes

Issue fixed, good job! One small glitch: double separator

@tsvietOK

One small glitch: double separator

Fixed

And why all pinned(by me) items are placed before default pinned items like Music, Downloads etc.?

That's the magic of not awaiting async methods, should be fixed

@gave92 Seems like there is no option to create a folder inside the MTP device.

@tsvietOK

Right click on empty space > New > Folder should work
But the menu "+" button on the menu bar is incorrectly disabled
On right click menu you can also click "New text file/New bitmap" but those do not work.

Do you confirm all this?

"+" button is disabled
There is no such menu items at all

@gave92 Update: Check in grid view mode

@tsvietOK thanks I can confirm there is no New options in Grid View Mode

There is New Bitmap and New Text document menu item exists in context menu in GridViewBrwoser mode

@tsvietOK hopefully it's now fixed

@gave92 Great work! :shipit:


Impossible to apply changes to a zone with a txt entry

Hi,
When using PDNS-Admin with PowerDNS 3.4 I can't apply changes to a zone with a txt entry.
Here's the error received:
{ "msg": "Record mg.epic-sys.io/TXT v=spf1 include:mailgun.org ~all: Parsing record content: Data field in DNS should start with quote (") at position 0 of 'v=spf1 include:mailgun.org ~all'", "status": "error" }
The field is quoted in the DNS and the changed record is unrelated. The json sent includes the quotes so it must be somewhere between the app and PowerDNS but my python is not good enough to track it.

Are you sure you are quoting using " marks rather than ' marks? I've found " marks work.

You have to quote your txt record with double quotes ""

Found the issue while looking into the database. Some unrelated records where not quoted. It was not obvious because the error was for another but yeah. Fixed the database and now everything works. Thanks


Adding info about test keys provided by Google

Test keys make local development and functional testing a breeze. Please see https://developers.google.com/recaptcha/docs/faq for more information.

Do you think we should make these the defaults? That way users will be able to leave their settings empty in development and not copy paste them.

That's a better idea. I'll close the PR and open a new one with those changes


Add support for storing base url for environment in config file

Overview
Add support for storing base url for environment in config file
Why is this needed?
Rather than having to specify the base url in testing I would like to be able to override the value in my config file on a per profile basis
Assumptions

N/A

Implementation Details
Out of Scope

N/A

Acceptance Criteria

N/A

This is now supported by adding either production_base_url or sandbox_base_url to the config section for a particular profile. This setting can only be overridden at a profile level, not a global level.


Next version preparation

This PR includes projects and tools clean up to prepare next release.

Close because some build errors on CI


3D chart doesn't compile: Undefined is not an object

Bug or Feature Request?
Bug
Description
On attempting to build, I get the following error:
TypeError: undefined is not an object (evaluating 'Highcharts.chart')
src/wizard/node_modules/react-jsx-highcharts/dist/es/components/HighchartsChart/HighchartsChart.js:9

How to reproduce
Copied over the example code from website, tweaking data locations to be local to the file.
Live demo demonstrating bug
Live Demo
Versions

React JSX Highcharts version: "^8.0.0"
Highcharts version: "^4.0.1"
React version: "^16.12.0"

Nevermind, it looks like I was missing the HighchartsProvider!


[ Feature Request ] Support batch in Image Preview node

If we have a batch of images in an Image Preview node, it only displays the first one.
This would be very useful in workflows with IPAdaper / FaceID
If this is possible, try to add a small margin between the images.

Yes. Right now we explicitly select just one image, so this should be on
the roadmap
On Tue, 5 Nov 2024 at 11:14 AM, JorgeR81 @.***> wrote:

If we have a batch of images in an Image Preview node, it only
displays the first one.
This would be very useful in workflows with IPAdaper / FaceID
If this is possible, try to add a small margin between the images.
batch.jpg (view on web)
https://github.com/user-attachments/assets/01d7db4f-808a-4e19-bfff-1cd09048970c
—
Reply to this email directly, view it on GitHub
https://github.com/chrisgoringe/cg-controller/issues/289, or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABBLMAY2XJO6IF4XFVIQJ4LZ7AEX5AVCNFSM6AAAAABRFL2MUOVHI2DSMVQWIX3LMV43ASLTON2WKOZSGYZTIMJTHE3TQMA
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

This is working, but the clicking area, on the left, is not disabled at 1/n
So you can click it to get 0/n

It's fixed.

Nice! 🚀

Maybe the last thing I would add is the ability to start over from the beginning when you reach the last image and vice versa.

yeah, thats nicer. Done.


Missing @types definition

Hello,
I'm using typescript for my API, and i'm surprised that Aclify doesn't provide @types definition files.
This would be great !
I'm trying to get the first example import {...} from "@aclify/aclify" but with no success....
For info node acl from OptimalBits had one here
Cheers!

I'm closing this PR, the package has been rewritten.

glad to hear and see that the plugin has been rewritten !
But it seems that the package still missing types definition file.
import * as Aclify from "@aclify/aclify";

Throws TS Error
Could not find a declaration file for module '@aclify/aclify'. <EMAIL_ADDRESS>implicitly has an 'any' type.
      Try `npm install @types/aclify__aclify` if it exists or add a new declaration (.d.ts) file containing `declare module '@aclify/aclify';`

    8 import * as Aclify from "@aclify/aclify";

created PR to fix this #47

Thank you @olivierto for this PR. 🙏
It has been merged.


MongoDB: "revision" field type should be casted to number when createSnapshot

Hi.
I've experienced wrong order issue while querying snapshots as I stored my snapshots with "string" typed revisions, and I think the library should force revision inputs in number type because string typed revisions cause sort issues when we query documents like this:
> db.snapshots.find({ aggregateId: "NEWSTREAMID", context: "User_Snapshot"}).sort({revision: -1}).pretty()
{
        "_id" : "5f2138f1f3969e93542e2b2c",
        "id" : "5f2138f1f3969e93542e2b2c",
        "streamId" : "NEWSTREAMID",
        "aggregateId" : "NEWSTREAMID",
        "aggregate" : "User",
        "context" : "User_Snapshot",
        "commitStamp" : ISODate("2020-07-29T08:53:05.651Z"),
        "revision" : "24",
        "version" : null,
        "data" : "24"
}
{
        "_id" : "5f213990f3969e93542e2b2e",
        "id" : "5f213990f3969e93542e2b2e",
        "streamId" : "NEWSTREAMID",
        "aggregateId" : "NEWSTREAMID",
        "aggregate" : "User",
        "context" : "User_Snapshot",
        "commitStamp" : ISODate("2020-07-29T08:55:44.575Z"),
        "revision" : "124",
        "version" : null,
        "data" : "124"
}
...

document with revision "24" comes out first even if there's a document with revision "124" and there's no errors nor warnings when I save my snapshot with string-value revisions.

How are you creating snapshots?
How do you call es.createSnapshot()

it’s the same revision value passed when creating the snapshot

@adrai
Yes I found it out yesterday. What I want to say is that since the store implementation (at least mongodb implementation) assumes that revisions are in number format, it's better to throw errors when people try to push revision value in another format rather than number.

Feel free to provide a PR.


Pre doesn’t respect dark theme on iOS

Not sure if this is intentional (or already fixed, I saw something in #4 related to it), but reporting in case it’s unintended…
Thanks for your library, definitely something I’ll want to use!

This is intentional to match <code> but im on the fence about it. Give me a few days and ill play around with some options of having it with a dark background.


ui_report: Migrate ui_report.message to ui_report.client_error.

As requested
Testing plan:
GIFs or screenshots:

Closing, as this commit should be part of #17122.

Closing, as this commit should be part of #17122.


Can't add data when sending eGLD via Elrond Ledger App => Can't delegate eGLD

Dunno if this is a Ledger app related issue or a wallet one but we can't add data using Wallet with Ledger authentication. It seems to be mandatory when sending to staking address.
Is it something planned ?

Sorry, didn't saw we needed to enable Contract Data in settings in the Ledger

@palnap I have the same problem, What is the solution?

Hi, just go the Elrond app on your ledger, go to settings and enable contract data.Envoyé depuis mon appareil Galaxy
-------- Message d'origine --------De : Mauricio Hernan Cabrera @.> Date : 06/10/2022  14:30  (GMT+01:00) À : ElrondNetwork/ledger-elrond @.> Cc : palnap @.>, Mention @.> Objet : Re: [ElrondNetwork/ledger-elrond] Can't add data when sending eGLD via Elrond Ledger App => Can't delegate eGLD  (#20)
@palnap I have the same problem, What is the solution?
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you were mentioned.Message ID: @.***>


refactor(golang): mv templates into files for embedding

Move templates into files for embedding and better maintainability.

 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.


Enhancement: support a script block as the replacement operand of the -replace operator to enable dynamic substitutions

Wishful thinking:
# Script block as replacement operand allows dynamic substitutions.
# TBD: how to refer to the match at hand in the script block.
> 'Increment these numbers: 1 2 3' -replace '\d+', { [int] $Args.Value + 1 }
Increment these numbers: 2 3 4

Sometimes string replacements need to be dynamic, i.e., the replacement string must be derived from the match at hand.
Consider the following (contrived example):
You want to increment all numbers in an input string; e.g.:
'Increment these numbers: 1 2 3' -> 'Increment these numbers: 2 3 4'
Currently, -replace doesn't support this type of replacement, because the replacement operand can only statically refer to the match at hand ($& for the match in full, $1 for the 1st capture group's match (if any), ...).
Thus, to achieve this kind of substitution the .NET Framework must currently be used:
> [regex]::Replace('Increment these numbers: 1 2 3', '\d+', { param($match) [int] $match.Value + 1 })
Increment these numbers: 2 3 4

Needless to say, this is neither PowerShell-like nor simple.

Supporting the syntax as proposed at the top would greatly simplify such operations.
Syntax details
The simplest approach is to use [regex]::Replace() as-is behind the scenes, and let  the script block reference the match at hand - which is passed as a [System.Text.RegularExpressions.Match] instance - via $Args (or, optionally, an explicitly defined parameter via param()).
The alternative is to stick with the $&, $1, ... syntax, but that would require behind-the-scenes manipulation of the script block.
Backward-compatibility considerations
Currently, attempting to use a script block as the replacement operand simply stringifies that block (i.e., uses its string contents as the replacement string).
Since this behavior isn't useful, it's hard to imagine anyone relying on it, so this change probably falls into Bucket 3: Unlikely Grey Area
Environment data
Written as of  PowerShell Core v6.0.1

This is approved in in review, see https://github.com/PowerShell/PowerShell/pull/6029

To allow this call would also be appreciated :
'Increment these numbers: 1 2 3' -replace '\d +', {[int] $ Args.Value + 1}, $Count, $StartAt

@LaurentDardenne Please open new issue for your suggestion.


[sw/silicon_creator] Add fuzzer for ROM_EXT bootstrap

This PR depends on #18929. Ignore all but the last commit.

I'm working on splitting this into two fuzzers: one for ROM bootstrap and one for ROM_EXT bootstrap.
I have a working proof of concept uploaded, but I want to clean it up a bit tomorrow.


'TsdFrame' object has no attribute 'reshape'

When trying to compute the scipy.stats.sem of a TsdFrame, this error is thrown:
---------------------------------------------------------------------------
AttributeError                            Traceback (most recent call last)
Cell In[65], line 8
      5 fr = counts_smooth / bin_size
      7 mean_fr_nap = np.mean(fr, axis=1)
----> 8 error_fr_nap = sem(fr, axis=1)

File /opt/conda/lib/python3.9/site-packages/scipy/stats/_axis_nan_policy.py:495, in _axis_nan_policy_factory.<locals>.axis_nan_policy_decorator.<locals>.axis_nan_policy_wrapper(***failed resolving arguments***)
    490     # New shape is unchanged for all axes _not_ in `axis`
    491     # At the end, we append the product of the shapes of the axes
    492     # in `axis`. Appending -1 doesn't work for zero-size arrays!
    493     new_shapes = [shape[:-n_axes] + (np.prod(shape[-n_axes:]),)
    494                   for shape in shapes]
--> 495     samples = [sample.reshape(new_shape)
    496                for sample, new_shape in zip(samples, new_shapes)]
    497 axis = -1  # work over the last axis
    498 NaN = _get_nan(*samples)

File /opt/conda/lib/python3.9/site-packages/scipy/stats/_axis_nan_policy.py:495, in <listcomp>(.0)
    490     # New shape is unchanged for all axes _not_ in `axis`
    491     # At the end, we append the product of the shapes of the axes
    492     # in `axis`. Appending -1 doesn't work for zero-size arrays!
    493     new_shapes = [shape[:-n_axes] + (np.prod(shape[-n_axes:]),)
    494                   for shape in shapes]
--> 495     samples = [sample.reshape(new_shape)
    496                for sample, new_shape in zip(samples, new_shapes)]
    497 axis = -1  # work over the last axis
    498 NaN = _get_nan(*samples)

AttributeError: 'TsdFrame' object has no attribute 'reshape'

Ok solved in #253. That issue was more troublesome than I thought but in the end, it improved the API.


Add unit tests for github.py

Unit tests for github.py  should go under the tests/ folder.
The API calls to GitHub will have to be mocked out.

All yours!


Be friendly to csh

att

csh is too old school and people often complain about it: http://www.faqs.org/faqs/unix-faq/shell/csh-whynot/
Personally not a big fan of csh because of the lack of documentation.
Won't fix for now -- unless you want to drive the effort :)


[bug]: confirmed funds gone after sweeping all tx

Hi,
im using LND 0.15.4-beta commit=v0.15.4-beta-1-g56476b632.
I had about 22mio sats offchain balance that was displayed by LND as confirmed, i didnt move those for a long time.
I tried to open a new channel and lnd said that my onchain balance is too low. I couldnt explain why, because there was no pending tx and no unconfirmed balance, so i decided to sweep all transactions, assuming that this would free up funds that are locked in script tx.
The sweep was successfully broadcasting a TX but not my unconfirmed balance is only 122k sats and the rest is not displayed anymore.
There is now only a single UTX left in my wallet that is the sweep tx.
How can i recover the remaining?

Can you share some transaction IDs please? Otherwise it's hard to say. Also, is this Neutrino or bitcoind? Updating to the latest version is definitely also a good idea.


[HUDI-4108] Clean the marker files before starting new flink compaction

Tips

Thank you very much for contributing to Apache Hudi.
Please review https://hudi.apache.org/contribute/how-to-contribute before opening a pull request.

What is the purpose of the pull request
(For example: This pull request adds quick-start document.)
Brief change log
(for example:)

Modify AnnotationLocation checkstyle rule in checkstyle.xml

Verify this pull request
(Please pick either of the following options)
This pull request is a trivial rework / code cleanup without any test coverage.
(or)
This pull request is already covered by existing tests, such as (please describe tests).
(or)
This change added tests and can be verified as follows:
(example:)

Added integration tests for end-to-end.
Added HoodieClientWriteTest to verify the change.
Manually verified the change by running a job locally.

Committer checklist

[ ] Has a corresponding JIRA in PR title & commit

[ ] Commit message is descriptive of the change

[ ] CI is green

[ ] Necessary doc changes done or have another open PR

[ ] For large changes, please consider breaking it into sub-tasks under an umbrella JIRA.

@danny0405 please take a look when free.

@hudi-bot run azure

@hudi-bot run azure

The test failure is not caused by this, so i would just merge it soon ~

We found an issue which might be related with PR, detail in https://issues.apache.org/jira/browse/HUDI-4880
May I ask if there is anyone can double check if it is the root cause? Really appreciate


Potential Contrast methods to implement as plugins

Only two worth picking from this page is (Weber and Michelson).
This APCA one is currently only sRGB specific. We'd have to wait for it to mature more.

I have not compared or researched Weber and Michelson to see if they add anything compelling compared to the already implemented WCAG 2.1 contrast. The APCA sounds promising, but I think it is too early. I think they are considering it for CSS, or at least some future variant of it.
There is no guarantee we'll implement all of these, nor is there a timeline to do so, but something to consider. We'll add more if we find anything else useful.

Playground with Weber and Michelson contrast: https://facelessuser.github.io/coloraide/playground/?source=https%3A%2F%2Fgist.githubusercontent.com%2Ffacelessuser%2Fec0829e0af84ff2227d0a2e1de2662a5%2Fraw%2F7ea9ebc585c81f9ea81edcb2901c95c7da9761af%2Fcontrast.py

I released Weber and Michelson as experimental plugins in ColorAide Extras: https://github.com/facelessuser/coloraide-extras

Closing this as we implement the others in extras and then created a specific issue for APCA.


readme.md

Current Translation:
https://microsoftlearning.github.io/MCT-User-Guide/
Suggested Translation:
https://microsoftlearning.github.io/MCT-User-Guide-JA/

Hello contributor,
Thank you for your contribution! We are processing your suggestions and we will update you shortly.
Kind regards,
Microsoft Worldwide Learning Team

Hello contributor,
Thank you for your contribution! We are processing your suggestions and we will update you shortly.
Kind regards,
Microsoft Worldwide Learning Team

Hello contributor,
Thank you for your contribution! We are processing your suggestions and we will update you shortly.
Kind regards,
Microsoft Worldwide Learning Team


[WIP] add client var to use userland DiscordClient in library code

This PR modifies startSession to assign a reference to a variable that points to a user populated DiscordClient so that we can use it in library code, this allows us:

to introduce procs like defer and other useful sugar that doesn't need the discord.api prefix
to (?) potentially refactor a bunch of procs that would not need the prefix either

New sugary procs are noticable by their short naming scheme.
What we need to complete this PR:

[ ] tests that include the added procs
[ ] documenting them
[ ] decide if this is the design that we really want

If you're adding helper procs I think adding them to the dimscord/helpers.nim file would be ideal.
If you're adding the helper procs based of only just message and interaction (Message.react/edit/reply), then I think the PR is more suited for dimscmd.
If somebody wants to say editGuild, banUser, getGuildAuditLogs etc they still have to use discord.api prefix.

The way my lib worked was by wrapping types in a generic type that looked like this (I can't remember the exact implementation)

type
  EasyDiscord*[T] = object
    data: T
    api: RestApi

This allowed me to add a reference to the API without needing to modify dimscord
I was doing something similar too but by using inheritance, because I needed a base type for implementing a generic waits table in order to wait for an event. It seems like a better design than globals indeed and we would "kill two birds with one stone" if we do it this way.

After seeing #106 I think inheritance might be the way to go :+1:

Also please note that if we go with inheritance, this PR becomes a breaking change

Ready for review and validation,  please refer to my past comments if you wonder why I removed inheritance and opted for another approach.
also @ire4ever1190 why the confused react ?

Confused why you moved from inheritance.
I'm reading the messages, but I still think a global variable is a bad idea. I also find the getClient which also acts as a setter to be weird. I assume it was so the client couldn't be reassigned but that is still possible like so
let foo = newDiscordClient("Some token")
getClient()[] = foo[]

Confused why you moved from inheritance.

Well that is because it's harder to implement and has a big shotgun which is due to a limitation of this approach:

First of all, the shotgun in question is that for objects that are obtained by using the API and assuming there is no global var involved, their client field will be nil (or none DiscordClient) because all the getters that rely on the API do not have the data, unless you do discord.stuff(myobj, args) instead of myobj.stuff(args) and assign discord to the client field before you return the object but then what's the point ? You might as well just do discord.getStuffFromApi(myobj.id, args) which is exactly the sort of syntax this PR aims to make nicer.
For objects that come from event handlers and are cached, the client field would be present if we modify every event handling procs. But not every object come from event dispatchers, for example Application ... So this mean the only plan is to not do anything about it instead, no helping procs for Application and more.
You already know but having to use inheritance means turning some objects into ref object if we plan to implement https://github.com/krisppurg/dimscord/issues/105. I feel like it's too early to decide if we're really going to use this approach and instead I suggest to take a step back instead and think through all our options which is also part of why I changed my mind because I though I was being too eager.

Now, because of point no.1, it could be that we need Option[DiscordClient] instead, but this doesn't change much unfortunately. The cost of using inheritance for everything must not be neglected too.

btw, I would be fine with inheritance, but I really don't know what to do about those issues, would appreciate some guidance here

In regards to the limitations

If obtained through the API then the client field shouldn't be null. Would require updating a lot of the API procs but mostly a matter of making sure to assign the client field and also taking the client as a proc instead

proc getChannelPins*(discord: DiscordClient,
        channel_id: string): Future[seq[Message]] {.async.} =
    ## Get channel pins.
    result = (await discord.api.request(
        "GET",
        endpointChannelPins(channel_id)
    )).elems.mapIt() do:
        m = newMessage(it)
        m.client = discord
        m

This is assuming they need the actual DiscordClient and not just the RestAPI. @krisppurg are there any procs that would require the full client or is passing around the RestApi object enough? If just passing RestApi around is enough then ignore the next paragraph (besides maybe the with proc, that could still be handy)
Since that is a massive breaking change we could also just add a with proc like below and then save changing parameters for a 2.0 breaking update
# We make it generic so the original type isn't erased
proc with[T: DiscordClient](e: T, client: DiscordClient): T {.inline.} =
  e.client = client
  e

Things like commands can still be got via the api with things like getApplicationCommands so would be good to have helpers for those. But since they are also constructed manually, it would just require the user to also pass the client when constructing or use something like the with proc I mentioned

Smart move to take a step back and plan. Currently I think the design is fine but guess we won't know until we try. I'll see if I can get an MVP done today to give us a better idea on how it will work

(If Krisp says that passing around RestApi is enough, then replace all usages of client with api in my previous message)

nvm, current MVP of awaiting would require client anyways

Re:

@krisppurg are there any procs that >would require the full client or is passing >around the RestApi object enough? If just >passing RestApi around is enough then >ignore the next paragraph (besides maybe >the with proc, that could still be handy)

Im pretty sure there are no procs that require the full client, RestApi is enough as the necessary information it has the api version and the discord bot token.

further developement of the inheritance approach happens here: https://github.com/krisppurg/dimscord/pull/113


Fix missing webview provider crash on Android

Summary
I applied the changes requested in this PR: https://github.com/facebook/react-native/pull/29089
We upgraded to RN 0.62.2 on our latest release and started to see again the "Failed to load WebView provider: No WebView installed" (see below for Crashlytics screenshot)

This crash had been fixed by https://github.com/facebook/react-native/pull/24533 but https://github.com/facebook/react-native/pull/26189 (added in 0.62) reverted the fix
Indeed the exception raised in Crashlytics is actually a AndroidRuntimeException and MissingWebViewPackageException is only part of the message.
For instance, in the screenshot above, the exception message is android.webkit.WebViewFactory$MissingWebViewPackageException: Failed to load WebView provider: No WebView installed
Now these crashes are quite tricky to reproduce, so to be on the safe side, I'm filtering out all exceptions containing WebView as suggested by @thorbenprimke on the original fix.
If my reasoning is correct, it should fix @siddhantsoni 's issue as well, since WebView is included in MissingWebViewPackageException
But following that reasoning, I am not sure https://github.com/facebook/react-native/pull/26189 fixed @siddhantsoni 's issue, so @siddhantsoni if you could check that this PR also fixes your issue, that would be great!

Changelog
[Android] [Fixed] - Fix missing WebView provider crash in ForwardingCookieHandler

Test Plan
I created a version of react native with this patch applied
"react-native": "almouro/react-native#release/062-2-fix-missing-webview-provider"

Before the fix ~0.1% of our users were impacted on Android, no new crashes have occurred after the update.
This is putting back what was already in place and working for us, but making the check wider to catch more errors.

@RodolfoGS how did you test your code? since i can't update to 0.67 i'm using patch-package, but the app keeps crashing and decompiling the code I see that it didn't apply any changes


Does not work without Sodium

When I launch Minecraft without Sodium in the mods folder, this error window appears:
GLFW error 65544: EGL: Failed to clear current context: An EGLDisplay argument does not name a valid EGL display connection.

Please make sure you have up-to-date drivers (see aka.ms/mcdriver for instructions).

@TotoShampoin which gpu do you have?

NVidia GeForce RTX 3050 Mobile

Apparently this error seems like it, and it looks like nvidia driver issue
You can try to set __GL_THREADED_OPTIMIZATIONS to 0 and check if it resolves the issue for you


Features on screen saver control

Since I'm currently using tetra to create tools rather than games, the
These features are necessary.

I am not a native speaker of English, so I appreciate this suggestion.

Thank you for the PR 😄