Keycloak reports Invalid parameter: id_token_hint during logout

Sometimes (usually when the user is already logged in for a while) during logout Keycloak reports Invalid parameter: id_token_hint and does not redirect back to the application again, leaving the user at an error page where no meaningful link is shown.
Is this a bug? How could this be avoided?

When the user session is expired and cannot be renewed, the user session will be locked out. The error you referenced is a hint that there is no client side id_token, but you are using a post logout redirect.
Please check if you haven't accidentially overwritten the exposeIdToken with false and are still using the post logout redirect.
The next version will also change the error behavior to hide most of the errors from the user and instead clearing the session and redirecting to the login page.

You can already try the new behavior in 1.0.0-beta.1.


deeplinks handler for accepting and generating thread invites

In the IPFS Camp workshop, creating invites to a thread was a lot more complicated than it needed to be. Obviously, this will be easier once there is a desktop GUI for go-textile, however until then, and even still, it would be nice to have the ability that when creating a thread, there was an invite URL generated with the output, or at least a deeplink to generate an invite
I would imagine this to work like textile://invite/thread/peer or something to that accord to generate, then textile://join/thread/inviteid to join.

Might be good to have this over in github.com/textileio/desktop as well...


Confirm all flags are added

[x]    -h          Display this help and exit

The globbing on this is strange Need to test the !  masks work

[x]    -f FILE     Target file(s) to upload
-f "path/to/file"     only upload this file
skips searching unless provided patterns below
-f '!*.bar'           ignore all files at pattern .bar
-f '.foo'            include all files at pattern .foo
Must use single quotes.
This is non-exclusive, use -s ".foo" to match specific paths.

[ ]    -s DIR       Directory to search for coverage reports.
Already searches project root and artifact folders.
[ ]    -t TOKEN     Set the private repository token
(option) set environment variable CODECOV_TOKEN=:uuid
-t @/path/to/token_file
-t uuid
[ ]    -n NAME      Custom defined name of the upload. Visible in Codecov UI
[ ]    -e ENV       Specify environment variables to be included with this build
Also accepting environment variables: CODECOV_ENV=VAR,VAR2
-e VAR,VAR2
[ ]    -X feature   Toggle functionalities
[ ]                 -X gcov          Disable gcov
[ ]                 -X coveragepy    Disable python coverage
[ ]                 -X fix           Disable report fixing
[ ]                 -X search        Disable searching for reports
[ ]                 -X xcode         Disable xcode processing
[ ]                 -X network       Disable uploading the file network
[ ]                 -X gcovout       Disable gcov output
[ ]    -N           The commit SHA of the parent for which you are uploading coverage. If not present,
the parent will be determined using the API of your repository provider.
When using the repository provider's API, the parent is determined via finding
the closest ancestor to the commit.
[x]    -R root dir  Used when not in git/hg project to identify project root directory

This should be removed as the codcov.yml file can only by located in the repo root for new notify

[ ]    -y conf file Used to specify the location of the .codecov.yml config file

[x]    -F flag      Flag the upload to group coverage metrics
-F unittests        This upload is only unittests
-F integration      This upload is only integration tests
-F ui,chrome        This upload is Chrome - UI tests
[ ]    -c           Move discovered coverage reports to the trash
[x]    -Z           Exit with 1 if not successful. Default will Exit with 0

I'd like to remove xcode fuctionally and make this a pure uploader
-- xcode --

[ ]    -D           Custom Derived Data Path for Coverage.profdata and gcov processing
Default '~/Library/Developer/Xcode/DerivedData'
[ ]    -J           Specify packages to build coverage.
This can significantly reduces time to build coverage reports.
-J 'MyAppName'      Will match "MyAppName" and "MyAppNameTests"
-J '^ExampleApp$'   Will match only "ExampleApp" not "ExampleAppTests"

I'd like to remove gcov fuctionally and make this a pure uploader
-- gcov --

[ ]    -g GLOB      Paths to ignore during gcov gathering
[ ]    -G GLOB      Paths to include during gcov gathering
[ ]    -p dir       Project root directory
Also used when preparing gcov
[ ]    -k prefix    Prefix filepaths to help resolve path fixing: https://github.com/codecov/support/issues/472
[ ]    -x gcovexe   gcov executable to run. Defaults to 'gcov'
[ ]    -a gcovargs  extra arguments to pass to gcov

-- Override CI Environment Variables --
   These variables are automatically detected by popular CI providers

[x]    -B branch    Specify the branch name
[x]    -C sha       Specify the commit sha
[x]    -P pr        Specify the pull request number
[x]    -b build     Specify the build number
[x]    -T tag       Specify the git tag
-- Enterprise --
[x]    -u URL       Set the target url for Enterprise customers
Not required when retrieving the bash uploader from your CCE
(option) Set environment variable CODECOV_URL=https://my-hosted-codecov.com
[x]    -r SLUG      owner/repo slug used instead of the private repo token in Enterprise
(option) set environment variable CODECOV_SLUG=:owner/:repo
(option) set in your codecov.yml "codecov.slug"
[ ]    -S PATH      File path to your cacert.pem file used to verify ssl with Codecov Enterprise (optional)
(option) Set environment variable: CODECOV_CA_BUNDLE="/path/to/ca.pem"
[ ]    -U curlargs  Extra curl arguments to communicate with Codecov. e.g., -U "--proxy http://http-proxy"
[ ]    -A curlargs  Extra curl arguments to communicate with AWS.
-- Debugging --
[x]    -d           Don't upload, but dump upload file to stdout

This can probably be removed as I'm not using color

[ ]    -K           Remove color from the output

[ ]    -v           Verbose mode

Also add the undocumented pre-signed PUT override flag.

pinging @drazisil-codecov

All v1 flags have been added. Marking this done.


-C flag

Using wlogout --help claims that you can specify a css file with the -C flag or --css flag. --css works just fine but -C gives wlogout: invalid option -- 'C'. Just a minor bug that doesn't remove any functionality from wlogout.

Fixed in 1.2


fix extrude using shift + space in Blender 4+

thanks to user kegoyo in HEAVYPOLY discord server. i think this is a breaking change for previous versions of Blender.

Thank you kegoyo and @doommchips!


Accept command line parameters from environment variable

Some tools have a handy (hidden?) feature that you can pass them command line parameters through the environment.
For example, the Microsoft C++ compiler accepts them through _CL_ and the linker through _LINK_. This can be super handy for adding temporary flags when you don't (easily) control the invocation such as when it's inside a script, batch file, or makefile. For example in the case of the compiler you can use it to preprocess a file with the same defines as the real build, or to get timing information.
This would also be useful for MSBuild. Right now I'm looking at the build of dotnet/runtime. I want to temporarily add a verbose logger like /flp6:v=diag. Right now to do this I have to dig through a chain of batch files, shell scripts, powershell scripts, and/or cmake files to find the 10 or more places where MSBuild may be launched within these and paste in the flag. This also adds noise in my diff. Later, I have to revert it as well.
If I could just do set _MSBUILD_=/flp6:v=diag it would solve all these problems.
I offer a proposed implementation in case you believe this is worthwhile.

We talked about this in PR triage today. BuildXL and other higher-order build engines would like to be better able to cache MSBuild invocations, and we took #5038 to better log when environment variables affect a build invocation. Doing this would mean they would have to know about _MSBUILD_ for their cache invalidation rules outside of that rule.
That's not necessarily a deal breaker but we're having a hard time weighing the costs and benefits, especially with Directory.Build.rsp as an option.
We'll leave this open to collect upvotes and other feedback, but I don't think we'll take the PR for now.

This shouldn't be anything new for build caching as such systems must already consider _CL_ and _LINK_ (either to respect them or document they are not respected)
Fair enough though, thanks for considering!

I would like to have this feature very much but the response files satisfy my needs for now.


higher rank trait bounds + fn traits

TODO

[x] print fn returns as part of the trait bound an not a separate alias relate
[x] the FnOnce here seems to be wrong, it should be FnMut
[x] there's a missing for<'a> around the FnMut bound

closed by #15


Enabled the "unhandled error" banner to be dismissed

I'd love it if this banner was able to be dismissed. My Internet is having issues right now so this pops-up a lot for me, but shortly after the app starts working again.

For me this error is constant and I would also like to dismiss it since everything seems to be working fine despite the error showing.


2.0 Token API enhancements

Rancher versions:
rancher/server: version 2.0
These changes should be done to Token /v3/tokens APIs after TP2 release:

listTokens and getToken (by id) - return all tokens(login and derived, and even expired)
add "expired" (bool) field and "expiresAt"(null if ttl=0, created+ttl as a datetime otherwise) on v3.Token
treat ttl=0/null == never expires
have a background thread to delete expired tokens

@cjellick @vincent99 logged as per discussion

@cjellick need review/merge of above PRs


fix(label): govuk modifier classes no longer overridden

:tada: This PR is included in version 3.3.2 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This PR is included in version 3.5.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Implement ETL scripts for blocks, transactions, and actions

The scope of this task is:

Implement a job for exporting blocks, transactions and actions using export_blocks_and_transactions.py  as a basis. Result should be in 3 CSV files: blocks.csv, transactions.csv, actions.csv.
Implement tests using test_export_blocks_job.py as a basis.
Implement get_block_range_for_date using get_block_range_for_date as a basis.
Optional
Implement Dockerfile
Write documentation in README using Exporting the Blockchain as a basis.
Test the scripts on a bigger range of blocks (5 days) on latest data to make sure the script don't fail.

Notes/questions:

There are a few EOS clients in Python to choose from https://github.com/Netherdrake/awesome-eos#python. The simplest one with minimal dependencies is preferable.
Does EOS api allow batching requests into a single HTTP payload?
Does this api https://developers.eos.io/eosio-nodeos/reference#get_block return full transactions or only hashes?
Are block timestamps in EOS monotonic? If yes get_block_range_for_date can be optimized. Read this for reference https://twitter.com/EvgeMedvedev/status/1073844856009576448.
Reuse blockchainetl package from bitcoin-etl https://github.com/blockchain-etl/bitcoin-etl/tree/master/blockchainetl. Don't worry about copy-pasting the code for now. This package will be moved to a separate repo later and included as a dependency.
Read this article to understand the overall architecture https://cloud.google.com/blog/products/data-analytics/ethereum-bigquery-how-we-built-dataset.

@medvedev1088 I am currently reading through the plan and previous ETL jobs written for other blockchain networks.

@aelbuni I'm using pytest to run unit and integration tests https://github.com/blockchain-etl/bitcoin-etl#running-tests.
If you run pytest -vv in bitcoin-etl it will run tests against mock data. Here is an example of such data https://github.com/blockchain-etl/bitcoin-etl/tree/master/tests/resources/test_export_blocks_job/bitcoin/block_1.
If before running the tests you specify provider uri e.g.
export BITCOINETL_BITCOIN_PROVIDER_URI=http://user:pass@localhost:8332
pytest -vv

it will also run the same tests against the actual node.

@aelbuni Hi. Any progress update?


[Event Hubs] September 2023 Release Prep

Summary
The focus of these changes is to prepare the core Event Hubs library for the September 2023 release.

API change check
API changes are not detected in this pull request.


Windows has no build user group

Description
Avoids managing build user group on Windows.  Since the block that creates the group is set to ignore_failure true, there is no reason to try and operate on build group for Windows platform, since it will most likely fail to create the group on Windows.
Signed-off-by: Eric G. Wolfe<EMAIL_ADDRESS>

