Add html_root_url to documentation.

When generating docs with cargo doc, it is often preferred to link external dependencies to existing hosted documentation (docs.rs) instead of regenerating the docs. Currently, running cargo doc leaves users with the option of either generating all documentation for gdnative (which is around 200M), or not including links to gdnative. Neither of these options is particularly desirable as one duplicates documentation and the other doesn't include document links.
Ideally, when users run cargo doc, they should only generate their own documentation, and have http links to external documentation on docs.rs (or wherever documentation is located).
Adding html_root_url to the project would allow users of this library to directly link to docs.rs: https://doc.rust-lang.org/rustdoc/the-doc-attribute.html?highlight=link#html_root_url.
Note: There is an open ticket to default to checking docs.rs, but that is old and doesn't seem to be actively developed: https://github.com/rust-lang/rust/issues/42301

Thanks for the suggestion! Honestly, I'm not sure how "often preferred" and "ideal" that is. I myself, for instance, have always preferred all my documentation local, simply because it's faster that way, and 200M isn't really that large for a modern computer. Truth to be told, I never for once cared how large the documentation was.
As a result, I'm hesitant to support this proposal since it has the potential to be controversial. I would prefer to let this gather more opinions before doing anything. Are there any "big" libraries that have this behavior by default?

Interesting. My perspective is coming from uploading the documentation to
S3/Pages for viewing. For uploading reasons, you'd want it to be small, but
I see where you are coming from. Seems like this is more of a rustdoc thing
than a Godot thing.
On Mon, May 4, 2020, 20:17 toasteater<EMAIL_ADDRESS>wrote:

Thanks for the suggestion! Honestly, I'm not sure how "often preferred"
and "ideal" that is. I myself, for instance, have always preferred all my
documentation local, simply because it's faster that way, and 200M isn't
really that large for a modern computer. Truth to be told, I never for once
cared how large the documentation was.
As a result, I'm hesitant to support this proposal since it has the
potential to be controversial. I would prefer to let this gather more
opinions before doing anything. Are there any "big" libraries that have
this behavior by default?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/GodotNativeTools/godot-rust/issues/339#issuecomment-623811348,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AEFGXF4T2RKQLWAC3OBV2K3RP6AM7ANCNFSM4MZFKIFQ
.

That's certainly a rare use case, then. You may make a local change on your own copy if you want to, but I'm afraid I can't see this getting into upstream, unless it's shown to be a more common practice among well-known crates, or if more users show their interest in it.
I'm keeping this open for now, but feel free to close the issue if you no longer need it in upstream.


nodePackages: update to latest

Description of changes

Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] 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/)
24.11 Release Notes (or backporting 23.11 and 24.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

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Result of nixpkgs-review pr 336926 at 81213480 run on x86_64-linux 1

5 packages failed to build:

discourse
discourseAllPlugins
styx
styx.lib
styx.themes

47 packages built successfully:

akkoma-frontends.akkoma-fe
authentik
commitlint
create-cycle-app
elmPackages.elm-doc-preview
elmPackages.elm-pages
elmPackages.elm-spa
emanote
emanote.data
emanote.doc
emojione
fast-cli
homepage-dashboard
image_optim
joplin
lessc
lighthouse
lunarvim
mastodon-bot
matrix-appservice-irc
mpvScripts.mpv-cheatsheet
mx-puppet-discord
node2nix
postcss-cli
pscid
pulp
shout
sloc
theLoungePlugins.plugins.closepms
theLoungePlugins.plugins.giphy
theLoungePlugins.themes.flat-blue
theLoungePlugins.themes.flat-dark
uppy-companion
vimPlugins.coc-docker
vimPlugins.coc-go
vimPlugins.coc-markdownlint
vimPlugins.coc-metals
vimPlugins.coc-r-lsp
vimPlugins.coc-rust-analyzer
vimPlugins.coc-sh
vimPlugins.coc-spell-checker
vimPlugins.coc-stylelint
vimPlugins.coc-sumneko-lua
vimPlugins.coc-tslint
vimPlugins.coc-vetur
vscode-extensions.rust-lang.rust-analyzer
wasm-text-gen

discourse and styx are broken on master.
I will merge it as soon as ofborg's evaluation is done. Do not merge PRs related to node-packages, since it will cause conflicts again.


Clean up font sizes on cards

Currently we're using two font sizes for small text on cards, somewhat arbitrarily:

Description on featured banner: 14px
"App of the Day": 16px
App of the day description: 16px
"Get the latest versions of all your favorite apps, for any Linux distribution": 16px
Tabs: 16px
Title on regular app cards: 16px
Summary on regular app cards: 14px

It'd be good to clean this up a bit, either by using the same size for all of them or by having a clear rule which size to use where.

I think that the font being bigger on the hero/highlight elements then on the app cards is fine.
The only thing I think that would make sense, is aligning description on featured banner to 16px to harmonize it between the other description texts.

I think that the font being bigger on the hero/highlight elements then on the app cards is fine.

Not sure, I'd have tried going the other way and making them all 14, that'd also help to avoid strings wrapping too early e.g. on the app of the day card. The question is at that point, why not do it for the tabs as well?

I think it's fine like this. The only one that's "surprising" might be the Get the lastest versions... text, as that is slightly bigger compared to other description texts.


Move Cypress tests about the Wikidata extension to the extension's directory

We now have Cypress tests which cover parts of the Wikidata extension. They are currently located in main/tests/cypress/cypress/integration/extensions/wikidata.
We should ideally have them in extensions/wikidata, so that they are clearly separated from the core software.
This is related to the project of moving this extension to an external repository (#5282), which will require this.
Running the Cypress tests in the external repository will require running OpenRefine there, so the setup will probably be quite different.

Those tests depend on main/tests/cypress/cypress/support/ext_wikidata.js which should also be migrated.


Review of Advance Network Vignette

Codecov Report

Merging #7 (d912c5a) into main (5c98904) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main       #7   +/-   ##
=======================================
  Coverage   94.84%   94.84%
=======================================
  Files          21       21
  Lines        2481     2481
=======================================
  Hits         2353     2353
  Misses        128      128

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 5c98904...d912c5a. Read the comment docs.


Add Arrowheads to Arrows

This PR adds start and end decorators to arrows.

A user can add decorators to arrow elements.
Each arrow element can have a start and end decorator.
A decorator may be null, arrow, dot, or bar.
By default, an arrow uses only an end decorator of arrow.

Notes

I've implemented a strategy to correctly render arrows without decorators, such as those from earlier projects; however I'm sure there is an existing approach to handle this sort of thing. I appreciate any notes or guidance here!
This PR also includes small changes to the icons in the panel.

How about a dropdown instead of cycling?

For dropdown we have to make a custom element to support icons and it will be pain to make it for mobile.. can we have the first iteration of this feature in that form?

Also for the first version I would just support the arrow and not the rest, and in a separate PR choose which endings we want to add as a start.

Also for the first version I would just support the arrow and not the rest, and in a separate PR choose which endings we want to add as a start.

I think we can leave it if it's implemented already, at least it will force us to implement the dropdown sooner.
One thing that bothers me a bit is that when you remove caps and then create new arrows it remembers the setting and effectively makes new arrows into lines. I'm not sure whether we should do anything about it though.

Thanks for the feedback.

Also for the first version I would just support the arrow and not the rest, and in a separate PR choose which endings we want to add as a start.

@lipis I've gone ahead and removed the other two decorators as you suggested. This PR now only proposes the ability to add or remove arrow decorators to the start and end of an arrow.
I believe this approach was also suggested in the other discussion. However, given the uncertainty around the selection UI, I wanted to have more than one option: when there are only two options, the buttons function just like a checkbox, so we really needed 3+ to judge the cycle interaction. Let's refer back to this preview deploy when discussing that interaction.
@dwelle As for the cycle interaction itself, I'm not 100% sure on whether it's the right approach. Most design tools have a panel to select line caps / decorators. I don't think a graphical select would be the right choice—hard to implement, overkill for few decorators and wouldn't scale easily to many decorators.
As an alternative to the cycle button, we could instead use a native text-based select but still show the icon of the selected decorator.

That said, I this a cycle button like the one implemented here is a fine choice for maybe three or four cap styles. And it works for one style, too, as implemented here—as mentioned above, it works just like a checkbox with only two options.

Could you check up on how I'm handling older arrows without start / end decorators? I'm unsure whether that part of this PR is in line with the rest of the project.

Could you check up on how I'm handling older arrows without start / end decorators? I'm unsure whether that part of this PR is in line with the rest of the project.

I'll do a review of the code, but no sooner than tomorrow.

Run npm run test:update

When two arrows and one rectangle are selected, the decorators don't show the actual status.

Good catch, fixing in the next commit. Tests are also updated and passing.
One of the tests in move.test.tsx, rectangles with binding arrow, has the UI creating a line element instead of an arrow. I fixed this in the test, but was this intended?

One of the tests in move.test.tsx, rectangles with binding arrow, has the UI creating a line element instead of an arrow. I fixed this in the test, but was this intended?

Let's wait for @dwelle on this one

Strangely that same test (rectangles with binding arrow) passes locally but fails when run by the Github action. :shrug:

Robots are failing us 🤷 🤖

One of the tests in move.test.tsx, rectangles with binding arrow, has the UI creating a line element instead of an arrow. I fixed this in the test, but was this intended?

I haven't written that one, but ultimately it doesn't matter whether it's a line or an arrow because in the test we're binding it via an API which is agnostic (works with either).

...ultimately it doesn't matter whether it's a line or an arrow because in the test we're binding it via an API which is agnostic (works with either).

Makes sense. The test only fails because it expects the "arrow" to have a "arrow" end decorator, but that decorator is only applied to new arrow elements, not to lines. Unless were explicitly testing whether binding also works on lines, I'd just make it an arrow.

@steveruizok you went a bit too far on my suggestion to remove the "migration" --- we still wanna retain the [null, null] for non-arrows (for now) — hence the failing snapshots.

I believe this was to catch arrows being copy and pasted between an older pre-decorators version and PR's version. I suppose this would never occur in production, though, so I'll remove this bit.

That's a good point. I thought we're restoring on paste, but apparently not. Will create an issue.

A few updates:

I've changed all the names (both labels and in the API) to arrowheads / Arrowheads. The name is used by Adobe, Sketch, and several other apps. A robust user research study (a Twitter poll) confirms that the name is preferred by Excalidraw's users.
I've added event tracking. Changing the starting arrowhead will fire arrowhead start and changing the end arrowhead will fire arrowhead end. Both events will fire values of either none, arrow, or any of our future arrowhead names.
Simplified the ButtonIconCycle component to be a fully controlled component.
Minor other issues resolved above.

I think it's good to go!

:heart: :rocket:

Is this feature released? I'm unable to see it.

Is this feature released? I'm unable to see it.


Issue with labels / values not being interpolated resulting in malformed / missing data

Hi!
Info:
@coveo/atomic-react": "^0.4.11" (have tried many version after v3+, including beta)
Issue
Seeing non interpolated variables
I see results and the templating for results is working ok for now, but the facet and the query summary / info has some of the labels / values not interpolated.

code:

<AtomicSearchInterface engine={engine}>
        <Row
          marginTop={5}
          marginBottom={5}
          justifyContent={'center'}
          flexGrow={1}
        >
          <AtomicSearchBox>
            <AtomicSearchBoxQuerySuggestions />
          </AtomicSearchBox>
        </Row>
        <Row display="flex">
          <Col lg={3}>
            <AtomicFacetManager>
              <AtomicFacet field="customer_facing_source" label="Source" />
              <AtomicFacet field="language" label="Language" />
            </AtomicFacetManager>
          </Col>
          <Col lg={9}>
            <Row>
              <Col lg={8}>
                <AtomicQuerySummary enableDuration></AtomicQuerySummary>
                <AtomicBreadbox></AtomicBreadbox>
              </Col>
              <Col justifyContent={'end'} lg={4}>
                <AtomicSortDropdown style={{ justifyContent: 'end' }}>
                  <AtomicSortExpression
                    label="Relevance"
                    expression="relevancy"
                  />
                  <AtomicSortExpression
                    label="Date"
                    expression="date descending"
                  />
                </AtomicSortDropdown>
              </Col>
            </Row>
            <Row justifyContent={'center'}>
              <AtomicNoResults></AtomicNoResults>
              <Divider marginTop={2} />
              <AtomicResultList
                template={(result: any) => <ResultTemplate result={result} />}
              ></AtomicResultList>
            </Row>
            <Row marginTop={5}>
              <Col>
                <AtomicResultsPerPage />
              </Col>
              <Col>
                <AtomicPager />
              </Col>
            </Row>
          </Col>
        </Row>
      </AtomicSearchInterface>

Expected Behavior
values / labels to be filled in appropriately

Hello, this is due to the way static assets needs to be made available in the public directory of your app.
There's a high level explanation in the README available here: https://github.com/coveo/ui-kit/tree/master/packages/atomic-react#static-assets---languages-and-svgs
Cheers !

What does http://localhost:3001/lang/en.json resolve to in your case ?

@olamothe
resolves with values.

Humm so that should be good... 🤔
And when you load the search page which contains the placeholder values, you should see that it should send a fetch request to load that en.json file, which I assume return some kind of 404 error.
What path is it requesting ? Is it trying to load from somewhere else than localhost:3001/lang/en.json ?

Hah, you're right it is 404ing, i missed that.
Looks like it's trying to prepend the current page path.

Alright, if it's possible to make this path resolve to the json dictionary file with Gatsby ( sorry, I'm not an expert with that technology ) then it should be good.
There is currently no way to custom Atomic React to look at a specific path for the language file.
I will log something in our internal ticketing system to see if we can make this easier to do properly, or even all automatic.
I can see it causing problems for many implementers since it's always going to be always different.

Hmmm I'll see what I can do, we have another ticket to solve a similar issue with paths in gatsby but it's not ideal.
Is there any danger in making atomic-react fetch it's variables from absolute rather than relative path?

Is there any danger in making atomic-react fetch it's variables from absolute rather than relative path?

There should not be any danger, no.

Temporary Solution
Using patch-package to patch @coveo/atomic v1.30.3 for
getAssetPath('./lang/')}{{lng}}.json

