[FEATURE] Adding A Share Section on the profile page

Description
Here's the current linkfree profile view

Here's bio.link share section

Similar share section needs to be added to linkfree.

Start with the copy link setting.
Then perhaps you could add the twitter option of sharing
Then maybe linkedin and the rest step-by-step

Screenshots
No response
Additional information
No response

I suggest we can use Javascript navigator.share to implement this feature. Can I be assigned this issue please?

I suggest we can use Javascript navigator.share to implement this feature. Can I be assigned this issue please?

@shoebilyas123 I have assigned this to you

I suggest we can use Javascript navigator.share to implement this feature. Can I be assigned this issue please?

@shoebilyas123 I have assigned this to you

Thank you. I'll try to raise a PR by tomorrow.

Hi, everyone i have understand the issue fully and can work on it ASAP.

Hi, everyone i have understand the issue fully and can work on it ASAP.

@kunalshokeen051 before requesting issues to be assigned to you, please check the Assignees box at the top of the page - this issue has been assigned to someone else. By checking before commenting this will avoid extra notifications for community members and maintainers as well.
Here is a link to our Contributing Guide which you may want to take a look at so that you can see the guidelines for contributing to this project.

Thoughts on this share profile UI? Any suggestions for improvement?


EventHub 2024-05-01-preview

ARM (Control Plane) API Specification Update Pull Request

[!TIP]
Overwhelmed by all this guidance? See the Getting help section at the bottom of this PR description.

[!NOTE]
As of January 2024 there is no PR assignee. This is expected. See https://aka.ms/azsdk/pr-arm-review.

PR review workflow diagram
Please understand this diagram before proceeding. It explains how to get your PR approved & merged.

Click here to see the details of Step 1, Breaking Changes review
If you are in purview of Step 1 of the diagram, follow the Breaking Changes review process.
IMPORTANT! This applies even if you believe your PR was mislabeled, for any reason, including tool failure.

Click here to see the details of Step 2, ARM review
See https://aka.ms/azsdk/pr-arm-review.

 Click here to see the diagram footnotes
Diagram footnotes
[1] See ARM review queue (for PR merge queues, see [2]).
[2] public repo merge queue, private repo merge queue (for ARM review queue, [1])
The ARM reviewer on-call engineer visits the merge queue twice a day, so the approximate ETA for merges is 12 - 24 hours.

Purpose of this PR
What's the purpose of this PR? Check the specific option that applies. This is mandatory!

[ ] New resource provider.
[ ] New API version for an existing resource provider. (If API spec is not defined in TypeSpec, the PR should have been generated using OpenAPI Hub).
[ ] Update existing version for a new feature. (This is applicable only when you are revising a private preview API version.)
[ ] Update existing version to fix OpenAPI spec quality issues in S360.
[ ] Other, please clarify:

edit this with your clarification

Due diligence checklist
To merge this PR, you must go through the following checklist and confirm you understood
and followed the instructions by checking all the boxes:

[ ] I confirm this PR is modifying Azure Resource Manager (ARM) related specifications, and not data plane related specifications.
[ ] I have reviewed following Resource Provider guidelines, including
ARM resource provider contract and
REST guidelines (estimated time: 4 hours).
I understand this is required before I can proceed to the diagram Step 2, "ARM API changes review", for this PR.

Additional information

 Viewing API changes
For convenient view of the API changes made by this PR, refer to the URLs provided in the table
in the Generated ApiView comment added to this PR. You can use ApiView to show API versions diff.

Suppressing failures
If one or multiple validation error/warning suppression(s) is detected in your PR, please follow the
suppressions guide to get approval.

Getting help

First, please carefully read through this PR description, from top to bottom. Please fill out the Purpose of this PR and Due diligence checklist.
To understand what you must do next to merge this PR, see the Next Steps to Merge comment. It will appear within few minutes of submitting this PR and will continue to be up-to-date with current PR state.
For guidance on fixing this PR CI check failures, see the hyperlinks provided in given failure
and https://aka.ms/ci-fix.
For help with PR workflow diagram Step 2 (ARM review), see https://aka.ms/azsdk/pr-arm-review.
If the PR CI checks appear to be stuck in queued state, please add a comment with contents /azp run.
This should result in a new comment denoting a PR validation pipeline has started and the checks should be updated after few minutes.
If the help provided by the previous points is not enough, post to https://aka.ms/azsdk/support/specreview-channel and link to this PR.

 Please get a breaking changes approval from Azure Breaking Changes Reviewers<EMAIL_ADDRESS>and ARM can review the changes afterwards. Once you have the approval, please remove the "ARMChangesRequested" label from the PR to make the PR visible to the ARM reviewers.

/azp run

/azp run

There are a bunch of breaking changes reported on this PR. Please get a breaking changes approval from Azure Breaking Changes Reviewers<EMAIL_ADDRESS>and ARM can review the changes afterwards. Once you have the approval, please remove the "ARMChangesRequested" label from the PR to make the PR visible to the ARM reviewers.

Hey @ms-henglu , Now there is no breaking on the PR
Can you pls review the PR now.

/azp run

Hey @mikeharder , can we suppressed the Swagger LinDiff, Avacado Pipeline error here?
Also Where can I get the sdk-for-js Approval

/azp run

Hey @mikeharder , can we suppressed the Swagger LinDiff, Avacado Pipeline error here? Also Where can I get the sdk-for-js Approval

Whats the reason for not being able to fix the lintdiff errors? There are a ton of violations that cannot be approved. https://github.com/Azure/azure-rest-api-specs/pull/28450/checks?check_run_id=23489128434

/azp run

Hey @mikeharder , can we suppressed the Swagger LinDiff, Avacado Pipeline error here? Also Where can I get the sdk-for-js Approval

Whats the reason for not being able to fix the lintdiff errors? There are a ton of violations that cannot be approved. https://github.com/Azure/azure-rest-api-specs/pull/28450/checks?check_run_id=23489128434

Hey @rkmanda, I have copied from 2023-01-01-preview and renamed it to 2024-01-01-preview with few more addition. The point is the previous version is also we got approval. this LintDiff error are coming from the files that are exact same as 2023-01-01-preview.

/azp run

Hey @mikeharder , can we suppressed the Swagger LinDiff, Avacado Pipeline error here? Also Where can I get the sdk-for-js Approval

Whats the reason for not being able to fix the lintdiff errors? There are a ton of violations that cannot be approved. https://github.com/Azure/azure-rest-api-specs/pull/28450/checks?check_run_id=23489128434

Hey @rkmanda, I have copied from 2023-01-01-preview and renamed it to 2024-01-01-preview with few more addition. The point is the previous version is also we got approval. this LintDiff error are coming from the files that are exact same as 2023-01-01-preview.

For the reference I am adding the Old PR
https://github.com/Azure/azure-rest-api-specs/pull/26672
This is the PR where we got LintDiff error approved.
If we fix something here many SDK which will generate from this Swagger will cause of backward compatibility of SDK.
CC - @mikeharder

/azp run

/azp run

/pr RequestMerge


Which branch should I pull request on?

I find the branch mrubyc3.1 is under active development, but I can't test it using mrubyc-test on account of some bugs which I can't fix for now.  I'm not using docker, instead I just install mrubyc-test gem directly .
However, I can test the master branch well, and I've already implemented some lacking pieces of the code.  Can I pull request on the master branch?

@ccmywish Could you push your branch to ccmywish/mrubyc  so that we can see it? If it looks able to be merged into master regarding our current development strategy, we are happy to do it. Even if not, we will be willing to find a good way to solve a problem

@hasumikin Hi, I've pulled a request here #126


What with Raspberries ?

what is the fate of vitmalina ?
what's wrong with him ?
my email<EMAIL_ADDRESS>

@mrg2001 / @vitmalina Not a w2ui issue, this topic can probably be closed.


This also needs a followup on the docs repo, since this file is deployed to the website: https://github.com/nginx/unit-docs/blob/main/Makefile#L58

          This also needs a followup on the docs repo, since this file is deployed to the website: https://github.com/nginx/unit-docs/blob/main/Makefile#L58

Originally posted by @thresheek in https://github.com/nginx/unit/issues/1430#issuecomment-2353556039

What is the AC here @ac000 @callahad ? the ticket needs more context, ideally without having to jump to other tickets/PRs


Suggestion: If init is called without a name, throw instead of console.error

