Error 'Maximum call stack size exceeded' when i called player.src('xxx')

Browser: Chrome 64.0.3282.186
It is ease to occur, just call:
player.src('https://d2zihajmogu5jn.cloudfront.net/bipbop-advanced/bipbop_16x9_variant.m3u8')

or call:
player.dispose();

And you cloud reshow this bug in the example of master branch.

Smells like an error I get when updating webpack.config. Have you try rebuilding the dist. I'll check if I forgot to commit the correct dist.

I had try to rebuild the dist. It is not work. And I exec npm run dev in the example, browsing http://localhost:8080/example/index.html, the error still occurred.

I'll investigate this when I have access to my work PC.

This PR should fix your issue https://github.com/streamroot/videojs5-hlsjs-source-handler/pull/58
Nice catch 😉

Thanks, it work!


fix: fix invalid parameter error in ChannelListProvider

External Contributions
This project is not yet set up to accept pull requests from external contributors.
If you have a pull request that you believe should be accepted, please contact
the Developer Relations team<EMAIL_ADDRESS>with details
and we'll evaluate if we can setup a CLA to allow for the contribution.
For Internal Contributors
<TICKET_ID>
Description Of Changes
Add a brief description of the changes that you have involved in this PR
Types Of Changes
What types of changes does your code introduce to this project?
Put an x in the boxes that apply_

[ ] Bugfix
[ ] New feature
[ ] Documentation (correction or otherwise)
[ ] Cosmetics (whitespace, appearance (ex) Prettier)
[ ] Build configuration
[ ] Improvement (refactor code)

I don't know whether this PR will be accepted or not, but I was facing this Invalid Parameter bug in ChannelListProvider. So I started debugging and found this potential bug. Was not able to throughly test it, still raised the PR as it seems to be a more of a typo. Please review it. @sravan-s

@banerjeesouvik Thanks for the contribution, will include in next release

Thank you @banerjeesouvik we are going to apply it through this PR
https://github.com/sendbird/sendbird-uikit-react/pull/252


fix cdn check for node

@thomasjball brought it to my attention that mkc was failing with a weird error when trying to fetch github repo dependencies. turns out it was crashing on this code because window isn't defined in node.
this will need to be ported over to micro:bit stable

actually, might not need to go to stable. not sure if this code ever got there in the first place


Add CC0 License to repo

In response to #5

Add CC0 License to repo. Try to make explicit the intentions of original author of this code.


Add CONTRIBUTING and LICENSE files

Resolves: https://github.com/klutchell/unbound-docker/issues/18

If you were interested in contributing the next Unbound release, would this guide help @churchofnoise ?

If you were interested in contributing the next Unbound release, would this guide help @churchofnoise ?

Will look into it, and definitely interesting in contributing!


Added blurple color

Relevant Issues

Closes #1123
Description
 Added an entry for `blurple`.
Did you:

[X] Join the Python Discord Community?
[X] Read all the comments in this template?
[X] Ensure there is an issue open, or link relevant discord discussions?
[X] Read and agree to the contributing guidelines?

As per my message on Discord (sending here for posterity sake), would be nice to also add og_blurple (#7289DA) as that's what the pydis branding uses.


[TEST] Add integration test for training script

Do we want the tests to run in a dev version of the code only (a clone of the git repo), or should they run for pip installs too?

I think dev version is fine. Would consider the walkthrough as a more approachable test a user can carry out.

The reason I ask is because if we want the tests to run when the code is pip installed, we would need to add example/ to the MANIFEST.in (or pyproject.toml equivalent config setting) so they are copied to pythonX/site-packages/wandb_preempt/. Not sure if we should do that anyway.

Maybe the best place for the example is inside the docs/ folder?

I think its good to have it in a separate dir from docs

Feel free to merge or propose further changes


Package reorganization

Reorganized and simplified target related package and module structure. Should make it easier to add new targets by reducing the number of files to edit. Separates framework modules from target-specific modules.

Moved target, coresight_target, and memory_map to pyOCD.core.
Moved semihost and svd to pyOCD.debug.
Moved flash_cortex_m, flash_kinetis, and target_kinetis to pyOCD.target.family.
Merged target-specific flash modules into target modules.
Renamed target modules with CMSIS-Pack device names, unless the target does not have a pack available.

@mbed-bot: TEST

cortex_test was failing for me due to the problem solved by #278. But it's strange that, for instance, #279 passed even though it doesn't have that fix. Not sure what's going on there.

Those two PRs have been merged, so if you rebase this I can give testing another try.

@c1728p9 Rebased.

@mbed-bot: TEST

@c1728p9 Thanks for rerunning the tests. I'm really curious why the cortex_test is failing.

It looks like its related to the software breakpoints:
 16:49:30 ------ Testing Software Breakpoints ------
16:49:30 Installed software breakpoint at 0x20000001
16:49:30 Exception 'module' object has no attribute 'target' when testing board<PHONE_NUMBER>20312054434a38313037313830303097969903
16:49:30

Thanks for looking into it!

That commit should fix it. The problem was a reference to pyOCD.target.target.Target that moved to pyOCD.core.target.Target, in cortex_test itself.

@mbed-bot: TEST


Add new environment variable for test iteration identifier

This PR introduces a new environment variable, TMT_TEST_ITERATION_ID, which is accessible to the test script. This variable combines a unique run ID and the test's serial number. The value of this variable should be different with each new test execution.
Currently, I use a timestamp for the unique run identifier, saved when the run is initialized. I am not sure if this is the best solution, due to its length, so I'd appreciate your feedback on that. Maybe some random string, or even the run ID from the workdir itself (run-XXX) would be better.
Resolves: #2632
Pull Request Checklist

[x] implement the feature
[x] write the documentation
[ ] extend the test coverage
[ ] update the specification
[ ] adjust plugin docstring
[ ] modify the json schema
[ ] mention the version
[ ] include a release note

Currently, I use a timestamp for the unique run identifier, saved when the run is initialized.

The timestamp would be probably safe enough, although there is a risk that two tmt runs can easily have the same "unique" ID. On the other hand, it's very unlikely they would overlap in terms of guests...

I am not sure if this is the best solution, due to its length, so I'd appreciate your feedback on that. Maybe some random string, or even the run ID from the workdir itself (run-XXX) would be better.

An obvious choice would be https://docs.python.org/3/library/uuid.html#module-uuid, probably https://docs.python.org/3/library/uuid.html#uuid.uuid4 to be specific. I would avoid the workdir itself, that one may easily conflict or be puzzling - a meaningless string seems like the best choice to me. Timestamp might be a good choice for now, and we can tweak it later if needed.

I confirm. Thank you.

Looks good, thanks for implementing this! Looking at the pull request checklist I'd suggest to cover two more items: include the version in the doc and mention the new environment variable in the release notes.

Both added in b85e52006a010100bbbb75de0f60eff974315223.


feat: add customProperties to endPaint to do custom reporting

This PR addresses issue https://github.com/Zizzamia/perfume.js/issues/68, by providing a way to log custom properties from within Perfume

Because it might be going to conflict with https://github.com/Zizzamia/perfume.js/pull/70, I need to first change
log(metricName: string, duration: number)
sendTiming(metricName: string, duration: number)

to
log({ metricName: string, duration: number, gaTimingVar: string, customProperties: any })
sendTiming({ metricName: string, duration: number, customProperties: any })

which will support more custom requirements and future PRs.
Sounds good?

This feature will be available with v4.4.0.
Really appreciated the feedback 🔥🔥🔥

Cool stuff!
On Sat, Nov 30, 2019, 8:28 PM Leonardo Zizzamia<EMAIL_ADDRESS>wrote:

Closed #69 https://github.com/Zizzamia/perfume.js/pull/69.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Zizzamia/perfume.js/pull/69?email_source=notifications&email_token=AA7GZM5O3T5HBO3TE46LBY3QWM4NPA5CNFSM4HOF5YG2YY3PNVWWK3TUL52HS4DFWZEXG43VMVCXMZLOORHG65DJMZUWGYLUNFXW5KTDN5WW2ZLOORPWSZGOVF3PMZY#event-2843145831,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AA7GZMZKGDGKISSTFAAUTNDQWM4NPANCNFSM4HOF5YGQ
.


Graph file doesn't exist

I can't find VGG_Origin model file in the repository . Instead of .pb file I can only see a shell file there. Can someone please tell me how to load the model.
The code works with mobilenet model though.

@immkapoor you can download from here: https://pan.baidu.com/s/1XT8pHtNP1FQs3BPHgD5f-A#list/path=%2Fsharelink1864347102-902260820936546%2Fopenpose%2Fopenpose%20graph%20model%20coco&parentPath=%2Fsharelink1864347102-902260820936546

@saimj7 Thank you

The link is invalid. Can you republish it

The link is invalid. Can you republish it


Node 0.10.40 is required by Meteor 1.2

Meteor 1.2 apps can break if using node 0.10.36. Changing mup to use 0.10.40. Meteor 1.0-1.1 apps should experience no problems.

I just modified the node version in mup.json and it works.


Disambiguate readme wording

Looks good, but the repo is configured to run prettier on markdown so it's failing out. Should just have to run the yarn target and re-push

That's what I get for writing in the Github interface.


remove extra readme added in #1650

Sorry, this one was my fault. Just didn't see the extra zero-length file when reviewing!

[x] Installs cleanly on a fresh build of most recent master branch
[x] Upgrades cleanly from the most recent release
[x] Updates documentation relevant to the changes

Cancelled the ci build


Test functions on custom test source sets are treated as non test functions

Expected Behavior
Any test function should allow to use backticks and spaces even if it is not on an "official" test source set
Observed Behavior
I have created custom source sets for different type tests, testIntegration and testFunctional and the functions reported on those source sets are marked by detekt, for example:
@Test
fun `android configuration cache clean v1_0_0`() { // marked by detekt
    gradleTestKitTest("gradle-features/android configuration cache clean v1_0_0") {
        beforeTest()
        testConfigurationCache(expectTaskOutcome = TaskOutcome.SUCCESS)
    }
}

Steps to Reproduce
The repo with the function mentioned above can be found here

I know it is not a good solution, but Detekt should check if a source set contains the word test or Test in order to apply some rules or others as, I think, it is impossible to know if a custom source set or configuration is for tests or not.

I asume that you are using the default configuration. There that rule have this configuration:
    excludes: ['**/test/**', '**/androidTest/**', '**/commonTest/**', '**/jvmTest/**', '**/androidUnitTest/**', '**/androidInstrumentedTest/**', '**/jsTest/**', '**/iosTest/**']

As you can see functionalTest is not listed there so it doesn't apply. You can just add it and everything should work as expected.

Since there is nothing actionable here, I close this issue. If not, please leave a comment.

I missed this notification.
I think the rule should find for *Test instead of hardcoding some of the multiplatform targets.
@schalkms @BraisGabin should I open a different issue or this is valid for this feature request?

@JavierSegoviaCordoba If #5412 does not already fit for your use case, please add your thoughts to this issue.

@andersfischernielsen to my knowledge, there is no merging but rather a overriding mechanism with buildUponDefaultConfig = true.

That's a shame - I was hoping we could extend the existing excludes rather than replace them. But thanks nonetheless!

@andersfischernielsen related discussion, which essentially proposes a solution to your scenario.
https://github.com/detekt/detekt/issues/5977#issuecomment-1522218952


Differing units between Windows and darwin

I took a look at how darwin is getting the disk info, and see that it uses df -P, which I believe outputs in counts of 512 bytes.
Comparing this with what Windows is doing with wmic, I think Windows outputs in plain bytes. This should be properly documented if this is really the case.
For context, I've experienced false positives on darwin in a platform-agnostic app.

Interesting.
I will check that this weekend.
thanks for the report.

i just looked intoo it and you are right.
This is also true for linux.
Should we all base on 512 bytes like linux / darwin or in bytes like windows ?

I feel basing in bytes like Windows feels more natural.
As long as the unit is properly documented and honored on all platforms then either option is fine.


add support for pointer events

add support for pointer events
https://developer.mozilla.org/en-US/docs/Web/API/Pointer_events
related to: https://github.com/purescript-halogen/purescript-halogen/pull/794

I would be happy if web-pointerevents could be moved into purescript-web. I can help anyway.

I have invited you to purescript-web. Once you accept the invitation, can you try transferring the repo?

https://github.com/purescript-halogen/purescript-dom-indexed/releases/tag/v11.0.0


Simultaneous edits to an issue/PR lead to data loss

Description
TL;DR: When two people edit an issue/PR description simultaneously, the last one to save "wins".
This scenario is not uncommon:

User files a bug report.
Developer sees the report, and wants to edit it to clarify some wording or clean up some markdown.
User sees a typo, and edits the report.
Developer knows the system, so is faster, and saves their edits first.
User saves their typo fix
Now all the edits done by the developer are reverted by the edit from the user.

This problem is quite serious, because:

There is nothing at all that warns about overwriting someone else's changes.
There are no historical comments "user X changed the description".
There is no way to see the history of description changes.

So not only is there data loss, it remains undetected until someone revisits the page and actually notices their work is gone.
A possible first step could be to use an etag to detect simultaneous edits. Then at least a user can be warned that they'll overwrite someone else's changes.
It would be better to have some way to merge those changes, but that's a whole different issue to solve. I think the most important is to prevent these losses; how to solve such collisions gracefully can be dealt with later.
Gitea Version
00b18ab42fdd49a437249f57c0b9086fd0ee1d03
Can you reproduce the bug on the Gitea demo site?
Yes
Log Gist
No response
Screenshots
No response
Git Version
No response
Operating System
No response
How are you running Gitea?
Self-built from Git.
Database
PostgreSQL

The only solution I can see is to completely refactor the issue edit feature to use AJAX, then various problems could be resolved and it's more friendly to users if something wrong happens.

That would be a good idea (to use the API for more aspects of the web interface), but it is far from the only solution.
A simple solution would be to:

Define a function H() that can compute some etag for the to-be-edited data. In this case, since the description can be edited all by itself, a hash of that description would be good.
Include this hash into the form, and when POSTing the form, send it along in a If-Match: header.
Before saving to the database, call H() and compare it to the value of the If-Match: header. If they do not match, refuse to overwrite the data and re-render the edit template with the submitted data (so their edits are not lost) but with the new etag in there.
The user can open the issue in another tab (Gitea can provide a link with target="_blank" for convenience) to see what the other person did, and manually merge the differences.
A second "Save" can then overwrite; JS can be added to add yet another confirmation that older data should be overwritten.

This doesn't require a big refactor of the edit feature, and does provide an initial step to protect against this hidden data loss.

re-render the edit template with the submitted data

That's the hardest (nearly impossible at the moment) part without AJAX. That's why I proposed AJAX 😂

Hmmm then it looks like an inconvenient decision was made when designing the form -- either do it in JS all the way, or do it the traditional HTTP/HTML way. Sitting somewhere in the middle of those two is troublesome indeed.
In any case, I think this is a high priority issue, as the safety of data in issue descriptions is kind fo central to Gitea. If this can't be solved quickly, maybe some pre-flight request made with JS before the POST happens can be added on short notice? A quick OPTIONS request that can be handled to check for collissions is not perfect, but at least it narrows the window of undetected losses to a few seconds (the time between the OPTIONS and the POST request).

We can have a version column for the issue table, so only one could update that column, and another will fail, but the form content will be kept on his web browser.

@lunny do it server-side (ajax) or dont do it at all.

Thanks for addressing this :)