and
getAssetPath(`./assets/${remainder}.svg`)

to be absolute path instead of relative seems to have solved this issue as a temporary work around.
Locking to version "@coveo/atomic-react": "0.4.11",
and in resolutions package.json i have
"@coveo/atomic": "1.30.3"

Everything seems to be working fine so far, in the case of discussing this change for future package updates internally.
Thank you for the help debugging @olamothe .

Glad you were able to patch the issue.
The team has logged an internal ticket to see how it could be improved in the future.
Most probably a property that can be passed to the interface to specify the path that should be used.
Thanks for reporting !


PRM-1966 Further content changes from remaining welsh review

AT PR to go with changes hmrc/penalties-acceptance-tests#217

Fixed ✅


Update zio-schema, zio-schema-derivation, ... to 0.4.11

About this PR
📦 Updates

dev.zio:zio-schema
dev.zio:zio-schema-derivation
dev.zio:zio-schema-protobuf

from 0.3.1 to 0.4.11
Usage
✅ Please merge!
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

⚙ Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "dev.zio" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "30 days" },
  dependency = { groupId = "dev.zio" }
}]

labels: library-update, early-semver-major, semver-spec-minor, commit-count:1

Superseded by #337.


x/sys/unix: Gettimeofday broken on Darwin: doesn't set the time.

What version of Go are you using (go version)?

$ go version
go version go1.14 darwin/amd64

Does this issue reproduce with the latest release?
Yes
What operating system and processor architecture are you using (go env)?
go env Output
$ go env
go env
GO111MODULE=""
GOARCH="amd64"
GOBIN=""
GOCACHE="/Users/watsonladd/Library/Caches/go-build"
GOENV="/Users/watsonladd/Library/Application Support/go/env"
GOEXE=""
GOFLAGS=""
GOHOSTARCH="amd64"
GOHOSTOS="darwin"
GOINSECURE=""
GONOPROXY="code.cfops.it,github.com/cloudflare"
GONOSUMDB="code.cfops.it,github.com/cloudflare"
GOOS="darwin"
GOPATH="/Users/watsonladd/go"
GOPRIVATE="code.cfops.it,github.com/cloudflare"
GOPROXY="https://proxy.golang.org,direct"
GOROOT="/usr/local/Cellar/go/1.14.2_1/libexec"
GOSUMDB="sum.golang.org"
GOTMPDIR=""
GOTOOLDIR="/usr/local/Cellar/go/1.14.2_1/libexec/pkg/tool/darwin_amd64"
GCCGO="gccgo"
AR="ar"
CC="clang"
CXX="clang++"
CGO_ENABLED="1"
GOMOD=""
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"
PKG_CONFIG="pkg-config"
GOGCCFLAGS="-fPIC -m64 -pthread -fno-caret-diagnostics -Qunused-arguments -fmessage-length=0 -fdebug-prefix-map=/var/folders/ps/_z7r371n1kdf72n71mlyf28h0000gp/T/go-build470647461=/tmp/go-build -gno-record-gcc-switches -fno-common"

What did you do?
package main

import (
	"fmt"

	"golang.org/x/sys/unix"
)

func main() {
	time := unix.Timeval{}
	err := unix.Gettimeofday(&time)
	if err != nil {
		panic(err)
	}
	fmt.Printf("sec=%d\n", time.Sec)
	fmt.Printf("%+v\n", time)
}

What did you expect to see?
The current time
What did you see instead?
All zeros

I imagine that this was broken when the Darwin support was converted to use libc.  Someone probably needs to drop the various Gettimeofday implementations in syscall_darwin_CPU.go and just call the libc function.

https://golang.org/src/cmd/vendor/golang.org/x/sys/unix/syscall_darwin_386.go#L27
https://golang.org/src/cmd/vendor/golang.org/x/sys/unix/syscall_darwin_arm.go#L25
These type conversion are wrong in Gettimeofday. (comment too)
gettimeofday returns (sec int64, usec int32, err error)
but sec is converted from int64 to int32.

@wbl thanks for the report. I've sent a fix at https://golang.org/cl/232803 and this works on darwin/amd64 for me with your code snippet. Does this fix the issue for you as well?

I might need some step by step instructions on how to check out the tree and run the newly built tool against my example: not that experienced with gerrit.
@Lekensteyn was the original discoverer of this issue: I just passed it on.

@wbl you should be able to get the CL into your local clone of golang.org/x/sys using git fetch "https://go.googlesource.com/sys" refs/changes/03/232803/2 && git checkout FETCH_HEAD

It worked for me! Thanks for your instructions.


[Backport 1.x] Add empty or custom SDKNamedXContentRegistry

Description
The Anomaly Detection project test framework makes frequent use of NamedXContentRegistry.EMPTY in constructors for test instances, or provides a minimal registry created from other sources.
Since the constructors will be taking SDKNamedXConetentRegistry parameters as part of #518, we need an equivalent to pass for either the EMPTY case or to set a custom internal registry.
Backport of https://github.com/opensearch-project/opensearch-sdk-java/pull/529
Issues Resolved
Part of #518 with the remainder of the work on AD Extension repository.
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Codecov Report

Merging #535 (824786c) into 1.x (e025a51) will increase coverage by 0.23%.
The diff coverage is 100.00%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@             Coverage Diff              @@
##                1.x     #535      +/-   ##
============================================
+ Coverage     66.74%   66.98%   +0.23%
- Complexity      191      194       +3
============================================
  Files            34       34
  Lines           827      833       +6
  Branches         26       26
============================================
+ Hits            552      558       +6
  Misses          263      263
  Partials         12       12

Impacted Files
Coverage Δ

...a/org/opensearch/sdk/SDKNamedXContentRegistry.java
100.00% <100.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


general refactoring

 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.:x: KeyStorke sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.

Thanks for taking the time to work on this! It's always great to see that others have use for the tools we use.
In terms of the PR, I'm good with the general structure changes, however, most of the other changes will cause linter violations. I've not set up a CI system for this yet, so it won't be flagged on the PR. Those issues will need to be corrected before we can move further.
If the move into various modules could be done on a commit by commit basis too, it would make reviewing a lot easier and cleaner.

Of course, I will close this PR and make again another on the week ok? I'll try to do PR simply and more readable for your comfortable :)


Typo in WebUI?

Short summary about the issue/question:
It seems that there is a typo in WebUI
Brief what process you are following:
Start a NNI experiment, then you can see the typo like this:

"Metric" not "Metrc"?
nni Environment:

nni version: 0.5.1
nni mode(local|pai|remote): local
OS: Linux
python version: 3.6
is conda or virtualenv used?: no
is running in docker?: no

@lvybriage please recheck and help to fix this typo?

There Lines:
https://github.com/Microsoft/nni/blob/master/src/webui/src/components/overview/SuccessTable.tsx#L6
https://github.com/Microsoft/nni/blob/master/src/webui/src/components/trial-detail/TableList.tsx#L13
https://github.com/Microsoft/nni/blob/master/src/webui/src/components/TrialsDetail.tsx#L95

Yes, it should be modified in the recent pr

@PurityFan  @leelaylay
Thanks.
Had fixed in pr 790.

Solved by https://github.com/Microsoft/nni/pull/790


feat: better create protocol options