https://github.com/mixpanel/mixpanel-js/blob/c0c19e7a374fbcebeb3944e6410087ea4c4ec35e/src/mixpanel-core.js#L210
Currently when Mixpanel.init is called without a name param (even though it's listed as optional in the API reference), the function just returns and an error is logged to the console. It would be helpful from a DX and TypeScript point of view if you would actually throw an error:
if (_.isUndefined(name)) {
    throw new Error('You must name your new library: init(token, config, name)');
}

This would have 2 advantages:

It would force the users of the library to provide a name property to the library; not just silently fail to the console. This would make it more obvious to new developers that there is a problem and what the problem is.
It would make the return type for TypeScript much more clear. It could just be a Mixpanel instance instead of possibly being undefined (see the discussion over at DefinitelyTyped).

If you agree with these changes, I can put up a PR.

Closing because the init function I referenced isn't the one used directly by the public API but is instead this one and it does have an optional name param


Explanation of the BigInteger.js codebase?

Hi there, I am digging into the codebase and wondering if you could do a quick breakdown of what the major parts are. For the first thing, what is the need for SmallInteger and Integer and NativeBigInt, as opposed to BigInteger?

It uses different functions for calculating operations depending on the situation. If the environment supports BigInteger natively, then it will take advantage of that and use NativeBigInt.
Otherwise, if the number is small enough to use native JavaScript numbers to perform the calculation, it will use SmallInteger. Otherwise it calculate with arrays of JavaScript numbers.


ImportError: cannot import name 'build_info' when using command import tensorflow as tf

I can successfully follow the tutorial of installing tensorflow (CPU version) with Anaconda yesterday. However, after trying to apply the GPU version and download CUDA9.0, CuDNN v7.1.4 for CUDA9.0, suddenly everything gets wrong,,,

PLEASE CAN ANYONE HELP ME...T_T  I am new to it as I work for it new because of my final year project.

I'm not sure how this happened, but it sounds like a messed up installation.
Note that the conda packages are community supported (see https://www.tensorflow.org/install/install_linux#use_pip_in_anaconda).
Could you use pip instead?

@asimshankar thank you for your reply! I am so happy that people are so helpful!!
I solve the problem by deleting the tensorflow folder in my C:\user\myusername and download it again and now the GPU version work for me!
However, when I follow the Simple Audio Recognition tutorial from https://www.tensorflow.org/tutorials/sequences/audio_recognition to the last step, I need to run these command "python tensorflow/examples/speech_commands/freeze.py
--start_checkpoint=/tmp/speech_commands_train/conv.ckpt-18000
--output_file=/tmp/my_frozen_graph.pb"
When I just enter "python tensorflow/examples/speech_commands/freeze.py " my cmd got python error and show OP_REQUIRES failed at save_restore_tensor.cc:170 : Invalid argument: Unsuccessful TensorSliceReader constructor: Failed to get matching files on : Unknown: FindFirstFile failed for: ./Application Data : 存取被拒。
; Input/output error
What does it mean? Can anyone help me please!!

@polariskying,
Can you confirm if you're still facing the issue? Thanks!


proposal to make Thulie as codeowner of website

Reason/Context
I don't think we need to clarify the reason to make @thulieblack as the CODEOWNER of the website as she is the Community Manager of the AsyncAPI :sweat_smile:. The discussion goes on whether should we make Thulie as Codeowner for all the files or for only markdown files as we did for @alequetzalli.
Description
We need the following things to be done to make @thulieblack as CODEOWNER:

Give maintainer/admin access to the @thulieblack
Update the CODEOWNERS file to add Thulie

Inviting @derberg @alequetzalli @thulieblack for the discussion to add their points :smile:

Yes, while she is contributing her time as a community manager she will def be able to do her work easier if she has codeowner rights. Especially with merging, otherwise it'll be a mess everytime. While she is not a core docs contributor who will automatically merge docs, she will need merging rights to do her work such as scheduling blog posts and beyond.
I don't have an opinion on what degree she is given codeowner rights... if she is not touching JS she may not need more than mardown access, but I don't think it should matter which one.

@akshatnema will you work on a PR? technically best would be to make @thulieblack owner of community and blog pages. And technically (again) I think you will also have to add @alequetzalli to the same configuration so she is still an owner there. You know what I mean?


Not clear if multiple activations are possible within an Assignment Period

Ive read through all the docs for PIM and it does not clarify if an assigned role can be activated multiple times or just the once.
E.g   if the assignment period is set to expire after 3 months and activation duration is set to 8 hours, can a user activate the role Monday morning, work with it that day for 8 hours until the duration is over and then activate it again the following day (multiple times in 3 months)? Or is it possible to just do a single activation only and further activation would require an admin to reenable?

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

ID: 43125fc1-f1fb-fa8c-69a6-7866a1b3a1fe
Version Independent ID: 174dbe2c-7469-a63a-ffd6-757272ce5c50
Content: Activate my Azure AD roles in PIM - Azure Active Directory
Content Source: articles/active-directory/privileged-identity-management/pim-how-to-activate-role.md
Service: active-directory
Sub-service: pim
GitHub Login: @curtand
Microsoft Alias: curtand

@philrice , Thanks for raising the issue. We will investigate and update you.

@philrice We are trying to get more information internally on this and hence it is taking more time. We will update this thread once we have more information.

@philrice, Thank you for reaching out. Your anticipation is correct to some extent. Let me walk you though with the example you shared. Once you create a role assignment, you specify the expiration time for the role to be active for users (when the role is activated once) is specified under the Azure AD roles - Settings. So, this statement "if the assignment period is set to expire after 3 months and activation duration is set to 8 hours, can a user activate the role Monday morning, work with it that day for 8 hours until the duration is over and then activate it again the following day (multiple times in 3 months)" holds true. But, the second part "Or is it possible to just do a single activation only and further activation would require an admin to reenable?". Everytime the user himself has to active the same and if an Approver is alinged for this role, then user has to wait for the Approver to approve his request manually.
Hope this helps.

Closing this thread for now, feel free to re-open this if any more queries around this ask.


make build: render the mappings.yml file before aws-stack.yml

Rendering aws-stack.yml depends on the mappings.yml file existing. I previously setup that dependency directly (see PR #706), however that broke CI where we want to manually manage the mappings.yml file.
This seems like it might be a way to ensure make build works in our development environment (as documented in the README) while also keeping the CI pipeline working.

Here's the relevant part of the CI pipeline that #706 broke: https://github.com/buildkite/elastic-ci-stack-for-aws/blob/87d7f1dc472cc71e407fbe4167ad6f499f36d033/.buildkite/steps/publish.sh#L26
In that script we download the correct mappings.yml file from buildkite artifacts and place it in the suitable place on disk where make build/aws-stack.yml will find and use it. We don't want make to rebuild it with us-east-1 only!

Unfortunately non-master branches only push AMIs to us-east-1 (see https://github.com/buildkite/elastic-ci-stack-for-aws/blob/87d7f1dc472cc71e407fbe4167ad6f499f36d033/.buildkite/steps/copy.sh#L103-L112) so to test this fully we have to merge and see what happens
:crossed_fingers:

I've confirmed the aws-stack.yml built after this merged has all regions in the mappings :tada:
See the annotation at the top of this build: https://buildkite.com/buildkite-aws-stack/buildkite-aws-stack/builds/2102#_


#49: added minVoteTime

Maybe not the best solution but it works.

Hi reCoNico... :)
sry can you resync the repo and add the PR/s again. Had to many changes in the develop branch - now we are off. So it's best to do a clean PR (to be seen for monthly counts and less merging).
thank you @reConNico

re-implemented into new code structure


Add BS5 support

As hinted in #349, this PR isn't finished yet, depending on the feedback I'd resolve additional outstanding points and add to this PR.
That being, said, what's already included, is ready to be reviewed. It looks like a lot, but the largest part of the changes is in the tests, and that's mostly a lot of small snippets, like added/removed/moved classes and slight markup structure changes.

I've also updated my UI test app for this, however somehow composer isn't able to require commit hashes anymore for me, but that's a problem for tomorrow.

The flexbox attempt for inline forms wasn't the proper way, so if anyone's looking at it, just ignore that for now, I'll try to fix that next using rows/cols.

So I've decided to go for auto-sizing for inline forms, using columns that stack based on window size felt wrong.
However I'm not going to check off that point, as people might have thoughts about this solution.

AFAICT switches are just checkboxes with one additional class, so I've chosen to go with using a template variable instead of adding new templates. If anyone would like to see this solved using additional templates instead, speak now or forever hold your peace!

Here goes color controls. Why the concrete color() method you may ask? Because just like some of the other controls it doesn't require extra markup, just an additional class, so this way people can also create them separately, just like with text(), select(), etc.
They have problems with the auto-sizing inline forms, and just like with the validation problem I hope that it's going to be fixed in soon. If there's no progress some time soon, I'll maybe try to fix it myself.

In true just do it and apologize later fashion, here's Bootstrap Icons, FontAwesome as the default is gone, and the Icon font assets are now being included by default, making them work out of the box. Using FontAwesome (or any other icon library for that matter) is of course still possible.
Currently Bootstrap Icons do not ship with sizing classes, so I've added a custom CSS file with classes for now. I've borrowed them from FontAwesome, they seem to work fine, and it keeps things familiar. Maybe the people over at Bootstrap are open to adding sizing classes, I might at least open an issue later on.
ps. I'll ignore stickler's line length complaints for now, there's lots of old content hat exceeds the limit, but isn't checked. I'll clean up things once the upgrade is finished.

Here's what it looks like:

And what it can look like when used in for example flash messages:

And here's icon support for flash messages, so one doesn't have to hack them in with the message, or reimplement the flash element template.
The defaults look like the manually built examples in my previous comment.

Floating labels are neat... they don't work well for select controls in inline forms though, at least in cases where the label text is wider than the widest option text:

There isn't really much that could be done about it, at least not with the current method of laying out inline forms where the controls shrink to the minimum size required, with the labels being absolutely positioned they simply cannot contribute to the controls minimum size.
There's workarounds on the user's end, like specifying a fixed/minimum width for the control container, or building the inline layout manually, but I'm afraid I don't really have any good idea right now how this could be made to work better out of the box.

If anyone's wondering, this is what help text and errors for inlined checkboxes/switches in non-inline layouts looks like:

It's generally nice to have if one wants it, and allowing this will also make adding support for aria-describedby and, possibly custom help classes easier.

The other night I added autocomplete for bootstrap icons - I opened a PR now ( https://github.com/FriendsOfCake/bootstrap-ui/pull/350 ) for FormatHelper::icon()
I bet the HtmlHelper::icon() in this repo could also benefit from some kind of IdeHelper task that provides optional autocomplete for IDEs like PHPStorm on those magic strings (icon names).
This way you can just get the possible names from typeahead dropdown.

 Remember kids, if you just add enough code you'll eventually reach the minimum coverage ratio 😄

So that's it for now, I'm not going push further changes (except after review feedback of course). If anyone wants to review this big pile of changes in peace - now's your chance :)

So if there aren't any objections I'd merge this soon, and then maybe we can release an alpha/preview to possibly garner some feedback.

So here we go... as requested, I'll tag you now that it's merged @ADmad 😛

@ndm2 Do you need to actually do anything? :slightly_smiling_face: You can start the alpha/beta as you see fit.

Not really, I guess... you just said "tag me when it's merged" :) If you're cool with it, then I'll draft an alpha pre-release.

Did you want to try to make a PR for the sandbox, to showcase some live demo cases of it?

Maybe later when things are finalized, I'm not familiar with the Sandbox app yet.


Fix typo in forcefield.py

PR Summary:
I think I found this typo in forcefield.py:
 if not all([a.id for a in structure.atoms][0]):
     raise ValueError('Not all atoms in topology have atom types')

I think that [0] should not be there since it kills the purpose of all().
PR Checklist

[ ] Includes appropriate unit test(s)
[ ] Appropriate docstring(s) are added/updated
[ ] Code is (approximately) PEP8 compliant
[ ] Issue(s) raised/addressed?

Here is the commit where it got added. Between 0.7.3 and 0.7.4.
I think this is indeed a typo.


singleton: forward attributes

This only works in the long form with a named variable because of
constraints on where the vis metavariable can be used.
close #521

The visibility thing might be useless as this ends up in a closure. I'm open to input.

Supporting attributes looks good and is definitely really useful. I can't tell when the visibility spec could ever make a difference though - the only way anything outside of the macro itself can refer to the storage is by the &mut returned from the macro, so I don't think it's material whether the underlying static is pub or not. Unless there's some other compiler check I'm missing about how a reference to a static gets used?

Oh, true, that wasn't possible before but I suppose it is now. I think ideally the static should not be accessible from outside the macro... but using an export_name attribute and separate extern already requires unsafe so perhaps we can allow that.
I don't think there's any point including the visibility specifier as it seems like it can never make any difference, so I'd rather remove it.

@adamgreig removed.


Inaccurate total population of 51 million

The total population listed for Nepal using our current data set is 51 million. That seems very inaccurate!
Googling shows estimates between 26 and 30 million.
See https://beta.nepalmap.org/profiles/country-NP-nepal/
The inaccurate data are here: https://github.com/Code4Nepal/nepalmap_federal/blob/dev/sql/population.sql#L1750-L1751
Screenshot

@pratimakandel - would you like to take a look at this and submit a pull request within a week or so if you have time?

Fixed by #56
Thank you, @pratimakandel

screenshot after update


Laravel 9 support

why remove support for laravel 9?

@TavoNiievez @Naktibalda Any plans to support Laravel 9 soon? I would like to update but this is blocking me


TypeError when checking mime-types

Hello everyone.
I have an issue when trying to check for wrong mime-types, I have a simple function that checks if the mime-type is included in an array of mime-types, and responds with an 404 is type doesn't match, and otherwise I let formidable handle the file.
const isInvalidMimeType = fileMimeType => {
  let mimeTypes = [...]

  return (!mimeTypes.includes(fileMimeType)) ? true : false
}

form.onPart = function (part) {
  if (part.filename) {
    if (isInvalidMimeType(part.mime)) {
      return res.status(404).json({ 'status': 404, 'message': 'Wrong file type'})
    } else {
      form.handlePart(part)
    }
  }
}

My issue is that if it is the wrong filetype, formidable still tries to upload the file, so I get;
TypeError: Cannot read property 'path' of undefined

That error in turn kills the entire application, is there anyway to let formidable know that the file isn't suppose to be uploaded and just return the response?
You can find the whole project in here in this repo, repo:uploadz

is there anyway to let formidable know that the file isn't suppose to be uploaded and just return the response?

I'm pretty sure we discussed this in few issues for aborting/canceling the upload.

Thank you @tunnckoCore, returning req.destory() fixed the issue of the app destroying itself.
form.onPart = function (part) {
  if (part.filename) {
    if (isInvalidMimeType(part.mime)) {
      res.status(404).json({ 'status': 404, 'message': 'Wrong file type'})
      return req.destroy()
    } else {
      form.handlePart(part)
    }
  }
}

The fixed code it anyone wants it.

Great to hear that :)


⚠ RPC server - Figment Solana has degraded performance

In f565b87, RPC server - Figment Solana ($FIGMENT_SOLANA_RPC_URL) experienced degraded performance:

HTTP code: 429
Response time: 220 ms

Resolved: RPC server - Figment Solana performance has improved in d2b9eff.


[JENKINS-50483] - Make the plugin compatible with Jenkins 2.102+ (JEP-200)

https://issues.jenkins-ci.org/browse/JENKINS-50483
More info about JEP-200: https://jenkins.io/blog/2018/03/15/jep-200-lts/

[x] - Facelift the plugin so that in can be tested with Plugin Compat Tester
[x] - Add a smoke test to reproduce the issue. There is no other tests in the plugin
[x] - Fix the issue, stop persisting PrintStream within the AWSDviceFarmReporter. It is a bug in the plugin, build objects should not be persisted on the project level
[x] - Add Jenkinsfile fo ci.jenkins.io

@reviewbybees @jglick @nikhil-dabhade

@nikhil-dabhade Hi, do you still maintain the plugin? If no, would it be fine if I take temporary ownership to deliver the release?

I am going forward with Ownership transfer and merges according to the discussion in https://groups.google.com/forum/#!topic/jenkinsci-dev/aTuFiNHhTLY


Add autonomous bookstore client.

Change-Id: Ic582892c4d23712dd968c4fb555255de05ce6f83

What is this for?

This client allows for an easy and automated way to hit all the endpoints of the demonstration/test bookstore project. It's useful for both simulating traffic to a demo cluster and as a means to test whether the endpoints are acting as expected. It's integrated in a later commit of mine into the Jenkins end-to-end tests and replaces all the current manual curl calls.

LGTM (LGTM1 label added)

@mkibbe1993 @martinmaly I'm confused why we'd need this in e2e tests for a service broker, and I suspect I'm missing something fundamental to how they work. Can you point me to somewhere that explains what the tests are doing (maybe I missed it in-code somewhere)?

@arschles @mkibbe1993 We have a documentation gap. Before we fill it ... the goal of the e2e test is to spin up the k8s cluster, start up the catalog in it, and then use the catalog to spin up a service, hook a client to it, and expose it to traffic.
So not only does it test that catalog started and works correctly, but also that the application started (and hooked up) by the catalog works as expected.
I think that this kind of e2e test is within scope of our work (especially as we add support for SIPs etc, we'll need to ensure they work properly). At the same time, it wouldn't be a goal to spin up overly complex applications. This simple bookstore, IMO, strikes the right balance.

@martinmaly @mkibbe1993 I see! That answers a lot of my questions. I certainly don't need a full architecture diagram of e2e tests at this point, but I'd love to see some documentation in the next few weeks (maybe after e2e is in and working?). Can you create a GH issue for that?
LGTM (added LGTM2 label)

Veto pending further discussion. Rationale to follow...

@krancour ... not to bee too strict about this but so far your veto is unqualified and therefore unenforceable.

@martinmaly I'm typing up my rationale right now. I wanted to get the veto in before a 3rd LGTM came in.

imo, there's one more moving piece here than is necessary. The "client" here is a test program that drives requests to an app, which in turn relies on the service? Why not cut out the (web) app entirely and combine the idea of the client and app into a single, simpler component-- something that is deployed to the cluster (like an app) and makes assertions itself (like the client) about its own ability to communicate with the provisioned service?

@krancour Thanks for the suggestion. I wonder how the test would drive this self-validating component and get its success status, logs etc.
We considered ideas such as this for, for example, making sure that catalog components were deployed correctly (spinning up watcher container in the cluster). This is similar, but I am not sure how to get the right detail of feedback from this app.
If I understand correctly, you'd propose to turn this application into a k8s app and e2e test would spin it up rather than run locally?

I like that idea of combining the client & app, but it feels like an optimization, and not blocking to get e2e tests up and running. @krancour objections?

My thoughts are what we perhaps lose structural simplicity we gain in functionality. Since this is just a simple autonomous user for the bookstore app, it has applicability beyond just the e2e tests; it also allows us to easily automate user traffic and poke at a demo app at will through a simple call to the script. @martinmaly also suggested to me that it would be useful when we expand beyond just running in k8s.
I think the idea of a client/app hybrid is interesting, but it seems like a optimization that'll take time to implement that in turns narrows usability without too much gain, and at the moment, like @arschles said, we have something already working and optimizing on that would block getting e2e tests up.

To clarify on the 'running beyond k8s' bit ... I can imagine that we'll want a client to our test which doesn't run in k8s cluster (as well as one that does).

I'm amenable to the idea of having something sooner than later, so I'll remove my veto, but I'm going to open a separate issue to capture the need for an optimization here.

LGTM

My only comment on this one is that let's try to comments like https://github.com/kubernetes-incubator/service-catalog/pull/110#issuecomment-265000516 in the original comment for the PR rather than a "Change-Id" - which doesn't mean a lot to readers.

Why does this belong in script? -- it should go in hack or under contrib

doi - missed that - agreed. @mkibbe1993 can you open an PR to move it to where the samples are?

I'm fine with putting it in contrib although test may seem more appropriate since that's what it is - a test.

@martinmaly I'm ok with create a "tests" top level dir for non-unit tests - if that's where this belongs.

I'm fine with test/script.  Do not make test and tests


How to uncompress a 7z file in a installer exe file?

    "version": "2.1.5",
    "homepage": "http://listen1.github.io/listen1/",
    "license": "MIT",
    "architecture": {
        "64bit": {
            "url": "https://github.com/listen1/listen1_desktop/releases/download/v2.1.5/Listen1_2.1.5_win64.exe",
            "hash": "bf250170523c5a2d39c230fdfd94c3b0956cbdd17e59a18855ba84d1855f6709",
            "extract_dir": "$PLUGINSDIR/app-64.7z/"
        },
        "32bit": {
            "url": "https://github.com/listen1/listen1_desktop/releases/download/v2.1.5/Listen1_2.1.5_win32.exe",
            "hash": "a5a6e40696f94c8126c4fca1c497889634319318e0fe94345e6faaa282655f3b",
            "extract_dir": "$PLUGINSDIR/app-32.7z/"
        }
    },
    "bin": "Listen1",
    "shortcuts": [
        [
            "Listen1.exe",
            "Listen1"
        ]
    ],
    "checkver": {
        "github": "https://github.com/listen1/listen1_desktop/"
    },
    "autoupdate": {
        "architecture": {
            "64bit": {
                "url": "https://github.com/listen1/listen1_desktop/releases/download/v$version/Listen1_$version_win64.exe"
            },
            "32bit": {
                "url": "https://github.com/listen1/listen1_desktop/releases/download/v$version/Listen1_$version_win32.exe"
            }
        }
    }
}

https://github.com/lukesampson/scoop-extras/blob/master/bucket/marktext.json

https://github.com/lukesampson/scoop-extras/blob/master/bucket/marktext.json

Thank you for your prompt reply.
I have added pre_install, post_install, installer items, but it doesn't work. Listen1desktop installer exe file has the same dir structure as marktext installer exe file.
{
    "version": "2.1.5",
    "homepage": "http://listen1.github.io/listen1/",
    "license": "MIT",
    "extract_dir": "\\$PLUGINSDIR",
    "pre_install": "Get-ChildItem \"$dir\" -Exclude 'app-64.7z', 'app-32.7z' | Remove-Item -Force -Recurse",
    "architecture": {
        "64bit": {
            "url": "https://github.com/listen1/listen1_desktop/releases/download/v2.1.5/Listen1_2.1.5_win64.exe",
            "hash": "bf250170523c5a2d39c230fdfd94c3b0956cbdd17e59a18855ba84d1855f6709",
            "installer": {
                "script": "Expand-7zipArchive \"$dir\\app-64.7z\" \"$dir\""
            }
        },
        "32bit": {
            "url": "https://github.com/listen1/listen1_desktop/releases/download/v2.1.5/Listen1_2.1.5_win32.exe",
            "hash": "a5a6e40696f94c8126c4fca1c497889634319318e0fe94345e6faaa282655f3b",
            "installer": {
                "script": "Expand-7zipArchive \"$dir\\app-32.7z\" \"$dir\""
            }
        }
    },
    "post_install": "Remove-Item \"$dir\\app-64.7z\", \"$dir\\app-32.7z\"",
    "bin": "Listen1.exe",
    "shortcuts": [
        [
            "Listen1.exe",
            "Listen1"
        ]
    ],
    "checkver": {
        "github": "https://github.com/listen1/listen1_desktop/"
    },
    "autoupdate": {
        "architecture": {
            "64bit": {
                "url": "https://github.com/listen1/listen1_desktop/releases/download/v$version/Listen1_$version_win64.exe"
            },
            "32bit": {
                "url": "https://github.com/listen1/listen1_desktop/releases/download/v$version/Listen1_$version_win32.exe"
            }
        }
    }
}

Is there any documentation to help me how to generate JSON file for scoop?

You need to rename that exe file into 7z. url...Listen.exe#/dl.7z Then extraction will be possible.


BUG: erroneous detail coefficients returned by downcoef with level > 1

As pointed out by @jairongfu in #72, it seems that there is still a problem for downcoef with detail coefficients and level > 1.  Unfortunately, the tests i added in #72 only tested the approximation case!
example reproducing the problem:
import pywt
import numpy as np
from numpy.testing import assert_array_almost_equal
r = np.random.randn(16)
a2, d2, d1 = pywt.wavedec(r, 'haar', level=2)

assert_array_almost_equal(a2, pywt.downcoef('a', r, wavelet='haar', level=2))
assert_array_almost_equal(d2, pywt.downcoef('d', r, wavelet='haar', level=2))

The second assert_array_almost_equal call above fails, but the first passes.

I found the bug and will submit a pull request shortly.  For the downcoef case, double_dec_a should be called up until the last level (when double_dec_d would be called).

Fixed in gh-98.


Is the "Open" button the most natural required action?

In most pages or sections where we have a list of resources (projects, datasets, workflows, tasks), the required action to open the resource-specific view is to click the "Open" button in the "Options" column.
While testing fractal-web with @marcodeltufo, we realized that another possibility would be to make the resource name a link, so that the "Open" button would be replaced by clicking the link (or maybe the two could co-exist). The downside of this option, however, is that it is less homogeneous with the fact that multiple options ("Open", "Edit", "Delete", ...) are grouped together.
I don't have a strong opinion here, but I do think that clicking the resource name looks like a natural action to do in order to get to the resource page.

I agree that a link in the name looks more natural, as many web UIs use that approach.

I'm fine with the change overall. Let's also drop the Id column for workflows though to make it a bit more uniform


Ability to spread the files over several providers to get more storage

Hi,
I am not sure if this ever would be inside the scope of rclone.
But it would be awesome if had a local cache of all files and where they were stored and the ability to spread it out over multiple configs/providers. This would be handy for people that were using more than 10TB.

Interesting idea!  Probably beyond the scope of rclone, but maybe one day!

It could handled simple round-robin and on fail, try of the other provided storage options.
 $ rclone unsafesync /giantpool/ hubic1:/giantpool-part,hubic2:/giantpool-part --i-know-this-is-dangerous

There are a lot of union filesystems that can do this.  Perhaps use that in hte meantime.

This is now possible with https://rclone.org/union
I suggest closing this issue.

I agree - I'll close this now :-)

Housekeeping is fun 😸


TypeError: Cannot read property 'connection' of undefined

Hi, I'm trying to create express server extending horizon.io following docs. How can I get connection configuration to run rethink query?
I tried
const conn = horizon._reql_conn.connection();

as it is in a documentation, but it says
TypeError: Cannot read property 'connection' of undefined

Server version: 2.0.0
Client version: 2.0.0

Got it, it should be called on horizonServer instance and look somehow like this horizonServer._reql_conn.connection().


[Docs] The link is broken in https://docs.meshery.io/concepts/logical/models

Current State
The link of components examples in https://docs.meshery.io/concepts/logical/models is broken (404).
Desired State
We should find and link another model which have components definition.

Contributor Guides and Handbook

📚 Instructions for contributing to documentation

Meshery documentation site and source

🛠 Meshery Build & Release Strategy
🎨 Wireframes and designs for Meshery UI in Figma (open invite)
🙋🏾🙋🏼 Questions: Discussion Forum and Community Slack

cc @MUzairS15

Thank you, @RipulHandoo, for your contribution. My apologies, @leecalcote, I was unavailable at that time due to classes.


Update 853-Car-Fleet.py

Reverse is a parameter in the sorted() function and setting it to be true is equivalent to sorted()[::-1] but more intuitive

Thanks, @UWDavidWu!


Docs: missing default values for many tsconfig.json options

Page URL: https://www.typescriptlang.org/tsconfig
Issue: many of the options are missing the "Default" value on the side, such as allowJs and checkJs
Recommended Fix: add default values for every option

#2049 fixes this 👍🏻


Does this project support unity2017?

if not support, can yours add a example for unity 2017? because my project is create with unity2017, please

It hasn't been tested with Unity 2017


[Feature] Add versioning and optimistic concurrency handling

Note: will be an issue as prisma doesn't fully support this at the moment:
https://www.prisma.io/docs/guides/performance-and-optimization/prisma-client-transactions-guide#optimistic-concurrency-control

We have added transactions for handling donations amounts and vaults increments/decrements.
Closing the ticket with the suggestion to create separate issues for the specific logic in future


log-viewer is producing warnings when triming null values

in src/Logs/Log.php, line 114:
$this->message = trim($matches[static::$regexMessageKey] ?? null);

This produces a lot of warnings in laravel.log when there is no $matches[static::$regexMessageKey], because function trim expects a string.
There should be a previous condition to trim the message only when it is not null.

hey @AComasSamcla , thanks for the report! Fixed in v3.12.0 :)


Overlapping map boundaries when merging some maps

Problem
The map is incorrect as:
Boundaries overlap

European boundaries overlap and show the boundaries as distinct polygon shapes (see Switzerland), labels are overlapped and/or hidden

Some labels are outside the continent

Water bodies must be shown and coloured blue

Spread link for viewing the map: https://view.spreadviz.org/?output=94d05c76-0585-43f9-99d8-9e13973ebba0/f97a0171-7836-4f7c-8dc0-d4eeef938d21.json&maps=DK,FR,RU,BE,UK,PT,SE

Not sure if we can do sth about this @jpmonettas sifted the internet and the darkweb for the most comprehensive set of (free) geojson maps. Closing?


On what basis are these Images Hosted?

I am confused will the Images get hosted by the Hosting data Discord Api or the PC running it?

The default behaviour is to store the image on the machine you host the bot on.

Okay so If I'm correct this stores all files into the PC so we can see what they have put in? Correct?

Yes. By default all attachments sent to the bot via DMs are stored on the PC, and then linked in the thread using the bot's built-in web server (used to attachments and logs).
If you want to change this so they're reposted on a discord channel instead and then linked from there, you can set the following 2 options:
{
    "attachmentStorage": "discord"
    "attachmentStorageChannelId": "channel id here"
}


added custom error classes

Description
Created a new "errors" directory at https://github.com/dapr/js-sdk/tree/main/src/types , created the "HttpNotSupportedError" and "GrpcNotSupportedError" classes and included them where needed in the codebase.
Issue reference
Resolves #297

Hi @jriffs could you please fix the DCO error? See https://github.com/dapr/js-sdk/pull/497/checks?check_run_id=14054037012

Hi @jriffs could you look at the build?

Overall the PR LGTM!

IF the last remarks are fixed (and the test pass) LGTM for me as well

Hi @jriffs could you look at the build?

Hi @shubham1172 , Thank you for the earlier pointers. i would absolutely appreciate if you could help me with pointers on how to resolve the issues with the build. this is my first time contributing to open source and i really don't know what the issue is or how to resolve it.
Thank you.

Hi @jriffs, sure! You can checkout this SO link to rebase with master https://stackoverflow.com/questions/7929369/how-to-rebase-local-branch-onto-remote-master
Regarding build, the linter is failing.

You can fix that by running npm run pretty-fix.

Hi @jriffs, sure! You can checkout this SO link to rebase with master https://stackoverflow.com/questions/7929369/how-to-rebase-local-branch-onto-remote-master
Regarding build, the linter is failing.

You can fix that by running npm run pretty-fix.

@jriffs looks like you deleted the .gitignore and checked a lot of files which shouldn't be here. Either you can fix it here in this PR, or alternatively you could create a fresh PR from the latest main branch - that would be easier and we will be able to check-in your first OSS contribution ⚡

@jriffs looks like you deleted the .gitignore and checked a lot of files which shouldn't be here. Either you can fix it here in this PR, or alternatively you could create a fresh PR from the latest main branch - that would be easier and we will be able to check-in your first OSS contribution ⚡

@shubham1172 Yes I think that would be best. I do have a couple of questions,
Do I run npm run build before I commit and push the code, if not what commands or processes should I run after writing the code, so I don't encounter the issues I had before (linter failing … etc.).
Thank you.

@jriffs looks like you deleted the .gitignore and checked a lot of files which shouldn't be here. Either you can fix it here in this PR, or alternatively you could create a fresh PR from the latest main branch - that would be easier and we will be able to check-in your first OSS contribution ⚡

@shubham1172 Yes I think that would be best. I do have a couple of questions, Do I run npm run build before I commit and push the code, if not what commands or processes should I run after writing the code, so I don't encounter the issues I had before (linter failing … etc.). Thank you.

Hi @jriffs, you can run npm run build locally and if it throws an error with prettier, you can run npm run pretty-fix. Also make sure to sign your commit by committing with -s flag - git commit -sm "my commit message".

@jriffs looks like you deleted the .gitignore and checked a lot of files which shouldn't be here. Either you can fix it here in this PR, or alternatively you could create a fresh PR from the latest main branch - that would be easier and we will be able to check-in your first OSS contribution ⚡

@shubham1172 Yes I think that would be best. I do have a couple of questions, Do I run npm run build before I commit and push the code, if not what commands or processes should I run after writing the code, so I don't encounter the issues I had before (linter failing … etc.). Thank you.

Hi @jriffs, you can run npm run build locally and if it throws an error with prettier, you can run npm run pretty-fix. Also make sure to sign your commit by committing with -s flag - git commit -sm "my commit message".

@shubham1172 okay, thank you. I have created a new pull request #508


labyrinth added

PR Description 📜
Please include summary related to the issue you have fixed and describe your PR in brief over here by specifying the issue number on which you were working below
Fixes #1028

Mark the task you have completed ✅

[x] I follow CONTRIBUTING GUIDELINE & CODE OF CONDUCT of this project.
[x] I have performed a self-review of my own code or work.
[x] I have commented my code, particularly in hard-to-understand areas.
[x] My changes generates no new warnings.
[x] I have followed proper naming convention showed in CONTRIBUTING GUIDELINE
[x] I have added screenshot for website preview in assets/images
[x] I have added entries for my game in main README.md
[x] I have added README.md in my folder
[ ] I have added working video of the game in README.md (optional)
[x] I have specified the respective issue number for which I have requested the new game.

Add your screenshots(Optional) 📸

Thank you soo much for contributing to our repository 💗

Thank you @S-ishita ,for creating the PR and contributing to our GameZone 💗
Review team will review the PR and will reach out to you soon! 😇
Make sure that you have marked all the tasks that you are done with ✅.
Thank you for your patience! 😀

Thank you  @S-ishita , for your valuable time and contribution in our GameZone 💗.
It’s our GameZone, so Let’s build this GameZone altogether !!🤝
Hoping to see you soon with another PR again 😇
Wishing you all the best for your journey into Open Source🚀

Hey @S-ishita,
You can't work on a closed issue 👀!
Follow the proper guideline and make a new PR again 😀.
Happy Hacking 💗


Cover refresh unavailable

The Refresh option for covers is unavailable. This seems to be a result of how the cover details (managed vs. custom, etc) are most likely overwritten when the game worker returns as it carries a potentially outdated copy of that data.
Need to ensure proper synchronization on the app.second.tex_cover object between the main thread, the cover worker, and the game worker.

Should hopefully be fixed (mostly) in https://github.com/SpecialKO/SKIF/commit/ae3f9eac2abc719d57335675b4257391c8ac04d9


Add description fields for Activity, Fishery, Gear, or Catch

This is required for https://github.com/WildAid/o-fish-ios/issues/285

FWIW - if possible - the ideal is to have the "other" value just save in the field, as if it was a menu item. so, if menu item is 'Other' (or a translation thereof) then save the 'Description' field to the actual field (e.g. Activity=Other, description='Sailing', save 'Sailing' to Activity in the database).

This can be done in the ViewModel, no need for a separate field.


Integrate DeviceProfile Resources / Commands Name Change

Update the example device profile and device-sdk-go, core-contract.
Resources -> DeviceCommands
Commands -> CoreCommands

Fix by https://github.com/edgexfoundry/device-mqtt-go/pull/43


Validate HighQC in Timeout Messages on Receipt

Current dependencies on/for this PR:

feature/proof-of-stake

PR #649

PR #841   👈

This stack of pull requests is managed by Graphite.


Use Gem.add_to_load_path if available

This is a follow up to https://github.com/rubygems/rubygems/pull/2749.
What is your fix for the problem, implemented in this PR?
My fix is to unify all LOAD_PATH manipulation in a single place, that uses rubygems logic if available.

@bundlerbot r+


Trained model release

Hi, are you planning to release the trained model?

Hi, since DIP re-trains a model anew for every image, we are not releasing the pre-trained models as that will be too many. However, replicating the results should be straightforward, please see #1 for details.


Authentication/Authorization aligned with ASP.NET API

This is more of a question/feature request.
HTTP triggered Functions are great for API implementation. With the Consumption plan or client-side architected applications, HTTP triggered functions are a great choice for backend API implementation. Anyone coming from the ASP.NET API world is familiar with Authentication/Authorization concepts. Unfortunately, there's no equivalent for Azure Functions (In-Proc and Isolated SDKs). The authentication out-of-the-box is solely for Function invocation and is not related to Authentication providers. Authorization with Authorize attribute, supporting Roles and Policies have not equivalent whatsoever. This leads developers to re-invent the wheel and build their own AuthN/AuthZ for Functions.
It would be great if these would be provided by Functions SDK and aligned with ASP.NET API apis.

any plans to implement this?

Planned and Comitted

Any ETA on this?

@SeanFeldman  @fabiocav
Do you have any new information on this. This feature would be really appreciated.

@kjartanvalur, I don't. I've raised this issue and am waiting for the feature to be added to replace the hand-rolled implementation.

An update on the plan for this feature would be appreciated as it would help to manage expectations.

Any update for this feature?

@SeanFeldman Thanks for raising this issue. It's something we're also interested in. Our team is currently looking to protect REST APIs implemented using Azure Functions and would prefer to leverage an approach similar to what ASP.NET offers, particularly in terms of using Authorize attributes, roles, and policies for both authentication and authorization.
Is there any update or ETA on when we can expect such a feature to be available in Azure Functions worker model?
Appreciate any insights


Radio buttons used for expanding/collapsing

On https://docs.api3.org/beacon/v0.1/reference/beacon-ids.html radio buttons are used for expanding/collapsing, which is odd. Chevrons would be better.

Removed the TryIt feature which used the radio buttons. Added a chevron to expand and collapse the Beacon.


Instructions do not work for VS 2022

These instructions did not work for VS 2022. For example, in Deploy your bot to Azure App Service, reaching step 3c, "Select Create a new Azure App Service by the green plus sign", VS 2022 did not offer that option. There was no green plus sign. After entering my credentials 3 or 4 times I gave up and switched to VS 2019. That matched these instructions. After creating a new app service in 2019, I switched back to VS 2022 and successfully deployed.

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

ID: 4a0e7068-9f6c-5b21-f12f-f44d27c17acd
Version Independent ID: 49c118b3-3e55-b321-26be-f4f668745f72
Content: Tutorial: Voice-enable your bot - Speech service - Azure Cognitive Services
Content Source: articles/cognitive-services/Speech-Service/tutorial-voice-enable-your-bot-speech-sdk.md
Service: cognitive-services
Sub-service: speech-service
GitHub Login: @eric-urban
Microsoft Alias: eur

@BruceHaley
Thanks for your feedback! We will investigate and update as appropriate.

@eric-urban
Could you please review add comments on this, update as appropriate.

@BruceHaley
Thank you for bringing this to our attention.
I've delegated this to content author @eric-urban, who will review it and offer their insightful opinions.

@BruceHaley - Thanks for flagging this! We have an internal work item to refresh this article top down. I'll close this to avoid duplicates.

#please-close


[5.8] Sort auto discovered packages

Service providers of auto discovered packages are registered in order of the declaration in vendor/composer/installed.json. This is in alphabetically order.
I am currently working at a project where it is necessary to register the service providers in the order they are declared in composer.json resp. the composer.json file of metapackages.
To preserve compatibility it should be sorted optionally by an extra key in the root composer.json file.
"extra": {
    "laravel": {
        "dont-discover": [],
        "sort-dependencies": true
    }
}

Wouldn't this break if someone reorders your composer.json file? I don't think the order of composer.json entries are supposed to matter, and there's a setting for composer to automatically order them. I wouldn't expect such a thing to also break my application...

Seems like a case where you'd want to disable auto discovery to get more fine-grained control

If you set composers sort-packages flag it would have still the same behavior like it is right now, in alphabetically order.
My intention was to control the order of packages in the composer.json files of metapackages. And this is for me the perfect place. I describe in one file which packages should be loaded and also in which order.
With the default Laravel´s composer.json there is no different behavior. But you can enable this functionality by setting the sort-dependencies flag in the extra:laravel section.

Seems like a case where you'd want to disable auto discovery to get more fine-grained control

In my case disabling auto discovery is not a good solution. I'm developing a web interface for several network devices. All of them have the same basic interface and i extending it with their custom packages.
With this implementation, i can clone my project and generate a specific web interface when I integrate the metapackage with composer require company/wifidevice-bundle.
This bundle requires company/core, company/autobackup etc.
Another device bundle requires company/core, company/sd-card etc.
And here is the problem: company/core needs always be the first registered service provider.
If i disable auto discovery, i need to tell Laravel manually which service providers should be loaded.
But I don't know which packages are implemented, because the root project is the base for every device bundle.

The PackageManifest::build() method is already parsing vendor/composer/installed.json to find all declared service providers. This file contains all the dependencies of the application, and we should be able to use this to sort the providers according to dependency order instead of alphabetical order. I believe this would work with your use-case assuming all the packages have their dependencies properly declared.

This is exactly what this PR is doing.
The PackageManifest::build() method checks if sort-dependencies is set to true and if so, it takes the already parsed packages from installed.json and sort it by dependencies instead of alphabetical order.

Ah, ok. Now i know what you mean.
I will change this.

I have also changed the composer key from sort-packages to sort-providers.

You should never depend on the registration order of service providers. They should be able to be registered in any order.


Deploy test on bebop

Execute test suite regularly on bebop and report to testing dashboard.

Tests are running daily. Using jump host and crontab to start tests.

Updated cronjob to reflect new repository name.

Thank you!


Implement login API and Github auth flow

This PR covers the initial login API, merging it would close #94, close #95, close #96, close #97, and close #98.
Some of that was covered by Adam's PR #104 but this closes things off somewhat.
Still to do, #99 and then #100 though the latter probably needs help from a flathub org member.
In addition, some refactoring will need to occur as we add further authentication schemes.  Refactoring to use authlib may make sense as we gain other oauth based authenticators, just to ensure consistency in handling secrets etc.  For now the client secret in this branch is only for a test app that isn't even hosted in the flathub org.  We need to decide how to handle client secrets going forward.

For the 2 Alembic migrations, couldn't they of been done in one migration instead of two separate?

@flyingpimonster Yeah, either we need to sort out a non-transparent image for the github logo, or else we should defer logo selection etc. to the frontend and remove it from the login kind list.  But I don't think that should block this PR.


Resolve-Path -relative visits parent directory without reason

When resolving the relative path for the current directory, the parent directory is visited without reason.
The implementation in PathIntrinsics.NormalizeRelativePath(String, String) needs fixing.
The issue is additionally relevant for #8435. See https://github.com/PowerShell/PowerShell/issues/8435#issuecomment-527763563 where I commented a workaround with the usage of System.IO.Path.GetRelativePath(String, String)
Steps to reproduce
$ pwd
c:\tmp
$ Resolve-Path -Relative ., (pwd), ../tmp
..\tmp
..\tmp
..\tmp

Expected behavior
$ pwd
c:\tmp
$ Resolve-Path -Relative ., (pwd), ../tmp
.
.
.

Environment data
$ $PSVersionTable

Name                           Value
----                           -----
PSVersion                      7.0.0-preview.4
PSEdition                      Core
GitCommitId                    7.0.0-preview.4
OS                             Microsoft Windows 10.0.17763
Platform                       Win32NT
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0…}
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>
WSManStackVersion              3.0

