The OnNetworkLatencyUpdate event is processed for the first time, and the latency value must be 0.

Whether it is internal or external network
I would like to get a correct ping before the OnPeerConnected event call

I have used this library in my project. Will this problem be processed? If handled, I will not modify the source code. Thank you.

Ehm... Ping calculated only after connection. Becuase this logic inside NetPeer.Update method. And for precise ping you must wait until special packets will be sent to calculate RTT.

You can do this by Unconnected messages.
Send one from client and measure response (by sending unconnected message from server) time.


Weave (1.0.3) ran out of memory

We recently encountered a out of memory issue with weave 1.0.3, which was supposed to fix it.
Here are the full logs of what happened:
https://gist.github.com/MaximeHeckel/e551bf4648b9539f46fb
It happened twice today while trying to implement the -initpeercount to our weave-daemon to handle connection issues

Any idea for how long weave was running before it died?
Is the (dead) container still around? If so, please run docker inspect on it.

It just restarted, so it was pretty new.
No, sadly the node has been cleaned and terminated almost an hour ago.

Some more info which may be relevant:

That host had a peer count value of 7  and 7 nodes were deployed.
This happened another time on another node (exact same logs) who had a peer count value of 5 (because only 5 nodes were launched at that time)
I’m testing on DO instances of 1GB of memory (what our users are using the most), so I guess it must have reached the limit.

Some things are evident from inspection of the log:

the allocation which fails is for 320 bytes, so
some goroutine numbers are in the 600,000s, but there are only 70 goroutines listed, which suggests a lot of churn
there are only 8 connection attempts listed in this log excerpt
none of the other goroutines appear to be doing anything surprising

Places where weave 1.0.3 creates goroutines:

LocalConnection.Start() - logs "completed handshake; using protocol version ..."
LocalConnection.Shutdown() - no logging
LocalConnection.run()  - always comes after "completed handshake" log as above
ConnectionMaker.Start() - no logging; only called once
ConnectionMaker.connectToTargets() - logs "attempting connection"
Forwarder.Start() - no logging
ForwarderDF.Start() - no logging
GossipSender.Start() - no logging
LocalPeer.Start() - no logging; never exits; only called once
Router.sniff() - logs "Sniffing traffic"; never exits
Router.listenTCP() - no logging; never exits; only called once
Router.listenUDP() - no logging; only called once
Routes.Start() - no logging; never exits; only called once
Allocator.Start() - no logging; never exits; only called once

So, if something is causing a lot of goroutines to be created and destroyed with no logging, it must be one of:

LocalConnection.Shutdown()
Forwarder.Start()
ForwarderDF.Start()
GossipSender.Start()


CIP-???? | Trustless BTC Bridge

Abstract
This proposal defines eBTC: a trustless, one-way bridge allowing BTC to enter Cardano as a native token. Users burn BTC on Bitcoin and present cryptographic proofs (NIPoPoW/SPV) to a Cardano smart contract, which mints a corresponding amount of eBTC. This solution removes reliance on custodians and oracles, enabling purely decentralized conversion and unlocking BTC’s liquidity for Cardano DeFi.

I'm converting this into Draft review status until it complies with the CIP format as described in CIP-0001 (https://github.com/cardano-foundation/CIPs/pull/955) with about 100 examples of CIPs in the repository to imitate if there is any uncertainty.  See here for factors that make a good CIP: https://github.com/cardano-foundation/CIPs/wiki/2.-CIPs-for-Reviewers-&-Authors
The term CPS is originally in the title although this appears to be a plan for a particular method.  @siran if you think this is instead a class of possible approaches to BTC wrapping on Cardano that could generate multiple CIPs, please say so here and we'll try to get you started converting your draft into a proper CPS draft.

Thanks so much for your comments and patience. This is my first attempt at a CIP. So i'll check and imitate previous CIPs.
I do not know if this is a CIP or a CPS. Nor if it would devolve into multiple CIPs. I would be happy just with it being discussed :).
I'll try to read around to meet standards. Tried Discord server but didn't get response.
Thanks.

@siran it's still not in the form of CPS a which has different sections than a CIP.  If you could please refer to it consistently as a CIP (as opposed to this Discord comment), and change the title of your containing directly to something like CIP-XXXX or CIP-BTC-bridge, it will help potential reviewers.
Also you should include the Cardano Forum post in the CIP header (https://forum.cardano.org/t/ebtc-a-trustless-btc-bridge-on-cardano/141117) now that discussion has emerged there.
I'll leave it as a CIP in the PR title since that's definitely what it appears to be.  FYI here is the specification for the CPS (which I don't think it's in your best interest to use): https://github.com/cardano-foundation/CIPs/blob/master/CIP-9999/README.md


Capturing Issue Description in Sentry

Related to

Related to #1604 by @sarayourfriend
Description
Not sure on when how to reproduce this issue, but this should return the whole exception to provide more context.

I'm not sure what the scope of this PR is, @ramadanomar, but it would be nice to re-write the handleMediaError function in light of the Axios handling errors page.
For instances of Axios errors:
In case the error has a response property
The request was made but has received a response that is not in 2xx range.

set the message to "Error fetching ${mediaType}. Request failed with status code ${error.response.status}"
capture the event in Sentry

In case the error doesn't have a response property, but has a request property
The request was made but no response was received.
error.request is an instance of XMLHttpRequest in the browser and an instance of http.ClientRequest in node.js

set the message to "Error fetching ${mediaType}. Could not receive a response from the server"
capture the event in Sentry

Otherwise
Something happened in setting up the request that triggered an Error

set the message to "Error fetching ${mediaType}. Unknown Axios error"
capture the event in Sentry.

Non-axios errors
I have no idea if those can happen, but we should still add a branch to capture those with Sentry and set the message to "Error fetching ${mediaType}. Unknown error"
I wonder how much we want to capture in Sentry. Do we want to capture all failed requests in Sentry, @WordPress/openverse-frontend? Would capturing the event with toJSON make sense, or would it be too much information?

I'm not sure what the scope of this PR is, @ramadanomar, but it would be nice to re-write the handleMediaError function in light of the Axios handling errors page.

Hey! Thanks for the feedback. My scope was to provide more context to audio errors. I am pushing an updated handleMediaError function right now. As of toJSON, i am not sure when this error is called to determine how much info we need. The new handleMediaError function should allow for easy changes in case we need to edit the level of detail we require.

Thanks for the changes @obulat . I resolved them.

@obulat From my perspective all failed requests that we don't know how to explicitly handle should be captured in Sentry so that we can be aware of them and either (a) handle those explicitly or (b) explicitly ignore them as non-issues or unsolveable. Does that seem like a reasonable standard?

Any idea on how i could mock the sentry capture event without importing nuxtjs/sentry module? tried accesing it through this.$nuxt.$sentry , this.$sentry, $nuxt.$sentry and $sentry but they all seem to be undefined. Another method i tried was mocking the function call, something similar to
jest.mock('axios', () => ({
  ...jest.requireActual('axios'),
  isAxiosError: jest.fn((obj) => 'response' in obj),
}))

but couldn't seem to be able to deconstruct the event properly. Any pointers on how i could approach this? Thanks!

@ramadanomar, to mock Sentry centrally, you can add this snippet to test/unit/setup-after-env.js:
import Vue from 'vue'

Vue.prototype.$nuxt = {
  context: {
    $sentry: {
      captureException: jest.fn(),
    },
  },
}

You can also add any other functions that need to be mocked.
I found this solution here: https://github.com/vuejs/pinia/discussions/1252

Then, for example, on line 385 in test/unit/specs/stores/media-store.spec.js you could use the mock like this:
      expect(mediaStore.$nuxt.$sentry.captureEvent).toHaveBeenCalledWith({
        message:
          'Error fetching audio from API. Request failed with status code: 500',
        extra: { error },
      })

Please note that you would also need to add captureEvent to the mock above for this to work correctly.

@ramadanomar, I've opened a PR in your fork with some more tests for different branches of error handling. I would really be happy to approve after those tests are added.

I've merged your pr @obulat. Thanks for the help!


Undefined Error after uploading opencart-2checkout

I uploaded the files as recomended.But now I am having a problem which gives an error when I add products to my shopping cart.
At notification it says UNDEFINED for the products I added to my cart while it is a success.
please find the pic as attached.

my open cart version is <IP_ADDRESS>

This does not appear to be related to the module but it could be some incompatibility with your theme. Were you able to setup your 2Checkout configuration in the woocommerce admin?


/selectedCueName - feature request

In the new Mitti version 1.5.13 the OSC feedback message "/selectedCueName" was added. Can you please include this in the module and add it as a text variable?
Thanks!

Working on adding it with https://github.com/bitfocus/companion-module-imimot-mitti/pull/10

Working on adding it with https://github.com/bitfocus/companion-module-imimot-mitti/pull/10

Implemented with the merge of https://github.com/bitfocus/companion-module-imimot-mitti/pull/10

Implemented with the merge of https://github.com/bitfocus/companion-module-imimot-mitti/pull/10


Performance of get_owned_ids

Extract from Gitter:

Roderik van der Veer @roderik Nov 02 20:29:
Is anyone experiencing very slow “get_owned_ids”? I've got 5 addresses with combined 80 assets in them, and when i do get_owned_ids on each of them (concurrently) i still end up with about 4.5 seconds to get all the results back.  We should consider creating a secondary index for this query

Is there any update on this? I've run out of options with some 600 assets...

@roderik currently working on it. I will cc you when I make the pull request

#1125 Added secondary indexes for inputs and outputs which makes the database queries faster, and the execution time pretty much constant as the number of documents grow since we no longer perform full table/collection scans for those queries.
We still have a performance bottleneck in get_owned_ids because for each transaction that the database returns we still need to check the votes table to see if the transaction is on a valid block.
We are still looking at solutions for this and we have a task on the rodmap to address the performance issues of all the queries.

@roderik We recognize that it's important to make this query fast. We haven't forgotten! BigchainDB 2.0 works very differently and is coming out very soon. We have good reason to expect that this query will run much faster in BigchainDB 2.0. For one, there are no more invalid blocks to go through and weed out! We'll be doing some performance benchmarking soon after the release of 2.0, and then we'll know how fast it is, and we can start figuring out how to make it even faster.


Test against Ruby 2.6 and RubyGems 3.0

Context
https://www.ruby-lang.org/en/news/2018/12/25/ruby-2-6-0-released/
I had to change the rubygems specification to get Bundler to install the dependencies.
Change

Add 2.6.0 to the list of Rubies to build against.

OK. LOL. https://twitter.com/colby_/status/1080703022760484865 (thread)


bp_me_axil_master: Support unaligned writes

Branch / Commit Hash
top_dev: af8beff
Area
bp_me_axil_master module
https://github.com/black-parrot/black-parrot/blob/top_dev/bp_me/src/v/network/bp_me_axil_master.sv#L113
Current behavior
Instead of m_axil_araddr_o  = io_cmd_header_cast_i.addr[0+:axil_addr_width_p];,
we might want something like m_axil_araddr_o  = {io_cmd_header_cast_i.addr[2+:axil_addr_width_p], 2'b00};(assuming bus width == 32),
so that we issue aligned memory access in AXIL. After getting the 4-byte read data, bsg_bus_pack will then select the appropriate subset of data and send it back to bp I/O cmd.
If this is correct then I can make a PR for this.

Are you trying to access unaligned registers on the AXI bus? It seems unlikely that software would require this

For example if BP wants to read a 16-bit data from address 0x1002. This BP i/o cmd will translate into a 32-bit read at 0x1002 on AXIL bus in our current implementation.

Yes I understand. Is there a device that you’re using that has 16-bit registers?

https://elixir.bootlin.com/linux/v5.15/source/drivers/net/ethernet/litex/litex_liteeth.c#L78
I guess when Linux wants to read out the Ethernet packet from IO, it might emit that kind of memory access. Say the incoming packet size is 0x79, the kernel might eventually have to do a 1-byte read from 0x79 instead of doing 4-byte accesses throughout the process.

The data buffer isn’t a fifo in the controller? We’re reading from a block of memory?

Yeah we are reading from a block of memory. This is compatible with Litex Ethernet core. It is not like, we read data from a fixed 32-bit memory address (if this is what you mean about the FIFO)

DIscussed offline, but for posterity: the axi spec dictates that clients should ignore the bottom bits such that all requests return data aligned to the bus width. Therefore there is no need to truncate the addresses for reads.
However, writes require shifting the strobes appropriately. I'll leave the issue open for someone to add this feature if they need it

Closing issue as axi modules have moved to subsystems


Round up Amount div Weight

Compute the ceiling so that the fee-rate is conservative.  Discussed here https://github.com/rust-bitcoin/rust-bitcoin/pull/3430

Pull Request Test Coverage Report for Build<PHONE_NUMBER>4
Details

23 of 23 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.02%) to 82.6%

Totals

Change from base Build<PHONE_NUMBER>1:
0.02%

Covered Lines:
19045

Relevant Lines:
23057

💛  - Coveralls

On https://github.com/rust-bitcoin/rust-bitcoin/pull/3430 you commented that "We should be rounding up, not down.".  Truncating division is basically "round down" from what I can tell.  Floor and round down behave differently when negative numbers are involved but this is all u64 types.  I'm confused if you are advocating for leaving the mpl Div<Weight> for Amount  method as it is and to round up with the checked variant?  I think that would be confusing so probably misunderstanding.

Yes. #3430 added a new method. My comment was about that, not /.

Yes. https://github.com/rust-bitcoin/rust-bitcoin/pull/3430 added a new method. My comment was about that, not /.

I see.  What's the justification to round up in the checked version https://github.com/rust-bitcoin/rust-bitcoin/pull/3430 but round down when using /?


Update README.md sample was broken

The sample given for Multiple permissions was broken. The compiler doesn't like doing Dexter.checkPermission inline because it has no return value. (This might have changed with a new version of Dexter?).
Creating the permission list separately solved it.

Ouch! It's a typo, the first line can be removed (no need to create a collection, let's keep the example as simple as possible), do you mind updating the PR with that change?