This is a feature request
Problem
Currently options for creating a node are convoluted and spread in the code base.
It get harder to handle and requires some TS hacks to prevent from compilation.
Proposed Solutions
For starters we should split building of options and creation of libp2p which is main consumer of options.
const createOptions = buildCreateOptions(options);
const libp2p = await createLibp2pNode(createOptions);
const wakuNodeOptions = buildWakuNodeOptions(createOptions);

Second step is to consolidate leftover option updates specific for WakuNode:
https://github.com/waku-org/js-waku/blob/9b0f1e855aa3a1f7b9aec3a4c726568d37595c28/packages/sdk/src/waku.ts#L96
Notes
Draft PR: https://github.com/waku-org/js-waku/pull/2072

another concern to be addressed - https://github.com/waku-org/js-waku/pull/2056#discussion_r1682637811

another concern to be addressed - https://github.com/waku-org/js-waku/pull/2056#discussion_r1682637811


Sinead Coughlan's QuickStart topic

Proposing a commit that comprises -

My QuickStart topic (my_first_use_topic_sineadc.md)
Updated index.md with link to #1.

Thanks, including absolute links (html) was an oversight on my part! I've updated the file with relative links.


PC / Stuck at Deadwood Canyon 12*

I'm pretty sure it should not apply debufs to Poena, because it seems that it is all just getting reflected back to the player, and then she one-shots me all the time, and I can't do anything with 3 ice-debufs on me.

I should note that I don't have too much progress in game (~almost all weapons maxed with +10 ench.)
But for some of locations it works much worse than for the others
Here's my config
// =============================================================================
// ================================================ CONFIG =====================
// =============================================================================

// ---- THRESHOLD VALUES ----
var smiteThreshold = 6               // Min number of foes to activate Blade of the Fallen God
var dmgDebuffThreshold = 7           // Total damage debuff on yourself before switching out of low-damage weapons
var splashThreshold = 3              // Total number of foes needed to switch to 'splashMeleeWeapon' (below)
var foeArmorThreshold = 50           // If foe's armor exceeds this, switch to 'antiArmorMeleeWeapon' (below)
var fastArmorShieldMax = 14          // Max armor of "Armor/sec"-enchanted shield, if you have one ('fastArmorShield', below)
var maxPickPocketBuffs = 1          // Max number of pick-pocket buffs you can have (see your Skeleton Arm's stats for this number)
var maxOverAvgToExitMult = 1.15      // If, during a speed run ('SHIFT+S' hotkey toggle), your current time exceeds your average time
                                     //   multiplied by this number, you'll auto-exit the stage (to prevent your average
                                     //   taking a hit).  Set to 0 to disable.
var devourFireRuneThreshold = 3      // The number of ignition stacks on a non-boss foe required to trigger Cinderwisp's devour.
                                     //   Set to 0 to never trigger devour against non-bosses.
var devourFireRuneBossThreshold = 10 // Same as above, but for bosses.
                                     //   Set to 0 to never trigger devour against bosses.

// ---- ARSENAL ----
// If you don't have a weapon for the indicated slot, or you don't want to change your weapon in the
//  situation described, set the value to 'false' (without any quotes).

var splashMeleeWeapon = "big aether sword dU +10"    // Name of melee splash weapon for when foes exceed 'splashThreshold'
var healthHammer = "vigor hammer dL +8"               // Name of rune hammer enchanted with health bonus for defensive potion combo
var antiArmorMeleeWeapon = "heavy hammer +10" 			// Name of melee weapon to equip vs. strong armor
var debuffDamageWeapon = "poison sword dP +10"      // Name of weapon to apply debuff_damage (poison)
var debuffDoTWeapon = "fire sword dF +11"           // Name of weapon to apply debuff_dot (fire)
var debuffChillWeapon = "ice sword dI +10"          // Name of weapon to apply debuff_chill (ice)
var magicRangedWeapon = "ice wand dI +11"           // Name of magical ranged weapon (able to hit physical-immune enemies)
var lifeStealMeleeWeapon = false                // Name of health-stealing melee weapon
var lifeStealRangedWeapon = false               // Name of health-stealing ranged weapon
var lifeStealShield = "vigor shield ah +8"          // Name of heal-when-hit shield
var icePillarWeapon = "big aether sword dU +10"      // Name of weapon to use when facing ice pillars/wall in Icy Ridge
var fastArmorShield = "compound shield +10"     // Name of "Armor/sec"-enchanted shield
var strongArmorShield = "compound shield +11"   // Name of "Armor"-enchanted shield
var iceWandA = "ice wand dI +11"                   // Name of first speed-enchanted chill wand for smite combo
var iceWandB = "ice wand dI +11"                   // Name of second speed-enchanted chill wand for smite combo
var iceWandBossA = "ice wand dI +11"             //TODO why lowstar???// Name of first low-star speed-enchanted chill wand for smite combo vs. bosses
var iceWandBossB = "ice wand dI +11"          	//TODO  why lowstar???// Name of second low-star speed-enchanted chill wand for smite combo vs. bosses
var stoneHammerA = "stone hammer +10"            //Name of first stun-enchanted stone hammer for stun-locking
var stoneHammerB = "stone hammer +10"            // Name of second stun-enchanted stone hammer for stun-locking

// ---- QUEST FLAGS ----
// Toggle these on to activate special script logic related to satisfying certain quest objectives.

var isOnPPKillQuest = false    // When true, Skeleton Arm will only activate when it can kill its target.

// ---- HOTKEY OVERRIDES ----
var rangedOverride = "repeating crossbow +10"       // Ranged Weapon to equip when [X] held down

// *** IMPORTANT *** --- You have to use this with a nonmagical ranged weapon to defeat R.I.Pieces,
//                         the mid-level boss in high-level Halls runs, to kill it by shooting through the
//                         ghosts it spawns; otherwise, you risk being overrun.


I want a C++ example

I haven't found an example based on C++. Can someone give me one?Thank you.

Unfortunately, we do not have c++ examples as they are not an easy way to learn. However, it should be fairly simple to rewrite our examples in Python to c++ code. The API is the same.
Furthermore, we do not have the resources to implement such examples as we are focusing our limited resources on extending the capabilities of Crocoddy.
But this should not block you as you could also learn how to write c++ code by reading the unit-tests code.


Implement a link graph view

For many this is the "killer feature" of a that allows bi-directional linking: the ability to show all your pages and their links in a graph. See Obsidian for examples of this.
I'm sure there are off-the-shelf JS libraries that make this fairly straight forward.

Work in progress by @bbroeksema: https://github.com/bbroeksema/silverbullet-graphview

sneak preview. For me I'll do the 3D version. What would you guys rather like in the public plug? 2D or 3D? It's a matter of CPU load...
Currently I'm trying to work on making it easier to navigate.  I think I'll go with some kind of smart search where you can anchor some of the nodes and those and their neighbors will get highlighted. Then you can interpret the relationship between those.
At the moment

the yellow ones are the top 4% connected nodes
the red ones are those running from and to the current node
the grey ones are files that are linked but not created yet

Opening the graph view takes roughly 3-4 seconds at the moment because I'm completely rebuilding the graph view (and reading all files from disk because of that). @zefhemel is there a good way to get a connected graph data structure from the existing indexer? I feel like we're duplicating code here.
Also, being able to slide the side panels open and close would be cool, it's feeling stiff atm. Maybe take inspiration from Obsidian? Even though I don't agree on all design choices, the new 1.0 UI with sliding panes and its layouting options is quite cool.

Has anyone else not been able to get the silverbulletmd/silverbullet-graphview plug working? I'm seeing a "show graph view" option but anytime I try to use it I get the following error in the console:
Unregistered syscall store.get
    at graphview.plug.js:1:1620
    at graphview.plug.js:1:1668

On further inspection, it's also throwing a different error at me on page load:
Unregistered syscall index.queryPrefix
    at graphview.plug.js:1:1620
    at graphview.plug.js:1:1668

^ confirming that this is still happening after updating to the latest version (cc @zefhemel)

Yeah it's been broken and unmaintained. Somebody needs to fork and fix it.

Nope


Add additional cloud metadata

The goal of this PR is to add aws_account_id, aws_image_id and aws_availability_zone in the host plugin. The cloud harvester provides and interface for any of the supported clouds, in order to retrieve the previous values new methods were added.
The PR implements the methods for AWS, Azure and Alibaba

I might be wrong but I think hostinfoplugin runs just once: when the agent is started.
cloudHarvester.GetCloudType() will return AWSType only after it succesfully make a request to AWS. So if the hostinfo plugin runs before cloud detection then aws metadata will not be available until agent restart.
Can you please confirm/infirm this?

@cristianciutea You are right, hostinfoplugin runs just once the agent is started: https://github.com/newrelic/infrastructure-agent/blob/master/cmd/newrelic-infra/newrelic-infra.go#L430
But before registering any plugin, during the creation of the Agent object, we try to detect the Cloud harvester: https://github.com/newrelic/infrastructure-agent/blob/master/internal/agent/agent.go#L273
The scenario that would fail is if the cloud harvester is not detected after some time the agent has been running. Do you think the cloud region we are currently reporting will also be affected? https://github.com/newrelic/infrastructure-agent/blob/master/internal/plugins/linux/hostinfo.go#L218

https://github.com/newrelic/infrastructure-agent/blob/master/internal/agent/agent.go#L273

yes, it could happen that the agent starts and cloud is not detected yet:
https://github.com/newrelic/infrastructure-agent/blob/76f07e131a4aefd52ca65ed2ca03aac12f5d883a/pkg/sysinfo/cloud/cloud.go#L113
Since the region field is in the same situation, I would suggest opening a new issue to fix this

Make cloud metadata more reliable: https://github.com/newrelic/infrastructure-agent/issues/1088


start debugging with options

Don't we need to update the call to startDebugging in packages/office-addin-debugging/src/commands.ts as well?

This review is currently set up to only add a new function startDebuggingWithOptions and not make any breaking changes.
I'm happy to completely replace the existing startDebugging with the new signature if that's preferable.

Don't we need to update the call to startDebugging in packages/office-addin-debugging/src/commands.ts as well?

This review is currently set up to only add a new function startDebuggingWithOptions and not make any breaking changes.
I'm happy to completely replace the existing startDebugging with the new signature if that's preferable.

I think let's just update the existing method and not add a new one for the sake of simplicity

I think let's just update the existing method and not add a new one for the sake of simplicity

Rather than use the master branch in your fork, you should make the a named topic branch and use that.

Rather than use the master branch in your fork, you should make the a named topic branch and use that.

/AzurePipelines run

/AzurePipelines run

I think there might be other callers of the startDebugging() function, perhaps in test code. Can you please just double-check?

I think there might be other callers of the startDebugging() function, perhaps in test code. Can you please just double-check?

Rather than use the master branch in your fork, you should make the a named topic branch and use that.

I was looking at the README in this repository root, and I couldn't find the document explaining the preferred way of making contributions.
Would you be able to link me to the appropriate document?

Rather than use the master branch in your fork, you should make the a named topic branch and use that.