[TFLITE]Select op support for tflite frontend

Added the support of select op in tflite frontend.
@FrozenGene @masahi please help me to review and merge this PR. TIA
Note: tflite where op is same as select in tflite.
Some part of code taken from #5474

@FrozenGene could you please help to merge this PR. TIA

LGTM. @siju-samuel After thinking about this pr, I think we should make get_expr and exp_tab.new_const be private and only public get_tensor_expr. Other places use get_expr / exp_tab.new_const should be replaced as get_tensor_expr. Wish you could consider this suggestion after this pr.

Thanks @siju-samuel @u99127 merged now.

LGTM. @siju-samuel After thinking about this pr, I think we should make get_expr and exp_tab.new_const be private and only public get_tensor_expr. Other places use get_expr / exp_tab.new_const should be replaced as get_tensor_expr. Wish you could consider this suggestion after this pr.

@FrozenGene Thanks. I will optimize all this and will do a PR.

LGTM. @siju-samuel After thinking about this pr, I think we should make get_expr and exp_tab.new_const be private and only public get_tensor_expr. Other places use get_expr / exp_tab.new_const should be replaced as get_tensor_expr. Wish you could consider this suggestion after this pr.

@FrozenGene, @siju-samuel, Considering this fix as part of this refactoring https://github.com/apache/incubator-tvm/pull/5528.
Please let us know your opinions on other refactoring aspects as well.

@maheshambule Thanks.
You can consider it as part of #5528. .


Rename Class

currently Sbt is not very informative, possibly renaming it to what it actually is would be more useful to the users of this class :)

