restrict hostname checks to softlayer only

Apparently adding hostname prefix field in the properties for softlayer broke AWS because of extra hostname that is not allowed.  This PR moves the logic to a separate function and modified the tests to hopefully better isolate these special cases.
Signed-off-by: David Chung<EMAIL_ADDRESS>@JacobFrericks @ndegory

LGTM, with this patch I was able to successfully deploy on AWS using the Terraform plugin.

LGTM I'm still able to spin up VSIs in SoftLayer using the magic key to have configurable hostnames.

Thanks guys for confirmation!


multiple refactoring + enhancements

Description
features / fixes:

var.bucket_name has been added to allow consumers complete control over the naming of buckets. This is a required variable if var.create_cos_bucket is true.
var.cos_instance_name is now required if var.create_cos_instance is true to allow consumers complete control over the naming of the COS instance.
var.existing_cos_instance_id must now be used to pass an existing COS instance.
var.existing_cos_key_ring_name is now available to create a key in an already existing key ring. Previously the module only supported creating a new key ring.
var.cos_key_name now takes a string input instead of a list, as only 1 key can ever be passed into the bucket for encrpytion.
var.enable_key_protect_metrics has been added to allow consumers to enable or disable metrics on a newly provisioned Key Protect instance.

docs:

readme cleanup
variable descrption cleanup

examples:

moved the old bucket-without-tracking-monitoring example into the new existing-resources example
moved the old complete-multiple-buckets example into the complete example
complete:

no longer provision logdna (not used by example)
now creates 2 buckets

existing-resources:

a new example that provisions COS, Key Protect, Key Ring + Key outside of the terraform-ibm-cos module
terraform-ibm-cos will just create the bucket

tests:

All tests now run in pr_test.go (deleted other_test.go)
removed code duplication
tests now use existing resource group "geretain-test-cos-base"

BREAKING CHANGES:

var.bucket_infix and var.environment_name have both been removed. Use var.bucket_name to create the bucket name now. Previously the bucket name was named like "${var.environment_name}${local.infix}-bucket-${var.region}".
You now need to use var.cos_instance_name for naming the COS instance to be provisioned. Previously, the COS instance was named like "${var.environment_name}-cos".
var.existing_cos_instance_id must now be used to pass an existing COS instance.
var.cos_key_name now takes a string input instead of a list.
key_protect_instance_id output was actually outputting the GUID, so the output has been renamed to key_protect_instance_guid.

Check the relevant boxes:

[x] Bug fix (nonbreaking change that fixes an issue)
[x] New feature (nonbreaking change that adds functionality)
[x] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[x] Examples or tests (addition or updates of examples or tests)
[x] Documentation update
[ ] CI related update (pipeline, etc.)

Checklist

[x] If relevant, a test for the change has been added or updated as part of this PR.
[x] If relevant, documentation for the change has been added or updated as part of this PR.

Merge

Merge using "Squash and merge".
Make sure to use a relevant conventional commit message that is based on the PR contents. The commit message determines whether a new version of the modules needs to be released, and if so, which semver number to use).

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


 Otherwise it's pretty much a copy-paste job compared with the one in main
It also alters the long_read logic to be the default - I think the same AF/AD/AC annotations need to be applied from variant_qc as they aren't populated from the combiner, so we just do that for all runs now.

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 26.78%. Comparing base (6ccf588) to head (88c8bf8).

Additional details and impacted files
@@              Coverage Diff               @@
##           rd_combiner_8    #1038   +/-   ##
==============================================
  Coverage          26.78%   26.78%
==============================================
  Files                  9        9
  Lines               1725     1725
==============================================
  Hits                 462      462
  Misses              1263     1263

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


built front end searchbar component and started search landing page

will work on the api request later
chekc on figma to see what i think the search page should look like

Will wait for full feature commit until resolving merge conflicts and building errors.
Left some comments on the Figma.

should be stable version, check merge again


Example number generated for Panama is someone's personal phone number

I have a web design client that asked me to look into this issue for him. He keeps seeing his personal cell phone number appear as the example/placeholder number in tel inputs on apps and websites (+507 6001-2345) and also regularly receives calls from the users of these apps and sites, many times they are angry and looking for support. I found that Jack O'Connor's intl-tel-input plugin was displaying this number, contacted him, and he said the number is generated by libphonenumber.
The correct number for Panama should be +507 6000-1234, I've tested it and it is disconnected.

Thank you for bringing this to our attention!
We regret the inconvenience and will fix this ASAP.
We do our best to avoid this from happening by, for example, sourcing our
example numbers from ITU docs where such numbers are provided by the region's
telecom authority. Where this is not present, we have to resort to fake numbers
that fit the valid ranges, and at the time of addition, we try to ensure these
are unassigned.
Sometimes these numbers do get assigned later, and at that point it is through
such reports that we know to take action.
Please note that you have submitted this information with the knowledge that we
may use, modify, reproduce, publicly display, and distribute the information in
accordance with our open source policies. Please let us know if you did not
intend to submit the information for such use.
Thanks for using the library!

Fixed in release 8.8.2.


Create local volumes instead of hostPath

Create local volumes instead of hostPath ones. Closes #85 .
This needs some documentation changes at the least.

The code part LGTM. We do need a bit more documentation updates.
Also, I am not using Velero so it would be helpful if someone else can give this PR a test with Velero.

I will merge this for now. Let's test it for a few days before releasing the next version.


[DispatchFormation] Fix CollapseDimensions for dynamic dimensions (and other things)