I was looking at the README in this repository root, and I couldn't find the document explaining the preferred way of making contributions.
Would you be able to link me to the appropriate document?

That's the standard way in Github to work so we didn't document this specifically in the repository, but I can see how explaining better how others can contribute would be good. Let's do this in a separate PR. Basically, we you just create a branch in your fork, make the PR from the branch, and then we merge.

That's the standard way in Github to work so we didn't document this specifically in the repository, but I can see how explaining better how others can contribute would be good. Let's do this in a separate PR. Basically, we you just create a branch in your fork, make the PR from the branch, and then we merge.

@wandyezj I don't think we document that the preferred method is to create a branch off of a fork.  I suppose we could add that to the "Contributing" section in the ReadMe

@wandyezj I don't think we document that the preferred method is to create a branch off of a fork.  I suppose we could add that to the "Contributing" section in the ReadMe

@wandyezj Could go through the comments and mark them resolved if you've addressed them? I think there are a few comments I made that are still outstanding.

@wandyezj Could go through the comments and mark them resolved if you've addressed them? I think there are a few comments I made that are still outstanding.

Also, you should merge master into this.

Also, you should merge master into this.

/AzurePipelines run

/AzurePipelines run

Before we merge this, I want to get a clear sense if this is a breaking change (i.e. will will need to update other template and tooling to accommodate this change. I am OK with a breaking change but just want to know what, if any, additional changes we will need to make to accommodate this change

Before we merge this, I want to get a clear sense if this is a breaking change (i.e. will will need to update other template and tooling to accommodate this change. I am OK with a breaking change but just want to know what, if any, additional changes we will need to make to accommodate this change

When this gets published, the major version of this package should be updated.
Other packages which consume this will need to the latest version, and then update to call StartDebugging() with the options object instead of just the parameters. startDebugging(manifestPath, etc.) gets replaced with:
startDebugging({ manifestPath, enableLiveReload: false, etc. });
It's not a difficult change.

When this gets published, the major version of this package should be updated.
Other packages which consume this will need to the latest version, and then update to call StartDebugging() with the options object instead of just the parameters. startDebugging(manifestPath, etc.) gets replaced with:
startDebugging({ manifestPath, enableLiveReload: false, etc. });
It's not a difficult change.


Create a Github Integration application

[ ] Create a Github Integration application
[ ] Configure it to request for events while installing

Completed. Now the integration is present at https://github.com/integration/gitnotif


ffmpeg: Update to 7.0, enable iOS platform and add support for libdrm

Also enable mediacodec support on Android
Try to compile on iOS

Before adding new features and new modules, please go to issues to submit the relevant feature description first.
Write good commit messages and use the same coding conventions as the rest of the project.
Please commit code to dev branch and we will merge into master branch in feature
Ensure your edited codes with four spaces instead of TAB.

增加新特性和新模块之前，请先到issues提交相关特性说明，经过讨论评估确认后，再进行相应的代码提交，避免做无用工作。
编写友好可读的提交信息，并使用与工程代码相同的代码规范，代码请用4个空格字符代替tab缩进。
请提交代码到dev分支，如果通过，我们会在特定时间合并到master分支上。
为了规范化提交日志的格式，commit消息，不要用中文，请用英文描述。

The remaining issue seems to come from a bug from xcode 15.0 and 15.0.1 (see https://github.com/arthenica/ffmpeg-kit/issues/900) which is fixed in xcode 15.1. how should that be handled?


Fix content type isn't multipart/form-data bug on push

Seeing this again in CI.

https://github.com/replicate/cog/runs/2713873111

More detail from discord:

it happened locally for me just now! it was with hello world so i think it's only when there's a small amount of data

presumably something like Close() getting called before a writer has a chance to write the form data. maybe it doesn't happen when there is a lot of data because the zipping is slow and isn't closing it immediately.


doesnt work :(

After the last update, when I inject the cheat in default mode nothing happens (I tried to open the menu on insert), in debug mode it crashes and the console is empty.
please help

worked fine on latest commit, pull it lol


Remove keybinding from package.json

Package.json still references ALT+F keybinding that was removed

Seems as though other .txt files are being included as FitNesse files. Need to modify that as well.


parse into Excel

Ilan is looking for Twitter data that contains the following columns:

handle (text)
user's name (text)
content of the tweet (text)
whether it had an image (boolean: calculated)
image URL (text: url)
tweet's timestamp (datetime or text?)
number of retweets (integer)
list of hashtags (text: list)
list of mentioned users (text: list)
isRT (boolean: calculated)
isMT (boolean: calculated)

I think the "calculated" columns are ones we have to determine the value, meaning that it wasn't provided directly by twitter.

"retweeted" is available in the Twitter JSON result, so it need not be calculated.


Push karma-coverage changes upstream?

Have you talked with the karma-coverage team about pushing your changes upstream that allow you to support arbitrary instrumentation and have more control over the the passed options?
I npm installed isparta + 6to5 + your karma-coverage fork to quickly get coverage working on a project with es6 modules that wasn't working with any of the esprima-harmony/esprima-fs based workarounds I tried.
At the very least having isparta becoming a valid option against the main karma-coverage repo would be nice.

Hey there @johlrich :)
Yes I though about it
But I didn't take the time to actually go through the Karma tests to prove that it's working  :P

Alright I made a PR on the official karma-coverage repo
We will see if somebody cares or not...


aiohttp is not actually a dependency

and requests was removed erroneously

Thanks, it was a leftover from async conversion tryout.

Where do you use this library for if I may ask?

Where do you use this library for if I may ask?

We packaged it in Nixpkgs as a dependency of Home Assistant. See https://github.com/NixOS/nixpkgs/pull/284893 for why I made this PR.

I wonder, why are you building packages from unreleased and unused modules? I know there is a version 0.0.44 in the code but there is no release 0.0.44 posted yet, nor is that version used yet in the home assistant integration linked to it.

You did release 0.0.44 on PyPI: https://pypi.org/project/pyfireservicerota/
As long as the component tests pass, we're fine with using a newer version than Home Assistant.

Understood, taken note of it, didn't know this flow was in place, thank you.


Default role of user created from client

Similar to #35 but I want to do it from the client side.  I tried to follow the examples on #35 but it didn't work.  This is the closest that I can make it run but it fails in the end with an update failed: Access denied.
The approach I was taking was to do it via callback event since that is the only time I can be sure that the user was created or had an explicit error.  I have verified that there are values in the console log.
In fact for a short while the user does have that role, but the update gets rolled back due to update failed.

Template.createAdminUserPage.events({
    'submit form': function (event, template) {
        event.preventDefault();
        var email = $('[name=email]').val();
        var password = $('[name=password]').val();

        Accounts.createUser({email: email, password: password}, function (err) {
            if (err) {
                console.log(err);
            } else {
                console.log("user id = "  + Meteor.userId());
                console.log("user = "  + Meteor.user());
                Roles.addUsersToRoles(Meteor.user(), 'admin');
            }
        });
    }
});

I found a bit of a workaround, but seems to work.  The idea is to do the job on the server which is what is done on #35 to begin with.  I used a the Meteor.call on the client to call a Meteor.methods on the server.  Keeping it open for now in case there is a correct way of doing it on the client side that I am missing.  (Apologies I just started learning Meteor last week while I was stuck at home with a cold).

Meteor.methods({
    /**
     * This will create the initial admin user.  This will only work if one does not presently exist, if one does exist it will throw an error.
     * @param email
     * @param password
     */
    createInitialAdminUser: function (email, password) {
        if (Roles.getUsersInRole('admin', null, {reactive: false, limit: 1}).count() != 0) {
            throw new Meteor.Error("An existing admin user is already present");
        }
        var userId = Accounts.createUser({email: email, password: password});
        Roles.addUsersToRoles(userId, ['admin']);
    },
    /**
     *
     * @returns {boolean} true if there is at least one admin user
     */
    hasAdminUser: function() {
        var c = Roles.getUsersInRole('admin', null, {reactive: false, limit: 1}).count();
        return c != 0;
    }
});

Yes, in fact the best way is to add do all changes inside Meteor methods.


fix(jest-each): fixes error message with incorrect count of missing arguments

Summary
FIxes incorrect number of missing arguments in jest-each, when used with templates.
Test plan
   each`
      a    | b    | expected
      ${0} | ${0}
    `.test('returns $expected when given $a and $b', ({a, b, expected}) => {
      expect(a + b).toBe(expected);
    });

should fail with a message showing 1 missing argument, however it is returning the number of arguments given in the incomplete row (2)

Thanks for the PR @flakolefluk! Could you sign the CLA? 🙂

Thanks for the PR @flakolefluk! Could you sign the CLA? 🙂

@SimenB I will as soon as I get confirmation from my employer on how to proceed.

I don't know, I don't have access 😅 Maybe @JoelMarcey can confirm? (And give the bot a kick if it's stuck?)

@SimenB / @JoelMarcey is there any update on the CLA? thanks!

tried merging in main, maybe that triggers something. if not, I'll try closing and reopening

This should be good to go now, though might take a few hours for our tooling to detect it properly and update the PR (or me commenting might trigger it?).

Woo!

Thank you @zpao and @SimenB for pushing to get this merged!


ci: update error reference docs

This PR is auto-generated by a nightly GitHub action to update the error reference docs from source code in withastro/astro.

Lunaria Status Overview
🌕 This pull request will trigger status changes.

Learn more
By default, every PR changing files present in the Lunaria configuration's files property will be considered and trigger status changes accordingly.
You can change this by adding one of the keywords present in the ignoreKeywords property in your Lunaria configuration file in the PR's title (ignoring all files) or by including a tracker directive in the merged commit's description.

Tracked Files

Locale
File
Note

en
reference/error-reference.mdx
Source changed, localizations will be marked as outdated.

en
reference/errors/prerender-client-address-not-available.mdx
Source added, will be tracked.

en
reference/errors/route-not-found.mdx
Source added, will be tracked.

Warnings reference

Icon
Description

🔄
The source for this localization has been updated since the creation of this pull request, make sure all changes in the source have been applied.


Error: Calling Formula.sha1 is disabled!

→ brew cleanup                                                                                >>>>>>>>>>
Error: Calling Formula.sha1 is disabled!
Use Formula.sha256 instead.
/usr/local/Homebrew/Library/Taps/bradley-curran/homebrew-tap/quiver2jekyll.rb:4:in `class:Quiver2jekyll'
Please report this to the bradley-curran/tap tap!
If reporting this issue please do so at (not Homebrew/brew or Homebrew/core):
https://github.com/bradley-curran/homebrew-tap/issues

Pull Request #1 addresses this and should be merged.
Meanwhile, you could make this change manually by editing the formula here:
/usr/local/Homebrew/Library/Taps/bradley-curran/homebrew-tap/quiver2jekyll.rb
and making the change listed in PR #1 .


[Feature] k8s 1.31 Support for rancher/shell

Add k8's 1.31 support for rancher/shell
Depends on: https://github.com/rancher/helm/pull/52
First shell image with support for 1.31 is: v0.3.0-rc.2

k8s 1.31 Support for rancher/shell has been validated successfully on 2.10.0-alpha7


Add LeaderBoard API endpoint

API endpoint that provide a list of reachable nodes ranked according to their Peer Index (PIX) as of the latest snapshot retrieved.  PIX values are calculated only for activated nodes.
Peer Index (PIX) is defined ad described in this document:
https://bitnodes.earn.com/nodes/leaderboard/#peer-index
This is needed to re-enable a source of seeding BUCash was using, see https://github.com/BitcoinUnlimited/BitcoinUnlimited/pull/928 for more details

API endpoint that provide a list of reachable nodes ranked according to their Peer Index (PIX) as of the latest snapshot retrieved.  PIX values are calculated only for activated nodes.
Peer Index (PIX) is defined ad described in this document:
https://bitnodes.earn.com/nodes/leaderboard/#peer-index
This is needed to re-enable a source of seeding BUCash was using, see https://github.com/BitcoinUnlimited/BitcoinUnlimited/pull/928 for more details


Include concaveHullByLength

Hi there, @twpayne
Would it be possible to include concaveHullByLength in the library?

Refer to https://libgeos.org/doxygen/3.11/classgeos_1_1algorithm_1_1hull_1_1ConcaveHull.html#ada45f68340212afa6984e310f7a560a9

I've opened an issue: https://github.com/libgeos/geos/issues/849

It's supposed to be included in libgeos, according to https://github.com/libgeos/geos/issues/849#issuecomment-1474525181

That PR was merged three days ago. The latest version of GEOS is 3.11.2, which was released five days ago, so it is not in a release version of GEOS. You have to wait for a released version of GEOS (of course you can use an unreleased version locally).


handling service registry client errors

The current interface which the various service discovery clients implement is the following:
type ServiceDiscovery interface {
	Services() []*Service
	GetService(hostname string) (*Service, bool)
	Instances(hostname string, ports []string, labels LabelsCollection) []*ServiceInstance
	HostInstances(addrs map[string]bool) []*ServiceInstance
	ManagementPorts(addr string) PortList
}

pilot/proxy/envoy/discovery.go calls this interface and updates its cache accordingly.  With the current interface, if the client experiences an error calling the consul/eureka/etc server, it will currently return an empty array, which discovery.go will updates its cache with and return the new empty result to the xDS API's
current suggestions would be to either:

add a return error for the interface methods so discovery.go ignores the empty array and returns what it has in the cache

caching or better/consistent error handling on the client side model.ServiceDiscovery implementations

Thoughts?

Lets start with 1. And possibly, along the way, split the handlers for consul, eureka, k8s, such that only one of the caches is cleared when service handlers are called, instead of clearing all caches.
There are several low hanging fruits in the cache and we need to have more finer grained cache invalidation.

The client of the registry already does caching (the discovery service that ultimately constructs the config). So adding another caching to the model adaptor seems redundant. Perhaps, returning an error indicator for "retrying later" is a good option then. I agree that returning an empty array on error is not a good behavior.

resolved by #1610


ERR The ID specified in XADD is equal or smaller than the target stream top item

Issue Description
RedisShake errors out whenever it receives XADD command
Environment

RedisShake Version：latest
Redis Source Version：latest
Redis Destination Version：latest
Redis 部署方式（standalone/cluster/sentinel）：standalone -> sentinel
Deployed on Cloud Provider：Kubernetes cluster (also tested using Docker Compose)

Logs
2024-05-29 08:07:18 INF load config from file: shake.toml
2024-05-29 08:07:18 INF log_level: [info], log_file: [/usr/src/app/RedisShake/bin/data/shake.log]
2024-05-29 08:07:18 INF changed work dir. dir=[/usr/src/app/RedisShake/bin/data]
2024-05-29 08:07:18 INF GOMAXPROCS defaults to the value of runtime.NumCPU [32]
2024-05-29 08:07:18 INF not set pprof port
2024-05-29 08:07:18 INF create SyncStandaloneReader: dev-redis-replicas-0.dev-redis-headless.dev-redis:6379
2024-05-29 08:07:18 INF create RedisStandaloneWriter: dev-redis-ha.dev-redis-ha:6379
2024-05-29 08:07:18 INF not set status port
2024-05-29 08:07:18 INF start syncing...
2024-05-29 08:07:18 ERR [writer_dev-redis-ha.dev-redis-ha_6379] receive reply failed. cmd=[xadd nlp:sync:output:4bcce72b-cb24-4b54-949e-867a9f769364:stream<PHONE_NUMBER>226-0 debug {"nlp_summary...], error=[ERR The ID specified in XADD is equal or smaller than the target stream top item]
			RedisShake/internal/writer/redis_standalone_writer.go:116 -> (*redisStandaloneWriter).processReply()
			runtime/asm_amd64.s:1695 -> goexit()

Could you check if there are any xadd operations being performed on the destination Redis instance? These operations might be causing the issue you’re encountering.

right there are XADD operations being performed. Does it mean that RedisShake can't sync those?

my bad, I did not read your question carefully. XADD operations are only performed on the source Redis, not destination. I found that you create Redis Stream with "0-1" https://github.com/tair-opensource/RedisShake/pull/808. I am using the same ID for the first entry. In that case RedisShake should not be creating the first entry for me, or should not be performing the ID comparison that is throwing the error.

I am first performing XREAD command with "0-0" and then XADD uses appended ID "0-1" (not set by default)

I can't think of a reason for now.

So I went ahead and deleted the stream from src and dest, restarted RedisShake and error changed:
2024-05-30 02:39:52 INF load config from file: shake.toml
2024-05-30 02:39:52 INF log_level: [info], log_file: [/usr/src/app/RedisShake/bin/data/shake.log]
2024-05-30 02:39:52 INF changed work dir. dir=[/usr/src/app/RedisShake/bin/data]
2024-05-30 02:39:52 INF GOMAXPROCS defaults to the value of runtime.NumCPU [32]
2024-05-30 02:39:52 INF not set pprof port
2024-05-30 02:39:52 INF create SyncStandaloneReader: dev-redis-replicas-0.dev-redis-headless.dev-redis:6379
2024-05-30 02:39:52 INF connecting to master node at dev-redis-ha-node-0.dev-redis-ha-headless.dev-redis-ha.svc.cluster.local:6379
2024-05-30 02:39:52 INF create RedisSentinelWriter: dev-redis-ha.dev-redis-ha:26379
2024-05-30 02:39:52 INF not set status port
2024-05-30 02:39:52 INF start syncing...
2024-05-30 02:39:52 ERR [writer_dev-redis-ha-node-0.dev-redis-ha-headless.dev-redis-ha.svc.cluster.local_6379] receive reply failed. cmd=[xadd nlp:sync:output:4bcce72b-cb24-4b54-949e-867a9f769364:stream MAXLEN 0 0-1 x y], error=[ERR The ID specified in XADD is equal or smaller than the target stream top item]
			RedisShake/internal/writer/redis_standalone_writer.go:116 -> (*redisStandaloneWriter).processReply()
			runtime/asm_amd64.s:1695 -> goexit()

before: receive reply failed. cmd=[xadd nlp:sync:output:4bcce72b-cb24-4b54-949e-867a9f769364:stream<PHONE_NUMBER>226-0
after: receive reply failed. cmd=[xadd nlp:sync:output:4bcce72b-cb24-4b54-949e-867a9f769364:stream MAXLEN 0 0-1 x y]

Try using the function to filter all stream class commands. https://tair-opensource.github.io/RedisShake/en/function/best_practices.html#过滤某类数据结构

Thank you, that's a good temporary fix


[Request] Two finger swipe on title bar to move windows

Is it possible to add two finger swipe on the title bar to move the windows to the next/previous desktop?

yabai does not care about user input at all aside from mouse actions and → skhd by the same author does not care about gesture inputs either, although this certainly would be a nice-to-have feature.
Your best bet here might be → BetterTouchTool, which is an application that does way too many things (some well, others not so much), and some of these things include mapping gestures to asynchronously executed external commands.
I don't think you can reliably detect whether the element under the cursor is a title bar, as there is no single commonly used accessibility role or subrole for title bars elements in macOS. Your best bet here might be detecting an AXTabGroup role for the frontmost element under the cursor using Apple Script.

Hyperdock does an ok job of two finger swipe, and I've really only found it to not work on the iMessage app.  Unfortunately, it doesn't seem like it is under development and I suspect it will fail to work under Catalina.  I don't have any idea how hyperdock decides what is a title bar or not.  I will see what I can do with AppleScript and BTT in the mean time.  I'll leave this issue open in the event someone has some insight.  Feel free to close it if it's of no interest regardless.

You can also just accept it if it's in the top 40px of your window or something like that. That's way easier to script and does not require you to use any Apple Script at all.
# grab current mouse xy coords
xy_coords=($(cliclick p | tr "," "\n"))

# set vertical threshold
h_threshold="40.0"

# compare with current window
# iff threshold is met move the window to the next space
yabai -m query --windows --window  | jq -re ".frame
    | (${xy_coords[1]} >= .x) and (${xy_coords[1]} <= .x + .w) and
      (${xy_coords[2]} >= .y) and (${xy_coords[2]} <= .y + ${h_threshold})" \
   && yabai -m window --space next

The example script uses cliclick to grab the mouse coords and jq for json parsing (brew install cliclick jq).

I started very briefly on gestures by hooking up the Apple multi-touch framework, but I shifted my interests to other fields and did not make anything of it.
If someone is interested in writing a system to recognize and turn custom gestures into actions feel free to take a look at what I started in skhd on the mt branch. I have made sure it is up to date with master, compiles properly and outputs some touch information.
I don't imagine that I'll be looking into this for quite a while, as there are far too many things I want to do.

I'll leave this issue open in the event someone has some insight. Feel free to close it if it's of no interest regardless.

I'll leave it open for some time so that it is noticable by other users, in case someone is interested in trying to implement this themselves. I'm not particularly interested in having it upstream in yabai though, but I would be open to a PR for this in skhd. This issue would maybe be better suited for follow-up in https://github.com/koekeishiya/skhd/issues/25


Local server extremely slow since 0.27

Describe the bug
Since version 0.27 slidev hangs (or is very slow) when running from a project with multiple slides.
To Reproduce
Steps to reproduce the behavior:

Clone https://github.com/fguisso/talks.git
Update the dependency of @slidev/cli to the current version (0.28.5)
run npm i
run npm run dev -- slides/top-10.md
See the process hanging/running at 100% cpu. It will finish after 1-2 minutes building

Desktop (please complete the following information):

OS: Linux fedora 5.16.8-200.fc35.x86_64
Browser: any
Slidev version: 0.28.5

On windows the server will not even start if the markdown files are in a subdirectory with error:
[vite] Internal server error: EMFILE: too many open files
Last known working version was 0.27.13. It may be the same error: Does it start normally if you put top-10.md in the project root directory?

I run macOS and have the same problem

This is still a problem - bump.

Just tried it with 0.36.11 - still the same behaviour

Hi! First and foremost, thanks for the awesome tool :fire: . For anyone bumping into this issue, opening a exclusive window (with a single tab) solved this for me (using firefox).


docs: change H in GitHub to uppercase in web page button

Fix H in GitHub to uppercase.

 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.


[Bug]: Table widget -> Toggle Editable Mode via JS -> does not work

Is there an existing issue for this?

[X] I have searched the existing issues

Description
In my table widget, if I make a column editable via the toggle button, it works fine.
But if I use the JS textbox, it does not work both ways. true or false.
Loom video link attached.
Steps To Reproduce
Link data to your table widget.
Try making any column editable via the JS textbox.
This problem will arise.
Public Sample App
No response
Environment
Production
Severity
High (Blocker to building or releasing)
Issue video log
https://www.loom.com/share/4fee727483ef4649a5db3003068a2c45?sid=e0444d9a-b738-48e7-ba43-864a9c7a6000
Version
Cloud

@carefine actually the editable field is not meant to be js enabled since the inter linked properties would not be configurable. Is there a use case you have for having this field as editable?

Yes. The use case is that the table must be editable based on timestamp of the row creation.
I want the editing to be allowed for upto 15mins after a new row is inserted.

I found a workaround for this.
First set the toggle switch to enabled.
Then setup the inter-linked properties.
After that use the js field to auto-enable/disable.
It then works as expected.

@carefine That's pretty interesting. Can you tell me a bit more about the type of app you are building that requires such a condition to be enforced? It also seems that this condition should be on the row level and not the column level if I understand correctly.

I'm building an Invoice recording app.
The user uploads a PDF of the Supplier's Invoice document.
The PDF base64 is then pushed to a lambda function where it then converts the multipage pdf to a single jpg image file stitched below each other.
This image file is then pushed to ChatGPT's API aong with the newest-release syntax for Structured-Output response.
(The ChatGPT Api does not accept pdf at the moment. Hence, needed to convert it to jpg.)
The near-perfect JSON is then returned to Appsmith.
The JSON is then inserted to the AWS RDS MYSQL db.
The data is displayed in a Table Widget.
ChatGPT OCR capabilities are 99% perfect. But there is some discrepency sometimes.
Its critical that a few fiels like Invoice Number, Supplier Name and a few other are perfectly matching to the actual document.
So downstream activities dont get affected.
Now here is where editable table comes into play.
I want the user to be able to make these changes to these few fields only upto 15 mins after the API response.
So that there is a time-bound pressure on him to complete it.
If this time-bound pressure is not there, then given the 10's of invoices we get daily, mistakes may remain uncorrected and affect downstream activites.
Hence, the row level timestamp is used and column level selected fields are made editable.
Its turning out to be a neat app.
All very much thanks to you and your splendid team at appsmith.


Eliminate range-based for loops from C++ code

The range-based for is a C++11 feature that is not supported in VS 2010 or gcc versions below 4.6. There are 6 uses of range-based for in the grpc source code and about 16 in the test (of which all but 1 are in the qps test). Since this is purely syntactic sugar, I would like to take a crack at it.

Well, trying to support gcc 4.4 has at least one other complication: gcc 4.4 doesn't support lambda expressions. This is hardly an issue in the main code; these are only used in thread_pool.cc . However, I believe that lambdas are heavily used in the tests (particularly the qps test!)
Otherwise, I have a patch for range-based for issues ready to go.

Closed by PR #1123


Cherrypick 2023-05-10

Got a few more big refactors this time around. Most of the changes were technical.
On my side of things, I fixed some adjacent bugs while certain systems were getting rewritten. I also got almost every test passing locally, except PardonTest, which I think is due to having a non-UTC timezone. We'll see if GitHub tests pass 100%. I felt confident and even re-enabled the three Mail tests which were failing non-deterministically.
This update also introduces new behavior for the food-eating mice from upstream: HungerPrecondition, an HTNPrecondition which passes if the NPC's hunger meets a specificed threshold. I felt this was important to get in this update, given how the mice from upstream will eat without stopping.
Shoveling should work again too, mostly. I haven't accounted for planet tiles and how there is no regular snow tile, just a planet one. It works enough that people should be able to maintain subfloor wires/pipes on Glacier.
Kudzu got refactored, so I adjusted the upstream stats. It should hopefully be more manageable, but we can look at it later if it's an issue. It's all in YAML now.
Stations can have round-start gravity again. This is something I had to patch here. Mappers will have to do two things:

VV the grid's GravityComponent and set Enabled to true.
VV the grid's gravity generator and set GravityActive to true.

You can set the grid's gravity on upstream, but the GravityActive datafield was removed during a refactor back in December.
I added some entries in migration.yml to deal with any issues from upstream maps after synchronizing some of the core ones (nuclear shuttle, CentCom, etc.).
:cl:

add: Merged ~200 changes from upstream. Report bugs and regressions.
fix: Shoveling tiles should work again.
fix: Stations will have gravity at the start of the round again.
tweak: Mice will become hungry less often.
tweak: Adjusted kudzu growth speed, again.

With this update, all tests now pass. All future PR authors should take care to examine any test failures.


Various small fixes

Enabled type checking in VSCode
Various small fixes in type hints
Fixed a flaky address generator

Impact

[ ] New migrations
[ ] New/Updated API fields or mutations
[ ] Deprecated API fields or mutations
[ ] Removed API types, fields, or mutations
[ ] Documentation needs to be updated

Pull Request Checklist

[ ] Privileged queries and mutations are guarded by proper permission checks
[ ] Database queries are optimized and the number of queries is constant
[ ] Database migration files are up to date
[ ] The changes are tested
[ ] GraphQL schema and type definitions are up to date
[ ] Changes are mentioned in the changelog

@maarcingebala Removed that part of .editorconfig as it was never standardized and is not widely supported.


num_training_batches is inf in configure_optimizers

Bug description
The value of num_training_batches is inf when referenced in configure_optimizers(). It seems that it doesn't actually get its correct value until some point later. This causes a very hard-to-find issue because the training runs without error, except the loss is nan.
Something inside optim.lr_scheduler.CyclicLR actually sets the lr of the optimizer to nan.
It would be nice if:

This value was available configure_optimizers() was called, or
There was a warning if accessing it before it's set

How to reproduce the bug
import os

import torch
from torch.utils.data import DataLoader, Dataset

from pytorch_lightning import LightningModule, Trainer

class RandomDataset(Dataset):
    def __init__(self, size, length):
        self.len = length
        self.data = torch.randn(length, size)

    def __getitem__(self, index):
        return self.data[index]

    def __len__(self):
        return self.len

class BoringModel(LightningModule):
    def __init__(self):
        super().__init__()
        self.layer = torch.nn.Linear(32, 2)

    def forward(self, x):
        return self.layer(x)

    def training_step(self, batch, batch_idx):
        loss = self(batch).sum()
        self.log("train_loss", loss)
        return {"loss": loss}

    def validation_step(self, batch, batch_idx):
        loss = self(batch).sum()
        self.log("valid_loss", loss)

    def test_step(self, batch, batch_idx):
        loss = self(batch).sum()
        self.log("test_loss", loss)

    def configure_optimizers(self):
        optimizer = torch.optim.SGD(self.layer.parameters(), lr=0.1)
        print(f"{optimizer.param_groups[0]['lr'] = }")  # 0.1
        lr_scheduler = torch.optim.lr_scheduler.CyclicLR(
            optimizer=optimizer,
            base_lr=0.01,
            max_lr=0.1,
            step_size_up=self.trainer.num_training_batches * 1,  # problematic!
            step_size_down=self.trainer.num_training_batches * 2,  # problematic!
            cycle_momentum=False,
        )
        print(f"{optimizer.param_groups[0]['lr'] = }")  # nan
        return [optimizer], [lr_scheduler]

def run():
    train_data = DataLoader(RandomDataset(32, 64), batch_size=2)
    val_data = DataLoader(RandomDataset(32, 64), batch_size=2)
    test_data = DataLoader(RandomDataset(32, 64), batch_size=2)

    model = BoringModel()
    trainer = Trainer(
        default_root_dir=os.getcwd(),
        limit_train_batches=1,
        limit_val_batches=1,
        limit_test_batches=1,
        num_sanity_val_steps=0,
        max_epochs=1,
        enable_model_summary=False,
        enable_checkpointing=False,
    )
    trainer.fit(model, train_dataloaders=train_data, val_dataloaders=val_data)
    trainer.test(model, dataloaders=test_data)

if __name__ == "__main__":
    run()

Error messages and logs
The main hint something is wrong is actually tensorboard printing "NaN or Inf found in input tensor" - but even that doesn't come with a trace telling me who's printing this.
Environment

Current environment
#- Lightning Component (e.g. Trainer, LightningModule, LightningApp, LightningWork, LightningFlow):
#- PyTorch Lightning Version (e.g., 1.5.0):
#- Lightning App Version (e.g., 0.5.2):
#- PyTorch Version (e.g., 1.10):
#- Python version (e.g., 3.9):
#- OS (e.g., Linux):
#- CUDA/cuDNN version:
#- GPU models and configuration:
#- How you installed Lightning(`conda`, `pip`, source):
#- Running environment of LightningApp (e.g. local, cloud):

More info
No response

FWIW, I question the logic of "stale bots". Do you not want GitHub to be a place that users can inform you of potential problems? What does automatic closure achieve other than sweeping issues under the carpet? I'm much rather a human being apply the "won't fix" label, I'm OK with that.

I'm facing this one on v1.9.0. It seems to be a problem around DDP. One of the workers can get the correct value but the others fail and get inf.

I found estimated_stepping_batches property, which works fine with DDP.

have you found a workaround for this ?


[Question] OpenHab Integration?

Been using this for quite some time now. Thanks. Have started making the move with my devices over to openhab and I just spotted in your config.json that you have a part set out for it. Did you ever get to complete the integration and if so how did it work?

It was a while ago that I was working on that and I don't recall that I ever got it working to my satisfaction.  While you experiment with it, please let us know how it goes.

I have tried to use the sample above with openhab2:
2 things:
xxx.Sitemap: the fist statement must start with  the keyword "sitemap' ex : sitemap Garage label="My Garage"
There is a small bug in the openhab part of the controller.py:
def update_openhab(self, item, state):
......

   conn.request("PUT", "/rest/items/%s/%s" % (item, state))

   conn.request("PUT", "/rest/items/%s/state" % item, state)

I am totally new with openhab so I try to adapt the given code.
Thanks to Lamping7

@Gilles94500

Above is only a snippet from my complete .sitemap file, which has many other things, like a frame for my thermostat controls.
Because I'm using the HTTP binding to query the garage door controller twisted server, I'm not using the openHAB portion of controller.py.

@Lamping7
Thanks, so I could disable the openhab section in the config.json file.
I know that it is not an openhab forum, but I don't know how to ask this question in the openhab forum because I have not added this using the openhab menus. Based on your extract, how can I add a color (green when closed and red when open), because everything is grey. (Or point me to a link)
.
Thanks
Gilles

What do you mean? You want the icons to change colors?

Yes , I would like to use different icons based on the status.
Until now, I used the controller.py script developed by Andrew. As I am getting old I use a set of icons (green icon when the status is “closed”, amber when “opening” or “closing” and red when “open”) . I prefer to see a color that to try to read a status:) .
For security purpose, as you have explained above, I would prefer to connect to  Openhab instead. On top of that, I think that Openhab is able to show the status of my SONOFF devices.
So I have set up a Ubuntu and have installed Openhab. I am trying to make it work. Later, I will install Openhab on the raspberry.
Gilles
From: Anthony Lamping<EMAIL_ADDRESS>Sent: jeudi 26 avril 2018 14:11
To: andrewshilliday/garage-door-controller<EMAIL_ADDRESS>Cc: Gilles94500<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [andrewshilliday/garage-door-controller] [Question] OpenHab Integration? (#69)
What do you mean? You want the icons to change colors?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/andrewshilliday/garage-door-controller/issues/69#issuecomment-384618129 , or mute the thread https://github.com/notifications/unsubscribe-auth/AQHIgePynyzD59gtpbJdeGbqNOwWv9_6ks5tsblQgaJpZM4QKSBq .  https://github.com/notifications/beacon/AQHIgSaUYa0X0Ak2T3GDMNvgUJ0FabFtks5tsblQgaJpZM4QKSBq.gif

I did this for one of my thermostat status items. Basically you need to create your own icons using your favorite image editor -- gimp for Linux works well. Then you have to name them appropriately... It took me a little while to figure this out. If your going to make your own icons, I can share a working example of how to use it.

I have made 4 colored icons  that I use with the controller.py (garage-door-controller\www\img).
I have read in the openhab doc that to use my icons, I need to copy them in the  $OPENHAB_CONF/icons/classic/ directory but  I don’t know how to set them depending on the status(closed, opening, open, closing, closed). If you could give me some hints it would be very helpful.
From: Anthony Lamping<EMAIL_ADDRESS>Sent: jeudi 26 avril 2018 14:32
To: andrewshilliday/garage-door-controller<EMAIL_ADDRESS>Cc: Gilles94500<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [andrewshilliday/garage-door-controller] [Question] OpenHab Integration? (#69)
I did this for one of my thermostat status items. Basically you need to create your own icons using your favorite image editor -- gimp for Linux works well. Then you have to name them appropriately... It took me a little while to figure this out. If your going to make your own icons, I can share a working example of how to use it.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/andrewshilliday/garage-door-controller/issues/69#issuecomment-384623942 , or mute the thread https://github.com/notifications/unsubscribe-auth/AQHIgV6senDEWeEjWwBfMbYbUIXKUbiIks5tsb5XgaJpZM4QKSBq .  https://github.com/notifications/beacon/AQHIgY5DMSsqnWM9uoYnaMay1Q6Z02dpks5tsb5XgaJpZM4QKSBq.gif

So the magic happens in .rules file where it sets the string that is used to find the icon file defined in the .items file.
rule "EquipmentStatus"
when
  Item ecobee_equipmentStatus changed
then
  if ((ecobee_equipmentStatus.state != null) && (ecobee_equipmentStatus.state != NULL)
        && (ecobee_equipmentStatus.state != "")) {
    switch ecobee_equipmentStatus.state.toString {
      case "fan"		: ecobee_runningEquip.postUpdate("fan")
      case "compCool1,fan"	: ecobee_runningEquip.postUpdate("cool")
      case "auxHeat1,fan"	: ecobee_runningEquip.postUpdate("heat")
    }
  } else {
   ecobee_runningEquip. postUpdate("off")
  }
end

Inside my .items file is (the <> is the important part):
String ecobee_runningEquip		"Status" <hvac>				{ autoupdate="false" }

Icons are here and named to match the rules:

Got it.
I have renamed my icons accordingly and it, woks.
Y:\>cd icons

Y:\icons>dir
 Le volume dans le lecteur Y s'appelle openHAB2-conf
 Le numéro de série du volume est 4910-FB5D

 Répertoire de Y:\icons

24/04/2018  10:29    <REP>          .
30/04/2018  09:03    <REP>          ..
28/04/2018  17:14    <REP>          classic
               0 fichier(s)                0 octets
               3 Rép(s)  76 237 279 232 octets libres

Y:\icons>cd classic

Y:\icons\classic>dir
 Le volume dans le lecteur Y s'appelle openHAB2-conf
 Le numéro de série du volume est 4910-FB5D

 Répertoire de Y:\icons\classic

28/04/2018  17:14    <REP>          .
24/04/2018  10:29    <REP>          ..
14/01/2017  08:17               859 garage.png
18/12/2017  14:44               284 readme.txt
14/01/2017  08:17             1 042 garage-opening.png
14/01/2017  08:16               955 garage-closing.png
14/01/2017  08:16               890 garage-closed.png
14/01/2017  08:17               859 garage-open.png
               6 fichier(s)            4 889 octets
               2 Rép(s)  76 237 279 232 octets libres

Y:\icons\classic>


split address to multiple key

WARNING! NEEDS TO REFETCH ALL OF DOCKER OSM DATABASE
To apply this change, if not, the server will always gives error
To refetch it

Delete database of osm
Delete volume of all docker osm
Redeploy docker osm

CHANGES
change addr:full to use

addr:housenumber
addr:street
addr:city
addr:country
addr:postalcode

The info of address

Updated form

When empty

Country can be selected and searched

My Example:

On OSM

Note:
i'm using master.apis.dev.openstreetmap.org to test
https://master.apis.dev.openstreetmap.org/node/4319572847#map=19/-6.95801/107.64781

One suggestion: It is normally not necessary to specify the country of an address, because it is most of the time obvious from its location inside of a country.
Also looking at https://master.apis.dev.openstreetmap.org/node/4319572847, it seems like the addr:city field has a wrong value of Indonesia rather than the actual name of the city (Bandung, I assume).

thank you @tyrasd
i'm sorry, i paste wrong url
https://master.apis.dev.openstreetmap.org/node/4319572849
this is the correct one  based on my test changes in here
https://user-images.githubusercontent.com/4530905/77432921-489d7700-6e11-11ea-8cb1-bd36e7a32479.png

ah yes, this one looks fine. :+1:

UPDATE:

Relayouting of address input
Call nominatim to get suggestion of address

Demo:
https://drive.google.com/file/d/19rCNDoiIUyMfjeGvuzhKqCzhVRRf7UYB/view?usp=sharing

@meomancer this has been reverted, could you please created new PR, but this time against develop?

hi @nixilla
https://github.com/healthsites/healthsites/pull/1391
could you make develop as default? it's always to be master at default and sometimes i forget to change it


ThrottleStop 9.5 download failed, link not working

Bug Report
Package Name: throttlestop (9.5) [64bit] from extras bucket
Current Behaviour
ThrottleStop 9.5 from extras bucket doesn't download
Expected Behaviour
Should download normally
Additional context/output
Installing 'throttlestop' (9.5) [64bit] from extras bucket
The remote name could not be resolved: 'nl1-dl.techpowerup.com'
URL http://nl1-dl.techpowerup.com/files/ThrottleStop_9.5.zip is not valid

Possible Solution
Update the link?
System details
Windows version: 10
OS architecture: [e.g. 32bit, 64bit]
Scoop Configuration

{
    "last_update":  "2022-10-28T18:03:15.2959753+05:30",
    "aria2-enabled":  true,
    "scoop_branch":  "master",
    "scoop_repo":  "https://github.com/lukesampson/scoop"
}

extras/gpu-z and extras/nvcleanstall too.

> scoop install throttlestop gpu-z
Installing 'gpu-z' (2.51.0) [64bit] from extras bucket
GPU-Z.2.51.0.exe (9.0 MB) [===================================================================================] 100%
Checking hash of GPU-Z.2.51.0.exe ... ok.
Linking ~\scoop\apps\gpu-z\current => ~\scoop\apps\gpu-z\2.51.0
Creating shim for 'GPU-Z'.
Creating shortcut for GPU-Z (GPU-Z.exe)
'gpu-z' (2.51.0) was installed successfully!
Installing 'throttlestop' (9.5) [64bit] from extras bucket
ThrottleStop_9.5.zip (1.7 MB) [===============================================================================] 100%
Checking hash of ThrottleStop_9.5.zip ... ok.
Extracting ThrottleStop_9.5.zip ... done.
Running pre_install script...
Linking ~\scoop\apps\throttlestop\current => ~\scoop\apps\throttlestop\9.5
Creating shim for 'ThrottleStop'.
Creating shortcut for ThrottleStop (ThrottleStop.exe)
Persisting Logs
Persisting ThrottleStop.ini
'throttlestop' (9.5) was installed successfully!

works now.


Lines 165 ff.: Why are the update methods before the create methods?

Comment on Line 159 should say,  # update checkbox question
But, “question” should be replaced with “item” throughout.
Why is code so different between update & create methods?
Should this code be moved to the model files for the appropriate question types?

This code is so different between update and create methods because

The create methods form a layout of sorts for the question (multichoice or true/false) and have at least one of the key indexes be correct.
The update methods split the multiple choice questions into checkbox or radio sections. Checkboxes can have multiple correct answers, which is the reason for its split from radio - which can only have one.
The update methods work on a single question choice, another reason why update and create code is so different. They take an existing choice and just update its attributes, such as its txt and if it is correct.


SUBMARINE-717. Add NFS support for persistent volume

What is this PR for?

User can use their own NFS server to store persistent storage data.
For developers, who might not have an existing nfs server, they can create an nfs server in kubernetes cluster by the yaml provided in dev-support/.
Added the guidance of nfs usage in docs.
Store MySQL data in PV.

What type of PR is it?
[Feature]
Todos

[ ] - Store notebook data in nfs
[ ] - Store tensorboard data in nfs

What is the Jira issue?
https://issues.apache.org/jira/projects/SUBMARINE/issues/SUBMARINE-717
How should this be tested?
https://issues.apache.org/jira/projects/SUBMARINE/issues/SUBMARINE-717
Questions:

Does the licenses files need update? No
Is there breaking changes for older versions? No
Does this needs documentation? No

@tangzhankun help me review this PR. Thanks!

@tangzhankun help me review this PR. Thanks!

@tangzhankun I have tested it with kind and minikube. They both work fine :)

Will merge if no more comment


Font generator occasionally clips "i" incorrectly

Roboto:

I've tried using both Source Sans 3 and Roboto, as well as DirectWrite and Freetype. Not sure why there's a little blob under lowercase "i", especially since it doesn't always happen. I can provide my font JSON if needed.

Some more examples:
Source Sans 3

Noto Sans


Roll engine 206cab6e7013..ea28bfa13e9d (11 commits)

https://github.com/flutter/engine/compare/206cab6e7013...ea28bfa13e9d
git log 206cab6e701392fe0433f15a248516e94d4f8025..ea28bfa13e9d614c126cc3cfe5238c666afdc7d6 --no-merges --oneline
ea28bfa13 Custom RTL handling for ghost runs, NotoNaskhArabic test font (flutter/engine#8683)
1c9457ced Revert "Only allow mappings for ICU initialization. (#8656)" (flutter/engine#8682)
b4ed3039f Revert "RTL handling for ghost runs, NotoNaskhArabic test font (#8638)" (flutter/engine#8681)
8b5f77630 Remove support for downloading dynamic patches. (flutter/engine#8663)
1bcb96bf3 Add framework test in engine presubmit checks (flutter/engine#8664)
6e79dcd0c RTL handling for ghost runs, NotoNaskhArabic test font (flutter/engine#8638)
c37d45981 Roll src/third_party/skia 1c0b05afdef8..a76fad2d494a (4 commits) (flutter/engine#8665)
2e4f0a4a7 Put the testing lib in the flutter namespace. (flutter/engine#8661)
1239df96a Allow native bindings in secondary isolates. (flutter/engine#8658)
b0cbce417 Replace ThreadLocal with ThreadLocalUniquePtr<T> (flutter/engine#8659)
bd8c5b135 Only allow mappings for ICU initialization. (flutter/engine#8656)
The AutoRoll server is located here: https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+/master/autoroll/README.md
If the roll is causing failures, please contact the current sheriff (liyuqian@google.com), and stop
the roller if necessary.

Auto-roller completed checks. About to merge.


Pseudo syntax isn't compatible with IE < 8

In IE < 8, the colon char isn't valid in CSS selectors, even if escaped, it seems.  For instance, the class
box:h_D(n)! will apply correctly in IE 8+, but will not apply in IE < 8.  The only way to get the class to apply is to modify it to remove the colon, from what I've found so far.
Simple test case here:
https://gist.github.com/src-code/4c17ecfbcac43e0ca12a
@dnewcome @thierryk @renatoi @3den

To be clear, other escaped chars such as ( and ! don't seem to cause problems - only colons.

So here's a few ideas on how to address this, assuming we can't find a way to make IE < 8 accept the colons:

Introduce a new char for pseudos, and deprecate :.
Ignore it, and leave it up to developers to cope with archaic browseres by writing custom styles outside of Atomizer
???

honestly, since this is for older browsers, I'd lean towards #2, instead of changing the API again.

@redonkulus I'm leaning that way too.  We could just document the issue as a known limitation of old-browser support.

I agree with, the api is fine as it is. RIP: IE 7

Agree not to change the API. Using a char different than what is standard in CSS will just make the Atomizer syntax more confusing for little benefit.

Ok, it seems we're all on the same page here.  We won't fix pseudos in IE < 8, and leave it up to developers to cope through custom styles.  I'll add a note to our docs for this.

https://github.com/yahoo/acss-site/pull/90


Warning shows up when running ember s

I'm getting this warning when running ember s:
Warning: ignoring input sourcemap for vendor/popper.js/popper.js because ENOENT:
no such file or directory, open ' <...> /tmp/source_map_concat-input_base_path-YBKBBjEI.tmp/vendor/popper.js/popper.js.map'

Steps to reproduce:

ember new my-project
cd my-project
ember install ember-attacher
ember s

@tmeloliveira luckily this is just extra verbosity, and doesn't affect anything, but agreed that it is not ideal.
I'm looking into this now to see if I can simply ignore that file in ember-popper's imports


Problem finding mk_sched in notebook code for "Agent-based modeling via graph rewriting" blog

Hi,
Our CEPHIL team has been having trouble running block 17 of the following notebook code for "Agent-based modeling via graph rewriting" blog.
https://nbviewer.org/github/AlgebraicJulia/AlgebraicRewriting.jl/blob/compat_varacsets/docs/src/Dynamic Tracing.ipynb
Specifically, we get an error message indicating that "mk_sched" cannot be found.  Here's the exact code and error message:

My project .toml file is attached as a txt file (so it can be attached_); is it adopted from the July7_2023 commit (reflecting the July 7 date of the associated blog post (https://blog.algebraicjulia.org/post/2023/07/graphical-schedule/#cb3-2).
Project.toml.txt
I further attach my complete .ipynb file, again as a .txt file so that it can be attached.
ACT-ABM-BlogExample-Jan-2023_With-July7_2023_Project_toml.ipynb.txt
Thanks so much for any guidance that can be offered!

Hey! I’m not a developer on this package, but I have used it in the past and I can help you interpret this error message.
The Julia error that you got says that the types of the arguments that you gave to the mk_sched function are not correct, but it did find a candidate solution. The first arguments in black: mk_sched(::NamedTuple, ::NamedTuple, were matched correctly. Julia would use this definition of mk_sched if not for the remaining types, which it could not match on.
The first argument that it didn’t find (the first type in red) was something with the type ::Names. As an example on how to create a Names object, you can see this line from the tests: https://github.com/AlgebraicJulia/AlgebraicRewriting.jl/blob/ddce48efb155564e766db31b540e5abb7dca3259/test/schedules/Eval.jl#L38
And where it is used here: https://github.com/AlgebraicJulia/AlgebraicRewriting.jl/blob/ddce48efb155564e766db31b540e5abb7dca3259/test/schedules/Eval.jl#L70
So, we should create our Names object here like:
N = Names(Dict("S" => S, "I" => I, "G” => G));
And pass this as the third argument to mk_sched.
For an example of a similar dynamics, check out this docs page:
https://algebraicjulia.github.io/AlgebraicRewriting.jl/dev/generated/lotka_volterra/
You can ctrl+f in this document for Names and mk_sched.
I hope this helps! ~ Luke

Thanks Luke! I should note that, as of an hour ago, there's a new release of AlgebraicRewriting (0.3.1) to accompany a new release of Catlab! The docs page Luke linked is important because that file gets run automatically with the tests whenever we push code (the code in the blog post does not get run, so it can become out of date).
An important difference in this latest release so the rewriting simulation can be fast via the use of in-place rewriting (rather than Catlab's normal pushout infrastructure which is a pure, non-mutating function). For this to work, there's a keyword you need to pass in when creating your ACSet data type @acset_type MyACSetType(MySchema, part_type=BitSetParts).


⚠ ClamAV has degraded performance

In 4632816, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 65 ms

Resolved: ClamAV performance has improved in 462f7c9.


3rd CAN interrupt?

I've finally forked and am refactoring a simpler I/O version only of the code, and noticed there was an interrupt only for the first two chips (not used in the writing).  Is there an interrupt for the 3rd chip?  Also schematics or something similar with the hardware details would be helpful.  When my code does something useful I'll contribute it back.

@etx: Any updates on this?

@Tuckie We only have one version :) Well, we have some different colored enclosures but they are sold out ATM.


Native support for pre-receive hooks.

How do I use pre-receive git hooks on Gogs ? (If they exist in the first place. If they don't,it would be awesome to have them).

User has to be given permission in order to use git hooks.

Admin can give permission in users dashboard

I'm talking about 'pre-receive' hook.
The current user has permission to use git hooks (I've tried it with a post-receive hook).

I see, my bad. No, not yet, but can be added I think.

:trollface:

Please test out develop branch, I've added more Git hooks in the list.

Close as implemented.


How to create TFTensor batch of dtype string ?

Hi,
I have a model that takes as input a tensor of dtype string. I can successfully perform predictions with a single example using TFTensor.CreateString to create a string tensor of size 1.
However I would like to be able to evaluate a batch of examples at once, but can't find a way to create a batch of examples from a list of string ?
Alternatively, is there a way to create a rank 1 TFTensor from a list of rank 0 TFTensors ?
Thanks !

Hi, did you find a solution for this issue?


What is the argument for 4K footage ?

I am currently using:
python inference_video.py --model-type mattingrefine --model-backbone resnet50 --model-checkpoint ./model/PyTorch/pytorch_resnet50.pth --model-refine-sample-pixels 320_000 --video-src ./input/Raw.avi --video-bgr ./input/Empty_Background.png --output-dir ./output --output-types com fgr pha err ref
So far it works VERY WELL.
But sometimes when the gap is small, the cut out will not be made probably because the sample is too low ?
What modifications should I make to the command line statement that I use on top to make it really good for 4K footage ?
I don't care if it is not real time, I care more about the cut out quality, it can take AS LONG AS IT TAKES, Quality comes first :D

4K I would use --model-backbone-scale 0.125. Sample pixels 320000 is fine.

Can I confirm that the whole statement would be:
python inference_video.py --model-type mattingrefine --model-backbone resnet50 --model-checkpoint ./model/PyTorch/pytorch_resnet50.pth --model-backbone-scale 0.125 320_000 --video-src ./input/Raw.avi --video-bgr ./input/Empty_Background.png --output-dir ./output --output-types com fgr pha err ref
Thanks.

You missed --model-refine-sample-pixels. Other than that it's good.

Where do I put that ? Before or after the --model-backbone-scale 0.125 320_000 ?
Can you give me the full complete statement ?

This should be good, but I haven't tested it myself.
python inference_video.py --model-type mattingrefine --model-backbone resnet50 --model-checkpoint ./model/PyTorch/pytorch_resnet50.pth --model-backbone-scale 0.125 --model-refine-sample-pixels 320_000 --video-src ./input/Raw.avi --video-bgr ./input/Empty_Background.png --output-dir ./output --output-types com fgr pha err ref


Fix reorg race condition that can cause rare crashes

Fixes issue #408.
This bug was introduced by PR #393, which changed how txids are
determined. That PR changed each call to the zcash getblock call into a
pair of calls, the first to get the raw block data, the second to
retrieve the txids in the block. (Unfortunately, you can't get both in a
single getblock RPC.) But this ordering introduced a timing window in
which the block at the given height can change, if a reorg occurred
between the two calls.
This PR reorders the getblock calls, so that the first call gets the
transaction IDs, which also happens to return the block hash, so then
the second getblock call can specify the block hash, rather than the
height. This ensures that the two RPC calls return consistent data,
definitely the same block.

Force-pushed to improve comment as a result of review comment (thanks, @daira).