As you said "Sbt" is not very informative...
I just want that users put few characters on their code to use this.
I found that putting below codes meets my requirement.
"using StrOpe = StringOperationUtil.OptimizedStringOperation;"
( And change "Sbt" to "StrOpe". "StrOpe" is short for ”StringOperation" )

also changed the namespace...


Implement parsing of conditions

Statements such as %if, %else and %endif are currently treated as regular section content.
That can cause issues, especially with conditions spanning across multiple sections - if a section is deleted, modified or moved, it can lead to syntax errors as the condition block could be broken apart.
Figure out how to deal with this.

Still relevant. Let us know if you need this (either in Specfile or for Packit).

I am hitting this when parsing amavis.spec. I get this TB
Traceback (most recent call last):
  File "/home/msuchy/projects/license-validate/./convert-spec-universal.py", line 65, in <module>
    with specfile.tags(section) as tags:
         ^^^^^^^^^^^^^^^^^^^^^^
  File "/home/msuchy/.local/lib/python3.12/site-packages/specfile/context_management.py", line 133, in __call__
    self.values[key] = next(self.generators[key])
                       ^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/home/msuchy/.local/lib/python3.12/site-packages/specfile/specfile.py", line 292, in tags
    tags = Tags.parse(section, context=self)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/home/msuchy/.local/lib/python3.12/site-packages/specfile/tags.py", line 497, in parse
    lines = process_conditions(list(section), macro_definitions, context)
            ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/home/msuchy/.local/lib/python3.12/site-packages/specfile/conditions.py", line 133, in process_conditions
    result.append((line, branches[-2]))
                         ~~~~~~~~^^^^
IndexError: list index out of range


FYI: Characteristic {char_specifier} was not found with old firmware

Documenting in case anyone runs into this:
Found Polar device
Model Number: H10
...
File: bleak/__init__.py", line 769, in write_gatt_char
    raise BleakError("Characteristic {char_specifier} was not found!")
bleak.exc.BleakError: Characteristic {char_specifier} was not found!

This was fixed by updating the firmware (using Polar Flow, Polar Beats did not show the update).

Thanks @ar-jan, I've updated the README to mention the use of Firmware 5.0.0 or later


Add kubernetes_feature_enabled metric to cluster-info dump

Add kubernetes_feature_enabled metric series to track whether each active feature gate is enabled.

https://github.com/kubernetes/kubernetes/pull/112690 was merged in v1.26 cycle.  It can fix partially on the issue https://github.com/kubernetes/kubernetes/issues/87869 and I wondered if we can add the metrics info into kubectl cluster-info dump
Originally posted by @pacoxu in https://github.com/kubernetes/kubernetes/issues/87869#issuecomment-1465626709
kubectl get --raw /metrics  | grep kubernetes_feature_enabled

/kind feature

good idea.

/close
We will let the KEP drive this.


How can I use useDragControls

When using import { useDragControls } from "svelte-motion" it tells me Failed to fetch dynamically imported module<EMAIL_ADDRESS>The only thing I can import is import Motion from "svelte-motion/src/motion/MotionSSR.svelte" but I don't understand why. Can I help with this Repository in any way? Or is it simply not possible?

Interesting, sounds like a problem with vite dev mode. I'll try to replicate the problem.
I'm happy to receive pull requests.

Oh alright, I might look into that. I was unsure if there is any compilations because of the new SvelteKit and Framer simply not handling this library. It uses popmotion to handle all the animations, right?


Check if user is signed into iTunes Store

Question/Feature request:
If a user is reinstalling the app, and on a previous install made non-consumable in app purchases, i would like to attempt to restore the functionality that the user has previously paid for.
I'm currently able to query if the user has previously purchased any extra content. However, if the user is not logged into the iTunes store, they will be greeted with an iTunes login. Although very few users wont be logged in, i really wouldn't want that to happen – my app asking the user to log into the iTunes Store immediately on the first launch would look really fishy.
Ideally i would only attempt to check the previous purchases if the user is already logged into iTunes, and i'm able to check the previous purchases silently.
Is this possible to do using react-native-iap?
Link to question on Stack Overflow
Version of react-native-iap
2.4.0-beta6
Version of react-native
0.56.0
Platforms you faced the error (IOS or Android or both?)
iOS
Expected behavior
Actual behavior
Tested environment (Emulator? Real Device?)
Steps to reproduce the behavior

@augustbjornberg how are you able to query their purchases without prompting them to login? getPurchaseHistory prompts iTunes login even if they're signed in. Is there another method?
And, not that this is a fix, but if they just downloaded your app it will be exceedingly rare that they won't be signed in because they would have to be signed in to download it. Such a rare edge case as to possibly not be worth your time, though I understand where you're coming from. But from what I can see there is no method to check this. Would be interested to hear if you find otherwise.

My case I am not logged in to the device, and in the application I am logged in to an account and I can not loggout

I think this couldn't be fixed since apple doesn't have the solution. See link.


Support call collapsing

We want to support call collapsing as documented here.
The class in Hystrix that supports call collapsing is HystrixObservableCollapser

Hi. Are there any news about this implementation?

Any news about Request collapsing?


Fix git alias conflict with github-cli

Description

Fixed type _command_exists by replacing it with _binary_exists
Motivation and Context

Fixes #1893
How Has This Been Tested?

Tested on local system with sourcing
Screenshots (if appropriate):
Before

After

Types of changes

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[x] My code follows the code style of this project.
[x] If my change requires a change to the documentation, I have updated the documentation accordingly.
[x] I have read the CONTRIBUTING document.
[x] If I have added a new file, I also added it to clean_files.txt and formatted it using lint_clean_files.sh.
[x] I have added tests to cover my changes, and all the new and existing tests pass.

The thing is, you are probably correct we should check for the binary, but the current syntax is also correct, if gh is already aliases/defined, maybe we do not want to re-alias it

@NoahGorny Remember once I had a problem with github-cli binary being overridden by alias.
https://github.com/Bash-it/bash-it/issues/1774
Maybe we have missed out (intentionally or due to workload)  this alias

@NoahGorny Ping

Seems like the real question in your setup is:
How is _command_exists gh failing at this point in the bashit startup, since it appears that it should always be present in your /usr/bin ?
Your "Before" section didn't include a type bh before sourcing git.aliasse.bash

I got your point now after running this example in the terminal. _command_exists is the right thing here


Fix cgpbigwig_bamcoverage step and update workflow example of rnaseq.cwl

This PR has two updates. I finally confirmed that the current rnaseq.cwl is now working with these updates.
(1) Fix definitions/tools/bam_to_bigwig.cwl as a subsequent update of #1007
The current rnaseq.cwl was broken in the cgpbigwig_bamcoverage step, because of invalid MarkedSorted.bai requirement. This PR fixes this broken step.
(2) Update rnaseq.cwl workflow example for the issue #958
The previous workflow example was outdated. I updated the inputs YAML file, and I confirmed that this workflow example for rnaseq.cwl is perfectly working now.

@sridhar0605, could you take a look at this PR? As we discussed, I fixed the broken rnaseq.cwl workflow, and it is now working with the updated workflow example.

Thank you for this review, Chris. I got your point to use built-in example datasets in the future, but not from gscmnt. I am going to merge it, once I discuss with @sridhar0605. He is working on for the rnaseq workflow and I am harmonizing with his work.


Can we replace GYB with macros in Swift to enhance expressiveness?

Since Swift now supports macros, can we consider replacing GYB with macros to achieve the same capabilities? This would make Swift more 'Swift'.

Why have you closed this?

Why have you closed this?

My original intention was to tidy up some issues I proposed that have been left undiscussed for a long period. I assumed that the community might not consider these as good ideas.

Could you elaborate on what you meant by replacing GYB with macros? If you were referring to the metaprogrammed standard library Swift sources, then I think we can leave this open as a task, but I imagine we might eventually want to generate those using the swift-syntax SyntaxBuilder library rather than macro-expanding the declarations.

Yes, what I want to replace is the GYB in the metaprogrammed Swift standard library sources.
Additionally, using the swift-syntax SyntaxBuilder would be a good idea.

gyb does straight textual substitution, and as such, cannot easily be replaced with macros in general.


feat(offer): refinement and implementation of new features

What does it do
Describe the technical changes you did.
Usage
If applicable, include a small code snippet in which you demonstrate how to use the new feature.
Why is it needed
Describe the issue you are solving.
Related issue(s)/PR(s)
Add any related issue/pull request here.
Additional context
Add any other context about the problem here.

@AdrianAlzamoraG Is this ready for review? If it is, request my review.

@dalbitresb12 not yet, I managed to fix that the chekbox list appears but you can only select all of them and not each one. Also when I want to apply filter it doesn't work and now the table loading icon is not shown anymore.

Let's merge this so that I can rewrite the component using <script setup> and maybe this will fix most of the bugs. What do you think? @AdrianAlzamoraG

I agree with your idea @dalbitresb12 you can proceed with the changes.


Should be able to pass in a string for live_start_date

this use to work, then broke when improving the timezone stuff
Basically any valid/parsable time string should be valid.  any string that 'just works' when passed to pd.Timestamp() is probably a reasonable bar

Definitely.


Application destroys when Playing a video using SurfaceView in Flutter when a call attended

The problem is Screen record Blocking used by surface view destroying the app in android 9.Flutter doesn't support secure flag for video record blocking. So, I used surface view instead of it. But, the problem is, if a call received at the time of playing a video on the player the app being restarted when going back to the player after the call disconnected. But, when I disabled the surface view it works well. But, this problem doesn’t happen on android 10 devices.Also, I've used exoplayer for video playback.
Steps to Reproduce.

Play video
Call to the device from any other device
Attend the call and disconnect the call

This is a sample repo to reproduce the issue
https://github.com/suhail-zartek/flutter_player_issue
Logs
Doctor summary (to see all details, run flutter doctor -v):
[✓] Flutter (Channel stable, 1.20.2, on Mac OS X 10.15.6 19G73, locale en-IN)
[!] Android toolchain - develop for Android devices (Android SDK version 30.0.1)
! Some Android licenses not accepted.  To resolve this, run: flutter doctor --android-licenses
[✓] Xcode - develop for iOS and macOS (Xcode 11.6)
[✓] Android Studio (version 4.0)
[✓] Connected device (1 available)
! Doctor found issues in 1 category.

Hi @suhail-zartek
Does this happen only Android 9 devices? Have you tried other devices with different Android versions such as Android 10?
Can you please run the app in debug mode by executing flutter run --verbose while the cable is connected pickup call and provide complete logs with the crash
Thank you

@TahaTesser I've checked on android 8,9,10. This not happening in android 10. Also, it doesn't happen on every phone even in android 8 or 10. Few of them faced this issue. One of the devices which faced this issue was redmi 6 pro.

Hi @suhail-zartek
Thanks for confirming it's a device-specific issue, Can you please run the app in debug mode by executing flutter run --verbose while the cable is connected pickup call and provide complete logs with the crash to better address the issue
Thank you

Without additional information, we are unfortunately not sure how to resolve this issue.
We are therefore reluctantly going to close this bug for now.
Please don't hesitate to comment on the bug if you have any more information for us; we will reopen it right away!
Thanks for your contribution.


Incompatibility With NL Apparel Paper Pattern

Hugs Log.txt

Looking at it I dont think I have any time to add compatibility with such a mod. I think they will have it easier fixing it on their end as this mods size-definitions is a public property that can be used simply on their end to set their drawsizes.


Pretty-print for C-set transformations

The default show method dumps all the type information and is therefore very unreadable.

Would something like this be helpful to implement?

CSetTransformation with elements V = 3↦2, E = 2↦3

┌────┬──────┬───┐
│  V │ refl │ → │
├────┼──────┼───┤
│  1 │   1  │ 2 │
│  2 │   3  │ 2 │
│  3 │   2  │ 1 │
├────┼──────┼───┤
│  1 │   2  │ * │
│  2 │   3  │ * │
└────┴──────┴───┘

┌────┬─────┬─────┬───┐
│  E │ src │ tgt │ → │
├────┼─────┼─────┤───┤
│  1 │   1 │   1 │ 3 │
│  2 │   2 │   2 │ 2 │
│  3 │   3 │   3 │ 3 │
│  4 │   2 │   1 │ 1 │
├────┼─────┼─────┤───┤
│  1 │   1 │   2 │ * │
│  2 │   1 │   1 │ * │
│  3 │   2 │   2 │ * │
└────┴─────┴─────┴───┘

That's an interesting idea. I was thinking that for the default show method, we would have a short and simple display mirroring the constructor, something like
ACSetTransformation((V=[...], E=[...]), [dom], [codom])

where [dom] and [codom] are some abbreviated representations of the domain and codomain ACSets, perhaps just showing their sizes.
But then we could also have a show method for MIME type text/plain with richer, tabular display, possibly along the lines you suggest.
This would agree with the conventions on the show method for ACSets themselves (see CSetDataStructures).

The PR above resolves the most immediate problem but I'll this issue open because I like the idea of having a show method for MIME type text/plain along the lines that Kris suggests.


Ax Upcoming Events Section

not sure if it's adding any value at this point - especially since it has been "more information coming soon" since its inception
just a thought

Just did a major refactor with a separate API directory to update events! Hopefully this will help and be more consistent


No confirmation or feedback of correct solution in Movie section

When building the objects in the Movie section, there is no way to test or complete each level.

Playing the movie tests completeness.

There is no play bar on the first game:

The very first level requires no animation, so it will automatically show successful completion as soon as you construct the figure shown (correct shapes in the correct colours in the correct locations).


CRUD for Otomi GitOps

WHY
To provide users and easy way to edit files in git repo.
User Story
AS A A team member
I WANT TO to create and modify and delete manifests in Otomi console
SO I can perform gitops
The values repo is going to have a new manifests directory.
values/
├── env
└── manifests
    └── <namespace>
        └── <resource-type>
            └── <manifest-x>.yaml

The otomi-api and otomi-console will implement CRUD for the /v1/manifests/<namespace>/<resource-type> endpoint
The otomi-api will perform URL path based access control based on team membership E.g.
team-a can perform CRUD on /v1/manifests/team-a/ but not /v1/manifests/team-b/
Acceptance criteria
GIVEN Otomi values repo and Otomi Console
WHEN a user performs CRUD operation on the /v1/manifests/<namespace>/<resource-type> endpoint
THEN the corresponding file env/manifests/<namespace>/<resource-type>/<resource-name> is modified
Definition of done

[ ] Relevant PRs are merged
[ ] Tested by peer
[ ] Updated documentation reviewed by peer
[ ] Short demo video recorded and stored on google drive (if applicable)

Hey team! Please add your planning poker estimate with Zenhub @Ani1357 @dennisvankekem @ElderMatt @ferruhcihan


Implement Response.Reader

Currently, there are a few high-level Response methods to inspect body:

Body - reads the whole body and returns it as String
Text - reads the whole body and returns it as String
Form - reads the whole body and returns it as Object
JSON - reads the whole body and returns it as Value
JSONP - reads the whole body and returns it as Value

All these methods internally use Response.getContent to retrieve body.
Now, we want to add a more low-level method that would allow user to read body manually:
func (r *Response) Reader() io.ReadCloser

This method will return a reader, which can be used to read response body. Unlike the methods listed above, it should be able to work with infinite responses too, and does not place any limitations on response format.
This method will be mutual exclusive with the methods listed above. User should either call Reader or one of those methods.
Here is how it should work:

check if the chain is failed; if it is, it should return nil
check if getContent was already called by checking contentState; if it's not contentPending, then it should report failure and return nil
check if the underlying type of resp.Body is bodyWrapper; if so, it should call bodyWrapper.DisableRewinds, to prevent bodyWrapper from trying to read the whole body into memory, and thus to allow infinite response
set contentState to some new state, say, contentHijacked, which would mean that Response does not own body content anymore
return resp.Body

Also getContent should be modified to report failure if contentState is contentHijacked.
We should also modify Body/Text/Form/etc, so that they will path method name (e.g. "Text()") to getContent, and modify getContent to store this method name. Then, when Response.Reader will report failure that getContent() was already called, it will be able to use that method name to explain user which exact method caused conflict, like this:
cannot call Reader() because Text() was already called

We use the same approach in Request.setType and Request.setBody.
Unit tests should be updated:

we need to add a new subtest to TestResponse_BodyDeferred to cover contentHijacked
we need to add a test for Response.Reader
we need to add tests that checks how Reader and Body/Text/Form/etc are mutually-exclusive

We also need to add integration test to e2e/chunked_test.go, which will stream a large (e.g. a megabyte) response from server to client, and use Response.Reader to retrieve that response.

I'd like to take this one

Thanks!

Looks #246 will be internal client of Response.Reader


maybe(): 1 argument only to be coherent with Array()

https://github.com/laurentpayot/runtime-signature/commit/bea298d664e06b0223f83e4a9b7c8f8087e49a0e


Problem when there is latex in the title of an article

This can happen like here:

link to the article: https://arxiv.org/abs/2106.09428

We could try to detect the presence of $ and then double them and display the result with Kramdown


Markdown support for multiple types of Header, italics, Bold, images, links

an issue always came up when running that relates to line 87 in skilie.py
return pathlib.Path(file_path.split('.')[X]).stem
where X should be a 0 or 1 depending on the parsing
further investigation is needed as a separate potential bug

implements #10


ADD: Generic cosmos txs to perform privileged zEVM system contracts calls via policy group account

There are a few privileged operations with regard to the system contracts on zEVM.
For example, these of ZRC20:  and of system contract itself.
Would need to have the policy account able to make those txs. These txs would need to be cosmos txs that calls these contracts as "FUNGIBLE MODULE" EOA.
It would be tedious to create a separate cosmos txs for each privileged  contract functions. Much nicer to have a generic cosmos tx which could handle all of them.

Closing as it seems no longer relevant with the current state of the project. Feel free to reopen it.


Wind

MMG model is changed to take wind force into account.
data.jl

Added functions for ship superstructure data get_structure_params

mmg.jl

Added wind_force_and_moment_ coefficients to calculate wind force and moments coefficients based on Fujiwara's formula (https://www.jstage.jst.go.jp/article/jjasnaoe/2/0/2_0_243/_article/-char/ja/).
Add time-series data of wind speed and direction u_wind_list,ψ_wind_list to the arguments of existing functions.

ShipMMG.jl

Changed due to changes in data.jl,mmg.jl

test_mmg.jl

Changed due to the changes in data.jl,mmg.jl,ShipMMG.jl

Other changes

Change Psi from uppercase to lowercase (Ψ→ψ)

OK. I will show the example implementation of using Mmg3DofWindForceMomentParams in this branch.

@hinata235
Before my updating, please check and update the test code.
If the test code runs, some error occurred.
Following command shows the example for running test code.
julia > ] # package mode
(@v1.9) pkg> activate .
(ShipMMG) test

