OMF format: Refactor relocation code and add type 2 and 3

This is a refactoring of the code that do OMF relocations.
This is based on comments from Ryan, saying that handling should be move towards the loader class.
Also relocation type 2, segment relocation and type 3 segment:offset is implemented, which was the base goal.
Moving the code means that the special state class is unneeded, as variables can now be local inside the relocation function.
Also the fixuptarget class is unneeded, but was used by OmfModuleEnd.
But since that OmfModuleEnd isn't using the information, it is changed to simply skip the information rather than reading it.
This could be split to a separate commit.
Also OmfData is changed to a abstract class, as some functions are identical in OmfIterationData and OmfEnumerationData.
It also makes keeping the link for the lastdata block simpler.
resolveFrame is simply removed as the information wasn't used in later processing.
I have added a catch of IndexArrayOutOfBounds in the relocation code. This may be a bad idea as this will try to recover a pass of  a bad file, rather than just failing.
The file attached to #4793 also shows another special thing. Some symbols are present in both an EXTDEF and PUBDEF, meaning that the current code, will procedure the same name both as a label inside the code, and as an external thunk.
The relocations in this case points to the external symbol, so all calls points there although the code exists.
So processing order is changed to public first, then externals, allowing externals to check for a PUBDEF definition and reusing the symbol, rather than creating an external thunk, which is what a linker would do.
For now the search loops through all PUBDEFs record to check the symbol. This could be optimized by creating a list of public symbols or try using the program symboltable. But the current implementation works.
In #4777 some relocations points to external names like 'FIWRQQ', which are handled special by linkers.
So an array of these are created in OmfSymbol and the relocation code for external target checks for these names, and skip the relocation if a matching name is found. This could be an arraylist instead.
This could be split to a follow up commit, rather than being a part of this.
Can I find the eclipse formatter in git repo, so I do proper formatting?
If not which format of "if" constructs are preferred, with {} or without for one liners?
(Fixes: #4777, #4793)

in the ghidra/eclipse directory, there should be a GhidraEclipseFormatter.xml file.

It should all be fixed now.

The formatter reports no more changes and I have been through all files.
Have added the missing static in OmfSymbol.

@Gravelbones, would you mind rebasing on the latest master and addressing the conflicts?  I'm trying to get us up to date on our OMF contributions.

Working on it, but now the code breaks on any import.
Seems to be reading too much.

Doh, I prolly broke something. I'll take a look tomorrow morning. I don't have a lot of samples to test with.

Not your mistake. And don't spend time on it.
It is a combination of old mistakes, new mistakes by me, and the fixup fix from #4912 not being correct either.
It should not be read1or2 based on 16/32, but readIndex based on high bit being set in first bit.
This will throw off any 32 bit load, by 1 byte very quickly.
Patch should be ready in about 12 hours, after I get home from work. :)
I got plenty of OMF files, and also found some new bugs, which hasn't been reported or fixed yet. :)

Ok, thanks!

Someone should really learn to do better testing. :)
"& 0xfe" broke things worse, so ALL records were marked "UNKNOWN".
This is because the value is tested against an internal representation, which is signed extended. Doh!
So the proper fix should be "& ~1" which doesn't interfere with sign extension.
All other comment types than the 3 mentioned should skip the bytes of the record, which wasn't done.
The error only showed if there were comment records with a length > 0 and not one of the 3. Did I mention bad testing?
Now there is a skip of the bytes in the default part of the switch statement.
The fixup subrecord where using read1or2bytes when reading subrecords, because the standard says 1 or 2 bytes.
But the function readIndex also reads 1 or 2 bytes, but is dependent on the first byte having high bit set for 2 bytes.
So this would make the function read 2 bytes every time in 32 bit mode, where it should still read only one byte.
This have been checked against the OpenWatcom objdmp program.
So the fix from #4912 was wrong too, but also the code before was wrong. Hopefully it is right now.
Comdat entries which are unsupported can have fixup records.
So if those fixup come after an iterated or enumerated datablock, it would apply the fixup to the wrong datablock.
If only a comdat block was seen, it would do a Null Pointer Exception, because lastDataBlock was null.
So a test for null in lastDataBlock is added in relocation code, and lastDataBlock is set to null in unsupported.
There should be no unsupported blocks between datablocks and fixup records in a normal case.
Adding proper handling of comdat is next in line.
externsyms could be left as null if no external symbols are defined.
Making it possible that the relocation code could try to use a null value.
Only bad data should trigger it. To make sure it has a value, it is initialized at the start.
In OmfLoader I would really like to remove the log messages in line 380, 391 and 403 or thereabout.

But when loading a OMF library with 1000s of records, you get lots of these, in essence making it harder to see Unsupported and Unhandled messages that is important.
I will assume that if the functions fail, a Exception will be thrown.
Otherwise a reverse test would be a proper message, rather than it succeeded.

I can not do much about OmfFixupRecord.
The major problem is that you included #4912 before this patch, but #4912 were based on my changes in this patch, which I now try to reintegrate.
Also this may be the source of those "final var" which I did not do.

This is the best I can do to split the commit

Thanks for supporting/maintaining this @Gravelbones.  This is an area where we lean a bit more on the community for help with.


Fingering with voltas

It seems to me that the program reads the partition continuously from start to end. It would be nice if it could take into account prima volta, seconda volta, etc..

this would actually be a bit difficult to implement..


New send button position

Fixes #2046 and #2044

I really like this.
Sent with GitHawk

@Huddie me too!
Sent with GitHawk


add tests to latex definitions files

verify consistence between all languages and verify possible duplicates

needs rebasing on https://github.com/chairemobilite/transition/pull/1158


Update tutorial | deepedit deepgrow monailabel

Update some information according to latest contribution guide.

Hi @mingxin-zheng , I saw the new copyright check is in effective, I'm adding deepedit, deepgrow and monailabel files with the latest copyright guide. But I see a lot of more are not covered. Do we add copyright license to all files in this PR? Or we handle them in a separate PR? Thank you!

Hi @tangy5 , the copyright check is not "required" so we only need to add it to the files modified in the PR.

Can we remove the cell block "Library versions used" in the deepedit/infoANDinference.ipynb if it is not necessary? @tangy5

Also, the copyright header in the monai label notebooks can be updated with the version with indentation. Please find the template in the link. Thanks!


Appease clippy

API docs are being generated and will be shortly available at: https://godot-rust.github.io/docs/gdnative/pr-1067

API docs are being generated and will be shortly available at: https://godot-rust.github.io/docs/gdnative/pr-1067


Added Unit Tests And Validations

In this PR I :

Added validations for  User and Post models.
Wrote unit tests for User, post, like, and comment models.

@Reem-lab Good day, and thank you for the amazing review, but the tests are passing on my machine and in my db the column name is author_id, please how do i go about this. You'd find screenshots attached to this.

Hi @Othniel2471, the tests are passing on my computer

@prolajumokeoni  Thank you for the kind review


10oct scc compliance it fix

PR summary

PR Checklist
Please make sure that your PR fulfills the following requirements:

[ ] The commit message follows the Angular Commit Message Guidelines.
[ ] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features)

Current vs new behavior

Does this PR introduce a breaking change?

[ ] Yes
[x] No

Other information

raised a new PR in correct format. So closing this one


Add Flexibility to Callonce Inside Scenario Outline

Currently Callonce has the same functionality regardless of where it's placed in a feature file. The ask is to allow it to behave differently if invoked from inside a Scenario Outline. Once the Scenario Outline completes, clear the cache for the callonce file, so any subsequent Scenarios in the feature file can call the same file without getting the cached response needed only for the Scenario Outline.
Below shows how it could work.
Feature: Callonce inside a Scenario Outline to allow cache to be cleared for subsequent scenarios use of the file.

Background:

* url demoBaseUrl
* def name = "Bob"

Scenario Outline: Cache partnerUserId response for every row in Scernario Outline, but not every Scenario in the feature
* callonce read('get_partner_user_id.js')
    Given path 'cats'
    When method post
		And request {'#(name)', <partnerUserId>}
    Then status 200
    And match $.name == "Bob"
		And match $.partnerUserId == '#String'
		* def $.partnerUserId = unique

Examples:
| partnerUserId      |
| '#(partnerUserId)' |
| '#(partnerUserId)' |

Scenario: Create new partnerUserId value as cache has been cleared at end of Scenario Outline
* call read('get_partner_user_id.js')
    Given path 'cats'
    And request { name: '#(name)', kittens: '#(partnerUserId)' }
    When method post
    Then status 200
    And match $.name == "Bob"
    And match $.partnerUserId != unique

unable to understand issue and what is expected. putting on hold and requesting community support

Added example to hopefully clarify the ask.

I'll admit defeat, I tried to understand this for a while and gave up. While this remains open for community intervention, I'd like to remind you that the best chance of getting this resolved is:

simplify the example so that it does the minimum possible to replicate the issue
have the example as part of a zip file that includes the entire maven project (you can use the quickstart as a starting point)
clearly explain what the expected behavior is and what the actual behavior is, ideally printed to the log as part of the working example

all the best.

closing due to inactivity


Fix: completed and operationOnFlight wouldn't be updated

closes #163

Codecov Report

Merging #164 (aebe907) into main (ae37686) will increase coverage by 0.00%.
The diff coverage is 100.00%.

:exclamation: Current head aebe907 differs from pull request most recent head e9fc98f. Consider uploading reports for the commit e9fc98f to get more accurate results

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@           Coverage Diff           @@
##             main     #164   +/-   ##
=======================================
  Coverage   99.13%   99.14%
=======================================
  Files          24       24
  Lines        1511     1514    +3
=======================================
+ Hits         1498     1501    +3
  Misses         13       13

Impacted Files
Coverage Δ

qtgql/codegen/py/runtime/queryhandler.py
100.00% <100.00%> (ø)

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


Add subfolders for 3.6 and 3.8, add xmlsec for 3.8

In procurement, we're building our own version of xmlsec because of a bug.  we're working from 3.8.
this pull request proposes adding a new folder hierarchy so that we can have packages for different versions of alpine.
it includes a first package for 3.8 for xmlsec based on what we're doing merged with the 3.6 xmlsec that we're compiling.  I have made some "educated" guesses about what that APKBUILD script is and how it works but I don't really know what I'm doing.
presumably if this is merged, some sort of script that runs on jenkins somewhere will need to be udpated to build and push multiple versions' images...
cc @bobthemighty

Cool, I need to test this and if this works, we can try and publish this.

the bob suggested we / someone should pair on this before merging it

Let's do that.

this may still be a good idea, but as far as I'm concerned the urgency is gone bc we can use the https://github.com/madedotcom/alpine-python instead, can close if y'all want.


pull request for e2e test at 12/17/2016 02:15:16

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


⚠ Cioos Pacific CKAN Sitemap has degraded performance