happens also on 5.1.17763.592

Notice, really there are some NormalizeRelativePath() methods.


Add clvm-traits and clvm-derive crates

This PR adds two crates, one that provides ToClvm and FromClvm macros and implementations for various common types, and another one that is re-exported in the other that adds derive macros for these traits.

Sounds good, I'll look into it


IRremote incompatiable with johnCenaMelody

IRremote library and johnCenaMelody function result in a conflict in complication.
Board: ARDUINO Nano
Library Version: 2.1.0
Protocol: Sony (if any)
Code Block:
Tone.cpp.o (symbol from plugin): In function `timer0_pin_port':
(.text+0x0): multiple definition of `__vector_7'
libraries/Arduino-IRremote-master/IRremote.cpp.o (symbol from plugin):(.text+0x0): first defined here
collect2: error: ld returned 1 exit status
exit status 1
Error compiling for board Arduino Nano.

/*
This example code is in the public domain.
 http://www.arduino.cc/en/Tutorial/Tone
 */

#include <IRremote.h>
#include "pitches.h"

int speakerPin             = 4;

void setup() {
  pinMode(speakerPin, OUTPUT);
  johnCena();
}
// Main loop most of the code is in the sub routines
void loop(){
}

void johnCena() { // plays john cena song
  // notes in the melody:
  int melody[] = {
    NOTE_A4, NOTE_B4, NOTE_G4, 0, NOTE_A4, 0,NOTE_C5, NOTE_B4, NOTE_G4, 0, NOTE_A4
  };

  // note durations: 4 = quarter note, 8 = eighth note, etc.:
  int noteDurations[] = {
    4, 6, 6, 8, 1, 4, 4, 6, 6, 8, 1
  };
....

https://github.com/z3t0/Arduino-IRremote/issues/103
says that Tone is trying to use same interrupt as IRremote.

Need to integrate NewTone Library from https://bitbucket.org/teckel12/arduino-new-tone/wiki/Home
The notes from pitches needs to be identified and rewritten.

#8


[KBM Editor] Support WM_DPICHANGED event

Summary of the Pull Request
What is this about:
The KBM Editor didn't support the WM_DPICHANGED event.
What is include in the PR:
Add helper methods to the DPI aware lib. Cleanup existing methods.
Add support to WM_GETDPISCALEDSIZE and WM_DPICHANGED.
How does someone test / validate:
On a single monitor: open the KBM editor window (test both key and shortcut editors) and while the window is open, go to the Windows Settings and change the current DPI scaling. The window should resize accordingly.
On a dual monitor setup: set different DPI scaling factors for the monitors, open the editor window on one monitor and move it to the other monitor and vice versa.`
Quality Checklist

[x] Linked issue: https://github.com/microsoft/PowerToys/issues/11014
[ ] Communication: I've discussed this with core contributors in the issue.
[ ] Tests: Added/updated and all pass
[ ] Installer: Added/updated and all pass
[ ] Localization: All end user facing strings can be localized
[ ] Docs: Added/ updated
[ ] Binaries: Any new files are added to WXS / YML

[x] No new binaries
[ ] YML for signing for new binaries
[ ] WXS for installer for new binaries

Contributor License Agreement (CLA)
A CLA must be signed. If not, go over here and sign the CLA.

Result of moving the window from a monitor with 100% scaling to a monitor with 200% scaling before the fix:

and after the fix:

When moving between monitors with different scaling works well.
However, after changing scaling on the same monitor with the editor opened, it could be sized incorrectly.

@SeraphimaZ
did you open the editor on one monitor, move the window to another monitor and then changed the DPI, correct?
If you move the Settings window to the second monitor, open the Editor there and then change the DPI, does it works correctly?


Unable to select the GPU manually when deploying models

Describe the bug

THe GPU list is empty

Environment

GPUStack version: main f283864
OS: Ubuntu 22.04
GPU: NVIDIA 4090

Additional context
Add any other context about the problem here.

Verified on UI cd1d300


Opportunity Naming Settings validation shouldn't rely on contact/opp data existing

From the hub: https://powerofus.force.com/0D58000002F96ew

**lurch: add

Tracking W-008166

Included in beta release version 3.52 (Beta 11) and higher

Included in production release version 3.52


⚠ Bofrost.be has degraded performance

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

HTTP code: 200
Response time: 1696 ms

Resolved: Bofrost.be performance has improved in 3535126.


adding k3s-on-host feature

What type of PR is this?

[X] Add a new dev container collection
[ ] Update to an existing dev container collection
[ ] Documentation/spec update
[ ] Other containers.dev site update (UX, layout, etc)

Description
Adds the k3s-on-host devcontainer feature that will install k3s on the host machine and enable kubectl / helm interaction from within the devcontainer
Collection checklist

[X] Collection name
[X] Maintainer name
[X] Maintainer contact link (i.e. link to a GitHub repo, email)
[X] Repository URL
[X] OCI Reference
[X] I acknowledge that this collection provides new functionality, distinct from the existing collections part of this index.

Will reopen with a new PR


Add loop mode + improve controls

Hi, this is a wonderful plugin and I'm really impressed by the accuracy!
However, I find the midi display a bit cumbersome to navigate. Would you mind improving the controls as follows? (Copying the navigation in fl studio)
Mouse wheel - Scroll up/down
Ctrl+Mouse wheel - Zoom in/out
Middle mouse drag - Pan
Space - Play
It will also be really useful if you add a loop functionality so it continues to play after it ends. Maybe you can make it possible to select a portion and the plugin will only export that portion. Thanks!

Hi,
Sorry for the late replay and thanks for those ideas. They would for sure improve the piano roll UX. I'd love to have them in NeuralNote but I can't say when I'll have time to implement this.


Fix double writing without check inside VisualShader::connect_nodes

The value coming from is_port_types_compatible in https://github.com/godotengine/godot/blob/97b8ad1af0f2b4a216f6f1263bef4fbc69e56c7b/scene/resources/visual_shader.cpp#L1288 is garbaged in the following if conditions without checks, so maybe the calling to is_port_types_compatible is unnecessary or we have to add another check before writing again however, I think the second one applies.

Hmm, I think This change is wrong because if there are any reroutes,  a special checking function which is _check_reroute_subgraph is called instead. I don't even know how we can use these reroute nodes in the Visual Editor, anyway, clooosing...


fix: removed errant condition

There was an extra condition that was unnecessary for this rule.

:tada: This PR is included in version 1.5.1 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


ReferenceError: require is not defined

How to solve

Vite is built on top of ESM which does not support require, use import instead

Vite is built on top of ESM which does not support require, use import instead

But some packages use require internally. What to do at this time ?

See https://github.com/vitejs/vite/issues/826#issuecomment-694776869 and https://github.com/vitejs/vite/issues/826#issuecomment-713262444

Packages that use require internally are not supported, because that's a NodeJS feature that can't be shimmed with import syntax. Especially without bundling.


v0.8 panics when run as a child process

I found the magic smoke:
[stderr] panic:
[stderr] runtime error: invalid memory address or nil pointer dereference
[signal 0xb code=0x1 addr=0x28 pc=0x4a71db]

goroutine 7 [running]:
github.com/mholt/caddy/caddy.init.5.func1()
        /home/aj/go/src/github.com/mholt/caddy/caddy/sigtrap_posix.go:25 +0x1ab
created by github.com/mholt/caddy/caddy.init.5
        /home/aj/go/src/github.com/mholt/caddy/caddy/sigtrap_posix.go:42 +0x2b

goroutine 1 [syscall]:
syscall.Syscall(0x0, 0x0, 0xc8200ce200, 0x200, 0xc820116220, 0xc82000be30, 0x86fea0)
        /usr/local/go/src/syscall/asm_linux_amd64.s:18 +0x5
syscall.read(0x0, 0xc8200ce200, 0x200, 0x200, 0x410c38, 0x0, 0x0)
        /usr/local/go/src/syscall/zsyscall_linux_amd64.go:783 +0x5f
syscall.Read(0x0, 0xc8200ce200, 0x200, 0x200, 0x200, 0x0, 0x0)
        /usr/local/go/src/syscall/syscall_unix.go:160 +0x4d
os.(*File).read(0xc82002e008, 0xc8200ce200, 0x200, 0x200, 0xc820000180, 0x0, 0x0)
        /usr/local/go/src/os/file_unix.go:211 +0x53
os.(*File).Read(0xc82002e008, 0xc8200ce200, 0x200, 0x200, 0x200, 0x0, 0x0)
        /usr/local/go/src/os/file.go:95 +0x8a
bytes.(*Buffer).ReadFrom(0xc8200cbad0, 0x7f7e9eb82348, 0xc82002e008, 0x0, 0x0, 0x0)
        /usr/local/go/src/bytes/buffer.go:173 +0x23f
io/ioutil.readAll(0x7f7e9eb82348, 0xc82002e008, 0x200, 0x0, 0x0, 0x0, 0x0, 0x0)
        /usr/local/go/src/io/ioutil/ioutil.go:33 +0x154
io/ioutil.ReadAll(0x7f7e9eb82348, 0xc82002e008, 0x0, 0x0, 0x0, 0x0, 0x0)
        /usr/local/go/src/io/ioutil/ioutil.go:42 +0x51
github.com/mholt/caddy/caddy.CaddyfileFromPipe(0xc82002e008, 0x0, 0x0, 0x0, 0x0)
        /home/aj/go/src/github.com/mholt/caddy/caddy/caddy.go:321 +0xe5
github.com/mholt/caddy/caddy.LoadCaddyfile(0xaf4ed0, 0x0, 0x0, 0x0, 0x0)
        /home/aj/go/src/github.com/mholt/caddy/caddy/caddy.go:287 +0x3c0
main.main()
        /home/aj/go/src/github.com/mholt/caddy/main.go:74 +0x4c1

goroutine 17
[stderr]  [
[stderr] syscall
[stderr] , locked to thread
[stderr] ]:

[stderr] runtime.goexit()
        /usr/local/go/src/runtime/asm_amd64.s:1696 +0x1

goroutine 5 [syscall]:
os/signal.loop()
        /usr/local/go/src/os/signal/signal_unix.go:22 +0x18
created by os/signal.init.1
        /usr/local/go/src/os/signal/signal_unix.go:28 +0x37

goroutine 6 [chan receive]:
github.com/mholt/caddy/caddy.init.4.func1()
        /home/aj/go/src/github.com/mholt/caddy/caddy/sigtrap.go:16 +0x143
created by github.com/mholt/caddy/caddy.init.4
        /home/aj/go/src/github.com/mholt/caddy/caddy/sigtrap.go:32 +0x2b

goroutine 8 [select, locked to thread]:
runtime.gopark(0xaf5540, 0xc82002d728, 0x9ff690, 0x6, 0x42da18, 0x2)
        /usr/local/go/src/runtime/proc.go:185 +0x163
runtime.selectgoImpl(0xc82002d728, 0x0, 0x18)
        /usr/local/go/src/runtime/select.go:392 +0xa64
runtime.selectgo(0xc82002d728)
        /usr/local/go/src/runtime/select.go:212 +0x12
runtime.ensureSigM.func1()
        /usr/local/go/src/runtime/signal1_unix.go:227 +0x353
runtime.goexit()
        /usr/local/go/src/runtime/asm_amd64.s:1696 +0x1

[close] 2 null```

Here's the test case:
'use strict';

function run() {
  var spawn = require('child_process').spawn;
  var caddypath = '/usr/local/bin/caddy';
  var caddyfilepath = '/srv/walnut/Caddyfile';
  var caddy = spawn(caddypath, ['-conf', caddyfilepath]);

  caddy.stdout.on('data', function (str) {
    console.log('[stdout]', str.toString('utf8'));
  });

  caddy.stderr.on('data', function (errstr) {
    console.error('[stderr]', errstr.toString('utf8'));
  });

  caddy.on('close', function (code, signal) {
    console.log('[close]', code, signal);
    clearInterval(caddy._interval);
    run();
  });

  caddy._interval = setInterval(function () {
    caddy.kill('SIGUSR1');
  }, 5 * 1000);
}

run();

Oh, and my Caddyfile:
https://lds.io {
  gzip
  tls /srv/walnut/certs/live/lds.io/fullchain.pem /srv/walnut/certs/live/lds.io/privkey.pem
  root /srv/walnut/init.public/
  proxy /api http://localhost:4080
}

