pythonPackages.pillow: Add libxcb support, convert to generic; pythonPackages.pillow-simd: Init at 7.0.0.post3

Motivation for this change
Things done

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

[ ] NixOS
[ ] macOS
[ ] other Linux distributions

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

This is a semi-automatic executed nixpkgs-review. If you find some bugs or got suggestions for further things to search or run please reach out to SuperSandro2000 on IRC.
Result of nixpkgs-review pr 108098 run on x86_64-linux 1

3 packages built:

python37Packages.pillow-simd
python38Packages.pillow-simd
python39Packages.pillow-simd

This is a semi-automatic executed nixpkgs-review. If you find some bugs or got suggestions for further things to search or run please reach out to SuperSandro2000 on IRC.
Result of nixpkgs-review pr 108098 run on x86_64-linux 1

3 packages built:

python37Packages.pillow-simd
python38Packages.pillow-simd
python39Packages.pillow-simd

Result of nixpkgs-review pr 108098 run on x86_64-darwin 1

3 packages failed to build:

python37Packages.pillow-simd
python38Packages.pillow-simd
python39Packages.pillow-simd

Result of nixpkgs-review pr 108098 run on x86_64-darwin 1

3 packages failed to build:

python37Packages.pillow-simd
python38Packages.pillow-simd
python39Packages.pillow-simd

Given this is intended to be compatible with upstream, I can't help but wonder if a generic.nix-approach would be better to avoid duplicating the package specification between pillow and pillow-simd, but that's probably more a question for the pillow maintainers, since it pushes maintenance onto them a bit.

Given this is intended to be compatible with upstream, I can't help but wonder if a generic.nix-approach would be better to avoid duplicating the package specification between pillow and pillow-simd, but that's probably more a question for the pillow maintainers, since it pushes maintenance onto them a bit.

/rebase-staging

/rebase-staging

Given this is intended to be compatible with upstream, I can't help but wonder if a generic.nix-approach would be better to avoid duplicating the package specification between pillow and pillow-simd, but that's probably more a question for the pillow maintainers, since it pushes maintenance onto them a bit.

The code between pillow 6, 8 and simd is almost identically and I just went ahead and created a generic.nix and added me as a maintainer.

Given this is intended to be compatible with upstream, I can't help but wonder if a generic.nix-approach would be better to avoid duplicating the package specification between pillow and pillow-simd, but that's probably more a question for the pillow maintainers, since it pushes maintenance onto them a bit.

The code between pillow 6, 8 and simd is almost identically and I just went ahead and created a generic.nix and added me as a maintainer.

@ofborg eval

@ofborg eval

/rebase-staging

/rebase-staging

I thought I could avoid the staging here but whatever.

I thought I could avoid the staging here but whatever.

Please mark PRs as draft while rebasing.

Please mark PRs as draft while rebasing.

Please mark PRs as draft while rebasing.

@roberth rebases are now automated via github actions.

Please mark PRs as draft while rebasing.

@roberth rebases are now automated via github actions.

@Mic92 I was referring to this manual rebase https://github.com/NixOS/nixpkgs/pull/108098#event-4174211571

automated via github actions.

Judging by the timeline here and the workflow definition, I don't see a reason why it wouldn't suffer from the same problem, though I'll give it the benefit of the doubt until I see it happen again. The spam issue didn't occur with all rebases after all.

@Mic92 I was referring to this manual rebase https://github.com/NixOS/nixpkgs/pull/108098#event-4174211571

automated via github actions.

Judging by the timeline here and the workflow definition, I don't see a reason why it wouldn't suffer from the same problem, though I'll give it the benefit of the doubt until I see it happen again. The spam issue didn't occur with all rebases after all.

@Mic92 I was referring to this manual rebase #108098 (comment)

automated via github actions.

Judging by the timeline here and the workflow definition, I don't see a reason why it wouldn't suffer from the same problem, though I'll give it the benefit of the doubt until I see it happen again. The spam issue didn't occur with all rebases after all.

Fair enough. I only the bot one.

@Mic92 I was referring to this manual rebase #108098 (comment)

automated via github actions.

Judging by the timeline here and the workflow definition, I don't see a reason why it wouldn't suffer from the same problem, though I'll give it the benefit of the doubt until I see it happen again. The spam issue didn't occur with all rebases after all.

Fair enough. I only the bot one.

@ofborg eval

@ofborg eval

@ofborg eval

@ofborg eval


finding containing string in ors

Model.find({
"or": [
{"field1": {"contains": "query"}},
{"field2": {"contains": "query"}},
{"field3": {"contains": "query"}}
]
});

Hi @calidion, what's your question / issue?

@dmarcelino
currently find criteria only support exact matched search,
which is
a = b

but sometimes we need searches like
a like "%b%"

and can be arranged in side or array.
turn this
Model.find({
"or": [
{"field1":  "query"},
{"field2": "query"},
{"field3": "query"}
]
});

to this
Model.find({
"or": [
{"field1": {"contains": "query"}},
{"field2": {"contains": "query"}},
{"field3": {"contains": "query"}}
]
});

Have you tried using or with contains in waterline? It's supported so it should work. There are some tests for it in waterline-adapter-tests and it has come in other issues like https://github.com/balderdashy/waterline/issues/978#issuecomment-96598798. Is this happening with a specific adapter or more than one?

i have tried.
It was anticipated to be able to work, but turned out to be not.
so I rewrote it with raw sql queries.

Which adapter were you using? Did you try another adapter?

@dmarcelino  mysql adapter.  haven't tried any another adapter.

@calidion, I've submitted PR balderdashy/waterline-adapter-tests#76 which adds a test for OR with multiple contains mimicking your scenario and the test passes in all adapters including mysql as you can see in: https://travis-ci.org/balderdashy/waterline-adapter-tests/jobs/63937899#L1657
The test itself:
      it('should work correctly when OR is used with multiple contains modifiers', function(done) {
        Queryable.User.find({
          type: 'or test',
          or: [
            { first_name: { contains: 'user0' } },
            { last_name:  { contains: 'user1' } },
            { title:      { contains: 'title2' } }
          ]
        })
        .sort('first_name')
        .exec(function(err, users) {
          if(err) return done(err);

          assert(Array.isArray(users));
          assert.strictEqual(users.length, 3);
          assert.equal(users[0].first_name, 'OR_user0');
          assert.equal(users[1].last_name, 'contains_user1');
          assert.equal(users[2].title, 'contains_title2');
          done();
        });
      });
    });

Are you using the latest versions of waterline and sails-mysql?

@dmarcelino
"sails-mysql": "0.10.6"
waterline are not included in the package, but the sails version is the most recent one.

You should be able to check your waterline version by doing npm ls waterline. If it's not the latest (v0.10.22) you can update it by going into sails folder, cd node_modules/sails and doing npm update waterline.
If that's not the issue is it possible that the data you have in your DB is not what you expect it to be?

@dmarcelino
should be the lastest version.
    options.online = {'!' : true};

and this won't work properly too.

I would like to know how to get the full sql when every sql executed or error occurred.

Set environment variable LOG_QUERIES=true.

@dmarcelino
I don't think we need to set this every time.
I think in debug mode, this option will always be true.

Not sure what you mean @calidion, LOG_QUERIES=true is for logging the SQL queries that sails-mysql produces, in response to what you asked earlier.

I mean no need to set this PARAMETER when debugging.
throw whole sql whenever queries failed.

@calidion, have you found the root cause of the issue? Are you able to create a test like the one in https://github.com/balderdashy/waterline/issues/1019#issuecomment-105207246 that reproduces this?

