open street map IDs

i THINK we are going to want some way to connect public goods and other artifacts to the items recorded in open street maps. @sujitmhj do you have access to OSM IDs?  can they be added to the data?
i need to figure out how to show these in the map.

@UpamanyuDatta thanks for the details!  @sujitmhj had showed me Kobo Toolbox, i had just forgotten the name.

Sujit and I had a chat with Sabin
and Sarita, who will go to the field sometime next week. The plan is to
give them some prototype that they can use to have a conversation with.

is that plan show this UI?

Dear Trebor,
I am throwing in some visual analyses into a Shiny app, as a prototype that
can be used to anchor discussions. Hopefully folks at Dhankuta will use
that as a starting point and provide more concrete recommendations that we
can incorporate into the React app. The React app will be the mainstay,
while this app a temporary repository of some analyses.
I just deployed the Shniy app at
https://upamanyudatta.shinyapps.io/LocalDataDraft1/
It has two main tabs, one using maps as the visual platform and the other
using a combination of chart and list as the visual-analytical platform.
The latter is intended more for municipal/ward functionaries, but am
curious to see if it finds purchase with tole committee members too.
Anyway, in the first tab, you can plot all assets, or assets by type. Each
asset would have some info displayed, which are coming from the Kobo data.
We can overlay toles into this, and request the distance to the nearest
asset of a certain type. An addition from our last discussion is that we
can filter a particular asset type by sub categories, and rerun the
analysis. For example, it can plot all educational institutes and all
toles, and show the most remote toles. But you can filter certain
educational institutes by category, for example colleges, and see  how far
each tole is from the closest college.
In the second tab, we can view ward and municipal level budget allocations,
overall and across major programs. We can also weight the allocations by
ward and municipal level demographic info (such as population, area,
population density) to maybe have a discussion about equity.
Please take a look when you have time, and let me know what you think. I am
going to be away till Sunday afternoon, but maybe we can have a chat on
Sunday evening if that works for you to touch base?
Upu
On Wed, Mar 15, 2023 at 11:38 AM Robert Harris @.***>
wrote:

@UpamanyuDatta https://github.com/UpamanyuDatta thanks for the details!
@sujitmhj https://github.com/sujitmhj had showed me Kobo Toolbox
https://www.kobotoolbox.org/, i had just forgotten the name.
Sujit and I had a chat with Sabin
and Sarita, who will go to the field sometime next week. The plan is to
give them some prototype that they can use to have a conversation with.
is that plan show this UI?
—
Reply to this email directly, view it on GitHub
https://github.com/trebor/sangai/issues/3#issuecomment-1470269255, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AEF3ZS2N54XIDJ42CGT6TGDW4HPA7ANCNFSM6AAAAAAVVXQNRE
.
You are receiving this because you were mentioned.Message ID:
@.***>


Clean-up Target configuration

There is room for potential gradle improvemetns. See:
https://github.com/touchlab/Kermit/pull/36#discussion_r416838847

HMPP dependent, revisit once thats working

HMPP and target rework done


Crash when trying to view the KAMI-ingredient researches when it's turned off

Like, we know that it's rare that one turns it off, but, like, we only installed the ttinkerer to test how the mod works
(it's really a shame that it's an obligate dependency, considering how almost all the others are optional)

Unfortunately, disabling of KAMI is not supported. Kinda silly that you install addon about expansion of other addons, and then cut half of their content ;)


chore: fix lock file

Caused by https://github.com/instana/nodejs/commit/8efd39332ddb8dcbf6347c95c89c6e14b21fec28
Caused by https://github.com/instana/nodejs/commit/67d27f3b312bf7b74490e5eb47b82f909ea99c4c


Adding the possibility to translate the msg.Data before output.

We have decided to use the nats-cli tool to assist in debugging and monitoring NATS stream messages stored in cbor format. However, we would prefer the messages displayed in an easier-to-read formatting. The same goes for JSON, which we would like to have a better readable representation too.
In response to my inquiry #636 about adding an encoder/decoder, @ripienaar suggested that this could be achieved by passing the output through a CLI utility that performs the translation. This would allow users to add their own translators or formatters as needed.
I have created a proof of concept implementation for this approach, and I hope that it will be incorporated into the tool in the future.
I added a template for {{subject}} which can be used to let the translator know about the subject, so one could write it in a way that it uses internal knowledge to assist in decoding the data without the need to add information to the messages. There could also be a template for header fields or maybe also other information (the stream name, for example). I didn't want to add much more until the general idea gets the green light.

I added a 'stream' template and pondered about adding the headers, as one could use them especially to describe the data.
I am also unsure of the argument name "translate", in our code base this would probably something like "cvt" or "flt".
The following is not part of this PR
I noticed that the code has many missing error checks ("will never happen" / "if that breaks, I am fleeing the country already"). We made a simple "must" package (using generics) to create must.Ok(), must.OkOne(), must.OkSkipOne() (and so on) which simply check and panic or return/ignore one return value.
We also have an alternative package that checks for the errors and on error lists just the file and line of the caller in the source and os.Exit(1). This is what we use for CLI tooling that is used by people that freak out when they get a panic with stack-dump. :)
This way it is effortless and readable to cover all error checks without having all the single if statements for the error checks. I would suggest adding something like this and could volunteer.

Hmm mind pointing at a few missing error checks?

Hmm mind pointing at a few missing error checks?

There are quite a lot

That’s a test :) fairly acceptable imo as following assertions would not pass had there been an error.
There are in general vast amounts of unchecked errors you are right - like fmt.Printf returns unchecked errors. So I would be curious if one’s if actual consequence outside of tests.

As usual, adding something like that opens up plenty of ideas. I would rather go with a minimalistic approach first and see what emerges from that. It should be easy to add more later on. For adding something like JSON information, one may create a temp file with that JSON, add the filename as template and remove the JSON after the translator has run.

That’s a test :) fairly acceptable imo as following assertions would not pass had there been an error.

I would add them, especially in tests. That makes it easier to see where it breaks if it breaks.
This is what I get when I quickly filter out '*_test.go':

This only comes to my mind because we have a zero warning policy for all code (using GoLand) and I also plan a global golangci-lint config to be mandatory for all pushed code into the CI/CD chains. But I know that I am quite strict with that in our company. We should have more tests though :)

In this case I would fetch that command. It’s old old code that was just copy and pasted in here.
I wouldn’t want to cha get the code style at all. Rather just add checks where they are of consequence.

I really didn't want to steer up the error handling stuff. As mentioned earlier. This PR was made to extend the ways of using the wonderful nats-cli tool with data that's encoded in some binary format. I actually wrote some "universal translator" implementation today that also uses AVRO model info stored in the stream to decode messages. It helps already by allowing to use "hex", "raw escaped strings", "CBOR", "AVRO" and "JSON" formats. Using "GOB" turned out to neither be smaller than JSON nor is debug printable without recompiling the tools and adding all the structures (or writing a debugging decoder from scratch). I didn't use AVRO before, but found that this may be the best way for us to store large amounts of data consisting of many messages (which makes gob rather bloated). It seems that Kafka has a point in using it. The possibility to store the structure as text and don't need to recompile when updating the format, makes it quite interesting when storing many messages with the same (binary) format.

About that "must" package:
I like how the code looks when using it while prototyping or even for production:
dec := cbor.NewDecoder(os.Stdin)
var cborNative any
must.Ok(dec.Decode(&cborNative))
mappedKeys := must.OkOne(convertMapKeysToStrings(cborNative))
jsonData := must.OkOne(json.MarshalIndent(mappedKeys, "", "  "))
fmt.Printf("%s\n", jsonData)

We have now two packages. One that just uses panic() for the errors and another one that uses log/slog and os.Exit().
Because of this conversation, I decided to make them both exchangeable. I find that much better than having the if(err) { panic(err) }. We do forbid variable shadowing too, and as a result, you need to name many of your errors when using just "err" all the time.

I'll review the data translation towards end of week, thank you.
I would not be interested in anything like the must package here, happy to have PRs adding idiomatic go error handling where its lacking now.

OK. I consolidated the code and think that the generalized output function should result in the same user experience while unifying the edge cases (like nil message body). I did not rename the source file which contains that code to make it easier to see my changes. I guess it should be renamed to "output_msg_body.go" before merging.
What do you think about it overall now?

Good stuff thank you, do you mind signing this with gpg or your ssh key? Please squash to one commit and sign that commit.

Good stuff thank you, do you mind signing this with gpg or your ssh key? Please squash to one commit and sign that commit.

I actually never used signing before. I hope I did that right.

Yup signature looks good

Thank you


Could not debug coffee-script app

Hy,
i have troubles debugging a simple express-app written in coffeescript.
Here is the source:

I start the script with coffee --nodejs --debug app.coffee and i can see the script in node-inspector, but the start of the script is different. It is not possible to set a break-point!

If i compile the coffeescript to javascript with following command coffee -c -m app.coffee and start the generated js file with node --debug app.js everything works just fine.
Does anybody has the same problem? Or is that an limitation of node-inspector.
OS: Mac OS X Mavericks
node 0.10.30
node-inspector 0.7.4
coffee-script 1.8.0
express 4.0.0
body-parser 1.0.1
Greetings
Tonka

@bajtos ,Hi , I can not debug too, It just said: $ debugger listening on port 5858, but actually nothing happened when I run $ coffee --nodejs --debug app.coffee

@TangMonk of course the chrome was not opened.
Steps to debug:

$ node-inspector &
$ coffee --nodejs --debug app.coffee
Manually load http://<IP_ADDRESS>:8080/debug?port=5858 in your Chrome browser.

Note that you will see the transpiled javascript, not the original coffeescript sources - see #224.

@bajtos thanks ! :+1:

@bajtos thanks a lot!


Examples and template for #32 and #444

This provides a template for documenting examples and the examples for verbatimLabel.
This has been reviewed by @peterdesmet (thanks Peter) for formatting and discussion around the filenames etc. This will result in a page on https://dwc.tdwg.org/examples/verbatimLabel appearing as follows:

Very nice!


feat: add rectAreaLightHelper to vLightHelper directive

FYI, I think the old branch can be deleted:
https://github.com/Tresjs/cientos/pull/214


Shows break progress bar and time remaining

fixes #71

Codecov Report

Merging #74 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #74   +/-   ##
=======================================
  Coverage   90.12%   90.12%
=======================================
  Files           5        5
  Lines          81       81
  Branches       11       11
=======================================
  Hits           73       73
  Misses          8        8

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 b125d39...dec8b0a. Read the comment docs.


Dataset with time range 4029-01-16 12:00:00 ... 4114-12-16 12:00:00

The dataset /badc/cmip6/data/CMIP6/ScenarioMIP/THU/CIESM/ssp585/r1i1p1f1/Amon/rsus/gr/v20200806 has a time range from 4029-01-16 12:00:00 ... 4114-12-16 12:00:00.
This is the only one in the latest inventory (v20210301) that has a time range so far in the future. Should this be included in the inventory?

tagging @RuthPetrie @agstephens