https://ldsconnect.org {
  gzip
  tls /srv/walnut/certs/live/ldsconnect.org/fullchain.pem /srv/walnut/certs/live/ldsconnect.org/privkey.pem
  root /srv/walnut/init.public/
  proxy /api http://localhost:4080
}

https://oauth3.org {
  gzip
  tls /srv/walnut/certs/live/oauth3.org/fullchain.pem /srv/walnut/certs/live/oauth3.org/privkey.pem
  root /srv/walnut/init.public/
  proxy /api http://localhost:4080
}

This works fine when run not as a child and it also works fine with 0.7.6

It seems caddy was reading the Caddyfile from stdin, when the USR1 signal arrived, the global caddyfile was still nil and thus the panic.

Thanks! That had crossed my mind, but I the default for 'process.stdin' was changed to paused and so I thought stdin was as well.
Why does it read from stdin when '-conf' is specified? Is that for a new feature in 0.8 or a new bug?

When the process forks, it has to transfer the Caddyfile to the child process. It does this using a pipe. Specifying -conf just tells the original process where to load the Caddyfile, and subsequent reloads will also load that file, but the re-loaded file is still transferred over a pipe after forking.
I will add a nil check on that line to prevent this panic in the future... but am still not sure what to do if nil.

Just got around to testing this...
Yep, it seems to be working. :-)

