Source freshness filters with string quotations break upload_sources macro in Snowflake (v1.0.0 beta prelease)

We have a source with a filter applied to the freshness test:
dayname(updatedat) not in ('Sunday', 'Monday')

This causes the upload_sources macro to output the freshness column as...
'{"warn_after": {"count": null, "period": null}, "error_after": {"count": 1, "period": "day"}, "filter": "dayname(updatedat) not in ('Sunday', 'Monday')"}'

...which causes the following error:
Runtime Error
  Database Error in operation sftw_dw-on-run-end-0 (./dbt_project.yml)
    001003 (42000): SQL compilation error:
    syntax error line 311 at position 150 unexpected 'Sunday'.
    syntax error line 313 at position 16 unexpected ''9e625675-40e7-4dc5-953e-b6f8e9824e82''.
    syntax error line 323 at position 12 unexpected ')'.

After some manual testing in Snowflake, the single quotes need to be escaped either with a backslash...
'{"warn_after": {"count": null, "period": null}, "error_after": {"count": 1, "period": "day"}, "filter": "dayname(updatedat) not in (\'Sunday\', \'Monday\')"}'

...or a double single-quote...
'{"warn_after": {"count": null, "period": null}, "error_after": {"count": 1, "period": "day"}, "filter": "dayname(updatedat) not in (''Sunday'', ''Monday'')"}'

I spent a few hours trying to figure out a fix by using replace on the filter key in the freshness dict, but that took me to string formatting hell.
For the time being, I can get around this by updating the filter to use dayofweekiso.

Thank you for reporting @MichaelGFagan !


cleaning for Irish (ga-IE)

LGTM

The linter doesn't like lambdas, I'll change them to defs

That seems ok now! Thank you @jimregan!

@jimregan feel free to share with us the result you get and eventually the trained model (if you want/can). I'm going to merge it!


refspace for allen mouse

Provide the referencespace data for allen mouse atlas

Data or at least a first version is implemented:
https://siibra-api-master.apps-dev.hbp.eu/v1_0/atlases/juelich%2Fiav%2Fatlas%2Fv1.0.0%2F2/spaces/minds%2Fcore%2Freferencespace%2Fv1.0.0%2F265d32a0-3d84-40a5-926f-bf89f68212b9


Flake8 and pbr

Also enable flake8 on TravisCI

Coverage remained the same at 40.81% when pulling<PHONE_NUMBER>539481d387cea0e78d593d868ca5a4 on toabctl:flake8-fixes into 367eda8c378035a5f99f7c221e26b174652e7152 on saschpe:master.

Coverage remained the same at 40.81% when pulling<PHONE_NUMBER>539481d387cea0e78d593d868ca5a4 on toabctl:flake8-fixes into 367eda8c378035a5f99f7c221e26b174652e7152 on saschpe:master.

Coverage decreased (-40.81%) to 0.0% when pulling 5560e16ddac8a37a99498b4d48a9d10a4854fbf7 on toabctl:flake8-fixes into 367eda8c378035a5f99f7c221e26b174652e7152 on saschpe:master.

Coverage decreased (-40.81%) to 0.0% when pulling 5560e16ddac8a37a99498b4d48a9d10a4854fbf7 on toabctl:flake8-fixes into 367eda8c378035a5f99f7c221e26b174652e7152 on saschpe:master.

Sorry, but no. Pbr is not pythonic, it's wrong in too many ways. Off-topic but gemspec isn't declarative either...
Same goes for flake8, I do want an AUTHORS file and I fail to see the reason for an empty ChangeLog file. It's a git repo after all.


hot_mode_dev_cycle_linux__benchmark hotRestartMillisecondsToFrame trending upwards, now overwhelmingly red

Difficult to tell a culprit from this.

What is the app being restarted in this benchmark? @jonahwilliams @liyuqian

Old flutter gallery: https://github.com/flutter/flutter/blob/master/dev/devicelab/lib/tasks/hot_mode_tests.dart#L25

Does that app remain fixed, or does it grow over time? Also, would this benchmark be expected to get slower over time as new code is added to the framework?

The gallery used for this benchmark isn't being developed anymore. I suspect that hot restart will gradually slow down as the framework code size increases, since we have to sync the full dill, but I don't think we measure the size of the gallery's dill file to verify

@kf6gpe Do we have logs from the benchmark runs anywhere? Maybe we should add dill size to the logs the benchmark spits out so that we can decide whether to rebaseline. For now, it sounds like we should rebaseline. I'll remove 'regression' and 'TODAY'.

@zanderso Good question. I don't know. @liyuqian Do you know?

I believe the logs should be in the GCP console somewhere (see this example). @godofredoc @digiter : do you know how to download logs of a particular perf test for a range of commits?

This is probably the same regression as https://github.com/flutter/flutter/issues/57323. Tentatively looking at whether it was a CFE change


Repackage maven dependencies into more meaningful coordinates and document contents of the different dependencies

Q
A

Bug or feature request?
feature request

Which Swagger-Core version?
2.0.0-rc2 ?

Which Java version?
all

Which JAX-RS framework & version?
2.0.0

I have setup a jersey project with the new Swagger Core JAX RS and was struggling a bit to find which dependencies to use.
According to the project setup tutorial I only need to use:
compile 'io.swagger:swagger-jaxrs2:2.0.0-rc2'
But the openapi annotations were missing so I ended up adding them with an additional dependency.
compile 'io.swagger:swagger-jaxrs2:2.0.0-rc2'
compile 'io.swagger.core.v3:swagger-jaxrs2:2.0.0-rc3'

(Is this even the correct way to set it up?)
I had similar issues in the past when setting up the previous version of JAX RS swagger.
IMHO Part of the difficulties in the setup process, stem from the confusing group/artifact ids used and maybe wrong maven coordinates in the setup pages.
I would suggest to remap them to something more meaningful and along the way add a care free all you need dependency if it does not exist yet.
Example:
compile 'io.swagger:jax-rs2-all:<version>' // contains all dependencies to run the open api servlet + annotations
compile 'io.swagger:jax-rs2-open-api-servlet:<version>' // contains just the servlet
compile 'io.swagger:jax-rs2-open-api-annotations:<version>' // contains just the annotations

While doing so it would also be nice to have just one common group id as currently it seems like there are 2, namely "io.swagger" and "io.swagger.core.v3".
Since the project is targeting to generate openapi specifications it would probably be nice to take the opportunity to unify everything under the group id "org.openapi"
Since I know this is not something that can be changed over night, it would be nice to at least get a github page, that explains what can be found in which dependency and for what it can be used.
That at least would dissolve the confusion short term.

Version 2.0.0 has recently been released along with a new wiki, which covers dependencies and integration; swagger-core group id is now io.swagger.core.v3 for all modules. Closing ticket, please reopen if it doesn't address your concerns.


daylight saving different results between safari/chrome

Hello
I ran this line in chrome and Safari and I get different result:
Safari seems to be wrong is there a workaround?
In Safari
moment("2015-03-08 07:30").toDate() => Sun Mar 08 2015 08:30:00 GMT-0700 (PDT)
In Node or Chrome
moment("2015-03-08 07:30"). toDate() => Sun Mar 08 2015 07:30:00 GMT-0700 (PDT)

Same issue. Using moment.utc("2015-03-08 07:30") fixes this.

Looks like a bug in Safari:
> new Date(2015, 2, 8, 7, 30)
< Sun Mar 08 2015 08:30:00 GMT-0700 (PDT)

Not much we can do about that, unfortunately; Moment is reliant on the underlying browser knowing what time it is.


feat: daemon call leaveHost when exit

Signed-off-by: bigerous<EMAIL_ADDRESS>
Description

Related Issue

Motivation and Context

Screenshots (if appropriate)
Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] Documentation Update (if none of the other choices apply)

Checklist

[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] I have read the CONTRIBUTING document.
[ ] I have added tests to cover my changes.

Codecov Report

Merging #1788 (5169188) into main (2556582) will decrease coverage by 0.21%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main    #1788      +/-   ##
==========================================
- Coverage   43.80%   43.58%   -0.22%
==========================================
  Files         128      128
  Lines       15885    15884       -1
==========================================
- Hits         6958     6923      -35
- Misses       8396     8431      +35
+ Partials      531      530       -1

Flag
Coverage Δ

Object-compatibility-e2etests
?

e2etests
?

unittests
43.58% <ø> (-0.22%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

client/daemon/storage/local_storage.go
30.11% <ø> (+0.05%)
:arrow_up:

client/daemon/peer/peertask_piecetask_poller.go
34.42% <0.00%> (-14.21%)
:arrow_down:

client/daemon/peer/peertask_conductor.go
56.35% <0.00%> (-0.66%)
:arrow_down:


[SPARK-18973][SQL] Remove SortPartitions and RedistributeData

What changes were proposed in this pull request?
SortPartitions and RedistributeData logical operators are not actually used and can be removed. Note that we do have a Sort operator (with global flag false) that subsumed SortPartitions.
How was this patch tested?
Also updated test cases to reflect the removal.

Please also merge this into branch-2.1 to minimize backport conflicts ...

Test build #70509 has started for PR 16381 at commit b734cfa.

Test build #70509 has finished for PR 16381 at commit b734cfa.

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

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

Merged build finished. Test PASSed.

Note that this is code from the initial Spark SQL commit!

LGTM. Merging to master/2.1. Thanks!


Cursor should be pointer on menu list

Is your feature request related to a problem? Please describe.

A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]

Describe the solution you'd like

A clear and concise description of what you want to happen.
Cursor without pointer:

Cursor with pointer:

Go ahead
@arbazalam01


Fix Select selection not working on mobile device - radix fix no longer required

Problem solved
Short description of the bug fixed or feature added

PR-Codex overview
This PR focuses on simplifying the ref handling in the SelectPrimitive.Content component of the select.tsx file by removing a custom ref function and replacing it with a direct assignment. It also enhances the component's class names for better styling based on its state.
Detailed summary

Removed custom ref handling logic.
Directly assigned ref to SelectPrimitive.Content.
Updated className with various state-based styles for animations and positioning.

✨ Ask PR-Codex anything about this PR by commenting with /codex {your question}

#5321  👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @MananTank and the rest of your teammates on  Graphite


.srec extension auto match

could you add the file of .srec extension to auto match. sometimes S-record file is not .19 extension.

Of course, I'm sorry for the late reply

This is released in 0.0.3.

Thank you for your work.


Swith to native DRF exception handling

To use the DRF exception handler