/cc @chef-cookbooks/engineering-services


Counfusion with position of non-local block

Hi, Thank you for your wonderful job!
I am studying non local block aiming to apply this on another network and your work is a good inspiration. However, I found the position of non-local block in your ResNet_Video_nonlocal is different from the original paper and its repos( repo1 & repo2 ).
 I consider non-local as a module to connect pixels in order to expand the receptive field of convolution layer, so I think it affect the convolution layer behind it.
What's your opinon about its position and why it work?
Looking forward to your reply!

Because the original paper didn't clarify the position it places the non-local layer, I just have tried different places. I think that because I didn't put it behind the last Res block, there are still conv layer of the next stage behind the non-local layer I put.

Thank you for your reply!


GUVNOR-2854: Add test cases

This pull request adds three additional test cases that help to test new
feature more properly.

Fail is not related to this PR.
Jenkins please retest this.

Hi @Rikkola. Please could you have a look when you have a moment?


Removed cf-tables from ignore.json

Removing cf-tables from the ignore.json to ensure cf-tables is picked up by the generator.
Removals

Removed cf-tables from ignore.json

Review

@Scotchester @ascott1 @contolini - do you mind checking this over?

Checklist

[ ] Changes are limited to a single goal (no scope creep)
[ ] Code can be automatically merged (no conflicts)
[ ] Code follows the standards laid out in the front end playbook
[ ] Passes all existing automated tests
[ ] New functions include new tests
[ ] New functions are documented (with a description, list of inputs, and expected output)
[ ] Placeholder code is flagged
[ ] Visually tested in supported browsers and devices
[ ] Project documentation has been updated (including the "Unreleased" section of the CHANGELOG)

Special Thanks
Thanks to @ascott1, @Scotchester, and @contolini for advice on CF!
Animated GIF
Removing tables is like:

Minor comment :+1:

:+1:

This kills the table :+1:

:+1: :clap:

After Voltrazord, the ignore file is no longer used.


Uncorrect source maps after minification

Source maps after minification should remaps source maps before.
For example.

We have two original files like index.ts and component.tsx
We have some webpack ts loader, for example swc-loader to transpile ts files.
After transpiling we minificate a final bundle.

Actual source maps, produced by minificator, contains information only about already bundled code.
Expected behavior: source maps after minification should contains information about index.ts and component.tsx
There are several packages that can remap source maps. For example https://github.com/mozilla/source-map or https://github.com/7rulnik/source-map-js or https://github.com/ampproject/remapping

Should be fixed in 2.1.2. Let me know if it works or not.


Timeout Option

Hi,
Your library is working perfectly, thank you.
I just couldn't handle one case which is about the timeout.
I am initializing timeout durations like this.
xmodem.Options = new XModemProtocolOptions
{
 SenderInitializationTimeout = 3000,
 ReceiverInitializationTimeout = 3000,
 ReceiverTimeoutDuringPacketReception = 3000
 };

After this, I am calling xmodem.PacketToSend and xmodem.Send();
Then I am detaching the data cable. So, COM port stays but there is no more Data cable. Timeout is not triggering. Then xmodem.Send() throws InvalidOperationException and this locks my UI totally.
Even try-catch handles the error.
How can I handle COM port exists but there is no data cable. Which means xmodem cannot send data.
Thanks

I'm sorry. I'm not really understanding. Are you allowing a couple of packets to transfer, and pulling the plug to simulate a lost connection?


Would you explain the meaning of "highest compression coefficient" of Jpeg?

Hello, Ando.
Thank you for sharing the great repo.
Thanks to this repo, I can understand paper.
I have a question about JPEG compression.
In the paper, they randomly picked the quality factor.
But your implementation does not have and you left the message
"Jpeg the same as the Jpeg layer from the paper. It is a differentiable approximation of Jpeg compression with the highest compression coefficient."
Are you meaning current implementation is equal to Q = 50?
Thank you.

Hi,
The Jpeg layer in here is exactly the Jpeg-Mask layer from the paper.
This is the differentiable approximation of Jpeg with Q=50.
Are you sure about the jpeg quality being a varied in the paper? I haven't re-read the paper for a while and I could not find the varying strength of Jpeg layer. In figure 10, they show the accuracy vs Jpeg compression Q, but that's not the noise layer Q (i.e. Jpeg-Mask), but the real Jpeg compression Q which they used to validate the model.

Thank you for fast reply.
I totally agree with you.
Maybe, I would misunderstand the paper's point.
In page 6 of the paper, they said quality factor Q in (0,100).
And end of the paragraph, they mention one more JPEG has quality factor.
But their approximation experiment, they compare with Q = 50.

You're right, they do mention Q as variable on page 6.
However, Figure 10 explicitly says:
"The specialized JPEG model is trained on the differentiable approximation JPEG-Mask, and the plot shows performance on actual JPEG."
Figure 11 is less clear, whether the Jpeg figure is diff. approximation or real Jpeg compression.
Anyway, if you want variable-Q jpeg, you can do the it relatively easily. The noise_layers/jpeg_compression.py has a class JpegCompression class, which has the following constructor:
def __init__(self, device, yuv_keep_weights = (25, 9, 9)):
...

the (25,9,9) are the number of coefficients in Y,U,V channels which Jpeg-Mask with Q=50 keeps. If you want compression with another Q-level, just look up the number of coefficients jpeg keeps for that level of Q, and create JpegCompression with yuv_keep_weights = (y_q, u_q, v_q) instead of (25,9,9).
Hope this helps.

Thank you so much!
It really helps me.
Have a great day!


how can I log logger name in shared file?

Hello!
I would like to have one file transport with many loggers. And i would like to have logger name on each log line to separate loggers when read file. So my question is how to pass logger names into this single file?

You can pass a formatter for each transport or a filter for each logger:
const logger = new winston.Logger({
    filters: [
      function(level, msg, meta) {
        return `[logger-1]: ${msg}`;
        }
      },
    ],
})

@the-noob
Can you show full configuration with 2 loggers wich write to same file please

const winston = require('winston');

const getOptions = name => ({
  transports: [
    new (winston.transports.File)({
      filename: 'somefile.log',
    }),
  ],
  filters: [
    (level, msg, meta) => {
      return `[${name}]: ${msg}`;
    },
  ],
});

const log1 = new winston.Logger(getOptions('log1'));
const log2 = new winston.Logger(getOptions('log2'));

log1.info('some message in log 1');
log2.info('some message in log 2');

Please close the issue if this solves your problem :)

@the-noob thak you for your answer.
unfortunally this configuration creates two separate file transports.
I'm looking for solution when one tansport shared by both loggers.
I found that two transports witch shared same file not stable on Windows. Looks like first file transport locks that file.

@alexey2baranov
Can't see an easy way to get the logger instance from the transport but, based on this SO answer you can do:
const winston = require('winston');

const originalGetLogger = winston.Container.prototype.get;// winston.loggers.get(loggerName);
winston.Container.prototype.get = (name, options) => {
  const logger = originalGetLogger.call(winston.loggers, name, options);
  logger.rewriters.push((level, msg, meta) => {
    meta.logger = name;
    return meta;
  });
  return logger;
};

const options = {
  transports: [
    new (winston.transports.File)({
      filename: 'somefile.log',
    }),
  ],
};

const log1 = winston.loggers.get('log1', options);
const log2 = winston.loggers.get('log2', options);

log1.info('some message in log 1');
log2.info('some message in log 2', {extra: 'meta'});

will produce
{"logger":"log1","level":"info","message":"some message in log 1","timestamp":"2017-10-01T10:57:02.102Z"}
{"extra":"meta","logger":"log2","level":"info","message":"some message in log 2","timestamp":"2017-10-01T10:57:02.104Z"}

@the-noob
This code is very close with my code in project  it works unstable on windows . Sometimes second logger does not write to file . Sometimes second logger writes after 10 seconds after event. So log file become unusefull.
My code very same with Your code generates two separated file transports.  Looks like first transport locks file. I.m looking how to share single transport by both loggers. This feature is able in log4j and his clones. I hope winston have it too.

@the-noob
Oh i see your way now. I need investigate your answer


Remove default air temperature and wind speed from ModelChain.run_model etc.

Currently, ModelChain.run_model requires only irradiance as input. Air temperature and wind speed are optional, and if not provided, pvlib defaults to using 20C and 0m/s respectively. ModelChain method docstrings describe this behavior so it is not hidden from the user.
Although convenient for some use cases, supplying default air temperature and wind speed allows ModelChain.run_model() to complete without error if a user has data for e.g. air temperature in the weather input but not in the temp_air column. This behavior is less explicit than desired, IMO.
Similar concerns with ModelChain.run_model_from_poa and ModelChain.run_model_from_effective_irradiance.
Recommended solution: require air temperature and wind speed as inputs for ModelChain.run_model methods, except in the case where cell_temperature is provided as input.
Alternatives: leave as is.

We'd need to add some warning/deprecation machinery to address the change in required weather, so it would also be a good opportunity to change the name from temp_air to air_temperature or temperature (#894).


[BUG] Canceling expansion results in a volume expansion error

Describe the bug
Tested on v1.6.x-head (longhorn-manager 275f7ce). Expanding volume through UI and then Cancelling expansion through UI results in volume stuck in expansion error:
Expansion Error: the expected size<PHONE_NUMBER>0 of engine test-1-e-0 should not be smaller than the current size<PHONE_NUMBER>4
Note: You can cancel the expansion to avoid volume crash

Try to cancel the expansion again, it shows another error:
unable to cancel expansion for volume test-1: volume expansion is not started

To Reproduce

Create a 20 Gi volume through UI, and attach it to a node

Expand the volume size to  21 Gi through UI

Before the expansion complete, cancel the expansion through UI

Observe the volume gets stuck in Expansion error

Try to cancel the expansion again,

Expected behavior

Support bundle for troubleshooting
supportbundle_573c79f2-662c-4099-a9d2-c609dd5c42f3_2024-09-16T03-13-19Z.zip
Environment

Longhorn version: v1.6.x-head
Impacted volume (PV):
Installation method (e.g. Rancher Catalog App/Helm/Kubectl): kubectl
Kubernetes distro (e.g. RKE/K3s/EKS/OpenShift) and version:

Number of control plane nodes in the cluster:
Number of worker nodes in the cluster:

Node config

OS type and version: ubuntu 24.04
Kernel version:
CPU per node:
Memory per node:
Disk type (e.g. SSD/NVMe/HDD):
Network bandwidth between the nodes (Gbps):

Underlying Infrastructure (e.g. on AWS/GCE, EKS/GKE, VMWare/KVM, Baremetal):
Number of Longhorn volumes in the cluster:

Additional context

Workaround and Mitigation
https://github.com/longhorn/longhorn/issues/9466#issuecomment-2352006846

@derekbit , I've added the auto-e2e-test label, and I encourage @longhorn/qa to submit a PR when they identify a bug. This will help developers reproduce the issue and verify the fix more efficiently.

This issue seems to be triggered when the cancellation happens between the actual expansion completion and the engine/volume CR update.
I don't think the check @derekbit mentioned is the key. Actually, Longhorn cannot do anything with this case but just repeatedly reports that the cancellation request is invalid as the expansionist already done.