Thanks @DanielMartinus !


[Aborea] Create character sheet generation script

Submission Checklist
Required

[x] The pull request title clearly contains the name of the sheet I am editing.
[x] The pull request title clearly states the type of change I am submitting (New Sheet/New Feature/Bugfix/etc.).
[x] The pull request makes changes to files in only one sub-folder.
[x] The pull request does not contain changes to any json files in the translations folder (translation.json is permitted)

Changes / Description

Implemented full script to generate the Roll20 character sheet.
Added functions to generate Spruchlisten HTML, compile Sass to CSS, and combine HTML and JS files into a final output.
Included logic to update the version in the JSON file if HTML or CSS files have changed.

Since I have seen a Makefile in some sheets, I have now also created a script that builds the character sheet for me, so that errors can be avoided and the simple yaml view makes it easy for inexperienced people to customize the spell list.

Character Sheet Info Roll20 Internal Use only.

this is just a nice to have checkbox, the skills of the spells are all still under the skills tab, these are temporary checkboxes that are changed while playing anyway.

I don't know the game or even the language very well, but it seems like these exist as indicators to mark the spells as "learned?" Players will need to go through and re-mark each spell they knew if this update proceeds as-is.
Maybe "known" spells change from session to session - I'll trust your judgement

the spell list height is listed under 07_fertigkeiten.html, the changes should not cause any problems.


Docs - incorrect file paths for database setup

The file paths for example database setup scripts in the System Administrator Guide > Setting up the WCT database are prepended with db/latest/setup/.
https://github.com/WebCuratorTool/webcurator/blob/daede2e615a1625b108793d281c1fcd0c0e30d4b/webcurator-docs/guides/system-administrator-guide.rst?plain=1#L202
The files are in webcurator-db so this should read webcurator-db/latest/setup/.

I understand now, these file paths make sense for the released version, but not if you clone the repository and try to build from source.


Click "more" on Android Firefox would toggle pin/unpin of the memo

Describe the bug
On the android firefox, when clicking the three dot icon on each memos. There is a chance (high chance in my case) to pin/unpin the memo. No false click involved.
I've tested it on the chrome android as well, and didn't have such problem. I can provide more testing on other case if it helps for diagnose.
Steps to reproduce

Open the memo service (logged in) at Android firefox browser.
click three dot icon for arbitrary memo.
There is a chance that the memo is unpinned (when its pined) or pinned (when vice versa)

The version of Memos you're using
0.19.1
Screenshots or additional context
No response

Maybe it has been fixed with the latest version, please try again in https://demo.usememos.com

Maybe it has been fixed with the latest version, please try again in https://demo.usememos.com

Thanks Steven, I had a brief testout and this problem really didn't take place!
Out of curiosity, is the demo following the branch rather than release? My instance is at least release but quiet different from the demo.

Out of curiosity, is the demo following the branch rather than release? My instance is at least release but quiet different from the demo.

The demo site is based on the main branch which is the release candidator.


v6 breaks node14 support

Environment
Node 14
Reproduction
I'm using a nuxt 2 project
Describe the bug
 ERROR  arr.at is not a function
index.cjs:29
  return arr.length > 0 ? arr.at(-1) : null;
https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Array/at
.at is supported starting at node v16.6
Package.json still required node14+
 "engines": {     "node": ">=14.21.3"   }
Additional context
No response
Logs
No response

I came across the same issue and managed to resolve it by using Yarn's resolutions field in my package.json. This approach allows you to override the version of a transitive dependency like webpackbar. Here's how you can do it:
Add the resolutions Field in package.json
In your package.json file, add a resolutions field specifying the version of webpackbar that you want to use. For example, if you want to force version 5.0.2, your package.json should include:
{
  // ... other configurations
  "resolutions": {
    "webpackbar": "5.0.2"
  }
}

Install Dependencies with Yarn
After adding the resolutions field, run yarn install in your project directory. Yarn will respect the specified resolutions and install webpackbar version 5.0.2, regardless of the version required by other dependencies.
Verify the Dependency Version
You can then check if the correct version of webpackbar is installed using yarn list webpackbar.
This method is straightforward with Yarn as it supports the resolutions field natively. However, remember that overriding dependency versions can sometimes lead to compatibility issues, so make sure to test your application thoroughly after making such changes.
Hope this helps!


Ethernet driver for STM32H5xx

Ethernet support for STM32H563, or H5 in general.
On Zephyr v3.4.0 board support is added, based on this chip: "nucleo_h563zi", but Ethernet support is not included.
zephyr\drivers\ethernet\eth_stm32_hal.c supports the series: F2, F4, F7 and H7, but not H5.
Needed for MCUBOOT
We are working on a framework for a customer, who wants to perform firmware updates over Ethernet. We are trying to use mcuboot/mcumgr for this purpose.
The developing work is executed on the eval-board: Nucleo-H563ZI (illustrated below)
I am quite new at Zephyr and would appreciate any input.
NUCLEO-H563ZI
Documentation

I have compared the Ethernet register map for the H7 and H5 series.
Have not found any differences.
stm32h5(63/73/62)
stm32h7(23/33/25/35/30)
Hence, it should be possible to group H7 and H5


Make execution async

[x] I have added the tests to cover my changes.
[ ] I have updated the documentation and CHANGELOG accordingly.
[ ] I have read the CONTRIBUTING document.

PR: https://github.com/AgnostiqHQ/covalent-awslambda-plugin/pull/52


Refresh api?

According to https://github.com/evcc-io/evcc/discussions/2212#discussioncomment-2580957, https://github.com/flobz/psa_car_controller is able to refresh the api session using security pin, e.g. for updating SoC. Seems this is not currently implemented in this module?

Security pin authentication is not implemented yet because of the complexity

If it help I'd volunteer for evcc but would need basic understanding of the flow.

Without access to an otp device it is really hard
https://github.com/flobz/psa_car_controller/tree/master/psa_car_controller/psa/otp

Also ich benutze flobz schon länger und funktioniert prima.Auch diese ABRP Unterstützung ein traum.


typo loading input module: imtc -> imtcp

Signed-off-by: Christoph Badura<EMAIL_ADDRESS>

@bad,
I'm in the process of updating the controls to conform to the latest benchmarks, v1.1.0, as well as updating the layout to match the Ubuntu 18 role. I can add this fix into that work as well, so I'm going to close this PR. I have the fix temporarily in benchmarks_v1.1.0_updates and the goal is to have all of those updates merged into devel by the end of the week, then into main mid-week next week. Thanks for catching this one!
George

@georgenalen,
excellent. I'll wait until the changes are in devel.
I have a list of notes that want to discuss.  We should figure out what the best way to discuss them is.
--chris

@bad,
Once I get the current stuff situated, updates for the new benchmarks version and layout, we should go over the notes. There were a fair number of fixes for Ubuntu 18 that I rolled through a couple weeks ago that I'm going to apply to Ubuntu 20 if needed. So hopefully between your notes and double checking fixed issues in Ubuntu 18 we can have the role working really well.
George


basic auth not working in Node 8.x

This code works in Node 7.x:
const req = require("http").request({
    auth: "user:pass",
    hostname: "temp",
    method: "GET",
    path: "/auth/index.html"
});

req.once("response", function(response) {
    console.log(response)
    response.once("end", () => done());
})
.on("error", error => console.log(error));

req.end();

