THRIFT-3000 .NET implementation has trouble with mixed IP modes

Client: C#
Patch: sharpdevel icode666@github & Jens Geyer
TcpListener and TcpClient are created based on the capabilities of the used runtime framework. For windows the changes automatically handle IPv4 and IPv6 sockets. In mono it behaves as before.
When using TcpListener and TcpClient it depends on the network configuration if IPv4 or IPv6 is used. By upgrading the framework to .NET 4.5 the DualMode can be set on the sockets of the listener and the client. The sockets then try to establish IPv6 sockets before they fallback to IPv4

Did this complete a CI build?  Someone suggested in analysis of my pull request to THRIFT-2850 that this changeset is the root cause.

Sure it did:
https://builds.apache.org/job/Thrift-Compiler-Windows/
https://builds.apache.org/job/Thrift/
Travis is only an indicator with unfortunalety some limitations. That's not to say that Travis is not a good tool, but it is not the sacred last word either. Anyway, I'm going to revert that one. Seems we need some more work here.

@Jens-G,  can you take a look at #459 before reverting ?
As to the Travis-CI failure, I'm aware that "true" CI is the Jenkins but it would be nice if Travis failures get more attention, because it is easily accessible to all contributors, not only to maintainers.
To name a few key differences:

I can run builds and tests on my own Travis-CI account and verify PRs before submitting, not for Jenkins
I can suggest working Travis-CI test changes though PRs but don't know any way to do it for Apache Jenkins servers

IMO, @jeking3's way of being very serious about Travis failure is good way to reduce  not only occasional regressions but also maintainers' work.
What is good about Jenkins setups ?

IMO, @jeking3's way of being very serious about Travis failure is good way to reduce not only occasional regressions but also maintainers' work.

Absolutely.

What is good about Jenkins setups ?

That's where the real stuff happens ;-)

How about having unified base for testing ?
Can we have build scripts inside the repository and invoke them from both Jenkins and Travis/Appveyer ?

They both call the usual suspects: bootstrapsh && configure && make &&... or cmake. The environment  makes the difference.

No, Jenkins not calling cross tests made the difference.
In fact, the log shows that Jenkins doesn't even build following languages at all (Building : no):

C (GLib)
Ruby
Haxe
Haskell
Erlang
Go
D
NodeJS

I couldn't find any evidence of make check being run either.
I have to say it does not do the job well.
The real problem is that Jenkins on Apache servers cannot evolve as fast as the code base because of its closed nature.

Von: Aki Sukegawa
Gesendet: 25.04.2015 06:13
An: apache/thrift
Cc: Jens Geyer
Betreff: Re: [thrift] THRIFT-3000 .NET implementation has trouble with mixed IP modes (#450)
No, Jenkins not calling cross tests made the difference.
In fact, the log shows that Jenkins doesn't even build following languages at all (Building : no):

C (GLib)
Ruby
Haxe
Haskell
Erlang
Go
D
NodeJS

I couldn't find any evidence of make check being run either.
I have to say it does not do the job well.
The real problem is that Jenkins on Apache servers cannot evolve as fast as the code base because of its closed nature.

Reply to this email directly or view it on GitHub:
https://github.com/apache/thrift/pull/450#issuecomment-96126819

we need to transition our jenkins build over to using the docker containers so we have more control over them and then can run whatever we like inside that. on my list for pre 1.0


Snapshot on multipe devices only saving last screenshot

Expected behaviour
I am trying to run a tablet emulator and phone emulator. I expect two screenshots in my /app/screenshots/debug folder (1 for tablet and 1 for phone)
Actual behaviour
Only getting one screenshot, but the test is executing on both simulators. Seems like the last finishing emulator is the only screenshot thats saved.
Steps to reproduce
./emulator -avd Pixel_C_API_30
./emulator -avd PixelXL_Emulator
./gradlew executeScreenshotTests -Precord
Version of the library
com.karumi:shot:5.14.1
Java 11

Shot runs the screenshot tests on all the devices adb finds.
However, Shot also supports to specify the folder where the screenshots are saved.
Moreover, using Android Serial we can decide on which emulators/devices the commands will be executed.
If we put all together, you can get it working as expected on a Mac as described in this tweet.
https://twitter.com/gio_sastre/status/1484201050450055169?s=46&t=0OpCcA9bfgI0DJA__8tsUQ
@pedrovgs Maybe makes sense to add to the README how to get Shot working on multiple devices? I can gladly add a PR for that


DelistingEventProvider always adds a day to delisting time

Expected Behavior

Some logic on when to shift the date and not too, possibly if there is a time component don't shift. If there is not do shift.
But really we should define exactly when these events should be emitted so we can control that.
Actual Behavior

Currently our delisting provider always adds a day to the delisting DateTime it receives before emitting the event.
This is troublesome because some delistingDates do have time components and others do not. It also raises the question of when do we want the delisting to be processed and emitted.
It appears the initial implementation was to move options (which in backtesting with GOOG options I see has 12AM delisting time) delisting to end of day. But I also noticed with Futures like ES where we have a time value (ex 1:30PM) adding a day shifts it to the next day entirely.
                if (!_delisted && eventArgs.Date > DelistingDate.Value)
                {
                    _delisted = true;
                    var price = eventArgs.LastBaseData?.Price ?? 0;
                    // delisted at EOD
                    yield return new Delisting(
                        eventArgs.Symbol,
                        DelistingDate.Value.AddDays(1),
                        price,
                        DelistingType.Delisted);
                }

Potential Solution

WIP
Reproducing the Problem

Process a delisting using one of our delisting regressions and watch the DelistingProvider create the Delisting data
Checklist

[x] I have completely filled out this template
[x] I have confirmed that this issue exists on the current master branch
[x] I have confirmed that this is not a duplicate issue by searching issues

[x] I have provided detailed steps to reproduce the issue

Note on this issue, because of this behavior contracts that are expired are still subscribed till EOD.

Closing for now, no impact detected


Consider removing NamingPolicy

I'd like to consider removing NamingPolicy from Rpm: now it's possible to create Rpm instance with sha-256 digest and NamingPolicy with sha-1 digest, which does not make any sense to me. Also, now we are calculating hexes for metadata files twice in MetadataFile#save, that is inefficient.
The other option can be to remove digest dependency from NamingPolicy: it would either be plain (without hex prefix) or prefixed.

@g4s8/z please, pay attention to this issue

@olenagerasimova why it's not possible to create Rpm with SHA256 or SHA1? As I know our clients are using SHA1 file prefixes with SHA256 checksums in repomd.xml. Also, it's like a standard to save rpm metadata files with SHA256 checksums.

@g4s8 ok, closing it then


crates.io FTBS

The version on crates.io fails to build from source.  https://github.com/helixbass/tree-sitter-grep/pull/63 fixes this.
> cargo install tree-sitter-grep
...
error[E0308]: mismatched types
  --> /home/jayvdb/.cargo/registry/src/index.crates.io-6f17d22bba15001f/tree-sitter-grep-0.1.0/src/language.rs:77:16
   |
77 |         Css => tree_sitter_css::language(),
   |                ^^^^^^^^^^^^^^^^^^^^^^^^^^^ expected `tree_sitter::Language`, found a different `tree_sitter::Language`
   |
   = note: `tree_sitter::Language` and `tree_sitter::Language` have similar names, but are actually distinct types
note: `tree_sitter::Language` is defined in crate `tree_sitter`
  --> /home/jayvdb/.cargo/registry/src/index.crates.io-6f17d22bba15001f/tree-sitter-0.22.6/binding_rust/lib.rs:54:1
   |
54 | pub struct Language(*const ffi::TSLanguage);
   | ^^^^^^^^^^^^^^^^^^^
note: `tree_sitter::Language` is defined in crate `tree_sitter`
  --> /home/jayvdb/.cargo/registry/src/index.crates.io-6f17d22bba15001f/tree-sitter-0.20.10/binding_rust/lib.rs:43:1
   |
43 | pub struct Language(*const ffi::TSLanguage);
   | ^^^^^^^^^^^^^^^^^^^
   = note: perhaps two different versions of crate `tree_sitter` are being used?

Ok thanks. Yes I can likely go ahead and merge #63 and release another version of tree-sitter-grep to crates.io when I pull back up the tree-sitter-lint/tree-sitter-grep stuff

You mind want to checkin a Cargo.lock as part of the next release, as cargo install --locked uses that if it is available, which can help keep a crates.io release working for longer

Reading through https://github.com/helixbass/tree-sitter-grep/pull/63 , it looks like it isnt ready yet.
I could try to get together a simpler PR that only fixes the dependency pins so that they all use the same tree-sitter.

Ok ya I've basically been adding stuff needed for tree-sitter-lint in #63 (eg a library-level crate API) but no idea if it's in a "coherent" state (from the perspective of tree-sitter-grep) or not
Sure it sounds like you've diagnosed the problem with trying to build the currently-released tree-sitter-grep version so ya feel free to open a PR that just remedies that and I'd be happy to merge and release a new version to crates.io

The problem appears to be that the Cargo.toml here uses
https://github.com/helixbass/tree-sitter-rust/commit/781a8d9 for tree-sitter-rust , which is an a commit in the series https://github.com/helixbass/tree-sitter-rust/commits/macro-invocation-contents/
If I instead use tree-sitter-rust = "0.20.3" , the build works.
But you were using a fork for a reason, and I dont see any upstream issues/PRs by you at https://github.com/tree-sitter/tree-sitter-rust/issues?q=author%3Ahelixbass OR  https://github.com/tree-sitter/tree-sitter-rust/pulls?q=author%3Ahelixbass
I see https://github.com/helixbass/tree-sitter-rust/commit/018531fcc09d43ae48a3d87b2d2b34023f8e994f changed the grammar, and test test_macro_contents depends on this feature.
Still, it feels like it was a "nice to have", but not worth using a git dependency, and ultimately causing a FTBS now, and will be a regular maintenance problem until it is merged upstream.
So, I have created https://github.com/helixbass/tree-sitter-grep/pull/71

And now in #71 Css and Elisp languages fail to build sigh


more complex passwords

change things so that the password for user1-userX are no longer just "openshift" as it always displays the scary messages.
consider same for minio/minio123

Should be done here: https://github.com/rhpds/agnosticv/blob/4422f5ea396e356cd3f3492b00f3843cbf4e49f4/sandboxes-gpte/openshift-ai-unleashed/common.yaml#L64
Will do it once the new catalog item is created


Watch for changes to assets

I was taking a look at #3873 and noticed that changes to assets were not reflected in the review app unless I manually ran npm run build:package.
Watch the govuk/assets folder and re-run the assets task when there are changes.
This means that changes to the assets folder are applied to the dist folder (and therefore are reflected in the review app) without having to manually run the build:package task.

Thanks @36degrees this is brilliant 🎉
For context, we spotted a list of things we didn't watch in this PR:

https://github.com/alphagov/govuk-frontend/pull/3498#issuecomment-1535033842

But last week we added Gulp .watch() for component data so this PR might complete that list?


update mixin, multiple additions

adding first
adding reduceRight
adding some
adding every
adding findIndex
adding size
adding Object.assign
adding now
adding templates

@AutoSponge Looks good. Can you merge from master or rebase so this can be merged. Thanks!

Please squash everything into one commit.


Best Efforts query needs to be optimized

Following on the success of https://github.com/SplitTime/OpenSplitTime/issues/264, we should optimize the Best Efforts query as well.

Cumulative effects of best efforts optimization can be seen here:


Upgrade core packages to 1.7.0

Changes
Upgrade to version 1.7.0 of:

OpenTelemetry
OpenTelemetry.Exporter.OpenTelemetryProtocol
OpenTelemetry.Extensions.Hosting
OpenTelemetry.Instrumentation.AspNetCore
OpenTelemetry.Instrumentation.Http

Merge requirement checklist

[ ] Unit tests added/updated
[x] CHANGELOG.md file updated for non-trivial changes
[ ] Changes in public API reviewed (if applicable)

With these changes, we can remove setting OTEL_SEMCONV_STABILITY_OPT_IN here: https://github.com/grafana/grafana-opentelemetry-dotnet/blob/main/src/Grafana.OpenTelemetry.Base/TracerProviderBuilderExtensions.cs#L34
Instead we should set OTEL_DOTNET_EXPERIMENTAL_ASPNETCORE_ENABLE_GRPC_INSTRUMENTATION now, to avoid breaking users who are using a gRPC server: https://github.com/open-telemetry/opentelemetry-dotnet/pull/5130/files#diff-eef1a6fbb9155013c4a21ba1f3da8ade5f0ac7e08d23bd761575b4ecd70ac86aR28


Instrumentation tests

Tried to implement many tests. Only intrumentation test at the moment though.

fix #27


refactor(landingpage): tweak footer design

Closes #998
Currently working with no design to abide to. More traditional variant of this is in #1053
Preview:
https://github.com/inovex/elements/assets/81302108/37b35103-e97a-44b8-bcfc-d5b30b7bb4ac

Change wording:

Getting Started -> Guides
Erste Schritte -> Guides


Start navigating in an existing browser?

I would like to use chromeless to sign into my account on a website, but there is a recaptcha on the login page.
Is it possible to tell chromeless to use an already opened browser instead of starting a new one (so that I could manually solve the recaptcha and then only launch the script)?

Hey, any update on this? Can I ask chromeless to start navigating in an existing browser, or how to deal with recaptcha in general?


Add localization support via Mattermost Translation Server

Add localization support via Mattermost Translation Server.
The new mobile apps have localization support (although it'll only be in English/Spanish for the beta launch).
We should explore options to support localization of the desktop app, which would consist of all app-specific strings like the menu bar, Settings page, etc.
Best way is to work with Elias from Mattermost team and get it hooked up to the Translation Server that already has a vibrant localization community.

I can help, if needed for french translation.

That would be great @LAfricain!
We have existing translations for the webapp and mobile hosted here, if you're interested helping: https://translate.mattermost.com/projects/mattermost/
The desktop app translations will most likely be handled there too

The desktop app translations will most likely be handled there too
I am especially interested in the desktop version...

Haha got it :) Will let you know when those are added. Thanks again for offering your help

Hello there. I'm the guy in charge of the French translation for almost 2 years now, I I have just noticed this. Since the French translation is complete for the server, webapp and mobile apps, I definitely think this is is feature needed. When the webapp is set to french and we need to use the desktop app, this looks as if the product was still a draft and not polished enough.
What do you require to make this happen? Parsing things etc? and playing with the Pootle API? This is not a problem for me if you need work force. cc @enahum
Link to the discussion on the chat: https://pre-release.mattermost.com/core/pl/m6kuy95sfjfp9jcre3trsj4zxr

Moved this to a Story Jira ticket: https://mattermost.atlassian.net/browse/MM-14058.


React to moving web.config

We need to update the IISExpress deployer to not  after moving web.config to the application root so that our sample apps can be tested.
cc @muratg @Eilon @DamianEdwards @Tratcher

:shipit:


js-is-stuck does not seem to be added when sticky is supported

I'm using the latest version of chrome which supports position: sticky.  Unfortunately I'm noticing that js-is-stuck does not seem to be added.  Is this a known issue?
I'm defining my sticky element with jquery
$('#element').stickybits({useStickyClasses: true});

The element is sticking correctly but the isStuck just does not work.

Digging in, I think the issue has something to do with "stickyBitStart".  Essentially that is smaller then it should be and as a result the js-is-sticky class gets added before the element is actually sticky.

Dang—no fun!!! This probably happened when I fixing another issue. Ma bad. I'm on it!!!

Interesting.  So I haven't replicated it yet in code pen because it requires a more advanced use case I think.  Basically it seems like the calculation of stickyBitStart changes depending on your scroll position on the page.  I put a console.log underneath the line
var stickyBitStart = this.el.getBoundingClientRect().top;

Then I refreshed the browser a couple of time with different scroll positions.  Depending on my scroll position when I refresh, this value changes significantly.  It is only correct when I'm scrolled to the very top of the page.  Because this value is wrong, the classes are added at the wrong times (before or after the element becomes sticky)
I fixed this by using the jquery method offset().top "offset().top".  This strategy produces a consistent number thats not based on where you are scrolled on the page.  You also need to take into account the offset thats passed in.  So this line became
var stickyBitStart = $(this.el).offset().top - (offset || 0)

And the last problem seems to be when developing responsive sites.  If you change the width of the browser, the start and stop positions of these elements need to change too.  As a result I think the calculations of stickyBitStart and stickyBitStop need to be dynamic.  So in other words there will likely need to be a method to generate these numbers on the fly rather than set them as constant variables.
Thoughts?

My initial thought is:

Can you make a  codepen to show the issue?
Once I can see the issue too, can we make a codepen together and fix the issue?

If you can make a codepen showing the issue, then I will work with you to make it work how we want.
I think/hope with the stickbits .cleanup method, we can get your issue working perfectly. 👌

@ishields, thanks for you notes. I'm going to cancel this issue.
Reach out if you have anymore notes/questions/ideas. 🙏

Hi @yowainwright, it still doesn't apply the js-is-stuck class in Firefox. I installed stickybits 1.3.5

@wernerkrauss 😔 Do you have link you can share? ...Or is the issue visible in demos? 🙏

@wernerkrauss I'm not seeing that issue here...?
https://codepen.io/yowainwright/pen/36449183ca18af18f3005734b0251769/


Cannot read STL file

When dragging and dropping or opening an STL file, PVG reports "Specified file is not currently supported" and then brings up a raw image reader dialog that reports "Effective size does not match total size!"
A sample file is at:
https://gitlab.kitware.com/cmb/smtk/blob/master/data/model/3d/stl/kettle.stl
This bug was reported by
Haocheng Liu

vtk.js only support binary stl files. We never get a chance to implement the text version of that format.

vtk.js 7.0.1 bring that support


[Bug]: A link in "Reverse image search" document leads to a 404 error

Is there an existing issue for this?

[X] I have searched the existing issues

Current Behavior

Go to https://docs.towhee.io/tutorials/reverse-image-search/
Scroll down to bottom
Click the link "Please visit this page a full list of image embedding pipelines Towhee supports."

Then got a 404 page

Expected Behavior
A right page should be displayed
Steps To Reproduce
As above

Environment
- Towhee version(e.g. v0.1.3 or 8b23a93):
- OS(Ubuntu or CentOS):
- CPU/Memory:
- GPU:
- Others:

Anything else?
No response

I think it is https://docs.towhee.io/tutorials/reverse-image-search, where do you find the link?

I think it is https://docs.towhee.io/tutorials/reverse-image-search, where do you find the link?

Yes, updated the description

Thanks @timshen, and updated it with #103.


⚠ Bofrost.de has degraded performance

In 74e7328, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 1050 ms

Resolved: Bofrost.de performance has improved in 05e42a0 after 13 minutes.


PUBDEV-6180: fixed rendering of SE base models ids

https://0xdata.atlassian.net/browse/PUBDEV-6180

@seb-h2o yes, please commit that file. I'll have to make sure the release process commits that automatically.

@seb-h2o please remove the build/js/flow.js. That file is generated during release process. I won't do any harm (it'll be rewritten), just to be consistent with rest of the PRs. Thanks!


