Improving EnsoGL docs

Pull Request Description
This PR adds EnsoGL docs about debug display modes.
Checklist
Please include the following checklist in your PR:

[x] The documentation has been updated if necessary.
[x] All code conforms to the
Scala,
Java,
and
Rust
style guides.
All code has been tested:

[x] Unit tests have been written where possible.
[x] If GUI codebase was changed: Enso GUI was tested when built using BOTH
./run ide build and ./run ide watch.

Quite clear for me, maybe someone who does not know EnsoGL well should also review it?

I included these docs in the code in this PR: https://github.com/enso-org/enso/pull/4003


.NET core support!

Also moved the test project over to dotnetcore and xunit, nunit has a dotnet runner but it was throwing hard exceptions and only testing one framework. The xunit dotnet runner tests both full desktop clr as well as netstandard1.3 (core clr).
Fixed tests due to 2.0 changes, the first emit() now always sends directly irregardless of BatchPostingLimit or period. The different timer implementations also do not guarantee that this first emit() will only send one document over.
Super helpful to have @nblumhardt and @merbla in person at NDC to help out :D cheers guys!
@mivano I ported over the powershell build script as far as I could as I'm not sure how you do releasing and appveyor so this needs another pass over to suit your deployment model.
It builds and tests.  It also packs nuget packages in the same location but please double check the dlls have the correct version applied, the nuget metadata is valid. same nuget id with casing, are the dlls signed as we expect them too?

LGTM! :+1: Added some very minor notes.
Not related directly to the PR, but what do you think about making dev the default branch in this repository? We've switched from master to dev in most/all of the other repos so that new work is always visible (makes liveness obvious) and so that new PRs will target it by default. Just a thought.

Thanks guys! Looking good. I picked the items from @nblumhardt but the feature branch building does not work correct with the 4.1.0-* trick. Getting this now:
4.1.0-feature/do-00152' is not a valid version string.
It looks like it is the same approach as used with Serilog itself, so maybe @nblumhardt has a tip here? We need to extend this to handle feature branches named feature/?

@mivano ah, I see - you have a branch name with a slash in it... I think it'd be easy enough to replace any instances of / in the branch name with - and get the script working, we've only not accounted for that case because it hasn't been an issue in the other repositories (yet) :-)

I 'fixed' the versioning by using gitversion. That gives better versions per branch.

@nblumhardt I agree that switching to dev as default branch makes sense. I will make the switch.

+1 on gitversion solves two problems in one go :), pitty it needs to change the working directory but I guess failures here leaving you in a diff older as the one we started should be super rare.

I m a bit new to dotnetcore so it can most likely be implemented in a better way, but this made sure you could run it both locally and on the build server.

Took some time, but merged to master and it has packages now: https://www.nuget.org/packages/Serilog.Sinks.Elasticsearch
Thanks for the all the great work!

Awesome @mivano thanks for taking this PR to 100% :tada: :balloon: :fireworks: :cake: :ship: :it:


Store logs in channel

Bot should keep a special channel and forward there any messages that triggered the ban. It is useful for discovering ban history.
Short bot logs could also be kept in the channel for ease of access.

I think that we can include message text when we ban someone in reply to message. And add something like “reason” option later


feature/#1289_2 - normalizing user agent and disabling preventive downloads

… for MAPNIK
Around TileSourcePolicy:

"normalized" user agent (package name + "/" + version)
disabling preventive download

Impacted classes:

DefaultConfigurationProvider: new member String mNormalizedUserAgent and its getter; new method computeNormalizedUserAgent(Context) to compute its value
IConfigurationProvider: added method String getNormalizedUserAgent()
MapTilePreCache: added a check in method search - if the tile source policy says "we don't try to preventively download tiles", there won't be any download attempt
MapTileProviderBasic: added the MapTileDownloader in the pre-cache tile providers
TileDownloader: in downloadTile, uses the normalized user agent if the tile source policy says so
TileSourceFactory: edited the TileSourcePolicy related to MAPNIK (new parameters)
TileSourcePolicy: now using int flags instead of booleans (therefore new constructor); new methods acceptsPreventive and normalizesUserAgent