Fixes CollapseDimensions to work properly on dynamic dimensions
Fixes CollapseDimensions thinking that the first input map in a linalg.generic represents the whole iteration space (why :( )
Fixes CollapseDimensions to not assume that a loop for a producer is not collapsable if the consumer doesn't contain it. This always happens for reduction dimensions.

Depends on:

https://github.com/llvm/llvm-project/pull/118202
https://github.com/llvm/llvm-project/pull/118203
https://github.com/llvm/llvm-project/pull/118208

Need to add tests

Depends on:

https://github.com/llvm/llvm-project/pull/118202

I think it makes sense to split this PR up since https://github.com/iree-org/iree/pull/19326 tries to fix the second point in the PR description and there is a PR dependent on that change. The dynamic dim fix might take a while since upstream changes are needed.

Fixes CollapseDimensions thinking that the first input map in a linalg.generic represents the whole iteration space (why :( )

I haven't looked at the fix here extensively, but it might make sense to use that instead. Also, for future reference, here is a dispatch we want to remove unit dims from https://gist.github.com/IanWood1/88007e15ae0ea2aa80a64407bc5897ad


Improve testing setup.

Don't test the compilation of files for which the Emacs version is
too low for the UI. For example, Vertico requires Emacs 27.1, so we
shouldn't bother testing vertico-prescient.el with Emacs versions
less than that. This makes implementations a bit simpler.
To do that, explicitly pass the version argument
for the docker make target.
Don't raise an error for long line of package requirements.

✨ Looks fine, although I do have a comment at #156 (comment). I looked at the actual runtime and I see building the Docker image only takes 15 seconds, so I guess it doesn't matter too much. Is the motivation for splitting the make targets out into a matrix to report easier to read results on the GitHub status checks?

Yes, but I realized that I was focusing on the wrong thing when I wrote this. For #156, it only needs that the "Package Require" line can be longer than 80 characters. I meant to make a new PR for just that, but accidentally pushed that change to the main branch instead of creating a PR. My apologies.

No worries!


Go over how login and refresh errors are reported.

My own testing shows this is marginally better.
I am not sure how the code before worked with refresh since  it was testing on e.status and strings were being returned or on objects the in the HTTP response field was statusCode, not status.
Feel free to try out and see what you get. This can be a huge time waste though.
There also seems to be a lot of opportunity to DRY code, but alas I didn't have the patience for that either.

Pull Request Test Coverage Report for Build 396

13 of 19 (68.42%)  changed or added relevant lines in 4 files are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage decreased (-0.1%) to 79.054%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

lib/login.js
4
5
80.0%

lib/requester.js
0
5
0.0%

Files with Coverage Reduction
New Missed Lines
%

lib/requester.js
1
60.56%

Totals

Change from base Build 394:
-0.1%

Covered Lines:
249

Relevant Lines:
307

💛  - Coveralls


Fix case when root component is ignored from allComponentResultsFromRoot when there are no variants

Before this PR
When DependencyGraphUtils#allComponentResultsFromRoot is called with a root that has has no variants (ie GCV has not managed to put a dependency constraints platform on it, like if the configuration has been copied so not visible to GCV), it is ignored as part of GCV platform checks due to a logic error.
After this PR
==COMMIT_MSG==
Root components with no variants are no longer spuriously excluded from DependencyGraphUtils#allComponentResultsFromRoot.
==COMMIT_MSG==
Possible downsides?

Released 0.5.0


Missing Validation Messages for Address Form

When the user attempts to register an address with invalid form data by clicking the submit button, they need feedback to indicate what the errors are in the form.

Closing as Invalid address message is now showing on invalid form submission

I see a generic error message "invalid address", which suffices for now.
But it requires a more detailed error message in future versions.

See #6592

See #6594 as well.


Dependency on old GDAL version 2.2

GDAL has had a 2.4 version release, but this package is still requiring < 2.3, I believe the simple fix is:
@conda-forge-admin, please rerender
@santoshamohan

Hi! This is the friendly automated conda-forge-webservice.
I just wanted to let you know that I rerendered the recipe in conda-forge/tuiview-feedstock#25.

Closed by #25


error obtaining VCS status: exit status 128

Describe the bug
This error message appeared when trying to use gitleaks on pre-commit and gitleaks failed to scan the commit
### version information

pre-commit version: 3.3.3
git --version: git version 2.38.1
sys.version:
    3.11.4 (main, Jul 25 2023, 17:07:07) [Clang 14.0.3 (clang-14<IP_ADDRESS>.1)]
sys.executable: /usr/local/Cellar/pre-commit/3.3.3/libexec/bin/python3.11
os.name: posix
sys.platform: darwin

### error information

An unexpected error has occurred: CalledProcessError: command: ('/usr/local/go/bin/go', 'install', './...')
return code: 1
stdout: (none)
stderr:
    go: downloading github.com/gitleaks/go-gitdiff v0.8.0
    go: downloading github.com/rs/zerolog v1.26.1
    go: downloading github.com/spf13/viper v1.8.1
    go: downloading github.com/lucasjones/reggen v0.0.0-20200904144131-37ba4fa293bb
    go: downloading github.com/spf13/cobra v1.2.1
    go: downloading github.com/charmbracelet/lipgloss v0.5.0
    go: downloading github.com/fatih/semgroup v1.2.0
    go: downloading github.com/h2non/filetype v1.1.3
    go: downloading github.com/petar-dambovaliev/aho-corasick v0.0.0-20211021192214-5ab2d9280aa9
    go: downloading golang.org/x/sync v0.0.0-20210220032951-036812b2e83c
    go: downloading github.com/lucasb-eyer/go-colorful v1.2.0
    go: downloading github.com/mattn/go-runewidth v0.0.13
    go: downloading github.com/muesli/reflow v0.2.1-0.20210115123740-9e1d0d53df68
    go: downloading github.com/muesli/termenv v0.11.1-0.20220204035834-5ac8409525e0
    go: downloading github.com/spf13/pflag v1.0.5
    go: downloading github.com/fsnotify/fsnotify v1.4.9
    go: downloading github.com/hashicorp/hcl v1.0.0
    go: downloading github.com/magiconair/properties v1.8.5
    go: downloading github.com/mitchellh/mapstructure v1.4.1
    go: downloading github.com/pelletier/go-toml v1.9.3
    go: downloading github.com/spf13/afero v1.6.0
    go: downloading github.com/spf13/cast v1.3.1
    go: downloading github.com/spf13/jwalterweatherman v1.1.0
    go: downloading github.com/subosito/gotenv v1.2.0
    go: downloading gopkg.in/ini.v1 v1.62.0
    go: downloading gopkg.in/yaml.v2 v2.4.0
    go: downloading github.com/rivo/uniseg v0.2.0
    go: downloading golang.org/x/sys v0.0.0-20211110154304-99a53858aa08
    go: downloading golang.org/x/text v0.3.6
    go: downloading github.com/mattn/go-isatty v0.0.14
    error obtaining VCS status: exit status 128
    	Use -buildvcs=false to disable VCS stamping.
    error obtaining VCS status: exit status 128
    	Use -buildvcs=false to disable VCS stamping.

To Reproduce
Steps to reproduce the behavior: https://github.com/gitleaks/gitleaks#pre-commit
Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
Basic Info (please complete the following information):

OS: Mac OS 13.5 (22G74)
Gitleaks Version: v1.17

Additional context
Add any other context about the problem here.
cc @zricethezav

Running pre-commit install-hooks worked for me.
https://github.com/pre-commit/pre-commit/issues/2722#issuecomment-1527611050

Running pre-commit install-hooks worked for me.
pre-commit/pre-commit#2722 (comment)

That worked, thank you!


How to generate qr code with transparent background?

Hello,
I am generating qrcode as below. Even though I made the background transparent, it casts as black.
public static Bitmap CreateQrCode(string code, int quality)
{
      QRCodeGenerator qrGenerator = new QRCodeGenerator();
      QRCodeData qrCodeData = qrGenerator.CreateQrCode(code, QRCodeGenerator.ECCLevel.Q);
      QRCode qrCode = new QRCode(qrCodeData);
      Bitmap qrCodeImage = qrCode.GetGraphic(quality, Color.Black, Color.Transparent, true);

     return qrCodeImage;
}

I think you shouldn't use transparent in your QR Code image. It's violate the QR Code spec. The white-part has it's purpose.

Hi @saklanmazozgur ,
Your code produces a perfectly fine transparent image. I think the problem lies in (the not shown) part where you display the code.
To keep transparency you have to save the Bitmap in a format that supports transparency, e.g. PNG. If you save the Bitmap as bmp, transparency will get lost. But that's not QRCoders fault, but how BMP works.
If your problem is missing transparency when showing the QRCode in a UI component, be aware that you have to set a transparency key for many of the components. E.g. for PictureBox it's done that way: https://stackoverflow.com/questions/19910172/how-to-make-picturebox-transparent
(For other controls it might be different, but can't help you here, without knowing how/where you are displaying the QR code.)


Rename list to array

Is your feature request related to a problem? Please describe.
List is a confusing name.
Describe the solution you'd like
Rename list into array.
Describe alternatives you've considered
None.
Additional context
None.

This might be trickier than expected since the work Array is a reserver keyword.

At second though, no. :joy:


Access Token in header issue

Hi, me and my team building an app with login functionality.
our app has access token but never expired.
so we want just change access token when login and logout repeatedly.
But after some research we found Moya doesn't provide auth api like change token, refresh token etc...
so is it okay to just insert access token in our TargetType header like below.
`
    var headers: [String: String]? {
    var headers = [String: String]()
    let accessToken = KeychainWrapper.standard.string(forKey: "access_token")
    switch self {
    case .signIn, .refreshToken:
        headers["Content_Type"] = "application/x-www-form-urlencoded; charset=utf-8"
        return headers
    case .getMyInfo,
         .getMyTeams,
         .getCurrentDirectoryFiles,
         .getStorageInfo,
         .getTeamLicenses:
        headers["Authorization"] = "bearer \(accessToken!)"
        return headers
    case .addFolder,
         .renameFile,
         .removeFiles,
         .copyFiles,
         .moveFiles:
        headers["Authorization"] = "bearer \(accessToken!)"
        headers["Content_Type"] = "application/json; charset=utf-8"
        return headers
    case .uploadImage:
        headers["Authorization"] = "bearer \(accessToken!)"
        headers["Content-Type"] = "multipart/form-data"
        return headers
    }
}`

Hi @dagad! What you described is one of the options you have to use an access token in your API and it's totally fine!
However, I believe that using our AccessTokenPlugin will give you a little bit more clarity and flexibility in the code. Please take a look at this tutorial about it and let me know if you have any more questions about it :)

Thanks you for reply @sunshinejr @pietrocaselani. i used AccessTokenPlugin


updated gulp to watch new file structure

DEV environment updates automatically when .json is modified.
gulp watches for modifications in .json in seed/challenges. This feature helps developer's productivity
But the file structure is modified in this release. So gulp file is modified to reflect the same.

:shipit:

Probably, this could've been like the following (but i'm not sure or if this might work or not)
challenges: [
    'seed/challenges/**.json'
]

Is that for recursive search in directories?

@gasperno yep! That's what the wildcard means. But i'm not sure if it works for that use case. You may run and check it.


undefined is not an object (evaluating 'this.onStarRatingPress.bind')

HI i'm having an error. I copied your example code.

sorry, the name of the method was wrong, I have update readme, please check code.


Add deno function for covid data

Fixes appwrite/appwrite#1923
Signed-off-by: Moulik Aggarwal<EMAIL_ADDRESS>

@aggmoulik  closing this PR for now since there has been no activity in quite a long time. Feel free to reopen it when you're ready to start working on it once again. 🙂


Add support for currency codes (very easy)

Great library, really appreciate it!
One tiny missing piece is support for currency codes (http://www.xe.com/iso4217.php).
What I would like to be able to do:
//in my config file
"currency": {
"format": {
"unit": "$",
"code": "USD",
"precision": 2,
"format": "%c%u%n"
}
Which would produce "USD$1,000.00"
There are other locales where you would need to set the format like this: "%n%c"  to get "100 SEK" for example.
I think all that is required is to add this line of code in a few places:
    .replace("%c", options.code)

@jfbrennan I believe locale and currency should be two separate concepts. In other words, there is not a 1:1 relationship between language and currency. So my vote would be to not add this feature to this gem.

This is not supported in I18n rails btw. There is just unit. You could include the code in the unit as well. I'm not sure how often there is a need to separate the two: "SEK 100 kr"? One option is to use two operations and do a translate and a toCurrency: I18n.t('number.currency.format.code') + ' ' + I18n.toCurrency(1000);


[Component] Add dividers in command menus

Description
Many types of applications often divide their menu items up to emphasise and group certain options. It would be useful if there were a way to do this in Aurora.
Current Result:
All items in command menus are grouped together.

Aim:
Dividers could be used to split up groups of options, like the HorizontalSeparatorProjection, but in a command menu.

Platform Info
Aurora Version: 1.1.0

This should be available already if you use multiple command groups. The horizontal separators should be shown between groups of commands in the menu.

Just what I was looking for, thank you.


AEM installation through maven content package plugin

Hi,
I've had some trouble interpreting the documentation regarding the deployment of the neba-delivery-aem package through maven:
http://neba.io/download.html
It mentions the content package maven plugin but it is not very clear how to include it in the project structure generated by the AEM maven archetype.
I've tried two separate approaches:

Create an entirely separate project for the NEBA delivery

Include an additional execution step in the "ui.apps" package
In both cases I've been unable to work with the provided sample configuration, it seems it is lacking the version tag. After including the version it works, otherwise it doesn't seem to resolve the artifact. Also the dependency part mentioned in the doc doesn't seem to be needed, altough it is standard practice.

separate project
 	<plugin>
 		<groupId>com.day.jcr.vault</groupId>
 		<artifactId>content-package-maven-plugin</artifactId>
 		<executions>
 			<execution>
 				<configuration>
 					<groupId>io.neba</groupId>
 					<artifactId>io.neba.neba-delivery-aem</artifactId>
 					<packaging>zip</packaging>
 					**<version>${lib.neba.delivery.aem.version}</version>**
 					<targetURL>http://${aem.host}:${aem.port}/crx/packmgr/service.jsp</targetURL>
 					<failOnError>true</failOnError>
 				</configuration>

 				<goals>
 					<goal>install</goal>
 				</goals>
 			</execution>
 		</executions>
 	</plugin>

"ui.apps"
            <plugin>
                 <groupId>com.day.jcr.vault</groupId>
                 <artifactId>content-package-maven-plugin</artifactId>
                 <executions>
                     <execution>
                         <id>install-package</id>
                         <goals>
                             <goal>install</goal>
                         </goals>
                     </execution>
 					<execution>
 						<id>install-neba</id>
 						<configuration>
 							<groupId>io.neba</groupId>
 							<artifactId>io.neba.neba-delivery-aem</artifactId>
 							<packaging>zip</packaging>
 							**<version>${lib.neba.delivery.aem.version}</version>**
 						</configuration>
 						<goals>
 							<goal>install</goal>
 						</goals>
 					</execution>
                 </executions>
                 <configuration>
                     <targetURL>http://${aem.host}:${aem.port}/crx/packmgr/service.jsp</targetURL>
                     <failOnError>true</failOnError>
                     <failOnMissingEmbed>true</failOnMissingEmbed>
                 </configuration>
              </plugin>

Also may be worth mentioning that AEM 6.1 SP2 solves the issue #69 with javax.inject being incorrectly exported by Sling Models / Java Scripting API bundles, we had to include the geronimo spec bundle for providing the javax.inject dependency through the "embedded" section in the content package plugin.
Cheers, João

Hi @joaotab , thanks a lot for the hints and testing effort!
Indeed, We've not yet thought about a best-practice of integrating into the Adobe's default AEM archetype.
Regardless, I have updated the download docs with regard to your findings and - hopefully - made the part with the dependencies more clear. Could you please review?
Regarding the AEM SP2: I believe the issue is still existing in AEM 6.2 at the moment, but indeed we may be able to remove the customized version range in NEBA. At the moment, NEBA will accept any javax.inject package in the range [0,2).
I didn't get what you meant regarding the deployment of the javax.inject geronimo spec bundle via the embedded dependencies - was this something your own app was depending on?

Hi @olaf-otto, thanks, I see you've included the version in the doc. My original issue was that I had to dig around to find out why it wasn't being deployed and it was about the missing version.
Regarding the inject, it is in fact related with the sample code from the AEM archetype, I just realized we still have some of it lying around. We've switched to resource and autowired for osgi and spring wiring in the meantime.


support matrix configuration for integTests, which defines a list of integTests that should run against a type of artifact

This was covered as part of #338


Make setSentryUnhandledExceptionHook public

Problem Statement
We have a mixed KMP and native app, meaning that the limited configuration options available via this library, limit what we can enable on both the Android and iOS sides.
On the iOS side, AFAICT the only thing library does is called SentrySDK along with setting an unhandled exception handler.
Solution Brainstorm
Since I would rather call SentrySDK.start {} in Swift, can we make the setSentryUnhandledExceptionHook function public?

hi @chrisbanes thanks for the FR, I think this makes sense to expose it, we'll discuss internally and make it public in the next SDK version. Will keep you posted!

This is now released with 0.6.0


test(pods-communication) add tests for basic pod communication

Add test case which creates a pod with two containers and tests if a http-client on one can access nginx on the other container
Closes DCOS_15051

hm, external tests alre included in this one? thats wrong.


what happens when substrate node down (/ misconfigured) and procaptcha cannot talk to it?

Currently there's a quiet error.

only 1 node for rococo, can't have redundancy!


Every Incoming packets are sent to handler even if the hash is not valid

Hello,
Based on my tests, it seems Decodepacket calls the handler even if the hash is insane.
I think the good behaevior would be to silently drop malformed packet (perhaps notify but NOT call the handler)
The following decode check the hash correctly before calling the handler.
tofgau
func DecodePacket(Secret string, buf []byte) (p Packet, err error) {
//	fmt.Printf("\n\ndecEntr*%v", buf)
if len(buf) < 20 {
return nil, errors.New("invalid length")
}
p = &Packet{Secret: Secret}
p.Code = PacketCode(buf[0])
p.Identifier = buf[1]
copy(p.Authenticator[:], buf[4:20])
//read attributes
b := buf[20:]
for len(b) >= 2 {
length := uint8(b[1])
if int(length) > len(b) {
return nil, errors.New("invalid length")
}
attr := AVP{}
attr.Type = AttributeType(b[0])
attr.Value = append(attr.Value, b[2:length]...)
p.AVPs = append(p.AVPs, attr)
b = b[length:]
}
//验证Message-Authenticator,并且通过测试验证此处算法是正确的
//Verify Message-Authenticator, and tested to verify the algorithm is correct here
//	err = p.checkMessageAuthenticator()
//Tofgau 201812 : this is not used anymore

//tofau : Dump Original Buffer
//fmt.Printf("\n\n****BUF0 %x", buf)
oldAuth := p.Authenticator
//fmt.Printf("\n****PKHASH %x", oldAuth)

//Duplicate the buffer and white the hash part
tmp := make([]byte, len(buf))
copy(tmp, buf)
var white [16]byte
copy(tmp[4:20], white[:])

//tofau : Calculate a hash on this new buffer concatenated with the secret
hasher := crypto.Hash(crypto.MD5).New()
hasher.Write(tmp)
hasher.Write([]byte(p.Secret))

calculatedHash := hasher.Sum(nil)

//tofau :
//fmt.Printf("\n****MYHASH %x", calculatedHash)

if !hmac.Equal(calculatedHash, oldAuth[:]) {
	//fmt.Printf("\n\nINVALID PACKET")
	return p, ErrMessageAuthenticatorCheckFail
} else {
	//fmt.Printf("\n\nVALID PACKET")
}

return p, nil

/* supressed by tofgau
if err != nil {

	return p, err
}

return p, nil
*/

}

Can you add a test and send a pull request?
Sorry, I do not have radius test environment right now.

I will try, i am new to github.
Thks
Le sam. 1 déc. 2018 à 02:16, bronze1man<EMAIL_ADDRESS>a écrit :

Can you add a test and send a pull request?
Sorry, I do not have radius test environment right now.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/bronze1man/radius/issues/17#issuecomment-443386109,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AqNM4v9D7oh9cNC9IVIMnwuVIVeVZSU9ks5u0dhmgaJpZM4Y7KJO
.


Upgrade to Google API v37.1.0?

Hi, thank you for the great job.
Do you plan to upgrade the requirements to the latest googleads-php-lib version v37.1.0 (201809)?

Hi @giv13 ,
Thanks for your message.
Yes, I'll upgrade the googleads dependency to its latest version as soon as I get some free time.
I can not provide an ETA but I'll keep this issue opened and update it once it's done.

Hi @giv13 ,
I've just updated the package with the lastest google ad version.
Hope this helps :)

Many thanks! But I found one mistake. In file src/Reports/Format.php in row 14 from use Google\AdsApi\AdWords\Reporting\v201710\DownloadFormat; to use Google\AdsApi\AdWords\Reporting\v201809\DownloadFormat;

Closing this issue.
Feel free to open it back if needed.


feature: improve file_transfer

Describe the solution you'd like
Our templates should show off best practices. Currently, file_transfer falls short. E.g., it has hanging code that is never used; it doesn't send Responses as a result of successful operations; etc.

#190


addText with hyperlink and addImage cannot co-exist in a slide?

Hello Team,
Not sure if this is a bug or PowerPoint's limitation but I have tried and tested adding both hyperlink text and image in a same slide using addText and addImage respectively, which will cause an error as below when trying to open the downloaded ppt:

However, the PowerPoint shows no error if the slide only exists either hyperlink text or image.
Thanks!

Please provide the PowerPoint version, the PptxGenJS version, and the code being used.

I have tested on PowerPoint version 2010 and 2016.
PptxGenJS version used is 2.4.0
Code used is as below:
`var pptx = new PptxGenJS();
var slide = pptx.addNewSlide();
var position = {
x: 0.4,
y: 1.0
};
    var rows = [];
    for (var m = 0; m < 4; m++) {
        rows.push({
            text: "Text " + m,
            options: headerOption
        });
    }

    slide.addText(rows, position);
    slide.addImage({ path:'https://upload.wikimedia.org/wikipedia/en/a/a9/Example.jpg', x:1.5, y:1, w:6, h:4 });`

Issue has been resolved. Target release 2.5.0


How to obtain raw bin1 x gene count matrix from GEM file?

I would like to obtain the "cell x gene" raw expression count matrix, where the cells in this case are the individual nanoballs (bin1).
For reference, I am working with the Mouse_brain_Adult_GEM_bin1.tsv.gz file located here:
https://db.cngb.org/stomics/datasets/STDS0000058
So far I have been able to read in the file using:
import stereo as st
data_path = 'Mouse_brain_Adult_GEM_bin1.tsv.gz'
data = st.io.read_gem(
        file_path=data_path,
        sep='\t',
        bin_type='bins',
        bin_size=1,
        is_sparse=True,
        )

data

StereoExpData object with n_cells X n_genes = 42147676 X 26177
bin_type: bins
bin_size: 1
offset_x = 3225
offset_y = 6175
cells: ['cell_name']
genes: ['gene_name']

From here, are their any commands or workflows to extract the raw counts matrix? I tried:
data.tl.raw and data.raw but their values are None.
Any help would be greatly appreciated. Thanks!
Brendan

Hi @UglyRay7,
Thanks so much for the quick reply. Yes, it seems the raw count matrix is stored in data.exp_matrix and I can access it that way. I tried data.tl.raw_checkpoint however data.tl.raw was still empty afterwards. But nonetheless, I can still get to the raw count data. Thanks!
Brendan


need help with a script for a timeBar in 0.6.3 (lua)

Describe your problem here.
I apologize if there was the same thing to this but i tried my best to look everywhere for it, so let me get straight into it..
So what i'm trying to do is when the timebar is disabled (in the game settings), the thing that's covering the timebar should also be disappeared
(just incase, this is a picture of what im talking about)

as you seen the picture above, i want that thing to disappear when you set your timeBar disabled.
i really tried everything the best i could, if this is easy to do then i really need to learn more lmfao
and here's a lua script i made for the thing
function onCreatePost()
	makeLuaSprite('Time', 'timebar-seq/time9');
	setObjectCamera('Time', 'hud')
	addLuaSprite('Time', true)
	setObjectOrder('Time', getObjectOrder('timeBar') + 1)
	setProperty('timeBar.visible', true)
	setTextFont('timeTxt', 'Comfortaa-Bold.ttf')
end

function onUpdatePost(elapsed)
	setProperty('Time.visible', getProperty('timeBar.visible'))
	setProperty('Time.alpha', getProperty('timeBar.alpha'))
	scaleObject('Time', 1, 1.1)
	setProperty('Time.x', getProperty('timeBar.x') - 40)
	setProperty('Time.y', getProperty('timeBar.y') - 27)
end

function onUpdate()
if timeBarType == Disabled then
removeLuaSprite('Time')
end
end

i really tried stuff like setPropertyFromClass('ClientPrefs', 'TimeBarType') but it didn't work how i wanted it to be and i honestly wanted help on this.
Are you modding a build from source or with Lua?
Lua
What is your build target?
Windows x64
Did you edit anything in this build? If so, mention or summarize your changes.
No response

I think I figured it out, Here:
function onUpdate() if setProperty('healthBar.visible', false) then setProperty('Time.visible', false) end end
Here, if 'Hide HUD' is turned on (or if the timebar isn't visible), the cover should disappear with the timebar!

that removed the healthbar colors and the timebar’s cover is still there

Sorry, I didn't check for if the hud was on


transitions dataset doesn't fetch

Looks like it just needs to be metadata['DESCR']
data.datasets.fetch_adk_transitions_DIMS()

---------------------------------------------------------------------------
NameError                                 Traceback (most recent call last)
<ipython-input-5-8bdebcdd3177> in <module>
----> 1 data.datasets.fetch_adk_transitions_DIMS()

~/miniconda3/envs/mda19/lib/python3.6/site-packages/MDAnalysisData/adk_transitions.py in fetch_adk_transitions_DIMS(data_home, download_if_missing)
     87     return _fetch_adk_transitions(METADATA['DIMS'],
     88                                   data_home=data_home,
---> 89                                   download_if_missing=download_if_missing)
     90
     91 def fetch_adk_transitions_FRODA(data_home=None, download_if_missing=True):

~/miniconda3/envs/mda19/lib/python3.6/site-packages/MDAnalysisData/adk_transitions.py in _fetch_adk_transitions(metadata, data_home, download_if_missing)
    192                                records.N_trajectories))
    193
--> 194     records.DESCR = _read_description(DESCRIPTION)
    195
    196     return records

NameError: name 'DESCRIPTION' is not defined

Closed in https://github.com/MDAnalysis/MDAnalysisData/commit/475a486f9790664a379d308a5d2d5c317c1c28ea


Change in colour when using operations on BMP images

Are you using the latest version of sharp?

[x] I am using the latest version of sharp as reported by npm view sharp dist-tags.latest.

If you cannot confirm this, please upgrade to the latest version and try again before opening an issue.
If you are using another package which depends on a version of sharp that is not the latest, please open an issue against that package instead.
What is the output of running npx envinfo --binaries --system --npmPackages=sharp --npmGlobalPackages=sharp?
  System:
    OS: macOS 12.2
    CPU: (8) arm64 Apple M1 Pro
    Memory: 76.86 MB / 16.00 GB
    Shell: 5.8 - /bin/zsh
  Binaries:
    Node: 18.16.0 - ~/.nvm/versions/node/v18.16.0/bin/node
    Yarn: 1.22.17 - /opt/homebrew/bin/yarn
    npm: 9.5.1 - ~/.nvm/versions/node/v18.16.0/bin/npm
  npmPackages:
    sharp: 0.32.5 => 0.32.5

What are the steps to reproduce?
Please run the below code snippet that reads a bmp file, performs a flip operation using sharp, and saves the output as bmp file. The output image changes to red color.
What is the expected behavior?
The color of the input image should not change. But it keeps changing to red. Even when using other transformations like rotate, resize etc.
Please provide a minimal, standalone code sample, without other dependencies, that demonstrates this problem
const sharp = require("sharp");
const bmp = require("@vingle/bmp-js");
const { readFileSync, writeFileSync } = require("fs");

const buffer = readFileSync("./input.bmp");
const bitmap = bmp.decode(buffer, true); // true: decode alpha channel

sharp(bitmap.data, {
    raw: {
        width: bitmap.width,
        height: bitmap.height,
        channels: 4,
    },
})
    .flip()
    .toBuffer({ resolveWithObject: true })
    .then(({ data, info }) => {
        const bitmap = bmp.encode({
            data,
            width: info.width,
            height: info.height,
        });

        writeFileSync("output.bmp", bitmap.data);
    });

Please provide sample image(s) that help explain this problem
input
output

Thanks for your reply.
Setting toRGBA as false in bmp.decode fixes the issue.
const bitmap = bmp.decode(buffer, false);

But I'm not sure whether this would work for all images.

Perhaps try https://www.npmjs.com/package/sharp-bmp ?

@AmeyMore98 Were you able to make any progress with this?

@lovell Yes, I solved this by using (bmj-js)[https://github.com/mooyoul/bmp-js] for encoding the images. Thanks for the help.
Sorry, didn't get a chance to update here.


'build' command

Build command to build application and library.
Folder structure for ltd projects are as follow:
$LTD_HOME
|
+--caches
|    +--project0
|    +--project1
|
+--project0
|    +--app
|    +--lib
|    +--inc
|
+--project1
     +--apps
     |    +--app0
     |    +--app1
     +--libs
     |    +--lib0
     |    +--lib1
     +--inc
     +--tests

Caches directory used to store all temporary and output files.
app and lib is used when the project only have 1 executable or 1 library. The name of the executable or library will be the project name.
apps and libs is used for multiple executables or multiple libraries in a project. The name of the folder will be used for the name of the binary.

Command format:
ltd build project_name

Issue is re-opened for some enhancements.


Message producer stop work with end of file

As soon as some file has been read, MessageProducer exits.
It's not an error, but this logis is in conflict with text file reading. The thing is: we stop reading of files only by user request (actually, it's tailing).
Ideal way - MessageProducer should not stop working with the end of the file.

added tail above producer


HDDS-1812. Du while calculating used disk space reports that chunk files are file not found

What changes were proposed in this pull request?

Copy DU from hadoop-common and customize it to ignore temporary chunk files.  Unfortunately the original implementation could not be simply extended.  Note that this means temp chunk files will not be counted towards HDDS space usage.  One possible alternative is ignoring errors from du (which returns with error due to the missing files), but that might make it more difficult to spot other problems.
Introduce an interface for creating various "space usage" implementations.  hadoop-common allows a single implementation to be configured, which is used for all volumes.  The factory interface allows creating different implementations or different parameters for different volumes.  Currently 3 factories are implemented:

pure du-based
pure df-based (this is actually based on java.io in both Hadoop Common and HDDS)
mock for testing -- this also helps avoid test failures due to "disk out of space" errors when allocating containers
Hybrid or other more intelligent implementation could be a future improvement.  Eg. one that would use du for shared volumes and df for dedicated ones.

Introduce an interface for the object that persists space usage info (to a file named scmUsed) to be able to:

skip persisting info from cheaper sources (ie. df)
use in-memory persistence for unit testing

Decouple caching and periodic refresh of space usage from its source.  Use a ScheduledExecutorService for refreshing disk usage info.  Use a "same thread executor" for unit tests.
Let container-service tests use code from HDDS common tests, which provide mock implementations of the new interfaces.  This also required:

Avoid using real config properties (eg. ozone.scm.client.bind.host) for testing config file generation.  The generated config file is picked up by other tests and causes failures.
Rename Log4J2 config file used by audit logger test to avoid creating an untracked audit.log (similar to HDDS-2063).  This would happen if some test starts components which use audit logger, and it picks up log4j2.properties by default.

https://issues.apache.org/jira/browse/HDDS-1812
How was this patch tested?
Added several unit tests, even for logic extracted from existing classes.  On compose-based cluster tested configuring each of the real factory implementations and configuring different refresh periods.  Tested du-based implementation by continuously creating/renaming files from shell.

Thanks @elek for the review and offline discussion.
I updated the patch to use @Config objects for the factories (both for specific implementations and the global one that creates it).  I agree that this makes the code more encapsulated, instead of scattering config info around several files (ozone-default.xml, HddsConfigKey, etc.)
I also renamed the factory config property a bit (...class to ...classname, since it's a string).


[util]: added script to download tokens / channels as csv file

@kunallillaney I will update for ndstore and can either merge or add separately.


[Batch mode] Cope with bugs that cause error  suppression.

In batch mode, any diagnostics located in non-primary files are suppressed, with the expectation that they will surface when the containing file is compiled as primary. Although this assumption should hold, it does not always. For example, certain inputs can interact with associated type inferencing to violate it.
This PR tracks the current primary input in the DiagnosticEngine, which passed it down to the DiagnosticConsumer. The FileSpecificDiagnosticConsumer uses this information to output the diagnostic into the appropriate primary's .dia file. It addresses rdar://49303574.
Needs https://github.com/apple/swift-lldb/pull/1431

@swift-ci please smoke test os x platform

@swift-ci please smoke test os x platform

Please test with following pull request:
https://github.com/apple/swift-lldb/pull/1431
@swift-ci please smoke test os x platform

This can't work as is. We emit diagnostics when we first see a problem, but we might first see a problem for declaration X1 in file X because we're trying to use X1 in file Y. Putting that diagnostic in Y's .dia file isn't correct because it won't always be Y that first tries to use X1.

Thanks! But I don't understand. Isn't this PR doing the same thing as in the pre-batch-mode, single-file case? That's what it seems to do in non-batch mode.
Also, what is the significance of "first"? Why is that important?

Glad the new factoring works. I have a bug to track down, then I'll rerun the tests.

The new refactoring led to a more complete change for how we handle non-primaries. I've pushed that so you can see it. Still plan to rename defaultDiagnosticLoc to one of your suggestions.

Please test with following pull request:
apple/swift-lldb#1431
@swift-ci please smoke test os x platform

Please test with following pull request:
apple/swift-lldb#1431
@swift-ci please smoke test os x platform

@jrose-apple I've finished the changes we spoke of, and also they led to a redoing on the subconsumer invariants. (Consumer cannot not be null anymore.) Check it out at your convenience and let me know. (Thanks for the great name!)
Also, I'm starting to look at the testing aspect. I want to include a test before landing this.
Thanks.

Please test with following pull request:
apple/swift-lldb#1431
@swift-ci please smoke test os x platform

Please test with following pull request:
apple/swift-lldb#1431
@swift-ci please smoke test os x platform

Please test with following pull request:
apple/swift-lldb#1431
@swift-ci please smoke test os x platform

Please test with following pull request:
apple/swift-lldb#1431
@swift-ci please smoke test os x platform

Please test with following pull request:
apple/swift-lldb#1431
@swift-ci please smoke test os x platform

Regarding testing, see https://github.com/apple/swift/pull/23735/files#diff-43eb268d973af29cc397746d4ed96b7a , which shows a modified test that is part of this PR. The test fails with the status quo and passes with this fix. Is that enough?

Please test with following pull request:
apple/swift-lldb#1431
@swift-ci please smoke test os x platform

Please test with following pull request:
apple/swift-lldb#1431
@swift-ci please smoke test linux platform

Please test with following pull request:
apple/swift-lldb#1431
@swift-ci please smoke test os x platform

Please test with following pull request:
apple/swift-lldb#1431
@swift-ci please smoke test linux platform

PS: Thanks for that "Implementation looks good"!

Please test with following pull request:
apple/swift-lldb#1431
@swift-ci please smoke test os x platform


Add repo docs

We want to make repodocs for the simulation and change the layout of the simulation folder structure a bit. These are efforts to get this simulator more 'out of early access'

I need to move the new socket example to it's new destination as well.


Fix data race in terminal.go

Should not execute wg.Add(1) inside of wait goroutine.

Same pr with https://github.com/chzyer/readline/pull/152


[iPhone] Artwork image may not show from left to right.

In some cases, depending on the device screen size, an artwork image might not be shown from side to side. Going into VIR mode and back changes the layout and fixes it.
Reproduction:

Load app on a small screen device (e.g. 4S)
Load an artwork (with presumably an aspect ratio that doesn’t jibe well with the device screen size) such as https://www.artsy.net/artwork/donald-baechler-the-edible-daydream
Notice that there are seemingly margins on both sides. Mind you, these are not margins around the image view, rather the image is centered in the image view while maintaining the aspect ration and thus showing white borders.

Enter VIR mode and leave it again
Notice that the image now shows from side to side. (In fact, the height of the image view is now larger than before.)

The issues seems related to this constraint.

This is intended. We want to always fit the artwork in the screen and make it
apparent that there is content below, so there is a constraint to limit the
height (and therefore width, depending on the ratio) of the artwork
On Mon, Jun 8, 2015 at 08:35, artsy/eigen
<EMAIL_ADDRESS>wrote:
Assigned #506 [https://github.com/artsy/eigen/issues/506] to @1aurabrown [https://github.com/1aurabrown] .
—
Reply to this email directly or view it on GitHub [https://github.com/artsy/eigen/issues/506#event-325184013] .[https://github.com/notifications/beacon/ADBlTjeKB1qR6iJrOQy9Rsjxqurng5Nmks5oRYOUgaJpZM4E4Njg.gif]

@1aurabrown That’s fair and a good point, but shouldn’t that be maintained after entering VIR mode?

The return from VIR seems weird. I will check this out, something isn't right
On Mon, Jun 8, 2015 at 08:38<EMAIL_ADDRESS><EMAIL_ADDRESS>wrote:
This is intended. We want to always fit the artwork in the screen and make it
apparent that there is content below, so there is a constraint to limit the
height (and therefore width, depending on the ratio) of the artwork
On Mon, Jun 8, 2015 at 08:35, artsy/eigen
<EMAIL_ADDRESS>wrote:
Assigned #506 [https://github.com/artsy/eigen/issues/506] to @1aurabrown [https://github.com/1aurabrown] .
—
Reply to this email directly or view it on GitHub [https://github.com/artsy/eigen/issues/506#event-325184013] .[https://github.com/notifications/beacon/ADBlTjeKB1qR6iJrOQy9Rsjxqurng5Nmks5oRYOUgaJpZM4E4Njg.gif]

Yes, something in this sequence is broken. Will check it out.
On Mon, Jun 8, 2015 at 08:40, Eloy Durán<EMAIL_ADDRESS>wrote:
@1aurabrown [https://github.com/1aurabrown] That’s fair and a good point, but shouldn’t that be maintained after entering
VIR mode?
—
Reply to this email directly or view it on GitHub
[https://github.com/artsy/eigen/issues/506#issuecomment-109979474] .[https://github.com/notifications/beacon/ADBlTi6TM92sWIgG2WPCFfSQlxsKEx8cks5oRYSsgaJpZM4E4Njg.gif]

@1aurabrown Well, testing with current master I can indeed no longer reproduce the originally described inconsistency. Feel free to close this one and create a new one about the animation.


Bump kernel version & scim2 version

Purpose

Bump kernel version to 4.10.18
Bump identity.inbound.provisioning.scim2.version to 3.4.91

PR builder started
Link: https://github.com/wso2/product-is/actions/runs/10250558321

PR builder completed
Link: https://github.com/wso2/product-is/actions/runs/10250558321
Status: success


[Bug] Git Connection issues

Is there an existing issue for this?

[X] I have searched the existing issues

Current Behavior
Connecting to Git

[ ]  The deploy key text should say "Paste this deploy key in your repository settings and give it write access. Learn more". This can be slightly more prominent, maybe place it above the deploy key instead of below?-->UI
[ ] When the user pastes the Wrong URL then paste the right one and click on generate SSH key the button doesn't work
[ ] there is a slight delay in the performance of the activity when the user add new widgets and click on deploy button under deploy tab it takes some time to enable the Commit and push button
[ ]  As the deploy key is displayed at the first instance now when there is an error in the SSh key the error feels like overlapped on the Deploy key section

Others

[x] #8655

Commit and Push

[ ]  We need to indicate when a branch has unpush commits ( git status API dependency ) [Out of scope as commit and push is an atomic operation]
[ ] The Commit and push functionality is broken when the user deletes the pages and try to commit and push
@abhvsn

Steps To Reproduce
New feature

Environment
Deploy Preview
Version
Cloud

The Commit and push functionality is broken when the user deletes the pages and try to commit and push

@abhvsn  updated the issue here, can you reply here

[x]  When there is no write access given when a user added a deploy key add a Related toast error for it @AnaghHegde (when ssh key is added without write access when tried connecting a relevant error should show up) still pending #8655

This is not possible because the connect CTA requires read only permission.

Unable to reproduce this

When the user pastes the Wrong URL then paste the right one and click on generate SSH key the button doesn't work

Closing this issue as mentioned issues are either resolved or aren't valid anymore due to new design changes.


Allow this action to run on branch

Currently, this action only allows running on PRs. Is it possible to allow this action to run on a specified branch? How will it detect when there is no change in dependencies, yet the issue is introduced by the old version being used?

You can specify the base/head where the diff of those two will be used to create the list of dependencies to review.
https://github.com/actions/dependency-review-action/blob/0659a74c94536054bfa5aeb92241f70d680cc78e/action.yml#L25-L30

The base/head git ref to be used for this check. Has a default value when the workflow event is pull_request or pull_request_target. Must be provided otherwise.

Ex: https://github.com/ctcampbellcom/WebGoat/actions/runs/8667866545

      head-ref: ${{ github.ref }}
      base-ref: ${{ github.event.repository.default_branch }}

dependency-review Not in the context of a pull request. Skipping comment creation.

Thank you to @felickz for the very thorough answer on the topic of running on branches!
I'll take the other part of this issue:

How will it detect when there is no change in dependencies, yet the issue is introduced by the old version being used?

That's not really the use case that Dependency Review is intended to solve. It sounds like what you want is something like Dependabot alerts, but with support for multiple branches. Is that right?

You can specify the base/head where the diff of those two will be used to create the list of dependencies to review.
https://github.com/actions/dependency-review-action/blob/0659a74c94536054bfa5aeb92241f70d680cc78e/action.yml#L25-L30

The base/head git ref to be used for this check. Has a default value when the workflow event is pull_request or pull_request_target. Must be provided otherwise.

Ex: https://github.com/ctcampbellcom/WebGoat/actions/runs/8667866545

      head-ref: ${{ github.ref }}
      base-ref: ${{ github.event.repository.default_branch }}

dependency-review Not in the context of a pull request. Skipping comment creation.

Thank you @felickz. I will give it a try based on your recommendation and let you know.

@juxtin - I was looking for something similar to Veracode SCA (https://github.com/marketplace/actions/veracode-dependency-scanning) where I can run the dependency review on a branch or any commit.


Problem at run jumpcuter

my envoriment is
(jumpcutter) C:\jumpc>pip list
Package            Version

certifi            2024.2.2
charset-normalizer 3.3.2
decorator          4.4.2
idna               3.6
imageio            2.34.0
imageio-ffmpeg     0.4.9
jumpcutter         0.1.6
moviepy            1.0.3
numpy              1.26.4
pillow             10.2.0
pip                23.0.1
proglog            0.1.10
requests           2.31.0
setuptools         65.5.0
tqdm               4.60.0
urllib3            2.2.1
(jumpcutter) C:\jumpc>jumpcutter -i corte_transcricao_job1_parte2_part3_intervalo_short_legendado.mp4 -o jump1.mp4
Running with the arguments:
Namespace(input=WindowsPath('corte_transcricao_job1_parte2_part3_intervalo_short_legendado.mp4'), output=WindowsPath('jump1.mp4'), cut='silent', magnitude_threshold_ratio=0.02, duration_threshold=0.5, failure_tolerance_ratio=0.1, space_on_edges=0.1, silence_part_speed=None, min_loud_part_duration=-1, codec=None, bitrate=None)
Traceback (most recent call last):
File "C:\Program Files\Python310\lib\runpy.py", line 196, in _run_module_as_main
return run_code(code, main_globals, None,
File "C:\Program Files\Python310\lib\runpy.py", line 86, in run_code
exec(code, run_globals)
File "C:\jumpc\jumpcutter\Scripts\jumpcutter.exe_main.py", line 7, in
File "C:\jumpc\jumpcutter\lib\site-packages\jumpcutter_main.py", line 115, in main
clip = Clip(str(input_path), args.min_loud_part_duration, args.silence_part_speed)
File "C:\jumpc\jumpcutter\lib\site-packages\jumpcutter\clip.py", line 11, in init
self.audio = Audio(self.clip.audio)
File "C:\jumpc\jumpcutter\lib\site-packages\jumpcutter\clip.py", line 81, in init
self.signal = self.audio.to_soundarray()
File "", line 2, in to_soundarray
File "C:\jumpc\jumpcutter\lib\site-packages\moviepy\decorators.py", line 54, in requires_duration
return f(clip, *a, **k)
File "C:\jumpc\jumpcutter\lib\site-packages\moviepy\audio\AudioClip.py", line 113, in to_soundarray
return stacker(self.iter_chunks(fps=fps, quantize=quantize,
File "C:\jumpc\jumpcutter\lib\site-packages\numpy\core\shape_base.py", line 216, in _vhstack_dispatcher
return _arrays_for_stack_dispatcher(tup)
File "C:\jumpc\jumpcutter\lib\site-packages\numpy\core\shape_base.py", line 209, in _arrays_for_stack_dispatcher
raise TypeError('arrays to stack must be passed as a "sequence" type '
TypeError: arrays to stack must be passed as a "sequence" type such as list or tuple.
(jumpcutter) C:\jumpc>

Downgrading to Numpy 1.24.0 works for me
pip install numpy==1.24.0

reproduced and fixed in https://github.com/emkademy/jumpcutter/pull/31


Upgrade Postgres

Acceptance Criteria

Update Postgres for the most recent version (currently version 15)

Steps for upgading postgres locally

take current db backup

docker container ls
docker exec -it {container_name} /usr/bin/pg_dumpall -U admin > dumpfile

stop container
delete container
delete image
delete volumes
change postgres version from 12 to 15 in sources/.docker/postgres/Dockerfile file
Delete the role query at the top of the dumpfile by opening it in textpad(eg: CREATE ROLE admin;
ALTER ROLE admin WITH SUPERUSER INHERIT CREATEROLE CREATEDB LOGIN REPLICATION BYPASSRLS PASSWORD 'xxxxxxxxxxxxx’;)

make db
docker exec -i {container_name} psql -U admin -d aest < dumpfile
Upgading postgres in openshift analysis

Our postgres image for openshift is build using the BCGov patroni repository https://github.com/BCDevOps/platform-services.git
This repo contains the upgrade till postgres 12.5, the current version in out openshift dev and test are 12.5

So to upgrade our current database we may need additional libraries to be build in the patroni image, and deployment should happen.

Sample commands to deploy patroni

Prerequisite: You should have a role in your github to create Rolebindings access in openshift
Goto devops folder in our current repo in Terminal
make oc-build-patroni -p NAMESPACE=0c27fb-tools
make init-patroni -p NAMESPACE=0c27fb-dev
make oc-deploy-patroni -p NAMESPACE=0c27fb-dev


Bug: Clicking "load" on the design switcher without setting an ID will soft-lock the switcher.

Repro:
Launch an app, set the figma key.
Expand the design switcher, click "Change", then click "load" without setting a new ID. You won't be able to click Change any more.

Should also check that clicking "load" with an invalid ID doesn't break.


storage: add gzip option

Fixes #751
Fixes #752
This adds automatic gzipping to file uploads:
// With upload:
bucket.upload('./input.html', { gzip: true }, function(err, file) {
  // file was gzipped!
  // file.metadata.contentEncoding = 'gzip'
});

// Or with streams:
var file = bucket.file('input.html');
var fs = require('fs');

fs.createReadStream('./input.html')
  .pipe(file.createWriteStream({ gzip: true }))
  .on('complete', function(metadata) {
     // file was gzipped!
     // metadata.contentEncoding = 'gzip'
  });

Looks good to me!


Missing ServiceConnectorCreator

When I attempt to do something like the following:
@CloudScan
public class OAuthCloudConfig extends BaseOAuthConfig {

    @Autowired
    private SsoServiceInfo ssoServiceInfo;
}

I get a BeanNotFound exception, which seemed a bit weird.
Looking deeper, in the log file I see the following Unable to create service for p-identity during service scanning. Skiping.
This is because the spring-cloud-sso-connector project expects there to be a org.springframework.cloud.service.ServiceConnectorCreator class, but there is not for the SSO service.

Hi,
The reason why we don't create a ServiceConnector is because we inject all the configuration through https://github.com/pivotal-cf/spring-cloud-sso-connector/blob/master/src/main/java/io/pivotal/spring/cloud/SsoServiceCredentialsListener.java
You can use the CloudFactory to retrieve the SsoServiceInfo: https://github.com/pivotal-cf/spring-cloud-sso-connector#java-applications.
Thanks!

Yeah, I understand that's possible (and frankly it's probably a more sensible approach). I guess the question is why is it not the same as the other services? Have you found that the typical use-case for SsoServiceInfo is different from the rest (e.g. Mongo, MySQL, Rabbit, etc), such that you're better served with this approach?
I think this approach is more sensible WRT to non-CF services (e.g. all I have to do is add some environment variables and PRESTO! SSO is now configured), at least until we get the ability to specify labels and tags and the like on Cloud Foundry, but it's kinda annoying having two different ways to do things.

Yes,  the use case is different since the services you mentioned have a DataSource in common. For the SSO service we don't have only one entity that configures/describes Spring OAuth 2.0 like the DataSource.


Add contributing.md file

I would like to add this file because it is a crucial document in open-source repositories, providing clear guidelines and instructions for anyone interested in contributing to the project. It outlines how contributors can get involved, the contribution process, and the repository's coding standards and rules. Please assign this task to me.

It already exists, check in docs dir
you may look to enhance it

Okay, I will enhance the file.

The above mentioned file doesn't exist
I say me and amulya work on it


LUIS Authoring Release Version 3.0

This PR will replace the PR https://github.com/Azure/azure-sdk-for-js/pull/16720.
In this PR, I have regenerated the LUIS Authoring SDK.

With this regeneration, the swagger service version is updated from 3.0-preview -> 3.0. The same has been reflected in README.md table.
The version of the SDK is updated from 4.0.0-preview.3 -> 4.0.0-preview.4. There are breaking changes in this PR. But since this is a preview release, that should not be a problem. Note: I am not sure if the release is going to be a preview release. After discussion with @mariam606, I may update the version to 4.0.0.
I have used the following command to regenerate the SDK:

autorest --typescript --license-header=MICROSOFT_MIT_NO_VERSION --typescript-sdks-folder=C:\Users\sarajama\Projects\azure-sdk-for-js --use=<EMAIL_ADDRESS>--package-version=4.0.0-preview.4 --title=LUISAuthoringClient https://raw.githubusercontent.com/Azure/azure-rest-api-specs/350cb8b8ca52d093010395c52130a5723abf717e/specification/cognitiveservices/data-plane/LUIS/Authoring/readme.md --tag=authoring_3_0

Note: The above command uses a swagger that is still in PR stage and has not been merged (That is the reason for Do Not Merge label on this PR). Once the swagger PR is merged, I need to regenerate the SDK again and update this PR. At this point, this PR is to give an overall idea of the upcoming changes.
FYI - @ramya-rao-a @mariam606 @kayousef

There are tests for this package that would need to be run manually by the LUIS team and make any changes if needed.
It is better if the original PR https://github.com/Azure/azure-sdk-for-js/pull/16720 is used for those purposes.
Please consider closing this and providing this PR as a pointer to the folks at https://github.com/Azure/azure-sdk-for-js/pull/16720 as to how to re-generate the package

Closing this PR. This will be used as reference only


Update Faraday dependency to < 0.14

Fixes #315
Latest version of faraday is 0.13.1

Coverage remained the same at 96.0% when pulling ba65f8216774b2877881f4feba8a6feecba5cfec on zacharywelch:zw/update-faraday-0.13.1 into 452a0853286fc0d2919037598ac891dd3e7b1d19 on intridea:master.

+1 for this. Looks like failures are related to Rubocop, which seems odd since you didn't change any code. Is that a known issue?

+1 for this. Anything I can do to help get it done?

@zacharywelch can you run rubocop -a to fix the rubocop violations?  I don't want to break the build on master.

Hi @pboling, the build still fails even after autocorrect because of rubocop differences. Are you ok w/ the rubocop fixes in #322?

@zacharywelch thanks for the heads up on that PR.  Merged it.  We are now good here.


[OSDEV-1120] Correct formatting of downloaded excel file of moderation events

OSDEV-1120 - Moderation. New Moderation Queue Page that copied from Dashboard / Facility Claims (FE).
In this PR was correct formatting of the downloaded excel file of moderation events.

@mazursasha1990 could we merge it?


How can I make a strike_popover open onClick instead of onHover?

It's cool to have popovers that open on mouseover and close on mouseout. But for those that can only be closed by clicking X there should be an option to open them on click. Otherwise the user moves over the screen and multiple popovers open and overlap without ever closing.
Could you add an option or does this already exist?

I am having a similar situation here. Is there an option to display the popover dialog onClick? I just want to display the popover on demand, when the users clicks on it.

I solved the issue on my own :) using pure css.
myComponent.cmp

            <c:strike_popover >
            </c:strike_popover>

myComponent.css
    .THIS #container {
       width: 20px;
       height: 20px;
       position: relative;
    }

    .THIS #curtain {
       width: 100%;
       height: 100%;
       position: absolute;
       top: 0;
       left: 0;
       z-index: 10;
    }

myComponentController.js
   open: function(cmp, evt, helper) {
        var curtain = document.getElementById("curtain")
        curtain.style.zIndex = 0;

        setTimeout(function() {
            curtain.style.zIndex = 10;
        },100);
    },

Love this solution. Thanks for sharing.


[e2e]: docker-based workflow_dispatch main default slsa3

Repo: https://github.com/slsa-framework/example-package/tree/main
Run: https://github.com/slsa-framework/example-package/actions/runs/4663506593
Workflow file: https://github.com/slsa-framework/example-package/tree/main/.github/workflows/e2e.docker-based.workflow_dispatch.main.default.slsa3.yml
Workflow runs: https://github.com/slsa-framework/example-package/actions/workflows/e2e.docker-based.workflow_dispatch.main.default.slsa3.yml
Trigger: workflow_dispatch
Branch: main
Date: Tue Apr 11 03:20:15 UTC 2023

Repo: https://github.com/slsa-framework/example-package/tree/main
Run: https://github.com/slsa-framework/example-package/actions/runs/4664654855
Workflow file: https://github.com/slsa-framework/example-package/tree/main/.github/workflows/e2e.docker-based.workflow_dispatch.main.default.slsa3.yml
Workflow runs: https://github.com/slsa-framework/example-package/actions/workflows/e2e.docker-based.workflow_dispatch.main.default.slsa3.yml
Trigger: workflow_dispatch
Branch: main
Date: Tue Apr 11 06:49:59 UTC 2023
Tests are passing now. Closing this issue.


Add better sample

The sample should show the features of the library in a way that's simple and easy to understand.

Better samples have been added.


[Concurrency] Add support for 'async' closures.

Closurea can become 'async' in one of two ways:

They can be explicitly marked 'async' prior to the 'in'
They can be inferred as 'async' if they include 'await' in the body

@swift-ci smoke test
https://github.com/apple/swift-syntax/pull/235

https://github.com/apple/swift-syntax/pull/235
@swift-ci smoke test


Update Gateway API to use beta instead of alpha resource versions.

Is your feature request related to a problem? Please describe.

While the Gateway API was updated to use version 0.5 (#5376), cert-manager still targets the experimental (v1alpha2) resources instead of the new stable (v1beta1) versions.
Describe the solution you'd like

Update the controller to target the current version instead of the previous version.
Environment details (remove if not applicable):

cert-manager version: 1.10.0

/kind feature

We just briefly chatted about this and @irbekrm wasn't sure if there was a reason we might have waited with changing to use v1beta1.
I'm not super familiar with this part of the codebase but this doesn't seem like a huge amount of work to do? Tagging as a good first issue, and I think it sounds like we should ideally work to upgrade to v1beta1!


fix(VariablePlugin): call editor.moveToEndOfNode if extending a variable

Issue #179
fix(VariablePlugin): if extending a variable, call editor.moveToEndOfNode(prev) where prev is the variable node so that the editor inserts the text inside of the variable node.
Flags

After updating the markdown-editor dependency, a test for rendering links correctly is failing. I'm skipping the test for now, but I think all of the formatting tests in that file should be moved to markdown-editor since they are testing that module's logic

Related Issues

Pull Request https://github.com/accordproject/markdown-editor/pull/203

After updating the markdown-editor dependency, a test for rendering links correctly is failing. I'm skipping the test for now, but I think all of the formatting tests in that file should be moved to markdown-editor since they are testing that module's logic

I agree, I shouldn't have approved these to be put in this repo in the first place.


python312Packages.aiogram: 3.13.0 -> 3.13.1

Diff: https://github.com/aiogram/aiogram/compare/refs/tags/v3.13.0...v3.13.1
Changelog: https://github.com/aiogram/aiogram/releases/tag/v3.13.1
Description of changes

Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Result of nixpkgs-review pr 344046 run on x86_64-linux 1

4 packages built:

python311Packages.aiogram
python311Packages.aiogram.dist
python312Packages.aiogram
python312Packages.aiogram.dist


Merge V0.5 milestone

Coverage increased (+0.5%) to 79.725% when pulling 180dd1608c27ee8694b65b514b585a6137cd662b on V0.5 into 095e74ed4afdc0a790449c35e22c3d063708f134 on master.


Development

Coverage remained the same at 100.0% when pulling da427f9ed319069be066f5668dab244db5acafd3 on development into 4c079355619bfbbe0730292650ffd449ab994102 on master.

Coverage remained the same at 100.0% when pulling 42ca3b2a66a2be48883cf1a5368b2153080963ea on development into 4c079355619bfbbe0730292650ffd449ab994102 on master.


Dialogue Preview doesn't show correct outcome of an option in some cases

Dialogue Preview, when set to show you what cues, etudes or scenes will be triggered by a dialogue option, can sometimes show the wrong one even when you do meet the requirements for an alternate cue to be played.
The most noticeable instance I've had to help friends debug  is with Daeran's final companion quest when you have to decide Daeran's fate when it comes to an inquisition tribunal , where it will show a dialogue option as being on the bad or happy route even when actually selecting it will put you on the good or lobotomy route.
Specifically, you can check this with the blueprint Answer_0018.7f610161f30f6b74aaa0967a8d8e27e9.
The good or lobotomy route is in cue_0032.219b6fb6e2eacdb4895a4a20ee68a0d1, the bad or happy route is in cue_0030.b81f18abf6968ac47bb86d727f57c9c3. However even when Daeran's trust counter is above the requirement to be on the former, the latter and the events it sets up will be what's previewed to the player.

I am working on a new Etude Editor and am probably going to improve dialog preview as part of this effort.


Update README.md

Hello,
I have updated the readme with some information about this plugin that is relevant to iOS developers and users.
More information about the rejection of apps using private api's on ios:
https://forum.ionicframework.com/t/ios-application-rejected-with-reason-non-public-apis/46717

This was removed in version v2.0.1. See fc3483e028a4b5ac325257a626130aa1e979e5a5


[skwasm] Platform view has incorrectly positioned when used in a widget with ClipBehavior not set to none in a listview

Steps to reproduce

Wrap video player in ClipRRect and ListView
Compile with --wasm

Expected results
Video should be correctly positioned in the black box
Actual results
Video gets clipped incorrect.
Code sample
Code sample
import 'package:flutter/material.dart';
import 'package:video_player/video_player.dart';

void main() {
  runApp(const MyApp());
}

class MyApp extends StatelessWidget {
  const MyApp({super.key});

  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      title: 'Video Player App',
      theme: ThemeData(
        primarySwatch: Colors.blue,
      ),
      home: const MyHomePage(),
    );
  }
}

class MyHomePage extends StatefulWidget {
  const MyHomePage({super.key});

  @override
  State<MyHomePage> createState() => _MyHomePageState();
}

class _MyHomePageState extends State<MyHomePage> {
  late VideoPlayerController _controller;

  @override
  void initState() {
    super.initState();
    _controller = VideoPlayerController.networkUrl(
      Uri.parse(
        'https://flutter.github.io/assets-for-api-docs/assets/videos/butterfly.mp4',
      ),
    )..initialize().then((_) {
        setState(() {});
      });
  }

  @override
  Widget build(BuildContext context) {
    return Scaffold(
      body: ListView(
        children: [
          const SizedBox(height: 200),
          _controller.value.isInitialized
              ? Container(
                  clipBehavior: Clip.hardEdge,
                  decoration: BoxDecoration(
                    border: Border.all(color: Colors.black),
                    borderRadius: BorderRadius.circular(16),
                  ),
                  child: AspectRatio(
                    aspectRatio: _controller.value.aspectRatio,
                    child: VideoPlayer(_controller),
                  ),
                )
              : Container(),
          const SizedBox(height: 20),
          for (int i = 1; i <= 20; i++)
            ListTile(
              title: Text('Dummy Entry $i'),
            ),
        ],
      ),
    );
  }

  @override
  void dispose() {
    super.dispose();
    _controller.dispose();
  }
}

Screenshots or Video

Screenshots / Video demonstration
https://github.com/user-attachments/assets/6f07cfea-cc4d-4f69-9da3-cd3b7e436b5f

Logs
Logs
[Paste your logs here]

Flutter Doctor output
Doctor output
Doctor summary (to see all details, run flutter doctor -v):
[✓] Flutter (Channel master, 3.27.0-1.0.pre.214, on macOS 14.6.1 23G93 darwin-arm64, locale en-DE)
[✓] Android toolchain - develop for Android devices (Android SDK version 34.0.0)
[!] Xcode - develop for iOS and macOS (Xcode 16.0)
    ✗ CocoaPods installed but not working.
        You appear to have CocoaPods installed but it is not working.
        This can happen if the version of Ruby that CocoaPods was installed with is different from the one
        being used to invoke it.
        This can usually be fixed by re-installing CocoaPods.
      For re-installation instructions, see
      https://guides.cocoapods.org/using/getting-started.html#installation
[✓] Chrome - develop for the web
[✓] Android Studio (version 2023.3)
[✓] Android Studio (version 2021.3)
[✓] Android Studio (version 2022.2)
[✓] IntelliJ IDEA Community Edition (version 2024.2.3)
[✓] VS Code (version 1.94.2)
[✓] Connected device (3 available)
    ! Error: Browsing on the local area network for Heißer Spot. Ensure the device is unlocked and attached
      with a cable or associated with the same local area network as this Mac.
      The device must be opted into Developer Mode to connect wirelessly. (code -27)
[✓] Network resources

! Doctor found issues in 1 category.

Reproducible using the code sample provided above. In the code sample shared above, if the clipBehavior of the Container is set to none, it works as expected. Any other clip behavior causes it to be misplaced.
This seems like a different manifestation of https://github.com/flutter/flutter/issues/155417. I can confirm that #155417 still works on the master channel.
Labeling for further investigation.

flutter doctor -v
[✓] Flutter (Channel stable, 3.24.4, on macOS 15.0.1 24A348 darwin-arm64, locale en-US)
    • Flutter version 3.24.4 on channel stable at /Users/deanli/dev/stable
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 603104015d (4 days ago), 2024-10-24 08:01:25 -0700
    • Engine revision db49896cf2
    • Dart version 3.5.4
    • DevTools version 2.37.3

[!] Flutter (Channel master, 3.27.0-1.0.pre.259, on macOS 15.0.1 24A348 darwin-arm64, locale en-US)
    • Flutter version 3.27.0-1.0.pre.259 on channel master at /Users/deanli/dev/master
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 5879a5cba1 (6 hours ago), 2024-10-27 18:09:20 -0700
    • Engine revision 10e50bd42c
    • Dart version 3.7.0 (build 3.7.0-71.0.dev)
    • DevTools version 2.40.1


STN  example not working

Traceback (most recent call last):
File "train.py", line 16, in
model = STN()
File "/home/vision/STN/STN.py", line 29, in STN
x = Conv2D(32, (3, 3), padding='same')(interpolated_image)
File "/home/vision/.virtualenvs/dl4cv/lib/python3.5/site-packages/tensorflow_core/python/keras/engine/base_layer.py", line 812, in call
self.name)
File "/home/vision/.virtualenvs/dl4cv/lib/python3.5/site-packages/tensorflow_core/python/keras/engine/input_spec.py", line 155, in assert_input_compatibility
' input tensors. Inputs received: ' + str(inputs))
ValueError: Layer conv2d_2 expects 1 inputs, but it received 4 input tensors. Inputs received: [<tf.Tensor 'bilinear_interpolation/Placeholder:0' shape= dtype=float32>, <tf.Tensor 'bilinear_interpolation/Placeholder_1:0' shape=(30,) dtype=float32>, <tf.Tensor 'bilinear_interpolation/Placeholder_2:0' shape=(30,) dtype=float32>, <tf.Tensor 'bilinear_interpolation/Placeholder_3:0' shape=(1,) dtype=float32>]

Same problem :/ cant find the problem

same problem.


Check Pica3 representation of table numbers

Fixing the DDC notation pattern caused the Pica3 representation to change. For our monitoring example, it changed from
5420 [23]700.90440747471-G--7-T1--09044-T1--074-T2--7471$Acoli-ana

to
5420 [23]700.90440747471-G--7-T1--09044-T1--0901-0905%3A074-T2--7471$Acoli-ana

The included numbers changed slightly (even though the actual result on the web UI did not change), and it now includes a table number with : which is encoded here as %3A.
Note that I will change the monitoring for now so that it won't bug us about this. I might change it back as soon as this is fixed.

I think it's an encoding issue with the : in the notation. When comparing the JSON:
❯ diff 0.4.1.json 0.5.1.json
183c183
<         "uri": "http://dewey.info/class/1--0901-0905:07/e23/",
---
>         "uri": "http://dewey.info/class/1--0901-0905%3A07/e23/",
185c185
<           "T1--0901-0905:07",
---
>           "T1--0901-0905%3A07",
193c193
<         "uri": "http://dewey.info/class/1--0901-0905:074/e23/",
---
>         "uri": "http://dewey.info/class/1--0901-0905%3A074/e23/",
195c195
<           "T1--0901-0905:074",
---
>           "T1--0901-0905%3A074",
204c204
<             "uri": "http://dewey.info/class/1--0901-0905:07/e23/"
---
>             "uri": "http://dewey.info/class/1--0901-0905%3A07/e23/"

I will look into it.

This is a bug in jskos-tools: https://github.com/gbv/jskos-tools/issues/41
However, it opened up discussion on how to handle these kinds of table range notations in general: https://github.com/gbv/jskos-data/issues/47


Adding support for InfluxDB

Adding support for InfluxDB

@all-contributors please add @TFyre for adding code to support InfluxDB


$.browser is undefined

I'm getting this error:
TypeError: $.browser is undefined
Line 300

Hashchange 1.3
jQuery 2.1.0
Firefox 27 (this won't matter, but just for completeness)
I think the error speaks for itself.

so what we can do??

Just don't use this plugin and use the native hashchange event. With many of our projects supporting no MSIE at all, the need for IE-specific code has gone down the toilet. Microsoft has deemed MSIE deprecated as well, in favour of Edge. And so should you.

Remove use of $.browser (removed in jQuery 1.9) and use regex
it work 👍

Just don't use this plugin and use the native hashchange event.

This should be the first line of the README.


[improve][build] Remove versions that are handled by netty-bom

Motivation
netty-bom includes the correct Netty tcnative version which is compatible with a particular Netty version. Because of this reason, it's an unnecessary overhead to duplicate the information in Pulsar pom.xml .
Modifications

remove netty-tc-native.version from Pulsar's pom.xml
remove unnecessary dependency management dependencies that are included in netty-bom
remove unnecessary definition of version for a netty dependency

Documentation

[ ] doc
[ ] doc-required
[x] doc-not-needed
[ ] doc-complete

Matching PR in forked repository
PR in forked repository: https://github.com/lhotari/pulsar/pull/107

/pulsarbot run-failure-checks

Codecov Report

Merging #18629 (49292b4) into master (cd85a67) will decrease coverage by 17.02%.
The diff coverage is 16.66%.

@@              Coverage Diff              @@
##             master   #18629       +/-   ##
=============================================
- Coverage     47.39%   30.36%   -17.03%
+ Complexity    10479     5783     -4696
=============================================
  Files           698      618       -80
  Lines         68070    58577     -9493
  Branches       7279     6094     -1185
=============================================
- Hits          32264    17789    -14475
- Misses        32228    38302     +6074
+ Partials       3578     2486     -1092

Flag
Coverage Δ

unittests
30.36% <16.66%> (-17.03%)
:arrow_down:

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