Coverage remained the same at 100.0% when pulling 646d8c34e26c7f318cc2c4b3b1e58b953328985d on Ivan-Feofanov:patch-1 into e62f1f92a621575174172e970da624d367ac0cf6 on JamesRitchie:master.

Coverage remained the same at 100.0% when pulling 646d8c34e26c7f318cc2c4b3b1e58b953328985d on Ivan-Feofanov:patch-1 into e62f1f92a621575174172e970da624d367ac0cf6 on JamesRitchie:master.

Coverage remained the same at 100.0% when pulling 646d8c34e26c7f318cc2c4b3b1e58b953328985d on Ivan-Feofanov:patch-1 into e62f1f92a621575174172e970da624d367ac0cf6 on JamesRitchie:master.

Coverage remained the same at 100.0% when pulling 646d8c34e26c7f318cc2c4b3b1e58b953328985d on Ivan-Feofanov:patch-1 into e62f1f92a621575174172e970da624d367ac0cf6 on JamesRitchie:master.


Helm chart dependencies do not follow best practices

Is your feature request related to a problem? Please describe.
Charts do not follow common guideline: chart templates bring dependencies into the main chart instead of using Helm dependency
Describe the solution you'd like
Move services like prometheus, alert-manager, zookeeper  to a chart dependencies, That would simplified helm chart support and usage.
Describe alternatives you've considered
Do not see other options, cause the issue is that Chart structure is inconsistent and does not follow best practices
Additional context
As a chart reference with a lot of dependencies GitLab helm chart could be taken..

+1
This would also fix the improper use of Prometheus as a deployment.  Prometheus needs to be a stateful set due to it's pvc.


Since the last update I have to go through 2 Alfred search results and press Enter twice to open a note

Since the last update I have to go through 2 Alfred search results and press Enter twice to open a note.
See Screen Recording 2020-12-07 at 10.19.05.mov.zip.
Is there any way to make the search results appear directly after typing "os " so that you can open notes with fewer keystrokes and just 1 alfred search results pane?
Thanks

I'm not sure if I can fix that easily without requiring users to manually tweak that part of the workflow (I introduced environment variables in the last update to remove the need for users to do that). I'll look into it and will ask on the Alfred forums to see if it's possible. The "problem" here is that I'm trying to also get Alfred to search for files only inside your vaults, which requires an extra step.
But if you don't mind the workflow also listing Markdown files that are not inside your vault, we can remove that extra keystroke. Try this version and let me know if you like it? The original keywords os, or, and ot are now oss, orr, and ott—they will only list files inside your vaults. The updated os, or, and ot will now work with just 1 keystroke, but will list all markdown files on on your computer.

Thanks Hause, it works better for me!
There is something funny going on with the searches, though, but I'm not
sure how to explain it. See the attached video which shows what happens
when I type: "os[tab]start typing a note's partial title[tab][enter]". Then
at 00:07 I do almost the same thing except that I don't type the first
[tab]. When I do that, Alfred shows me my desired note first ("O- 2020 -
GROW - USGlobalInvestors") and I can immediately open it. It seems that if
I type a full word present in a note's title along with the 2 tabs,
sometimes it also work as expected "os[tab]GROW[tab][enter]", but not for
some notes, e.g. "os[tab]graftech[tab][enter]" won't leave a note that has
"something Graftech.md" in its title.
If this description is unclear, please let me know. But if you don't have
more time for this, the new version does really work better for me, this
last comment is an attempt to make the plugin work in a more predictable
fashion with fewer keystrokes, that's all. :-)
Thanks!
On Mon, 7 Dec 2020 at 17:25, Hause Lin<EMAIL_ADDRESS>wrote:

I'm not sure if I can fix that easily without requiring users to manually
tweak that part of the workflow (I introduced environment variables in the
last update to remove the need for users to do that). I'll look into it and
will ask on the Alfred forums to see if it's possible. The "problem" here
is that I'm trying to also get Alfred to search for files only inside your
vaults, which requires an extra step.
But if you don't mind the workflow also listing Markdown files that are
not inside your vault, we can remove that extra keystroke. Try this
version
https://www.dropbox.com/s/gal6atk046h3vjt/Obsidian.alfredworkflow?dl=1
and let me know if you like it? The original keywords os, or, and ot are
now oss, orr, and ott—they will only list files inside your vaults. The
updated os, or, and ot will now work with just 1 keystroke, but will list
all markdown files on on your computer.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/hauselin/obsidian-alfred/issues/9#issuecomment-740024546,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAAQ6NLU6E562DPLLIAEWN3STT6VXANCNFSM4UQFK2FQ
.

Great. I might update the github version too if that's preferred.
Can you attach your video again? It doesn't look like it's been uploaded. Thanks!

Thanks Hause.
Here is the video, now uploaded via Github instead of reply-by-email:
Screen Recording 2020-12-08 at 17.37.59.mov.zip

Hm... I'm not 100% what the problem is, though I did find a "bug." Try checking the box below if it isn't already checked. Click on the os File Filter action to open up that pane. Let me know if it works. If not, try removing the oss, orr, and ott keywords. They might be interferring with things too...

Oh, yeah, that seems to have fixed it. Thanks! :)


Update docs for New partition grammar

A document change request is generated from
https://github.com/GreptimeTeam/greptimedb/pull/3347

close as completed


Layers not getting processed

During demo I noticed that occasionally a requeue after delay does not work
This should be addressed by implementing https://github.com/fidelity/kraan/issues/8
in the mean time, restarting the kraan-controller causes the layers to be reprocessed

not seeing this issue anymore, will be addressed by https://github.com/fidelity/kraan/issues/8


interaction with sccache changed, all deps rebuilt when calling "cargo build" twice

cargo 1.36.0-nightly (c4fcfb725 2019-05-15)
release: 1.36.0
commit-hash: c4fcfb725b4be00c72eb9cf30c7d8b095577c280
commit-date: 2019-05-15