Ok. That's not a workaround, that's correct code. Glad it's working now! ^_^ Pushing a fix for the panic in a moment.

I reread what you wrote ^^, but I'm still not clear.
Why is checking stdin when -conf is used correct? When it forks does it still have -confin the arguments so there's no way to know it was a fork?
Also, for the sake of sanity, please do print an error message if it gets a signal before the config is read. In my case a useful error message would have directed me to close stdin. :-)

Because pipe has precedence over the conf flag.

This still sounds like a bug to me.
Why would the default behavior use implicit configuration that overwrites an explicit configuration?
I see that you've even commented that you're doing this on purpose, but I'm not understanding why.
It seems like loader should accept an argument to the file specified by -conf and then loader should read from the pipe if the filename is nil.
Then if you're calling it as the new process you don't pass the filename, but if the user has said "this is what I want, here's -conf, you're not ignoring the user's intent".

Piping data into a process is an explicit action.

There's something that you understand that I don't understand or vice versa.
If I just run caddy from the command line with -conf, it doesn't halt and wait for user input.
If I write a node program for the command line and I want to read from stdin I have to explicitly tell it to read from stdin in node.
As a user I did not specify the "read from stdin" flag in caddy (typically this would be the - character for most programs that support the option), but it decided to read from stdin anyway.
I agree that you as the developer are explicitly opening and reading from the os.Stdin, but I disagree that I as the user specified to send data to stdin. I gave it a commandline argument.
This is where I'm confused. From my experience, it did something I explicitly told it not to do by virtue that I gave it -conf.
Furthermore, I'm trying to understand why stdin should have priority over flags. I don't understand your use case and it is counter-intuitive to my experiences with other programs.
Not that I'm spawning a child process from a node cluster in my day-to-day, but on the few occasions that I have I've never run into this before.
I understand that you are intending to do what you're doing on purpose. My knowledge is incomplete as to why this is good or necessary or both because I haven't experienced a program behaving this way before.

