UI: image with Absolute position don't render correctly

Bevy version
main since https://github.com/bevyengine/bevy/pull/10690
What you did
Run example game_manu or overflow_debug
What went wrong
Images that have position_type: PositionType::Absolute in their style are not displayed correctly

@nicoburns any idea?

Hmm... no that's weird. It looks like it's about the right size in the x axis but it's getting stretched by about a factor of 2 in the y axis :/

That because width is set, but height isn't. If you remove width from style, you'll see  big icons with correct proportions.

@mockersf I set height same as width and got

correct result. Is this change good enough solution, or someone should dig deeper and figure out why taffy changed behaviour?

For overflow_debug same thing: height is set, but width isn't. If add width: Val::Px(400.0),, then result

looks correct

But if this solution get approve, then I think we should add some notes to Migration guide, because such things could be in bevy users code

Or better add some check/warning in bevy_ui::layout::convert::from_style(...) -> taffy::style::Style

Is this change good enough solution, or someone should dig deeper and figure out why taffy changed behaviour?

No, images should keep their ratio when you set only one of their dimension

@mockersf The problem hides in taffy, because at some point size fills None by measured_size which is done componentwise. So there Size(Some(45), None) becomes Size(45, 100) which have no correct ratio.

I think the issue here is with the image measure function. It ought to read the image node's Style and use the size, min_size and max_size as part of it's computation. For most (all?) other "leaf" nodes this is automatically applied by Taffy, but that doesn't work for images because of their inherent aspect ratio.

I agree with your desire for universal approach, but IMO min_size and max_size is for clamping but not for ratio description, if there is width or height set only, image should be abble grow or reduce unlimitedly but proportionally. I read the contribution.md page of taffy, it says that behaviour must be similar as in chrome css render. That's why IMO we should make html example first (test?) for taffy and reproduce this issue. If chrome behaves as we expect, then this is the basis for creating a well-formed issue in taffy project.

I didn't notice that bevy doesn't inform taffy about node types, that's why taffy can't make correct decision about size behaviour. I think Style::aspect_ratio is correct approach, I will make PR based on it.

@mockersf May I ask you for a review?

@alice-i-cecile Just wanted to call attention to this issue. It's a really not great regression which I believe it caused by the interaction between Taffy and Bevy's measure function. I don't think the fix should be too hard (either need to expose taffy's Style to the measure function or inherent aspect ratios to Taffy - the former is probably an easier quick fix), but it's a bit of an awkward one because I think it will require breaking changes in Taffy and then another Taffy upgrade in Bevy.
(but luckily Taffy doesn't have any other unreleased changes sitting around, so the upgrade should be quite straightforward).
What kind of release date is 0.14 targeting?

We're looking at 1-2 weeks for the release candidate. I agree that this is important to resolve.


Make client APIs extensible.

fix #206
Changes:

Make public methods virtual
Change private transport client and API path to protected

It would also be beneficial to allow extending the HttpApiTransport but I think that deserves its own issue/pull request.

Looks good, I noticed some private members that we might want to make protected, as you have done for some of the client classes but not all. I stopped leaving comments but worth checking:

DatabaseApiClient
GraphApiClient
DocumentApiClient

Also TransactionApiClient

Update:

Rebased on master to resolve conflicts.
Add protected keyword to remaining variables missed in previous commit
Make ArangoDBClient extensible. Marked API clients properties as virtual and transport variable as protected.


[FLINK-12343]add file replication config for yarn configuration

What is the purpose of the change
This pull request add file replication config for Flink yarn configuration. This is helpful to accelerate the bootstrap time of containers when Flink applications needs 100+ containers by changing the default replication number (such 3 for hdfs).
Brief change log

Add a file-replication in YarnConfigOptions
Apply the file-replication  value for each of  application master local resources
Apply the default file replication value 3 for task manager local resources

Verifying this change
This change is a trivial rework / code cleanup without any test coverage.
Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): (no)
The public API, i.e., is any changed class annotated with @Public(Evolving): (no)
The serializers: (no)
The runtime per-record code paths (performance sensitive): (no)
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Yarn/Mesos, ZooKeeper: (no)
The S3 file system connector: (no)

Documentation

Does this pull request introduce a new feature? (yes)
If yes, how is the feature documented? (docs)

@rmetzger @tillrohrmann
Please have a look this PR when you have time.

I tend to disagree. I think the problem is well defined, the configuration parameter name is okay (not great) and the idea of how to solve the problem is also good.
However, there are some issues with the implementation approach: The replication factor is set with each call of the utility. I would rather set it once globally.
Secondly, the code seems to use a default value (3), potentially ignoring a global configuration param set by HDFS.
@flinkbot approve description

Sorry for having to push back here. The current implementation should not work because FileSystem#setReplication only works on existing files. Thus, this PR should effectively be a no-op. If you move the setReplication call after the copyFromLocalFile, then it only changes the replication afterwards.
The underlying problem imo is that we have not agreed on a proper way to solve this problem which should be part of the JIRA discussion. By first agreeing on the overall approach we save a lot of time reviewing PRs which don't work. I hope that this will soon be properly reflected in the contribution guidelines @rmetzger.
One solution could be to temporarily overwrite the dfs.replication value in the Hadoop configuration to a smaller value. However, we must make sure that we properly reset this value. Morever, we definitely need tests to validate the solution.

Having looked more at this, I agree with you. We need to go back to step 1 and discuss the overall approach in JIRA.
setReplication is a no-op, because the file doesn't exist at the time of the call.
I agree that setting the dfs.replication factor is a good direction to consider.
Also, we need to carefully think through the files we want to be affected by this setting. I guess all files uploaded by the yarn client and application master, but not checkpointing or any other files written by runtime operators.
@HuangZhenQiu Can you look into all these aspects and propose a solution in JIRA?

@rmetzger @tillrohrmann
I think we can set the hdfs.replication in YarnConfiguration of AbstractYarnClusterDescriptor. As, this configuration is only used in client side, so will not impact the runtime file replications. The reason I initially choose to use the setReplication method is that our org will use S3 for long term to submit job to different cluster management system. It looks S3AFileSystem doesn't implement the method. I think it is good to use hdfs.replication initially. How do you think?

Hi all,
Actually, I think it might be a good idea to change the replication with setReplication in each call of the utility after uploading the file with copyFromLocalFile.

With larger replication, uploading files may takes longer to create all the replicas. On the other hand, if we set the file replication afterwards, the dfs will create more replicas without blocking the client.
It is easy to control which files should be affected while preserve the dfs default replication for the others, by passing an argument into the utility.

FYI, MapReduce does the similar thing also in this way.

@xintongsong
Thanks for the feedback. Actually, I also tested current solution in which the replication number is set in the yarn configuration for the filesystem. It doesn't work. I will change the revision back by using the replication argument in the utility function. Once it is verified, I will update the PR.

@xintongsong Thanks a lot for taking a look at the PR.
Isn't it a good thing (in this case), that the copyFromLocalFile blocks until the file has been replicated? If it happens asynchronously, we might run into a situation where the files are not yet replicated, and the deployment of the YARN cluster won't benefit from a higher replication.

It is easy to control which files should be affected while preserve the dfs default replication for the others, by passing an argument into the utility.

Of course, we should only set the replication factor of the files uploaded by the YARN client and AM, needed for the deployment, not the operation of Flink.
This should be validated through a test in Flink.

@xintongsong  @tillrohrmann @rmetzger
Thanks for the informative comments. The verified the solution end to end. It works. I will continue with adding an integration test in flink-yarn-tests module in the PR later.

@HuangZhenQiu What's the state of the test?

@zentol Working on it this weekend. I will push a change soon.

@rmetzger @tillrohrmann
I was blocked by setting up the secured MiniDFSCluster in integration tests, so took a little more time on the PR. Please review it at your most convenient time.

@xintongsong @wangyang0918
Thanks for your suggestions. I changed the PR accordingly. Please kindly review it.

Thank you @HuangZhenQiu , I think this PR looks really good now. Will try reach a committer to get this in.
@tillrohrmann , would you please check on this?

@HuangZhenQiu
My bad, there is a test failure. Would you please take a look.

@xintongsong
It is wired that the default value of  dfs.replication  in YarnConfiguration  is 1. The setting will not be used for starting MiniYarnCluster, in which the dfs.replication will be set as 3 by default. It caused the error. Thus, I have to set dfs.replication  in YarnConfiguration to make the test case pass.

@xintongsong
It is wired that the default value of dfs.replication in YarnConfiguration is 1. The setting will not be used for starting MiniYarnCluster, in which the dfs.replication will be set as 3 by default. It caused the error. Thus, I have to set dfs.replication in YarnConfiguration to make the test case pass.

Hi @HuangZhenQiu
I'm afraid we cannot simply set the dfs.replication in YARN_CONFIGURATION.  We have to find the root cause why the replication of ResourceManager#createTaskExecutorContext is different from flink client. They should have the same configuration. I will help to find out the root cause.

Hi @HuangZhenQiu
After investigation, i got the following conclusions.

The the default value of dfs.replication is 1 just because MiniDFSCluster only start 1 datanode. So the replication in configuration is updated to 1 by MiniDFSCluster.
The settings of dfs.replication is not used by MiniYarnCluster and flink application. Because you write the hdfs configuration to core-site.xml, it should be hdfs-site.xml. And also hdfs-site.xml should be added to local resource just like yarn-site.xml so that all containers(jm/tm) could access it.

What do you think?

@wangyang0918
Thanks for the investigation. I will verify it and make change accordingly tonight.

Hi @tillrohrmann,
FYI

FileSystem#SetReplica() happens asynchronously. The blocks will be populated or removed in the background. You could check the ClientProtocol.java#L276 for details. Also the hdfs client implements an internal waitForReplication to support block call.
A datanode could serve multiple files downloading from nodemanager. Generally, the bandwidth becomes a bottleneck. So increase the replication could reduce the localization duration.

Ah I think I misunderstood the problem the whole time. @HuangZhenQiu did you intend to increase the replication factor so that the download bandwidth is higher? I always thought that you wanted to decrease it so that the replication finishes faster. If this is the case, then it makes sense to do like you proposed.

@tillrohrmann @wangyang0918  Yes. Thanks for pointing out theoretic feasibility. We do this for a use case that needs 800 containers for backfill with streaming mode. So both the starting TM executor with Flink RM and Jar downloading from hdfs. We internally tried this change, and find the reduce the start time as the jar download parallelism is increased. We also waiting for this https://issues.apache.org/jira/browse/FLINK-13184 to merge.  BTW, build always fail due to some unstable test cases. Let me rebase master and try again.

I think there is a port conflict in the newly added test. Ideally we would start the DFSMiniCluster with a random port. Moreover, there are still unaddressed comments which would be good to resolve @HuangZhenQiu.

@xintongsong @tillrohrmann @rmetzger
Would you please take one more round of look? I found secured test cases will fail due to timeout after creating DFS, thus I simply only enable MiniDFS for YarnITCase.


Add $data parameter to deleteList

This PR is one of a set.  All PRs for this set listed here:
https://github.com/zfcampus/zf-rest/pull/59
https://github.com/zfcampus/zf-content-negotiation/pull/29
https://github.com/zendframework/zf2/pull/7140
Together these PRs will add support for deleteList($data) where it was deleteList()
This PR will be the tracker for the set.
https://github.com/zendframework/zf2/pull/7140
does not need to be merged for this PR to work.  However, revisiting this in the ancestor is a good idea.
https://github.com/zfcampus/zf-content-negotiation/pull/29
may be merged safely.
https://github.com/zfcampus/zf-rest/pull/59
uses an optional $data argument and may be merged safely.  Override for onDispatch for delete without an id is included.

This PR and https://github.com/zfcampus/zf-content-negotiation/pull/29 are ready for merge.

Can I get some feedback or movement on this PR?