Latest Release Missing?

For some reason my version of the visualizer thinks the latest version is a previous release.
I assume this is because the latest release was removed or something?
Hope this helps.

After looking at it, [MeasureWebParser] in MeasureUpdateChecker.inc, checks https://github.com/MarcoPixel/Monstercat-Visualizer/releases/latest.
Which appears to be missing update 2.1.0

I was wondering why 2.0.2 was showing. I wasn't paying attention and installed it before I realized what happened. I don't see anything in the releases past 2.0.2. Luckily I still had the download file for 2.1.0 in case I needed to reinstall

Huh, interesting...
It seemed like GitHub did "hide" the latest release for some reason, but not remove it entirely.
Anyways, it should be fixed now :)


How to prevent a NodeTask from failing the build (CI)

I have set up protractor-tests for our angular projects.
On jenkins the gradle script uses gradle-node-plugin to run the protractor tests.
task update_webdriver(type: NodeTask){
    script = file('node_modules/protractor/bin/webdriver-manager')
    args = ['update']
    npmInstall.dependsOn 'protractor_copy'
}
update_webdriver.dependsOn 'npmInstall'

task protractorTests(type: NodeTask) {
    script = file('node_modules/protractor/bin/protractor')
    args = ['protractor-conf.js']
    ignoreExitValue = false
}
protractorTests.dependsOn 'update_webdriver'

This is quite fast, but the problem I got: when a test fails, the gradle-job fails and is shown as error on jenkins (red), but I want it to behave like junit-tests where failing tests only makes the build UNSTABLE (yellow).
I already asked at Protractor(1178), I can change the status code Protractor returns on test-failure, but returning code 0 when all tests run but with failures is maybe not the correct solution. Is there a non-zero code where the gradle-task still succeeds? Or how to configure it to behave as desired?

Hello,
Was any enhancement done to resolve this issue? I am facing similar problem. I have a task after running my tests which is not carried out if tests fail.


Can it run on RTX 4090 by adjusting hyperparameters such as batch size?

Can it run on RTX 4090 by adjusting hyperparameters such as batch size?

Should work with no modification at all!


reinstall spiderpool issue

after spiderpool is uninstalled, some CR is blocked in deleting state because finalizer
if install  spiderpool again, the  deleting CR will make it fail to install
it could optimize to delete these deleting CR in init pod

already done in v0.6


Support TypeScript 4.7 "moduleResolution": "node16"

TypeScript can't find the type definitions for this package if I set the moduleResolution option to node16 in my tsconfig.json.
Adding a "types" key to the package.json exports section fixes the problem:
{
  "exports": {
    ".": {
      "types": "./index.d.ts",
      "require": "./index.cjs",
      "import": "./index.mjs"
    }
  }
}

This needs to be patched in both temporal-polyfill and temporal-spec packages.
I can send PRs if you want.

Released in v0.1.0

I realized this needed fixing in the temporal-spec package as well. Fixed here: https://github.com/fullcalendar/temporal/commit/e955c2b460d7fdc0321b283cb5d36b7700c509b3
Released in v0.1.1


Not showing any PRs in workspaces

Issue Type: Bug

Create a workspace with more than one repos clones into it
Create some PRs for the second repo in the workspace list
Go to Source Control and in GITHUB PULL REQUESTS panel observe the values to be 0

If you only open the repo as a folder, the PRs are visible, but if you open it in a workspace the values are 0.
Extension version: 0.1.4
VS Code version: Code 1.27.1 (5944e81f3c46a3938a82c701f96d7a59b074cfdc, 2018-09-06T09:14:32.385Z)
OS version: Darwin x64 18.0.0

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-4980HQ CPU @ 2.80GHz (8 x 2800)

