chore(gitattributes): Add "hex" diff tag to binary files

This adds a diff tag to binary files in .gitattributes,
enabling git to use an external tool to generate diffable output for binary files
by adding a handler to the local or global git config, i.e:
[diff "hex"]
  textconv = hexdump
  binary = true

Change-Type: patch

Amazing! I must confess I'm really curious about what you're using this for :)

Amazing! I must confess I'm really curious about what you're using this for :)

Hehe 😄
The story behind it: I got a bit frustrated with the test .dmg images in #1248 differing from their source image (the raspberry test image file), and no nice way to diff them. So I started looking for ways to get git to do binary diffs, and found the gitattribute tags and textconv settings deep in the docs.
Then I got a bit frustrated with how slow hexdump was, and wrote hxd, which streams, is pageable and 2x as fast as hexdump – and that's what I'm using now to do bindiffs on test images in Etcher :)

Cool stuff. I'm thinking if we could somehow make use of these diffs for something else. Do you it would make sense to display something like that (or allow it to save it somehow) to the user when there is a validation error? Alternatively, if most users won't be able to make sense out of it, would it make sense to send it to us so we can inspect it?

@jhermsmeier Would it be worth adding hxd as a devDependency, and then setting up the unit-tests so that if one of the image-writing-tests fails, it automatically outputs the hex diff of the failed file? (or is that best still left as a manual step, due to the large amount of output it'd be likely to generate?)

I don't think so – we'd have to write a bunch more code in the tests to even display the diffs – it'll be a lot easier & faster to just run the failing ones locally and inspect the output.

Perhaps you could add some notes to one of the docs, explaining how to do this hex-diff for any image that fails the unit-tests? ;)

@lurch yup, good idea! Added in https://github.com/resin-io/etcher/pull/1380


feat(controller): add a option to limit the number of log lines

Usage: deis apps:logs -n <number>
I used a URL query parameter instead of a JSON body because that is generally considered better practice in a GET request. If you would like me to change to using a JSON body I can.
Do you want a API bump for this, or since 1.3 hasn't been released yet, add to that?
Closes #1264

Do you want a API bump for this...?

I think it's fine to just fold it in to the 1.3 API since it hasn't been in an official release, but I defer to @bacongobbler.

I agree that it's fine to fold it into v1.3

looks like the integration tests need to be fixed

I'm getting weird test failures now that seem to be related to ceph not binding to the monitor. Any advice?

yeah that's an odd issue. Let me re-run the build and see if that's okay. Jenkins has a temper these days

I shipped off job 576 on your PR but it failed on the recent integration test changes: https://ci.deis.io/job/test-ec2/576/consoleFull#682192135502066c60b2236c09cc9bf8

I'm getting weird test failures now that seem to be related to ceph not binding to the monitor. Any advice?

From the test logs, ceph version 0.80.9 is Ceph Firefly, which is quite old. I'm not sure how your builds are using such an old version of Ceph.

I have rebased recently. On top of that, I believe that PR's are run from merge of the PR commits on top of master, so that shouldn't affect anything.

PR's are run from merge of the PR commits on top of master

You would think so... But that's apparently not how the Jenkins PR builder plugin works, and this is defensible behavior I'm told. It confuses me, but in practice you have to rebase early and often. It's probably worth git pulling down latest master and rebasing this against that once more. Otherwise we're a bit clueless. :grey_question:  Sorry!

You would think so... But that's apparently not how the Jenkins PR builder plugin works, and this is defensible behavior I'm told.

Good to know, thanks for the tip!

I'm really confused with the way you guys are using your utils.Execute command.
For example in line 99 of the apps_test.go file:
	utils.Execute(t, cmd, params, false, "created initial release")

You're setting the fail flag to false but you are also setting an expectation for the result. However, looking at the execute function itself, if you set the fail flag to false, the expectation is never checked.
Is the purpose of utils.Execute only to check if a command doesn't throw an error, or to check that  the error thrown is the expected one? If so, why are you passing an expect string with a sane looking value (which is deceptive, as the string looks realistic but often is not actually contained in the output at all), instead of nothing or just an empty string? What is the use case of utils.Execute vs utils.CheckList which seems to do almost exactly the same thing but also checks for a string is contained if the command passes.

I've always been confused, too. @smothiki, do you understand the rationale behind the function signatures?

Agreed that those test helper funcs are basically inscrutable. I have a refactoring in progress that discards them and starts over with more obvious code, and I will revisit that work soon.
Thanks for persevering @Joshua-Anderson. I'll re-review this today.

LGTM, reviewed and tested locally with vagrant.

Code LGTM. Thanks so much @Joshua-Anderson!


Browser opening leaves url bar empty if the internal url is invalid

If browser was opened internaly on invalid url, the url bar becomes empty.
It can be reproduced by changing the default page to invalid url.

I found it while i was creating my pull request #3 in which i added ability to open browser with non-default page by command.
Firstly i thought it was bug that i've created, but then i've tested it the way i mentioned earlyer and got sure i had not.

commit https://github.com/Blobanium/MCBrowser/commit/1e345d9c32f367617bf12db25ac01c22911f91ee does not solves the problem? the bug is still here. The problem is somewhere on calling BrowserScreen, not on which request u giving to it. When u type incorrect url in urlBar it firstly tryes to reach it, and after fail sets the urlBar to what u've been writing to it, so the thing where it sets text to urlBar does not calls on entering incorrect url manually.


Polyfill not working on Safari 15 (iOS & macOS)

I just compared the behavior with the previous versions of Safari, both mobile and desktop and smooth scrolling no longer works via this polyfill.
I can provide a CodePen if needed, let me know.

it is working http://iamdustan.com/smoothscroll/
The issue is safari itself - it changed its behaviour. If you have say snapping enabled then the plugin won't scroll smoothly anymore b/c Safari seems to ignore scrollTop/scrollLeft values if they fall behind the threshold.

Everything worked fine prior to Safari 15, but now the behavior: "smooth" doesn't seem to be respected any more.

Same issue, smoothscroll broke down on safari 15

Hey all, after some tedious testing I found that Polyfill does in fact work on IOS safari 15. The actual issue is that Safari In macOS Monterey and IOS 15 doesn't seem to be working with scroll-snap, and the reason that the example site works is because it actually isn't using scroll snap (which a lot of people use this Polyfill with). This might be why using an anchor worked for @plamenh, and why @apithy-isidro-martinez is having his bug with Vue Horizontal.
TL;DR
css scroll-snap is broken in MacOS Monterey and IOS 15 and will cause elements to jump if used.

I too use vue-horizontal and scroll-snap is also broken with Safari 15 on macOS Catalina :/

I had the same issue with this polyfill on macOS Monterey and Safari 15. Thankfully it's just one of the many bugs of Safari 15. I tried this polyfill on the next version of Safari, v. 15.4 (release 134), that's currently being tested as a public beta and the polyfill works as expected (at least for my use case)! So please don't freak out (like I did) and install the Safari Technology Preview to test your website.

Just a heads up, it is still broken for me in safari 15.4

Works for me in 15.4 using scrollBy in a container that uses scroll-snap

Got the same issue! 👍

Just a small update from me, it may be helpful to some:

The polyfill seems to work on Safari 15-15.1-15.2 but it won't work if the element is hidden (for a example if it's overflowed). You can verify that it works by visiting this page: https://codepen.io/diyifang/embed/MmQyoQ?height=265&theme-id=0&default-tab=js,result&embed-version=2
For my use case, I had to remove "scroll-snap-type" from the parent element and it works again now.

From my experience, this polyfill works -without any modifications- on many old versions of Safari and it also works on the upcoming v. 15.4 which has been in development for months now and should be released soon.

Still not working on my iPhone…

thank you @zac-keakie ,
I apply your solution to my Ionic project.
I put this below lines in file polyfill.ts  and work fine:
import smoothscroll from 'smoothscroll-polyfill';
smoothscroll.polyfill();
window.__forceSmoothScrollPolyfill__ = true;
Tested on IOS 15.4

Sharing my experience:
For our scroller, we originally had CSS overflow:hidden on the target element and behaviour: smooth in the JS scroll event. This worked in all browsers, and using this polyfill made it work in Safari.
For iOS 15.4, this polyfill ceased working and we removed it, hoping native smooth scrolling would kick-in for Safari. This was not the case: native smooth scrolling in Safari was not working right away. Instead, simply adding CSS overflow-x:auto to the scroll container did the trick; and seemingly has had no effect on other browsers. We're not sure why the container originally needed overflow:hidden in the first place; maybe it was a safety measure to prevent any bleed (something I would have done).
In addition, we're fine with older-iOS-version-users not having smooth scroll in the same context, however it would have been nice to continue legacy support.

Sharing my experience:
Before iOS 15.4: For our scroller, we originally had CSS overflow:hidden on the target element and behaviour: smooth in the JS scroll event. This worked in all browsers, and using this polyfill made it work in Safari.
After iOS 15.4: This polyfill ceased working and we removed it, hoping native smooth scrolling would kick-in for Safari. This was not the case: native smooth scrolling in Safari was not working right away.
Solution: Simply adding CSS overflow-x:auto to the scroll container did the trick for native smooth scrolling in Safari; and seemingly has had no effect on other browsers. We're not sure why the container originally needed overflow:hidden in the first place; maybe it was a safety measure to prevent any bleed (something I would have done). In addition, we're fine with older-iOS-version-users not having smooth scroll in the same context, however it would have been nice to continue legacy support via this polyfill.

Absolutely. This fix is great unless you need to disable X overflow, in which case the native smoothscroll still does not work and you need to force use of the polyfill instead.

This worked for me;
useEffect(() => {
  smoothscroll.polyfill();
}, []);

@media not all and (min-resolution: 0.001dpcm) {
  @supports (-webkit-appearance: none) {
    scroll-snap-type: unset;
  }
}

Here a polyfill for scrollIntoView that fixes this for scroll-parents with scroll-snap-type set:
const { elementScrollIntoView } = await import('seamless-scroll-polyfill')
Element.prototype.scrollIntoView = function (options) {
  const duration = options?.behavior === 'smooth' ? 250 : undefined
  if (duration) {
    // The scroll-snap-type will interfer with polyfill smooth-scrolling,
    // so disable it on the scroll parent for the duration of the scroll.
    const { style } = this.parentElement
    style.setProperty('scroll-snap-type', 'none', 'important')
    setTimeout(() => {
      style.removeProperty('scroll-snap-type')
    }, duration)
  }
  elementScrollIntoView(this, options, { duration })
}


Support for Angular 19

Hello, any plans on supporting angular 19 soon.
This is what I get, when trying to install it Angular 19 project:
npm error code ERESOLVE
npm error ERESOLVE unable to resolve dependency tree
npm error
npm error While resolving<EMAIL_ADDRESS>npm error Found<EMAIL_ADDRESS>npm error node_modules/@angular/common
npm error<EMAIL_ADDRESS>from the root project
npm error
npm error Could not resolve dependency:
npm error peer<EMAIL_ADDRESS>from<EMAIL_ADDRESS>npm error node_modules/@omnedia/ngx-marquee
npm error   @omnedia/ngx-marquee@"*" from the root project
npm error
npm error Fix the upstream dependency conflict, or retry
npm error this command with --force or --legacy-peer-deps
npm error to accept an incorrect (and potentially broken) dependency resolution.
npm error
npm error
npm error For a full report see: ....
Thanks in advance