I'm reasoning about this and because I've not created a program to behave this way, I may lack some fundamental background knowledge.
Perhaps the case is that the default state of stdin is paused, but when you use something like cat filename.txt | caddy then cat is explicitly choosing to open stdin, which caddy can detect.
And also what you're saying is that running just caddy would leave stdin paused, but cat filename.txt | caddy would cause it to open and that cat filename.txt | caddy -conf ./Caddyfile would ignore the explicit behavior that the user specified -conf and instead choose the explicit behavior of cat.
I would argue that since the case of cat filename.txt | caddy -conf ./Caddyfile is generally not intuitive either to humans or to machines, it should throw an error and quit rather than choosing to obey either the user or the piper.

Caddy only reads from the pipe if there is one. It is the parent process' responsibility to open that pipe or not.

I would argue that since the case of cat filename.txt | caddy -conf ./Caddyfile is generally not intuitive either to humans or to machines, it should throw an error and quit rather than choosing to obey either the user or the piper.

Yeah, don't do that. Choose one or the other.

But... that's ambiguous and arbitrary. That's... confusing.
That means that people have to guess why something happened the way that it did and spend time debugging it (and your time explaining it).

How is it arbitrary and confusing?
If you have a Caddyfile to pipe into Caddy, then open a pipe, send the bytes, and close the pipe.
If instead you have a Caddyfile on disk for Caddy to load, then use -conf to specify it.