GPU Status
2d_canvas: enabledchecker_imaging: disabled_offflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onnative_gpu_memory_buffers: enabledrasterization: enabledvideo_decode: enabledvideo_encode: enabledwebgl: enabledwebgl2: enabled

Load (avg)
2, 2, 2

Memory (System)
16.00GB (3.09GB free)

Process Argv
/private/var/folders/by/z23k5y5n0gj156wj520kxy4c0000gn/T/AppTranslocation/1D71EB54-F5F2-4C95-975F-9958290B1607/d/Visual Studio Code.app/Contents/MacOS/Electron

Screen Reader
no

VM
0%

This extension doesn't yet support multi-root folders, please see https://github.com/Microsoft/vscode-pull-request-github/issues/386


Add axis_option argument for matplotlib plots

This PR addresses #38 for matplotlib plots (i.e. plot method of Embedding and EmbeddingSet).

The thing that amazes me the most is that it's been a week now and this issue has not been resolved yet (whether it's Github Actions or PyPI server)!

Thanks for submitting a pull request 🚀 @akelad will take a look at it as soon as possible ✨

Urgh. This is getting annoying I agree. This time it seems to be tensorflow that is having trouble getting the proper installation. I wonder what is causing this.

I'm guessing that it might be a pip issue, looking at their updates on pypi. It all seems very recent.

@koaning It seems it happens even if we use wget to download the package wheels: https://github.com/actions/virtual-environments/issues/1354#issuecomment-669911376

Urgh. I guess we'll need to wait for a fix then. Good to confirm this lies outside of our control tho.

This is a nice one to merge because I had need of it today. I'll be ignoring the tests for now because of the flaky pip/github actions situations. Again, @mkaze thanks!


ocaml-libvirt: update to <IP_ADDRESS>

Merge Checklist