In 3559620, Cioos Pacific CKAN Sitemap (https://catalogue.cioospacific.ca/sitemap/sitemap-1.xml) experienced degraded performance:

HTTP code: 200
Response time: 19600 ms

Resolved: Cioos Pacific CKAN Sitemap performance has improved in f3393ce after 48 minutes.


fix(studio): update image version to 20231023

What kind of change does this PR introduce?
Bug fix, feature, docs update, ...
What is the current behavior?
Please link any relevant issues here.
What is the new behavior?
Feel free to include screenshots if it includes visual changes.
Additional context
Add any other context or screenshots.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
5 unchanged lines in 1 file lost coverage.
Overall coverage decreased (-0.03%) to 56.917%

Files with Coverage Reduction
New Missed Lines
%

internal/gen/keys/keys.go
5
12.31%

Totals

Change from base Build<PHONE_NUMBER>:
-0.03%

Covered Lines:
5188

Relevant Lines:
9115

💛  - Coveralls


Plugin Support for OpenSearch

AWS has released OpenSearch with ALv2 license: https://github.com/opensearch-project/OpenSearch
Is there any plan to support this analyzer for OpenSearch ?

I have created a fork and refactored it to support OpenSearch: https://github.com/soosinha/opensearch-analysis-thaichub2 .
Would you like to import this as a separate repository in your project?


Added Travel Packing Calculator

Fixes Issue🛠
Closes #1322
Description👨💻
The Travel Packing Calculator generates customized packing lists tailored to your travel plans, provides real-time weather updates, and suggests top tourist spots at your destination. It features a user-friendly interface, supports multiple activities, and includes smart packing tips to ensure a seamless travel experience.
Type of Change📄

[x] New feature (non-breaking change which adds functionality)

Checklist✅

[x] I am an Open Source contributor
[x] I have performed a self-review of my code
[x] My code follows the style guidelines of this project
[x] I have commented on my code, particularly in hard-to-understand areas

Screenshots/GIF📷

@Rakesh9100 This PR is reviewed & can be merged.


OraclizeAPI not compatible with solc ver > 4.0

When I try to compile with Truffle 2.0.1 which uses solc >4.0 it raises multiple compilation errors.

 Error: Expected token Semicolon got 'RBrace'

This is the due to the fact that you now have to append semicolon like _; in modifier.

 Error: Member "value" not found or not visible after argument-dependent lookup

After fixing #1, I still get the following errors.
OraclizeI.sol:97:16: Error: Member "value" not found or not visible after argument-dependent lookup in function (uint256,string memory,string memory) returns (bytes32) - did you forget the "payable" modifier?
        return oraclize.query.value(price)(0, datasource, arg);
               ^------------------^

For the second issue, the important part of the error is did you forget the "payable" modifier?

As you can see here, we have solidity 0.4 compatible version


Site permissions and dependent permissions table are broken.

Hi Team,
It seems that the "Site permissions and dependent permissions" section is completely broken during the edit.
Please check the previous version of table and fix the documentation.
SharePoint OnPrem table looks correct.
https://learn.microsoft.com/en-us/sharepoint/sites/user-permissions-and-permission-levels#site-permissions
Detail

Permission
Description
Dependent permissions

Manage Permissions
Create and change permission levels on the website and assign permissions to users and groups.
Approve Items, Enumerate Permissions, Open

View Web Analytics Data
View reports on website usage.
Approve Items, Open

"Approve Item" has nothing to do with Manage Permissions. So does View Web Analytics Data.
This is just a couple of examples. Overall, the permission dependency tables are smaller than what is specified in SPS.

Document Details
⚠ Do not edit this section. It is required for learn.microsoft.com ➟ GitHub issue linking.

ID: 2f072d4a-23e6-d73f-397e-4ffc000570a9
Version Independent ID: 1ee08317-82cd-9f23-a1e9-d2380fb7ff78
Content: Understanding permission levels in SharePoint - SharePoint in Microsoft 365
Content Source: SharePoint/SharePointOnline/understanding-permission-levels.md
Service: sharepoint-online
GitHub Login: @MikePlumleyMSFT
Microsoft Alias: mikeplum

Thanks for your feedback, I created a PR to correct the table


ECS Service force new deployment

Resolves #518

should we add a test for it?

We could add a tests that if true after the successful apply it goes back to false. Or do you have something more elaborate in mind?

Yeah, that sounds good.


ARROW-2975: [Plasma] Fix TensorFlow operator compilation with pip package

This uses pyarrow.get_include() and pyarrow.get_library_dirs() instead of pkg-config to build the TensorFlow operator. The latter is not available in pip packages.

I need a little help with the build system here. It works in normal operation, but not in the tests, since the arrow libraries are not available in the directories in pyarrow.get_library_dirs(). I created a JIRA here: https://issues.apache.org/jira/browse/ARROW-2976

Hm, does this impact the RC that is out? Can you comment on the vote thread?

rebased on #2398 now

Codecov Report

Merging #2368 into master will increase coverage by 2.19%.
The diff coverage is 88.88%.

@@            Coverage Diff             @@
##           master    #2368      +/-   ##
==========================================
+ Coverage   84.67%   86.87%   +2.19%
==========================================
  Files         293      237      -56
  Lines       45459    42708    -2751
==========================================
- Hits        38492    37101    -1391
+ Misses       6926     5607    -1319
+ Partials       41        0      -41

Impacted Files
Coverage Δ

python/pyarrow/plasma.py
87.5% <88.88%> (-0.22%)
:arrow_down:

go/arrow/math/int64_amd64.go

rust/src/memory.rs

rust/src/array.rs

go/arrow/math/float64_sse4_amd64.go

go/arrow/memory/buffer.go

go/arrow/math/int64_avx2_amd64.go

go/arrow/math/float64_avx2_amd64.go

go/arrow/math/uint64_sse4_amd64.go

rust/src/buffer.rs

... and 48 more

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 5c97cd6...41708bc. Read the comment docs.


[GCP] Make URL/URI parsing more comprehensive

Resources that are global have an extra global parameter (e.g. global/networks/<vpc-name> for VPCs). Moreover, getVPCURL generates partial URI. Even though the compute API allows for partial URIs, there may be edge cases around using partials. We should generate full URIs for everything even if it may add a little bit of redundancy.
Also, we should decide on a naming convention (URL vs URI). Resolved by #164
Resources

https://cloud.google.com/apis/design/resource_names
https://cloud.google.com/compute/docs/api/how-tos/api-requests-responses

Closing since all components have been resolved by #331.


39 refactor to blockly generator

Fixes #39

This change is

For the most part, this looks great. A few easy issues noted below.
I think we should keep the term CodeGeneratorService (or at least BlocklyCodeGeneratorService if you really think it needs "Blockly" in the name, a fair point for something listed in the manifest). Code generator is the term we use in the docs to describe the JavaScript functions this runs.
Relatedly, in my mind, "Blockly generator" invokes the idea of those JavaScript functions. I think the BlocklyGenerator you have below is really a helper or manager for the code generation service. Hence, CodeGenerationManager or ...Helper is probably a better name.
Thanks a bunch!!

Reviewed 9 of 9 files at r1.
Review status: all files reviewed at latest revision, 10 unresolved discussions.

blocklylib-core/src/main/java/com/google/blockly/android/AbstractBlocklyActivity.java, line 308 at r1 (raw file):

        mClipDataHelper = onCreateClipDataHelper();
        mBlocklyGenerator = new BlocklyGenerator(
            getBlockDefinitionsJsonPaths(), getGeneratorsJsPaths());

getBlockDefinitionsJsonPaths() and getGeneratorsJsPaths() should be called and passed in during onRunCode(), so an app can change the available block set or output language based on other parts of the app (e.g., a output language selector).
You probably want to pass in the this Context for the below onResume()/onPause() suggestions.

blocklylib-core/src/main/java/com/google/blockly/android/AbstractBlocklyActivity.java, line 429 at r1 (raw file):

        super.onResume();
        Intent intent = new Intent(this, BlocklyGeneratorService.class);
        bindService(intent, mBlocklyGenerator.getCodeGenerationService(), Context.BIND_AUTO_CREATE);

I suggest wrapping these two lines in CodeGenerationManager.onResume().  This, and the matching onPause() below, should complete the transition to a higher level API, with no direct references to the Service.

blocklylib-core/src/main/java/com/google/blockly/android/AbstractBlocklyActivity.java, line 456 at r1 (raw file):

    protected void onPause() {
        super.onPause();
        unbindService(mBlocklyGenerator.getCodeGenerationService());

And this line could be moved to CodeGenerationManager.onPause().

blocklylib-core/src/main/java/com/google/blockly/android/AbstractBlocklyActivity.java, line 765 at r1 (raw file):

    /**
     * Runs the code generator. Called when user selects "Run" action.

Please add:
Gets the latest block definitions and generator code by calling {@link #getBlockDefinitionsJsonPaths()} and {@link #getGeneratorsJsPaths()} just before invoking generation.

blocklylib-core/src/main/java/com/google/blockly/android/WorkspaceFragment.java, line 109 at r1 (raw file):
Previously, ChrisA89 wrote…
This is the logic that used to be called by onRunCode (above). I've extracted it here and it's now called in the BlocklyGenerator (below).

Move this into Workspace, instead of WorkspaceFragement. Add a JavaDoc comment when you do.

blocklylib-core/src/main/java/com/google/blockly/android/codegen/BlocklyGenerator.java, line 18 at r1 (raw file):

 * Client Side class responsible for connecting to the {@link BlocklyGeneratorService}.
 */
public class BlocklyGenerator {

CodeGenerationHelper or Manager. Reasoning detailed at the top of this review.

blocklylib-core/src/main/java/com/google/blockly/android/codegen/BlocklyGenerator.java, line 46 at r1 (raw file):

    }

    public ServiceConnection getCodeGenerationService() {

Probably don't need this if we implement the onResume()/onPause() methods suggested above.

blocklylib-core/src/main/java/com/google/blockly/android/codegen/BlocklyGenerator.java, line 58 at r1 (raw file):
Previously, ChrisA89 wrote…
The callback could be passed in the constructor, but I think it makes the outcome of the method clearer if passed in here.

The first argument should be a Workspace, since there is nothing particular about the UI/Fragment that is needed here.

blocklylib-core/src/main/java/com/google/blockly/android/codegen/BlocklyGenerator.java, line 74 at r1 (raw file):
Previously, ChrisA89 wrote…
This didn't used to be logged, but I think it's useful.

:thumbsup:

blocklylib-core/src/main/java/com/google/blockly/android/codegen/BlocklyGeneratorService.java, line 50 at r1 (raw file):

 * to generate code.
 */
public class BlocklyGeneratorService extends Service {

Revert rename.

Comments from Reviewable

I'll sort this out now. The rename was mainly because issue #39 asked for it.

Oh, looky there. Huh... Sorry for the confusion. Should be easy to fix. Thanks!

:LGMT:


Bring back old version

Can you bring back the old version of instant inventory this no one feels so buggy and weird.

I agree - I much preferred the old version. This one is broken.

Could you describe whats broken with the new version?
Try changing the advanced settings It might be the new Min Changed Millis, try setting that to 0. I appreciate feedback on what the default advanced values feels the best.
You can also disable a specific feature you don't like in the plugin config

Shift click wield to instantly dropping a item feels terrible. This is used for staff in p1 of verzik.
When you switch gear sometimes it instantly switches and then sometimes it just disappears for a tick then switches. Its extremely inconsistent .

I've trained changing all the settings to make it like the old version but it just feels broken.

Thanks for the feedback. This update shouldn't have changed any existing behaviour, except for the Min Changed Millis mentioned above.

Shift click wield to instantly dropping a item feels terrible

This seems to be the same bug as issue #13

When you switch gear sometimes it instantly switches and then sometimes it just disappears for a tick then switches. Its extremely inconsistent

This seems like a new bug. Do you get the different behaviour for the same items? Or do different items behave differently? I'm trying to pinpoint what goes wrong

1) When I try to [SHIFT] + [LMB] to to 'Wield' bolts, I can't even do that (have to disable plugin, as I'm prevented from wielding them).
2) Group ironman bank storage; Upon depositing items to the group storage, with this plugin enabled, the items actually go to the storage, however the items "appear" on my inventort still. having the option to still "deposit" them, even though they don't actually exist in my inventory anymore.

Will be fixed in the next version, and is also a duplicate of #13
This is a different issue, could you create a new issue for this?


[WASM] Make panic errors available from JS

Also, the demo checks if the error it catches is a WASM RuntimeError and displays the correct error nicely.

Codecov Report

Merging #532 into master will decrease coverage by 0.01%.
The diff coverage is 0%.

@@            Coverage Diff            @@
##           master    #532      +/-   ##
=========================================
- Coverage   48.41%   48.4%   -0.02%
=========================================
  Files          69      69
  Lines       14375   14380       +5
  Branches     3591    3593       +2
=========================================
  Hits         6960    6960
- Misses       5554    5559       +5
  Partials     1861    1861

Impacted Files
Coverage Δ

wasm/lib/src/lib.rs
0% <0%> (ø)
:arrow_up:

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 c675a59...878c131. Read the comment docs.

Happy to merge this one, can you resolve the conflict?

Done!


dialyzer: enable -Wunmatched_returns

This warning checks for unmatched returns, i.e., for exhaustiveness on
structured return values (but not unions of just atoms or integers).

🚀


Support newer versions of 'rpm' that use Sqlite for the db instead of BerkeleyDB

What would you like to be added:
Cataloger support for RPM dbs using sqllite instead of just berkelydb. This would enable support for recent Fedora releases among others.
Why is this needed:
To correctly create SBoMs from containers using newer versions (4.16+) of the rpm package.
Additional context:

This can be identified in the image by 'rpmdb.sqlite' file being present in /var/lib/rpm/ or /usr/sysimage/rpm/

Just a note: We're seeing this cause Syft not to show any RPM packages in fedora:33 and later.

Any idea when this feature will be implemented?

@sli720 We don't have an expected date for this to land yet. We do accept PRs from the community, and we'd happily fields questions from anyone who wants to take a shot at this! 😃

We are currently using a fork of https://github.com/knqyf263/go-rpmdb; however, upstream now supports these new formats.  I spoke some with @wagoodman regarding this and his idea was to work on getting our enhancements submitted upstream and then remove our fork

@westonsteimel I've created https://github.com/tofay/go-rpmdb/commits/upstream-plus-anchore-changes which adds the anchore customizations to the latest upstream commit and gets the tests passing. I've also verified that syft can use this to scan images with sqlite RPM dbs.
I've made two commits - one for the extra file information added in various PRs to the fork, and one for the change to make the epoch optional.
This should be a useful basis for anchore to submit the changes upstream? Note the commits didn't apply cleanly, so would need review/author fix-up as necessary.
Could syft also use that alongside upstreaming to resolve this issue? (perhaps by moving that branch into anchore/go-rpmdb)

Can I submit a PR to anchore/go-rpmdb and leave upstreaming to your team please? You can better speak to the changes and their use case in syft/grype.
I think that requires you creating a branch of anchore/rpm-db, matching the latest upstream, that I can push my changes to?

Hi, I'm a maintainer of go-rpmdb. I'm welcome any contribution😄

Anchore's changes to go-rpmdb have been upstreamed now, and I've raised https://github.com/anchore/syft/pull/1018 which removes the use of anchore's fork.


 I didn't see it.

@IngridPuppet I wasn't on the right tack i am not supposed to touch the handler files i was to add those logic in the plugin file

So i am going to remove all what is in the handler and implement in the plugin


update github.com/containers/{buildah,image,storage}

Signed-off-by: Valentin Rothberg<EMAIL_ADDRESS>

LGTM

This should wait until buildah gets fully updated.  Also want to get in storage 1.13.1

We're waiting on a fix to logout -a by Qi.

I updated https://github.com/containers/libpod/pull/3704 with c/image v3.0.2.

We need to finish rebasing buildah to 3.0.2 and then we can get this updated.

Closing in favor of #3704


Update packit with currently supported upgrade paths

That' an adoptation of leapp-repository's packit update and refactoring introduced in PR1176.
OAMG-10855

/packit build

/packit copr-build


Provide a .forRoot() method on ClarityModule to prevent reimport of ClrResponsiveNavigationService in lazy-loaded feature modules

Select one ...  (check one with "x")
[ ] bug
[ ] feature request
[x] enhancement

Expected behavior
ClarityModule needs a .forRoot() method so that it cooperates nicely with lazy-loaded feature modules.
Actual behavior
The current implementation results in separate instances of ClrResponsiveNavigationService per injector scope of lazy loaded feature modules
Reproduction of behavior
For reference, see https://angular.io/docs/ts/latest/cookbook/ngmodule-faq.html#!#q-for-root
Environment details

Angular version: 2.2

Clarity version: 0.7

OS and version: OS X El Capitan

Browser: [all | Chrome XX | Firefox XX | IE XX | Safari XX | Mobile Chrome XX | Android X.X Web Browser | iOS XX Safari | iOS XX UIWebView | iOS XX WKWebView ]

Thanks for filing this issue @JohannesRudolph. @youdz will try to get this fixed ASAP. Assigning this to him.

Actually, @hippee-lee will take care of this one!


Finish code refinement

No checkstyle warnings

No PMD warnings

As of commit 8afca88, PMD passed, checkstyle passed, with a total of 85% branch coverage achieved.


In x86 targets, replace volatile asm with immintrin calls

This improves the unrolling behaviour on clang quite significantly, where previously it would ignore the -funroll-loops and get stuck on the asm calls.

Doesn't compile on clang; it defines __rdtsc instead of _rdtsc

Apparently both GCC and Clang have __rdtsc defined as a built-in even though neither document it as such. The definition is also entirely missing from <immintrin.h> which means that MSVC needs its own include <intrin.h>.


Allow multiple users to score the same topic

Is your feature request related to a problem? Please describe.
I want to see other people's opinions on my topic, and I want to be able to share my opinion on other people's topics. Ideally, I'd be able to identify differences in opinion at a glance.
Describe the solution you'd like
Scoring seems like a good first step for this. Ideas:

dropdown with select options for each user (logged-in user called "me"?). maybe a multi-select with a "select all" checkbox
bee-hive-like clustering of scores when multiple users are selected:

maybe a max # selected scores for this to avoid bee-hives overlapping everywhere
on hover/click of score that isn't yours: helper text that says the name of the user whose score this is
instead of bee-hive, could average the scores, use a gradient for the score color, that's weighted based on the scores (simple 3-color based on min/avg/max seems good enough), then hover/click could show the names of each user and their chosen score:

instead of gradient, could use the color of the average, and add an indicator (yin-yang symbol?) to convey if there's significant difference in scores (idk based on standard deviation?), with hover/click showing each user's scores

Averaging & using a gradient seems good to me. Bee-hive sounds nice but harder and performance might be bad for many graph parts at once. Indicator also seems easier than gradient but would take up more space? Though it might be nice to introduce the concept of this kind of indicator, because others might have good ideas for other kinds of indicators like this.
Describe alternatives you've considered
Additional context
Technical ideas

Good idea from @alextilot:

pie chart to show score distributions
can be size of current score, then on-hover/-click can expand to size of current score pie, and have on-hover/-click to view usernames that gave that score


 That happens a lot when you work with experimental stuff. So I have to open Task Manager, hunt down the right Visual Studio instance, and kill it.
And then if ***** starts "Windows Error Reporting" to gather data about the crash. Which keeps my files locked for quite some time longer.
Proposed technical implementation details (optional)
Not really sure about the details, but I would just simply like to be able to kill VS and kill it completely. If you can make it restart loading the same solution, so much the better.

I wonder if it would be also nice to be able to select a folders/files in explorer and kill any processes which are currently have any handles to them.

I think this was discussed in the most MSDev show. Featuring Clint 😀
Joost van Schaik
Windows Development MVP & Windows Insider MVP
"The world as I envision it in my head is such a more interesting place"

From: Andrey Nekrasov<EMAIL_ADDRESS>Sent: Friday, February 7, 2020 1:12:59 PM
To: microsoft/PowerToys<EMAIL_ADDRESS>Cc: Joost van Schaik<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [microsoft/PowerToys] Quick kill button (#1228)
I wonder if it would be also nice to be able to select a folders/files in explorer and kill any processes which are currently have any handles to them.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/microsoft/PowerToys/issues/1228?email_source=notifications&email_token=AA7QDH6BEIW26RIZTPHND6DRBVF4XA5CNFSM4KRBXLAKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOELCXD3Y#issuecomment-583365103, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AA7QDH6EREG2HP7TMCVL4Y3RBVF4XANCNFSM4KRBXLAA.

cc @crutkas :)

Hi @LocalJoost
thank you for your feedback, this feature request is tracked in this issue https://github.com/microsoft/PowerToys/issues/322
Feel free to add you comments there. Closing this as duplicate.
Thanks.

Darn. I had searched for duplicates but had missed this one
Joost van Schaik
Windows Development MVP & Windows Insider MVP
"The world as I envision it in my head is such a more interesting place"

From: Enrico Giordani<EMAIL_ADDRESS>Sent: Friday, February 7, 2020 1:45:29 PM
To: microsoft/PowerToys<EMAIL_ADDRESS>Cc: Joost van Schaik<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [microsoft/PowerToys] Quick kill button (#1228)
Hi @LocalJoosthttps://github.com/LocalJoost
thank you for your feedback, this feature request is tracked in this issue #322https://github.com/microsoft/PowerToys/issues/322
Feel free to add you comments there. Closing this as duplicate.
Thanks.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/microsoft/PowerToys/issues/1228?email_source=notifications&email_token=AA7QDH4KA6GWMMSWOHJGAI3RBVJWTA5CNFSM4KRBXLAKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOELCZQFA#issuecomment-583374868, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AA7QDH3LRHE2G44VPC3TC7LRBVJWTANCNFSM4KRBXLAA.

No problem. We should do a better job in setting issue names, to make it easier to find them.


Disappearing cursor

My cursor blinks and then fades completely away...

The more I click around, the opacity returns, but when I let it sit, it blinks and then fades away.

Hey, thanks for reporting the issue. I'm very busy right now, so I don't know when I have some time to take a look at this.

Looks like setting pulseDuration to 0 fixed it for me, I think I had it set to something weird, closing this for now, thanks!

I'm guessing you had it set to something ridiculously high (several thousands)? Can't think of another explanation that would cause this behavior. Anyway, glad to hear you solved it :)

Correcto! Lol, thanks again!


Resolution not changing and Distant Horizons glitch when using shaders

Resolution not changing
Downloaded the mod, changed the resolution while on menu to 0.1, started a world, the resolution was applied, but after opening the mod settings while in game and changing it reverted back to the original resolution and stop working, only after closing and reopening the world it started working again.
Image 1: Resolution theoretically at 0.1 of 4k (384x203) but as you can see on the background everything is crystal clear.

Distant horizons with shaders issue
Downloaded distant horizons installed this shader and every time I load a world the distant horizons low poly chunks start glitching, but when I remove the resolution mod the world load fine and no glitch is visible.
Image 2: Glitch distant horizons low poly chunks when using this mod

Extra info
System Details Report

Report details

Date generated:                              2024-06-12 11:04:52

Hardware Information:

Hardware Model:                              ASUSTeK COMPUTER INC. PRIME X570-PRO
Memory:                                      32.0 GiB
Processor:                                   AMD Ryzen™ 9 5950X × 32
Graphics:                                    AMD Radeon™ RX 6900 XT
Disk Capacity:                               2.5 TB

Software Information:

Firmware Version:                            5013
OS Name:                                     Fedora Linux 40 (Workstation Edition)
OS Build:                                    (null)
OS Type:                                     64-bit
GNOME Version:                               46
Windowing System:                            Wayland
Kernel Version:                              Linux 6.8.11-300.fc40.x86_64

Game info
Mods that I'm using
animatica-0.6+1.20.jar
auto-clicker-fabric-86.0.0.jar
bettermounthud-1.2.3.jar
capes-1.5.4+1.20.5-fabric.jar
cloth-config-14.0.126-fabric.jar
continuity-3.0.0-beta.5+1.20.5.jar
Controlify-2.0.0-beta.10+1.20.6.jar
cubes-without-borders-1.1.0+1.20.jar
Debugify-1.20.6+1.0.jar
DistantHorizons-2.1.0-a-1.20.6-noForge.jar
dynamic-fps-3.4.3+minecraft-1.20.5-fabric.jar
e4mc_minecraft-5.0.0-fabric.jar
enhancedblockentities-0.10+1.20.6.jar
entityculling-fabric-1.6.5-mc1.20.6.jar
entity_model_features_fabric_1.20.5-2.0.2.jar
entity_texture_features_fabric_1.20.5-6.0.1.jar
fabric-api-0.100.0+1.20.6.jar
fabric-language-kotlin-1.11.0+kotlin.2.0.0.jar
fabricskyboxes-0.7.4+mc1.20.5.jar
fabrishot-1.13.1.jar
fastquit-3.0.0+1.20.6.jar
ferritecore-6.1.1-fabric.jar
firstperson-fabric-2.4.1-mc1.20.6.jar
fsb-interop-1.4.0+mc1.20.5-build.53.jar
ImmediatelyFast-Fabric-1.2.17+1.20.6.jar
indium-1.0.31+mc1.20.4.jar
iris-1.7.0+mc1.20.5.jar
language-reload-1.6.1+1.20.6.jar
lithium-fabric-mc1.20.6-0.12.3.jar
main-menu-credits-1.2.0.jar
mixintrace-1.1.1+1.17.jar
modelfix-1.14-fabric.jar
modernfix-fabric-5.18.0+mc1.20.6.jar
modmenu-10.0.0-beta.1.jar
morechathistory-1.3.1.jar
moreculling-1.20.5-0.25.0.jar
NoChatReports-FABRIC-1.20.6-v2.7.0.jar
notenoughanimations-fabric-1.7.3-mc1.20.6.jar
optigui-2.3.0-beta.2+1.20.5.jar
paginatedadvancements-2.4.2.jar
polytone-1.20.6-1.19.2-fabric.jar
puzzle-1.6.2+1.20.6.jar
reeses_sodium_options-1.7.2+mc1.20.5-build.103.jar
resolution-control-plus-plus-1.20.6-3.0.2.jar.disabled
rrls-4.0.7+mc1.20.6-fabric.jar
sodium-extra-0.5.5+mc1.20.6-build.118.jar
sodium-fabric-0.5.8+mc1.20.6.jar
sound-physics-remastered-fabric-1.20.6-1.4.2.jar
YetAnotherConfigLib-3.4.4+1.20.6-fabric.jar
yosbr-0.1.2.jar
Zoomify-2.13.5+1.20.6.jar

Resolution not changing
Downloaded the mod, changed the resolution while on menu to 0.1, started a world, the resolution was applied, but after opening the mod settings while in game and changing it reverted back to the original resolution and stop working, only after closing and reopening the world it started working again.
Image 1: Resolution theoretically at 0.1 of 4k (384x203) but as you can see on the background everything is crystal clear.

Distant horizons with shaders issue
Downloaded distant horizons installed this shader and every time I load a world the distant horizons low poly chunks start glitching, but when I remove the resolution mod the world load fine and no glitch is visible.
Image 2: Glitch distant horizons low poly chunks when using this mod

Extra info
System Details Report
Report details

Date generated:                              2024-06-12 11:04:52

Hardware Information:

Hardware Model:                              ASUSTeK COMPUTER INC. PRIME X570-PRO
Memory:                                      32.0 GiB
Processor:                                   AMD Ryzen™ 9 5950X × 32
Graphics:                                    AMD Radeon™ RX 6900 XT
Disk Capacity:                               2.5 TB

Software Information:

Firmware Version:                            5013
OS Name:                                     Fedora Linux 40 (Workstation Edition)
OS Build:                                    (null)
OS Type:                                     64-bit
GNOME Version:                               46
Windowing System:                            Wayland
Kernel Version:                              Linux 6.8.11-300.fc40.x86_64

Game info
Mods that I'm using
animatica-0.6+1.20.jar
auto-clicker-fabric-86.0.0.jar
bettermounthud-1.2.3.jar
capes-1.5.4+1.20.5-fabric.jar
cloth-config-14.0.126-fabric.jar
continuity-3.0.0-beta.5+1.20.5.jar
Controlify-2.0.0-beta.10+1.20.6.jar
cubes-without-borders-1.1.0+1.20.jar
Debugify-1.20.6+1.0.jar
DistantHorizons-2.1.0-a-1.20.6-noForge.jar
dynamic-fps-3.4.3+minecraft-1.20.5-fabric.jar
e4mc_minecraft-5.0.0-fabric.jar
enhancedblockentities-0.10+1.20.6.jar
entityculling-fabric-1.6.5-mc1.20.6.jar
entity_model_features_fabric_1.20.5-2.0.2.jar
entity_texture_features_fabric_1.20.5-6.0.1.jar
fabric-api-0.100.0+1.20.6.jar
fabric-language-kotlin-1.11.0+kotlin.2.0.0.jar
fabricskyboxes-0.7.4+mc1.20.5.jar
fabrishot-1.13.1.jar
fastquit-3.0.0+1.20.6.jar
ferritecore-6.1.1-fabric.jar
firstperson-fabric-2.4.1-mc1.20.6.jar
fsb-interop-1.4.0+mc1.20.5-build.53.jar
ImmediatelyFast-Fabric-1.2.17+1.20.6.jar
indium-1.0.31+mc1.20.4.jar
iris-1.7.0+mc1.20.5.jar
language-reload-1.6.1+1.20.6.jar
lithium-fabric-mc1.20.6-0.12.3.jar
main-menu-credits-1.2.0.jar
mixintrace-1.1.1+1.17.jar
modelfix-1.14-fabric.jar
modernfix-fabric-5.18.0+mc1.20.6.jar
modmenu-10.0.0-beta.1.jar
morechathistory-1.3.1.jar
moreculling-1.20.5-0.25.0.jar
NoChatReports-FABRIC-1.20.6-v2.7.0.jar
notenoughanimations-fabric-1.7.3-mc1.20.6.jar
optigui-2.3.0-beta.2+1.20.5.jar
paginatedadvancements-2.4.2.jar
polytone-1.20.6-1.19.2-fabric.jar
puzzle-1.6.2+1.20.6.jar
reeses_sodium_options-1.7.2+mc1.20.5-build.103.jar
resolution-control-plus-plus-1.20.6-3.0.2.jar.disabled
rrls-4.0.7+mc1.20.6-fabric.jar
sodium-extra-0.5.5+mc1.20.6-build.118.jar
sodium-fabric-0.5.8+mc1.20.6.jar
sound-physics-remastered-fabric-1.20.6-1.4.2.jar
YetAnotherConfigLib-3.4.4+1.20.6-fabric.jar
yosbr-0.1.2.jar
Zoomify-2.13.5+1.20.6.jar

The developer is trying to fix this bug.
You can check it in my issue #9


GitClient failing

 jsconfig dir in: /opt/code/local/stdorg/config
Traceback (most recent call last):
  File "jstest.py", line 47, in <module>
    compare('j', jgen, dj, 2)
  File "jstest.py", line 45, in compare
    compare("%s.%s" % (tree, subname), subobj1, subobj2, depth-1)
  File "jstest.py", line 43, in compare
    subobj1 = getattr(obj1, subname)
  File "/usr/local/lib/python3.6/dist-packages/jumpscale.py", line 18, in _lazyprop
    setattr(self, attr_name, fn(self))
  File "/usr/local/lib/python3.6/dist-packages/jumpscale.py", line 1452, in currencylayer
    return JumpscaleLibClientsCurrencylayerCurrencyLayer()
  File "/usr/local/lib/python3.6/dist-packages/jumpscale.py", line 1253, in JumpscaleLibClientsCurrencylayerCurrencyLayer
    return _CurrencyLayer()
  File "/usr/local/lib/python3.6/dist-packages/JumpscaleLib/clients/currencylayer/CurrencyLayer.py", line 22, in __init__
    parent=None, template=TEMPLATE)
  File "/home/lkcl/src/jumpscale_core/Jumpscale/tools/configmanager/JSBaseClassConfig.py", line 36, in __init__
    self, instance=self._instance, data=data, template=self._template, ui=self._ui)
  File "/home/lkcl/src/jumpscale_core/Jumpscale/tools/configmanager/ConfigManager.py", line 271, in _get_for_obj
    data=data)
  File "/home/lkcl/src/jumpscale_core/Jumpscale/tools/configmanager/Config.py", line 28, in __init__
    if not j.sal.fs.exists(self.path):
  File "/home/lkcl/src/jumpscale_core/Jumpscale/tools/configmanager/Config.py", line 69, in path
    j.data.text.toStr(j.tools.configmanager.path),
  File "/home/lkcl/src/jumpscale_core/Jumpscale/tools/configmanager/ConfigManager.py", line 69, in path
    self.init()
  File "/home/lkcl/src/jumpscale_core/Jumpscale/tools/configmanager/ConfigManager.py", line 517, in init
    cpath, giturl = self._findConfigRepo(die=False)
  File "/home/lkcl/src/jumpscale_core/Jumpscale/tools/configmanager/ConfigManager.py", line 206, in _findConfigRepo
    self.logger.info("remote url: %s" % g.remoteUrl)
  File "/home/lkcl/src/jumpscale_core/Jumpscale/clients/git/GitClient.py", line 84, in remoteUrl
    raise j.exceptions.Input("There is not remote configured for this repository")