@taiga4112
I ran the above test code and no error occurred in my case.
Here is my operating environment.
It may be due to the difference in Julia's ver.
Julia> versioninfo()
Julia Version 1.8.2
Commit 36034abf260 (2022-09-29 15:21 UTC)
Platform Info:
OS: macOS (arm64-apple-darwin21.3.0)
CPU: 10 × Apple M1 Max
WORD_SIZE: 64
LIBM: libopenlibm
LLVM: libLLVM-13.0.1 (ORCJIT, apple-m1)
Threads: 1 on 8 virtual cores

I am not interested in your PC's execution environment.
Anyhow,  I cannot find the Mmg3DofStructureParams params in the test code from your GitHub PR branch. Because of this reason, I can get the following error.
mmg.jl KVLCC2_L7 turning: Error During Test at /home/mitsuyuki/Dropbox/workspace/dev-ShipMMG/ShipMMG.jl/test/test_mmg.jl:3
  Got exception outside of a @test
  MethodError: no method matching mmg_3dof_simulate(::Mmg3DofBasicParams{Float64}, ::Mmg3DofManeuveringParams{Float64}, ::StepRangeLen{Float64, Base.TwicePrecision{Float64}, Base.TwicePrecision{Float64}, Int64}, ::Vector{Float64}, ::Vector{Float64}, ::Vector{Float64}, ::Float64; u0::Float64, v0::Float64, r0::Float64)

  Closest candidates are:
    mmg_3dof_simulate(::Mmg3DofBasicParams, ::Mmg3DofManeuveringParams, ::Mmg3DofStructureParams, ::Any, ::Any, ::Any, ::Any, ::Any; u0, v0, r0, x0, y0, ψ0, ρ, algorithm, reltol, abstol)
     @ ShipMMG ~/Dropbox/workspace/dev-ShipMMG/ShipMMG.jl/src/mmg.jl:543

  Stacktrace:
    [1] macro expansion
      @ ~/Dropbox/workspace/dev-ShipMMG/ShipMMG.jl/test/test_mmg.jl:16 [inlined]
    [2] macro expansion
      @ /opt/julias/julia-1.9.2/share/julia/stdlib/v1.9/Test/src/Test.jl:1498 [inlined]
    [3] top-level scope
      @ ~/Dropbox/workspace/dev-ShipMMG/ShipMMG.jl/test/test_mmg.jl:5
    [4] include(fname::String)
      @ Base.MainInclude ./client.jl:478
    [5] macro expansion
      @ ~/Dropbox/workspace/dev-ShipMMG/ShipMMG.jl/test/runtests.jl:9 [inlined]
    [6] macro expansion
      @ /opt/julias/julia-1.9.2/share/julia/stdlib/v1.9/Test/src/Test.jl:1498 [inlined]
    [7] top-level scope
      @ ~/Dropbox/workspace/dev-ShipMMG/ShipMMG.jl/test/runtests.jl:7
    [8] include(fname::String)
      @ Base.MainInclude ./client.jl:478
    [9] top-level scope
      @ none:6
   [10] eval
      @ ./boot.jl:370 [inlined]
   [11] exec_options(opts::Base.JLOptions)
      @ Base ./client.jl:280
   [12] _start()
      @ Base ./client.jl:522

Why can your test code run without error? It's quite strange for me...

@taiga4112
It's my mistake. I had the wrong branch. I ran the test again on the correct branch and confirmed the same error.
I will modify test_mmg.jl and commit again.

#54

@hinata235
Still there are many errors after running your test code.
Please send PR before you check your code by running test code.

@hinata235
This page shows the result of running your test code on GitHub server.
Please check it.

@hinata235
This PR is one of the implementation of Mmg3DofWindForceMomentParams instead of Mmg3DofStructureParams. Please check it.
The point of this PR is to separate between the definition of core parameters for calculating wind force & moment and the estimation method of this core parameters.


fix: address crowdin issues

Checklist:

[X] I have read freeCodeCamp's contribution guidelines.
[X] My pull request has a descriptive title (not a vague title like Update index.md)
[X] My pull request targets the main branch of freeCodeCamp.
[X] I have tested these changes either locally on my machine, or GitPod.

Closes #XXXXX

Resolves issues brought up in Crowdin.

Congrats on your first pull request (PR)! 🎉
Thank you, for your contribution to the page! 👍
We are happy to accept these changes, and hope you will consider providing future contributions. 📝


chore: revert temporary disable graphql and core contracts check

coverage: 76.089%. remained the same
when pulling a79733535c90fda4edafcf0bd89f52f7c2a8fa15 on feat-revert-temporary-disable-graphql-and-core-contracts-check
into c216cd29949213f9c3732110bd57ce2cbc2e28f4 on development.


Create a standard library

Standard exploding/imploding dice
BoL
ORE

Most can be salvaged from java-legacy.

exploding, BoL: d46f60a680ac1a6ad6c3f1719a97c8bf19800f4f

Closing. Open an issue for each additional library.


AssetIndex: add support for QuerySpec filtering

What this PR changes/adds
Adds support for QuerySpec filtering and AssetSelectorExpression, i.e. filtering on Asset properties.
Why it does that
This is necessary for generating and validating ContractOffers.
Further notes

Currently not all operators support prepared statements, e.g., the IN operator.
sorting is not yet implemented

Linked Issue(s)
Closes #1014
Checklist

[x] added appropriate tests?
[x] performed checkstyle check locally?
[x] added/updated copyright headers?
[x] documented public classes/methods?
[x] added/updated relevant documentation?
[x] added relevant details to the changelog? (skip with label no-changelog)
[x] formatted title correctly? (take a look at the CONTRIBUTING and styleguide for details)

Codecov Report

Merging #1021 (6ab94c4) into main (3305340) will increase coverage by 0.07%.
The diff coverage is 86.53%.

@@             Coverage Diff              @@
##               main    #1021      +/-   ##
============================================
+ Coverage     63.10%   63.17%   +0.07%
- Complexity     2597     2632      +35
============================================
  Files           638      640       +2
  Lines         13862    13973     +111
  Branches        971      980       +9
============================================
+ Hits           8748     8828      +80
- Misses         4666     4694      +28
- Partials        448      451       +3

Impacted Files
Coverage Δ

...aspaceconnector/sql/asset/index/SqlAssetIndex.java
82.03% <86.27%> (+3.27%)
:arrow_up:

...ector/sql/asset/index/PostgresSqlAssetQueries.java
93.33% <100.00%> (ø)

...spi/types/domain/transfer/ProvisionedResource.java
87.50% <0.00%> (-12.50%)
:arrow_down:

...sfer/core/transfer/TransferProcessManagerImpl.java
71.29% <0.00%> (-2.07%)
:arrow_down:

...connector/transfer/core/CoreTransferExtension.java
0.00% <0.00%> (ø)

.../transfer/core/provision/ProvisionManagerImpl.java
100.00% <0.00%> (ø)

...r/spi/types/domain/transfer/ProvisionResponse.java
0.00% <0.00%> (ø)

...handlers/AddProvisionedResourceCommandHandler.java
0.00% <0.00%> (ø)

...provision/azure/blob/ObjectStorageProvisioner.java
100.00% <0.00%> (ø)

... and 7 more

Continue to review full report at Codecov.

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

Why don't force a Criterion object to have a Collection as the right operand instead of a string and then build the subSelect for the "in" criterion as "leftoperand IN (?, ?, ? ..." with a question mark for every collection item.
This will probably avoid the sql injection issue from the bottom. (because, doing so, every operator will support prepared statement operators)