Closing this as we were unable to reproduce (PR balderdashy/waterline-adapter-tests#76), anyway if an issue exists it's probably in the MySQL adapter and that should be discussed in https://github.com/balderdashy/sails-mysql.


fin-principal match fin-jwt

This has fin-principal match fin-jwt in request.  eg, check cookie, then header.
I also deleted getUserFromRequest, didn't look right

line 206 and 210 of keycloak.js should be a configuration parameter

@jrmerz updated with parameterized

I'm curious, if a fin application where to allow keycloak based username/password login (ie third party with no SSO wired up), would auto appending a default domain be incorrect?  It's an edge case, an I live where this is going, but just something to think about.

Yeah, I'm also a bit concerned that from an fcrepo standpoint<EMAIL_ADDRESS>is still a string and not a URI.  I basically copied eduroam, TBH, but we may need to review later

I'm curious, if a fin application where to allow keycloak based username/password login (ie third party with no SSO wired up), would auto appending a default domain be incorrect? It's an edge case, an I live where this is going, but just something to think about.

One more thought, can we add an enabled flag for adding the @[domain] to the principle roles list and set it to false by default?

Only adds if  eg PRINCIPAL_ADD_DOMAIN='ucdavis.edu'


Documentation on available methods

Hi, is there a documentation page on all the available methods in this library? I am specifically looking for options to iterate over OWL axioms in functional syntax using python operators and some of the examples in Jupyter Notebooks indicate that this would be the perfect library for me.
Thanks

I need to get one in place -- got side tracked but should get back to it.
One of the things we still need to get finished is "convenience methods" -- the extra ones that I've been adding to the Ontology itself that takes care of some of the messier constructors and the like.  Any advice or suggestions are welcome


Support office documents and check for macros

https://github.com/decalage2/oletools could prove useful

Started analyzing with commit 6509f418feb7684c46b76eefb28b82baccbef286

Automatic Macro detection implemented with quicksand in commit 3254923e12367a4b4d82db7b03d805e4cefcee68. However, Macro removal can still be implemented.

Added multiple YARA rules for detecting Office Documents with Macros.
A few selected rules:

rule office_document_vba : maldoc
Contains_VBA_macro_code (Detect a MS Office document with embedded VBA macro code - by evild3ad)
rule macrocheck : maldoc (Identify office documents with the MACROCHECK credential stealer in them. - by Fireeye Labs)
rule Contains_hidden_PE_File_inside_a_sequence_of_numbers : maldoc (https://www.welivesecurity.com/2016/01/04/blackenergy-trojan-strikes-again-attacks-ukrainian-electric-power-industry/  - by evild3ad)


Added table footer methods and fixed table caption on top

Single commit to add table footer methods and fix table caption on top (now on bottom).
I already fixed doc example and test module.

Thanks so much for opening a PR! It looks like some of the tests are failing in the automatic workflow, so I'm going to merge this into a staging branch in this project while I investigate.


Add standalone option to install Cave Story (NXEngine)

The reason why the script has to write another script (Starting at line 43, ending at line 47) is because the executable can't be called from afar and you have to cd into the directory, and then you execute the executable from in there to be able to work.

Thanks.


Add ibc-go v9 to chains running tests in CI

Closes: #4151
Description

This PR adds ibc-go v9.0.0-beta.1 to the matrix of chains running the tests in the CI.
Disable denom trace test with ibc-go v9 since the gRPC query DenomTrace has been replaced with queries Denom and Denoms, https://github.com/cosmos/ibc-go/blob/release/v9.0.x/modules/apps/transfer/keeper/grpc_query.go#L27-L80.

PR author checklist:

[x] Added changelog entry, using unclog.
[x] Added tests: integration (for Hermes) or unit/mock tests (for modules).
[x] Linked to GitHub issue.
[ ] ~Updated code comments and documentation (e.g., docs/).~
[x] Tagged one reviewer who will be the one responsible for shepherding this PR.

Reviewer checklist:

[ ] Reviewed Files changed in the GitHub PR explorer.
[ ] Manually tested (in case integration/unit/mock tests are absent).

Thanks!


hande empty maven projects - or figure out an easy way to disable the f-m-p plugin executions on a project in a multi-maven project?

in fabric8-devops we configure the f-m-p plugin in a profile: https://github.com/fabric8io/fabric8-devops/blob/master/pom.xml#L334-L380 which means the plugin is then active for all child projects.
It turns out we don't want to generate any resources or perform a docker build for one of these child projects:
https://github.com/fabric8io/fabric8-devops/tree/master/collector-utils
we could totally refactor things and add a level of indirection; but I wonder if we can figure out an easy way to disable f-m-p for this project? Maybe a maven property or a configuration or something? I tried re-defining the executions (to an empty list of goals) in this project; or tried redefining the fmp3 profile; neither worked.
For now I'm gonna keep this project on the 2.x maven plugin as a hack ;) it'd be nice to figure out a nice way around this one really.
e.g. maybe the f-m-p should ignore any project which there is not a Container generated or configured? As generating an empty Deployment with no containers just causes issues?

You can set certain skip properties / config options on the sub-modules.
E.g. fabric8.skip will skip the resource execution, docker.skip the build execution ...
We have to harmonize the property names, though, but the general mechanism is to let the Mojo Run but do nothing if configured so.
Would this be sufficient here ?

yeah - that'd be fine.
Though I so think in the case that there's no Container configured / generated we shouldn't generate a Deployment yaml file either (and generate a nice warning that no container could be discovered or is configured - then mention that the fabric8.skip property could be used to remove the warning).

so kinda relates to https://github.com/fabric8io/fabric8-maven-plugin/issues/207

to disable a build failing from not being able to access the docker daemon we currently have to specify 2 properties:
mvn clean install -Ddocker.skip.build=true -Ddocker.skip=true

We probably want to just have one property we can used to skip docker from the fabric8-maven-plugin build (e.g. just docker.skip?)


Navigating with command + key behaves differently with backspace than arrow keys

Check for existing issues

[X] Completed

Describe the bug / provide steps to reproduce it
When a line is wrapped in the editor to be shown across multiple lines navigating with command pressed behaves differently depending if arrow keys are used or backspace.
When pressing command + left arrow the cursor moves to the left most position in the wrapped line as expected. With command + shift + left arrow it highlights the wrapped line as it's displayed on the screen.
However using command + backspace ignores the wrapping and deletes the entire line up to the line break in the file. This is pretty unintuitive and seems like a bug.
Environment
Zed: v0.115.3 (stable)
OS: macOS 14.1.2
Memory: 16 GiB
Architecture: aarch64
If applicable, add mockups / screenshots to help explain present your vision of the feature
The issue was tested with a markdown file where the file level lines are wrapped and shown on multiple lines. One example could be the below line
Lorem ipsum dolor sit amet, consectetur adipiscing elit. Sed accumsan non quam ut
vestibulum. Nunc eget eleifend mauris, eget ultrices risus**<cursor here>**

If pressing command + left arrow the cursor will be right before the v in vestibulum, but if pressing command + backspace the entire line is deleted.
If applicable, attach your ~/Library/Logs/Zed/Zed.log file to this issue.
If you only need the most recent lines, you can run the zed: open log command palette action to see the last 1000.
No response

Hey @magnuspaaske - totally get the inconsistency here, but I just wanted to ask, if you were to fix this, would you change the behavior of cmd-delete to match the behavior or say cmd-shift-left, or vice versa?

@JosephTLyons I would change the behaviour of cmd-delete to match that of cmd-shift-left. Might want to hide it behind a setting (or not depending how customizable things should be). FWIW I also checked it on Sublime Text and found the same behaviour


Running the CoseSignTool without any arguments hits an exception rather than printing the helptext

I downloaded the 1.1.0 version of the CoseSignTool and the required .NET dependencies. When I ran the CoseSignTool.exe without any arguments it hit an exception:
%> CoseSignTool.exe
Unhandled exception. System.IndexOutOfRangeException: Index was outside the bounds of the array.
   at CoseSignTool.CoseSignTool.Main(String[] args) in D:\a\CoseSignTool\CoseSignTool\CoseSignTool\CoseSignTool.cs:line 27

This is unexpected, I expected the tool to print out it's help text.
Note that when I passed in --help it worked as expected:
%> CoseSignTool.exe --help

*** CoseSignTool ***
A tool for signing, validating, and getting payload from Cose signatures.

Usage:
    ...

Thank you very much for identifying these issues! This has now been fixed as part of https://github.com/microsoft/CoseSignTool/pull/68 :)


Configurable method text underline of step points (3.0.84)

In this version the color of the step point underline was made slightly darker (at my request).  I think I like this - certainly much easier to add breakpoints - but it's slightly more distracting and might impact code readability.
It might be nice to make this user-configurable.

Customer did not care for the darker underlines.  That makes this feature much more important.

Note - color restored to gray from darkGray for stepPoint underline in 3.0.88

I think we can allow this to not be further changed.


Enabling CI for Mac, Linux and PSCore on Windows

Integrated with TravisCI for both Linux and OSx platforms. Added required build, test actions and .travis.yml file.
Integrated with AppVeyor for PowerShellCore validation on Windows. Added required functionality in build.psm1 file to bootstrap the PowerShellCore on Windows AppVeyor VM.
Enabled all PowerShellGet tests on Linux, OSx with PS Core and on Windows with both PS and PSCore.
Added Contributing.md and Issue_Template.md
Updated README.md with the build status badges from AppVeyor and TravisCI for both master and development branches.
Changed minimum required version of PackageManagement module back to <IP_ADDRESS> so that PowerShellGet module works fine on Linux and Mac OSx platforms.
Renamed PublishModuleIsNotSupportedOnNanoServer errorid to PublishModuleIsNotSupportedOnPowerShellCoreEdition, similarly renamed PublishScriptIsNotSupportedOnNanoServer to PublishScriptIsNotSupportedOnPowerShellCoreEdition.
Fixed an issue in Update-Module and Update-Script cmdlets to show proper version of current item being updated in Confirm/WhatIf message.
Updated Test-ModuleInstalled function to return single module instead of multiple modules.
Updated ModuleCommandAlreadyAvailable error message to include all conflicting commands instead of one. Corresponding changes to collect the complete set of conflicting commands from the being installed.
Also ensured that conflicting commands from PSModule.psm1 are ignored in the command collision analysis as Get-Command includes the commands from current local scope as well.

@KrishnaV-MSFT @jianyunt : Please CR this PR.

Hi @bmanikm, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!
You've already signed the contribution license agreement. Thanks!
The agreement was validated by Microsoft and real humans are currently evaluating your PR.
TTYL, MSBOT;

:shipit:


Load content for the projects' page.

After login page, the user go to the projects' page at projects.html. It is a page where all projects recorded are shown to user and he has to pick one of them to work or create a new one. Those projects are recorded in the database and has a model in model/Projects.java class.
The option to create a new project is in a button bellow the table.

Up to this point, API can CRUD projects by the requests but the front-end is not done yet. I'll let the front-end to other issue.


Add built-in command for selecting the editor canvas when it is iframed

Is your feature request related to a problem? Please describe.
On WordPress 6.3 and above, the editor canvas can be iframed if there are no blocks registered with an API version lower than 3. In a future version of WordPress this will be standard no matter what blocks are registered. More info on this can be found here: https://make.wordpress.org/core/2023/07/18/miscellaneous-editor-changes-in-wordpress-6-3/#post-editor-iframed
This makes it harder as any selectors that try and get elements within the editor canvas don't work as you'd expect as they are now inside an iframe.
Describe the solution you'd like
There have been a few projects that implement the workaround documented in this Cypress blog post.
For example, on NewsPress' text only mode plugin:
const getIframeDocument = () => {
	return (
		cy
			.get( 'iframe[name="editor-canvas"]' )
			.its( '0.contentDocument' )
			.should( 'exist' )
	);
};

Cypress.Commands.add( 'getIframe', () => {
	return (
		getIframeDocument()
			.its( 'body' )
			.should( 'not.be.undefined' )
			.then( cy.wrap )
	);
} );

Cypress.Commands.add( 'getIntroParagraph', () => {
	return cy.getIframe().find( '.textOnly_content-intro' );
} );

It would be handy to have this as a command provided by the wp-cypress plugin to save having to add and maintain this across all projects that need it.
Describe alternatives you've considered
N/A
Additional context
N/A

+1, it would be great to abstract this as a built-in command as people will begin to hit this issue more and more.
My implementation of a solution was taken from the same blog post you linked to in your OP, which mentions that combining the its commands into a single command will ensure that the selector is re-tried should the page re-render during a test. Otherwise, you can encounter an issue where the iFrame reference is no longer valid:
/**
 * Custom command to select the block editor iframe
 *
 * @example cy.getBlockEditor().find('.editor-post-title__input').type('Edit 1');
 * @see https://www.cypress.io/blog/2020/02/12/working-with-iframes-in-cypress
 */
Cypress.Commands.add('getBlockEditor', () => {
  return cy
    .get('iframe[name="editor-canvas"]')
    .its('0.contentDocument.body')
    .should('not.be.empty')
    .then(cy.wrap);
});

There is also the cypress-iframe package which may be worth looking into, but even if we used that package I feel a custom command for WP Cypress would be beneficial to shorten:
cy.iframe('iframe[name="editor-canvas"]').find('.some-button').should('be.visible').click()

to:
cy.blockEditor().find('.some-button').should('be.visible').click()


[ASCII-1391][E2E] Avoid a panic in secrets e2e tests

What does this PR do?

Only check the check list content if it is not empty.
Motivation

Avoid a hard panic which immediately fails the test and prevents a later retry.
The assert package doesn't stop the test directly in case of failure but returns a boolean.
Additional Notes

Possible Drawbacks / Trade-offs

Describe how to test/QA your changes

/merge


S02-158. Update dockerfile on Dev

WELCOME TO DEVOPS-EASY-LEARNING PULL REQUEST SYSTEM
Description of the change
Update dockerfile on Dev
Checks and balances

[ ] Tests added and run
[ ] (External) Interfaces documented
[ ] Has security implications (describe below)

Type of change

[ ] Bug fix (non-breaking change that fixes an issue)
[x] New feature (non-breaking change that adds functionality)
[ ] Breaking change (fix or feature that would cause existing
functionality to not work as expected)

Related stories, issues and pulls
S02-158
Security considerations
None

hey Kelly please another env

@kmigap add volumes as well please @patdady verify


Update flake and don't build tests when building main application

bors merge

bors merge

bors merge

bors merge

bors merge


Reset of playwright dialog event listener before registration of new one

Motivation/Description of the PR

Playwright Alert popup handling code register multiple event listeners when multiple scenarios executed in one browser which causes error situation and error logging to console when an alert appear on the page.
Resolves #2945.

Applicable helpers:

[ ] WebDriver
[ ] Puppeteer
[ ] Nightmare
[ ] REST
[ ] FileHelper
[ ] Appium
[ ] Protractor
[ ] TestCafe
[X] Playwright

Applicable plugins:

[ ] allure
[ ] autoDelay
[ ] autoLogin
[ ] customLocator
[ ] pauseOnFail
[ ] puppeteerCoverage
[ ] retryFailedStep
[ ] screenshotOnFail
[ ] selenoid
[ ] stepByStepReport
[ ] wdio

Type of change

[ ] :fire: Breaking changes
[ ] :rocket: New functionality
[X] :bug: Bug fix
[ ] :clipboard: Documentation changes/updates
[ ] :hotsprings: Hot fix
[ ] :hammer: Markdown files fix - not related to source code
[ ] :nail_care: Polish code

Checklist:

[ ] Tests have been added
[ ] Documentation has been added (Run npm run docs)
[ ] Lint checking (Run npm run lint)
[X] Local tests are passed (Run npm test)

Thanks!


Data Science with Python : Item Based Filtering System

Welcome to 'DSWP' Team, good to see you here.
This issue will helps readers in acquiring all the knowledge to learn about getting involved with Item Based Filtering System. Tutorial to Item Based Filtering System and how it's applied using sample code.
To get assigned to this issue, add your serial numbers mentioned in the spreadsheet of "Data Science with Python", the approach one would follow and choice you prefer (Documentation, Audio, Video). You can go with all three or any number of options you're interested to work on.
This issue is created for beginners and who aren't assigned to any issues yet.
If you had referred any resources, add them up in "DS Resources". Similarly if you had used datasets, include them in "DS Datasets".
Domain : Machine Learning
Mentors Assigned : Dhruv Bajaj / Peehu Saxena
Points to Note :
The issues will be assigned on a first come first serve basis, 1 Issue == 1 PR.
"Issue Title" and "PR Title should be the same. Include issue number along with it.
Changes should be made inside the Datascience_With_Python/ directory & Datascience_With_Python branch.
Follow Contributing Guidelines & Code of Conduct before start Contributing.
This issue is only for 'GWOC' contributors of 'DSWP' domain.
All the best. Enjoy your open source journey ahead. 😎

@prathimacode-hub
I would like to contribute to this issue.
Name: Akshat Runwal
Serial number: 443
Type: Documentation
Approach: The document will include the concept of Item Based Filtering in Data Science and then the explanation of the same with the help of an example(i.e. Recommendation System)

Issue assigned to @runwalakshat5 for documentation

@prathimacode-hub , Can you please assign me this issue for video contribution mam.

Issue assigned to @ashish-reddy-20-08 for audio contribution


Validate input

Invalid payment requests seem to cause an exception (at least during development, not sure if that is the same in the production build).

Fixed with some recent PR.


Deprecate primaryColorBrightness

In line with https://flutter.dev/go/material-theme-system-updates and https://github.com/flutter/flutter/issues/91772, deprecates primaryColorBrightness.
List which issues are fixed by this PR. You must list at least one issue.
If you had to change anything in the flutter/tests repo, include a link to the migration guide as per the breaking change policy.
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 ///).
[ ] I added new tests to check the change I am making, or this PR is test-exempt.
[ ] All existing and new tests are passing.

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

@HansMuller Can you please review before I write a breaking change doc?


Fails to work with NextJs

Followed Demo experience, tried to run with NextJS default configuration
Getting error You may need an appropriate loader to handle this file type, currently no loaders are configured to process this file. See https://webpack.js.org/concepts#loaders

As far I see

Ecctrl is not transpiled

Any suggestions, would be highly appreciated

UPD: when i'm trying to do it via 'transpilePackages' in next.config.js in complains about three.js

I had the same issue, I resolved it by adding ecctrl to the transpilePackages in next.config.js.
Such as const nextConfig = {     transpilePackages: ['three', 'ecctrl'], }

tried, three complains then.
what version of Next you are using?

There.
"ecctrl": "^1.0.25", "next": "13.4.19", "three": "^0.156.1", "react": "18.2.0", "react-dom": "18.2.0",
What is three complaining about ?

@ecaii  ah ok we are still on 13.2.4, This is issue is within nextjstranspilePackage they have fixed it on 13.4v.
I think easier @ErdongChen-Andrew  is just to ship dist folder fully compiled as npm module

Thanks @Magvin , I ended up generating different versions of the main file and publishing the dist folder. This resolved all the errors, and now everything works smoothly 😁.

Legend thanks


Prepare for release 3.6.0

Description

Added a 3.6.0 version section to CHANGES.rst.
Related Issue

Motivation and Context

How Has This Been Tested?

Screenshots (if appropriate):
Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[x] I have run the pre-commit run command to format and lint.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[x] I have read the CONTRIBUTING document.
[ ] I have added tests to cover my changes.
[x] I have added my name and/or github handle to AUTHORS.rst
[x] I have added my change to CHANGES.rst
[x] All new and existing tests passed.

@treyhunner @rossmechanic Could one of you publish 3.6.0 to PyPI? 🙂

@treyhunner @rossmechanic Could one of you publish 3.6.0 to PyPI? 🙂

Published 👍


Bugg in BusServiceProvider.php

Laravel Version: 8
PHP Version: 7.3
Database Driver & Version: I don't know.

Description:
Bug found in: /vendor/laravel/framework/src/Illuminate/Bus/BusServiceProvider.php
The code below has an bug:
    protected function registerBatchServices()
    {
        $this->app->singleton(BatchRepository::class, DatabaseBatchRepository::class);

        $this->app->singleton(DatabaseBatchRepository::class, function ($app) {
            return new DatabaseBatchRepository(
                $app->make(BatchFactory::class),
                $app->make('db')->connection(config('queue.batching.database')),
                config('queue.batching.table', 'job_batches'),
            );
        });
    }

Steps To Reproduce:
Remove the , that is on the line: config('queue.batching.table', 'job_batches') or else /password/email will not work.

That is valid PHP syntax as of PHP 7.3. It is likely your CLI PHP is not PHP 7.3 or higher.

See the screen for PHP version I'm using


Implement Decodable for Vec

Bitcoin Core REST endpoint /rest/headers/{count}/{hash}.{bin/hex} requires deserializing a Vec<BlockHeader>.

Which RPC are you using? Usually, the Encodable, and Decodable attributes are implemented for types that are propagated via p2p network.

This one https://github.com/bitcoin/bitcoin/blob/master/doc/REST-interface.md#blockheaders. Decode also works for the other REST endpoints for getting binary or hex data of a BlockHash, Block, Tx, Vec, etc. But Vec is missing.

Actually, BlockFilter is also not Decodable, but you can use Vec<u8> for https://github.com/bitcoin/bitcoin/blob/master/doc/REST-interface.md#blockfilters.

On closer inspection, it actually needs to be Box<[BlockHeader]> as it doesn't start with a VarInt. Updated to just add Decodable. Otherwise there's no way to implement this trait from external packages if you want to deserialize the binary output from that REST endpoint.

I see what you mean, this response is from a specific implementation, and not part of bitcoin consensus. So it doesn't belong in consensus::encode.


Static files are exponsing my src

Static files are exponsing my src
Hi there, I'm facing a problem on deployment with creat-react-app: my src files are expose, how can I remove them from statics without eject?
My source tree

This is odd. Can you share what you your CRA tree and the scripts in your package.json?

This looks like the development environment, which is expected. This shouldn't happen in production.

This happens here too, but if I remove the map files, everything's fine. I know that source maps are helpful for debuggers and browsers, but are they really necessary in production? If the answer is yes, how could we also keep them but avoid having the files exposed?
Using node 9.3.0 and express with nodemon.

You can add a postbuild script in package.json that moves the files to a different directory or deletes them.

@gaearon  I believe the aim here is not to prevent  reverse engineering but rather make it harder to do so. Any code can be reverse engineered the question is how easy you want it to be.


Re-exporting default from a dependency in JavaScript produces an invalid declaration file

Bug Report
🔎 Search Terms
Re-export, default, declaration, module.exports, Identifier expected ts(1003), js, JavaScript.
🕗 Version & Regression Information

This is the behavior since version 4.7. Prior versions have compiler errors due to not supporting Node16/NodeNext modules.

⏯ Playground Link
Workbench Repro
Switch to the “Debug” tab and look at /index.d.ts. @showEmittedFile did not work.
💻 Code
// @checkJs
// @module: Node16
// @moduleResolution: Node16
// @declaration
// @emitDeclarationOnly

// @filename: index.js
export {default as mod} from 'a';
// @filename: package.json
{"private": true, "type": "module"}
// @filename: node_modules/a/package.json
{"name": "a", "version": "0.0.0"}
// @filename: node_modules/a/index.js
exports.a = 123;
// @filename: node_modules/a/index.d.ts
export const a = 123;

Other cases like this:

package.json: "type": "commonjs"
(Keep ESM-like module syntax despite being in CJS. No compiler errors due to moduleDetection: auto.)
index.js: export {default} from 'a';
(Declaration file will have as default)
Indirect re-export
(Only the declaration file that re-exports a dependency will be invalid.)
node_modules/a/index.d.ts may be provided separately (e.g., by a @types/ package).
Any other @module
Any other @moduleResolution except classic
(Does not resolve node_modules/a.)

Edit: Only some combinations of module+moduleResolution seems to cause this. I will try to test it and update tomorrow.
🙁 Actual behavior
Generated index.d.ts is invalid because, instead of re-exporting default as mod, it re-exports "/path/to/node_modules/a/index" as mod. Firstly, there is no export on a/index.js with that identifier. Secondly, TypeScript does not even support “arbitrary module namespace identifier names” (#40594), so the resulting declaration file cannot be subsequently parsed anyway. (Errs Identifier expected. ts(1003).)
🙂 Expected behavior
Generated index.d.ts should simply re-export default as mod, just like the JavaScript file.

Update on which @module, @moduleResolution, and "type" cause the bug. Columns represent @moduleResolution and rows represent @module.
Symbols:

Empty: Compiler error,
Dash: Impossible case,
‘B’: Bug.

CJS from CJS
node
node16
nodenext
bundler

AMD

-

CommonJS

-

ES2015

B

ES2020

B

ES2022

B

ESNext

B

Node16
B
B
B
-

NodeNext
B
B
B
-

System
B
B
B
-

UMD

-

CJS from ESM
node
node16
nodenext
bundler

AMD

B
B
-

CommonJS

B
B
-

ES2015

B
B
B

ES2020

B
B
B

ES2022

B
B
B

ESNext

B
B
B

Node16
B
B
B
-

NodeNext
B
B
B
-

System
B
B
B
-

UMD

B
B
-

ESM from CJS
node
node16
nodenext
bundler

AMD

-
-
-

CommonJS

-
-
-

ES2015

-
-
B

ES2020

-
-
B

ES2022

-
-
B

ESNext

-
-
B

Node16
B
-
-
-

NodeNext
B
-
-
-

System
B
-
-
-

UMD

-
-
-

ESM from ESM
node
node16
nodenext
bundler

AMD

-

CommonJS

-

ES2015

B

ES2020

B

ES2022

B

ESNext

B

Node16
B

-

NodeNext
B

-

System
B

-

UMD

-


SoapyLMS7 initialization takes around 1.5 seconds

The SoapyLMS7 constructor takes about 1.2 seconds to open and initialize the LimeSDR, most of which is on initializing default values of both channels in both directions (0.760 seconds).
The first call to setSampleRate() to set the application's sample rate then takes about 0.200 seconds: 0.160 seconds on rfic->SetInterfaceFrequency() and 0.040 seconds on _conn->UpdateExternalDataRate(). This totals to about 1.5 seconds to go from
zero to samples on one channel. I haven't timed the internal functions yet, so I'm not sure where the bottlenecks are, but this does seem a little bit long and strange. Speeding this up, if possible, would benefit prototyping uses that have frequent restarts.

Update: Fixed with the latest software and fw/gw images using faster FPGA SPI implementation. :-)

Awesome, thanks.


[BUG] import error with python 3.8 and numpy < 1.20

Describe the bug
when installing k-means-constrained in python3.8 environment with numpy < 1.20 (i.e. 1.19.5) on linux, import fails due to numpy binary incompatibility.
Details:
I think the root cause is that during package installation (PEP-517) the numpy that is used to compile the package is  numpy>1.13 (as described in pyproject.toml). on python 3.8 this will collect 1.20 (in python 3.6 it will collect 1.19.5).
so the package is built with numpy1.20 but if my environment has a different numpy (1.19.5) there is some binary incompatibility issue.
maybe its better to remove numpy from pyproject.toml and let it use the installed numpy instead?
Minimum working example
Python 3.8.9 (default, Apr  3 2021, 01:00:00)
[GCC 7.5.0] on linux
Type "help", "copyright", "credits" or "license" for more information.
>>> from k_means_constrained import KMeansConstrained
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/home/e101364/views/1v_main/Qstreams/provision-algobb/assaf/lib/python3.8/site-packages/k_means_constrained/__init__.py", line 4, in <module>
    from .k_means_constrained_ import KMeansConstrained
  File "/home/e101364/views/1v_main/Qstreams/provision-algobb/assaf/lib/python3.8/site-packages/k_means_constrained/k_means_constrained_.py", line 18, in <module>
    from .sklearn_import.metrics.pairwise import euclidean_distances
  File "/home/e101364/views/1v_main/Qstreams/provision-algobb/assaf/lib/python3.8/site-packages/k_means_constrained/sklearn_import/metrics/pairwise.py", line 10, in <module>
    from k_means_constrained.sklearn_import.metrics.pairwise_fast import _sparse_manhattan
  File "k_means_constrained/sklearn_import/metrics/pairwise_fast.pyx", line 1, in init k_means_constrained.sklearn_import.metrics.pairwise_fast
ValueError: numpy.ndarray size changed, may indicate binary incompatibility. Expected 88 from C header, got 80 from PyObject

Versions:

Python: 3.8.9
Operating system: linux (ubuntu18.04)
k-means-constrained: 0.5.2
numpy: 1.19.5
scipy: 1.6.3
ortools: 9.0.9048
joblib: 1.0.1
cython (if installed):

Thanks

I had the same issue on Python 3.8.5, requirements.txt needs to be updated.

Hi both,
Thanks for the bug report. Sorry it's taken me so long to get to it. I have updated the dependencies and corresponding tests. The latest version v0.6.0 should work: https://pypi.org/project/k-means-constrained/0.6.0/
Please let me know if you have any issues 😃
Josh


Peering Request

Full Name
AS400442
ASN
400442
PeeringDB Integration

[X] My PeeringDB Entries are current
[ ] Peer at ALL EXCHANGES (and skip the Peer Address(es) field below)

Peer Address(es)
<IP_ADDRESS>
2602:FBC7:4::1
Multihop

[X] Multihop Peering Request

Thank you for your interest in RouteViews.
We'd be happy to set up a multi-hop peering with you, using our route-views7 collector. Details:
IPv4: <IP_ADDRESS>
IPv6: 2001:468:d01:33::113
ASN:  6447
FQDN: route-views7.routeviews.org

Please let us know when you have set up the configuration and we will set up our end.
Thanks!

Sorry for the delay; routes configured.
-------- Original Message --------
Subject: Re: [routeviews/issues] Peering Request (Issue #171)
Date: Friday, October 18, 2024 18:21 CDT
From: Philip Smith @.>
Reply-To: routeviews/issues @.>
To: routeviews/issues @.>
CC: as400442 @.>, Author @.***>
References:
Thank you for your interest in RouteViews.
We'd be happy to set up a multi-hop peering with you, using our route-views7 collector. Details:IPv4: <IP_ADDRESS>
IPv6: 2001:468:d01:33::113
ASN:  6447
FQDN: route-views7.routeviews.org
Please let us know when you have set up the configuration and we will set up our end.
Thanks!
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you authored the thread.Message ID: @.> [ { @.": "http://schema.org", @.": "EmailMessage", "potentialAction": { @.": "ViewAction", "target": "https://github.com/routeviews/issues/issues/171#issuecomment-2423361454", "url": "https://github.com/routeviews/issues/issues/171#issuecomment-2423361454", "name": "View Issue" }, "description": "View this Issue on GitHub", "publisher": { @.": "Organization", "name": "GitHub", "url": "https://github.com" } } ]>

Hi
Our side is done and sessions are up - thanks for the support!
<IP_ADDRESS>    4     400442    953202         5 170362762    0    0 00:02:12       954993        0 Vulpine Networks
2602:fbc7:4::1          4     400442     90460         5 98074093    0    0 00:02:12       203401        0 Vulpine Networks
route-views7.routeviews.org#


[BEAM-6170] Change Nexmark stuckness warnings to not fail pipeline

The monitoring in the NexmarkLauncher watches the pipeline results and source for events.  If no events are consumed from the source or published to the output for over
STUCK_WARNING_DELAY it prints an warning message:
"WARNING: streaming query appears to have been stuck for %d min.",
However it also adds this to the errors list, which causes the pipeline to terminate with an exception.  There is a separate STUCK_TERMINATE_DELAY which is used to terminate the pipeline.  It seems inconsistent to cause the pipeline to fail with the warning timeout.  It is also not entirely accurate to say the pipeline is stuck since it may be busy processing stages other than the input or output.
R: @echauchot

@kennknowles @lgajowy @scwhittle I'm a bit busy on the spark runner. I plan to take a look before the end of the week.

starting the review

retest this please

Thanks Etienne and Lukasz, I made that change as you suggested. I also cleaned up some comments and made logging a little more consistent. Let me know what you think

Run Java PreCommit

Run Java PreCommit

Failure is unrelated (beam-sdks-java-io-hadoop-format), merging


Scrubbable samples list (Flashlight) WIP

A WIP to make scrolling (and scrubbing) through images a wonderful experience.

Closing. Will open a new PR for the branch flashlight when the feature is close to completion.


AWS Application Balancer with Nextcloud

Hi all,
I'm currently using NPM for my services on AWS with ALB.
Nextcloud and other web services are working fine.
I also configured the Nextcloud config.php with 'overwriteprotocol' => 'https' .
However, I have to enter like this: https://nextcloud.mydomain.com to get the the Nextcloud site.
If I enter just: nextcloud.mydomain.com, it would not redirect to Nextcloud.
It does not auto redirect from HTTP to HTTPS.
I'm using only HTTP as my listener for the ALB.
environment setup:
Docker:
NPM: 80:80; 443:443
http://<IP_ADDRESS>:81
Nextcloud (Linuxserver.io): 4443:443; 8080:80
https://<IP_ADDRESS>:4443
Am I missing something ?

Hi,
I don't know about your proxy configuration, but when you configure a proxy redirection host, you need to select the option 'force ssl redirection'. I case you redirected to your nextcloud using a custom location, there's currently a bug that can be worked around as described here

I found the issue.
I was using the Load Balancer and one of the Security Group didn't allow direct access to port 80.
So by allowed it direct access both to ports 80, and 443, it worked.


CRYP-25: remove an ethereum wallet from an account

Can you please also attach screenshots of the screens.


[optimization] use omitEmpty for all structs in service/interfaces.go

https://github.com/indeedeng/iwf/pull/370


[FEATURE] track time for every single page, instead for whole site

and record top 10 pages every day for history, which always means valuable contents.

Hi, @huxh, thanks for your submission~
Maybe it is not entirely necessary to track any pages by default, I think.
There is little memory  allowed for each extension to store data, while most of the data stored by single page is useless.
But this extension supports you manually defining one VIRTUAL site with ANT patterns to track any pages you wanna.
Just like the screenshot below

And then, URLs hitting this pattern will be tracked into this VIRTUAL site, and you can view its statistics like other normal sites.


We would really like to invite you for a technology live about this paper~

Hey~We are the team of '自动驾驶之心'.
Currently, we have accumulated a large number of fans in the fields of autonomous driving perception, positioning, planning and prediction, deployment and acceleration, hardware design, etc. They are distributed in over 300 autonomous driving companies and well-known laboratories both domestically and internationally. We have held over 100 open-source sharing sessions in the past two years. We sincerely hope to invite your team to join in a live broadcast. Look forward to your reply!
WeChat: AIDriver002!

Thanks for your attention! I have add your WeChat for further connection.


Terminal widths not correctly restored after reload

Testing #117265
Version: 1.54.0-insider
Commit: bd74308b5abbbb2f5a5df0f0d8a3bef606652d36
Date: 2021-02-23T05:17:41.287Z
OS: Linux x64 5.8.0-43-generic

open a terminal and split it
change the size to make one smaller than the other
reload window with the command from the command prompt. Terminals have the same width again

I hit this as well last week, you reloaded too quickly after changing the size as saving the state is debounced. Closing as designed

I've waited over 10s, multiple times and I never seem to get the split size restored.

@eamodio I can't reproduce that. Can you pls attach a gif?

Can now repro this thanks


Deaths and intensive care patients by counties and age

If its possible please show  deaths and intensive care patients by counties and age.
Also total number how many patients have been in intensive care (includes those who have already recovered or died).

Hold in mind explanation about "distance and speed" #87.

PS. Total intensive care recovered patients are cases that doctors need to analyze deeply.

We don't have this data available, but if it becomes available we will release it / make a visualization.

As this is related to #186, I am merging this issue into that one.


Allow wildcards for read-only token whitelist

Summary
Based off the feedback discussion I created when first testing out read-only tokens https://github.com/tinacms/tinacms/discussions/2671
My workflow for my company is to have people branch off main, create their document, and merge back to main. In fact we have a rule in our repo where you cannot merge directly to main.
Every branch will be named based on the author's decision, I cannot predetermine the set of branches that will be created.  The whitelist must be dynamic

This has been added! https://tina.io/docs/graphql/read-only-tokens/#wild-card-matching

amazing, thanks!


Functional components usage

I used the following steps below to implement my Calculator app

First I created a new react project using the commands;
-  npx create-react-app math-magic,
- Set up eslint and style lint.
Created a component directory that contained a calcultor.js file as a parent component and also a logic directory, that contained the calculate.js and operate.js files respectively, which I imported in the calculator.js file.
The Calculator component in calculator.js is defined as a function using the function keyword. This is the syntax for defining a functional component in React.
Created 1 child component (keys-function) where I made the UX design for the calculator function keys, then I  imported it in the calculator.js file that I finally used in the app.js.
I used the useState hook which I imported into the calculator.js, to initiate the state of the app.
I created an onClickhandler method which I called in the buttons to implement the functions.

STATUS: APPROVED :tada: :clap: :champagne:
Hi @Rudy-Menekam ,
You are amazing :smiley: :clap: !  There is nothing else to say other than... it's time to merge it :shipit:
Congratulations! 🎉

Highlights :clap: :tophat:

Good use of React hooks :clap:

Optional suggestions :small_airplane:
Every comment with the [OPTIONAL] prefix won't stop the approval of this PR. However, I strongly recommend you to take them into account as they can make your code better. Some of them were simply missed by the previous reviewer and addressing them will really improve your application.
Cheers and Happy coding!👏👏👏
Feel free to leave any questions or comments in the PR thread if something is not 100% clear.
Please, remember to tag me in your question so I can receive the notification.

As described in the Code reviews limits policy  you have a limited number of reviews per project (check the exact number in your Dashboard). If you think that the code review was not fair, you can request a second opinion using this form.


Fix crash when publisher tries to modify its internal state after being stopped

We've seen a fatal exception occur when using the publisher SDK. It looks like the publisher is stopped but then it itself tries to access its own state which leads to the PublisherPropertiesDisposedException being thrown. This should not happen as after stopping the publisher should perform no operations on its own.
Fatal Exception: w3.i0: Cannot access the publisher properties after it's disposed.
       at com.ably.tracking.publisher.DefaultCorePublisher$Properties.getTrackableStateFlows(CorePublisher.kt:700)
       at com.ably.tracking.publisher.DefaultCorePublisher$updateTrackableState$1.invokeSuspend(CorePublisher.kt:475)
       at kotlin.coroutines.jvm.internal.BaseContinuationImpl.resumeWith(ContinuationImpl.kt:33)
       at kotlinx.coroutines.DispatchedTask.run(DispatchedTask.kt:106)
       at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1167)
       at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:641)
       at java.lang.Thread.run(Thread.java:923)

Just a question on this, Is there  a mechanism that prevents use of disposed properties? If not I suppose that might be a race condition issue where trackableStateflows still emits updates after it was stopped. I will investigate the potential of this happening

The mechanism you mention is the error throwing that happens 😉 When you try to access disposed properties they will throw the PublisherPropertiesDisposedException


Creating a database for tubestations

Creating a  database of tube stations including the following data:

Name
Location
Lines
Changes

Database is done, I just need to add to project and write code to copy user's old data to the new database file.

Done in b68a0d960df23d5f26ba5c6f4097157d14bae413


[WIP]: Add TessTargetPixelFile class

This PR allows one to do:
from lightkurve import TessTargetPixelFile
filename = ("https://archive.stsci.edu/missions/tess/ete-6/tid/00/000/"
            "004/104/tess2019128220341-0000000410458113-0016-s_tp.fits")
tpf = TessTargetPixelFile(filename)

Codecov Report

Merging #27 into master will decrease coverage by 0.87%.
The diff coverage is 67.61%.

@@            Coverage Diff             @@
##           master      #27      +/-   ##
==========================================
- Coverage   90.98%   90.11%   -0.88%
==========================================
  Files          14       14
  Lines        1320     1325       +5
==========================================
- Hits         1201     1194       -7
- Misses        119      131      +12

Impacted Files
Coverage Δ

lightkurve/tests/test_targetpixelfile.py
100% <ø> (ø)
:arrow_up:

lightkurve/targetpixelfile.py
75.9% <66.32%> (-5.51%)
:arrow_down:

lightkurve/lightcurve.py
87.72% <85.71%> (-0.21%)
: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 918014e...b59be0e. Read the comment docs.

@mirca Do you mind if we get rid of the tpf.n_good_cadences property?  I think it has lost its purpose now that we automatically apply the quality mask on everything, one can simply do e.g. len(tpf.flux) or tpf.time.shape and get the same answer?
(Now that we're close to the 1.0 release, now is a good time to remove features we don't need or want to support.)

@barentsen agreed, go for it! =D


[SPARK-20716][SS] StateStore.abort() should not throw exceptions

What changes were proposed in this pull request?
StateStore.abort() should do a best effort attempt to clean up temporary resources. It should not throw errors, especially because its called in a TaskCompletionListener, because this error could hide previous real errors in the task.
How was this patch tested?
No unit test.

Test build #76844 has started for PR 17958 at commit e10101e.

@zsxwing @marmbrus

Test build #76845 has started for PR 17958 at commit 1e36134.

Test build #76845 has finished for PR 17958 at commit 1e36134.

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

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/SparkPullRequestBuilder/76845/
Test PASSed.

Test build #76844 has finished for PR 17958 at commit e10101e.

This patch fails Spark unit tests.
This patch does not merge cleanly.
This patch adds no public classes.

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

Build finished. Test FAILed.

Test build #3711 has started for PR 17958 at commit 1e36134.

Test build #3711 has finished for PR 17958 at commit 1e36134.

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

Test build #76880 has started for PR 17958 at commit 820e4d5.

Test build #76880 has finished for PR 17958 at commit 820e4d5.

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

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/SparkPullRequestBuilder/76880/
Test PASSed.

LGTM. Merging to master and 2.2. Thanks!


Brownouts / Power usage increase on new version of library

Good news - no more green boogies.
Bad news - the power consumption seems to have  increased.
Before I could run plasma test on USB for days without issue, now it lasts about 1/2 second before rebooting without external power.
New library draw -
10% coverage (Conway game of life)
5v/ .46A - Brightness 50
5v/ .32A - Brightness 20
5v/ .27A - Brightness 10
5v/ .25A - Brightness 5
100% coverage (plasma)
5v/ .36A - Brightness 5
5v/ .48A - Brightness 10
5v/  .59A - Brightness 15
5v/ .70A - Brightness 20
Over that = brownout/reset.
Win some, lose some!
Will add in a cap to see if that improves USB power brownout, otherwise will need to add in some code to try modulate power if its going to go over hard limits.

I'll look at this again in the coming days. That is too much of a
regression.
I like my RGB panels, but prefer if they're more energy efficient.
On Tue., 30 Jul. 2019, 5:02 am csloz<EMAIL_ADDRESS>wrote:

Good news - no more green boogies.
Bad news - the power consumption seems to have increased.
Before I could run plasma test on USB for days without issue, now it lasts
about 1/2 second before rebooting without external power.
New library draw -
10% coverage (Conway game of life)
5v/ .46A - Brightness 50
5v/ .32A - Brightness 20
5v/ .27A - Brightness 10
5v/ .25A - Brightness 5
100% coverage (plasma)
5v/ .36A - Brightness 5
5v/ .48A - Brightness 10
5v/ .59A - Brightness 15
5v/ .70A - Brightness 20
Over that = brownout/reset.
Win some, lose some!
Will add in a cap to see if that improves USB power brownout, otherwise
will need to add in some code to try modulate power if its going to go over
hard limits.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/mrfaptastic/ESP32-RGB64x32MatrixPanel-I2S-DMA/issues/17?email_source=notifications&email_token=AC3TMKIW6TX55AMLNX46IT3QB64LTA5CNFSM4IHYFNAKYY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4HCGE5QA,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AC3TMKN336OQSBRCY5G6PUTQB64LTANCNFSM4IHYFNAA
.

Resolved with better library defaults.


⚠ Fortnite has degraded performance

In fad1bf7, Fortnite (https://fortnite.com) experienced degraded performance:

HTTP code: 200
Response time: 1393 ms

Resolved: Fortnite performance has improved in 723b7fe.


Adding FsPermission serialize / de-serialize methods

@ibuenros and @pcadabam could you both take a look?

Addressed comments.

LGTM


All animals

Assignee Tasks

[x ] added unit tests (or none needed)
[x ] written relevant docs (or none needed)
[x ] referenced any relevant issues (or none exist)

Guidelines
Please add a description of this Pull Request's motivation, scope, outstanding issues or potential alternatives, reasoning behind the current solution, and any other relevant information for posterity.

Created All Animals vertical

created API route for /api/animals
seeded db with two animals
created AllAnimals React Component
connected AllAnimals to Redux Store

Tests:

route test
component test
redux store test

you need exact. CLOSED!


Promise giving "Cannot read property 'base' of undefined"

Use node 8.9.1
npm init
npm i -S twitter

Create an index.js file with:
const Twitter = require('twitter');
const twitter = new Twitter({
    consumer_key: 'x',
    consumer_secret: 'x',
    access_token_key: 'x',
    access_token_secret: 'x',
});

twitter.get('favorites/list').then((tweet) => { console.log(tweet); });

Run the script
node index.js

Error:
/Users/nelsyeung/Sites/test/node_modules/twitter/lib/twitter.js:119
  if (typeof params.base !== 'undefined') {
                    ^

TypeError: Cannot read property 'base' of undefined
    at Twitter.__request (/Users/nelsyeung/Sites/test/node_modules/twitter/lib/twitter.js:119:21)
    at Twitter.get (/Users/nelsyeung/Sites/test/node_modules/twitter/lib/twitter.js:236:15)
    at Object.<anonymous> (/Users/nelsyeung/Sites/test/index.js:10:9)
    at Module._compile (module.js:635:30)
    at Object.Module._extensions..js (module.js:646:10)
    at Module.load (module.js:554:32)
    at tryModuleLoad (module.js:497:12)
    at Function.Module._load (module.js:489:3)
    at Function.Module.runMain (module.js:676:10)
    at startup (bootstrap_node.js:187:16)

This error does not appear when running with a callback.

Same issue here

Seems like a pretty simple bug...I made a PR https://github.com/desmondmorris/node-twitter/pull/270

@HarryEMartland  thanks .you just saved me .it did solve the issue for me .

@HarryEMartland can you explain why your solution worked?

@neer17 If you look at the documentation on requests, you'll see that it expects a params parameter for the function call: client.get(path, params, callback);. If we omit this params parameter, it'll fail on the following block of code within node-twitter, because params doesn't exists thus cannot access params.base:
if (typeof params.base !== 'undefined') {
  base = params.base;
  delete params.base;
}

Calling client.get with an empty config object {}, e.g., client.get(path, {}, callback); means that we are getting around the problem that params doesn't exist by setting it to {}. I hope this is clear enough.


avoid forced/implicit conversion

avoid forced/implicit conversion inside the WebsocketHandler::send

Duplicate of #59.
Sorry that I didn't get to review that other PR earlier, but thanks for pointing out that issue!


Letters missing

Hi @ankush-me ,
why is some words missing in the corner of bounding box ?
This "k" and "t", can it be rotated, so that the letters looks in each other?
I use my own words, length is 15 because I want to generate sinus shape text (wavy text).

@ankush-me  is it possible to do something that bounding boxes don't overlap themselves??


JSX Support

Reference:
https://github.com/sindresorhus/ava/issues/274

Now I'm working with it.
refs: [WIP][experimental] Big refactoring: Embed assertion's AST and tokens to make runtime side parser unnecessary

@twada I saw that the PR you mentionned has been merged. What is the progress on this issue then?

@MoOx
If you are using AVA, you can use JSX tags once this PR  has been released.
If you are using power-assert directly, you can now use JSX tags through babel-plugin-espower with option embedAst: true.

To be documented though.

Close this issue since this feature is documented, and power-assert also suggests configuration option on parse error.


Intro message in Spawn has wrong server in title.

Says "Orion" when playing on Titan.

Fixed from the next reboot, thanks for the report! :)


Fix E2E timeout since there are too many programs on cloud

Summary
https://github.com/Qiskit/qiskit-ibm-runtime/runs/7316951924?check_suite_focus=true#step:5:50
Details and comments
Fixes #

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.3%) to 62.674%

Totals

Change from base Build<PHONE_NUMBER>:
0.3%

Covered Lines:
2883

Relevant Lines:
4600

💛  - Coveralls


Solution for failed pip dependency in ubuntun 9.5 pycrypto

Empire Version
2.5
OS Information (Linux flavor, Python version)
ubuntu 9.5
Expected behavior and description of the error, including any actions taken immediately prior to the error. The more detail the better.
Empire launch failed because of
Screenshot of error, embedded text output, or Pastebin link to the error
Failed building wheel for pycrypto
Command "/usr/bin/python -u -c "import setuptools, tokenize;file='/tmp/pip-install-ohqoTY/pycrypto/setup.py';f=getattr(tokenize, 'open', open)(file);code=f.read().replace('\r\n', '\n');f.close();exec(compile(code, file, 'exec'))" install --record /tmp/pip-record-wGkBoa/install-record.txt --single-version-externally-managed --compile" failed with error code 1 in /tmp/pip-install-ohqoTY/pycrypto/
Any additional information
apt-get install build-essential libssl-dev libffi-dev python-dev solves above issue

@madisettisunil how about to update Ubuntu? ;)

Yeah, that ubuntu is super old. Closing.


support sending photo in direct message

Codecov Report

Merging #902 into master will decrease coverage by 0.41%.
The diff coverage is 14.58%.

@@            Coverage Diff             @@
##           master     #902      +/-   ##
==========================================
- Coverage   46.25%   45.84%   -0.42%
==========================================
  Files          28       28
  Lines        2897     2936      +39
==========================================
+ Hits         1340     1346       +6
- Misses       1557     1590      +33

Impacted Files
Coverage Δ

instabot/api/api.py
53.51% <13.04%> (-0.79%)
:arrow_down:

instabot/bot/bot_direct.py
9.48% <13.04%> (+0.71%)
:arrow_up:

instabot/bot/bot.py
75.19% <50%> (-0.14%)
: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 5986198...a09a0b4. Read the comment docs.

Great pull request! Great work!
May I ask you to arrange direct-messages-related templates into a specific folder? Thanks.

Sure, do you mean example scripts?


[BUG] Maximum execution time exceeded

Hello,
I installed the plugin on my shared hosting, because openssl doens't work locally. The homepage works fine, but whenever I'm trying to visit another page on my website. It get's in an endless loop and after that I get the message from Bolt that my maximum execution time is exceeded.
Details:

Latest version of Translate
Installed via Bolt CMS
Bolt version 3.2.3
PHP version 5.6
Webserver: Apache

Translate config:
# The locales the site should be available in
locales:
    nl_NL:
        label: Nederlands
        slug: nl
    en_GB:
        label: Engels
        slug: en

routing_override: true
menu_override: true
url_generator_override: true
translate_slugs: true
use_accept_language_header: true

I think it has something todo with the routing, but I'm not an expert in Bolt..
Anybody had this problem before and can help me?
Thanks in advance.

Hmm, I've never seen it go looping off like that, Could you perhaps check the debug profiler so see if that gives us any more info?

Hi, thanks for answering! Where can I find that.
This is the url: https://test.inchplastics.nl/nl/pagina/over-ons. So you can check by yourself if necessary.

Hmm, it looks like it's hitting that execution time in monolog, but the only place we use the logger is in the menu, here: https://github.com/AnimalDesign/bolt-translate/search?utf8=✓&q=logger, so I'm guessing it's something to do with the menu. Could you post your menu.yml? Also, the debugger can be turned on and off in the main config.yml and is then shown on the bottom of every page.

Debug is turned on in the main config.yml, but for some reason it doens't show up when the error is thrown.
My menu.twig (the most important, the rest is just html):
```

            {% setcontent pages = 'pages/latest/6' orderby 'id'  %}

              {% for page in pages %}
                {% if loop.first %}{% endif %}
                {{ page.title|trimtext(12) }}
                {% if loop.last %}{% endif %}
              {% endfor %}

```
As I see now, my menu.yml is never changed (just default stuff), maybe that's the problem?
main:
    - label: Home
      title: This is the first menu item.
      path: homepage
      class: first
    - label: Second item
      path: entry/1
      submenu:
          - label: Sub 1
            path: entry/2
          - label: Sub 2
            class: menu-item-class
            path: entry/3
          - label: Sub 3
            path: entry/4
          - label: Sub 4
            class: sub-class
            path: page/3
    - label: All pages
      path: pages/
    - label: The Bolt site
      link: http://bolt.cm
      class: last

Is it possible to temporarily disable the logger?

Any solution? If you want, I can make a repo from my current site, if that helps.

@RickvanB Sorry, I haven't had the time to look deeper at this :(
I haven't been able to reproduce it in a fresh install, but I'll check it out more in the coming days. Sorry for the delay!

@SahAssar Thanks for you answer and for looking into it deeper! Is it a possibility to temporarily disable the logger? So that I can move on with the website.

@RickvanB We've been having one of those nightmare projects at work, so still no luck on this. Could you perhaps try the latest dev-master of translate? A bunch of small issues have been fixed in that, and this might have been effected.

@SahAssar I've updated Translate to 4.0.2, but still the same problem. When I set debug to true I get an empty response from Chrome. If debug is false I get only an error page, which is probably the error in my first post.

@RickvanB Yeah, sorry, I botched that release... :/ I tagged 4.0.3 which should fix that WSOD problem, could you try that?

@SahAssar I installed 4.0.4 yesterday and it seems like it's working again! 👍 . Only thing that I need to find out now what my slug for english pages is. Anyway, many thanks for your help!

Great, closing this then! :)


Import error when running Github Action workflow for data validation

Overview
In our repository the validation workflow is failing with an import error when running frictionless validate inquiry.json --json. See the error report.
Our frictionless.yaml workflow for Frictionless Repository is pretty much the default one: https://github.com/okfn/publicbodies/blob/master/.github/workflows/frictionless.yaml

Same issue observed yesterday when validating using the frictionless github action, the problem seems to be an import for the termui python library:
From: https://github.com/jmcastagnetto/covid-19-peru-vacunas/actions/runs/2055954381
[validate](https://github.com/jmcastagnetto/covid-19-peru-vacunas/runs/5730354742?check_suite_focus=true)
Cannot validate inquiry: Command failed: frictionless validate inquiry.json --json
Traceback (most recent call last):
  File "/usr/local/bin/frictionless", line 5, in <module>
    from frictionless.__main__ import program
  File "/usr/local/lib/python3.10/site-packages/frictionless/__init__.py", line 20, in <module>
    from .program import program
  File "/usr/local/lib/python3.10/site-packages/frictionless/program/__init__.py", line 1, in <module>
    from .api import program_api
  File "/usr/local/lib/python3.10/site-packages/frictionless/program/api.py", line 1, in <module>
    from typer import Option as Opt
  File "/usr/local/lib/python3.10/site-packages/typer/__init__.py", line 12, in <module>
    from click.termui import get_terminal_size as get_terminal_size
ImportError: cannot import name 'get_terminal_size' from 'click.termui' (/usr/local/lib/python3.10/site-packages/click/termui.py)

YAML at: https://github.com/jmcastagnetto/covid-19-peru-vacunas/blob/main/.github/frictionless.yaml
Using the same YAML definition and running locally on my laptop, the frictionless validation works.

I also encountered this problem today. It works if I pin click to version 8.0.4.
This issue seems to be related to the 8.1.0 release of click on the 28th of March (https://pypi.org/project/click/#history)

It seems to be related to the click 8.1.0 version as mentioned here also.

With the info given by @AT-SyT, I checked my local click, and I have 8.0.3, had not updated it yet (and for now, I won't)

I just released Typer 0.4.1 that should handle the Typer issue. :rocket: :nerd_face:

I can confirm that now the action is working again (did not downgrade click)
See: https://github.com/jmcastagnetto/covid-19-peru-vacunas/actions/runs/2066912541

I too have just re-run the latest job and it is now working.
Unless anyone raises any objections or concerns, I'm soon going to close this issue as solved.


regs3k: Add results count to search header

Adds Showing X matches out of XXX total results text per GHE#106:

Adjusts the tags that appear under search results per GHE#113:

Makes the search results page a little nicer on smaller screens by shortening Regulation X into Reg X:

Testing

Visit the regs3k search page and search for something.
Verify the above screenshots match your experience.

Checklist

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

BTW we'll soon have automated acceptance tests for all this UI stuff.


Logic Apps Standard built-in Service Provider SQL connector Dynamic Content output is not visible in new designer

Describe the Bug with repro steps
In a specific Standard Logic Apps workflow we do not see the built-in Service Provider SQL connector Dynamic Content output in the subsequent action. The same works fine with fallback URL([https://aka.ms/lafallback]. Below are the details:
Unexpected New designer where the previous Service Provider SQL connector Dynamic content output is not visible example:

Expected Old designer where the previous Service Provider SQL connector Dynamic content output is visible example:

The Logic Apps Resource has VNET Integration enabled with a route table
Further the SQL Server is also has private endpoint enabled.
Tried to replicate the same scenario but could not replicate it with simple Logic Apps Standard VNET Integration and SQL Private endpoint setup
In this specific issue Standard Logic Apps there is a route table routing traffic to Virtual Appliance. But the same setup works fine with old fallback designer.

What type of Logic App Is this happening in?
Standard (Portal)
Are you using new designer or old designer
New Designer
Did you refer to the TSG before filing this issue? https://aka.ms/lauxtsg
Yes
Workflow JSON
No response
Screenshots or Videos
No response
Browser
Chrome
Additional context
No response
AB#24467304

I also have this issue in Safari, Chrome, Edge.

Hi @Eric-B-Wu, the details are already shared with you internally, also sent to you over mail at<EMAIL_ADDRESS>

Hi @Eric-B-Wu , hope you are doing well. Do you have any updates further on the issue scenario?

talked offline with customer, seems like it was an intended change from old designer to new designer - that being said, if this causes confusion we are considering a "See All Tokens option to show type-filtered tokens" as seen in: https://github.com/Azure/LogicAppsUX/issues/1815

closing this issue as resolved after discussion with customer


Training data examples

Thanks for sharing the amazing project! Can you provide examples of training data to better understand the format of the training data

We are working on releasing our dataset soon! Thanks for your patience, when our dataset is ready, we will also prepare a sample file (including 8 instruction pairs with corresponding in-context samples) to facilitate researchers to quickly prototyping.

Thanks for sharing the amazing project! Can you provide examples of training data to better understand the format of the training data

we have release our paper and dataset, please check the new readme!


Add terminology page to explain the words we use

To explain

[x] Agent (Ruby & Elixir) & system agent (Rust)
[x] Alerting / Notifications
[x] API
[x] API key
[x] AppSignal vs Appsignal vs appsignal
[x] Application (App)
[x] Blog
[x] Changelog
[x] Configuration
[x] Events
[x] Environments
[x] Errors
[x] Instrumentation
[x] Instrumentation events
[x] Integration
[x] Markers
[x] Metadata
[x] Metric
[x] Organizations
[x] Owners
[x] Performance issue
[x] Push API
[x] Push API key
[x] Ruby Magic
[x] Sample
[x] Stroopwaffles
[x] Tags
[x] Teams
[x] Transaction
[x] User account

Let me know if I missed anything so far

@tombruijn good initiative! You might want to take some extra effort to explain the difference between the API key and the Push API key, because these both show up in the interface if I am correct and might be confused sometimes :)

Oh and maybe add "Ruby magic" as explanation?

The normal API key is already on the list :)
I'll add the ruby magic one as well.
Other suggestions are welcome!

Noticed that, just meant that maybe you can "cross-reference" them to make it absolutely clear that there are two and how they're different 👍

Will do!

Terminology updated with the last few missing topics. Please review, but also feel free to update the content yourselves. I've kept the content simple and high-level, not diving too deep in certain topics and referring to pages with more detailed explanations instead.

Terminology updated. Please re-review @jeffkreeftmeijer @matsimitsu


Update the game over message

Update the game over message so people know how to play again!

An essential change


Update displacement filter to support rotation of displacement map pixels

Description of change

Updated displacement filter to support rotation of displacement map pixel values. Allows displacement map pixels to displace in local-space instead of always in global space. For more info, please see https://github.com/pixijs/pixi.js/issues/5491
Pre-Merge Checklist

[ ] Tests and/or benchmarks are included
[ ] Documentation is changed or added
[X] Lint process passed (npm run lint)
[X] Tests passed (npm run test)

Could you also make a public API to make it easier to set rotation? Setting a 2x2 matrix via uniforms isn't that intuitive for most folks.

@ivanpopelyshev Hmm, it did look fine there, but I realized the playground I put together didn't have any displacement in the y axis, so you couldn't tell it was doing something weird. If you look at this playground and you change them to inverted, you can see spikey displacement at the top and bottom of the edges of the sprite: https://www.pixiplayground.com/#/edit/M~8Ld47o6vOWvFTxmvvOX
@bigtimebuddy Not sure I understand here. End users shouldn't have to do anything for this to work, it's automatic based on the rotation of the displacement sprite.

Oooh you’re right! Sorry I misinterpreted the code. Makes sense

Cool cool, no worries.


Trial use of Swirrl's AOT-compiles csv2rdf binary in docker

Swirrl provide a (beta) version of csv2rdf which is AOT using GraalVM (I think) which is designed to limit the startup-overheads which should reduce the amount of time we spend in behaviour tests.
I believe that @ajtucker did some work to bring this into a docker image. If not, that needs to be created as part of this task. N.B. do not publish it to gsscogs/csv2rdf:latest (give it a seperate tag at least so no one accidentally uses it).

[ ] Alter the csv2rdf behaviour test functionality in the devtools package and ensure that it uses the special docker container containing the AOT-compiled binary.
[ ] Do some benchmarking to show us the performance improvements in behaviour testing (if any) with files of various sizes (ideally).

https://github.com/GSS-Cogs/csvcubed/pull/464


Consider adding SetBytesLE and BytesLE

Motivation
Not every protocol will use big endian when serialising field elements. This means that when using gnark-crypto, one must reverse the bytes before and after de/serialisation.
Suggestion
Adding a SetBytesLE and BytesLE wrapper method which reverses the bytes would be useful. If breaking changes are allowed, then change SetBytes/Bytes to SetBytesBE/BytesBE method might make for a better/more explicit API
Observation
It seems that Big endian was possibly used as a default due to the usage of big.Int internally in the field element code

Yep, seems reasonable. We had to do that couple of times on forks / branches for compatibility purposes.
Lots of Golang based code indeed use Big endian 👍 .


Update dlp-conditions-and-exceptions.md

@chrfox please review

@aamalhot  - This can't be merged as is.  1) remove your name from the PNG and send me the png. Graphics have to be uploaded to a specific folder in GitHub, links out will not work, also https links will have difficulties with airgap scenarios.
Chris

@aamalhot please implement the suggested changes. Thanks


ARM 32 ELF export fails for unchanged program: Relocation at [...] exceeds file length

Describe the bug
ARM 32 ELF export fails for unchanged program with error Relocation at [...] exceeds file length
To Reproduce

Import attached program (ARM 32 ELF, is open source, part of DD-WRT).
Try to export the program: a window pops up with the error.

Expected behavior
The unchanged program should be successfully exported.
Attachments
libutils.so.tar.gz
Environment (please complete the following information):

OS: linux mint 19.3
Java Version: openjdk 11
Ghidra Version: 10.1-BETA
Ghidra Origin: official

Thanks we are taking a look.

There was a relocation that was only partially file-backed, so that case had to be handled.  The fix is working it's way through to you.

thanks so much for the immediate help!
no way to get my hands on a test build, is there?

No, we don't distribute test builds.  If you are able to build Ghidra on your own I can tell you what you need to change.

thanks! i'm downloading the tree and deps as i write.

i was able to build Ghidra master :)

I can tell you what you need to change

please!

In this zip file is a new AbstractLoaderExporter.java file that you can drop in where the old one was at
./Ghidra/Features/Base/src/main/java/ghidra/app/util/exporter/AbstractLoaderExporter.java

AbstractLoaderExporter.zip

thanks sooo much!!
for documentation purposes...
diff --git a/Ghidra/Features/Base/src/main/java/ghidra/app/util/exporter/AbstractLoaderExporter.java b/Ghidra/Features/Base/src/main/java/ghidra/app/util/exporter/AbstractLoaderExporter.java
index e2e51654f..901e9e8e8 100644
--- a/Ghidra/Features/Base/src/main/java/ghidra/app/util/exporter/AbstractLoaderExporter.java
+++ b/Ghidra/Features/Base/src/main/java/ghidra/app/util/exporter/AbstractLoaderExporter.java
@@ -25,9 +25,11 @@ import ghidra.app.util.opinion.Loader;
 import ghidra.framework.model.DomainObject;
 import ghidra.program.database.mem.AddressSourceInfo;
 import ghidra.program.database.mem.FileBytes;
+import ghidra.program.model.address.Address;
 import ghidra.program.model.address.AddressSetView;
 import ghidra.program.model.listing.Program;
 import ghidra.program.model.mem.Memory;
+import ghidra.program.model.mem.MemoryBlockSourceInfo;
 import ghidra.program.model.reloc.Relocation;
 import ghidra.util.Conv;
 import ghidra.util.HelpLocation;
@@ -88,24 +90,32 @@ public abstract class AbstractLoaderExporter extends Exporter {
                }

                // Undo relocations in the temp file
-               // NOTE: not all relocations are file-backed
+               // NOTE: not all relocations are file-backed, and some are only partially file-backed
                String error = null;
                try (RandomAccessFile fout = new RandomAccessFile(tempFile, "rw")) {
                        Iterable<Relocation> relocs = () -> program.getRelocationTable().getRelocations();
                        for (Relocation reloc : relocs) {
-                               AddressSourceInfo info = memory.getAddressSourceInfo(reloc.getAddress());
-                               if (info == null) {
+                               Address addr = reloc.getAddress();
+                               AddressSourceInfo addrSourceInfo = memory.getAddressSourceInfo(addr);
+                               if (addrSourceInfo == null) {
                                        continue;
                                }
-                               long offset = info.getFileOffset();
-                               byte[] bytes = reloc.getBytes();
+                               long offset = addrSourceInfo.getFileOffset();
                                if (offset >= 0) {
-                                       if (offset + bytes.length > fout.length()) {
+                                       MemoryBlockSourceInfo memSourceInfo = addrSourceInfo.getMemoryBlockSourceInfo();
+                                       byte[] bytes = reloc.getBytes();
+                                       int len;
+                                       for (len = bytes.length; len > 1; len--) {
+                                               if (memSourceInfo.contains(addr.add(len - 1))) {
+                                                       break;
+                                               }
+                                       }
+                                       if (offset + len > fout.length()) {
                                                error = "Relocation at " + reloc.getAddress() + " exceeds file length";
                                                break;
                                        }
                                        fout.seek(offset);
-                                       fout.write(bytes);
+                                       fout.write(bytes, 0, len);
                                }
                        }
                }

@ryanmkurtz just letting you know that the fix worked and you are absolutely awesome for sharing the early fix. i was able to pinpoint a longstanding issue affecting some routers thanks to your help. you rock!!

Great! I'm going to reopen this and let it get closed automatically by the real commit that will eventually make its way to GitHub.

Was actually closed by 67569a918f020020ebb61faaff00599b1ab46709.  Note the final solution was a little different than the above code.


Panic when receiving a reset on a stream in the process of closing

This is the cause of upstream issue runconduit/conduit#607.
It appears that when a stream is in the process of transitioning from Half-Closed (remote) to Closed, with a Trailers frame queued to end that stream, and a Reset (Canceled) frame is received, h2 will hit an expect in prioritize.rs and panic.

Here's the expect that fails:
https://github.com/carllerche/h2/blob/f8baeb7211985834acaf7ecc70548aafd5d9ef6d/src/proto/streams/prioritize.rs#L693-L694
And the relevant logs from @bourquep's panicking Conduit instance:
E  TRCE h2::proto::streams::state send_close: HalfClosedRemote => Closed
E  TRCE h2::proto::streams::send send_trailers -- queuing; frame=Headers { stream_id: StreamId(107), stream_dep: None, flags: HeadersFlag { end_stream: true, end_headers: true, padded: false, priority: false } }
E  TRCE h2::proto::streams::prioritize schedule_send; StreamId(107)
E  TRCE h2::proto::streams::store Queue::push
E  TRCE h2::proto::streams::store  -> first entry
E  TRCE h2::proto::streams::prioritize reserve_capacity; stream=StreamId(107); requested=0; effective=0; curr=0
E  TRCE h2::proto::settings send_pending_ack; pending=None
E  TRCE h2::codec::framed_read poll
E  TRCE h2::codec::framed_read poll; bytes=13B
E  TRCE h2::codec::framed_read decoding frame from 13B
E  TRCE h2::codec::framed_read     -> kind=Reset
E  DBUG h2::codec::framed_read received; frame=Frame::Reset(Reset { stream_id: StreamId(107), error_code: CANCEL })
E  TRCE h2::proto::connection recv RST_STREAM; frame=Reset { stream_id: StreamId(107), error_code: CANCEL }
E  TRCE h2::proto::streams::prioritize clear_queue; stream-id=StreamId(107)
E  TRCE h2::proto::streams::prioritize dropping; frame=Frame::Headers(Headers { stream_id: StreamId(107), stream_dep: None, flags: HeadersFlag { end_stream: true, end_headers: true, padded: false, priority: false } })
E  TRCE h2::proto::settings send_pending_ack; pending=None
E  TRCE h2::codec::framed_read poll
E  TRCE h2::proto::streams::prioritize try reclaim frame
E  TRCE h2::proto::streams::prioritize poll_complete
E  TRCE h2::proto::streams::prioritize schedule_pending_open
E  TRCE h2::proto::streams::prioritize pop_frame
E  TRCE h2::proto::streams::prioritize pop_frame; stream=StreamId(107)
E  thread 'main' panicked at 'must be scheduled to reset', /checkout/src/libcore/option.rs:891:5
E  note: Run with `RUST_BACKTRACE=1` for a backtrace.

More information on the specific case where this issue occurs may be found in runconduit/conduit#607.

I believe this is because the stream is in the Closed(Cause::EndStream) state already when it is reset?

Do you mean that we've received a RST_STREAM from the peer at about the same time we were trying to close it up locally?

@seanmonstar yeah, that's what appears to be happening in the logs.

hawkw/h2@7b80ac05184c04025ce9bb42033066d0d6bc8096 adds a test in stream_states.rs that reproduces the crash.


chore(main): release megaparse 0.0.42

:robot: I have created a release beep boop
0.0.42 (2024-11-08)
Features

sdk: new version (e377cd6)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/QuivrHQ/MegaParse/releases/tag/megaparse-v0.0.42 :sunflower:


Offline listening (Qobuz)

Hi,
I really enjoy the Qobuz app, but the missing offline listening feature really is a bummer. When I have an unstable connection I cannot listen at all...
Actually it's the only thing missing apart from maybe support for switching audio devices.
Thanks for making this, I will purchase a lifetime license in an instant if this feature is added!

Hello, @JoeKays. Nuvola uses Qobuz web player, which doesn't support offline listening - see Is it possible to listen to music without internet connection? Since Nuvola doesn't have direct access to the music, I cannot implement such a feature in Nuvola itself.


Display confluence changes in activity stream

Hello,
I can observe that plugin jira don't allow confluence changes to be displayed with activity stream :
https://github.com/RoadieHQ/roadie-backstage-plugins/blob/main/plugins/frontend/backstage-plugin-jira/src/api/index.ts#L333
Display this changes could be interresting for me. That's why I want to provide a patch allowing to configure the jiraCard component to display confluence changes with activity stream.
Regards,
Stéphane MORI

@stephane-mori I think it might be a little more clear if you were to add a new option to the getActivityStream function. It could be called "filterConfluenceActivity" and default it to true.. What do you think?


[PIE-2154] Change complex words

Signed-off-by: Steven Gregg<EMAIL_ADDRESS>

Pull Request Description
Some complex words should be changed to simple words.
Fixed Issue(s)

PIE-2154

Will replace with a new PR