Jumpscale.errorhandling.JSExceptions.Input: ERROR: There is not remote configured for this repository ((type:input.error)
[Fri10 11:59] - ErrorHandler.py   :257 :j.core.errorhandler  - ERROR    - There is not remote configured for this repository

there is no remote because there's no... remote.
as in, the repository /opt/code/local/stdorg/config is indeed local.... because it's been created... locally.
so ConfigFactory is assuming that there's a remote repo.... when there isn't one.


Document profile guided optimization

For performance sensitive environments using profile guided optimization (PGO) is very useful, so reference it from the website and performance sensitive environment documentation.
Also add duk-pgo.O2 and duk-perf-pgo.O2 test targets to the Makefile. The difference to plain GCC -O2 is almost 20% on tests/perf/test-mandel.js:
# -O2 without PGO.
$ time ./duk.O2 tests/perf/test-mandel.js
[...]

real    0m3.061s
user    0m3.059s
sys     0m0.000s

# -O2 with PGO.
$ time ./duk-pgo.O2 tests/perf/test-mandel.js
[...]

real    0m2.488s
user    0m2.487s
sys     0m0.000s

On Google V8 benchmark the plain -O2 result is around 305, while -O2 + PGO is around 376.
The training set for duk-pgo.O2 is quite dummy and doesn't involve e.g. regexps, so these results are probably possible to improve on by using a better profile generation test set.

@fatcerberus It would be interesting to hear how MSVC PGO (https://msdn.microsoft.com/en-us/library/e7k32f4k.aspx) affects Minisphere -- in case you haven't tried that yet. It seems MSVC won't be getting much in the way of static hot/cold attributes and PGO is recommended instead.

The -O2+PGO build can also be smaller (this will obviously depend on the compiler):
 356973	   1468	    328	 358769	  57971	duk.O2.pgo
 380836	   1436	    216	 382488	  5d618	duk.O2.master

@svaarala I actually haven't tried PGO yet, mainly because I haven't felt the need.  minisphere isn't really CPU-bound at all; by default it even throttles to 60fps (this is configurable by game code of course) so you have ~16ms to get everything done between frames, which in almost all cases is more than enough.  I honestly don't think I would benefit much from a JIT engine.  Cell would of course, but I'm not going out of my way to embed V8 just for the build tool ;-)

Ok, maybe that transpilation pass then? That seemed to be a performance bottleneck in your case?
I don't know how complex using PGO is with MSVC but with GCC it was trivial: https://github.com/svaarala/duktape/blob/master/Makefile#L373-L386.
First pass with -fprofile-generate, then just run the program, and second pass with -fprofile-use.

To be clear, I'm interested in the impact of PGO with MSVC in general. But since you have an actively developed codebase and there was discussion about a Minisphere benchmark run, I thought it might be relatively easy for you to try that out.
I'll look into MSVC PGO at some point anyway (just for documenting the bare essentials) and I can test it then too.

I'm still investigating how to use PGO with MSVC, but here's the result of a profiling run on Cell (note: optimized with /O1 "Minimize Size") while transpiling ES6 code with Babel:

The profile looks quite a bit different with Babel compared to TypeScript.  With TypeScript, I found that I spent a large portion of my execution time in duk__vm_arith_add() (i.e. string concatenation), while here, that function is nowhere to be found.  Bytecode dispatch dominates of course, but property retrieval accounts for a decent chunk too.

No idea why I was optimizing Cell for size - switching to /O2 vastly improved my transpilation speeds :)

How much is "vast"? :-)
Re: that string concatenation issue, how dominating is duk__vm_arith_add() in practice? Just ballpark-wise; in the pathological cases it can be something like 90+%.