In 8.x, it logs this:
{ Error: Nock: No match for request {
  "method": "GET",
  "url": "http://temp/auth/index.html"
}

Any nock contributors actually care?

It looks like it's related to issue 925.

@stevenvachon @jscheffner Are you still seeing this issue using the latest master of nock? #925 should be fixed on that commit

Should be fixed in 9.0.14. Please reopen if issue persists.

Fixed! Thank you.


eslint spaced-comment: Allow sprockets directives

@lemonmade

eslint complains if you use a sprockets directive, like
//= require_tree .

This make it recognize = as a special character that comes before the directive

Thanks :+1:


[Enhancement] Implement navigator.notification and navigator.mozNotification

See: https://wiki.mozilla.org/Services/Notifications/Push/API

Please check https://github.com/DangerCove/html5-notifications/issues/1.
The demo is Chrome only to keep it as comprehensible as possible and prevent adding exceptions for unsupported methods (like createHTMLNotification).

I've added your link to the 'not supported'-message.


refactor: select style for model & metric

This PR does some simple styling for model & metric dropdown.
Since some feature of native select depends on browser default settings, such as the blue color when hovering over options. There are actually few options to style.

awesome!


Support for multiple highlighting groups in one pattern

This is a redo of #192 which had me screw up the branch's git commit history. I'm pasting the description from the original PR here.

This PR implements the second part of #126, which enhances the syntax highlighting engine to support multiple groups in a single pattern. It also includes an improved C syntax, with precise highlighting for #include <abc>, struct def, and union ghi, along with some alterations to make it more in line with what the C standard specifies (namely a leading - is not considered to be part of a number literal anymore, but rather a separate operator).
This PR will likely be followed up by another one that'll improve syntaxes for other languages such as Lua, fixing errors like the or keyword in a or {} or return in return "abc" being highlighted like a function call.
Usage
This can be used in syntax files, like so:
syntax.add {
  …,
  patterns = {
    { pattern = "a()b", type = {"keyword", "keyword2"} },
    -- equivalent to
    { pattern = "()a()b", type = {"keyword", "keyword2"} },
  },
  …
}

Side note: I think we should probably document how to write syntax files somewhere, as browsing the source code might be quite daunting for newcomers if they want to learn about these more advanced features of the highlighter.
Limitations
Currently this doesn't work with span patterns (used in matching eg. block comments /* abc */) and subsyntaxes. The code for these is quite complicated since @adamharrison's PR #160, and I just couldn't find my way around adding this feature there.

Off-topic, but for some reason the dev branch seems to be very unstable at the moment, often crashing when opening the application up. Any news on why that's happening?

I've been meaning to go in there and clean up some of the code (the indentation is messed up, at the very least), I could add a decent amount of comments in there while I'm at it to explain things a little better, or maybe restructure some of it to be more clear. I'll look into it.

@adamharrison

I've been meaning to go in there and clean up some of the code (the indentation is messed up, at the very least)

I already fixed that for you in the master branch, so no worries.
I'd appreciate if you could somehow make the code a bit less repetitive. Currently doing any sort of refactoring with token parsing is a bit of a chore, hence why this PR does not apply to span patterns.

Yeah; I was really just trying to fit it in the existing paradigm, but something could probably be done to compress it. I'll look into it tonight.

@liquidev Do you want me to make the changes on your branch, or are you happy merging changes I make on master?

I guess that settles that. I'll make the changes on master.

For the complexity in relation with the @adamharrison code, well, we will see. I hope we can reconcile the two modifications at the end.

I like this, for the C syntax is a good improvement.
On the other side the real thing for the syntax is this:
https://www.sublimetext.com/docs/3/syntax.html
The ability of enter in a new context to enable other kind of patterns is very powerful and is needed in many cases to have accurate highlighting. The approach using highlight groups offer the same thing but in a limited way because everything has to fit into a single pattern even if you are allowed to have multiple groups. You cannot fit a whole sub-language into a pattern.
In some way the modification of @adamharrison goes in the direction of the context but only allow to choose another complete language. You cannot ember a mini sub-language inside a syntax file. May be we can develop something along these lines.

In some way the modification of @adamharrison goes in the direction of the context but only allow to choose another complete language. You cannot ember a mini sub-language inside a syntax file. May be we can develop something along these lines.

You actually can. See my language_liquid.lua plugin for a reference. The syntax property takes either a string, like ".js", or a table, in which case it treats it as the syntax. You don't need to register it globally.

You actually can. See my language_liquid.lua plugin for a reference (https://raw.githubusercontent.com/franko/lite-plugins/master/plugins/language_liquid.lua). The syntax property takes either a string, like ".js", or a table, in which case it treats it as the syntax. You don't need to register it globally.

Great, I completely missed that this was possible with your modification! I am going to play with this now. You have given some syntax file that use this feature ?

Great, I completely missed that this was possible with your modification! I am going to play with this now. You have given some syntax file that use this feature ?

You should be able to see it just by using the plugin and copying and pasting any liquid file; there's an example of liquid here you can copy and paste: https://getshogun.com/learn/shopify-liquid-tutorial

I like this stuff, if only you didn't use these ugly bit32 operations... :-) ok I am joking!

Yeah, lol. I'll abstract that away a bit. I really think it does make the code cleaner and keeps it lightweight, but I agree it could definitely be more descriptive about what it's doing when it runs them. I'll work on that now.

I may prepare a new release but only with the stuff in the master branch.
Make sure your code who lives there is in good shape and include any small modifications you may want to see in the next release. I have merged the new max-tabs-with-scrolling-buttons thing, wheather @liquidev likes it or not :-)

Yeah, lol. I'll abstract that away a bit. I really think it does make the code cleaner and keeps it lightweight, but I agree it could definitely be more descriptive about what it's doing when it runs them. I'll work on that now.

Don't get mad trying to change the code, I was joking! I mean if the code is okay, it's okay.

Off-topic, but for some reason the dev branch seems to be very unstable at the moment, often crashing when opening the application up. Any news on why that's happening?

I have just fixed a bug that was actually causing some crashes both on master and dev branch. The fix is currently in the master branch and I will merge it into the dev branch.
It was maybe the cause of the crashes you have seen in the dev branch. I think that some of its modifications were revealing the bug which was lurking into the code since a very long time. It was inherited by the rxi's code thought I believe in the original implementation the bug never really happened, it was just a bug-prone design decision. I believe the new approach should be inherently immune from the bug we had.


Warning: Failed to fetch attestation

Not an error but a warning in our logs:
Failed to fetch attestation: Failed to deserialize as Attestation: error decoding response body: attestation missing: attestation missing

Not sure if this is a red flag for something.

This means we tried to fetch an attestation that does not exist yet. Can be a transient error of the oracle. We should probably extend the logs with the event ID.

closing this in favor of https://github.com/itchysats/itchysats/issues/1966


Use workspace inheritance

The new cargo feature "workspace inheritance" allows to define fields like version, authors, homepage, etc... in the root Cargo.toml and applied them to all crates in the workspace.

I would like to try this one 😃


Updating DigitalTwins README for #21105

Packages impacted by this PR

dtdl-parser

Issues associated with this PR

#21105

Describe the problem that is addressed by this PR
Updates the dtdl-parser README based upon comments from #21105
What are the possible designs available to address the problem? If there are more than one possible design, why was the one in this PR chosen?
Made the changes as requested in the issue.
Are there test cases added in this PR? (If not, why?)
Not necessary as updating the README.
Provide a list of related PRs (if any)

#21105

Command used to generate this PR:**(Applicable only to SDK release request PRs)
Checklists

[ ] Added impacted package name to the issue description
[ ] Does this PR needs any fixes in the SDK Generator?** (If so, create an Issue in the Autorest/typescript repository and link it here)
[ ] Added a changelog (if necessary)

API change check for @azure/dtdl-parser
API changes are not detected in this pull request for @azure/dtdl-parser


[Egress provider - S3] Support KMS encryption for artifacts

Summary
AWS offer the ability to use KMS (Key Management Service) encryption keys to encrypt new objects into an S3 bucket. There is more documentation here, but the key benefits of this are:

Centrally create, view, edit, monitor, enable or disable, rotate, and schedule deletion of KMS keys.
Define the policies that control how and by whom KMS keys can be used.
Audit their usage to prove that they are being used correctly.

The security controls in AWS KMS can help you meet encryption-related compliance requirements. You can use these KMS keys to protect your data in Amazon S3 buckets.

As such it would be a useful option given the sensitive data which dotnet monitor exports (e.g. memory dumps) to be able to enable this for dotnet monitor deployments.
For KMS encryption it provides two options:

No specific key provided ("AWS managed key") - AWS will create a KMS key for the bucket and uses that one for future PUT operations. This gives limited control as to what can be done with that key going forwards
A specific key provided ("customer managed key") - the customer creates and manages their own KMS key and has full control over actions relating to it

This PR allows for both options while also allowing users to continue to use the default encryption which AES256 encryption.
It introduces two new configuration options:

useKmsEncryption: A boolean flag which controls whether the Egress should use KMS server side encryption.
kmsEncryptionKey: If UseKmsEncryption is true, this specifies the arn of the "customer managed" KMS encryption key to be used for server side encryption. If no value is set for this field then S3 will use an AWS managed key for KMS encryption.

Documentation for how to specify server side encryption with KMS can be found at https://docs.aws.amazon.com/AmazonS3/latest/userguide/specifying-kms-encryption.html
I could not see a sensible way to unit test these changes as it would require mocking out the S3 client which is not currently possible in the existing test harness and would require a significant refactor to make it possible. However, if you think this is needed I would be happy to follow your guidance

Release Notes Entry
The S3 egress provider now provides the ability to use KMS encryption keys to encrypt artifacts placed in S3 buckets

@schmittjoseph thanks for the speedy review. Just checking, do I need to do anything to make sure this ends up in the release/8.x branch / the dotnet monitor nightly build? 😄

@schmittjoseph thanks for the speedy review. Just checking, do I need to do anything to make sure this ends up in the release/8.x branch / the dotnet monitor nightly build? 😄

Nope! This will automatically be available in the next 8.1-preview nightly build.  As for official builds, it'll be available in both the 8.1 and 9.0 releases once they are available.

Great thanks very much!


pull request for e2e test at 01/14/2017 12:15:12

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


Create MongoDB pymongo backend

You can run scripts/lint and scripts/test  to see what causes the Travis build to fail :wink:

Great stuff, tho we'd certainly want it as a third party package, rather than built-in.
(I think we'll eventually do the same for the existing backends too)

cool, i was thinking about it when i see all the options of ORM`s for SQL and NOSQL databases
I will extract that to an external package
Thanks for the tip


Stalled in Step 7

Issue loading the messages, the request loading them never returns, it stalls...

it happens that reading the friendlychat-android issues I found a similar stalling in Step 7, and it suggests the checking of the rules settings, supposedly in Step 6. In this friendlychat-web codelab, it is not there any reference in Step 6 to rules... changing the settings of the database from private to public solved my issue, allowing me to continue. I'll further read the docs on the rules to make it better than public.

Thank you for the note about changing from private to public. I was having the same issue and this worked for me.


win10 cmake problems

I want warp-ctc to run on windows, so I use windows to compile this. when I run cmake for tensorflow-banding ,I have the following problem. I want to know how to solve it.
My env: py3.5  tf1.10.0.

-- Selecting Windows SDK version  to target Windows 10.0.17134.
-- cuda found TRUE
CMake Warning at CMakeLists.txt:50 (FIND_PACKAGE):
By not providing "FindTorch.cmake" in CMAKE_MODULE_PATH this project has
asked CMake to find a package configuration file provided by "Torch", but
CMake did not find one.
Could not find a package configuration file provided by "Torch" with any of
the following names:
TorchConfig.cmake
torch-config.cmake

Add the installation prefix of "Torch" to CMAKE_PREFIX_PATH or set
"Torch_DIR" to a directory containing one of the above files.  If "Torch"
provides a separate development package or SDK, be sure it has been
installed.
-- Torch found Torch_DIR-NOTFOUND
-- Building shared library with GPU support
CMake Error: The following variables are used in this project, but they are set to NOTFOUND.
Please set them or make sure they are set and tested correctly in the CMake files:
CUDA_curand_LIBRARY (ADVANCED)
linked by target "test_gpu" in directory E:/download/code-20190509T021058Z-001/code/warp-ctc
-- Configuring incomplete, errors occurred!
See also "E:/download/code-20190509T021058Z-001/code/warp-ctc/build/CMakeFiles/CMakeOutput.log".

请问 您解决了吗？

Same error!


Revert "Fix broken tests (#85)" because the PR actually breaks tests

This reverts commit dde4eb0097974e15a82f680260e42a57034701a6.
Closes issue #87

@Vinai your comments make sense. I am assuming that you were testing on a local setup? I was testing against https://hyva-demo.elgentos.io/ since I don't have a local Hyva setup (We are still on Luma for now...) and this suite was set to run again that site.
I had made the (perhaps wrong) assumption that https://hyva-demo.elgentos.io/ was Hyva standard setup running Magento 2.4 with Hyva 1.1.17. Perhaps there has actually been a change to the site that caused the tests to break and instead of changing the tests the site needs to be updated? @peterjaap has there been any recent changes to https://hyva-demo.elgentos.io/. If changes have been made then yeah we should revert my PR and update the site.
I did confirm again that all tests still pass in CI with my version. (See screenshot below) I ran this against my own Cypress Project since I don't think I have permissions to run in this repo.

@zephanhazell I am running the tests against a fresh M2 + Hyvä install in CI in addition to a local installation.
It would indeed be interesting to know in what way https://hyva-demo.elgentos.io/ is different.

@Vinai I put the version numbers on the homepage;

This shop runs on:

Magento 2.4.5 with sample data
Hyvä theme 1.1.17 & Hyvä React Checkout 1.1.1
ElasticSuite 2.10.11 + Hyvä compatibility module 1.1.1
Hypernode Falcon S (PHP 8.1 / MySQL 8.0 / ElasticSearch 7.6 / Varnish 6)

So maybe it has something to do with Elasticsuite?
It's still on my todo list to spin up a vanilla Magento + Hyva instance inside Github Actions but haven't come round to it yet.

I agree Elastic is the likely culprit.

I am closing this issue because as far as I can see, nothing is broken at the moment. Here's the latest run result; https://dashboard.cypress.io/projects/8vuidn/runs/e0a3cf9c-9653-4b03-97cc-04178dd6adb8/test-results/044b4dba-9ee4-43ec-8db0-6fcf12c95df5


Hack to delete feature params for deploying to persistent orgs.

This new flow makes it easy to delete feature parameters from a local repo so that you do not hit the 25 feature param limit in (some?) persistent orgs.

@prescod What do you think we should do with this PR?

I really created it to make things a bit easier for the NPSP team (including broader team like CCE etc.) We could clean it up and merge it or turn it into a Quip doc. You can consider it a POC and make the decision about whether to operationalize it or abandon it...

@prescod If you're ok with my updated flow, let's merge this into master so it's available on any branch.


IAM role for VPC flow logs fails when using template in second region

When you attempt to use this template with the same VPC name in a second region within the same AWS account, the template will fail due to an IAM role having already been created. The IAM role created by this template allows VPC flow logs to create and put logs to a CloudWatch logs group
As a workaround, you can set the VPC name to a unique value within your account (for example. by including the region identifier in the name).  By doing so, the IAM role name will include the region identifier and be unique across your regions.
An alternative approach that is to move the IAM role creation from this template to another template that addresses global AWS resources.  This approach would result in a single IAM role vs a role per region and per VPC.

Fixed IAM and multi-region with same VPC name issue by qualifying IAM role name with region identifier.
As mentioned above, a broader solution would be to move the IAM role creation out of the VPC template so that only a single IAM role be created and maintained.


Messenger

title: Keyboard shortcuts
intro: 'Nearly every page on {% data variables.product.product_name %} has a keyboard shortcut to perform actions faster.'
redirect_from:

/articles/using-keyboard-shortcuts/
/categories/75/articles/
/categories/keyboard-shortcuts/
/articles/keyboard-shortcuts
versions:
free-pro-team: ''
enterprise-server: ''

About keyboard shortcuts
Typing ? in {% data variables.product.product_name %} brings up a dialog box that lists the keyboard shortcuts available for that page. You can use these keyboard shortcuts to perform actions across the site without using your mouse to navigate.
Below is a list of some of the available keyboard shortcuts.
Site wide shortcuts

Keyboard shortcut
Description

s or /
Focus the search bar. For more information, see "About searching on {% data variables.product.company_short %}."

g n
Go to your notifications. For more information, see {% if currentVersion == "free-pro-team@latest" or currentVersion ver_gt<EMAIL_ADDRESS>%}"About notifications{% else %}"About notifications{% endif %}."

esc
When focused on a user, issue, or pull request hovercard, closes the hovercard and refocuses on the element the hovercard is in

Repositories

Keyboard shortcut
Description

g c
Go to the Code tab

g i
Go to the Issues tab. For more information, see "About issues."

g p
Go to the Pull requests tab. For more information, see "About pull requests."{% if currentVersion == "free-pro-team@latest" or currentVersion ver_gt<EMAIL_ADDRESS>%}

g a
Go to the Actions tab. For more information, see "About Actions."{% endif %}

g b
Go to the Projects tab. For more information, see "About project boards."

g w
Go to the Wiki tab. For more information, see "About wikis."

Source code editing

Keyboard shortcut
Description

e
Open source code file in the Edit file tab

control f or command f
Start searching in file editor

control g or command g
Find next

shift control g or shift command g
Find previous

shift control f or command option f
Replace

shift control r or shift command option f
Replace all

alt g
Jump to line

control z or command z
Undo

control y or command y
Redo

cmd + shift + p
Toggles between the Edit file and Preview changes tabs

For more keyboard shortcuts, see the CodeMirror documentation.
Source code browsing
| Keyboard shortcut | Description
Street map messenger location

Why:

What's being changed:

Check off the following:

[x] All of the tests are passing.
[x] I have reviewed my changes in staging.
[x] For content changes, I have reviewed the localization checklist
[x] For content changes, I have reviewed the Content style guide for GitHub Docs.

lint Expected — Waiting for status to be reported
Required
staging Expected — Waiting for status to be reported
Required
test (content) Expected — Waiting for status to be reported
Required
test (meta) Expected — Waiting for status to be reported
Required
test (rendering) Expected — Waiting for status to be reported
Required
test (routing) Expected — Waiting for status to be reported
Required
test (unit) Expected — Waiting for status to be reported
Required street map location messenger


[DO NOT MERGE] Starting work on retrieving meetings from calendar

Work in progress.

@sarangjo great to see this work underway. Please note that much of the sdk is being refactored over the next month and this will need to be implemented very differently by the time that refactor is complete. I hope to have examples of the new layout sometime in the next two weeks.

@udaysrinath Is this still being worked on?

No, I will make another PR for this. Can you please close this PR? @sarangjo is not around, could you close it @ianwremmel ?


Query user to get user id when changing password

Note: To use this new way of updating a user the API version needs to be 4.7.5 or greater.

For https://github.com/guardian/alerta/pull/153


PMACTrajectoryPart needs to insert a point mid way through turnaround

Can use run-up calculations to determine it's position. Need to set all GPIOs to zero during this point

Fixed in #146


Use Github Actions for manual build test

Submisison Checklist

[x] Run tests: ./runCmdStanTests.py src/test
[x] Declare copyright holder and open-source license: see below

Summary:
Fixes #884 by removing Circle CI and replacing it with Github Actions. If this is approved we also need to remove the Circle CI webhook in the repo settings. Otherwise Circle CI test fails as this PR removes the test yml file.
How to review:

check https://github.com/stan-dev/cmdstan/pull/883/commits/6351b31f593a5eccdde8438208719945687c0230 that the Circle CI passed even with error
see that Github actions failed on the same error as it should: https://github.com/stan-dev/cmdstan/runs/726799350
check the Github Actions setup. Its really intuitive, so no experience needed there
see that test pass if error removed and the manual PDF is built on success: https://github.com/stan-dev/cmdstan/actions/runs/121292187

Copyright and Licensing
Please list the copyright holder for the work you are submitting (this will be you or your assignee, such as a university or company): Rok Češnovar, Univ. of Ljubljana
By submitting this pull request, the copyright holder is agreeing to license the submitted work under the following licenses:

Code: BSD 3-clause (https://opensource.org/licenses/BSD-3-Clause)
Documentation: CC-BY 4.0 (https://creativecommons.org/licenses/by/4.0/)

This is ready for review. I updated the top-level comment of this PR with all the info.

@ahartikainen mind looking at this when you have a few minutes. You are my go-to Github Actions expert :)

Btw, you can also upload pdf to release if you want to set up that
https://github.com/marketplace/actions/upload-files-to-a-github-release
- name: Upload build result to release
      uses: svenstaro/upload-release-action@v1-release
      if: success() && startsWith(github.ref, 'refs/tags/v')
      with:
        repo_token: ${{ secrets.GITHUB_TOKEN }}
        file: doc/cmdstan-guide-*.pdf
        tag: ${{ github.ref }}
        overwrite: true
        file_glob: true

Thanks Ari! I think Nic manages the release from Jenkins so lets leave that for now.  But if we move creating releases to G.A. that will definitely be very handy.
@serban-nicusor-toptal can you please approve for @ahartikainen (I dont understand why Ari does not have approve permissions and who can give them to him).
Also Nic, do you have permission to remove Circle CI from cmdstan? Based on this link it should be:
On GitHub, go to repo settings, Webhooks and Services, and delete the CircleCi webhook.

Should be done, let me know if everything is working fine.

Great, will try. Will push a whitespace change to this branch.

Yes, seems that did it. Thanks. Please approve when you have the chance.

@serban-nicusor-toptal you will have to approve also :)
@mitzimorris can you please give Ari permissions to approve here? I am asking him to review CI/Python related stuff quite often and it seems silly to me he cant approve.


False positive error with nested parallelism

Describe the bug
Certain cases of nested parallelism cause false positive validation errors from this library.
I've tracked it down to the JSONPath in nextAndDefaultTargets, $.[Next,Default] for "container" states, not working as I think it's intended to.
To Reproduce
I've added a test case in the associated PR.
Expected behavior
No validation error for the attached test case, which AWS happily accepts.
Version:
2.2.1

Released in v 2.2.2

Also released in v3.0.2


Validators are not under TraceFrame

https://github.com/ApeWorX/evm-trace/blob/ba8eddea539e9fa5dcb9f7d520619000298032ff/evm_trace/base.py#L18-L32

Needs to be indented in one


[Create ride request] 'Search a ride' page must be opened after choosing new filters in previously created ride

Environment: Android Samsung Galaxy A 10s
Reproducible: always
Build found: v1.0.428
Preconditions:

'Car' app is open
User is logged
The main page is open

Steps to reproduce:

Go to 'Search for a ride' page
Put text on text fields:  'From' (e.g. Ternopil) press 'Confirm' button, 'To' (e.g. Krakow, Poland), and press the 'Confirm' button.
Scroll down to 'Departure time' and choose day and time (e.g. December 1, 14;45), press 'Done' button
Tap 'Search' button
In 'Search result' page go to 'Request' on the right top corner on the page
In 'Request a ride' pop-up choose 'With new filters'
Tap the 'Yes, create' button at 'Are you sure' pop-up message

Actual result
The 'Create ride request' page is open
Expected result
"Create a Ride Request" page is open with the Map view
Labels to be added
"Bug" ("bug"), Priority ("pri: ")

https://user-images.githubusercontent.com/91321998/142193096-3aad15dc-a53a-476e-9f25-024528a4237d.mov

Does not correspond to the mock-ups.

Verified


Allow service import to work with Gateways

This is not complete solution and is a bit hacky but is a start
to be able to have service import work at least in some basic
cases.
Also fixed a bug where replySub would not be removed from
connection's list of subs after delivery.
Signed-off-by: Ivan Kozlovic<EMAIL_ADDRESS>/cc @nats-io/core

Coverage remained the same at 91.777% when pulling 75d322f3027ee39cd3c339846e09aec90198bfcd on gw_service_imports into 0bb8562930b89d1f17274f9a65153aead326645f on master.


Refactor show cmd

Show task in a vertical table (instead of horizontal), like the Vim plugin:

KEY
VALUE

DESC
my desc

TAGS
+tag +tag2

...

Added in 8cc71ac:

Feature added in v0.4.0 :tada:


Reference doesn't render in Preview (haven't tested publishing) when > 2 spaces are inserted before it

Works:
  ((ref: [[daily.journal.2020-10-19]]))

Breaks:
    ((ref: [[daily.journal.2020-10-19]]))

Desktop (please complete the following information):

OS: macOS Catalina

this is because >2 spaces is an alternative syntax in markdown to specify a code block

Oh, right. One of my least favorite parts of the spec.
It's not just >2, but >= 4.
I could imagine a scenario where a reference is embedded in a code block, so this might render as a the rendered ref wrapped in a code block. Thoughts?

yep, i stand corrected. >= 4.
as for wrapping a reference in a code block, that would require a bunch of additional parsing logic to pull off.
currently we make sure to not parse links of any kind inside code blocks because we assume the user wants the literal link.
in the future, we'll probably end up customizing the rendering of code blocks to add additional functionality (eg. ipython like code execution within blocks). I think the place to make this change is probably within how we parse code blocks. see example below

This is still a problem with the V2 Preview and ![[]] reference syntax:
    ![[daily.journal.2020-10-19]]


Issues with spec text diff rendering?

Is anyone else having issues with how the spec text is rendering? New sections are not being highlit as green in the rendered version, so I end up having to read this a lot closer.

In general, I don't tend to highlight new sections in green, just changed text in existing sections. Maybe this is a bad convention; @mathiasbynens has been highlighting new sections in his specs; maybe I should switch to that.

I think one of the issues is that this is also out of date with the existing spec.
It is the first time i saw this convention and I found it a bit surprising... but it is also really annoying to annotate everything.


Query profile command with address can't be used

The code here
https://github.com/desmos-labs/desmos/blob/master/x/profile/client/cli/query.go#L33
will make the subcommand become
desmoscli q profile [address

I think the solution is as as simple as changing from [address or moniker] to [address_or_moniker]

I will fixi it today


mirrorlist: google maps API requires an API key

Google Maps returns "The Google Maps API server rejected your request. This service requires an API key.".
According to https://stackoverflow.com/questions/38462525/google-maps-static-api-is-asking-for-api-key, all new servers online after June 22nd 2016 require an API key.
I don't know if adding an API key is feasible or not, due to restrictions from google on number of requested maps per day, or if it would be better to look at implementing a different provider.
(I'm willing to do either in a PR)

I'm sure adding a configuration field for the key should be fine.

Exactly, adding the API key to the configuration file should be the way to go. In the long term, I'm in favor of supporting other maps backend like OSS but that's not very high priority.


AnimatedContainer on beta doesn't animate

A simple width change of an AnimatedContainer doesn't seem to run on the latest version of the beta channel of flutter.
Using the simple example of:
Column(
  mainAxisAlignment: MainAxisAlignment.spaceAround,
  children: <Widget>[
    RaisedButton (
      child: Text('Animate!'),
      onPressed: () {
        setState(() {
          w = 250.0;
        });
      },
    ),
    AnimatedContainer (
      duration: Duration (seconds: 1),
      width: w,
      height: 40,
      color: Colors.red,
    ),
  ],
)

Upon pressing the button the value of the width will change (seen by printing it), however the animation will not occur. This does not happen when running on Flutter (Channel master, v1.3.11-pre.39, on Mac OS X 10.14.3 18D109, locale de-DE).
Flutter Doctor
Doctor summary (to see all details, run flutter doctor -v):
[✓] Flutter (Channel beta, v1.2.1, on Mac OS X 10.14 18A391, locale en-GB)
[✓] Android toolchain - develop for Android devices (Android SDK version 28.0.3)
[✓] iOS toolchain - develop for iOS devices (Xcode 10.1)
[✓] Android Studio (version 3.3)
[✓] Connected device (1 available)

• No issues found!

An addition to this is that it does not work on beta v1.3.8 either
Flutter Doctor
[✓] Flutter (Channel beta, v1.3.8, on Mac OS X 10.14 18A391, locale en-GB)
• Flutter version 1.3.8 at /Users/rhysduggan/flutter
• Framework revision e5b1ed7a7f (2 weeks ago), 2019-03-06 14:23:37 -0800
• Engine revision f4951df193
• Dart version 2.2.1 (build 2.2.1-dev.0.0 571ea80e11)
[✓] Android toolchain - develop for Android devices (Android SDK version 28.0.3)
• Android SDK at /Users/rhysduggan/Library/Android/sdk
• Android NDK location not configured (optional; useful for native profiling support)
• Platform android-28, build-tools 28.0.3
• Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
• Java version OpenJDK Runtime Environment (build 1.8.0_152-release-1248-b01)
• All Android licenses accepted.
[✓] iOS toolchain - develop for iOS devices (Xcode 10.1)
• Xcode at /Applications/Xcode.app/Contents/Developer
• Xcode 10.1, Build version 10B61
• ios-deploy 1.9.4
• CocoaPods version 1.5.3
[✓] Android Studio (version 3.3)
• Android Studio at /Applications/Android Studio.app/Contents
• Flutter plugin version 33.4.1
• Dart plugin version 182.5215
• Java version OpenJDK Runtime Environment (build 1.8.0_152-release-1248-b01)
[✓] Connected device (2 available)
• Shrek's Swamp • f88a459b4e691bbc686e6e1d4c8b9d3a1817285d • ios • iOS 12.0
• iPhone XS Max • F9EC2B7B-C5B7-49D1-9E08-2F26C302A14B     • ios • iOS 12.1 (simulator)
• No issues found!

Hi @shaydoth,
Thank you for filing this issue. Can you include a complete runnable sample of your code?
Here I have made the code you provided runnable, but I have not been able to reproduce your issue:
import 'package:flutter/material.dart';

void main() => runApp(MyApp());

class MyApp extends StatelessWidget {
  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      title: 'Issue Demo',
      home: MyHomePage(),
    );
  }
}

class MyHomePage extends StatefulWidget {
  MyHomePage({Key key}) : super(key: key);
  @override
  _MyHomePageState createState() => _MyHomePageState();
}

class _MyHomePageState extends State<MyHomePage> {
  double w = 10.0;

  @override
  Widget build(BuildContext context) {
    return Scaffold(
      appBar: AppBar(title: Text('Issue Demo')),
      body: Center(
        child: Column(
          mainAxisAlignment: MainAxisAlignment.spaceEvenly,
          children: <Widget>[
            RaisedButton(
              child: Text('Animate!'),
              onPressed: () {
                setState(() {
                  w = w == 10.0 ? 250.0 : 10.0;
                });
              },
            ),
            AnimatedContainer(
              duration: Duration(seconds: 1),
              width: w,
              height: 40,
              color: Colors.red,
            ),
          ],
        ),
      ),
    );
  }
}

If your code is different and you are still experiencing this issue, please share it so we can better identify the issue. Thank you!

The no-response bot would have closed this by now. If you would like to return and provide more information to address your issue, please feel free to re-open. :)


feat: Adding Secure Web Proxy to BigQuery example

Hey folks.
This PR adds the Secure Web Proxy (SWP/SWG) to the BigQuery example.

Still need to update the source for secure_web_proxy module.
This update will be done as soon as the module is merged:
https://github.com/GoogleCloudPlatform/terraform-google-cloud-functions/pull/34

/gcbrun

/gcbrun

/gcbrun

/gcbrun

/gcbrun

/gcbrun

/gcbrun

/gcbrun

/gcbrun

/gcbrun


Breakage after gdext multi-thread sanitizing.

Test project can't be opened from editor (or exported), producing error:
$ godot--headless --path tests/godot --export-debug Linux/Headless
Initialize godot-rust (API v4.2.stable.official, runtime v4.2.stable.official)
Godot Engine v4.2.stable.official.46dc27791 - https://godotengine.org

Registering saver!
Registering loader!
WARNING: Custom cursor shape not supported by this display server.
     at: cursor_set_custom_image (servers/display_server.cpp:505)
Entered tree!
Added plugin!
WARNING: Instantiated WithBundledGd used as default value for ExportTestNode's "bundle_res" property.
     at: class_get_default_property_value (core/object/class_db.cpp:1743)
WARNING: Instantiated WithExtGd used as default value for ExportTestNode's "ext_res" property.
     at: class_get_default_property_value (core/object/class_db.cpp:1743)
WARNING: Instantiated TestResource used as default value for WithBundledGd's "first" property.
     at: class_get_default_property_value (core/object/class_db.cpp:1743)
WARNING: Instantiated TestResource used as default value for WithBundledGd's "second" property.
     at: class_get_default_property_value (core/object/class_db.cpp:1743)
WARNING: Instantiated TestResource used as default value for WithExtGd's "first" property.
     at: class_get_default_property_value (core/object/class_db.cpp:1743)
thread '<unnamed>' panicked at 'assertion failed: `(left == right)`
  left: `ThreadId(1)`,
 right: `ThreadId(2)`: attempted to access binding from different thread than main thread; this is UB - use the "experimental-threads" feature.', /home/kosin/.cargo/git/checkouts/gdext-76630c89719e160c/bfc3a98/godot-ffi/src/binding/single_threaded.rs:74:13

As no multithreading is used explicitly, I'm trying to pinpoint the cause. Have to be something with EditorPlugin or EditorExportPlugin, as only these are called in-editor - #[gditest] tests on Scene init level are running without error.
Probably caused by https://github.com/godot-rust/gdext/pull/581

Issue isn't really possible to circumvent - Godot uses ResourceFormatLoader on Editor startup with additional, spawned threads. For now the only fix is to use the gdext with feature experimental-threads enabled in gdext-dependent gdextensions introducing custom ResourceFormatLoader - so the one using gd-props must have it with the same feature.
Better described in there gdext issue. Upstream could introduce some special cases with thread safety - if so, it will be made clear in linked issue.


Match property not working correctly on windows

Describe the issue that you're seeing. Any Loom videos or screenshots usually help a lot!
Collection structure:
content/page1.md
content/page2.md
content/posts/post1.md
content/posts/post2.md
content/authors/author1.md
content/authors/author2.md

And collections defined as:
collections: [
{
  name: "pages",
  path: "content",
  match: {
    include: "*",
  }
},
{
  name: "posts",
  path: "content/posts",
  match: {
    include: "**/*",
  }
},
{
  name: "authors",
  name: "content/authors",
  match: {
    include: "**/*",
  }
}

When you try to open any item in any of these collections when running in local dev on windows, it gives an error like: [Error] GetDeocument failed: Unable to fetch, errors: Unable to find collection for file at content\posts\post1.md
Based on the error message it looks like it is probably just an issue with the directory separators.
Reproduction
na
Steps to reproduce
No response
System Info
windows 11

Validations

[X] Follow our Code of Conduct
[X] Read the docs.
[X] Check that there isn't already an issue that reports the same bug.

This is happening in Windows 10 as well after upgrade from 1.5.6 to 1.5.20

This is the output in the terminal
GraphQL request:3:11 2 |         query($collection: String!, $relativePath: String!) { 3 |           document(collection:$collection, relativePath:$relativePath) {   |           ^ 4 |             ... on Document { Unable to find collection for file at exampleSite\content\english\author\john-doe.md


Feature: Image test

Type of feature
🍕 Feature
Current behavior
N/A
Suggested solution
N/A
Additional context
No response
Code of Conduct

[X] I agree to follow this project's Code of Conduct

Contributing Docs

[X] I agree to follow this project's Contribution Docs

Hey, @isabensusan Hope you're doing well. Can you explain the feature you need?

hey @Himanshuch8055 !! This issue was only a quick test to try out Github's image embeds on issues. I'll close it now, so pls ignore! :)

ok....  :)


[Youtube] Having issues with videos downloaded from youtube movies.

Checklist

[x ] I'm reporting a broken site support
[x ] I've verified that I'm running youtube-dl version 2019.05.20
[x] I've checked that all provided URLs are alive and playable in a browser
[x ] I've checked that all URLs and arguments with special characters are properly quoted or escaped
[x ] I've searched the bugtracker for similar issues including closed ones

Verbose log

❯ youtube-dl --verbose s7_qI6_mIXc
[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: [u'--verbose', u's7_qI6_mIXc']
[debug] Encodings: locale UTF-8, fs utf-8, out UTF-8, pref UTF-8
[debug] youtube-dl version 2019.05.20
[debug] Python version 2.7.10 (CPython) - Darwin-18.6.0-x86_64-i386-64bit
[debug] exe versions: ffmpeg 4.1.3, ffprobe 4.1.3, rtmpdump 2.4
[debug] Proxy map: {}
[youtube] s7_qI6_mIXc: Downloading webpage
[youtube] s7_qI6_mIXc: Downloading video info webpage
[youtube] s7_qI6_mIXc: Downloading MPD manifest
WARNING: [youtube] s7_qI6_mIXc: Skipping DASH manifest: ExtractorError(u'Failed to download MPD manifest: HTTP Error 404: Not Found (caused by HTTPError()); please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.',)
[debug] Default format spec: bestvideo+bestaudio/best
[debug] Invoking downloader on u'https://r2---sn-o097znlr.googlevideo.com/videoplayback?id=d6e5a8a51c0e5e16&aitags=142%2C143%2C144%2C145%2C146%2C161%2C222%2C223%2C224%2C225%2C226%2C227%2C273%2C274%2C275%2C279%2C280%2C317%2C318%2C357%2C358%2C359%2C360&itag=227&source=yt_media&requiressl=yes&mm=31%2C29&mn=sn-o097znlr%2Csn-n4v7sn76&ms=au%2Crdu&mv=m&pl=40&ei=xF3jXNX7DtLukgbVhL6QBQ&gcr=us&initcwndbps=2243750&mime=video%2Fmp4&aid=c6fd0e1d17810c44&asource=youtube&gir=yes&clen=5681871035&dur=5748.033&lmt=1548769415160173&mt=1558404466&fvip=1&keepalive=yes&c=WEB&ip=2603%3A3024%3A1a4e%3A0%3Ab060%3A6705%3A8e9c%3A2298&ipbits=0&expire=1558426148&sparams=ip%2Cipbits%2Cexpire%2Cid%2Caitags%2Csource%2Crequiressl%2Cmm%2Cmn%2Cms%2Cmv%2Cpl%2Cei%2Cgcr%2Cinitcwndbps%2Cmime%2Caid%2Casource%2Cgir%2Cclen%2Cdur%2Clmt&signature=C7525CD22B372EC488A55F7D455BCD9239CAD2AF.91291E016979BAC4C0B2FE0755369222FF8F5B18&key=yt8&ratebypass=yes'
[download] Resuming download at byte 523264
[download] Destination: In The Mix-s7_qI6_mIXc.f227.mp4
[download]   0.2% of 5.29GiB at  4.30MiB/s ETA 20:58^C

Description

Trying to download a movie to use for a school project, and the video failed. so i deleted and tried with another movie, with VLC this time, and the video just didnt play, apart from some audio, at the beginning. No credentials needed. As its the terminator movie, free on YouTube.  (https://www.youtube.com/watch?v=s7_qI6_mIXc). Unsure of a suggestion on how to fix DASH manifest: ExtractorError stuff

I just noticed this too today with several of them from https://www.youtube.com/playlist?list=PLHPTxTxtC0ibVZrT2_WKWUl2SAxsKuKwx. I think it is related to "manifest: HTTP Error 404: Not Found (caused by HTTPError())" error as shown below:
$ youtube-dl -v  https://www.youtube.com/watch?v=jlb6Qrvgt0Y
[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: ['-v', 'https://www.youtube.com/watch?v=jlb6Qrvgt0Y']
[debug] Encodings: locale UTF-8, fs utf-8, out UTF-8, pref UTF-8
[debug] youtube-dl version 2019.05.20
[debug] Python version 3.4.2 (CPython) - Linux-3.16.0-8-amd64-x86_64-with-debian-8.11
[debug] exe versions: ffmpeg 3.2.10-1, ffprobe 3.2.10-1, rtmpdump 2.4
[debug] Proxy map: {}
[youtube] jlb6Qrvgt0Y: Downloading webpage
[youtube] jlb6Qrvgt0Y: Downloading video info webpage
[youtube] jlb6Qrvgt0Y: Downloading MPD manifest
WARNING: [youtube] jlb6Qrvgt0Y: Skipping DASH manifest: ExtractorError('Failed to download MPD manifest: HTTP Error 404: Not Found (caused by HTTPError()); please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.',)
[debug] Default format spec: bestvideo+bestaudio/best
[debug] Invoking downloader on 'https://r4---sn-a5meknlz.googlevideo.com/videoplayback?id=4d2eacfd70b5b81d&aitags=142%2C143%2C144%2C145%2C146%2C161%2C222%2C223%2C224%2C225%2C226%2C227%2C273%2C274%2C275%2C279%2C280%2C317%2C318%2C357%2C358%2C359%2C360&itag=227&source=yt_media&requiressl=yes&mm=31%2C26&mn=sn-a5meknlz%2Csn-q4flrnek&ms=au%2Conr&mv=m&pl=14&ei=QanpXKCBGJeCkgbSxpTwAQ&gcr=us&initcwndbps=1766250&mime=video%2Fmp4&aid=8e56fa42bbe0b746&asource=youtube&gir=yes&clen=3476185375&dur=4745.578&lmt=1549681906065708&mt=1558816811&fvip=4&keepalive=yes&c=WEB&ip=<IP_ADDRESS>&ipbits=0&expire=1558838689&sparams=ip%2Cipbits%2Cexpire%2Cid%2Caitags%2Csource%2Crequiressl%2Cmm%2Cmn%2Cms%2Cmv%2Cpl%2Cei%2Cgcr%2Cinitcwndbps%2Cmime%2Caid%2Casource%2Cgir%2Cclen%2Cdur%2Clmt&signature=D9138CD810BC0C8AE9CA9AEFB1671B9DB65132F7.31C4397908C1D29E41AB588A9F86074F9FE35AED&key=yt8&ratebypass=yes'
[download] Destination: Code - Debugging the Gender Gap-jlb6Qrvgt0Y.f227.mp4
[download]   3.0% of 3.24GiB at  9.93MiB/s ETA 05:23


FIXED ALL MINOR ISSUES

FIXED ISSUE:  #234
IMROVED FONT VISIBILITY
ADDED NEW BACKGROUND IMAGES

Fixes #234


livecoding(LiveEdu), hitbox, beam etc

Pls add support for other platforms especially for livecoding(LiveEdu)
thanks in advance

This project is suspended for now, don't think that I will add new features in the near future


[DB Designer] -> [Online Design Tools]

[DB Dessigner]
Free Database Designer online
Link: https://app.dbdesigner.net/
Checklist:

[x] I have performed a self-review of submitted resource and its follows the guidelines of the project.

I don't know if only frontend design resources are accepted.
But if you are a full stack developer this tool can be helpfull.


First cluster implementation

Goal
The goal of this change is to add high availability for bosun service. Currently it’s possible to have only one active bosun node running checks and sending alert. If this node goes down or becomes unavailable there’s a high chance that checks won’t run for some time, alerts are not fired, incidents aren’t created and notifications — delivered. It then take some time for a sysadmin to switch all the checks to another node. So in the end we’re minimising human intervention needed in failure scenarios and reducing possible downtime of service.
Changes overview
We implemented cluster for bosun.
Cluster would only have one ‘leader’ at a time, all other nodes are followers (so this is an implementation of a model with with 1 master and multiple standby nodes).
‘Master’ node executes the checks and sends notifications, ‘follower’ nodes don’t do neither (they run with ‘no-checks’ and ‘quiet-mode’ options enabled). This also adds a new (optional) dependency raftdb to store state and perform leader election.
Configuration
Config example:
#node 1
[ClusterConf]
    	MetadataStorePath = "/var/db/bosun_raft"
    	RPCListen = "<IP_ADDRESS>:10000"

#node 2
[ClusterConf]
    	MetadataStorePath = "/var/db/bosun_raft"
    	RPCListen = "<IP_ADDRESS>:10000"
    	Members = ["<IP_ADDRESS>:10000"]

#node 3
[ClusterConf]
    	MetadataStorePath = "/var/db/bosun_raft"
    	RPCListen = "<IP_ADDRESS>:10000"
    	Members = ["<IP_ADDRESS>:10000"]
# or
# Members = ["<IP_ADDRESS>:10000", "<IP_ADDRESS>:10000", "<IP_ADDRESS>:10000"]

MetadataStorePath - is a path to store cluster data e.g. raft database and snapshots (snapshots are empty all time in current implementation). If this path doesn’t exist we try to create it automatically
RPCListen - host and port for serf listener (https://github.com/hashicorp/serf). Host should be the address that will be available for other nodes of the cluster (so shouldn’t be loopback address!). Port will be a primary port for cluster. There will also be another opened extra port for raft protocol (https://github.com/hashicorp/raft). Raft port will be RPCListen + 1.

Additionally an api endpoint was implemented to expose statistics and current cluster status:
# curl http://<IP_ADDRESS>:8070/api/cluster/status
{
  "State": 2,
  "StateString": "Leader",
  "Nodes": [
	{
  	"Address": "<IP_ADDRESS>:10022",
  	"ID": "<IP_ADDRESS>:10022",
  	"State": "Follower"
	},
	{
  	"Address": "<IP_ADDRESS>:10012",
  	"ID": "<IP_ADDRESS>:10012",
  	"State": "Leader"
	}
  ],
  "Stats": {
	"applied_index": "6",
	"commit_index": "6",
	"fsm_pending": "0",
	"last_contact": "0",
	"last_log_index": "6",
	"last_log_term": "4",
	"last_snapshot_index": "0",
	"last_snapshot_term": "0",
	"latest_configuration": "[{Suffrage:Voter ID:<IP_ADDRESS>:10022 Address:<IP_ADDRESS>:10022} {Suffrage:Voter ID:<IP_ADDRESS>:10012 Address:<IP_ADDRESS>:10012}]",
	"latest_configuration_index": "6",
	"num_peers": "1",
	"protocol_version": "3",
	"protocol_version_max": "3",
	"protocol_version_min": "0",
	"snapshot_version_max": "1",
	"snapshot_version_min": "0",
	"state": "Leader",
	"term": "4"
  }
}

I love the concept, and I'm excited to try it out. A few questions before I dive in:

Does this do anything to sync alert rules and config between nodes? That could be a nice upgrade if possible.
Is there any way to specify node priorities? We have two nodes currently in different data centers, and will almost always want the primary dc node to be the master.
I assume this is like most other raft implementations in terms of consensus requirements. So we'd need 3 nodes minimum to be able to do any automated failover? Is there any tunability to the quorom requirements and so forth?

Thanks for questions! Current realisation is just very basic first step. It means that we will not sync any data between nodes yet. Basically we are managing alert rules and config file by configuration management (puppet), and it makes configs equal. Manage of system config from application (like sync from another nodes) can be bad idea, because users can define for slaves different endpoints as instance. I also don't see normal possibility to sync alert rules between nodes with current implementation of those rules. Now we are managing rules via config file. I don't think that rewrite some config files from application is a good idea, because it can make management of configs very difficult (like it was made in redis sentinel - it is really very difficult to manage config like that from chef/ansible/puppet). May be we will think about moving rules to some db storage then?
About priorities of nodes it can be difficult, because raft algorithm - https://github.com/hashicorp/raft/issues/211. But there is exists 2 different models - wan and lan. I think for different datacenters it could work. I will take a look on this.
It is indeed just another implementation of raft from hashicorp. Yes, for automated failover we should have at least 3 nodes. With another way it is possible to split brain:) Here is some description of this (from authors of library) - https://www.consul.io/docs/internals/consensus.html.

Hey @kylebrandt @captncraig !
Any objections to merging this?

Hi @captncraig ! Is there some problem with this MR?:( We really need this, because currently it is possible to have only one node without any possibility of HA...

Hey, @captncraig , @kylebrandt !
Any objections to this?
Manual failover is quite inconvenient and does increase time to recover :(

Hey @svagner ! I've just checked and it seems this has conflicts now. Shall we resolve them?

New implementation is in https://github.com/bosun-monitor/bosun/pull/2472


Fixed gravatar request in HTTPS

Now the app queries the HTTPS endpoint of gravatar when HTTPS is enabled.
Closes #711.

@jfresco I'm not sure how to test it. Does mongoose-gravatar takes care of all the other cases?
Also, this brakes the Facebook registration:
/app/lib/registration/index.js:
   21    user.email = profile.emails[0].value;
   22    user.emailValidated = true;
   23:   user.avatar = profile.imageUrl;
```

Fixed @mjlescano.


Manually revert #90017

This is a revert of #90017, reviewers please note this is manual not automatic.
Will nominate for cherry pick.
Internal golden file changes are expected.
Fixes https://github.com/flutter/flutter/issues/95270
Fixes https://github.com/flutter/flutter/issues/96087
Fixes https://github.com/flutter/flutter/issues/94906
Re-opens https://github.com/flutter/flutter/issues/89255
Re-opens https://github.com/flutter/flutter/issues/71008
FYI @yjbanov & @dnfield
Pre-launch Checklist

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

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

Gold has detected about 1 new digest(s) on patchset 2.
View them at https://flutter-gold.skia.org/cl/github/96246

Also, if at all possible, can we add some test(s) that would avoid the regressions found?

Also, if at all possible, can we add some test(s) that would avoid the regressions found?

The reported issues have good test cases to include when this is fixed or reverted. 👍

Gold has detected about 1 new digest(s) on patchset 3.
View them at https://flutter-gold.skia.org/cl/github/96246

I am going to close this since it will not be able to cherrypick a fix into the latest stable release 2.8.

Fix forward is in https://github.com/flutter/flutter/pull/96295


First program linking issues

Hi,
I am following the Copper book, and I am currently having problems with the very first program (http://japaric.github.io/copper/first/build.html).
The program compiles, but the objdump output is the following:
$arm-none-eabi-objdump -CD target/thumbv7m-none-eabi/debug/app

target/thumbv7m-none-eabi/debug/app:     file format elf32-littlearm

Disassembly of section .text:

00000000 <_reset-0x8>:
   0:	20010000 	andcs	r0, r1, r0
   4:	00000009 	andeq	r0, r0, r9

For some reason the main function is optimized away at some point.
If I make rustc emit the assembly file, I can see the function is there, with the right symbol (_reset), so I guess that the problem comes while linking.

@yuri91 are you using a mac? Does the gcc you are using come from the gcc-arm-embedded tap/cask? Someone reported (japaric/discovery#34) that that gcc has some bug like the one you reported here. Could you try the gcc from the px4/px4 tap? (Sorry, I don't have the exact instructions to do that as I don't access to a mac)

I am not using a mac, I am using arch linux. The version of arm-none-eabi-gcc that I had installed from the community repo was 6.3.
I looked the version currently shipped with Ubuntu, and it is 4.8 or 4.9.
So I tried the 4.9 myself, and It works.
Thank you very much for your help, I didn't think it could be an actual bug in the linker.
Also, I looked into the other two related issues (japaric/discovery#34 and japaric/f3/issues/75), and I think that they have a 6.x version of GCC too.

I'm using Arch as well with gcc 6.3 but I can't repro your problem. I get:
$ arm-none-eabi-objdump -Cd target/thumbv7m-none-eabi/debug/bar

target/thumbv7m-none-eabi/debug/bar:     file format elf32-littlearm

Disassembly of section .text:

00000000 <_reset-0x8>:
   0:   20010000        .word   0x20010000
   4:   00000009        .word   0x00000009

00000008 <_reset>:
   8:   b083            sub     sp, #12
   a:   e7ff            b.n     c <_reset+0x4>
   c:   202a            movs    r0, #42 ; 0x2a
   e:   9001            str     r0, [sp, #4]
  10:   9000            str     r0, [sp, #0]
  12:   e7ff            b.n     14 <_reset+0xc>
  14:   e7fe            b.n     14 <_reset+0xc>

Does changing the loop in main to
    loop {
        unsafe {
            ::core::ptr::read_volatile(0x0 as *const u32);
        }
    }

makes the linker preserve the _reset symbol? With gcc 6.3 I mean. There's a LLVM bug when you have loops without side effects but that's supposed to only occur when optimizations (+LTO) are enabled. Which is not the case here but just to discard the possibility.

Ok, my bad:
I was actually using yet another version of GCC (6.2.1) from the AUR.
Back when I installed that package (probably a year ago or so) there wasn't the one in [community] yet.
So the actual package in [community] with GCC 6.3 works, like the 4.9 one.

Good to know it's solved. I'll add a note to the book suggesting trying a different linker if you get a binary that's too different from what's in the book.

Hi,
I had the same issue with gcc 4.9. It seems to come from the fact that the entry point has the same name as the variable _reset you define in the ld script. here is 2 ways to fix it:

use KEEP(*(.text._reset))
or a better alternative
rename your entry point to _start or something else (both in the ld script and the main.rs) or rename the _reset variable in the ld script

Hope it helps

I am also experiencing this issue on arch. I'm using

arm-none-eabi-gcc from community ( 6.3.1 20170330)
xargo 0.3.6
rustc 1.18.0-nightly (7627e3d31 2017-04-16)

and compiling in 02-hardware with xargo build --target thumbv7em-none-eabihf --verbose which produces this binary: https://oolong.tahnok.me/cdn/app
I cloned this repo, so the code should be correct. Tried changing the entry point and changing the loop to that unsafe read_volatile, but same output.

The copper book is pretty outdated at this point and won't have time to update in the short term. You are better off starting with this blog post.


Update bindings to work with panic-less marlin

This PR updates the bindings to support the marlin PR o1-labs/proof-systems#132. That PR modifies some functions to avoid panics lower down the stack, which allows WASM to surface these errors to javascript (panics are 'aborts' in WASM, so the process just crashes otherwise).
This PR uses unwrap() within the bindings themselves, which will be caught by ocaml-rs and surfaced as an OCaml exception. This matches the old behaviour; this PR should be a no-op.
Checklist:

[ ] Document code purpose, how to use it

Mention expected invariants, implicit constraints

[ ] Tests were added for the new behavior

Document test purpose, significance of failures
Test names should reflect their purpose

[ ] All tests pass (CI will check this if you didn't)
[ ] Serialized types are in stable-versioned modules
[ ] Does this close issues? List them:

The work from the submodule needs to be redone. Closing.


Allow gem to work with MongoDB on Heroku

I'm under the impression that this gem won't work with MongoDB on Heroku: https://github.com/lgromanowski/letsencrypt-plugin/issues/39 because Heroku is a read-only file system and this either requires the challenge directory or ActiveRecord?
Related because I can't create files on Heroku - (or unrelated, I'm not sure?) I can't figure out how to make the private key available on Heroku without checking it into the source code. I don't think checking it into the source code is the right way to go, but we're an open source app, so...

Hi,
on Heroku you should use DB as a challenge storage (challenge_dir_name should be commented in config file) and after that please run rake letsencrypt_plugin task on heroku - it should connect with letsencrypt servers and print certificates on console.

If you outline how and where to look and make this work and what kind of tests to write for this I would gladly write a PR.

Code for challenge is placed in Challenge Model: https://github.com/lgromanowski/letsencrypt-plugin/blob/master/app/models/letsencrypt_plugin/challenge.rb:
module LetsencryptPlugin
  # if the project doesn't use ActiveRecord, we assume the challenge will
  # be stored in the filesystem
  if LetsencryptPlugin.config.challenge_dir_name.blank? && defined?(ActiveRecord::Base) == 'constant' && ActiveRecord::Base.class == Class
    class Challenge < ActiveRecord::Base
    end
  else
    class Challenge
      attr_accessor :response

      def initialize
        full_challenge_dir = File.join(Rails.root, LetsencryptPlugin.config.challenge_dir_name, 'challenge')
        @response = IO.read(full_challenge_dir)
      end
    end
  end
end

Place which should be changed is Challenge::initialize() method in "else" branch - at this moment that code tries to read challenge from a file.

Cool, thanks!


ActionView::Template::Error in single_events#show

Exception in Hacken.in
ActionView::Template::Error in single_events#show
undefined method `color' for nil:NilClass
View on Bugsnag
Stacktrace
app/views/modules/categories/_indicator.html.haml:1 - _app_views_modules_categories__indicator_html_haml__2871570490134160349_86693600
app/views/single_events/show.html.haml:15 - _app_views_single_events_show_html_haml___862817483225247207_87524460
app/controllers/single_events_controller.rb:11 - show

View full stacktrace

Looks like we expect every Category to have a colour while this isn't always the case:
.indicator{style: "background-color: #{category.color};", title: category.title}

I think that I coded that a while ago. The question is: Why is this assumption not valid? We should probably enforce this. What do you think?

@moonglum Yep – I already thought about adding a colour / color wrapper to the Category model, but I don't know the nuts and bolts of Category / SingleEvents. :)


texLive: xelatex should depend on lmodern fonts

I've encountered this error: https://tex.stackexchange.com/questions/652/xetex-not-working-on-ubuntu-10-04
I don't have time to dig deeper into this now, but it looks like lmodern fonts are needed for xelatex's fontspec to work. I've triggered this bug with just
\documentclass{article}

\usepackage{polyglossia}

#287 shows no error on your input with just texlive.combine { inherit (texlive) scheme-basic xetex; };

Hmm, actually it does produce problems when adding \begin{document} \end{document, as polyglossia isn't found. Adding just that one isn't enough, too. There do seem to be some underspecified dependencies, but I just took those from upstream... I know Fedora generator does try some automatic dependency detection when building the binary packages.

So, do you know what extra TeX Live packages are needed for xelatex? I can add them to our overrides so that texlive.combine always includes them. I don't see what better to do right now about it.

Sorry, I've forgotten about this issue.
With this;
  inherit (pkgs.texlive) scheme-basic xetex polyglossia euenc xunicode xetex-def;

The document builds fine, so polyglossia needs euenc but I'm unsure if polyglossia or euenc needs xunicode, same with xunicode for xetex-def.

(triage) @abbradar any news? I recall @vcunat restructured the texlive packages, if you were using the old ones above maybe the problem is solved now.

I can add several dependencies which we've more or less determined (polyglossia -> euenc, euenc -> xunicode) and close this later.

@abbradar Are you still intending on adding those dependencies?

Ough, unfortunately I have several very tight deadlines approaching. Thank you for reminding (I forgot!) but I won't be able to look into this till 10/08...

Ping @abbradar

Right.
Oh, I haven't seen you here for a very long time :-)


Move the project editor repository

This is being done in conjunction with moving nunit-gui. See nunit/nunit-gui#204.

Isn't this project entirely separate from the test runner GUI? I don't see it in nunit/nunit-gui#204.

Yes. But it's one of the gui projects addressed in the governance issue. I started with "projects" and talked about the gui team, which covers this project as well, but when discussing the details we focused on nunit-gui as being most important.
It's not covered in nunit/nunit-gui#204 that's a separate repository, although the same team is involved. As you know from experience, I'm fanatic about sticking to the topic in a given repo or issue. 😄

Steps to get the project editor up and running

[ ] Use GitHub to move the repository under my personal account. GitHub will forward references.

[ ] Invite @nunit/gui-team members who want to continue working on this as collaborators.

[ ] Move wiki pages and create links as needed to and from the NUnit project.

[ ] Update the ci builds so they work under the new account.

[ ] Figure out how to handle the MyGet feeds.

This is working. THere will be future changes, of course, but I'm closing this.


Remove underscores from module name (i.e. cstar_ocean -> cstar)

[x] Closes #7

Superceded by #47


Signatures / MACs: AAD generation missing?

Looking to fix the big TODO around _verify_signature (which currently returns successfully for all cases), I'm lacking the AAD context to verify these hashes.
What's the plan around that function? Is the expectation that it has enough context to build the relevant plaintext/AAD itself, or should it grow an argument and have the callers provide that?

Having worked a bit on an actual implementation (with which I can verify the MAC in message3), I found that while of peers there's always full information (ID_CRED_mine, CRED_mine, my private key), of the peer there's only ID_CRED_peer and the public key of the peer, but not the consensus encoding of the CRED_peer (which typically takes the form {1: 1, -1: 4, -2: key, "subject name": something}). Are there any plans where that should be transported yet?
CC @TimothyClaeys as I'm not sure you're subscribed here.

Hm, seems ther is a distincguished place for those (_peer_cred, _remote_authkey), just that there's some paths through _parse_credentials that make them identical.
On the plus side, this is getting into a PR'able shape slowly, maybe we can talk more precisely when reviewing that.

I do now have working signature_or_mac_3 verification (both versions); PR probably makes most sense after #9 is processed and my stuff is rebased onto there.

Okay, I think you can PR your changes. in the meantime I'll add some additional test vectors from test-vectors-05.


cri-o: 1.18.3 -> 1.19.0

Motivation for this change
Update CRI-O to the latest version.
cc @NixOS/podman
Things done

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

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

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

@GrahamcOfBorg build cri-o

I think we can remove sed -i '/version.buildDate/d' Makefile and socat from the wrapper?

I think we can remove sed -i '/version.buildDate/d' Makefile and socat from the wrapper?

Yes, we can also remove the patchShebangs as well as the deprecated manage_ns_lifecycle option from the module :innocent:

@GrahamcOfBorg build cri-o


1 setting up GitHub actions

in python-app.yml of .github/workflows dir:

removed original manual action yml file to create Jira tickets;
add dvc install
add src/tests dir for unit tests path (have not been created yet)

only small changes as described in python-app.yml action file,
removing original manual.ymal file for jira tickets

...


Fixes for multi-level discriminators

Fixes a few bugs for multi-level discriminator structures:

Types that change the polymorphic discriminator generate getters for all discriminators.
Enum discriminators now use the right type when there are multiple levels of discriminators.

Thanks Alan!


feat: add radio into components #26

Alaska Airlines Pull Request
close #26
adding<EMAIL_ADDRESS>to auro-formkit
By submitting this Pull Request, I confirm that my contribution is made under the terms of the Apache 2.0 license and I have performed a self-review of my own update.
Summary by Sourcery
Add the auro-radio component to the project, complete with build scripts, documentation, and tests, enhancing the form kit with radio button functionality.
New Features:

Introduce the auro-radio component, a custom HTML element for creating radio buttons with associated labels, supporting accessibility standards.

Enhancements:

Add support for the focus-visible library to enhance accessibility by managing focus states.

Build:

Update build scripts to include tasks for the new auro-radio component, such as building, testing, and generating documentation.

Documentation:

Add comprehensive documentation for the auro-radio component, including usage examples, API details, and customization options.

Tests:

Introduce a suite of tests for the auro-radio component to ensure functionality and accessibility compliance.

team decided to merge without PR


Timestamp option in output properties doesn't use predefined timezone

Current Behavior
I'm using Node-RED in Home Assistant from official add-ons. Node-RED displays correct times in flows (under nodes), but when I choose to fill some output value by timestamp in  Current state node, it didn't respect my timezone, so it returns incorrect timestamp.

Expected Behavior
Option Timestamp should use the same timezone as other parts of Node-RED.
Steps To Reproduce
No response
Example flow
No response
Environment

Node-RED version: 3.0.2
Node.js version: I don't know
npm version: I don't know
Platform/OS: Home Assistant official add-on

This node screenshot is not a Node-RED core node and we do not maintain it.
You will need to raise this with the HA team.

Ah, sorry, I didn't know that. I will crate it in HA, thank you.


Improve UX of next/previous

If in the current state we're making use of previous/next recordings, and one of the two is not available, show the button but disabled. Currently if there's a next recording available but not a previous one we show only the next button which can be a bit confusing and visually does not look great.
We should however keep the logic that if neither next/previous is available, the buttons should be gone.

Ideally we'd also add some sort of autoplay functionality which automatically plays next as current one ends. :)

My thoughts exactly! Forgot to add the note here.

I think all the latest changes @rcmarron has done makes this obsolete, closing.