Hey, as soon as I find some time I'll add support for 19.* this week.

Updated with commit 24395a4 and release 2.0.0


workflows

JIRA Ticket
Ticket: https://ihm-it.atlassian.net/browse/IHRACP-5340
Action: update readme
Description
Major Changes:
update readme
Better shalls
How To Test
just read it
Test Data
Related PRs

Do we want GHA in this ticket scope.

need to update readme


In Strictly greater number

yourr code is invalid for input like
1 1 1 1 1 1 1 1 1 1

Thank You for taking Interest In My repo
I checked this Can I assign you to do correct it


AAv4 Template and public pages

Full update to AAv4 templates and public pages.
The screenshots for the readme you have to update yourself :-)

I'm going to merge then then make a few changes


Fix propagation of optimized predicates on nested projections

Which issue does this PR close?

Closes #3073.
Rationale for this change

This PR prevents the removal of predicates without referencing columns (WHERE FALSE, WHERE 1=1, etc.) that might have been created during the column name replacing phase (on filter pushdown optimizer when dealing with projections specifically).
What changes are included in this PR?
Columnless predicates are now collected into a separate entity, stripped away from the actual list of filters when switching the projection/filter and then re-applied.
Are there any user-facing changes?
This should fix the bug referenced in #3073

I'm not sure if this fixes the bug in issue #3073? I'm not totally convinced we should not propagate filters without column (e.g. constants), the result should remain the same whether it is propagated or not. Issue #3073 seems to be about a filter expression with a column, that somehow doesn't filter the row out.
Could we add a test for #3073 here?

Issue https://github.com/apache/arrow-datafusion/issues/3073 seems to be about a filter expression with a column, that somehow doesn't filter the row out.

@Dandandan since the example in #3073 uses an in-memory table, the filter where f.a=2 got replaced with where 1=2 in a step of the optimizer (which, at least for me, makes perfect sense) when we are rewriting columns here:
https://github.com/apache/arrow-datafusion/blob/873b071dff1a6099d30abdd24437e083a60e2686/datafusion/optimizer/src/filter_push_down.rs#L396

I'm not sure if this fixes the bug in issue https://github.com/apache/arrow-datafusion/issues/3073? I'm not totally convinced we should not propagate filters without column (e.g. constants), the result should remain the same whether it is propagated or not.