Just eyeballing it, switching from size to speed optimization seemed like about a 2x difference.  I wasn't exaggerating :)
I'd have to find the post with the TypeScript profiling results again, but if I remember correctly, duk__vm_arith_add() was maybe about 30% of the execution time?  Nowhere near 90% though.

Ok, 30% sounds like a case where it's maybe doing some string concatenation using res += x but it's not one of the pathological cases like:
var res = '';
for (var i = 0; i < 1e5; i++) {
    res += String.fromCharCode(i);
}

This takes 10 seconds on this x64 laptop, and interns ~4.5GB of string data. The equivalent:
var res = [];
for (var i = 0; i < 1e5; i++) {
    res.push(String.fromCharCode(i));
}
res = res.join('');

takes 0.15 seconds and only interns ~200kB of string data.
Real code mostly falls in between (though that pathological case does happen in char-by-char accumulation). Building a result from snippets (say, XML tags) is a common case which is in between.
To be clear, I'm not saying the application code is at fault -- one might reasonably expect char-by-char concatenation to be faster than using a temporary array. But currently it does lead to pathological behavior (same as in Lua, incidentally, and where I've been bitten by it :-).

@fatcerberus If you want to see how much more performance you can achieve, you could try prepending:
--option-file config/examples/performance_sensitive.yaml