From https://view.es-doc.org/?renderMethod=name&project=cmip6&type=cim.2.designing.NumericalExperiment&client=esdoc-url-rewrite&name=ssp585 it seems this is incorrect and the time range should be<PHONE_NUMBER>12

Running ls /badc/cmip6/data/CMIP6/ScenarioMIP/THU/CIESM/ssp585/r1i1p1f1/Amon/*/gr/latest/ shows there are others with the same time range:
/badc/cmip6/data/CMIP6/ScenarioMIP/THU/CIESM/ssp585/r1i1p1f1/Amon/rlus/gr/latest/:
rlus_Amon_CIESM_ssp585_r1i1p1f1_gr_402901-411412.nc

/badc/cmip6/data/CMIP6/ScenarioMIP/THU/CIESM/ssp585/r1i1p1f1/Amon/rsdt/gr/latest/:
rsdt_Amon_CIESM_ssp585_r1i1p1f1_gr_402901-411412.nc

/badc/cmip6/data/CMIP6/ScenarioMIP/THU/CIESM/ssp585/r1i1p1f1/Amon/rsus/gr/latest/:
rsus_Amon_CIESM_ssp585_r1i1p1f1_gr_402901-411412.nc

/badc/cmip6/data/CMIP6/ScenarioMIP/THU/CIESM/ssp585/r1i1p1f1/Amon/rsutcs/gr/latest/:
rsutcs_Amon_CIESM_ssp585_r1i1p1f1_gr_402901-411412.nc

This has been removed from the inventory for now

@martinjuckes: should this dataset be included in the latest C3S inventory? It is the only data in the 41st century.


chore: enable integration tests for windows

oops, sorry just merged #2683 disabling windows. Can re-enable it now just by adding into the matrix

oops, sorry just merged #2683 disabling windows. Can re-enable it now just by adding into the matrix

No worries, this was just a quick test for why we're disabling windows since I forgot 😅

Mehul got our Windows stuff lined up today. 🦸
This looks stale so I'll close.


Prometheus Metrics about Postgres

Postgres Exporter can run as a sidecar to gather the metrics from postgres pod and provide the /metrics endpoint

Probably not needed as we will get rid of Postgres short term

no postgres no more


Running into errors attempting to build on Windows

Hello all, I'm attempting to build registrator on Windows and currently getting the following build error. Any input at all would be helpful.

What version of docker are you running?

Server Version: 17.06.2-ee-8 - Running on Windows 2016 1607

What version of registrator are you running?

Attempting to build from master/release which is v7.
Description of the problem:
Build for Windows isn't completing, stuck on some etcd error, here is my hobbled together Dockerfile.

Sending build context to Docker daemon  103.9kB
Step 1/5 : FROM golang:1.10-windowsservercore
 ---> 8520b84033e1
Step 2/5 : SHELL powershell -Command $ErrorActionPreference = 'Stop'; $ProgressPreference = 'SilentlyContinue';
 ---> Using cache
 ---> bcb291ebca85
Step 3/5 : COPY . /work/go/src/github.com/gliderlabs/registrator
 ---> aa024572bd3b
Removing intermediate container 890d313f4edb
Step 4/5 : RUN cd C:/work/go/src/github.com/gliderlabs/registrator;     $env:GOPATH='C:\work\go';       git config --global http.https://gopkg.in.followRedirects true;         go get;
go build -ldflags '-X main.Version=v7' -o C:/registrator.exe;   cd c:/;         rm /work -Force -Recurse;       & C:/registrator.exe --version;
 ---> Running in e9d78ce2ddd8
# github.com/coreos/go-etcd/etcd
..\..\coreos\go-etcd\etcd\response.generated.go:99:6: r.EncodeArrayStart undefined (type codec.genHelperEncDriver has no field or method EncodeArrayStart)
..\..\coreos\go-etcd\etcd\response.generated.go:107:6: r.EncodeMapStart undefined (type codec.genHelperEncDriver has no field or method EncodeMapStart)
..\..\coreos\go-etcd\etcd\response.generated.go:111:6: z.EncSendContainerState undefined (type codec.genHelperEncoder has no field or method EncSendContainerState)
..\..\coreos\go-etcd\etcd\response.generated.go:119:6: z.EncSendContainerState undefined (type codec.genHelperEncoder has no field or method EncSendContainerState)
..\..\coreos\go-etcd\etcd\response.generated.go:121:6: z.EncSendContainerState undefined (type codec.genHelperEncoder has no field or method EncSendContainerState)
..\..\coreos\go-etcd\etcd\response.generated.go:130:6: z.EncSendContainerState undefined (type codec.genHelperEncoder has no field or method EncSendContainerState)
..\..\coreos\go-etcd\etcd\response.generated.go:142:6: z.EncSendContainerState undefined (type codec.genHelperEncoder has no field or method EncSendContainerState)
..\..\coreos\go-etcd\etcd\response.generated.go:144:6: z.EncSendContainerState undefined (type codec.genHelperEncoder has no field or method EncSendContainerState)
..\..\coreos\go-etcd\etcd\response.generated.go:157:6: z.EncSendContainerState undefined (type codec.genHelperEncoder has no field or method EncSendContainerState)
..\..\coreos\go-etcd\etcd\response.generated.go:170:6: z.EncSendContainerState undefined (type codec.genHelperEncoder has no field or method EncSendContainerState)
..\..\coreos\go-etcd\etcd\response.generated.go:170:6: too many errors
# github.com/coreos/go-etcd/etcd
..\..\coreos\go-etcd\etcd\response.generated.go:99:6: r.EncodeArrayStart undefined (type codec.genHelperEncDriver has no field or method EncodeArrayStart)
..\..\coreos\go-etcd\etcd\response.generated.go:107:6: r.EncodeMapStart undefined (type codec.genHelperEncDriver has no field or method EncodeMapStart)
..\..\coreos\go-etcd\etcd\response.generated.go:111:6: z.EncSendContainerState undefined (type codec.genHelperEncoder has no field or method EncSendContainerState)
..\..\coreos\go-etcd\etcd\response.generated.go:119:6: z.EncSendContainerState undefined (type codec.genHelperEncoder has no field or method EncSendContainerState)
..\..\coreos\go-etcd\etcd\response.generated.go:121:6: z.EncSendContainerState undefined (type codec.genHelperEncoder has no field or method EncSendContainerState)
..\..\coreos\go-etcd\etcd\response.generated.go:130:6: z.EncSendContainerState undefined (type codec.genHelperEncoder has no field or method EncSendContainerState)
..\..\coreos\go-etcd\etcd\response.generated.go:142:6: z.EncSendContainerState undefined (type codec.genHelperEncoder has no field or method EncSendContainerState)
..\..\coreos\go-etcd\etcd\response.generated.go:144:6: z.EncSendContainerState undefined (type codec.genHelperEncoder has no field or method EncSendContainerState)
..\..\coreos\go-etcd\etcd\response.generated.go:157:6: z.EncSendContainerState undefined (type codec.genHelperEncoder has no field or method EncSendContainerState)
..\..\coreos\go-etcd\etcd\response.generated.go:170:6: z.EncSendContainerState undefined (type codec.genHelperEncoder has no field or method EncSendContainerState)
..\..\coreos\go-etcd\etcd\response.generated.go:170:6: too many errors
& : The term 'C:/registrator.exe' is not recognized as the name of a cmdlet,
function, script file, or operable program. Check the spelling of the name, or
if a path was included, verify that the path is correct and try again.
At line:1 char:325
+ ... r.exe; cd c:/; rm /work -Force -Recurse; & C:/registrator.exe --versi ...
+                                                ~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : ObjectNotFound: (C:/registrator.exe:String) [],
   ParentContainsErrorRecordException
    + FullyQualifiedErrorId : CommandNotFoundException

The command 'powershell -Command $ErrorActionPreference = 'Stop'; $ProgressPreference = 'SilentlyContinue'; cd C:/work/go/src/github.com/gliderlabs/registrator;        $env:GOPATH='C:\wo
rk\go';         git config --global http.https://gopkg.in.followRedirects true;         go get;         go build -ldflags '-X main.Version=v7' -o C:/registrator.exe;   cd c:/;         rm
 /work -Force -Recurse;         & C:/registrator.exe --version;' returned a non-zero code: 1

I'm not sure if this is an issue with registrator but more on the lines of the etcd imports. I commented out etcd imports since we only plan on using it with consul and was able to build successfully.


chore!(v0): update v0 for upcoming ckb 2023 hardfork

Updated rust, ckb deps, ckb-vm, ckb-sdk and ckb-cli. The ckb builtin indexer is required (and the standalone indexer is no longer supported).
Breaking change is only in gw-tools: the ckb indexer url arg is removed.

[ ] A new image would need to be built. It should have a newer ckb-cli.
[ ] Kicker will also need some small updates.

The ckb builtin indexer is required (and the standalone indexer is no longer supported in the next version of Godwoken v0.
cc @Simon-Tl

I've tested this with https://github.com/godwokenrises/godwoken-kicker/commit/c65db3158db0995dcec46a09170ec24b38e432fc and v0 works fine.
Now we just need to add a github workflow to build and push the image.

please update the image used in https://github.com/godwokenrises/godwoken/blob/0.10-rc/devtools/fetch-binaries.sh too.

Please fix CI @blckngm

We should update to ckb 0.111.0 and ckb-sdk 3.0. Will update later.


[Enhance](array function) add support for DecimalV3 for array_enumerate_uniq()

Proposed changes
Issue Number: close #xxx
Problem summary
Describe your changes.
Checklist(Required)

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

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


Managing open windows

Currently on macOS, Station will open links in new windows, however the window toolbar doesn't report them and the 'bring all to front' command doesn't perform any function on main or subsequent windows.
Tabbed browsing would be great. But in the meantime control over windows with hotkeys please?
Stationversion 2.7.3
Electron25.9.8
Chrome114.0.5735.289
Node18.15.0
v8<IP_ADDRESS>-electron.0

Try Ctrl + (Cmd + for MacOS) to switch focused window.

Cmd + is zoom in on MacOS
Is there a missing key here? i.e Cmd and xyz
On Tue, Mar 12, 2024 at 3:17 AM Viktor @.***> wrote:

Try Ctrl + (Cmd + for MacOS) to switch focused window.
—
Reply to this email directly, view it on GitHub
https://github.com/getstation/desktop-app/issues/351#issuecomment-1988832893,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AXPQQVSPHJXUU6BKRB5RER3YXXKJZAVCNFSM6AAAAABEISH73OVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSOBYHAZTEOBZGM
.
You are receiving this because you authored the thread.Message ID:
@.***>

--
Fraser Walker https://www.monikeraudio.com/
Audio Engineer
m +61 403 727 194

Cmd + is zoom in on MacOS Is there a missing key here? i.e Cmd and xyz

[Cmd + `(backticks)]

Yep that's working, cheers!
On Tue, Mar 12, 2024 at 8:49 PM Viktor @.***> wrote:

Cmd + is zoom in on MacOS Is there a missing key here? i.e Cmd and xyz
[Cmd + `(backticks)]
—
Reply to this email directly, view it on GitHub
https://github.com/getstation/desktop-app/issues/351#issuecomment-1991207589,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AXPQQVRYZAGE7OPWBLWGQ6TYX3FRLAVCNFSM6AAAAABEISH73OVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSOJRGIYDONJYHE
.
You are receiving this because you authored the thread.Message ID:
@.***>

--
Fraser Walker https://www.monikeraudio.com/
Audio Engineer
m +61 403 727 194

#178


Mobile Responsive Design

This doesn't have a description, but I will second its existence, since the site actually does not work on mobile at all for some pages (major cutoffs). Would be huge to see mobile support :3

Thanks for the feedback. I have fixed it and confirmed that it works fine, at least on my Android.


Method parameter names scoping change for nameof

Proposal is here: nameof(parameter)
A motivation for this is to enable nameof to name a parameter in an attribute constructor.
Effectively, the nameof(p) could find p as a parameter name to a function defined in the current scope.
Tasks:

[ ] Update the nameof language reference page to show the new example.

@jcouv  This doesn't seem to be in .NET 6 preview 5.
The roslyn feature status page shows it as in the main branch, but without a release target. Can you clarify? Is this available yet?

The feature status page says the feature will go straight into main (no feature branch needed since small feature) but is still "in progress".
I still want to get it into C# 10 (will aim for dev17.0p3).

I dropped the .NET 6 release label, and removed this issue from the related projects. I'll leave it open to address when the feature is available.


Handle 404 redirects outside of chapters

When someone lands on a page that can't be found, they are shown this image. We should have a friendly redirect or display a landing page that points them back to savingsatoshi.com.

It looks like we have something in place if a user lands in a url in the chapter section:

First we need to discuss how we want to handle 404 redirects. Then we can send the issue to design if needed.
This is not necessary for the demo.

We can have a catch-all 404 page instead of handling it on a per page basis, maybe @GBKS can make a fun design.
Something like this:

But related to Bitcoin

Holocat yelling at them would be pretty funny.

Here's an old error page from the times of the Legends of Lightning pitch.

Hey is this issue open can I work on it?


Typo in Javascript Modulo Exercise: "var remainder" should just be "remainder"

Finding a Remainder in JavaScript
https://www.freecodecamp.com/challenges/finding-a-remainder-in-javascript
Issue Description
As the directions clearly state, the correct answer to the exercise will be: [the chosen variable] set equal to 11 % 3.
The problem is that the exercise only accepts var remainder = 11 % 3 as the correct answer, when the variable remainder is already declared in the line above and does not need to be declared again.
Browser Information
Error encountered in Chrome 51.0.2704.79 on Mac 10.11.5.
Code Entered When Issue Encountered
var remainder;
remainder = 11 % 3;

Screenshot

Closing because of duplicate of #8976.


encode_path on url

Hi, I had to monkey path to add encode_path to the url method, like this:
module AzureEncodingPatch
  include CarrierWave::Utilities::Uri

  def url(options = {})
    path = ::File.join @uploader.azure_container, encode_path(@path)
    if @uploader.asset_host
<EMAIL_ADDRESS>    else
      uri = @connection.generate_uri(path)
      if sign_url?(options)
        @signer.signed_uri(uri, false, { permissions: 'r',
                                         resource: 'b',
                                         start: 1.minute.ago.utc.iso8601,
                                         expiry: expires_at}).to_s
      else
        uri.to_s
      end
    end
  end
end

Otherwise, images with spaces in it were failing. Should I create a PR?

@krismichalski URI.encode could also be used I suppose!

Hi @allaire, great catch!
Yes, if you could create PR with encode_path patch it would be great.
It would be even more awesome if you could add a test case for files with spaces in them.
Unfortunately, I no longer have access to Azure, so I cannot develop it by myself.

@krismichalski Good, will try to get this done soonish!


Fixing the bug when a segment has no vector field present for disk based vector search

Description
Fixing the bug when a segment has no vector field present for disk based vector search
The check will ensure that if there are segments with no vector field the disk based vector search is not crashing.
Related Issues
Ref: https://github.com/opensearch-project/k-NN/pull/2278
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.

Since this problem happened due to fieldinfo being null will raise a separate PR for fixing that so that in future we don't face this issue.


Fix error condition

Thank you for the examples, made this small change to get them to build.

Thanks!


How do I create a subfolder? Or is not possible?

How do I create a subfolder? Or is not possible?

That's currently not possible, but planned: https://github.com/electerious/Lychee/issues/254


Quick Open View doesn't work consistently

On Mac from Command Palette:

Open Command Palette, type "Quick Open View"
Select by clicking: the quick open view appears
Select using enter: the quick open view flashes and then disappears :x:

On Mac from Keybinding:

Open "Quick Open View" using Ctrl+Q
View disappears when Ctrl key is released :x:
Navigation works when holding down Ctrl and pressing Q
Navigation does not work when holding down Ctrl and pressing arrow keys :x:

On Windows:

Open Command Palette, type "Quick Open View"
Select by clicking: the quick open view appears
Select using enter: the quick open view flashes and then disappears :x:
Navigation works by pressing arrow keys without holding down Ctrl

@aefernandes there are some commands which only really work when invoked via keybinding ("quick navigate" commands). It is true that you cannot really execute them via mouse, so they should probably either be supported or filtered out (though it is still good to show them for discoverability reasons).

Actually I think a problem is that some of these commands don't work via keybinding, but only mouse.
I expect Quick Open View to show me the menu below whether I run the command via command palette + enter, command palette + click, or the quick open keybinding. Right now it only works as expected through command palette + click. (The brief flash and disappear that occurs for the other options is visually jarring.)

Closing "as designed".


Android O Emojis?

I see the SVG folder of this repository is active and updated on the recent days. I wonder if these http://emojipedia.org/google/android-o-beta/ will be uploaded to this repository also or they will not be part of noto project anymore? Thanks

The current plan is to provide them.  No date for that yet.

Better save a copy of the nice blobs :D
2017-05-30 20:28 GMT+02:00 dougfelt<EMAIL_ADDRESS>
The current plan is to provide them. No date for that yet.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/googlei18n/noto-emoji/issues/141#issuecomment-304966805,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABI4QNICAx6TxFKHKLVqdcWhdroZrCAhks5r_F_jgaJpZM4NopeH
.

:-) I plan to tag the repo before we update to the new design.

@dougfelt Could this be reopened so that we have a way to subscribe and get notified when that happens? (and only then close the issue as "resolved")

Sure.

Will they be released this year? Maybe alongside Android O stable?

That's the plan.

Is this repo going to keep the current "blob" design or be replaced with the new design now that Android Oreo is out?

The repo will have the new design.

New design has been posted.


Opensource : 3.2 customising GitHub profile

Hello Supervisor/ Mentor, Can I start working on this issue I just created? 😅

@LegendSumeet Sure , Happy documenting

can I also do this or only one can only do it??

can I also do this or only one can only do it??

Create a new issue which you want to work under open_source_domain @lohith1392

Ok thank you

@Aryamanz29 Is there any list of issues to select or our own???

Its must be inside opensource domain


GREENPLUM OR POSTGRE SQL‘s  sql edit & debug

use gp or pg sql edit ,while exists some errores only Position ,like this  SQL 错误 [42601]: ERROR: syntax error at or near "SELECT"
Position: 51

hi @lctlover ,
cannot catch your issue. Could you clarify it a bit more?
Thanks in advance

there are not any updates for a long time
The issue is closed.
If it is still actual for you - feel free to reopen it


Making a bench using IPA semi honest protocol

Results

Batchsize
Time Taken

1000
18.18sec

10000
190secs

100K
2469sec

It does not

Validate results

One last thing - it's mad about an unused import: use rand::thread_rng; can you fix that?

I had fixed that in local but didnt push it. Should be good to go now


Remove unused import in extensions/git/src/api/git.d.ts

Fix #200800
By the way, please for the love of every gods this planet, make #198592 happen, it's impossible to use the API without documentation NOR source files the API uses

@Malix-off, thank you very much for your contribution. I will go ahead and close this as there is very little value add here.
If you'd like to help out, check out our Contribution Guide:

Issues labeled help wanted are good issues to submit a PR for. Issues labeled good first issue are great candidates to pick up if you are in the code for the first time.

Waht.
Okay it's little value added but it still value added, I do not care having my name tied to the commit, you can take credit idc I just want the clutter removed


New test configs: where, limit, warn_if, error_if

resolves #3258
Description
tests:
  +warn_if: ">10"
  +error_if: ">100"
  +where: "date_col = current_date"
  +limit: 10

Adds four new test configs:

limit: Simple enough, templated out in the materialization. This is mostly a complement for #3316
where: The way I did this was unbelievably hacky—see the test cases for yourself—but it works, and in a way that should be backwards compatible with all existing schema/generic tests, without them needing to change any part of their SQL definition. (This config doesn't make sense for one-off tests.)
warn_if, error_if: The user supplies a python-evaluable string (e.g. >=3, <5, ==0, !=0) and dbt will compare the fail count/calc against it.

By default, set to >0. (I realize now this should probably be !=0)
Interaction with severity: A little tricky, but I actually think they work reasonably together. By default, severity: error, and dbt checks the error_if condition first; if not error, then check the warn_if condition; if the result meets neither, it passes the test. If the user sets severity: warn, dbt will skip over the error_if condition entirely and jump straight to warn_if.

TODO

Add integration tests
Cleanup: Could we wrap the where logic in a special internal macro, so long as it's included in the schema test parsing context?
Cleanup: Any way to DRY up the manual-set configs within unique + not_null shortcut?

For the future
I got stuck trying to implement relative/percentage warn_if and error_if. That's ok! That piece, while compelling, can very easily come in a later issue.

If the user supplies error_if: >5%, it's easy enough to error_if.replace('%', '/100') and compare against >5/100 instead.
The issue was, I couldn't figure out how to grab the denominator, i.e. the count of the "full model" (ideally with the where condition included). At materialization time, we can access test_metadata.kwargs.model, but this is an unrendered Jinja string, and even with the as_native filter, I couldn't figure out how to "extra-render" it.

Checklist

[x] I have signed the CLA
[x] I have run this code in development and it appears to resolve the stated issue
[ ] This PR includes tests, or tests are not required/relevant for this PR
[ ] I have updated the CHANGELOG.md and added information about my change to the "dbt next" section.

Also need to update:
https://github.com/fishtown-analytics/dbt/blob/85e2c897945e6372fe3657075b2c5ea4b068d3e0/core/dbt/contracts/graph/compiled.py#L122-L126


-fmodules, -fmodule-name, -fmodules-cache-path missing on Windows

It appears that we drop the modules related feature flags when building for Windows.  This can yield issues with builds and unnecessary build differences.  Investigate why the flags are dropped and add them if appropriate

We don't build with modules for Android either. I'm about to change it so that only Apple platforms build with modules, because once clang adds its own builtin modules, the rest of the platforms break due to their lack of module support. (i.e. Linux only works right now by luck)
I'm not sure how important it is to build non-Apple platform SPM C-based targets using modules. If package authors really want, they can add the flags themselves in the unsafe flags. If we want to make it work in SPM internally, then we need to do what the Swift compiler does, which is to inject module maps into the OS/SDK via a vfs.


Rename tensor tests for clarity

Rename Tensor_host tests to Tensor_image_host, and rename Tensor_hip tests to Tensor_image_hip to clarify the type of data for that subcategory of tests.
This matches other test suite naming convention like in 'Tensor_voxel_host' or 'Tensor_audio_host' or 'Tensor_misc_host'.

@r-abishek - conflicts needs resolved


diff_drive_controller generates wrong odom and robot base frame names

Since the last sync (https://discourse.ros.org/t/new-packages-for-ros-2-humble-hawksbill-2022-12-16/28806) there are two issues:

Default frame name of the robot base frame is odom. This used to be base_link and it was a meaningful name. I don't think we should change defaults within the single ROS distro. I guess the problem is here: https://github.com/ros-controls/ros2_controllers/blob/42f6c1470508ca2480b954eead58a87e7e827434/diff_drive_controller/src/diff_drive_controller_parameter.yaml#L49
Namespaces are passed as prefixes to the odom and robot base frame names. I think I read somewhere that adding prefixes to frame names in multi-robot systems is a bad practice. Anyways, like in the first issue, the behavior shouldn't change within a single ROS distro.
Even adding a leading slash to the frame names didn't make prefixes go away. My temporary solution is to use the snapshot version:wget -O /tmp/diff_drive_controller.deb http://snapshots.ros.org/humble/2022-11-23/ubuntu/pool/main/r/ros-humble-diff-drive-controller/ros-humble-diff-drive-controller_2.12.0-1jammy.20221108.202153_amd64.deb && \
sudo apt install -y /tmp/diff_drive_controller.deb && \
rm -f /tmp/diff_drive_controller.deb

It looks like the changes are made on purpose in https://github.com/ros-controls/ros2_controllers/pull/461. I am not sure whether it is a good decision, especially considering that the change breaks the previous behavior.

@lukicdarkoo you are right. I don't know how I missed this "/" issue with frames. I think we should use "_" as separator between prefix and frame name and use it only if there is controller namespace provided.
@delihus maybe you would like to open another PR 😉

@lukicdarkoo sorry, this change slipped through into humble unfortunately…

@destogl I don't think the namespace should affect frame names. So far I haven't seen any package doing it in ROS 2.
Instead, there should be a parameter, like frame_prefix. Or, simply not allowing prefix is also fine. A user can add prefix when defining frame names anyways.
For example in Nav2, a namespace adds a prefix to every topic, including tf and tf_static. It makes transforms isolated and no need to add prefixes to frame names.

Hi, I also agree that #461 should be reverted.
I think that having a prefix option could be a way of doing it for those who only have 1 /tf topic and want to add differentiation.
In my situation I use remapping to make a new /tf topic for each namespace as described in this issue on Geometry2. This is also how they do it on the Nav2 project, so having nothing would be best for me!

hi, anyone following this problem?
I guess I'm struggling with same issue .
I applied a namsapce on the node. But /tf publishes frame id with namespace as a prefix.
All I need is just remove the namespace on frame id for TF.

Didn't someone mention a config parameter would be added for the frame prefix?  My current bot setup is to namespace my nodes, topics, and the tf topics.  I do not want to prefix my frame names.  The current code is automatically prefixing my frames when the node has a namespace.
It seems like we just need a config param for the frame  prefix, and default  it to nothing.  I though I read this was going to happen, but I can't seem to find it again.
My current fix was to clone the package and remove the prefixing code, but that's not maintainable.  I'd be glad to help if nobody else is working to fix this.


Fix lint errors

Codecov Report

:exclamation: No coverage uploaded for pull request base (dev@dedf2e1). Click here to learn what that means.
The diff coverage is 0%.

@@          Coverage Diff          @@
##             dev     #50   +/-   ##
=====================================
  Coverage       ?   0.03%
=====================================
  Files          ?     287
  Lines          ?   10697
  Branches       ?    3273
=====================================
  Hits           ?       4
  Misses         ?   10693
  Partials       ?       0

Impacted Files
Coverage Δ

lib/common/components/SelectFileModal.js
0% <ø> (ø)

lib/editor/components/map/EditorMap.js
0% <ø> (ø)

lib/editor/actions/active.js
0% <ø> (ø)

...er/components/reporter/components/PatternLayout.js
0% <ø> (ø)

lib/common/user/Auth0Manager.js
0% <ø> (ø)

lib/alerts/components/AlertPreview.js
0% <ø> (ø)

...r/components/reporter/components/DateTimeFilter.js
0% <ø> (ø)

lib/common/components/StatusMessage.js
0% <ø> (ø)

lib/manager/components/FeedSourceDropdown.js
0% <ø> (ø)

lib/common/components/StatusModal.js
0% <ø> (ø)

... and 61 more

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 dedf2e1...b6c83ec. Read the comment docs.


bug: Multiple cursors keybinding ma move the cursor to the wrong position

Did you check docs and existing issues?

[X] I have read all the vscode-neovim docs
[X] I have searched the existing issues of vscode-neovim
[X] I can reproduce the issue with an empty init.vim/init.lua and no other vscode extensions (when applicable)

Neovim version (nvim -v)
v0.10.0-dev-9af03bc
Operating system/version
Microsoft Windows 11 Home 10.0.22621 N/A Build 22621
Describe the bug
https://github.com/vscode-neovim/vscode-neovim/assets/54128430/4fd2439b-6bcc-4134-9efd-eda645edae09
Steps To Reproduce
this is ascii, but this is not 🦄😊😂🤣    |
this is ascii, but this is not 🦄😊😂🤣    |
this is ascii, but this is not 🦄😊😂🤣    |

To reproduce: Vjjma
Expected Behavior
After I press ma, the cursors should move to the end of line

Thanks for the fast fix.👍👍👍BTW, when can I update it from vscode market?

Soon.
However, there are still issues in visual block mode, and currently, there are no plans to fix them :(


Fetch the json representation of the dashboard

On our public doc, for the fetch we only provide the possibility to do this using Azure Portal.
Is there another way yp fetch this information? It is not specific on our public doc.
https://docs.microsoft.com/en-us/azure/azure-portal/azure-portal-dashboards-create-programmatically#fetch-the-json-representation-of-the-dashboard

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

ID: 47fb1c00-f103-92c2-32e9-eefae46a6ce9
Version Independent ID: 33555263-7be5-7164-310d-e97e6bea6704
Content: Programmatically create Azure Dashboards - Azure portal
Content Source: articles/azure-portal/azure-portal-dashboards-create-programmatically.md
Service: azure-portal
GitHub Login: @JnHs
Microsoft Alias: jenhayes

@MonikaReddy-MSFT  is there any updates here?
Thank you :)

Hi @pumpkinclarinnet-MSFT, yes you should be able to capture the JSON programmatically as well. I'm working to confirm the details and will update the doc to include that.

@JnHs were you able to get any updates?

@pumpkinclarinnet-MSFT I apologize for the delay. I've updated the topic to mention that you can use REST APIs or other methods to retrieve details about the dashboard resource. This works the same way as for other resources in Azure.
I'll close this issue now, but please let us know if you have further feedback. Thanks again for taking the time to share your feedback and help us improve our content.
#please-close

@JnHs  what do you mean by other methods?


Feature Request: message_id

I think it is a good idea to add message_id to the array that is returned from the getMessage() function and more specifically to formatMessage().
This field identifies message between sessions and is important. #

Sounds great. I'll be sure to add this

Check your note with the new source code.

This feature was added by @sergey144010. Closing this issue!


usage: Unclear how to unset lspconfig keybinding

First, thanks for developing and publishing this, it's really great as a starting point for neovim.
I was checking the plugins/example.lua trying to check how to unset premade keybindings by LazyVim. In particular, I want to use the K keymap in normal mode for moving this around with mini.move. I'm doing this by adding the following to my init.lua
require("mini.move").setup({
  mappings = {
    left = "H",
    right = "L",
    down = "J",
    up = "K",
    line_left = "H",
    line_right = "L",
    line_down = "J",
    line_up = "K",
  },
})

However, the on_attach call by lspconfig that LazyVim preconfigures in keymaps.lua takes precendence so K doesn't call the mini.move function.
As a sanity check
:nmap K
n  K           *@<Lua 807: /usr/local/Cellar/neovim/0.8.1/share/nvim/runtime/lua/vim/lsp/buf.lua:41>
                 Hover
n  K           * <Cmd>lua MiniMove.move_line('up')<CR>
                 Move line up

What is the intended way of disabling particular keybindings done vim LazyVim in this way?

Don't know if this is the most efficient way but I copied the entire plugins/lsp/init.lua file in my plugins/ folder (I did plugins/lsp.lua where I put some other lsp related plugin configs) because there were some other things I wanted to change. Relevant changes below:
  {
    "neovim/nvim-lspconfig",
    -- other relevant lazy.nvim settings here, including opts table
    config = function(plugin, opts)
      -- other config stuff here

      -- setup formatting and keymaps
      require("lazyvim.util").on_attach(function(client, buffer)
        require("lazyvim.plugins.lsp.format").on_attach(client, buffer)
        -- require("lazyvim.plugins.lsp.keymaps").on_attach(client, buffer)
        require("config.lsp_keymaps").on_attach(client, buffer)
      end)

    -- Other config stuff here
    })
    end
  }

This just overrides the on_attach call for keymaps I think you're referring to, while keeping the format opts. The I add a call to config/lsp_keymaps.lua (where the optional keymaps.lua/options.lua etc files are in the runtime path) that just has a copy of plugins/lsp/keymaps.lua, and from there I can comment out/edit whatever.
Again, not sure that's the most efficient way, but I like it since I can update to any other changes I want to follow with little hassle, since lazy.nvim makes updates so easy to follow.

I'll see for an easier way to override those mapping tomorrow!

I just pushed some change to make it easier to delete/add/change lsp keymaps:
  {
    "neovim/nvim-lspconfig",
    init = function()
      local keys = require("lazyvim.plugins.lsp.keymaps").get()
      -- change a keymap
      keys[#keys + 1] = { "K", "<cmd>echo 'hello'<cr>" }
      -- disable a keymap
      keys[#keys + 1] = { "K", false }
      -- add a keymap
      keys[#keys + 1] = { "H", "<cmd>echo 'hello'<cr>" }
    end,
}

The keymap definition is the same as for the lazy.nvim keys property


Merge deployment doc changes to master

Merging https://github.com/wso2/docs-apim/pull/3953 to master.

 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.


Click and drag scrollbar and release mouse outside scroll area will trigger a click on document.

Click and drag scrollbar and release mouse outside scroll area will trigger a click on document.
While browser's native scrollbar would not trigger a click. Is that possible to fix this?
JSFiddle: https://jsfiddle.net/gbfLazpx/975/

This behavior also happens when you release mouse over the rail. It will trigger a click on the rail which in IE and Chrome makes the scrollbar jump to the bottom.

We can prevent an event from being propagated or doing default behaviour, but cannot prevent other events in the same level. That's why we cannot do anything on document click. Basically there is no easy (or even hard) way to prevent document click from being fired.
However @andrewtibbetts's example seems like fixable. I will upload a fix for it.


Running MTGJSONv4 on Windows

Using Python 3.7, I did the following:

Extracted the contents of mtgjson-master.zip to directory "C:\Users\a046369\Desktop\mtgjson4-master"
Executed "pip3 install -r requirements.txt".  Several messages about files being downloaded and installed
From the directory "C:\Users\a046369\Desktop\mtgjson4-master" I typed:
"C:\Users\a046369\AppData\Local\Programs\Python\Python37-32\python.exe" -m mtgjson
I got the error in the attachment

I didn't see a python3.exe file as the original documentation stated.

Python3 -m mtgjson4

I tried that first.  This is the message I got.  Where is that executable?  pythonw -m mtgjson4 didn't work either.

You need the path for python3... check out google on how to exec a python script

I found the fix for this: remove : from the logfile name generation. Windows doesn't like colons in filenames, apparently.


multi-dimensional average fails to apply partial weights to outermost cells

I found this while comparing heat content (vertical-mean temperature) between two different models; I was seeing an unexpectedly large difference that was inconsistent with other calculations.  This is urgent since it silently gives wrong results; needs to be fixed right away and distributed to the community, with warnings about the incorrect answers in v7.11-7.4.  It'd be good to add regression tests to catch this as well.
This bug first appears in v7.11, dated 4/19/2017.  (It was not in v7.1, dated 2/16/2017.)  May be related to the new memory management, and/or the fix for #1590.
First create a little 2-d field:
 	NOAA/PMEL TMAP
 	FERRET v7.4 (optimized)
 	Linux 2.6.32-696.23.1.el6.x86_64 64-bit - 04/18/18
 	 5-Jul-18 14:06

yes? can mode verify
yes? def ax/edge/x=0:2:1 xax; def ax/edge/y=0:2:1 yax; let a = x[gx=xax] + y[gy=yax]
yes? list/noh a
 0.5 / 1:  1.000  2.000
 1.5 / 2:  2.000  3.000

Then do a 2-d average, three different ways.  For this variable on a rectangular grid with no missing values, averaging is a linear operation, so order shouldn't matter and all three should give the same answer to within machine precision.
yes? let b = a[x=@ave]; let c = a[y=@ave]
yes? list/noh a[x=@ave,y=@ave], b[y=@ave], c[x=@ave]
I / *:     2.000  2.000  2.000

Good.  But now constrain the region to omit half the rightmost cell:
yes? list/noh/x=0:1.5 a[x=@ave,y=@ave], b[y=@ave], c[x=@ave]
I / *:     2.000  1.833  1.833

The right answer is (1 + .52 + 2 + .53)/(1+.5+1+.5) = (1+1+2+1.5)/3 = 5.5/3 = 1.833.  So the first result, from the simultaneous 2d average, is wrong -- it's not properly applying the partial-edge weights for the rightmost cell.

This was indeed in the memory management and improvements allowing larger regions for transformations by doing the calculation in chunks.  As part of that, the grid-box-sizes are collected, then the code says "trim if partial box at low end ... high end". But then it goes and gets the wrong information for the grid box edges to compare with the requested region ends.
So this is an easy fix.  Surprisingly we didn't have a test for multi-dimensioned transformations with endpoints other than at grid box edges. I've added these sorts of tests.
The same code starts off the computation for multi-dimensional averages, definite integrals, and standard deviations, so the bug and the fix apply to all of these.
We'll get you a test executable in the next couple of days and get this out to the public with an announcement about the bug.


Something went wrong.

Microsoft PowerToys version
0.80.1
Installation method
GitHub
Running as admin
None
Area(s) with issue?
General
Steps to reproduce
Was watching a YouTube video in Chrome Browser and it was as if the video driver crashed as everything froze for about 30 seconds.  When everything recovered a window popped up saying "PowerToys ran into an issue. Something went wrong. Please file a bug..."
I don't know if this was anything to do with PowerToys or if it was just letting me know something went wrong somewhere else but it said to file a bug.  Hoping something in here would be helpful to you.
2024-06-16.txt
✔ Expected Behavior
No response
❌ Actual Behavior
No response
Other Software
No response

It seems to be a display driver issue.  The display driver crashed for whatever reason and needed to restart.  PowerToys must have detected the fatal exception and notified me once the display driver recovered.


Master branch and v2.0.0 tag produce different totp tokens

I have been banging my head on this for a while and need someone to either tell me what silly thing I'm doing wrong, or help me figure out why the master branch is broken:
package.json
{
  "dependencies": {
    "speakeasy_v200": "https://github.com/speakeasyjs/speakeasy#4e0aa91dee166306e07ea11e371d5a5df95f4391",
    "speakeasy_master": "https://github.com/speakeasyjs/speakeasy#cb58351b7eb829719dbee2df2ede53d97721e"
  }
}

main.js
let speakeasy_v200 = require('speakeasy_v200');
let speakeasy_master = require('speakeasy_master');

let secret = 'xhwb2ilzt5gqyxrz';

// The v2.0.0 branch produces a code which
// DOES match that produced by
// Google Authenticator, FreeOTP, etc
let token_v200 = speakeasy_v200.totp({
  secret: secret,
  encoding: 'base32'
});
console.log('v2.0.0 = %s', token_v200);

// The master branch produces a code which
// DOES NOT match anything...
let token_master = speakeasy_master.totp({
  secret: secret,
  encoding: 'base32'
});
console.log('master = %s', token_master);

if(token_v200 !== token_master) {
  console.log('WAT?!?!?!?!');
}

Now, install and run:
$> yarn install
...
$> node main.js
v2.0.0 = 922665
master = 616379
WAT?!?!?!?!

Any thoughts, comments, hints, suggestions, second set of eyes, etc, etc are seriously appreciated. Thanks!

Any thoughts/comments from the maintainers?

There are a LOT of changes between master and 2.0.0 and there hasn't been a release in a while, the API probably changed.

The base32 secret shown in your example is not a multiple of 8 in length - therefore this bug may be cased by #135


PLEASE HELP!!! Seeing some twisted-python error in my carbon-cache console.log file

Hello community.
I'm seeing below errors in my console.log file. Its concerning that when this error occurs my carbon-cache will stop writing to the disk which causes my memory to be filled up and system invokes OOM. Please guide me how to fix this issue.
==> /var/log/carbon/console.log <==
21/11/2024 14:50:33 :: Unhandled Error
Traceback (most recent call last):
File "/usr/lib/python3/dist-packages/twisted/python/threadpool.py", line 250, in inContext
result = inContext.theWork()
File "/usr/lib/python3/dist-packages/twisted/python/threadpool.py", line 266, in
inContext.theWork = lambda: context.call(ctx, func, *args, **kw)
File "/usr/lib/python3/dist-packages/twisted/python/context.py", line 122, in callWithContext
return self.currentContext().callWithContext(ctx, func, *args, **kw)
File "/usr/lib/python3/dist-packages/twisted/python/context.py", line 85, in callWithContext
return func(*args,**kw)
---  ---
File "/usr/lib/python3/dist-packages/carbon/writer.py", line 189, in writeForever
writeCachedDataPoints()
File "/usr/lib/python3/dist-packages/carbon/writer.py", line 98, in writeCachedDataPoints
(metric, datapoints) = cache.drain_metric()
File "/usr/lib/python3/dist-packages/carbon/cache.py", line 187, in drain_metric
metric = self.strategy.choose_item()
File "/usr/lib/python3/dist-packages/carbon/cache.py", line 84, in choose_item
metric_name, _ = max(self.cache.items(), key=lambda x: len(itemgetter(1)(x)))
builtins.RuntimeError: dictionary changed size during iteration

What's your version? I believe fix was released with version 1.1.5 - https://github.com/graphite-project/carbon/issues/822

@deniszh
I removed the apt package and now only have package from pip. But now I'm facing the below issue. Can you please help me. I'm new to these stuffs.
/usr/local/bin/carbon-cache.py
/usr/lib/python3/dist-packages/requests/init.py:89: RequestsDependencyWarning: urllib3 (2.2.3) or chardet (3.0.4) doesn't match a supported version!
warnings.warn("urllib3 ({}) or chardet ({}) doesn't match a supported "
Traceback (most recent call last):
File "/usr/local/bin/carbon-cache.py", line 28, in
from carbon.util import run_twistd_plugin  # noqa
ModuleNotFoundError: No module named 'carbon'


Results reproducibility

Hi,
I'm trying to reproduce the results presented in the paper using the WN18RR dataset. However, I'm running into some kind of trouble since I'm not able to replicate the results. Using the default parameters, i.e., running only python main.py the obtained results are, for example:
final results
acc: 1.0000   mrr: 0.4864   mr: 3.9359   h1: 0.3086   h3: 0.5443   h5: 0.7096

What I'm doing wrong?
Also, the other models' differ greatly from everything reported so far, as an example, the RotatE results for the WN18RR dataset reported in the original paper are mmr: 0.440  h1: 36.1  h3: 48.3  h10: 58.1, while in the PathCon paper are mmr: 0.799  h1: 0.735  h3: 0.823. How do they were evaluated?
Thanks in advance.

Hi! For your first question, the result is quite weird since we never get a result of accuracy 1.0. I just ran the code again and below is the result:

Could you please post the logs  printed during training just as above? For your second question, the result reported in our paper is for relation prediction (given h and t, predict r) while in previous papers they do tail or head prediction (given h and r, predict t, or given t and r, predict h).

Thanks for the answer. I was able to reproduce after re-cloning the repo. I think I ran into some problems with the data and the generation process.


fix(src): Multiple updates

Closes #93
Closes #1
Closes #133
Closes #142 (Yeah, this extension had a bug but someone closed it before fixing it.)
Checklist:

[x] Updated extVersionCode value in build.gradle for individual extensions
[x] Updated overrideVersionCode or baseVersionCode as needed for all multisrc extensions
[x] Referenced all related issues in the PR body (e.g. "Closes #xyz")
[x] Added the isNsfw = true flag in build.gradle when appropriate
[x] Have not changed source names
[x] Have explicitly kept the id if a source's name or language were changed
[x] Have tested the modifications by compiling and running the extension through Android Studio
[x] Have removed web_hi_res_512.png when adding a new extension
[x] Have made sure all the icons are in png format

I had tested anime ID with different animes and they worked perfectly, that's why I had sent the ticket to close

I had tested anime ID with different animes and they worked perfectly, that's why I had sent the ticket to close


Filter expiry candidates in database

Small improvement to filter out candidates by max age in the database, instead of in Ruby.
I'm not entirely sure why this was done this way before, but it seems to pass the tests.

Added a comment to explain this here - https://github.com/rails/solid_cache/commit/ec6b58c2fbb09cff41a3b9151090cc9db64bfee2


How to set the value of “subscriberPrimary“ to false

When I joined the room successfully, the value of “subscriberPrimary“ in the returned JoinResponse is true, and then the datachannel will be opened. I want to adjust this value to false. Where should I set it

this is an internal state. could you say more about what you are trying to accomplish by not using subscriber as the main connection?


zplug update --self does not find zplug git repository

Problem Summary
When trying to run  % zplug update --self as suggested in the docs, I get this error:
fatal: Not a git repository: '/home/gregor/.zplug/repos/zplug/zplug/.git'
When manually checking out the repository into the expected location, it works, but I guess zplug should check out the repo transparently in the background?
System Information
4.8.0-28-generic #30-Ubuntu SMP
zsh 5.2 (x86_64-ubuntu-linux-gnu)
git version 2.9.3
zplug version 2.3.3
Minimal zshrc
using standard unchanged init.zsh

It is a known problem. Please wait until you fix it

Oops. My bad

Close this issue. See #311 for the reason.


Filter by attribute of related entity

Maybe this is possible already but it would be great to be able to filter by an attribute of a related entity
Maybe, have something like
bookings?include=status&filter[starting-at]=le:2017-01-01&filter[status.name]=eq:active

@nialljt1 this is not currently supported. i'll take some time this week to think about the best way to implement it and try to get it in this weekend.

Hi Jared,
Thanks for the response and thanks for all the good work.
Niall
On 5 April 2017 at 14:37, Jared Nance<EMAIL_ADDRESS>wrote:

@nialljt1 https://github.com/nialljt1 this is not currently supported.
i'll take some time this week to think about the best way to implement it
and try to get it in this weekend.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/Research-Institute/json-api-dotnet-core/issues/85#issuecomment-291863579,
or mute the thread
https://github.com/notifications/unsubscribe-auth/APQT4rfDrJodfP__BYtsslyz9X0oEmr3ks5rs5kigaJpZM4M0PXv
.

@nialljt1 this has been merged into the pre-release package 1.3.2-alpha1-0101. Can you pull this version and verify it fixes your issue?

Hi Jared,
I pulled this version and confirmed that this allows me to filter by a
related entity and thus fixed my issue.
Thank you very much.
On 11 April 2017 at 14:18, Jared Nance<EMAIL_ADDRESS>wrote:

@nialljt1 https://github.com/nialljt1 this has been merged into the
pre-release package 1.3.2-alpha1-0101
https://www.myget.org/feed/research-institute/package/nuget/JsonApiDotNetCore.
Can you pull this version and verify it fixes your issue?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/Research-Institute/json-api-dotnet-core/issues/85#issuecomment-293258689,
or mute the thread
https://github.com/notifications/unsubscribe-auth/APQT4gA35VfkUw08hyRy3llKpwydQR61ks5ru32IgaJpZM4M0PXv
.


RB performance benchmarks

What is the expected behavior?
Let's add randomized benchmarking from Ignis as one of the benchmarks under test/benchmarks. It is helpful for tracking performance.

How do you feel about adding some applications benchmarks (e.g. VQE, operator estimation, hamiltonian simulation)? I can start a separate bug to discuss if you think that could be valuable.

@dongreenberg I definitely think that having some application benchmarks would be useful. In the first WIP version of the benchmark suite I had a vqe benchmark (based on terra's optimizers module which were removed in 0.7) but removed it by the time things finalized because the terra optimization module it depended on was removed.

@ajavadia - I started a PR with RB performance benchmark.

How can I run the performance benchmark?
Is there some link to the performance results and graphs?
Which RB configurations would you suggest to test?
I started with 2-qubits RB, with 50 different Clifford lengths (going from 1 Clifford to 200 Cliffords, spaced linearly) and 20 seeds.

@ShellyGarion You run the benchmarks using the asv. I normally run things locally for testing with just asv dev but you can also use asv run especially if you want to test across the history. You can see current benchmark results here: http://<IP_ADDRESS>/

@ShellyGarion I'm not sure varying the seed affects things much from a performance point of view (you can try). The Clifford lengths are good. Maybe you can add 1Q RB as well.
@dongreenberg yes please add a couple applications too. Maybe one with a huge circuit, and one with many small circuits.

@mtreinish - the link for current benchmark results  http://<IP_ADDRESS>/ does not work for me anymore

Yes, I implemented a more permanent hosting solution for the benchmark results since I left that comment last month. You can view the benchmark results here: https://qiskit.github.io/qiskit/

@ajavadia - I added several more parameters:

1Q, 2Q, and simultaneous 1Q&2Q (where the 1Q is3 times more than 1Q).
1, 5, 10 seeds (need to check if no. of seeds affects the performance).
Clifford lengths: 1 to 200 in steps of 4, 1 to 500 in steps of 10.

The RB performance benchmark is now reduced to the following parameters:

1Q, 2Q, and 3Q simultaneously 1Q&2Q, namely: rb_pattern=[[[0]], [[0, 1]], [[0, 2], [1]]]

with the following default parameters:

1 seed
Clifford lengths: 1 to 200 in steps of 4, namely: length_vector = np.arange(1, 200, 4)

I think that this old issue could be closed @ajavadia @mtreinish
Here is a link to RB performance test:
https://github.com/Qiskit/qiskit/blob/master/test/benchmarks/randomized_benchmarking.py

Yep, this has been done since #306 and can be closed.


ignore the sample directory in bower

prevent bower from downloading the sample code on bower install.

ah good idea -- thanks!


Cycle_in_directed_graph_dfs.cpp

You can follow me on github

Sure, thanks man
On Mon, 4 Oct 2021, 22:58 Sonu, @.***> wrote:

Merged #105 https://github.com/sonumahajan/All_Program_helper/pull/105
into main.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/sonumahajan/All_Program_helper/pull/105#event-5405569535,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AV4ISCU7OXK2H64FEEUEGHLUFHP4RANCNFSM5FJVZA5A
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.


gRPC: add more unit tests for Stream and Datastore

Can you do a sync? Some changes I've saw in your other merged PR and do not belong to this PR.

@zxu123 Hmm, looks correct to me. Can you point me to the duplicate changes? (perhaps you mean very similar tests between grpc_stream_test.h and stream_test.h?)


Added Bayesian quadratic discriminant model

Hello Sam and thanks for this again!
One thing though is that we'd rather have fewer models in MLJModels rather than more if that makes sense bc everything that's here needs to be maintained here.
Exposing things in MultivariateStats makes sense but here  you're extending MultivariateStats within MLJModels which is not very maintainable in the long term.
Given your recent input, would you be interested in doing a DiscriminantAnalysis.jl package, maybe a fork of the existing one  or just your own one with a different name say BayesianDiscriminantAnalysis or whatever and just have an interface to MLJ?
That way you can own that package, it's easier to maintain in the long run, and it also makes DA models more discoverable; further it also allows to ditch the naming "BayesianDA*" for just "*DA" which would not be ambiguous anymore
What do you think? and I'm happy  to  help  you towards this.
Basically what  I'm suggesting would be:

have a dedicated and efficient package just for DA models
profit from that  opportunity to make everything work with n x p convention
make it possible to add QDA models  and other extensions that could exist
remove the ambiguity "Bayesian*DA" and just "*DA"

Okay.

Thanks! I'll close this PR then, please ping me from your package if I can help


Adds support for aws api orchestration

Codecov Report

Merging #347 (efd6de7) into main (2b36067) will decrease coverage by 0.05%.
The diff coverage is 81.95%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
Additional details and impacted files

@@            Coverage Diff             @@
##             main     #347      +/-   ##
==========================================
- Coverage   84.91%   84.86%   -0.05%
==========================================
  Files          62       64       +2
  Lines        3593     3654      +61
==========================================
+ Hits         3051     3101      +50
- Misses        416      424       +8
- Partials      126      129       +3

Impacted Files
Coverage Δ

pkg/testsupport/test_httpclient.go
0.00% <0.00%> (ø)

...atorproviders/amazonwebservices/amazon_provider.go
90.62% <87.50%> (+2.83%)
:arrow_up:

...ratorproviders/amazonwebservices/cognito_client.go
91.45% <91.45%> (ø)

...rproviders/amazonwebservices/amazon_http_client.go
100.00% <100.00%> (ø)

...atorproviders/openpolicyagent/aws_bundle_client.go
88.88% <100.00%> (+3.43%)
:arrow_up:

...hestratorproviders/openpolicyagent/opa_provider.go
93.75% <100.00%> (ø)

Continue to review full report in Codecov by Sentry.

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


changelog parsing fails:  invalid type: null, expected a string

I'm toying around with the project; I set my repo to 'https://github.com/betwixt-labs/bebop' and when building I get:
↪ >o_o< INFO: Adding changelog...

  × Failed parsing response when fetching releases from Github.
  ├─▶ error decoding response body: invalid type: null, expected a string at line 1 column 238414
  ╰─▶ invalid type: null, expected a string at line 1 column 238414

hey @AndrewMD5 - thanks for filing and trying out the project. i've reproduced and am gonna dig in :) if you want- we have a discord you can join https://discord.gg/5m7tqr2eGr


Update the rubric

The rubrics have gotten out of sync with some of the important changes we've made in the last couple of years ((branch protection, using pull requests):

The ZenHub entry should probably be updated to reflect the use of protected branches and pull requests.
The continuous integration entry should probably be removed because our use of protected branches arguably sorts that out now.

We might want to replace the continuous integration entry with something that addresses their sensible use of branch and pull requests. They often don't build these around single issues, but instead do things like front end vs. back end, or structuring them around which member of the team is going to work on them. It would be useful if they got some feedback on that sometime early.
Something like this should be done for many/most of the subsequent labs and iterations as well.

It looks like @kklamberty made some changes last year, and I just made a few on my own that I should run past her sometime. I removed the continuous integration item, and added a new item about effective use of branches and associate PRs to organize work into small, focused blocks. If we're both happy with where things are, then I could close this.


No mention of Premium

This article doesn't mention Power BI Premium at all.  Would be great if you could update it to reflect whether it does/doesn't apply to Premium also.  Thanks

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

ID: 0fdac5f7-cd43-34c5-5905-dceeb9813952
Version Independent ID: 6fd53f84-dbb2-2a9c-2745-ea1999779113
Content: Embed Power BI content into an application for your customers - Power BI
Content Source: powerbi-docs/developer/embed-sample-for-customers.md
Service: powerbi
GitHub Login: @markingmyname
Microsoft Alias: maghan

@chrisweis Please check out this article - https://docs.microsoft.com/en-us/power-bi/developer/embed-sample-for-your-organization.  It discusses Premium similar to this article.
Hope this helps.
Thank you.

We will now proceed to close this thread. If there are further questions regarding this matter, please tag me in your reply. We will gladly continue the discussion and we will reopen the issue.


CHEBI:76255 wrong structure

The functional parent indometacin has a methyl group that is missing here. Please fix.

Now fixed. Thanks


tflite quantization to fp16 only quantized input

Hi, I have a model which is simply 3 layer fully connnected layers (the first layer will have resample on weights)
but I got a tflite graph that only quantized input:

it only quantized the weights, but all other operations were dequantized and calculate in float32 way.
Why? I want the whole operations inference in fp16 since am runing on ARMv8.2 above CPU!

Hi @jinfagang ! Could you please attach a minimal stand alone code too?

@mohantym  Hi, I might can only add the tflite model. But Can u point out why it goes like this? the model arch is like this:
link
the whole graph is actually very simple:
https://github.com/sebastianstarke/AI4Animation/blob/e2cd539816b1cb1fa0a57e9d21df21d48467b313/AI4Animation/SIGGRAPH_2017/TensorFlow/PFNN/main.py#L82

@jinfagang ! Could you point towards the location of Input.txt  and Output.txt too?

@mohantym Hi, the input is jus a random [bs, 1024], and output is just a random of [s, 980].
For exportation, just give it a random input and we can the graph. For the input name, I think you can specific anyname in node

float16 weights will be upsampled to float32 weights prior to first inference to have minimal impact on latency and accuracy and also it reduces the model size significantly.
To know more, refer this document.Thanks!


Alias '/careers' -> '/jobs' not working

The alias '/careers' for the /jobs page doesn't seem to be working correctly.
For example, the link to https://2i2c.org/careers/ on the Get Involved section results in a 404 page.

Thank you so much for spotting these link bugs!


:arrow_up: chore: pnpm up all packages

Description
Concise description of proposed changes
Testing
Explain the quality checks that have been done on the code changes
Additional Information

[ ] I read the contributing docs (if this is your first contribution)

Your ENS/address:

Current dependencies on/for this PR:

main

PR #217   👈

This comment was auto-generated by Graphite.


Accept array in is?

(is @[foo bar baz] note) would be equivalent to  (is foo note) (is bar note) ...

Would it be better to have a separate are macro for this? While this looks helpful in the abstract, it also seems to confuse the semantics of is. To be clear, it doesn't have to be are and could be something that makes the relationship to is clearer (e.g. is-multi, is-group, iss, etc).

Yeah, I think that’s right. I’m not a fan of those bdd-style libraries that try to give you an English word for everything you might want to say, but the aim of keeping a simple and consistent semantics for the function itself is quite worthwhile.

Yeah, I'm similar. I'm a worried that are is a little too clever by half. I didn't really like the other alternatives, though. If it was a separate macro, do you have a name you'd prefer? is-all? is-every?

each-is maybe? each is already a word

That one sounds good to me: clear and easy to understand.


taskfile does not work with nvm

That might come as a weird bug report but you are the first person that used nvm inside taskfile and I was looking to do the same. Sadly, I soon found out that your implementation does not really work (at least not for me).
I am on macos with m1 and when that is what I got:
$ task build:local
bash: line 4: nvm: command not found

nvm is installed and working locally but I do not know why it did not work with your cool NODE_ENV approach.
PS. That is also related to https://github.com/go-task/task/discussions/773

For me both run fine. I created #5 to fix it for you. I think it's a safe change.


Is Namespace 'Gibbed.ProjectData' Missing Type or Namespace Name 'Manager'?

I'm trying to build Gibbed.Disrupt, and i have some problem.
C:\Users\[REDACTED]\files\Gibbed.Disrupt\projects\Gibbed.Disrupt.FileFormats\ProjectHelpers.cs(35,30): error CS0234: 命名空間 'Gibbed.ProjectData' 中沒有類型或命名空間名稱 'Manager' (是否遺漏了組件參
考?) [C:\Users\[REDACTED]\files\Gibbed.Disrupt\projects\Gibbed.Disrupt.FileFormats\Gibbed.Disrupt.FileFormats.csproj]
    8 個警告
    1 個錯誤

How are you cloning the repository? The repository has submodules that need to be initialized/cloned as well.
ie
git clone --recurse-submodules https://github.com/gibbed/Gibbed.Disrupt.git
if you've already cloned the repository you can do
git submodule init
git submodule update

Thanks for helping
Now I have a new issue:Is .net Framework 4.6 or higher can be used to retarget?
I'm stuck on MSB3644 now,and net40/45 installation isn't able to proceed

There shouldn't be any issues retargetting to a newer framework version.

Huge thanks
In the end it just a dependency issue, Just need to type these CLI command to XCompress/Ndesk/Gibbed.IO/Gibbed.ProjectData
dotnet add package Microsoft.NETFramework.ReferenceAssemblies.net40 --version 1.0.3 dotnet add package Microsoft.NETFramework.ReferenceAssemblies.net45 --version 1.0.3
Thank you


Running h5_to_faiss.py script fails with invalid ELF header on swigfaiss.so

@HendrikStrobelt
I have generated the states.h5 file and now want to create the indices for encoder and decoder. Running the h5_to_faiss.py script fails on _swigfaiss.so with  invalid ELF header.
I have tried reinstalling the faiss library, installing different versions of faiss library, also gpu version and they don't seem to solve the original problem.
It seems like _swigfaiss.so is corrupted, if so, is there a location where i can download latest version from.
Is there any alternate workaround for generating the indices apart from using this scrips ?
Snapshot of Error:

Thanks !
Mohammed Ayub

Presently I just got around this error by installing faiss in another conda environment. Running only those lines from h5_to_faiss.py that extract the indices and context separately in Jupyter notebook. Successfully generated those 3 files (encoder.faiss, decoder.faiss and context.faiss)
Assuming we don't require the swigfaiss files in the same folder.

@mohammedayub44  Can you show more details about how to get around this error? Thanks

@Leslie-Fang
If you look at the h5_to_faisss.py all we need the faiss library installed (version)
conda install -c pytorch faiss-cpu  this should install version 1.6.0 in your new environment. As long as you can run import faiss you can run the below step.
I just wanted to see what each step is doing so I broke it out in the Jupyter file like below (running it three times for different outputs as suggested here

Hope that helps.

@mohammedayub44 Thanks for the information
It seems faiss.py in the scripts is useless. Remove it would fix the problem

@mohammedayub44 Thanks for the information
It seems faiss.py in the scripts is useless. Remove it would fix the problem


Add an Auto categorize feature like IDM and FDM

I like how IDM and FDM use folders to organize files according to their type
so like:
for zip,rar > Compressed
for mp3 > Music
for exe,msi > Programs
and so on
it would be a really good QOL feature

@Khalylexe Hi there thank you. I think this is duplicate of #26.


cleanup ConnectToController and PasswordToController

The code in ConnectToController and PasswordToController is not factored well and should be cleaned up.
https://github.com/microsoft/azuredatastudio/pull/12217#discussion_r488383714

fixing with original PR.


Add error handling for pose estimation

Adding error handling to pose estimation in case of perception failure. The pose estimation task will try to get the pose up to 3 times and will fail if this does not happen.
Behaviour is to go to a predefined position which is determined to likely be safe(currently set to the restock position).
For the time being, we should aim to ensure that there is always a good pose given to the robot(make sure a person is in a good position which shoulders clearly in view). This functionality should be important but with our demo being a video and the limited time, the error handling is not critical.
Merging int v1 test day so we have all of the code changes on a stable branch

Will fix formatting once approval is given, otherwise formatting changes will clutter the PR

Deprecated for https://github.com/nightingale-project/nightingale/pull/57 which integrates directly into main


Artisan command for "refetching" user avatars for existing users

Added an artisan command to search for new (gr)avatar for given user but deletes previous avatar, even if no new one is found.
Solves #1893.

Hi @shelmz, Thanks for offering this. Looks functionally good.
A few notes from my side:

[ ] I think the command should be named using "Refetch" instead of "Reset" since people may instead expect it to be reset to the default empty state. Fetching enforces the idea we'd be "fetching" the image from elsewhere.
[ ] The $signature needs to be updated since it still refers to reset MFA.
[ ] The User is being injected in via the constructor but not used, can be removed (Including the comment on line 27).
[ ] Ideally we'd add a test to cover the functionality (example). Just say if not comfortable though and I can add this afterwards.
[ ] I think we should have an optional --users-without-avatars flag which will run this for all users without avatars assigned. This would considerably help the scenario given in  #1893 where they presumably have many existing users without avatars, rather than them needing to run this command for each id/email desired.

Since there's been no follow-up I'm going to close this but would be happy to re-open or accept a new PR if the above notes can be taken into account.


Support defining warnings for DevWorkspaces

Description
The DevWorkspace Operator defines a DevWorkspaceWarning condition, but it is used and surfaced to users in an inconsistent way. We should define a general warning mechanism for DevWorkspaces that enables notifying users that there are potential problems without failing the DevWorkspace entirely. For example, this could look like
NAME    DEVWORKSPACE ID             PHASE     INFO
plain   workspace1735ae3bdb3d46bd   Running   Workspace is running (2 warnings)

with some relatively simple way to view what those warnings are via the API.
Additional context

Closed by https://github.com/devfile/devworkspace-operator/pull/1064


Explore adding option to import Fleet-only clusters into Rancher

Fleet-only clusters are great for lightweight, IoT, and edge use cases, but some users may want an easy solution or process to import them into Rancher as well.

There is a manual process to enable this which should be automatable:

Register Fleet cluster
Create an empty Rancher cluster object
Convert the import manifest into a git repository
Add the fleet.cattle.io/managed: false labels to the resources in the repository
Create a GitRepo resource referring to the git repository and the target Fleet cluster
Clean up original Fleet resources after cluster has been successfully imported.

workaround exists, dropping team/fleet
/cc @agracey (edge use case)

Putting team/fleet back on this but removing the milestone, if they feel that they want to tackle this work, they can. Or they can close the issue (with workaround).

/cc @davidstauffer


Origin: move out of internal

As origin is now saved as part of quasiquotes, let's make sure it abides by MIMA standards (which are not currently applied to internal code).
Similarly, remove TokenStreamPosition and inline in Origin.Parsed. Helps with #3425.

@tgodzik @bjaglin i am having second thoughts about this. is my understanding even true? if origin is not a public field, does it really have to abide by any interface when quasiquotes are involved?

I think we use Origin in quite a few places outside of scalameta so it makes sense to move it out of internal.

(I see there is very active progress on this massive effort, thanks a lot for keeping backward compatibility in mind despite the complexity!)

@bjaglin thanks for chiming in. a further question:

internal.Impl.unapply was problematic as that was the final result of this expansion;
however, origin is "hidden" somewhere inside the tree and not directly exposed

after some thought, it looks to me no different from any other internal type, at least as far as it is used in the construction of a scalameta tree, so either all of them are bad or none are...
for instance, InternalTree is a base class of Tree, and yet it's  underinternal package and hence outside of Mima
similarly, .syntax is implemented using TreeSyntax and that is also internal; so if you have code which uses this method, is it at risk if TreeSyntax changes?

I see your point, it's hard to tell what's really contractual at macro expansion time... Did you have a look at code generated by the quasiquotes macros with -Vprint:typer? I doubt that there is anything internal at the moment, but if there is, then I agree about your statement "either all of them are bad or none are".

... Did you have a look at code generated by the quasiquotes macros with -Vprint:typer?

i don't really know how to do that, truth be told... 😊

There is probably an easier way to do it, but what I did last time was to add -Vprint:typer to scalacOptions in a test module, run a full compile (ignoring the verbose output), touch a file exercising a macro by adding some dummy code, and check the output of that incremental compilation hunting for references.
And ... now that I write this, I realize javap -p -v on class files generated from macro-expanding sources is probably even easier to identify what really gets emitted and thus should be backward compatible. So that would be my recommendation!


Move sensor read to common code

Summary:

Move code about sensor reading from platform code to common code.
Modify sensor reading value to 4 bytes, 2 bytes for integer, 2 bytes for fraction.
Provide README file describing how to add new device.

Notice:

BMC needs to support parsing 4-byte accuracy sensor reading. Otherwise, sensor-util command would print wrong value.

Test Plan:

Build code: Pass
Test OEM sensor reading command: Pass

Log:
root@bmc-oob:~# bic-util slot1 0xe0 0x23 0x9c 0x9c 0x00 0x01 0x00
9C 9C 00 26 00 00 00 C0

@GoldenBug @garnermic
This PR need to be phased in with BMC code change, otherwise it will cause sensor reading/reporting issues for all the BIC sensors. BMC portion is planned to be PRed by today taiwan time and not yet pull requested.
For this moment, (before one day of release), I would suggest that we don't merge them (both BMC/BIC port) today.
Wiwynn is discussing with ODMB to get more detail test status to get more confidence for the pull request.

Hi @GoldenBug,
Thanks for your suggestion and we modified the DIMM temperature and the return code of the sensor initialization function.
The sensor naming isn't impacted the firmware behavior, so I will create another issue to track.
Except for the sensor naming, do you have other concerns about the code logic of this PR?
ODMA had been run the AC/DC cycle testing over 1000 without failed and the ODMB also passed the same test with 250 cycles.
We have enough confidence for the PR by the test result.

Could you please rebase this then I can get it reviewed again?
Thanks!

We've forced updated the PR to rebase the main branch.
Please help to review again, thank you.

Hi @GoldenBug,
We have submitted a commit to revise the comment you provided, including:

Extend sensor names to follow the same pattern.
"sen" -> "sensor" and "snr" -> "sensor"
Modify function return value, return 0 if success, non-0 if fail.
Remove unnecessary goto.
Add 0x69 in pmbus.h

Please help to review again, thank you.

I still see a lot of SNR uses. Please look at extending all the use cases of SNR.
We may need to change SNR in other files this commit didn't touch and that's ok.

OK,  we will change all SNR to sensor.

Hi @GoldenBug,
We have submitted two commits with changes including:

merge to main branch.
Extend the name of variables, functions and macros from "snr" or "sen" to "sensor".

Please help to review again, thank you.

Hi @GoldenBug,
We have submitted two commits with changes including:

Merge to main branch.
Add path of common sensor code into CMakeList.txt of yv35_bb.
Apply Clang format to all change files.

Please help to review again, thank you.


[AutoPR automation/resource-manager] Swagger change for Update configuration dynamic group

Created to sync https://github.com/Azure/azure-rest-api-specs/pull/4165

(message created by the CI based on PR content)

This PR has been merged into https://github.com/Azure/azure-sdk-for-python/pull/3125


remove dtostrf from code to make it less AVR specific.

Some unit test need printHelpers - e.g. timing
/github/home/Arduino/libraries/printHelpers/printHelpers.h:270:3: error: 'dtostrf' was not declared in this scope
dtostrf(val, length, decimals, buf);
^~~~~~~
Also look into this warning
/github/home/Arduino/libraries/printHelpers/printHelpers.h:260:26: warning: ISO C++ forbids converting a string constant to 'char*' [-Wwrite-strings]
if (isinf(val)) return "";

fixed in 0.2.0 version

fixed in 0.2.0 version


Incorrect number of replicas when editing replicas on the model list

Describe the bug

Steps to reproduce
https://github.com/user-attachments/assets/8a257886-8931-4e59-a1d6-6f319ac3d23e
Expected behavior

Environment

GPUStack version: fa073a7
UI version: b6d5b45
OS: macOS 14.6.1
GPU: Apple M2, 24G

Additional context
Add any other context about the problem here.

Environment

GPUStack version: main(528ab87)
UI version: 2cd676f
OS: macOS 14.6.1
GPU: Apple M2, 24G

Test Result: pass


Wishlist css

Cursor in drag / drop in wishlist is a pointer and not a cross.
When dragging items, they should keep their size and not shrink

duplicate issue

See #35 and #36


ListItem dropdown options menu

Felt like the buttons on the list of items were getting cluttered. So I replaced them all with a button that when clicked displays a dropdown menu with the different actions. I just gave it some basic styling for now, and we could maybe add some icons to the different options in the future.
I also created two new functions claimItem and deleteItem in RoomRoute that are called when those options are clicked in the menu. All they do is print to the console right now.

I think it looks pretty good, definitely less cluttered than it was before. That being said, I changed the color of the "Delete" option to red (I think I made it the same color as Bootstrap's default, just because) to draw more attention to it. We could also make the background of the button red instead of the text, if it looks better.

Looks good


Annoyingly 1 Station from TuneIn Radio not working

What version of Music Assistant has the issue?
2.0.0b85
The problem
I have been using Music Assistant Beta 2 for ages and been using TuneIn to play my favourite music station on my google device. This has now stopped working. All other stations work. And it works directly from TuneIn. I have tried to delete TuneIn and re add it and update the database but no luck. I have included the log which contains many errors when I click on this station.
How to reproduce
Play Radio Essex using the TuneIn App within Music Assistant
Relevant log output
2024-01-24 15:47:12.572 ERROR (MainThread) [aiohttp.server] Error handling request
Traceback (most recent call last):
File "/usr/local/lib/python3.11/site-packages/aiohttp/client_reqrep.py", line 965, in start
message, payload = await protocol.read()  # type: ignore[union-attr]
^^^^^^^^^^^^^^^^^^^^^
File "/usr/local/lib/python3.11/site-packages/aiohttp/streams.py", line 622, in read
await self._waiter
File "/usr/local/lib/python3.11/site-packages/aiohttp/client_proto.py", line 224, in data_received
messages, upgraded, tail = self._parser.feed_data(data)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "aiohttp/_http_parser.pyx", line 557, in aiohttp._http_parser.HttpParser.feed_data
aiohttp.http_exceptions.BadHttpMessage: 400, message:
Expected HTTP/:
b'RPC-ERR authentication failed\n'
  ^

The above exception was the direct cause of the following exception:
Traceback (most recent call last):
File "/usr/local/lib/python3.11/site-packages/aiohttp/web_protocol.py", line 452, in _handle_request
resp = await request_handler(request)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/usr/local/lib/python3.11/site-packages/aiohttp/web_app.py", line 543, in _handle
resp = await handler(request)
^^^^^^^^^^^^^^^^^^^^^^
File "/usr/local/lib/python3.11/site-packages/music_assistant/server/controllers/streams.py", line 482, in serve_queue_item_stream
await set_stream_details(self.mass, queue_item=queue_item)
File "/usr/local/lib/python3.11/site-packages/music_assistant/server/helpers/audio.py", line 277, in set_stream_details
streamdetails: StreamDetails = await music_prov.get_stream_details(
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/usr/local/lib/python3.11/site-packages/music_assistant/server/providers/tunein/init.py", line 230, in get_stream_details
url_resolved, supports_icy = await resolve_radio_stream(self.mass, stream["url"])
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/usr/local/lib/python3.11/site-packages/music_assistant/server/helpers/audio.py", line 541, in resolve_radio_stream
async with mass.http_session.head(
File "/usr/local/lib/python3.11/site-packages/aiohttp/client.py", line 1187, in aenter
self._resp = await self._coro
^^^^^^^^^^^^^^^^
File "/usr/local/lib/python3.11/site-packages/aiohttp/client.py", line 601, in _request
await resp.start(conn)
File "/usr/local/lib/python3.11/site-packages/aiohttp/client_reqrep.py", line 967, in start
raise ClientResponseError(
aiohttp.client_exceptions.ClientResponseError: 400, message="Expected HTTP/:\n\n  b'RPC-ERR authentication failed\n'\n    ^", url=URL('https://playoutonestreaming.com/proxy/essex?mp=/stream')
Additional information
No response
What version of Home Assistant Core are your running
2024.1.5
What type of installation are you running?
Home Assistant OS
On what type of hardware are you running?
Raspberry Pi

I just tried the url at the end of your log post and it works ok in my browser. Out of interest have you tried entering that url manually on the radio stations view?

Yes I tried it manually in a browser and it worked. Thats why I thought maybe there was some sort of bug in music assistant.

Yeah I didn't mean in the browser I meant via the MA radio stations view. The menu in the top right has the option to manually add a stream URL. However, I just tried it and that doesn't work either but with a different error. Might take a while until this can get looked at unfortunately. If you can find a different stream URL then try that. All I can surmise at the moment is that the stream URL changed and that has caused a problem...

I have reproduced the issue - will be fixed in the beta (beta89) later today.


chore(ci): don't run e2e in GKE clusters

What this PR does / why we need it:
Due to the issues with running E2E tests on GKE we have (https://github.com/Kong/kubernetes-ingress-controller/issues/3331), we decided to switch all the tests to Kind for now, until we find a working solution for the problems described in the mentioned issue.
Issue for tracking the enablement of GKE E2E tests back once the problems are figured out: https://github.com/Kong/kubernetes-ingress-controller/issues/3348

Which issue this PR fixes:
Workaround for https://github.com/Kong/kubernetes-ingress-controller/issues/3331.

Can you run e2e on this branch and kink it here? I am sceptical about this working on kind 1.26

Can you run e2e on this branch and kink it here? I am sceptical about this working on kind 1.26

It's still running: https://github.com/Kong/kubernetes-ingress-controller/actions/runs/3874399771/jobs/6605545199. Why are you skeptical about that? Did we have issues with Kind v1.26.0 in the past?

Can you run e2e on this branch and kink it here? I am sceptical about this working on kind 1.26

It's still running: https://github.com/Kong/kubernetes-ingress-controller/actions/runs/3874399771/jobs/6605545199. Why are you skeptical about that? Did we have issues with Kind v1.26.0 in the past?

Nope, I tried running kind with 1.26 last week and it didn't manage to deploy the cluster successfully.
BTW: this

It's still running: https://github.com/Kong/kubernetes-ingress-controller/actions/runs/3874399771/jobs/6605545199.

Is running e2e targeted which is a different workflow than what you're changing ( e2e and release) 😬 . (My opinion on this is unnecessarily complicated and we should extract the common parts out and parametrize those workflows - as I believe you suggested already at some point @czeslavo )

@pmalek as you suggested I've removed v1.26 tests for now as there's still no official Kind release for this k8s version. It's mentioned in https://github.com/Kong/kubernetes-ingress-controller/issues/3348 that using Kind v1.26 may be the solution as well.


Service fails to start when installing from source with systemd

When installing from source on Debian 8, the service will fail to start because systemd does not recognize it.  Have not tested other systemd distros yet.  Would assume it does.
Running systemctl daemon-reload works with the current init.d script or a systemd unit file could be generated instead.  Would require a platform version check either way.

can you see if this is still a problem in the 3.x release?