Yes. Adding a warning log & a KB that asks the users to redo the expansion is all we can do.

After rethinking this issue, the behavior is expected and we only need a KB for this issue. WDYT @shuo-wu @innobead

I think updating the PR that includes the hint log is nice-to-have.


Libnetwork to validate Ipam returned addresses

Closes #655
Signed-off-by: Alessandro Boch<EMAIL_ADDRESS>

@mrjana Taken care of your comment. Also updated the ipam doc accordingly.
PTAL when you get a chance.

LGTM

@aboch @mrjana I understand the intention of this PR. But, I think we should hold on this till we understand the use-cases better. Maybe it is better to let the user decide this instead of hard-coding such a logic in libnetwork core. WDYT ?

@mavenugo
I am fine to remove this restriction. As we discussed offline, we might want to support a null ipam driver option in future.
Please close #655 then.
I am ok to close this PR now.

Thanks @aboch @mavenugo
+1 for less restrictive behavior and null IPAM support. In our use case (assigning secondary addresses from ENI in VPC+L3 routing on host) it seems pretty tricky to separate out the logic between two drivers.


Conflict between this action (pushing to gh-pages) and github-pages bot?

Whenever I make a push to the repo, both nikola-action and the github-pages bot get run, despite having .nojekyll file in the main (source) branch which is correctly copied to the gh-pages branch. Is that a problem with this action? How can it be fixed? Do I have some other problem in my setup?
The workflow file is very plain, here: https://github.com/eudoxos/brzdime/blob/main/.github/workflows/main.yml
This is how both action are always triggered:

These are the settings:

Any ideas what is up?

This seems to be the correct behavior. The deployment of a branch to GitHub Pages seems to happen as an action nowadays. There seems to be a way to deploy directly from a GitHub Action, but it’s in beta.
Are you experiencing any issues with the deployment?

Okay, I understand now. The build action would be deleting CNAME in the gh-pages branch, and since GitHub keeps the hosting domain setting in there, the setting would be reset after each rebuild and the website disappeared.
The solution was to put both CNAME and .nojekyll into files/ so that they appear in output/ and are commited to gh-pages.
Perhaps a documentation tip?
Thank you for your work on Nikola and everything else!


Allow Bugsnag to be configured through appSettings instead of required custom configuration section

The high-level reasons for these changes were to simplify the initial code necessary to immediately start using Bugsnag in projects with the fewest steps as possible. There are 3 central commits to this pull request. As a result, to start using Bugsnag in this fork, I only need:

Bugsnag package to be installed in the project
An appSetting called "bugsnag.apiKey" in the web.config or app.config
The "Bugsnag" namespace imported for the class being managed (ie: using Bugsnag;)
Calling the "Notify()" extension on the exception (ie: ex.Notify();)

As the existing projects are written today, for a product to implement BugSnag using the provided clients in the library (such as WebAPIClient, WebMVCClient, or WPFClient), the product must use the defined custom configuration sections. These custom configuration sections are clean and strongly typed - great in many scenarios. However, it creates a barrier for some development teams who may not be comfortable with custom configuration sections (which can be difficult to document and train how to use). It also means additional steps to begin using Bugsnag in a competitive market where you want your product to be as easily and quickly integrated as possible.
These minor changes create a new implementation of IConfigurationStorage to derive the Bugsnag settings from the appSettings section of an app.config or web.config file for a project. This means that once Bugsnag is installed from nuget, developers only need to add the api key to the appSettings of their project to immediately begin using Bugsnag.
The source code follows the existing naming convention of the IConfigurationStorage model with camel case applied to a prefix for the application name (bugsnag), then a dot, and finally the camel case of the configuration setting name. To define the api key, the key in the app setting would be "bugsnag.apiKey".
Several other popular error logging frameworks leverage extension methods to simplify the code base when reporting errors. It can also make things easier out of the box to report exceptions. This commit defines an ErrorExtensions static class that once a product includes "Bugsnag", the "Notify" method is extended onto any Exception objects that are available. Reporting an error now minimizes how much knowledge the developer must understand about the internals of Bugsnag (ie: BaseClient or the various static client classes). The developer must only import Bugsnag, then a single line such as "ex.Notify()" will send the exception.
The configuration for this extensions attempted to leverage the existing information provided by the various clients and done in a way to work with existing documentation on the Bugsnag site. The ErrorExtensions use a new enum, ClientTypes, to identify which client should be used to pass on the exception (and thus take advantage of the auto-appended details of the session or client). The client must be configured so that an MVC project won't use the Wpf client. The client type is configured within the Start method of each of the clients as well as the methods used when configuring global error handling. Therefore, if the user follows the Bugsnag docs for global handling on mvc/api/wpf projects, or calls the start method on the desired client when the app starts, the underlying settings are configured automatically.

Please let me know what the process of reviewing pull requests. I'm happy to answer any questions you have. This would be an extremely useful set of features for our team.

Just my two cents......
I think its a great idea for using extension methods for reporting individual exceptions!
I also agree that its really quick, easy and familiar to add
<add key="bugsnag.api" value="xxxxxxxxxx">

to any app.config or web.config and the client is configured! The app storage looks good.
Its looks like there is not much control over what client is used for the extension methods. The methods rely heavily on the initial client (WPF, WebAPI or MVC) being started up as expected before any extension methods are called and the extension static class is initialized. But in cases when people want to use them in a custom way e.g. use there own BaseClient, or only use the Notify() methods on the web clients, the extension methods I think could use different clients with potentially different config (is that true?). Also I guess there is no way to specify or swap out the client to use, but that might be OK for this use.
Also if you use the SingletonBaseClient set the client and custom configure it, the first time you call an extension method, it would be written over with a brand new one (is that right?)
Definitely think these additions would makes it easier and quicker to start using Bugsnag.
What do you think @snmaynard ?

Any idea whether this will be a candidate to include? We'd love to start using these features natively out of nuget across all our apps!

@CodeHex, @snmaynard - Any idea whether this can be included in a future version? I would still love to see something like this out of the box. I'm happy to try to refactor the approach if you think another way may be better.

@jwallace1983 thanks for your contribution, sorry it has taken us so long to get around to addressing this PR. We are about to release a new version of the notifier which will make this PR impossible to merge. Let us know how you find the new version


Cortex 3.0 upgrade issue

Hi Team,
We were at cortex 2.0, and now upgrading to Cortex 3.0. However when I run the existing code after the pull I face these issues while table generation.

Any suggestions would help.
Thank you

hi @papiyachatterjee , without any more details on your implementation, I'd guess this is because your setting.yaml config file in 2.0 had the default for vbrp  as RUNTIME) and is now switched to behave as a table. I'd recommend not merging configuration files and only adding new values instead, as  your configurations will work for your needs better than the examples we include in them.
If switching the value back to runtime does not fix it, please provide more context on what vbrp looks like today.

Closing this issue as inactive. If there are still open questions, please let us know and we will be happy to assist you.


correction to send size with S3 request

when testing I was initially unable to send files to S3 without this additional change.  I figured I should contribute back to the community as this was helpful for my project.

true, I believe 234 may have undone the initial setting of content length.  I remember at the time getting error from S3 about missing content length before the change, we have been a build from this branch since the change was made.  you're correct it would be cleaner to have removed 232 and moved down to 235 instead of duplicate lines.

The order of calls doesn't concern me either way, but reviewing the code of ObjectMetadata I don't see any reason to suppose that #setContentType() and #setContentLength() would impact one another in any fashion. Any chance you could double check that the relevant functionality is still broken for you using latest master?

I'm pretty sure this report must have been a duplicate of #1654 and am recommending it be closed.


libusb and libgtk for Ubuntu 14.04

When building on Ubuntu 14.04, I had to make sure to have libusb-1.0-0-dev and libgtk-3-dev installed in order to build from source without errors. When re-running the cmake command with those two packages installed, this is my output:
-- The C compiler identification is GNU 4.8.2
-- Check for working C compiler: /usr/bin/cc
-- Check for working C compiler: /usr/bin/cc -- works
-- Detecting C compiler ABI info
-- Detecting C compiler ABI info - done
-- Found PkgConfig: /usr/bin/pkg-config (found version "0.26")
-- checking for module 'libusb-1.0'
--   found libusb-1.0, version 1.0.17
-- checking for module 'gtk+-3.0'
--   found gtk+-3.0, version 3.10.8
-- Configuring done
-- Generating done
-- Build files have been written to: /home/brian/sensei-raw-ctl/build
If you you try to build it on Ubuntu and if it works without issues with those two packages installed via apt or aptitude, then update the README. Thanks!

Eeeeh, sort of invalid. I don't want to put any distribution-specific package information in the README. I wouldn't mind if someone made a pull request with Debian packaging files, though. The best thing, of course, would be to patch the few major issues with this application (handle USB errors a bit better and rearrange the GUI to make more sense) and get it included in Debian.
However it's a very low priority task for me.


Routable Page url reversing honour WAGTAIL_APPEND_SLASH setting

Urls generated when the WAGTAIL_APPEND_SLASH is false should look like this: /archive/year/2020. Notice that there is no trailing slash at the end.
This works fine with the exception being the reverse_subpage() method on RoutablePageMixin.
This pull request fixes reverse_subpage to take this setting into account.

Closing this after discussion with @gasman
RoutablePage URL reversing should ignore WAGTAIL_APPEND_SLASH and respect what @route says. Which makes this PR invalid.


2016纪念版-0.8.2 添加了语法分析器的中间过程输出

2016纪念版
添加了json输出接口
方便Gui调试器的开发
添加了扩展包管理器的链接属性支持
添加bdwgc库

Current coverage is 72.56%

Merging #33 into master will decrease coverage by -1.06% as of fa2b08d

@@            master     #33   diff @@
======================================
  Files           16      16
  Stmts          857     893    +36
  Branches         0       0
  Methods          0       0
======================================
+ Hit            631     648    +17
  Partial          0       0
- Missed         226     245    +19

Review entire Coverage Diff as of fa2b08d

Powered by Codecov. Updated on successful CI builds.


It appears that the back button in detail and settings page header doesn't work

Perhaps it's my lack of knowledge.

Update to the newest version of the Nuget package. It is working now, as of version 1.1.5. You just need to do a clean/rebuild and refresh your project from the solution explorer after you have updated the Nuget package, and then you should be good to go.

Thanks. I'll do that


[SPARK-21567][SQL] Dataset should work with type alias

What changes were proposed in this pull request?
If we create a type alias for a type workable with Dataset, the type alias doesn't work with Dataset.
A reproducible case looks like:
object C {
  type TwoInt = (Int, Int)
  def tupleTypeAlias: TwoInt = (1, 1)
}

Seq(1).toDS().map(_ => ("", C.tupleTypeAlias))

It throws an exception like:
type T1 is not a class
scala.ScalaReflectionException: type T1 is not a class
  at scala.reflect.api.Symbols$SymbolApi$class.asClass(Symbols.scala:275)
  ...

This patch accesses the dealias of type in many places in ScalaReflection to fix it.
How was this patch tested?
Added test case.

Test build #80151 has started for PR 18813 at commit 8a4b63d.

Test build #80151 has finished for PR 18813 at commit 8a4b63d.

This patch fails PySpark unit tests.
This patch merges cleanly.
This patch adds no public classes.

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

Merged build finished. Test FAILed.

retest this please.

Test build #80157 has started for PR 18813 at commit 8a4b63d.

