Assertion error with no details on Mac OS - tool doesn't work

Describe the bug
Assertion error without any details
To Reproduce
Steps to reproduce the behavior:

Run the following code in Jupyter Notebook for Mac OS:

import whoisdomain as whois
print(whois.query('google.com'))

Expected behavior
Whois details
Screenshots

Desktop:

OS: MacOS 13.5

Dependencies

whoisdomain = 1.20240129.1
jupyter = 1.0.0
python = 3.12
whois binary = 5.5.23 (installed via homebrew)

You did see in the readme:
Notes for Mac users
https://github.com/mboot-github/WhoisDomain#notes-for-mac-users

it has been observed that the default cli whois on Mac is showing each
forward step in its output, this makes parsing the result very unreliable.
using a brew install whois will give in general better results

On Fri, Jul 12, 2024, 16:12 Nikita Madebeykin @.***>
wrote:

Describe the bug
Assertion error without any details
To Reproduce
Steps to reproduce the behavior:

Run the following code in Jupyter Notebook for Mac OS:

import whoisdomain as whois
print(whois.query('google.com'))
Expected behavior
Whois details
Screenshots Monosnap.populate-domain-info.ipynb.notebooks.2024-07-12.18-09-31.png
(view on web)
https://github.com/user-attachments/assets/9e4ca308-ede4-4ca0-a574-23eaad37037f
Desktop:

OS: MacOS 13.5

Dependencies

whoisdomain = 1.20240129.1
jupyter = 1.0.0
python = 3.12

—
Reply to this email directly, view it on GitHub
https://github.com/mboot-github/WhoisDomain/issues/33, or unsubscribe
https://github.com/notifications/unsubscribe-auth/A7CCKLG6PJJVNBK44FDUEO3ZL7P6DAVCNFSM6AAAAABKZAIFSKVHI2DSMVQWIX3LMV43ASLTON2WKOZSGQYDKNZQGY2TMMY
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

You did see in the readme: Notes for Mac users https://github.com/mboot-github/WhoisDomain#notes-for-mac-users - it has been observed that the default cli whois on Mac is showing each forward step in its output, this makes parsing the result very unreliable. - using a brew install whois will give in general better results
…
On Fri, Jul 12, 2024, 16:12 Nikita Madebeykin @.> wrote: Describe the bug Assertion error without any details To Reproduce Steps to reproduce the behavior: 1. Run the following code in Jupyter Notebook for Mac OS: import whoisdomain as whois print(whois.query('google.com')) Expected behavior Whois details Screenshots Monosnap.populate-domain-info.ipynb.notebooks.2024-07-12.18-09-31.png (view on web) https://github.com/user-attachments/assets/9e4ca308-ede4-4ca0-a574-23eaad37037f Desktop: - OS: MacOS 13.5 Dependencies - whoisdomain = 1.20240129.1 - jupyter = 1.0.0 - python = 3.12 — Reply to this email directly, view it on GitHub <#33>, or unsubscribe https://github.com/notifications/unsubscribe-auth/A7CCKLG6PJJVNBK44FDUEO3ZL7P6DAVCNFSM6AAAAABKZAIFSKVHI2DSMVQWIX3LMV43ASLTON2WKOZSGQYDKNZQGY2TMMY . You are receiving this because you are subscribed to this thread.Message ID: @.>

No I havent seen that part, thanks for clarifying!

You did see in the readme: Notes for Mac users https://github.com/mboot-github/WhoisDomain#notes-for-mac-users - it has been observed that the default cli whois on Mac is showing each forward step in its output, this makes parsing the result very unreliable. - using a brew install whois will give in general better results

No I havent seen that part, thanks for clarifying!


DS-157 add button element

Jira
https://pegadigitalit.atlassian.net/browse/DS-157
Summary
Introduces the second Bolt element named Button.
Details

Button element is added to packages and PL docs.
Small updates were made to Text Link element to keep the two elements consistent.
Added mode specific CSS vars that will eventually replace theming.

How to test
Run the branch locally and read through everything under PL > Elements > Button. Make sure all the docs make sense. Create a dummy page and try to use the twig include and HTML, ensure all props work as intended.

@danielamorse Good catch. I need to investigate further on the shadows.


[Fix-7510] Remedy the issue with clicking the breadcrumb on the page of file resources.

Purpose of the pull request
This PR will close #7510 .
Brief change log
This issue is caused by missing a parameter of 'id' when requesting the backend api.
Verify this pull request
This change added tests and can be verified as follows:

Manually verified the change by testing locally.

Codecov Report

Merging #7747 (4906d6e) into dev (b2e6b69) will decrease coverage by 0.07%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##                dev    #7747      +/-   ##
============================================
- Coverage     41.04%   40.96%   -0.08%
+ Complexity     3674     3668       -6
============================================
  Files           641      641
  Lines         26659    26659
  Branches       2995     2995
============================================
- Hits          10942    10922      -20
- Misses        14668    14689      +21
+ Partials       1049     1048       -1

Impacted Files
Coverage Δ

...er/master/processor/queue/TaskResponseService.java
50.00% <0.00%> (-18.19%)
:arrow_down:

...rver/master/processor/queue/TaskResponseEvent.java
93.54% <0.00%> (-6.46%)
:arrow_down:

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 b2e6b69...4906d6e. Read the comment docs.


feat(nix_flake): add viewBox

This allows to use width and height to be used to scale the icon.
REF: https://github.com/catppuccin/jetbrains-icons/issues/17

sure thing , we merge this fast so it can be used on jb, upcoming implementation of all icons will have the same viewbox and width/height so no problem anymore :)


Harmony imports/exports slow down rebuilds

Do you want to request a feature or report a bug?
I want to discuss webpack behavior that may either be a bug or end up being a feature request.

What is the current behavior?
The number of harmony import/exports in an application increases rebuild time all code changes, even trivial ones that do not affect imports/exports.
This can be seen in CPU profiles of rebuilds, in applyPluginsBailResult1 (under seal). The more harmony exports there are in an application, the more time is spend on this phase checking each one.
There is a reproduction below, where this can be see. The reproduction contains two precompiled Angular applications: one in JIT mode and one in AOT mode. In Angular, AOT mode generates more files than JIT mode. These files contain more harmony exports.
The JIT application rebuild spends 70.5/66.0 ms in this phase, whereas the AOT application takes
253/200.1 ms. Larger applications spend more time in this phase, even on rebuilds where no
import or export was altered (e.g. adding console.log(1);).
Here is a screenshot of the applyPluginsBailResult1 phase in the JIT application:

And here is the same for the AOT application:

The 160ms difference might not seem that drastic, but this is just a toy application. In bigger applications
this phase takes much longer.
In the case of Angular.io, a full rebuild in AOT mode total 4.5s, of which around 1.3s is spent on applyPluginsBailResult1. By comparison, the same code change takes only some 0.15s on the JIT (less imports/exports) version.
CPU profile of Angular.io JIT rebuild:

CPU profile of Angular.io AOT rebuild:

I understand harmony imports/exports enable better optimizations and need to be checked because of that, but this seems a very steep increase in rebuild time that is outside of the control of loader/plugin authors. Perhaps there is a way to reduce the time spent on this phase.
If the current behavior is a bug, please provide the steps to reproduce.
A reproduction can be found in https://github.com/filipesilva/webpack-harmony-exports, together with CPU profiles for the seen behaviour.
Please note there are no loaders, all code is ES5 with ES2015 imports/exports.

What is the expected behavior?
Rebuilds that do not affect import/exports should not incur such a significant rebuild time due to harmony import/export checks.
If this is a feature request, what is motivation or use case for changing the behavior?
Rebuild speed is of utmost importance for application development.
Please mention other relevant information such as the browser version, Node.js version, webpack version and Operating System.
Chrome 61, node v6.11.1, webpack 3.6.0, Windows 10
/cc @d3viant0ne

hmm ok... I see what's the slow part here...
I guess this can be improved with some refactoring...

Is this something I can help with?

You can try the PR if you want too?

Using your PR with the repro in my post, the JIT application spent roughly the same time (70ms) in the applyPluginsBailResult1 phase:

The AOT application spent about half the time (109ms) as previously:

On JIT Angular.io roughly the same time was spent (80ms):

On AOT Angular.io roughly third of the time (500ms) was spent:

So your PR seems to reduce the harmony export checks time by at least half in small applications, and more than that in bigger applications. In Angular.io that's a 1s reduction in rebuilds, which is amazing.
Awesome work @sokra!


Docs

Lots of test commits trying to figure out the issue.

We should add a link to the docs's main page into the repository's README, and/or update the links to specific pages in the Documentation section.


Cannot read property 'NgGrid' of undefined (angular2 - RC5)

Note I'm on angular2 RC5*
Trying to follow the demo and then the documentation.  they are not in sync for one thing so I'm not sure what to do.
Doc only mentions importing the following (nothing about any "NgGridModule"):
import { NgGrid, NgGridItem } from 'angular2-grid';
The demo does the following in the app.module file:
import { NgGridModule } from 'angular2-grid';
When I do that I get error that NgGridModule is not exported by dist/main.js.
When I try to just add  NgGrid, NgGridItem  to my declaration in my app.module.ts it builds but then throws

Cannot read property 'NgGrid' of undefined