Please @monsieurtanuki how can I modify these classes in my project ? (I ask this question because I know that it's impossible to modify a class of OSMdroid in an android project)

@akoua

if you have a problem with the maps not being displayed, the fix is probably just to set a relevant user agent, using something like this: Configuration.getInstance().setUserAgentValue(context.getPackageName());
if you really want the latest version of those classes, you can get the full sources and include them in your code (which makes obvious the fact that I'm not a git expert)
if you want to modify those classes in order to contribute to osmdroid, welcome!

I hope this helps.

I have an app which was being blocked as it was using a unique string rather than the package name for the user-agent value,
Once I used: OpenStreetMapTileProviderConstants.setUserAgentValue(getApplicationContext().getPackageName()): instead, that resolved the problem. The problem is, when our tested tried reporduce the problem on the same device with version of the app without the aforemenioned change, it wasn't blocked. Why would this happen? shouldn't it have blocked it once it had seen the invalid user-agent value?


Get latest Nexus changes (not yet deployed)

Please see #1425 for all the Nexus API changes up to v0.3.0 (currently deployed).
This PR depends on that, and added the remaining API changes, which are not yet deployed.
There is a breaking change, and out code is adapted accordingly.

No, scratch that. Actually, this latest change is also part of Nexus v0.3.0, so there is no need for this separate PR.


Update dependencies, primarily aiohttp

Description
Title. Alleviating a known security vulnerability in aiohttp.
Checklist

[x] If code changes were made then they have been tested.

[ ] I have updated the documentation to reflect the changes.
[ ] I have updated the changelog with a quick recap of my changes.

[ ] This PR fixes an issue.
[ ] This PR adds something new (e.g. new method or parameters).
[ ] This PR is a breaking change (e.g. methods or parameters removed/renamed)
[ ] This PR is not a code change (e.g. documentation, README, ...)
[x] I have read and agree to the Developer Certificate of Origin for this contribution

Superceded by #23.


API uses POSIXError

The code uses POSIXError which I think is not an actual thing an implementation can create, right? It is just a namespace (at least on macOS, maybe that is different on Linux?).
Is that meant to be POSIXErrorCode?
https://github.com/swift-server/http/blob/develop/Sources/SwiftServerHttp/HTTPCommon.swift#L72
Note: This is reported as an Issue, because the error part of the API cannot be implemented. (My suggestion would be to not expose 'POSIX' at all but instead use an own HTTPError type which wraps the underlying socket error)

I agree. Mentioning POSIX at all seems like an implementation detail.

This was removed/fixed in #19


When searching for code, options Match and Fuzzy are switched

Description
When searching for code under Explore, selecting Fuzzy as search type performs a matched search, while selecting Matchperforms a fuzzy search.
For example, in my organization, searching for flightcase with Fuzzy yields 3 results, but flightcas yields none.
Searching for flightcase with Match yields 3 results, as does searching for flightcas.
See attached screenshots
Gitea Version
1.21.5
Can you reproduce the bug on the Gitea demo site?
No
Log Gist
No response
Screenshots

Git Version
2.30.2
Operating System
Raspbian GNU/Linux 11 (bullseye)
How are you running Gitea?
Running official arm release with systemd on raspbian
Database
MySQL/MariaDB

Not sure if it's a bug of 1.21.x, but for 1.22

The default indexer was changed to git grep via #29998
https://github.com/go-gitea/gitea/pull/29706 is about fixing the fuzziness of the former default indexer bleve.

I haven't tested it so I can't guarantee the result. But it may be fixed.

We close issues that need feedback from the author if there were no new comments for a month. :tea:


Add image path as variable

Add an image path as a variable so we can move the icon images to a different folder easily.

I agree, this makes using bootstrap as a submodule much easier.
I think this issue was mentioned once before. #1073

I'm sold—adding this and the white icons as separate variables.

Awesome, thanks.
On Feb 4, 2012, at 11:55 PM, Mark Otto
<EMAIL_ADDRESS>wrote:

I'm sold—adding this and the white icons as separate variables.

Reply to this email directly or view it on GitHub:
https://github.com/twitter/bootstrap/issues/1536#issuecomment-3815821


First API routes

This one adds new routes to the application. Provivided API has yet limited functionnalities but allows search by vendor/product, description and allow to fetch vulnerabilities as JSON, detailled or not.
API routes and roles are detailled bellow.

URL route
Role
HTTP methods

/api/v1/<cve_id>
View vuln as JSON
GET, HEAD, OPTIONS

/api/v1/details/<cve_id>
View detailled vuln as JSON
GET, HEAD, OPTIONS

/api/v1/product/<vendor_id>/<product_id>
Search CVE ids by vendor and product
GET, HEAD, OPTIONS

/api/v1/search/description:<description>
Search CVE ids by description
GET, HEAD, OPTIONS

/api/v1/search/vendor:<name>
Search vendor by name
GET, HEAD, OPTIONS

/api/v1/search/product:<name>
Search product by name
GET, HEAD, OPTIONS

/api/v1/search/vendor_or_product:<name> /api/v1/search/product_or_vendor:<name>
Search products where vendor or product matches name
GET, HEAD, OPTIONS

/api/v1/search/vendor:<vendor>/product:<product> /api/v1/search/product:<product>/vendor:<vendor>
Search products where vendors matches vendor and products matches product
GET, HEAD, OPTIONS

PR is mostly for review and not yet mature to be merged. Feedbacks are welcome.

This looks great, thank you so much Ewen!
Looking forward to deploy and try it out soon :).

I've fixed some very small things in 4f2f69eb15432635a4cbdd756adee503b34fd923 and have pushed this to prod. Some examples:

https://www.vulncode-db.com/api/v1/CVE-2013-7470
https://www.vulncode-db.com/api/v1/details/CVE-2013-7470
https://www.vulncode-db.com/api/v1/search/vendor:microsoft
https://www.vulncode-db.com/api/v1/search/product:kernel
https://www.vulncode-db.com/api/v1/search/vendor:microsoft/product:windows

Again, amazing work Ewen thanks a lot for helping out, this is really appreciated.


Performance issues on Android

We have decided to remove this plugin from our project due to its effect on app performance on Android devices. We were experiencing frame drop in large lists while scrolling which went away after removing the plugin. Right now I don't have enough time to provide more info regarding this issue.

Hi @Amir-P the flutter SDK has since been upgraded to its latest version (2.1.2) which has updated Android and iOS dependencies that tackle several performance improvements and bug fixes, it would be extremely helpful if you could give this update a try and monitor your app's performance, here's a changelog with the latest updates and here you'll find the integration docs.


Add a CSS variable to allow the d2l-navigation shadow to be hidden

Changes
This pull request adds the --d2l-navigation-shadow-drop-border-display CSS variable to allow the shadow to be hidden on specific pages.

I think this is only needed for one page, similar to what was needed for the old gradient in grades.  Exposing the attribute would mean the navbar MVC renderer would need to be updated with an option to configure that, and that page would need to specify the special navbar type.  Still do-able, but not as clean as an override in css.  But you're right, this allows people more flexibility than we need


scope(build) aarch64 als target

Blocked by #17 and #4. Building for arm64-v8a android is no problem, however the android emulator is not able to run these binaries on an x86_64.

@fhilgers would it be a solution to use nox-player for development?
https://www.bignox.com/blog/how-to-connect-android-studio-with-nox-app-player-for-android-development-and-debug/#:~:text=Nox App Player is a,idea and eclipse as well.

The idea of using aarch64 was to be a close as possible to the android system used on cars (thats why we use android 13 instead of 15). As nox player is pretty heavily customized i think it would be more appropriate to just run integration test on arm64 runners.
After #17 and #4 is merged, we can build for android on both systems. Then we can setup ci on arm64 systems.


Feat/zeppelinhub ws client

Add zeppelinhub websocket client

[ ] Move Message from zeppelin-server to zeppelin-zengine
[ ] Bring Classes related to zeppelinhub
[ ] tests
[ ] more

thanks for addressing this part of migration and refactoring and looks good to me! let's merge it and i'll take over from there and combine with what i did


Scatter-gather documentation errors

The “Defining scatter-gather processes” documentation in docs/snakemake/rules.rst says

Thereby, scatter.split("splitted/{scatteritem}.txt") yields a list of paths "splitted/1-of-n.txt", "splitted/2-of-n.txt", …, depending on the number n of scatter items defined. Analogously, gather.split("splitted/{scatteritem}.post.txt"), yields a list of paths "splitted/0.post.txt", "splitted/1.pos.txt", …, which request the application of the rule intermediate to each scatter item.

For this to work, one assumes that both gather.split and scatter.split would need to generate the same list of filenames. So surely either both of them should be of the form 1-of-n, 2-of-n, etc or both of them should be of the form 0, 1, …, n-1.
There is also a minor typo: 1.pos.txt.
It would also be useful if the documentation noted the Snakemake version in which this facility became available, which appears to be v5.25.0.

Any practical example of using scatter/gather.split? In the documentation, the "rule split" has no input file. Is it possible to split an input file into n parts, using linux command, split -n -d? Thanks!

PR #1441 fixed the minor typo, but the filename inconsistency remains.


Using GitHub actions concurrency mode to reduce CI queue time

In my past contributions, I've found that GitHub actions are heavily queued because they are under-resourced. This PR will use concurrency mode to ensure that only one commit on each PR will run tests. This way when we do multiple commits, the tests from the previous commit will automatically break and only the tests from the last commit will run. This will greatly save our testing resources. Especially if the contributor is unable to stop the GitHub actions.

My PR itself is an example of this. I added two commits in a short period of time and the first one would be quickly cancelled.


clarify behaviour for id fields

@eavanvalkenburg : Thanks for your contribution! The author, @jseb225, has been notified to review your proposed change.

Thanks for your contribution @eavanvalkenburg - I sent you a separate email with feedback. Please let me know if you have any questions about recommended changes before we merge.

@eavanvalkenburg : Thanks for your contribution! The author, @jseb225, has been notified to review your proposed change.


Some features

Add agents
Add ingame medal changer
Some fixes

very nice

very nic

Can't add agents.

fixed

Error while building?

Help needed

sorry, my bad

Help needed

fixed

I'm guessing Medal only affects the visual of Client ? still doing god's work

I'm guessing Medal only affects the visual of Client ? still doing god's work

client side only

knife logo on kill message doesnt change, whilst it works on normal osiris. pls fix @blackMasterLord

knife logo on kill message doesnt change, whilst it works on normal osiris. pls fix @blackMasterLord

fixed

@blackMasterLord since dk added agent patches. remove or nah

Dude can you make an other pr with only the Better Vote Revealer?

Can someone send me dll of this completed ? everytime i get error.. my discord is - uzdiiilekk_#3312

https://github.com/blackMasterLord/Osiris-Some-features here's the project, if u keep getting errors try updating visual studio.

Can someone send me dll of this completed ? everytime i get error.. my discord is - uzdiiilekk_#3312

https://github.com/blackMasterLord/Osiris-Some-features here's the project, if u keep getting errors try updating visual studio.

@blackMasterLord do you think you are able to add lobby things and also troll options? Like you can see people locations, spam the lobby chat or send them 200 pop up errors?

@blackMasterLord is it possible for you to link me the branch for this pr?

can u instead make the better inv changer menu a seperate pr? @blackMasterLord


Renovate Approve Bot for Bitbucket Server

I have forked this repository to yieldlab/renovate-approve-bot-bitbucket-server and ported it to support Bitbucket Server. But I feel like it belongs to the renovatebot organization. Would you like to take it over?
Maybe it would also makes sense to merge both approve bots into a single one that can be used for Bitbucket Cloud and Server. WDYT?

If there's a lot of common platform code then combining them would make sense, otherwise keeping them separate is also OK

Basically only getPullRequests() and approvePullRequest() are different, the rest is pretty much the same.

I think if you can merge the two and submit a clean pull request here, it would be great. Ideally default to Bitbucket Cloud so that existing users don't need to make configuration changes afterwards


Make tgmath tests generic over TGMath protocol, extend to Float80

Define most of the test machinery in terms of a internal TGMath protocol that refines BinaryFloatingPoint. Add a little bit of gyb support to automate adding additional types, as it is likely these tests will need to support Float16 and Float128 in the future. Make all the input values exactly representable in all of the types in question, to avoid representation errors, loosen default tolerance to 3 to make porting to new platforms less noisy.

@swift-ci Please smoke test

@swift-ci Please smoke test Linux.

@swift-ci please smoke test.

@swift-ci please smoke test and merge.

@swift-ci please smoke test

@swift-ci please smoke test and merge.

@swift-ci Please smoke test.

@swift-ci Please smoke test.

Sure enough, we should remove the ldexp test, since we've deprecated it and scalbn is the thing now.

@swift-ci Please smoke test.

This no longer checks that CGFloat implementations match Float on 32-bit and Double on 64-bit.  Was that intentional?

That’s not what this test is for; ~everything will fail to build if that’s ever violated anyway, so it probably doesn’t need a test, but it’s fairly well covered already.


hacker.itAbbr was renamend to hacker.abbreviation

So using @fake(type:hackerAbbr) does not work currently.

@MrGreenTea Fix released in<EMAIL_ADDRESS>🎉


Migrating Source Code Files to GitHub

Commit Source Code to GitHub
Now that our files are organized, the next step is to migrate our source code files to GitHub. This process will involve setting up a .gitignore file to exclude unnecessary files (e.g., data files) and committing our source code to the repository.
Objective:
Migrate all source code files to GitHub while excluding data files.
Deliverable:

Create new branch in which you commit all files; then, create a pull request. The issue is successfully completed when the pull request is merged.

Instructions:
ONLY ONE TEAM MEMBERS handles the migration. Others can watch and give feedback/help.

Setting Up .gitignore:

In your repository's root directory, create a file named .gitignore.
Inside .gitignore, list the types of files you wish to exclude from the repository (e.g., *.csv, *.xls, and any other non-source code files).
This step prevents unnecessary files from being tracked by Git and keeps the repository clean.

Development Branch:

Create a new branch using Git on your local computer for this migration process. This branch will be where all initial commits are made, allowing us to keep the main branch clean and stable.
Use the following commands to create and switch to the development branch:git branch development
git checkout development

Committing Files:

Add your source code files to the staging area using git add . (This command adds all files except those listed in .gitignore).
Check with your instructor on whether the commit looks "ok" (do this before proceeding!)
Commit your changes with a meaningful message using git commit -m "Commit message".
Push your changes to GitHub using git push origin development.

Create a Pull Request:

Happy with your development branch? Then create a pull request and merge it subsequently.

Ik heb geprobeerd (o.a. met chatgpt) om de source code te migreren, maar het lukt me niet. Ik heb de .gitignore file gemaakt, maar het lijkt alsof die nu in de main branch staat. Ik heb waarschijnlijk iets fout gedaan maar weet niet zo goed wat.


How to use Vue data on framework7 initialization

Hey Vladimir, is there a way to use Vue component data on Framework7 initialization?
Example:
...
data: {
  root: '#app'
  ...
},
framework7: {
  root: this.root,
  ...
}
...

I figured out, until beforeMount() it's possible to assign values like this:
beforeMount: function () {
  this.$options.framework7.root = '#app'
}

Mhm, so I can change all data like shown above, but not the theme.
In the code I see that I could pass it by the plugin params:
`vue.use(require('../vendor/framework7-vue/framework7-vue.js'), {theme: 'material'})``
But why is it not possible to change the theme later on?
With the following try I will have some kind of a mix ...
beforeMount: function () {
  this.$options.framework7.theme = 'material'
}

F7 got init in mounted hook, so you do it right, you can change it in hooks that fires before mounted

In regards to the above commit, is this released? And should we be grabbing the new version? I'm new and just now starting to get into the data binding portion so I'm just curious.

@nolimits4web - I thought so, but Material will be init correctly only if I pass the argument on framework7: {} or as use... component param.
The approach with beforeMount does not work.
Or does it work for you well?

@nolimits4web
I use the following code:
...
created: function () {
  this.$options.framework7.material = true
}
...

The result is a mix of iOS and Material style:

If I add vue.use(require('../framework7-vue.js'), {theme: 'material'}) the outcome is correct:

That's strange, isn't it?
Dennis

@nolimits4web - Ist it by design or not wanted?

So, now that I've actually gotten into the data, I'm seeing an interesting pattern with my issue and I think it can be resolved by creating an event and callback handler (or if anyone knows how to make mounted wait, please let me know, I'm fairly new to Vue). My issue is that in created: I initialize my connection to my deepstream.io server (websocket data streaming server), but that doesn't finish executing until after mounted: is already in progress. The data in mounted:, where I specify DOM elements and framework7 in initialized, depends on the data that has not yet been grabbed from the deepstream.io connection since it has not yet finished opening the connection. It might be enough to use beforeMount: though I'm not sure if mounted: waits for beforeMount: to finish executing? I figured it would for created: but I was apparently wrong about that...


Main Returns the PRs that it creates

In some situations it would seem useful to do post processing after CompatHelper runs based on the PRs that were created during the run. I propose we just return a vector of PRs

bors try

bors try

bors r+


Update CSM culling section because of PR 7677

I had forgotten to submit it!

Arf, my update of the transparency page is also there!


Why do I need to call the create method when using NaxParameter[T].

The current usage is as follows:
  NaxScope.create(xlen = 32)
  val frame = new MyPlugins(plugins).framework

Can the parameter configuration and auto-negotiation mechanism be generalized and independent.
like this:
https://github.com/chipsalliance/cde/blob/master/cde/src/chipsalliance/rocketchip/config.scala
Thanks!

Hi,
So, there is quite a few different experiments in the way to define things in Nax, including NaxParameter an NaxScope.
SpinalHDL implement a way to define variable for a given portion of the hardware elaboration, that's what the NaxScope is about.
Then on the top of this, NaxParameter is a way to define a new value related to the current NaxScope. It's a bit like having a global variable, but instead of being global it is related to the current NaxScope. That way you can elaborate multiple NaxRIscv in parallel without conflicting.
So the reasons why you need to call  NaxScope.create(xlen = 32) before using NaxParameter is to create the host of all the future NaxParameter for a given CPU instance.
In general, i think it could be generalized more, i was planning to go back on it to clean things up.
About the rocketchip config, i'm don't realy know how they manage their parametrization.
Looking at https://github.com/chipsalliance/cde/blob/aa49948fe412840a0786a70df3dedf18ef002479/cde/tests/src/SanityTest.scala#L10, it seems that they pass the database explicitly around, which isn't the same case.
Do you have some practical demo ?

Hi,
So, there is quite a few different experiments in the way to define things in Nax, including NaxParameter an NaxScope. SpinalHDL implement a way to define variable for a given portion of the scala runtime, that's what the NaxScope is about.
Then on the top of this, NaxParameter is a way to define a new value related to the current NaxScope. It's a bit like having a global variable, but instead of being global it is related to the current NaxScope. That way you can elaborate multiple NaxRIscv in parallel without conflicting.
So the reasons why you need to call NaxScope.create(xlen = 32) before using NaxParameter is to create the host of all the future NaxParameter for a given CPU instance.
In general, i think it could be generalized more, i was planning to go back on it to clean things up. About the rocketchip config, i'm don't realy know how they manage their parametrization.
Looking at https://github.com/chipsalliance/cde/blob/aa49948fe412840a0786a70df3dedf18ef002479/cde/tests/src/SanityTest.scala#L10, it seems that they pass the database explicitly around, which isn't the same case.
Do you have some practical demo ?

This is a great project, you did a great job.
It took me half a year to use it.
Thanks!!!
I haven't done the above project for more than a year, but I think it's good to learn a little Chisel's excellent projects to learn from each other's strengths.
I try to restore its intent
its definition is here.
https://github.com/chipsalliance/rocket-chip/blob/86a2f2cca699f149bcc082ef2828654a0a4e3f4b/src/main/scala/subsystem/RocketSubsystem.scala
case object RocketTilesKey extends Field[Seq[RocketTileParams]](Nil)

RocketTilesKey extends FieldSeq[RocketTileParams]
RocketTilesKey: key name
Seq[RocketTileParams]: key type
Nil: key default value
This is the specific usage in engineering,
like this:
https://github.com/chipsalliance/rocket-chip/blob/master/src/main/scala/subsystem/Configs.scala
class WithRV32 extends Config((site, here, up) => {
  case XLen => 32
  case RocketTilesKey => up(RocketTilesKey, site) map { r =>
    r.copy(core = r.core.copy(
      fpu = r.core.fpu.map(_.copy(fLen = 32)),
      mulDiv = Some(MulDivParams(mulUnroll = 8))))
  }
})

class WithRV32 extends Config((site, here, up) => {
case XLen => 32
case RocketTilesKey => up(RocketTilesKey, site) map { r =>
r.copy(core = r.core.copy(
fpu = r.core.fpu.map(_.copy(fLen = 32)),
mulDiv = Some(MulDivParams(mulUnroll = 8))))
}
})
New configuration class: WithRV32, which needs to inherit from the Config class.
(site, here, up) :
site: global key-value Map
here: current location identifier
up: forward search identifier
up(RocketTilesKey, site) will find the closest RocketTilesKey.
RocketTilesKey: Seq[RocketTileParams] type, so it can be map.
Modify parameters like this.
For details, it is recommended to refer to the official documentation of chipyard

t I think it's good to learn a little Chisel's excellent projects to learn from each other's strengths.

Yes i think that's right, different origine / path can lead to different solutions, diversity of backgrounds is a +

I found some doc about the SpinalHDL ScopeProperty here :
https://spinalhdl.github.io/SpinalDoc-RTD/master/SpinalHDL/Other language features/scope_property.html?highlight=scopeproperty

I just pushed a cleanup for the NaxScope thing, which make the parent classes reusable in other contexts.
So, overall it seems it is a quite different paradigme.
Seems like the NaxRiscv aproache is more like having kind of a database that you can enrich by running code with side effects, in a very not functional programming way.
Note that the NaxScope purpose isn't realy to provide parameters, but instead to be a way for the scala code which elaborate NaxRiscv to share variables without having to pass them manualy all around (ex : no use of implicit). It could also be used to pass parameters from the toplevel, but most of the time, parameters pass around the Plugin's constructors themself.
For instance :
https://github.com/SpinalHDL/NaxRiscv/blob/main/src/main/scala/naxriscv/Parameters.scala#L61
Which is loaded by the FetchCachePlugin, and could be refer everywhere durring scalarun durring NaxRiscv elaboration without implicit.

Thank you very much for your detailed explanation and optimization of this.


feat: Extend API's types

If we use this feature to extend interfaces from another library, I think we could make it so that the @ftrack/api entity types would automatically reference generated schema types if possible.
Let's say we created a third package which just contained the EntityTypeMap type. That type would be entirely empty and would be published as a package called @ftrack/schema. This package would be reference both by @ftrack/ts-schema-generator and @ftrack/api.
In @ftrack/api, the interface could be extended with a default base key that represents untyped entities.
And then in @ftrack/ts-schema-generator we also extend the interface by instead emitting .d.ts files which then contain the types.
That would be insanely awesome. Then you would never need type-casts, even with types from the official API client!
As a bonus, we could define a union-type for all generated entities too, for cool type discrimination.

Was thinking since it was an unclosed question :)
But yes, something similar sounds very much in line with what I'm thinking about. I'm taking some inspiration from the type extendability they are using in material-ui

Glad we're on the same page! This is going to be awesome! You guys rock! 😍
Didn't know MUI did something similar!

Wait, wow you're right. The link indeed doesn't have an answer. I swear there was one last time I looked 🤣
Maybe I should just go full into vacation mode and enjoy my vacation 😅 I think I'm bad at doing each thing half-focused 😅

@gismya can you share more about what your thoughts with this were? Then I'll get a PR started!

I tried my best over at this PR: https://github.com/ftrackhq/ftrack-javascript/pull/143
Let me know what you think!

I assume we can close this now?


[New Project]: Blog Website

I want to add a Fully Responsive Blog website using developed using HTML, CSS and Javascript.
The website will be static with various features such as:
Multiple sections covering blogs, carousel, header, footer and a navbar.
Hover animations
Fading effects
Scroll to top functionality
Fully Responsive on all size of devices.
I would like to contribute by working the proposed project. Please assign me the task under GSSoC'23

assigned to @abhinav-m22
Include screenshot or deploy link to assess your project and after that include that to main code with following the contributing.md guidelines


Explain worker exit codes when possible

The :ide:compileTestGroovy task failed.View task in console log
Failed to run Gradle Worker Daemon
> Process 'Gradle Worker Daemon 87' finished with non-zero exit value 137

There is a meaning to exit code 137 (and also for a number of others). We should print that explanation for known exit codes in the error message.

Currently our builds fails with a similar message, but a 100 exit value. it would be great if we could get any pointer to what this actually means.

Hello! We are still interested in this issue, as my mental map of these worker exit codes is very small :)
While the issue is old, the result would be great to have

Hello! We are still interested in this issue, as my mental map of these worker exit codes is very small :)
While the issue is old, the result would be great to have


[CoW AMM Performance] 10k growth for daily rebalancing portfolio

On top of the comparison with the Uniswap reference pool, another interesting reference for CoW AMM performance is the value that a daily rebalancing portfolio (ie a 50:50 ETF that re-allocates once a day) would have over time.
This query achieves this using a recursive SQL statement. It invests 10k on the starting day at the opening price, and then for every consecutive day takes the closing price of the previous day to assess its allocation and perform a rebalancing.
Test Plan
Dune run: https://dune.com/queries/4055484
To cross check, I printed the closing prices for each day and redid the calculation manually for a short time period.

Looks good. I wonder whether we can work with work with price_open and price_close of each day so that we don't have to look in the previous day. This would help avoid the last join as we could carry the balances along with the prices in the balances table.

So weirdly enough, prices open of day x + 1 and price close of day x are not guaranteed to be the same. I don't think this join is very expensive, so I wouldn't do it because of that. Do you think it would make the query overall simpler to comprehend?

So weirdly enough, prices open of day x + 1 and price close of day x are not guaranteed to be the same. I don't think this join is very expensive, so I wouldn't do it because of that. Do you think it would make the query overall simpler to comprehend?

Yeah true, i checked myself GNO, that i expected to be really identical but actually there were some small variations. As i am ignorant about "performance improvements" in sql, if you think it's fine, then let's stick to your implementation. Readbility is fine.

Btw, do we want to add an end date as well?

Morningstar graphs are always from a start date until the current day, so I left it like that for now, but it would be trivial to add if we wanted to.

There seems be be an issue with maximum recursion length: starting from 2024-08-29 00:00:00 gives the error
Error: Recursion depth limit exceeded (10). Use 'max_recursion_depth' session property to modify the limit. [Execution ID: 01J7BNRGBD56MYACCVJZCMP4KS].

The code looks clean though.
Using closing and opening prices looks a bit strange. It is supposed to represent rebalancing between closing and opening, right?

Note that a recursive approach is not required here: for the total value of reserves one gets
current_value(day + 1) = current_value(day) * (p1.price_close(day + 1) / p1.price_close(day) + p2.price_close(day + 1) / p2.price_close(day)) / 2

with initial condition
current_value(start) = 10000 * (p1.price_close(start) / p1.price_open(start) + p2.price_close(start) / p2.price_open(start)) / 2

(using opening prices, which does seems a bit unnatural when written like this). Thus, current_value can be computed using cumulative products of (sums of) daily price changes
(p1.price_close(day + 1) / p1.price_close(day) + p2.price_close(day + 1) / p2.price_close(day)) / 2

Balances can be reconstructed from the total value using prices.

Using closing and opening prices looks a bit strange. It is supposed to represent rebalancing between closing and opening, right?

The goal was to have the first day already capture some price movement (ie. pretending it was deposited at the beginning of the day), but I agree this is a bit confusing.

Note that a recursive approach is not required here:

You are right, great idea! Will rewrite the query based on that insight.


Windows compile problem

Windows 7 x64
Visual Studio Community 2015
Cuda 8.0
Project: cpu_tromp
File: pthread.h
Line: 324
Error code: C2011
Description: timespec: redefinition of type "struct"
How to solve that?

just add _TIMESPEC_DEFINED in Preprocessor Definitions or you can remove the redefinition codes.


[CLI] - Don't exit on bad configuration input

Currently, if the user provides bad input to a CLI configuration, such as leaving the name of the project blank, the CLI exits with an error code.
This is inconvenient for the user. It forces the user to re-run an entire process due to a minor issue.
Adjust the configuration process to let the user retry until the value is acceptable.

This was done in a recent PR.


SSL Certificate pinning broken for Android 5+

CertificateTrustManager.checkServerTrusted() expects authType to be RSA, but starting with Android 5+,
        if (!(null != authType && authType.equalsIgnoreCase("RSA"))) {
            throw new CertificateException("PublicKeyManager: AuthType is not RSA");
        }

See comment in: http://developer.android.com/about/versions/android-5.0-changes.html#ssl
App is making wrong assumptions about cipher suites used to connect to server
For example, some apps contain a custom X509TrustManager that breaks because it expects the authType parameter to be RSA but encounters ECDHE_RSA or DHE_RSA.

Exception stack trace:
01-05 14:18:08.777 22583 22600 W Countly : javax.net.ssl.SSLHandshakeException: PublicKeyManager: AuthType is not RSA
01-05 14:18:08.777 22583 22600 W Countly : 	at com.android.org.conscrypt.OpenSSLSocketImpl.startHandshake(OpenSSLSocketImpl.java:328)
01-05 14:18:08.777 22583 22600 W Countly : 	at com.android.okhttp.internal.http.SocketConnector.connectTls(SocketConnector.java:103)
01-05 14:18:08.777 22583 22600 W Countly : 	at com.android.okhttp.Connection.connect(Connection.java:143)
01-05 14:18:08.777 22583 22600 W Countly : 	at com.android.okhttp.Connection.connectAndSetOwner(Connection.java:185)
01-05 14:18:08.777 22583 22600 W Countly : 	at com.android.okhttp.OkHttpClient$1.connectAndSetOwner(OkHttpClient.java:128)
01-05 14:18:08.777 22583 22600 W Countly : 	at com.android.okhttp.internal.http.HttpEngine.nextConnection(HttpEngine.java:341)
01-05 14:18:08.777 22583 22600 W Countly : 	at com.android.okhttp.internal.http.HttpEngine.connect(HttpEngine.java:330)
01-05 14:18:08.777 22583 22600 W Countly : 	at com.android.okhttp.internal.http.HttpEngine.sendRequest(HttpEngine.java:248)
01-05 14:18:08.777 22583 22600 W Countly : 	at com.android.okhttp.internal.huc.HttpURLConnectionImpl.execute(HttpURLConnectionImpl.java:433)
01-05 14:18:08.777 22583 22600 W Countly : 	at com.android.okhttp.internal.huc.HttpURLConnectionImpl.connect(HttpURLConnectionImpl.java:114)
01-05 14:18:08.777 22583 22600 W Countly : 	at com.android.okhttp.internal.huc.DelegatingHttpsURLConnection.connect(DelegatingHttpsURLConnection.java:89)
01-05 14:18:08.777 22583 22600 W Countly : 	at com.android.okhttp.internal.huc.HttpsURLConnectionImpl.connect(HttpsURLConnectionImpl.java)
01-05 14:18:08.777 22583 22600 W Countly : 	at ly.count.android.sdk.ConnectionProcessor.run(ConnectionProcessor.java:175)
01-05 14:18:08.777 22583 22600 W Countly : 	at java.util.concurrent.Executors$RunnableAdapter.call(Executors.java:423)
01-05 14:18:08.777 22583 22600 W Countly : 	at java.util.concurrent.FutureTask.run(FutureTask.java:237)
01-05 14:18:08.777 22583 22600 W Countly : 	at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1113)
01-05 14:18:08.777 22583 22600 W Countly : 	at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:588)
01-05 14:18:08.777 22583 22600 W Countly : 	at java.lang.Thread.run(Thread.java:818)
01-05 14:18:08.777 22583 22600 W Countly : Caused by: java.security.cert.CertificateException: PublicKeyManager: AuthType is not RSA
01-05 14:18:08.777 22583 22600 W Countly : 	at ly.count.android.sdk.CertificateTrustManager.checkServerTrusted(CertificateTrustManager.java:55)
01-05 14:18:08.777 22583 22600 W Countly : 	at com.android.org.conscrypt.Platform.checkServerTrusted(Platform.java:117)
01-05 14:18:08.777 22583 22600 W Countly : 	at com.android.org.conscrypt.OpenSSLSocketImpl.verifyCertificateChain(OpenSSLSocketImpl.java:556)
01-05 14:18:08.777 22583 22600 W Countly : 	at com.android.org.conscrypt.NativeCrypto.SSL_do_handshake(Native Method)
01-05 14:18:08.777 22583 22600 W Countly : 	at com.android.org.conscrypt.OpenSSLSocketImpl.startHandshake(OpenSSLSocketImpl.java:324)
01-05 14:18:08.777 22583 22600 W Countly : 	... 17 more

Hi @suzukieng
Thanks for your report. The fix will be part of next SDK version which is expected to be out this month.

Hi @suzukieng
I'm sorry, but we moved release of new SDK to later date. 16.02 release won't have this issue fixed.

Fixed a while ago in #4da1354


Electron Cash --- SIGHASH FORKID error

When trying to send BCC/BCH I'm having an error :
The transaction was rejected by network rules. (16: mandatory-script-verify-flag-failed (Signature must use SIGHASH FORKID ))

what version? what kind of wallet?  screenshot?

Hi fyookball, I am having the same issue... I have tried multiple servers. Some people say its because I have 2fa but I don't know how to get rid of it... The error comes straight after I google authenticate.
Any assistance would be greatly appreciated!

This is a known 2FA issue. See below from the 2FA multisig provider: TrustedCoin
I'm the TrustedCoin admin. For a variety of reasons, we have no immediate plans to add support for BCC to our co-signing service. I suggest you modify the Trustedcoin plugin in the Electron Cash release to:
Not allow totally new 2FA wallets to be created
Allow existing 2FA wallets to be restored from seed only with the 2nd-factor disabled (i.e. restored with 2 keys in the wallet thus allowing unilateral spending).
When opening an existing 2FA wallet with only 1 of the 3 keys in it, a dialog should be shown informing the user they must restore from seed to regain access to their wallet.

For anyone out there with a similar issue not wanting to wait for an update,this is what I did to fix the issue.

Got on a different computer that already had version 2.9.2 on it
opened application and whatever wallet was already on that computer
File -> New/Restore
Gave a new wallet name
Standard wallet
I already have a seed
entered the seed that I saved from when I first opened electron cash (2.9.0) on the first computer
It then told me I had 2fa on that wallet and asked if I want to disable -> yes.
I then could see the balance, chose the casharia network and sent the balance (with max tx fee for increased speed) to another wallet to test - success, no error!

1 confirmation so far but still not able to see it in the receiving wallet, will update later.

Received in other wallet!

Did you solve the issue? If not, call<EMAIL_ADDRESS>Maybe I can help you. I rarely go here.


Feature Request - add a JSON checker that is called prior to compiling

Add a call to a JSON checker prior to compiling a board.
I find I have a tendency of making JSON errors when defining boards/components.
Would be great if instead of getting a python error that the JSON files be checked for silly errors - such that it makes fixing them easier then parsing python errors.
Thanks!
Peter

I'm starting to work on the next version of PCBmodE at branch cinco-master.
I intend on adding this feature to Cinco.


If you are making your website open-source, please mention it in your website.

Its a good thing that you want to make your project open-source, but please mention the same in your website and link the github repository to it.

@Vedant-Manjrekar Good idea If you wish you can work on it

Sure 👍🏽

@Arun9739 Done 👍🏽

@Vedant-Manjrekar If you are still working on this issue, then plz make sure to finish early.


feat(dispatch): updated example simulation runs for improved support for data visualizations and metadata

Also closes #2728

:tada: This PR is included in version 3.2.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Update identity-provider-generic-openid-connect.md

Signing-in to an Azure AD tenant requires that the user performing the sign-in flow already has an attribute such as object ID that will identify them on the Azure AD tenant they are signing-in to. Object IDs of every user is different on every Azure AD tenant irrespective of their root Azure AD tenant, this attribute is not transferred from directory to directory and only a sign-up process will create this attribute for users on Azure AD.
Without objectId claim the signin flow return the error AADB2C90037: An error occurred while processing the request. Please contact administrator of the site you are trying to access.

#sign-off


Added TM Forum blog post from Tarus on opennms.org

@JessiDoesStuff Are you finished and can I publish the article with "Merge pull request"?  Thanks a lot for your help.


Decoding in HF Transformers incorrect due to blanket pre-truncating output tokens

This line pre-truncates input tokens so we can only de-tokenize the new tokens -> https://github.com/normal-computing/outlines/blob/main/outlines/models/hf_transformers.py#L160
However, when we batch in multiple prompts of different token length, we left-pad them for uniformity. Therefore, the pre-truncation is invalid. We should instead keep track of the padding, and individually pre-truncate and decode. Else, we can batch-decode the entirely and then pre-truncate before returning.

This issue is incorrect. Left-padding ensures all input prompts are of same length, and it is indeed correct to pre-truncate all of them at once, because all new tokens indeed start from the same index.


An error occurred when the time data was saved to the database

package main

import (
	"database/sql"
	"fmt"
	"time"

	_ "github.com/mattn/go-oci8"
)

func main() {

	db, err := sql.Open("oci8", "**")
	if err != nil {
		panic(err)
	}
	if _, err := db.Exec("create table testa(id varchar2(10) primary key,d date)"); err != nil {
		panic(err)
	}
	defer func() {
		_, err := db.Exec("drop table testa")
		if err != nil {
			panic(err)
		}
	}()
	tm, err := parseWithLocation("Asia/Shanghai", "0001-01-01 00:00:00")
	if err != nil {
		panic(err)
	}
	fmt.Println(tm)
	if _, err := db.Exec("insert into testa(id,d)values(:1,:2)", "123", tm); err != nil {
		panic(err)
	}

}

const TIME_LAYOUT = "2006-01-02 15:04:05"

func parseWithLocation(name string, timeStr string) (time.Time, error) {
	locationName := name
	if l, err := time.LoadLocation(locationName); err != nil {
		println(err.Error())
		return time.Time{}, err
	} else {
		lt, _ := time.ParseInLocation(TIME_LAYOUT, timeStr, l)
		fmt.Println(locationName, lt)
		return lt, nil
	}
}

output:
Asia/Shanghai 0001-01-01 00:00:00 +0805 LMT
0001-01-01 00:00:00 +0805 LMT
panic: timeToOCIDateTime for column 1 - error: ORA-08192: 在固定表上不允许闪回表操作

Correct if you do not have a time zone.

What happens if you try it with pure SQL inside the database? Does it work? What happens if you use the column type TIMESTAMP WITH TIME ZONE?

There is no problem with plain SQL insert into testa(id,d)values('123', TIMESTAMP '0001-01-01 00:00:00 +8:00'), but if the data type is changed to TIMESTAMP WITH TIME ZONE and the parameters are still passed, there is a problem.

Could be a bug. Need someone that would be willing to look into it more and see about fixing it.

@mattn
I am not actively programming in Go anymore and will probably not be in the near future, so going to unwatch this repository. Will be back in your hands.
Thank you for all the wonderful code that you have written! :)

@MichaelS11 Thanks for your great help.


How to download the full logs of a job execution in azkaban?

Hi,
I have a job execution that has lot more log lines than what show up in the azkaban web portal's job logs page. When I click "refresh" on the logs page, I only get the lastest logs but not the logs from the begininig of the execution.
I tried couple of things.

Get the execution logs using the rest api mentioned here http://azkaban.github.io/azkaban/docs/latest/#api-fetch-execution-job-logs
But this gets the same set of log lines as present in the web portal logs page. Not the full and complete logs from the execution

I tried to SSH into the azkaban server and go to this path /opt/azkaban/azkaban-exec-server-2.7.0/executions/647/MyFlow/
Here I find a couple of log files in the format _job.647.MyJob.log, _job.647.MyJob.log.1, _job.647.MyJob.log,2, _job.647.MyJob.log.3 etc
But these logs also have same data as we get from the portal and the rest api not the full logs of the job.

I tried downloading the log data from the mysql database which we are using as our datastore as follows
echo "use azkaban; select log from execution_logs where exec_id=647" | mysql -N --raw -uuser -ppass > mysql.gz
And then wehn I go $> gunzip mysql.gz, I get an error message saying .."doesnt look like a tar archive or trailing garbage ignored"

I'm out of options now.. how can I get the full logs for a job from the time it started to the end of the job execution?

@erwa
Anthony, do you have any clue about this?

Hi Vikram,
I think you can use the Azkaban AJAX URL to fetch a big log. Something like https://<host>:<port>/executor?execid=<execid>&jobId=<jobid>&ajax=fetchExecJobLogs&offset=0&length=2147483647&attempt=0
Also I believe if the log is too big (I think the limit is configurable), the log starts rolling and the beginning of the log will be overwritten.

I tried this..but it doesn't work when the log file is rotated when it gets
too big. Even when offset is 0 it doesn't get the beginning of the log as
expected.
Is it possible to get this from MySQL dB?
On Feb 18, 2016 11:39 AM, "Anthony Hsu"<EMAIL_ADDRESS>wrote:

Hi Vikram,
I think you can use the Azkaban AJAX URL to fetch a big log. Something
like https://
:/executor?execid=&jobId=&ajax=fetchExecJobLogs&offset=0&length=
2147483647&attempt=0
Also I believe if the log is too big (I think the limit is configurable),
the log starts rolling and the beginning of the log will be overwritten.
—
Reply to this email directly or view it on GitHub
https://github.com/azkaban/azkaban/issues/599#issuecomment-185884868.

I think Azkaban only retains a fixed amount of logs. See https://github.com/azkaban/azkaban/blob/master/azkaban-execserver/src/main/java/azkaban/execapp/FlowRunnerManager.java#L186


The animation gets wrapped up with words "echo/read" command

I've cloned your repo as instructed.
But when i run the script below, the animation gets wrapped up with letters (see in video)
I want the animation to always work until the end of script and without wrapping with any letters/words.
#!/usr/bin/env bash

source ~/Desktop/david/bash_loading_animations/bash_loading_animations.sh
trap BLA::stop_loading_animation SIGINT

BLA::start_loading_animation "${BLA_braille_whitespace[@]}"

echo -ne "Write something: "
read sth

echo "You have written: $sth "

BLA::stop_loading_animation

https://user-images.githubusercontent.com/84121715/179161886-b1774de3-af64-4949-b401-9e900968fe19.mp4

This is out of range of the project for now as it would require a lot of work to make it function properly. You can take a look at this repo which has a persistent progress bar. If you wish to implement it, though, feel free to open a pull request.


[fix][cli] Fix logging noise while admin tool exit

Motivation
Pulsar admin tool unnecessary logs noise while existing the process which gives bad user experience and should be removed.
./bin/pulsar-admin topics list test1/test2
null

Reason: java.util.concurrent.CompletionException: org.apache.pulsar.client.admin.internal.http.AsyncHttpConnector$RetryException: Could not complete the operation. Number of retries has been exhausted. Failed reason: Connection refused: localhost/<IP_ADDRESS>:8080
2023-03-21T17:53:57,334-0700 [main] WARN  org.apache.pulsar.common.util.ShutdownUtil - Triggering immediate shutdown of current process with status 1
java.lang.Exception: Stacktrace for immediate shutdown
	at org.apache.pulsar.common.util.ShutdownUtil.triggerImmediateForcefulShutdown(ShutdownUtil.java:55) ~[pulsar-common-3.0.0-SNAPSHOT.jar:3.0.0-SNAPSHOT]
	at org.apache.pulsar.common.util.ShutdownUtil.triggerImmediateForcefulShutdown(ShutdownUtil.java:50) ~[pulsar-common-3.0.0-SNAPSHOT.jar:3.0.0-SNAPSHOT]
	at org.apache.pulsar.admin.cli.PulsarAdminTool.exit(PulsarAdminTool.java:315) ~[pulsar-client-tools.jar:3.0.0-SNAPSHOT]
	at org.apache.pulsar.admin.cli.PulsarAdminTool.execute(PulsarAdminTool.java:305) ~[pulsar-client-tools.jar:3.0.0-SNAPSHOT]
	at org.apache.pulsar.admin.cli.PulsarAdminTool.main(PulsarAdminTool.java:280) ~[pulsar-client-tools.jar:3.0.0-SNAPSHOT]

Verifying this change

[ ] Make sure that the change passes the CI checks.

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

Added integration tests for end-to-end deployment with large payloads (10MB)
Extended integration test for recovery after broker failure

Does this pull request potentially affect one of the following parts:

If the box was checked, please highlight the changes

[ ] Dependencies (add or upgrade a dependency)
[ ] The public API
[ ] The schema
[ ] The default values of configurations
[ ] The threading model
[ ] The binary protocol
[ ] The REST endpoints
[ ] The admin CLI options
[ ] The metrics
[ ] Anything that affects deployment

Documentation

[ ] doc
[ ] doc-required
[x] doc-not-needed
[ ] doc-complete

Matching PR in forked repository
PR in forked repository:

Since we will start the RC version of 3.0.0 on 2023-04-11, I will change the label/milestone of PR who have not been merged.

The PR of type feature is deferred to 3.1.0
The PR of type fix is deferred to 3.0.1

So drag this PR to 3.0.1

/pulsarbot rerun-failure-checks

I like this change. Let me see if I can help nudge this patch.

Merging..
Thanks for taking care of this!


fix: correctly static site generation

🔗 Linked issue
resolve #1812 #1804
related #1721

❓ Type of change

[ ] 📖 Documentation (updates to the documentation or readme)
[x] 🐞 Bug fix (a non-breaking change that fixes an issue)
[ ] 👌 Enhancement (improving an existing functionality like performance)
[ ] ✨ New feature (a non-breaking change that adds functionality)
[ ] ⚠ Breaking change (fix or feature that would cause existing functionality to change)

📚 Description

📝 Checklist

[x] I have linked an issue or discussion.
[ ] I have updated the documentation accordingly.

@kazupon Do you mind using edge channel for now? The issue should be fixed in edge-channel. The next release will come out really soon (Tomorrow, maybe)

@farnabaz
Sure!
I've added edge channel, so that has worked!
Thanks!


Chapters Grapho doesn't work

In the v.1.2.15 when you want to create the chapters diagram the sw goes in stuck.

You are right. Man, I have no idea why, but it's broken again. I'll fix it as soon as I can.


Fix bug: ParserBase has no virtual destructor.

The destructor of ParserBase frees the expat parser. If this
destructor is non-virtual, the destruction of a derived parser does not
clean up properly.

Thanks!


Show Bays in sidebar

As a user I want to see the bays in the sidebar to get a better understanding which IEDs are part of which bay.
ACs:

[x] show the bay in the sidebar above the IED
[x] when selecting multiple IED – the bay will be displayed always directly above each IED
[x] It is possible to see which IEDs are part of which bays

implemented with #232


Doesn't work on OSX Ventura

The app install without problems. When I try to run it, it just doesn't do anything, doesn't open also doesn't crash.
I'm using the M1.
Happy to provide more context given some info on how to provide logs here.
Thanks for the amazing work this app is ⭐!

It was also automatically uninstalled as a few other apps I had once I updated it.
Cheers folks!

The app doesn't even start for me.

Ahhh okay
Sorry, I was confused.
Ahn, yeah. Thank you!
Let me know if I can help to debug it. I have no clue about objective c but happy to generate logs 😅

The app is in general fully M1/M2 and Ventura compatible. Here are a couple of questions to track down any problems:

where is the app installed? (should be the /Applications folder)
while it's running and you notice that your Mac still falls asleep:

open Activity Monitor from the /Applications/Utilities folder or by using Spotlight
enter caffeinate into the search bar in the top left, there should be exactly one entry
when you deactivate KYA the caffeinate entry should disappear

please paste the output of the pmset -g command when you notice any problems (you can type this into the Terminal app from the /Applications/Utilities folder or by using Spotlight)

Thanks 🙂

It's installed under /Applications
I open the app

When I open the activity monitor and search for caffeinate nothing shows up
If I try to uninstall the software osx says it's in use
When I open htop and search for keepinyoueawake it shows up
I killed it, uninstalled, installed again. Same problem:
It seems to be running just fine but I have no GUI.
Here's the output:

pmset -g
System-wide power settings:
 SleepDisabled		0
Currently in use:
 standby              1
 Sleep On Power Button 1
 hibernatefile        /var/vm/sleepimage
 powernap             1
 networkoversleep     0
 disksleep            10
 sleep                1 (sleep prevented by powerd, coreaudiod)
 hibernatemode        3
 ttyskeepawake        1
 displaysleep         0
 tcpkeepalive         1
 lowpowermode         0
 womp                 0

Thanks!!

OK, thanks! This looks like it's running but it's not active. This looks normal so far 👍.
Do you use any tools that manage the macOS menu bar and hide items by default? Can you quit these and check if the icon shows up? The expectation is, when you run KeepingYouAwake it shows up as coffee cup icon in the menu bar at the top of your screen.
Also when there are a lot of status items in the menu bar, macOS sometimes hides them (especially on Macs with a camera notch). If that's the case, can you try quitting some of these apps until (hopefully!!) the KYA icon appears? You can relaunch and reorder them afterwards.

Okay!
One of these things... Fixed my issue. Maybe I had to reset the computer? But now it works.
Thank you for the feedback!!
I guess I'll close it now as I can't reproduce it.
Cheers and thanks again!

@newmarcel I just had the same behavior of no KeepingYouAwake icon on menu bar. Closing other apps helped.
Is this the best builtin behavior on Ventura? Is there a recommended "menu bar icons manager" you can suggest? 🙏


use user/pass flags

We already use global flags for docker specific stuff. This patch enables --username and --password to be passed down to containers/image to setup docker's registries auth.
Fixes #253
@mtrmac @cyphar PTAL
Signed-off-by: Antonio Murdaca<EMAIL_ADDRESS>

I am not really thrilled with the generic option names: if --username and --password never worked, nothing stops us from naming them --docker-username / --docker-password.
I am worried that w really shouldn’t send the same password to several different servers. OTOH the distinction really isn’t between transport types, but between individual hostnames, and that would be a horrible UI, so a generic --username and fervently hoping that the users know what they are doing might be the best I can do.
This should also restore TestCanAuthToPrivateRegistryV2WithoutDockerCfg and perhaps some of the neighboring tests, though.
After that, I guess, LGTM.

(As for “we already do use generic option names for Docker-specific options”, that's partly legacy. And partly using --cert-path for an unwanted service is not a vulnerability, but sending a password to an unwanted service is. But, really, I am bad at designing UIs,)

This should also restore TestCanAuthToPrivateRegistryV2WithoutDockerCfg and perhaps some of the neighboring tests, though

Sort of.. I have to mock an empty Docker cfg first. I'll add that though, thx.

I am not really thrilled with the generic option names: if --username and --password never worked, nothing stops us from naming them --docker-username / --docker-password.

We used to have user/pass flag in the very first skopeo versions :/

Yeah; I am not thrilled with --password but I can live with that.

@mtrmac what if we add --src-username and --src-password and the same for dest. That's not horrible to me and IIUC should solve your concern around wrongly sending a password to a wrong server.

That’s great idea. We can very confidently expect that the sets of servers underlying a transport(atomic: native API + Docker registry API, docker: native API + sigstore) must be under a shared control.
Yes to --{src,dest}-{username,password}, please.

Alright, do note that I'll make these new flags available only to copy. inspect and the now deprecated layers command will continue to behave as usual (w/o any user/pass flag set).

Or wait, I can add username and password flag locally for inspect/layers commands. I'll do this way.

@mtrmac PTAL, before going ahead and add docs/tests/containers/image bits, I would like to have your ack on the direction with copy. I'll also fix layers|inspect once we agree on copy.

I don't like options with dashes ("-") in them.  Could we look at other tools like ssh? Would
--src dwalsh/password@registry --dest mitr/foobar@remote

be better?

@rhatdan fine, but --src isn't clear enough because it's not saying we're talking about credentials

We don't need to specify the registry also, that's implied by src and dest in the command and we don't need that (something like that is how the Docker auth config file works but that's another story, here in the cli we directly honor user/pass combination without checking the registry they'll be used to authenticate against)

@rhatdan @mtrmac BTW, I'm fine compacting "username:password" in the cli as it  doesn't make sense to have 2 flags for user/pass when you must always use both. I'll rework this PR once we agree on flags naming though.

@runcom It would help me if you gave a couple of examples of what you are proposing for a CLI.

Like --src-creds runcom:password - modulo no dash in the flag

I like that better

BTW, I'm fine compacting "username:password" in the cli as it doesn't make sense to have 2 flags for user/pass when you must always use both. I'll rework this PR once we agree on flags naming though.

That feels a bit off, as a general security habit I like to avoid formatting/parsing and in-band signalling. More importantly, it may make sense to provide an username but not a password (e.g. when authenticating using a Kerberos ticket, or a smartcard)

BTW, I'm fine compacting "username:password" in the cli as it doesn't make sense to have 2 flags for user/pass when you must always use both. I'll rework this PR once we agree on flags naming though.
That feels a bit off, as a general security habit I like to avoid formatting/parsing and in-band signalling. More importantly, it may make sense to provide an username but not a password (e.g. when authenticating using a Kerberos ticket, or a smartcard)

alright, no parsing then, any idea regarding Dan's issue on naming?

Why wouldn't --cred dwalsh
Cause the passwd checks to happen to use kerberos like sshd does?
I guess if you had a case where there was a password without a user then we have a problem.

Why wouldn't --cred dwalsh
Cause the passwd checks to happen to use kerberos like sshd does?

Good point, that works fine.

Why wouldn't --cred dwalsh
Cause the passwd checks to happen to use kerberos like sshd does?
Good point, that works fine.

I probably missed something, are we ok with --creds username:password? I didn't understand Dan's example :confused:

I don't like options with dashes ("-") in them.

This could also be handled by providing aliases:
cli.StringFlag {
    Name: "src-username, suser"
    // …
}

and then
skopeo copy --suser mitr --spass sourcepass $source --duser Miloslav --dpass destpass $dest

(or even --su/--sp/--du/--dp but that’s probably too unreadable).
OTOH it is more typing than
skopeo copy --screds mitr:sourcepass $source --dcreds Miloslav:destpass $dest

runcom I am just giving a username not a password
--creds dwalsh
Means Username = dwalsh No password
--creds dwalsh:foobar
Means Username=dwalsh Password=foobar.

Overall, I like the separation of username/password but it does mean quite a bit of extra typing.
WRT the too-long-option names, we can provide shortcuts but it would still be nice to at least allow using long readable ones (--dcreds is also pretty cryptic).
= overall I don’t have a strong opinion on the single/two-argument decision.

We can very confidently expect that the sets of servers underlying a transport(atomic: native API + Docker registry API, docker: native API + sigstore) must be under a shared control.

Sadly, we can’t, when uploading to docker.io for docker/distribution vs. company-specific sigstore.
Ouch.
(This does not immediately affect this PR because that is only setting a docker/distribution username:password. But any more such cases, and telling the user to run docker login so that we have a hostname-associated configuration stored and not a bazillion of command-line options will start to look pretty good.)

runcom I am just giving a username not a password
--creds dwalsh
Means Username = dwalsh No password
--creds dwalsh:foobar
Means Username=dwalsh Password=foobar.

Yup, thanks @rhatdan - I was asking if @mtrmac is fine with that also, Miloslav are you?

I don’t have a strong opinion on creds vs. username+password; separating the source and destination still seems worthwhile to me.

@mtrmac I was talking just about the "user:pass" format which should be parsed

@mtrmac I was talking just about the "user:pass" format which should be parsed

Yeah, I don’t have a strong opinion. If you like that, go for it.

@mtrmac @rhatdan PTAL again, before going ahead and add docs/tests/containers/image bits, I would like to have your ack about copy. I'll also fix layers|inspect once we agree on copy.
The patch right now allows:
$ skopeo copy --src-creds dwalsh:password --dest-creds runcom docker://busybox docker://runcom/credentials

Missing changes to man pages and bash completions.

Yup, if @mtrmac ack on the changes done so far, I'll update man, docs and everything else (adding tests also)

This is ready to be reviewed fully. @mtrmac @rhatdan added man changes, README changes, tests, completions (it's auto-generated so...). What's missing is to create a PR for containers/image changes, I'll do that shortly after this has been reviewed.

Still missing bash completion fixes.

Still missing bash completion fixes.

where is bash completion? hack/make/bash_autocomplete auto generates the bash completion on make install - not sure what I should fix :confused:

Looks like it only generates one level of bash completions.  Lets ignore this for now and we can open an issue to have a much more complete bach completions.

Looks like it only generates one level of bash completions. Lets ignore this for now and we can open an issue to have a much more complete bach completions.

ack, there's already an issue upstream which tracks this https://github.com/urfave/cli/issues/188

@mtrmac @rhatdan updated

Bash Completions https://github.com/projectatomic/skopeo/pull/256

Bash Completions #256

thanks Dan, we'll merge this one and re-iterate on #256

my bad, I have to fix containers/image and revendor here. I'll do it asap, sorry


Options for ReasonML formatting?

It's really difficult to read code with 2 spaces for me. It would be really nice if there was an option to change to something else.
Another person also wrote a nice article why having higher indentation can help for readability and how it invites us to write less nested code.
https://simplystatistics.org/2018/07/27/why-i-indent-my-code-8-spaces/

This other plugin seems to respect 4-space indentation if you use an .editorconfig file:
https://marketplace.visualstudio.com/items?itemName=freebroccolo.reasonml
Indentation level should be a configurable setting.


test(runtime-dom): add tests for nodeOps

Add unit tests for nodeOps.

Thanks but I don't think code simple as that really need unit tests. These code paths will be covered in higher level tests anyway.


Doesn't animate when destination page elements have set Transition.Group

Bug Information
Version Number of Plugin: 2.2.0
Device Tested On: Android API 29
Simulator Tested On: none
Version of VS: Community 2019 16.6.3
Version of Xamarin: <IP_ADDRESS>2
Versions of other things you are using:
Xamarin.Forms: <IP_ADDRESS>0
Prism.Dryloc.Forms: <IP_ADDRESS>2
Steps to reproduce the Behavior
On destination page, set both Transition.Group and Transition.Name
Expected Behavior
Animation will happen.
Actual Behavior
No animation when navigating to destination page.
Code snippet
Source page template of collectionview:
  <BoxView
    sharedTransitions:Transition.Group="{Binding Id}"
    sharedTransitions:Transition.Name="ColorBox"
    BackgroundColor="{Binding Color}" />

Destination page:
  <BoxView
    sharedTransitions:Transition.Group="{Binding Data.Id}"
    sharedTransitions:Transition.Name="ColorBox"
    BackgroundColor="{Binding Data.Color}" />

Elaboration
I want to animate an element between multiple pages. So the flow will be something like>
Page 1: CollectionView which shows all elements
Page 2: Details page. Animate a view from collectionview to a view on this page.
Page 3: Advanced details page. Animate the same view from page 2 to this page.
If you want, I can put up a sample project.

I managed a workaround by setting/removing when navigating to/from page 2.

Almost working. When going back from page 2 to page 1, the elements which I've changed the group to a value and back to null again, are not animated.


Send accepts_incomplete correctly on (de)provision

Fixes #655

LGTM if CI is happy

I've verified the fix against the broker from #655, thanks @pmorie


Fix Custom CSS sytlesheet syntax hred to href

The current yaml example in the documentation uses hred, it should likely be href.

Thanks!


Run a new build for CI fix

Change to run a new build since an image script failed in the build infrastructure.
Refs:

https://prow.ci.openshift.org/view/gs/origin-ci-test/logs/branch-ci-stolostron-governance-policy-addon-controller-main-latest-image-mirror/1729172398034391040

/cc @dhaiducek


Refactoring user_can_vote method.

Why set coding? I didn't see any utf8 char

@wagnerluis1982 I think we should always define the file encoding to avoid conflicts.
In this case we have another reason that is the ’ (apostrophe) character.
https://www.python.org/dev/peps/pep-0263/

Ok, but in this case, ’ could be replaced by the ascii '.

Single quotes are only used around words - they come in pairs, and are not part of any word.

http://english.stackexchange.com/questions/36046/apostrophe-vs-single-quote

Hi, I'm working on issue #91 and this PR changes the same method that I need.
Although the unnecessary utf8 apostrophe, I think the changes here are in good shape.

@wagnerluis1982, it’s a good practice to maintain a development branch updated with the master.

@victor-torres @wagnerluis1982 yes, it is! But to avoid more commits (unnecessary) in the history I think you want rebase instead of merge.

@mauricioabreu that was my intent to say, I always rebase my code when the master has updates.


Solarized Extended -> Awesome Styler

Submit a Plugin update
Rebranding )
Solarized Extended -> Awesome Styler
Github releases checklist

[x] a legal package.json file.
[x] a valid CI workflow build action for Github releases. (Optional for theme plugin only).
[x] a release which includes a release zip pkg from a successful build.
[x] a clear README file, ideally with an image or gif showcase. (For more friendly to users, it is recommended to have English version description).
[x] a license in the LICENSE file.

@xyhp915, the last one 🚀


(DOC) fix table

Move Copy previous results to? to its own row

Thanks for the contributions @VladimirAlexiev !


DatabaseBackend should return TxSet in hash order

https://github.com/stellar/go/pull/2720 fixes a bug introduced in #2710: the order of transactions in LedgerCloseMetaV0.TxSet and results&meta in LedgerCloseMetaV0.TxProcessing are ordered differently. We should update DatabaseBackend to return LedgerCloseMetaV0.TxSet in a correct hash order.

For future reference, from communication with @jonjove "hash order" means

lexicographical byte-wise comparison (of the transaction hash) based on https://en.cppreference.com/w/cpp/container/array/operator_cmp where the template parameter T = std::uint8_t


Checkout - Grand total Discount wrong calculation

See Screenshot

That looks like the discounts have been somehow skipped in the more recent changes to the cart code.
I'll try to look at this soon.
(I'm personally not using discounts on projects yet, that feature was very loosely put together)

Me neither, just testing it anyway. :)

may I add to this context, that the discount is now displayed nowhere in the order view (/dashboard/store/orders/order/1), just the grand total is decreased accordingly:

this could lead to some irritations. :)

This is weird, I was sure we had some mention of discounts on this screen, perhaps it's just not being passed through to display anymore.

just seen that the previous point is already an issue/enhancement: https://github.com/VividWeb/vivid_store/issues/206

Closing legacy Discount system tickets.


Added gitter link to README

Having people know a gitter channel exists is a good way to get them to join :)

Merged. Thanks @omervk!


textarea element

Hi, does this support  elements? Thanks!Q

No, unfortunately there is no way to customise scrollbars of textarea elements directly

Hi, are you sure? this plugin has the option:
https://mobius1.github.io/MiniBar/
Could you look into it and see if you can add the same option? I think its worth it considering your plugin is one of the best and lightest out there. Please :)

Thanks for providing an example. That lib uses an interesting trick to solve the limitation. Unfortunately don't have time right now to add support for it, but I'll reopen the issue so it won't get lost.

Hi Alberto, any chance you would have some free time to try to implement this new feature? I really love this scrollbar! I would like to use it for textareas too.


protobuf error?

Hi,
I'm a newbie and just started learning about sawtooth. I tried to run the containers as instructed in the README.md, but I got these errors on the console;
simple-supply-subscriber    | TypeError: __new__() got an unexpected keyword argument 'serialized_options'

simple-supply-rest-api      | TypeError: __new__() got an unexpected keyword argument 'serialized_options'

simple-supply-tp            | TypeError: __new__() got an unexpected keyword argument 'serialized_options'

Googling around I think it is related to protobuf (protoc version and protobuf runtime version). How can I fix these errors?
Thanks.
Best regards,

hi Faizol, i am also getting the same error . struggling to find out the solution... ;(
Tried all the option mentioned on various blogs to update the python protobuf version, still no luck?
Did you find any solution ?

@RakeshDevUK
Hi,
The problem happened as the protobuf runtime lib version supplied/installed by Dockerfile is older than protoc. So to solve the problem, just make sure that the containers update the runtime lib by modifying Dockerfile in their respective folder.
For Dockerfile in processor folder, make sure to also install "python3-pip" (add to the end of line 22). Then for Dockerfile in all three folders (processor, rest_api, simple_supply_subscriber), add "RUN pip3 install -U protobuf" to update the installed protobuf runtime lib.
Hope that helps.

@faizol  perhaps create a pull request with the fix?

This issue is closed, but for the record,  it looks the protobuf library is older than the protoc compiler.
You can have a protobuf library newer than your protoc compiler, but not older (the protobuf library is backward compatible, but not forward (future) compatible.

A fix for this error is in PR #30.
Basically the fix is to remove the serialized_options arguments from the protobufs/*/*_pb2.py files..
The serialized_options argument requires Protobuf 3.6.0 or above to support serialized_options,
but repo.sawtooth.me is only at Protobuf 3.4.1 for Sawtooth 1.0 and 1.1 at least.

@RakeshDevUK
Hi,
The problem happened as the protobuf runtime lib version supplied/installed by Dockerfile is older than protoc. So to solve the problem, just make sure that the containers update the runtime lib by modifying Dockerfile in their respective folder.
For Dockerfile in processor folder, make sure to also install "python3-pip" (add to the end of line 22). Then for Dockerfile in all three folders (processor, rest_api, simple_supply_subscriber), add "RUN pip3 install -U protobuf" to update the installed protobuf runtime lib.
Hope that helps.

working fine after these changes.


[#643] Overwrite trials and records with the new data

NOTE: This depends on https://github.com/opentrials/processors/pull/123 being merged first.
For example, consider the case that some data is removed from the source (e.g.
it used to have "target_sample_size" but doesn't anymore). If we don't clear
the row when updating the data, this field wouldn't be cleared, and so we'd end
up with a mix of old and new data.
Fixes opentrials/opentrials#643

This PR isn't needed anymore, as I wrote a better test for the case we're checking on #661 (e.g. https://github.com/opentrials/processors/blob/e26fb217d1e7ab2cd3366ccb2852fecc0168bbb3/tests/processors/base/writers/test_trial_and_record.py#L244-L266)


Remove duplicated items

Problem
I noticed that the pattern list includes some duplicated values. For example,
$ echo addicts | alex
<stdin>
    1:1-1:8  warning  `addicts` may be insensitive, use `people with a drug addiction`, `people recovering from a drug addiction` instead
    1:1-1:8  warning  `addicts` may be insensitive, use `people with a drug addiction`, `people recovering from a drug addiction` instead

⚠ 2 warnings

Solution

I added a validation script to ensure the YAML resource files have no duplicated values. https://github.com/wooorm/retext-equality/commit/6a5dcb13a5087f85d259a57d259833c7fa9a1db9

If a YAML file includes duplicated values, it throws an error like below and won't write files.

Error: "ablist.yml" has duplicated value(s):
{
  "type": "simple",
  "considerate": "person with learning disabilities",
  "inconsiderate": "learning disabled"
}
{
  "type": "simple",
  "considerate": [
    "turned off",
    "person with a disability",
    "people with disabilities"
  ],
  "inconsiderate": [
    "disabled"
  ]
}

I removed duplicated values in accordance with the validation, and rebuilt files. https://github.com/wooorm/retext-equality/commit/2f54a05163538f48351f946dc22249a96e5c3001

Thanks a lot @shinnn! Great work!


Revert to 2.0.6 because failed to upload to maven central

What is the necessity of this update? What is updated?
I reverted the library to 2.0.6 because it failed to upload to maven central.
How did you do the test? (Not required for updating documents)

LGTM


PublicKey null, after connecting and ready

Im really liking this package, you take away a lot of the work but I've noticed a small bug / "growing pain" where the walletState is outputing the following:
{
  ...,
  connecting: false,
  connected: true,
  ready: true,
  publicKey: null
}

I would expect the publicKey to be set before the connecting flag is set to false after actually connecting. If you agree, I'd be happy to open a PR for this.

I agree... but I can't tell from your description where this is happening or how to reproduce this. Which package is this in?

I'm guessing it's this? https://github.com/solana-labs/wallet-adapter/blob/f14d5b30f7bdd2f47ee03c0744298f44b8803210/packages/react/src/WalletProvider.tsx#L70-L76
I think we may want to refactor this so we can do something like
const { connected, publicKey, autoApprove, ready } = adapter;
setState({ connected, publicKey, autoApprove, ready });

This will cause the state to be updated atomically to prevent inconsistencies.

@grumpyoldman-io please review: https://github.com/solana-labs/wallet-adapter/pull/33

Another solution would probably be to re-order the functions on line 73-75, but the proposed solution is way sturdier, at the cost of having a wee bit complexity in the state.

Fixed by #33


No module named 'Foundation' on M1

Hi,
I'm testing the latest universal commits on M1 and I'm getting
Traceback (most recent call last):
  File "/Library/installapplications/installapplications.py", line 29, in <module>
    from Foundation import NSLog
ModuleNotFoundError: No module named 'Foundation'

Am I doing something wrong? If I install rosetta the 2.0.2 intel version works.
edit: 2.1 Alpha 1 also works so this might be only on the latest code.

I'm confused by your edit, but yes you must be using absolutely latest commit with the available python framework.

Sorry, I see now that the edit is a little weird.
What I meant to say was that the latest commit I used today with built python framework returned the above error.
When using 2.1 Alpha1 from december this does not occur.

That makes sense. Closing ticket.


Generate different barcode with the Gem

Hi there,
I using this Gem to generate PDF417 barcode png. It works as expected, though the output image is different from those generated by online barcode generators. Just wondering if there is any other variance to this barcode format or if there is anything I need to be aware of when setting up the option arguments. Thanks.
Here is an example generated by the gem and the code for it.

PDF417.new(barcode_text,aspect_ratio: 0.9).to_png(margin: 0, x_scale: 1, y_scale: 28)
And here is another example from the online generator with same data.

As far as I know, those are both PDF417 compliant barcodes.  There are some differences between them but I think that is all about the options set on the left hand side of the online generator.  The data mode, error correction level, row and column count will all change the barcode.  Visually, one thing I note in the online example is that many of the rows are similar which suggests that for the length of data you're encoding you don't need 30 rows.
The gem will try to get a best row and column fit based on the data.  I remember that it always uses one data mode but I forget what mode that is.  Lastly, it has a default error correction level but I forget what that one is too.
The y scale in your ruby gem example is probably larger than it needs to be too.  Maybe try 2 or 3?
As for what else to be aware of with the option arguments, I've always gone with 'scannable' as my top goal which depends a lot on the scanners you'll be using, the paper, and the environment they'll be used in.  If it's a rough environment where the barcode can get obscured or destroyed a high error correction would be great.  If it's on a small part that is protected from smudges then you'd want something smaller.  It will probably take some experimentation to find the best fit.

Thanks James! I think you are right. At the end of day, it's all about if the barcode is 'scannable'. Will do some experiments.


Performance issue of LowCardinality on group-by with multi-keys

Hi. I have tested the performance of LowCardinality feature. I found that the performance has improved a lot when we use group-by with only one key. But if I use multiple keys, the performance did not improve much. For example, I have a query:
select device_brand, count() from medium group by device_brand, app_name
where device_brand and app_name are LowCardinality(String) type. I get almost the same running time:

I read the most of the code about the LowCardinality and found it cannot handle the case of group-by with multi-keys in the current version.  The engine selects serialized aggregation method instead of low_cardinality_key_string so that we convert indexes to full column.
I wonder if the LowCardinality will support group-by with multi-keys. And is there any plans for improving LowCardinality since the current version seems not very helpful for many kinds of queries.

Well, that's a very good question. The case with multiple LowCardinality keys can actually be improved, but it' s more difficult than in case with single key and potential speedup is lower.
There are two main optimizations for GROUP BY LowCardinality, and both of them optimize access to hash table (when for aggregation key we need to find the position of corresponding data):

Hash of aggregation key is cached for dictionary. So, when we search in hash table, we don't calculate hash function result, and only need to find corresponding cell.
The result of hash table find() function is also cached, and if we have a few different keys in local dictionary, we can mostly skip the hash table step.

For multiple LowCardinality keys the first optimization is partly possible. We still can cache hash function result for each LowCardinality column, and if we use combination of hashes as result hash value, and it's computable fast enough, we can have some profit.
The second one is much more difficult as soon as the total number of different keys is a multiplication of each column. If it's much less than the total number of rows, it will probably be possible to create similar cache but with two-level address scheme (each dictionary to temporary index, and tuple of temporary indexes to final result). But, we need a fallback in case we have a lot of different tuples.
Right now I can't say when (if) it will be implemented.

Well, that's a very good question. The case with multiple LowCardinality keys can actually be improved, but it' s more difficult than in case with single key and potential speedup is lower.
There are two main optimizations for GROUP BY LowCardinality, and both of them optimize access to hash table (when for aggregation key we need to find the position of corresponding data):

Hash of aggregation key is cached for dictionary. So, when we search in hash table, we don't calculate hash function result, and only need to find corresponding cell.
The result of hash table find() function is also cached, and if we have a few different keys in local dictionary, we can mostly skip the hash table step.

For multiple LowCardinality keys the first optimization is partly possible. We still can cache hash function result for each LowCardinality column, and if we use combination of hashes as result hash value, and it's computable fast enough, we can have some profit.
The second one is much more difficult as soon as the total number of different keys is a multiplication of each column. If it's much less than the total number of rows, it will probably be possible to create similar cache but with two-level address scheme (each dictionary to temporary index, and tuple of temporary indexes to final result). But, we need a fallback in case we have a lot of different tuples.
Right now I can't say when (if) it will be implemented.

Thanks for your reply. I will continue to pay attention to this feature and look forward to your progress in the future.


CSS Prettify Transform bug: "border-radius: .2rem .2rem 0 0;" => "border-radius: .2rem.2rem00;"

"border-radius: .2rem .2rem 0 0;"
is transformed to
"border-radius: .2rem.2rem00;”
Using chrome.

Could you give a bit more info about how to repro this? I this part of a larger CSS file?

When I beautify the following on http://jsbeautifier.org/ I get unchanged output:
<style>
    border-radius: .2rem .2rem 0 0;
</style>

@buildbreakdo  ?

@bitwiseman Look into reproducing this today, switched libraries and need to revert to repro

@bitwiseman Couldn't reproduce this again unfortunately.. will switch back to using js-beautifier, if I see it again, will be sure to properly document repro steps.

@buildbreakdo - Cool, thanks.


extension for University of Mannheim

mbs.education Extension for the University of Mannheim, Deutschland.

added domain for University of Mannheim

@ir2pid Please provide us with an URL of a page or another proof (.pdf or a screenshot are OK) showing that the university recognizes the domain which you are submitting as an official email domain for the students.

@ir2pid You might miss my previous comment. Please reply.

@ir2pid You might miss my previous comment. Please reply.

@ir2pid I am sorry, this request remains unanswered for a while. I have to close it without merging. Please reopen and comment if you have an update regarding the abovementioned issue. Thank you for the understanding!


Background image lazy load and callbacks

Hello,
I saw in docs, callback_finish wont be fired and load class will not ba added into bg images.
I also tried other hooks like callback_finish with no luck. Is there a way to add class or some callback after BG image is loaded?
I'd like to make transition between placeholder and new loaded BG.
Thanks :)
By now Im using just stying with attribute &[data-was-processed="true"] { since this is added after image is loaded, but if it is added, maybe there is way to add something else with some callbacks?

Is there a way to add class or some callback after BG image is loaded?

You can try using callback_reveal, it fires when div or img appears on screen, the callback_finish fires when all images is loaded.

Ok, thanks

Hey @isuke01, @wamk9,
things changed on version 15.
I found a way to manage background images's load/error event, so now background images are now getting the loaded or error class and calling callback_loaded and callback_error.
Take a look at the changelog and to the upgrading guide if you're interested.
Have a nice day.


Add option to spec for plplot 3D options plus code to use it

This is for issue #57.
I added a bit of text to the doc comments in the mli file.  Not sure if that's more than you want there.
The commit history is kind of odd looking (to me, at least).  Not worth explaining.

I ran it through the examples in the tutorial. The only files that differed were the ones using random data, but visually they looked OK.  I'll PR the examples file, too.  I'm trying to separate different kinds of additions into different PRs.

Great! Thanks @mars0i . It is an awesome PR and thanks for the contribution. Could you also revise the tutorial https://github.com/ryanrhymes/owl/wiki/Tutorial:-How-to-Plot-in-Owl%3F and contribute a small example there if possible. 👍
Ideally, it is better not to expose raw Plplot configurations to Owl users in order to maintain a consistent plotting configuration. I will do some minor refactoring to wrap them up. but it is not a big issue at the moment. :)

Sure, I'd be happy to contribute an example.  I want to wait until the interface has stabilized.
(I'm using Owl for a presentation at a week-long conference that starts a week from now.  The code has been changing pretty quickly--I'm not complaining, but I might need to stop contributing for a couple of weeks and just make use of my fork, which has some additional changes, until after the conference.  We'll see.  Working on Owl can be a nice way to take a break.)

Sure, this is not urgent at all. I would also recommend using your own fork for the presentation. As you mentioned, the master branch is changing quite fast at the moment.
Good luck with your presentation.


Windows issue

I have installed gtk+-2.0 using mysy2.
But got a build error like this:

go version is 1.13.4/amd64
Any help here?

See https://github.com/golang/go/wiki/InvalidFlag
You must specify flags to CGO_LDFLAGS_ALLOW.


Option to automatically accept all transfers

Thank you for fantastic utility!
👉  This is not a duplicate of #92 - There are many more people who are not sharing their WIFI network with unknown people.  (Also if someone has both macOS and Android devices, chances are that Android is used for development testing and therefore may not have approved Contact there).
And while #92 is being considered, it might be easier to add this option which will be tremendous value.
The connection and negotiation takes time and you have to sit glued to mac waiting for notification to accept. And if you missed it, the Android device won't wait long.
I have lost and had to re-initiate countless transfers because of it.
Please add option to auto accept from all!

You can check my fork of this app. Auto download and some other goodies
https://github.com/XRayAdamo/NearDrop

You can check my fork of this app. Auto download and some other goodies https://github.com/XRayAdamo/NearDrop

This is perfect, thanks! Rather simple changes that make the entire experience perfect.


chore(ops): update

What this PR does / why we need it:
Which issue(s) does this PR fixes?:

Fixes #
Sample Links & Screenshots:

Link:

Desktop Screenshot

Mobile Screenshot

Additional comments?:
Developer Checklist:

[ ] Read your code changes at least once
[ ] Tested on multiple web browsers
[ ] Tested responsiveness (e.g, iPhone, iPad, Desktop)
[ ] No console errors
[ ] Unit tests*
[ ] Added e2e tests*

Welcome to Codecov :tada:
Once merged to your default branch, Codecov will compare your coverage reports and display the results in this comment.
Thanks for integrating Codecov - We've got you covered :open_umbrella:


[WIP] Merge PRs

⚠WIP ⚠

Checks when setup
Individual check status
Mergeability (conflict, etc)
Button w/ different types

Still TODO:

Interactions
Create a configure store to set the default merge type per repo
Send merge API request

cc @orta 😉

This is very relevant to my interests

Slick!

Will followup w/ a11y after we get some feedback


568 operation converter optimizer

closing and picking it up here (https://github.com/Tmonster/duckdb/pull/108). Need to support filter pushdown through semi and anti joins


Added status checks

Added ability to generate a status check and the ability to overwrite the latest one.
This resolves #33 and closes #26 as it merged it into one ticket.
There are some limitations as we can not select the check suite that the check will belong (see https://github.com/paritytech/review-bot/issues/33#issuecomment-1662034503), but the name will be constant, making it validable.

Example of the status check

@mutantcornholio @mordamax any objections or should I merge it?


Adjust listener for kaltura

Fixes #1202 and related to #1082.
Strangely it is necessary to wait a bit for the player to be ready, even after the onReady event fired. Otherwise the added event listeners would just be ignored as there is no warning or error message.
After my adjustments I can now consistently reproduce seeking (in-player and via react-player), play/pause (in-player and via react-player).
Nevertheless there is still no support for:

playbackRate
volume update (setting inside the player won't propagate to react-player)
controls option

Manual investigation (console.log(this.player.events)) of the API shows, that the internal player only supports these events:

"ready"
"play"
"pause"
"ended"
"timeupdate"
"progress"
"error"

Hi @Yasamato .
Thank you so much for this!!!
I'm confirming you've fixed the seeking action.
With this PR, I'm having some trouble with the Play/Pause toggle, but I believe this is fixed if you add "autoplay" to the Kaltura iframe:
<iframe   ref={this.ref}   src={this.props.url}   frameBorder="0"   scrolling="no"   style={style}   allowFullScreen   allow="encrypted-media;autoplay"   referrerPolicy="no-referrer-when-downgrade" />

I'm on a Mac and I use Chrome as a browser.

Published in 2.10.0


A few broken links on developer guide in documentation

The link to developer guide - operators in how to create new operator is down. The operator documentation is here.
The link to doxygen api in system design is down. Actually where is doxygen documentation?
The API reference part of execution engine is down. Also related to doxygen.
I am also interested in expanding the C++ operator developing guide to the new unified API defined in operator_utils.h. Any other resources to look at?

Thanks for bring this up. These should be fixed by https://github.com/dmlc/mxnet/pull/2065
You are more than welcomed to add guide of newly added operator_util.h I can help review and answer any of your questions. Thanks!

Please see pr #2136 for the new doc.


Fix javadoc following HSEARCH-1404

See #1442

Waiting for the CI...

worked fine locally. Merged!


docs: change "exception hierarchy" for exception

ResponseSystemError -> InternalServerError and add ForbiddenError

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
5514

Relevant Lines:
6558

💛  - Coveralls


Disable Function Versioning?

@JonRoma unfortunately testing has shown that It is not possible to use $LATEST in variable cloudfront_lambda_origin_request_arn. Amazon, or Terraform, requires a specific version. I think we should consider dropping versioning, or better allow the end user to decide on what behavior he desires. For us I think we would prefer no version so we get automatic updates without having to reapply each distribution. But this is something that needs to be thought about and documented.

@JonRoma function versioning is required for CloudFront: https://advancedweb.hu/how-to-use-lambda-edge-with-terraform/.

@ddriddle, did we complete and document a workaround for this deficiency in the Amazon API? If so, this issue is redundant and can be closed. Otherwise, let's add a comment listing what still needs doing.

This has been addressed. Nothing left to do.
In summary, versioning is required for lambda@edge functions, and we do need to remember for future work that we will always need to create an alias latest for all of our lambda@edge functions.