require-default-props: Allow disabling "must have default value" warning but keeping "forbid defaults for required props"

Right now there doesn't seem to be a way to do this.

Having to specify a null/undefined default for a prop where you're perfectly fine with it being undefined when not specified feels a bit pointless
This is made worse for a prop where you want to allow null, because a required prop will fail runtime proptypes validation when its value is null (even if it's explicitly set to null

So I'd like to just ditch the part of having to specify default for optional props, but still prohibit setting defaults for required ones.

Some of those cases are why https://npmjs.com/airbnb-prop-types has explicitNull, but an option as you describe seems feasible.


[App]: Video logged in media folder but not displayed in Media

Current Behavior
Hi,
I have an error where some videos are logged in at each step and do get recorded in the media folder.
However, when trying to display them in the media object they do not appear. Only the first two (or first depending of the log) do get displayed.
On another run they do get displayed with a slider to select between steps
Expected Behavior
Been able to display all videos
Steps To Reproduce

go on https://wandb.ai/nicolas-dufour/torchrl/runs/2r54ati0?workspace=user-nicolas-dufour
Look at media
Look at media folder

Screenshots
No response
Environment
OS: Linux
Browsers: Chrome and Safari
Version: 0.12.21
Additional Context
No response

@nate-wandb Hiu, yeah sorry, i ahd deleted it by error, i reproduced it here: https://wandb.ai/nicolas-dufour/torchrl/runs/btslaz6v?workspace=user-nicolas-dufour

Hi @nicolas-dufour, thank you for the reproduction. I see the issue as well. I'll report this to our engineering team as a bug and follow up with you once we are able to get this resolved.
Thank you,
Nate

Was there ever a resolution reached? I'm seeing a similar issue with image logging at https://wandb.ai/dyunis/deq_diffusion/runs/c85nrhug?workspace=user-dyunis including the missing images in wandb-summary.json.

Hi @dyunis, it looks like this hasn't been resolved. I will raise this again to our engineering team so we can get a fix on this. Thank you for providing your workspace so we have another example of this behavior.
Thank you,
Nate

Thanks @nate-wandb, I really enjoy working with wandb and appreciate you looking into it.

This seems related to #4830

As of today, I stil have the problem that the logged videos are in the media folder, but don't appear (whereas images do)

I also have this problem. The videos are in the media folder but do not show up in the workspace.

Hi @Nate711, looks like this ticket got buried, and the examples provided above by the user aren't available anymore.
Could you please send me a code snippet of you reproing this as well as the workspace where you are experiencing this behavior?

Hi Wanted to follow up with you regarding this thread!

@ArtsiomWB This is probably due to my code specifying the step like with https://github.com/wandb/wandb/issues/4830#issuecomment-1431935492
wandb.log(
        {
            "eval/video/command/vx": vx,
            "eval/video/command/vy": vy,
            "eval/video/command/wz": wz,
            "eval/video": wandb.Video(filename, format="mp4", fps=fps),
        },
        step=current_step,
    )

In another place I have
wandb.log(metrics, step=num_steps)
Is there a way to log both data to the same step that does work?

Is there a way to log both data to the same step that does work

The step has to either be the same for the epoch of metrics you are logging such as:
wandb.log(
        {
            "eval/video/command/vx": vx,
            "eval/video/command/vy": vy,
            "eval/video/command/wz": wz,
            "eval/video": wandb.Video(filename, format="mp4", fps=fps),
        },
        step=current_step,
    )

wandb.log(metrics, step=current_step)

Or has to be monotonically increasing. Unfortunately, there is no way of writing it as a previous step, or a step located in the future to go straight back to the previous step.
If you are interested in using a separate x-axis that is not just step, you can customize it using define metric

@ArtsiomWB I was providing the same step number to both calls to .log. Once I deleted the Video panel and recreated it, all the media showed up.

Thanks for the update @Nate711 !


Add Build system components to Staging

Components of the App Studio Build System
In production, this should probably be on tenant clusters only.
The App Studio Build System is composed of the following
components:

OpenShift Pipelines.
AppStudio-specific Pipeline Definitions in build-templates for building images. <-- there would be multiple ways of getting these in here.
AppStudio-specific ClusterTasks.

Long-term, (2) and (3) would be pulled from some remote catalog.
As a non-admin user, one would have access to the Pipeline definitions in build-templates.
To be able to use them in one's personal namespace, the UI or the client would need to run the equivalent of the following command:
Usage
oc get pipelines -n build-templates -o yaml | sed 's/namespace: .*/namespace: YOUR-NAMESPACE/' | oc apply -f -

Testing out this PR
Install OpenShift GitOps and create the following Application
apiVersion: argoproj.io/v1alpha1
kind: Application
metadata:
  name: build

spec:
  project: default

  source:
    path: components/build
    repoURL: https://github.com/sbose78/infra-deployments.git
    targetRevision: build-service-init ## <-- points to my fork's branch

  destination:
    namespace: default # not relevant yet since every object has an explicit namespace.
    server: https://kubernetes.default.svc

  syncPolicy:

    # Comment this out if you want to manually trigger deployments (using the
    # Argo CD Web UI or Argo CD CLI), rather than automatically deploying on
    # every new Git commit to your directory.
    automated:
      prune: true
      selfHeal: true

    syncOptions:
    - CreateNamespace=true

    retry:
      limit: 50 # number of failed sync attempt retries; unlimited number of attempts if less than 0
      backoff:
        duration: 15s # the amount to back off. Default unit is seconds, but could also be a duration (e.g. "2m", "1h")

Since this doesn't use sync waves yet, the subscription.yaml might need a nudge since it needs to be sync'd first.

/test appstudio-e2e-deployment


docs(compile_models.rst): Get Started, target iphone

As @tqchen  says: Likely you need to limit seq Len and use q3 for llama 2 models
https://github.com/mlc-ai/mlc-llm/issues/858

sorry @baiyutang . for the red pajama model, we recommend q4. We only recommend q3 for llama 2 models

sorry @baiyutang . for the red pajama model, we recommend q4. We only recommend q3 for llama 2 models

limit seq Len this, is there a recommanded max-seq-len value for  llama 2?


Make Markdown guide mobile compatible

Doesn't look great right now with mobile devices

Added a responsive tag


added asset lib for rke2 zarg config: will be prepping for tests next

Description
Added RKE2 1.21.11 as a distribution package for Zarf
Related Issue
378-ks-alternative-example
Fixes #378
Type of change

[x] New feature (non-breaking change which adds functionality)

Checklist before merging

[ ] Tests have been added/updated as necessary (add the needs-tests label)
[ ] Documentation has been updated as necessary (add the needs-docs label)
[ ] An ADR has been written as necessary (add the needs-adr label) [ 1 2 3 ]
[x] (Optional) Changes have been linted locally with golangci-lint. (NOTE: We haven't turned on lint checks in the pipeline yet so linting may be hard if it shows a lot of lint errors in places that weren't touched by changes. Thus, linting is optional right now.)

@saintskeeper what do you think about adding a test job for RKE2 like we do k3s/d, kind?

Will be working on that tomorrow  @jeff-mccoy. I might need to touch base with you on your workflow, but I intend to build out an e2e for RKE2.

E2Es are set up to accept an env var TESTDISTRO, of which one of the valid values is provided. Creating a new GitHub Actions run that stands up the RKE2 cluster, then uses provided, should be a good way to do it. Here's an example of one that uses provided, though for RKE2 I guess we'd do a zarf package deploy, followed by the test run? Not sure I've ever tried that before, doing a zarf package deploy without first doing a zarf init...

Thanks @RothAndrew for the added context!

Closing this issue as stale, will keep #378 open


Video bug that happens when the video is loaded but the play isnt triggered and the user switches tabs.

Version

Phaser Version:
3.70 but the issue occurs in 3.55 and 3.60 as well

Operating system: Windows

Browser:
Opera GX, Chrome, Brave (Those are the ones I tested it on)

Description

its basically a video bug. I have games that have videos in them that are loaded at the start. And I have a play button to start playing them. If the user changes tab on pc and goes back to the game window it starts playing the sound of the video without playing the actual video. Also if u press the play button after the bug occurs its just a black screen with the sound keep playing but it breaks the whole game without errors and forces people to restart the window.
Sometimes it doesn't happen when you do it but it happens often enough to be annoying.
Example Test Code

create() {
this.video= this.add.video(0,0,'oliviaVideo').setOrigin(0)
let playButton = this.add.image(this.config.width/2,this.config.height/2-30,'playButtonVideo').setInteractive().setDepth(5000).setScale(.3).setOrigin(.5);
playButton.on("pointerup",()=>{
this.video.play()
playButton.destroy()
})
}
Additional Information

This is now handled properly via its own Video.getFirstFrame method, and no longer exhibits the issue described here.


[QUESTION] How can i give custom index name while dumping output to ElasticSearch?

I wanted to store the output into Elastic Search. I read all the documentation, apart from providing hostname for Elastic Search, i didn't find how to give custom index name while dumping data into Elastic Search. How can i provide?

You have to edit the index name when you create the index, so when you are in DevTools and you copy&paste from index-tweets.json and others, change twint to what you prefer. You will have something like PUT mycustomindex ...
Also you will have to edit _index in twint/elasticsearch.py
I'll make this easier asap

Thank you.
Yeah, I think it will be better if we can provide through command line along with elastic search host name. If user doesn't give, the default can be used or else it can be overridden from command line.

Sure, thank you for the hint!

For the index of tweets -it or --index-tweets, for followers/following -if or --index-follow and for users profile -iu or --index-users.
If you user Twint as module:
import twint
c = twint.Config()
Index_tweets = "..." # change
Index_follow = "..."  # values
Index_users = "..."  # here
...

About the Elasticsearch hostname you have just to change -es or --elasticsearch at your needs (e.g. --elasticsearch myhostname.io:1234) or using Twint as module c.Elasticsearch = "myhostname.io:1234"

Ok now it should be fixed, there are default both for indexes name and essid (this one is an empty string)
Should be fixed, in case feel free to reopen and/or comment


Navbar support?

Hello! I love your library very much. However, I didn't find a component like a navbar for navigation... Is there a plan for adding it? Thanks a lot!!

if you're talking about components, yes I have... Actually, I make them before vechaiui... about 60 components for the marketing page but honestly I'm considering whether to publish them  🥲

Oh... I think if the design fits for vechaiui, it would be fine! A proper navbar would make this library more complete.


Driver-side metrics

We should gather and expose some metrics about driver operations, e.g. how many queries were sent, how many of them resulted in an error, what was their latency, etc.
Implementation detail: @piodul since session is immutable, what's the rusty way of keeping mutable metrics inside it? RefCell?

When I was working on the topology refreshes I had a thought that this could be done in a more efficient way -- with less synchronization -- if Session query/prepare/execute would take a mutable reference to the Session. I.e. we have a guarantee that there's only one user of Session at a time.
We could then make Sessions Cloneable and the clones could share some data structures (e.g. connections). But Sessions know their own internals so they can do the sharing more efficiently.


Bigint PK migration -- HIL core and all extensions

In response to #315
Migrations for core HIL and all extensions
Some migrations checked for table existence using sa.MetaData(...), which caused hangs in the CI

Replaced with db.inspect(...)

Just rebased this to master

why are diffs from the other merged PRs shown in this one? can someone with a higher gitfu rank explain?

I feel like we've hit this before. @henn, do you recall what we figured out?

Can we just squash them when we merge it?

Also I'm working on adding a migration test too.

I'm having some auth issues with the migration test, for now I've left the new database dump so it can be visually inspected.

There are some tweaks needed to the db dump, which (I think) are described in the migrations docs. Some are for documentation, others are workarounds which may solve your auth issue.

I'll add the documentation parts.  I think I've deleted the necessary lines mentioned in the docs, my auth issue is just figuring out how to populated the 'fresh' db in the migration test.  I think I'll try something similar to initial_db in auth/database.py

I'm thinking I should redo that database dump actually. How thorough should the test be? I was thinking about adding one instance of every object since everything was upgraded.

Quoting Ian Ballou (2017-07-26 14:29:34)

I'm thinking I should redo that database dump actually. How thorough
should the test be? I was thinking about adding one instance of every
object since everything was upgraded.

Sounds good.

Pretty much done with the migration test. Only issue is with the User table, since the password hash is different every time a fresh database is created.  Thus, the migration always fails when comparing the user hashed password between the psql dump and the fresh DB. Does anyone know any workarounds?

Have a look at what I did for the headnode uuids

@zenhack  want to look at the latest commit?

Everything should be all set now.


GetFileFromPathForUserAsync doesn't list possible exceptions

The doc page for GetFileFromPathForUserAsync doesn't list the possible exceptions that it could throw. The example catches some exceptions but they are normally listed explicitly in the documentation. For example, the doc page for GetFileFromPathAsync lists the possible exceptions that could be thrown.

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

ID: df1cf6ee-4e1e-26d8-2724-f8cde062bf48
Version Independent ID: af6d83e3-911a-a2f9-75eb-51a81b8af54f
Content: StorageFile.GetFileFromPathForUserAsync(User, String) Method (Windows.Storage) - Windows UWP applications
Content Source: winrt-api-build/xml/Windows.Storage/StorageFile.xml
Product: uwp
Technology: winrt-reference
GitHub Login: @lastnameholiu
Microsoft Alias: alholiu

Hello. We apologize for the delay in resolving this issue. As we transition to a new feedback system, we are closing all outstanding issues. Any issues that can be addressed will be tracked using a different internal tool, and we regret that we won't be able to provide further updates on this particular matter. We value your input and assure you that all feedback is reviewed. We recommend visiting the Microsoft Q&A site, where your issue may have already been discussed.


feat(mv3): Redirection Tests

In this PR:

Adding support for MV3 specific interaction in Redirection Cycle.
Tests:

https://github.com/ipfs/ipfs-companion/blob/rc/3.0-mv3/test/functional/lib/ipfs-request-gateway-redirect.test.js added MV3 equivalent
https://github.com/ipfs/ipfs-companion/blob/rc/3.0-mv3/test/functional/lib/ipfs-request-protocol-handlers.test.js added MV3 equivalent.
https://github.com/ipfs/ipfs-companion/blob/rc/3.0-mv3/test/functional/lib/ipfs-request-dnslink.test.js This only tests if the redirect did not happen and the tab got refreshed after, right now refresh happens momentarily after. https://github.com/ipfs/ipfs-companion/pull/1239 changes that to a message on the bus to tell when to refresh page, the tests will be added along with that.
https://github.com/ipfs/ipfs-companion/blob/rc/3.0-mv3/test/functional/lib/ipfs-request-gateway-recover.test.js similar interaction is already testest in redirect testing

@whizzzkid note that CI tests are emitting an error: https://github.com/ipfs/ipfs-companion/actions/runs/5597804498/job/15162779434?pr=1236#step:11:97

Ok. I made some updates, so maintenance of tests should be a lot easier (addressing most of @lidel and my concerns).
https://github.com/ipfs/ipfs-companion/pull/1247 merges into https://github.com/ipfs/ipfs-companion/pull/1246 which merges into https://github.com/ipfs/ipfs-companion/pull/1236

@whizzzkid lmk when this is ready for me to peek at again!


pydantic.errors.PydanticImportError: pydantic:NoneStr has been removed in V2.

Installed via my PKGBUILD on Manjaro GNOME (unstable branch) using python-pydantic 2.3.0-1:
❯ gext list
Traceback (most recent call last):
  File "/usr/bin/gext", line 5, in <module>
    from gnome_extensions_cli.cli import run
  File "/usr/lib/python3.11/site-packages/gnome_extensions_cli/cli.py", line 10, in <module>
    from .commands import (
  File "/usr/lib/python3.11/site-packages/gnome_extensions_cli/commands/disable.py", line 7, in <module>
    from ..icons import Color
  File "/usr/lib/python3.11/site-packages/gnome_extensions_cli/icons.py", line 11, in <module>
    from .schema import AvailableExtension, InstalledExtension
  File "/usr/lib/python3.11/site-packages/gnome_extensions_cli/schema.py", line 11, in <module>
    from pydantic import BaseModel, Field, NoneStr  # pylint: disable=no-name-in-module
    ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/lib/python3.11/site-packages/pydantic/__init__.py", line 210, in __getattr__
    return _getattr_migration(attr_name)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/lib/python3.11/site-packages/pydantic/_migration.py", line 295, in wrapper
    raise PydanticImportError(f'`{import_path}` has been removed in V2.')
pydantic.errors.PydanticImportError: `pydantic:NoneStr` has been removed in V2.

For further information visit https://errors.pydantic.dev/2.3/u/import-error

Hi
It looks like you have installed gnome-extensions-cli with pydantic 2.x
If you take a look at the https://github.com/essembeh/gnome-extensions-cli/blob/47b5c80b7660362771c15910b0d81809de48a63c/pyproject.toml#L18
The pydantic requirement is >= 1.10.4 and < 2.0.0, see Poetry documentation
I don't know PKGBUILD, but you may have to configure your build to respect the poetry requirements and not the latest version?

Yes, I realized that this project requires an older version of Pydantic. I was letting you know as any distro that has moved on to 2.XX cannot use this program.
On rolling release distributions like Manjaro (based on Arch), packages are updated to the latest versions. If some package requires a legacy dependency version that cannot be updated to the new version, a legacy dependency mayh be kept around; however only as long as it's required.
To make this work, one could add a python-pydantic1 package at say v1.10.12 (last 1.XX version) package to the AUR (Arch User Repository) that this could depend on. I've let the AUR maintainer know. I was just curious and attempted to package it properly.

I know that pydantic 2 has been out for weeks now, there is not too much code to adapt, and I plan to upgrade it asap.
But keeping all the requirements up to date few days after their releases is an endless time consuming task.
That's why I would suggest to try installing python 3rd party  apps like gext using pipx.
It create isolated virtualenv for each app with needed version of requirements.
Maybe you'll find this tool as useful as I do ;)

Could you please test the branch bump-pydantic ?

Yes, it works.


Could this be implemented in PLpgSQL instead of Rust?

I think there are two barriers to adoption in the way this is written:

it has its own migration format that doesn't seem like we can seamlessly integrate with existing migrations that do arbitrary DDL queries like modify constraints and triggers
being Rust, we need separate build/deployment code to deploy it alongside our app

I wonder if this is necessary?  If there were PLpgSQL functions we could call to perform the various actions, then we could call those functions from our existing migration tools.  The actions could be passed to PLpgSQL functions in jsonb format.

Hi!
I'm designing Reshape to be a full-featured migration tool so by design it's not made to be called from other migration tools. The reason I went with a custom migration format is precisely to avoid arbitrary DDL queries. I want all migrations that Reshape performs to be zero-downtime without the user having to write those queries themselves, which avoids a lot of effort and potential foot guns.
Regarding having Reshape deployed separately, I'm not sure if that can be avoided given how it works. Every migration is a two-phase process: the migration is first started and then completed. In between, you can deploy your application changes gradually without worrying about breakages and the deployment can take however long it needs. This also means that Reshape has to be explicitly built into your deployment process to achieve true zero-downtime. My vision is that Reshape is built into your CI/CD (or other deployment process) and not your application.
Does this make sense? :)

I get what you mean but I still don't understand how it would be impossible to achieve your goals with PLpgSQL functions.
As far as arbitrary DDL queries, it's not like reshape would have to support performing arbitrary DDL via reshape.  A SQL migration file could contain DDL statements along with calls to PLpgSQL functions provided by reshape that only accept operations that reshape supports.
As far as the two-phase process, assuming that reshape stores migration status in the database (if it doesn't, why not?) is there any reason the deployment process couldn't call PLpgSQL functions to perform migrations, instead of running a Rust executable??

Sorry, should have been a bit more clear. I don't think there is anything stopping Reshape from being implemented in PL/pgSQL. Given the things I mentioned in the previous comment, I believe it's simpler though to have it implemented as a separate client rather than embedded in the database. Simpler both for end users who can use Reshape like most other migration tools (write some files and run a CLI script) and for me as a maintainer as PL/pgSQL is unpleasant and hard to version.

As far as the two-phase process, assuming that reshape stores migration status in the database (if it doesn't, why not?) is there any reason the deployment process couldn't call PLpgSQL functions to perform migrations, instead of running a Rust executable? Running migrations either way requires access to the db.

It does store them in the database so no reason why that wouldn't work but the same reasoning as above applies.

(It seems overambitious to make Reshape be a full-featured migration tool that supports absolutely any possible change one would wish to make to the DB? I mean what about creating and altering domains, enum types, triggers, rules, altering views themselves, creating extensions, etc...)

I aim to eventually support all the changes that actually can be achieved in zero-downtime (enum changes for example). For things were zero-downtime changes can't be automated or are custom in some way, I'm planning on adding an escape hatch to let users run arbitrary SQL queries. Using this escape hatch would of course be at the users own risk.
Ruby on Rails migrations work similar to this with a DSL for most changes and the option to write custom SQL for whatever isn't supported. In their case the DSL isn't strictly necessary though as they don't do much more than run the equivalent DDL directly: https://edgeguides.rubyonrails.org/active_record_migrations.html.

@fabianlindfors well, it's your project of course, for what it's worth I'd be more inclined to use reshape if I have the option of integrating it with my existing migration system.  Which would mean something like

SELECT reshape_add_migration('migration_name', $$[
  {
    "actions": {
      "type": "create_table",
      "table": "users",
      "primary_key": "id",
      "children": [
        {
          "actions.columns": {
            "name": "id",
            "type": "INTEGER",
            "generated": "ALWAYS AS IDENTITY"
          },
        },
        {
          "actions.columns": {
            "name": "name",
            "type": "TEXT"
          }
        }
      ]
    }
  },
  {
    "actions": {
      "type": "add_index",
      "table": "users",
      "name": "name_idx",
      "columns: ["name"]
    }
  }
]$$::jsonb);

And then later
SELECT reshape_migrate();
-- ...
SELECT reshape_complete();

I appreciate the feedback! It would be a bit hard to implement this now with the current design but I'll keep it in mind for the future. It's still early in development so if I hear this from more users, I might change direction  :)

Yup!  I definitely think it's a neat project, I was just like...hrm, darn, wish it was a bit less of a hassle to integrate with my existing stuff.  Though I know you discouraged using it in production

Thanks! Using it in production right now is definitely not advised but I appreciate anybody willing to take it for a spin and offer some feedback. The more people doing that, the faster Reshape can be production-ready.
If you every do take Reshape for a spin on something else, I'd love to hear your thoughts!


[UI] The Learn Git And Github page is not Responsive

Category

[ ] Resource Addition
[ ] Documentation
[ ] Codebase
[x] User Interface
[ ] Feature Addition

Description
The Learn Git And Github page is not Responsive.

@Sriparno08 assign me this issue


upgrade to rocksdb 5.6.1

@siddontang @zhangjinpeng1987 @follitude PTAL
Signed-off-by: Cholerae Hu<EMAIL_ADDRESS>

LGTM

It's not just about ci. Should update rust-rocksdb too.

rust-rocksdb updated.

travis-build/Makefile should be updated too.

Updated.


AttributeError while downloading (transcoding?) from Livestream.com

The following happened:
[http @ 0x1e2b2a0] HTTP error 404 Not Foundtime=316.85 bitrate=2126.1kbits/s
http://api.new.livestream.com/accounts/9197973/events/3195646/videos/57095862.m3u8?dw=0&hdnea=st=1405970094~exp=1405970994~acl=/i/9197973_1198167_1cd4ee13_1@43651/*~hmac=f45e90d8234b39a08570cbeb699ffcfc836324ae7b40ce168f5eb61139b35a26: Input/output error
frame= 9556 fps=104 q=-1.0 Lsize=   83452kB time=319.95 bitrate=2136.7kbits/s
video:78064kB audio:5094kB global headers:0kB muxing overhead 0.353446%
[avconv] 85454550 bytes
[download] Downloading video #2 of 10
Traceback (most recent call last):
  File "/usr/bin/youtube-dl", line 9, in <module>
    load_entry_point('youtube-dl==2014.07.21', 'console_scripts', 'youtube-dl')()
  File "/usr/lib/python2.7/site-packages/youtube_dl/__init__.py", line 857, in main
    _real_main(argv)
  File "/usr/lib/python2.7/site-packages/youtube_dl/__init__.py", line 847, in _real_main
    retcode = ydl.download(all_urls)
  File "/usr/lib/python2.7/site-packages/youtube_dl/YoutubeDL.py", line 1052, in download
    self.extract_info(url)
  File "/usr/lib/python2.7/site-packages/youtube_dl/YoutubeDL.py", line 527, in extract_info
    return self.process_ie_result(ie_result, download, extra_info)
  File "/usr/lib/python2.7/site-packages/youtube_dl/YoutubeDL.py", line 645, in process_ie_result
    extra_info=extra)
  File "/usr/lib/python2.7/site-packages/youtube_dl/YoutubeDL.py", line 564, in process_ie_result
    return self.process_video_result(ie_result, download=download)
  File "/usr/lib/python2.7/site-packages/youtube_dl/YoutubeDL.py", line 770, in process_video_result
    format['ext'] = determine_ext(format['url']).lower()
  File "/usr/lib/python2.7/site-packages/youtube_dl/utils.py", line 855, in determine_ext
    guess = url.partition(u'?')[0].rpartition(u'.')[2]
AttributeError: 'NoneType' object has no attribute 'partition'

Command run:
youtube-dl http://new.livestream.com/internetsociety2/hopex2

Latest version, installed from pip minutes ago. youtube-dl failed to continue after this uncaught exception, breaking the download of the remainder of the channel. I can reproduce the problem reliably; it always breaks on this video.

It's working now for me, so I'm closing the issue. If you can reproduce the error with the last youtube-dl version, please comment here or open a new issue.


Bandwidth

Implement bandwidth measurement
Implement mock latency and signal strength measurement
Implement mock location
Perform measurements and report them to the backend when receiving an FCM query

closes #27


[EX-6] Implement peers in Jellyfish RN client

Please focus on the js/ts code, native stuff is just there because of the new deps. I'll try to bump deps in a separate PR since this is confusing enough already.

Also you've probably tried to update RN and changed some native code but not updated all the way. You're making a bit of a mess imho. If you need help with this I'm available.


Would you display the details when the compile failed?

It's hard to find out the reason of compile failed through restler-fuzzer/Compile/StdErr.txt, this file only contained the exception of source code, It would be better if the error can point to a specific api file.
eg.
Unhandled exception. System.Exception: non-array should always have a property name
   at Restler.Compiler.SwaggerVisitors.GenerateGrammarElements.extractPropertyFromObject(String propertyName, JsonObjectType objectType, FSharpOption`1 exampleObj) in /home/horsefall/work/restler-fuzzer/src/compiler/Restler.Compiler/SwaggerVisitors.fs:line 177
   at Restler.Compiler.SwaggerVisitors.declaredPropertyParameters@389.Invoke(KeyValuePair`2 item) in /home/horsefall/work/restler-fuzzer/src/compiler/Restler.Compiler/SwaggerVisitors.fs:line 392
   at Microsoft.FSharp.Collections.Internal.IEnumerator.choose@171.System-Collections-IEnumerator-MoveNext() in F:\workspace\_work\1\s\src\fsharp\FSharp.Core\seq.fs:line 179
   at Microsoft.FSharp.Core.CompilerServices.RuntimeHelpers.takeInner@266[T,TResult](ConcatEnumerator`2 x, Unit unitVar0) in F:\workspace\_work\1\s\src\fsharp\FSharp.Core\seqcore.fs:line 268
   at Microsoft.FSharp.Collections.SeqModule.IsEmpty[T](IEnumerable`1 source) in F:\workspace\_work\1\s\src\fsharp\FSharp.Core\seq.fs:line 702
   at Restler.Compiler.SwaggerVisitors.generateGrammarElementForSchema(JsonSchema schema, FSharpOption`1 exampleValue, Boolean generateFuzzablePayloadsForExamples, FSharpList`1 parents, FSharpFunc`2 cont) in /home/horsefall/work/restler-fuzzer/src/compiler/Restler.Compiler/SwaggerVisitors.fs:line 479
   at Restler.Compiler.Main.allResponseProperties@637.GenerateNext(IEnumerable`1& next)
   at Microsoft.FSharp.Core.CompilerServices.GeneratedSequenceBase`1.MoveNextImpl() in F:\workspace\_work\1\s\src\fsharp\FSharp.Core\seqcore.fs:line 371
   at Microsoft.FSharp.Core.CompilerServices.GeneratedSequenceBase`1.System-Collections-IEnumerator-MoveNext() in F:\workspace\_work\1\s\src\fsharp\FSharp.Core\seqcore.fs:line 403
   at Microsoft.FSharp.Collections.SeqModule.TryHead[T](IEnumerable`1 source) in F:\workspace\_work\1\s\src\fsharp\FSharp.Core\seq.fs:line 1370
   at Restler.Compiler.Main.requestDataSeq$cont@601(Config config, String ep, KeyValuePair`2 m, RequestId requestId, FSharpOption`1 exampleConfig, Unit unitVar) in /home/horsefall/work/restler-fuzzer/src/compiler/Restler.Compiler/Compiler.fs:line 645
   at Restler.Compiler.Main.getRequestData@557-1.GenerateNext(IEnumerable`1& next) in /home/horsefall/work/restler-fuzzer/src/compiler/Restler.Compiler/Compiler.fs:line 601
   at Microsoft.FSharp.Core.CompilerServices.GeneratedSequenceBase`1.MoveNextImpl() in F:\workspace\_work\1\s\src\fsharp\FSharp.Core\seqcore.fs:line 371
   at Microsoft.FSharp.Core.CompilerServices.GeneratedSequenceBase`1.System-Collections-IEnumerator-MoveNext() in F:\workspace\_work\1\s\src\fsharp\FSharp.Core\seqcore.fs:line 403
   at Microsoft.FSharp.Core.CompilerServices.RuntimeHelpers.takeInner@266[T,TResult](ConcatEnumerator`2 x, Unit unitVar0) in F:\workspace\_work\1\s\src\fsharp\FSharp.Core\seqcore.fs:line 268
   at System.Collections.Generic.List`1..ctor(IEnumerable`1 collection)
   at Microsoft.FSharp.Collections.SeqModule.ToArray[T](IEnumerable`1 source) in F:\workspace\_work\1\s\src\fsharp\FSharp.Core\seq.fs:line 825
   at Restler.Compiler.Main.generateRequestGrammar(FSharpList`1 swaggerDocs, MutationsDictionary dictionary, Config config, FSharpList`1 globalExternalAnnotations, FSharpOption`1 userSpecifiedExamples) in /home/horsefall/work/restler-fuzzer/src/compiler/Restler.Compiler/Compiler.fs:line 704
   at Restler.Workflow.generateGrammarFromSwagger(String grammarOutputDirectoryPath, FSharpOption`1 swaggerDoc, Config config) in /home/horsefall/work/restler-fuzzer/src/compiler/Restler.Compiler/Workflow.fs:line 156
   at Restler.Workflow.generateRestlerGrammar(FSharpOption`1 swaggerDoc, Config config) in /home/horsefall/work/restler-fuzzer/src/compiler/Restler.Compiler/Workflow.fs:line 247
   at Program.main(String[] argv) in /home/horsefall/work/restler-fuzzer/src/compiler/Restler.CompilerExe/Program.fs:line 37

Does anyone know non-array should always have a property name means in a swagger file?

Hello @h0rs3fa11,
This means that there is a type for which RESTler expects a property name, but it did not find one in the spec.  Arrays are an exception, because the array element does not have a name.  There may be another case missing.
Could you please build the mp/diag branch and paste what is printed in the exception?  I added more diagnostic info to see which type is failing.
We'll also add the diagnostic on which method and endpoint is failing, but this will take a little more time.
Thanks,
Marina

It's strange, I compile succeeded after pulling the newest code

That's strange.  Was it an older version, or the commit right before?
(I'd like to keep this issue open until the additional logging is added, so please do not close it.)
Thanks!

I don't remember the version that I used before. And I will not to close this issue.
Thank you for your reply!


[📚] Your documentation feedback title (Firebase Dynamic Links Domain Not Configured Error on iOS)

Please describe your documentation issue or suggested improvement in detail here and provide links to any pre-existing/relevant documentation and screenshots if necessary:
The error message "DYNAMIC_LINK_NOT_ACTIVATED : FDL domain is not configured" means that the Dynamic Links domain for your project is not configured. This is required for iOS, but not for Android.
Dynamic Links are a feature of Firebase that allow you to create links that can open your app on any platform. When you send an email verification link to a user, Firebase will use Dynamic Links to open the link in the user's app, regardless of the platform.
If the Dynamic Links domain for your project is not configured, then Firebase will not be able to create the link. This will cause the error message that you are seeing.
To fix this error, you need to configure the Dynamic Links domain for your project. You can do this in the Firebase Console. Once you have configured the domain, you should be able to send email verification links without any errors.
As for why Dynamic Links are not recommended anymore, it is because Firebase has a new feature called App Links. App Links are a more modern and secure way to open your app on any platform. If you are not using Dynamic Links for anything else, then you can safely disable them.
Here are the steps on how to configure the Dynamic Links domain for your project:
Go to the Firebase Console.
Click on the "Dynamic Links" tab.
Click on the "Domain settings" tab.
Enter the domain name for your project.
Click on the "Save" button.
Once you have configured the domain, you should be able to send email verification links without any errors.
I hope this helps!

Hi @Ramghosh16RG, are you responding to a particular issue or reporting an issue with the documentation? If the former, please comment on that issue instead. If the latter, kindly share which part of the documentation you are reporting the issue with. Your issue as it stands now is not clear on what it is trying to achieve.
Thank you


fix: don't add service name inside

A service name may contain an invalid character.

r? @yuaanlin


Add option to hide npc aggro lines when out of combat. SupahSoftware

This adds a toggle setting in the NPC Aggression Timer plugin to hide the aggro lines when not in combat.

@Adam- Sorry to bug you again. Do I need to do anything to get the workflow started and get this merged in?


fix imports order

I hereby agree to the terms of the CLA available at: https://yandex.ru/legal/cla/?lang=en

Pull request type

Please check the type of change your PR introduces:

[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting, renaming)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] Documentation content changes
[x] Other (please describe):

What is the current behavior?
import style only

Codecov Report

Merging #284 (1275d5e) into master (1ef9790) will decrease coverage by 17.16%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##           master     #284       +/-   ##
===========================================
- Coverage   57.63%   40.46%   -17.17%
===========================================
  Files         133      133
  Lines       15350    15350
===========================================
- Hits         8847     6212     -2635
- Misses       5951     8707     +2756
+ Partials      552      431      -121

Flag
Coverage Δ

1.14.x
28.95% <ø> (ø)

1.15.x
28.95% <ø> (ø)

1.16.x
28.95% <ø> (ø)

1.17.x
40.46% <ø> (-17.11%)
:arrow_down:

1.18.x
28.95% <ø> (-0.08%)
:arrow_down:

connection
12.81% <ø> (ø)

discovery
11.96% <ø> (ø)

e2e
20.97% <ø> (-24.92%)
:arrow_down:

integration
20.97% <ø> (-24.92%)
:arrow_down:

macOS-latest
28.95% <ø> (ø)

ratelimiter
14.76% <ø> (ø)

scripting
15.33% <ø> (ø)

table
?

ubuntu-latest
40.46% <ø> (-17.11%)
:arrow_down:

unit
28.95% <ø> (-0.08%)
:arrow_down:

windows-latest
?

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

Impacted Files
Coverage Δ

internal/conn/grpc_client_stream.go
40.50% <ø> (-21.52%)
:arrow_down:

internal/repeater/repeater.go
82.72% <ø> (-7.28%)
:arrow_down:

internal/table/session.go
43.41% <ø> (-32.34%)
:arrow_down:

internal/scheme/config/config.go
6.25% <0.00%> (-81.25%)
:arrow_down:

internal/deadline/context.go
0.00% <0.00%> (-80.00%)
:arrow_down:

internal/table/statement.go
0.00% <0.00%> (-75.00%)
:arrow_down:

log/table.go
0.62% <0.00%> (-69.06%)
:arrow_down:

internal/table/config/config.go
1.29% <0.00%> (-63.64%)
:arrow_down:

trace/table_gtrace.go
0.08% <0.00%> (-63.03%)
:arrow_down:

table/table.go
7.31% <0.00%> (-60.17%)
:arrow_down:

... and 47 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 1ef9790...1275d5e. Read the comment docs.


Monitored item delivers errors in clients log.

Monitoring "Item\Trend 000001" with UAExpert delivers every second this log-messages in UAExpert's log output:
Error: Number of acknowledgement results (0) does not match number of acknowledgements (2)
Error: UaSessionPrivate::publishCallback - List of available sequence numbers is empty, at least current sequence number has to be available

Fixed: https://github.com/nauful/LibUA/commit/4a2ee9a48438d8aceeca55840320a05c2cb440a6

How do you reproduce this? Acknowledgements are being sent back, perhaps a configuration issue that I can investigate.

Closing, could not reproduce.


Enable MBEDTLS_AES_ROM_TABLES in config-no-entropy

Enable the MBEDTLS_AES_ROM_TABLES option in the
configs/config-no-entropy.h to place AES lookup tables in ROM. This
saves considerable RAM space, a resource that is very limited in small
devices that use this configuration.
NOTES:

This change does not need to be backported to mbed TLS 1.3 and 2.1
This change is related to this mbed OS PR that requires MBEDTLS_AES_ROM_TABLES to be defined for some tests to pass due to limited RAM in some devices.

@andresag01 MBEDTLS_AES_ROM_TABLES  saves RAM, but htis means that the device itself should have the AES tables as part of their ROM code. Enabling this configuration will reduce RAM space, but if hte devices won't have the tables in ROM, AES will simply not work.
Am I missing something?

sorry, I was confused between ROM code. forget my comment

@RonEld: Thanks for reviewing and good point, I have added a ChangeLog entry.

Reviewed, approved and merged.


AKS publishing tools fails on VS 2017 Enterprise 15.6.5

AKS publishing tool installation fails with message This extension is not installable on any currently installed products. I am running VS 2017 Enterprise 15.6.5. The installation log has following message

Supported Products :
Microsoft.VisualStudio.Community
: [15.0.27701.1,16.0)

@lisaguthrie Have you seen this issue before? Any suggestions for Rahul?

I am experiencing same issue. Not seen any response to this thread.

@rahulvmarathe -- Rahul, it looks like you’re having an issue with installation. If your Visual Studio installation fails, see Troubleshooting Visual Studio installation and upgrade issues for step-by-step guidance.
We also offer a Live Chat (English only) support option for installation-related issues.
Here are a few more support options:
-Report product issues to us via the Report a Problem tool that appears both in the Visual Studio Installer and in the Visual Studio IDE.
Suggest a feature, track product issues, and find answers in the Visual Studio Developer Community.
Use your GitHub account to talk to us and other Visual Studio developers in the Visual Studio conversation in the Gitter community.


Milliseconds since epoch are not formatted as integers.

Fiddle here http://jsfiddle.net/tutuca/e3s21Lab/, although it fails to run.
I have packaged the minimal case that you can run (download)
So, the gist of it is that moment().format('x') doesn't return a valid integer so sinon.fakeTimer fails with a non obvious error (I'd probably report it there too.)
describe("Test sinon.fakeTimes with Moment.js", function() {
    var today, tomorrow, ticks, clock;
    beforeEach(function(){
        // get milliseconds since UNIX epoch for this date.
        ticks = parseInt(moment('01/15/2015').format('x'));
        // set the timers to start at expected time.
        clock = sinon.useFakeTimers(ticks);

    });
    afterEach(function(){
        clock.restore();
    });
    it("date is set in some arbitrary time.", function() {
        today = new moment();
        tomorrow = today.add(1, 'days');
        expect(tomorrow.format('MM/DD/YYYY')).toBe('01/16/2015');
    });
});

See how I needed to wrap the time with parseInt?
Thing is, that format returns something that looks like a number, so it should be a number so I don't have to rely on parseInt for it to work, no?
Opinions?

It definitely needs to stay a string. format() returns a string representation of the date, and it would be really confusing if it did that for every format string except one. Further, a user might even combine "x" with some other piece of formatting and then it would have to be a string anyway, so the special case here would be "if the format string only contains 'x', return a number, otherwise a string", which is not a good idea.
However, getting an integer is really easy and doesn't require parseInt: use moment().valueOf() or even the cheekier but shorter +moment(). If you wanted seconds instead of millis, there's moment.unix(). We got you covered!

interesting all of the methods moment().month() returns an integer


mod: add wizard for playground

What problem does this PR solve?
What is changed and how it works?
Check List
Tests

Unit test
Integration test
Manual test (add detailed scripts or steps below)
No code

Code changes

Has exported function/method change
Has exported variable/fields change
Has interface methods change
Has persistent data change

Side effects

Possible performance regression
Increased code complexity
Breaking backward compatibility

Related changes

Need to cherry-pick to the release branch
Need to update the documentation

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@e4ac3d4). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master      #61   +/-   ##
=========================================
  Coverage          ?   36.57%
=========================================
  Files             ?       22
  Lines             ?     1080
  Branches          ?        0
=========================================
  Hits              ?      395
  Misses            ?      621
  Partials          ?       64

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 e4ac3d4...d8d11c2. Read the comment docs.


[Feature Request] Formalize the Console Backend

What is the feature ?

Now we just demonstrated it's possible draw plots on console without actually changing the API in this example.
But it seems we still needs to tweak something and make it a built-in backend.

Why do you need the feature ?

Plotting in console may be very interesting for someone who use SSH which doesn't have a easy to access the image.

What is the use case ? See above

We can make improvement on https://github.com/plotters-rs/plotters-console


ci: split tasks

Still hacky but i'm getting fed up of the CI debugging..

ACK 7dcf3001b99c6a266c112a7bb1fd95d126d3408f

ACK e9c1e94136a77a56a1f3a6b70f05591a2582996e


Interfaces realMouseUpOptions and realMouseDownOptions are inconsistently cased with other interfaces in the project

In order to maintain consistency across exported types, realMouseUpOptions and realMouseDownOptions should use pascal case. This is also the recommended/documented casing across the official TypeScript handbook.
Thanks for your work on this project! Hugely helpful.

:tada: This issue has been resolved in version 1.13.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Create script to automate milestone creation and migration across sprints

I'm tired of manually creating milestones and moving issues.
Fix it.

Does zenhub already support this? https://www.zenhub.com/guides/managing-cross-repo-milestones

Well, that's convenient! Thanks, @dlebauer.


SyntaxError: await is only valid in async functions and the top level bodies of modules

Using almost identical code from here, I get the error in the title. Of course, I replaced the filenames with my own, but no luck.
I also imported both eloquent-ffmpeg and fs so I really don't know what the issue is considering, from what I can see, there's nothing different.
var ffmpeg = require('eloquent-ffmpeg'),
  fs = require('fs');

The answer is in the error message, commonjs doesn't support top-level awaits so you have to wrap your code in an async function.
async function convertMyVideo() {
  // ...
}
convertMyVideo();

// This is fine but note that ffmpeg will be a namespace not a function
var ffmpeg = require('eloquent-ffmpeg');
// If you just want the ffmpeg function
var { ffmpeg } = require('eloquent-ffmpeg');

I see, what is commonjs? Does this mean there is a way to avoid needing to wrap it in an async function? I have started learning JS recently so apologies for the basic questions. :]

Node.js supports commonjs modules, which use require and exports and ES6 modules, which use import and export. You can use es modules instead of commonjs by adding "type": "module" to your package.json or by changing the file extension from .js to .mjs. I recommend you read the official guides Node.js ECMAScript modules and Node.js commonjs modules.

Thank you, king. And now for my final question.......
i have currently this bit of code:
        async function streamVideo() {
            const cmd = ffmpeg();
            cmd.input(fs.createReadStream(stream));
            cmd.output(fs.createWriteStream('./'+Date.now()+'.flv'))
                .format('flv');
            const proc = await cmd.spawn();
            await proc.complete();
            console.log("Stream has finished.");

cmd.input(fs.createReadStream(stream)); is supposed to launch streamlink for nodejs and read its buffer output (I had to manually modify it to do this), like so:
	let link = message.match(/[-a-zA-Z0-9@:%_\+.~#?&//=]{2,256}\.[a-z]{2,4}\b(\/[-a-zA-Z0-9@:%_\+.~#?&//=]*)?/gi);
        let stream = new Streamlink(link).quality('best').pipe();

The modification I made to the streamlink script was adding this pipe()... method? function? I'm still learning -w-:
   pipe(done) {
      this.isLive((live)=>{
         if(!live){
            this.emit('err', 'Is not live.');
            return;
         }
         var args = [];
         if(!this.outputLoc){
            args.push('-O');
         }
         args.push(this.stream);
         args.push(this.qual+',worst');
         this.startTime = Math.floor(Date.now()/1000);

         this.live = spawn('streamlink', args);
          this.live.stdout.on('data', (d)=>{
             console.log(d);
         });

         this.live.on('close', (code, st)=>this.end(code, st));
      })
      return this;
   }

Running these though, I get:
TypeError [ERR_INVALID_ARG_TYPE]: The "path" argument must be of type string or an instance of Buffer or URL. Received an instance of Streamlink

So it appears to not be recognising it as a buffer. Very weird! :(


clearAxis bug

Create a Surface trace, add a couple of columns to the Z key, then try to clear that dropdown... app gets into inconsistent state.

Could not reproduce


Resize

Hi, Sorry to bother you once again but I have 2 issues for which I wanted your help.

when I open neovim the dashboard appears as the center but upon resizing the terminal  while keeping neovim open the dashboard does not adjust itself at the center.
you can see in the screenshots below the column numbers and all written. I want that to be removed . Thanks

fix it later


Examples: section on engineering design

Engineering design is a big application area of convex optimization. It would be great to have a section in the examples on just this one topic.

There are some engineering design problems here:
https://web.stanford.edu/~boyd/cvxbook/bv_cvxbook_extra_exercises.pdf


place autocomplete not working

Hello,
place autocomplete feature seems not working.
In #80 place autocomplete support was added, and it also added IvoryGoogleMapBundle:Form:places_autocomplete_widget.html.twig to twig.form.resources. Later in #159 all this stuff was removed from DependencyInjection/IvoryGoogleMapExtension.php.
Now we have {% block places_autocomplete_widget %} in https://github.com/egeloen/IvoryGoogleMapBundle/blame/master/Resources/views/Form/place_autocomplete_widget.html.twig#L1 but block prefix in https://github.com/egeloen/IvoryGoogleMapBundle/blob/master/Form/Type/PlaceAutocompleteType.php#L144 is place_autocomplete (difference in additional S at the end of place).
Downside is that form theme is not loaded automatically, but after all if you add it manually to your config/template it is not used because of bad block prefix name.
Am I misunderstanding something and using this bundle not properly or is it bug? If so just let me know and maybe I can submit PR with fix.

Thanks for reporting it! Indeed, I forgot to remove the "s" from the twig block when refactoring... I should probably introduce more robust tests about templates...
Be aware, the form_theme is still loaded automatically but via a compiler pass: https://github.com/egeloen/IvoryGoogleMapBundle/blob/master/DependencyInjection/Compiler/RegisterFormResourcePass.php

Thanks for information. Compiler pass is working as you wrote, so probably I had some mess in code before which results in not working properly.
Fix is also submitted.


fix BSSingleton.source access with multi-thread

dispatch_apply(1000, dispatch_get_global_queue(DISPATCH_QUEUE_PRIORITY_DEFAULT, 0), ^(size_t idx) {
[inject getInstance:[TestClass class]]; // which bind to BSSingleton scope, because BSSingleton's source is nonatomic, will crash inside [scope scope:provider], the BSSingleton BSScope protocol implement.
});

We are seeing a bug where multiple background thread activities are causing a crash with the stack trace as seen below. Can we get someone to talk about this fix for the ones that involve the [BSSinglton scope:] call? With regards to inFlightKeys I am wondering if it's related to the Thread specific dictionary. Let me know what you think.

the BSSingleton scope, because BSSingleton's source is nonatomic, [scope scope:provider] will try to use the source of BSSingleton, step is

retain it
use it
set the source to nil
release it

But in the multi-thread environment, [singleton scope:] will set the singleton's source property, [singleton provide:injector:] will reset the source to nil if instance is nil, and the source property is not marked as atomic, so getter and setter will case the multi-thread problem.


[improve][monitor]Add prometheusRawMetricsProvider support

Motivation
Currently, Pulsar Prometheus metric framework can generate new metrics from PrometheusRawMetricsProvider, and the current PrometheusMetricsProvider implementation doesn't support this interface, which leads to new created PrometheusMetricProvider instance can not integrate with the current metric system.
Modification
Pulsar prometheusMetricsProvider supports the following features.

Support label
Support PrometheusRawMetricsProvider interface.

This Pr just migrated from the BookKeeper Prometheus metrics provider.
Documentation
Check the box below or label this PR directly.
Need to update docs?

[x] doc-required
(Your PR needs to update docs and you will update later)

[ ] doc-not-needed
(Please explain why)

[ ] doc
(Your PR contains doc changes)

[ ] doc-complete
(Docs have been already added)

Matching PR in the forked repository
PR in forked repository: https://github.com/hangc0276/pulsar/pull/4

@asafm @tjiuming  Would you please help take a look? thanks.

@asafm @tjiuming Would you please help take a look? thanks.

Done @hangc0276

For future readers of this PR:
Why is this PR needed?
Pulsar uses Bookkeeper Client, which has metrics. The way to get those metrics is to implement the BK Metric library API, which Pulsar has already done through PrometheusMetricsProvider, PrometheusStatsLogger, etc.
The problem with that implementation was that the API itself supported having labels per metric, but the implementation ignored the labels provided to it.
This PR aims to fix and add support for the labels given to it via the API methods.
The impact is that now Pulsar /metrics endpoint, which exposes the Pulsar metrics in Prometheus format, will contain the Bookkeeper client metrics with labels (which were previously stripped).

@asafm @tjiuming Would you please help take a look? thanks.

Done @hangc0276

@asafm Thanks for your review, I addressed all the comments, please help take a look again, thanks a lot.

For future readers of this PR:
Why is this PR needed? Pulsar uses Bookkeeper Client, which has metrics. The way to get those metrics is to implement the BK Metric library API, which Pulsar has already done through PrometheusMetricsProvider, PrometheusStatsLogger, etc. The problem with that implementation was that the API itself supported having labels per metric, but the implementation ignored the labels provided to it.
This PR aims to fix and add support for the labels given to it via the API methods. The impact is that now Pulsar /metrics endpoint, which exposes the Pulsar metrics in Prometheus format, will contain the Bookkeeper client metrics with labels (which were previously stripped).

@asafm Thanks for your explanation. Another purpose of this Pr is to support plugin metrics integrate with Pulsar broker's metrics through the interface.
https://github.com/apache/pulsar/blob/63d4cf20e7b9c9bd24d3fcd5ba7397f0d185ce57/pulsar-broker/src/main/java/org/apache/pulsar/broker/PulsarService.java#L904-L914
https://github.com/apache/pulsar/blob/63d4cf20e7b9c9bd24d3fcd5ba7397f0d185ce57/pulsar-broker/src/main/java/org/apache/pulsar/broker/PulsarService.java#L1673-L1682

How do you provide such support in your PR - that I haven't figured out yet

BTW: Force push is not good - I lose all context - I can't see only the changes you've added

BTW: Force push is not good - I lose all context - I can't see only the changes you've added

Sorry, I just rebased the master, and it needs force push, otherwise, the push will fail

This PR changed the metric name for BK client, and it can only be released in the major version and needs to update the doc. /cc @Anonymitaet

I just wandering that why don't we use Prometheus lib, I think this implementation is hard to maintain


log: Remove unused V function

This function does not seem to be used, and as it import "golang/glog"
it can conflict with other packages using the "-v" flag globally.

Actually, that won't work vitess is using glog as well.


Feature request: Infected attachments (doc, xlsx etc)

In the readme I only saw "Auto renaming attachments on the fly" but not other mentions of attachments.
Would be interesting to know how much of your users will enable macro's for example.
Macro could show scareware or an educational message for the end-user.

You can use {{CID}} to name attachments on-the-fly to uniquely identify them. If you use doc or excel file attachment, the macro can be used to read own file name. Check https://stackoverflow.com/questions/1895616/how-to-get-the-excel-file-name-path-in-vba/1896057
Then you can use macro to send web request to the Quick tracker link. Eg: https:///qt?tid=xxx&cid=
Then the Quick tracker tracks attachment open.


In use version set and import map lockfile extraction

Currently the lockfile version extraction from import maps used in the htmlGenerate function works best with the direct scoped form of import maps which is not the default output (where scopes correspond to package boundaries for versioning resulting in duplication).
Instead the version resolver should be updated in the installer to deal in terms of an "in use package version set", where resolutions are prioritised from this pool based on the resolution rules, using new versions only when semver forces this or when the latest: true option is used.
The version collection from the import map would then work seamlessly with the version resolution in prepopulating this in use version set.
//cc @vovacodes this feature will likely solve your use case for the lockfile here.

Added in https://github.com/jspm/generator/pull/134.


http-01 challenge: disable TCP connection reuse

If TCP connection reuse (KeepAlives) are left on then, in a sequence
of challenges arising from a multiple-domain certficate, challenges
after the 1st can fail due to reusing the now defunct tcp connection
used in the first challenge. This has been observed when using the Go
standard library reverse proxy to forward the challenges to Lego.
Fixes #107

LGTM.


How to connect

How does an outside device connect to this..
all i get on starting the server is:
index.js:69 Listening on :::41574
but when i use my local ip from my network it says:
VM3493:1 WebSocket connection to 'ws://<IP_ADDRESS>:41574/' failed: Error during WebSocket handshake: Unexpected response code: 404

What's an outside device for you?
Your client needs to be on the same network.
Also you need a pure web socket client implementation to be able to connect.
socket.io's client won't do as far as I know for instance.

I've same problem and try it with socket.io and with pure WebSocket but the result was the same - the 'handshake error 404'.
And I've try to remove some properties from 'start' method options: "origins", "protocols" and "tcpNoDelay" (I've copypasted them from README.md example) and It works for me
HIH


Bug: OP_VERIFY (and equivalents) in some cases are incorrect when disproving.

Description
Since the groth16 verifier program was written for normal execution some of the OP_VERIFY and OP_EQUALVERIFY checks are incorrect when included in the segmented version and taproot.
Example
OP_VERIFY in hinted_check_tangent_line.
        let script = script! {                    // x, y
            { Fq::copy(0) }                                         // x, y, y
            { Fq::double(0) }                                       // x, y, 2y
            { hinted_script1 }                                      // x, y, alpha * (2 * y)
            { Fq::copy(2) }                                         // x, y, alpha * (2 * y), x
            { hinted_script2 }                                      // x, y, alpha * (2 * y), x^2
            { Fq::copy(0) }                                         // x, y, alpha * (2 * y), x^2, x^2
            { Fq::double(0) }                                       // x, y, alpha * (2 * y), x^2, 2x^2
            { Fq::add(1, 0) }                                       // x, y, alpha * (2 * y), 3 * x^2
            { Fq::neg(0) }                                          // x, y, alpha * (2 * y), -3 * x^2
            { Fq::add(1, 0) }                                       // x, y, alpha * (2 * y) - 3 * x^2
            { Fq::is_zero(0) } OP_VERIFY                            // x, y
            { hinted_script3 }
        };

When Fq::is_zero(0) puts 0 on the stack the script fails due to the OP_VERIFY. In that case the disprove script should be executable on-chain.
Solution
Replacing OP_VERIFY with OP_SUCCESS ~(even automatically) might be possible~ is NOT possible because OP_SUCCESS marks any transaction valid without exeuting the script.
~We need to negate the OP_VERIFY input but~ have to be careful to not negate it when it is used in the Winternitz and hashing part of the script and when it is used to validate a hint.

An example where we do not negate the OP_VERIFY is in tmul as a hint is verified there.
// Correction/validation
// r = if q < 0 { r + p } else { r }; assert(r < p)
{ T::push_u32_le(&Fq::modulus_as_bigint().to_u32_digits().1) } // {MODULUS} -> {r} {0/1}
{ T::fromaltstack() } OP_FROMALTSTACK // {MODULUS} {r} {0/1}
OP_IF { T::add_ref(1) } OP_ENDIF      // {MODULUS} {-r/r}
{ T::copy(0) }                        // {MODULUS} {-r/r} {-r/r}
{ T::lessthan(0, 2) } OP_VERIFY       // {-r/r}

We noticed that negating the flag and using OP_VERIFY doesn't give us what we need and thought about it a bit more.
Let's assume the operator creates a proof such that all bit commitments are correct and all parts of the script can be executed with the exception that it fails at the OP_VERIFY in hinted_check_tangent_line.
For a verifier to be able to show that the OP_VERIFY failed, the disprove script has to be adjusted to be spendable when the stack element consumed by OP_VERIFY (let's call it the flag) is false.
However, there are other OP_VERIFY when we check that the segment's blake3 hashes for the input are not equal which in our setting wouldn't allow the script to be spendable if these are checked.
So what we want is to wrap everything after the OP_VERIFY in a OP_IF block and depending on the flag either do the normal execution and hash checks or clear the stack and put OP_TRUE.
For automatic chunking that would be a fine solution but for the manual effort right now it seems like that would be messy. So we are wondering if we can somehow postpone or get rid of the remaining OP_VERIFY checks or carry the flag through until the end while avoiding to fail with the OP_VERIFY from the final stack hash comparison.

I checked the code and the final output hash checks don't use OP_VERIFY but leave one flag indicating inequality on the stack:
https://github.com/BitVM/BitVM/blob/cbfdd6076514836f25aae7b9ca653b3e141403a6/src/chunker/segment.rs#L136-L146
With the stack analyzer we could find out how many items are left on the after the hash check and interpret all remaining values as flags that have to be true. Then OP_BOOLAND them until there is only one item left on the stack.

The initial example is a hint verification aswell as pointed out by @Hakkush-07. We have to make sure that all other OP_VERIFY are also part of a hint verification or the winternitz signature check. If so the problem does not exist.


Forced casting generates errors

Decompiling a script with Champollion and recompiling it with Caprica inserts forced casting (explicit coercion) in a bunch of places (variable as bool, variable as float, etc...). Just noticed you mentioned this already, so editing this ticket. The explicit coercion appears to be generating errors.

Enable logging (so can see the stack traces)
in Fallout.ini, under [Papyrus]:
bEnableLogging=1
bEnableTrace=1
bLoadDebugInformation=1

Decompile workshopscript.pex in Champollion and recompile the resulting workshopscript.psc

Go to a Settlement, and move a Settler to another Settlement
or

Find an idle companion at a Settlement, ask him/her to join you, then dismiss him/her to a different Settlement

errors are generated in \Documents\My Games\Fallout 4\Logs\Script\

(note that the vanilla script generates a couple of warnings and errors on its own, but recompiling generates a bunch of new ones that appear to be casting related)

Which version of Caprica? There have been some recent improvements to the implicit casting that was being done.

v0.1.0. I've been grabbing the latest from Nexus.

This should be fixed in v0.1.1.


Bring up to date with vRP master.

Changes from https://github.com/ImagicTheCat/vRP/commit/78ca6f2acc0d047e3f95c46c8320436192ce3ccd

Thanks for help!

No problem. I like your way of thinking regarding the menu system.


quickly - there is an easy way to trigger it from windows; make a shortcut to obsidian://quick-capture and assign a hotkey to the shortcut

can you please open a PR for the README.md or simply add the instructions here and I will add to the README.md


[Suggestion]Support to paste a deep-level directory path to clipboard

Storage Explorer Version: master/20180419.2
OS Version: Windows 10/Mac/Linux
Steps to Reproduce:

Launch Storage Explorer and expand Storage Accounts.
Open one file share and go to a deep-level directory.

Suggestion:
Support to paste the deep-level directory path so that share the files/directories to others.

Duplicate of #45.


Generator not respect nillable

Describe the bug
if generate type of class properties - you not respect nillable (nullable) property in wdsl
Expected behavior
when you found nillable property in wdsl add ? to type of property
Sorry for my english

\vendor\wsdltophp\packagegenerator\src\Model\StructAttribute.php
line 170 ++
public function isNillable(): bool
 {
     return 'true' === $this->getMetaValue('nillable');
 }

\vendor\wsdltophp\packagegenerator\src\File\Struct.php
line 66:
$isNullable = $attribute->isNillable();
if (false === $isNullable) {
   $isNullable = (false === $attribute->isRequired() && false === $attribute->isArray());
}

line 162:  same as previous

Hi,
Can you paste a generated class having nillable properties not generated as you expect?
thx

Any chance you test the Docker image or the phar file from feature/issue-250?

Probably on the weekend - I'm migrating to a new computer

I found some time - it works, thank you very much !


Localized conversion fails: wrong expected type

I tried to use the -localization argument during a conversion, but it fails.
Steps:

Extract all pak files from the Paks folder (I only put DataStrm-part1.pak): pak-extracter.exe -assets Paks -output Extracted\DataStrm-part1
Extract all pak files from the Paks folder (I only put server.pak): pak-extracter.exe -assets Paks -output Extracted\server
Convert datables contained in server.pak while providing localizations from DataStrm-part1: datasheet-converter.exe -input "Extracted\server\server\sharedassets\springboardentitites\datatables" -output "Converted\server-localized" -format json -localization "Extracted\DataStrm-part1\localization\fr-fr" -keep-structure
Output:

2023/02/05 11:07:21 The number of threads is set to 3
2023/02/05 11:07:22 localization.New: expected element type <resources> but have <Workbook>

Am I doing something wrong?
Note: I extracted both pak files as a two steps process because I initially experimented with files one by one. I could have done it as a single step.

the french localization contains some kind of legacy file. fixed https://github.com/new-world-tools/new-world-tools/releases/tag/v0.5.7


error running example

Error occurred when executing DiffusersPipelineLoader:
File "E:\StableDiffusion\ComfyUI_windows_portable\ComfyUI\execution.py", line 154, in recursive_execute
output_data, output_ui = get_output_data(obj, input_data_all)
File "E:\StableDiffusion\ComfyUI_windows_portable\ComfyUI\execution.py", line 84, in get_output_data
return_values = map_node_over_list(obj, input_data_all, obj.FUNCTION, allow_interrupt=True)
File "E:\StableDiffusion\ComfyUI_windows_portable\ComfyUI\execution.py", line 77, in map_node_over_list
results.append(getattr(obj, func)(**slice_dict(input_data_all, i)))
File "E:\StableDiffusion\ComfyUI_windows_portable\ComfyUI\custom_nodes\ComfyUI-Diffusers\nodes.py", line 31, in create_pipeline
StableDiffusionPipeline.from_single_file(
File "E:\StableDiffusion\ComfyUI_windows_portable\python_embeded\lib\site-packages\diffusers\pipelines\pipeline_utils.py", line 708, in save_pretrained
save_method(os.path.join(save_directory, pipeline_component_name), **save_kwargs)
File "E:\StableDiffusion\ComfyUI_windows_portable\python_embeded\lib\site-packages\diffusers\models\modeling_utils.py", line 520, in save_pretrained
safetensors.torch.save_file(
File "E:\StableDiffusion\ComfyUI_windows_portable\python_embeded\lib\site-packages\safetensors\torch.py", line 281, in save_file
serialize_file(_flatten(tensors), filename, metadata=metadata)
File "E:\StableDiffusion\ComfyUI_windows_portable\python_embeded\lib\site-packages\safetensors\torch.py", line 475, in _flatten
return {
File "E:\StableDiffusion\ComfyUI_windows_portable\python_embeded\lib\site-packages\safetensors\torch.py", line 479, in
"data": _tobytes(v, k),
File "E:\StableDiffusion\ComfyUI_windows_portable\python_embeded\lib\site-packages\safetensors\torch.py", line 439, in _tobytes
return data.tobytes()

Thank you for the report. Maybe it's an error when loading the model?
I think the possibility is that the browser was not updated when the model was deleted from the directory.

Yes, the error occurs in the Diffusers Pipeline Loader. I have uninstalled and reinstalled this node, still got this error.

where are we supposed to save / how do we set the path for the diffusers model folders?


Proposal: New DockManager / Dockable Windows Control

Summary
It would be nice to have a new DockManager control similar to how Visual Studio and AvalonDock works.
Rationale
This type of control can be used for many LOB-type apps and has been missing as a 1st-class control for some time.  With my limited knowledge of where WinUI is headed, it does seem like a control that would be helpful.

Would this control need a DockablePanel control which contains XAML that fills the space, and then that panel handles the dragging and triggers the hint icons on the DockManager panel which contains the app's UI?

@mdtauk Great point.  I think there will need to be a few supporting controls, as well.  Similar to TabItem under the TabControl.

Bringing @harvinders proposal's description over here:
Proposal: Dockable Windows

Summary
Add support for dockable windows that can be docked to a window like what we have in Visual Studio
Rationale

This is one of a popular Desktop apps modes, which UWP is currently lacking
Allows us to make apps who's layout is personalised by the user

Scope

Capability
Priority

Dock window
Must

Dock window container control, inside which the dock window docks
Must

Dock window container (inside) dock options (top,bottom,center,left,right)
Must

Undock options (float)
Must

Dock options to any window (outside) (top,bottom,left,right)
Should

Related feature request - #1080
We definitely need additional innovation in this space but won't be able to begin implementation until after WinUI 3.

@YuliKl

We definitely need additional innovation in this space but won't be able to begin implementation until after WinUI 3.

That is completely understandable.  I'm very glad the team is considering this control.  Look forward to WinUI 3.0 and future goodness 😄

Both WinForms (dockpanelsuite) and WPF (AvalonDock) have widely used third-party controls for docking. Another good example would be UWP Docking Control by Syncfusion. Having a control like this would make easier to migrate apps to WinUI.

Is there any progress on this feature? It seems like there hasn't been update in over a year and I would say that this seems like a pretty critical feature is Microsoft are truly aiming to make WinUI 3 the new default UI for Windows apps. Right now I am in the early stages of writing some graphical profiling software and evaluating what software is available out there, and not having an easy method of moving panels/dockable windows between areas to match the users needs seems like a huge oversight. I asked about this feature in #4784 and was told to put forward a feature request but it seems like the request is already here, it's just not made any progress.
They way I see it is that if I wanted to have what most would consider a pretty standard feature in software tools I would be best off using some combination of SDL2/Dear Imgui or a third party library for WPF/WinForms. This means that I wouldn't be able to take advantage of other WinUI 3 features which is a shame as for the most part WinUI 3 seems like a solid UI framework.
Is this feature still in consideration, or would it be something that is likely to be left to third parties again as with dockpanelsuite and AvalonDock?

Well that's a bummer. WinUI 3 certainly looks like the WPF savior we all have been waiting for. I'm honestly very excited, but... creating a modern Desktop app without docking panes?! You have got to be kidding me!
This means we again have to go through the hassle of:

Bloating our app with 10 MB of sloppy third-party code (they never get the docking stuff right).
Paying some license for a control which should have been there in the first place.
Maintaining some abandoned DockManager toolkit.
Writing our own docking system. Sure, but that means less time for cool features on our Windows app, and in the end less profit for Microsoft.

Name one other UI platform where you immediately need to look for commercial alternatives! iOS? Android? 😆
"Now introducing WinUI 4 for Android. Scroll viewer? Nah, it's on our todo-list."

Just to keep the ball rolling on the fact that this is an absolutely essential feature.
I've been keeping an eye on the whole WinUI3 / Windows App SDK for a while and in particular docking windows / mdi. This is an absolutely essential feature for desktop productivity apps and any tool I have been considering writing of late. Quite literally every program I use for development on a daily basis uses this interface paradigm and it is absolutely essential.
Have I mentioned it's absolutely essential? I think I did.
For me, quite honestly, until there is SOME kind of docking / mdi type support (even 3rd party) it really does just feel like a toy for creating mobile-phone-wannabe-apps.
MFC, some 30 years on, is still more attractive for those wanting to develop C++ based productivity apps on Windows. I'm not interested in moving to C# just for the UI only to jump through all the hoops required to then leverage the C++ codebase I require under the hood.
 Make it happen. :)

Commenting to keep this alive - this is absolutely essential for us to consider moving off WPF.

Any update on this?
I have a new project for which this is required.
Played around with UWP control from SyncFusion. Time to jump on WinUI3 but this control is essential for the project.

+1

I definitely want to update this feature request.  Docking panels is an integral part of my MFC desktop application that I want to convert to WinUI3.

I definitely want to upvote this feature request. Docking panels is an integral part of my MFC desktop application that I want to convert to WinUI3.

Same things. We also want to convert WPF to WinUI3.

Please implement this feature. WinUI3 looks very promising, as UWP looked back in the day. But both are lacking the docking panes / MDF feature which is a bummer and is the sole reason to not move over from WPF.
A true, modern desktop application not only has to look good but also needs a lot of flexibility for the user to justify the need of a desktop application for him in the first place. There are a lot of business applications out there that need this kind of flexibility.

@fl0aty for enterprise apps I cannot upvote your comment enough. It's basically a showstopper not having this for us.

ImGui is a completely different UI paradigm, though, it's not always appropriate. The lack of support for this is totally bizarre to me. The only real option currently is having a C# front end with C++ back end, or using QT.

Another random revisit of this topic as I consider using what UI API might be useful again in C++ land. And nothing appears to have changed on this absolutely essential feature. 🙄
It might be time to bite the bullet and use Qt… or just dig out the MFC libraries again.

came in 2023, seeing posts over years and still no progress really scares me to full dive to WinUi. Hope this feature will be out one day...
Having Official solution is always my preference over opensource or 3rd party solution. For now perhaps going back to win32 is the answer.

I am puzzled by the lack of evolution of the Windows UI development stack.  WPF and WinUI seemed promising but more difficult to use than WinForms and MFC and did not have some of the critical features needed for rich business style applications.  I kind of lost interest in Windows development for that reason.  I'd love to see some sort of roadmap for Windows UI enrichment.

Ditto carmangary!  What the heck?  Maybe I will move to QT.  Can QT host a Win32 window?

I honestly think the WinUI 3 tab control should be generalized to this. AvalonDock supports simply having them all fill their container as layered documents and then it basically becomes a tab control with tabs to change views.
The obvious benefit of this is that all WinUI apps would automatically gain docking support, although this could of course be opt-in on the control e.g. IsDockable.

Its gone already 5 years since the proposal... dont feel like any cares....

Seems as though Microsoft stopped caring about desktop developers, at least.  Azure is their cash cow now.

Now, three years later, I have to say that I no longer care about WinUI dock managers 😝.
We spent a few weeks trying to migrate a tiny version of our WPF application to WinUI. We're not convinced, to say the least.
Maybe in another three years...

Starting with .NET 9, WPF will have built-in support for good Windows 11 styling and controls, so a more sensible way forward here for complex LOB applications etc. might be WPF with AvalonDock  or a similar mature docking system. You can get decent Windows 11 ribbons via FluentRibbon 11.0 onwards, or Coho.UI.

The whole point for me is that WinUI naively supports C++. If I wanted to jump through hoops and use C#, WPF, etc, then that's always been there. If you have a codebase you want to leverage in C++, it's far nicer to have the option to keep everything in C++. MFC or QT are really the only sensible solutions to do this. It really is insane, imo, that this has been left to stagnate as it has. I guess using C++ is not something the masses do these days?

I agree with the above.  I have and still support a multimedia application written in C++ using MFC that has docking tabs.  I spent a lot of time porting it from C# due to C# being a pain to deal DirectX and also because it was so easily hackable due to ease of decompilation, even when obfuscated.  There was a noteable increase in application performance when moving to C++.
C# is cute and capable for a lot of stuff but it is not a good option for the type of development I want to do.  I'm not asking for much.  Just have WinUI match the level of capabilities that MFC had decades ago and improve upon it.

This hits the point home for me. Can we just have the same level of features we had 20 years ago, please?

VS2022 version 17.10.4 - Tried to create Blank App, C++ WinUI 3 in Desktop.  VS Created 911 files in 356 folders.  Tried to build and had 8 errors.  Good grief.

从 .NET 9 开始，WPF 将内置对良好 Windows 11 样式和控件的支持，因此对于复杂的 LOB 应用程序等，更明智的前进方式可能是使用 WPF 与 AvalonDock  或类似的成熟停靠系统。您可以通过 FluentRibbon 11 及更高版本（在我撰写本文时处于 alpha 阶段）或 Coho.UI 获得不错的 Windows 11.0 功能区。如果您依赖早于 .NET 9 的 .NET 版本，您可以将 WPFUI 用于 Windows 11 样式，它支持低至 .NET Framework 4.6.2，实际上是 .NET 9 采用和集成的库。

I don't think so. Because Wpf has a lot of history problems.Such as using too many reflections, therefore is not able to support AOT. Meanwhile wpf is still a DiectX 9 thing, when Winui3 is based on DirectX 12, and Avalonia 11+ or C++ QT is based on Vulkan.
Just open a solution in vs2022's lastest version, which is a wpf app, and close this solution. Congratulations, you find a 100% memory leak.


Persist multi-translations

Hi,
i have a problem (with Symfony2 and Doctrine2).
We fill a form (for a language), and with checkboxs we choose to copy this datas into others languages.
Like this:
$establishment->setTranslatableLocale($language);
$establishment->setHash(uniqid('', true));
$em->persist($establishment);
$em->flush();

// Copy datas to other languages
$languagesToCopy = $form->get('copylanguage')->getData();
foreach ($languagesToCopy as $languageToCopy) {
    $establishment->setTranslatableLocale($languageToCopy);
    $establishment->setHash(uniqid('', true));
    $em->persist($establishment);
    $em->flush();
}

I try to use a HASH to force the persist / flush. But I can't persist the 3 languages (per example), only 2/3 or 1/3 ...
I don't know why
thanks for help

Closing due to age and lack of activity. If you are interested in this issue or are still experiencing any problems described here, please add a comment!


Support EC2 subnet id in config. (Required for new AWS acccounts.)

Impossible to run EMR jobs without specifying this for newer accounts.

Fixes #25


add EliminateSubqueryAliases

Which issue does this PR close?

Closes #5407.
Rationale for this change

What changes are included in this PR?

Are these changes tested?

Are there any user-facing changes?

This PR is blocked by alias schema.
Due to we hope to keep same name, we will add some alias like col1 as t1.a, but its field is t1.a, original field is t1.a.
I'm resolving this problem.


[BD-46] docs: add ADR for i18n support

Add ADR for i18n support.
JIRA: PAR_412

Thanks for the pull request, @viktorrusakov! I've created BLENDED-1050 to keep track of it in Jira.   More details are on the BD-46 project page.
When this pull request is ready, tag your edX technical lead.

So many new checks 😮

This seems sensible - there are a few things I'd want to sanity check:

That the same react-intl version and code in node_modules is being bundled into the host MFE and that we're not increasing our bundle size. (I think with tree shaking and having Paragon built preserving import statements, this will be fine)
That the same instance of react-intl, through frontend-platform, is being used at runtime.  To my knowledge, weird inconsistent stuff can happen if there are more than one instance of react-intl initialized into the page.

Assuming the impact here is that nearly all MFEs will just use AppProvider and pass the paragon messages into the app via initialize, that seems like a fine API/footprint for this change.

Good questions, thanks for the feedback!

That the same react-intl version and code in node_modules is being bundled into the host MFE and that we're not increasing our bundle size. (I think with tree shaking and having Paragon built preserving import statements, this will be fine)

I believe keeping react-intl version in sync with frontend-platform's one makes it work correctly, I've tested the build with frontend-app-payment and having react-intl as a dependancy in Paragon does not increase the bundle size (tested the build with prod config).

That the same instance of react-intl, through frontend-platform, is being used at runtime.  To my knowledge, weird inconsistent stuff can happen if there are more than one instance of react-intl initialized into the page.

Hmm, I'm not sure that can happen since we will have only one version of react-intl in the bundle, so it should be one instance if I understand correctly. I've added translations for some components and they worked correctly inside frontend-app-learning when testing it locally, I haven't noticed any issues. Would you like me to check something specific?

@viktorrusakov 🎉 Your pull request was merged!  Please take a moment  to answer a two question survey so we can improve your experience in the future.

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


fix: Debugged PACKAGE.html problem #697

Title and Issue number
Title : fixed the page linking problem in PACKAGE.html and addressed the CSS issues in titles and subtitles.
Issue No.: #697
Code Stack : HTML, CSS, JS
Close #697
Description
I have rectified the page linking problem in package.html. Additionally, I addressed slight CSS issues that were causing titles and subtitles to appear off-center and not in their original form. Specifically, I centered the titles and subtitles in both package.html and index.html, making necessary adjustments in styles.css.
Video/Screenshots (mandatory)
package.html now,

By clicking 'Book Now' under Maldives and many other places, it showed the error 404, but I have rectified that now

Center aligned the topics and subtopics in index.html

Type of PR

[x] Bug fix
[ ] Feature enhancement
[ ] Documentation update
[ ] Other (specify): ___________

Checklist:

[x]  I have performed a self-review of my code.
[x]  I have commented my code, particularly in hard-to-understand areas.
[x]  I have tested the changes thoroughly before submitting this pull request.
[x]  I have provided relevant issue numbers, screenshots, and videos after making the changes.
[x]  I have gone through the  contributing.md file before contributing

Additional context:
Are you contributing under any Open-source programme?
yes

Hi @apu52 , this PR is ready for review. Kindly, tell me if any further changes are required.
This is my first contribution to GSSoC'24. Kindly, merge it or suggest me more changes 🙏

Sure @apu52 I will do it


Build warning with Rust 1.81.0

What should we add?
When building Qiskit with the latest Rust release, 1.81.0, a future incompatibility warning is raised at the end of the build:
warning: the following packages contain code that will be rejected by a future version of Rust: ra_ap_stdx v0.0.188

Looking at the cargo report this is caused by code in the crate that might lead to undefined behavior in a future release of Rust.
This isn't a direct issue in Qiskit, as the ra_ap_stdx dependency gets pulled in by: https://github.com/Qiskit/openqasm3_parser but I opened so we can track it on the Qiskit side as people building qiskit will encounter this. It also will let us track upgrading the openqasm3_parser version once this is addressed over there.
The upstream tracking issue is: https://github.com/Qiskit/openqasm3_parser/issues/229

The most recent version of ra_ap_stdx that is compatible with rust v1.70 is 0.0.190. But version 0.0.190 still causes the warning to be printed when compiling with rust v1.81.0
We need to find a workaround.


Dev version: Use last release + 0.0.1 + '-dev'

This version will resolve to < 0.0.7, which allows us to use meaningful deprecation versions, as discussed in #157.
This approach maintains the ability for Circle to replace the version on release, as well as allows meaningful version comparisons for git repositories. The additional work of updating the dev version number occurs after the release, so there's no risk of this resulting in a need for a quick release cycle because we screwed up the version number.
@oesteban mentioned a new system he's using in MRIQC. I'm okay with switching to that if that seems like a better option.
Fixes #163

It's been a while since a release, anyway. If this goes in, I'd say let's do a 0.0.7 release, to make sure I'm right about how this will work.
If it does work, I'd say let's move to either this system or MRIQC's system for FMRIPREP, just so we don't have three different systems going.

I think for FMRIPREP it may make sense a more elaborate versioning. But versioneer could be an overkill.
Let's replicate this simple solution into FMRIPREP and if versioneer does not raise any problems for mriqc then we can switch FMRIPREP knowing it will work well.
+1000 to a new release of niworkflows.

Looks like deprecation tests against the nipype version, not the niworkflows version. I'm going to submit a PR that makes the version they check against interface-specific (nipype by default).

Relevant line: https://github.com/nipy/nipype/blob/f39e222499cca6647d732e9164a75339f1f3a6a7/nipype/interfaces/base.py#L447

Good catch!

This should be good to go, pending a final test.


entity.file not return when first time

Describe the bug
await entity.file not return when first time but originFile had content.
Same image second time file had content.
Another image is same situation.
Flutter version
Flutter (Channel stable, 1.22.4, on macOS 11.0.1 20B29 darwin-x64, locale zh-Hans-CN)
Smartphone (please complete the following information):

Device: iPhoneXR
OS: iOS14

v0.5.8

If this method does not throw an exception, it should always contain content.
You need to provide a more complete reproduction to illustrate the steps you took to encounter this problem.

Seem those image on icloud #359

@CaiJingLong @AlexV525 I'm having the same issue for video files only.
I've tried
File file  = await entity.file;

and
  PMProgressHandler handler = PMProgressHandler();
  handler.stream.listen((event) {
    print(
        'loading $asset => ${event.progress}, ${event.state} -- ${DateTime.now().millisecondsSinceEpoch}');
  });
  File file = asset.loadFile(isOrigin: false, progressHandler: handler);

Both result in a File object being returned instantly, but with fileSize = 0. After further testing, I believe it's downloading the video file from iCloud, as if you wait long enough and then reload the file, it works. This makes sense since fetchFullSizeImageFile method is configuring PHImageRequestOptions to be synchronous. Video doesn't have that same option, however, the progress handler doesn't seem to be working properly.
I'm not entirely sure what the exact problem is, but I'd be happy to help collaborate on getting it fixed. Let me know how I can help.

We're currently scheduled to refactored the iCloud part, and will cost maybe 1 or 2 months for it. Let us know if you have any ideas in how to implement the whole iCloud feature in a more proper way.

@AlexV525  #558 has made pulling the file successful now. I haven't been able to test it heavily with iCloud (#377), but so far looks like that is fixed as well.
The progress handler is still not 100% working however. It reports back prepare and success states, but never loading, even when a file takes many seconds to load. Added an issue #565
In terms of suggestions for iCloud feature or iOS in general, here are a few.

Move the project over to Swift (or at least the more complicated areas). I believe this will allow more developers (such as myself) contribute to bug fixes and feature improvements.

While iOS and Android share some similarities in terms of their APIs, they depart pretty quickly when it comes to some newer features (Live Photos, Slow-Mo videos,  etc). To provide a user experience that iOS users expect, more API features need to be supported in this library.

For example, it'd be difficult to add LivePhoto support into this library since AssetEntity is limited to fetching one file, were as a LivePhoto is actually a Photo and Video file. The library API interface needs to support these differences, but not be overly complicated.
Two possible solutions:

Inheritance

  AssetEntityiOS asset = AssetEntityiOS.fromId(id);

AssetEntity asset = AssetEntity.fromId(id);
asset.iOS.isLivePhoto
asset.iOS.livePhotoKey

3. Expose  `PHVideoRequestOptions`, `AVAssetExportSession` settings similar to how image thumbnail creation has `ThumbOption`. There are times when quality matters and other times speed is preferred.

@AlexV525 #558 has made pulling the file successful now.

Sounds good, closing this issue.

Move the project over to Swift (or at least the more complicated areas)

No matter we refactored in which language, it'll be a huge work. I personally have no experience in Objective-C, but I would recommend to refactor after these critical issues been resolved.
About Live Photos and other more specific issues, we'll track them separately.

1.2.8, iCloud photos still hangs


ISPN-13278 SoftIndexFileStore does not support purge or expiration events

Also includes changes to have stores provide metadata and values to listeners.
https://issues.redhat.com/browse/ISPN-13278
https://issues.redhat.com/browse/ISPN-13295
https://issues.redhat.com/browse/ISPN-13334

Some thread leak checks have failed

It looks like there are a number of related failures

Updated and addressed the review comments.

One related failure https://ci.infinispan.org/job/Infinispan/job/PR-9564/7/testReport/junit/org.infinispan.expiration.impl/ExpirationFileStoreDistListenerFunctionalTest/testSimpleExpirationMaxIdle__SOFT_INDEX_/

Nearly there... :)
org.infinispan.expiration.impl.ExpirationFileStoreListenerFunctionalTest.testSimpleExpirationLifespan[ SOFT_INDEX].Timeout
org.infinispan.expiration.impl.ExpirationFileStoreListenerFunctionalTest[ SOFT_INDEX].Undelivered

Nearly there... :)
org.infinispan.expiration.impl.ExpirationFileStoreListenerFunctionalTest.testSimpleExpirationLifespan[ SOFT_INDEX].Timeout
org.infinispan.expiration.impl.ExpirationFileStoreListenerFunctionalTest[ SOFT_INDEX].Undelivered

Oops, typo. Should be fixed now.

Thanks @wburns


Is SniHandler vulnerable to partial Client Hello message attack?

Suppose an attacker send a partial Client Hello message and leave the connection open, would SniHandler wait indefinitely for the remaining part of the message?
I was looking for a timeout that would close the connection if the remaining part of the message does not arrive within a timeout limit, but I did not find one in the class hierarchy.
Just to ensure that I did not miss anything.
Netty version
4.1

I don't think any of the codec-level handlers enforce timeouts. You are expected to use an IdleStateHandler to define and enforce your own timeouts, if needed.

I haven't looked at the details, but maybe SslHandler's setHandshakeTimeout might come in handy: https://netty.io/4.1/api/io/netty/handler/ssl/SslHandler.html#setHandshakeTimeout-long-java.util.concurrent.TimeUnit-

I am not sure if IdleStateHandler is granular enough. I can use IdleStateHandler to close the connection if no traffic for sometime after the handshake. "setHandshakeTimeout" is closer to what I need as it enforce that the handshake completes within a limit. It seems that we need another level of timeout to enforce that all packets of a message received within a limit.

just put the IdleStateHandler in front of the SniHandler that will do the trick.


Naming of pre-processed DWI data overlapping raw DWI data

After merging #5, we'll need to address this https://github.com/bids-standard/bids-bep016/pull/5#issuecomment-528219145

I don't think this is an issue anymore.


Fix to concat_files function

When generating .agg files with ASNs that have v4 and v6 prefixes, I was seeing that the last line in the .agg file did not contain a \n, resulting in the v6 address being appended to the line, resulting in .agg files that looked like this:
[irrpt]$ more db/11178.agg
<IP_ADDRESS>/23
<IP_ADDRESS>/23
<IP_ADDRESS>/24
<IP_ADDRESS>/24
<IP_ADDRESS>/24
<IP_ADDRESS>/23
<IP_ADDRESS>/24
<IP_ADDRESS>/23
<IP_ADDRESS>/24
<IP_ADDRESS>/242001:1900:3015::/48

This causes config file generation to be broken. Juniper example:
policy-options {
    replace: policy-statement CUSTOMERv6:11178 {
        term prefixes {
            from {
                route-filter <IP_ADDRESS>/242001:1900:3015::/48 prefix-length-range /242001-/48;
            }
            then next policy;
        }
        then reject;
    }
}

Adding a newline character to the concat_files function fixes this problem.

Thanks so much or catching this!