Here's the litmus test:

Go find a few programmers (or people that have used pipes in the command line).
Show them this text cat wow.txt | caddy -conf awesome.txt.
Ask them what they think might happen.
Then show them this text cat /etc/a/Caddyfile | caddy -conf /etc/b/Caddfile.
Ask them what they think might happen.

It's arbitrary because:

the choice wasn't made by the user (or rather the user unknowingly made two conflicting choices)
it isn't empirically deterministic.

If it discrete - say there were a unix standard for this behavior (and there actually is: it's to throw an error, same as the python zen) we wouldn't have had this conversation because I would have known "the standard unix way is to read from a pipe first and the standard unix way is to open a pipe to a child process with stdin open".
But neither of those statements are true. For any given program x it may or may not use stdin or commandline flags. For any program y it may or may not open a pipe when running a child process.
You just arbitrarily decided to do one. I didn't know what that was. It was documented. There was actually a bug that caused a panic because you hadn't foreseen that another person would encounter a situation that would arbitrarily decide to do something different.
Now, I'm sure you have a reason for wanting it one way or the other, so it doesn't seem to be arbitrary to use because you're the arbitrator in this case.

I'll have Caddy throw an error if both a pipe is open and the -conf flag is used.

I love you Matt.

You think things through very critically and once you've decided on you don't just go with what someone says, you make them work to prove its legitimacy. I genuinely appreciate that about you.
:-)

P.S. I can PR if you'd like. You've shown me where the code is and I know how to test it locally.
Let me know.

Thanks - I actually want to move the code that calls LoadCaddyfileFromPipe into the loader() function in main.go, since I think that should be up to the discretion of the users of the caddy package, not forced. So I'll take care of this one. But the pidfile issue could use your help when you have a chance. :+1:


Unable to set annotations on measures

Describe the bug
Using mymeasure.SetAnnotation("aaa","1") does not result in the creation of an annotation.
To Reproduce
In the following script, everything works without the WHERE clause, but with the WHERE clause it can't find any measures with the AUTOGEN annnotation = "1"
This script for the most part lifted from useful snippets.
var MacroTable = Model.Tables["_Macro measures"]; //table holding the measures to be created
foreach(var m in MacroTable.Measures.Where(m => m.GetAnnotation("AUTOGEN") == "1").ToList())  //This result in zero model changes, since the annotations are not being created in the line later below
{
m.Delete();
}
string query = "EVALUATE 'Fred metadata'";  //table of measure names, format strings, default aggregations, folder names
using (var reader = Model.Database.ExecuteReader(query))
{
while(reader.Read())
{
string mySeriesID = reader.GetValue(0).ToString();
string myMeasureName = reader.GetValue(19).ToString();
string myAggregation = reader.GetValue(18).ToString();
string myCategory = reader.GetValue(20).ToString();
string myFormatString = reader.GetValue(21).ToString();
string myExpression =
"VAR DS = FILTER('Fred datasets',[SeriesID] = "" +
mySeriesID + "")VAR myResult = " +
myAggregation + "(DS,[Value]) RETURN myResult";
var myNewMeasure = MacroTable.AddMeasure(myMeasureName, myExpression, myCategory);
myNewMeasure.FormatDax();
myNewMeasure.FormatString = myFormatString;
myNewMeasure.SetAnnotation("AUTOGEN","1");   //this seems not to run
}
}
Expected behavior
Annotations against all measures in the table with Name = "AUTOGEN" and Value = "1".
Observed behavior
It seems that no annotations are being created, so the WHERE filter is returning an empty list.
Application specifics

Tabular Editor version: 2.14.2]
Semantic Engine used: Power BI Desktop
Model Compatibility Level: not sure, but should be quite recent

Additional context
Hopefully I'm over looking something simple, like a syntax error, in which case, sorry for troubling you...

Tried simpler test and still zero model changes.  If Annotations are successfully created, will I be able to see it in the properties pane of the measure or will I have to query?  Here is the simpler script:
_var myTable = Model.Tables["MacroMeasures"];
foreach(var m in myTable.Measures.ToList())
{
m.SetAnnotation("AUTOGEN","1");
}