When using sccache (via RUSTC_WRAPPER), running cargo build ; cargo build will cause all dependencies to be rebuilt even if nothing changed between the two "cargo build" runs.
(I'm not very sure what is actually happening, since the second run is consistently faster than the first run)
cargo clean ; cargo build ; cargo build

   Compiling serde v1.0.80
   Compiling ryu v0.2.7
   Compiling itoa v0.4.3
   Compiling serde_json v1.0.39 (/home/matthias/vcs/github/serde_json)
    Finished dev [unoptimized + debuginfo] target(s) in 4.43s
   Compiling itoa v0.4.3
   Compiling serde v1.0.80
   Compiling ryu v0.2.7
   Compiling serde_json v1.0.39 (/home/matthias/vcs/github/serde_json)
    Finished dev [unoptimized + debuginfo] target(s) in 1.48s

This is not the case when sccache is not used or when RUSTC_WRAPPER is unset:
   Compiling serde v1.0.80
   Compiling ryu v0.2.7
   Compiling itoa v0.4.3
   Compiling serde_json v1.0.39 (/home/matthias/vcs/github/serde_json)
    Finished dev [unoptimized + debuginfo] target(s) in 11.65s
    Finished dev [unoptimized + debuginfo] target(s) in 0.01s

According to my bisect, 6b28a0c0505f19a574a44bffdeccbc07a2d25980 is the commit that caused this strange behaviour.

Just to be clear, the ticket is about none-pipelined compilation (unless there is something that turned it on by default that I missed).

Is this related to #6978 ?

I was personally unable to reproduce this yesterday, @matthiaskrgr can you detail how to reproduce this? (for example what version of sccache, what project, perhaps a minimal example, etc)

I'm able to reliably reproduce it.  I'm fairly certain this is because Cargo is now producing both rlib and rmeta files, even when pipelining is not enabled.  Sometimes sccache detects it (presumably some kind of race?), but usually the rmeta files don't get created.  I would think this issue needs to be filed on sccache?  Unless @alexcrichton wants to change it so that Cargo can handle rmeta files mysteriously disappearing.  There was a similar issue in the Clippy repo where it was deleting rmeta files, causing everything to be rebuilt (already fixed https://github.com/rust-lang/rust-clippy/pull/4115).

I use sccache from the git repo https://github.com/mozilla/sccache , commit ce983d5d1b8d35c71e6b99a6a9b17ed17a6dad1a
build it in release mode and copy it into ~/.cargo/bin/
I have RUSTC_WRAPPER set as /home/matthias/.cargo/bin/sccache
I can repro for example building https://github.com/serde-rs/json at 5b5f95831d9e0d769367b30b76a686339bffd209
 matthias@t470  ~/vcs/github/serde_json   master  cargo build
   Compiling ryu v0.2.7
   Compiling serde v1.0.80
   Compiling itoa v0.4.3
   Compiling serde_json v1.0.39 (/home/matthias/vcs/github/serde_json)
    Finished dev [unoptimized + debuginfo] target(s) in 4.16s
 matthias@t470  ~/vcs/github/serde_json   master  cargo build
   Compiling itoa v0.4.3
   Compiling ryu v0.2.7
   Compiling serde v1.0.80
   Compiling serde_json v1.0.39 (/home/matthias/vcs/github/serde_json)
    Finished dev [unoptimized + debuginfo] target(s) in 1.13s

Ah thanks for confirming @ehuss and for the extra info @matthiaskrgr. If this is an issue with sccache then I think yeah we can move it to that repository, and it sounds like that's the case?

I looked into this briefly  and found that sccache isn't caching the rmeta outputs, but there's an existing workaround that pointed to https://github.com/rust-lang/rust/issues/54852 and testing shows that rustc does not in fact list the rmeta filename when calling it with --print=file-names, so that is likely the underlying issue!

This has been fixed on the sccache side :+1:


[Fix rubocop#10969] Allow Lint/AmbiguousBlockAssociation `AllowedPa…

[Fix rubocop#10969] Allow Lint/AmbiguousBlockAssociation AllowedPattern to match on
send_node.source instead of method_name

[x] The PR relates to only one subject with a clear title and description in grammatically correct, complete sentences.
[x] Wrote good commit messages.
[x] Commit message starts with [Fix #issue-number] (if the related issue exists).
[x] Feature branch is up-to-date with master (if not - rebase it).
[x] Squashed related commits together.
[x] Added tests.
[x] Ran bundle exec rake default. It executes all tests and runs RuboCop on its own code.
[x] Added an entry (file) to the changelog folder named {change_type}_{change_description}.md if the new code introduces user-observable changes. See changelog entry format for details.

I'm planning to cut a new release very soon, so it'd be nice to update this changelog entry and wrap up this bugfix in time for the next release.

Please let me know if I should make any further changes. Thank you. 🙇

Thanks!


Update README.md

Replace this paragraph with a description of what this PR is changing or adding, and why. Consider including before/after screenshots.
List which issues are fixed by this PR. You must list at least one issue.
If you had to change anything in the flutter/tests repo, include a link to the migration guide as per the breaking change policy.
Pre-launch Checklist

[ ] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[ ] I read the Tree Hygiene wiki page, which explains my responsibilities.
[ ] I read the Flutter Style Guide recently, and have followed its advice.
[ ] I signed the CLA.
[ ] I listed at least one issue that this PR fixes in the description above.
[ ] I updated/added relevant documentation (doc comments with ///).
[ ] I added new tests to check the change I am making, or this PR is test-exempt.
[ ] All existing and new tests are passing.

If you need help, consider asking for advice on the #hackers-new channel on Discord.

GitHub turns a go-link into something unclickable.


feat(webpack): enable filesystem-based caching

Production build of packages/spec/integration/react…

with cold webpack cache: ~6.3s
with warm webpack cache: ~2.3s

Bors merge bot cheat sheet
We are using bors-ng to automate merging of our pull requests. The following table provides a summary of commands that are available to reviewers (members of this repository with push access) and delegates (in case of bors delegate+ or bors delegate=[list]).

Syntax
Description

bors merge
Run the test suite and push to master if it passes. Short for "reviewed: looks good."

bors merge-
Cancel an r+, r=, merge, or merge=

bors try
Run the test suite without pushing to master.

bors try-
Cancel a try

bors delegate+
Allow the pull request author to merge their changes.

bors delegate=[list]
Allow the listed users to r+ this pull request's changes.

bors retry
Run the previous command a second time.

This is a short collection of opinionated commands. For a full list of the commands read the bors reference.

bors merge


No error messages if mandatory fields are not provided

If some mandatory fields are not provided, no error messages are shown at all. For this to work the following HTML element should be added under the field:
<span class="error" id="error_<field name>">

Thanks for the hint!


Failed AJAX requests returning status code 200 OK

I have another problem with AJAX requests. Since we enabled it in PR https://github.com/chromelyapps/Chromely/pull/157 it was all good. But today I was needed to handle errors on API requests, and found that all failed requests will have status code 200 with no content. Because of this I am not able to know in JS code is request was failed (no connection, non-OK API response) or it is just a normal response.
It can be reproduced with sending any request on non-exist API e.g.:
https://non-exist-api.com/get
URL scheme should be registered as AJAX:
new UrlScheme("default-api", string.Empty, string.Empty, "non-exist-api.com", UrlSchemeType.Ajax, false)

@donotcodeit
I think that the setting here ...is being blocked by the callback in External Http Handler
Compare with what is done for for
Local Http Handler
If this is something you want to look into, I will suggest that the callback [continue, cancel] be removed.
Additoanlly this should be done here

if (_httpResponseMessage == null)
{
   response.Status = something with error (NOT OK);
   response.StatusText = some reason;
   return;
}

*** Note: I did not prove that to be correct yet, just comparing the two approaches.
Thanks.

I can confirm that removing these lines: https://github.com/chromelyapps/Chromely/blob/8684d310c1dc6cff48c37cb2e471900b08e062b8/src/Chromely.CefGlue/Browser/Handlers/ExternalRequestSchemeHandler.cs#L114-L115 fixing unexpected behavior.
@mattkol can you include this to the next build? I will fix it locally for now.

@donotcodeit makes sense.
Thanks.

@mattkol sorry I was wrong. Those lines should be replaced with:
if (_httpResponseMessage == null)
{
    response.Error = CefErrorCode.Failed;
    return;
}

Or we'll just get null ref exception.


Move ContractRegistry to ApiSession

This will make the ContractRegistry a session-dependent entity even though it won't be a LiveEntity.
And while we are here, move ApiSession.getLiveContract to the associated ApiSession.ContractRegistry instance.
This will allow more fluent calls such as session.contractRegistry[<name_of_loaded_contract>].getLiveAt({ id }).
Special considerations should be thought for web-runtime contexts. Specifically, the strato-rollup plugin will need to be updated.
We should do this after #33 but lets start discussing it: How will the resulting API be like? Any concerns?

Maybe it's best if we live ApiSession.getLiveContract where it is (for flexibility: let the dev get a hold on a LiveContract with ABI not necessarily stored in the ContractRegistry) and have a new LiveContract retrieval method inside the ContractRegistry itself.


yiiActiveForm ajax validation duplicate error messages in summary

Right now there is inconsistency between server-side and client-side version of displaying error summary as server-side version shows only unique error messages and client-side shows all of them. As far as I can tell there's also no way to alter that behavior on client-side as summary rendering is done after the  afterValidate event was triggered.
What steps will reproduce the problem?
ActiveForm with AJAX validation with multiple validation errors with same message (e.g. "Field is required").
What is the expected result?
Display only unique error messages and skip duplicates.
What do you get instead?
All error messages no matter if they is a duplicate or not.

If someone uses old yii2 versions and cannot update by some reasons:
$this->registerJs(<<<JS

$('div.error-summary ul').on('DOMSubtreeModified', function () {
    let uniqueErrors = {};
    $(this).find('li').each(function () {
        let thisVal = $(this).text();
        if (thisVal in uniqueErrors) {
            $(this).remove();
        } else {
            uniqueErrors[thisVal] = true;
        }
    })
});

JS
);

If someone uses old yii2 versions and cannot update by some reasons:
$this->registerJs(<<<JS

$('div.error-summary ul').on('DOMSubtreeModified', function () {
    let uniqueErrors = {};
    $(this).find('li').each(function () {
        let thisVal = $(this).text();
        if (thisVal in uniqueErrors) {
            $(this).remove();
        } else {
            uniqueErrors[thisVal] = true;
        }
    })
});

JS
);


start and enable in any case

this partially reverts changes of commit 1fd0a0356b482e385882e7d9916e74af04c8d00f to make starting and enabling in systemd work again
/cc @egmont1227 @brotzeit

Also brauchen wirs doch.


Connection issue

The client automatically gets disconnected after a certain amount of time.  When running on iOS, there seems to be an issue there too. It's unable to automatically reconnect when the connection is lost. The client also gets disconnected when using the Poxa console. Events can still be intercepted by the server when reconnecting.

Issue similar to https://github.com/edgurgel/poxa/issues/156
I'm running on iOS instead.

Setting the ACTIVITY_TIMEOUT worked.


storage: compaction group cache should be updated on scan

    /// This function could only be called once during the lifecycle of `LocalStreamManager` for
    /// now.
    pub async fn build_actors(&self, actors: &[ActorId], env: StreamEnvironment) -> Result<()> {
        // Ensure compaction group mapping is available locally.
        dispatch_hummock_state_store!(self.state_store(), store, {
            store.update_compaction_group_cache().await?;
        });
        let mut core = self.core.lock();
        core.build_actors(actors, env)
    }

Consider the case: the actor is build on CN 1, 2, but batch row seq scan is issued on CN 3. In this case, we will panic at get_compaction_group_id because the cache is not populated.
We meet this issue when running ./risedev ctl table scan <table>. @BugenZhao will have a PR to fix the case for risectl. But I think it's better to have a push / pull based mechanism for updating compaction group.

cc @zwang28


Hotfix Question

This question is with regards to the "Releasing Hotfixes" section of the following article about the Azure DevOps Release Flow process: https://docs.microsoft.com/en-us/azure/devops/learn/devops-at-microsoft/release-flow#releasing-hotfixes.
The article mentions that hot fixes are done in branches of master and then cherry-picked to the release being hot fixed (such as releases/M129). My question is what do you do if the area that needs to be hot fixed has already been touched by another commit that is for a future release (such as releases/M130) that is now in master? If I apply a fix to master and then cherry-pick it to M129, won't the other changes that were applied for M130 to the same code also be cherry-picked?
Thank you.

Thanks for your question. It looks like your question isn't an issue with the documentation, but an issue or question specific to your implementation or a bug in the product itself.
For help with a specific task, here are a couple of options where you might consider asking your question:

Azure DevOps on Stack Overflow
Azure DevOps Support Bot

If you've encountered an issue with the product itself, to make sure it gets in front of the right people, please submit your bug here.
If you have an idea for improving the product, please submit your idea here.


WhiteIntel Data Breach Search Engine is Added

WhiteIntel is dark-web empowered search engine which monitors data leaks from information stealer malwares.

Hello!
Thanks for your contribution.
README.md is auto-generated though.
It'd be awesome if you could make this addition in intel.json.

Hi,
Thank you for the responses. I've sent a new one.
Best regards.


lost transition animations

Hi, I just got the min.js file and I get notification but without the animations.  Do I need to bring in a dependency?

Yes, my plugin relies on custom css for its intro and exit animations. For this I recommend using this wonderful resource called Animate.css by Daniel Eden. He has put together a wonderful set of css animations that work perfectly with this plugin.
This isn't listed as a dependency because it isn't technically required. You could always write your own css animations or use a different library for the fact.

I am going to assume this was solved. If it wasn't please feel free to comment on here and I'll re-open it.
Thank you for the support.


pull request for e2e test at 12/23/2016 17:15:12

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


RotationAxisConstraint doesn't work well with BoundsControl

Describe the bug
RotationAxisConstraint and BoundsControl don't work well together in a number of ways. Link to the suggested solution adopted by UXT is in Additional context.
Examples (see GIFs below too):

If the constraint is configured to allow rotation around X (both in local and world space), the object will only rotate up to a certain degree, then it'll start rotating back. However, when no axes are constrained, you can rotate the object 360º.
If the constraint is configured to rotate around the Z axis in local space, you can roll with any edge handle, but you should only be able to do so with handles on the side edges.
You can configure the component to constrain a single axis, which leads to unexpected rotations.

To reproduce

Go to BoundsControlExamples.
In the scene's hierarchy, click on SceneContent > CoffeeCups and select one of them.
Modify configuration of the RotationAxisConstraint, both Constraint On Rotation and Use Local Space For Constraint.
Play the scene.
Manipulate the actor via BoundsControl and observe how it behaves.
Repeat 3-5 with multiple combinations and you'll see the issues mentioned.

Expected behavior
Assuming that the object was sliced through the center by 3 perpendicular planes, each edge handle should rotate around the plane it's on, unless it must constraint in world space, as the projection of the rotation might lead to a certain rotation on another axis. Also, the configuration shouldn't allow constraining a single axis, as rotating around the other two axes can achieve the same rotation you're trying to restrict.
Screenshots

Your setup (please complete the following information)

Unity 2019.4.6f1
MRTK v2.5 (commit 1ba4258f74)

Target platform (please complete the following information)

HoloLens 2

Additional context
We have fixed this in UXT by using swing-twist decomposition (instead of Euler decomposition) in the constraint and slightly modifying how BoundsControl uses it. Still works perfectly fine with Manipulator.

Un-stale-ing for MRTK3.

We appreciate your feedback and thank you for reporting this issue.
Microsoft Mixed Reality Toolkit version 2 (MRTK2) is currently in limited support. This means that Microsoft is only fixing high priority security issues. Unfortunately, this issue does not meet the necessary priority and will be closed. If you strongly feel that this issue deserves more attention, please open a new issue and explain why it is important.
Microsoft recommends that all new HoloLens 2 Unity applications use MRTK3 instead of MRTK2.
Please note that MRTK3 was released in August 2023. It features an all-new architecture for developing rich mixed reality experiences and has a minimum requirement of Unity 2021.3 LTS. For more information about MRTK3, please visit https://www.mixedrealitytoolkit.org.
Thank you for your continued support of the Mixed Reality Toolkit!


Support for Xiaomi Smart Doorbell 3

Device model - madv.cateye.mowl3g
The unique model of your device, e.g zhimi.fan.za5.
Device name - Xiaomi Smart Doorbell 3
The friendly name of the device used by the manufacturer, e.g. Smartmi Standing Fan 3.
Does the device require a MiCloud connection? - Do not know
Yes/No/Do not know
Most devices which were introduced to the market in 2020 and later does not require MiCloud.
Anything special about the device? - The plugin finds the device, but fails to poll and logs "[Smart Doorbell 3] Device not supported! Using a generic device with limited properties! Consider requesting device support!"
Does it already work with the plugin with a limited scope (can be controlled using the auto created switch)?

What do you expect from the doorbell to work? Video will not be possible.

Hi @merdok ! Thanks for your answer. Would be great to have at least notification that someone at the door, if video is not possible.

As far as I can see, it should be possible to expose a sensor which would trigger on the camera motion sensor detection. I can try to add that.


[FEATURE REQUEST] - Add ability to export layer as tar

Similar to https://github.com/wagoodman/dive/issues/336
Can we get a "save layer" as a tar to the host filesystem? This can be useful when inspecting how files change across layers as files are modified/deleted.

Seems to be related to https://github.com/wagoodman/dive/issues/333


Remove fraction file completely

Fraction file slows down the computations on metacentum (solved) and on boinc. Jirka and Martin decided to completely remove this because it has not any significant information.

And update docuentation accordingly.

Done (new implementation of eacirc)


futileness, chronogenetic

This pull request was generated by the 'mq' tool
[test]
flake rate: 0.4
logical conflict every: 100
sleep for: 300s
[pullrequest]
requests per hour: 30
deps=[c,f]

/trunk merge


File Manager

File manager would be fun. https://github.com/cpojer/mootools-filemanager

As a file uploader is already included in Grocery CRUD library, another file manager will be at lower priority for this repo.
Thanks for the suggestion though :)


Upper limit of geocoding in Excel

Hi,
I just want to ask about the upper limit of geocoding in Excel.
For example, the Bing API can geocode up to 10k addresses from one account.
How about the Nominatim. Is it limited at some point?
Thanks

There doesn't seem to be an enforced rate limit. But they ask that we keep it to light use, specifically 1 request per second.
Bulk-geocoding is also discouraged, unless

it's a small task (often true in geocode-excel)
limit your requests to a single thread (true in geocode-excel)
limited to 1 machine only, no distributed scripts (including multiple Amazon EC2 instances or similar) (true in geocode-excel)
Results must be cached on your side (not yet implemented in geocode-excel)


Custom tooltip not working

The custom tooltip function seems to not be working properly, and the demo is not showing any tooltip: https://codepen.io/apexcharts/pen/NBdyvV
After digging into the code, it seems that the issue is that when calling https://github.com/apexcharts/apexcharts.js/blob/022f242b99629bdd4a12cd70e54aaec90cd87dbb/src/modules/tooltip/Labels.js#L27
The method arguments { i, j } are missing.
When added, it's ok.
Thanks.

Ouch! The last tooltip refactoring caused this.
Thanks for reporting. Don't hesitate to create a PR for these obvious issues ;)
Lack of tests may scare away some people, but I will add as I get more time.

Fixed in https://github.com/apexcharts/apexcharts.js/commit/cf8eb08be9c2ce3beeef8e0402117cb960e936b2

True! Will do next time. Thanks for the quick fix, as always :)


WIP Make it possible to use a variable number of clients

What does this PR change?
For the moment, not much... This is work in progress

The present flag was only used to avoid the usage of empty map. This flag doesn't have any impact, so Eric point one big problem: if soneone set the flag to false, it will take no effect. So removing the flag and just use empty map

The present flag was only used to avoid the usage of empty map. This flag doesn't have any impact, so Eric point one big problem: if soneone set the flag to false, it will take no effect. So removing the flag and just use empty map

The empty set is also a set... I don't think that anyone with computing background will be disturbed by contructs like
  client = {  }

Impossible to have a variable number of clients, reducing the focus.


chore: release main

:robot: I have created a release beep boop
accesskit_c: 0.2.0
0.2.0 (2023-04-01)
⚠ BREAKING CHANGES

Improve C bindings package directory structure (#239)

Bug Fixes

Improve C bindings package directory structure (#239) (44c27e7)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/AccessKit/accesskit/releases/tag/accesskit_c-v0.2.0 :sunflower:


[SCHEMA] Add objects and rules for sessions and scans files

Closes #978. This continues from Thursday's schema conversation.
Changes proposed:

Add object definitions for the scans and sessions files to objects/suffixes.yaml.
Create a new rules file for "tabular metadata files", such as participants.tsv, scans.tsv, and sessions.tsv files. The file is currently called tabular_metadata.yaml.

The proposed structure formalizes the assumption that presence or absence of allowed entities also determines where they should be located in the dataset (e.g., if a file requires the sub entity, then it MUST go in the subject subfolder).

Remove cases of required: false from rules/top_level_files.yaml, given that the default should be false moving forward.
Move object definitions in objects/top_level_files.yaml to objects/suffixes.yaml. These filenames are essentially suffixes without entities, even though some of them have underscores in the names (e.g., dataset_description), which are normally prohibited for suffixes.

I just think this organization is a little cleaner, overall.

To do:

[ ] Discuss the proposed changes.
[ ] Update the rendering code to work with the new rules.
[ ] Add a test or two for the new rules.

I was also working on this and trying to make it work with our validator.
I created a draft PR (including the suffix description from here): https://github.com/bids-standard/bids-specification/pull/987
I thought the suggestion was to treat these files similarly to datatypes and not create a separate YAML file for them, but rather have the hierarchy level defined by the presence of the subject and session entities.
If the subject or session entities are present, so are the respective directories — that seemed very elegant.

In as far as anybody finds this relevant at this point, this is how we could ~seamlessly integrate directory addition with all the other datatypes:
https://github.com/TheChymera/bids-specification/blob/e92d094a20135992cc4f9d8c7ee27e0c94bc1316/tools/schemacode/schemacode/validator.py#L81-L103
I've tried to base this code closely off of render.py and maybe we could use similar functions for both (or the same, if there's an easy way to output either regex or human-readable styling)... in any case this approach is usable with the logic seen here: https://github.com/bids-standard/bids-specification/blob/f8882915a55613b26d60d1789a75da037c8dd63a/tools/schemacode/schemacode/render.py#L152

If the suggestion was to treat these files similarly to datatypes and not create a separate YAML file type for them, perhaps the YAML documenting them can live inside the datatype directory? Might also be good as an intermediary step, i.e. those are files which behave the same in as far as for the time being others still do not.

I think I'd prefer to avoid having them in the datatypes folder and to instead pursue adding a datatype field, as proposed by (I believe) @effigies. Then, these files would either have datatype: None or just nothing, with the default being that datatype is None.

In as far as anybody finds this relevant at this point, this is how we could ~seamlessly integrate directory addition with all the other datatypes:
https://github.com/TheChymera/bids-specification/blob/e92d094a20135992cc4f9d8c7ee27e0c94bc1316/tools/schemacode/schemacode/validator.py#L81-L103

That's awesome! I haven't had a chance to take a close look at either of your code snippets, but I will try to soon.

I'd prefer to avoid having them in the datatypes folder and to instead pursue adding a datatype field

@tsalo so you'd like to flatten the datatype file hierarchy as well and have everything live under rules/ ?

I've tried to base this code closely off of render.pyand maybe we could use similar functions for both

Also, do you have any input on this? I think it would be cool if we could have a minimal validator which draws upon the BIDS documentation functions and is thereby in sync with it by design.

@tsalo so you'd like to flatten the datatype file hierarchy as well and have everything live under rules/ ?

@TheChymera I meant more that the subfolders would still exist, but for convenience, rather than as an actual source of information.

Also, do you have any input on this? I think it would be cool if we could have a minimal validator which draws upon the BIDS documentation functions and is thereby in sync with it by design.

That sounds like a good idea, and the code itself looks solid. I think the actual discussion of the minimal validator should probably be a part of the schema meeting, but using similar patterns for rendering and regex construction seems like a solid plan.

Closing this in favor of #1006.


Transformers do not work on imported grammars

I have a grammar that I want to use on its own (for parsing logical formulas), but also as a part of another grammar (so you can do something with the formulas), which I am currently doing by importing the frist grammar into the second. I have also written a transformer that transforms trees from the first grammar into the formulas, but in writing the second transformer I ran into a problem.
Because the second grammar imported the rules from the first grammar as formula.negation etc, the transformer is now looking for an attribute formula.negation when transforming a negation, which leads to the following two problems:

formula.negation is not a valid method name in Python, meaning that it is a bit hard to actually define this function.

Now I cannot simply do something like self.formula = FormulaTransformer() in the second transformer to use the first one, as the data property on the node suggests.

This makes dealing with transformers and imported grammars a bit counterintuitive, but this might also be expected behaviour.
If you would want to solve this, I would suggest splitting the data property on .s, and then seeing if there is a property for each of the found substrings, replacing line 28 of visitors.py with something like:
f = self
for attr in tree.data.split('.'):
    f = getattr(f, attr)

Fetch latest master. Imported rules are now module__rule, so they should be accessible to the Transformers.

Thanks, that did the trick! I was even able to reuse the old transformer by overriding __getattr__ and  checking for __ in the middle of the attribute name.


The output of default example is wrong

I used the default template to render the video and downloaded the video. The output is below.

@LLSean thanks for pointing this out! You can get a correct output when first fetching a github repository (say redotvideo/pluto), but nonetheless, the default should of course also be correct. I will start working on a fix today.

Thanks for flagging! This will be fixed with the next release.


Add CryptoFont

Is your feature request related to a problem? Please describe.
Currently some of the fonts provide minimal coverage of crypto fonts. As the market is growing in crypto, and I think there will be more demand to support a large set of currencies.
Describe the solution you'd like
I think adding CryptoFont would be great! It is open source and the developer seems to be actively adding in currencies as needed. We could add it as a typical font like the others.
Describe alternatives you've considered
I have a personal small repo, just getting started, that has the glyphs exported out already. Technically, I'd like to make this more automated with a script, since the current project does not seem to have have them exported in a json, only a css file.
My repo is here: react-native-vector-icons-cryptofont
Additional context
Happy to put in some elbow grease to get this font implemented.
Also, I have a issue opened at CryptoFont: https://github.com/monzanifabio/cryptofont/issues/16

We’ve just released a major new version with significant changes and improvements. If you’re still experiencing this issue after upgrading, please open a new issue with updated details. We're closing all the very old issues to get on top of things. Thanks for your patience!


chore: remove lock file flag for cargo build

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 51.634%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
711

Relevant Lines:
1377

💛  - Coveralls


CredentialsFromJSON fails without specifying scopes

Setting up firestorm using the google.CredentialsFromJSON is a bit messy. The scopes are specified in a internals files (which I'm not allowed to import). So I've had to copy and pate this code into mine to get the login to work.
scopes
// FirebaseScopes is the set of OAuth2 scopes used by the Admin SDK. var FirebaseScopes = []string{ 	"https://www.googleapis.com/auth/cloud-platform", 	"https://www.googleapis.com/auth/datastore", 	"https://www.googleapis.com/auth/devstorage.full_control", 	"https://www.googleapis.com/auth/firebase", 	"https://www.googleapis.com/auth/identitytoolkit", 	"https://www.googleapis.com/auth/userinfo.email", }
FYI, It works without specifying the scopes when loading from file, but I'm loading my json from an encrypted location, and I don't want to write the unencrypted value to disk.
The following example works, 'j.ServiceAccount' contains json key file text, admit the scopes and it will fail to login.
cred, err := google.CredentialsFromJSON(ctx, j.ServiceAccount,
		"https://www.googleapis.com/auth/cloud-platform",
		"https://www.googleapis.com/auth/datastore",
		"https://www.googleapis.com/auth/devstorage.full_control",
		"https://www.googleapis.com/auth/firebase",
		"https://www.googleapis.com/auth/identitytoolkit",
		"https://www.googleapis.com/auth/userinfo.email")

	if err != nil {
	   //logging
	}

	sa := option.WithCredentials(cred)
	app, err := firebase.NewApp(ctx, nil, sa)
	if err != nil {
		 //logging
	}

	client, err := app.Firestore(ctx)
	if err != nil {
		 //logging
	}
	defer client.Close()

	_, _, err = client.Collection("users").Add(ctx, map[string]interface{}{
		"first": "Ada",
		"last":  "Lovelace",
		"born":  1815,
	})
	if err != nil {
	    //logging
	}

Don't specify any scopes. firebase.NewApp() adds all the required scopes by default:
https://github.com/firebase/firebase-admin-go/blob/1d2a52c3c8195451b5ad2e0a173906bd6eb9529d/firebase.go#L138-L140

Follow up at https://github.com/google/google-api-go-client/issues/284


ADD HOPERS

@JeremyParish69 since we removed stuff  the token is not displaying anymore

did svg is mandatory ?


net: ipv4_autoconf: Fix requested IPv4 address in ARP packet

Unspecified address <IP_ADDRESS> was used as a requested IPv4 address
because the ARP message was generated second time. So for IPv4
autoconf ARP message, generate the message only once.
Signed-off-by: Jukka Rissanen<EMAIL_ADDRESS>

Codecov Report

Merging #9209 into master will decrease coverage by 0.01%.
The diff coverage is 10%.

@@            Coverage Diff             @@
##           master    #9209      +/-   ##
==========================================
- Coverage   52.75%   52.74%   -0.02%
==========================================
  Files         201      201
  Lines       25293    25296       +3
  Branches     5286     5288       +2
==========================================
- Hits        13344    13343       -1
- Misses       9820     9823       +3
- Partials     2129     2130       +1

Impacted Files
Coverage Δ

subsys/net/l2/ethernet/ethernet.c
51.19% <0%> (-0.16%)
:arrow_down:

subsys/net/l2/ethernet/arp.c
54.05% <33.33%> (-0.81%)
:arrow_down:

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 528f7f8...c2f8909. Read the comment docs.


Feature/Redis Trigger

Added Redis trigger support in addition to existing http in the shim.
I'll add integration test support for this and also the existing "spin-outbound-redis" in next PR.
We are currently using a private version of this shim in our repo to demo wasm/containers messaging using Redis in K3d. https://github.com/suneetnangia/wasm-orchestration-with-spin
P.S. Let me know if you want to make any changes. Thanks

All done @Mossaka. Thanks

I am going to merge this in and we can have follow ups resolved in future PRs.

lgtm!
Would you like to add the spin-inbound-redis image to the release pipeline? @suneetnangia

Can do, but was wondering if I should add integration tests for redis based images (both inbound and outbound (existing)) prior to release, to ensure images always work when we make changes?

@suneetnangia it's always a good idea to add more integration tests, specifically for redis based images.


Settings

Hi.
Can anyone share their settings for PHP, CSS and JS lint??
It can use the native PHP or nodejs,
Please do share, I will change the path to the correct one, matching my own pc

@ATouhou if you're looking for info from and interaction with fellow SL (and ST) users, the forum might be a better place: https://forum.sublimetext.com/


python3Packages.nose: fix cross-compiling

Description of changes
Nose was running the host platform python 2to3, which wouldn't work when cross-compiling
Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
22.11 Release Notes (or backporting 22.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[ ] Fits CONTRIBUTING.md.

Came up with something similar in #190167


Fix Mixin Errors + Update Traditional Chinese

Fixed the mixins due to deobf names. This is the only way I know to fix this and please tell me if there are any other ways to fix this.
Didn't update to pre4 as I tested this on pre4 and it works.
P.S. Updated Traditional Chinese

Oh modmuss50 says that pre4 contains the deobf names so this pr is kinda useless as you can just update to pre4, welp

Thank you anyway, I guess I'll just skip pre3 then.


Allow sources to fail individual results in a batch fetch

PREVIEW ONLY: STILL NEED TO WRITE DOCS AND TESTS
Some RPC systems (like rest.li used at LinkedIn) allow a batch response to contain both successful and failed entries. For example, if I fetch the profiles for ids 1 and 2, it's possible that 1 could return a profile and 2 will return an UnauthorizedAccess. Visualized:
Map(
  1 -> Entity(Profile),
  2 -> Error(UnauthorizedAccess)
)

Currently I have only two choices to deal with this:

Remove all error entries from the Map. But then you lose all the error information which makes it difficult to debug why things are missing.
If I need access to the error, I can fail the entire fetch, but then you lose the profiles that were successfully returned.

This change fixes that by introducing the ability to explicitly define a Try in the resulting map. If the Try is a Failure, then that Clump will be failed. Otherwise it will behave as normal.
Added a .sourceTry method to create these types of sources. Everything else stays the same. Now if you use .sourceTry then the example above changes to the expected behaviour:
profileSource.get(1) will return a Clump containing the Profile.
profileSource.get(2) fails the Clump with an UnauthorizedAccess exception.
A good side effect of this is that .sourceSingle now handles exceptions much better as well. If one call to the single source throws an exception it only fails that Clump instead of failing everything in that "batch" fetch. I also took some time to refactor the single source creation into a parameterize method following the pattern of the other types of sources.

In the future this means we can also add the ability to retry individual resources. So for example, if the batch fetch returns the following map:
Map(
  1 -> Entity(Profile),
  2 -> Error(DatabaseTimeout),
  3 -> Error(DatabaseTimeout)
)

We could send another fetch request with just keys 2 and 3.

It sounds like a good change. Would the retry mechanism also be changed to retry only the failed keys?

Well now there's two types of failures:
The whole request fails (as in the Future[Map[K,V]] is in a failed state, maybe some sort of network unavailable exception) or an individual value is marked as failed (marked as failed by the remote server).
I think there'd have to be separate retries for both.
In the case of sourceSingle it's sort of weird because individual entries and the entire request are always the same.
Do you think we need two different retry mechanisms then?

Considering that the retry mechanism receives a partial function and we can configure it for each type of exception, I think it is generic enough to be used for both individual and batched failures. I don't see need for a separate mechanism.

[x] Document .sourceTry method
[ ] Update README with new functionality
[ ] Add tests for individual failings in sourceSingle
[ ] Add tests for individual failings in sourceTry

@fwbrasil @williamboxhall - added tests and docs. This is ready for a review now whenever you have time.

I can't review in details, but it looks good to me.
:+1: if you need this asap


[Example] Named Entity Recognition using MT-DNN

Close stale issue. Will open new issue when we finish integrating MTDNN/SMART.


Question: Is there a way to list the models or check the model?

Hi! I want to check if a model is an existing one or to be able to list the available models using the SDK. Is there any method to do this? If not, any available API endpoint?
I went through the documentation but I couldn't find anything. I really appreciate any help you can provide.

Here is the documentation page where you can find the model list:
https://docs.anthropic.com/claude/docs/models-overview
and If you want to get the model list using Python SDK you can use the below code to list the models:
import inspect
import anthropic
models = inspect.signature(anthropic.Anthropic().messages.create).parameters['model'].annotation
print("Anthropic models", models)

@muhammadahmedzahid is correct. There is not an API to list models at this time.

This will actually get you a list, not a string like the suggestion above produces:
from typing import get_type_hints

list(get_type_hints(anthropic.Anthropic().messages.create)['model'].__args__[1].__args__)

However, both methods are quite brittle; a function signature change would probably break them.


Add Civo to Packaged Kubeflow distribution

I recently contributed the Kubeflow distribution to Civo Cloud: https://github.com/civo/kubernetes-marketplace/pull/342 . Here's a link to the Kubeflow distribution on Civo: https://www.civo.com/marketplace/Kubeflow
I would love to add this, if this sounds good?

@Rishit-dagli thanks for this.   I am interested in this offer.   Will you be adding it to the list Kubeflow distributions on this page ?  https://www.kubeflow.org/docs/started/installing-kubeflow/

@jbottum Yes, that is what I was intending. I will create a PR soon, please feel free to assign this issue to me

@Rishit-dagli if Civo is still interested in being listed on the "Installing Kubeflow" table, please raise a new PR that conforms to the new table format.
If you need an example, here is the PR which added deployKF to the table: https://github.com/kubeflow/website/pull/3542
Also, as @jbottum was saying, you will only be able to list the distribution in the table, we will not be accepting new documentation per https://github.com/kubeflow/website/issues/3074.


Need maintainers?

Hi there,
I notice this plugin hasn't been updated in a while... would it be a good idea to add some more maintainers to help?

I made a fork https://github.com/Cap-go/native-audio and will pay maintainer with Capgo business

You can join the capacitor community in discord and ask there if you are interested, I just updated a few things and merged an user PR that added Capacitor 6 support


Allow localizing front-end messages.

Hi.
The project uses a resource file for messages. This makes it very easy to add localized versions to the source code. A quick search reveals that some people really like to have the localization option.
I'm not an expert in .net localization, but I guess we have 3 options:
1: Add localized resource files in the same project as the main
2:Create localization assemblies.
3:Allow loose localized resource files.
Which one is the preferred approach?

@HaoK can you follow up for the Identity ask. For the Web app, we don't have a localization story yet.

@rustd @HaoK I guess I can contribute this. But I need to be briefed on your preferred method.

@AlirezaHaghshenas Have you taken a look at the IdentityErrorDescriber service?  That is what you might be able to plug in to localize identity specific application facing errors.

@HaoK  Ok, I'll give it a try. I'll ask you in case I'm unsure about the approach.

Which source should I make changes to? The V2.x in CodePlex or V3.x in Github?

Feature only exists in v3 (Github)
From: AlirezaHaghshenas<EMAIL_ADDRESS>Sent: Saturday, March 14, 2015 3:32 AM
To: aspnet/Identity
Cc: Hao Kung
Subject: Re: [Identity] Allow localizing front-end messages. (#379)
Which source should I make changes to? The V2.x in CodePlex or V3.x in Github?
—
Reply to this email directly or view it on GitHubhttps://github.com/aspnet/Identity/issues/379#issuecomment-80288338.


Use with flatpak

This is an odd one.  I tried to install Chrome via Flatpak ( https://flathub.org/apps/com.google.Chrome ).
You'll notice the command to execute it is flatpak run com.google.Chrome.  However, bing-rewards doesn't like this:
[user@host]$ bing-rewards -c 2
Doing 2 desktop searches
Unexpected error: [Errno 2] No such file or directory: 'flatpak run com.google.Chrome'
ERROR: Chrome could not be found on system PATH
Make sure it is installed and added to PATH,or use the --exe flag to give an absolute path

I tried putting the flatpak command in a script, but it errors differently:
[user@host]$ bing-rewards -c 2
Doing 2 desktop searches
Traceback (most recent call last):
  File "/usr/bin/bing-rewards", line 8, in <module>
    sys.exit(main())
  File "/usr/lib/python3.10/site-packages/bing_rewards/__init__.py", line 331, in main
    desktop()
  File "/usr/lib/python3.10/site-packages/bing_rewards/__init__.py", line 310, in desktop
    search(
  File "/usr/lib/python3.10/site-packages/bing_rewards/__init__.py", line 233, in search
    chrome = subprocess.Popen(
  File "/usr/lib/python3.10/subprocess.py", line 971, in __init__
    self._execute_child(args, executable, preexec_fn, close_fds,
  File "/usr/lib/python3.10/subprocess.py", line 1847, in _execute_child
    raise child_exception_type(errno_num, err_msg, err_filename)
OSError: [Errno 8] Exec format error: '/home/user/runchrome'

Any idea what "Exec format error" may mean?  It would help me get around this.

sigh  You know, this happened before.  I troubleshoot for a long time, come up short, so I open an issue, then suddenly I find an answer.
To solve, make the script to run the flatpak look like this:
#!/bin/sh
flatpak run com.google.Chrome

Ok, I am glad you found a fix.
This issue is due to the way the python process launches a new chrome instance. It is expecting to launch an executable called "chrome" or similar directly. It expects a path to a file on the system.
When run a Flatpak app, you actually are launching the flatpak program and passing command line parameters 'run' and 'com.google.Chrome'. I don't use flatpak myself, but I am assuming that additional arguments to the flatpak command are passed to the running process.
You may need to format your script like flatpack run com.google.Chrome -- $@ or similar depending on how flatpak handles arguments sent to the launched process. This will be necessary to launch chrome with the --user-agent flag that the script uses to set a mobile user agent.


Create install_Docker_en.md

翻译Install_Docker_en.md英文文档

建议全部改完统一提一次commit

 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.

 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.


Recording local video on ios

Hi,
we are currently trying to record local video tracks (TVILocalVideoTrack), but i guess there are some missing methods.
...
var localVideoTrack: TVILocalVideoTrack?
...

    func startRec() {
        let documentsURL = FileManager.default.urls(for: .documentDirectory, in: .userDomainMask)[0]
        let filepath = documentsURL.appendingPathComponent("temp2")
        var recordingDelegate:AVCaptureFileOutputRecordingDelegate? = self

        videoFileOutput = AVCaptureMovieFileOutput()

        self.captureSession?.addOutput(videoFileOutput!)

        videoFileOutput?.startRecording(to: filepath, recordingDelegate: recordingDelegate!)
    }

when i try to addOutput to localVideoTrack i got an error.
We also found similar problem #https://github.com/twilio/video-quickstart-swift/issues/123
but its kindly different.
Any help will be appreciated.

Hi @swift2geek,
You can add a TVIVideoRenderer to TVILocalVideoTrack using addRenderer: for the purposes of recording. This interface is fairly generic and not related to AVCaptureSession APIs offered by Apple.
There is some excellent information in https://github.com/twilio/video-quickstart-swift/issues/123 looking at recording remote video Tracks using AVAssetWriter.
I also provided a code snippet which demonstrates using TVIVideoRenderer to record still frames from a TVILocalVideoTrack in https://github.com/twilio/video-quickstart-swift/issues/142.
Hopefully this is helpful to you, but I can provide more info if needed. I also have some WIP code for Video Track recording here: https://github.com/ceaglest/video-quickstart-swift/blob/task/renderer-example/VideoRendererExample/VideoRenderers/ExampleVideoRecorder.swift
Best,
Chris

Hey @swift2geek,
Were you able to get recording up and running? Is there anything else I can help with?
Best,
Chris

Hi
Did this discussion continue since then? Is there currently a solution for recording the local video?
Thanks

@lucasalgarra As @ceaglest mentioned on this thread, we have VideoRendererExample PR which demonstrates writing your own custom renderer. We plan to add more example quickstarts in the future but, at present, we do not have a sample app for custom video renderer, so I recommend using VideoRendererExample PR as a reference to implement your used case.
Let me know if you have any questions.

I am sorry @lucasalgarra, I am kicking myself for not getting the PR merged and officially adding the VideoRendererExample. The ExampleVideoRecorder class in the PR is a good starting point, and should be fine for local video. For remote video that is VP8 or VP9 the recorder won't support the I420 pixel format. If the local video is from the camera then it will be NV12 and is supported.
We generally recommend using Group Rooms for audio/video recording since they have a rich set of APIs for managing recordings and composing audio/video Tracks.

Closing an old ticket. Feel free to reopen or file another ticket if you run into any issues.

Closing an old ticket. Feel free to reopen or file another ticket if you run into any issues.


chore(sdk): clean up targets in Makefile

Description
Added make lint to sort imports, format the code, and execute flake8 and mypy.
Testing
Manually ran make lint.
Checklist

[x] Include reference to internal ticket "Fixes WB-NNNN" and/or GitHub issue "Fixes #NNNN" (if applicable)
[x] Ensure PR title compliance with the conventional commits standards

closing as stale


chore(docs): update logger chart to deis-logger-beta1

ping @sgoings; @mboersma mentioned you have a release checklist hiding somewhere. Mind adding this to the checklist? :)

superseded by #173


Sync java options in example protos

This makes the protos identical between this repository and the java
repository.
Fixes grpc/grpc-java#1381

Can one of the admins verify this patch?

Can one of the admins verify this patch?

FYI, the copyright year shouldn't be bumped just because the file is changed. It's meant to be when the file was first created.

@dsymonds, that disagrees with check_copyright.py:

examples/protos/helloworld.proto modified 2016 but copyright only extends to 2015
examples/protos/route_guide.proto modified 2016 but copyright only extends to 2015

I don't know who made that check, I'm just trying to follow the rules.

Maybe it's trying to tell you to write 2015-2016 instead?
But the check is broken. You can read go/copyright (internally) or http://www.copyright.gov/eco/help-publication.html.

@dsymonds, updated to 2015-2016. Reading the script it appears to support the year range and I see multiple other usages of that in the codebase. Is 2015-2016 correct in your mind?
Based on the link you provided, it seems that 2016 would be correct:

For a revised version, the publication date should be the date the revised version was first published, not the date the original version first appeared online.

and:

Give the year in which creation of this work was completed. Also, this should be the year this particular version was completed. Do NOT refer to earlier or later versions.

If the check is still broken in your mind, would you mind filing an issue? I would, but I don't think I'm knowledgeable enough to do so.

Yes, 2015-2016 would be fine, though IMO leaving it as 2015 would have been fine too.
The relevant phrase is Give the year in which creation of this work was completed., and it was completed last year. You're just making a change to what was previously considered completed; it wasn't "incomplete" last year.
Happy to file an issue, but I don't know where. Who owns check_copyright.py?
@dberlin might have words to say.

@dsymonds, an issue in this repo looks good. At the very least that'll help track down where it came from. But it seems it might have been created by @ctiller. We can discuss in an issue once we have one.

Sure. Filed #5051.

You shouldn't bother updating dates. It's pointless, and if we are still
using this in 80 years, we should all feel bad :)
On Wed, Feb 3, 2016 at 3:06 PM, David Symonds<EMAIL_ADDRESS>wrote:

Yes, 2015-2016 would be fine, though IMO leaving it as 2015 would have
been fine too.
The relevant phrase is Give the year in which creation of this work was
completed., and it was completed last year. You're just making a change
to what was previously considered completed; it wasn't "incomplete" last
year.
Happy to file an issue, but I don't know where. Who owns
check_copyright.py?
@dberlin https://github.com/dberlin might have words to say.
—
Reply to this email directly or view it on GitHub
https://github.com/grpc/grpc/pull/5041#issuecomment-179521357.


add changie and start linting

Codecov Report

Merging #75 (5ca2d89) into main (377fa37) will increase coverage by 5.09%.
The diff coverage is 6.35%.

@@           Coverage Diff            @@
##            main     #75      +/-   ##
========================================
+ Coverage   0.00%   5.09%   +5.09%
========================================
  Files          5       9       +4
  Lines        309     530     +221
========================================
+ Hits           0      27      +27
- Misses       309     500     +191
- Partials       0       3       +3

Impacted Files
Coverage Δ

mathf/bounds.go
0.00% <0.00%> (ø)

mathf/ease.go
0.00% <0.00%> (ø)

mathf/gametime.go
0.00% <0.00%> (ø)

mathf/ticker.go
0.00% <0.00%> (ø)

mathf/timer.go
0.00% <0.00%> (ø)

mathf/transform.go
0.00% <0.00%> (ø)

mathf/tween.go
0.00% <0.00%> (ø)

mathf/vec2.go
0.00% <0.00%> (ø)

mathf/misc.go
71.05% <71.05%> (ø)

graphics/label.go

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 e5b00b7...5ca2d89. Read the comment docs.


[BUG]: No full streams found!

Program version
5.1.5
Operating System
Windows
Type
GUI
Service
Crunchyroll
Command used
None
Show ID
G6J0G49DR
Episode
14
Console Output
[S2E1] - Last Chance [✓ Japanese, ✓ English, ✓ German, ✓ Portuguese, ✓ Spanish, ✓ Hindi, ✓ Russian, ✓ French]
Requesting: [G2XU07K9E] Tower of God Season 2 - 1 - Last Chance
Chapter request successful
[WARN] No full streams found!
[ERROR] Download error: Unable to download episode 14 from G6J0G49DR

Additional Information
I used the GUI one. A few days ago, it worked fine but yesterday it began acting up and now that is the message I get.

You are missing a CDM or mp4decrypt

Thanks. But now it won't work again.


Distinct query breaks typed collections

for {
  collection <- db.getCollection("name")
  res <- collection.distinct("column").first
} yield ()

This will fail if the column is not a Document which I'd expect it normally is not. So I ended up having to do this to make it work:
for {
  collection <- db.getCollection[String]("name", MongoDatabaseF.DefaultCodecRegistry)
  res <- collection.distinct("column").first
} yield ()

The issue with this is that now the type of the collection has to change to conform to the distinct query, so I can't do a find using the same collection. It seems like it might be a good idea to make aggregations pass a Decoder implicit for distinct that will decode the BsonValue into the required type.

Turns out mongo's distinct supposed to return only values (and not the entire document as I mistakenly though) for a specified field across a single collection.
I've updated distinct implementation to allows specifying the type of the returned field. In addition to that I also added a method for adding more codecs to the collection's codec registry.
val result = for {
  coll   <- db.getCollectionWithCirceCodecs[Person]("people")
  _      <- coll.insertMany[IO](List(person("John", "Bloggs"), person("John", "Doe")))
  addresses <- coll.withAddedCirceCodecs[Address].distinct[Address]("address").all[IO]
} yield addresses

sweet! Also I found an OOM bug that I can submit a PR for if you like. It may not be implemented the way you want, but can at least show you.

Though it would be cool if distinct required the Codec so that you can't call distinct unless you have a deserializer for that type. If you forgot to call withAddedCirceCodecs it would fail at runtime.

Yes, that would have been nice. Tbh, the only reason why I didn't do it the way you've described is because I couldn't come up with a neat name for that method (needs to be different from distinct).
I'll think about it and try to do something tonight 👍

I followed your suggestion and added another method for doing distinct query with codecs:
for {
  coll <- db.getCollectionWithCodec[Person]("people")
  _    <- coll.insertMany[IO](List(person("John", "Bloggs"), person("John", "Doe")))
  distinctAddresses <- coll.distinctWithCodec[Address]("address").all[IO]
  _                 <- IO.println(distinctAddresses)
} yield ()


Update words snippet for JavaScript

Ignore any special characters instead of only \W. This adds support to any languages. Not only English.

@VasiliyRusin Unfortunately I don't host the snippets. They are taken from another open-source repo. Here is the original file https://github.com/30-seconds/30-seconds-of-code/blob/master/snippets/words.md
If you'd like to make changes to snippets, that's where you should go and open a PR.


Move host IO to coordinate (0,0)

This PR resolves issue #61
Tested with PR #30 of bsg_manycore repo.

@leonardxiang This is a lot of changes for just moving the host interface. Was anything else included in this commit?

@leonardxiang This is a lot of changes for just moving the host interface. Was anything else included in this commit?

Hi, Dustin,
This PR is on top of the PR #207 which is the priority 0 as we discussed yesterday.


Simple Piano Using java gui

Simple Piano Using java gui #4998

Close #4998

Checklist:

[x] I have mentioned the issue number in my Pull Request.
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have created a helpful and easy to understand README.md

@MohitGupta121 @pranjay-poddar Review this pr

@MohitGupta121 @pranjay-poddar Can you review my pr and merge it.


trig functions

The usual suspects are really needed.
cos, sin, tan, arccos, arcsin, arctan, and atan2.

yep! they are on my TODO list. They likely will be implemented similarly to the Tan example; i.e., as continued fractions.


WIP: Upload packages along with AWS advanced templates

High Level Description
This includes packages in the artifacts that get uploaded along with generated AWS templates.
This also fixes an issue where failed downloads to tempfile locations during node setup cause an incidental error that hides the original download error.
Related Issues

DCOS-14381 Various reports of failure when using templates generated by --aws-cloudformation

Checklist for all PR's

[ ] Included a test which will fail if code is reverted but test is not. If there is no test please explain here:
[x] Read the DC/OS contributing guidelines
[x] Followed relevant code rules Rules for Packages and Systemd

Checklist for component/package updates:
If you are changing components or packages in DC/OS (e.g. you are bumping the sha or ref of anything underneath packages), then in addition to the above please also include:

[ ] Change log from the last version integrated (this should be a link to commits for easy verification and review): example
[ ] Test Results: [link to CI job test results for component]
[ ] Code Coverage (if available): [link to code coverage report]

@mesosphere-mergebot bump-ee

Ready for review @amitaekbote @orsenthil.


Messy 1.9.4 port.

TTF font rendering still isn't there, but it runs on 1.9.4, now.

Does this port currently function in the same way as the original, or is this just a partial port to 1.9x?


Option for english translation?

It would be ideal, if there was an english translation for the player.
Cheers..

Done. https://github.com/DIYgod/DPlayer/commit/4cf303ce26f691be5e3a3722bce6151067671375


Fixed: Card content is hidden behind the tab-bar (#289)

Related Issues

#289
Short Description and Why It's Useful

changed padding for main in find facility , find group and parking , variable css was overriding , defined for each page specifically , verified changes for different resolutions
Screenshots of Visual Changes before/after (If There Are Any)

Contribution and Currently Important Rules Acceptance

[x] I read and followed contribution rules

@ymaheshwari1


Remove sourcery

What
As per #76, this removes Sourcery templates and code in favor of Swift 4.1 automatic Equatable conformance.
Why
Code generation for Equatable is not necessary anymore as Swift 4.1 features automatic conformance for Equatable. While Sourcery is reliable, using native Swift auto-conformance makes for a simpler toolset when getting started with Danger, and is significantly less likely to require maintenance between major Swift versions.
How

Remove templates, sourcery.yml, AutoEquatable protocol, and conformance declarations
Add Equatable with Xcode's regex find and replace as below

Concerns

Adding Equatable conformance in the Test target did not work, meaning it had to be added directly in the Main target

This adds a small amount of clutter to the model objects affected

Some model objects feature an "ID" or otherwise unique property that is technically the only parameter needed to define equatability

These model objects may have this more technically "correct" conformance added, causing tests to miss all the other parameters
For now, I believe this doesn't present a major problem, but it may cause tests coverage to drop dramatically if not watched
One solution may be to write a Danger rule that's triggered by func == (rhs: ...), reminding the developer that the tests require a workaround

CI failing at Danger JS, but I don't understand what "Danger JS did not include fails" means. Perhaps the config file has some kind of error?
Error:  Error: Results passed to Danger JS did not include fails.
{}
    at /Users/travis/.nvm/versions/node/v6.12.3/lib/node_modules/danger/distribution/dsl/DangerResults.js:20:19
    at Array.forEach (native)
    at Object.validateResults (/Users/travis/.nvm/versions/node/v6.12.3/lib/node_modules/danger/distribution/dsl/DangerResults.js:16:24)
    at Executor.<anonymous> (/Users/travis/.nvm/versions/node/v6.12.3/lib/node_modules/danger/distribution/runner/Executor.js:162:41)
    at step (/Users/travis/.nvm/versions/node/v6.12.3/lib/node_modules/danger/distribution/runner/Executor.js:32:23)
    at Object.next (/Users/travis/.nvm/versions/node/v6.12.3/lib/node_modules/danger/distribution/runner/Executor.js:13:53)
    at /Users/travis/.nvm/versions/node/v6.12.3/lib/node_modules/danger/distribution/runner/Executor.js:7:71
    at __awaiter (/Users/travis/.nvm/versions/node/v6.12.3/lib/node_modules/danger/distribution/runner/Executor.js:3:12)
    at Executor.handleResults (/Users/travis/.nvm/versions/node/v6.12.3/lib/node_modules/danger/distribution/runner/Executor.js:158:16)
    at Object.<anonymous> (/Users/travis/.nvm/versions/node/v6.12.3/lib/node_modules/danger/distribution/commands/utils/runDangerSubprocess.js:108:47)
The command "DEBUG="*" danger process .build/x86_64-apple-macosx10.10/debug/danger-swift" exited with 1.

This feels like a danger-js bug tbh

Seems to be ok on the Linux CI and my local machine

I shipped an update with danger-js for some improved logging, but I don't think that should block this PR


Update text and URLs for analytics.md

Description
Update text for better translation.
Update URL for the correct analytics dashboard.
Issue
Closes: #

/retitle [pt-br] Update docs/contribute/analytics.md

/cc


Minimal factions functionality.

I'd like to have a way to manage people in groups, and there aren't many good plugins out there that do that very well but I can see Essence having a good place with this feature.
The commands could be something like these:

/essence:group create [name] ~ creates a group with the name if that group name is not taken
/essence:group join [name] ~ sends a request to join the group (group members only, group creator would be told that they cannot join their own group)
/essence:group kick [name] ~ kicks a player out of the group (group creator only)
/essence:group rule allowFriendlyFire true ~ allowFriendlyFire is an example name but this would be a great feature to have friendly fire built-in, if true is not provided then it would just provide the current state of the rule. Rule configuration can either be available for OPs or just normal players depending on how people want their server to be.

I hope this is a good suggestion.

Brilliant idea! I’ll get that in the next update - could take some time though!

Some other commands may include:

/essence:group home [name]
/essence:group sethome [name]
/essence:group delhome [name]
/essence:group homes

Some other commands may include:

/essence:group home [name]
/essence:group sethome [name]
/essence:group delhome [name]
/essence:group homes

Opened as #23

Hi, I've uploaded a snapshot version (may contain bugs) to Hangar, Modrinth, and GitHub. If you give it a try let me know what you think. https://hangar.papermc.io/LewMC/Essence/versions/1.3.0-SNAPSHOT


How to avoid the import path update disaster when upgrading major versions?

In our code, we start to use Go modules for v2. Then we had a breaking change (https://github.com/casbin/casbin/pull/472#issuecomment-638114123) and based on Semver, we need to bump to v3. But this requires all our source code files to update all import paths from v2 to v3. And it also requires all our users' code that imports our library to change their import paths from v2 to v3. This is a disaster. And our users are not happy: https://github.com/casbin/casbin/pull/478#issuecomment-637548747 It not only requires a lot of work for each major release, but also makes the git history looking bad.
For other package managers like npm (node.js) or pip (python), it just needs to update the version in the dependency file, no need to update source code.
Can we have a good solution to avoid this import path update disaster for major version releases?

Expect
casbin go.mod
module github.com/casbin/casbin

Let's say v10.0.0 has been released in GitHub.
Developer can get the dependency by go get github.com/casbin/casbin@v10 and import it into own project by import "github.com/casbin/casbin.

Expect
casbin go.mod
module github.com/casbin/casbin

Let's say v10.0.0 has been released in GitHub.
Developer can get the dependency by go get github.com/casbin/casbin@v10 and import it into own project by import "github.com/casbin/casbin.

What about large projects that use various versions of a package?
Some packages break compatibility but instead migrate to newer version instantly we import both modules with different versions and add a notice to users that function is deprecated and will be dropped in X versions of the program.
This is why we use the import with vX suffix in the source code, this allows to use two or more versions, example:
import casbinV1 "github.com/casbin/casbin when v1
import casbinV2 "github.com/casbin/casbin/v2 when v2
And go.mod:
require (
    github.com/casbin/casbin v1.0.1
    github.com/casbin/casbin/v2 v2.0.5
)

Take this into account before make a proposal.

Why we doesn't  consider using github.com/casbin/casbin replace version with vX suffix and support multiple versions?
go.mod
require (
    github.com/casbin/casbin v2.0.5
    // github.com/casbin/casbin v1.0.0
    github.com/casbin/casbin/v2 v2.0.5
    github.com/casbin/casbin/v3 v3.0.0
)

There have been other issues on this topic before; have you tried searching?
I would also suggest avoiding dramatic words like "disaster" when filing issues, so that we can just focus on the topic at hand.

Relate issues: https://github.com/golang/go/issues/32014, https://github.com/golang/go/issues/32816
There is a lot of work to do from vX to v(X+1), so I have a bit puzzle, how to update dependency paths when major releases are released :(

Also #25518 is related.
Personally automates the version bump can be a nightmare. Not all developers breaks the compatibility, some deprecate functions before make code obsolete.

@mvdan sorry, I have fixed my wording. Like @nodece @xhit said, I also noticed those issues but didn't get a very clear solution for it. Any advice? Or will Go fix this issue in future?

@hsluoyz Does #38762 address your concerns?

@toothrot It seems to meet our expect.

I would like to present my ideas here on how to install dependency of different version here.
Let's assume there are multiple versions like v1, v2, module is github.com/nodece/casbin.
Expect
things1
If the user does not install this dependency, you can get the latest version of v2 by go get github.com/nodece/casbin and import it by github.com/nodece/casbin,  but it gets v1 version now :(
I hope that get the dependency by go get github.com/nodece/casbin then convert module to  github.com/nodece/casbin.
things2
How resolve the import with vX suffix in the source code, example:
import casbinV1 "github.com/nodece/casbin when v1
import casbinV2 "github.com/nodece/casbin/v2 when v2

I hope that the get the dependency by go get github.com/nodece/casbin@v2 then convert module to  github.com/nodece/casbin/v2.
I hope that the get the dependency by go get github.com/nodece/casbin@commit_id then convert module to  github.com/nodece/casbin.

I'm not sure I understand. Do you mean that the package import path (in .go files import statements), the URL, and the module URL should refer to different packages?
One of the primary design goals of modules is that different major versions are different packages to add clarity around breaking API changes.

One of the primary design goals of modules is that different major versions are different packages to add clarity around breaking API changes.

I see, but we need to update import path (in .go files import statements) when major version has been release.
It looks unfriendly :(

Hi there,
Unlike many projects, the Go project does not use GitHub Issues for general discussion or asking questions. GitHub Issues are used for tracking bugs and proposals only.
For asking questions, see:

Stack Overflow with questions tagged "go"

The Go Forum, a web-based forum

Gophers Slack, use the invite app for access. The #general channel is a good starting point.

Go Community on Hashnode with questions and posts tagged with "go"

The golang-nuts mailing list

IRC channel #go-nuts on Freenode

Please ask the question on one of the above forums.
(Quoted from https://golang.org/wiki/Questions)


Add notifyStash global function

Thank you for this pull request! Please check this document for how the Jenkins project handles pull requests.

@mscharp CREDENTIAL_ID_IN_JENKINS seems like bad naming for a variable. Wouldn't be more appropiate to use something containing SLACK_TOKEN ?

As mentioned in the comment above that section, it's not a variable, it's a placeholder.

@orrc in this case is even worse, putting credentials inside the SCM is a clear no-no. Please try to find a way to keep credentials out of the code that goes into the repository.
Read http://arstechnica.co.uk/security/2016/04/hacking-slack-accounts-as-easy-as-searching-github/ to get an idea about how bad this is.

Also have a look at my approach  at https://github.com/jenkinsci/pipeline-examples/pull/26/files -- clearly not ideal but at least it does not put the token inside the code.
I think that using environment variables or Jenkins API to access these credentials are both "good enough" security approaches. Still, inviting people to put their tokens inside Jenkinsfile, would clearly be not a good idea.

This code isn't actually putting credentials in the code.  There is a "withCredentials" plugin that will use the username/password that has been set up in Jenkins.  The CREDENTIAL_ID_IN_JENKINS is just a placeholder for the hash key that you access those credentials by.  So, whoever uses this code would replace it with a hash string.  I've attached an image of what the credentials look like in Jenkins.

And, you are right.  There are other ways to access the credentials that you'd need.  However, having it hardcoded into this script isn't a big concern for us, as the user associated with those credentials doesn't have rights to do much.

Ahh, sorry that would have been ok because this ID is useless outside Jenkins. I had the impression that was the slack token.

I'd guess this is a workaround until the following is implemented? https://github.com/jenkinsci/stashnotifier-plugin/issues/92

@mscharp how can I use this in a Jenkinsfile?

@mscharp perhaps the commit sha bit can be integrated in the function, see https://github.com/jenkinsci/pipeline-examples/blob/912c73791fdc1a6faa1ab159c4719e33eba18688/pipeline-examples/gitcommit/gitcommit.groovy

Even with the stash notifier plugin working (and supposedly it's working now), I find this is a very interesting example. Maybe make it more generic, like "calling a REST interface with build results"? The actual example can stay pretty much the same.

Note, the http-request-plugin has a Pipeline step and allows POST a request body (JSON or whatever). Furthermore, it makes it much easier to handle the response from your REST call.

I was wondering how to do a POST w/ auth info securely as well, so an example of this would be helpful.


new version support

1.19.1?

Yes it has been updated: https://cdn.discordapp.com/attachments/918649771844042752/1002310019976532098/essential-client-1.19.1-1.2.4-BETA-280722.jar
If you want to keep up with future pre releases I post them on my discord :)

thanks :)


[Tooltip] Fix disableTouchListener behavior

[x] I have followed (at least) the PR section of the contributing guide.

The mouseOver event was making the tooltip visible (on a touch) with disableTouchListener=true
Closes #20245

I realized which this diff only work for the first render. I'm trying to get a better approach

@weslenng Does the latest version of the pull request match your expectations?

@weslenng Does the latest version of the pull request match your expectations?

It solves the problem, so yes.
I was kind of sad that I couldn't get to the solution 😥

@weslenng Don't worry, it's an opportunity to see what was missing :)


@yuki24 => Markdown supported SuperArticle footer

Related to: https://github.com/artsy/positron/issues/1116 and https://github.com/artsy/positron/pull/1123
We need to add markdown support in the "Super Article" footer for an upcoming sponsorship. Here's an example of a super article.

LGTM 👍🏻


Aliases don't take CIDR notation for networks anymore

Important notices
Before you add a new report, we ask you kindly to acknowledge the following:

[X] I have read the contributing guide lines at https://github.com/opnsense/core/blob/master/CONTRIBUTING.md
[X] I am convinced that my issue is new after having checked both open and closed issues at https://github.com/opnsense/core/issues?q=is%3Aissue

Describe the bug
The add or edit dialog for aliases doesn't take the CIDR notation for networks anymore.
To Reproduce
Steps to reproduce the behavior:

Go to 'Firewall->Aliases'
Click on '+ for adding a new or edit an existing'
Add or change a network wird CIDR notation in the field content
See error Entry "<IP_ADDRESS>/24" is not a valid hostname

Expected behavior
The add or edit dialog accepts networks in CIDR notation as described here: https://docs.opnsense.org/manual/aliases.html.
Describe alternatives you considered
Screenshots

Relevant log files
Additional context
Environment
Software version used and hardware type if relevant, e.g.:
OPNsense 22.7.4 (amd64, OpenSSL).

wrong type, choose network. a host is an address without a subnet (https://docs.opnsense.org/manual/aliases.html#hosts)

Thanks for not laughing at me.