to your configure.py run.  It's not a dramatic difference but can be 10-20% too.

Thanks, I'll try that.  I'm already using fastints but I see there's a bunch of other low-level optimizations in that file, so it might be worth a try.


Data still there after dropping

Bug report
System info: [Include InfluxDB version, operating system name, and other relevant details]
InfluxDB 1.2.0, amd64. OS: Ubuntu 16.10 amd64.
Steps to reproduce:

Have a table condition with some data
Drop data with DROP SERIES and DROP MEASUREMENT commands
SELECT data again

Expected behavior:
The data returned should be empty.
Actual behavior:
Still see old data
Additional info:
Log: https://hastebin.com/orizobiboh.md
influx-debug.tar.gz

Update: The issue still there after upgrading to InfluxDB v.1.2.2
Log: https://hastebin.com/yojumiteri.md

Would you be able to backup your test_plantinghouse database and attach it to the issue?  influxd backup -database test_plantinghouse /path/to/backup/to.

Here you are:
test_plantinghouse.zip

Upgraded Ubuntu to 17.04.
The problem still exists.

Looks like the repro steps that were in hastebin are gone.  I'm not sure how repro this.  I ran a few drops and deletes on the attached database and they all worked.


:seedling: Changed cron to run everyday

Please check if the PR fulfills these requirements

[ ] Tests for the changes have been added (for bug fixes / features)
No

What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)
Feature

What is the current behavior? (You can also link to an open issue here)
Cron does not run everyday

What is the new behavior (if this is a feature change)?
With the latest fix for roundrobin token usage, the cron can run everyday.

Does this PR introduce a breaking change? (What changes might users need to make in their application due to this PR?)
None

Other information:
None

We still need more tweaks like gitcache before we take 10,000 repositories.

Should we scale the system instead ? like from 2K to like 10K repos from gs://ossf-criticality-score/all.csv ?

This should help people to get daily scorecard results. I understand there might not be many teams using this. But that should be our goal IMO.


multichar line sep on server socket

This test verifies that the sockets can operate on
multichar line seperators for calls to get.
This should already have been verified by test 5.
Somehow that test passes when this one does not.
Test 5 seems to the on the client socket.
jnthn mentioned that MOAR does not like multichar
line sepearators. But the problem seems to persist
for Parrot and JVM backend in a slightly different
manner.
I only tested this patch on Linux.

Conflicts not solved, probably best to close it.

Let us know if there's still an issue, preferably by filing a bug report.


New Metric Release: Event Location Inclusivity

This issue is created to collect comments on the Event Location Inclusivity metric
This metric can be found here: <>
CHAOSS Metric Quality Checklist
This checklist is used for new metrics to ensure we follow CHAOSS quality standards and processes.
Process

[ ] Add the new metric to release notes issue in working group repo
[x] Update the Metrics Spreadsheet to indicate that the metric is under community review.

When above steps are completed:
Content Quality

[x] Required headings are filled in
[x] If any, ensure links to other CHAOSS metrics work
[x] Optional headings that have no content are removed
[x] Contributors section lists those contributors that want to be named
[x] The name of the metric is the same in all locations

Technical Requirements

Message that the metric will be part of the next regular release is at top of page:

This metric is a release candidate. To comment on this metric please see Issue [#[put the respective Issue Number here]](URL to issue). Following a comment period, this metric will be included in the next regular release.

Metric file name is the full metric name and only contains lower case letters and hyphens (“-”) for spaces
Images are included using markdown and absolute links (as described in the metrics template)
Images have at least one empty line above and below them
Ensure images are placed in image folder and followed naming convention
Ensure tables within metric are converted as images and placed in the image folder (both original MD and screenshotted PNG format) and follow the naming convention
No HTML code in the metrics markdown.

No feedback during the period. Closing issue.


feat(backend): remove iota from backend

🍰 Pullrequest

Remove Iota from backend.
Issues

None

Todo

[X] None

Why nit just revert the merge commit? Doesnt work?

I get this message while try reverting it:
Sorry, this pull request couldn’t be reverted automatically. It may have already been reverted, or the content may have changed since it was merged.


🐛 Bug Report: Documentation error in Getting-Started causes compilation error in local

📜 Description
In the documentation page https://backstage.io/docs/getting-started/config/authentication/
under section:
Add sign-in option to the frontend
Duplicate import for SignInPage causes compilation error in local
packages/app/src/App.tsx
import { githubAuthApiRef } from '@backstage/core-plugin-api';
<-- Below import is a duplicate and causes compilation errors -->
import { SignInPage } from '@backstage/core-components';
👍 Expected behavior
Code should compile correctly after following the instructions in the documentation
👎 Actual Behavior with Screenshots
Compilation error in local:

👟 Reproduction steps

In the page https://backstage.io/docs/getting-started/config/authentication/ follow the instructions
start the app with "yarn dev"

📃 Provide the context for the Bug.
Following the documentation getting-started should run flawlessly to save time
🖥 Your Environment
Running backstage in the local on WSL2
👀 Have you spent some time to check if this bug has been raised before?

[X] I checked and didn't find similar issue

🏢 Have you read the Code of Conduct?

[X] I have read the Code of Conduct

Are you willing to submit PR?
Yes I am willing to submit a PR!

@rghara do you want to raise a PR with some suggested changes for new users that could be helpful? :pray:

@benjdlambert sure, I'll look into it.


Refactor deployment to just use docker-compose

The deploy_docker.py scripts works, but is cryptic to maintain.  Now that docker-compose can do almost everything we need, we should switch to that.  There needs to be a migration path as well.
We have a docker-compose setup in the devops directory, but this currently doesn't handle running HistomicsTK or other docker algorithms and has some other limitations.

There is a work-in-progress of this on the https://github.com/DigitalSlideArchive/digital_slide_archive/tree/docker-compose branch.

The docker-compose script can handle everything that the deploy_docker.py script can do.  If there is still substantial value in a command-line starting script, it would probably be better to emit a docker-compose yaml file and then defer to docker-compose for actions.
deploy_docker.py has some complex options, such as using a localhost version of rabbitmq, mongodb, and memcached.  It would be useful to know if any of these are being used and need to be maintained, or if we can discard some of the deployment options.


[WIP] fix: add git ignore, remove unused file

Signed-off-by: LaurenceLiZhixin<EMAIL_ADDRESS>
Add .gitignore
Remove unused file
Fix: ci

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


Added timestamp support

I added timestamp support based on moment module, so you can customize the output format

I really appreciate the effort, but I ended up doing the support for this in another commit: https://github.com/kimmobrunfeldt/concurrently/commit/2c785cb9743bc643dab83cfa248573c79d5e1549 because it solved another issue too.
I'm going to close this, but you will get a support for time stamps in next release.


Add secrets block to workflows and support more workflow triggers

Hi @npalm,
Thank you for this library! I was working on a similar tool then I found this action-docs which seems smarter than mine :)
I have a couple of suggestions, and I will be happy to implement them here:

Support workflow_dispatch in addition to workflow_call
Add secrets block for workflow_run parameters

Great to see you find this library. And love the open source mindset to work together.
Feel free to great PR's! Any improvement is welcome!


List of -Type

Please could we have a list of possible value for the field type ?
-Type Choice
-Type Text
etc

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: b38e3d82-0583-ad60-4cac-6407ce43615c
Version Independent ID: e114139d-1dad-eecd-fe0e-29f834096d7c
Content: Add-PnPField (sharepoint-pnp)
Content Source: sharepoint/sharepoint-ps/sharepoint-pnp/Add-PnPField.md
GitHub Login: @techwriter40
Microsoft Alias: kirks

@officedocsbot assign @yogkumgit

@get-itips  Thank you very much for the contribution and updating the documentation with PR. @seaSide974 Hope this update is helpful for you. Thanks for taking out some time to open the issue. Appreciate and encourage you to do the same in future also.

@officedocsbot close


Fixes for infoj entry and filter issues

Issue

A infoj entry of type:integer will not be formatted correctly, neither formatting nor rounding is applied.
This was due to a check in line 30 of numeric.mjs that was conditional on entry.formatterParams.locale. However, there was no default applied.
I also noticed that providing a maximumFractionDigit of >2 when using certain locales (e.g. en-GB) was not applied.
The numericFormatter.mjs was not checking if the type was integer so it was not formatting the value in the filter in this case.

Fixes

The default locale is set to en-GB, but can be overriden in the configuration of an infoj entry.
If a maximumFractionDigit of >2 is applied, then the minimumFractionDigit (if not provided in the configuration), is set to the same value, so that this is applied.
numericFormatter.mjs now checks if the type if integer and sets the default locale to en-GB.

How to Test
Please test the various configuration options - examples below but this may not be exhaustive.

type: integer + formatterParams.locale + filter: integer
type: numeric + round: 3 + filter: numeric
type: integer + filter: integer
type: numeric + filter: numeric

 {
      "field": "X",
      "type": "integer",
      "prefix": "£",
      "inline": true,
      "formatterParams": {
        "locale": "ar-EG"
      },
      "filter": {
        "type": "integer"
      }
    },
    {
      "field": "Y",
      "type": "numeric",
      "round": 3,
      "prefix": "£",
      "inline": true,
      "skipNullValue": true,
      "filter": {
        "type": "numeric"
      }
    },
    {
      "field": "Z",
      "type": "numeric",
      "prefix": "£",
      "inline": true,
      "skipNullValue": true,
      "filter": {
        "type": "numeric"
      }
    },
    {
      "field": "A",
      "type": "integer",
      "prefix": "£",
      "inline": true,
      "skipNullValue": true,
      "filter": {
        "type": "integer"
      }
    },

I noticed the prefixes and suffixes werent not applying so i corrected that.
Prefixes and suffixes shouldnt apply in the filter input, nor will rounding, precision or min/maximumFractionDigits.
Looks good to me otherwise.

Updated the config docs to highlight this change. All ready to merge!
https://github.com/GEOLYTIX/xyz/wiki/Workspace-Configuration#integer-1


K6 CRD object doesn't progress past started

The k6 CRD object does not progress past stage: started even after all the downstream jobs have finished.

Hi @MarkSRobinson,
I apologize for the delay in response. Thank you for opening the issue but what you described is not a bug: current implementation of K6 CRD has support for only 2 stages of runner execution, created and started. Upon re-deploy of the test, the stages loop in this way:
"" →  "created" → "started" → "" →  "created" → "started" → ...
Stages will likely be extended with the work done in #79 and #86 but right now this is the expected behavior.
I'm going to close this for now. If there's another case to look into, please feel free to open a new issue.


Allow Monolog v2.x

Hi,
Allowing both v1 and v2 of monolog will pass your tests.
Fixes #12

Removing monolog as a dependency is the better option. This package doesn't rely on anything in Monolog in the first place. At best, include it as a dev-dependency for generating a log for testing.


Trouble excluding wildcard CNAME

Bug Description
I have a wildcard domain name for my Minecraft servers, *.mc.drewburr.com, and have external dns CNAME this record out in CloudFlare. I also have this webook looking at my services and creating the same records internally. Since Unifi does not support wildcard CNAMEs and the logs are flooded with errors, I tried to exclude this domain using REGEXP_DOMAIN_FILTER_EXCLUSION: \*.* and REGEXP_DOMAIN_FILTER_EXCLUSION: \*.mc.drewburr.com to no avail. I've also tried this with a double escape and no escape, to no avail.
Could I get assistance with understanding how to properly exclude this record?
ExternalDNS Configuration
external-dns:
  provider:
    name: webhook
    webhook:
      image:
        repository: ghcr.io/kashalls/external-dns-unifi-webhook
        tag: v0.3.3
      env:
        - name: UNIFI_HOST
          value: https://<IP_ADDRESS>
        - name: UNIFI_EXTERNAL_CONTROLLER
          value: 'false'
        - name: UNIFI_USER
          valueFrom:
            secretKeyRef:
              name: unifi-credentials
              key: username
        - name: UNIFI_PASS
          valueFrom:
            secretKeyRef:
              name: unifi-credentials
              key: password
        - name: LOG_LEVEL
          value: info
        - name: REGEXP_DOMAIN_FILTER_EXCLUSION
          value: \*.mc.drewburr.com
      livenessProbe:
        httpGet:
          path: /healthz
          port: http-webhook
        initialDelaySeconds: 10
        timeoutSeconds: 5
      readinessProbe:
        httpGet:
          path: /readyz
          port: http-webhook
        initialDelaySeconds: 10
        timeoutSeconds: 5
  extraArgs:
    - --ignore-ingress-tls-spec
    - --ingress-class=traefik-internal
    - --ingress-class=nginx-internal
  policy: sync
  sources: ['ingress', 'service']
  txtOwnerId: default
  txtPrefix: k8s.
  domainFilters: ['drewburr.com']

  serviceMonitor:
    enabled: true

ExternalDNS Logs
Defaulted container "external-dns" out of: external-dns, webhook
time="2024-12-17T06:04:26Z" level=info msg="config: {APIServerURL: KubeConfig: RequestTimeout:30s DefaultTargets:[] GlooNamespaces:[gloo-system] SkipperRouteGroupVersion:zalando.org/v1 Sources:[ingress service] Namespace: AnnotationFilter: LabelFilter: IngressClassNames:[traefik-internal nginx-internal] FQDNTemplate: CombineFQDNAndAnnotation:false IgnoreHostnameAnnotation:false IgnoreIngressTLSSpec:true IgnoreIngressRulesSpec:false GatewayNamespace: GatewayLabelFilter: Compatibility: PublishInternal:false PublishHostIP:false AlwaysPublishNotReadyAddresses:false ConnectorSourceServer:localhost:8080 Provider:webhook ProviderCacheTime:0s GoogleProject: GoogleBatchChangeSize:1000 GoogleBatchChangeInterval:1s GoogleZoneVisibility: DomainFilter:[drewburr.com] ExcludeDomains:[] RegexDomainFilter: RegexDomainExclusion: ZoneNameFilter:[] ZoneIDFilter:[] TargetNetFilter:[] ExcludeTargetNets:[] AlibabaCloudConfigFile:/etc/kubernetes/alibaba-cloud.json AlibabaCloudZoneType: AWSZoneType: AWSZoneTagFilter:[] AWSAssumeRole: AWSProfiles:[] AWSAssumeRoleExternalID: AWSBatchChangeSize:1000 AWSBatchChangeSizeBytes:32000 AWSBatchChangeSizeValues:1000 AWSBatchChangeInterval:1s AWSEvaluateTargetHealth:true AWSAPIRetries:3 AWSPreferCNAME:false AWSZoneCacheDuration:0s AWSSDServiceCleanup:false AWSZoneMatchParent:false AWSDynamoDBRegion: AWSDynamoDBTable:external-dns AzureConfigFile:/etc/kubernetes/azure.json AzureResourceGroup: AzureSubscriptionID: AzureUserAssignedIdentityClientID: AzureActiveDirectoryAuthorityHost: CloudflareProxied:false CloudflareDNSRecordsPerPage:100 CoreDNSPrefix:/skydns/ AkamaiServiceConsumerDomain: AkamaiClientToken: AkamaiClientSecret: AkamaiAccessToken: AkamaiEdgercPath: AkamaiEdgercSection: OCIConfigFile:/etc/kubernetes/oci.yaml OCICompartmentOCID: OCIAuthInstancePrincipal:false OCIZoneScope:GLOBAL OCIZoneCacheDuration:0s InMemoryZones:[] OVHEndpoint:ovh-eu OVHApiRateLimit:20 PDNSServer:http://localhost:8081 PDNSAPIKey: PDNSSkipTLSVerify:false TLSCA: TLSClientCert: TLSClientCertKey: Policy:sync Registry:txt TXTOwnerID:default TXTPrefix:k8s. TXTSuffix: TXTEncryptEnabled:false TXTEncryptAESKey: Interval:1m0s MinEventSyncInterval:5s Once:false DryRun:false UpdateEvents:false LogFormat:text MetricsAddress::7979 LogLevel:info TXTCacheInterval:0s TXTWildcardReplacement: ExoscaleEndpoint: ExoscaleAPIKey: ExoscaleAPISecret: ExoscaleAPIEnvironment:api ExoscaleAPIZone:ch-gva-2 CRDSourceAPIVersion:externaldns.k8s.io/v1alpha1 CRDSourceKind:DNSEndpoint ServiceTypeFilter:[] CFAPIEndpoint: CFUsername: CFPassword: ResolveServiceLoadBalancerHostname:false RFC2136Host: RFC2136Port:0 RFC2136Zone:[] RFC2136Insecure:false RFC2136GSSTSIG:false RFC2136CreatePTR:false RFC2136KerberosRealm: RFC2136KerberosUsername: RFC2136KerberosPassword: RFC2136TSIGKeyName: RFC2136TSIGSecret: RFC2136TSIGSecretAlg: RFC2136TAXFR:false RFC2136MinTTL:0s RFC2136BatchChangeSize:50 RFC2136UseTLS:false RFC2136SkipTLSVerify:false NS1Endpoint: NS1IgnoreSSL:false NS1MinTTLSeconds:0 TransIPAccountName: TransIPPrivateKeyFile: DigitalOceanAPIPageSize:50 ManagedDNSRecordTypes:[A AAAA CNAME] ExcludeDNSRecordTypes:[] GoDaddyAPIKey: GoDaddySecretKey: GoDaddyTTL:0 GoDaddyOTE:false OCPRouterName: IBMCloudProxied:false IBMCloudConfigFile:/etc/kubernetes/ibmcloud.json TencentCloudConfigFile:/etc/kubernetes/tencent-cloud.json TencentCloudZoneType: PiholeServer: PiholePassword: PiholeTLSInsecureSkipVerify:false PluralCluster: PluralProvider: WebhookProviderURL:http://localhost:8888 WebhookProviderReadTimeout:5s WebhookProviderWriteTimeout:10s WebhookServer:false TraefikDisableLegacy:false TraefikDisableNew:false}"
time="2024-12-17T06:04:26Z" level=info msg="Instantiating new Kubernetes client"
time="2024-12-17T06:04:26Z" level=info msg="Using inCluster-config based on serviceaccount-token"
time="2024-12-17T06:04:26Z" level=info msg="Created Kubernetes client https://<IP_ADDRESS>:443"
time="2024-12-17T06:04:27Z" level=error msg="Failed to do run once: soft error\nfailed to apply changes with code 500"