when it hits this code
// **from NgGrid.module.js**
var main_1 = require('../main');
var NgGridModule = (function () {
    function NgGridModule() {
    }
    NgGridModule = __decorate([
        core_1.NgModule({
            declarations: [main_1.NgGrid, main_1.NgGridItem, main_1.NgGridPlaceholder],
            entryComponents: [main_1.NgGridPlaceholder],
            exports: [main_1.NgGrid, main_1.NgGridItem]
        }),
        __metadata('design:paramtypes', [])
    ], NgGridModule);
    return NgGridModule;

It seems the require('../main') returns undefined so main_1 is undefined.
Any idea what's up?

Hmm... now I'm using the release version of Angular 2 now so there may be some things that are off. In my app.module I am importing NgGridModule from angular2-grid/dist/main, and I'm importing NgGridModule in my imports group.

can you share your code for your importing?  I'm still a bit green - new to angular2 and typescript but I really dig it ;)

Ahh, I've not tried angular 2 grid with Ionic so I am not sure if there is anything weird there.
This is an abbreviated version of my app.module where I removed basically everything

import { AppComponent }  from './components/app.component';
import { NgGridModule } from 'angular2-grid/dist/main';
@NgModule({
imports: [
NgGridModule,
],
declarations: [
AppComponent,
providers: [ ],
bootstrap: [ AppComponent ]
})
export class AppModule { }

Did you also setup systemjs for angular2-grid?

I was trying to get this to work with Ionic 2 as well. I was able to get it working by doing
var NgGrid = require('../directives/NgGrid').NgGrid; var NgGridItem = require('../directives/NgGridItem').NgGridItem; var NgGridPlaceholder = require('../components/NgGridPlaceholder').NgGridPlaceholder; var NgGridModule = (function () {     function NgGridModule() {     }     NgGridModule = __decorate([         core_1.NgModule({             declarations: [NgGrid, NgGridItem, NgGridPlaceholder],             entryComponents: [NgGridPlaceholder],             exports: [NgGrid, NgGridItem]         }),         __metadata('design:paramtypes', [])     ], NgGridModule);     return NgGridModule; }());

thanks!  but  what file is that from?  is that compiled .js file?   Can you
share what you did exactly in your app.module to pull it in and then what
you did the component you used it in?
On Wed, Nov 2, 2016 at 10:27 PM, havenotfear<EMAIL_ADDRESS>wrote:

I was trying to get this to work with Ionic 2 as well. I was able to get
it working by doing
var NgGrid = require('../directives/NgGrid').NgGrid;
var NgGridItem = require('../directives/NgGridItem').NgGridItem;
var NgGridPlaceholder = require('../components/NgGridPlaceholder').
NgGridPlaceholder;
var NgGridModule = (function () {
function NgGridModule() {
}
NgGridModule = __decorate([
core_1.NgModule({
declarations: [NgGrid, NgGridItem, NgGridPlaceholder],
entryComponents: [NgGridPlaceholder],
exports: [NgGrid, NgGridItem]
}),
__metadata('design:paramtypes', [])
], NgGridModule);
return NgGridModule;
}());
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/BTMorton/angular2-grid/issues/157#issuecomment-258057642,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AB7NvqcO_YbFb0G9f804s3mhL_8RAXTwks5q6VSFgaJpZM4Kkkpd
.

Nevermind I get what you mean.   So I used your approach and now at least I can declare it and I'm not getting any more errors.  But of course I'm manually editing NgGrid.module.js which will be overwritten if/when I need to re-install npm modules right?
This is the complete file I have now in case it helps anyone (need a pull request probably yeah?)
Updated file: node_modules/angular2-grid/dist/modules/NgGrid.module.js
"use strict";
var __decorate = (this && this.__decorate) || function (decorators, target, key, desc) {
    var c = arguments.length, r = c < 3 ? target : desc === null ? desc = Object.getOwnPropertyDescriptor(target, key) : desc, d;
    if (typeof Reflect === "object" && typeof Reflect.decorate === "function") r = Reflect.decorate(decorators, target, key, desc);
    else for (var i = decorators.length - 1; i >= 0; i--) if (d = decorators[i]) r = (c < 3 ? d(r) : c > 3 ? d(target, key, r) : d(target, key)) || r;
    return c > 3 && r && Object.defineProperty(target, key, r), r;
};
var __metadata = (this && this.__metadata) || function (k, v) {
    if (typeof Reflect === "object" && typeof Reflect.metadata === "function") return Reflect.metadata(k, v);
};
var core_1 = require('@angular/core');
var main_1 = require('../main');

var NgGrid = require('../directives/NgGrid').NgGrid;
var NgGridItem = require('../directives/NgGridItem').NgGridItem;
var NgGridPlaceholder = require('../components/NgGridPlaceholder').NgGridPlaceholder;
var NgGridModule = (function () {
function NgGridModule() {
}
NgGridModule = __decorate([
core_1.NgModule({
declarations: [NgGrid, NgGridItem, NgGridPlaceholder],
entryComponents: [NgGridPlaceholder],
exports: [NgGrid, NgGridItem]
}),
__metadata('design:paramtypes', [])
], NgGridModule);
return NgGridModule;
}());

exports.NgGridModule = NgGridModule;
//# sourceMappingURL=data:application/json;base64,eyJ2ZXJzaW9uIjozLCJzb3VyY2VzIjpbIm1vZHVsZXMvTmdHcmlkLm1vZHVsZS50cyJdLCJuYW1lcyI6W10sIm1hcHBpbmdzIjoiOzs7Ozs7Ozs7O0FBQUEscUJBQThDLGVBQWUsQ0FBQyxDQUFBO0FBQzlELHFCQUF5RixTQUFTLENBQUMsQ0FBQTtBQU9uRztJQUFBO0lBQTJCLENBQUM7SUFMNUI7UUFBQyxlQUFRLENBQUM7WUFDUixZQUFZLEVBQU0sQ0FBRSxhQUFNLEVBQUUsaUJBQVUsRUFBRSx3QkFBaUIsQ0FBRTtZQUMzRCxlQUFlLEVBQUcsQ0FBRSx3QkFBaUIsQ0FBRTtZQUN2QyxPQUFPLEVBQVcsQ0FBRSxhQUFNLEVBQUUsaUJBQVUsQ0FBRTtTQUN6QyxDQUFDOztvQkFBQTtJQUN5QixtQkFBQztBQUFELENBQTNCLEFBQTRCLElBQUE7QUFBZixvQkFBWSxlQUFHLENBQUEiLCJmaWxlIjoibW9kdWxlcy9OZ0dyaWQubW9kdWxlLmpzIiwic291cmNlc0NvbnRlbnQiOlsiaW1wb3J0IHsgTmdNb2R1bGUsIE1vZHVsZVdpdGhQcm92aWRlcnMgfSBmcm9tICdAYW5ndWxhci9jb3JlJztcbmltcG9ydCB7IE5nR3JpZCwgTmdHcmlkSXRlbSwgTmdHcmlkSXRlbUNvbmZpZywgTmdHcmlkSXRlbUV2ZW50LCBOZ0dyaWRQbGFjZWhvbGRlciB9IGZyb20gJy4uL21haW4nO1xuXG5ATmdNb2R1bGUoe1xuICBkZWNsYXJhdGlvbnM6ICAgICBbIE5nR3JpZCwgTmdHcmlkSXRlbSwgTmdHcmlkUGxhY2Vob2xkZXIgXSxcbiAgZW50cnlDb21wb25lbnRzOiAgWyBOZ0dyaWRQbGFjZWhvbGRlciBdLFxuICBleHBvcnRzOiAgICAgICAgICBbIE5nR3JpZCwgTmdHcmlkSXRlbSBdXG59KVxuZXhwb3J0IGNsYXNzIE5nR3JpZE1vZHVsZSB7fSJdLCJzb3VyY2VSb290IjoiL3NvdXJjZS8ifQ==


Shift+Tab no longer works

Contour Terminal version
<IP_ADDRESS>47
Installer source
Github: release page
Operating System
Windows 10.0.26100
Architecture
x86-64
Other Software
Tried opening cmd, pwsh, and nvim none of them seem to receive Shift+Tab correctly anymore
Steps to reproduce
press Shift+Tab within any application which has Shift+Tab bindings
Expected Behavior
Shift+Tab working like previous versions
Actual Behavior
contour itself seems to receive Shift+Tab fine but the shell receives a regular Tab
Additional notes

no custom config file was used
can only reproduce using <IP_ADDRESS>47 due to https://github.com/contour-terminal/contour/issues/1684
likely an effect of https://github.com/contour-terminal/contour/pull/1579

relevant contour terminal debug *:
previously:
[2024-12-30 08:55:21.8839874.883987] [gui.input] Character Press event received: Shift '\t'
[2024-12-30 08:55:21.8845888.884588] [vt.input] Sending Shift "\t" Press.
[2024-12-30 08:55:21.8849575.884957] [pty.output] Sending bytes: "\e[Z"

now:
[2024-12-30 11:20:19.550819] [gui.input] Key Press event received: Shift Tab
[2024-12-30 11:20:19.551802] [vt.input] Sending Shift "Tab" Press.
[2024-12-30 11:20:19.552115] [pty.output] Sending bytes: "\t"

Hm, so it looks like we do not apply modifier correctly, since for tab we need to send "\t" but for Shift tab "\e[Z"


Point to "drm_sched_can_queue" workaround

As discussed below, this changes the kernel reference to point to a workaround commit for the recent GPU related crashes made by @asahilina. Currently testing myself but feel free to merge when you believe it is sufficiently tested.
https://github.com/AsahiLinux/linux/commit/fd34a0c8a30d41bfea10829d1f9193f6c9150dc1
https://github.com/AsahiLinux/linux/issues/309
https://github.com/tpwrules/nixos-apple-silicon/issues/218

Thank you, but I much prefer to use tags if possible. I've done a new release with the official tag.


There is no way to add a node

Most diehard enthusiasts run their own node and want their wallet to connect to it.
Describe the solution you'd like
There should be a section for adding a custom node. So the node selection area should have "Testnet/Mainnet/Custom" and in Custom, a person can connect to their own node by manually entering the node information.
Describe alternatives you've considered
Instead of a white list of NGL approved nodes, the app pulls from a list of all the most commonly known nodes on the network and lists those in a searchable way.

Fixed.


pinning GHA workflow

Question
I can update the docs if there's a clear answer here. It looks like there will be a change with the next major version of how things work (both semantic-release itself, as well as the action; and hopefully the default branch for this project will also change to main at some point?)
https://python-semantic-release.readthedocs.io/en/latest/automatic-releases/github-actions.html
My question is: if I pin the python-semantic-release/python-semantic-release@ action to a tag (v9 or v9.8.7 or whatever), since it's building the Dockerfile directly from the code, that should avoid a situation where the update to the next major version breaks the GH action, right? Does it make sense to provide an example like that in the docs to avoid unexpected surprises for people using the action?
Configuration

Semantic Release Configuration

Additional context

git log --oneline --decorate --graph --all -n 50

My question is: if I pin the python-semantic-release/python-semantic-release@ action to a tag (v9 or v9.8.7 or whatever), since it's building the Dockerfile directly from the code, that should avoid a situation where the update to the next major version breaks the GH action, right?

Yes, it is my recommended to use the locked major version tag at a minimum. I use mostly full tags. If you are truly security conscious, go with the GitHub Action docs recommendation and use the direct hash of the tag (because tags can be moved).

Does it make sense to provide an example like that in the docs to avoid unexpected surprises for people using the action?

Yes, I intended to update the doc to remove any example that points at master. That is a very bad practice and should be removed. My recommendation is to follow suit for what I did with python-semantic-release/upload-to-github-release to have the build modify the version in the example. That way the example in the docs always follows gives the latest version as the example (full tag).

the default branch for this project will also change to main at some point?

Unlikely, as I cannot make that change, only the owner can, because there are push rules imposed on the default branch.

I made a PR to update the docs; let me know how that looks to you.
It would be convenient if there were a moving major or major / minor combo tag cut as well (like v9), but pretty sure that's not an option right now, right?

I made a PR to update the docs; let me know how that looks to you.

Thanks! I'll take a look.

It would be convenient if there were a moving major or major / minor combo tag cut as well (like v9), but pretty sure that's not an option right now, right?

No, unfortunately not. The problem is that PSR is not officially published to the GitHub Actions Marketplace. If it was then we could use a simpler specification to include shorter aliases that GitHub Actions would automatically resolve for us. In that case I think it would only be specified as @9 without the v. But currently since the project is not published to the marketplace, that means the spec we are using is owner/repo@tag and it interprets the repository in its raw form.

:tada: This issue has been resolved in version 9.8.7 :tada:
The release is available on:

PyPi package
GitHub release

No, unfortunately not. The problem is that PSR is not officially published to the GitHub Actions Marketplace. If it was then we could use a simpler specification to include shorter aliases that GitHub Actions would automatically resolve for us. In that case I think it would only be specified as @9 without the v. But currently since the project is not published to the marketplace, that means the spec we are using is owner/repo@tag and it interprets the repository in its raw form.

@relekang, would you be able to review and accept the developer agreement for publishing to GitHub Actions Marketplace for the organization? This should provide users with shorthand aliases and prevent the need from including the owner name in the uses: directive.  I currently get the following error message:

@relekang, idk if you saw my message above but here is a reminder if you have time to address it, thanks!

@codejedi365 I accepted it now

@relekang, awesome, thank you. Once the org accepted the agreement, then each member must accept the agreement, so I did, and now it looks like I can publish a release to the GitHub Action Marketplace.  Just FYI, the action name will have to change slightly as it can't match the org name, any other user, or an existing action.


Problems with React 16.0

First of all, thanks for this awesome component! Works like a charm for us.
It seems like it is not compatible with React 16.0, yet? I just naively tried to update my project and pages without Truncate just work, but pages with Truncate are giving me this error:
Uncaught Error: Element ref was specified as a string (target) but no owner was set. You may have multiple copies of React loaded.

I assume this is because Truncate's peer-dependencies? Any chance to bump them to 16.0?

The integrated example stopped working after upgrading to React 16.0.
It breaks when clicking on the (more) link with this error:

Uncaught DOMException: Failed to execute 'insertBefore' on 'Node': The node before which the new node is to be inserted is not a child of this node

(I'll be making a fork and then pull request soon)
For me two things needed fixing for react 16. The issue I brought up here: https://github.com/One-com/react-truncate/issues/62
And wrapping the text on this line in a span tag: https://github.com/One-com/react-truncate/blob/master/src/Truncate.js#L319

Hey guys, thank you very much for reporting.
I haven't been active in this repo for a good amount of time, but I guess some maintenance is needed now and I don't want to be a roadblock here.
I'll look to devote some time now to fix the issues and highly appreciate code contributions.

I'm currently working on a fork and will give some pull requests, but a lot of the testing stuff is out of date and updating them is taking longer than I thought.

PR here https://github.com/One-com/react-truncate/pull/64

Updated the playground with react@16 and<EMAIL_ADDRESS>

Thanks a lot to everyone involved!

@PabloSichert you need to update peer dependencies:
https://github.com/One-com/react-truncate/blob/master/package.json#L34

Looks like you removed it when it got squashed:
https://github.com/One-com/react-truncate/pull/65/files#diff-b9cfc7f2cdf78a7f4b91a753d10865a2R35
vs
https://github.com/One-com/react-truncate/pull/64/files#diff-b9cfc7f2cdf78a7f4b91a753d10865a2R35

Made a PR: https://github.com/One-com/react-truncate/pull/66/files

Hi @jonricaurte thanks for following up. I intentionally reverted the version bump after reading https://github.com/One-com/react-truncate/pull/64#issuecomment-335392053.
react-truncate is supposed to still work with<EMAIL_ADDRESS>For that reason react-truncate didn't get a major version bump.
Did I miss something?

@PabloSichert now it should work for both:
https://github.com/One-com/react-truncate/pull/66/files#diff-b9cfc7f2cdf78a7f4b91a753d10865a2
An example of another repo using this:
https://github.com/eiriklv/react-masonry-component/blob/master/package.json#L39

Without the above changes, with react 16 you will always get the following in the console:
warning<EMAIL_ADDRESS>has incorrect peer dependency "react@<= 15.x.x".

@jonricaurte I'm sorry, lol. I read that as >= instead of <=. You're absolutely right.


Update depgraphs, rbe, shapemap, shapepath, ... to 0.2.32

Updates

es.weso:depgraphs
es.weso:rbe
es.weso:shapemap
es.weso:shapepath
es.weso:shex
es.weso:shextest
es.weso:wikibaserdf
es.weso:wshex

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

Files still referring to the old version number
The following files still refer to the old version number (0.2.2).
You might want to review and update them manually.
build.sbt
src/main/scala/es/weso/shexs/Main.scala
website/yarn.lock

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

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

labels: library-update, early-semver-minor, semver-spec-patch, old-version-remains, commit-count:1

Superseded by #561.


Alt+Enter hotkey sometimes solving and skipping questions instead of just solving them.

With the large re writing of all the code, this shortcut was removed.


Recording - Open Channel/ iOS - Add extra data to message

AddExtraDataMessageOpenChannel
Link to documentation:
https://sendbird.com/docs/chat/v4/ios/guides/open-channel#2-send-a-message

@justinbaldwin possible visualization here that contains an emoji (star, etc) that can categorize messages. I.e. in Slack how you can star a message.
@petru-gherghel


Missed REQUEST_URI in server variables

Too many frameworks used $_SERVER['REQUEST_URI'] variable to determine requested uri (at least symfony and symfony-based FWs).
I though that code https://github.com/spiral/roadrunner/blob/master/src/PSR7Client.php#L137 should contains something like that:
$server['REQUEST_URI'] = $rawRequest['ctx']['uri'];

Feel free to submit PR. We are expecting your code to rely on PSR-7 Uri, but I see the value of adding such fallback.

Added in upcoming version.


Cosmos Spark Connector: Cannot convert Json into InternalRow

Cosmos Spark Connector
Hi,
I am using the new Azure Cosmos DB spark connector for the Cosmos DB Sql api, but I am having trouble with it inferring the schema before it being loaded.

cosmos_endpoint: str = "##############"
cosmos_key: str = "#######"

cosmosEndpoint = cosmos_endpoint
cosmosMasterKey = cosmos_key
cosmosDatabaseName = "cosmosDatabaseName"
cosmosContainerName = "cosmosContainerName"

cfg = {
   "spark.cosmos.accountEndpoint" : cosmosEndpoint,
   "spark.cosmos.accountKey" : cosmosMasterKey,
   "spark.cosmos.database" : cosmosDatabaseName,
   "spark.cosmos.container" : cosmosContainerName,
}

# Configure Catalog Api to be used
spark.conf.set("spark.sql.catalog.cosmosCatalog", "com.azure.cosmos.spark.CosmosCatalog")
spark.conf.set("spark.sql.catalog.cosmosCatalog.spark.cosmos.accountEndpoint", cosmosEndpoint)
spark.conf.set("spark.sql.catalog.cosmosCatalog.spark.cosmos.accountKey", cosmosMasterKey)

df = (
    spark
    .read
    .format("cosmos.oltp")
    .options(**cfg)
    .option("spark.cosmos.read.inferSchema.enabled", "true")
    .load()
)

df.show()

But I get the following error
Exception: Cannot convert Json '{"A":1,"partitionKey":"Hello","array":[1,2,3],"object_array":[{"A":[1,2,3],"B":[4,5,6]},{"A":[1,2,3]},{"B":[1,2,3]}],"id":"be76ea5d-690a-470a-b2ba-de5ea677bc63","_rid":"lnUoALlRjwYDAAAAAAAAAA==","_self":"dbs/lnUoAA==/colls/lnUoALlRjwY=/docs/lnUoALlRjwYDAAAAAAAAAA==/","_etag":"\"2d006de7-0000-3000-0000-60b787320000\"","_attachments":"attachments/","_ts":1622640434}' into InternalRow
Caused by: RuntimeException: Error while encoding: java.lang.RuntimeException: The 1th field 'B' of input row cannot be null.
Caused by: RuntimeException: The 1th field 'B' of input row cannot be null.

I've noticed if I define the structure beforehand it works, but I would prefer it to infer the Schema as the previous version, is there a solution for this?
The maven coordinates are com.azure.cosmos.spark:azure-cosmos-spark_3-1_2-12:4.1.0

@kushagraThapar could you assist @inscorep-Louis with their question?

@moderakh  - please take a look at this .

@ealsur this is schema inference related could you please take a look?

This is probably due to the object_array attribute having elements with and without B property.
According to the config page, the schema inference has a configuration called forceNullableProperties: https://github.com/Azure/azure-sdk-for-java/blob/master/sdk/cosmos/azure-cosmos-spark_3-1_2-12/docs/configuration-reference.md#schema-inference-config, which is false by default.
Schema inference samples documents, if it finds the property being null for some documents and not null for others, it marks it as nullable on the Schema, but if the sampled documents all have the property, then it marks it as not nullable.
In your case, it looks like the sampled documents might have all had items with B property in object_array but when processing the entire container, it found documents that did not. If this is expected, you can just set spark.cosmos.read.inferSchema.forceNullableProperties to true in the config.
The reason for this being a config is that some customers might not expect documents missing properties, and it's better to err but be aware of the situation and override it with a config.

Closing as by-design, and the config can be changed if the scenario is expected.

Hi,
Thank you this seems to resolve the error.
I would like ask another question. Is it possible to infer the Schema of the data selected after the query, as it seems that it is currently doing it beforehand?
I created a test container with two Json documents with similar schemas.
{
    "A": 1,
    "B": 2,
    "partitionKey": "Numbers",
    "id": "2f8bc7cc-3af2-4d42-a813-b50e281324f7"
},
{
    "A": 1,
    "C": 2,
    "partitionKey": "Numbers",
    "id": "da3491cb-0083-47cb-a54e-78dc9325102d"
}

If I run the following code I still get column C, even though if it used the data it received from the query it would have inferred that there were only columns id, A, B and partitionKey.
# Import packages
import pyspark.sql.functions as F

cosmosEndpoint = cosmos_endpoint
cosmosMasterKey = cosmos_key
cosmosDatabaseName = cosmos_db
cosmosContainerName = "test"

# Set configurations
cfg = {
   "spark.cosmos.accountEndpoint" : cosmosEndpoint,
   "spark.cosmos.accountKey" : cosmosMasterKey,
   "spark.cosmos.database" : cosmosDatabaseName,
   "spark.cosmos.container" : cosmosContainerName,
}

# Configure Catalog Api to be used
spark.conf.set("spark.sql.catalog.cosmosCatalog", "com.azure.cosmos.spark.CosmosCatalog")
spark.conf.set("spark.sql.catalog.cosmosCatalog.spark.cosmos.accountEndpoint", cosmosEndpoint)
spark.conf.set("spark.sql.catalog.cosmosCatalog.spark.cosmos.accountKey", cosmosMasterKey)

# Read data from Cosmos DB
df = (
    spark
    .read
    .format("cosmos.oltp")
    .options(**cfg)
    .option("spark.cosmos.read.inferSchema.forceNullableProperties", "true")
    .option("spark.cosmos.read.inferSchema.includeTimestamp", "false")
    .option("spark.cosmos.read.inferSchema.enabled", "true")
    .load()
    .where(
        F.col("id") == "2f8bc7cc-3af2-4d42-a813-b50e281324f7"
    )

)

print(df.explain())
display(df.show())

== Physical Plan ==
*(1) Project [A#2192, partitionKey#2193, id#2194, B#2195, C#2196]
+- BatchScan[A#2192, partitionKey#2193, id#2194, B#2195, C#2196] Cosmos ItemsScan: sigma.test
 - Cosmos Query: SELECT * FROM r WHERE NOT(IS_NULL(r['id'])) AND r['id']=@param0
 > param: @param0 = 2f8bc7cc-3af2-4d42-a813-b50e281324f7

None
+---+------------+--------------------+---+----+
|  A|partitionKey|                  id|  B|   C|
+---+------------+--------------------+---+----+
|  1|     Numbers|2f8bc7cc-3af2-4d4...|  2|null|
+---+------------+--------------------+---+----+

@inscorep-Louis Schema inference works by sampling the first 1000 documents and creating the schema based off that, then executing the query.
There are different configuration options (https://github.com/Azure/azure-sdk-for-java/blob/master/sdk/cosmos/azure-cosmos-spark_3-1_2-12/docs/configuration-reference.md#schema-inference-config).
You can disable schema inference (spark.cosmos.read.inferSchema.enabled) and provide your desired schema, you can change the sampling size (spark.cosmos.read.inferSchema.samplingSize), you can use a custom query to tell the inference which documents should be used to infer it (spark.cosmos.read.inferSchema.query).
If you are always selecting a single document or just a document with that id, maybe you can provide the schema you want to use or apply the same filter in the inference query (like SELECT * FROM d WHERE d.id = {some value you want to use}).


No rule to make target 'menuconfig'. Stop. (IDFGH-2120)

Hi,
I have been following "https://exploreembedded.com/wiki/Setting_up_Bare_Metal_Development_Toolchain_for_ESP32" to be able to work on ESP-32 using Eclipse.
When I try running the "make menuconfig", it shows me the following error: "make: *** No rule to make target 'menuconfig'. Stop."
My IDF_PATH is "D:/SaahilESP/esp-idf".
Msys has been extracted in C drive.
How do I resolve this issue? Please help! TIA.

@saahilshroff Thanks for reporting, and also saw you left a message https://github.com/espressif/esp-idf/issues/1703#issuecomment-549265126.

Correct! Could you please help me out?

@Alvin1Zhang, look at this:

In your msys32 directory, run mingw32.exe not msys2.exe
Run make menuconfig in your project directory NOT in esp-idf directory
If you get warnings about missing python dependencies, run pip install -r $IDF_PATH/requirements.txt

If you plan to use Make build system, I suggest you take a look at the Legacy Getting Started Guide for GNU Make. The version of ESP-IDF used in the link you've provided is release/v3.1 which is a relatively old version of ESP-IDF.

Hi @Dazza0 .
Thank you for replying.
I could now successfully run the 'make menuconfig' and this is how I configured it. (The ESP-32 is connected to COM24)

Now, when I use "make" to compile this test code, the output on the screen:

Similarly, when I use "make flash" and hold "BOOT" button while it flashes, the output on the screen:

The on-board LED doesn't glow. Could you please help me out with this as well?
[Note: The "blink" code is as follows:
#include <stdio.h>
#include "freertos/FreeRTOS.h"
#include "freertos/task.h"
#include "driver/gpio.h"
#include "sdkconfig.h"
#define BLINK_GPIO 2
void blink_task(void *pvParameter)
{
gpio_pad_select_gpio(BLINK_GPIO);
gpio_set_direction(BLINK_GPIO, GPIO_MODE_OUTPUT);
while(1)
{
gpio_set_level(BLINK_GPIO, 0);
vTaskDelay(1000 / portTICK_PERIOD_MS);
    gpio_set_level(BLINK_GPIO, 1);
    vTaskDelay(1000 / portTICK_PERIOD_MS);
}

}
void app_main()
{
xTaskCreate(&blink_task, "blink_task",
configMINIMAL_STACK_SIZE, NULL, 5, NULL);
}
]

Won't be able to help unless I know which board/development kit you're using and which GPIOs are connected to which LED on the board. Have you tried changing #define BLINK_GPIO 2 ?

I am using ESP-32.
Currently, I am just trying to blink the on-board LED, which is internally connected to GPIO 2.

I am using ESP-32
ESP32 is the chip in the board, not the board itself. I mean the actual development board you're using (e.g. WROVER Kit, Pico Kit). See this link for all the official Espressif Boards.

It is possible that you may have also purchased a third party board (i.e., not made by Espressif). In that case, a link to the seller and their board schematic would he helpful.

Using ESP32-WROOM-32 (ESP32-DevKitC)

I am using ESP-32

ESP32 is the chip in the board, not the board itself. I mean the actual development board you're using (e.g. WROVER Kit, Pico Kit). See this link for all the official Espressif Boards.
It is possible that you may have also purchased a third party board (i.e., not made by Espressif). In that case, a link to the seller and their board schematic would he helpful.

Using ESP32-WROOM-32 (ESP32-DevKitC)

There is no LED connected GPIO 2 on ESP32-DevkitC. Please refer to the schematic. The LED on  ESP32-DevkitC is connected to the power supply, not a GPIO.

Hi @Dazza0 .
I checked the schematic out and could not find an LED connected.
However, using Arduino, I was able to blink an onboard LED. Kindly refer below for the code and output. (I referred to _https://circuitdigest.com/microcontroller-projects/getting-started-with-esp32-with-arduino-ide_ when I was using Arduino IDE)
CODE:
#define led 2
void setup()
{
pinMode(led,OUTPUT);
}
void loop()
{
digitalWrite(led,HIGH);
delay(5000);
digitalWrite(led,LOW);
delay(500);
}
OUTPUT
(It appears Green in the picture since Red&Blue LED are beside each other. Red indicates that the board is powered. Blue color signifies the onboard LED that I have been talking about)

Hence, "#define BLINK_GPIO 2" for the code that I am trying to flash on the board using the Make-Build system.

That is not an ESP32-DevKitC. That is a third party board. Judging from the image, it looks like an HW-573 or NodeMCU. But if the works on Arduino, I guess there is indeed.
I've looked through the instructions in the link you have provided. Are you actually flashing you development board using make flash or make flash monitor commands?
If so and it's still not working, add a couple of printf() statements in your blink_task() to check if it's actually running.

Hey @Dazza0.
The "blink" project that I had downloaded as suggested by the link that I was following didn't actually contain the code for blinking the LED on Pin-2.
Works fine now.
Kudos to you for the fast and patient replies.
Closing this issue.
Thank you very much! :)

Hi @Dazza0 .
I checked the schematic out and could not find an LED connected.
However, using Arduino, I was able to blink an onboard LED. Kindly refer below for the code and output. (I referred to _https://circuitdigest.com/microcontroller-projects/getting-started-with-esp32-with-arduino-ide_ when I was using Arduino IDE)
CODE:
#define led 2
void setup()
{
pinMode(led,OUTPUT);
}
void loop()
{
digitalWrite(led,HIGH);
delay(5000);
digitalWrite(led,LOW);
delay(500);
}
OUTPUT
(It appears Green in the picture since Red&Blue LED are beside each other. Red indicates that the board is powered. Blue color signifies the onboard LED that I have been talking about)

Hence, "#define BLINK_GPIO 2" for the code that I am trying to flash on the board using the Make-Build system.

Correction: The development board shown in the image is NodeMCU-32S.


Visualize Lane Markings in OpenDrive Standalone Mode

Currently, with version 0.9.9, the OpenDrive standalone mode does not visualize lane markings (e.g. see following image from the release notes ).
Can they be added? This increases the number of usecases CARLA could be used for. Also it eases transitioning from any other simulator to CARLA, as many other tools already have support of OpenDRIVE. So transferring assets from any other simulator to CARLA would get easier.

The lane markings are there, but not rendered. We will add them in future versions of the tool.

Yes, this is in our roadmap. the main difficulties here are how to generate these LaneMarkings as separate meshes of the main road so they can be also added into the semantic segmentation rendering. Since we also want to change how the semantic segmentation works, we must wait until it's completely done so maybe the separation of these meshes will not be necessary.

Is there any update?

Is there any progress on this? please let me know.

Is there any update? please let me know.


⚠ Bofrost.be has degraded performance

In db479a8, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 1404 ms

Resolved: Bofrost.be performance has improved in 1bac8d3 after 9 minutes.


ARROW-6898: [Java] Fix potential memory leak in ArrowWriter and several test classes

Related to ARROW-6898.
ARROW-6040 fixed the problem that dictionary entries are required in IPC streams even when empty, which only writes dictionaries when there are at least one batch. In this way, if we write empty stream and invoke ArrowWriter#close, the dictionaries are not closed leading to memory leak (they are closed after the write operation), and it’s really hard to debug, this problem was found by TestArrowReaderWriter#testEmptyStreamInStreamingIPC when I tried to close allocator after the test.
Besides, there are several test classes have potential memory leak without closing allocator/vector/buf etc.

cc @emkornfield, it's better to include this one into 15.1.

+1 thank you, yes I think it would be ok to go into 0.15.1 .

Sorry, I missed it but This still has a potential leak if an exception is thrown in this code: private void ensureDictionariesWritten()

Sorry, I missed it but This still has a potential leak if an exception is thrown in this code: private void ensureDictionariesWritten()

Remove close logic from ensureDictionariesWritten() and make it all in ArrowWriter#close?

that or move the try/finally block outside the loop and call AutoCloseables.close() on the list at once.

that or move the try/finally block outside the loop and call AutoCloseables.close() on the list at once.

I will provide a hotfix later, thanks for catching this.


[Robust] List of sandbox violations

EDIT: Once every linked PR is merged, the only remaining problems are the usage of reflection and the DMI parser needing to be moved to the server.
These all need to be fixed so OpenDream can go on the SS14 hub:

[x] [ERRO] res.typecheck: Sandbox violation: Access to type not allowed: [System.Drawing.Primitives]System.Drawing.Point
[x] [ERRO] res.typecheck: Sandbox violation: Access to type not allowed: [System.Runtime]System.Reflection.CustomAttributeExtensions
[x] [ERRO] res.typecheck: Sandbox violation: Access to type not allowed: [System.Web.HttpUtility]System.Web.HttpUtility
[x] [ERRO] res.typecheck: Sandbox violation: Access to type not allowed: [System.Runtime]System.Reflection.FieldInfo
[ ] [ERRO] res.typecheck: Sandbox violation: Access to type not allowed: [System.IO.Compression]System.IO.Compression.DeflateStream
[x] [ERRO] res.typecheck: Sandbox violation: Access to type not allowed: [System.Drawing.Primitives]System.Drawing.Size
[ ] [ERRO] res.typecheck: Sandbox violation: Access to type not allowed: [System.IO.FileSystem]System.IO.File
[ ] [ERRO] res.typecheck: Sandbox violation: Access to type not allowed: [System.Runtime]System.IO.Path
[x] [ERRO] res.typecheck: Sandbox violation: Access to type not allowed: [System.Console]System.Console
[x] [ERRO] res.typecheck: Sandbox violation: Access to type not allowed [System.ComponentModel]System.ComponentModel.CancelEventArgs
[x] [ERRO] res.typecheck: Sandbox violation: Access to type not allowed:[System.Collections.Specialized]System.Collections.Specialized.NameValueCollection
[ ] [ERRO] res.typecheck: Sandbox violation: Access to type not allowed: [System.Runtime]System.IO.BinaryReader
[x] [ERRO] res.typecheck: Sandbox violation: Access to method not allowed: string [System.Runtime]System.Convert.ToString(char)
[ ] [ERRO] res.typecheck: Sandbox violation: Access to type not allowed: [System.Runtime]System.IO.SeekOrigin
[x] [ERRO] res.typecheck: Sandbox violation: Access to type not allowed: [System.Text.Json]System.Text.Json.Serialization.JsonIgnoreAttribute
[x] [ERRO] res.typecheck: Sandbox violation: Access to method not allowed: unsigned int32 [System.Runtime]System.Convert.ToUInt32(string, int32)
[ ] [ERRO] res.typecheck: Sandbox violation: Access to type not allowed: [System.Runtime]System.BitConverter
[x] [ERRO] res.typecheck: Sandbox violation: Access to type not allowed: [System.Drawing.Primitives]System.Drawing.Color
[x] [ERRO] res.typecheck: Sandbox violation: Access to method not allowed: int32 [System.Runtime]System.Convert.ToInt32(string, int32)
[x] [ERRO] res.typecheck: Sandbox violation: Access to type not allowed: [System.Net.Primitives]System.Net.HttpStatusCode
[x] [ERRO] res.typecheck: Sandbox violation: Access to type not allowed: [System.Runtime]System.Globalization.NumberStyles
[ ] [ERRO] res.typecheck: Sandbox violation: Access to type not allowed: [System.IO.Compression]System.IO.Compression.CompressionMode

Note that these are all in either Content.Client or Content.Shared. The server isn't sandboxed.

https://github.com/space-wizards/RobustToolbox/pull/2042

Reflection fixed in https://github.com/wixoaGit/OpenDream/pull/422

All issues are fixed, just waiting on content and engine PR merges


Close ParquetFileReader when footer is read

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/6270/
Test FAILed.

jenkins, test this please

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/Alluxio-Pull-Request-Builder/6274/
Test PASSed.

alluxio-bot, merge this please

alluxio-bot, cherry-pick this to branch-2.1 please


add syncState status string inprogress to SyncStatus class - 1907

[Auto generated]
Number: [#1907]

openshift-python-wrapper[pull_request][20680c30-4410-11ef-81b3-3aa65975d3ce][PR 1907]: Closing issue for PR: add syncState status string inprogress to SyncStatus class.
PR was closed.


Improvement: Add dairy products to the Cooking calculator

Producing Pot of cream, Pat of butter or Cheese on a Dairy churn from a bucket of milk grants 18, 40.5 resp. 64 cooking xp. Note: Pat of butter may also be made from a Pot of cream for  22.5 xp.

Cheese, butter and cream made from milk was added in #10542


Added center orientation to master layout

Describe your PR, what does it fix/add?
This adds support for a new orientation for master layout.  It is very close to the request in #1250 .
Is there anything you want to mention? (unchecked code, possible bugs, found problems, breaking compatibility, etc.)
For my workflow I prefer my main work window to be centered.    This may be a bit awkward for some folks as this leaves empty space on the right side.  If there is a strong preference I can change this to split into halves when only two windows are open.
Only other issue I found is that the mouse resize operation is a bit wonky when in this mode.
The following was added to the master layout config:

orientation: added "center"
always_center_master: bool - only used for center orientation - if true, while in center orientation the master window will always occupy the center of the screen, otherwise center orientation behaves like left orientation until three windows are open
layoutmsg: orientationcenter

Is it ready for merging, or does it need work?
See above.

I can change this to split into halves when only two windows are open.

you can make this a config option in the master: cat.

I can change this to split into halves when only two windows are open.

you can make this a config option in the master: cat.

Good idea. I will take a look today and update the pull request.

Hi @vaxerski, your suggestion has been added.

Thanks!

@vaxerski You can probably close #1250 as well.


WIP Feature/helpdesk

Followup from PR #896 https://github.com/EuroPython/epcon/pull/896
Because had to rename the branch...

We are going to use https://zammad.org/ for the helpdesk in 2020 and hopefully won't have to implement or maintain any helpdesk related code directly in the EuroPython website in the future. Thanks @artcz for the hard work on this. 🤗


Fix default icon color constants reversed brightness documentation

Fixes the reversed default icon color constants brightness documentation for kDefaultIconLightColor and kDefaultIconDarkColor.
FIX: https://github.com/flutter/flutter/issues/130230
Pre-launch Checklist

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

This'll need a secondary review from @HansMuller for the autosubmit to pass.


Allow User to choose Authentication Challenge when performing 2FA with Personal Capital

Currently this is hard coded to email, but there is another option sms available.

Implemented in 3d4f1a7e525d1ba94c682976cda87e8415dcde67


Enable objects controlled by PAPI

Enable objects that the PAPI controls for the extensions. We need to be able to handle things like webviews, resources, etc.
Depends on #30

@tjcouch-sil, It seems like some of this task has already been done as part of #67. Is that right?
It also seems like the Webview aspect of this task was split off into #44.


No spaces check - Feature req

Include No spaces check
i am going to borrow your source code until you do it, then i promise i will delete it and use your package
just let me know

@sators Done <3

Reference #55

This is now available in v1.7


Test with 1.0-rc1 TOML spec tests

https://github.com/iarna/toml-spec-tests, see https://github.com/bobfang1992/pytomlpp/issues/20#issuecomment-647534319

OK, here's an update about conformance. I've spent the last few days bringing the tests from both test sets into the toml++ test suite, and fixed a number of small conformance bugs I identified in the process. You should be able to update your test code to remove the exceptions for the following invalid inputs:

BurntSushi/float-leading-zero
BurntSushi/float-leading-zero-neg
BurntSushi/float-leading-zero-pos
BurntSushi/float-no-leading-zero
BurntSushi/float-no-trailing-digits

The fact that they were previously failing was an error on my part; I was too relaxed with allowing omission of some digits in float values.
Additionally, you should add an exception to exclude BurntSushi/datetime from the set of valid inputs because the test data itself is incorrect (see here).
Finally, since I've also added in tests for the TOML 1.0.0 spec from iarna's suite, you might want to re-evaluate if you even need to bother since it might be a bit of work (and is all about to be replaced with the official toml-lang/conformance anyway). If you do want to, there's a few exceptions you'll need to add to the invalid cases because they ignore the TOML spec's provision for sub-millisecond precision in fractional time values, thus making them incorrect:

iarna/spec-date-time-6
iarna/spec-date-time-local-2
iarna/spec-time-2

(more info here)
The latest toml++ in master contains all the conformance fixes, so update at your leisure ^_^

Hey, thanks for this. I will update the library soon. I think I will still try to include both test suite in this repo and I will add more unit tests on my side to improve the quality of this repo.

Hi, thanks for the input. I will keep what we have for now and later change it to the merged conformance tests. I am now updating toml++ and said tests.

closed via 54add69dfd9f56ac86669ec0badd042ad698bb79


MethodAccessException same as Issue 384

Please fill in the following fields:
Unity editor version: 2019.1.8f1
Firebase Unity SDK version: 6.2.1
Firebase plugins in use (Auth, Database, etc.): Messaging, DynamicLInks
Additional SDKs you are using (Facebook, AdMob, etc.): Facebook, GameAnalytics, GameOfWhales
Platform you are using the Unity editor on (Mac, Windows, or Linux): Windows
Platform you are targeting (iOS, Android, and/or desktop): ios + android
Please describe the issue here:
Try to update Firebase from version 5.5.0 to 6.2.1. after update on any Scene recieve a lot of such errors:
MethodAccessException: Method Google.RunOnMainThread.Run(System.Action,bool)' is inaccessible from method Firebase.Editor.NetSDKChecker..cctor()'
Firebase.Editor.NetSDKChecker..cctor () (at Z:/tmp/tmp.lUJ3Ncf6L3/firebase/app/client/unity/editor/src/NetSdkChecker.cs:21)
MethodAccessException: Method Google.RunOnMainThread.Run(System.Action,bool)' is inaccessible from method Firebase.Editor.GenerateXmlFromGoogleServicesJson..cctor()'
Firebase.Editor.GenerateXmlFromGoogleServicesJson..cctor () (at Z:/tmp/tmp.lUJ3Ncf6L3/firebase/app/client/unity/editor/src/GenerateXmlFromGoogleServicesJson.cs:88)
Update to latest play-services-resolver (v<IP_ADDRESS>) as it was a solution in issue 384 didn`t help.

Hi @IvzhenkoOleksii
You seems to used an older version of Play Services Resolver which has not expose Google.RunOnMainThread().
https://github.com/googlesamples/unity-jar-resolver/commit/cea8035e3cec7551779f2b0e8a2ee56dec2de17d#diff-8b0683bb75d78519adeed5d3b7684470
The Play Service Resolver coming with 6.2.1 should includes this change already.
Could you try to

Close your editor
Remove Play Service Resolver entirely. (Under Assets/PlayServicesResolver)
Open Unity Editor
Reimport the latest Play Service Resolver
https://github.com/googlesamples/unity-jar-resolver/releases/tag/v1.2.124

Shawn

Hi, it seems to help. Thanks


Weird error in REPL: bad constant pool index: 0

Version(s)
1.0.4
Describe the bug
Confusing and unexpected error in simplest REPL command.
To Reproduce
scala-cli
Welcome to Scala 3.3.0 (21, Java OpenJDK 64-Bit Server VM).
Type in expressions for evaluation. Or try :help.

scala> val s = ""
error while loading AccessFlag,
class file /modules/java.base/java/lang/reflect/AccessFlag.class is broken, reading aborted with class java.lang.RuntimeException
bad constant pool index: 0 at pos: 5189
error while loading ElementType,
class file /modules/java.base/java/lang/annotation/ElementType.class is broken, reading aborted with class java.lang.RuntimeException
bad constant pool index: 0 at pos: 1220
2 errors found

Expected behaviour
To not see two errors, or at least get actionable error messages.

I've updateted scala-cli to 1.0.5 and it is fixed now:
scala-cli
Welcome to Scala 3.3.1 (21.0.1, Java OpenJDK 64-Bit Server VM).
Type in expressions for evaluation. Or try :help.

scala> val s = ""
val s: String = ""

scala>

looks like a JDK 21 thing
note that 1.0.5 defaults to 3.3.1 — I think on 1.0.4, scala-cli -S 3.3.1 would probably have also worked for you

it's a bit unusual that there was a fairly long delay between Scala 3.3.1 coming out and Scala-CLI 1.0.5 coming out with 3.3.1 as the default. there is some discussion at https://github.com/VirtusLab/scala-cli/issues/2394 about the prospect of avoiding such delays in the future

Thank you for the context! Keep up the good work :)


Add Turing Experiment Engine XP Plugin

What this PR does / why we need it:
This PR adds the following:

Auto-conversion of Segmenter value from request payload from string to respective Segmenter type. This is required since Segmenter type information is most suited to be left in Segmenter Runner's Transform method, rather than the Experiment Engine plugin itself.
Experiment Engine plugin for Turing.
CICD build/publish step to expose XP Experiment Engine plugin for first-class support in Turing.

Which issue(s) this PR fixes:

NONE

Why is the cmd package removed? It's still useful for development.

Could you elaborate a little more on what you mean by useful for development?


Model scaffold doesn't import all of the helper functions for model definition

Description
Using the following command:
❯ cargo loco generate scaffold todos title:string! done:bool! user:references

I get the following error:
    Finished dev [unoptimized + debuginfo] target(s) in 0.67s
     Running `target/debug/loco_leptos_todomvc-cli generate scaffold todos 'title:string'\!'' 'done:bool'\!'' 'user:references'`
added: "migration/src/m20240214_213812_todos.rs"
injected: "migration/src/lib.rs"
injected: "migration/src/lib.rs"
added: "tests/models/todos.rs"
injected: "tests/models/mod.rs"
   Compiling migration v0.1.0 (/opt/dev/projects/loco_leptos_todomvc/migration)
error[E0423]: expected function, found builtin type `bool`
  --> migration/src/m20240214_213812_todos.rs:14:26
   |
14 |                     .col(bool(Todos::Done))
   |                          ^^^^ not a function
   |
help: consider importing this function instead
   |
1  + use loco_rs::schema::bool;
   |

The generated file doesn't have that use statement and the scaffolding migration process fails
use sea_orm_migration::{prelude::*, schema::*};

#[derive(DeriveMigrationName)]
pub struct Migration;

#[async_trait::async_trait]
impl MigrationTrait for Migration {
    async fn up(&self, manager: &SchemaManager) -> Result<(), DbErr> {

To Reproduce
Run the above command in your Loco project
Expected Behavior
A new generated model and controller scaffolding

I reviewed this one and it looks like it was reproduced under 0.3.1.   Due to #457 this issue has been fixed in 0.3.2.  @slyons if you have a minute to confirm with the new crate it would be useful.
In the process though I noticed that #457 incorrectly updated the documentation to say to use boolean, boolean! rather than bool, bool!, so we need to fix that.

closing this issue.
if you stil have a problem fell free to repoen or create a new one


Fix weapon cooldown serialization while mapping

fixes #15113 by removing next-attack times from the dev map file
Prevents next attack & next fire times from being set while the entity is paused.
Adds warning logs to try prevent repeat incidents for guns & melee
Removes some bad TimeOffsetSerializer uses.

These also might be related: #15096 #13008


markdown blockquote not interpreted

In a document like:
<pre class='metadata'>
...
Markup Shorthands: markdown yes
</pre>

> Markdown blockquote

the markdown blockquote (> ...) syntax is not recognized, and verbatim >s appear in the output document.

Yeah, I haven't added blockquotes to Bikeshed's markdown syntax yet.  I presume you'd like me to do so? ^_^

And done!


Show alert limits in graphs

Description:
Show alert limits in graphs on history view page in app.
Refer to WebUI card for details: https://github.com/ruuvi/com.ruuvi.station.webui/issues/204
iOS card: https://github.com/ruuvi/com.ruuvi.station.ios/issues/1547

Library doesn't support this currently, needs internal discussion.

@andreevdenis let's improve the visibility of the alert limit lines

Line width 2px, colour is same as used in alerts

@andreevdenis as discussed on channel the line looks too thick now, can it still be adjusted?

QA passed


Lightbulb Tasks

Steps to reproduce
I've tried to run my bots code, however, i get the following error, due to importing tasks from lightbulb
Expected result
It should have displayed the usual hikari text, instead it printed the traceback
Actual result
Traceback (most recent call last):
  File "main.py", line 29, in <module>
    from lightbulb.ext import tasks
ImportError: cannot import name 'tasks' from 'lightbulb.ext' (unknown location)

System info
hikari-lightbulb (2.1.3)
hikari (2.0.0.dev108) [6dbab9a6]
located at /home/runner/Lynx-Main/venv/lib/python3.8/site-packages/hikari
CPython 3.8.12 GCC 10.3.0
Linux 355a27b5f37f 5.13.0-1027-gcp #32~20.04.1-Ubuntu SMP Thu May 26 10:53:08 UTC 2022 x86_64
~/Lynx-Main$

Further info
No response
Checklist

[X] I have made sure to remove ANY sensitive information (bot token, passwords, credentials, personal details, etc.).
[X] I have searched the issue tracker and have made sure it's not a duplicate. If it is a follow up of another issue, I have specified it.

You are using an outdated version of lightbulb. Please update to the latest by using pip install -U hikari-lightbulb
Tasks where added on release 2.2.0 and you are using 2.1.3 (https://github.com/tandemdude/hikari-lightbulb/releases/tag/2.2.0)

You are using an outdated version of lightbulb. Please update to the latest by using pip install -U hikari-lightbulb
Tasks where added on release 2.2.0 and you are using 2.1.3 (https://github.com/tandemdude/hikari-lightbulb/releases/tag/2.2.0)
Thank's alot, your a lifesaver yknow!


Castle Windsor registration

Hi team,
How can I register the MongoRepository class in Castle Windsor?

I have zero experience with Castle Windsor so I wouldn't know. But let us know when you find out?


exec step

Added exec step which allows to run executables or shell scripts as part of the pipeline, see updated README and tests

Pull Request Test Coverage Report for Build 995

3 of 15 (20.0%)  changed or added relevant lines in 2 files are covered.
22 unchanged lines in 2 files lost coverage.
Overall coverage decreased (-8.09%) to 18.65%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

datapackage_pipelines/lib/exec.py
0
12
0.0%

Files with Coverage Reduction
New Missed Lines
%

.tox/py36-plyvel/lib/python3.6/site-packages/datapackage_pipelines/specs/resolver.py
11
0.0%

.tox/py36-sqlite/lib/python3.6/site-packages/datapackage_pipelines/specs/resolver.py
11
0.0%

Totals

Change from base Build 993:
-8.09%

Covered Lines:
1666

Relevant Lines:
8933

💛  - Coveralls

This feels like it should be an extension and not part of the core lib.
That extension could have more than one exec, eg one for running pre commands, one for running finalization commands etc.


Add xz support

Adds a -x flag with XZ support.
XZ with default args can produce files half the size of gz on average.
130M zabbix_cfg_<IP_ADDRESS>_20160728-1316_db-3.0.0.sql.gz
53M zabbix_cfg_<IP_ADDRESS>_20160728-1319_db-3.0.0.sql.xz

Any thoughts on this merge?

Oh sorry, I forgot to answer after reviewing it. It's a very nice idea!
I just wanted to remove the redundant messages in lines 336/343 and 348/349 before merging.

Hmm good idea.  I didn't really like that when I added it anyway.  I'll see about getting that cleaned up tomorrow.

I now rewrote your implementation a bit and added another option for "no compression".
You will find the changes in the new release 0.8.2.
Thank you very much for your commitment to improve this script!


List individual Wodehouse novels in wanted ebooks list

List each Wodehouse novel on the wanted ebooks list, so that new producers can pick one more easily.

Thanks!


Problem with Cashout

I was updating the image yesterday and followed to procedure as prescribed in README. Closed channels no problem, but had trouble moving funds out through the GUI. When prompted to enter amount, I left it blank to empty wallet fully, entered my address, and got back "OK. That worked :)", but then my balance never changed and lncli listchaintxns did not show it either (last txns were the ones to close channels). Tried several times and same thing happened every time. Since then I managed to send it from cli manually tweaking fees until in worked, final balance was 0 satoshis.
I'm guessing BBcashoutWallet.sh doesn't catch the error somehow.

Yes it looks like the cashout script needs update on error catching. Adding it to my list for the next release. thx.

I tested and upadted the CASHOUT script. It work on my tests now and also uses multiple tries to get the fees right. Will be part of next sd card relase. I am closing the ticket for now.


[SPARK-37326][SQL][FOLLOWUP] Fix the test for Java 11

What changes were proposed in this pull request?

This PR fixes an issue that the test added in SPARK-37326 (#34596) fails with Java 11.
https://github.com/apache/spark/runs/4381645820?check_suite_focus=true#step:9:11681
The reason is that the error message DateTimeFormatter was changed as of Java 9.
https://bugs.openjdk.java.net/browse/JDK-8085887
http://hg.openjdk.java.net/jdk9/jdk9/jdk/rev/28df1af8e872
Why are the changes needed?

To keep the build stable.
Does this PR introduce any user-facing change?

No.
How was this patch tested?

Tests with the following command success.
JAVA_HOME=/path/to/java11/ build/sbt -Phive -Phive-thriftserver "testOnly org.apache.spark.sql.execution.datasources.csv.CSVv*Suite"

cc: @sadikovi @MaxGekk

Test build #145819 has started for PR 34771 at commit 514a5e8.

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/50294/

Kubernetes integration test status failure
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/50294/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/50294/

Test build #145819 has finished for PR 34771 at commit 514a5e8.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/145819/


chore: Update deleteChangesets.mjs to track changeset deletions

Description 📝
The deletion of changesets were not being tracked in git because we weren't using git rm.
Major Changes 🔄

Asynchronously deleting files using git rm ${filePath}

Preview 📷

See this commit which shows files: d5af96f

@bnussman-akamai It could, what do you think about adding https://www.npmjs.com/package/simple-git?


Please add AitrVip extractor for TRAnimeİzle

Source information
TrAnimeİzle 13.11
Source language
Turkish
Steps to reproduce
Use tranimeizle
Expected behavior
a
Actual behavior
a
Aniyomi version
<IP_ADDRESS>
Android version
Android 11
Other details
No response
Acknowledgements

[X] I have searched the existing issues and this is a new ticket, NOT a duplicate or related to another open or closed issue.
[X] I have written a short but informative title.
[X] I have updated the app to version <IP_ADDRESS>.
[X] I have updated all installed extensions.
[X] I have tried the troubleshooting guide.
[X] If this is an issue with the app itself, I should be opening an issue in the app repository.
[X] I will fill out all of the requested information in this form.

Use the correct issue template.

Use the correct issue template.

I was editing it... Is it still would be valid?

It would be preferable if you use the correct issue template

https://github.com/aniyomiorg/aniyomi-extensions/issues/2650 this one is OK, i guess..


1.19.4

Describe the bug
It's not on 1.19.4, the latest MC version
To Reproduce
Steps to reproduce the behavior:

Try to download this mod/See it's raining
See it's not for 1.19.4/Cover yourself in oil
Weep/Fly

Expected behavior
It exists for 1.19.4
Screenshots
N/A
Setup (please complete the following information):

OS: Arch btw
Minecraft Version: 1.19.4
Version: Any

Additional context
Add any other context about the problem here.

This is not a bug report and you know it

Should I open a feature request then? I thought it was a bug, since "Dope Version Support" is one of the features and clearly it has a bug in it (aka no Dope Version Support).


Website theme improvements

I extracted the (IMO) ready to merge pieces from #46 and the navbar CSS from #51. The idea is we can merge all this now then rebase other PRs for further hacking.

FYI @zregvart any objections?


Change function signature of 'load' in format loaders from 'linesToLoad' to 'instancesToLoad'

Loaders in 'format' now has 'linesToLoad' as parameter which assumes data is organized line by line. However it is not always the case, and the 'load' function should know the exact data format and handle it by itself.

resolved through #26


Invisible Textscreen

When i load my server up have the textscreen addon installed have the sbox commadn up i press 3D2D Textscreen and when i spawn it is invisible and i can pick it up plz help

Please try adding "-dxlevel 95" to your Garry's Mod launch options. (Steam -> Right click Garry's Mod -> "Set launch options...")

Do you type any text in settings?

@Rorke any more information on this? Additionally, have you tried the addon by itself in a vanilla environment (all other addons disabled)?

Closing this due to inactivity. Please create a new issue if you have any further information.


Fix the broken NuGet build.

It's been a few days of us not building NuGet packages - here's what happened.
We hit a CI outage earlier in the week caused by an upstream Azure DevOps issue (a bad nuget push package was pushed out). Our build was broken for a couple of days, during which we still had changes going in (mostly we were judging by green-ness of mrtk_pr, which is identical to CI except it doesn't produce and publish NuGet packages).
The problem is, without coverage on the NuGet building, we didn't know that we broke the nuget packages until the upstream nuget push package got fixed.
The asset retargeter assumes that every single class/object in the MRTK is prefixed by the MRTK namespace. This is a fairly reasonable assumption to make, and it throws when it finds something that violates it (so that we can then fix it). The fix is thus to make sure that the class in question is put into an MRTK namespace (the example demos one)
I will follow up here and do work to make sure that the build breaks when this happens, instead of just showing warnings.

/azp run mrtk_pr


What is the actual size of the NEO Blockchian?

Hi, I am making a wallet an need to download the blockchain, but I have a slow internet connection.
I was wondering what the actual size of the Blockchain is and, ev. the last blocknr.
I looked everywhere but didn't find this information
Thanks for looking at this!
Erwin

Currently the MainNet Chain sits at around 2GB.
Please use bootstrap files to accelerate the syncing process. You find these here:
https://github.com/CityOfZion/awesome-neo

Thank You!
2017-12-28 20:29 GMT+01:00 fabwa<EMAIL_ADDRESS>
Currently the MainNet Chain sits at around 2GB.
Please use bootstrap files to accelerate the syncing process. You find
these here:
https://github.com/CityOfZion/awesome-neo
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/neo-project/neo-gui/issues/119#issuecomment-354343765,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AfxsHqM0fM2vGFIpdBPExhtXaEZqdSs6ks5tE-wKgaJpZM4RNSR7
.


A timer with 24-hour timeout fires immediately

Describe the bug
In Mesh implementation in net.c there is a ivu_timer that has a 24-hour timeout, yet it fires immediately causing invalid behavior of IV Update Procedure. After changing the timeout to 10 seconds the timer was behaving as expected. Possibly the large value of timeout is causing invalid behavior. I believe that this issue is related to the recent changes in k_timeout_t.
To Reproduce
Run MESH/NODE/IVU/BV-01-C test in PTS and follow the instructions.
It should be also possible to reproduce it by testing against another Zephyr node:
Provision two nodes. Enable IV Update test mode in both. Start IV Update procedure in one of them and observe the behavior of the other one.
I verified the invalid behavior by looking at debug logs.
Expected behavior
ivu_timer firing after 24 hour period.
Screenshots or console output
<snippet>
[00:00:17.298,980] <dbg> bt_mesh_beacon.secure_beacon_recv: net_idx 0x0000 iv_index 0x0000000b, current iv_index 0x0000000a
[00:00:17.298,980] <dbg> bt_mesh_net.bt_mesh_net_iv_update: IV Update state entered. New index 0x0000000b
[00:00:17.299,011] <dbg> bt_mesh_net.bt_mesh_net_beacon_update: NetIndex 0 Using current key
[00:00:17.299,041] <dbg> bt_mesh_net.bt_mesh_net_beacon_update: flags 0x02, IVI 0x0000000b
[00:00:17.299,530] <dbg> bt_mesh_net.ivu_refresh: IVU in Progress for 24 hours
[00:00:17.299,652] <dbg> bt_mesh_net.ivu_refresh: IVU in Progress for 48 hours
[00:00:17.299,865] <dbg> bt_mesh_net.ivu_refresh: IVU in Progress for 72 hours
[00:00:17.300,018] <dbg> bt_mesh_net.ivu_refresh: IVU in Progress for 96 hours
[00:00:17.300,048] <dbg> bt_mesh_net.bt_mesh_net_iv_update: Normal mode entered
[00:00:17.300,109] <dbg> bt_mesh_net.bt_mesh_net_beacon_update: NetIndex 0 Using current key
[00:00:17.300,109] <dbg> bt_mesh_net.bt_mesh_net_beacon_update: flags 0x00, IVI 0x0000000b
[00:00:17.300,567] <dbg> bt_mesh_beacon.beacon_send:
[00:00:17.300,567] <dbg> bt_mesh_beacon.secure_beacon_send:
[00:00:17.300,598] <dbg> bt_mesh_net.ivu_refresh: IVU Normal mode for 24 hours
[00:00:17.300,720] <dbg> bt_mesh_net.ivu_refresh: IVU Normal mode for 48 hours
[00:00:17.300,842] <dbg> bt_mesh_net.ivu_refresh: IVU Normal mode for 72 hours
[00:00:17.300,964] <dbg> bt_mesh_net.ivu_refresh: IVU Normal mode for 96 hours
</snippet>

Environment

OS: Linux
Toolchain: Zephyr SDK
HW: nRF52840DK
SHA: d96a1a5fef19729b47bc8bc17335ad5ae1f852a6

@pabigot does this sound familiar?

Can you post the .config for the application, and an indication of where the timeout is being set, if you have it?
My blind guess is that this is an overflow bug in the code computing the timeout.  If you have a 32 bit quantity that goes negative and construct a timeout out of it, the newer code will clamp to zero in a macro (the older code would just give you whatever junk was computed, cast to an unsigned value).

The bt_mesh.ivu_timer is initialized in subsys/bluetooth/mesh/net.c.
Here is the config file: config.txt

It sounds familiar because (a) somewhere there's an issue that reported this well over a year ago; and (b) because we've been discussing it on slack. https://zephyrproject.slack.com/archives/C18PLHN8H/p1592386718046500
Ensuring the calculation was done with unsigned values, and converting to an absolute timeout, apparently didn't fix the problem.  This should be reproduced in a standalone example to confirm there's no bug in absolute timeouts; then there are fixes needed to ensure that the transformation doesn't result in a signed intermediate.  E.g. K_HOURS should not be scaling by a signed integer.

@andyross This reproduces the problem
#include <zephyr.h>
#include <sys/printk.h>

#define INTERVAL K_HOURS(24)  // change to 18 and it works (nordic-specific boundary)

static struct k_delayed_work work;
static void do_work(struct k_work *wp)
{
	struct k_delayed_work *w = CONTAINER_OF(wp, struct k_delayed_work, work);

	printk("%" PRIx64 " now\n", k_uptime_get());
	k_delayed_work_submit(w, INTERVAL);
}

void main(void)
{
	k_delayed_work_init(&work, do_work);
	k_delayed_work_submit(&work, K_NO_WAIT);
	while (true) {
		k_sleep(K_HOURS(24));
	}
}

Cool, thanks.  Reproducers are always the magic.  Two spots were still 32 bit, fix submitted.
The constructor macros weren't actually the problem here, it was the dticks field that was wrong.  And that's why it was subtle to reproduce: you needed to have a very long timeout inserted into the queue after an existing queued timeout.

I tested the fix and it works as expected. Well, I didn't wait for 24 hours but it doesn't fire immediately anymore :)

Excellent, thanks.  As I read the bug, we were actually pretty lucky to have discovered it.  The rollover was in a tick delta, which is below the clamping I mentioned earlier and in code that assumes unsigned numbers.


Allow spreading Parameters<T> into a named tuple

Suggestion
Allow spreading named tuples into named tuples, and make Parameters<T> a named tuple.
🔍 Search Terms
parameters named tuple
✅ Viability Checklist
My suggestion meets these guidelines:

[ ] This wouldn't be a breaking change in existing TypeScript/JavaScript code
[x] This wouldn't change the runtime behavior of existing JavaScript code
[x] This could be implemented without emitting different JS based on the types of the expressions
[x] This isn't a runtime feature (e.g. library functionality, non-ECMAScript syntax with JavaScript output, new syntax sugar for JS, etc.)
[x] This feature would agree with the rest of TypeScript's Design Goals.

Not expert enough with Typescript to know whether this would be a breaking change. Definitely wouldn't break any JavaScript code, but might "break" some existing usage of the Parameters<T> utility? Breaking could be avoided if there was some sort of logic of "if spreading this named tuple into another named tuple, keep the names, but if spreading it into an unnamed tuple, drop the names", maybe?
⭐ Suggestion
Say I have this function:
type ClampFn = (value: number, range: [min: number, max: number]) => number;

Now I want to create a named tuple, which includes all its parameters, and its return type at the end. Doing it manullay works of course:
type ManualType = [value: number, range: [min: number, max: number], returns: number]

But doing it via the Parameters<T> and ReturnType<T> utilities gives the error that "Tuple members must all have names or all not have names":
type InferredType = [...Parameters<ClampFn>, returns: ReturnType<ClampFn>];
                     ^^^^^^^^^^^^^^^^^^^^^^

Would be great if Parameters<T> returned a named tuple, and if it was possible to spread named tuples into named tuples and keeping the names (maybe already possible?). To prevent breaking, it should also be possible to spread a named tuple into an unnamed tuple (maybe already possible?).
https://www.typescriptlang.org/play?ts=4.7.2#code/C4TwDgpgBAwgNgQwLZgGIDsoF4oAoBuCcArhAFxTrFIBGEATgDRT0LoDm5UA2kgJboKVWg2ZIEADyHU69ALoBKbAD5KMhgG4AUFtCQoAWTbEiAFXDQc3QiS7DZzVhy68B0kUyjipaj3McQwMT06ADO7rJyuhZQAJLoAGYM9BAAJub6VgB0OQAKCKxIgQyhADzwyGjoygFBIeFQAEqBwegZEOWIKBjKctpAA
📃 Motivating Example
I've started using the test.each feature of Jest, and this would be very handy when defining the type of test-cases you expect to pass into the each function.
Currently have to do it manually, to get named parameters and auto-complete in the test-function parameter list:
    it.each<[input: number, range: [min: number, max: number], result: number]>(
      [
        [0, [1, 3], 1],
        [1, [1, 3], 1],
        [2, [1, 3], 2],
        [3, [1, 3], 3],
        [4, [1, 3], 3],
      ]
    )('%p clamped within %p should be %p', (input, range, result) => {
      expect(clamp(input, range)).toBe(result);
    });

💻 Use Cases
If I could spread Parameters<T> into a named tuple, then I could easily "lock" the type of test-cases with the actual function arguments, and get correct parameter names in the named tuple directly from the function:
    it.each<[...Parameters<typeof clamp>, result: ReturnType<typeof clamp>]>(
      [
        [0, [1, 3], 1],
        [1, [1, 3], 1],
        [2, [1, 3], 2],
        [3, [1, 3], 3],
        [4, [1, 3], 3],
      ]
    )('%p clamped within %p should be %p', (input, range, result) => {
      expect(clamp(input, range)).toBe(result);
    });

@weswigham do you see any blockers here? Inclined to mark this 'help wanted'

Note that you can spread Parameters<T> into a named tuple, just use an ignored label:
type InferredType = [...params: Parameters<ClampFn>, returns: ReturnType<ClampFn>];
//    ^? type InferredType = [value: number, range: [min: number, max: number], returns: number]

See also https://github.com/microsoft/TypeScript/issues/39941


Add FLAN-T5

This is ready for review again. PTAL

@teetone could you take another look? This PR is blocked on your review.

Thanks!


Use java-test-fixtures plugin

Incorporate the java-test-fixtures plugin to share common testing code across the project and delete commons-testing module, which acts as a replacement for it at this point.
Unfortunately, the plugin, at the moment of writing this issue, is only available for pure java and kotlin modules, meaning it cannot be used for android modules. There is a feature request on the Google Issue Tracker to support java-test-fixtures plugin.
A good resource about what text-fixtures-plugin is and what it can do can be found here.

https://youtrack.jetbrains.com/issue/KT-50667


Setting environment variables when activating conda env

Conda packages can have scripts to set and unset environment variables when the environment is activated and deactivated.
https://docs.conda.io/projects/conda/en/latest/user-guide/tasks/manage-environments.html#saving-environment-variables
These get added to .CondaPkg/env/etc/conda/activate.d/ and .CondaPkg/env/etc/conda/deactivate.d/
Currently from what I can tell CondaPkg.withenv(f::Function) doesn't run these scripts.

Indeed, most Conda packages do not use this feature, so it hasn't been implemented by CondaPkg.
Are there particular packages you're having issue with.

Indeed, an earlier version of CondaPkg did this, but it was not super reliable. It could be reimplemented though.


Fix raitile to work with Rails 4.2.1

I don't see anything in this PR that's specific to Rails 4, other than coercing a nil asset_host into an empty string.
I think we need to allow handling nil/empty Emoji.asset_host values to do the right thing in cases where a Rails app doesn't have an asset_host configured.
Also, for future reference please don't reformat so much of the class if the actual change was a one line fix in the railstie.
Thanks for taking the time to submit this PR and raise the issue, but I don't think this PR is the right solution so I'm going to close it.


Incompatable with Architectury API

Crash report: https://pastebin.com/FEPgREjp
Architectury API seems to conflict with the mixin which this mod uses. I have tried running the game with and without Sodium, as well as any other mods that came up in the stack trace.

Fixed in 830dd96e4e490f7bd40defabda9a189f9c9677f8


Unify duplicate code in MMLU and MMLUBranch

Followup from #19
MMLU and MMLUBranch have a block of identical code - this should be unified into a single function both classes can call

I think this is fixed here: https://github.com/instructlab/eval/pull/52/files


Less menues after "Close other tabs"

Godot version:
v3.2.stable.official
OS/device including version:
Windows 7 ultimate (version 6.1.7601 Service Pack 1 compilation 7601)
Issue description:
Sometimes when I open a text editor or switch to it I get it in a different "mode". In that "mode" I see less menu entries (only "File" and "Debug" instead of the five usual File/Search/Edit/Go To/Debug) and I'm not able to do some actions (e.g. ctrl-F for find doesn't work, and that's why I realize it's bugged). It apparently works fine apart from that two symptoms.
It's a very small issue because I simply switch to another text editor (or to 2d mode and back to script mode) and the problem is gone.
I'm not able to reproduce it, looks like random to me; it's more frequent when I just opened the Godot editor, though.
Steps to reproduce:
Can't reproduce it.

Can you take a screenshot and upload it next time you manage to reproduce the bug?

Yes, I will. Sorry for opening the issue without the screenshot (I had taken it and accidentally lost it, then decided to open the issue nevertheless).
I'm pretty sure it will be a normal screen with only File and Debug menues though.

I got the bug, this time after doing Close Other Tabs within the text editor (it probably doesn't have nothing to do with the issue though, because I got the bug in another circumstances too).
Here goes the screenshot. I recorded a video too, it's here in mkv format.

Got it two more times, always after Close Other Tabs.
I still can't reproduce it though.

Lately I'm getting it only after Close Other Tabs and sometimes with very high frequency (looking almost reproducible) so I changed the issue title. You can see in this video how I got it three times in a row.
But it doesn't happen always, and I'm pretty sure I got it in other circumstances too.

I changed the issue description as I'm getting the same bug in Linux (Ubuntu 20.04, 3.2.1.stable.official).


Update dcos diagnostics

High-level description
Update dcos-diagnostics with fixes for new unreleased API. Changelog is not required since this API was not announced yet.
Corresponding DC/OS tickets (required)

DCOS_OSS-5467 Include optional file in the bundle
DCOS_OSS-5470 Deletion of cluster bundle doesn't delete node bundles
DCOS_OSS-5449 Empty files in a bundle
DCOS_OSS-5456 Cluster Bundle Handler includes any bundles on the master nodes
DCOS_OSS-5454 diagnostics list breaks when no bundles have been created

Related tickets (optional)

DCOS-ID JIRA title / short description.

Checklist for component/package updates:
If you are changing components or packages in DC/OS (e.g. you are bumping the sha or ref of anything underneath packages), then in addition to the above please also include:

[x] Change log from the last version integrated (this should be a link to commits for easy verification and review): example
[x] Included a test which will fail if code is reverted but test is not. If there is no test please explain.
[x] Test Results: link to CI job test results for component
[x] Code Coverage (if available): link to code coverage report

@mesosphere-mergebot label Ready For Review

@mesosphere-mergebot request-review @drewvanstone @br-lewis @makkes @bamarni @rgo3

@mesosphere-mergebot changelog-not-required Changelog is not required since this API was not announced yet.

:wave: @drewvanstone @br-lewis @makkes @bamarni @rgo3  Can you review, please?


Add test for update to Add-PoshGitToProfile

This test checks that the command creates the profile parent dir (or any in the path) if they don't exist.  The command was modified to create any dir in the path by the addition of the -Force parameter on New-Item.
Follow-up on #449

Simplify missing profile parent test

For your consideration... since we already have cleanup infrastructure (for the record Remove-Item -Recurse works fine on file paths), the test seems cleaner without the try...finally.

Nice!  I got hung up $profilePath being a generated filename (not a dir) but yeah, deleting the file and usurping that name for the dirname does make this cleaner.  Thanks!


Latest version wasn't released to npm

This is a follow up of https://github.com/microsoft/playwright/issues/5591#issuecomment-811837895
Version was bumped but not released to npm
Thanks

cc @aslushnikov

@haikyuu thank you for the heads up! Fixed.

most welcome  @aslushnikov


Added docs for agit-setup

This PR Adds Documentation for the new Agit feature. Agit allows creating PR directly while pushing code.
Close #21018

:rocket:


function call optimization

Some functional languages with immutable data support an optimization where a function calls with identical arguments will only be called once and the return value is cached.
Is that currently a feature of Power-fx?  Or is it planned?

Answered my own question, output properties on a component accomplish this purpose.  It would be nice to see the language keep this ability as it evolves to support more than canvas apps.

Answered my own question, output properties on a component accomplish this purpose. It would be nice to see the language keep this ability as it evolves to support more than canvas apps.


.deb

Is your feature request related to a problem? Please describe.
Installing from AppImage requires right clicking the app and setting permissions, which can be laborious.
Describe the solution you'd like
.deb packages are a better default, double-click to install without any extra messiness
Additional context
This should be easy enough by adding "deb" to the list of "targets" in package.json

rel https://github.com/beakerbrowser/beaker/issues/1568

Would this post of mine be helpful?
It's not a Debian Package, but BeakerBrowser is available in Flatpak and the repository "Flathub".
Flatpak must be installed beforehand, but if you install it with Flatpak's CLI command or on Flathub's page, it will even register in the application menu.
Flatpak Official Web site
Flathub app detail about Beaker
In addition, my environment is Ubuntu 20.04.


Testing user story 75

related to https://github.com/ebi-ait/dcp-ingest-central/issues/75#issuecomment-735724883
Testing errors
First submission
Tried to upload a super broken spreadsheet. Didn’t work. It pointed to errors, although they are pretty well pointed out.
Trying with another sheet with the error “UnexpectedUuidEntity” solved.
Fatal errors make the spreadsheet unloadable and you have to look at the json response. In this case, it was a boolean error (NOO instead of yes/no, which I guess ingest can’t convert), and integer conversion errors (96O6 instead of 9606).
Fixing those ones for the next iteration to see if it loads the spreadsheet.
Adding a non-existing link also breaks the import with the error:
A link from a biomaterial with id specimen_ID_1 to a biomaterial with id, "not_a_real_donor_id", is not found in the spreadsheet.

Only showing that 1 error. Going to fix it and then see if it can load the spreadsheet
Last submission
https://dev.contribute.data.humancellatlas.org/submissions/detail?uuid=5b0f1d6d-6d45-4c18-9273-e023b5f93d30
Now it’s showing the errors as intended. Immediately, the first thing I noticed is that project and submission errors are in a different format, which is not ideal. Screenshot attached.

Going type by type:
Biomaterials:
Clicked to show biomaterials. Shows only the donor, even though the specimen also should have biomaterial errors.
For now, looking at the metadata errors:

Can we get the user friendly name in the errors?
Also, about the linking, it shows unlinked, but there is no intuitive way to know how to fix the linking. My first thought would be to go to edit the biomaterial and link it there, but we have to go to processes.
Why is body mass index a boolean?
About the biomaterial_core error: Can we state which additional properties are they trying to add?
Other than that, the other errors are easy to spot.
File errors:
Metadata errors are easy to spot, kind of the same comments as the previous one.
I like the metadata errors being different from the “file missing” and stating both, but there is a bug in the file metadata/data validation: When the metadata is invalid, it won’t list the missing data file. In the specific example, I would have expected “1 metadata error, 2 files missing” but it’s stating only 1 file missing.
I uploaded an invalid file for the file with the invalid metadata. No changes.
Corrected the metadata in the UI and it is validating now. After correcting the metadata and uploading an incorrect file, no error shows up, but the file is invalid. It just goes to invalid without specifying “invalid data file”. I think this is another bug!
So far I have come up with this. Happy to test further if needed?

Hi, Enrique!!!
These look awesome.
cc @aaclan-ebi and @jacobwindsor so that we can maybe break these down and see what to do about them.
I think I need to think of more options of visualising errors.
Gabs


[IO-764] IOUtils.write() throws OutOfMemoryError/NegativeArraySizeException while writing big strings

If Strings with a certain size are written to a stream, this fails with an exception on OpenJDK 11.
java.lang.NegativeArraySizeException: -1283060862
at java.base/java.lang.StringCoding.encodeUTF8(StringCoding.java:904)
at java.base/java.lang.StringCoding.encode(StringCoding.java:449)
at java.base/java.lang.String.getBytes(String.java:964)
at org.apache.commons.io.IOUtils.write(IOUtils.java:3251)
at org.apache.commons.io.FileUtils.writeStringToFile(FileUtils.java:3541)
at org.apache.commons.io.FileUtils.writeStringToFile(FileUtils.java:3524)

This PR fixes the problem by using a ByteBuffer instead getBytes.
Unfortunately, this increases test time significantly. One workaround would be to disable the test and only enable it if needed - but testing whether big Strings can be written is not possible without handling such a big string.

Unfortunately, this does not work on GH Actions since the RAM is too small. Should I increase the RAM for the test or disable the test?

https://issues.apache.org/jira/browse/IO-764

Thanks for the hints. I've removed my test, added the comments, removed closing of the channel and fixed the checkstyle problems, and I rebased to latest master state.
Getting the behaviour by mocking is not possible from my point of view, because the problem happens when getBytes produces an array that is too big for handling - returning this array from a mocked getBytes would result in the same ressource usage as having it inside a String.
Is this ok from your point of view?

Hi @DaGeRe
Thank you for the update.
Something is wrong with this PR. It contains unrelated changes you did not author.

Thanks for the hint, these were included on rebasing.
To fix this, I branched freshly from master, made the change and committed via force-push.
Is this ok from now from your point of view?

Now it git master.


RedirectURL?

Hello,
What do I need to use for redirect URL?

You can setup a reply callback in Azure portal which can be any url, then
use that url on RedirectUrl
On Thu, Aug 9, 2018, 5:33 AM Amine BIZID<EMAIL_ADDRESS>wrote:

Hello,
What do I need to use for redirect URL?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/judedaryl/nativescript-azure-auth/issues/1, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ANA05-g4xSfQjNGlGsNJDPGVrsIKQ4oNks5uO1lCgaJpZM4V0svm
.


FluentDConfigCheck does not get scheduled du to inherited anti-affinity rules

I have added the following pod affinity rules to my fluentd config to ensure that the pods are spread over the nodes:
    fluentd:
      affinity:
        podAntiAffinity:
          requiredDuringSchedulingIgnoredDuringExecution:
          - labelSelector:
              matchExpressions:
                - key: "app.kubernetes.io/name"
                  operator: In
                  values:
                    - fluentd
                - key: "app.kubernetes.io/component"
                  operator: In
                  values:
                    - fluentd
            topologyKey: "kubernetes.io/hostname"

Now the fluentd-configcheck pods stay in pending state. It seems that they inherit the configuraton of the fluentd statefulsets.
18m         Warning   FailedScheduling    pod/logging-operator-fluentd-configcheck-e97a6f8f     0/8 nodes are available: 2 node(s) didn't match Pod's node affinity/selector, 3 node(s) didn't match pod affinity/anti-affinity rules, 3 node(s) didn't match pod anti-affinity rules, 3 node(s) had taint {CriticalAddonsOnly: true}, that the pod didn't tolerate.

Is there a way to make sure that the fluentd pods are distributed, but the configcheck pods are still scheduled?

Thanks, I have removed the affinity rules for now.
My other idea, to reduce the number of fluentd replicas, to be able to run the configheck periodically, works only as long as all the nodes are available and ready.
I think it was useful being able to configure the configcheck sperately.

Can't we just drop the pod anti affinities altogether for fluentd configcheck in here? https://github.com/kube-logging/logging-operator/blob/e0331c4b508ff54e8b0958d29ace7e8d7427674b/pkg/resources/fluentd/appconfigmap.go#L222

yes, I don't think it makes sense to use the affinity rules in the configcheck pod

I still don't think there is a good reason to apply affinity rules to the configcheck pod, but I also don't want to break this for those who might use it (even if it's accidental right now).
The good solution would be to add override options for the configcheck pod, but the logging resource is already very big and require restructuring.
So the options I see:

we disable inheriting affinity rules (affinity and antiaffinity as well) for configcheck pods (backwards incompatible)
we add a flag to control weather we want to allow inheriting the rules or not, but then:

enable flag (disable inheriting by default, which could break existing code, let users opt-in)
disable flag (enable inheriting by default, but let users enable it, which is not a good experience if most of the time it's not needed)

we live with this and say configcheck pod requires an extra node in this specific case.

What do you think?
also cc @ahma @tarokkk

Any updates?

After this time I would simply go with this: https://github.com/kube-logging/logging-operator/pull/1787


Making an alert from a Redux action?

Thank you and congratulations on the project, it is great!
A question:  I want one of my Redux actions to show an alert.
Currently, my Redux action sets a Redux property, then a component, in its componentWillReceiveProps method:

Sees the updated property and calls this.props.alert.show
Then it dispatches a Redux action to reset the property. (If it doesn't reset the property then there is an infinite loop.)

This is all working fine, but is there a more direct way for an action to call alert.show?
Many thanks,
Larry

Hey @LarryKlugerDS,
So you are actually looking for a way to show an alert based on a prop update.
I would recommend you to do it in the componentDidUpdate lifecycle (the way you are doing it also works, but componentWillReceiveProps is not recommended anymore, see here).
Hope it helps!