All boxes should be checked before merging the PR (just tick any boxes which don't apply to this PR)

[x] The toolchain has been rebuilt successfully (or no changes were made to it)
[x] The toolchain/worker package manifests are up-to-date
[x] Any updated packages successfully build (or no packages were changed)
[x] Packages depending on static components modified in this PR (Golang, *-static subpackages, etc.) have had their Release tag incremented.
[x] Package tests (%check section) have been verified with RUN_CHECK=y for existing SPEC files, or added to new SPEC files
[x] All package sources are available
[x] cgmanifest files are up-to-date and sorted (./cgmanifest.json, ./toolkit/scripts/toolchain/cgmanifest.json, .github/workflows/cgmanifest.json)
[x] LICENSE-MAP files are up-to-date (./LICENSES-AND-NOTICES/SPECS/data/licenses.json, ./LICENSES-AND-NOTICES/SPECS/LICENSES-MAP.md, ./LICENSES-AND-NOTICES/SPECS/LICENSE-EXCEPTIONS.PHOTON)
[x] All source files have up-to-date hashes in the *.signatures.json files
[x] sudo make go-tidy-all and sudo make go-test-coverage pass
[x] Documentation has been updated to match any changes to the build system
[x] Ready to merge

Summary

ocaml-libvirt: update to <IP_ADDRESS>
Change Log

../SPECS-EXTENDED/ocaml-libvirt/ocaml-libvirt.signatures.json
../SPECS-EXTENDED/ocaml-libvirt/ocaml-libvirt.spec
../cgmanifest.json
ocaml-libvirt-<IP_ADDRESS>-1.azl3.src.rpm.log
Does this affect the toolchain?

NO
Associated issues

#xxxx

Links to CVEs

https://nvd.nist.gov/vuln/detail/CVE-YYYY-XXXX

Test Methodology

Pipeline build id: xxxx

This package built successfully without runtime errors.


size of emoticons are slightly different

Hi,
When I use emoticons in a single line of text, the height returned is always slightly bigger than that containing text only.
Found a similar SO question with no answer
http://stackoverflow.com/q/11691495/2954866
Could you please look into the issue?
I want to restrict the height of the label to three lines if the text is too long. Due to different height for emoticons, the text gets truncated.

I'm experiencing the same issue. Any idea where the problem could be?

Also experiencing this issue in iOS 7.0 - 8.1

Hello All, I have face same issue. Please help me how to resolved. I have strict deadline.

Finally i got solution .

Thank you @JayMehta711 for find the solution.

Hello All,
I have found one solution for that please refer below link.
http://stackoverflow.com/a/29429606/2058242

Hello,
3.1.18 should fix this problem.

Hi Sebastien,
Thanks for addressing this. Unfortunately, the fix has opened another bug with emoticons which I will open as a separate issue.


Using nvim-cmp with firenvim cause problem

FAQ

[X] I have checked the FAQ and it didn't resolve my problem.

Issues

[X] I have checked existing issues and there are no open or closed issues with the same problem.

Neovim Version
0.7.0dev
Minimal reproducible config
https://github.com/tin-pham/NeoVimLua
It inside lua/options/cmp.lua
Description
It show this error when I use firenvim on text box on the browser (markdown textbox like reddit, github work fine. Other textbox not markdown will cause this)

Steps to reproduce

Use nvim-cmp with firenvim
If the textbox is not markdown (facebook, googletranslate) it will cause this

Expected behavior

It just like the image above

Actual behavior
It will just writing normal like this without that text being annoy pop up all over again

Additional context
I have open issue on the github site of the firenvim author, and have some discussion with the admin
I can turn off nvim-cmp and start using firenvim on the web but when I start coding, it can be annoy to turn it on again, so maybe my config have something that cause this problem??
https://github.com/glacambre/firenvim/issues/1330

Having the same issue in JupyterLab.
:set lines = 20 fixed it for me

@130ndim That works but like, looks wierd and doesn't fit the input box / text area. I would imagine on larger text boxes, the height of the nvim box would be too small as well. There must be another way?

I've created an autocmd to set lines = max(20, box lines). I use it mostly to edit JupyterLab cells, so I'm not comfortable with default 1 line window anywas


docs: Improve the types of Billboard&BillboardCollection.add and Label&LabelCollection.add

Description
It improves the types of Billboard&BillboardCollection.add and Label&LabelCollection.add.
Fix a doc bug that they both do not contain disableDepthTestDistance in option, which should be contained.
Add types about the constructOptions of Label and Billboard. They are copy from BillboardGraphics / LabelGraphics.
Future improvement:
I think it may be need a little discussion that The BillboardGraphics and LabelGraphics already have similar type declarations. Should I
use the type declarations here directly in Billboard and Label,
or declare them in Billboard and Label and then reference them elsewhere,
or simply copy the type declarations and use them in both Label and LabelCollection?
I believe both approaches would not have any impact on the usage of the code (including TypeScript users). However, a less optimal solution might cause confusion during future maintenance.
Issue number and link
see #11813.
Testing plan
The PR is only about docs. Run build-docs and build-ts, and check the effect.
Before: BillboardCollection.add and LabelCollection.add have no intellisense.
After:  They should have correct intellisense.
Run npm run build-docs to confirm the document should be correct to show the information.
Author checklist

[x] I have submitted a Contributor License Agreement
[x] I have added my name to CONTRIBUTORS.md
[ ] I have updated CHANGES.md with a short summary of my change
[ ] I have added or updated unit tests to ensure consistent code coverage
[x] I have update the inline documentation, and included code examples where relevant
[x] I have performed a self-review of my code

Hi @s3xysteak, I don't see a CLA on file on file for you. We'll need this before we can review this PR. See CONTRIBUTING.md for instructions on submitting a CLA.
Let us know once that's submitted. Thanks!

@syzdev I'm closing this PR due to inactivity. If you get the chance to submit a CLA, please feel free to re-open. Thanks!


I can't run it if I change any other ecr20 token address

Thank you for providing such a good program, but I run a little problem, please help solve it. I changed another ecr20 token address, and i can't run my program.
from uniswap import Uniswap
from web3 import Web3
provide = 'https://mainnet.infura.io/v3/02c03b06ce7344d1bbde374834b349de'
address = '0xfd895ACd6960054B25Fe8CDdaA0dc271EF18716f'
private_key = '***'
uniswap_wrapper = Uniswap(address, private_key, provide, version=2)
eth = "0x0000000000000000000000000000000000000000"
pickle = Web3.toChecksumAddress("0xbd17b1ce622d73bd438b9e658aca5996dc394b0d")
print(uniswap_wrapper.get_eth_token_input_price(pickle, 1*10**18))

The error code is：
Traceback (most recent call last):
  File "D:\python\python38\lib\site-packages\web3-5.12.0-py3.8.egg\web3\contract.py", line 1509, in call_contract_function
    output_data = web3.codec.decode_abi(output_types, return_data)
  File "D:\python\python38\lib\site-packages\eth_abi\codec.py", line 181, in decode_abi
    return decoder(stream)
  File "D:\python\python38\lib\site-packages\eth_abi\decoding.py", line 127, in __call__
    return self.decode(stream)
  File "D:\python\python38\lib\site-packages\eth_utils\functional.py", line 45, in inner
    return callback(fn(*args, **kwargs))
  File "D:\python\python38\lib\site-packages\eth_abi\decoding.py", line 173, in decode
    yield decoder(stream)
  File "D:\python\python38\lib\site-packages\eth_abi\decoding.py", line 127, in __call__
    return self.decode(stream)
  File "D:\python\python38\lib\site-packages\eth_abi\decoding.py", line 142, in decode
    start_pos = decode_uint_256(stream)
  File "D:\python\python38\lib\site-packages\eth_abi\decoding.py", line 127, in __call__
    return self.decode(stream)
  File "D:\python\python38\lib\site-packages\eth_abi\decoding.py", line 198, in decode
    raw_data = self.read_data_from_stream(stream)
  File "D:\python\python38\lib\site-packages\eth_abi\decoding.py", line 305, in read_data_from_stream
    raise InsufficientDataBytes(
eth_abi.exceptions.InsufficientDataBytes: Tried to read 32 bytes.  Only got 0 bytes

The above exception was the direct cause of the following exception:

Traceback (most recent call last):
  File "D:/workspace/python/selen/test_uniswap.py", line 13, in <module>
    print(uniswap_wrapper.get_eth_token_input_price(Web3.toChecksumAddress(pickle), 1*10**18))
  File "D:\python\python38\lib\site-packages\uniswap_python-0.4.2-py3.8.egg\uniswap\uniswap.py", line 82, in check_version
    return f(self, *args, **kwargs)
  File "D:\python\python38\lib\site-packages\uniswap_python-0.4.2-py3.8.egg\uniswap\uniswap.py", line 293, in get_eth_token_input_price
    price = self.router.functions.getAmountsOut(
  File "D:\python\python38\lib\site-packages\web3-5.12.0-py3.8.egg\web3\contract.py", line 954, in call
    return call_contract_function(
  File "D:\python\python38\lib\site-packages\web3-5.12.0-py3.8.egg\web3\contract.py", line 1532, in call_contract_function
    raise BadFunctionCallOutput(msg) from e
web3.exceptions.BadFunctionCallOutput: Could not decode contract function call getAmountsOut return data b'' for output_types ['uint256[]']

I don't know what's going on, it can run normally!


feat(construct rewrite): rewrite aws-kinesisfirehose-s3

aws-kinesisfirehose-s3 as written has limited functionality and can only connect with data sources using constructs that tightly couple source and destination.
A rewrite will allow:

Decoupling source and destination - we can write aws-kinesisfirehose-s3 and aws-kinesisfirehose-amazonredshift once, and aws-lambda-kinesisfirehose and aws-iot-kinesisfirehose once and mix and match sources and destinations. As the number of sources and destinations grows, this will reduce the number of constructs required to fulfill more use cases.
More functionality - the new version will allow us to add helpers such as a transformation Lambda function to be applied to all incoming data, rather than going through all the steps required to create a processor (the construct will do that work).

The new construct must be functionally equivalent to the existing aws-kinesisfirehose-s3 if none of the new arguments are specified.
Use Case

Proposed Solution

Other

[ ] :wave: I may be able to implement this feature request
[ ] :warning: This feature might incur a breaking change

This is a :rocket: Feature Request

Changing direction to adapt current construct rather than rewrite.


First and last column cropped in bar chart

Hi, I'm building a bar chart with multiple columns. The first and last column are cropped when using 'datetime' as type on the x-axis. See screenshot. When I switch to 'numeric' as x-axis type, there is no issue.

It's similar to these previous issues: #540 and #280.
Is this a bug or can I fix it with adapting the styling?
Thanks so much.

I am using the latest version of  "vue-apexcharts": "^1.6.2", and still facing this cropped issue @brianlagunas @lucismedia


[AKS] allow setting --pod-cidr and --network-plugin-mode in az aks update command

This checklist is used to make sure that common guidelines for a pull request are followed.
Related command

az aks update: a cluster can be updated from Azure CNI to Azure CNI Overlay.

For new extensions:

[ ] My extension description/summary conforms to the Extension Summary Guidelines.

About Extension Publish
There is a pipeline to automatically build, upload and publish extension wheels.
Once your pull request is merged into main branch, a new pull request will be created to update src/index.json automatically.
You only need to update the version information in file setup.py and historical information in file HISTORY.rst in your PR but do not modify src/index.json.

AKS

Queued a pipeline to validate the newly added test case.


Add different types of volumes to the glossary

Some source material: https://success.docker.com/KBase/Different_Types_of_Volumes
Don't add these as separate entries, just expand on what's in the existing entry for volumes

I'll take this one up!


Go back from test not working for inner class

If you have test class like this:
[TestFixture]
public class MyClassTests
{
    [TestFixture]
    public class GroupedTests
    {
           //Have cursor here
    }
}

Navigating from the inner class above will not work as it will try to find Grouped class instead of MyClass...

Fixed in 0.7+


Investigate en_gf reset issue

There seems to be an issue where clk_adc is "X" if the en_gf defaults to 1.  I think the problem seems to be in V2T_clock_gen, although I haven't investigated this issue much.  Having en_gf default to 0 solves the problem, but then we have to remember to write en_gf to 1 once the chip starts up.  It would be good to get to the bottom of this problem in case it is masking a more serious issue.

Per discussion yesterday, en_gf will be enabled as part of the reset sequence:

De-assert ext_rstb
De-assert int_rstb (over JTAG)
Assert en_gf
Assert en_v2t


Update README to point to correct image

The image should be alpine instead of gliderlabs/alpine.

I'm not sure about this. This project is the gliderlabs/alpine image. We supply the tarballs for the official repository. But we were first gliderlabs/alpine and still are. The argument against this is because the specific official docs are at https://github.com/docker-library/docs/tree/master/alpine so those docs would reference alpine while ours would gliderlabs/alpine.
Though, I'll leave this open for more feedback. @tianon thoughts?

My confusion arose when I saw the latest tag in gliderlabs/alpine is 3.4 while the official alpine tag is the latest 3.6 - just like this repo. Will the gliderlabs/alpine image be kept up-to-date with the latest tags?
Additional confusion was felt after seeing that Glider Labs is mentioned as the maintainer in the official image.

That is a good question. I think the last two releases were pull requests and maybe they just neglected to update both. I'll get them updated to they are in sync.


TTN Socket

Hi, Jan
You plan to implement a socket to connect to the TTN (The Things Network). I started to investigate how to implement this connection, but I think it will be easier for you to do. Already in your plans with this library?

Not in LoRaLib - I want to keep this library just as a control for SX127x series LoRa chips. I did however investigate implementing LoRaWAN support in KiteLib, since that already has means to separate protocol implementation from the communication module (e.g. MQTT and HTTP protocols over ESP8266). If at some point I implement LoRaWAN or TTN, it will be in KiteLib, not here.


feat: grpc consistent hashing

Description

Add grpc consistent hashing.

Related Issue

Motivation and Context

Screenshots (if appropriate)
Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] Documentation Update (if none of the other choices apply)

Checklist

[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[x] I have read the CONTRIBUTING document.
[ ] I have added tests to cover my changes.

Codecov Report

Merging #1554 (f918297) into main (fbb6f1d) will increase coverage by 1.57%.
The diff coverage is 60.65%.

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

@@            Coverage Diff             @@
##             main    #1554      +/-   ##
==========================================
+ Coverage   43.29%   44.86%   +1.57%
==========================================
  Files         118      116       -2
  Lines       10919    10511     -408
==========================================
- Hits         4727     4716      -11
+ Misses       5708     5309     -399
- Partials      484      486       +2

Flag
Coverage Δ

Object-compatibility-e2etests
?

e2etests
?

unittests
44.86% <60.65%> (+1.57%)
:arrow_up:

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

Impacted Files
Coverage Δ

scheduler/config/dynconfig.go
27.83% <13.04%> (-8.07%)
:arrow_down:

client/config/dynconfig_local.go
42.50% <42.50%> (ø)

scheduler/resource/seed_peer_client_mock.go
34.21% <42.85%> (+7.12%)
:arrow_up:

scheduler/resource/seed_peer_client.go
91.17% <50.00%> (-2.95%)
:arrow_down:

client/config/dynconfig_manager.go
68.75% <68.75%> (ø)

internal/dynconfig/dynconfig.go
73.80% <71.42%> (+12.40%)
:arrow_up:

client/config/dynconfig.go
75.00% <76.92%> (+9.56%)
:arrow_up:

pkg/slices/slices.go
100.00% <100.00%> (ø)

scheduler/service/service.go
77.35% <100.00%> (ø)

client/daemon/peer/peertask_piecetask_poller.go
47.10% <0.00%> (-3.63%)
:arrow_down:

... and 4 more


Form preserver iterates through wrong elements

PHP Version
8.1
Shopware Version
<IP_ADDRESS>
Expected behaviour
When you add data-form-preserver="true" to a form the value is saved in local storage and  filled up with the stored value after a page reload.
Actual behaviour
When you have the other elements in the form besides input fields. The input fields (ex. div) are not taken into account.
How to reproduce
Working example:
<form id="test-form" data-form-preserver="true">             <input name="firstname" type="text">             <input name="lastname" type="text">         </form>
Not working example:
<form id='test-form' data-form-preserver='true'>             <input name="firstname" type='text'>

                     <input name="lastname" type='text'>

         </form>

Hi @wannevancamp, thank you for creating a PR to fix this issue. 👍
You changes were merged to trunk and the PR was marked as "Accepted": https://github.com/shopware/platform/commit/9b0a663aefba7dd37ccf4ec39943a6363f491585


Pressing tab after command makes it impossible to delete first 3 letters (fish shell).

After typing a command and pressing tab for completions I am unable to delete the first 3 letters of this command. I can't delete those letters with Ctrl + U either. It also makes Alacritty ignore them. Even if I press enter without deleting those letters, the command won't execute. Example:
$ ls

Then I press tab and some completions show up. Sometimes pressing tab first time places four spaces instead of showing completions.
$ ls     some-dir

Then I delete everything I can and ls  is left. After I press enter a new line happens.
$ ls
$

I'm using fish shell on MacOS 10.14.1.

@micouy Can you repro this with bash? What is the output of echo $TERM and alacritty --version?

Also please make sure you've tested the default config with alacritty --config-file /dev/null to make sure this is not an issue with the tabspaces you've set.

Can't reproduce it with bash because it only displays completions below the current line. Fish shell puts the first option in the current line. Running it with --config-file /dev/null seems to solve the problem but I've just downloaded alacritty.yml shipped with 0.3.3 release and it happened again.
echo $TERM returns xterm-256color.
alacritty --version returns alacritty 0.3.3 (cc3e4d3).
fish --version returns fish, version 3.0.0.

If --config-file /dev/null fixes it, it's likely an issue with your config. If you download a new config file, please make sure it's getting loaded correctly. I'd recommend just deleting the existing file and letting Alacritty generate one itself, that way it's guaranteed to be the correct config file for the Alacritty version.
For best possible compatability, I'd also recommend running the alacritty terminfo instead of xterm-256color, that might fix some issues.

If you download a new config file, please make sure it's getting loaded correctly.

@micouy You can launch alacritty -v to find out which config is being loaded.
You can see line similar to this one INFO] Configuration loaded from "/home/kchibisov/.config/alacritty/alacritty.yml"

It still happens after automatic generation. I checked alacritty -v and it loaded it from the correct location.
It works after running env TERM=terminfo alacritty. But I usually run it like a normal MacOS app, not from the terminal. Can I also make it work like that?

You need to add TERM: alacritty in env section of your config.
https://github.com/jwilm/alacritty/blob/76dd84bef87f77c67bf5c23ee627163e93786b5d/alacritty.yml#L13

I do not recommend setting the TERM to alacritty in the env section manually. If it is not loaded with the default config, that means you do not have the terminfo of Alacritty installed. Manually setting it to Alacritty in this case will break things. Instead of setting the TERM to alacritty yourself, you shouldn't set it to anything and install the terminfo file.

@chrisduerr do you have any ideas why --config-file /dev/null works and generated one doesn't? They should be identical as far as I remember.

OK, I think I've installed the terminfo file (I'm not sure if I did it correctly because I'm not familiar with terminfo files) and it works so far.
Thank you :). Closing.

Reopening. Alacritty works just fine if I run it like this:
open -a /Applications/Alacritty.app

or this
/Applications/Alacritty.app/Contents/MacOS/alacritty

but I didn't realize that the error still occurs if I open it like a normal MacOS app (from Spotlight Search, from Dock or from Finder). Sorry for that. In all cases the config is loaded from the same location. In all cases echo $TERM returns alacritty (IDK if that's the proper way of checking it so correct me if I'm wrong).

@micouy Could you provide your outputs of locale when you're starting from /Applications/Alacritty.app/Contents/MacOS/alacritty and from "sportlight launch"?

Launched like a normal app (faulty):
LANG=
LC_COLLATE="C"
LC_CTYPE="C"
LC_MESSAGES="C"
LC_MONETARY="C"
LC_NUMERIC="C"
LC_TIME="C"
LC_ALL=

Launched from a terminal:
LANG=
LC_COLLATE="C"
LC_CTYPE="UTF-8"
LC_MESSAGES="C"
LC_MONETARY="C"
LC_NUMERIC="C"
LC_TIME="C"
LC_ALL=

The first locale is odd, as you can see, don't change these variables for now, we'll come to them latter if you'll have problems.
Try launching your shell as login, since it's macOs.
shell:
  program: /usr/local/bin/fish
  args:
    - --login

duplicate #2566

@chrisduerr do you have any ideas why --config-file /dev/null works and generated one doesn't? They should be identical as far as I remember.

but I didn't realize that the error still occurs if I open it like a normal MacOS app

That explains the problem.

Changed the config and lauched it again. Nothing happened. locale returns the same output. Also there's a difference in displaying Unicode (screenshots from Kakoune).

Also there's a difference in displaying Unicode (screenshots from Kakoune).

It's expected, since your locale doesn't support unicode stuff. As a work around you can add
LC_CTYPE: "UTF-8" to env section in alacritty.yml. The second locale seems a little bit odd to me too, so probably some locale issue on your system is already presented, I don't remember how macOs handles it's locale stuff though.

Yeah this is a locale issue and the related issues should help out. Setting your locale properly will resolve this. Making sure to always launch a login shell is a good way of making sure your system has a chance to set the locale properly.


Bot keeps bumping ruff-pre-commit to incompatible release

Hi,
Firstly, great service, this tool is a great help keeping code in check.  One little irritation however, the bot seems to always choose the latest version of each hook, even when that version is incompatible with the project.
Our project targets multiple versions of Python, including some that are no longer maintained such as Python 3.6… just because 3.12 is the latest does not mean every computer with Python installed magically updates itself and some production environment are very conservative regarding updates.
(I've had to argue with a manager about Python 2.7 for example, and I have colleagues that still happily load Debian Stretch onto new boxes despite it being ancient today.)
As such, we try to limit ourselves to features supported by Python 3.6, chosen as a lower practical limit that still supports 'modern' features, whilst still allowing a wide compatibility base.
The problem is the pre-commit bot insists on bumping ruff-pre-commit.  The last version to support Python 3.6 is v0.0.233.  When it bumps this and proposes a pull request, the PR tests fail because… surprise surprise… the version of ruff used doesn't support this version of Python.
https://results.pre-commit.ci/run/github/659078614/1697485172.NGow0yyXQNaO8Fti5MGycw
https://github.com/diazona/setuptools-pyproject-migration/pull/108 is an example, where I've then had to revert one of the version bumps.
I had a look through the documentation and issues, couldn't see anywhere to "pin" a hook version.  Is there a method to pin this particular hook to a compatible version?
(Previously raised https://github.com/pre-commit/pre-commit.com/issues/887)

closed this as completed

I must be dense, what was the resolution to this issue?

I fixed the link in the original post to link to the correct spot where you'll find several duplicates and an explanation


rdar://105261237 (llbuild test failure in BuildDBBindingsTests.testCouldNotOpenDatabaseErrors)

Apparently we don't run tests for Swift bindings when we smoke test (smoke tests only run lit tests) and I forgot to run Swift binding tests before merging.

@swift-ci please smoke test


Fixes importHandler to not rely on a file mapping since the file mapping will not work for transitively loaded files.

I fixed the situation where the a collision was occurring for the protofiles. Now getting an elements name should be covered for all cases except when a protofile is loaded both directly and transitively (like it will be for the only for the test case when grpc-reroute-to-interface is used)

Current coverage is 82.47% (diff: 81.81%)

Merging #482 into master will increase coverage by 0.01%

@@             master       #482   diff @@
==========================================
  Files           206        206
  Lines          7703       7697     -6
  Methods           0          0
  Messages          0          0
  Branches       1013       1013
==========================================
- Hits           6352       6348     -4
+ Misses         1050       1049     -1
+ Partials        301        300     -1

Powered by Codecov. Last update f1eacb9...bfc969f

Looks like some codecov is lost on this CL. Hold off on review until I get some test cases.

To be fair, it seems from the codecov delta summary that it is complaining because your change removes 6 lines of previously-tested code, not because it adds any untested code. IMO, just deleting unneeded code should not cause the check to fail, since the "spirit" of the check is to prevent merging untested code in.
/cc @bjwatson

@Landrito How is this related to https://github.com/googleapis/toolkit/pull/480, which makes similar test case changes?
@geigerj I agree with the spirit of the check here, since it's forcing us to advance the needle on test coverage. However, I'm okay with ignoring a scenario like this (just removing previously tested code) in the interest of time.

@bjwatson @geigerj I think this issue was fixed by my most recent commit on #470. Closing for now.


SSL Verification Issue

Software Versions

Python: 3.10.12
OS: Linux
Kivy: 2.2.1
Kivy installation method: pip

Describe the bug
The SSL Certificate verification failed.
Expected behavior
It connecting to the server.
To Reproduce
Use SSL
Code and Logs and screenshots
[SSL: CERTIFICATE_VERIFY_FAILED] certificate verify failed: self signed certificate in certificate chain (_ssl.c:997) (configured timeouts: socketTimeoutMS: 20000.0ms, connectTimeoutMS: 20000.0ms),ac-7nhuys9-shard-00-00.jnw3vvs.mongodb.net:27017: [SSL: CERTIFICATE_VERIFY_FAILED] certificate verify failed: self signed certificate in certificate chain (_ssl.c:997) (configured timeouts: socketTimeoutMS: 20000.0ms, connectTimeoutMS: 20000.0ms),ac-7nhuys9-shard-00-01.jnw3vvs.mongodb.net:27017: [SSL: CERTIFICATE_VERIFY_FAILED] certificate verify failed: self signed certificate in certificate chain (_ssl.c:997) (configured timeouts: socketTimeoutMS: 20000.0ms, connectTimeoutMS: 20000.0ms), Timeout: 30s, Topology Description: <TopologyDescription id: 65574f17a4fd7d122525f3da, topology_type: ReplicaSetNoPrimary, servers: [<ServerDescription ('ac-7nhuys9-shard-00-00.jnw3vvs.mongodb.net', 27017) server_type: Unknown, rtt: None, error=AutoReconnect('ac-7nhuys9-shard-00-00.jnw3vvs.mongodb.net:27017: [SSL: CERTIFICATE_VERIFY_FAILED] certificate verify failed: self signed certificate in certificate chain (_ssl.c:997) (configured timeouts: socketTimeoutMS: 20000.0ms, connectTimeoutMS: 20000.0ms)')>, <ServerDescription ('ac-7nhuys9-shard-00-01.jnw3vvs.mongodb.net', 27017) server_type: Unknown, rtt: None, error=AutoReconnect('ac-7nhuys9-shard-00-01.jnw3vvs.mongodb.net:27017: [SSL: CERTIFICATE_VERIFY_FAILED] certificate verify failed: self signed certificate in certificate chain (_ssl.c:997) (configured timeouts: socketTimeoutMS: 20000.0ms, connectTimeoutMS: 20000.0ms)')>, <ServerDescription ('ac-7nhuys9-shard-00-02.jnw3vvs.mongodb.net', 27017) server_type: Unknown, rtt: None, error=AutoReconnect('ac-7nhuys9-shard-00-02.jnw3vvs.mongodb.net:27017: [SSL: CERTIFICATE_VERIFY_FAILED] certificate verify failed: self signed certificate in certificate chain (_ssl.c:997) (configured timeouts: socketTimeoutMS: 20000.0ms, connectTimeoutMS: 20000.0ms)')>]>

Additional context
Used pymongo. Included certifi and specified certifi.where() in the request.

I was told to create this issue in the support channel.

I am having trouble seeing where. Please link to it.
Please include some minimum code that demonstrates the problem. Please include the complete log file.
Is this running a Kivy app on Linux, or running Buildozer on Linux?
If Buildozer, did you follow the advice at https://github.com/Android-for-Python/Android-for-Python-Users#ssl-certificate_verify_failed at some point. Did you clean your buildozer install?
[This is the sort of discussion that is best done in the support channel.]

Buildozer and followed the advice at Android-for-Python/Android-for-Python-Users#ssl-certificate_verify_failed.
Cleaned .buildozer
I think the person who sent the message deleted it.

2023-11-18 12-43-34 debug log.txt

Closing issue as stale.


Organize admin list pages

More attributes can be visible in the List pages of the admin panel. This is useful in searches. Add more attributes for models that have many entries:

[x] PaperSubmissions
[ ] PaperSubmissionImages
[ ] Assignments

latest commit in develop fixes this


Multiple Instruments not receiving ticks

Connection closed: 1006 - connection was closed uncleanly (I dropped the WebSocket TCP connection: close reason without close code)
when i subscribe list of instruments i received above error. then how receive multiple instruments ticks
`# Assign the callbacks
watch_list = [int(i) for i in tsht.range("A2").expand('down').value]
[779521, 738561, 1102337]
def on_connect(ws, response):
ws.subscribe([watch_list])
ws.set_mode(ws.MODE_QUOTE, [watch_list])
kws.on_ticks = on_ticks
kws.on_connect = on_connect
kws.connect()`

Parameter of ws.subscribe should be a list.  From the looks, it seems your passing a list of list.
Try:
ws.subscribe(watch_list)
ws.set_mode(ws.MODE_QUOTE, watch_list)

@vijaykz Thanks now its working😀🤝👍


Game Request

Hi! I have come again for some game requests of quite popular games XD

Fire Emblem Warriors: Three Hopes
Live A Live
Demon Slayer -Kimetsu no Yaiba- The Hinokami Chronicles
TRIANGLE STRATEGY
Pokémon Legends: Arceus

Appreciate all the work and updates so far!

hello so here are my responses to your games:

alright i'll add this
sadly i'm going to have to say this game isn't big enough to add in the current game list, it will have to wait until v2.0.0
all right i can add this too
i will add this too
this game has already been added lol

so my final response is i'll be adding Fire Emblem Warriors: Three Hopes, Demon Slayer -Kimetsu no Yaiba- The Hinokami Chronicles, and TRIANGLE STRATEGY

Oh, didn't realize about Arceus lol. Anyways, thanks! Can't wait for the next update

added them


JSON from Angular to Play Controller gets $oid stripped on Angular 1.2.13

Trying to write an update command and while sending the JSON back to the Play controller, Angular 1.2.13 strips out any $ characters, like the $oid which will cause validation errors.
Updating to the latest Angular version (1.3.6) fixed this issue.
If I get to it, I'll fork, update, test and submit a pull request.

Thanks for the Pr Andrew, its been merged and the latest greatest is in master.


Environment variables won't load in presets file

Brief Issue Summary
After upgrading to v1.19.49 the $env{} won't work in CMakePresets.json file. Reverting version to v1.18.44 will fix the problem
CMake Tools Diagnostics
No response
Debug Log
No response
Additional Information
No response

@amir-s2 It is very difficult to fix any issues if there is no information to investigate with. What about your env isn't working?
Please provide a repro project, repro steps, and the output of the CMake Diagnostics Log command.
Also, we've recently merged some regression fixes into the v1.20.2 pre-release channel.

I have something like this as my default preset:
  "configurePresets": [
    {
      "name": "default",
      "hidden": true,
      "cacheVariables": {
        "CMAKE_TOOLCHAIN_FILE": "$env{VCPKG_ROOT}/scripts/buildsystems/vcpkg.cmake",
        "VCPKG_MANIFEST_NO_DEFAULT_FEATURES": "ON"
      }
    },
    ...
    ]

The variable "$env{VCPKG_ROOT} would evaluate to nothing with the latest version. It's would be loaded correctly in the last version

pre-release seems fine btw I've just checked that :))

Great! I'll close this issue. We hope to patch the official version of the extension sometime next week. Thanks!

@gcampbell-msft We verified this issue on CMake tools:v1.20.2 (pre-release) and it has been fixed.
CMake Tools Diagnostics.txt
Actual result：


Answer to exercise 1-4 ch.5

Hi @axel22
Pls see my answers

LGTM
The Cellular Automaton example could be further optimized, but it looks correct to me!

Hm, it looks like this broke the build, due to some issues with JavaFX.
https://travis-ci.org/concurrent-programming-in-scala/learning-examples/builds/83789552

Hi @axel22,
I added sbt dependency for JavaFX 8, but travis.yml uses oraclejdk7.
Could you use JDK 8 ?

Scala 2.11 provides only experimental support for JDK 8, and is mainly targeting JDK 7. I feel a bit anxious about changing that.


Possibly add GitHub Discussions to Repo?

I've seen that there are lots of interested people and also possible contributors.
For that reason, I think it is possibly beneficial to use the new GitHub discussion feature.
This could be for topics that are not inherently issues, but more for clarification, sharing ideas on design and sketching rough feature ideas.

Thank you, I never know it.

After tested it for a week, I found out that using "Issues" only is more centralized and it is more well-known for most Github user. So I disabled it again.


update the Riot screenshot on Matrix.org to one that doesn't say that E2E is doomed

c.f. https://www.reddit.com/r/privacy/comments/5jqe9g/encrypted_messengers_why_riot_and_not_signal_is/dbi9nso/

done!


Re-implemented filepath, changepath, bug fixes

Coverage decreased (-2.5%) to 81.811% when pulling 9d6e0341a4bd0c9b877328ab2d858e0054760d23 on DaoHanLim:master into 8e0b44d143125af6a095d1153eb331d0afeff7eb on CS2103JUN2017-T4:master.


Spatio-temporal data support

Is your feature request related to a problem? Please describe.
Support framework for spatio-temporal data.
Describe the solution you'd like
Integrating spatio-temporal forecasting methods
Describe alternatives you've considered
Integrating spatio-temporal forecasting methods
Additional context
GluonTS

Hi @moghadas76, could you provide more information about what feature you would like to have added?
E.g., which particular algorithms you would like to have and perhaps also post some references, this would be very helpful.


Social account error html - 3rdparty

when entering the url 'accounts/3rdparty/', I get this error

any solution?

Fixed in 1.2.1

sorry for the comment, I'm kind of new to github. Have you fixed the bug or are you still going to fix it?

It's been fixed

Oh, it's fixed now, thank you very much for your hard work.

Thanks for finding the bug


fix yq version check

What this PR does / why we need it:
At my mac, the yq --version output is not matching the grep pattern.
yq --version
yq (https://github.com/mikefarah/yq/) version v4.30.8

Which issue(s) this PR fixes:

Fixes #
What type of PR is this?
/kind bug

Special notes for your reviewer:
Does this PR introduce a user-facing change? Then add your Release Note here:

fix yq 4 version check

Documentation:

NONE

/assign @wurbanski

not needed as mla is now installed via insaller


Updated linting url

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

:exclamation: No coverage uploaded for pull request base (main@7f87380). Click here to learn what that means.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #605   +/-   ##
=======================================
  Coverage        ?   63.61%
=======================================
  Files           ?       50
  Lines           ?     1891
  Branches        ?        0
=======================================
  Hits            ?     1203
  Misses          ?      688
  Partials        ?        0

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


$in with empty collection as argument results in MongoQueryException (version 1.0.1)

When running following query
            final Query<GridEventActive> query = getDataStoreWithMorphia()
                    .find(GridEventActive.class)
                    .field("eventId")
                    .in(event.getRelatedEvents());

where event.getRelatedEvents() is an empty HashSet
the Mapper tries to find a suitable converter and encode this argument.  I noticed this resulted in using the IterableConverter, which ends up returning null in its final statement.
return !values.isEmpty() || getMapper().getOptions().isStoreEmpties() ? values : null;
com.mongodb.MongoQueryException: Query failed with error code 17287 and error message 'Can't canonicalize query: BadValue $in needs an array' on server

Seconded. Using $in with an [] is a valid Mongo Query. Not sure why [] is turned into $null.
I've had to add a bunch of if(collection.isEmpty()){ return default }

I'm having trouble recreating this one:
        getMorphia().getMapper().getOptions().setStoreEmpties(false);
        final Query<Photo> query = getDs()
                           .find(Photo.class)
                           .field("keywords")
                           .in(new ArrayList<String>());
        System.out.println("************ query = " + query);

************ query = { "keywords" : { "$in" : [ ]}}
What am I missing?  i tried passing in an empty HashSet, too.

I can't seem recreate this so I'm closing for now.  If we get a test case recreating the issue, we can reopen this.


new logo update on every navbar

Related Issue: UI Related Issue
Fixes:  #2302
Description
The logo updated also the size has been changed
Type of PR

[ ] Bug fix
[X] Feature enhancement
[ ] Documentation update
[ ] Other (specify): _______________

Screenshots / videos (if applicable)

Checklist:

[X] I have made this change from my own.
[X] My code follows the style guidelines of this project.
[X] I have performed a self-review of my own code.
[X] My changes generate no new warnings.
[X] I have tested the changes thoroughly before submitting this pull request.
[X] I have provided relevant issue numbers and screenshots after making the changes.

Close Issue : #2302

@RitiChandak
I do all changes check it and merge it!

@RitiChandak I can't receive points for 2 issues.
even if both are merged


[BEAM-13015] Simplify DataStreamsDecoder by moving hasNext logic to next

Also delete a bunch of dead code.

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Choose reviewer(s) and mention them in a comment (R: @username).
[ ] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue, if applicable. This will automatically link the pull request to the issue.
[ ] Update CHANGES.md with noteworthy changes.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.

GitHub Actions Tests Status (on master branch)

See CI.md for more information about GitHub Actions CI.

R: @y1chi

Run Java PreCommit


Simconnect needs updating. After today's MSFS2024 update - appears to be a version mismatch.

The was a game update today and teleporting on longer works in MSFS 2024. Looks like a version mismatch. Can someone update it please?
When you try to teleport, it gets an error:
msfs2020-go/vfrmap
readme: https://github.com/lian/msfs2020-go/blob/master/vfrmap/README.md
issues: https://github.com/lian/msfs2020-go/issues
version: v0.0.7 (2020-09-05_04:33:49)
connected to flight simulator!
flight simulator info:
codename: SunRise
version: 12.1 (282174.999)
simconnect: 12.1 (0.0)
new browser connection
SIMCONNECT_RECV_ID_EXCEPTION simconnect.RecvException{Recv:simconnect.Recv{Size:0x18, Version:0x6, ID:0x1}, Exception:0x14, SendID:0x99, Index:0x1}
SIMCONNECT_RECV_ID_EXCEPTION simconnect.RecvException{Recv:simconnect.Recv{Size:0x18, Version:0x6, ID:0x1}, Exception:0x14, SendID:0x99, Index:0x2}
SIMCONNECT_RECV_ID_EXCEPTION simconnect.RecvException{Recv:simconnect.Recv{Size:0x18, Version:0x6, ID:0x1}, Exception:0x14, SendID:0x99, Index:0x3}
remove browser connection

BUMP can someone explain how to fix it?

Same issue here.

same here with exact same error message.

Bump!!!

Try this https://github.com/watsonpaul80/MSFS2024-VFR-MAP/releases/tag/v1.0.0 I have it working on mine now

Try this https://github.com/watsonpaul80/MSFS2024-VFR-MAP/releases/tag/v1.0.0 I have it working on mine now

Why is it that this one is instantly blocked by any antivirus programs?

Try this https://github.com/watsonpaul80/MSFS2024-VFR-MAP/releases/tag/v1.0.0 I have it working on mine now

Why is it that this one is instantly blocked by any antivirus programs?

For the same reason that is outlined on the official golang website https://golang.org/doc/faq#virus and which lian mentions at the bottom of the page.

someone with brain needs to make it work in career :).

Im a noob mate thought i had it working but its not sorry :( I installed the new SKD and it worked but for some reason no teleporting now

msfs2020-go/vfrmap
readme: https://github.com/lian/msfs2020-go/blob/master/vfrmap/README.md
issues: https://github.com/lian/msfs2020-go/issues
version:  ()
connected to flight simulator!
flight simulator info:
codename: SunRise
version: 12.1 (282174.999)
simconnect: 12.1 (0.0)
new browser connection
SIMCONNECT_RECV_ID_EXCEPTION simconnect.RecvException{Recv:simconnect.Recv{Size:0x18, Version:0x6, ID:0x1}, Exception:0x14, SendID:0x332, Index:0x1}
SIMCONNECT_RECV_ID_EXCEPTION simconnect.RecvException{Recv:simconnect.Recv{Size:0x18, Version:0x6, ID:0x1}, Exception:0x14, SendID:0x332, Index:0x2}
SIMCONNECT_RECV_ID_EXCEPTION simconnect.RecvException{Recv:simconnect.Recv{Size:0x18, Version:0x6, ID:0x1}, Exception:0x14, SendID:0x332, Index:0x3}
yeah same error in career, I believe MSFS 2024 SDK API has different teleporting method or something.

https://forums.flightsimulator.com/t/career-mode-exploit-players-able-to-teleport-reposition-using-simconnect-api/681208/7
This is the answer, the Microsoft team is prohibiting the use of teleportation in career mode, they consider it an exploit, or abuse of teleportation to gain xp and credits.
But, they didn't think that this part is boring, and we want the best missions.

Flying a MISSION for 4-6 hours and getting 30K, or asobo forbid crash your plane, only retirees, single dudes or without jobs or families can do. Companies are good that you can make nice money, but it's VERY easy to be owing millions from crashes, maintenances, etc. The career mode is full of bugs, not balanced, and they going to push us to live in this game? No way, Josue.

You're right, but if we succeed the first time, there will always be a way, we'll just wait a little.

Please Vote here if you want teleport option in career mode in MSFS2024:
https://forums.flightsimulator.com/t/bring-back-the-ability-to-use-sim-connect-with-vfrmap/684747/13

I posted there as well, but yeah... I think devs disabled this on a code level, so only they can add it now.

Just for your info, here is another Thread where you are able to vote:
Please allow the unrestricted use of mods, even in Career Mode
https://forums.flightsimulator.com/t/please-allow-the-unrestricted-use-of-mods-even-in-career-mode/686678/16

Doesn't work with the latest release. Hope to see an update soon!

Try this https://github.com/watsonpaul80/MSFS2024-VFR-MAP/releases/tag/v1.0.0 I have it working on mine now

Tried this one too and same issue, not working with the latest release.

So, Asobo thinks the teleport function as a exploit and cheating...i really doubt that they will gonna change that by request of some guys on the official forums.
We need to contact the original VFRMAP developer and ask him for a bypass.
And since its working on free flight mode, it's not broken, it's probably not to complex to make a update to put it working again...

@leoleo12321 I have contacted the VFRMAP dev. The problem is that they completely removed the ability for 3rd Party software to modify the game state like anti-cheat. There's no way to send any command back to the game engine. The only way is for Msfs devs to allow it again

They disabled devkit entirely when you enter the career mode, so you can't use simconnect any longer to teleport. VFRMAP was using simconnect to pass the new coordinates to the plane in the game. Since all the external access to developer mode tools is now disabled for good, teleport will never work again. Good for those who raked a lot of money while it did.


Fix Mobile Renderer - Shadow Disabled and User Vertex Lighting flags

Fixes #98102 Forward Mobile Renderer fix. Corrects preprocessor directive, also removes redefinition of method names as parameters which must predate the ubershader work.
To test toggle the two shader flags in Mobile mode and watch the console to confirm no fireworks, visual checks look good:
USE_VERTEX_LIGHTING - uber shader bug
Shadows Disabled - preprocessor order bug
Adjusted both here as the issue crept in under the vertex lighting change and I wanted to confirm no gremlins with the two interacting, was confused when I couldn't switch on the vertex lighting at all.

Thank you. Will pull this to recompile the fork as soon as it is merged. Leaving this comment so as to not to forget.

Confirmed the issue and tested the PR to confirm it fixes it. Changes look good to me.

Cool is anything else required to merge this?

Thank you! And congratulations on your first merged pull request!!


chore : GH-40 Increase test coverage.

Codecov Report

Merging #41 (a9a8290) into main (f4143b8) will decrease coverage by 0.24%.
The diff coverage is 16.88%.

@@             Coverage Diff              @@
##               main      #41      +/-   ##
============================================
- Coverage     28.37%   28.13%   -0.24%
+ Complexity       37       36       -1
============================================
  Files            44       45       +1
  Lines          1667     1834     +167
  Branches        213      245      +32
============================================
+ Hits            473      516      +43
- Misses         1147     1271     +124
  Partials         47       47

Files
Coverage Δ

...e/jerryokafor/ihenkiri/ui/navigation/Transition.kt
0.00% <ø> (ø)

...etwork/datasource/DefaultMoviesRemoteDataSource.kt
100.00% <100.00%> (ø)

...enkiri/core/network/model/response/NetworkMovie.kt
79.31% <ø> (ø)

...afor/ihenkiri/core/network/service/MovieListApi.kt
100.00% <ø> (ø)

...n/me/jerryokafor/core/ui/components/MoviePoster.kt
36.95% <ø> (ø)

...kiri/feature/moviedetails/MoviesDetailViewModel.kt
100.00% <100.00%> (+74.50%)
:arrow_up:

...yokafor/ihenkiri/ui/navigation/BottomNavigation.kt
0.00% <0.00%> (ø)

...n/me/jerryokafor/ihenkiri/ui/screens/HomeScreen.kt
0.00% <0.00%> (ø)

.../kotlin/me/jerryokafor/core/data/util/ImageUtil.kt
0.00% <0.00%> (ø)

.../jerryokafor/core/ui/components/MovieListFilter.kt
0.00% <0.00%> (ø)

... and 8 more


Remove unused retire-with-permanently-down flag

Circuit breaker failed twice in a row, giving up.


The crates' avrdude.config not compatible with AVRDUDE 7.1

It seems like this file: https://github.com/Rahix/avr-hal/blob/main/ravedude/src/avrdude/avrdude.conf is not compatible with AVRDUDE 7.1: https://github.com/avrdudes/avrdude/discussions/1636
You get these warnings:

avrdude warning: ATmega329's flash writepage misses a necessary address bit a13 [/tmp/.avrdude-OHDaQR.conf:5780]
avrdude warning: a4 would normally be expected to be a5 [/tmp/.avrdude-OHDaQR.conf:11367]

Is there a way to update that avrdude.conf file or use the one in /etc/ instead?

Sorry for the late reply!  Yeah, this is a really unfortunate situation.

Is there a way to update that avrdude.conf file or use the one in /etc/ instead?

Both are tricky:

We can't just update to an avrdude.conf for avrdude 7.X as that will break users who are still using avrdude 6.X.
We can't rely on the host avrdude.conf file (e.g. from /etc/) because there are some board specific changes we had to add to the config.  I'd much prefer to use the system config instead of our custom one, but I don't see how we can get the necessary changes in there.

I think in the long term, the proper solution is to stop relying on the host avrdude and instead manage a vendored version that we can control.  Then we always know what avrdude version we use and can always ship a matching config.  It also means one less thing for the user to install.  Arduino does the same thing, I'm pretty sure they had to learn the same lesson in the past ^^
As an immediate bandaid, I've created #508 to at least get ravedude working again.  Not pretty, but maybe it's enough for now...


Exception in 3D Secure handling in some cases (might be 3DSv1 handling related)

General information

SDK version: 3.79.1
Environment: Production
Browser and OS Edge 91.0.864 on Windows 10, Chrome 91.0.4472 on Mac OS X 10.14.6

Issue description
We've had some clients complaining about not being able to go over the 3D Secure authentication step. We have identified the error described below as the cause in these cases. It seems to happen for clients with credit/debit cards that use 3DSv1. We weren't able to replicate the issue in a sandbox environment even when forcing the ThreeDSecure client to use 3DSv1.
The problem is that this._verifyCardPromisePlus is null when _handleV1AuthResponse is called here: https://github.com/braintree/braintree-web/blob/main/src/three-d-secure/external/frameworks/base.js#L250
This the stack trace, captured by our exception tracking platform:
TypeError: Cannot read property 'resolve' of null
  at _handleV1AuthResponse(./node_modules/braintree-web/dist/browser/three-d-secure.js:2112:33)
  at _handleAuthResponseFromV1Fallback(./node_modules/braintree-web/dist/browser/three-d-secure.js:2760:8)
  at apply(./node_modules/braintree-web/dist/browser/three-d-secure.js:2770:12)
  at apply(./node_modules/braintree-web/dist/browser/three-d-secure.js:537:37)
  at dispatch(./node_modules/braintree-web/dist/browser/three-d-secure.js:734:51)
  at n.onmessage(./node_modules/braintree-web/dist/browser/three-d-secure.js:787:16)

Thanks for the report. Can you share your client side code? I assume you're setting the SDK up to use 3ds v2 and your customers' bank is going through the v1 flow instead.

That's right, the setup is made with version: 2. Here's a simplified version of our setup / code, but please note that we weren't able to replicate the error with the sandbox env, and we haven't found a credit card that triggers the issue with the production environment.
import BraintreeClient from 'braintree-web/client';
import BraintreeThreeDSecure from 'braintree-web/three-d-secure';

import { BRAINTREE_CC_ENDPOINT } from 'config';
import { generatePaymentMethodNonce } from 'api/payments';
import { checkEnrollStatus, addFrame, removeFrame } from 'utils/three-d-secure';

const updateCreditCard = (token, ccFormData, customerEmail, amount, planName) => {

	let braintreeClient;
	let threeDSecureInstance;

	BraintreeClient.create({authorization: token}, (error, client) => {

		if (error) {
			return console.error(error);
		}

		braintreeClient = client;

		BraintreeThreeDSecure.create({
			version: 2,
			client: braintreeClient
		})

		.then(threeDSecure => {

			threeDSecureInstance = threeDSecure;

			return braintreeClient.request({
				method: 'post',
				endpoint: BRAINTREE_CC_ENDPOINT,
				data: ccFormData
			});

		})

		.then(({creditCards}) => {

			let {
				nonce
			} = creditCards[0];

			// This is where the card is vaulted on the server side
			return generatePaymentMethodNonce({
				nonce: nonce,
				plan: planName
			});
		})

		.then(response => {

			let {
				nonce,
				bin,
				authRegulationEnv
			} = response;

			return threeDSecureInstance.verifyCard({
				amount: amount,
				nonce: nonce,
				bin: bin,
				email: customerEmail,
				additionalInformation: {
					acsWindowSize: '03'
				},
				// We've added these only for testing with version: 1
				// I don't think these are called when using version: 2
				addFrame: addFrame,
				removeFrame: removeFrame
			});
		})

		.then(({threeDSecureInfo}) => {

			if(checkEnrollStatus(threeDSecureClient)) {
				console.log('success');
			} else {
				console.warn('failed 3dsecure verifyCard');
			}
		})

		.catch(error => {
			console.error(error);
		});
	});
};

Thank you!

Thanks for the info. We'll take a look.

Can you pass along your public merchant id? I have an inkling about what's going wrong and want to confirm.

I believe this is our merchant id: czwt3x4rnqxh9dtb Let me know if I can help with other details.

We have analytics events fire for when the v1 fallback flow gets used. These typically only fire when the SDK for our 3DS provider (Cardinal) fails to set up correctly. Here are the analytics events we see for your integration for the last 7 days:

kind
count

web.three-d-secure.v1-fallback.cardinal-sdk-setup-failed.songbird-js-failed-to-load
2

web.three-d-secure.v1-fallback.cardinal-sdk-setup-timeout
2

web.three-d-secure.v1-fallback.cardinal-sdk-setup-error.number-10001
1

web.three-d-secure.v1-fallback.cardinal-sdk-setup-error.number-0
2

web.three-d-secure.v1-fallback.cardinal-sdk-setup-error.number-10004
64

For songbird-js-failed-to-load, we fire that here: https://github.com/braintree/braintree-web/blob/3da6d477feb0f02bec9fd5dad89de3fea4fd2d27/src/three-d-secure/external/frameworks/songbird.js#L365-L367
That typically occurs when the code that pulls in the Cardinal SDK fails. (maybe the CDN was unreachable, or the device lost internet connect while trying to pull down the script)
For cardinal-sdk-setup-timeout, we fire that here: https://github.com/braintree/braintree-web/blob/3da6d477feb0f02bec9fd5dad89de3fea4fd2d27/src/three-d-secure/external/frameworks/songbird.js#L358
And that should only fire if it takes longer than 60 seconds to initialize the Cardinal SDK.
The last batch of events, cardinal-sdk-setup-error.number-* fires here: https://github.com/braintree/braintree-web/blob/3da6d477feb0f02bec9fd5dad89de3fea4fd2d27/src/three-d-secure/external/frameworks/songbird.js#L464
And should only really happen if the callback that is supposed to be used to finish the 3DS Challenge gets called during the setup process. It's a weird quirk of the Cardinal SDK where this callback can be called in the case of a terminal error.
The documentation for the specific error codes are provided here: https://cardinaldocs.atlassian.net/wiki/spaces/CC/pages/100401153/Error+Codes
Though, since 0 is undocumented and the other 2 don't give us much to go on, it's hard to draw any conclusions here.
To my knowledge, it should be impossible to get to the point where it's calling this._verifyCardPromisePlus.resolve without having this._verifyCardPromisePlus set, but that's clearly not the case here.
I think our next step is to get specific timestamps in UTC so we can figure out exactly what failure flow is occurring, which should give us a better idea of how to diagnose why the state is incorrect.

Thank you for the insights. I was able to gather the timestamps for three customers we've had where this error happened. For one of them we also captured an error message about cardinal, right before the exception happened, but we didn't capture any other information about the error, only the error message. Please find the timestamps below:
# Customer 1 - Edge 91.0.864, Windows
Jul 21, 2021 12:19:47 PM UTC -TypeError: Cannot read property 'resolve' of null
Jul 21, 2021 12:19:43 PM UTC -A general error has occurred with Cardinal. See description for more information.
Jul 20, 2021 4:45:18 PM UTC - TypeError: Cannot read property 'resolve' of null
Jul 20, 2021 4:44:28 PM UTC - TypeError: Cannot read property 'resolve' of null
Jul 20, 2021 1:03:33 PM UTC - TypeError: Cannot read property 'resolve' of null
Jul 19, 2021 1:02:13 PM UTC - TypeError: Cannot read property 'resolve' of null
Jul 19, 2021 1:01:53 PM UTC - TypeError: Cannot read property 'resolve' of null
Jul 15, 2021 8:31:38 AM UTC - TypeError: Cannot read property 'resolve' of null
Jul 15, 2021 8:31:15 AM UTC - TypeError: Cannot read property 'resolve' of null

# Customer 2 - Chrome 91.0.4472, MacOS
Jul 20, 2021 7:07:19 PM UTC - TypeError: Cannot read property 'resolve' of null

# Customer 3 - Chrome 91.0.4472, MacOS
Jul 18, 2021 10:17:15 PM UTC - TypeError: Cannot read property 'resolve' of null

We've just released a beta version that we think will resolve the issue. (hard to tell, of course, since we can't replicate it) Can you upgrade your version to 3.79.2-beta.1 and let us know if you are still getting the TypeError?

We released the beta to npm as well, so should be as easy as updating the package.json

Thank you! We have now updated the version in our builds, we're watching for any errors, I'll keep you posted.
Have a nice weekend!

@adig any changes on your end?

@crookedneighbor apologies for the delay. Everything looks good on our end, we haven't seen the error again.
If you feel comfortable with the changes, you can close this issue and apply them to the stable release.
Thank you 🙏

Awesome, glad to hear it! We're hoping to get a release out this week.

This is out in v3.80.0


test suite failing on Node.js 8 / 10 / 11

Had to skip it from CITGM test suite

I'd assume that is the cause yes. our CI was also failing for a while and is green since #53 was merged so it should be fine again.

Please lmk once a new release is out and we can revert the skip on CITGM
On Thu, Nov 29, 2018, 5:33 AM Renée Kooi<EMAIL_ADDRESS>wrote:

I'd assume that is the cause yes. our CI was also failing for a while and
is green since #53 https://github.com/nearform/node-clinic-flame/pull/53
was merged so it should be fine again.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/nearform/node-clinic-flame/issues/55#issuecomment-442785218,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAecV-DD3SsiruQlPe8x_4-McKrdSJhkks5uz7fogaJpZM4Y5AOE
.


[v1] do fix CI

What type of PR is this?

bug

What this PR does / why we need it:
This really fixes the CI for v1 branch. The previous attempt at that (#1379) fell short
due to two issues; both are fixed here.
Which issue(s) this PR fixes:
none
Special notes for your reviewer:
none
Testing
Check the CI.
Release Notes
none

Working fine now.
 Once merged, I will rebase https://github.com/urfave/cli/pull/1383 and make it ready for review and merge.


Searching a username with special character show no result

For example I tried to search Aporue-Sutol .. it showed no result. Searching just Aporue showed up.

I thought you couldn't have special characters in usernames?

I think we can have - .. I have seen few people with it. Not sure about other characters.


Cannot build esbuild bundle

Describe the bug
It appears the path used in
<script data-entry-point type="module" src="./App.js"></script>

Needs to exist on disk for esbuild to find it.
To Reproduce
I have a config like this:
{
  "$schema": "https://raw.githubusercontent.com/AngelMunoz/Perla/v0.22.1/perla.schema.json",
  "index": "./index.html",
  "fable": {
    "project": "./src/MyProject.App/MyProject.App.fsproj",
    "extension": ".js",
    "outDir": "./out"
  },
  "devServer": {
    "port": 8900,
    "mountDirectories": {
      "./out": "",
      "./out/fable_modules": "/fable_modules",
      "./images": "/images"
    },
    "useSSL": true,
    "watchConfig": {
        "directories": [ "./out", "./images" ]
    },
    "liveReload": true
  }
}

I'm mounting the physical out folder on the root and then i"m pointing my src="./App.js" of my data-entry-point to it.
Expected behaviour
It would be nice if there was some validation of whether the file passed to esbuild exists.
If it doesn't, it could look into the physical folder (out in my case) that is mapped to the root of the dev server.
Desktop (please complete the following information):

OS: Windows

Additional context
Add any other context about the problem here.
Mounting "./out": "", doesn't appear to include any subfolders. This might be a different bug or expected behaviour, I'm not sure.

if you switch the path of your script to
<script data-entry-point type="module" src="./out/App.js"></script>
does it work?

Yes, that works for build but breaks during serve.

It would be nice if there was some validation of whether the file passed to esbuild exists.
If it doesn't, it could look into the physical folder (out in my case) that is mapped to the root of the dev server.

Yeah, I think that's a good thing to check before starting the build

Yes, that works for build but breaks during serve.

What version are you on? I had an issue to mount directories on the root but it was for *nix systems and fixed in 0.22.1

Mounting "./out": "", doesn't appear to include any subfolders. This might be a different bug or expected behaviour, I'm not sure.

I'll have a look this week and see what I can do about it but it should mount everything under out, not just the first level

What version are you on?

I'm on 0.22.1
Thanks for the quick response.

I have you a workaround in case you can actually work with "out" in the urls, testing with this config:
{
  "fable": {
    "autoStart": true,
    "project": "./src/App.fsproj",
    "extension": ".js",
    "outDir": "./out"
  },
  "devServer": {
    "mountDirectories": {
      "./out": "/out"
    }
  }
}

<!-- assuming that your sass script outputs to ./out/styles.css -->
<link rel="stylesheet" data-entry-point href="./out/styles.css" />

<script data-entry-point src="./out/Main.js" type="module"></script>

that should make it possible to both build and serve in this case "out" works as the usual "src" file so perhaps
the reason your directories were not working reclusively is due the watchConfig option that you added. That config setting is very fragile and can break all of the detection I was planning to switch that eventually but haven't done anything there

the reason your directories were not working reclusively may be due the watchConfig option that you added. That config setting is very fragile and can break all of the detection I was planning to switch that eventually but haven't done anything there

Not really sure I follow there, I would expect these to be different settings for different goals.
Your workaround will work but I've raised a PR with how I'd like to see it fixed.
If you agree I can finish it up.

Not really sure I follow there, I would expect these to be different settings for different goals.

In dev we use a FileSystemWatcher which is a very limited API that doesn't support globbing patterns so one has to almost list by hand what do you want it to watch but you are right looking at the code it might have been something else.

Your workaround will work but I've raised a PR with how I'd like to see it fixed.
If you agree I can finish it up.

Yes I don't mind, feel free to send it and I'll gladly merge and publish

The new version has been released! Once again thanks for this contribution
https://www.nuget.org/packages/Perla/0.23.0
https://github.com/AngelMunoz/Perla/releases/tag/v0.23.0


Create sufficient logging.

Make sure to log all relevant event's for all endpoints.

Check if it's possible to set the log output destination to a log file url set in the config.yml file.
If not possible, set the log output destination to standard out.


AttributeError: 'tuple' object has no attribute 'permute'

May I use train again_ Gan_ Social_ LSTM_ The following errors occurred during the training of the MHSA model：
Traceback (most recent call last):
File "/mnt/f/pycharm-professional/pytorch-project/VectorNet-GAN/mapfe4mp-master/train.py", line 143, in
model_trainer(config, logger)
File "/mnt/f/pycharm-professional/pytorch-project/VectorNet-GAN/mapfe4mp-master/model/trainers/other/trainer_gan_social_lstm_mhsa.py", line 271, in model_trainer
losses_d = discriminator_step(hyperparameters, batch, generator,
File "/mnt/f/pycharm-professional/pytorch-project/VectorNet-GAN/mapfe4mp-master/model/trainers/other/trainer_gan_social_lstm_mhsa.py", line 489, in discriminator_step
pred_traj_fake = relative_to_abs(pred_traj_fake_rel, obs_traj[-1])
File "/mnt/f/pycharm-professional/pytorch-project/VectorNet-GAN/mapfe4mp-master/model/datasets/argoverse/dataset_utils.py", line 394, in relative_to_abs
rel_traj = rel_traj.permute(1, 0, 2)
AttributeError: 'tuple' object has no attribute 'permute'
Excuse me, class TrajectoryGenerator (nn. Module): Is this part of the code correct and why is it returning pre d_ Traj_ Fake_ Rel returns a tuple, but tuples cannot use relative_ To_ ABS function

Hi. I am modifying the repo to have a stable version. Thank you!


Fix #161. Update Interpreting CircleCI Results wiki to GitHub Actions

@gp201 This fixes #161
Is there any way I can access the GitHub actions job workflow diagram? Because I haven't removed the CircleCI job workflow diagram yet.

@gp201 PTAL

@gp201 PTAL

@gp201 PTAL, thanks.

@deadex-ng Just a reminder, please make sure to assign folks when you are asking for a review.

@gp201 and @Ash-2k3 I was talking to Chris about this issue last Sunday. He said I could just add an image of the logs.
I have added that and PTAL.

Thanks @deadex-ng the changes LGTM. I think you would also want to update https://github.com/oppia/oppia-web-developer-docs/blob/c88044ec40b924c8787afb8068b1c8b99eacb40f/Debugging.md?plain=1#L14C66-L14C66, with the link to new wiki page you created. Ping me once you are done with it, so that we can merge it.

@Ash-2k3 PTAL


Gumshoe IE 11/Edge issues - undefined

Hello fellow Devs,
I have been implementing Gumshoe onto a project at work and found that its functioning across all browsers bar IE (surprise surprise!) and im wondering what im doing wrong, had a look through the documentation and can see that it is supported, however when i run the script at the end of my document to Initialize gumshoe, it flops with 'gumshoe' is undefined, even though the script is loaded on the page (tried it in the head and footer etc) and i can see this in my network tab with a 200 response (also again, its working on other browsers, fully functional as expected).
Im running this on Internet Explorer 11/Edge, and simply making the call to gumshoe.init() at the end of my page when the document is ready.. any thoughts on the issue?
Thanks in advance!

any chance you're in a different scope than gumshoe was loaded in?
I'd inspect the window object to see if gumshoe is attached to that prior to calling gumshoe.init.
And as an alternative, you can feed Gumshoe a Promise. Here's how we init gumshoe. See the inline comment I left there for you:
  /**
   * Initializes this module.
   *
   * @public
   * @method init
   */
  function init (opts) {

    options = _.extend(options, opts || {});

    var setup = function setup () {

      if (!window.gumshoe) {
        exception('Gumshoe: window.gumshoe is undefined. setup() cannot continue and Gumshoe will not send events.');
        return;
      }

      // The script is added to the page in dom.chassis/templates/bottom.handlebars
      log.info('Using Gumshoe v' + window.gumshoe.version);

      transport.init(options);

      // tell gumshoe to use our transport
      window.gumshoe({ transport: 'gilt-svc-event' });

      // send the pageView event - all pages will send this event.
      send('page.view');
    };

    // @Amckenzie93 - if gumshoe exists on window, move ahead.
    // otherwise, use a Promise (we're using when.js) and setup after
    // gumshoe has signaled it's ready. Gumshoe recognizes the promise
    // if one is setup. this helps avoid race conditions.
    if (window.gumshoe) {
      setup();
    }
    else {
      window.gumshoe = { ready: when.defer() };

      when
        .all(window.gumshoe.ready)
        .then(setup);
    }
  }


Wr/data bugs

What does this PR do?
fixes various bugs with data
lots of the QA steps will be found in the associated WI
What issues does this PR fix or reference?
@W-10290520@
@W-10290524@
@W-5380834@

QA notes
force:data:soql:query -t -q "select id, Metadata from RemoteProxy"
✅ has valid output
same, but with the description field being invalid json "description": "Can I nest some {invalid JSON} here"
✅ has valid output (treats the invalid json as string)
same, but the description field is actual json "{"foo":"bar"}"
✅ escapes it as JSON "description": "\"{\"foo\":\"bar\"}\""
same, but with nested JSON and no outer quotes {"foo":"bar", "nested": {"a": 15}}
✅ fully escaped "description": "{\"foo\":\"bar\", \"nested\": {\"a\": 15}}"

force:data:soql:query -q "SELECT Amount, Id, Name, (SELECT Quantity, ListPrice,PriceBookEntry.UnitPrice, PricebookEntry.Name,PricebookEntry.product2.Family FROM OpportunityLineItems) FROM Opportunity" with setup per WI
✅ 👎🏻 that's an awful looking table, BUT it does have all the values