https://github.com/zendframework/zf2/pull/7140 is also ready for merge.

@TomHAnderson I merged all the PRs related to this one:

https://github.com/zfcampus/zf-content-negotiation/pull/29
https://github.com/zendframework/zf2/pull/7255


SqueezeDet is SSD on one layer

Recently, I compared the detection pipeline of SqueezeDet and Single Shot Detection(By Wei Liu https://arxiv.org/abs/1512.02325). I found that squeezeDet is equal to the  SSD only applied on one feature layer, basically. There are some minor difference except this but really I found nothing more.
@BichenWuUCB What is your opinion? Can you spot some more difference? Thank you.

@ymshao Thanks for your question.
Since SqueezeDet paper and code repository were released, we received several questions about comparing SqueezeDet with SSD, YOLO, etc. I'd like to take this chance to provide a response to these questions and share some thoughts.
The short answer is, yes, they are similar.
And in case someone is interested in a longer answer:
Our work was initially inspired by YOLO -- we used a one-stage detection pipeline to ensure fast inference speed. And to reduce parameters in fully connected layers, we used a convolutional layer as output for both region proposal and classification.  By the time the work is done, we thought that this was a novel idea. But later, we saw SSD from ECCV and we realized that SDD used an idea similar to ConvDet and it's been published on arxiv much earlier. We respect the originality of SDD using convolution as detection output layer. For a heated research topic like object detection, low hang fruits are usually picked much earlier than one would expect.
But here is a more important point. The trend of research in object detection has been primarily focusing on pushing the state-of-the-art accuracy higher and higher. Meanwhile, however, problems such as inference speed, energy efficiency, model size, etc. are not adequately addressed. These are critical problems if we want NN models to get out of computer vision lab's workstation GPUs and be deployed on portable devices such as autonomous vehicles, mobile phones, etc.
So, the motivation for us to build SqueezeDet is to design a model deployable on embedded devices. All our design decisions are focusing on how to make it faster, how to reduce model parameters, how to reduce memory footprint, how to reduce energy consumption while keeping the same level of or even better accuracy. We are happy that after integrating many efficient design ingredients (SqueezeNet, one-stage detection, ConvDet, etc.), SqueezeDet achieved significant progress in all above aspects. Within less than half a year, SqueezeDet has been adopted by a lot of mobile apps, embedded vision processors, etc. That's what we are really happy to see.

@BichenWuUCB  Thank you for such a comprehensive explanation, it really help a lot. You are right object detection is a so hot research subject that usually people are really testing and do experiments on very similar ideas. But only the one who can obtained the best accuracy (or release the code and paper earliest) get credit. The point is to make it work. And your code is not only work but very elegant, and I reproduced similar results as stated in the paper.
I believe the researchers in this area will pay more attention on the speed, energy cost, model size in addition to accuracy. And you work is really a very good start.
Thank you for your reply again.


HOW TO ADD ANIMATED ICONS (source)

Describe your problem here.
I NEED THIS FOR MY PORT
Are you modding a build from source or with Lua?
Source
What is your build target?
Windows x64
Did you edit anything in this build? If so, mention or summarize your changes.
No, i did not.

ehhhhhhhhhhhhhhhhhhhhhhhhh ill just use a lua script and put it in mods/scripts lol

Bro why did you close it then reopen it if you solved your own question...

i did it so if anyone KNOWS how tp do it in source, THEY CAN TELL ME.

iconP1.frames = Paths.getSparrowAtlas('icons/[ICONNAME]')
iconP2.frames = Paths.getSparrowAtlas('icons/[ICONNAME]')

iconP1.animation.addByPrefix('normal', 'normal', 24, true);
iconP2.animation.addByPrefix('normal', 'normal', 24, true);

I might be a Lua person, but I know how to make animations with Haxe.

Should I put them in PlayState?

Yes.

iconP1.frames = Paths.getSparrowAtlas('icons/[ICONNAME]')
iconP2.frames = Paths.getSparrowAtlas('icons/[ICONNAME]')

iconP1.animation.addByPrefix('normal', 'normal', 24, true);
iconP2.animation.addByPrefix('normal', 'normal', 24, true);

I might be a Lua person, but I know how to make animations with Haxe.

What is P1 And P2 mean?

IconP1: the player's icon
IconP2: the opponent's icon

in what line do i put his on for psych engine 0.6.3?


New package: AlgamesDriving v0.1.0

Registering package: AlgamesDriving
Repository: https://github.com/simon-lc/AlgamesDriving.jl
Created by: @simon-lc
Version: v0.1.0
Commit: 39128b9750d223cf37e32482ec6586ff4968d2b4
Reviewed by: @simon-lc
Reference: https://github.com/simon-lc/AlgamesDriving.jl/commit/39128b9750d223cf37e32482ec6586ff4968d2b4#commitcomment-46672839

@simon-lc the package can't be loaded :point_up:
[noblock]


Revisit pagination in TransactionDao.find

We do not have transaction index information for transparent transactions, so we can't guarantee that we can compare transactions by index within a block. In https://github.com/zcash/ZcashLightClientKit/pull/1275 we make it so that if the find api is used to query multiple pages, there may be duplicates if a user receives multiple transparent transactions in the same block; in a future release it would be good to reconsider how pagination is performed here and make sure that we can generate a stable order in all circumstances.

The issue described in https://github.com/zcash/ZcashLightClientKit/pull/1275#discussion_r1330736655 should be addressed when we fix this issue.


Store OAuth tokens encrypted on disk

Currently the authorization and refresh tokens are stored in plaintext on disk, creating a token compromise risk if an attacker gains access to the database where information is persisted across application restarts. This could lead to compromise of PII for any account whose tokens were accessed this way.
This is unlikely but easy to deny; API access secrets are already being handled so adding an encryption key to the secrets is a trivial expansion and adding encryption/decryption steps to the disk operations is not difficult.

fixed in https://github.com/axleantilles/EDFleetTracker/commit/c4f8ff032dfe167ead658f79b733eab230dfa76f


Enhancement:  Icon Notification

This notification would have to show like a small number on it, to show that there is a note taken when you re-open a specific URL for site specific URLs.
The small number would look like just as the HTTPS Everywhere addon icon has when it encrypts sites, it has numbers on the blue icon S.
Thank you so much for such an awesome addon!!! this is such a time saver and organizer!

Also, maybe you can allow bullet points to be activated by default if the user wishes.  And each bullet point will be the number of notes taken which will = the icon numbers shown.
eg.
if there are 4 bullet points in a site specific URL, then when checking the icon, it will show the small number 4 in the top right or bottom right corner. To show that 4 notes have been taken for that specific site URL.

Thanks for the suggestion. I worked on it a bit, and I think it's going to be a very useful feature. It will definitely be added soon.

wow thats really awesome and useful!!

Notification badges have been added in version 1.1.2, now available on AMO. They're disabled by default, so you need to activate them on the options page.

I see that it works great! Awesome thank you


feature(multi_object_tracker): add acc estimation model

Description

This is just test development for prediction.
I added acc estimation to any classification tracker except Unknown object.
Related links

Tests performed

Notes for reviewers

Interface changes

Effects on system behavior

Pre-review checklist for the PR author
The PR author must check the checkboxes below when creating the PR.

[x] I've confirmed the contribution guidelines.
[x] The PR follows the pull request guidelines.

In-review checklist for the PR reviewers
The PR reviewers must check the checkboxes below before approval.

[ ] The PR follows the pull request guidelines.
[ ] The PR has been properly tested.
[ ] The PR has been reviewed by the code owners.

Post-review checklist for the PR author
The PR author must check the checkboxes below before merging.

[ ] There are no open discussions or they are tracked via tickets.
[ ] The PR is ready for merge.

After all checkboxes are checked, anyone who has write access can merge the PR.

I believe there was a problem with the position vibrating when acceleration was added. Has that been resolved?

Has that been resolved?

I just add acc estimation model and did not tune system noise.
But after last parameter tuning PR in odaiba, position vibrating seems suppressed in both rosbag and simulation.
Of course we need fine degradation check before main branch merge, it seems sufficiently work for test dev  branch now.


Add huggingface report template

Description
Add details and examples for markdown report.
Initiated a new template specifically for huggingface.
Before

After

demo: https://huggingface.co/spaces/ZeroCommand/test-giskard-bot/discussions/11
Related Issue

Type of Change

[ ] 📚 Examples / docs / tutorials / dependencies update
[ ] 🔧 Bug fix (non-breaking change which fixes an issue)
[x] 🥂 Improvement (non-breaking change which improves an existing feature)
[ ] 🚀 New feature (non-breaking change which adds functionality)
[ ] 💥 Breaking change (fix or feature that would cause existing functionality to change)
[ ] 🔐 Security fix

Checklist

[x] I've read the CODE_OF_CONDUCT.md document.
[x] I've read the CONTRIBUTING.md guide.
[ ] I've updated the code style using make codestyle.
[ ] I've written tests for all new methods and classes that I created.
[ ] I've written the docstring in Google format for all the methods and classes that I used.

Finished [GSK-2271] as well


[#392] noisy logger

Notes for Reviewer

Sets default log level to LogLevel:Warn.
Pre-Review Checklist for the PR Author

[x] Add sensible notes for the reviewer
[x] PR title is short, expressive and meaningful
[x] Relevant issues are linked in the References section
[x] Every source code file has a copyright header with SPDX-License-Identifier: Apache-2.0 OR MIT
[x] Branch follows the naming format (iox2-123-introduce-posix-ipc-example)
[x] Commits messages are according to this guideline

[x] Commit messages have the issue ID ([#123] Add posix ipc example)
[x] Commit author matches Eclipse Contributor Agreement (and ECA is signed)

[x] Tests follow the best practice for testing
[x] Changelog updated in the unreleased section including API breaking changes
[x] Assign PR to reviewer
[x] All checks have passed (except task-list-completed)

Checklist for the PR Reviewer

[ ] Commits are properly organized and messages are according to the guideline
[ ] Unit tests have been written for new behavior
[ ] Public API is documented
[ ] PR title describes the changes

Post-review Checklist for the PR Author

[ ] All open points are addressed and tracked via issues

References

Closes #392

@elfenpiff do you think you could sneak in the new benchmark images for the main README in this PR?

@elBoberido the benchmark images are already the newest ones. Take a look at the first image where iceoryx2 is close to zero and 10 times faster than iceoryx

Oh, I thought the one which compares the latency to the last release would be the default 😅


1.5.0 Final

Hi.
Is there any plan to release 1.5.0 final version in the next days, week, months?
Beta is out approx. 1 year now.
Regards.
Thomas Bosch

+1

+1

@elaatifi there's a roadmap for the next release?
I'm really interested to partecipate in development.

@andreabertagnolli really glad if you can help.
Take a look at the issues list and drop me a mail (elaatifi at google mail service) so we can have a little chat I would be glad help you work on some of the issues assigned to 1.5.0 release there was already a beta but more issues need to be fixed before final release.

I would like to add some fixes before the Orika 1.5 release.
On some points the MappingContext get be lost during Mapping (see draft orika-2016-11-08.txt).
I'm currently prepare a UnitTests to cover this changes.
Also #105 should be easy to fix.

cool! :)

can we estabilish a deadline date for the release?

I think we can go with 1.5.0, or do you think that's better release a beta1?

We can go with 1.5.0

Released


vite dev incompatible with vercel dev - ?import query param expects JS

Describe the bug
I am unable to use vite with vercel dev CLI command that allows local hosting of vercel serverless functions.
I believe the core issue is that vercel dev eagerly returns source files when they exist, so static assets i.e. /src/image.svg?import will immediately resolve to /src/image.svg. The vite server seems to use the ?import query param to hijack the request and return a small JS file with the asset exported
I'm not sure if the best path is for vercel dev to honor the ?import query param and handoff to vite, or if there should be an option to resolve the static asset URLs directly in vite's dev environment.
Reproduction

Create a new Vite project with an SVG import
Push to vercel using vercel CLI (or github)
Run locally using vercel dev

I have a more detailed breakdown in a discussion on the vercel github:
https://github.com/vercel/vercel/discussions/6356
System Info
Output of npx envinfo --system --npmPackages vite,@vitejs/plugin-vue --binaries --browsers:

  System:
    OS: macOS 11.4
    CPU: (8) arm64 Apple M1
    Memory: 310.05 MB / 16.00 GB
    Shell: 5.8 - /bin/zsh
  Binaries:
    Node: 15.14.0 - ~/.nvm/versions/node/v15.14.0/bin/node
    Yarn: 1.22.10 - ~/.nvm/versions/node/v15.14.0/bin/yarn
    npm: 7.7.6 - ~/.nvm/versions/node/v15.14.0/bin/npm
    Watchman: 4.9.0 - /opt/homebrew/bin/watchman
  Browsers:
    Chrome: 91.0.4472.101
    Firefox: 88.0
    Safari: 14.1.1

Used package manager: npm
Before submitting the issue, please make sure you do the following

[x] Read the Contributing Guidelines.
[x] Read the docs.
[x] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[x] Provide a description in this issue that describes the bug.
[x] Make sure this is a Vite issue and not a framework-specific issue. For example, if it's a Vue SFC related bug, it should likely be reported to https://github.com/vuejs/vue-next instead.
[x] Check that this is a concrete bug. For Q&A open a GitHub Discussion or join our Discord Chat Server.

mentioned pull request was merged


Can XLSX.read be used to parse a BLOB containing an XLSX generated in the browser's memory?

Hi all,
I've a 3rd party JS library that exports a JS grid as an XLSX file, and I can get the exported XLSX BLOB (ultimately built using JSZip) inside the browser before it's saved to the disk. I need to customize this file by adding some data to some cells.
I tried this:
// blobXLSX contains the XLSX file exported by the other library
var workbook = XLSX.read(blobXLSX, { type: "binary" });

But the workbook doesn't seem to be populated with the contents of the file (sheets, cell values, etc).
If I save the BLOB to the disk directly, without opening with JS-XLSX (e.g. using this approach), the contents are there though, so the BLOB does contain the XLSX.
All this happens in the browser, without any trips to the server.
I looked over the other issues in the list and the documentation, but didn't find a case like this. Any help would be appreciated.
Thanks

As explained in https://docs.sheetjs.com/#output-type , the "binary" type corresponds to a binary string.  If you can generate a Uint8Array or ArrayBuffer, you can read with the "array" type.  If you have an actual JS Blob object, you have to use FileReader to read the blob into a binary string:
var reader = new FileReader();
reader.onload = function(e) {
  var data = e.target.result;
  var workbook = XLSX.read(data, 'binary');
  /* DO SOMETHING WITH workbook HERE */
};
reader.readAsBinaryString(blob);

Thanks!
Is the full API documented somewhere?


Spotdl suddenly stoped working!!

System Operating System (OS)
Linux
Python Version
3.10 (CPython)
Install Source
Arch User Repository (Unofficial)
Install version / commit hash
3.9.6
Expected Behavior vs Actual Behavior
No response
Steps to reproduce
spotdl https://open.spotify.com/track/3B4etxytImrRKQgV3XHjnR\?si\=5e675eeb3e454132
Traceback
``spotdl https://open.spotify.com/track/3B4etxytImrRKQgV3XHjnR\?si\=5e675eeb3e454132
Traceback (most recent call last):
  File "/usr/bin/spotdl", line 33, in <module>
    sys.exit(load_entry_point('spotdl==3.9.6', 'console_scripts', 'spotdl')())
  File "/usr/bin/spotdl", line 25, in importlib_load_entry_point
    return next(matches).load()
  File "/usr/lib/python3.10/importlib/metadata/__init__.py", line 171, in load
    module = import_module(match.group('module'))
  File "/usr/lib/python3.10/importlib/__init__.py", line 126, in import_module
    return _bootstrap._gcd_import(name[level:], package, level)
  File "<frozen importlib._bootstrap>", line 1050, in _gcd_import
  File "<frozen importlib._bootstrap>", line 1027, in _find_and_load
  File "<frozen importlib._bootstrap>", line 1006, in _find_and_load_unlocked
  File "<frozen importlib._bootstrap>", line 688, in _load_unlocked
  File "<frozen importlib._bootstrap_external>", line 883, in exec_module
  File "<frozen importlib._bootstrap>", line 241, in _call_with_frames_removed
  File "/usr/lib/python3.10/site-packages/spotdl/__init__.py", line 1, in <module>
    from .__main__ import console_entry_point
  File "/usr/lib/python3.10/site-packages/spotdl/__main__.py", line 1, in <module>
    from spotdl.console import console_entry_point
  File "/usr/lib/python3.10/site-packages/spotdl/console/__init__.py", line 6, in <module>
    from spotdl.download import ffmpeg, DownloadManager
  File "/usr/lib/python3.10/site-packages/spotdl/download/__init__.py", line 1, in <module>
    from spotdl.download.tracking_file_handler import DownloadTracker
  File "/usr/lib/python3.10/site-packages/spotdl/download/tracking_file_handler.py", line 4, in <module>
    from spotdl.search import SongObject, song_gatherer
  File "/usr/lib/python3.10/site-packages/spotdl/search/__init__.py", line 3, in <module>
    from spotdl.search.song_gatherer import (
  File "/usr/lib/python3.10/site-packages/spotdl/search/song_gatherer.py", line 7, in <module>
    from spotdl.providers import (
  File "/usr/lib/python3.10/site-packages/spotdl/providers/__init__.py", line 3, in <module>
    from spotdl.providers import lyrics_providers
  File "/usr/lib/python3.10/site-packages/spotdl/providers/lyrics_providers.py", line 2, in <module>
    from bs4 import BeautifulSoup
  File "/home/soumyadeep/.local/lib/python3.10/site-packages/bs4/__init__.py", line 30, in <module>
    from .builder import builder_registry, ParserRejectedMarkup
  File "/home/soumyadeep/.local/lib/python3.10/site-packages/bs4/builder/__init__.py", line 314, in <module>
    from . import _html5lib
  File "/home/soumyadeep/.local/lib/python3.10/site-packages/bs4/builder/_html5lib.py", line 70, in <module>
    class TreeBuilderForHtml5lib(html5lib.treebuilders._base.TreeBuilder):
AttributeError: module 'html5lib.treebuilders' has no attribute '_base'. Did you mean: 'base'?
``

Other details
No response

Try update
sudo apt-get install python3-html5lib
pip3 install -U --force spotdl beautifulsoup4 html5lib


chore(deps): upgrade dev dependencies

Upgrades project dependencies. See details in workflow run.

Automatically created by projen via the "upgrade-dev-dependencies-2.x" workflow

⚪ Backport skipped
The pull request was not backported as there were no branches to backport to. If this is a mistake, please apply the desired version labels or run the backport tool manually.
Manual backport
To create the backport manually run:
backport --pr 3093

Questions ?
Please refer to the Backport tool documentation


⚠ Adacta-fintech.com has degraded performance

In d7339db, Adacta-fintech.com (https://www.adacta-fintech.com) experienced degraded performance:

HTTP code: 200
Response time: 990 ms

Resolved: Adacta-fintech.com performance has improved in 0ae4cc0.


CI: fix NCCL 2.10 unit test not covered

/test mini

@takagi Failed to backport automatically.

Not cleanly cherry-picked


Add modal to display error messages

As a user, I would like to see a modal overlay when an error occurs so I better understand my current situation.
Wireframes for in-app errors
https://xd.adobe.com/view/4f9b250e-27d3-4d8b-af48-c6b90520d0ee-1ac8/
Wireframes for errors during scanning
https://xd.adobe.com/view/cfc523f6-fb60-45fd-bb3d-a09fa3d04c98-9a96/
Acceptance Criteria
Given the Holder within the BC Wallet app
When BC Wallet malfunctions due to system errors
Then the Holder is presented with an error modal overlay describing the error event in layman terms,
And an error code
And an "Okay" button that dismisses the error modal overlay

@jleach I'm unsure as to how I might be able to test this as I'm not sure how to force malfunctions.


python3Packages.aiodocker: init at unstable-2022-01-20

Description of changes

Docker API client for asyncio https://github.com/aio-libs/aiodocker
Things done

Built on platform(s)

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

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

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

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

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

[x] Fits CONTRIBUTING.md.

@ofborg build python3Packages.aiodocker


db.xxx.select - High Memory Load

Hello,
we are using your solution to store a large set of objects with many properties on a web server.
But since the db file grow up to ~800MB we are facing enormous memory load issues when accessing the data (up th 3GB memory while reading 2000 records of the data).
For example we need to get a subset of the Product called ProductLite (simplified classes)
        public class ProductLite
        {
            public string Name { get; set; }
            public Guid Guid { get; set; }
        }

        public class Product
        {
            public long Id { get; set; }
            public string Name { get; set; }
            public string Number { get; set; }
            public Guid Guid { get; set; }
            public Details DetailList { get; set; }
        }

        public class Details
        {
            public int Id { get; set; }
            public Dictionary<int,string> Values { get; set; }
        }

The best method we have found so far is the following
var lites = db.ExecuteRawQuery<ProductLite>("db.Product.select $.Name, $.Guid");

public IEnumerable<T> ExecuteRawQuery<T>(string query)
{
     var items = _database.Engine.Run(query).Select(x => x.AsDocument);
     return items.Select(item => BsonMapper.Global.ToObject<T>(item));
}

But still 2,5GB memory while reading.
So the question is: what is the desired way to get the data without burning down the servers memory when 3 users access the database?
Best regards

Hi! With the objective of organizing our issues, we are closing old unsolved issues. Please check the latest version of LiteDB and open a new issue if your problem/question/suggestion still applies. Thanks!


feat: session handling refactoring. Step 1

Add browser.close command support.
Don't close browser when WS connection is closed.
Close browser after each e2e test.

Out of scope:

Reconnecting to existing BiDi sessions.

did you mean to mark a refactoring as feat:?

Indeed, it should have been chrore: or similar. I tend not to pay much attention to those, but will keep it in mind in future reviews.

better to use refactor: for refactorings


west.yml: Update telink hal version

To obtain Telink driver updates.

The following west manifest projects have been modified in this Pull Request:

Name
Old Revision
New Revision
Diff

hal_telink
https://github.com/telink-semi/hal_telink/commit/db1930caea14720577813ca6643695396084b2f5 (develop)
https://github.com/telink-semi/hal_telink/pull/18
telink-semi/hal_telink#18/files

Note: This message is automatically posted and updated by the Manifest GitHub Action.


Content packs documentation

added art
added document

Docs Build status updates of commit 073ec3f:
:clock10: Pending: waiting for processors (8 builds ahead of you)
:warning: Docs Build is busy, currently there are 8 builds ahead of this one, for more information you can view the Build queue graph on the Docs Portal.

Docs Build status updates of commit 073ec3f:
:warning: Validation status: warnings

File
Status
Preview URL
Details

docs/use-sample-data-packs-with-your-Office-365-developer-subscription.md
:warning:Warning

Details

docs/images/content-packs-01.png
:white_check_mark:Succeeded

docs/images/content-packs-02.png
:white_check_mark:Succeeded

docs/images/content-packs-03.png
:white_check_mark:Succeeded

docs/images/content-packs-05.png
:white_check_mark:Succeeded

docs/use-sample-data-packs-with-your-Office-365-developer-subscription.md

Line 59: [Warning] Invalid file link:(~/images/content-packs-04.png).

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.


LEA's view for configuring a cluster should be re-worked to enhance the usability

LEA's current cluster configurator is incomplete and some information get not displayed.
The picture below illustrates a possible solution:

PR #49 resolves this issue,


Allow to pass output file with ast via macro variable

By this, we can use GCC_PREPROCESSOR_DEFINITIONS setting to pass where dumped output file should be located. This simplifies usage of this script

This looks good to me! Just one small ask

Thanks!


Pre-process data

Hi,
When i try your network in my data, i need to normalize the intensity of the data to a zero-mean, unary-variance.  What does unary variance mean? Does unary variance equal unit variance? When i read the data you provide, i found the variance not equal 1 and the intensity outside the roi may less than zero and why this happens?
When i use your net to do a 2 class segmentation, i find in the log file that in validation process, Accuracy, sensitivity, specificity and dice are always the same between the 2 class, as follows:

2017-12-16 19:23:09.051506 >> VALIDATION: Epoch #0, Subepoch #0, Overall:	 mean accuracy:   	0.0098	=> Correctly-Classified-Voxels/All-Predicted-Voxels = 49/4992
2017-12-16 19:23:09.051549 >> +++++++++++++++ Reporting Accuracy over whole subepoch for Class-0 ++++++++ [Whole Foreground (Pos) Vs Background (Neg)] ++++++++++++++++
2017-12-16 19:23:09.051598 >> VALIDATION: Epoch #0, Subepoch #0, Class-0:	 mean accuracy:   	0.0098	=> (TruePos+TrueNeg)/All-Predicted-Voxels = 49/4992
2017-12-16 19:23:09.051643 >> VALIDATION: Epoch #0, Subepoch #0, Class-0:	 mean sensitivity:	0.0098	=> TruePos/RealPos = 49/4992
2017-12-16 19:23:09.051685 >> VALIDATION: Epoch #0, Subepoch #0, Class-0:	 mean specificity:	N/A	=> TrueNeg/RealNeg = 0/0
2017-12-16 19:23:09.051714 >> VALIDATION: Epoch #0, Subepoch #0, Class-0:	 mean Dice:       	0.0194
2017-12-16 19:23:09.051734 >> +++++++++++++++ Reporting Accuracy over whole subepoch for Class-1 ++++++++ [This Class (Pos) Vs All Others (Neg)] ++++++++++++++++
2017-12-16 19:23:09.051783 >> VALIDATION: Epoch #0, Subepoch #0, Class-1:	 mean accuracy:   	0.0098	=> (TruePos+TrueNeg)/All-Predicted-Voxels = 49/4992
2017-12-16 19:23:09.051823 >> VALIDATION: Epoch #0, Subepoch #0, Class-1:	 mean sensitivity:	0.0098	=> TruePos/RealPos = 49/4992
2017-12-16 19:23:09.051852 >> VALIDATION: Epoch #0, Subepoch #0, Class-1:	 mean specificity:	N/A	=> TrueNeg/RealNeg = 0/0
2017-12-16 19:23:09.051879 >> VALIDATION: Epoch #0, Subepoch #0, Class-1:	 mean Dice:       	0.0194
I am very confused how it happens.

Hi,
Regarding intensity normalization, yes, with unary-variance I meant unit-variance. I corrected it now in the readme, thanks.
Explanations on how to do the normalization can be found in previous issues #56 and #72 and in our papers.
For explanation about the case where class-0 scores are the same as class-1 scores in binary problems, please see previous issue #48 .
Thank you for your interest in our work and your feedback.
Kostas


Mislabeling of "sentence features" as "dense features" in documentation

You can see in the _get_sentence_features method in the rasa/nlu/classifiers/sklearn_intent_classifier.py file that the error message reads "No sentence features present. Not able to train sklearn policy."  This is misleading because sentence features are referred to as "dense features" in the documentation.
I have attached a zip folder containing test component code as well as my config.yml file. I used the test component code to test that the features were not present from the spacy_featurizer.  When outputting the features I found that they were empty, as you can see in these screenshots:

code.zip
@kearnsw

Thanks for raising this issue, @alopez will get back to you about it soon✨
Please also check out the docs and the forum in case your issue was raised there too 🤗

@koaning Are you working on this? If not, can you remove your assignment?

@alexyuwen I wasn't able to reproduce this on the latest rasa version 2.5.1 and also on main. I was able to successfully train a model with this config:
pipeline:
 # No configuration for the NLU pipeline was provided. The following default pipeline was used to train your model.
 # If you'd like to customize it, uncomment and adjust the pipeline.
 # See https://rasa.com/docs/rasa/tuning-your-model for more information.

    - name: SpacyNLP
    - name: SpacyTokenizer
    - name: SpacyFeaturizer
    - name: SpacyEntityExtractor
    - name: SklearnIntentClassifier
    - name: EntitySynonymMapper
    - name: FallbackClassifier
      threshold: 0.3
      ambiguity_threshold: 0.1

I also tried with
pipeline:
 # No configuration for the NLU pipeline was provided. The following default pipeline was used to train your model.
 # If you'd like to customize it, uncomment and adjust the pipeline.
 # See https://rasa.com/docs/rasa/tuning-your-model for more information.

    - name: SpacyNLP
    - name: SpacyTokenizer
    - name: SpacyFeaturizer
    - name: SpacyEntityExtractor
    - name: DIETClassifier
    - name: EntitySynonymMapper
    - name: FallbackClassifier
      threshold: 0.3
      ambiguity_threshold: 0.1

Command used: rasa train nlu
Let us know if the issue persists for you on the above mentioned versions . I am closing the issue until then.
Thanks!


Player Import Error from Older Blend file

Describe the bug
We are trying to update the export from an older Blender file. The goal is to export a gltf2 which we can used in Godot Engine.
To Reproduce
Steps to reproduce the behavior:

Go to https://github.com/godotengine/tps-demo/blob/master/player/model/player.blend
Click on export gltf2 and the "Apply Modifier" causes a stack track when enabled, and broken model when disabled.

blender_export_gltf_modifier.txt

Open in Godot Engine or Visual Studio Code's Three js import or even Blender
See error

Expected behavior
It should look like this.

Screenshots
If applicable, add screenshots to help explain your problem.

.blend file/ .gltf
If applicable, a zipped folder containing a .blend file for exporter issues and a .gltf file for importer issues. Please take care to include all resources, such as textures in the .zip file.
player.zip
Version

OS: Windows 10
Blender Version 2.83

I can confirm these issues on both Windows and Linux with Blender 2.83 stable. The player file comes from the Godot TPS demo, and you can also find more examples there (red_robot.blend, door.blend).

Hello,
Confirmed (seems to happen on 2.82 too)
Investigation in progress, but here are some first thoughts:
File has some big issues:

Some depend cycle on armature because of how piston are rigs. This can have an impact on how rigs are exported.
you assigned some armature animation on mesh object.

Problem seems to be linked to depgraph evaluation. I had a quick hack that solve the issue, but I need to investigate more, because I can't reproduce on simple example.

More info:
You have lots of non visbile objects on your scene that are exported, that explains why you see more object in three.js
Select visible meshes + armature and use "use_selection" to export only player+rig

Perhaps there should be an option in Blender to only export visible objects?

Currently, exporter is not well designed to export only part of hierarchy tree
See #193 discussion

Here is a simpler model as a test case:
https://github.com/godotengine/tps-demo/tree/master/door/model
The existing .dae file works fine. When exporting the blend as .glb it's very weird. If all of the non-visible parts are deleted, exporting works almost fine, but it is missing an animation (it has doorsimple_closing, doorsimple_closed-cycle, and  doorsimple_opening, but is missing  doorsimple_open-cycle).
By the way, it is perfectly possible that these models have problems on their own (I'm not the artist that created them), or maybe just problems caused by opening old Blender files in a newer version of Blender. I'm not knowledgeable enough in Blender to figure out which problems are bugs in Blender, which problems are bugs in this file, or how to fix either of those categories of bugs.

@aaronfranke The reason doorsimple_open-cycle isn't exported is because it isn't pushed to an NLA track or assigned as the active action. This exporter uses NLA tracks to organize how actions get arranged into animations. Set it up like this

and it will export.

Tested with #1378 + #1563 (selected object + apply modifiers)
All actions are exported with #1560
Still have some issue with negative frame, but is out of scope of #1378
Seems to work without problem in Godot 3.4.2

Great! Looking forward to testing this. Is this in the steam 3.0 beta daily build yet?

@fire : No, not yet.
You can find #1378 here : https://builder.blender.org/download/experimental/gltf_vtree/

Fixed by 69746b2b31eeee080d5dfd3c74f1e02742795ed2


recipes-graphics: mesa_git bump to rev aed8d30b50

Reviewed-by: Dmitry Baryshkov

does the build fail if we pass the PACKAGECONFIG?

does the build fail if we pass the PACKAGECONFIG?

This might be a cleaner solution:
PLATFORMS_remove = "drm surfaceless"

does the build fail if we pass the PACKAGECONFIG?

This might be a cleaner solution:
PLATFORMS_remove = "drm surfaceless"

probably. That said I was interesting to understand why we needed this change. I have now looked at the upstream commit, and I understand. But perhaps a better commit message would be helpful.

@ndechesne  @lumag I updated the commit to remove platforms instead.


Reducers and Actions

Highlights

[ ] No linter errors.
[ ] Followed correct git flow.
[ ] Added Redux Toolkit to the project.
[ ] Used the best practices for HTML & CSS.
[ ] Structured the application using professional way.
[ ] Defined action types for adding and removing a book.
[ ] wrote book's actions and reducer.
[ ] wrote categories actions and reducer.
[ ] Configured the Redux Store.

Hi @tanveerisonline,
While you made a great effort in this project, unfortunately, I cannot proceed to review your code.
Invalid Code Review Request
You have not implemented the correct gitflow. You should request a merge to develop, not the main branch. Kindly rebase your PR.
Your Code Review Request will be marked as invalid in your Dashboard, so please submit a new one once you are ready 🙏
Cheers and Happy coding!👏👏👏
Feel free to leave any questions or comments in the PR thread if something is not 100% clear.
Please, do not open a new Pull Request for re-reviews. You should use the same Pull Request submitted for the first review, either valid or invalid unless it is requested otherwise.

Invalid Code Review Request does not count into the code reviews limit.

Hi @tanveerisonline,
While you made a great effort in this project, unfortunately, I cannot proceed to review your code.
Invalid Code Review Request
Kindly change the base branch to the dev branch for this PR as requested earlier
Your Code Review Request will be marked as invalid in your Dashboard, so please submit a new one once you are ready 🙏
Cheers and Happy coding!👏👏👏
Feel free to leave any questions or comments in the PR thread if something is not 100% clear.
Please, do not open a new Pull Request for re-reviews. You should use the same Pull Request submitted for the first review, either valid or invalid unless it is requested otherwise.

Invalid Code Review Request does not count into the code reviews limit.


Call to undefined method Illuminate\Session\Store::first() in the login page

I am using Laravel 5.7. In the login page, when i hit login without any value in the form, i see this error.
I also see this when i provide some data in it too. It used to work well I dont know what's causing it.

Hello @gouseferoz,
Are you sure that the issue is related to laravel-u2f ?
The auth/login.blade.php file is not provided by this package.
What version of laravel-u2f are you using ? composer why lahaxearnaud/laravel-u2f
Arnaud

I can't reproduce your issue.
You can take a look at the project lahaxearnaud/laravel-u2f-example.
Perhaps I miss the point.
Arnaud


An unclosed client may cause Deno from exiting automatically

A not closed client with idle connections will prevent Deno from automatically exiting, after implementing idle timeout (#81).
This is a problem, especially in short-running applications.
For example:
const client = new Client().connect(/* ... */);
console.log(await client.query("SELECT ..."));
// End of file, but Deno won't exit

Deno won't exit after it finishes querying, because there is some timer (by setTimeout) that can close connection when it stayed idle for too long. However, after all other tasks finish, Deno still wait for these timers.
Unfortunately, I haven't found a solution to this problem. Seems that setTimeout and setInterval are the only APIs in Deno for the timer and currently there is no way to have Deno not wait for some specific timer. 😢
Workaround if you are writing short-running applications:

Set idleTimeout to 0 in Client.connect() options to disable the idle timeout, or
Call Client.close() or Deno.exit() after queries

Set idleTimeout to 0 in Client.connect() options to disable the idle timeout

Thanks.  That seems to do the trick.


401 Unauthorized

Hello cricle dev!
I applyed a test-apikey,and use this payments-sample-app.
I enter my test-api-key at payments-sample-app but It showed error message by any endpoint
error message : {"code":2,"message":"401 Unauthorized. errId: 2b78e9c7-6288-ffbf-55b1-a11791330f42"}
please help 😭

hey @kingsmao is this still an issue? Have you tried to generate a new test api key?

hi there @kristinfritsch  yes, but I don't use it anymore, I integrated the api manually


Docs TOC issues

Migrated from GitLab: https://gitlab.com/meltano/meltano/-/issues/3531
Originally created by @afolson on 2022-05-25 22:29:33

If the ToC exceeds the height of the browser viewport, there’s no way to scroll down to get at the lower links
If a long heading breaks over multiple lines, the list becomes hard to parse because it’s hard to tell apart 1 heading over multiple lines from multiple short headings on their own line
https://docs.meltano.com/tutorials/ Has a ghost TOC

~https://docs.meltano.com/getting-started does as well~

View 1 previous comment from the original issue on GitLab

Ghost TOCs are caused by the template being used on index.md pages. I'll make a new one to get rid of the TOC since it won't be needed on those pages for a while and it's easy to add back once it is.

A user reported in Slack  that it may be useful to have a more verbose Table of Contents. When you're on https://docs.meltano.com/concepts/plugins#how-to-use-12 for example, it's not clear that you're in the Mapper section or that there's even a Mapper section in the TOC.

Closing. This is now addressed in docs v2: https://afolson-docs.netlify.app/concepts/environments


Implement a bot reporting pre-commit results

What I'm trying to achieve
To view the pre-commit results on the PR comments.
Describe a proposed solution
Implement a bot as a GitHub application to do it for us.
This will report:

black
flake8
isort
trailing blank spaces
outdated GraphQL schema
outdated Typescript types

Other solutions I've tried and won't work
Travis, GitHub Actions, code climate...

@NyanKiyoshi this seems like great idea. Any ETA on when you might have time to implement it?

Hi @Satloff, it was actually supposed to be done a few days ago but I got material issues (power supply died) so I was unable to work on it.
I am not sure when it will happen. I might do it this weekend (I want to this task to be done during my free time, off from work–so it depends on my availability).
This will be implemented as a GitHub action (unless impossible to work something out with the current limitation of this feature).


Fixes for generic ObjC classes conforming to Swift protocols.

Fix a SILGen and IRGen bug that caused crashes and linker errors when ObjC generic classes conformed to Swift protocols. rdar://problem/27869089

@swift-ci Please smoke test

@rjmccall Mind reviewing this for 3.0?

LGTM.


set Selinux to permissive in engine during HE deployment

workaround for bug-url: https://bugzilla.redhat.com/show_bug.cgi?id=1788090#c12

selinux patch can be abandoned, fixed by https://gerrit.ovirt.org/107314 .


Migration to Python3 error: Class advice impossible in Python3. Use the @Implementer class decorator instead.

My raspberry has burnt last night after a electrical power outage. I have a spare Raspberry3 . Today only python3 is available.
I have used 2to3 to convert your script,
Now I get the following error
pi@raspberrypi:~/garage-door-controller $ python controller.py &
[1] 2070
pi@raspberrypi:~/garage-door-controller $ Traceback (most recent call last):
File "/home/pi/garage-door-controller/controller.py", line 20, in
class HttpPasswordRealm(object):
File "/home/pi/garage-door-controller/controller.py", line 21, in HttpPasswordRealm
implements(portal.IRealm)
File "/usr/lib/python3/dist-packages/zope/interface/declarations.py", line 706, in implements
raise TypeError(_ADVICE_ERROR % 'implementer')
TypeError: Class advice impossible in Python3.  Use the @implementer class decorator instead.
I have some python basic knowledge so I have asked google but I don't find any relevant answer to this problem.
Any help ?
Thanks
Gilles

ok , I found . Just replace "implement" with "implementer" 2 times

btw , I still get some other errors that I try to fix one by one.

I was wrong. It is not yet fixed. The authentication doesn't work. I have found a lot of records ion google about that, but nothing that I can apply to my case.
Any help ???
I don't want to open/close my garage doors from internet without security :)

Hello,
can someone help to migrate it to Python3 . The twisted security part with Zope doesn't work anymore.
I have found some sample but no way to make it work properly. It ignores the username/password and refuses the connection:
https://docs.twistedmatrix.com/en/stable/web/howto/web-in-60/http-auth.html

I give up and will try to rewrite it with flask instead of twisted.


M3U8 media playlist reading fixes

Note that this change includes the fix from #11. It fixes the way it reads lines, in particular the new code doesn't handle files with \r\n properly.

Hi @wolfcon and @alexsun, any chance you give this PR a look?

@noamtamim Srr. Im busy nowadays.

Thank you @wolfcon! When do you intend to release an updated pod?


perf(core): Disable debug mode in lua.stdlib

Benchmarking this commit shows a marked improvement:
$ make benchmark HEADSHA=1ce6438
cat time-1ce6438.json time-f568660.json
{ "real": "29.451", "user": "28.452", "sys": "0.880" }
{ "real": "146.859", "user": "145.438", "sys": "1.011" }

Coverage remained the same at 98.96% when pulling 28ec2d8f436585fb28dc6e8cc8c617b2b1490e0d on alerque:disable-stdlib-debug into f568660c19cb12bfd527bf301d59b61f6f0452d3 on simoncozens:master.


Ability to specify serviceContext for logging-winston

Originally reported by @vic3lord on https://github.com/GoogleCloudPlatform/google-cloud-node/issues/2435.
Having the same issue, also I cannot state serviceContext Object in logging-winston directly without using error-reporting.
https://github.com/GoogleCloudPlatform/google-cloud-node/blob/master/packages/logging-winston/src/index.js#L174
Let me know if I can do anything to help, even to provide info.

Hi @vic3lord. Is your application running on GCP or elsewhere? On GCP running apps, we should auto-discover the service. However, if you are running locally, on-premise or elsewhere, that's the case where would use the 'global' resource descriptor. Is that your case?

I am running it on gke, I want to separate each container name to be a different service both in cloud logging and error-reporting, since each container is an actual service.


Reporting keys into logs

https://github.com/cloudfoundry/log-cache/blob/df05c6457305c8b633f00cc9e4a8cbfa89e2cf03/cmd/nozzle/config.go#L23

Nevermind this is just the path.


Need help with a Taxon Name of a subspecies all terms non latin

I want the citations to be correct, when this taxon was cited completely non latin... Cranaë kükenthali kükenthali
But I am having a lot of trouble with the original combination and rank panel.
I know that the best practice is to create the taxon name well spelled and then clone it and create the originaly misspelled. But that is easy with the genus. Then when I create the species what do I use in the original combination? and then when I create the subspecies?
This is what I need to recreate

Right now... trying to solve this I have two Taxon Names duplicated... one Taxon Name with the types associated and no other info in its OTU and other taxon name, with all the citations an OTU with images but no types... Loosing my mind here.
Location
Edit taxon name
Screenshot, napkin sketch of interface, or conceptual description
No response
Your role
No response

Sorry about the label... can't erase it... It is just a question.

I updated the record for you to use as an example
https://sfg.taxonworks.org/tasks/nomenclature/browse?taxon_name_id=927555

You fixed it??? Wow! I will see all the details!!! Thank you so much!

Kirby cited the name as Cranaë kuekenthali. Still I am unable to create this combiantion. I will save this example for a TW session.

Go to the Edit page for the protonym of kuekenthali. Go down to Subsequent combination section. Drag the name kuekenthali to the species level. In the genus field, add 'Cranaë'. Add the citation. Click Save.


Clang Undefined Warnings

The following warnings are given by the -Wundef flag on Clang.
toml/datetime.hpp:23:5: error: '_POSIX_C_SOURCE' is not defined, evaluates to 0 [-Werror,-Wundef]
#if _POSIX_C_SOURCE >= 1 || defined(_XOPEN_SOURCE) || defined(_BSD_SOURCE) || defined(_SVID_SOURCE) || defined(_POSIX_SOURCE)
    ^
toml/datetime.hpp:38:7: error: '_MSC_VER' is not defined, evaluates to 0 [-Werror,-Wundef]
#elif _MSC_VER

Should these be macros be checked to see if they are defined?

Thank you for finding the issue. Here also you are right.
Would you like to make a PR to fix this?

I can make a PR, no problem 👍

Merged. Thank you!


Delete dead code in introspection engine

Noticed while working on https://github.com/prisma/prisma/pull/15834

We can delete the engines code when this is merged.


CSP Update

As of discord/discord-api-docs@61928fd19960fe807a2f56da353aa7a4ab123084, a breaking change will be deployed for all existing activities on August 28th.
This issue will remain open while I prepare a version that supports completely this change.

Testing this release in the version/1.2.0 branch.


Centralize bio error

The apps make use of some common library routines to do things like "dup stdout" or "open a file for writing"  The callers were not consistent in error-checking.  So, we do like app_malloc does and return a valid BIO or print the error stack and exit.
There were a couple of places where we needed to "silently fail" -- in cases where we tried to open a file (config file) and didn't care if it failed. That was handled be rewriting code to do the fopen first, and then failing if BIO_new_fp failed.  I also did the same thing for cmp.c, but I'm not sure if that's needed, so pinging @ddvo to look at that.  Also pinging @mspncp and @levitte who promised to review this. :)

Additional commit pushed adding more calls; you convinced me.

amended fixup pushed to catch missing check call.

are we having fun yet? :)

I hope that is not a show stopper,
but does this mean that the openssl command shell will exit
when the first file can't be read?

you mean interactive mode?  yes, it will exit.  that could be a show-stopper if folks think interactive mode is (still) important.  it could be fixed using setjmp/longjmp.

I must say it is a showstopper for me unless the interactive mode is dropped completely.
Why using setjmp/longjmp is better than just keeping the if (bio==NULL) in individual apps?
Memory allocation errors are quite some different beast than failure to open a file in the sense that they are usually unrecoverable so for these immediate exit from an application is somewhat acceptable.

Since #12023 is on the fast track, although the body isn't even cold yet, I assume that there's no show-stoppers any more.

Yeah, no more show-stoppers. I still a little bit dislike this approach but I am definitely not blocking this.

Thanks for the review!  I just resolved comments where I agreed. Fixup commit pushed.

There was a merge conflict (new function app_bail_out) so I rebased and merged/squashed many of the commits.  Any new feedback comes in, I'll just push fixups :)

@bernd-edlinger back from vacation yet? :)

I just added a fixup commit that I hope will settle the check_bio dispute

I don't think your commit addresses @ddvo's concern, but I will be happy if I am wrong.
I added another commit that replaces most "mode" variables with "bflags" when they are BIO flags.

Quite frankly, I'm not sure what use the BIO flags or the format number is for the user...  what are they supposed to do with it, go dive in header files?

I err on the side of more diagnostic info, even if not useful to the end-user.

I don't think your commit addresses @DDvO's concern, but I will be happy if I am wrong.

Recent changes go in the right direction by separating the file opening mode (r/w/a) from other detail, but there is still a mix of file content format and BIO flags, which should be separated as well.

I pushed a commit that does the separation, at least what I think is needed.  If it's not right, please add a commit here.

I pushed a commit that does the separation, at least what I think is needed. If it's not right, please add a commit here.

Thanks! Now the code is clean and the flags/format output is meaningful.
I would have expected that check_bio() would get both parameters (rather than adding a second function, check_bio_fmt()), which would be a bit less redundant and can be more informative in case both the flags and the format is present, but in case only one of them is available then the solution with two separate functions is of course more natural.

https://mta.openssl.org/pipermail/openssl-project/2020-July/002091.html

https://mta.openssl.org/pipermail/openssl-project/2020-July/002091.html

@richsalz, are you sure you did not confuse issues?
What has #12089 to do with this PR?

It appears to me that this PR has been closed and branch deleted by mistake!
Can you restore it?

@richsalz is absolutely free to withdraw his submissions as he sees fit and for whatever reason, even though we may find that unfortunate. He has stated his reasons in the openssl-users he linked.

Oh - meanwhile I noticed what the reason/background of this deletion is!
Sad that a merited contributor leaves, for an IMHO unessential reason.
And a waste of resources to kill PRs, on which also others have invested.


Filter Collection Objects - Filter by Individual Loan and generate DWCA/CSV

As a curator, I want to go to Filter COs
Then I want to search for the collection objects currently loaned on a SPECIFIC LOAN
Then I want to be able to generate a CSV or DWCA of the objects on that loan to be sent to loanee

@jlpereira new facet.  Param is loan_id[]=

Request reopen, not working in 0.21.2

Currently returns list of all records in project (Production, INHS Insect Collection)


Wrap revm Return status to proper error.

Describe the feature
Revm will refactor return type to be more friendly and when this happens errors should be properly wrapped inside reth
Additional context
No response

dependent on: https://github.com/bluealloy/revm/issues/309

revm v3.0.0 got released so this can be started

Can I work on this?

Can I work on this?

Go for it!

I've been thinking about this and dealing with some errors after bumping the revm version but
I don't think I quite get a general idea on how to wrap the revm errors.
Any ideas or tips on how to handle this?

@fkrause98 most of the changes are dealing with breaking changes of the new revm version, kudos if you did it!
error wrapping is mostly cleaning up this chunk of code: https://github.com/paradigmxyz/reth/blob/df6e3143c3bf3b7d46a185ff6fa950b69f429c08/crates/executor/src/executor.rs#L367-L391
is_success can be set with a use of this: https://github.com/bluealloy/revm/blob/v20/crates/primitives/src/result.rs#L40
execution::Error can probably be expanded with EVMError

Oh, ok!
Seems like what I've been doing does not strand too far from what you just said.
Thanks, I'll have a PR ready soon 😄.

@fkrause98 thanks for taking this on - can you pls push your branch and share your current progress? upgrading to revm 3.0 is P0 right now (both in Reth & Foundry) and would like to drive this to completion ASAP! thank you^^

Sure, here it is!
Please, let me know what is to be improved.


Remove version: Bitnami.Joomla version 4.2.3-0

[Automated] It returns code over 400 in all urls
Microsoft Reviewers: Open in CodeFlow

URL: https://downloads.bitnami.com/files/stacks/joomla/4.2.3-0/bitnami-joomla-4.2.3-0-windows-x64-installer.exe
OK
(Automated message - build 910)

Version still available.


fix issue with callable references in kotlin 1.4

@see https://youtrack.jetbrains.com/issue/KT-37435 and https://youtrack.jetbrains.com/issue/KT-37435

I can potentially also use a compiler flag to remove the optimization causing the issue


Fix CIS Scan For Startup Procs

A New PR
THANK YOU - We love to get PR's and really appreciate your time and help to improve this module
Accepting a PR
Before we accept the PR - please confirm that you have run the tests locally to avoid our automated build and release process failing. You can see how to do that in our wiki
https://github.com/sqlcollaborative/dbachecks/wiki
Please confirm you have 0 failing Pester Tests
[X] There are 0 failing Pester tests
Changes this PR brings
Some of the original code didn't get merged into the code.
Please add below the changes that this PR covers. It doesnt need an essay just the bullet points :-)

I think I resolved this in 1.2.14

I think I resolved this in 1.2.14


Corgio can't stand on the platform

Describe the bug
Corgio can't stand on the platform
Steps to reproduce the behavior

Navigate to https://arcade.makecode.com/
Click "Simple Extensions" on "Tutorial" -> "Start tutorial"
Finished the tutorial and run it

Expect behavior
Corgio can stand on the platform
Actual behavior
Corgio can't stand on the platform

Additional context

OS: Windows(rs6)
arcade version: 0.15.13
Microsoft MakeCode version: 5.30.40

Nice bug @Sheila111

Did you set them as walls in the last part of step 5?

Fixed right jwunderl?


architecture: move ecc/ package into its own repository

Once #15 and #13 are done would be a good time to move ecc into its own repository.
gnark would only contain Constraint Systems and Proving protocols. The crypto primitives they rely on would be reference in go.mod .
Current cs pacakge needs refactoring (move out subpackages, etc).

Do you mean just the code for the arithmetic in the elliptic curve or the pairing as well? For the pairing it's still possible but different path has to be taken according to the curve (for instance the BN12 family has some extra operations after the Miller loop to compensate the loop shortening).

@ThomasPiellard the whole ecc/ package, should be a different github repo once it's stable.
A project like goff, but to generate ecc tower + pairing. It will have a generated subfolder that other projects like gnark can reference through go modules. Question is should it be ecc only or should it be more general to support other crypto primitives gnark may need in the future?


[question] How to build this library on macOS

Describe the impediment
I want to use this library on macOS, But I could not use it correctly.
Trying to obtain
I download master zip from repo, then follow below steps.
mkdir build && cd build
mkdir install
cmake -DCMAKE_BUILD_TYPE=Release -DCMAKE_INSTALL_PREFIX=./install -DJWT_DISABLE_BASE64=ON -DJWT_DISABLE_PICOJSON=OFF -DJWT_BUILD_EXAMPLES=OFF -DOPENSSL_ROOT_DIR=/Users/xxx/openssl ..
make
make install

Desktop (please complete the following information): (Include/Attach if Applicable)

OS: macOS
Compiler: AppleClang
Version 12

Additional context
After cmake and make install, the generated files in install folder (jwt-cpp and picojson).
Then, I include these two folder into project, and try sample code.
#include <jwt-cpp/jwt.h>
#include <iostream>

std::string token = "eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXUyJ9.eyJpc3MiOiJhdXRoMCJ9.AbIJTDMFc7yUa5MhvcP03nJPyCPzZtQcGEp-zWfOkEE";
    auto decoded = jwt::decode(token);

The error is as below.
No matching function for call to 'decode'

Logs (Include/Attach if Applicable)
Click to expand log
-- The C compiler identification is AppleClang <IP_ADDRESS>00032
-- The CXX compiler identification is AppleClang <IP_ADDRESS>00032
-- Detecting C compiler ABI info
-- Detecting C compiler ABI info - done
-- Check for working C compiler: /Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/cc - skipped
-- Detecting C compile features
-- Detecting C compile features - done
-- Detecting CXX compiler ABI info
-- Detecting CXX compiler ABI info - done
-- Check for working CXX compiler: /Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/c++ - skipped
-- Detecting CXX compile features
-- Detecting CXX compile features - done
-- Found OpenSSL: /Users/xxx/openssl/lib/libcrypto.a (found suitable version "1.1.1l", minimum required is "1.0.1")
-- Configuring done
-- Generating done

Hmmm 🤔 looks good at first glance...
Can you please share your CMakeList.txt? As well as the verbose build command/output? That might give us more clues 🤞

CMakeLists.txt is from source code zip. I do not change anything.
cmake_minimum_required(VERSION 3.12)

# HUNTER_ENABLED is always set if this package is included in a project using hunter (HunterGate sets it) In this case
# we will use hunter as well to stay consistent. If not the use can supply it on configure to force using hunter.
if(HUNTER_ENABLED)
  include("cmake/HunterGate.cmake")
  huntergate(URL "https://github.com/cpp-pm/hunter/archive/v0.23.314.tar.gz" SHA1
             "95c47c92f68edb091b5d6d18924baabe02a6962a")
  message(STATUS "jwt-cpp: using hunter for dependency resolution")
endif()

project(jwt-cpp)

option(JWT_BUILD_EXAMPLES "Configure CMake to build examples (or not)" ON)
option(JWT_BUILD_TESTS "Configure CMake to build tests (or not)" OFF)
option(JWT_ENABLE_COVERAGE "Enable code coverage testing" OFF)
option(JWT_ENABLE_FUZZING "Enable fuzz testing" OFF)

option(JWT_EXTERNAL_PICOJSON "Use find_package() to locate picojson, provided to integrate with package managers" OFF)
option(JWT_DISABLE_BASE64 "Do not include the base64 implementation from this library" OFF)
option(JWT_DISABLE_PICOJSON "Do not provide the picojson template specialiaze" OFF)

set(JWT_SSL_LIBRARY_OPTIONS OpenSSL LibreSSL wolfSSL)
set(JWT_SSL_LIBRARY OpenSSL CACHE STRING "Determines which SSL library to build with")
set_property(CACHE JWT_SSL_LIBRARY PROPERTY STRINGS ${JWT_SSL_LIBRARY_OPTIONS})

set(JWT_JSON_TRAITS_OPTIONS boost-json danielaparker-jsoncons kazuho-picojson nlohmann-json)

list(APPEND CMAKE_MODULE_PATH "${CMAKE_CURRENT_LIST_DIR}/cmake")

if(NOT JWT_SSL_LIBRARY IN_LIST JWT_SSL_LIBRARY_OPTIONS)
  message(FATAL_ERROR "JWT_SSL_LIBRARY must be one of ${JWT_SSL_LIBRARY_OPTIONS}")
endif()

# If Hunter is enabled, we configure it to resolve OpenSSL and warn the user if he selected an option not supported by
# hunter. We fall back to the system library in this case.
if(HUNTER_ENABLED)
  if(${JWT_SSL_LIBRARY} MATCHES "OpenSSL")
    hunter_add_package(OpenSSL)
  elseif(${JWT_SSL_LIBRARY} MATCHES "LibreSSL")
    message(WARNING "Hunter does not support LibreSSL yet, the system library will be used (if available)")
  elseif(${JWT_SSL_LIBRARY} MATCHES "wolfSSL")
    message(WARNING "Hunter does not support wolfSSL yet, the system library will be used (if available)")
  endif()
  if(JWT_EXTERNAL_PICOJSON)
    message(WARNING "Hunter does not support picojson yet, the system library will be used (if available)")
  endif()
endif()

# Lookup dependencies
if(${JWT_SSL_LIBRARY} MATCHES "OpenSSL")
  find_package(OpenSSL 1.0.1 REQUIRED)
elseif(${JWT_SSL_LIBRARY} MATCHES "LibreSSL")
  find_package(LibreSSL 3.0.0 REQUIRED)
elseif(${JWT_SSL_LIBRARY} MATCHES "wolfSSL")
  find_package(PkgConfig REQUIRED)
  pkg_check_modules(wolfssl REQUIRED IMPORTED_TARGET wolfssl)
  list(TRANSFORM wolfssl_INCLUDE_DIRS APPEND "/wolfssl") # This is required to access OpenSSL compatability API
endif()

if(JWT_EXTERNAL_PICOJSON)
  find_package(picojson 1.3.0 REQUIRED)
endif()

set(JWT_INCLUDE_PATH ${CMAKE_CURRENT_SOURCE_DIR}/include)
set(JWT_HEADER_FILES ${JWT_INCLUDE_PATH}/jwt-cpp/jwt.h)
foreach(traits ${JWT_JSON_TRAITS_OPTIONS})
  list(APPEND JWT_HEADER_FILES ${JWT_INCLUDE_PATH}/jwt-cpp/traits/${traits}/defaults.h
       ${JWT_INCLUDE_PATH}/jwt-cpp/traits/${traits}/traits.h)
endforeach()

if(NOT JWT_DISABLE_BASE64)
  list(APPEND JWT_HEADER_FILES ${JWT_INCLUDE_PATH}/jwt-cpp/base.h)
endif()

add_library(jwt-cpp INTERFACE)
add_library(jwt-cpp::jwt-cpp ALIAS jwt-cpp) # To match export
target_compile_features(jwt-cpp INTERFACE cxx_std_11)
if(JWT_DISABLE_BASE64)
  target_compile_definitions(jwt-cpp INTERFACE JWT_DISABLE_BASE64)
endif()
if(JWT_DISABLE_PICOJSON)
  target_compile_definitions(jwt-cpp INTERFACE JWT_DISABLE_PICOJSON)
endif()

include(GNUInstallDirs)
include(CMakePackageConfigHelpers)
target_include_directories(jwt-cpp INTERFACE $<BUILD_INTERFACE:${JWT_INCLUDE_PATH}>
                                             $<INSTALL_INTERFACE:${CMAKE_INSTALL_INCLUDEDIR}>)

if(${JWT_SSL_LIBRARY} MATCHES "OpenSSL")
  target_link_libraries(jwt-cpp INTERFACE OpenSSL::SSL OpenSSL::Crypto)
endif()

if(${JWT_SSL_LIBRARY} MATCHES "LibreSSL")
  target_link_libraries(jwt-cpp INTERFACE LibreSSL::TLS)
endif()

if(${JWT_SSL_LIBRARY} MATCHES "wolfSSL")
  target_link_libraries(jwt-cpp INTERFACE PkgConfig::wolfssl)
  # This is required to access OpenSSL compatability API
  target_include_directories(jwt-cpp INTERFACE ${wolfssl_INCLUDE_DIRS})
  target_compile_definitions(jwt-cpp INTERFACE OPENSSL_EXTRA OPENSSL_ALL)
endif()

if(JWT_EXTERNAL_PICOJSON)
  target_link_libraries(jwt-cpp INTERFACE picojson::picojson>)
endif()

# Hunter needs relative paths so the files are palced correctly
if(NOT JWT_CMAKE_FILES_INSTALL_DIR)
  set(JWT_CMAKE_FILES_INSTALL_DIR cmake)
endif()

configure_package_config_file(
  ${CMAKE_CURRENT_LIST_DIR}/cmake/jwt-cpp-config.cmake.in ${CMAKE_CURRENT_BINARY_DIR}/jwt-cpp-config.cmake
  INSTALL_DESTINATION ${JWT_CMAKE_FILES_INSTALL_DIR} PATH_VARS JWT_EXTERNAL_PICOJSON JWT_SSL_LIBRARY)
write_basic_package_version_file(${CMAKE_CURRENT_BINARY_DIR}/jwt-cpp-config-version.cmake VERSION 0.6.0
                                 COMPATIBILITY ExactVersion)

install(TARGETS jwt-cpp EXPORT jwt-cpp-targets PUBLIC_HEADER DESTINATION ${CMAKE_INSTALL_INCLUDEDIR})
install(EXPORT jwt-cpp-targets NAMESPACE jwt-cpp:: FILE jwt-cpp-targets.cmake
        DESTINATION ${JWT_CMAKE_FILES_INSTALL_DIR})
install(DIRECTORY ${JWT_INCLUDE_PATH}/jwt-cpp DESTINATION ${CMAKE_INSTALL_INCLUDEDIR})
if(NOT JWT_EXTERNAL_PICOJSON AND NOT JWT_DISABLE_PICOJSON)
  install(FILES ${JWT_INCLUDE_PATH}/picojson/picojson.h DESTINATION ${CMAKE_INSTALL_INCLUDEDIR}/picojson)
endif()
install(FILES ${CMAKE_CURRENT_BINARY_DIR}/jwt-cpp-config.cmake ${CMAKE_CURRENT_BINARY_DIR}/jwt-cpp-config-version.cmake
        DESTINATION ${JWT_CMAKE_FILES_INSTALL_DIR})

if(JWT_BUILD_EXAMPLES)
  add_subdirectory(example)
endif()

if(JWT_BUILD_TESTS)
  add_subdirectory(tests)
endif()

if(JWT_ENABLE_FUZZING)
  add_subdirectory(tests/fuzz)
endif()

Below is make install output
$ make install
Install the project...
-- Install configuration: "Release"
-- Installing: /XXX//jwt-cpp-0.6.0/build/install/cmake/jwt-cpp-targets.cmake
-- Installing: /XXX//jwt-cpp-0.6.0/build/install/include/jwt-cpp
-- Installing: /XXX//jwt-cpp-0.6.0/build/install/include/jwt-cpp/.DS_Store
-- Installing: /XXX//jwt-cpp-0.6.0/build/install/include/jwt-cpp/traits
-- Installing: /XXX//jwt-cpp-0.6.0/build/install/include/jwt-cpp/traits/kazuho-picojson
-- Installing: /XXX//jwt-cpp-0.6.0/build/install/include/jwt-cpp/traits/kazuho-picojson/traits.h
-- Installing: /XXX//jwt-cpp-0.6.0/build/install/include/jwt-cpp/traits/kazuho-picojson/defaults.h
-- Installing: /XXX//jwt-cpp-0.6.0/build/install/include/jwt-cpp/traits/nlohmann-json
-- Installing: /XXX//jwt-cpp-0.6.0/build/install/include/jwt-cpp/traits/nlohmann-json/traits.h
-- Installing: /XXX//jwt-cpp-0.6.0/build/install/include/jwt-cpp/traits/nlohmann-json/defaults.h
-- Installing: /XXX//jwt-cpp-0.6.0/build/install/include/jwt-cpp/traits/danielaparker-jsoncons
-- Installing: /XXX//jwt-cpp-0.6.0/build/install/include/jwt-cpp/traits/danielaparker-jsoncons/traits.h
-- Installing: /XXX//jwt-cpp-0.6.0/build/install/include/jwt-cpp/traits/danielaparker-jsoncons/defaults.h
-- Installing: /XXX//jwt-cpp-0.6.0/build/install/include/jwt-cpp/traits/boost-json
-- Installing: /XXX//jwt-cpp-0.6.0/build/install/include/jwt-cpp/traits/boost-json/traits.h
-- Installing: /XXX//jwt-cpp-0.6.0/build/install/include/jwt-cpp/traits/boost-json/defaults.h
-- Installing: /XXX//jwt-cpp-0.6.0/build/install/include/jwt-cpp/traits/defaults.h.mustache
-- Installing: /XXX//jwt-cpp-0.6.0/build/install/include/jwt-cpp/jwt.h
-- Installing: /XXX//jwt-cpp-0.6.0/build/install/include/jwt-cpp/base.h
-- Installing: /XXX//jwt-cpp-0.6.0/build/install/include/picojson/picojson.h
-- Installing: /XXX//jwt-cpp-0.6.0/build/install/cmake/jwt-cpp-config.cmake
-- Installing: /XXX//jwt-cpp-0.6.0/build/install/cmake/jwt-cpp-config-version.cmake

This is generated Makefile
# CMAKE generated file: DO NOT EDIT!
# Generated by "Unix Makefiles" Generator, CMake Version 3.22

# Default target executed when no arguments are given to make.
default_target: all
.PHONY : default_target

# Allow only one "make -f Makefile2" at a time, but pass parallelism.
.NOTPARALLEL:

#=============================================================================
# Special targets provided by cmake.

# Disable implicit rules so canonical targets will work.
.SUFFIXES:

# Disable VCS-based implicit rules.
% : %,v

# Disable VCS-based implicit rules.
% : RCS/%

# Disable VCS-based implicit rules.
% : RCS/%,v

# Disable VCS-based implicit rules.
% : SCCS/s.%

# Disable VCS-based implicit rules.
% : s.%

.SUFFIXES: .hpux_make_needs_suffix_list

# Command-line flag to silence nested $(MAKE).
$(VERBOSE)MAKESILENT = -s

#Suppress display of executed commands.
$(VERBOSE).SILENT:

# A target that is always out of date.
cmake_force:
.PHONY : cmake_force

#=============================================================================
# Set environment variables for the build.

# The shell in which to execute make rules.
SHELL = /bin/sh

# The CMake executable.
CMAKE_COMMAND = /usr/local/Cellar/cmake/3.22.3/bin/cmake

# The command to remove a file.
RM = /usr/local/Cellar/cmake/3.22.3/bin/cmake -E rm -f

# Escaping for special characters.
EQUALS = =

# The top-level source directory on which CMake was run.
CMAKE_SOURCE_DIR = /Users/XXX/Downloads/jwt-cpp-0.6.0

# The top-level build directory on which CMake was run.
CMAKE_BINARY_DIR = /Users/XXX/Downloads/jwt-cpp-0.6.0/build

#=============================================================================
# Targets provided globally by CMake.

# Special rule for the target edit_cache
edit_cache:
	@$(CMAKE_COMMAND) -E cmake_echo_color --switch=$(COLOR) --cyan "Running CMake cache editor..."
	/usr/local/Cellar/cmake/3.22.3/bin/ccmake -S$(CMAKE_SOURCE_DIR) -B$(CMAKE_BINARY_DIR)
.PHONY : edit_cache

# Special rule for the target edit_cache
edit_cache/fast: edit_cache
.PHONY : edit_cache/fast

# Special rule for the target rebuild_cache
rebuild_cache:
	@$(CMAKE_COMMAND) -E cmake_echo_color --switch=$(COLOR) --cyan "Running CMake to regenerate build system..."
	/usr/local/Cellar/cmake/3.22.3/bin/cmake --regenerate-during-build -S$(CMAKE_SOURCE_DIR) -B$(CMAKE_BINARY_DIR)
.PHONY : rebuild_cache

# Special rule for the target rebuild_cache
rebuild_cache/fast: rebuild_cache
.PHONY : rebuild_cache/fast

# Special rule for the target list_install_components
list_install_components:
	@$(CMAKE_COMMAND) -E cmake_echo_color --switch=$(COLOR) --cyan "Available install components are: \"Unspecified\""
.PHONY : list_install_components

# Special rule for the target list_install_components
list_install_components/fast: list_install_components
.PHONY : list_install_components/fast

# Special rule for the target install
install: preinstall
	@$(CMAKE_COMMAND) -E cmake_echo_color --switch=$(COLOR) --cyan "Install the project..."
	/usr/local/Cellar/cmake/3.22.3/bin/cmake -P cmake_install.cmake
.PHONY : install

# Special rule for the target install
install/fast: preinstall/fast
	@$(CMAKE_COMMAND) -E cmake_echo_color --switch=$(COLOR) --cyan "Install the project..."
	/usr/local/Cellar/cmake/3.22.3/bin/cmake -P cmake_install.cmake
.PHONY : install/fast

# Special rule for the target install/local
install/local: preinstall
	@$(CMAKE_COMMAND) -E cmake_echo_color --switch=$(COLOR) --cyan "Installing only the local directory..."
	/usr/local/Cellar/cmake/3.22.3/bin/cmake -DCMAKE_INSTALL_LOCAL_ONLY=1 -P cmake_install.cmake
.PHONY : install/local

# Special rule for the target install/local
install/local/fast: preinstall/fast
	@$(CMAKE_COMMAND) -E cmake_echo_color --switch=$(COLOR) --cyan "Installing only the local directory..."
	/usr/local/Cellar/cmake/3.22.3/bin/cmake -DCMAKE_INSTALL_LOCAL_ONLY=1 -P cmake_install.cmake
.PHONY : install/local/fast

# Special rule for the target install/strip
install/strip: preinstall
	@$(CMAKE_COMMAND) -E cmake_echo_color --switch=$(COLOR) --cyan "Installing the project stripped..."
	/usr/local/Cellar/cmake/3.22.3/bin/cmake -DCMAKE_INSTALL_DO_STRIP=1 -P cmake_install.cmake
.PHONY : install/strip

# Special rule for the target install/strip
install/strip/fast: preinstall/fast
	@$(CMAKE_COMMAND) -E cmake_echo_color --switch=$(COLOR) --cyan "Installing the project stripped..."
	/usr/local/Cellar/cmake/3.22.3/bin/cmake -DCMAKE_INSTALL_DO_STRIP=1 -P cmake_install.cmake
.PHONY : install/strip/fast

# The main all target
all: cmake_check_build_system
	$(CMAKE_COMMAND) -E cmake_progress_start /Users/XXX/Downloads/jwt-cpp-0.6.0/build/CMakeFiles /Users/XXX/Downloads/jwt-cpp-0.6.0/build//CMakeFiles/progress.marks
	$(MAKE) $(MAKESILENT) -f CMakeFiles/Makefile2 all
	$(CMAKE_COMMAND) -E cmake_progress_start /Users/XXX/Downloads/jwt-cpp-0.6.0/build/CMakeFiles 0
.PHONY : all

# The main clean target
clean:
	$(MAKE) $(MAKESILENT) -f CMakeFiles/Makefile2 clean
.PHONY : clean

# The main clean target
clean/fast: clean
.PHONY : clean/fast

# Prepare targets for installation.
preinstall: all
	$(MAKE) $(MAKESILENT) -f CMakeFiles/Makefile2 preinstall
.PHONY : preinstall

# Prepare targets for installation.
preinstall/fast:
	$(MAKE) $(MAKESILENT) -f CMakeFiles/Makefile2 preinstall
.PHONY : preinstall/fast

# clear depends
depend:
	$(CMAKE_COMMAND) -S$(CMAKE_SOURCE_DIR) -B$(CMAKE_BINARY_DIR) --check-build-system CMakeFiles/Makefile.cmake 1
.PHONY : depend

# Help Target
help:
	@echo "The following are some of the valid targets for this Makefile:"
	@echo "... all (the default if no target is provided)"
	@echo "... clean"
	@echo "... depend"
	@echo "... edit_cache"
	@echo "... install"
	@echo "... install/local"
	@echo "... install/strip"
	@echo "... list_install_components"
	@echo "... rebuild_cache"
.PHONY : help

#=============================================================================
# Special targets to cleanup operation of make.

# Special rule to run CMake to check the build system integrity.
# No rule that depends on this can have commands that come from listfiles
# because they might be regenerated.
cmake_check_build_system:
	$(CMAKE_COMMAND) -S$(CMAKE_SOURCE_DIR) -B$(CMAKE_BINARY_DIR) --check-build-system CMakeFiles/Makefile.cmake 0
.PHONY : cmake_check_build_system

I appogize, my question was not clear. Let me try again!
I wanted to know how are you building the example code? I am curious how you are getting the compilation error. What build command are you using to get it?
You posted CMake logs so I assumed (incorrectly) you were building it separately. Also you have EXAMPLES=OFF in the configure which further lead me astray.

Sorry, I made a mistake.
When I compile, I disable DJWT_DISABLE_BASE64, so when decode, It shows No matching function for call to 'decode'.

I am glad you solved your issue!
If there is anything you can suggest to improve the example please let us know, always looking to improve ❤


Allow user to disable update prompt

I added an option, $DISABLE_UPDATE_PROMPT, which is checked for in "check_for_upgrade.sh" that will skip asking the user to confirm the update and just go ahead and do it if it's time to check for updates again.

This would a great addition. There are a lot of times when I ssh into a remote machine and I am prompted for an update. I always answer yes so this would definitely be a timesaver for me.

+1

Perhaps it should be named $AUTO_UPDATE or something similar? The current name could be confused with disabling update all together.

I chose DISABLE_UPDATE_PROMPT because DISABLE_AUTO_UPDATE is already used for the actual update checks already in oh-my-zsh.

Ah, makes sense.

Thanks for this. It'd be great if someone submitted another patch that would ask people, "do you want to update? yes / no... " and then say, "yes, don't ask me anymore" and do just that...


fix: adds alias to match recurly api docs

Hey there.
It looks like there's a slight mismatch between the API exposed here and the one documented in the README.md and Recurly.
I've created an additional and duplicate method to avoid breaking anyone's current implementation.

Heya,
Please let me know if there's any further information or changes you would like that is blocking this from being merged.
Many thanks,
Luke

Thanks @thgreasi I have updated the PR to include a note about removing the duplication at a later date. @umayr please let me know if there's anything I can do here.

👋 Just wanted to follow up on this PR and see if there was anything further I can do to support this being merged.

I will cut a release tomorrow morning :+1:

Many thanks @thgreasi

We thank you @thisislawatts


[BUG]  Nav bar not visible in login page

Description
The nav bar is not visible in the login and signup page
Screenshots

Any additional information?
No response
What browser are you seeing the problem on?
No response
Checklist

[X] I have checked the existing issues
[X] I have read the Contributing Guidelines
[X] I'm a GSSoC'24-Extd contributor
[X] I'm a Hacktoberfest'24 contributor
[X] I am willing to work on this issue (optional)

@somyadipghosh Please assign me this issue


How do I define my own color scheme for Table Heatmap?

Hi,
I would like to use custom color scales in Table Heatmap, just like the Montreal Weather 2014 Dataset example for pivottable.js. How do I do this in react-pivottable?

This code snippet shows how to use the makeRenderer to create a renderer: https://github.com/plotly/react-pivottable/blob/33c77e9bb9a82a47f6eba4e51284ca11a603b3cd/src/TableRenderers.jsx#L347-L354
You can import makeRenderer from TableRenderers to create your own renderer, and pass it to your react-pivottable (see README for an example)

This code snippet shows how to use the makeRenderer to create a renderer: https://github.com/plotly/react-pivottable/blob/33c77e9bb9a82a47f6eba4e51284ca11a603b3cd/src/TableRenderers.jsx#L347-L354
You can import makeRenderer from TableRenderers to create your own renderer, and pass it to your react-pivottable (see README for an example)

how do you import something that is not exported?  The makeRenderer() function needs to be exported in order to import it to customize it.

did you ever figure out how to do this @jheckman-rt  @deecay?

You should be able set a tableColorScaleGenerator prop that returns a colorscale function.

Here's an example that uses d3 scale chromatic and the tableColorScaleGenerator prop @nicolaskruchten mentioned.
import { interpolateRdYlGn } from "d3-scale-chromatic";

      <PivotTableUI
        data={data}
        onChange={(s) => setState(s)}
        renderers={Object.assign({}, TableRenderers, PlotlyRenderers)}
        tableColorScaleGenerator={(values) => {
          const min = Math.min.apply(Math, values);
          const max = Math.max.apply(Math, values);
          const range = max - min;
          return (x) => {
            const progress = (max - x) / range;
            const backgroundColor = interpolateRdYlGn(1 - progress)
            return { backgroundColor };
          };
        }}
        {...state}
      />

If you install d3 scale chromatic you can use any of their color scale functions which take a number from 0 to 1 and return a color.
If you want to roll your own, you can still follow the basic structure above. Essentially, you'll just pass a function to the tableColorScaleGenerator prop that accepts a list of values and returns another function which takes one of those values and returns a CSS-in-JS object like {backgroundColor: myCalculatedBgColor} with the appropriate background color for the passed value.

Thanks @nicolaskruchten , @ScottGuthart
Everything clear now. The code example was super helpful. I have decided to create a different kind of lighten function to preserve the gradient in the lighter area.
  function lighten (color, amount) {
    const r = d3.rgb(color).r;
    const g = d3.rgb(color).g;
    const b = d3.rgb(color).b;
    const newColor = d3.rgb(
      ( r + (255 - r) * amount ),
      ( g + (255 - g) * amount ),
      ( b + (255 - b) * amount ),
    );
    return newColor.toString();
  }

This thread is very useful. In my case, I used d3's scaleLinear function to define the min, max and the range of colors. Example:
`
const minColor = 'white'
const maxColor = 'blue'
const tableColorScaleGenerator = (values) => {
const min = Math.min.apply(Math, values);
const max = Math.max.apply(Math, values);
const myColor = d3
.scaleLinear()
.domain([min, max])
.range([minColor, maxColor]);
	return (x) => {
		return { backgroundColor: `${myColor(x)}` };
	};
};

`


Removed coverage reports from repo fixes #4

removes coverage from repo and adds ignore to .gitignore ⛔

If you have access to the mobile center you will see that this branch builds and passes all tests 🎉


TPA: Merge DAG nested loops

This is an implementation of the merging of the nested loops into a single Transition System. It is done via the same approach as the transformation of the whole instance into the single System, but is adapted to only merge loops in the Graph structure into a single node.

Ok, I think generally the code was moved and main upds are done
Pls take a look to understand if it's Ok architecturally or if some additional cleanup is needed!

Can you please also add tests for the new functionality?

Done, added a couple of tests on merging the loops!

Can you please rebase?

And squash the commits? I think this can be a single commit, no?


How to generate a batch Ramachandran plot?

Provide the command if possible.

I updated the README.md (commit 4c6267c) to illustrate more clearly how you can do that.


IsLocalHost: support host:port

Without this patch IsLocalHost does not work for URLs with port specified i.e. it works for http://localhost but does not work for http://localhost:80 or http://localhost:10000.
Fixes #487

LGTM


fix: Improvements to universal installer script

fixes #906
- What I did
Numerous fixes to get script working as expected on Linux
- How I did it
Test iterations on clean Ubuntu 22.04 and Amazon Linux 2023 VMs
- How to verify it
Run against a test VM of your choice.
- Description for the changelog
fix: Improvements to universal installer script

I've opened this mainly so that @XavierChanth can keep an eye on what I'm doing

Superseded by #913


Coloring the notification

Hi.
I'm wondering if is there any way to somehow change the color of notifications ?

Color of what specifically ?

I'm using push.js for windows desktop notification. the background color of notifications on windows 10 is too white. in addition to it, I'm using arabic language for notification body and the default font is so ugly.
I'm looking for a way to change background color and default font of windows desktop notifications.

No that is not possible according to the Notifications API spec.


New package: BellSoft.LibericaJRE.13 version 13.0.2+9

Pull request has been created with Komac v1.11.0 :rocket:

Resolve https://github.com/microsoft/winget-pkgs/issues/113719

Microsoft Reviewers: Open in CodeFlow

@wingetbot waivers Add Validation-Domain


Upgrade Oracle Java JDK to 8.51

Thank you for the contribution. It was merged directly as 1b4629b817f1b4d1d78051f1d5aa467e050a4cd5 to keep commit history cleaner. Your contribution is still credited to you.


Error facing docker installation locally

After locally installing ASL following "https://cybercentrecanada.github.io/assemblyline4_docs/installation/appliance/docker/", I was faced with incorrect credentials.
Environment :
Ubuntu 22.04.3
Fix
To fix it I needed to run this command at the very beginning of Docker installation. Which I did not find anywhere in the walk-through.
sudo apt install python3-pip -y
pip3 install pip -U
sudo pip install assemblyline_client
Proceed with Docker installation from link above.

Not sure I understand how installing the Assemblyline's Python client on the host would fix an invalid credentials error.
What's likely to have happened is if you tried to login before the bootstrap container initialized the admin account (which there can be a delay depending on how long it takes the dependencies to achieve a ready state), you would've encountered the invalid login error because the account doesn't exist in the system.
Can you provide more details if that wasn't the case?

Any updates on this issue?

Will close the issue. If there are any updates or if this issue still persists, feel free to re-open.