@ndr-brt there is already an issue to improve the Criterion API (#323), which could be used to tackle that.
As an intermediate fix, we could use a List as rightOperand - the Criterion would support that already, since it uses Objects. However, the store would then explicitly have to check for that, which seems like a fair trade-off. WDYT?

@paullatzelsperger the issue is known so no hurry with this, we can address it to #323

@ndr-brt I saw you already approved, but in the meantime I added handling for IN operators in this commit. Care to take a look?


adding more details on status template in doc

Pull Request Test Coverage Report for Build 5347

0 of 0   changed or added relevant lines in 0 files are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage decreased (-0.007%) to 89.219%

Files with Coverage Reduction
New Missed Lines
%

broker/data-access-layer/cf/src/TokenIssuer.js
1
98.55%

Totals

Change from base Build 5345:
-0.007%

Covered Lines:
9572

Relevant Lines:
10358

💛  - Coveralls


on licenses with lots of Devices this command takes to long profile.update!

New Issue Checklist

[x ] Updated fastlane to the latest version
[x ] I have read the Contribution Guidelines

Issue Description
on licenses with lots of Devices this command takes to long between 15-20 minutes
so if I try to added all Devices to each Provision Profiles.
I have hundreds of Provision profiles and it will take 150 hours to do this just to add one new device to each provision profiles. There has to be a better way. Cupertino code could do this in less then 20 minutes. What can I do to fix this so It is faster. I really need to to work.
in the function  def update!
profile = Spaceship::ProvisioningProfile.all(mac: mac?).find do |p|
p.name == self.name # we can use the name as it's valid
end
Complete output when running fastlane, including the stack trace and command used
INSERT OUTPUT HERE

Configuration Files
Please copy the complete content of your Fastfile and any other configuration files you use below:
Fastfile:
# INSERT CONTENT HERE

Environment
fastlane version (fastlane -v):
fastlane 1.66.0
Do you use bundler, rbenv or rvm?

I fine if I command these lines out of spaceship update! function
if you are going to add devices to more then 1 provision profiles
it will speed things up lot. as this call takes 10 -1 5 minutes each time.
so it I have a for loop to add to each provision profiles   then I do another
for loop to download all the provision profiles  after I make this call ONCE
I saved  many Hours
I did a test and it I do it your way with as many devices in my license it will take me 150 hours  to add 1 devices to all my provision profiles and download them all
if I removes those extra calls I can do as thing in 43 minutes
should I create a different function for large Licenses for other people to use?
I still like to know how to speed up this call.
Spaceship::ProvisioningProfile.all(mac: mac?).find
these are the calls you don't want on large license in update!
profile = Spaceship::ProvisioningProfile.all(mac: mac?).find do |p|
p.name == self.name # we can use the name as it's valid
end profile
return profile

This message was created automatically by mail delivery software.
A message that you sent could not be delivered to one or more of its
recipients. This is a temporary error. The following address(es) deferred:
<EMAIL_ADDRESS>Domain zyapps.com has exceeded the max emails per hour (469/400 (117%)) allowed.  Message will be reattempted later
------- This is a copy of the message, including all the headers. ------
Received: from github-smtp2-ext2.iad.github.net ([<IP_ADDRESS>]:40497 helo=github-smtp2b-ext-cp1-prd.iad.github.net)
by serveur.ex2-p10.com with esmtps (TLSv1.2:ECDHE-RSA-AES256-GCM-SHA384:256)
(Exim 4.87)
(envelope-from<EMAIL_ADDRESS>id 1bEKtN-002lci-6O
for<EMAIL_ADDRESS>Sat, 18 Jun 2016 20:22:54 +0200
Date: Sat, 18 Jun 2016 11:22:07 -0700
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=github.com;
s=pf2014; t=1466274127;
bh=RMhRssMpm/sNXpOYImxD8mURArzLU9x4r5mRi4NERfs=;
h=From:Reply-To:To:Cc:In-Reply-To:References:Subject:List-ID:
List-Archive:List-Post:List-Unsubscribe:From;
b=PQBrqiHqg1pUEXA+49FdD+k4JBLx5xN44a8igyLf4gZChrf/vizhQi/v34G8M78e9
SFXfQdNG+6cnTd5KVic+hY7N99B/FAgNBLcY2kLcMAKT1qxAsvX6x4w2yvheMTslkD
66Eq6kDBgdI2mAzrE8GscgwS7bPMx++z3RpWqKK0=
From: fastlane bot<EMAIL_ADDRESS>Reply-To: fastlane/fastlane<EMAIL_ADDRESS>To: fastlane/fastlane<EMAIL_ADDRESS>Cc:
Message-ID<EMAIL_ADDRESS>In-Reply-To<EMAIL_ADDRESS>References<EMAIL_ADDRESS>Subject: Re: [fastlane/fastlane] on licenses with lots of Devices this command
takes to long profile.update! (#3706)
Mime-Version: 1.0
Content-Type: multipart/alternative;
boundary="--==_mimepart_5765914fd52a6_1c273fa92afa12bc1320924";
charset=UTF-8
Content-Transfer-Encoding: 7bit
Precedence: list
X-GitHub-Sender: fastlane-bot
X-GitHub-Recipient: yanndupuy
List-ID: fastlane/fastlane <fastlane.fastlane.github.com>
List-Archive: https://github.com/fastlane/fastlane
List-Post<EMAIL_ADDRESS>List-Unsubscribe<EMAIL_ADDRESS>https://github.com/notifications/unsubscribe/AA3w3FHVd_pDxbkBeD5FvPYawOyg1N1aks5qNDdPgaJpZM4HvOkd
X-Auto-Response-Suppress: All
X-GitHub-Recipient-Address<EMAIL_ADDRESS>X-Spam-Status: No, score=-4.1
X-Spam-Score: -40
X-Spam-Bar: ----
X-Ham-Report: Spam detection software, running on the system "serveur.ex2-p10.com",
has NOT identified this incoming email as spam.  The original
message has been attached to this so you can view it or label
similar future email.  If you have any questions, see
root@localhost for details.
Content preview:  This issue will be auto-closed because there hasn't been any
activity for a few months. Feel free to open a new one
if you still experience this problem 👍 [...]
Content analysis details:   (-4.1 points, 5.0 required)
pts rule name              description

-5.0 RCVD_IN_DNSWL_HI       RBL: Sender listed at http://www.dnswl.org/, high
trust
[<IP_ADDRESS> listed in list.dnswl.org]
-0.0 RCVD_IN_MSPIKE_H2      RBL: Average reputation (+2)
[<IP_ADDRESS> listed in wl.mailspike.net]
-0.0 SPF_PASS               SPF: sender matches SPF record
0.0 HTML_MESSAGE           BODY: HTML included in message
1.0 HTML_IMAGE_ONLY_16     BODY: HTML: images with 1200-1600 bytes of words
-0.1 DKIM_VALID             Message has at least one valid DKIM or DK signature
-0.1 DKIM_VALID_AU          Message has a valid DKIM or DK signature from author's
domain
0.1 DKIM_SIGNED            Message has a DKIM or DK signature, not necessarily valid
X-Spam-Flag: NO
----==_mimepart_5765914fd52a6_1c273fa92afa12bc1320924
Content-Type: text/plain;
charset=UTF-8
Content-Transfer-Encoding: quoted-printable
This issue will be auto-closed because there hasn't been any activity for=
a few months. Feel free to [open a new one](https://github.com/fastlane/=
fastlane/issues/new) if you still experience this problem =F0=9F=91=8D

You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/fastlane/fastlane/issues/3706#issuecomment-226957395=
----==_mimepart_5765914fd52a6_1c273fa92afa12bc1320924
Content-Type: text/html;
charset=UTF-8
Content-Transfer-Encoding: quoted-printable
This issue will be auto-closed because there hasn't been any activity =
for a few months. Feel free to open a new one if you still experience this problem=
 =F0=9F=91=8D
&m=
dash;You are receiving this because you are subscribed to this thre=
ad.Reply to this email directly, view it on GitHub, =
or mute the thread.

----==_mimepart_5765914fd52a6_1c273fa92afa12bc1320924--

This message was created automatically by mail delivery software.
A message that you sent could not be delivered to one or more of its
recipients. This is a temporary error. The following address(es) deferred:
<EMAIL_ADDRESS>Domain zyapps.com has exceeded the max emails per hour (470/400 (117%)) allowed.  Message will be reattempted later
------- This is a copy of the message, including all the headers. ------
Received: from github-smtp2-ext8.iad.github.net ([<IP_ADDRESS>]:46227 helo=github-smtp2a-ext-cp1-prd.iad.github.net)
by serveur.ex2-p10.com with esmtps (TLSv1.2:ECDHE-RSA-AES256-GCM-SHA384:256)
(Exim 4.87)
(envelope-from<EMAIL_ADDRESS>id 1bEKtN-002lcq-C4
for<EMAIL_ADDRESS>Sat, 18 Jun 2016 20:22:56 +0200
Date: Sat, 18 Jun 2016 11:22:08 -0700
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=github.com;
s=pf2014; t=1466274128;
bh=AsIhnV0o9e/Ya6QLwZCHSrxdxekSvXfaU5nIk9XM5oQ=;
h=From:Reply-To:To:Cc:In-Reply-To:References:Subject:List-ID:
List-Archive:List-Post:List-Unsubscribe:From;
b=j5fb73XN41Am5ckqdFwFnCC+cf01qCVtn052iccyzkPRPs9VaGoUmGey3Sj3yuBlO
eDGhpY5/aXN2mVtpjHqLFaQflfHnl+QQrucW2PeKNIwPajwq8EltG0MBNE0DwGV3MJ
9p6wjPcrkPSDiANcv8vxQTCfvb09Waj6CaEMQ/eg=
From: fastlane bot<EMAIL_ADDRESS>Reply-To: fastlane/fastlane<EMAIL_ADDRESS>To: fastlane/fastlane<EMAIL_ADDRESS>Cc:
Message-ID<EMAIL_ADDRESS>In-Reply-To<EMAIL_ADDRESS>References<EMAIL_ADDRESS>Subject: Re: [fastlane/fastlane] on licenses with lots of Devices this command
takes to long profile.update! (#3706)
Mime-Version: 1.0
Content-Type: multipart/alternative;
boundary="--==_mimepart_576591501e9c5_43ca3fc216c9d2a063563c";
charset=UTF-8
Content-Transfer-Encoding: 7bit
Precedence: list
X-GitHub-Sender: fastlane-bot
X-GitHub-Recipient: yanndupuy
List-ID: fastlane/fastlane <fastlane.fastlane.github.com>
List-Archive: https://github.com/fastlane/fastlane
List-Post<EMAIL_ADDRESS>List-Unsubscribe<EMAIL_ADDRESS>https://github.com/notifications/unsubscribe/AA3w3DhqVLXAau6DEcWLJT6elELhBkTTks5qNDdQgaJpZM4HvOkd
X-Auto-Response-Suppress: All
X-GitHub-Recipient-Address<EMAIL_ADDRESS>X-Spam-Status: No, score=-4.1
X-Spam-Score: -40
X-Spam-Bar: ----
X-Ham-Report: Spam detection software, running on the system "serveur.ex2-p10.com",
has NOT identified this incoming email as spam.  The original
message has been attached to this so you can view it or label
similar future email.  If you have any questions, see
root@localhost for details.
Content preview:  Closed #3706. --- You are receiving this because you are subscribed
to this thread. Reply to this email directly or view it on GitHub: https://github.com/fastlane/fastlane/issues/3706#event-696785049
[...]
Content analysis details:   (-4.1 points, 5.0 required)
pts rule name              description

-5.0 RCVD_IN_DNSWL_HI       RBL: Sender listed at http://www.dnswl.org/, high
trust
[<IP_ADDRESS> listed in list.dnswl.org]
-0.0 RCVD_IN_MSPIKE_H2      RBL: Average reputation (+2)
[<IP_ADDRESS> listed in wl.mailspike.net]
-0.0 SPF_PASS               SPF: sender matches SPF record
0.0 HTML_MESSAGE           BODY: HTML included in message
1.0 HTML_IMAGE_ONLY_16     BODY: HTML: images with 1200-1600 bytes of words
-0.1 DKIM_VALID             Message has at least one valid DKIM or DK signature
-0.1 DKIM_VALID_AU          Message has a valid DKIM or DK signature from author's
domain
0.1 DKIM_SIGNED            Message has a DKIM or DK signature, not necessarily valid
X-Spam-Flag: NO
----==_mimepart_576591501e9c5_43ca3fc216c9d2a063563c
Content-Type: text/plain;
charset=UTF-8
Content-Transfer-Encoding: 7bit
Closed #3706.

You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/fastlane/fastlane/issues/3706#event-696785049
----==_mimepart_576591501e9c5_43ca3fc216c9d2a063563c
Content-Type: text/html;
charset=UTF-8
Content-Transfer-Encoding: 7bit
Closed #3706.
—You are receiving this because you are subscribed to this thread.Reply to this email directly, view it on GitHub, or mute the thread.

----==_mimepart_576591501e9c5_43ca3fc216c9d2a063563c--


jack module: set soundpipe to JACK sample rate

I'm not sure whether setting this in sp_jack_process() is too late for other modules, which may need the correct sample rate at init.
Otherwise the soundpipe client needs to access the JACK API directly to get the rate, in which case the jack example should be modified to show this (vs. current hard-coded 96000).

Is this ready to be merged? I'm happy to merge your code if it is helpful for you since it is a small localized fix, but this is a deprecated part of Soundpipe.

see comment in #99. both the JACK and Pi modules for realtime audio have been removed from Soundpipe.


[Query] CloudBlockBlob.OpenWriteAsync and number of blocks connection. [Azure Storage]

[Query] CloudBlockBlob.OpenWriteAsync and number of blocks
How number of blocks in blob are connected to OpenWriteAsync. if stream opened by
Stream stream  = CloudBlockBlob.OpenWriteAsync ;
treamWriteSizeInBytes = 4 MB
stream.Write() // 4 MB data
stream.Write() // 4 MB data
stream.Flush() // will this create 2 blocks in block blob or How it is connected.
Hitting 50000 block limit error on using stream.Write()  based writing to cloudbloclBlob

Hi, this looks like you're using the deprecated library. Please upgrade to our current version which is v12+
All issues regarding the deprecated library can be found and placed here.
https://github.com/Azure/azure-storage-net/issues
Also if you have any further issues after upgrading please clarify the question you have or the problems you are having. Thanks. I recommend following the form given when first making the issue.

Using Microsoft.Azure.Storage.Blob, Version=<IP_ADDRESS> and doing below steps :
CloudBlockBlob cloudBlob = new CloudBlockBlob(new Uri(FullPath));
cloudBlob.StreamWriteSizeInBytes = blockSize // 4 MB
var blobStream = cloudBlob.OpenWriteAsync(null, BlobOptions);
now on writing blobStream it will take block size = blockSize  or it wont take blockSize. Writes to blobStream can be any length
i.e.
blobStream.Write() 1 MB data
blobStream.Write() 2 MB data
blobStream.Write() 1 MB data
blobStream.Write() 1 MB data
blobStream.Write() 1 MB data
blobStream.Write() 1 MB data
block size is 4 MB and total written is 7 MB , will this create 2 blocks on block blob ? if not then how to make sure it creates 2 blocks on blockblob. for above write pattern.

Sorry to repeat myself.
Please upgrade to our current version which is v12+. You are using a deprecated library ( Version=11.2.3).
All issues regarding the deprecated library can be found and placed here.
https://github.com/Azure/azure-storage-net/issues
Also can you be more clear on what the question is when you make the issue on the other repo. I don't know what problem you're hitting with OpenWrite. Are you seeing two block on the block blob? or are you not? (Confusion can be easily solved if you followed the initial form given in the github issue when you initially made it).

Thanks for update. Sorry for confusion. with library  <IP_ADDRESS> i am seeing more than 2 blocks are getting created. This issue will go away by moving to v12+ ?

We do not see this issue in v12.
If you would like assistance with v11 please follow the statement in my previous replies.

v12 is for .netstandard 2.0. Is there anything simialr for .netframework 4.5

Unfortunately not. I would not recommend .net framework 4.5 either as support ended in 2016
https://docs.microsoft.com/en-us/lifecycle/faq/dotnet-framework#what-is-the-microsoft-lifecycle-policy-for-the--net-framework-

v12 will work with .net 4.6.1 ?

Moving to blob12 dll and doing like this :
BlobStreamOptions =
new BlockBlobOpenWriteOptions()
{
BufferSize = 8 * 1024 * 1024,
};
blobStream = cloudBlobClient.OpenWrite(true, BlobStreamOptions);
blobStream.Write() 1 MB data
blobStream.Write() 2 MB data
blobStream.Write() 1 MB data
blobStream.Write() 1 MB data
blobStream.Write() 1 MB data
blobStream.Write() 1 MB data
this will make each block to size of 8 MB ? Please confirm this part. Thanks for help.

Yes v12 works with .NET 4.6.1
and as stated before

We do not see this issue in v12.
I will make each buffer size / block size 8MB if you set BlockBlobOpenWriteOptions.BufferSize to 8MB.


Add reference to solutions architectures

Please add reference to https://learn.microsoft.com/en-us/azure/azure-netapp-files/azure-netapp-files-solution-architectures in
NAS use cases section

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

ID: eabe039d-7b67-0813-ecd2-4dcbdd8d07f2
Version Independent ID: 26e275ea-df62-64bd-a1c4-7e4d2193cc3f
Content: Understand NAS concepts in Azure NetApp Files
Content Source: articles/azure-netapp-files/network-attached-storage-concept.md
Service: azure-netapp-files
GitHub Login: @b-ahibbard
Microsoft Alias: anfdocs

@gvanteylingen
Thanks for your feedback! I've assigned this issue to the author who will investigate and update as appropriate.

The doc has been updated.
#please-close


docs/source/ipynb/simpleaircraft is broken

(due to API changes -- monify_up and init_printing no longer exist, etc)

I'm getting an AttributeError: 'numpy.float64' object has no attribute 'units' in the ractorpy block.

"Fixed" by realizing it was actually a deeper copy issue: #293


Annotations/import any eaf tier

[ ] tests
[ ] documentation
[ ] implementation

I would suggest having a params: dict argument for _import_annotation rather than new_tiers so we can support more parameters of that kind in the long run.
Then we can do:
df = converter.convert(path, filter, **params)
The signature of Converter.convert() should include **kwargs.
What do you think?

Yes that's a good idea, and also params: dict in import_annoation, or should I keep an explicit argument like new_tiers?
Also, is there a reason to use dict and not list (and then we call df.converter(path, filter, *params) with args in the signature of Converter.convert()) ?


Is this project still maintained, developed?

More of a question than an issue.
Is this project still maintained?  Is it under active development?
The last commit was 9 months ago, and there are several open issues without resolution.

I still receive email about issues, many of them are related to wkhtmltopdf and anyway I don't have much time left to give to this, but I'm open to pull requests, as long as they are tested.


Instruction to backend setting for Tensorflow user

Cifar10 dataset in keras comes with the dimension order of (3, img_row, img_col) which is not comparable to Tensorflow setting of (img_row, img_col, 3).
As more Tensorflow users come in, I think it would be great to add that instruction.

Corrected to PEP8 standard


Privacy module does not hide itself when not in use [Bug]

Privacy module does not hide itself when not in use when "audio-out" is omitted from configuration.
Config of privacy module:
  "privacy": {
    "icon-spacing": 14,
    "icon-size": 14,
    "transition-duration": 250,
    "modules": [
      {
        "type": "screenshare",
        "tooltip": true,
        "tooltip-icon-size": 24
      },
      //{
      //  "type": "audio-out",
      //  "tooltip": true,
      //  "tooltip-icon-size": 24
      //},
      {
        "type": "audio-in",
        "tooltip": true,
        "tooltip-icon-size": 24
      }
    ]
  },

(Opened OBS)

(no software is recording sound/screen or outputting audio)

If "audio-out" module is included in the config, privacy module disappears from waybar as expected

forgot to mention, I use arch with hyprland, ask me if you need more info

I can confirm this, I also use arch with hyprland. I don't think that should matter tho?

I can confirm this, I also use arch with hyprland. I don't think that should matter tho?

Arch probably don’t matter, hyprland might, however, in general pointing out your environment is good practice when reporting a bug.

Which version are you using? This was recently changed with 4fbd4f212a80009e92d179f9e96a50562891c2e8. I have not attempted a repro, but the code looks to me like it should work. ;)

Fair enough. Fwiw, the commit is newer than last waybar release (0.10.3). I have no knowledge about what specific distro packages have or do, though.

I just installed waybar throught the AUR, it does actually work now. I guess it will just take a bit until it arrives in the repo :)

Upgraded the package, issue resolved.


Error: HTTP error! status: 400

After 15 minutes of using ChatGPT it responds with "Error: HTTP error! status: 400"

There is a workaround where you disable the context feature (the green button in the upper right). Still, it would be nice if I could keep this feature.

I think this is caused because I haven't set a limit for the context of the chat. So, every time you ask something, all the previous messages will be sent to the API. ChatGPT has a limited context of 4096 tokens, so exceeding that limit will result in an error. This issue will be fixed, but until 7 July, I am occupied with exams.

+1 to this request. When working through some complex conversations that build on each other, that context is really important.

The response error is related to this issue.
{
  "error": {
    "message": "This model's maximum context length is 4097 tokens. However, you requested 4610 tokens (1610 in the messages, 3000 in the completion). Please reduce the length of the messages or completion.",
    "type": "invalid_request_error",
    "param": "messages",
    "code": "context_length_exceeded"
  }
}


PCBS micro-benchmark

This is micro-benchmark used to compare performance of writing/reading cached batches using PCBS and DefaultCachedBatchSerializer

@viadea

I spend some time building the jar, due to the terrible urm or maven repository network. Shall we add more comments to guide customers to download and install mvn package manually to save times? Such as
wget https://XXX/rapids-4-spark_2.12-22.06.0-20220512.134355-33-cuda11.jar

mvn install:install-file -DcreateChecksum=true -Dpackaging=jar -DgroupId=com.nvidia -DartifactId=rapids-4-spark_2.12 -Dversion=22.06.0-SNAPSHOT -Dclassifier=cuda11 -Dfile=/your-path/rapids-4-spark_2.12-22.06.0-20220512.134355-33-cuda11.jar

Do we need to add some sample datasets under datasets folder?  What are the considerations about parquet file when we test? For example， how does the parquet file size influence the performance.

I spend some time building the jar, due to the terrible urm or maven repository network. Shall we add more comments to guide customers to download and install mvn package manually to save time? Such as
wget https://XXX/rapids-4-spark_2.12-22.06.0-20220512.134355-33-cuda11.jar

mvn install:install-file -DcreateChecksum=true -Dpackaging=jar -DgroupId=com.nvidia -DartifactId=rapids-4-spark_2.12 -Dversion=22.06.0-SNAPSHOT -Dclassifier=cuda11 -Dfile=/your-path/rapids-4-spark_2.12-22.06.0-20220512.134355-33-cuda11.jar

This is not necessary I think because the file can be built locally or downloaded however the user wants and put in the lib folder

Do we need to add some sample datasets under datasets folder? What are the considerations about parquet file when we test? For example， how does the parquet file size influence the performance.

I think the data is publicly available and unnecessary to push into the repo. The files are huge

@razajafri is the demo specific to spark3.1.X?  it works well with spark 3.1.1 and 3.1.2, but failed with spark3.2.0 due to spark version mismatch
  at org.apache.spark.sql.execution.columnar.Util$.$anonfun$readWriteCache$1(Util.scala:98)
  at scala.runtime.java8.JFunction1$mcJI$sp.apply(JFunction1$mcJI$sp.java:23)
  at scala.collection.TraversableLike.$anonfun$map$1(TraversableLike.scala:286)
  at scala.collection.immutable.Range.foreach(Range.scala:158)
  at scala.collection.TraversableLike.map(TraversableLike.scala:286)
  at scala.collection.TraversableLike.map$(TraversableLike.scala:279)
  at scala.collection.AbstractTraversable.map(Traversable.scala:108)
  at org.apache.spark.sql.execution.columnar.Util$.readWriteCache(Util.scala:95)
  at org.apache.spark.sql.execution.columnar.Util$.runPcbsInternal(Util.scala:184)
  at org.apache.spark.sql.execution.columnar.Util$.runPcbs(Util.scala:190)
  at com.nvidia.BenchmarkPcbs$.main(RunPCBS.scala:28)
  at com.nvidia.BenchmarkPcbs.main(RunPCBS.scala)
  at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
  at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
  at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
  at java.lang.reflect.Method.invoke(Method.java:498)```

@razajafri can you please also make the spark master URL a variable?
such as ${BENCH_JAR} $url $data-path in pcbs-benchmark.sh, then we don't need to hardcode in Util.scala
.master("spark://raza-linux-2:7077")

@razajafri is the demo specific to spark3.1.X? it works well with spark 3.1.1 and 3.1.2, but failed with spark3.2.0 due to spark version mismatch
Exception in thread "main" java.lang.NoSuchMethodError: org.apache.spark.sql.internal.SessionState.executePlan(Lorg/apache/spark/sql/catalyst/plans/logical/LogicalPlan;)Lorg/apache/spark/sql/execution/QueryExecution;
  at org.apache.spark.sql.execution.columnar.Util$.$anonfun$readWriteCache$1(Util.scala:98)
  at scala.runtime.java8.JFunction1$mcJI$sp.apply(JFunction1$mcJI$sp.java:23)
  at scala.collection.TraversableLike.$anonfun$map$1(TraversableLike.scala:286)
  at scala.collection.immutable.Range.foreach(Range.scala:158)
  at scala.collection.TraversableLike.map(TraversableLike.scala:286)
  at scala.collection.TraversableLike.map$(TraversableLike.scala:279)
  at scala.collection.AbstractTraversable.map(Traversable.scala:108)
  at org.apache.spark.sql.execution.columnar.Util$.readWriteCache(Util.scala:95)
  at org.apache.spark.sql.execution.columnar.Util$.runPcbsInternal(Util.scala:184)
  at org.apache.spark.sql.execution.columnar.Util$.runPcbs(Util.scala:190)
  at com.nvidia.BenchmarkPcbs$.main(RunPCBS.scala:28)
  at com.nvidia.BenchmarkPcbs.main(RunPCBS.scala)
  at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
  at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
  at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
  at java.lang.reflect.Method.invoke(Method.java:498)```

Yes, the benchmark was run on Spark 3.1.1 and it's very specific to it for now. We can in future make changes to it

@nvliyuan I think I have addressed your concerns. PTAL

this is the perf result I get:
*****
PCBS
acc: true
Average write: 585
Time taken for frst read: 361
Average read (without first read): 142
acc: false
Average write: 3399
Time taken for frst read: 3619
Average read (without first read): 370
*******
DefaultSerializer
acc: true
Average write: 1429
Time taken for frst read: 946
Average read (without first read): 73
acc: false
Average write: 1298
Time taken for frst read: 1133
Average read (without first read): 54

Can I make a conclusion that if we enable PCBS(GpuInMemoryTableScanExec), we could get 5.8x(3399/585) speedups in ParquetCachedBatch write and 2.6x(370/1422) speedups in ParquetCachedBatch read if we enable the rapids plugin? And if we don't enable PCBS(GpuInMemoryTableScanExec),  there is no difference whether we enable rapids plugin?

this is the perf result I get:
*****
PCBS
acc: true
Average write: 585
Time taken for frst read: 361
Average read (without first read): 142
acc: false
Average write: 3399
Time taken for frst read: 3619
Average read (without first read): 370
*******
DefaultSerializer
acc: true
Average write: 1429
Time taken for frst read: 946
Average read (without first read): 73
acc: false
Average write: 1298
Time taken for frst read: 1133
Average read (without first read): 54

Can I make a conclusion that if we enable PCBS(GpuInMemoryTableScanExec), we could get 5.8x(3399/585) speedups in ParquetCachedBatch write and 2.6x(370/142) speedups in ParquetCachedBatch read if we enable the rapids plugin? And if we don't enable PCBS(GpuInMemoryTableScanExec), there is no difference whether we enable rapids plugin?

Yes

@nvliyuan PTAL

@nvliyuan PTAL

@razajafri thanks, the PR looks good to me

@razajafri since we just restructured the directories, I am not sure which directory this demo should belong to, I am confirming with @krajendrannv

we can put it under SQT+DF-Examples folder

will close this pr and re-draft one @razajafri @sameerz


Style disabled tabs?

Could disabled be added to slotProps?

disabled is a render prop on the Tab component so it's a value you have control over when you are writing the markup.
So you could use it to style the tabs as you like.


ci: add luarocks-tag-release workflow

Closes #11

@swarn One thing I forgot to mention: The way I've implemented this workflow, it only triggers if you push a tag, but not if you create a tag via GitHub Releases.
There's a more complicated setup that solves this problem, which we provide a tutorial for.
But I think for your case, this simple approach should suffice :smile:.

I agree!
Running into a little bug here, I think. I added the release tag, the ran the workflow manually. It uploaded a fzy-lua rock, rather than a fzy rock.

Ha, I added a name field, but now it uploads a main-1 rock to luarocks rather than a 1.0.1-1 rock. I'll stop messing with it until you get a change to check it out.

Oh yeah, I missed that :sweat_smile:
I see you've already added the name.

Ha, I added a name field, but now it uploads a main-1 rock to luarocks rather than a 1.0.1-1 rock. I'll stop messing with it until you get a change to check it out.

This is probably because you triggered it on the main branch.
If you trigger it on the tag, it should work (but it might use the old workflow file, so it might be best to delete the tag and create a new one).

fzy 1.0.2 is live on luarocks and your windows builds should all succeed now. Thanks for the help!

Oh, I know what it is.
The luarocks-tag-release action needs a rockspec template in your case.
I will add this in another PR


Integration test

Integration test to spin up Kafka, Postgres, Debezium source and Ably Sink connector.
Steps:

Kafka container has to be started first.
Postgres, Debezium source containers.
Register Postgres source connector using HTTP request.
Start Ably Sink connector with the correct bootstrap.servers(retrieved from kafka container)
Add configuration variable to write the messages into a file instead of calling Ably Batch API so that the order of messages can be validated.

Superseded by #131 and #137


Kcp hangs on invalid networkAddress

Any Mirror example with the HUD...type '123' into the address field and click Client.  Unity hangs.
Windows 10 / master

we have test coverage for that already:

might be kcp TRANSPORT for mirror related, will check later today

can't reproduce.
feel free to reopen but I think this was fixed a while ago


Cannot read property 'filter' of undefined

Seems as if thread can't be found when there's a symbol in title.
Error with stacktrace:
Unhandled rejection TypeError: Cannot read property 'filter' of undefined
    at ../node_modules/facebook-chat-api/src/listen.js:169:40
    at ../node_modules/facebook-chat-api/src/searchForThread.js:27:18
    at tryCatcher (../node_modules/bluebird/js/main/util.js:26:23)
    at Promise._settlePromiseFromHandler (../node_modules/bluebird/js/main/promise.js:507:31)
    at Promise._settlePromiseAt (../node_modules/bluebird/js/main/promise.js:581:18)
    at Promise._settlePromises (../node_modules/bluebird/js/main/promise.js:697:14)
    at Async._drainQueue (../node_modules/bluebird/js/main/async.js:123:16)
    at Async._drainQueues (../node_modules/bluebird/js/main/async.js:133:10)
    at Immediate.Async.drainQueues [as _onImmediate] (../node_modules/bluebird/js/main/async.js:15:14)
    at processImmediate [as _immediateCallback] (timers.js:371:17)

Come across error when threadName contains special symbols. Shouldn't be hard to reproduce, try with § in chat name.

Only happening when group chat has more than 4 participants.

Fixed in https://github.com/Schmavery/facebook-chat-api/commit/ca9beae7c26dbd626e840a8c5c5f666839be8dc4, my mistake.


An uploaded file doesn't immediately appear in uploaded file list

During submission, uploading a file from file system succeeds but the latest uploaded file doesn't appear in the "Files Uploaded" list. Current workaround to see it is to manually refresh by pressing "Upload" step in progress list.
Uploading from URL works as expected though and the new file is immediately listed.

fixed in #167

The originally described issue was not fixed.

Hi @marsav could you please describe your issue in more detail?
What you should see, when using either the drop zone or the choose files button, after you choose the file(s) is a javascript dialog prompting you to describe the file(s) and then start the upload. After the upload is finished the dialog should close and the page should be refreshed.
Does this not happen for you? Do you see some error in the javascript console? The fix in #167 changed some .js files, please make sure your browser is not using some older version from cache.

Hi @kosarko what you just described is the behaviour for UFAL theme, whereas using Mirage, I don't get a javascript dialog, but instead I choose the file using file chooser and fill the name - both on the workflow page and then submit it by pressing "Upload and add another" button. The first file uploads successfully and is shown in the list below. But when I upload the second file, the "Upload" page refreshes but the list is not containing the newly uploaded (second) file data.
The same happens if I continue an abandoned submission.
The only way to see that file in the list is to choose the same wortkflow step by pressing "Upload" button in top progress menu, or to press either "Prev" or "Next" and then return to the "Upload" section.
I tested it with fresh 5.3 build yesterday and in both Mirage and UFAL themes I get the same faulty behaviour. The .js files updated are UFAL-specific and doesn't seem to affect Mirage at all.

@marsav: are you trying to use lindat-dspace UFAL theme directly with "raw" DSpace?
For the moment, we are testing lindat-dspace as a standalone project only.

@vidiecan no, I am using lindat-dspace source base as a standalone project.

@marsav does this happen to you all the time, or just in some specific cases? I can see this happening when I upload the exact same file again. Is that your case too, or is it really with every file?


feat(frontend): add sol addresses to derived

Motivation
Make a derived for the sol addresses
Changes

Add sol address derived per network
add typing

Tests

No unit tests provided

bases on https://github.com/dfinity/oisy-wallet/pull/3998


official docker image for mlnet cli?

Hi! wonder if it would be possible to have an official docker image, either on dockerhub or Microsoft Container Registry (MCR).
That would be very cool for single line mlnet cli automl workloads. Can't get easier than that. Also safer to trust an official image.
(example: https://hub.docker.com/_/microsoft-azure-batch-shipyard)
thanks!

Transferring to modelbuilder for additional input.

@briacht and @JakeRadMSFT to comment 😊

@rquintino Can you share how you would use the docker image? Would it be apart of the CI?

Sorry for the long delay in response on this issue. @rquintino Do you have any comment on how you would use the Docker image?


codegen-libdeploy won't work for IPayableSystem

ISystem is hardcoded in LibDeploy.ejs and LibDeploy.sol
so running mud test gives error
Compiler run failed
error[7407]: TypeError: Type contract MyPayableSystem is not implicitly convertible to expected type contract ISystem.
  --> src/test/LibDeploy.sol:84:14:
   |
84 |     system = new MyPayableSystem(world, address(components));
   |              ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^

a possible fix:

in deploy.json, allow a new optional payable field for systems.
in LibDeploy.ejs, if this field is true, use IPayableSystem.

You can use ISystem, have an empty execute which does nothing, and add the payable function you need, e.g.
function doStuff(uint256 arg1, address arg2, ...) external payable {
...
}

execute is a bit confusing in that you don't really have to use it at the moment.
In v2 it'll be removed from the interface entirely
(https://github.com/latticexyz/mud/blob/alvrs/datamodel/packages/store/src/System.sol)


Leave signals alone

Fixes #51

Did you write multiple commits on purpose, or do you mean to rebase/squash this before merging?

Also, please briefly summarize your changes in the commit description, like mentioning the added exported name, and the changes that this fix will do to people's use of hint.
I assume you also considered exporting a new name instead of breaking backwards compatibility slightly. I would likely not introduce a new name either, as you explained how the old behavior wasn't really useful to anyone. In any case, please do mention that too in the commit message.

Did you write multiple commits on purpose

I did, but now that you mention it, I do remember that you always ask contributors to squash their commits. Maybe you should add a CONTRIBUTING.md file pointing out this policy, so you don't have to repeat it every time?
I have now squashed my commits.

:) I just prefer avoiding redundant commits. But of course, they can be useful to separate different pieces of work, which is why I was asking.
I don't mention it in the contribution guide because GitHub now has a web interface to squash easily, so I don't really have to depend on others doing it. But I still ask, in case they did mean to separate their work in multiple commits.

please briefly summarize your changes in the commit description

I have now modified my commit description to include this.

like mentioning the added exported name

I did not export any new name, rawRunGhcT is private to Control.Monad.Ghc

I assume you also considered exporting a new name instead of breaking backwards compatibility slightly.

I was hoping that my intention to modify the existing function was clear in #51, when I said we should change the default behaviour. I expected you might ask for a separate function then, but you didn't, so I went ahead and modified the existing function's behaviour.

I did not export any new name

Err, I meant to delete that part of my comment.

I was hoping that my intention to modify the existing function was clear

Yes - it is clear to me, but I was talking more in terms of someone reading the commit message and not having the full context. Thanks for updating the message.
Happy to merge once you've addressed the code comments.

In any case, please do mention [breaking backwards compatibility slightly] too in the commit message.

I have now modified my commit message to include this as well.

Happy to merge once CI is green. Seems like the tests don't build properly, perhaps the deps aren't set up right.

Let's see...
The ghc-7.10.3 build fails with Not in scope: ‘GHC.withCleanupSession’. I am guessing that the problem is that the runGhcT` code I have copied is from ghc HEAD, but this code has changed over the past few years. Indeed:

only ghc-8.2 calls withSignalHandlers
ghc-8.0 calls installSignalHandlers instead, which does not restore the handlers afterwards
ghc-7.10 does not call either, nor does it call withCleanupSession (I don't know what that cleans). I remember reading some ghc notes saying that the ghc API was no longer automatically installing signal handlers and that users now had to call installSignalHandlers if they want them, so it might not be possible to prevent runInterpreter from installing signal handlers when using ghc-7.10 :(

ghc-8.4 will roll out soon, and your policy is to only support the last three major ghc versions, so can we just drop support for ghc-7.10?

I adopted the three-version policy very recently, simply to keep the project easier to maintain in the long run. I did not imagine supporting more than three versions would be a problem this soon :)
Dropping 7.10 is fine, as long as you add 8.4, which is already present as a 8.4.0 git build in hvrs repo: https://launchpad.net/~hvr/+archive/ubuntu/ghc/+index?batch=75&memo=75&start=75
I also understand that the added code should run fine on 8.0, 8.2, and 8.4 without any CPP magic? Either way, please have a look at the failures on those versions.
Feel free to leave the dynamic build+test one with 8.2, the latest stable version.

To add to my comment above - I was planning on doing another Hint release sometime this spring, so I can simply make it line up with the GHC 8.4 release. That Hint release will add support for 8.4, and drop 7.8 and 7.10 (as per the new policy).

I have now modified the .travis.yml file, and I have updated the commit message to reflect the change.

Travis is still very unhappy :)

It might make sense to do the addition of the 8.4.0 git build in a separate pull request, as it's likely that it will require a few fixes here and there. At least, it should be its own commit.

At least, it should be its own commit.

See, this is why squashing commits is a bad idea! Now I have to manually separate the changes related to ghc-8.4 from the rest of my changes, whereas if the commits were separate, it would have been trivial to rebase cherry-pick it onto master :)
I will extract the changes (fortunately only the .travis.yml file changed so this will be easy) and create another PR.

Well, I never asked that you squash all changes :) I simply asked if you wanted to keep the commits separate or not. The commits you initially posted seemed like amends to previous commits, which is why I wondered if you meant to squash them before merging.

@gelisam please rebase - master now supports GHC 8.4 and dropped 7.10.

I have now rebased.


Unify message execution properties

We had duplicate information on message execution basics. This PR unifies the two places:

Move all the information to references "Properties of message executions on ICP". I think the original page here was originally duplicated from the security best practices on inter-canister calls. Since the latter evolved further in the meantime, I'm moving the content from security best practices to here, replacing the current content.
The security best practices on inter-canister now refer to the reference rather than providing the same content.
Added a more detailed introduction to the "Properties of message executions on ICP" and improved a few details

If you are submitting a Pull Request for adding or changing content on the ICP Developer Documentation, please make sure that your contribution meets the following requirements:

[x] Follows the developer docs style guide.
[x] Follows the best practices and guidelines.
[x] New documentation pages include document tags.
[x] New documentation pages include SEO keywords.
[x] New documents are in the .mdx file format to support the previous two components.
[x] New documents are registered in /sidebars.js, otherwise, it will not appear in the
side navigation bar.
[x] Make sure that the .github/CODEOWNERS file is
filled with new documents that you added. This way we can ensure that future Pull Requests are reviewed by the right people.

Thanks again for the reviews and good input, proceeding to merge this.


January 6th - Shooting at Fort Lauderdale,  Florida.

5 dead, 8 injured.

https://www.theguardian.com/us-news/2017/jan/06/fort-lauderdale-airport-shooting-florida


[CI] Add GitHub workflows to build maven and publish container.

The PullRequest workflow would automatically build the maven project and the container.
Release next automatically publishes a SNAPSHOT version with container
Release master builds maven release and a container image

Maven artifacts and container images are hosted at github packages, thus we and collaborators can pull maven dependencies from Githubs public repository.

This will be the start of a set of PullRequests, because workflows need to reside in the next and master branches in order for the release/release-next workflow to run.


Autorest Regen Preview to 3.0.0-alpha.20231212.7 by Dapeng Zhang from refs/pull/3838/merge

Triggered from https://github.com/Azure/autorest.csharp/pull/3838

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


Changing position of dropdown

One feature that I liekd from ui-select2 is the dropdown could be injected at the top-level of the DOM, so it would appear outside of DIV's with overflow:hidden, it also could switch between dropping down and dropping up.
Any roadmap to support this, or ideas to implement with CSS? I wouldn't mind working on a patch myself when I have time either.

+1
  for Bootstrap.

+1

<ui-select  class="btn-group dropup">  for Bootstrap.

+1

<ui-select  class="btn-group dropup"> </ui-select> for Bootstrap.


Updates to host.json configuration

More updates on host.json configuration
Removed the sentence to the specific GitHub issue

Hi @CarlosSardo. Looks like this PR hasn't been updated in a while. Were you still planning on working on it?

Hi @cgillum , I left this PR in draft on purpose (as described here).
Now that these changes will be released, I will submit it for approval.
Thanks!


feat(ast): add a dummy node_id field to all visitable AST types.

Part of #5689
A few places are creating these nodes without AstBuilder; For now I've assigned them directly but should be taken care of as part of making types non_exhustive.

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#5775  👈
#5774
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @rzvxa and the rest of your teammates on  Graphite

CodSpeed Performance Report
Merging #5775 will degrade performances by 3.87%
Comparing 09-15-feat_ast_add_a_dummy_node_id_field_to_all_visitable_ast_types (00ce823) with 09-15-fix_ast_tools_fix_miscalculation_of_enum_layouts (ea1c733)
Summary
❌ 3 regressions ✅ 26 untouched benchmarks

⚠ Please fix the performance issues or acknowledge them on CodSpeed.

Benchmarks breakdown
Benchmark	09-15-fix_ast_tools_fix_miscalculation_of_enum_layouts 09-15-feat_ast_add_a_dummy_node_id_field_to_all_visitable_ast_types	Change
❌	codegen[checker.ts]	20.8 ms	21.7 ms	-3.87%
❌	isolated-declarations[vue-id.ts]	391.8 ms	404.7 ms	-3.17%
❌	parser[cal.com.tsx]	24.5 ms	25.2 ms	-3.07%

Not bad!!

A few places are creating these nodes without AstBuilder; For now I've assigned them directly but should be taken care of as part of making types non_exhustive.

I cleaned up the parser a few days, let me ask @Dunqing to refactor the transformer 😀

Merging #5775 will degrade performances by 3.87%

Not bad!!

We can win most of this perf back if we optimize field layouts later on.


Investigate alternative DVC integrations

This ticket will serve as a container for all investigation into alternative DVC integrations.
Options to investigate (incomplete):

[x] 1. Transpile Python to Javascript (e.g Transcrypt, PyPy.js)
[x] 2. Use bridge library (e.g Boa, Pythonia)
[ ] 3. Improve existing integration
[ ] 4. Typescript rewrite
[ ] 5. Message passing through files
[ ] 6. Python-js IPC with named pipes (https://github.com/iterative/vscode-dvc/issues/967#issuecomment-955976810)
[ ] 7. RPC

Investigations will go in the comments and each comment can / will be updated over time.

Closing this as we exhausted the ideas for now and can come back later