Webhook Logs
external-dns-provider-unifi
version: v0.3.3 (2c45e95ebd3978e178c6c512ef0e04d747c2d9a9)

{"level":"info","ts":1734415466.741563,"caller":"dnsprovider/dnsprovider.go:45","msg":"creating unifi provider with no kind of domain filters"}
{"level":"info","ts":1734415467.0410144,"caller":"server/server.go:57","msg":"starting health server","address":"<IP_ADDRESS>:8080"}
{"level":"info","ts":1734415467.0410128,"caller":"server/server.go:44","msg":"starting webhook server","address":"localhost:8888"}
{"level":"error","ts":1734415467.6613457,"caller":"unifi/provider.go:79","msg":"failed to create endpoint","name":"*.mc.drewburr.com","type":"CNAME","error":"POST request to https://<IP_ADDRESS>/proxy/network/v2/api/site/default/static-dns/ returned 400: Invalid hostname","stacktrace":"github.com/kashalls/external-dns-unifi-webhook/internal/unifi.(*Provider).ApplyChanges\n\t/build/internal/unifi/provider.go:79\ngithub.com/kashalls/external-dns-unifi-webhook/pkg/webhook.(*Webhook).ApplyChanges\n\t/build/pkg/webhook/webhook.go:145\nnet/http.HandlerFunc.ServeHTTP\n\t/usr/local/go/src/net/http/server.go:2220\ngithub.com/go-chi/chi/v5.(*Mux).routeHTTP\n\t/go/pkg/mod/github.com/go-chi/chi/v5@v5.1.0/mux.go:459\nnet/http.HandlerFunc.ServeHTTP\n\t/usr/local/go/src/net/http/server.go:2220\ngithub.com/go-chi/chi/v5.(*Mux).ServeHTTP\n\t/go/pkg/mod/github.com/go-chi/chi/v5@v5.1.0/mux.go:90\nnet/http.serverHandler.ServeHTTP\n\t/usr/local/go/src/net/http/server.go:3210\nnet/http.(*conn).serve\n\t/usr/local/go/src/net/http/server.go:2092"}

UniFiOS Version
4.1.9
UniFi Network Version
8.6.9
ExternalDNS Version
0.15.0
Search

[X] I have searched the open and closed issues for duplicates

Code of Conduct

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

You did not search open and closed issues for duplicates.
See: https://github.com/kashalls/external-dns-unifi-webhook/issues/58
This is an issue internally within UniFi itself. A couple users have documented that this is in fact a validation error and not a backend dns error as it is perfectly capable of doing it in config if you set it.

You'll need to pass REGEXP_DOMAIN_FILTER as an ENV to exclude that subdomain. You may want to do this on external-dns's side though using --regex-domain-filter="^([a-zA-Z0-9-]+\.)?mc\.drewburr\.com$".


Cron on Debian

I had some problem to configure the cron on Debian 11 (bot installed with PyPi).
Solved with complete path to Python and Stork like this:
*/10 * * * * /usr/bin/python /usr/local/bin/pleroma-bot -c /path/to/config.yml -l /path/to/error.log 1> /dev/null

Maybe this information can be somewhere in the documentation

Note: In the terminal running pleroma-bot --help is working fine, no need of path...
This problem occurs only when configuring the cronjob

Hi!
I'm glad you were able to find a workaround!
Interesting, I've been using it on Debian 10 as a cronjob too. However I was using a virtual environment, so it's not all that relevant to your issue in particular.
In order to try to properly document this for others that may run into it, could you provide some info for me?
I'm curious, were you editing /etc/crontab directly or running crontab -e as the user that ran the pip install command?
Is there a PATH variable on your crontab and does it differ at all from what you get when running echo $PATH?
Oh, and what's your default python version?
$ which python
$ python --version

Looks like Debian 11 deprecated Python 2  but it says you can restore the /usr/bin/python symlink by installing either the python-is-python3 or python-is-python2 packages.
Perhaps the default in your system is Python 2 and the cron job is getting confused trying to find the pleroma-bot package, which was installed using Python 3.

Infos on Python:
/usr/bin/python
Python 3.9.2

This the default Python version on my system
I used pip3 command to install the bot
Result of $PATH:
/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin
I created and edited crontab with this command:
crontab -e
Hope this help

Thank you! I've added a note mentioning this on the README and on the Cron section of the documentation:
https://robertoszek.github.io/pleroma-bot/gettingstarted/automateit/#cron

NOTE: If you have issues with cron running the bot you may have to specify the full path of your Python executable
*/10 * * * * /usr/bin/python /usr/local/bin/pleroma-bot

Let me know if you run into any other trouble! I'll close this for now.
Feel free to re-open this or open a new issue if needed,
have a nice day!


Remove jest-cli from before_install (global) and use npm test instead of jest

Travis CI before_install uses npm install -g grunt-cli jest-cli which is installing<EMAIL_ADDRESS>.
Travis CI build will fail current tests<EMAIL_ADDRESS>$ jest
/home/travis/.nvm/v0.10.40/lib/node_modules/jest-cli/bin/jest.js:12
const fs = require('fs');
^^^^^
SyntaxError: Use of const in strict mode.

To make it consistent, we should just npm test and use jest from package.json instead of using jest-cli installed from npm install -g  jest-cl.
See Travis CI build error here -  https://travis-ci.org/GriddleGriddle/Griddle/builds/89018241

Awesome! Thanks!


Add support for UI themes/skins to Pàdé Converse

At the moment, Pàdé Converse has only a single UI theme/skin (black ink and white paper). Enable third-party themes/skins to be added via CSS by doing the following:

Add a new option to allow named themes to be selected.
Add same named css files to the plugins/css folder
Add code to dynamically load the selected css file after the main converse.css file when starting Converse.

Converse supports theming via CSS properties.
https://conversejs.org/docs/html/configuration.html#theme
There's no UI for choosing a theme though, it would be great to have one ;)

There's no UI for choosing a theme though, it would be great to have one ;)

Unfortunately, I am reusing the Browser Extension options UI for this and that will not translate into reuse directly from Converse. The CSS variables however, have been very useful in creating new themes with CSS that overrides converse.css.

This in now available in version 0.8.8.
There are 4 themes to choose from.

Add a new theme/skin involves a PR with new CSS file and a modification to append the theme/skin name to the UI list in manifest.js


Build container images for all architectures and publish multi-arch manifests

We currently build and publish only amd64 images. There are other popular architectures, e.g. arm64, that we should support.
At least CAPM3 and IPAM already supports building for multiple architectures easily with make docker-build-all. BMO and Ironic-image will need some work though.
The script for building must also be updated so that it builds multiple architectures and then creates a multi-arch manifest from them.

[ ] Support multiple architectures in BMO: https://github.com/metal3-io/baremetal-operator/issues/1715
[ ] Support multiple architectures in IPAM: https://github.com/metal3-io/ip-address-manager/issues/517
[ ] Build container images for multiple architectures and publish multi-arch manifests (this issue)

There is https://github.com/metal3-io/ip-address-manager/pull/507 from week ago btw

Thanks, I had not seen that!

Hmm that means we have some work to do in CAPM3 also. It was actually a mistake that I put IPAM above. I meant to put ironic-image, which needs a bit more work I think. I'll update.


Parse Log with hostname instead of ipv4/ipv6

I have a log-file where the ip is already resolved and instead of ip has a hostname as a field.
It seems this field is not understood by goaccess. As I understand the hostname field is manadatory, but is it really a requirement to hold an IP?
Token 'some.perfect-valid-hostname.example.com' doesn't match specifier '%h'
Does someone have an advice?

digging into the source I spotted --no-ip-validation  which seems to do the trick

That's right. --no-ip-validation will do the job, still you will need to specify %h where the ip has been resolved.

Closing this. Feel free to reopen it as needed.


How can I install normally?

Hello,
I don't use NPM and how can I installation?

You must use npm. It’s a node project.


Ability to perform Soak tests