Test build #80157 has finished for PR 18813 at commit 8a4b63d.

This patch fails SparkR unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

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

retest this please.

Test build #80160 has started for PR 18813 at commit 8a4b63d.

Test build #80160 has finished for PR 18813 at commit 8a4b63d.

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

Merged build finished. Test PASSed.

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

cc @cloud-fan @hvanhovell

ping @cloud-fan May you have time to look at this? Thanks.

ping @cloud-fan @hvanhovell Can you help to review this change? Thanks.

is it possible to dealias the Type at the very beginning and avoid touching so many places?

@cloud-fan I identified only one place getClassFromType we can remove the dealias. As there is only one and it's a public method, I prefer to keep it in case the future caller doesn't do dealias, except for you strongly feel we should remove it. Thanks.

LGTM

Test build #80374 has started for PR 18813 at commit 312c7b0.

Test build #80377 has started for PR 18813 at commit 031d1d3.

Test build #80374 has finished for PR 18813 at commit 312c7b0.

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

Merged build finished. Test PASSed.

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

Test build #80377 has finished for PR 18813 at commit 031d1d3.

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

Merged build finished. Test PASSed.

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

thanks, merging to master/2.2!

Thanks you @cloud-fan.


centos7测试环境环境配置里的SpeechSDK-Linux- 是什么，在哪里下载的？

https://learn.microsoft.com/zh-cn/azure/cognitive-services/speech-service/how-to-configure-rhel-centos-7
客户这边测试环境是centos7的，安装上面文档链接安装依赖，截图里面最后一步的环境配置里的SpeechSDK-Linux- 是什么，在哪里下载的？文档上没看到

centos7环境报这个错，是依赖没安装好的原因吗？
Caused by: java.lang.UnsatisfiedLinkError: com.microsoft.cognitiveservices.speech.SpeechConfig.setTempDirectory(Ljava/lang/String;)V
at com.microsoft.cognitiveservices.speech.SpeechConfig.setTempDirectory(Native Method)
at com.microsoft.cognitiveservices.speech.SpeechConfig.(Unknown Source)

Hi,
The part about SpeechSDK-Linux-<version> is "For C++ development only" as noted in comments. Are you using C++ or Java (or something else)?
For C++, you need to download the Speech SDK tar.gz package from https://aka.ms/csspeech/linuxbinary. The current latest such package is SpeechSDK-Linux-1.24.2.tar.gz so when you extract it, there will be a folder SpeechSDK-Linux-1.24.2 (i.e. <version> becomes 1.24.2, and the full path prefix /path/to... depends on where this folder is).
For Java, you need the Speech SDK jar package. First ensure that the environment is genuine CentOS (or RHEL) 7:
# cat /etc/centos-release
CentOS Linux release 7.9.2009 (Core)

Make sure the latest updates are installed (sudo yum updateinfo, sudo yum update).
Download Speech SDK Java samples from https://github.com/Azure-Samples/cognitive-services-speech-sdk/tree/master/samples/java/jre/console and test that they work in your environment:
# git clone https://github.com/Azure-Samples/cognitive-services-speech-sdk
# cd cognitive-services-speech-sdk/samples/java/jre/console/
# sed -i 's/1\.24\.0/1.24.2/g' pom.xml

Try, for example, the case with wav file input; edit the source file src/com/microsoft/cognitiveservices/speech/samples/console/SpeechRecognitionSamples.java and go to continuousRecognitionWithFileAsync() in that file and replace "Your..." strings with your subscription key, service region, and some wav file path. Then build and run (on the command-line):
# mvn --version
Apache Maven 3.8.6
Maven home: /opt/apache-maven-3.8.6
Java version: 1.8.0_352, vendor: Red Hat, Inc., runtime: /usr/lib/jvm/java-1.8.0-openjdk-<IP_ADDRESS>2.b08-2.el7_9.x86_64/jre

# mvn clean package

# mvn dependency:tree
...
[INFO] SpeechSDKDemo:SpeechSDKDemo:jar:0.0.1-SNAPSHOT
[INFO] +- com.microsoft.cognitiveservices.speech:client-sdk:jar:1.24.2:compile

# export LD_LIBRARY_PATH=/usr/local/lib64:$LD_LIBRARY_PATH
# java -jar ./target/SpeechSDKDemo-0.0.1-SNAPSHOT-jar-with-dependencies.jar

and in case continuousRecognitionWithFileAsync() was edited, select and run
 5. Speech continuous recognition using events with file.

Note that with current CentOS 7 x64 installations, /etc/ld.so.conf.d/local-x86_64.conf contains /usr/local/lib64 as a library search path by default. In that case you don't actually need to set LD_LIBRARY_PATH.
If the Java problem still persists, try one or more of the following:

Clear the local Maven repository, then rebuild the app/samples.# rm -rf "$HOME/.m2"

If the Linux distribution in use is not genuine CentOS/RHEL 7 but is based on it (and must be fully compatible), set a special environment variable to force using CentOS 7 specific Speech SDK binaries from the jar package.# export SpeechSDK_UseCentos7Binaries=true

One CentOS 7 user recently had an odd issue that required adding /usr/local/bin to PATH (although it should be already there by default), even for Java.# export PATH=/usr/local/bin:$PATH

Please let us know (in English, if possible) if suggestions above help.

Closed as answered, please open a new issue if more support is needed.


State: DOWN,  Error: context deadline exceeded

What did you do?
The metrics endpoint of my web application seems to be working. I can curl it and get good data. I've added my configuration to prometheus.yml
What did you expect to see?
I expect to see the metrics Prometheus has scrapped.
What did you see instead? Under which circumstances?
When I view the Prometheus targets page, my endpoint is listed but it's reported as down with the error, "context deadline exceeded".
Environment

System information:
Darwin 15.6.0 x86_64

Prometheus version:
prometheus, version 1.3.1 (branch: master, revision: be476954e80349cb7ec3ba6a3247cd712189dfcb)
build user:       root@37f0aa346b26
build date:       20161104-20:27:37
go version:       go1.7.3

Alertmanager version:
insert output of alertmanager -version here (if relevant to the issue)

Prometheus configuration file:

# my global config
global:
  scrape_interval:     15s # By default, scrape targets every 15 seconds.
  evaluation_interval: 15s # By default, scrape targets every 15 seconds.
  # scrape_timeout is set to the global default (10s).

  # Attach these labels to any time series or alerts when communicating with
  # external systems (federation, remote storage, Alertmanager).
  external_labels:
      monitor: 'codelab-monitor'

# Load rules once and periodically evaluate them according to the global 'evaluation_interval'.
rule_files:
  # - "first.rules"
  # - "second.rules"
  - 'prometheus.rules'

# A scrape configuration containing exactly one endpoint to scrape:
# Here it's Prometheus itself.
scrape_configs:
  # The job name is added as a label `job=<job_name>` to any timeseries scraped from this config.
  - job_name: 'prometheus'

    # Override the global default and scrape targets from this job every 5 seconds.
    scrape_interval: 5s

    # metrics_path defaults to '/metrics'
    # scheme defaults to 'http'.

    static_configs:
      - targets: ['localhost:9090']

  - job_name: 'my-job'
    scrape_interval: 20s
    static_configs:
      - targets: ['test-site.dev']
        labels:
          group: 'my-job-test'

  - job_name:       'example-random'

    # Override the global default and scrape targets from this job every 5 seconds.
    scrape_interval: 5s

    static_configs:
      - targets: ['localhost:8080', 'localhost:8081']
        labels:
          group: 'production'

      - targets: ['localhost:8082']
        labels:
          group: 'canary'

Thanks for the attention.

Does curl work from the Prometheus node as well? If so, try using tcpdump or Wireshark to find out what request Prometheus is sending. Something must be off.

@juliusv Thanks for your response but how do I check whether curl works from the Prometheus node? I'm working with only the Prometheus server. Hope I haven't missed out anything. Once again thanks.

You mean you already did the curl from the Prometheus server's node? If so, the Prometheus must be doing something differently when sending the request. Then I would suggest using tcpdump or Wireshark on either the Prometheus node or the node where the service is to see what is going on on the wire.

Sorry but this is my first time of using a tool like Wireshark so bear with me. When I captured from Loopback lo0 and filtered on "http", I saw requests from the Prometheus endpoint (localhost:9090/metrics) and my application (promtest.dev/metrics) - I opened both in my browser. No scraping requests from Prometheus to my promtes.dev showed up.
With my application, I got this from the prometheus endpoint:
# TYPE prometheus_target_scrape_pool_sync_total counter
prometheus_target_scrape_pool_sync_total{scrape_job="my-job"} 1
# HELP prometheus_target_sync_length_seconds Actual interval to sync the scrape pool.
# TYPE prometheus_target_sync_length_seconds summary
prometheus_target_sync_length_seconds{scrape_job="my-job",quantile="0.01"} 0.00011974
prometheus_target_sync_length_seconds{scrape_job="my-job",quantile="0.05"} 0.00011974
prometheus_target_sync_length_seconds{scrape_job="my-job",quantile="0.5"} 0.00011974
prometheus_target_sync_length_seconds{scrape_job="my-job",quantile="0.9"} 0.00011974
prometheus_target_sync_length_seconds{scrape_job="my-job",quantile="0.99"} 0.00011974
prometheus_target_sync_length_seconds_sum{scrape_job="my-job"} 0.00011974
prometheus_target_sync_length_seconds_count{scrape_job="my-job"} 1