There is already a similar work done by an earlier step of the filter pushdown optimizer which ensures that WHERE 1=2 (and other variants of constant/columns-less filters) are not propagated (since we can't rewrite them if there are no earlier projections to propagate, unlike regular filters). The main thing this PR does is allow this check to be also carried out in Projection part where filters might change due to how we propagate constants.
https://github.com/apache/arrow-datafusion/blob/873b071dff1a6099d30abdd24437e083a60e2686/datafusion/optimizer/src/filter_push_down.rs#L358-L368

Add test for https://github.com/apache/arrow-datafusion/pull/3037 and explain why changes are needed

Definitely, will be working on that!

Issue https://github.com/apache/arrow-datafusion/issues/3073 seems to be about a filter expression with a column, that somehow doesn't filter the row out.

@Dandandan since the example in #3073 uses an in-memory table, the filter where f.a=2 got replaced with where 1=2 in a step of the optimizer (which, at least for me, makes perfect sense) when we are rewriting columns here:
https://github.com/apache/arrow-datafusion/blob/873b071dff1a6099d30abdd24437e083a60e2686/datafusion/optimizer/src/filter_push_down.rs#L396

I'm not sure if this fixes the bug in issue https://github.com/apache/arrow-datafusion/issues/3073? I'm not totally convinced we should not propagate filters without column (e.g. constants), the result should remain the same whether it is propagated or not.

There is already a similar work done by an earlier step of the filter pushdown optimizer which ensures that WHERE 1=2 (and other variants of constant/columns-less filters) are not propagated (since we can't rewrite them if there are no earlier projections to propagate, unlike regular filters). The main thing this PR does is allow this check to be also carried out in Projection part where filters might change due to how we propagate constants.
https://github.com/apache/arrow-datafusion/blob/873b071dff1a6099d30abdd24437e083a60e2686/datafusion/optimizer/src/filter_push_down.rs#L358-L368

Isn't the issue then that the propagated filters without column are not added to the plan at all, even when we are at the bottom of a plan? E.g. a propagated where false still should be present regardless of whether it has columns in it.

Isn't the issue then that the propagated filters without column are not added to the plan at all, even when we are at the bottom of a plan? E.g. a propagated where false still should be present regardless of whether it has columns in it.

Exactly, at least that is my understanding of this issue. I thought as is, it would be similar to the existing behaviour from #225 but this time done on the projection level rather than filter level.
If it makes sense, I can also change the logic in issue_filters so that when pushing down we can issue filters for both used filters + all the column-less filters.
https://github.com/apache/arrow-datafusion/blob/873b071dff1a6099d30abdd24437e083a60e2686/datafusion/optimizer/src/filter_push_down.rs#L125-L145

@Dandandan I did a re-implementation using the approach I've described below (handling this on the issue_filters level rather than individual plans) in here. I'd be happy to include it here it makes more sense.

Yes, this makes most sense, and simplifies the implementation quite a bit.

Codecov Report

Merging #3228 (ad0a93d) into master (873b071) will increase coverage by 0.01%.
The diff coverage is 97.50%.

@@            Coverage Diff             @@
##           master    #3228      +/-   ##
==========================================
+ Coverage   85.91%   85.93%   +0.01%
==========================================
  Files         294      294
  Lines       53443    53469      +26
==========================================
+ Hits        45918    45946      +28
+ Misses       7525     7523       -2

Impacted Files
Coverage Δ

datafusion/optimizer/src/filter_push_down.rs
98.36% <94.73%> (+0.13%)
:arrow_up:

datafusion/core/tests/sql/projection.rs
97.38% <100.00%> (+0.41%)
:arrow_up:

datafusion/expr/src/logical_plan/plan.rs
78.55% <0.00%> (+0.17%)
:arrow_up:

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

Thanks again @isidentical


Is mapDispatch2Props really necessary?

I've searched for this topic but none result matched the case I'm going to talk about.
In react-redux, we use mapDispatch2Props to inject access to dispatch into components(containers) like:
// container.js
onSubmit() {
  this.props.addUser(this.state.formData);
}

function mapDispatch2Props(dispatch) {
  return {
    addUser(data) {
      dispatch({ type: ADD_USER, data });
    }
  }
}

However dispatch is actually accessible anywhere as long as you import the store, so we don't need this injection.
// utils.js
import store from '../store';

export function addUser(data) {
  store.dispatch({ type: ADD_USER, data });
}

// containers.js
import { addUser } from '../utils';

onSubmit() {
  addUser(this.state.formData);
}

I know this seems breaking the "all data should come in as props" rule of React, but importing and using util functions are very common. Say we need to display a timestamp in some specific format:
import { formatTime } from '../utils';

render() {
  return (
    <span>{formatTime(this.props.time)}</span>
  );
}

And this approach prevents your component(container) from being aware of redux stuff. So I wonder if it's a good replacement of mapDispatch2Props, especially when not taking ownProps.

import store from '../store';

I think you are not using server-side rendering.
When you do server-side rendering, store should be created per request to due with the thread-safe problem.
Thus store(also dispatch() and getState()) should be passed from context.

@kouhin No I'm not. That makes sense.
Then what about none-ssr occasions?

No, mDTP isn't strictly necessary. You're free to get access to dispatch or bind it to your action creators by other means, but for most it's a nice convenience. Especially in the object shorthand form.

Also please see the Redux FAQ entry on why we discourage importing the store directly.


RTX Code Release (PI-4): T-Route RnR Enhancements to PI-3 Deliverables through v2.2 Hydrofabric integration

Today, Raytheon demo'd the FIM-Services capability to:

Make T-Route compatible with the current version of the Enterprise Hydrofabric. The requirement is verified through a demonstration of the deployment of Replace and Route using Terraform IaC

Thus, this PR builds on the changes introduced in #837 to:

Enhance Replace and Route through the introduction of an API
Allow the Replace and Route API to be configurable to work with different versions of the hydrofabric

this change supports v2.2 and v20.1

Additions

Additional API data for v2.2 of the hydrofabric
Changes to the HYFeaturesNetwork.py function to support v2.2 routing
A Notebook for testing v2.2 hydrofabric integration

Removals

Closing #837 as this code enhances / builds from the solutions delivered in that PI

Changes

Additions to include the Open Database License (ODbL) within the LICENSE to credit Lynker Spatial for supplying the Hydrofabric v2.2 dataset used in our testing.

Testing

See test/LowerColorado_HYFeatures_v22/run_v22.ipynb to validate the v2.2 integration
See test/api/README.md for information on testing the

Screenshots

Notes

Todos

Checklist

[ ] PR has an informative and human-readable title
[ ] Changes are limited to a single goal (no scope creep)
[ ] Code can be automatically merged (no conflicts)
[ ] Code follows project standards (link if applicable)
[ ] Passes all existing automated tests
[ ] Any change in functionality is tested
[ ] New functions are documented (with a description, list of inputs, and expected output)
[ ] Placeholder code is flagged / future todos are captured in comments
[ ] Visually tested in supported browsers and devices (see checklist below :point_down:)
[ ] Project documentation has been updated (including the "Unreleased" section of the CHANGELOG)
[ ] Reviewers requested with the Reviewers tool :arrow_right:

Testing checklist
Target Environment support

[ ] Windows
[ ] Linux
[ ] Browser

Accessibility

[ ] Keyboard friendly
[ ] Screen reader friendly

Other

[ ] Is useable without CSS
[ ] Is useable without JS
[ ] Flexible from small to large screens
[ ] No linting errors or warnings
[ ] JavaScript tests are passing

Note:

A SIGNIFICANT amount of the code introduced is data. If there is a better solution than to include forcing files where each hour is contained in one csv, I'm happy to participate.


Add remove attribute method ot SessionWriter

Goal

Testing

Release Notes

WHAT:
WHY:
WHO:

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#693
#692  👈
#691
#690
master

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

Join @bidetofevil and the rest of your teammates on  Graphite


handle case where artifact was filled by igor

Currently Docker images filled by Keel are handled correctly in the plugin but docker image created by Igor are not supported and thus falls back to  ${account_name}-registry naming convention.
This should be fixed in Keel as well. It's going to require changes to keel-core and keel-artifacts. However, this plugin should be able to handle it since all the information needed to get the full image path is provided.
fixes #102

Tests are failing due to reasons unrelated to code change. Need to investigate next week.


Question: Authenticating API endpoint

Hello,
Is it possible to put authentication in front of the API ?
I would like to use oauth maybe with a lib like passport (http://passportjs.org/docs/overview) but maybe it's not so easy. So if you have any alternative roads :)
There is a 'boilerplate' repo for doing it directly with Serverless. Can it be an inspiration ?
Thanks for your answer !
BTW so far, Claudia seems a great tool to use AWS lambda

you should be able to use Cognito from AWS. Claudia will supply the cognito IDs with the request. Check out https://github.com/claudiajs/claudia-api-builder#the-request-object for more info on the request object.

Thanks !

if you get stuck somewhere because claudia isn't creating the right template, let me know what you'd like to create and we can integrate it. I haven't used Cognito with Api Gateway, so it might be possible that something isn't covered by Claudia yet

The request object documentation seems to have been moved here: https://github.com/claudiajs/claudia-api-builder/blob/master/docs/api.md


DB::()->delete() returns zero always

Laravel 6.18.24
PHP Version: 7.4
Database Driver & Version: Mysql
Thanks for developing Laravel.
I have recently updated to the Laravel 6.18.24 and realised that DB::()->delete() always returns zero. It used to be right but seems problematic with the new update.
Here is the code that I use:
 $dlt = DB::table('members')
            ->whereIn('id', array_XXX)
            ->where('id2', YYY)
            ->delete();
dd($dlt);
0

I should add that the deletion is successful however the return value in $dlt is zero, then any check on it fails and ends up misleading message to the client.
I am also aware of the bug in update() that returns zero if there is nothing to update. Please can you kindly take a look at this as well?

I get the number of deleted records back when I try to run a delete statement.

Thanks. Do you have the same version of Laravel as I have?

Yes.

Its weird. do you use ->whereIn('id', array_XXX) or where?

Tried both.

puzzled. Do you have any guess of the reason? I use redis cache, could it be a reason?

out of curiosity what is the output of whereIn()->delete(). Is it an array of record indexes?

@omidMolaverdi it's best that you try a support channel at this point as this issue tracker is for reporting bugs:

Laracasts Forums
Laravel.io Forums
StackOverflow
Discord
Larachat
IRC

Okay thanks a lot.


editorconfig: Extend with more current settings

I used the style in the existing files as a reference for consistency.

Can one of the admins verify this patch?

Can one of the admins verify this patch?

Can one of the admins verify this patch?

ok to test

Merged build finished. Test PASSed.

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


fix tiny typo in readme

You definitely should merge this in. 😏

Ha.....I see what you did there 👈 😄 👈


add unit tests for the cape feature extractor

addresses #1498 and #1546
Checklist

[ ] No CHANGELOG update needed

[x] No new tests needed

[x] No documentation update needed

The requested changes should be addressed now. I am unsure however why code_style tests are failing for base_extractor.py here but not in the other PR.

lets close this PR because the tests are added in #1546


Add Block and Transaction Structs

Description
Add Block and Transaction types as defined in StrakNet's Technical Documentation.
Changes:

Add Block Struct
Add Deploy Transaction struct
Add Invoke Transaction struct
Add Declare Transaction struct

Types of changes

New feature (non-breaking change which adds functionality)

Testing
Requires testing: Yes
Did you write tests??: No

@IronGauntlets

@IronGauntlets According to the docs here, deploy transaction and invoke transaction 0 is depreciated. We should be using invoke transaction 1 instead.

should the Transaction.go file be capitalized?

No, it should not. I have updated the file name.


Add MailChimp to styles library

As with other PR, I cannot figure out the latest zip archive link

@jdkato MailChimp should also now be ready

@jdkato MailChimp should also now be ready

The assets are missing, https://github.com/testthedocs/MailChimp/releases/latest/download/MailChimp.zip does not exist

This could help releasing the required MailChimp.zip https://github.com/testthedocs/MailChimp/pull/3
(There are other ways to do it.)


New Feat: Copy & Paste / Import & Export Button For Headers and Params

I have checked the following:

[X] I've searched existing issues and found nothing related to my issue.

Describe the feature you want to add
A nice quality of life feature be a copy and past button for request headers and params. Currently for each request if it has the same fields of another you need to individually copy and past the name value pairs separately. Having a simple copy and past button would be just a quality of life feature reducing the number of clicks and button presses needed when making new but similar request files.
Mockups or Images of the feature

I feel like this adding more button here would clutter up the UI. I feel like adding a Bulk edit more would better because it only adds one button and adds new functionality: https://github.com/usebruno/bruno/pull/706

Yeah I would agree if they were to go with a copy button per each it should be like a on hover button to show. though I think I like the bulk more.
honestly it just drives me crazy when there's a good handful of headers that need to be used for every request with no good way of bringing them over.

Closed as duplicate of #706


feat(macos): show checkmark/xmark in Dock icon on transfer success/failure

closes https://github.com/localsend/localsend/issues/1772
Uses https://github.com/localsend/localsend/blob/main/app/macos/Runner/Assets.xcassets/AppIcon.appiconset/logo-1024-mac-256.png plus checkmark\xmark from Apple's SF Symbols, and adding A little transparency and shadow using Adobe Photoshop. Inspired by WebStorm and Slack (see Materials).
Demo:
https://github.com/user-attachments/assets/695f8675-9d3f-4977-ad10-54fc4867c8dc
Materials:
logo.zip

@Tienisto Can you help me at which points in the code to integrate it on the Flutter side? Or maybe merge it like this and you'll take care of the Flutter part later?
It should actually be displayed as long as the error/sucess screen is still displayed in the app interface.

@Tienisto Note that currently this will do nothing, calls to TaskbarHelper.setTaskbarIcon must be added


Business Objects connector Questions

Tableau 9.1 windows user trying to connect to Business Objects QAAWS
The documentation had a few gaps I could have used some assistance with, considering I have never used any of the tools this project seems dependent on - below are the steps I took, and it looks like I am close, but not there 100% yet.  Please update the documentation to include these steps for the complete n00bs like myself.

download node.js and install it (https://nodejs.org/en/)
Update NPM by typing "npm install npm -g"
Install Coffee "npm install -g coffee-script" (CoffeeScript is a little language that compiles into JavaScript.)
Download the tableau-web-table-connector zip, and extract into a directory
Open a command shell as Administrator
run "npm install" from the administrator command shell while in the extracted directory
run "build-sap-bo.sh" from the administrator command shell while in the extracted directory
run "coffee coffee/sap_bo/proxy_server.coffee" from the administrator command shell while in the extracted directory

Once I get this working on my workstation, what needs to be done on our Tableau server to allow this to be refreshed on a schedule?

hey, you should import the generated HTML file to the Tableau Server
https://onlinehelp.tableau.com/current/server/en-us/datasource_wdc.htm#importing
If you have 10 mins I can show it over webex

Thanks so much for this document, I am going to give it a shot in one of our lower environments once I can successfully query from my workstation - if I have any trouble I may take you up on your offer.


cmd/go/internal/load: transient "unexpected directory layout" error

What version of Go are you using (go version)?
go version go1.9beta2 darwin/amd64
What operating system and processor architecture are you using (go env)?
GOARCH="amd64"
GOBIN=""
GOEXE=""
GOHOSTARCH="amd64"
GOHOSTOS="darwin"
GOOS="darwin"
GOPATH="/Users/tsuna/go"
GORACE=""
GOROOT="/usr/local/Cellar/go/1.9beta2/libexec"
GOTOOLDIR="/usr/local/Cellar/go/1.9beta2/libexec/pkg/tool/darwin_amd64"
GCCGO="gccgo"
CC="clang"
GOGCCFLAGS="-fPIC -m64 -pthread -fno-caret-diagnostics -Qunused-arguments -fmessage-length=0 -fdebug-prefix-map=/var/folders/8j/3m62t7m92958kqxt9s3chtrr0000gn/T/go-build954574771=/tmp/go-build -gno-record-gcc-switches -fno-common"
CXX="clang++"
CGO_ENABLED="1"
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"
PKG_CONFIG="pkg-config"

What did you do?
While trying to update a package:
$ go get -u github.com/uber/go-torch
unexpected directory layout:
        import path: github.com/mattn/go-colorable
        root: /Users/tsuna/go/src
        dir: /Users/tsuna/go/src/github.com/fatih/color/vendor/github.com/mattn/go-colorable
        expand root: /Users/tsuna/go/src
        expand dir: /Users/tsuna/go/src/github.com/fatih/color/vendor/github.com/mattn/go-colorable
        separator: /
$ go get -u github.com/uber/go-torch
<success>

What did you expect to see?
go get should've either failed consistently or succeeded from the first time.
What did you see instead?
A transient and somewhat confusing "unexpected directory layout" error.

@tsuna is this a regression from Go1.8?
/cc @rsc

I don't know, I've never seen this error before.  The only similar issue I've found on GitHub was alecthomas/gometalinter#242 and that was reported with go1.8 already, but I don't know if it's the same problem.

Aye aye, gotcha. Thanks for the update. I'll mark this as needing investigation and for Go1.10.

I'm seeing this on go1.8.3
 → dockerc run --rm api go version
Starting xxx_postgres_1 ... done
go version go1.8.3 linux/amd64

I can't see how @tsuna's report can happen unless there are symlinks involved. (Symlinks are basically unsupported.) Somehow the package in his $GOPATH/src/github.com/fatih/color/vendor/github.com/mattn/go-colorable has been loaded as if its fully-qualified import path were only "github.com/mattn/go-colorable" when in fact it should be "github.com/fatih/color/vendor/github.com/mattn/go-colorable". The most likely culprit for this kind of confusion is symlinks. I cannot reproduce the problem right now, although it should of course not be transient in any way.
I tried reproducing this with both Go 1.8 and Go 1.9, with no luck:
$ mkdir /tmp/gp1
$ GOPATH=/tmp/gp1 go1.9 get github.com/uber/go-torch
$ GOPATH=/tmp/gp1 go1.9 get -u github.com/uber/go-torch
$

If anyone can send a set of commands to reproduce this with a specific version of Go, starting with an empty GOPATH directory and not using Docker, that would be great.
@pbdeuchler since you didn't post any command output it's impossible to know whether you're really seeing the same problem @tsuna is or a different bug entirely.

I believe I am seeing the same issue with symlinks using Go 1.10-beta1 (was fine in Go 1.9.2).
In my case, my project is copied into /source in the docker golang:1.10-rc-alpine image.
I then symlink the directory to /go/src/github.com/user/someproject.
This is the output when I try to run go test -v $(go list ./... | grep -v /vendor/)t:
unexpected directory layout:
        import path: _/go/src/github.com/user/project/cockroachdb/cockroachdbpb
        root: /go/src
        dir: /go/src/github.com/user/project/cockroachdb/cockroachdbpb
        expand root: /go/src
        expand dir: /source/cockroachdb/cockroachdbpb
        separator: /

Sorry, you cannot use symlinks with the go tool, they break its concept of a single path to find a particular package.

On 10 Dec 2017, at 10:11, Francis Chuang<EMAIL_ADDRESS>wrote:
I believe I am seeing the same issue with symlinks using Go 1.10-beta1 (was fine in Go 1.9.2).
In my case, my project is copied into /source in the docker golang:1.10-rc-alpine image.
I then symlink the directory to /go/src/github.com/user/someproject.
This is the output when I try to run go test -v $(go list ./... | grep -v /vendor/)t:
unexpected directory layout:
import path: _/go/src/github.com/user/project/cockroachdb/cockroachdbpb
root: /go/src
dir: /go/src/github.com/user/project/cockroachdb/cockroachdbpb
expand root: /go/src
expand dir: /source/cockroachdb/cockroachdbpb
separator: /
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.


generate links with self on an index view

I plan on cutting a new release tomorrow evening!

@jeregrine nice, looking forward to it.

@jeregrine Hey there, any update on when to expect the new release?

@StareIntoTheBeard any chance you can squash your commits?  I think you may need to rebase on master, there's a lot of duplicated commits

Once we squash this and merge it I will cut a release


Typescript 3.7 support

We're not stuck on 4.1.3 as of https://www.github.com/googleapis/nodejs-storage/issues/952 (and actually are not having issues on 4.1.3 despite being on typescript 3.7.5).
What's blocking this restriction to be lifted?

:wave: there were breaking changes between TypeScript 3.6 and 3.7, which caused issues for users running Angular.
In the next major version bump (when we drop Node 8) we will unpin TypeScript.

TypeScript has been unpinned!


Restore the "physical model" shadow rendering implementation

It was temporarily disabled by https://github.com/flutter/flutter/pull/8636
The problem can be demo'd with the gallery Shrine app, see https://github.com/flutter/flutter/issues/8635
@jason-simmons

This also improved some of the performance benchmarks, most notably the average rasterizer frame time (though it made the worst case worse again). Presumably this is because painting shadows is now cheaper but we do it more often? If I'm right, maybe we can cache the shadows that get painted a few times? The regression here (which this revert reverted) is tracked in https://github.com/flutter/flutter/issues/8501.

I believe @jason-simmons did this?  Speculatively closing.


Locking issues are preventing continued operation with Spanner

We consistently have issues with locking related to our spanner db access. Seeming like these are related to queued jobs. The issues tend to disappear when we use the sync driver for the queued job processing. Here is an example of one of our exception traces which shows the result but not the likely cause of the issue.
Next Illuminate\Database\QueryException: Failed to open lock file. (SQL: select * from `users` where `id` = @p0 limit 1) in /nix/store/krcnq71n6yh724z8dq11lr68lq7gvbwf-composer-laravel-laravel/vendor/colopl/laravel-spanner/src/Colopl/Spanner/Connection.php:441
Stack trace:
#0 /nix/store/krcnq71n6yh724z8dq11lr68lq7gvbwf-composer-laravel-laravel/vendor/laravel/framework/src/Illuminate/Database/Connection.php(629): Colopl\Spanner\Connection->runQueryCallback('select * from `...', Array, Object(Closure))
#1 /nix/store/krcnq71n6yh724z8dq11lr68lq7gvbwf-composer-laravel-laravel/vendor/colopl/laravel-spanner/src/Colopl/Spanner/Connection.php(250): Illuminate\Database\Connection->run('select * from `...', Array, Object(Closure))
#2 /nix/store/krcnq71n6yh724z8dq11lr68lq7gvbwf-composer-laravel-laravel/vendor/laravel/framework/src/Illuminate/Database/Query/Builder.php(2149): Colopl\Spanner\Connection->select('select * from `...', Array, true)

Any help in understanding and overcoming this issue would be appreciated.

And are you trying to update the same user multiple times?
And Is this query run within a transaction?
If so, this is probably the correct behavior since Spanner will lock the row if you select it in a read-write transaction.

And are you trying to update the same user multiple times?
And Is this query run within a transaction?
If so, this is probably the correct behavior since Spanner will lock the row if you select it in a read-write transaction.

Where this is happening is within a transaction, but only a single insert statement within the transaction. It only happens when we have background jobs running. The jobs do not write to the users table, only read from it. If we change the jobs to run synchronous then the issue goes away. It seems we only ever have issues with locking when running jobs, and then we have lock issues elsewhere in the code which may be related.

Where this is happening is within a transaction, but only a single insert statement within the transaction. It only happens when we have background jobs running. The jobs do not write to the users table, only read from it. If we change the jobs to run synchronous then the issue goes away. It seems we only ever have issues with locking when running jobs, and then we have lock issues elsewhere in the code which may be related.

I believe reading will still lock the row in a read-write transaction.
QueryException should have an underlying exception.
I need to see that in order to understand it further.
You might want to try looking at the lock statistics to see what's happening on the server side.
https://cloud.google.com/spanner/docs/introspection/lock-statistics

I believe reading will still lock the row in a read-write transaction.
QueryException should have an underlying exception.
I need to see that in order to understand it further.
You might want to try looking at the lock statistics to see what's happening on the server side.
https://cloud.google.com/spanner/docs/introspection/lock-statistics

We have averted our issue temporarily by sync driver for jobs processing. In our many attempts to utilize jobs for background processing, we have failed with this library due to lock conditions for same row jobs. We even wrote a non overlapping job Middleware to avoid this issue, but at the end of the day, it is really no different than the sync driver. If time allows in the future, I may attempt a correct approach to conquer this problem... but for now it will ne synchronous processing.

I have no way of reproducing this so I'll go ahead and close it for now.
Please send a pull request if you have a solution to this problem.
Thanks.


[K-Bug] Incorrect #Exists instantiation

What component is the issue in?
Front-End
Which command

[ ] kompile
[ ] kast
[ ] krun
[X] kprove
[ ] kprovex
[ ] ksearch

What K Version?
v5.5.19-0-gb0c703a959
Operating System
MacOS (Apple Silicon/AMD)
K Definitions (If Possible)
exists-issue.k
module EXISTS-ISSUE-SYNTAX
  imports INT-SYNTAX

  syntax Int ::= F1(Int) [function, no-evaluators]
  syntax Int ::= F2(Int) [function, no-evaluators]

  syntax Int ::= FX(Int) [function]
endmodule

module EXISTS-ISSUE
     imports BOOL
     imports INT
     imports EXISTS-ISSUE-SYNTAX

     rule FX(I) => F1(I) +Int F2(I +Int 2)

     rule #Ceil(F1(I:Int))  => #Exists X:Int. { I #Equals 2 *Int X } [simplification]
     rule #Ceil(F2(I:Int))  => #Exists X:Int. { I #Equals 2 *Int X } [simplification]

     rule #Ceil(FX(I)) => #Ceil(F1(I)) #And #Ceil(F2(I)) [simplification]
endmodule

exists-example.k
requires "exists-issue.k"
requires "domains.md"

module EXISTS-EXAMPLE-SYNTAX
    imports EXISTS-ISSUE-SYNTAX

endmodule

module VERIFICATION
  imports EXISTS-EXAMPLE-SYNTAX
  imports EXISTS-ISSUE
endmodule

module EXISTS-EXAMPLE
  imports VERIFICATION

  claim
    <k>
      FX(I:Int) => I ...
    </k>
endmodule

Steps to Reproduce
kompile exists-issue.k --backend haskell
kprove exists-example.k

The claim is (wrongly) vacuously proven:
Claim proven during initialization: [REDACTED]/exists-example.k:18:5-20:9
    The left-hand side of the claim may be undefined.
#Top

because (I believe) the existentially quantified X is considered to be the same X when instantiated, instead of having two separate Xs, say X0 and X1. If one renames X in any of the #Ceil simplifications, one gets the expected result.
Expected Results
A failing proof, as the claim does not hold.

@PetarMax what happens if you write the rules like:
rule #Ceil(F1(I:Int))  => { I #Equals 2 *Int ?X }

@ana-pantilie It is not allowed by the front-end:
[Warning] Compiler: Variable '?X' defined but not used. Prefix variable name
with underscore if this is intentional.
	Source(/Users/petarmax/Projects/RV/exists-issue/exists-issue.k)
	Location(17,51,17,53)
	17 |	     rule #Ceil(F1(I:Int))  => { I #Equals 2 *Int ?X } [simplification]
	   .	                                                  ^~
[Error] Compiler: Cannot encode equations with existential variables to KORE.
 If this is desired, please use #Exists with regular variables.
 Offending variables: [?X]
 context: rule
#Ceil{Int,#SortParam}(`F1(_)_EXISTS-ISSUE-SYNTAX_Int_Int`(I))=>#Equals{Int,#SortParam}(I,`_*Int_`(#token("2","Int"),?X))
requires #token("true","Bool") ensures #token("true","Bool")
[UNIQUE_ID(7e9a206dd6ce5075947cf25f90d20a73db3736330a4676c6751710df48550b36),
org.kframework.attributes.Location(Location(17,11,17,55)),
org.kframework.attributes.Source(Source(/Users/petarmax/Projects/RV/exists-issue/exists-issue.k)),
org.kframework.definition.Production(syntax #RuleContent ::= #RuleBody
[klabel(#ruleNoConditions), symbol]), simplification, sortParams({Q0})]

@PetarMax is this blocking you? It seems that renaming the variables is a workaround. I agree that it's concerning that:

We are treating #Ceil rules as equations (equations should be between elements in the algebra of function-like terms, not about set-elements like #Ceil).
The variable X is being conflated between th erules.

It seems tha tthe backend should not be storing #Ceil simplification rules in the same bucket of rules as equational simplification rules.


Bump toi version in toix to 1.2.0

Bumps toi version in tox to the newest 1.2.0.

Codecov Report

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

@@          Coverage Diff          @@
##           master   #137   +/-   ##
=====================================
  Coverage     100%   100%
=====================================
  Files           3      3
  Lines         410    410
  Branches       56     56
=====================================
  Hits          410    410

Continue to review full report at Codecov.

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


AI Castles for the future

With 2.06 having the option to disable the AI from putting their production buildings on sleep, wouldn't this mean we can freely improve all AI castles such as giving Lords better eco/bread production?
That's just a thought.

Not bad idea

I had the thought too. There's still the issue of AIs buying and instantly selling resources they don't normally use. Pitch being the best example. Put pitch on say, one of Pig's castles and he will dig himself into a hole from buying and selling so much pitch for infinite.

Also, just because the AI doesn't put everything to sleep immediately does NOT mean that the economy scales up easily.
If the AI has no wheat or flour (and does not have tons of gold reserves for buying the resources right away either), this will still mean that a large bunch of millers / bakers are waiting for resources, doing nothing and consuming food/beer nonetheless.
And when low on gold, pretty much any AI character prioritizes recruiting troops over buying raw materials to keep production chains going.
So what does the fix for the AI's overly aggressive setting-everything-to-sleep-policy do? It improves the worst case endless cycle of wasting resources, but it does not fix general economic stupidity of the AI. Also, as @Panbutt already mentioned, it does not mean that you could now give e.g. inns to Richard: He will still not build hops farms, and even if he would, he'd sell hops immediately, and beer too, and never buy those resources either.

And another remark: With custom AIVs, one can generally try to increase in-castle economy of any AI, sure. But that comes at a cost: More expensive to set up, more space needed, probably harder to defend, possibly more susceptible to fire (unless you go the extra mile to take measures against that, which usually results at least in more space demand and higher population from additional wells).
Thus, for many AI characters, the "usability" of castles with larger economy is questionable and must be finetuned and tested on a case-by-case basis.


NFV fixes since Jojoba

This branch neutron-sync-master and neutron-sync-agent more robust and failure proof. Additionally their output is less verbose and now they print timestamped log messages instead.

I have merged this into the nfv branch for wider testing now. (Surprised Github doesn' show that? You can see it when examining the individual commits at least.)

Is SnabbBot process nfv btw? or do I need to ge my &*@! bench_env
working again? :)

I assume you mean if SnabbBot will run for PRs on the snabb-nfv repo? No
it won't. I can make it so tomorrow. For now SnabbBot will run every
minute (e.g. as fast as it can) on SnabbCo/snabbswitch though. That is
with a lock so that it won't interfere with itself.

Thanks. I would like to have SnabbBot runs on the nfv branch too please. I will PR this to master to trigger for now.

Merged into nfv branch. Will be included on a future PR from there to master.


Remove GWT and Errai module files

@manstis should this module be used by UI/GWT? I couldn't find any use across kie-wb-common, drools-wb and jbpm-wb. It is currently breaking the build due to invalid package. I saw your comment here https://github.com/droolsjbpm/jbpm/pull/746 and tried using super-source but couldn't get Errai to compile properly still.  Suggesting to remove it but feel free to close this PR if not reasonable.

@cristianonicolai OK. I dabbled around a bit; trying to provide a real fix for QName but it's more effort than we have time for at present (and since this module is not used client-side at the moment) I'll go with the simple approach of removing (for now).


Autocomplete for eslint.codeAction.disableRuleComment should insert separateLine instead of newLine

resolves #673
Previous Behavior
Creating a rule for eslint.codeAction.disableRuleComment will autocomplete with location: 'newLine' and a warning that newLine is not a valid location.
New Behavior
It should autocomplete with location: 'separateLine'

Good catch. Thanks!


Skip already customized operator bundle images

This patch adds a condition to verify if a operator bundle image needs to be updated or not.
The following scenario is broken in current version of the script, and is the workflow that will be used on CI:

A PR to *-operator is proposed
Job runs and build operator, bundle and catalog images.
Images are pushed to CI registry.
openstack-operator go.mod is updated with new api reference
custom-bundle.Dockerfile is modified to point to CI operator bundle image
Job builds openstack-operator images, and push to CI registry
Deploy steps validates openstack-operator image

When generating openstack-operator bundle, it fails to find the bundle image on user's quay.io account.
The new condition ignores bundle images that were already modified.

FYI, where I am replacing operator bundle image, before build openstack-operator[1].
It fails when github user doesn't have the image built in its quay registry.
In this case, CI is building and pushing to RDO registry.
[1] https://github.com/openshift/release/pull/35553/files#diff-f535e0344a0aeb7b9ebdd1d5273e0c372d24a40e397d1f1ea78a45bba82338d8R140-R144

/assign @dprince
Hi @dprince @stuggi - we need a way to avoid replacing image bundle path in bundle.Dockerfile when the api is set to a GH user commit. This is the scenario that CI will cover, since CI will build and push images to its own registry. We are manually doing the replacement in job steps.
Can you take a look on this PR?


Optimize WASM arguments and returns

Use LLVM arguments and returns where possible (no GC references) instead of passing them on the shadow stack. The argument optimization saves 3% optimized (5% compressed). The return change adds about 0.5% to the uncompressed file, but saves about 6% compressed. They both save size in debug.  This also greatly simplifies debugging and reading code. Also includes a fix to the class constructor runner where it was calling cctors with the wrong signature and some test fixes.

Thanks for the fast review!


Created Recipients object rather than it being a stdClass

the recipients property in MessageBird\Objects\Message is currently just a stdClass. This PR creates a Recipients Object in its place to allow better autocomplete in IDEs

@rfeiner can you have a look at this please?

Thanks for your contribution, @JohnstonCode! I personally think it would be super nice to add, however it is also a rather major change.
Do you think we could maybe keep a way to preserve the old behaviour so that existing users can gradually migrate instead of having to do one "big bang" change? Maybe add a parameter to the getter, and by default keep the old behaviour in the first step, and then in a next version switch the default to the Recipient object, before finally dropping the array return completely.
wdyt?

Yeah that makes sense. I will update the PR later today.

Looking back through the code i now remember why i did what i did. So the json_decode here turns MessageBird\Objects\Message->recipients not into an array but into a stdClass. So the changes i have proposed are purely to change the stdClass into Recipients all the fields will map the same so there should be no breaking changes unless people are checking that MessageBird\Objects\Message->recipients is a stdClass

Ah, you're right! That's on me for missing that, sorry. I will give this a spin later today and, if all goes well, merge it and release a new version.
Thanks again for your contribution!

No its my fault, I should have explained it better when i opened the PR.
Brilliant, thank you.

@modprobe did you manage to give it a try?

@JohnstonCode Merged and released with v1.19.1! Thanks again!


Add category buttons to homepage, create browse page. add category pictures

Description
Give a description of what this PR is about.
Jira Ticket

https://garrysjh.atlassian.net/browse/DIP-[your ticket number]

Fixed merge conflicts

Itemcard and vertical view listings still uses non-existent products placeholder

@W0ShiiSky @yongjin372k can yall make this link to the backend


[FR] add handling for scsi disks

Hey @PhilipMay  as promised, here is the output of smartctl --xall <scsi disk>
smartctl 7.2 2020-12-30 r5155 [x86_64-linux-5.10.103] (local build)
Copyright (C) 2002-20, Bruce Allen, Christian Franke, www.smartmontools.org
=== START OF INFORMATION SECTION ===
Model Family:     Samsung based SSDs
Device Model:     <device model>
Serial Number:    <device serial number>
LU WWN Device Id: 5 002538 e406ebf30
Firmware Version: <firmware version>
User Capacity:    960,197,124,096 bytes [960 GB]
Sector Sizes:     512 bytes logical, 4096 bytes physical
Rotation Rate:    Solid State Device
Form Factor:      2.5 inches
TRIM Command:     Available, deterministic, zeroed
Device is:        In smartctl database [for details use: -P show]
ATA Version is:   ACS-4 T13/BSR INCITS 529 revision 5
SATA Version is:  SATA 3.2, 6.0 Gb/s (current: 6.0 Gb/s)
Local Time is:    Tue Jun 21 07:33:36 2022 UTC
SMART support is: Available - device has SMART capability.
SMART support is: Enabled
AAM feature is:   Unavailable
APM feature is:   Unavailable
Rd look-ahead is: Enabled
Write cache is:   Enabled
DSN feature is:   Unavailable
ATA Security is:  Disabled, frozen [SEC2]
Wt Cache Reorder: Enabled
=== START OF READ SMART DATA SECTION ===
SMART overall-health self-assessment test result: PASSED
General SMART Values:
Offline data collection status:  (0x00) Offline data collection activity
                    was never started.
                    Auto Offline Data Collection: Disabled.
Self-test execution status:      (   0) The previous self-test routine completed
                    without error or no self-test has ever
                    been run.
Total time to complete Offline
data collection:        (    0) seconds.
Offline data collection
capabilities:            (0x53) SMART execute Offline immediate.
                    Auto Offline data collection on/off support.
                    Suspend Offline collection upon new
                    command.
                    No Offline surface scan supported.
                    Self-test supported.
                    No Conveyance Self-test supported.
                    Selective Self-test supported.
SMART capabilities:            (0x0003) Saves SMART data before entering
                    power-saving mode.
                    Supports SMART auto save timer.
Error logging capability:        (0x01) Error logging supported.
                    General Purpose Logging supported.
Short self-test routine
recommended polling time:    (   2) minutes.
Extended self-test routine
recommended polling time:    (  60) minutes.
SCT capabilities:          (0x003d) SCT Status supported.
                    SCT Error Recovery Control supported.
                    SCT Feature Control supported.
                    SCT Data Table supported.
SMART Attributes Data Structure revision number: 1
Vendor Specific SMART Attributes with Thresholds:
ID# ATTRIBUTE_NAME          FLAGS    VALUE WORST THRESH FAIL RAW_VALUE
  5 Reallocated_Sector_Ct   PO--CK   100   100   010    -    0
  9 Power_On_Hours          -O--CK   093   093   000    -    30796
 12 Power_Cycle_Count       -O--CK   099   099   000    -    70
177 Wear_Leveling_Count     PO--C-   098   098   005    -    128
179 Used_Rsvd_Blk_Cnt_Tot   PO--C-   100   100   010    -    0
180 Unused_Rsvd_Blk_Cnt_Tot PO--C-   100   100   010    -    2837
181 Program_Fail_Cnt_Total  -O--CK   100   100   010    -    0
182 Erase_Fail_Count_Total  -O--CK   100   100   010    -    0
183 Runtime_Bad_Block       PO--C-   100   100   010    -    0
184 End-to-End_Error        PO--CK   100   100   097    -    0
187 Uncorrectable_Error_Cnt -O--CK   100   100   000    -    0
190 Airflow_Temperature_Cel -O--CK   069   053   000    -    31
194 Temperature_Celsius     -O---K   069   047   000    -    31 (Min/Max 22/53)
195 ECC_Error_Rate          -O-RC-   200   200   000    -    0
197 Current_Pending_Sector  -O--CK   100   100   000    -    0
199 CRC_Error_Count         -OSRCK   100   100   000    -    0
202 Exception_Mode_Status   PO--CK   100   100   010    -    0
235 POR_Recovery_Count      -O--C-   099   099   000    -    42
241 Total_LBAs_Written      -O--CK   099   099<PHONE_NUMBER>24128
242 Total_LBAs_Read         -O--CK   099   099<PHONE_NUMBER>605
243 SATA_Downshift_Ct       -O--CK   100   100   000    -    0
244 Thermal_Throttle_St     -O--CK   100   100   000    -    0
245 Timed_Workld_Media_Wear -O--CK   100   100   000    -    65535
246 Timed_Workld_RdWr_Ratio -O--CK   100   100   000    -    65535
247 Timed_Workld_Timer      -O--CK   100   100   000    -    65535
251 NAND_Writes             -O--CK   100   100<PHONE_NUMBER>16576
                            ||||||_ K auto-keep
                            |||||__ C event count
                            ||||___ R error rate
                            |||____ S speed/performance
                            ||_____ O updated online
                            |______ P prefailure warning
General Purpose Log Directory Version 1
SMART           Log Directory Version 1 [multi-sector log support]
Address    Access  R/W   Size  Description
0x00       GPL,SL  R/O      1  Log Directory
0x01           SL  R/O      1  Summary SMART error log
0x02           SL  R/O      1  Comprehensive SMART error log
0x03       GPL     R/O      1  Ext. Comprehensive SMART error log
0x04       GPL,SL  R/O      8  Device Statistics log
0x06           SL  R/O      1  SMART self-test log
0x07       GPL     R/O      1  Extended self-test log
0x09           SL  R/W      1  Selective self-test log
0x10       GPL     R/O      1  NCQ Command Error log
0x11       GPL     R/O      1  SATA Phy Event Counters log
0x12       GPL     R/O      1  SATA NCQ Non-Data log
0x13       GPL     R/O      1  SATA NCQ Send and Receive log
0x30       GPL,SL  R/O      9  IDENTIFY DEVICE data log
0x80-0x9f  GPL,SL  R/W     16  Host vendor specific log
0xa5           SL  VS      16  Device vendor specific log
0xce           SL  VS      16  Device vendor specific log
0xe0       GPL,SL  R/W      1  SCT Command/Status
0xe1       GPL,SL  R/W      1  SCT Data Transfer
SMART Extended Comprehensive Error Log Version: 1 (1 sectors)
No Errors Logged
SMART Extended Self-test Log Version: 1 (1 sectors)
No self-tests have been logged.  [To run self-tests, use: smartctl -t]
SMART Selective self-test log data structure revision number 1
 SPAN  MIN_LBA  MAX_LBA  CURRENT_TEST_STATUS
    1        0        0  Not_testing
    2        0        0  Not_testing
    3        0        0  Not_testing
    4        0        0  Not_testing
    5        0        0  Not_testing
Selective self-test flags (0x0):
  After scanning selected spans, do NOT read-scan remainder of disk.
If Selective self-test is pending on power-up, resume after 0 minute delay.
SCT Status Version:                  3
SCT Version (vendor specific):       256 (0x0100)
Device State:                        Active (0)
Current Temperature:                    31 Celsius
Power Cycle Min/Max Temperature:     28/34 Celsius
Lifetime    Min/Max Temperature:     22/53 Celsius
Specified Max Operating Temperature:    70 Celsius
Under/Over Temperature Limit Count:   0/0
SCT Temperature History Version:     2
Temperature Sampling Period:         1 minute
Temperature Logging Interval:        10 minutes
Min/Max recommended Temperature:      0/70 Celsius
Min/Max Temperature Limit:            0/70 Celsius
Temperature History Size (Index):    128 (113)
Index    Estimated Time   Temperature Celsius
 114    2022-06-20 10:20    31  ************
 ...    ..( 46 skipped).    ..  ************
  33    2022-06-20 18:10    31  ************
  34    2022-06-20 18:20    30  ***********
  35    2022-06-20 18:30    31  ************
 ...    ..( 38 skipped).    ..  ************
  74    2022-06-21 01:00    31  ************
  75    2022-06-21 01:10    30  ***********
  76    2022-06-21 01:20    31  ************
 ...    ..( 13 skipped).    ..  ************
  90    2022-06-21 03:40    31  ************
  91    2022-06-21 03:50    30  ***********
  92    2022-06-21 04:00    31  ************
 ...    ..( 20 skipped).    ..  ************
 113    2022-06-21 07:30    31  ************
SCT Error Recovery Control:
           Read: Disabled
          Write: Disabled
Device Statistics (GP Log 0x04)
Page  Offset Size        Value Flags Description
0x01  =====  =               =  ===  == General Statistics (rev 1) ==
0x01  0x008  4              70  ---  Lifetime Power-On Resets
0x01  0x010  4           30796  ---  Power-on Hours
0x01  0x018  6<PHONE_NUMBER>28  ---  Logical Sectors Written
0x01  0x020  6<PHONE_NUMBER>  ---  Number of Write Commands
0x01  0x028  6<PHONE_NUMBER>  ---  Logical Sectors Read
0x01  0x030  6        47732631  ---  Number of Read Commands
0x01  0x038  6         3306000  ---  Date and Time TimeStamp
0x04  =====  =               =  ===  == General Errors Statistics (rev 1) ==
0x04  0x008  4               0  ---  Number of Reported Uncorrectable Errors
0x04  0x010  4               0  ---  Resets Between Cmd Acceptance and Completion
0x05  =====  =               =  ===  == Temperature Statistics (rev 1) ==
0x05  0x008  1              31  ---  Current Temperature
0x05  0x020  1              53  ---  Highest Temperature
0x05  0x028  1              22  ---  Lowest Temperature
0x05  0x058  1              70  ---  Specified Maximum Operating Temperature
0x06  =====  =               =  ===  == Transport Statistics (rev 1) ==
0x06  0x008  4             483  ---  Number of Hardware Resets
0x06  0x010  4               0  ---  Number of ASR Events
0x06  0x018  4               0  ---  Number of Interface CRC Errors
0x07  =====  =               =  ===  == Solid State Device Statistics (rev 1) ==
0x07  0x008  1               1  N--  Percentage Used Endurance Indicator
                                |||_ C monitored condition met
                                ||__ D supports DSN
                                |___ N normalized value
Pending Defects log (GP Log 0x0c) not supported
SATA Phy Event Counters (GP Log 0x11)
ID      Size     Value  Description
0x0001  2            0  Command failed due to ICRC error
0x0002  2            0  R_ERR response for data FIS
0x0003  2            0  R_ERR response for device-to-host data FIS
0x0004  2            0  R_ERR response for host-to-device data FIS
0x0005  2            0  R_ERR response for non-data FIS
0x0006  2            0  R_ERR response for device-to-host non-data FIS
0x0007  2            0  R_ERR response for host-to-device non-data FIS
0x0008  2            0  Device-to-host non-data FIS retries
0x0009  2           69  Transition from drive PhyRdy to drive PhyNRdy
0x000a  2           69  Device-to-host register FISes sent due to a COMRESET
0x000b  2            0  CRC errors within host-to-device FIS
0x000d  2            0  Non-CRC errors within host-to-device FIS
0x000f  2            0  R_ERR response for host-to-device data FIS, CRC
0x0010  2            0  R_ERR response for host-to-device data FIS, non-CRC
0x0012  2            0  R_ERR response for host-to-device non-data FIS, CRC
0x0013  2            0  R_ERR response for host-to-device non-data FIS, non-CRC

smartctl-data.zip

Hi @Jopaul-John - many thanks for your help.
Unfortunately, the device is somehow not a real scsi device. It tells in the json:
    "type": "sat",
    "protocol": "ATA"

From the smartctl doc:

sat[...] - the device type is SCSI to ATA Translation (SAT). This is for ATA disks that have a SCSI to ATA Translation Layer (SATL) between the disk and the operating system.

Can you please check if you could somehow give me the output of a "type": "scsi" disk?
Many thanks and sorry so much for all the trouble.

Here is the updated file
scsi.zip

done, closing


WebRTC to .NetCore API, GoogleSpeechApi return nothing

Hi,
I send my webrtc audio to my .netcore api.
Angular TypeScript
record() {
    this.isRecording = true;

    navigator.mediaDevices.getUserMedia({
      audio: true
    })
      .then((stream) => {
        var options = {
          audioBitsPerSecond: 128000,
          mimeType: 'audio/webm\;codecs=opus'
        }

        if (!MediaRecorder.isTypeSupported(options.mimeType)) options.mimeType = "audio/ogg;codecs=opus";

        this.stream = stream;
        this.mediaRecorder = new MediaRecorder(stream, options);
        this.mediaRecorder.onstop = e => {
          console.log("stop recording => OK");
          const blob = new Blob(this.chunks);
          this.chunks.length = 0;

          console.log("blob => OK");

          var reader = new FileReader();
          reader.readAsDataURL(blob)
          reader.onloadend = () => {
            this.sampleService.sendFile(<FileDto>{
              name: 'record.webm',
              content: reader.result.split(',')[1], //because first is data:;base64 and i dont want this in my content
              contentLength: 0,
              contentType: options.mimeType
            })
              .subscribe(result => {
                console.log("sendFile=> OK");
              }, error => {
                console.error(error);
              });
          }
        }
        this.mediaRecorder.ondataavailable = e => {
          this.chunks.push(e.data);
        }
        this.mediaRecorder.start();
      });
  }

  stop() {
    this.isRecording = false;

    this.stream.getAudioTracks().forEach(track => track.stop());
    this.stream.getVideoTracks().forEach(track => track.stop());

    this.mediaRecorder.stop();
  }

C#
        [HttpPost]
        [ProducesResponseType(200)]
        public async Task<IActionResult> SendFile([FromBody] FileDto file)
        {
                       var byteContent = Convert.FromBase64String(file.Content);

            var speech = await SpeechClient.CreateAsync();
            var response = await speech.RecognizeAsync(new RecognitionConfig()
            {
                Encoding = RecognitionConfig.Types.AudioEncoding.OggOpus,
                SampleRateHertz = 48000,
                LanguageCode = "en-CA"
            }, RecognitionAudio.FromBytes(byteContent));
            foreach (var result in response.Results)
            {
                foreach (var alternative in result.Alternatives)
                {
                    Console.WriteLine(alternative.Transcript);
                }
            }

            return Ok();
       }

How did I know about the good SampleRateHertz? I copy the byteContent to a localFile:
            var path = "C:\\Users\\Cedric\\Downloads\\Temp\\" + Guid.NewGuid().ToString();
            if (file.ContentType.Contains("webm"))
            {
                path += ".webm";
            }
            else
            {
                path += ".ogg";
            }
            System.IO.File.WriteAllBytes(path, byteContent);

First I played it to verify that I can listen my voice, and It works! Then I open the details tab property in the file properties and I saw 48 000 kHz
How did I know that Encoding is OggOpus?
When I created the mediaRecorder I specified in the option : mimeType: 'audio/webm;codecs=opus'
It was pain in the ass to understand that because I had no knowledge about Audio in general so for those who dont know that too, It could be helpfull.
Of course, The Google Environnment Variable is set in the LaunchSettings, I added in the environmentVariables section:
"GOOGLE_APPLICATION_CREDENTIALS": "C:\MYPATH\XXX-202418-645374f15cb1.json"
About the issue I have. on the audio recorded from WebRtc, I can cleary hear "This is a test". But I always have no answer from Google Speech, the Api doesnt failed, it just returns nothing.
Did I miss something? How can I make it work?

Are you able to publish the test file somewhere so that I can try to reproduce the problem? How the file is produced shouldn't be an issue, unless it's not actually an Ogg Opus file for some reason. (I'll check that as best I can when I've got the file.)

Hi,
I uploaded a file in my google drive account, let met know it you download it:
https://drive.google.com/open?id=176Xt_rd3ub8oyUed5mSo3ySJ4I-0pMsC
I installed MediaInfo (https://mediaarea.net/en/MediaInfo) to know which codec is used for my AudioFile. Result:
48kHz, 16bits, 1 channel, Opus

Done

Downloaded now, thanks.

Okay, I think I see the problem. You have an Opus audio stream, but it's within a WebM container. (MediaInfo shows that.) As per the documentation, the Cloud Speech API supports Opus within an Ogg container.
I've managed to get the sample file from https://people.xiph.org/~giles/2012/opus/ to work, for example. (I'm having trouble with a longer sample which needs a long-running operation, as tracked in #2140 but that's a different matter.)
Your Javascript code looks like it should be able to support Ogg, so I suggest you try using that instead.

@ranouf: I don't know, I'm afraid. I would look through all the supported formats at https://cloud.google.com/speech-to-text/docs/encoding and see what's supported in Chrome.
I'm going to close this issue now as it's not a problem with the client libraries as such. That doesn't mean I'm not sympathetic to your plight - it just means there's no change I could really make to the client libraries themselves to help you :(

Hi,
No pb, I understand.
To make an update for the next developer who could be interested by the same thing.
From I understand, the sound in webm is ogg. But as audio part is in a webm video container, Google speech can't access to the ogg audio part, to resolve this problem, I use ffmpeg to convert my webm to ogg.
Install-Package MediaToolkit
then
Download ffmpeg.exe from https://www.ffmpeg.org/download.html and Add it to your project with "Always copy properties"
then you can convert like this:
        if (file.ContentType.Contains("webm"))
        {
            using (var engine = new Engine("~/ffmpeg.exe"))
            {
                // Convert webm to ogg
                string command = $"-i \"{pathWebm}\" -vn -y \"{pathOgg}\"";

                engine.CustomCommand(command);
            }
        }

As i can see in MediaInfo (software):

the file:
https://drive.google.com/file/d/1iOVbSvp1vuxzNsb0Vzi1YGeogmIAUeus/view?usp=sharing (already shared with you, @jskeet )
But, I still dont have any result from Google Speech. @jskeet , what is the problem now?

As per your screenshot, that's now Ogg Vorbis, not Ogg Opus.

I updated the ffmpeg command like:
        if (file.ContentType.Contains("webm"))
        {
            using (var engine = new Engine("~/ffmpeg.exe"))
            {
                // Convert webm to ogg
                string command = $"-i \"{pathWebm}\" -vn -acodec libopus \"{pathOgg}\"";

                engine.CustomCommand(command);
            }
        }

Result:

## And now It works!!
Thanks a lot!

Hooray! Very glad to hear that :)


Explanation of Config connection/thread settings

I am fairly new to Redisson and had a few questions I would like clarified. I read through the documentation, but the description of config settings weren't very helpful to me.

For every RedissonClient, a connection pool is created with a set min idle size and max size. How is each connection exactly used? Does each operation use a connection? Are ops batched into one request and sent through the connection?

When we set the threads parameter (setThreads), what are these threads used for? Connecting to redis? Deserializing?

When we set the nettyThreads parameter, what are these threads used for?

What is relationship between nettyThreads, threads, and connectionPool? What are important things to keep in mind while tuning these parameters?

Thank you!

Connection is used per operation unless you use RBatch object. It allows to send multiple commands in batch.

Defineds threads are shared across all listeners of RTopic object, invocation handlers of RRemoteService object and RExecutorService tasks.

Netty threads are used for Redis response decoding and command sending.

You many need to increase nettyThreads if you use big connectionPool


Added mkvtoolnix

Thank you but this cask has already been removed.
We’re considering its addition to caskroom/unofficial.

@vitorgalvao can I ask why? I'm using MkvToolnix very often and I would like to install it via Cask.

Sure. The issue is the download url isn’t official, and such casks belong in the aforementioned repo. If you add it there you can still use homebrew-cask to install it. It’s simply a matter of tapping another repo.

remove mkvtoolnix, no official binaries, only built by individuals

If you want to install it from homebrew, you can simply run brew install --with-wxmac mkvtoolnix.

@iggyvolz that's not the sime brew install --with-wxmac mkvtoolnix will install cli tool (+ bunch of other dependencies). The one here is a binary version GUI tool.
@vitorgalvao in that case I will add it to homebrew-unofficial.


NamespaceAlreadyRegistered

If I delete every page in the cms and create a new page which is named like one before like home the it appears this error. NamespaceAlreadyRegistered

The seems related to an apphook that you have/had registered. Can you repeat this in 3.2.0?

Closing as we're unable to reproduce this on the latest version.


[CI][ETHOSN] Add ssh to the driver stack installation

Some use cases of the NPU integration rely on ssh being available within the docker image in order to launch an RPC server on a remote device. This commit adds ssh as a dependency on the NPU driver stack installation.
cc @ashutosh-arm @Liam-Sturge

There is an argument that this would be useful for development within the containers more generally. Wondering if the "[CI]" folks had any thoughts?

@driazati to confirm if its okay to move this to ubuntu_install_core.sh

Submitting this now. We can take up other changes in a follow up if needed.


add timeout to websocket connection and socket heartbeat

Add timeout to websocket connection and send socket heartbeat, else in case of no internet, await _ws!.ready is stuck indefinitely. This causes the socket not to try reconnecting while the internet is not present.
A similar case would happen in the previous versions of the library when await _ws!.ready was not added. It gets stuck indefinitely when sending a socket heartbeat.

Nice job @Neelansh-ns
I think a better solution would be to expose a method like: reconnect(), to allow the application to handle when the socket should or should not try the connection again. This allows the application to use the connection-plus plugin to check when the connection changes and then try to reconnect.
What do you think @Neelansh-ns  @matehat

I think a better solution would be to expose a method like: reconnect(), to allow the application to handle when the socket should or should not try the connection again. This allows the application to use the connection-plus plugin to check when the connection changes and then try to reconnect.

The issue this PR would solve is that it is adding a timeout to the ready future await _ws!.ready and similarly for sending the heartbeat. If this won't we there it takes around close 120 to 180 seconds to get a WebSocketException with the error message OS Error Time Out .
On receiving such an error, the _onSocketError callback is triggered which in turn calls _onSocketClosed which tries to reconnect as _shouldReconnect is true.
So the reconnect functionality is there, but it takes a sufficient amount of time awaiting the futures. Hence this PR adds a timeout for that.
I'll try to open a separate PR for the reconnect, which would require some refactoring.

@RodolfoSilva did you get a chance to check this?

Nice job @Neelansh-ns
I think a better solution would be to expose a method like: reconnect(), to allow the application to handle when the socket should or should not try the connection again. This allows the application to use the connection-plus plugin to check when the connection changes and then try to reconnect.
What do you think @Neelansh-ns @matehat

As mentioned by @Neelansh-ns, I think this issue is orthogonal to the need for a reconnect() method, which I think is also a good idea!


Use 62.5% body size in normalize (?)

https://snook.ca/archives/html_and_css/font-size-with-rem
https://www.freecodecamp.org/news/override-root-font-size-for-a-better-user-experience/
https://medium.com/@jagadishkamuni/respecting-font-size-preferences-rems-and-62-5-base-font-size-aleksandr-hovhannisyan-f0473d6c8410
There is a long history of doing html { font-size: 62.5% } body { font-size: 1.6rem } in projects as most (but not all!) user agents have 16px as the default base font. Resizing the base from 16 to 10 makes it a lot easier to work with rem units outside of --size-* as it is just a base10 decimal point shift away from the given design size. It’s been a good practice to use rems for many sizings as this project does since the default user agent is respected, however, in practice, I very often find myself with designs to implement whose sizes are always using pixels (from Figma or other software). Doing the conversions from 24px to 1.5rem is a much slower exercise than converting 24px to 2.4rem. It’s been slow enough that on more than one project I have used Open Props already, I swapped out the entire sizes file & converted everything to lighten the burden of reading/translating designs in practice.
Seeing that there is a 2.x on the horizon, I am curious if this has been considered. I appreciate that most values in the project are rem-based for respecting user agents, but I would personally love to see 62.5% as I did not see any existing discussions anywhere about the topic as a gauge of interest.

There is indeed a long history of it but I don't foresee a version of normalize shipping with the strategy. That ability to not use the sizes and roll your own is exactly the path to take if you're calculating rems to pixel equivalents.
Thanks for pinging and asking! Going to close for now.


Update fundraising.md

Love the fundraising doc!
I help run Celo bld.ai Web3 Studio.
Proposing some copy changes to refine the messaging.

Added on https://github.com/celo-org/docs/pull/421


Added onFracture event for informational callback when a fracturing occurs.

Resolves issue #4

To summarize, these changes add a way to trigger a fracture by code (CauseFracture) and adds an event (onTrigger) to the callbackOptions.  This event will be invoked when the object fractures, allowing one to react to the fracturing and know which collider (if any) might have caused it.  There may be additional information that is useful, but I only pass back the collider that caused it (could be null), the object which fractured, and the hit location (if a collision, else the fractured object's position).
My use-case was having AI react to a player destroying special objects and acting differently when it wasn't the player's fault.  I hope it helps!


AuthenticatedRouteMixin redirects to localhost when deployed

Hi, I am using Ember Simple Auth 1.3.0 with Ember-2.15.1+ ember-cli-fastboot-1.1.0. When i try to access my admin which require Login by AuthenticatedRouteMixin it redirects me to localhost/login. on a localhost it was works fine, but server. Has anyone faced this problem???

This is likely caused by wrong config. Can you share your settings for http://ember-simple-auth.com/api/classes/Configuration.html#property_authenticationRoute or http://ember-simple-auth.com/api/classes/AuthenticatedRouteMixin.html#property_authenticationRoute ?

@marcoow i did not override any of them only did change to routeAfterAuthentication here is my environment.js
    'ember-simple-auth': {
      routeAfterAuthentication: 'https://localhost:4200/admin'
    },
    'ember-cli-head': {
      suppressBrowserRender: true
    }
  };

  if (environment === 'production') {
    ENV['host'] = 'https://api.example.com'
    ENV['ember-simple-auth'] = {
      routeAfterAuthentication: 'https://www.zoog.mn/admin'
    }
  }

This cannot be in ESA - there is no mention of localhost anywhere in the code even (except for tests of course - see https://github.com/simplabs/ember-simple-auth/search?utf8=✓&q=localhost&type=). Do you maybe use a custom authenticator that might store a redirect URL or so?

Nope, i use oauth2 authorizer and authenticator. Also i used authenticator like this
export default OAuth2PasswordGrant.extend({
  serverTokenEndpoint: `${ENV.host}/auth/token`
});

application adapter is
export default DS.RESTAdapter.extend(DataAdapterMixin,{
  "host": `${ENV.host}`,
  authorizer: 'authorizer:oauth2'
});

on environment i used {{host}}
let ENV = {
    modulePrefix: 'my-web',
    podModulePrefix: 'my-web/pods',
    environment,
    rootURL: '/',
    host: 'http://localhost:3000',
    ....
}

  if (environment === 'production') {
    ENV['host'] = 'https://api.example.com'
    ENV['ember-simple-auth'] = {
      routeAfterAuthentication: 'https://example.com/admin'
    }
  }

does it affect them?

Are you sure you're deploying a production build?
I can't really see how this could be related to ESA - I'd look for something in your config or deployment setup.


features/CIV-5682-CCJ confirmation

JIRA link
https://tools.hmcts.net/jira/browse/CIV-5682
Change description
Develop the "Confirmation of CCJ submission" screen
Does this PR introduce a breaking change? (check one with "x")
[ ] Yes
[x] No

created a new one for pipeline issues;
https://github.com/hmcts/civil-citizen-ui/pull/1130


NPE when starting sbt console with scalac-profiling

I was trying out this guide to profile a project.
I added the following in sbt
  "-Ycache-plugin-class-loader:last-modified",
  "-Xplugin:$PATH_TO_PLUGIN_JAR",
  "-P:scalac-profiling:no-profiledb",
  "-P:scalac-profiling:show-profiles",
  "-P:scalac-profiling:sourceroot:$BLOOP_CODEBASE_DIRECTORY"

When I do console from sbt shell, I get a NPE here
as _.file returns null

@anish749 have you replaced PATH_TO_PLUGIN_JAR and BLOOP_CODEBASE_DIRECTORY with correct values?


How to send ESC Commands?

Hi....
I know that my question is not a phonegap-bluetooth-plugin
But I believe you can help me .
I would like to know how to send ESC Commands to a printer
Any feedback is appreciated!
Tks !

Hi,
I've no idea. Sorry. 😓

I appreciate the help! :)
The support my printer said I'm need to send the command as HEX string, not plain text.
But I have no idea how to do it

You'll need to convert the ESC string to a hexadecimal representation. You can try an online tool like this or just do the conversion in your code.

Great!!! I think I'm evolving.
I had already tried that way, but even so it prints the hex code...
I'm send the HEX code in an application developed by the company that distributes the printer and worked correctly ...
But in my application does not work :(
I 'm thinking I need to add some code at the beginning , to inform you that will be sent "HEX string" instead of "Plain Text"

hi @danicarla
have you found any solution for this issue?

@dannygunawan
Sorry.. Unfortunately you still can not print

@danicarla Just in case you still need this feature, I found another plugin as an alternative. [https://github.com/don/BluetoothSerial]

@dannygunawan Great!!!
When I have time.. I will test
Tks!


What's new in Windows app sdk 1.5preview1

Describe the bug
What's new in Windows app sdk 1.5preview1
Steps to reproduce the bug
None
Expected behavior
Please provide update information
Screenshots
NONE
NuGet package version
Windows App SDK 1.5 Experimental 2: 1.5.240124002-experimental2
Packaging type
Packaged (MSIX)
Windows version
Windows 11 version 22H2 (22621, 2022 Update)
IDE
Visual Studio 2022
Additional context
None

See here
https://ghost1372.github.io/wasdkOMD/omd/WinAppSDK1.5.240205001-preview1_v1.4.231219000.html

Release notes are now posted: https://learn.microsoft.com/windows/apps/windows-app-sdk/preview-channel#version-15-preview-1-150-preview1


Socket Gem Enchantments

Is it possible to add vanilla enchantments (i.e. Unbreaking or DURABILITY) with the socket gems? I know it's possible to do potion effects, and those are quite easy, but how (if it is possible) could you change it so it adds enchantments of unbreaking, fire aspect, etc.

It is possible, you just need to do something like the following:
socket-gems:
  Harden I:
    type: ARMOR
    enchantments:
    - UNBREAKABLE:5
    weight: 400
    prefix: Hard
    lore:
    - '&4This item brought to you by Nokia.'

still having an issue with it, it doesn't work with the way you sent. Any other ideas?

I'll look more into it when I'm not at work, but I'm fairly certain that is the correct format.

I am also attempting to find this out, and unable to locate how, this doesn't work for me as well.. =( hope to get a reply soon haha sorry for posting in his post, just wanted to mention that others don't know how to do this as well :P

sorry for my double post intending to bump it as I figured out the issue.. Jack you need to have a space after the enchantment type for example
socket-gems:
Harden I:
type: ARMOR
enchantments:
UNBREAKABLE: 5
weight: 400
prefix: Hard
lore:
- '&4This item brought to you by Nokia.'
NOT!
socket-gems:
Harden I:
type: ARMOR
enchantments:
UNBREAKABLE:5
weight: 400
prefix: Hard
lore:
- '&4This item brought to you by Nokia.'
:P sory the [code][/code] bbcodes didn't work when I tried to type them in so had to do the long string, hope you were able to understand haha, the author forgot to put a space after the enchantment and before the number.. works fine when you do that =)

Whoops, my bad. Forgot about the space after the enchantment name.

It seems not working properly. Here's my issue https://github.com/Nunnery/MythicDrops/issues/358


Add OpenSource.Science as a Community / Organization to map

Hi Richard
Can you add these guys to the list?
https://www.opensource.science/

Hi @CiaraFlanagan, thank you! I will take a look at it and add it to the list.


Better attachment when not using id

Hey,
This seems like a more reliable technique of finding the dropzone div within the template (using without id wasn't working for me otherwise).

Interesting that it didn't work, but :+1: this is a better way to do it. Thanks!


[backport] test: Stabilize roi_pooling_2d test by adjusting tolerance

This backports test: Stabilize roi_pooling_2d test by adjusting tolerance #3376

Please rebase.

jenkins, test this please

LGTM!


Sample site code?

Is the sample site code available for www.playliststar.com ?
I'm still not quite sure how I would run this on the backend of say, netlify, and still call it from my front end code. A few more instructions may help. Looks cool - Thanks!

Thanks for checking this out!
Try looking at the html example here:
https://jlxw.github.io/static-json-db/example.html

OK, I get how I can run it from the browser side.
But I'm still confused about how to run this backend on a JAM stack.
(That page is obviously host by Github Pages, but I don't see it listed anywhere in the gh repo, in any branches or tags... 🤷)

The Github Pages are in /docs :)
You'd still need Node to write to the database, then expose it via http or deploy it as a static site to be read by the browser.
I'll update the README to try and make this clearer.

Ah I understand a little bit better now.
The database is populated/written at deploy-time. It is read-only in the browser. I wondered if the browser could write data, too.

You would need to expose your own API to it to write data, handle its own authentication/security/etc


Adding optional import support to the VM.

This allows us to add functions that may not exist at runtime based on older versions, compile-time optional features, or different tooling.
This finally lets us run programs using the check module outside of iree-check-module, so tests can be easily passed to iree-run-module/iree-benchmark-module/etc.
Fixes #5731.

I can rename it to optional/required - I like that better anyway!

Renamed to optional | required.


Replace showThumbnailPreview with timlinePreview

Currently, the showThumbnailPreview  is rendered useless as the implementation is yet to be completed. Therefore I have replaced this property with timelinePreview which will display the expected current time when hovering over the video timeline.

:tada: This PR is included in version 1.4.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


[Snyk] Upgrade esbuild from 0.15.10 to 0.15.11

This PR was automatically created by Snyk using the credentials of a real user.Snyk has created this PR to upgrade esbuild from 0.15.10 to 0.15.11.
:information_source: Keep your dependencies up-to-date. This makes it easier to fix existing vulnerabilities and to more quickly identify and fix newly disclosed vulnerabilities when they affect your project.

The recommended version is 1 version ahead of your current version.
The recommended version was released 23 days ago, on 2022-10-14.

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open upgrade PRs.
For more information:
🧐 View latest project report
🛠 Adjust upgrade PR settings
🔕 Ignore this dependency or unsubscribe from future upgrade PRs

Codecov Report
Base: 81.00% // Head: 81.00% // No change to project coverage :thumbsup:

Coverage data is based on head (91eee2c) compared to base (3cf17de).
Patch has no changes to coverable lines.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #397   +/-   ##
=======================================
  Coverage   81.00%   81.00%
=======================================
  Files          31       31
  Lines         416      416
  Branches       61       61
=======================================
  Hits          337      337
  Misses         73       73
  Partials        6        6

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

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


mss to mls file type rename

Model Layout Sheet is better than Model Style Sheet which is something else we may add later in the form of a YAML file.  So it's time to dump the .mss extension and use .mls throughout.  This will be consistent with the documentation.

Done!


⚠ Bofrost.es has degraded performance

In 7b191eb, Bofrost.es (https://www.bofrost.es) experienced degraded performance:

HTTP code: 200
Response time: 1564 ms

Resolved: Bofrost.es performance has improved in 7a5b6f1 after 7 minutes.


[staging-next] android-tools: add patch to fix build against 6.0 kernel headers

Description of changes

Things done

Built on platform(s)

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

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

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

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

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

[ ] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 196146 run on x86_64-linux 1

2 packages built:

agi
android-tools

Is it an upstream patch? If not should it be forwarded there as well?

Agreed. If it's an upstream patch then fetchpatch is much preferred (AFAIK; and the source should definitely be documented in Nixpkgs). If this patch is from you then it should be sent upstream as well (and ideally it would even get merged so that we can use fetchpatch).

Upstream merged a similar patch in the meantime: https://github.com/nmeum/android-tools/pull/85
They're exactly the same, except they have additional diffs in usb_close() and register_device().  I'd expect their patch is more correct, but I haven't really reviewed either.

I wrote this patch with the help of a friend. I did not get around to submitting it upstream, but if they already have an equivalent patch submitted there that's great too.

They're exactly the same, except they have additional diffs in usb_close() and register_device().

You overlooked that my patch adds the allocation/free of heap-allocated child structs to the constructor/destructor of usb_handle. This is cleaner than what they are doing because it makes it impossible to accidentally delete the usb_handle without freeing the child structs too. But well, theirs will probably work too (except for leaking memory in some failure cases).

Oh right, I did overlook that when diffing.

Please someone else update or create a new PR for importing the upstream patch. I can't focus on this currently.
Feel free to request review from me then.

The upstream patch wouldn't apply as-is.  Maybe it will after the update.  Anyway, your patch should be OK and at least it fixes the build now.  Apparently, many nixpkgs users were affected.


Depend on tfjs-core 0.10.1

0.9.1 --> 0.10.1

This change is

Review status: 0 of 2 files reviewed at latest revision, all discussions resolved.

Comments from Reviewable


Readme had examples of attribute with multiple attributes

Thanks! Guess we need some tests for the readme :smiley:


wasm: fix standalone/multipart body conversion to JsValue

This is an attempt to fix the conflicting conversion code requried to turn a Body into a JsValue depending if the body is standalone or if it is part of a form (aka multipart/form-data).
The idea is to introduce a new variant to the crate::wasm::body::Inner enum that holds the actual content of a body. The new variant allows Body.to_js_value to have dedicated codepaths for standalone bodies and form parts. The new variant is contructed by the new Body.into_part method. That method is called by the crate::wasm::multipart::Part::new constructor and turns an Inner::Body into the new Inner::MultipartPart variant.
The drawback is that the new variant leads to some code duplication in the Body methods where it mostly shares code with the Inner::Bytes variant.
For clarity the PR renames the Inner::Multipart variant to Inner::MultipartForm. It also adds new test cases to cover the serialization of different bodies either standalone or multipart.
Thanks to @crapStone for pointing at the Inner enum in #1358 and @skystar-p for the original mutlipart conversion code in #1341.

Would this fix #1141 ?

@seanmonstar This is a critical fix for sending multipart requests via WASM and review of this code should be prioritized. I can confirm through my testing that multipart requests work correctly after applying this patch.


[BUG]

crash-2024-05-27_00.39.55-client.txt

Thank you, I will update to fix support for Panda's Extra Details.

Should be fixed in 3.2.2. Keep in mind that short and tall doors will no longer animate until a workaround can be looked into.