This might already be possible I just can't see it documented.
It would be useful to perform Soak tests (http://en.wikipedia.org/wiki/Soak_testing) using gatling.
I think this would require being able to run repeat a scenario either X times, for Z amount of time, or indefinitely with the ability to stop the tests, and then create the reports.

Hi Steve,
You can build loops and iterate :

a given number number of times --> times
a given period of time --> during
with a custom criteria --> asLongAs

https://github.com/excilys/gatling/wiki/Basic-Usage --> Repeating actions
https://github.com/excilys/gatling/wiki/Advanced-Usage#wiki-conditional-execution
If you want to run indefinitely, you can either use times(Long.MaxValue)
or asLongAs(true) and kill the process when you want. Then you can generate
the reports : launch Gatling with the following option : "-ro
<timestamp=name of the folder containing the simulation.log generated by
your run>"
https://github.com/excilys/gatling/wiki/Advanced-Usage --> Options in
Gatling
Cheers,
Stphane
2012/2/11 Steve Mosley <
<EMAIL_ADDRESS>

This might already be possible I just can't see it documented.
It would be useful to perform Soak tests (
http://en.wikipedia.org/wiki/Soak_testing) using gatling.
I think this would require being able to run repeat a scenario either X
times, for Z amount of time, or indefinitely with the ability to stop the
tests, and then create the reports.

Reply to this email directly or view it on GitHub:
https://github.com/excilys/gatling/issues/410

Thanks for the quick response. That sounds like it would do the trick right now.
But maybe a future enhancement would be to allow that repetition to be defined at the scenario level? ... just that way the same scenario's could be used for either short or long tests.
Thanks,
Steve

Hi Steve,
Actually, you don't reuse scenarios, you reuse chains, that you can combine
however you want:
https://github.com/excilys/gatling/wiki/Advanced-Usage#wiki-modularization
For example, you could have 1 common included file with 2 simulations:

included file :

val myChain = chain.exec(...).pause(...)....

simulation 1

include("myChainDefinition.txt")
val scn = scenario("My Scenario")
.loop(myChain).times(5)

simulation 2

include("myChainDefinition.txt")
val scn = scenario("My Scenario")
.loop(myChain).times(100)
Cheers,
Stephane
2012/2/12 Steve Mosley <
<EMAIL_ADDRESS>

Thanks for the quick response. That sounds like it would do the trick
right now.
But maybe a future enhancement would be to allow that repetition to be
defined at the scenario level? ... just that way the same scenario's could
be used for either short or long tests.
Thanks,
Steve

Reply to this email directly or view it on GitHub:
https://github.com/excilys/gatling/issues/410#issuecomment-3925830

Ah thanks, exactly what I needed ... I guess this is a non-issue then.
Cheers,
Steve


Quality profile name not valid

I get this error althoug in radarr the profile is named exactly the same (2160p_TRaSH).
[WRN] The following quality profile names are not valid and should either be removed or renamed in your YAML config [WRN] ["2160p_TRaSH"]

@Palle34 what was the solution to the problem? Just in case others come across this in the future.

There was no real problem..
It was simply my stupidity, as I used a port from another Radar instance ;D

Im seeing this on the latest version of Sonarr <IP_ADDRESS>4. Nothing has changed besides an update to Sonarr. Looks like recyclarr removed the custom formats associated with the quality profiles.
@rcdailey any tips to resolve this?

My issue was resolved by removing all existing custom formats and having recyclarr resync them. Not sure why this happened post upgrade but just an FYI for others if they come across this.

@l3uddz you should have opened a new issue instead of posting on an existing one. The whole point of the issue template is to gather the necessary information from you to allow me to answer your question.
If you still have your old cache and logs around, please open a new issue and I'm happy to look.


Page action should toggle entries to reading list

The page action button should ideally be able to remove a webpage from the reading list if it has already been added.
Also this is a great addon. I really like the idea of keeping it in a native style which is why I downloaded it. Thank you for making this addon.

I did this logic at the very beginning, but then I changed it - now clicking on the icon hoist a link in the list. I did it because it isn't possible to show in address bar the actual status (icon always in initial state) - so I had accidentally removed some of my articles.
I think about adding some indicator to show visited link in list. And maybe show some popup in opened link (but it requires access to all pages :( )

Oh, I see the problem. Maybe you could circumvent this issue by having two page actions? The current default (not added state) only shows when a url is not yet in the list and a new one (added state) only when it already is in the list. Would this be possible instead?
Also the read indicator in the list would be nice too. Would make it more like in Firefox for iOS.
And about the popup, the feature as well as the permission could be optional. This way the addon would be less scary for users.


open ue424 crash

when double click
ShooterGame\Content\Characters\HeroFPP\HeroFPP.uasset

i see,it's a good job if also run on editor asserts.

Yeah, I know. I'm working on that, but always interrupted with something else.

ShooterGame is Valorant so it should be cooked

ShooterGame is a free example at Epic Games Store. I use it myself for testing UModel's compatibility with new engine stuff.

Oh then Valorant uses the same name, sorry

Some UE3 games has "UDKGame" folder inside. It just depends on laziness/business of developers. Valorant devs used ShooterGame as a base and didn't get rid of that folder yet.


Land detector fixes

@AndreasAntener The changes here do not correlate (yet) with your issue and are merely style fixes.

No change with this, as expected I guess.
It's hard to pin down, but we think that it might have something todo with the motion on spinning up the props while arming. It didn't happen when we had no props on


(Poedit 3 beta) Separator in the splash screen is transparent

STR:

Open Poedit 3 beta 6345 (tested on Windows 10)
See the separator between the "Welcome to Poedit" and the recent file panel

AR:
It's transparent and I can see though it whatever there is behind that Poedit instance
ER:
Should probably be a solid color and not transparent.

Thanks; must be some off-by-one in RTL situation only. Took me a while to decode STR/AR/ER, but got there eventually :)

Current state of my understanding: this is some interaction between Poedit's TitlelessWindow and Widget's control used for recent files. In  wxDataViewMainWindow::OnPaint() it uses wxAutoBufferedPaintDC and this somehow doesn't copy the 1px boundary from the buffer. Changing the DC to plain old wxPaintDC works as expected.
Disabling TitlelessWindow fixes it too. Selectively removing its adjustments of origin and client area has no effect, but disabling handling of WM_NCCALCSIZE helps. This indicates that TitlelessWindow doesn't do enough to adjust client area, and it breaks something deep in wx or win32 handling of to-screen blitting (which wxBufferedDC does).
How RTL (broken) vs LTR (fine) comes into it is completely beyond my understanding 🤷♂

It turns out to be unrelated to Poedit 3 or even the welcome window — except that it makes the issue more visible. If you look in the Crowdin open window, you can see (in 2.4.3 too) that the selection in the list control, which is the same wxDataViewCtrl used for the recent files, is cropped off. The selection rectangle should have a darker outline as it has in LTR, but RTL the right side is cropped by 1px and the darker outline is missing:

→  https://trac.wxwidgets.org/ticket/19190

Do you know what graphics card you use? It may possibly be related to graphics drivers. (I reproduced inside Parallels VM, I assume you use Poedit on actual Windows hardware.)

Do you know what graphics card you use? It may possibly be related to graphics drivers. (I reproduced inside Parallels VM, I assume you use Poedit on actual Windows hardware.)

I have an on board intel graphics. Not sure about the exact one but I do know its driver is the most recent (or fairly recent at least. I keep its driver up to date on a regular basis). I can get the exact model if you'd like.
And yes the Windows is running on actual hardware.

This appears to be fixed.


Define page which is automatically loaded after successful login (for current version of Serene)

Hi all,
I would like to configure the page which is automatically loaded after successful login. By default, this is going to "/" which will go to the DashboardController and load the dashboard.
At the moment, I just have changed the return View(...) statement within the DashboardController Index() method to return the desired url - like this:
        public ActionResult Index()
        {
            //return View(MVC.Views.Common.Dashboard.DashboardIndex, new DashboardPageModel());
            return View("~/Modules/Content/Content/ContentIndex.cshtml");
        }

This works, but is there a better way?
With kind regards,
John

@belalAhmed
How are you passing the URL you want the user to land on ?
Can you show that code ... perhaps this should be a WIKI when it is done .. because I want some people to land on one 'sensitve' page and other users to land on a common page.

Hi @belalAhmed ,,
thank you for your feedback. I already looked at the button code - but I don't understand how to set the "returnURL" on the server-side. As @stixoffire already commented - Could you please elaborate how to set returnURL within the action response in the xyzEndpoint.cs?
Many thanks for your help.
With kind regards,
John

Hi all and @stixoffire
I think I have figured it out:
within AccountPage.cs, create a new class for the custom Serviceresponse:
    public class LoginServiceResponse  : ServiceResponse
    {
        public string redirectURL { get; set; }
    }

Also within AccountPage.cs, modify the Login ServiceResponse to be like this:
public Result<LoginServiceResponse> Login(LoginRequest request)
        {
            return this.ExecuteMethod(() =>
            {
                request.CheckNotNull();

                if (string.IsNullOrEmpty(request.Username))
                    throw new ArgumentNullException("username");

                var username = request.Username;

                if (WebSecurityHelper.Authenticate(ref username, request.Password, false))
                {
                    var sr = new LoginServiceResponse();
                    sr.redirectURL = "/Content/Content";   // *** Your custom redirect target URL goes here... ***
                    return sr;
                }

                throw new ValidationError("AuthenticationError", Texts.Validation.AuthenticationError);
            });
        }

And within AccountLogin.cshtml, change the LoginButton click function to be like this:
$('#LoginButton').click(function (e) {
	e.preventDefault();
	if (!_this.validateForm()) {
		return;
	}
	var request = _this.getSaveEntity();

	Q.serviceCall({
		url: Q.resolveUrl('~/Account/Login'),
		request: request,
		onSuccess: function (response) {
			var q = Q.parseQueryString();
			var returnUrl = q['returnUrl'] || q['ReturnUrl'];
			if (returnUrl) {

				// *** The standard returnURL querystring parameter is '/' so if it is something else, then you were logged out at some other page and after login again you must be redirected back to that page ***
				if (String(returnUrl) != '/') {

					var hash = window.location.hash;
					if (hash != null && hash != '#')
						returnUrl += hash;
					window.location.href = returnUrl;
				}
				else {
					// *** Standard returnURL queryString parameter, redirect to custom url given from server-side ***
					var redirectURL = response.redirectURL;
					var hash = window.location.hash;
					if (hash != null && hash != '#')
						redirectURL += hash;

					window.location.href = Q.resolveUrl(redirectURL);
				}

			}
			else {
				// *** in case returnURL is null, redirect to custom url given from server-side ***
				var redirectURL = response.redirectURL;
				var hash = window.location.hash;
				if (hash != null && hash != '#')
					redirectURL += hash;

				window.location.href = Q.resolveUrl(redirectURL);
			}
		}
	});
});

This should do it.
Hint: As I never have waited until I got logged out at an arbitrary url within the SF app, this is not yet fully tested.
Hope this helps somebody.
With kind regards,
John

Have quickly put this into the following Wiki article: https://github.com/volkanceylan/Serenity/wiki/Define-page-which-is-automatically-loaded-after-successful-login-dynamically-from-server-side
Regards,
John

Additional info: I found, that this works fine when the project is deployed into the root of the webserver directory. So far I was not able to make this working when the destination of the Serenity project on the webserver where it is being deployed is a subdirectory.
If somebody has a solution which works dynamically both within the visual studio environment and being deployed into a subdirectory of a webserver, please let me know and I will update the wiki.
With kind regards,
John

@JohnRanger just a question for you regarding the path assignments I notice you are using "/"
or "/content/content" Notice AccountLogin has ~/AccountLogin
The difference is one is referring to the site root and the other refers to application application .
So I am thinking, but have not tested ; is the solution is really to supply application root ~ for your redirects .
Rather than type up something nice - here is a nicely typed answer on StackOF.
/ Slash vs  ~/ Tilde Slash


Image not being resized

When performing a resize, then an extract and then an resize again, the final image is not being resized(the actual image is resized, but not the dimensions)
Clone my repo and test https://github.com/hornta/sharp-resize-bug
const sharp = require('sharp');
sharp('dog.jpg')
  .resize(50)
  .extract({
    left: 0,
    top: 0,
    width: 25,
    height: 25
  })
  .resize(1000)
  .jpeg()
  .toFile('bigDog.jpg');

bigDog.jpg results in a 25x25 pixels image when it should end up with a 1000px wide image

Hello, if you want to resize to 50 pixels wide, extract the top-left 25x25 region then resize to 1000 pixels wide, try something like the following:
sharp('dog.jpg')
  .resize(50)
  .extract({
    left: 0,
    top: 0,
    width: 25,
    height: 25
  })
  .toBuffer()
  .then(data => sharp(data)
    .resize(1000)
    .toFile('bigDog.jpg')
  );

Is there a reason I have to do it like that? Why is not my example working? I think most people expect it to work like my example

Although resize can be called multiple times to set/update options, the current implementation supports one resize operation per instance.
You may be interested in a discussion about an alternative API at #241.