I changed the target to "foo.dev" (which doesn't exist), restarted Prometheus and got exactly the same output as above.

If you still have issues with this, send a question to the prometheus-users https://groups.google.com/forum/#!forum/prometheus-users mailing list.


Cleanup coverage reports

This all looks good to me. I am a bit concerned about the 2 minutes it takes to upload. I assume all of that time is uploading the "lcov-report" folder?  What's the breakdown between that and the cobertura-coverage.xml?


401 Unauthorized error when registrying application

Hi, I keep getting the following message when I try to register the application:
Invoke-WebRequest : The remote server returned an error: (401) Unauthorized.
At C:\Sources\New-AadApp.ps1:262 char:21
... $response = Invoke-WebRequest -Uri $url -Body $body -Method Put -UseD ...
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
CategoryInfo : InvalidOperation: (System.Net.HttpWebRequest:HttpWebRequest) [Invoke-WebRequest], WebException
FullyQualifiedErrorId : WebCmdletWebResponseException,Microsoft.PowerShell.Commands.InvokeWebRequestCommand
Failed to register the application with the gateway
At C:\Sources\New-AadApp.ps1:270 char:5
throw $err
CategoryInfo : OperationStopped: (Failed to regis...ith the gateway:String) [], RuntimeException
FullyQualifiedErrorId : Failed to register the application with the gateway

It creates the application in the portal ok, but does not complete the registration apparently, so the AAD auth is not selectable in WAC.
Running ps as domain admin, and the AAD account used is a global admin.

---
#### Document Details

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

* ID: 9ae730e6-5e1e-1ae6-5cbd-9eb6d271690b
* Version Independent ID: a79daa4b-e8fb-0ed4-867c-430b3c61ceea
* Content: [Configuring Azure Integration](https://docs.microsoft.com/en-us/windows-server/manage/windows-admin-center/configure/azure-integration)
* Content Source: [WindowsServerDocs/manage/windows-admin-center/configure/azure-integration.md](https://github.com/MicrosoftDocs/windowsserverdocs/blob/master/WindowsServerDocs/manage/windows-admin-center/configure/azure-integration.md)
* Service: **windows-server-threshold**
* GitHub Login: @Haley-Rowland
* Microsoft Alias: **harowl**

my WAC URL ie. "management.domain.com" was different to my server name "myserver.domain.com".
After I added a CNAME in our internal DNS the registration worked.
I previously had an A record in place.
Hope that helps someone

Thanks @PaulStirling for sharing the details of your workaround!

Didn't work for me, still struggling with this issue.


[question/enhancement] override default markdownComponents recursively

About this issue
This is not a real issue but more of a question or enhancement proposal, because either I am not getting it (despite having checked the "Advanced usage" section in the README multiple times), or what I would like to do can not be done actually with the current version and from this discussion new evolution proposals can come. I candidate myself to help out if needed.
First of all, thanks for this awesome library: I am following the project since year and I think this is one of the "missing pieces" that are crucial for Kotlin Multiplatform development. Also, I really appreciated the possibility to provide custom MarkdownComponents that was introduced with 0.9.0, because it was really difficult to support Markdown dialectal extensions (e.g. spoilers with the ::: spoiler <title>\n<content>\n::: syntax).
But, from what I see, the user-provided components are used only at the top level and there is no easy way to use the newly provided component as a subcomponent of another one. Let me explain with an example: suppose I am providing my custom MarkdownComponent to render images, this works fine to handle an IMAGE element but if a TEXT element contains an inline image the default Image is used and not the custom one because MarkdownText directly uses the built-in Image Composable. Similarly, if I provide a custom MarkdownComponent to render TEXT blocks, this is not called inside a PARAGRAPH because MarkdownParagraph internally calls the default MarkdownText and  custom components are only queried in the handleElement function which is running of every node of the AST.
Shouldn't it be possible to retrieve MarkdownComponents from composition locals?
Feel free to close if not relevant and thank you in the meantime for this great project.
I'm currently using library version 0.12.0, Compose multiplatform 1.5.12, Gradle 8.6 with AGP 8.2.2
Details

[ ] Used library version
[ ] Used platform
[ ] Used support library version
[ ] Used gradle build tools version
[ ] Used tooling / Android Studio version
[ ] Other used libraries, potential conflicting libraries

Checklist

[ ] Searched for similar issues
[ ] Checked out the sample application
[ ] Read the README
[ ] Checked out the CHANGELOG
[ ] Read the FAQ
[ ] Checked out the MIGRATION GUIDE

Thank you very much @diegoberaldin for the question.
The note on top-level is not fully the case. The custom components apply to children and the childrens children as identified by the markdown parser. You can see the specific logic here:
https://github.com/mikepenz/multiplatform-markdown-renderer/blob/develop/multiplatform-markdown-renderer/src/commonMain/kotlin/com/mikepenz/markdown/compose/Markdown.kt#L56-L63
With that said, there is different handling at play depending on the item. For example an IMAGE top level is a very different component than an inline image.
https://github.com/mikepenz/multiplatform-markdown-renderer/blob/develop/multiplatform-markdown-renderer/src/commonMain/kotlin/com/mikepenz/markdown/compose/Markdown.kt#L94
vs
https://github.com/mikepenz/multiplatform-markdown-renderer/blob/develop/multiplatform-markdown-renderer/src/commonMain/kotlin/com/mikepenz/markdown/utils/AnnotatedStringKtx.kt#L71-L73
Which makes it unfortunately not as simple.
Also at this time items require a ColumnScope to be retrieved, however the specific composeables don't have the columnscope anymore.
The general idea was, in case you require a very different handling of paragraphs, that it is very likely it to be custom handled all-together - That said, you can use whichever custom MarkdownText you'd have in that case in your custom MarkdownParagraph to make it work the way needed.
In that specific example, top level MarkdownText get's a String whereas the one in the Paragraph gets an AnnotatedString already.
I had a look to check how simple / complex it would be to bring this feature in, unfortunately I think at this time it's not as feasible, due to the way things need to be different as a core component, vs when used as component within a component.

Ok thanks for answering.

With all of that said though, I am open to see improvements made on the flexibility of the library. Aiming to provide a great baseline, with the opportunity to expand for special cases! So if you, or someone else has a nice way to contribute some more flexibility I am happy to review pr's.
Also thanks again for the kind words!

Ok, I can fork and experiment, I'll let you know if I acheive some interesting result. With the current situation not only I managed to support spoilers (which are common on Lemmy) but also custom links (the so called "handles" that are used on Lemmy).

Ok, I can fork and experiment, I'll let you know if I acheive some interesting result. With the current situation not only I managed to support spoilers (which are common on Lemmy) but also custom links (the so called "handles" that are used on Lemmy).

May I ask how you have done so? I am also in the case where I need support of spoilers and other components not already in the base library.

I didn't have time to fork and pass custom handlers through composition locals, unfortunately. And in the meantime I managed to add support to custom spoilers using a workaround you can find here, line 78 and here.


Improving Contact page in Hackathon project page

Describe the bug

Improving Contact page in Hackathon project page
Expected behavior

Screenshots (Mandatory)

Additional context (Mandatory )
Are you contributing under any open-source program ?

I have been contributing under GSSoC'24

@apu52 Please assign me this issue under GSSoC'24

@apu52  I have resolved the previous issue.. Please assign me this issue

@SuhainaFathimaM I have assigned you . You can start working.


Create Hack Series

[x] Eventbright set up
[x] Venue sponsor #403
[x] Food sponsor #404

[x] Need to purchase the food #406

[x] 32 student signups #407
[x] Facilitators #408
[x] Confirm curriculum #409
[x] Swag pack #410

@MaxWofford, should someone be assigned to this issue?

Assigned myself!

Closing this because Hack Series is done. Way to end on a high note!


Prep release 2.2.18

Codecov Report

Merging #4680 (38864a5) into master (3d10536) will increase coverage by 0.00%.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #4680   +/-   ##
=======================================
  Coverage   95.26%   95.26%
=======================================
  Files         550      550
  Lines       60752    60752
=======================================
+ Hits        57877    57878    +1
+ Misses       2875     2874    -1

Flag
Coverage Δ

servertests
47.81% <ø> (ø)

test_responses
32.46% <ø> (ø)

unittests
95.18% <ø> (+<0.01%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

moto/awslambda/models.py
93.03% <0.00%> (+0.11%)
:arrow_up:

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 3d10536...38864a5. Read the comment docs.


Create issue_template.md

Provide users with more comprehensive instructions for reporting new issues, including link to Gitter group

@nwhitmont,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot


Missing Statistics [BUG]

Describe the bug
The number of drawbacks and scenarios taken does not appear to be counted in the statistics tab
To Reproduce
Steps to reproduce the behavior:

Create any new Jump
Take any new drawbacks or scenarios
Go to the Statistics tab and refresh

Expected behavior
Drawbacks and scenarios number should be counted, like perks, warehouse addons, and jumps
Screenshots

Alright, I have fixed this bug and Scenarios and Drawbacks should now be counted properly. This fix will be in the 1.3.3 update.


Code change required, need to be updated to accessToken from result.A…

The code block in Option 3: Call a downstream web API without the helper class should be updated to accessToken from result.AccessToken, as there is only one parameter, string accessToken in the function CallTodoListService. Open issue: https://github.com/MicrosoftDocs/azure-docs/issues/84251

@SibeeshVenu : Thanks for your contribution! The author(s) have been notified to review your proposed change.

#label:"aq-pr-triaged"


[FEATURE] Support static IP address

Describe the solution you'd like

The harvester bridge now only supports dynamic IP address requested from DHCP server. The static IP address also needs to be supported.
Support to configure static IP address for management network NIC. It should be able to specify in the installation process.

The harvester bridge now only supports dynamic IP address requested from DHCP server. The static IP address also needs to be supported.
We refactor Harvester network controller. You can use connman to modify IP address of physical NIC attached with bridge manually , then Harvester network controller can monitor the change of IP address and reset the IP address for bridge.

The harvester bridge now only supports dynamic IP address requested from DHCP server. The static IP address also needs to be supported.
We refactor Harvester network controller. You can use connman to modify IP address of physical NIC attached with bridge manually , then Harvester network controller can monitor the change of IP address and reset the IP address for bridge.


fix(proxy) disable service mesh by default and show a deprecation

warning when attempting to enable it
Service Mesh causes upstream HTTPS request to ignore proxy_ssl_*
directives due to its replacement of the Nginx SSL_CTX data
structure (#5112, #4955). This PR disables Service Mesh by default and
adds a test to confirm normal upstream TLS behavior. Enabling it
requires additional configuration service_mesh = on or environment
variable KONG_SERVICE_MESH=on and will cause a deprecation warning on
startup.
Service Mesh (alongside patches to OpenResty) will be completely removed
in the next major release.
Fixes #5112.
Fixes #4955.

Note: two failed mesh tests in plugin triggering spec, shouldn't affect the rest of the functionality. Will be addressed shortly.

didn't look at the code, got here from the slack notifcation; fix doesn't seem the appropriate type for the commit.

@Tieske Could you elaborate which one would be more appropriate?

@Tieske we are going with fix because the point of deprecating this right now is the fix the HTTPS problems. If we were deprecating this for product roadmap reasons, we could consider other scope types (perhaps even "feat" for the new config flag as in "add flag to make service mesh optional"), but it's not the case here.


added Intl polyfills

Signed-off-by: NeilSMyers<EMAIL_ADDRESS>Summary of Changes
Replace this text with a high-level summary of the changes included in this PR.
Related Issues
Please reference here any issue #'s that are relevant to this PR, or simply enter "N/A" if this PR does not relate to any existing issues.
Pull Request Checklist
This is just a reminder about the most common mistakes. Please make sure that you tick all appropriate boxes. But please read our contribution guide at least once; it will save you a few review cycles!
If an item doesn't apply to your pull request, check it anyway to make it apparent that there's nothing to do.

[ ] All commits contain a DCO Signed-off-by line (we use the DCO GitHub app to enforce this).
[ ] Updated LICENSE-3RD-PARTY.md for any added dependencies or vendored components.
[ ] Run prettier: npm run style-format
[ ] Updated documentation for changed code and new or modified features.

If you have any questions to any of the points above, just submit and ask! This checklist is here to help you, not to deter you from contributing!
PR template adapted from the Python attrs project.

To add some context--this PR fixes some issues with mediation, Aries Hooks event listeners, and some of the workflows. It adds to the additions in #81.


Update parser to be case-insensitive when checking commands

Updated AddressBookParser.java to be case-insensitive when checking commands by converting to lowercase.
Note: As such, in the future, any command classes should define their COMMAND_WORD in lowercase.

Codecov Report
Base: 65.08% // Head: 65.08% // No change to project coverage :thumbsup:

Coverage data is based on head (65938cb) compared to base (b3dcb56).
Patch coverage: 100.00% of modified lines in pull request are covered.

Additional details and impacted files
@@            Coverage Diff            @@
##             master      #92   +/-   ##
=========================================
  Coverage     65.08%   65.08%
  Complexity      463      463
=========================================
  Files           100      100
  Lines          1521     1521
  Branches        165      165
=========================================
  Hits            990      990
  Misses          492      492
  Partials         39       39

Impacted Files
Coverage Δ

...seedu/address/logic/commands/FindEmailCommand.java
100.00% <ø> (ø)

...du/address/logic/commands/FindLocationCommand.java
100.00% <ø> (ø)

.../seedu/address/logic/commands/FindNameCommand.java
100.00% <ø> (ø)

.../seedu/address/logic/parser/AddressBookParser.java
73.91% <100.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


README.rst not found during pip install

This is breaking pip install autobahn as well
Tag 2.02
Windows 10
Python 3.5.0
Work around: installing from source works fine, which makes it non-obvious how to fix this.
Problem:
PS C:\Users[user]\dev\autobahn> pip install txaio
Collecting txaio
Using cached txaio-2.0.2.tar.gz
Complete output from command python setup.py egg_info:
Traceback (most recent call last):
File "", line 20, in
File "C:\Users[user]\AppData\Local\Temp\pip-build-7zimen6i\txaio\setup.py", line 40, in
with open('README.rst') as f:
OSError: [Errno 22] Invalid argument: 'README.rst'
----------------------------------------

Command "python setup.py egg_info" failed with error code 1 in C:\Users[user]\AppData\Local\Temp\pip-build-7zimen6i\txaio
Oddly, pip install -e works fine:
PS C:\Users[user]\dev\txaio> pip install -e .
Obtaining file:///C:/Users/[user]/dev/txaio
Requirement already satisfied (use --upgrade to upgrade): six in c:\program files\python 3.5\lib\site-packages\six-1.10.
0-py3.5.egg (from txaio==2.0.2)
Installing collected packages: txaio
Running setup.py develop for txaio
Successfully installed txaio

Thanks for the report!
I also added a packaging-test so we can't make a broken package in the future.
@oberstet I bumped the version too, so should be able to do a release of this immediately.

@meejah thx for arranging things so I can proceed with minimal efforts;) https://pypi.python.org/pypi/txaio/2.0.3
a minor thing: the pypi "readme" doesnt look that pretty compared to https://github.com/tavendo/txaio#txaio .. not a big thing, maybe next release ..

AB Py / pypi readme: I think I recently fixed this https://pypi.python.org/pypi/autobahn .. already forgot again what I did exactly ..


Missing reflection_test.dae for raytracing reflections sample..

just testing your new raytracings sample I found missing this file..

sorry.. closing forgot to download additional assets..


feat: prettify existing snippets and add new ones

I've prettified existing snippets to make them follow the same style as these ones.
Also I've added new snippets for optional checks and disabling all warnings at once. 😄

Prettier runs in CI to ensure a consistent style. Most likely this is why it is failing.

Prettier runs in CI to ensure a consistent style. Most likely this is why it is failing.

I can't run Prettier as it fails when I open snippet file:
rror: Cannot find module 'prettier' from '/home/emilyseville7cfg/Documents/mine/bash/json-vscode-shellcheck'
	at e.exports [as sync] (/home/emilyseville7cfg/.vscode/extensions/esbenp.prettier-vscode-9.10.4/dist/extension.js:1:37268)
	at t.ModuleResolver.findPkg (/home/emilyseville7cfg/.vscode/extensions/esbenp.prettier-vscode-9.10.4/dist/extension.js:1:7353)
	at t.ModuleResolver.getPrettierInstance (/home/emilyseville7cfg/.vscode/extensions/esbenp.prettier-vscode-9.10.4/dist/extension.js:1:3313)
	at t.default.handleActiveTextEditorChanged (/home/emilyseville7cfg/.vscode/extensions/esbenp.prettier-vscode-9.10.4/dist/extension.js:1:9837)
	at handleActiveTextEditorChangedSync (/home/emilyseville7cfg/.vscode/extensions/esbenp.prettier-vscode-9.10.4/dist/extension.js:1:9439)
	at f.invoke (/usr/share/code/resources/app/out/vs/workbench/api/node/extensionHostProcess.js:63:145)
	at m.deliver (/usr/share/code/resources/app/out/vs/workbench/api/node/extensionHostProcess.js:63:2066)
	at d.fire (/usr/share/code/resources/app/out/vs/workbench/api/node/extensionHostProcess.js:63:1704)
	at /usr/share/code/resources/app/out/vs/workbench/api/node/extensionHostProcess.js:98:154048
	at f.invoke (/usr/share/code/resources/app/out/vs/workbench/api/node/extensionHostProcess.js:63:145)
	at m.deliver (/usr/share/code/resources/app/out/vs/workbench/api/node/extensionHostProcess.js:63:2066)
	at d.fire (/usr/share/code/resources/app/out/vs/workbench/api/node/extensionHostProcess.js:63:1704)
	at m.acceptDocumentsAndEditorsDelta (/usr/share/code/resources/app/out/vs/workbench/api/node/extensionHostProcess.js:98:13934)
	at m.$acceptDocumentsAndEditorsDelta (/usr/share/code/resources/app/out/vs/workbench/api/node/extensionHostProcess.js:98:12249)
	at a.N (/usr/share/code/resources/app/out/vs/workbench/api/node/extensionHostProcess.js:106:11223)
	at a.M (/usr/share/code/resources/app/out/vs/workbench/api/node/extensionHostProcess.js:106:10941)
	at a.H (/usr/share/code/resources/app/out/vs/workbench/api/node/extensionHostProcess.js:106:10034)
	at a.G (/usr/share/code/resources/app/out/vs/workbench/api/node/extensionHostProcess.js:106:9015)
	at /usr/share/code/resources/app/out/vs/workbench/api/node/extensionHostProcess.js:106:7803
	at f.invoke (/usr/share/code/resources/app/out/vs/workbench/api/node/extensionHostProcess.js:63:145)
	at m.deliver (/usr/share/code/resources/app/out/vs/workbench/api/node/extensionHostProcess.js:63:2066)
	at d.fire (/usr/share/code/resources/app/out/vs/workbench/api/node/extensionHostProcess.js:63:1704)
	at p.fire (/usr/share/code/resources/app/out/vs/workbench/api/node/extensionHostProcess.js:72:14907)
	at /usr/share/code/resources/app/out/vs/workbench/api/node/extensionHostProcess.js:122:16556
	at f.invoke (/usr/share/code/resources/app/out/vs/workbench/api/node/extensionHostProcess.js:63:145)
	at m.deliver (/usr/share/code/resources/app/out/vs/workbench/api/node/extensionHostProcess.js:63:2066)
	at d.fire (/usr/share/code/resources/app/out/vs/workbench/api/node/extensionHostProcess.js:63:1704)
	at p.fire (/usr/share/code/resources/app/out/vs/workbench/api/node/extensionHostProcess.js:72:14907)
	at MessagePortMain.<anonymous> (/usr/share/code/resources/app/out/vs/workbench/api/node/extensionHostProcess.js:122:14682)
	at MessagePortMain.emit (node:events:526:28)
	at MessagePortMain._internalPort.emit (node:electron/js2c/utility_init:5:364)

You can run it from the terminal as well. Check the scripts in package.json, there is one which calls prettier --write or something.

Why when I run npm install I get this:
npm ERR! Invalid comparator: ./see_vscode_vsce_issue_786

npm ERR! A complete log of this run can be found in:
npm ERR!     /home/emilyseville7cfg/.npm/_logs/2023-03-26T16_47_21_373Z-debug-0.log

with this log:
0 verbose cli /home/emilyseville7cfg/.nvm/versions/node/v19.3.0/bin/node /home/emilyseville7cfg/.nvm/versions/node/v19.3.0/bin/npm
1 info using<EMAIL_ADDRESS>2 info using<EMAIL_ADDRESS>3 timing npm:load:whichnode Completed in 1ms
4 timing config:load:defaults Completed in 1ms
5 timing config:load:file:/home/emilyseville7cfg/.nvm/versions/node/v19.3.0/lib/node_modules/npm/npmrc Completed in 1ms
6 timing config:load:builtin Completed in 1ms
7 timing config:load:cli Completed in 1ms
8 timing config:load:env Completed in 0ms
9 timing config:load:file:/home/emilyseville7cfg/Documents/mine/bash/json-vscode-shellcheck/.npmrc Completed in 0ms
10 timing config:load:project Completed in 2ms
11 timing config:load:file:/home/emilyseville7cfg/.npmrc Completed in 0ms
12 timing config:load:user Completed in 0ms
13 timing config:load:file:/home/emilyseville7cfg/.nvm/versions/node/v19.3.0/etc/npmrc Completed in 0ms
14 timing config:load:global Completed in 0ms
15 timing config:load:setEnvs Completed in 1ms
16 timing config:load Completed in 6ms
17 timing npm:load:configload Completed in 6ms
18 timing npm:load:mkdirpcache Completed in 0ms
19 timing npm:load:mkdirplogs Completed in 0ms
20 verbose title npm install
21 verbose argv "install"
22 timing npm:load:setTitle Completed in 1ms
23 timing config:load:flatten Completed in 2ms
24 timing npm:load:display Completed in 2ms
25 verbose logfile logs-max:10 dir:/home/emilyseville7cfg/.npm/_logs/2023-03-26T16_47_21_373Z-
26 verbose logfile /home/emilyseville7cfg/.npm/_logs/2023-03-26T16_47_21_373Z-debug-0.log
27 timing npm:load:logFile Completed in 3ms
28 timing npm:load:timers Completed in 0ms
29 timing npm:load:configScope Completed in 0ms
30 timing npm:load Completed in 13ms
31 silly logfile start cleaning logs, removing 1 files
32 timing config:load:flatten Completed in 0ms
33 timing arborist:ctor Completed in 1ms
34 silly logfile done cleaning log files
35 timing idealTree Completed in 80ms
36 timing command:install Completed in 84ms
37 verbose stack TypeError: Invalid comparator: ./see_vscode_vsce_issue_786
37 verbose stack     at Comparator.parse (/home/emilyseville7cfg/.nvm/versions/node/v19.3.0/lib/node_modules/npm/node_modules/semver/classes/comparator.js:38:13)
37 verbose stack     at new Comparator (/home/emilyseville7cfg/.nvm/versions/node/v19.3.0/lib/node_modules/npm/node_modules/semver/classes/comparator.js:22:10)
37 verbose stack     at /home/emilyseville7cfg/.nvm/versions/node/v19.3.0/lib/node_modules/npm/node_modules/semver/classes/range.js:133:47
37 verbose stack     at Array.map (<anonymous>)
37 verbose stack     at Range.parseRange (/home/emilyseville7cfg/.nvm/versions/node/v19.3.0/lib/node_modules/npm/node_modules/semver/classes/range.js:133:35)
37 verbose stack     at /home/emilyseville7cfg/.nvm/versions/node/v19.3.0/lib/node_modules/npm/node_modules/semver/classes/range.js:34:22
37 verbose stack     at Array.map (<anonymous>)
37 verbose stack     at new Range (/home/emilyseville7cfg/.nvm/versions/node/v19.3.0/lib/node_modules/npm/node_modules/semver/classes/range.js:34:8)
37 verbose stack     at Object.intersects (/home/emilyseville7cfg/.nvm/versions/node/v19.3.0/lib/node_modules/npm/node_modules/semver/ranges/intersects.js:3:8)
37 verbose stack     at OverrideSet.getEdgeRule (/home/emilyseville7cfg/.nvm/versions/node/v19.3.0/lib/node_modules/npm/node_modules/@npmcli/arborist/lib/override-set.js:53:18)
38 verbose cwd /home/emilyseville7cfg/Documents/mine/bash/json-vscode-shellcheck
39 verbose Linux 5.15.0-67-generic
40 verbose node v19.3.0
41 verbose npm  v9.2.0
42 error Invalid comparator: ./see_vscode_vsce_issue_786
43 verbose exit 1
44 timing npm Completed in 110ms
45 verbose unfinished npm timer reify<PHONE_NUMBER>399
46 verbose unfinished npm timer reify:loadTrees<PHONE_NUMBER>400
47 verbose unfinished npm timer idealTree:init<PHONE_NUMBER>401
48 verbose code 1
49 error A complete log of this run can be found in:
49 error     /home/emilyseville7cfg/.npm/_logs/2023-03-26T16_47_21_373Z-debug-0.log

I see that you are using Node 19. These are the versions we use:
https://github.com/vscode-shellcheck/vscode-shellcheck/blob/master/package.json#L293-L296

It looks kinda strange, that CI fails even after reformatting. 🤔

Add "nullary" to cspell.json

You can always open the CI logs and see what's going on. :)

They all passed! 🚀

Are you planning to add more after this PR? Just so I know whether to cut the next release now or not.

Are you planning to add more after this PR?

I guess everything I wanted already added. :)

Thanks for your contributions, @EmilySeville7cfg!


Support case sensitive keys

Use different method for escaping JCR names preserving the casing This closes #31

Any chance for having a release with this PR soon? Thanks a lot in advance.

Of course! you mentioned in the issue that you had to do some rework because of an edge case, didn't know you were ready with it ;)


Create an event making long press on any place of the map

Get coordinates where the user make a longPress
Display a blue container where the user generate an event
Display the new event modal
Associate the modal with the createEvent backend service

List of a few fixes/improvements:

If you click outside the modal without adding an event the icon is still shown in the map.
Change 'Description of the event' to 'Event description'.

String has already been change @anitanigma


Fix issue #5578 validateRequest method returning "true" even if the request path …

Fix issue #5578 validateRequest method returning "true" even if the request path is shorter than the collection url.

Thanks!

See #8358


Doubt Regarding Apparmor enabled using this script

@tommytran732  @AryunGupta @ajeyrgowda Total newbie here , I read the install.sh code but i wonder if i have to create apparmor profiles for each and every application i install in arch. Is there any advantage if we enable apparmor but don't use apparmor profiles.

There are basic profiles that come with just having apparmor it, though for it to really be effective you need to make your own profiles.

Thank you


fix 'ascii' codec can't decode byte 0xef in position 583: ordinal not…

… in range(128)
we had multiple errors like:
Unicodedecodeerror ascii codec can t decode byte 0xe3 in position 0 ordinal not in range 128
or
CRITICAL: 'ascii' codec can't decode byte 0xef in position 583: ordinal not in range(128)
setting utf-8 as default fixed the failure

the fix is mainly to set utf-8 as default fix. There were cases where we were seeing the error mentioned above and after I applied this change it is fixed. I have this in our environment for over 4 month and have not hit this error anymore.

Not sure why the entire file is being replaced, but that's a bit of a red flag for me.
We don't accept pull requests. This will need to be posted to https://reviews.reviewboard.org/, along with detailed info on the fix, and unit tests demonstrating the fix solves the core problem.
Thanks!

Is there any instruction on how to submit this. there is another fix that I have for Clearcase. What kind of proof also required. This has been working just fine with over 160 developer using these new changes

You'll just need to register an account on that server and, from the RBTools source tree in the correct branch, run:
rbt post

Then include the necessary info in the fields and publish. We also have more info at https://www.reviewboard.org/docs/codebase/dev/contributing-patches/
The unit tests need to demonstrate the problem. If we run just the tests without the patch, we should see that the tests fail, but with the patch, they should work.
Something to note is that Review Board/RBTools is used by at least hundreds of thousands of developers world-wide, and this means there's a wide range of configurations, encodings, operating systems, and other such things out there. Encoding-related changes can sometimes impact other users, and future changes can always regress earlier fixes, which is why we need the proof in terms of clear, reproducible, and self-explanatory unit tests. Knowing that it works for 160 developers is always good, but does nothing to guarantee it'll work for others, or keep working long-term.


CNN function Approximate

In experiments, have you ever compared the effect of CNN with different layers in this task? 4 is the minimum requirements considering performance？Thanks!

Thanks for the question. We do not claim the current network structure to be optimal. It's just a trade-off between output quality and development speed. The four-layer design is the outcome of a manual hyper-parameter search.


Add IBCX metadata for testnet

@junhoyeo Will this be added to ibc-assets (so that it's displayed on the Assets page)? Also will this be registered to Chain Registry (if needed, I can take a shot at this)?

@junhoyeo Will this be added to ibc-assets (so that it's displayed on the Assets page)?

@JeremyParish69 There's no need to add this token to ibc-assets, as /assets already displays the new token from Osmosis.

Also, will this be registered to the Chain Registry?

Since this token is only for the testnet, it's not necessary to register it in the Chain Registry for now.

If needed, I can take a shot at this.

But sure! I appreciate the offer.


kbld appends duplicate preresolved origin

What steps did you take:

Run kbld -f foo.yaml --imgpkg-lock-output lock.yaml
Rerun kbld with the resulting ImagesLock file: kbld -f foo.yaml -f lock.yaml --imgpkg-lock-output lock.yaml
Notice you now have a preresolved entry under kbld.carvel.dev/origins. So far so good.
Run kbld again with the lockfile: kbld -f foo.yaml -f lock.yaml --imgpkg-lock-output lock.yaml
Notice that the lockfile now has duplicate entries for preresolved

What happened:
In general, every time you run kbld while passing in the lockfile, kbld will append another preresolved entry.
Example:
---
apiVersion: imgpkg.carvel.dev/v1alpha1
images:
- annotations:
    kbld.carvel.dev/id: fluent/fluent-bit:x86_64-1.7.9
    kbld.carvel.dev/origins: |
      - resolved:
          tag: x86_64-1.7.9
          url: fluent/fluent-bit:x86_64-1.7.9
      - preresolved:
          url: index.docker.io/fluent/fluent-bit@sha256:8e3fdc011d3ff98502143650dfa3261860b11f1f289dbb3184c630f71236a366
      - preresolved:
          url: index.docker.io/fluent/fluent-bit@sha256:8e3fdc011d3ff98502143650dfa3261860b11f1f289dbb3184c630f71236a366
  image: index.docker.io/fluent/fluent-bit@sha256:8e3fdc011d3ff98502143650dfa3261860b11f1f289dbb3184c630f71236a366
kind: ImagesLock

What did you expect:
Stop after the first addition of preresolved.
Anything else you would like to add:
https://kubernetes.slack.com/archives/CH8KCCKA5/p1640125213244500?thread_ts=1640041938.237600&cid=CH8KCCKA5
Environment:

kbld version (use kbld --version): 0.32.0
Docker registry used (e.g. Docker HUB): n/a
OS (e.g. from /etc/os-release): macOS

If building images using docker: n/a

Docker CLI Version (e.g. docker version)

If building images using buildkit: n/a

Buildkit version (e.g. kubectl buildkit version):
Kubernetes version (e.g. kubectl version):

If building images using pack

Pack Version (e.g. pack version):

Vote on this request
This is an invitation to the community to vote on issues, to help us prioritize our backlog. Use the "smiley face" up to the right of this comment to vote.
👍 "I would like to see this addressed as soon as possible"
👎 "There are other more important things to focus on right now"
We are also happy to receive and review Pull Requests if you want to help working on this issue.

Hi @gabegorelick,
Thank you for pointing out this bug, I agree that the duplicate preresolved origin should probably not be there and could cause confusion. Additionally, in CI we want the same inputs to create the same outputs. For those reasons we agree that kbld should not append duplicate preresolved entries in the origins.

Should this issue be closed? The MR has been merged well over a year ago.


Determine if we have time to implement a Foundation CSS UI in Phase I or Not?

This isn't a story.

This is a task, determine if we can implement Foundation responsive UI framework or not during the time we have for phase I

We do.  I've already made it a part of the development environment.  I was going to explain that yesterday, but you didn't want to hear anything about the dev environment I configured.

Um, I thought you were not available, and needed to leave after 20 minutes?
Are you okay? You seem aggravated
I am just now trying to work on the development environment, and I have like all of these emails from you, so I’m sorry that my timing is not your timing on this project!
From: nevinsjr<EMAIL_ADDRESS>Sent: Sunday, May 22, 2016 4:35 PM
To: PennStateSoftwareTest/FinalProjectPennState<EMAIL_ADDRESS>Cc: Josh Sherman<EMAIL_ADDRESS>Assign<EMAIL_ADDRESS>Subject: Re: [PennStateSoftwareTest/FinalProjectPennState] Determine if we have time to implement a Foundation CSS UI in Phase I or Not? (#18)
We do. I've already made it a part of the development environment. I was going to explain that yesterday, but you didn't want to hear anything about the dev environment I configured.
—
You are receiving this because you were assigned.
Reply to this email directly or view it on GitHub https://github.com/PennStateSoftwareTest/FinalProjectPennState/issues/18#issuecomment-220854660 https://github.com/notifications/beacon/AMuN2GeJzaH9VkKvvP_TiMAoFahhawV0ks5qEL3kgaJpZM4IenWo.gif

ok great, the last discussion we had, was that we might need to take this out of scope for phase I, so that is why i had this task, great.


Add line separator attribute (CRLF)

We've been coding a lot on my current job, and came to agreement to make all files in CRLF line separators, since this is more fitable for our project.
I have a problem with persistent appearing in commit window with generated files

where the only difference seem to be only in separators

how can we change it from being generated in LF to CRLF?

I add use_crlf config, https://github.com/Lochinor/intl_utils
you  can delete .pub-cache\global_packages\intl_utils\bin\generate.dart-3.5.3.snapshot
them replace libs files at  .pub-cache\hosted\pub.dev\intl_utils-2.8.7
last add  use_crlf: true after flutter_intl like this:

Simple Used：
dev_dependencies:   intl_utils:     git:       url: https://github.com/Lochinor/intl_utils.git
run command
dart run intl_utils:generate

@Lochinor can you please submint a pull request on that?

this PR https://github.com/localizely/intl_utils/pull/134


Add optional deploy_freeze boolean

Hello @sbz and @felixgborrego!
We've had a couple of moments, where we wanted to prevent ourselves from accidentally deploying a certain service with a certain config. We've been thinking about adding an optional boolean deploy_freeze to the container. If that is set, and set to true, then it will print out a specific warning and exit the deploy.
I'd be very happy if you could take a look at that!
cc @relistan

thanks, I sounds good. let me run a quick test and release

Awesome, thanks Felix !

Thank you, Felix!


Support for Web API / WebRTC

Hi!
Does Dioxus support Web APIs like Web Audio etc. or WebRTC to capture audio, video and take photos?
Thanks for your great work!

This is not a dioxus thing - but rather an ecosystem issue.
Check this out:
https://github.com/wasm-peers/wasm-peers
That being said, we probably won't support this natively, and if we do, it'll be part of
https://github.com/dioxusLabs/sdk


[GOBBLIN-1883] Multiple tokens in ProxiedFileSystem

Dear Gobblin maintainers,
Please accept this PR. I understand that it will not be reviewed until I have checked off all the steps below!
JIRA

[x] My PR addresses the following Gobblin JIRA issues and references them in the PR title. For example, "[GOBBLIN-XXX] My Gobblin PR"

https://issues.apache.org/jira/browse/GOBBLIN-1883

Description

[x] Here are some details about my PR, including screenshots (if applicable):
While creating a proxied file system, sometimes we need multiple tokens. e.g. when there exists multiple namenodes

Tests

[x] My PR adds the following unit tests OR does not need testing for this extremely good reason:
trivial change

Commits

[x] My commits all reference JIRA issues in their subject lines, and I have squashed multiple commits if they address the same issue. In addition, my commits follow the guidelines from "How to write a good git commit message":

Subject is separated from body by a blank line
Subject is limited to 50 characters
Subject does not end with a period
Subject uses the imperative mood ("add", not "adding")
Body wraps at 72 characters
Body explains "what" and "why", not "how"

Codecov Report

Merging #3744 (99f0507) into master (69d7e0f) will increase coverage by 1.64%.
The diff coverage is 0.00%.

@@             Coverage Diff              @@
##             master    #3744      +/-   ##
============================================
+ Coverage     47.09%   48.74%   +1.64%
+ Complexity    10877     7981    -2896
============================================
  Files          2147     1483     -664
  Lines         84859    58999   -25860
  Branches       9416     6782    -2634
============================================
- Hits          39968    28760   -11208
+ Misses        41264    27582   -13682
+ Partials       3627     2657     -970

Files Changed
Coverage Δ

...rg/apache/gobblin/util/ProxiedFileSystemCache.java
0.00% <0.00%> (ø)

...rg/apache/gobblin/util/ProxiedFileSystemUtils.java
0.00% <0.00%> (ø)

...main/java/org/apache/gobblin/util/WriterUtils.java
44.96% <0.00%> (ø)

... and 684 files with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


[5.0] Database transactions do not seem possible

Calling \DB::beginTransaction() from a controller yields in the following error. But I can see Illuminate\Database\Connection contains the beginTransaction() function which Illuminate\Database\MySqlConnection extends so I don't see why this problem occurs.
ErrorException in DatabaseManager.php line 304:
call_user_func_array() expects parameter 1 to be a valid callback, class 'Illuminate\Database\MySqlConnection' does not have a method 'beginTransation'

in <snip>/vendor/laravel/framework/src/Illuminate/Database/DatabaseManager.php line 304
at HandleExceptions->handleError('2', 'call_user_func_array() expects parameter 1 to be a valid callback, class 'Illuminate\Database\MySqlConnection' does not have a method 'beginTransation'', '<snip>/vendor/laravel/framework/src/Illuminate/Database/DatabaseManager.php', '304', array('method' => 'beginTransation', 'parameters' => array()))
at call_user_func_array(array(object(MySqlConnection), 'beginTransation'), array()) in DatabaseManager.php line 304
at DatabaseManager->__call('beginTransation', array()) in Facade.php line 207
at DatabaseManager->beginTransation() in Facade.php line 207
at Facade::__callStatic('beginTransation', array()) in AgentController.php line 47
at DB::beginTransation() in AgentController.php line 47

Scratch that, seems to be possible when injecting the Illuminate\Database\DatabaseManager instance into the controller


StickBreakingProcess logpdf is ambiguous

ERROR: MethodError: Distributions.logpdf(::StickBreakingProcess{DirichletProcess{Float64}}, ::Float64) is ambiguous. Candidates:
  logpdf(d::StickBreakingProcess, x) in Turing.RandomMeasures at /home/msca8h/.julia/packages/Turing/eIa7P/src/stdlib/RandomMeasures.jl:41
  logpdf(d::Distribution{Univariate,S} where S<:ValueSupport, x::Real) in Distributions at /home/msca8h/.julia/packages/Distributions/tfkz4/src/univariates.jl:333
Possible fix, define
  logpdf(::StickBreakingProcess, ::Real)

This is the Turing compiler error,
using Plots, Random
using Turing
using Turing.RandomMeasures
    @model infiniteGMM(x, K_max) = begin
        # Base distribution.
        α   = 1.0
        μ0  = 0.0
        σ0  = 1.0
        αg  = 2.0
        βg  = 2.0
        rpm = DirichletProcess(α)

        # Latent assignments.
        N = length(x)
        z = tzeros(Int, N)

        # Infinite collection of stick pieces and weights.
        v = tzeros(Float64, K_max)
        w = tzeros(Float64, K_max)
        K = 0

        # Cluster locations.
        μ = tzeros(Float64, K_max)
        σ = tzeros(Float64, K_max)
        u = tzeros(Float64, K_max)

        for i in 1:N

            # Draw a slice ∈ [0,1].
            u[i] ~ Beta(1, 1)

            # Instantiate new cluster.
            while (sum(w) < u[i]) && (K < K_max)
                K += 1
                v[K] ~ StickBreakingProcess(rpm)
                σ[K] ~ Gamma(αg, βg)
                μ[K] ~ Normal(μ0, σ0*σ[K])
                w[K] = v[K] * prod(1 .- v[1:(K-1)])
            end

            # Find truncation point
            K_ = findfirst(u[i] .< cumsum(w))

            # Sample assignments.
            w_ = w[1:K_] / sum(w[1:K_])
            z[i] ~ Categorical(w_)

            # Draw observation.
            x[i] ~ Normal(x[z[i]], σ[z[i]])
        end
    end

Here is the code.

Should be fixed once the PR is merged.
You might want to look into using the size-biased representation instead of the stick-breaking as this should be more efficient for inference.
See: https://github.com/TuringLang/Turing.jl/blob/f2192c71c763f678b653e80e84363197835e2355/test/stdlib/RandomMeasures.jl#L274-L300
I'll add a tutorial on this and on the stick-breaking representation once I find the time.
For the paper see: https://pdfs.semanticscholar.org/e68a/9e22f650bcb8ffe2e6b0a8b13096a6595b31.pdf

@trappmartin Thanks!


Infinite loading when scrolling to the bottom

SoundCloud always puts a loading gif at the bottom of the page to indicate that there are more tracks to be loaded (whether you have scrolled to the bottom or not)
There seems to be a height detection mechanism that prevents more tracks from loading when using the userstyle, will look into it.

Do you encounter the same issue on your end?

No I don't get this issue. What Browser & theme version do you have?

Browser: Firefox Developer Edition v97.0b9
Theme version: v0.9.161

Seems like a problem on my end. Fixed #4


Reach Out Modal

https://trello.com/c/GxhB3dsV/162-pop-up-when-clicking-reach-out-on-results-page

IBM Cloud toolchain: Delivery Pipeline deployed LRP-web-app to dev, including commit 6337e62f33f3ebd89461b6a7a5b282b0f7e73946

IBM Cloud toolchain: Delivery Pipeline deployed LRP-web-app to test, including commit 6337e62f33f3ebd89461b6a7a5b282b0f7e73946

IBM Cloud toolchain: Delivery Pipeline deployed LRP-web-app to prod, including commit 6337e62f33f3ebd89461b6a7a5b282b0f7e73946


Added SGX Plug-in details and collaterals.

Can we get this merged over please if there are no comments?


Mod Checking for Install supported Mods

I understand the mod supports ores from multiple Mods, by any chance does check to see which mod are installed and prevent loaded assets for mod you are not using? I ask because i'm using the mod on my forge server, and notice error being spammed because "Botany Pots Ore Planting" can't find ores from other mods such as Botania, Thermal, Immersive, Allthemodium which are not on my server.
is that something you can fix, ot just not cause it report because the mods are not installe?
latest.log

If it does not affect the server startup, then you can ignore this error. After all, I can't do what you said, sorry.


Add type hints to gyazo.Api

Part of #31

Coverage increased (+0.2%) to 54.795% when pulling 28948bec8fee0026bb44c5fd2971325e74b63e8c on add-type-hints into d47d92313f8d1ebfddb125b6a3bc4c08761476c7 on master.


FIX: Import from root

https://github.com/pyvista/pyvista/pull/1310 broke things in pyvistaqt:
https://app.circleci.com/pipelines/github/mne-tools/mne-python/8243/workflows/ab89194f-358e-4ed6-9cd1-3e9ca7fdebfa/jobs/28518
This should fix it by importing rcParams from pyvista root directly

This one seems pretty trivial (and I tested locally with latest 0.30 on PyPi and latest PyVista master) so I'll go ahead and merge


How to add further file types

Is there a way to add a new file type, in particular Adobe Illustrator files to the config file?

Please visit the configuration document and find out which mime type to fill in from here.

v1.7.0 has released. Please follow the instructions in readme.


Create graphgen service for probabilistic fiber tracts

This involves making a design decision regarding how we want to represent our graphs - do we threshold fibers and have deterministic graphs? Do we have probabilistic graphs? We can discuss once I have stable probabilistic fiber generation, @willgray @jovo.

i know things - next m2g meeting, perhaps?
Short answer is save both.
Best,
Will
On Mon, Jul 20, 2015 at 12:58 PM, Greg Kiar<EMAIL_ADDRESS>wrote:

This involves making a design decision regarding how we want to represent
our graphs - do we threshold fibers and have deterministic graphs? Do we
have probabilistic graphs? We can discuss once I have stable probabilistic
fiber generation, @willgray https://github.com/willgray @jovo
https://github.com/jovo.
—
Reply to this email directly or view it on GitHub
https://github.com/openconnectome/m2g/issues/109.

most-def, though implementation is likely a week or two away so no huge
rush on the discussion if other things are more pressing :)
I agree, save both, is for sure where we want to start.
--
Greg Kiar<EMAIL_ADDRESS>On Mon, Jul 20, 2015 at 1:03 PM, William Gray<EMAIL_ADDRESS>wrote:

i know things - next m2g meeting, perhaps?
Short answer is save both.
Best,
Will
On Mon, Jul 20, 2015 at 12:58 PM, Greg Kiar<EMAIL_ADDRESS>wrote:

This involves making a design decision regarding how we want to represent
our graphs - do we threshold fibers and have deterministic graphs? Do we
have probabilistic graphs? We can discuss once I have stable
probabilistic
fiber generation, @willgray https://github.com/willgray @jovo
https://github.com/jovo.
—
Reply to this email directly or view it on GitHub
https://github.com/openconnectome/m2g/issues/109.

—
Reply to this email directly or view it on GitHub
https://github.com/openconnectome/m2g/issues/109#issuecomment-122952688.

migrated


Create a search panel in the patient list view in the reference app

Can I work on this?

This issue is not ready to be worked on yet.
Design discussion pending.

@jingtang10 Can you please provide a mockup for the search.


Bugfix/quantity input field

Related issues
closes #2398, #2437
Short description and why it's useful
Code from product page has been moved to separate component and used in cart as well. Also i get rid of "edit" option in cart, for simplicity user can quickly edit quantity in cart.
Screenshots of visual changes before/after (if there are any)
Before:

After:

Upgrade Notes and Changelog

[x] No upgrade steps required (100% backward compatibility)
[x] I've updated the Upgrade notes and Changelog on how to port existing VS sites with this new feature

That's awesome https://www.youtube.com/watch?v=9UJ1syXaNoQ
Please do resolve conflict and merge this in

@patzick resolve conflict and we can merge it in ;)