You have to enable unsupported Power BI features under File > Preferences, as annotations are not among the currently supported write operations of the External Tools integration of Power BI Desktop (although adding new annotations are unlikely to break anything, as long as you don't modify the annotations used by Power BI).

Great, thanks.  I must have missed that somewhere.  Everything worked as advertised.
Clearly this is not a bug but I can't figure out how to change the label, so I'll just close the issue.


Adding release notes for push task

release notes doc added and image used in release notes added in /pictures folder

@Matthew-Solomon please add a link of this release notes to the readme of your Push pipeline task just like the other pipeline task.


Test page

@cmpaul solved. please review when you're able.

@renderf0x Nice! Can you explain the fix? Looks like it was to set up a server that will build the bundle?

@cmpaul actually, having the Node server use webpack's api to do the build was part of the plan.
The fix here was a combination of taking the path out of the demo server webpack config json loader57e1792132ebe9f2426079624de65a12R13
so it falls back to the config file dir instead of the server __dirname,
and making sure that file specific loaders were included in the demo repo.
Webpack can be quite misleading in the error messages is gives you; sometimes "I can't find this file" really means "I can't load or read this file". In this case it was both, which made it extra blargh.


mvn2nix is not working ?

when i execute nix run -f https://github.com/fzakaria/mvn2nix/archive/master.tar.gz \         --command mvn2nix > mvn2nix-lock.json
or the command: nix run -f https://github.com/fzakaria/mvn2nix/archive/master.tar.gz --command mvn2nix
in the terminal, it returns the error message: Failed to execute goals [[package]]. Exit code: 1,
what's going on here and how do i fix this issue ?
note: it started installing from cachix and made some progress, but it seems like something broke during the installation.

i'm trying to use mvn2nix to install/build the Eclipse JDT Language Server, is that possible with this mvn2nix tool ?
here's the associated links:
https://github.com/eclipse/eclipse.jdt.ls
https://github.com/emacs-lsp/lsp-java

@zeta-00 thanks for your feedback and issue; also for trying the software.
You can add the -v switch (or -vvv) to control the verbosity of helpful messages which get printed to STDERR.
I will report back from trying that eclipse project.

@zeta-00 looks like that eclipse language server uses P2 layout something related to OSGi.
I will admit I'm not at all familiar with OSGi, Tycho or the P2 layout -- I'm sorry.
I don't think this tool will help in it's current incarnation there.

I have the same issue as well, tried to install mvn2nix running on macOS 10.15.6:
Running nix run -f https://github.com/fzakaria/mvn2nix/archive/master.tar.gz --command mvn2nix returns Failed to execute goals [[package]]. Exit code: 1.
There's no additional error information when I tried running with the -vvv flag.

@manojkarthick are you running it on a public repository ?
I'll be happy to look into it for you.

Thanks @fzakaria ! I'm trying to build https://github.com/apache/parquet-mr that build the parquet-tools CLI tool for working with files using the parquet file format.

@manojkarthick
2020-09-30 21:23:46,978 [Thread-1] INFO : [INFO] Downloaded from central: https://repo.maven.apache.org/maven2/org/codehaus/plexus/plexus-utils/3.0.14/plexus-utils-3.0.14.jar (239 kB at 3.7 MB/s)
2020-09-30 21:23:47,033 [Thread-1] INFO : [ERROR] thrift failed output:
2020-09-30 21:23:47,033 [Thread-1] INFO : [ERROR] thrift failed error: /bin/sh: thrift: command not found

I looked at it a bit more.

I started a nix-shell -p thrift
(My current version of nixpkgs has thrift at version 0.13.0)

Change the pom.xml to set that version, otherwise the build will fail.

index a84af6e76..2e9eeaabb 100644
--- a/pom.xml
+++ b/pom.xml
@@ -94,7 +94,7 @@
     <pig.version>0.16.0</pig.version>
     <pig.classifier>h2</pig.classifier>
     <thrift-maven-plugin.version>0.10.0</thrift-maven-plugin.version>
-    <thrift.version>0.12.0</thrift.version>
+    <thrift.version>0.13.0</thrift.version>
     <format.thrift.version>0.12.0</format.thrift.version>
     <fastutil.version>8.2.3</fastutil.version>
     <semver.api.version>0.9.33</semver.api.version>

Run the build

nix run -f https://github.com/fzakaria/mvn2nix/archive/master.tar.gz \
--command mvn2nix \
  --goals package --goals "-DskipTests=true" \
  --repositories https://conjars.org/repo/ \
  --repositories https://jitpack.io/ \
  --repositories https://repo.maven.apache.org/maven2/ \
  --repositories https://repository.cloudera.com/artifactory/ \
> mvn2nix-lock.json

I think that should get you further.

Thanks @fzakaria - I did try to install it with thrift initially. Sorry I forgot to mention that initially. Unfortunately that does not  fix the issue. The actual command I'm trying to run is:
nix run -f https://github.com/fzakaria/mvn2nix/archive/master.tar.gz \
--command mvn2nix \
  --goals package --goals "-DskipTests=true" \
  --repositories https://conjars.org/repo/ \
  --repositories https://jitpack.io/ \
  --repositories https://repo.maven.apache.org/maven2/ \
  --repositories https://repository.cloudera.com/artifactory/ \
  --repositories https://maven.twttr.com/ \
> mvn2nix-lock.json

I had to add https://maven.twttr.com to the list of repositories for the hadoop-lzo dependency. The error message returned is Failed to execute goals [[package, -DskipTests=true]]. Exit code: 1. Would you be able to guide me how you obtained the error message in https://github.com/fzakaria/mvn2nix/issues/25#issuecomment-701875359 ? That would help me with digging in a little further.

@Anderssorby @manojkarthick you can put -vvv after the call to mvn2nix to see more DEBUG output.
This issue is probably not the right format to do interactive messaging; I'm going to close it.
You can reach me over e-mail, Java channel on the NixOS discord server, fzakaria on freenode or the #nixos irc channel.


Cursor inconsistencies

There are some inconsistencies in the cursors we display in certain parts of Builder.

Different texts prompt the default, pointer, and text cursor style and it could be confusing.
As a general rule, we should show:

Pointer  for every interactive element
Text  for editable texts (like the map title in the header)
Crosshair  for adding geometries**
Move  for draggable elements (like when reordering layers or inputs for Pop-Ups)
Not-allowed  for disabled elements #13735
Default  for everything else (every non-interactive text or element)

In that sense, in the example...

Only the map title should show the text style
The tabs, inputs, icons, and links should show the pointer style
Everything else (labels, step title, description, breadcrumb...) should show the default style

**We don't have this implemented right now.

Stale issue. Closing 👋


lint scripts and utils directory

This PR adds linting for the scripts/ and utils/ directory and cleans up some linting issues in those folders.

Coverage remained the same at 73.611% when pulling 770df033df786ea389d28e269c4412406d9eb597 on hotfix/add-missing-lint-paths into 9b94a56a41219c8a473214b1864dbde59c1e5454 on develop.

easy 😄


Project-references type check with --noEmit fails without built files

TypeScript Version:  ^4.0.2

Search Terms: "has not been built from source file", "project-references", "typecheck without emit"
Description
In monorepo with project-references it's not possible to use tsc for type check only without emitting files.
Whereas calling tsc -b works without any issues. But there's no point in emitting files if we only need to perform type check of all related modules.
It seems that is should be possible because of: https://github.com/microsoft/TypeScript/pull/32028
Expected behavior:
tsc with --noEmit builds all referenced projects without emitting anything:
tsc --noEmit -p ./zoo

Found 0 errors.

Actual behavior:
It fails because no built files are found for referenced projects:
tsc --noEmit -p ./zoo

zoo/zoo.ts:1:32 - error TS6305: Output file '/Users/val/Desktop/patch/project-references-demo/lib/core/index.d.ts' has not been built from source file '/Users/val/Desktop/patch/project-references-demo/core/index.ts'.

> import { makeRandomName } from "@p/core";

Playground Link:
repro repo
git clone https://github.com/valerybugakov/project-references-demo

cd ./project-references-demo
npm i
npm run typecheck

Related Issues:
https://github.com/microsoft/TypeScript/issues/25613
https://github.com/microsoft/TypeScript/issues/25600
https://github.com/microsoft/TypeScript/issues/30661#issuecomment-526305644

A .d.ts can be subtly different from the .ts that produced it, so you might not get identical results between a "virtual" build like this and a proper one, so enabling this is a dangerous thing to do. We could produce the .d.ts files in-memory in a serial build, but that's just build mode, and you're not invoking build mode so we can't legally go and inspect the referenced project. I'm not sure what we could do here; this was one of the listed caveats of project references from the get-go.

@RyanCavanaugh  Thanks for the response. This functionality looks like what I want to achieve using --noEmit flag from the CLI. It doesn't use .d.ts files and relies on "virtual" build to provide better DX with type-checking.
If it's true, it means that the mechanism is already in place. It would be great to expose some flags for developers to use it. Please correct me if I confuse two unrelated features.

That PR is implemented by presenting the type checker with a virtual file system that builds .d.ts files on the fly; we can't enable it with a flag from tsc because it uses code specific to the language service process.

@valerybugakov Did you ever find a way to solve the problem you've described here?

@taylorkline, not the root cause. We partially solved the performance issue in our project by caching type check commands per Typescript project with nx.

I think this issue should be reconsidered, and opened again.
It appears that not only do we get TS6305 errors, any files in project references do not appear to be type checked. I could be wrong, and I will create a reproduction in the future.
Project references are used in the Vite template for React and TypeScript which is very popular.
Any execution of tsc to check the types of a project in these cases would dangerous, as while Visual Studio Code will check the types a give a false sense of safety any CI/CD or tool such as Vitest will silently fail.
In addition, it is not efficient to run tsc --build, where many of these build with their own bunder.

That PR is implemented by presenting the type checker with a virtual file system that builds .d.ts files on the fly; we can't enable it with a flag from tsc because it uses code specific to the language service process.

We could produce the .d.ts files in-memory in a serial build, but that's just build mode, and you're not invoking build mode so we can't legally go and inspect the referenced project.

I'm confused — so is this a technical limitation (as the first quote implies), or a design/correctness limitation?

I hope this topic gets revisited in the future. I just ran into this when I explored the idea of using project references to speed up type checking (via --noEmit).

The thing is that project references put on the table some unique functionality like project boundaries, which can be extra useful in monorepo's workflow,however, it currently overlooks its potential due to the necessity of building files, leading to a performance degradation when compared to the --noEmit check.
In the realm of modern monorepo development, there's a shift towards employing rapid compilers like SWC and esbuild with extracting type checking into a separate task. In such approaches, emitting any files through TypeScript (tsc) becomes unnecessary, further enhancing overall efficiency. I would really like to see someday this feature in your roadmap.

Run into this many times, I believe it should be fixed. There should be no reason to need to emit declaration files if you are just trying to type check and have noEmit set, and the errors are quite confusing.

I have a Vite SPA. In the main.ts entrypoint, I dynamically import some fixture data outside of the /src directory for mocking network requests:
  if (mock) {
    const { setupWorker } = await import("msw/browser");
    const { handlers } = await import("../fixtures");
    await setupWorker(...handlers).start({
      serviceWorker: { url: import.meta.env.BASE_URL + "mockServiceWorker.js" },
    });
  }

app/
├─ fixtures/
│  ├─ index.ts
│  ├─ some-data.json
├─ src/
├─ tsconfig.json

tsconfig.json
{
  "compilerOptions": {
    "composite": true,
    "isolatedModules": true,
    "jsx": "react-jsx",
    "lib": ["ESNext", "DOM", "DOM.Iterable"],
    "module": "ESNext",
    "moduleResolution": "Bundler",
    "noEmit": true,
    "noErrorTruncation": true,
    "noUncheckedIndexedAccess": true,
    "paths": { "@/*": ["./src/*"] },
    "resolveJsonModule": true,
    "rootDir": ".",
    "skipLibCheck": true,
    "strict": true,
    "target": "ESNext"
  },
  "include": ["src"],
  "references": [{ "path": "./tsconfig.node.json" }]
}

Oddly, this exact setup was working fine in a Vue Vite project, but not in a React one. If I run tsc, I get the error:
Output file '/Users/vincerubinetti/Desktop/molevolvr-new/app/fixtures/index.d.ts' has not been built from source file '/Users/vincerubinetti/Desktop/molevolvr-new/app/fixtures/index.ts'.

14     const { handlers } = await import("../fixtures");

I tried a bunch of tsconfig options, but couldn't find a way to prevent the error, except for actually -b building the whole project. I'd be fine biting this bullet with respect to execution time, but what I find unacceptable about this is polluting the repo with the emitted/built files. I guess I could output them to dist or something and .gitignore that. Or I guess I could @ts-ignore the dynamic import (which @typescript-eslint/ban-ts-comment throws a fit about), but I don't really like either of these workarounds.
Does anyone have a better option here? I don't even fully understand the problem here... I thought tsc would normally automatically build/transpile all the imported files to be able to type check them. Why is it failing on just the dynamic import? (Btw, yes I've tried includeing the fixtures directory, and also VSCode isn't showing any error for the dynamic import).


[Feature Request] Add ability to install plugin snapshots with ./bin/opensearch-plugin

Is your feature request related to a problem? Please describe
There are 3 parts to this feature request.

I would like to have the ability to install plugin snapshots of all plugins included in the core repository.

For example, I would like the ability to run an installation like ./bin/opensearch-plugin install repository-s3 from a local build of OpenSearch (i.e. the output of ./gradlew localDistro)
Currently, I receive output explaining that installation by name is not supported on snapshots:
./bin/opensearch-plugin install repository-s3
-> Installing repository-s3
-> Failed installing repository-s3
-> Rolling back repository-s3
-> Rolled back repository-s3
ERROR: attempted to install release build of official plugin on snapshot build of OpenSearch

This error occurs for a couple of reasons: 1) The sonatype snapshots repo does not have plugin zips published and 2) behind the scenes ./bin/opensearch-plugin searches for artifacts in Maven Central

I'd like to have the ability to install plugins from the default distribution of opensearch by name. i.e. ./bin/opensearch-plugin install opensearch-security

When installing with maven coordinates, I'd like to have the ability to specify the artifact repository so that its possible to fetch artifacts from https://aws.oss.sonatype.org/content/repositories/snapshots/org/opensearch/

Describe the solution you'd like
Support installing snapshot builds and configuration of the maven repository
Related component
Plugins
Describe alternatives you've considered
Keep performing local builds and install from a file
i.e.
./gradlew :plugins:repository-s3:assemble

./bin/opensearch-plugin install file:<OPENSEARCH_HOME>/plugins/repository-s3/build/distributions/repository-s3-3.0.0-SNAPSHOT.zip

Additional context
No response

[Catch All Triage - 1, 2, 3]