wasmparser: turn table index in the elements section into an Option

The wasmparser crate currently doesn't disambiguate between the two possible encodings of the table index in the elements section while wasm-encoder does. I'd like to be able to generate bit-exact .wasm blobs by parsing with wasmparser and piping everything to wasm-encoder, and this makes it impossible. So this PR makes wasmparser do what wasm-encoder does and store the index inside of an Option.

In isolation I think this is ok, but I don't think that wasmparser nor wasm-encoder is suitable for your goal of bit-exact wasm blobs. One major aspect neither handle is overlong encodings of leb128 values. For example wasm-ld will produce 5-byte encodings of all targets for call instructions to have the linker efficiently fill them in. The wasmparser crate has no means of conveying this information and similarly the wasm-encoder crate has no means of creating an overlong encoding.
Additionally this trick is not isolated to just table indices in element sections, but it's been pervasively used throughout wasmparser's development. For example memory indices is another case where lack-of-integer is silently translated to "you specified 0". Data segments also do something similar, and this is just what I remember off the top of my head so I think there's more.

In isolation I think this is ok, but I don't think that wasmparser nor wasm-encoder is suitable for your goal of bit-exact wasm blobs.

While this is a little unfortunate I don't really need this in all cases in general, just for a particular set of .wasm blobs I'm dealing with.
We're currently using parity-wasm for manipulating .wasm blobs and we'd like to migrate to wasmparser + wasm-encoder, and obviously having the new codepath be able to give out exactly the same .wasm as the old one makes this process easier.
This is somewhat offtopic, but may I ask - are there any plans to unify the types used by wasmparser and wasm-encoder? In isolation I think those two crates work pretty well, but when you want to mutate an existing .wasm blob translating between the wasmparser types and wasm-encoder types is a little tedious (especially translating wasmparser::Operation to wasm_encoder::Instruction which is a ~600 lines match), and seems a somewhat unnecessary when they're mostly the same. Couldn't we have e.g. a wasm-types crate and unify those types between wasmparser and wasm-encoder where it makes sense?

Ok if this is useful to you I think it's reasonable to include as it's relatively minor anyway, as long as you're ok with the other shortcomings that wasmparser has for that sort of use case.

are there any plans to unify the types used by wasmparser and wasm-encoder?

Currently not at this time. Historically this was also done by the walrus crate for the rustwasm working group, but that crate has sort of fallen out of style (and it's behind the times and lacking maintenance, so I wouldn't recommend it nowadays). Not to say that you're the first person who wants something like this, though, as I've also wanted this. Even in this repository the wasm-mutate crate translates between wasms and parts of wit-component do the same. I think it would be great for something like this to exist, but at this time I don't think anyone knows how best to construct this. Adding something like a wasm-types crate I think may be a bit too simple of a solution since there's a lot of subtelties about representation between wasmparser and wasm-encoder that are different and have repercussions on performance in various situations.
If you're interested, though, some macro soup can ease the translation today.

Ok if this is useful to you I think it's reasonable to include as it's relatively minor anyway, as long as you're ok with the other shortcomings that wasmparser has for that sort of use case.

Yes, I still think this should be useful!


(WIP) Add unit tests for ProfileDataCollector and ProfileDataExporter classes

You can convert this pull request to a draft since it's not ready for a final review/approval:
https://docs.github.com/en/pull-requests/collaborating-with-pull-requests/proposing-changes-to-your-work-with-pull-requests/changing-the-stage-of-a-pull-request#converting-a-pull-request-to-a-draft


Revert jackson update

Revert the update to the Jackson library back to 2.12.7.

Codecov Report

Merging #11126 (6a7516e) into master (7422edf) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master   #11126   +/-   ##
=======================================
  Coverage    0.11%    0.11%
=======================================
  Files        2203     2203
  Lines      118196   118196
  Branches    17887    17887
=======================================
  Hits          137      137
  Misses     118039   118039
  Partials       20       20

Flag
Coverage Δ

integration1temurin11
0.00% <ø> (ø)

integration1temurin17
0.00% <ø> (ø)

integration1temurin20
0.00% <ø> (ø)

integration2temurin11
?

integration2temurin17
0.00% <ø> (ø)

integration2temurin20
?

unittests1temurin11
?

unittests1temurin17
0.00% <ø> (ø)

unittests1temurin20
0.00% <ø> (ø)

unittests2temurin11
0.11% <ø> (?)

unittests2temurin17
0.11% <ø> (ø)

unittests2temurin20
0.11% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

Can we add some comments on why this is reverted?


Testing Performance

Just wanted to know if this is comparable to your model. and if not where could i possibly improve?

I will close this Issue. Just saw that it is actually comparable performance. Also for others to see that replicating is possible.
Keep up the good work !


let gpioa = p.GPIOA.split();

Some  HALs  want mut in    let mut gpioa = p.GPIOA.split();   and some do not.

The HALs do not provide a consistent API at this level. This has been resolved this by wrapping setup for different HALs and hardware in setup() functions that are board specific.


AZ400 - lab 18 - az webapp create

This is autogenerated. Please review and update as needed.
Describe the bug
Command Name
az webapp create Extension Name: appservice-kube. Version: 0.2.2.
Errors:
The command failed with an unexpected error. Here is the traceback:
__init__() got multiple values for argument 'linux'
Traceback (most recent call last):
  File "/opt/az/lib/python3.8/site-packages/knack/cli.py", line 231, in invoke
    cmd_result = self.invocation.execute(args)
  File "/opt/az/lib/python3.8/site-packages/azure/cli/core/commands/__init__.py", line 658, in execute
    raise ex
  File "/opt/az/lib/python3.8/site-packages/azure/cli/core/commands/__init__.py", line 721, in _run_jobs_serially
    results.append(self._run_job(expanded_arg, cmd_copy))
  File "/opt/az/lib/python3.8/site-packages/azure/cli/core/commands/__init__.py", line 713, in _run_job
    return cmd_copy.exception_handler(ex)
  File "/home/pierre/.azure/cliextensions/appservice-kube/azext_appservice_kube/commands.py", line 42, in _polish_bad_errors
    raise ex
  File "/opt/az/lib/python3.8/site-packages/azure/cli/core/commands/__init__.py", line 692, in _run_job
    result = cmd_copy(params)
  File "/opt/az/lib/python3.8/site-packages/azure/cli/core/commands/__init__.py", line 328, in __call__
    return self.handler(*args, **kwargs)
  File "/opt/az/lib/python3.8/site-packages/azure/cli/core/commands/command_operation.py", line 121, in handler
    return op(**command_args)
  File "/home/pierre/.azure/cliextensions/appservice-kube/azext_appservice_kube/custom.py", line 505, in create_webapp
    helper = _StackRuntimeHelper(cmd, client, linux=(is_linux or is_kube))
TypeError: __init__() got multiple values for argument 'linux'

To Reproduce:
Steps to reproduce the behavior. Note that argument values have been redacted, as they may contain sensitive information.

Put any pre-requisite steps here...
az webapp create -n {} -g {} -p {} -i {}

Expected Behavior
Environment Summary
Linux-5.4.0-1073-azure-x86_64-with-glibc2.28 (Cloud Shell), Common Base Linux Delridge (quinault)
Python 3.8.12
Installer: DEB

azure-cli 2.34.1

Extensions:
appservice-kube 0.2.2
ai-examples 0.2.5
ssh 1.0.0

Dependencies:
msal 1.16.0
azure-mgmt-resource 20.0.0

Additional Context

route to CXP team

@piegre It looks like you're running an old version of the appservice-kube extension from private preview (since your version number is 0.2.2). You should uninstall the extension with az extension remove -n appservice-kube and install the latest version (public preview) with az extension add -n appservice-kube. After this you can test the command again. Awaiting your reply.
Similar issue https://github.com/Azure/azure-cli-extensions/issues/4490

I am just using the azure CLI (shell.azure.com)
Le ven. 1 avr. 2022 à 12:30, navba-MSFT @.***> a écrit :

@piegre https://github.com/piegre It looks like you're running an old
version of the appservice-kube extension from private preview (since your
version number is 0.2.2). You should uninstall the extension with az
extension remove -n appservice-kube and install the latest version (public
preview) with az extension add -n appservice-kube. After this you can test
the command again. Awaiting your reply.
Similar issue #4490
https://github.com/Azure/azure-cli-extensions/issues/4490
—
Reply to this email directly, view it on GitHub
https://github.com/Azure/azure-cli-extensions/issues/4609#issuecomment-1085729961,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AYOX4UBUGPVTHSR4OFPHLBTVC3F2VANCNFSM5SH4LIJA
.
You are receiving this because you were mentioned.Message ID:
@.***>

@piegre shell.azure.com only upgrades the core-cli version & not the individual extensions, you seem to have this installed appservice-kube 0.2.2 - which overrides some behavior on webapp core-cli, so either delete this or upgrade this extension.
See following: https://docs.microsoft.com/en-us/cli/azure/extension?view=azure-cli-latest#az-extension-update

@piegre Could you please let me know if you had a chance to upgrade your appservice-kube extension. You can either uninstall and reinstall the extension as explained in my above comment. Or you can upgrade it using the command az extension update --name appservice-kube. Please let me know if you still face the same issue after following the above plan. Awaiting your reply.

Yes it worked, thanks.
It's a bummer to lose so much time during an intense training like az400,
just because the azure CLI doesn't include the latest packages.
Le lun. 4 avr. 2022 à 06:16, navba-MSFT @.***> a écrit :

@piegre https://github.com/piegre Could you please let me know if you
had a chance to upgrade your appservice-kube extension. You can either
uninstall and reinstall the extension as explained in my above comment. Or
you can upgrade it using the command az extension update --name
appservice-kube. Please let me know if you still face the same issue
after following the above plan. Awaiting your reply.
—
Reply to this email directly, view it on GitHub
https://github.com/Azure/azure-cli-extensions/issues/4609#issuecomment-1087093824,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AYOX4UGKQAQ7VWBF3H4CXNTVDJUK5ANCNFSM5SH4LIJA
.
You are receiving this because you were mentioned.Message ID:
@.***>

@piegre Thanks for getting back. I am glad that the issue is resolved. As mentioned earlier in the above comment, the shell.azure.com only upgrades the core-cli version & not the individual extensions. Having CLI extensions, will override the behavior of webapp core-cli. We will now proceed with closure of this issue. Please free to reopen this thread if you have any further questions on this.


Documentation ambiguous

The section that says 'Select the role you wish to assign to the application.' needs screenshots
For instance, when I try to add role assignment to example-app created above, I just can't find example-app in the list when I click on 'Select Members' on the role assignment page.
Either instructions are incorrect or unclear

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

ID: 11aaa091-1108-52f6-0066-0f4e939aeef4
Version Independent ID: dbc7e37e-87a7-c124-3e4c-47aa060e6cf3
Content: Create an Azure AD app and service principal in the portal - Microsoft Entra
Content Source: articles/active-directory/develop/howto-create-service-principal-portal.md
Service: active-directory
Sub-service: develop
GitHub Login: @rwike77
Microsoft Alias: ryanwi

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

@elbee19  Thanks for bringing this up.  Looks like the portal is a little different now.  In the section that says 'Select the role you wish to assign to the application.', Select Add > Add role assignment to open the Add role assignment page.
In the Role tab, select the role you want to added to the app in the list. For example "Contributor".  Select the Next button to move to the Members tab.  Select Assign access to-> User, group, or service principal and then select +Select members.
In the Select members window, search by the name of the app you created (for example "example-app").  Select the app from the list, then click the Select button.  Then click the Review + assign button.
Hope that helps, I'll update the article.

Doc updates are live, hope that helps.  Closing the issue now.  #please-close


fix bug    Memory leak caused by VlcMediaPlayer.Dispose()

VlcMedia.LoadedMedias will keep adding "VlcMediaPlayer instance" when call VlcMediaPlayer.Dispose()

Hi,
thanks for your contribution. After reviewing your code, I wonder what it's supposed to do and how it would work if you have two VlcMediaPlayer instances at a given time? it would clear the dictionary and leak other instance's media.
However, I agree that creating a new instance of a list for something that you want to remove is far from ideal (and probably leaks VlcMediaPlayer)
What do you think about VlcMedia.LoadedMedias.Remove(this);?

This PR and #348 are replaced by #358, a slightly different implementation. Thanks for your contribution


updating bower.json and package.json so tsd link picks up typscript t…

The tsd type definitional manager for typescript supports a link functionality to automatically pull in d.ts files from included npm and bower dependencies if they have a "typescript" entry. https://github.com/Definitelytyped/tsd#link-to-bundled-definitions. Branch "typescript-package-link" adds these entries to both json files.

Merged in dd4d8a56e1aa27c049683a16839f3814faa631c8


Add more committers

To make RailsAdmin development sustainable!

@codealchemy Thank you for your active contribution to RailsAdmin.
Are you interested in having a committer access to the project, so you can push your code freely?

👍🏼  that'd be great - thanks!

@jdufresne Thank you for your persistent interest on maintaining the project.
Are you interested in having a committer access, so your work like this can be done smoothly?

Hi @mshibuya I'd be happy to help out in that regard. Thanks!

@mshibuya I've been using this project for a long time now and I would love to support maintaining it and possibly even have folks from Codelitt as well. Is there anything I could start on? I'm not asking for committer access, just orientation about how I could help.

@kaiomagalhaes Great to hear that!
Currently we don't have something like a list of 'good first issue', but you can still take a look at open issues and work on the one you're interested. You don't need to totally solve the problem, just finding a clue or reproduction steps will be greatly helpful.
Another way of contribution is documentation. We use GitHub Wiki for that, but not all features are included there, especially for newly-implemented ones. You can add instructions on those features, or tips you found through your experience of using RailsAdmin!


updated: dates and statuses for milestones

Based on our talk I have updated the dates and statuses of Milestones in .md files.
Please check and pull. Cheers.

@devendran-m milestone dates updated acc. to @Anna-defdone ; please review.

Hi @yetson0 ,  I have updated it in the subsequent PRs. Therefore closing this.


dbuser parameter not set correctly

The first command works correctly as expected:
# pgbadger --format redshift /tmp/todel/26/useractivitylog_2020-03-26T08:53.gz --dbname vdb --outfile /tmp/myqn199.txt
[========================>] Parsed 612540 bytes of 612540 (100.00%), queries: 702, events: 0
LOG: Ok, generating txt report...

But when I add the dbuser parameter it does not return any query.
# pgbadger --format redshift /tmp/todel/26/useractivitylog_2020-03-26T08:53.gz --dbname vdb --dbuser root --outfile /tmp/myqn199.txt
LOG: Ok, generating txt report...d 612540 bytes of 612540 (100.00%), queries: 0, events: 0

I have checked that the file has queries executed by "root" user. For e.g.

'2020-03-26T07:55:08Z UTC [ db=vdb user=root pid=28961 userid=100 xid=36036362 ]' LOG: SET search_path TO "public"

Same problem as for RDS log format, commit 3e3c1ae fixes also this issue. Thanks for the report.


Back

How do i get back to choose the wallet, i choose Trezor instead of Ledger, and now i'm stuck with Trezor, no back button , not even in the menu, i tried to close the app and relaunch and it's the same.

Go to Settings, there's a dropdown to change preferred vendor.

You can do it from the login screen or inside your wallet.

closing


Add Dockerfile

Hi,
this Dockerfile builds a peerflix-server Docker image. I've added my fork to my hub account: https://hub.docker.com/u/fish/peerflix-server
If you're up for it, I'd suggest you sign up as well so you can provide official images.
Using this image is as simple as running:
docker run -p 9000:9000 fish/peerflix-server

You can always build the image locally by running docker build -t peerflix-server . in this repo.

I think it would be more efficient to install the package from NPM rather than building it from source. Also, I would probably add forever support and maybe an upstart script if it makes sense for docker.

@asapach The nice thing about building from souce is that the docker image automatically gets updated when someone pushes change to peerflix-server.
Using upstart in general is discouraged since a container should ideally just run one application and forever isn't really necessary, since docker can take care of restarting failed containers (see docker run --restart).

Next question: you probably would want to move the torrent storage(/tmp/torrent-stream) outside of the container into a separate volume.

Next question: you probably would want to move the torrent storage(/tmp/torrent-stream) outside of the container into a separate volume.

And maybe also the config folder:~/.config/peerflix-server/

@asapach I would keep ~/.config/peerflix-server in the container. If people need to customize the config, they can either create their custom docker image based off this image like this (with my fork as example)
FROM fish/peerflix-server
ADD config/ /home/app/.config/peerflix-server

Or they can bind-mount a directory from host with a config like this:
docker run -v path/to/config:/home/app/.config/peerflix-server ...

Thanks, merged. I'll need to play with this.

Created the Docker repo: https://registry.hub.docker.com/u/asapach/peerflix-server/


version 1.0.4 release

Hello Martin,
Could you please do a release with version 1.0.4 ?
thanks in advance!

There isn't anything impacted code with the 1.0.4 release.  It is just readme updates right now.


Any example with ellipsis?

I'm trying to use the ellipsis but I don't get it to work. There isn't any documentation around it.
What I'm trying to achieve is something like this:
https://stackoverflow.com/questions/3922739/limit-text-length-to-n-lines-using-css

Interesting feature, dynamic text is not being maintained as of now. If you want to fix it, you're welcome to do so :)


Dev

i really  want to create some projects am new to programming and there are no tutorial for what i need

Hi! Note that the project has moved to https://github.com/RiseRobotRise/Joyeuse
And it's not development ready yet, if you want help you can enter these chat rooms: https://discordapp.com/invite/zH7NUgz
https://discord.gg/rvgpduC


EasyCodingStandard - phar file

Hello, I tried to compile EasyCodingStandard as a phar file for using it standalone but it doesn't work.
I opened discussion on StackOverflow. There are also print-screens of my problem. Can you check it please? Thank you.
https://stackoverflow.com/questions/50052815/phar-doesnt-work-after-compiling

Hey, what issue do you try to solve?
Building phar is very complicated thing for Symfony-based applications, see very similar architecture at Rector:

https://github.com/rectorphp/rector/pull/389
https://github.com/rectorphp/rector/pull/449
https://github.com/rectorphp/rector/issues/177

@TomasVotruba I think for ECS it might be actually easier, as I think there will be no exceptions with namespace prefixing. Still I think he should think of prefixing only as it is much easier than fixing PHAR incompatibilities.

I see no prefixing in original question.

I assume he is creating PHAR coz of dependency conflicts.

I think problem with skip is caused by incompatibility of glob PHP method.

Thank you for replies guys. It seems that the problem is in doctrine/dependency-injection. I'm gonna write them to the issues. Maybe they will fix it :-)

@zener05 Wow, that's awesome! :rocket:
Thanks for sharing this with use, we got stuck on the very same problem

The PR in Symfony for reference: https://github.com/symfony/symfony/pull/26845

@TomasVotruba would it be possible to add a prebuild phar file to the github release page of EasyCodingStandard (like in https://github.com/phpstan/phpstan/releases)

Hi, why not!
Feel free to send PR

Good news everyone, the ecs.phar is here:
https://github.com/Symplify/Symplify/pull/1734


Set Up Continuous Integration Through Github Actions

CI implemented here through Github Actions, ensuring that linting and tests will be checked upon each merge into master.

Looks good! Thanks for setting this up :)

Oops sorry, I accidently re-requested your review cuz it was top on suggested reviewers lmao


[Bug Report] Selecting a date or time with mouse resets tabIndex

Element UI version
2.3.9
OS/Browsers version
Windows 10/Chrome- latest
Vue version
2.5.16
Reproduction Link
https://jsfiddle.net/7rkvvmw1/
Steps to reproduce
Tab into the date and/or time field
Select a date using the mouse
Hit tab
What is Expected?
The focus to go to the next field
What is actually happening?
The current tabIndex gets reset

It is an interesting case.Because if you select the the date or time field then you will unfocus the input where in the page. if you want to reach your expect that you can write a function bind on change event make cursor after you select.

@leslieSie I've created a solution based on your comment here: https://jsfiddle.net/amoafc0f/.  On change, you just need to call:
      	this.$refs.datePicker.focus();
        this.$refs.datePicker.pickerVisible = false;
I'm wondering if this should be the default behavior of this component though.

@ChuckFields  Good Job!!!!!

Well.. my workaround doesn't always work. If you select a date in the picker that is already selected, the change function does not fire and the focus does not get reset.
I should also note that this is an issue with the time picker as well.


gce cluster workload "Group by Node" view alignment bad

Create a gce cluster, deploy some workload (or update the cattle-system, kube-system namespace to default project), then use the "Group by Node"

Version - 2.0 master 4/25
Verified fixed


Marketplace publisher isn't aligned

The publisher isn't aligned which is very obvious with the new checkbox.

Probably need to set line-height on the element.

fyi @prashantvc

Thanks, we are aware of alignment issue :)
We will fix this issue in Dec iteration

Another one main page and search results

@prashantvc verified icon alignment issue on search results has been pushed.
Also, as discussed for publisher profile alignment issues leaving for UI revamp.

@anupkmr03 then let's go ahead and close this issue if the fix has been pushed for search?

Thanks @isidorn  for the confirmation closing this.


Release v5.11.0

I suspect we need this in Julia 1.10 and 1.11 to solve the Enzyme issues. Is the next step to bump the release on Julia master?


chore: dependencies upgrade

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

AWS CodeBuild CI Report

CodeBuild project: DDKIntegrationTestsD8472609-5SCHWjldmWcf
Commit ID: 08b8aa6e30310bb979c656e449aec02c168f6710
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository


Link to Sample: Query Metadata and Detect Changes is broken

The link goes to https://docs.microsoft.com/en-us/dynamics365/customer-engagement/developer/org-service/sample-query-metadata-detect-changes, but it redirects to https://docs.microsoft.com/en-us/powerapps/developer/common-data-service/org-service/metadata-option-sets
It should go here: https://docs.microsoft.com/en-us/powerapps/developer/common-data-service/org-service/samples/query-metadata-changes

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

ID: 6fccbc62-5a81-6b35-b133-f1ad2974045b
Version Independent ID: eb9c593b-6655-a49d-d754-c77f5d891ebc
Content: Retrieve and detect changes to metadata (Common Data Service) - Power Apps
Content Source: powerapps-docs/developer/common-data-service/org-service/metadata-retrieve-detect-changes.md
Service: powerapps
GitHub Login: @JimDaly
Microsoft Alias: jdaly

@phecke Can you fix this link please?

Link seems to have been fixed.


[WIP]add the log crate to enable logging

Signed-off-by: Lokesh Mandvekar<EMAIL_ADDRESS>@flouthoc PTAL

@lsm5 looks good, could you also add a sample debug log in teardown or setup command.

@lsm5 looks good, could you also add a sample debug log in teardown or setup command.

ack, will do.


SSL feed not working

Hi
This doesn't seem to work for SSL feeds. for e.g. https://www.bis.org/list/bis/index.rss. I tried in plunkr setting ssl option to true, but doesn't work. Is this a limitation?
Thanks

Want to back this issue? Post a bounty on it! We accept bounties via Bountysource.

Hm the problem might be the invalidity of the feed itself: https://validator.w3.org/feed/check.cgi?url=https%3A%2F%2Fwww.bis.org%2Flist%2Fbis%2Findex.rss
Please note that you only need to enable the ssl option if you are running on an SSL page. It's not for rendering feeds that are hosted on an SSL endpoint.

I'm not so sure. If I open the feed in IE, it seems to work fine. I've tried using another jquery plugin, that seems to bring the result - http://jquery-plugins.net/FeedEk/FeedEk_demo.html

I have similar problem. I'm using jquery-rss in an ssl-hosted page, so I have to enable ssl:true option, otherwise the browser blocks my request. because of mixed content (The feed I'm reading is hosted on a non-ssl server)
Anyway, I tried this in a plnkr, and if i enable ssl, the request sent by jquery-rss goes to https://feedrapp.herokuapp.com, and the response status is 503.
If I do not enable ssl, the request goes to http://www.feedrapp.info, and the response status is 200.
So I think the problem is somewhere around your ssl hosted service...

Oh crap! I will fix that. I think you can set a host to
https://feedrapp.info and enable ssl
László Huszta<EMAIL_ADDRESS>schrieb am Do. 22. Juni 2017 um
14:43:

I have similar problem. I'm using jquery-rss in an ssl-hosted page, so I
have to enable ssl:true option, otherwise the browser blocks my request.
because of mixed content (The feed I'm reading is hosted on a non-ssl
server)
Anyway, I tried this in a plnkr, and if i enable ssl, the request sent by
jquery-rss goes to https://feedrapp.herokuapp.com, and the response
status is 503.
If I do not enable ssl, the request goes to http://www.feedrapp.info, and
the response status is 200.
So I think the problem is somewhere around your ssl hosted service...
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/sdepold/jquery-rss/issues/92#issuecomment-310368762,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAE1OxCViYudQwvBYE4_vCpdUBhraHrrks5sGmFggaJpZM4HVJDj
.

Hello, I removed my previous comment, because I realised what caused my problem. Simply my version of the rss plugin was outdated. After updating to the newest, the ssl requests are nicely going to feedrapp.info.

At the moment feedrapp.info is down, it responds with status code 500. Are you aware of this problem?

Is it better now?
László Huszta<EMAIL_ADDRESS>schrieb am Fr. 23. Juni 2017 um
15:52:

At the moment feedrapp.info is down, it responds with status code 500.
Are you aware of this problem?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/sdepold/jquery-rss/issues/92#issuecomment-310671063,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAE1O6O7HUXf-6Ypn6jJzj9pLk1rwssKks5sG8MJgaJpZM4HVJDj
.

Perfect. I also figured that later
László Huszta<EMAIL_ADDRESS>schrieb am Fr. 23. Juni 2017 um
15:51:

Hello, I removed my previous comment, because I realised what caused my
problem. Simply my version of the rss plugin was outdated. After updating
to the newest, the ssl requests are nicely going to feedrapp.info.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/sdepold/jquery-rss/issues/92#issuecomment-310670921,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAE1OzzG5TUgEpvhE7YLjzF93E3aec6Eks5sG8LngaJpZM4HVJDj
.

Definitely better! :smile: All the feeds I'm using are working fine. Thanks for the quick response, and fix! Big respect!

🙂 I didn't fully figure out yet why this happens from time to time but I
will investigate in it
László Huszta<EMAIL_ADDRESS>schrieb am Fr. 23. Juni 2017 um
16:08:

Definitely better! 😄 All the feeds I'm using are working fine. Thanks
for the quick response, and fix! Big respect!
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/sdepold/jquery-rss/issues/92#issuecomment-310675158,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAE1O7DNYfp9E8QKMTEhQRB0JrX65gdxks5sG8bvgaJpZM4HVJDj
.

As I understand the feedrapp server is a node application. Do you use pm2 or foreverjs or similar process management tool to ensure it runs continuously? - obviously just a workaround, not a solution for the errors :slightly_smiling_face:

Pm 2 I use
László Huszta<EMAIL_ADDRESS>schrieb am Fr. 23. Juni 2017 um
16:17:

As I understand the feedrapp server is a node application. Do you use pm2
or foreverjs or similar process management tool to ensure it runs
continuously? - obviously just a workaround, not a solution for the errors
🙂
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/sdepold/jquery-rss/issues/92#issuecomment-310677144,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAE1O5HH-BySgqv_a5N3lB0CZ_9Jkqelks5sG8jegaJpZM4HVJDj
.


Feature/#52 reorg v and v menu

Added geoJSON, still work in progress

To test, build the branch.
Push .geojson files to the tablet's /sdcard/Download folder.  Launch the app.
Launch a map, exit and select Add GeoJSON from the navigation menu.  Add an overlay.  Touch Add GeoJSON.  Pick from the list of files.  The features in the file should appear on the map.

Reintegrate


Consider moving jOOQ configuration stuff into a jooq element

The plugin configuration looks like this:
<configuration>
        <!-- Plugin configuration -->
	<database>
	</database>

        <!-- delegation to flyway configuration -->
	<flyway>
	</flyway>

        <!-- delegation to liquibase configuration -->
	<liquibase>
	</liquibase>

        <!-- jooq configuration ? -->
	<generator>
		<database>
			<includes>.*</includes>
			<excludes>flyway_schema_history</excludes>
			<inputSchema>public</inputSchema>
		</database>
		<target>
			<packageName>org.jooq.codegen.maven.example</packageName>
			<directory>target/generated-sources/jooq</directory>
		</target>
	</generator>
</configuration>

I think it would be more clear to put jOOQ stuff inside a jOOQ element. I'm aware that this would produce an apparently unnecessary additional level of nesting, but:

Even in jOOQ, <generator/> isn't a top level element. While <jdbc/> isn't necessary in most cases, it may be in some (e.g. to pass additional properties to the JDBC connection)
There are elements like <skip/>, <logging/>, <onError/>, <onUnused/>, <configurationFile/>, and others that might be added in the future, which you won't support out of the box unless you nest jOOQ's configuration in a <jooq/> element. Note, you could add support for these in your own top level, but then that would be confusing as it wouldn't be clear if e.g. <skip/> belongs to jOOQ, or to you.
Who knows if you're going to add more additional elements in the future, e.g. to clean up / post-process generated code, etc. Those elements will also possibly conflict with jOOQ stuff.

Hi @lukaseder
Thanks for the suggestions. As you suggested, moving jOOQ related config inside <jooq>...</jooq>.
@romchellis would you like to work on these changes? If we can incorporate these changes by next week we can release 0.0.2 along with these changes.

Hi @sivaprasadreddy , @lukaseder .
I like suggested approach.
Actually I'am already looking on possible solutions.Not sure that it will be completed by monday, possibly by the end of next week.


Does this implementation support tiny yolo v3?

I tried to run tiny yolo v3 followed the instructions as listed for yolov3, but I got segmentation fault. Does anyone tried the tiny-yolo_v3

Yes,
I'm running tinyYolov3 without problems on a JetsonTX2


Verify Go Legacy Encryption with the standard Aries Python SDK

Build wrapper for Go crypto (primarily nacl Box) to match API of Python sodium box for interoperability with ACA-Py.
Part of #36
Related to #139

@troyronda assign me

Fixed by #292


R2 Support

Hello everyone,
I would like to know if you already have an ETA regarding R2 support.
Thank you

Hello,
Any ETA on native R2 support?
Thanks

Same here. Also, in the meantime, is there an example of how to access S3 inside a Rust worker, whether AWS S3 or R2?


Add WITH command

Thinking about it, this may need to be FOREACH_INSTANCE <iterator> IN <object> THEN

Added up being an extension for the standard FOREACH loop. WITH is potentially possible but the scoping issues make things tricky... maybe we'll revisit this one in the future.


Viskit Installation Problem

I met this problem while running pip install -e viskit
(mbpo) dingcheng@dingcheng-GP63-Leopard-8RE:~/Documents/mbpo$ pip install -e viskit
ERROR: File "setup.py" not found. Directory cannot be installed in editable mode: /home/dingcheng/Documents/mbpo/viskit

Any idea why that would happen? Thanks

It sounds like your viskit folder might be empty. Did you include the --recursive flag when cloning the repo? (See the first command in the installation instructions in the README.)


how to know page index is next or previous   `

What version of React-Table are you using?
6.7.4
I have a table with server-side rendering. I use loadMoreentries() function of graphql for page pagination. here :
<ReactTable
        manual
        data={tableData}
        columns={columns}
        loading={loading}
        onPageChange={pageIndex => loadMoreEntries({ pageIndex: pageIndex })}
        onFetchData={state => {
          const options = { where: JSON.stringify(state.filtered) };
          refetch(options);
        }}
      />

how i can know what was previouse page index?

You have to manage that in your own state (by collecting it in your onPageChange handler and storing it in store by you load the data)

Hmm, what I think the functionality should be is that if you set the page attribute to x, when you click next it should be page +1 and when you click previous it should be page - 1. State management should not be required. I consider it a bug that this is not the case.


feat: migrate Camera to new arch

PR adding Camera implementation on new arch and adding all the files to fabricexample so it is easier to test new things there.

I resolved the issues. You still need to trigger the jobs for CI, right?

@WoLewicki you have now access I think, so you can push it to a branch on this repo? then CI can run.

Ok, I'll check

@WoLewicki I'd like to update our example to rn 0.73 and will be requiring 0.73 for fabric going further. Is this an issue for you?
0.73 seems to have UnsafeMixed support for events/methods.


Allow descriptor type 12

The doc generation process was broken due to an uncaught exception.
Fixes the currently breaking build. @dcolligan @bwalsh @saupchurch

Thanks @mbaudis , it was a comment whitespace problem :/


chore: add guidance on using knock as template source of truth

Description
This PR adds a section to the docs on Why store all of my notification templates in Knock? It's currently marked as a draft for a first pass of feedback before we are ready to merge things.
Questions

Now that I've typed this up, it does seem a bit long to be buried on the page where it is currently. Does it feel like this should be somewhere else?

I wasn't sure about using the <Step> components here (the through line really gives them the "follow these steps" feel and that's not really what this is), but I liked that it indented things and made them look a bit more organized. I tried with just h4 headings and it looked a bit weird to me, but maybe it's better (see below)? Alternatively, we could do the h4 headings but without numbering.

Content: I found myself having a very difficult time not repeating myself/using the same language on the short explanations for all of these things ("unified", "centralized", etc.), in spite of the fact that they are all technically different reasons. Please feel free to wordsmith or suggest cuts/modifications. I'm also curious about the order of the items (maybe number 6 should be pulled up above 4 and 5?).

Lastly, with all of the above taken under consideration: does this feel compelling? i.e. would the information as presented motivate you to commit to the extra upfront effort with the knowledge that the advantages you would gain would be worth it?

My thoughts:

I feel as though this is the best place for this. I know we chatted about how it might also make sense as a blog post, but having it on this page feels like the best fit to me.
We also chatted about this one, and I think the <Step> component looks best. If we were to do just a h4 my vote would be to do so without the numbering.
At first glance, it read well to me! I personally like 6 being separated and love that the section ends with "and isn't possible with other templating solutions," as that felt more compelling to me.
I think so!

I like this addition a lot. Reading it, it feels like this is the right page for the content but the H2 feels a bit awkward to me alongside the rest of the table of contents of this page.
Can we try it in an FAQ? If it is a common question we hear from customers, then that feels like the right place to put it since it's a pattern we use on other pages in the docs (and one we should probably remember to add to docs like Audiences concept doc and keep updated as we get new questions).
If we do put it under an FAQ, it will be a longer one, but I think that's okay and we can take it out of  and just use an unordered list with bold intros at the start of each one.
E.g.

A consistent editing experience. When you manage all your...
Cross-channel engagement analytics. With Knock link and...

But yes, 100% compelling and an item we should callout to Hashim and sales team so they know it exists and can link to it from sales enablement assets DB in notion


Allow overriding group colors

For example, use the following lua config:
-- zephyr override
vim.g.zephyr = {
	override = {
		TSVariable = {fg='wheat'};
	};
}

Anything on this PR?

Is this project dead?

alive
--------------原始邮件--------------
发件人："tzachar @.>;
发送时间：2021年6月27日(星期天) 晚上8:01
收件人："glepnir/zephyr-nvim" @.>;
抄送："Subscribed @.***>;
主题：Re: [glepnir/zephyr-nvim] Allow overriding group colors (#17)
Is this project dead?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or unsubscribe.

Anything on this PR?

sorry I am in hospital when I go home I will check it
--------------原始邮件--------------
发件人："tzachar @.>;
发送时间：2021年6月27日(星期天) 晚上8:05
收件人："glepnir/zephyr-nvim" @.>;
抄送："Raphael @.>;"Comment @.>;
主题：Re: [glepnir/zephyr-nvim] Allow overriding group colors (#17)
Anything on this PR?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or unsubscribe.

Sorry to hear that. Hope you get well soon :)


oops ignore this, accidentally pushed into original, not my fork

just close the PR @Blatantist

_ No description provided. _

that's useless and the wrong way to interest shadowmario

its an accidental pr btw

kid named Close pull request

Why is this still opened
@Blatantist close this pr


Notebooks for Activation Atlas

Pull Request Test Coverage Report for Build 448

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 81.978%

Totals

Change from base Build 446:
0.0%

Covered Lines:
1492

Relevant Lines:
1820

💛  - Coveralls


copy of Mutex is not safe

common.SignatureData has MessageState field which contains Mutex.
when signing is done we need to get common.SignatureData from the end channel which copies the value inside the channel.
this operation is not safe in terms of concurrency handling.
I suggest that the end channel will be of *common.SignatureData (pointer of common.SignatureData) this way the copy will be by reference which is safe.
I also suggest that for compatible with the keygen process, the end channel of keygen will also be of type *LocalPartySaveData (pointer of LocalPartySaveData). even though LocalPartySaveData has no Mutex.

Thanks. https://github.com/bnb-chain/tss-lib/pull/251 is a related PR.

Great! when can I expect it to be published to https://pkg.go.dev ?

https://github.com/bnb-chain/tss-lib/releases/tag/v2.0.1 here is a new release. It should be updated on pkg.go.dev soon.

thanks a lot. it also made me switch to v2.
I didn't know I should.


v0.2.6

Coverage remained the same at 72.234% when pulling ae19e78629d4ed15efa5258e3327c60f96bf471d on v0.2.6 into b09ae61fe2fc97f1b3596b546353da5ebccebc3d on master.

Coverage remained the same at 72.204% when pulling 74a6ad68473ac295318c4b42ad982bf395ad80af on v0.2.6 into fcd23271948583082959ade315653d3476112bd2 on master.


Modifiers For Macro Arguments

Add Modifiers for macro arguments to distinct between register, address and immediate and pointer dereferencing to build a more complex macro instead of writing many different macros

The basis for distinction between registers and values are the enum's. Any variable can be checked if it's part of an enum with the "is" and "is not" operator. A value not part of an enum can be type cast with the "as" operator.
In the example language file the instructions accept the following. Where the first must be an register and the second can be een immediate or a register.
mov r0 23
mov r0 r1

But with to allow address distinction you could define en empty enum
enum address {
}

mov (123 as address) 54
mov (123 as address) r1

With the use of the "is" and "is not" in combination with argument expansion and a data type enum, you can perform argument matching, thus allowing a syntax like.
enum datatype {
   addr = 1
   imm = 2
   ptr = 2
}

mov addr 123 34
mov addr 123 r1

In the list of wishes to add are array's which would be handy for addresses as in most instruction sets you can use a base value, like a stack/base pointer and add or subtract an offset to it, mostly used to access arguments and local variables. This will also be needed to support x86 for example.
mov [sp 3] 45

An array in combination with enum types as modifiers can make all kind of constructions possible, without having to make use of argument matching.


Using --fast option runs out of memory even when regular option does not

Thank you for making TotalSegmentator available. It looks like an amazing resource!
While processing a dataset, I was running into CUDA out of memory issues for nii.gz files of around 150 MB. To process these, I tried to use the --fast option assuming that a smaller model would use less memory (and the README does indicate this). However, it had the opposite effect making files with size 92 MB, which process ok without the --fast, run out of memory:
torch.cuda.OutOfMemoryError: CUDA out of memory. Tried to allocate 6.58 GiB (GPU 0; 23.70 GiB total capacity; 13.64 GiB already allocated; 4.23 GiB free; 18.07 GiB reserved in total by PyTorch) If reserved memory is >> allocated memory try setting max_split_size_mb to avoid fragmentation.  See documentation for Memory Management and PYTORCH_CUDA_ALLOC_CONF
Exception ignored in: <totalsegmentator.libs.DummyFile object at 0x7f0e2c318d90>
AttributeError: 'DummyFile' object has no attribute 'flush'

Hey @bdzyubak!
As a fellow TotalSegmentator user, I believe this happened to me once while playing with the tool.
Are you sure other runs/other data did not clutter your GPU VRAM? As far as I can remember, that solved the problem for me. Besides that, I ran TotalSegmentator hundreds of times and didn't run into any issues (using even bigger .nii.gz volumes).


Updates Search tool documentation and screenshots

Updates screenshots
Updates textual references to drawer with Quick View

Docs Build status updates of commit f27d205:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

microsoft-edge/devtools-guide-chromium/media/search-tool/open-search-tool.png
:white_check_mark:Succeeded
View

microsoft-edge/devtools-guide-chromium/media/search-tool/search-tool-first-open.png
:white_check_mark:Succeeded
View

microsoft-edge/devtools-guide-chromium/media/search-tool/search-tool-open-in-sources.png
:white_check_mark:Succeeded
View

microsoft-edge/devtools-guide-chromium/media/search-tool/search-tool-regexp.png
:white_check_mark:Succeeded
View

microsoft-edge/devtools-guide-chromium/media/search-tool/search-tool-search-results.png
:white_check_mark:Succeeded
View

microsoft-edge/devtools-guide-chromium/search/search-tool.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


Feature/frontend

hmm.. Good 👍🏻

Bruh
Vào 22:59 T.2, 28 Th10, 2024 Jun @.***> đã viết:

Merged #4 https://github.com/JunCodera1/SoundBox/pull/4 into main.
—
Reply to this email directly, view it on GitHub
https://github.com/JunCodera1/SoundBox/pull/4#event-14964238293, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AY4G5U5TD5AMZISLVSABBELZ5ZNO5AVCNFSM6AAAAABQXNYVFCVHI2DSMVQWIX3LMV45UABCJFZXG5LFIV3GK3TUJZXXI2LGNFRWC5DJN5XDWMJUHE3DIMRTHAZDSMY
.
You are receiving this because you authored the thread.Message ID:
@.***>


pytest 6.1 compatibility

Please, update requirements to support pytest 6.1

Sorry for missing this issue! I just published version 1.0.5 to PyPI, which formally supports pytest 6.1 — and also relaxes the version pinning to allow all 6.x versions.

Nice, thank you!


Memory issue while using Array<Vector>

Hi,
I created a mfem::Array<mfem::Vector> my_array object and push back the individual vector to it within a loop, but I got some memory error after a few iterations:
rbf_prolong.bin(4423,0x10ee79e00) malloc: Incorrect checksum for freed object 0x7fd201e04178: probably modified after being freed.
Corrupt value: 0x300000000
rbf_prolong.bin(4423,0x10ee79e00) malloc: *** set a breakpoint in malloc_error_break to debug

Here is the sudo code:
Array<Vector> my_array; // this is outside the function and it's passed into the function by a reference

// inside function
my_array.LoseData();
int dim = 2;
Vector coord(dim);
for (int i = 0; i < mesh->GetNE(); i++)
{
  coord = // something;
  my_array.Append(coord);
}

The Array class is only intended for so-called trivial types, and Vector is not trivial. In fact, with the latest master, that code should give a compilation error
../mesh/../fem/../general/array.hpp:57:7: error: static_assert failed due to
      requirement 'std::is_trivial<mfem::Vector>::value'
      "type T must be trivial"
      static_assert(std::is_trivial<T>::value, "type T must be trivial");

since #2214 was merged.

I should add that an alternative approach that could work for your purposes is to use std::vector<Vector>.

Thank you very much. Let me try that


feat(jubako/arx): scaffold jubako/arx

jubako/arx: Store files and directory in an archive. Like tar, but faster and with direct random access
$ aqua g -i jubako/arx

Check List

[x] Read CONTRIBUTING.md

[x] Read OSS Contribution Guide

[x] All commits are signed

This repository enables Require signed commits, so all commits must be signed

[x] Avoid force push
[x] Do only one thing in one Pull Request
[x] Execute cmdx s to scaffold code
[x] Install and execute the package and confirm if the package works well

System Prerequisites
I don't know if you want to distribute tools which have a system dependency.
This one have the lib "libfuse2" as dependencies as we can see when we launch it first time :
$ arx
arx: error while loading shared libraries: libfuse.so.2: cannot open shared object file: No such file or directory

This can be fixed with "libfuse2" installed :
$ sudo apt-get install -y libfuse2

How to confirm if this package works well
Reviewers aren't necessarily familiar with this package, so please describe how to confirm if this package works well.
Please confirm if this package works well yourself as much as possible.
Command and output
$ arx
A file archive based on Jubako container.

Usage: arx [OPTIONS] [COMMAND]

Commands:
  create   Create an archive
  list     List the content in an archive
  dump     Print the content of an entry in the archive
  extract  Extract the content of an archive
  mount    Mount an archive in a directory
  help     Print this message or the help of the given subcommand(s)

Options:
  -v, --verbose...  Set verbose level. Can be specify several times to augment verbose level
  -h, --help        Print help
  -V, --version     Print version

Advanced:
      --generate-man-page [<GENERATE_MAN_PAGE>]
          [possible values: , create, list, dump, extract, mount]
      --generate-complete <GENERATE_COMPLETE>
          [possible values: bash, elvish, fish, powershell, zsh]

$ arx --version
arx 0.2.1

Thank you for your contribution!

📝 On macOS, I needed to install macFUSE.
$ arx --help
INFO[0000] download and unarchive the package            aqua_version=2.30.0-3 env=darwin/arm64 exe_name=arx package_name=jubako/arx package_version=0.2.1 program=aqua registry=standard
Downloading jubako/arx 0.2.1 100% |████████████████████████████████████████████████████████████████████████████| (2.1/2.1 MB, 28 MB/s)
INFO[0001] downloading a checksum file                   aqua_version=2.30.0-3 env=darwin/arm64 exe_name=arx package_name=jubako/arx package_version=0.2.1 program=aqua registry=standard
dyld[24298]: Library not loaded: /usr/local/lib/libfuse.2.dylib
  Referenced from: <D50CC246-DDA5-3E78-8A5D-C588EB7C4DC1> /Users/shunsukesuzuki/.local/share/aquaproj-aqua/pkgs/github_release/github.com/jubako/arx/0.2.1/arx-0.2.1-macos.tar.gz/arx-0.2.1-macos/arx
  Reason: tried: '/usr/local/lib/libfuse.2.dylib' (no such file), '/System/Volumes/Preboot/Cryptexes/OS/usr/local/lib/libfuse.2.dylib' (no such file), '/usr/local/lib/libfuse.2.dylib' (no such file), '/usr/lib/libfuse.2.dylib' (no such file, not in dyld cache), '/opt/homebrew/lib/libfuse.2.dylib' (no such file), '/System/Volumes/Preboot/Cryptexes/OS/opt/homebrew/lib/libfuse.2.dylib' (no such file), '/opt/homebrew/lib/libfuse.2.dylib' (no such file), '/usr/local/lib/libfuse.2.dylib' (no such file), '/usr/lib/libfuse.2.dylib' (no such file, not in dyld cache)
zsh: abort      arx --help

brew info macfuse

I don't know if you want to distribute tools which have a system dependency.

We accept tools which have dependencies, though aqua doesn't install them.

How can we warn users about it ?
And I can't test on a windows operating system to check dependency too..

How can we warn users about it ?

aqua doesn't have the feature to warn users about it.
What we can do for it is to write warning in the package's description and the release note.

v4.205.0 is out 🎉
https://github.com/aquaproj/aqua-registry/releases/tag/v4.205.0


feat: address few bug with linting and improve highlighting

Description
Addresses the following issues with linter

[x] 1. Lint errors are not displayed until control is within the editor
[x] 2. Disable all warnings
[x] 3. Global data not getting imported
[x] 4. For scanning/parsing issues - highlight an entire code block
[x] 5. Same errors are displayed on 2 different lines
[ ] 6. Typo on suggestion for nested properties (add multi level global data) // Not possible
[x] 7. === issue (remove this rule)

We are using positions from jshint instead of string matching. This gives us much better location on errors
Type of change

Bug fix (non-breaking change which fixes an issue)

How Has This Been Tested?
Manually tested
Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[x] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes

Fixes #7170
Fixes #7171
Fixes #6901
Fixes #6811
Fixes #6800
Fixes #7163
Test coverage results :test_tube:
:green_circle: Total coverage has increased
// Code coverage diff between base branch:release and head branch: FEATURE/Show-lint-errors-in-codeeditor

Status
File
% Stmts
% Branch
% Funcs
% Lines

:green_circle:
total
54.45 (0.01)
36.27 (0.02)
33.55 (-0.01)
55.01 (0.01)

:green_circle:
app/client/src/components/editorComponents/CodeEditor/index.tsx
67.91 (0.12)
40 (0.39)
52.5 (0)
68.7 (0.13)

:red_circle:
app/client/src/components/editorComponents/CodeEditor/lintHelpers.ts
91.78 (0)
65 (-13.57)
88.89 (-11.11)
89.83 (0.74)

:red_circle:
app/client/src/sagas/PostEvaluationSagas.ts
17.61 (-0.12)
0 (0)
0 (0)
21.55 (-0.19)

:green_circle:
app/client/src/workers/evaluate.ts
95.06 (0.54)
86.36 (6.36)
88.89 (-0.58)
94.94 (0.5)

@hetunandu @vnodecg not sure if the approach to highlight the entire block is an appropriate way to deal with parsing issues like seen in the screenshot below. User would be unable to pin point the issue if this was a longer code whereas the issue is to do with missing brackets. Our documentation and Google search aren't helpful in this case either.

@hetunandu @vnodecg not sure if the approach to highlight the entire block is an appropriate way to deal with parsing issues like seen in the screenshot below. User would be unable to pin point the issue if this was a longer code whereas the issue is to do with missing brackets. Our documentation and Google search aren't helpful in this case either.

If that is a case then lets not show any lint errors incase there is a parsing issue

Makes sense for most parsing errors. But in the case of missing bracket pairs, can we still do something because that's a common error and also something most IDE's do for UX


Adding a sass-loader option with chainWebpack requires double entry

What problem does this feature solve?
Given this example in the webpack part of the docs, you would assume that this is all you need to add a custom option for sass-loader, but that snippet only affects standalone .scss files handled by webpack.
Code inside .vue files in the <style lang="scss"> is managed by vue-loader and this change is not getting copied into the respective vue-loader.options.loaders.scss block automatically.
To achieve that, you currently need to do something like this:
var path = require('path')
var merge = require('lodash/merge')
var findIndex = require('lodash/findIndex')

module.exports = {
  chainWebpack: config => {
    config.module
      .rule('scss')
      .use('sass-loader')
      .tap(options =>
        merge(options, {
          includePaths: [
            path.resolve(__dirname, 'src/styles')
          ]
        })
      )
    config.module
      .rule('vue')
      .use('vue-loader')
      .tap(options => {
        var i = findIndex(options.loaders.scss, { loader: 'sass-loader' })
        merge(options.loaders.scss[i].options, {
          includePaths: [
            path.resolve(__dirname, 'src/styles')
          ]
        })
        return options
      })
  }
}

Notice the double entries; one for the scss rule and one for the vue rule. Webpack Chain is already hard enough to wrap your head around as it is :)
What does the proposed API look like?
I would simply expect that all changes added to top level extension rules like scss is automatically copied and applied to the same extension specific blocks inside the vue-loader options.
Is there any good reason why this should not be a sane default?

Update: this actually already works if I just use the shorthand config css.loaderOptions instead of trying to achieve it through chainWebpack as shown here.
Somebody has clearly thought this through already - awesome 🤘
Full example for others that stumble upon this:
module.exports = {
  css: {
    loaderOptions: {
      sass: {
        includePaths: [
          path.resolve(__dirname, 'src/styles')
        ]
    }
  }
}

The example in docs is slightly misleading and should perhaps be updated with the above example instead.

You don't need to add sass-loader yourself. Use css.loaderOptions in vue.config.js.


[#537] Add headerHashAnnotated

Closes #537

Looks reasonable, but the diffstat comes out to +4 stuck_out_tongue_winking_eye

Don't worry, I've got some changes coming that should be in the minus!

bors r+

Don't worry, I've got some changes coming that should be in the minus!

Heh, I figured as much 🙌


Load checkpoint with updated Trainer arguments

Just like the model, from docs:
# uses in_dim=32, out_dim=10
model = LitModel.load_from_checkpoint(PATH)

# uses in_dim=128, out_dim=10
model = LitModel.load_from_checkpoint(PATH, in_dim=128, out_dim=10)

I'd like to do e.g.
trainer.fit(model, ckpt_path..., detect_anomaly=False)

Is there a workaround in the meantime?
cc @borda

hey @OverLordGoldDragon !
why not do just
trainer = Trainer(..., detect_anomaly=False)
trainer.fit(model, ckpt_path=...)

Overridden by the checkpoint

checkpoint doesn't save any trainer arguments.

Strange, I saw trainer._detect_anomaly == True despite passing False, but now can't make an MRE. I'll reopen if I can confirm more definitively, thanks


Added dependancies.txt

Added the dependancies.txt file for the project with the installation instructions for Windows and Ubuntu users. If something is wrong, suggest changes.

Thanks OneEyedGhost this is great


Vim Ruby Version 2.3.7 but expected Version 2.5.3

I have used MacVim for years and absolutely love it. Thanks in advance for the help here.
I was excited for the new release because it fixed #257, which I reported in the past. However, after upgrading to MacVim 8.1.560 (152), I'm unable to use command-t in MacVim, because the expected version of Ruby doesn't match.
I get the following error:

Vim Ruby version: 2.3.7-p456
Expected version: 2.5.3-p105

I've tried brew uninstalling macvim, setting rbenv global 2.5.3, and then brew installing macvim. This did not work.
I've seen #779 and tried setting rubydll like it recommends. In order to do so, I installed ruby through brew
brew install ruby

In order to get a libruby.2.5.dylib, which I've found in the following path:
/usr/local/Cellar/ruby/2.5.3_1/lib/libruby.2.5.dylib

And I set rubydll as follows at the top of my .vimrc:
set rubydll=/usr/local/Cellar/ruby/2.5.3_1/lib/libruby.2.5.dylib

But I keep getting the error that vim is using Ruby version 2.3.7.
Would you have any ideas as to why vim is not using Ruby version 2.5.3?
Here's some system information:
macOS: 10.14.1
Homebrew: 1.8.4
rbenv: 1.1.1
rbenv ruby: 2.5.3
system ruby: 2.3.7
brew ruby: 2.5.3
Thanks again for all your hard work!

It sounds more like Cmd-T is expecting 2.3.7 to me. Did you rebuild the C components? One issue with the Cmd-T plugin is that the native component + ruby does mean there's more fussing needed to get it to work.
You can always do :ruby puts RUBY_VERSION in Vim to be sure what Vim is using.

I was able to compile Command-T against Ruby 2.3.7 to get that Command-T working for me, since MacVim uses Ruby 2.3.7.
Still not certain why I couldn't get MacVim to use Ruby 2.5.3, though.
The release notes say:
Script interfaces have compatibility with these versions:
…
Ruby 2.5

And this lead me to believe that I would need to have Ruby 2.5 installed in order to use MacVim, but this does not seem to be the case. Is it mentioned anywhere what version of Ruby MacVim requires?

Cool! Glad to see if working for you.
I'm not sure why MacVim keeps using the 2.3 library. Setting rubydll should work and you can try setting it at runtime instead of in your vimrc to see if it works.
As for the versioning, Vim is usually compiled against a particular Ruby version, so when we build MacVim we do the same thing. Since we compile with +ruby/dyn we load the library dynamically so that users who don't care about Ruby don't have to deal with it. Technically, the library you load in should match the version Vim is compiled with. See :h ruby-dynamic:

Unix ~
The 'rubydll' option can be used to specify the Ruby shared library file
instead of DYNAMIC_RUBY_DLL file what was specified at compile time.  The
version of the shared library must match the Ruby version Vim was compiled
with.

That said on my machine I find that it works whether I use a 2.3 or 2.5 library, so the changes may not be big enough.


check-meta: Another round of assorted performance improvements

Description of changes
This should also help with better NixOS evaluation performance because it improves value merging.
Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[ ] Fits CONTRIBUTING.md.

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

Because ofborg is posting the evaluation performance report with checkMeta = false, here are the deltas using checkMeta = true:
{
  "cpuTime": -4.220979540693733,
  "envs": {
    "bytes": -2.5812792390687775,
    "elements": -4.030667046102138,
    "number": -1.6404360669685047
  },
  "gc": {
    "heapSize": -1.9082648688065689,
    "totalBytes": -6.231961937741872
  },
  "list": {
    "bytes": -0.41426822450232237,
    "concats": -0.005216563647053363,
    "elements": -0.41426822450232237
  },
  "nrAvoided": 8.762861000104934,
  "nrFunctionCalls": 2.099203978456984,
  "nrLookups": -8.056222273737504,
  "nrOpUpdateValuesCopied": -0.0009998505487942566,
  "nrOpUpdates": -0.003548833390354389,
  "nrPrimOpCalls": -0.3193114471500138,
  "nrThunks": -10.87584626520588,
  "sets": {
    "bytes": -7.130238304130813,
    "elements": -5.420259487165595,
    "number": -20.12622857468783
  },
  "sizes": {
    "Attr": 0.0,
    "Bindings": 0.0,
    "Env": 0.0,
    "Value": 0.0
  },
  "symbols": {
    "bytes": -0.0001363888062968499,
    "number": -0.0006235891296029195
  },
  "values": {
    "bytes": -9.684489952698812,
    "number": -9.684489952698812
  }
}

https://github.com/NixOS/nixpkgs/pull/273935 is better regarding the performance of meta checks.
The changes in check-meta.nix from this PR could be dropped if https://github.com/NixOS/nixpkgs/pull/273935 was to get merged.
The other changes are still worthwhile for better NixOS evaluation performance.

Dropped the check-meta.nix bits in favour of the bespoke type checking PR.


Option to specify 'check' severity

There is an option to check individual checks via their identifiers.
It would also be good if you could select the checks by their severity.
For example only "--check HIGH" as in https://github.com/bridgecrewio/checkov with "checkov -d . --check MEDIUM".
My current config is:
- name: Run Checkov action
        id: checkov
        uses: bridgecrewio/checkov-action@master
        with:
          directory: .
          quiet: true
          soft_fail: false
          framework: all
          output_format: sarif
          download_external_modules: true
          log_level: WARNING
          check: LOW # Results in - no checks applied

This feature has been in Checkov itself for a few months now and for me is one of the most useful configuration options to have. Enabling this setting in the Github Action would be by far the lowest overhead to integrate checkov into existing IaC projects, and get meaningful but well-scoped feedback for new projects as well.
Hope to see it feature here soon! 👍

      - name: tests-checkov-sam
        if: hashFiles('**/serverless.yml') != ''
        uses: bridgecrewio/checkov-action@v12
        with:
          check: 'MEDIUM,HIGH,CRITICAL'

I reckon you need at least 2 and a coma XD

hey everyone, this works since it was added to checkov, but you need use an API key for severities like in checkov.


Added clarification on validation technical profile execution

Ran into an issue recently where we wanted to call some validation technical profiles to take advantage of the ContinueOnError attribute of a ValidationTechnicalProfile element. The documentation didn't make it immediately clear (to me) that there must be some input from the user in the calling self-asserted technical profile to execute, and B2C does not alert the user to why it skipped the validation technical profiles.
This update should make this clear and save someone else some time.

@kengaderdus
Can you review the proposed changes? IMPORTANT: When the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"

@stuart938503
Thank you for your contribution and patience. Looking at your PR, do you think it will be more appropriate to add your content suggestion the Validation technical profile article, and change it as shown below:
Azure AD B2C triggers a validation technical profile only when there's an input from a user. It's not possible to create an empty self-asserted technical profile that exists to call validation technical profiles only to take advantage of the ContinueOnError.
In Validation technical profile article, there's already the note shown below:

So, we can combine this note with your message.
Let's know what you think.
Thanks.

@stuart938503
Thank you for your contribution. We've incorporated your contribution to our product content in another PR since you didn't respond to our proposed change.
#please-close


replace go.rice with embed

Go now has standard way how to deal with embedded resources.
That means we can replace go.rice with the embed package

thanks a lot for raising this! :+1: will work on this asap


introduce and use action

private only for now

Coverage decreased (-2.07%) to 96.544% when pulling 2d1c0ede8007b59831bbd3a9dbaabc834d95436f on ro0gr:actions into acc16eb1cc263fdab13ead520abdb808decf5f5f on san650:v2-beta.


Use Templatus for F# code generation

https://github.com/kerams/Templatus

It looks like Templatus is not under active development atm. Maybe we can revive that, or ask @kerams, who maintains that repo.

Yeah, it was mostly just a toy and is not .NET Core compatible at its current state. Shouldn't be too hard to migrate and convert it to a proper tool, but I doubt there is interest as better options are available.

I just thought about moving from hard-coded F# generation to some more manageable and maintainable

Perhaps Myriad? I vaguely remember other code generators, not sure if they can easily be plugged in though. It's probably gonna be quite a massive change.

Maybe this is a good option https://github.com/edgarfgp/Fabulous.AST

What is the purpose of changing the codegen AST, is there anything we want to express but can't due to a limitation? I don't believe so. If there is, consider opening an issue for it.


Sourcify support

Hi, just encountered the project randomly and really liked it. It is in lines with what we are trying to achieve at https://sourcify.dev.
Sourcify is a decentralized verified contract repository and an open-sourced smart contract verification tool. We pin the Solidity metadata files and source files on IPFS, which are retrievable through the bytecode (See in action in the playground).
We also want to make it easier for the users to do "edge verification" because while getting contracts from IPFS through the bytecode is convenient, one still needs to make sure the source code behind the IPFS CID really compiles to this code. Although really unlikely, the contract deployer can still tamper with the CID that is normally placed by the Solidity compiler.
Would love to see native Sourcify support on evmc. Let me know how we can help, and happy to chat!
cc: @marcocastignoli

Hey dude let’s do this, I’m really up for doing a full integration with sourceify, I’m a huge fan of that project. let’s start up a telegram and we can discuss the feature set ? I have a few questions around it also! Telegram dev_josh ! speak soon


Geiser REPL only working with Guile and not other schemes  in Emacs.

I have Geiser installed in emacs.
When I open a scheme program  and run  C-c C-a  it starts a  guile  REPL that works as it should.
Now when I try   run-geiser and select     racket  I get a REPL but cannot seem to use the functions in the original module.

I have interactively tried to change  geiser-active-implementation  by changing order of scheme interpreters but that doesn't seem to work.

Will be happy to provide more information.

this is by design: your original module is attached to the Guile REPL,
not to the Racket one.  definitions are not interchangeable (the schemes
we support are different, and there's no good way a new scheme process can
load the scheme code that it was loaded in the other).
if you want to choose the original buffer to another scheme, use
geiser-set-scheme, bound to C-c C-s in scheme buffers.
to modify the scheme implementation that is associated with a file by
default, please take a look at
http://geiser.nongnu.org/geiser_4.html#The-source-and-the-REPL

Thank you for the clarification. This solves it!


Synology with Reolink

Hi I'm trying to configure "Homebridge Camera Ffmpeg".
I get ther following error:
[Camera ffmpeg] Snapshot from Rear Camera 2 at 480x270
[Camera ffmpeg] An error occurs while making snapshot request
[Camera ffmpeg] Start streaming video from Rear Camera 2 with 1280x720@299kBit
[Camera ffmpeg] An error occurs while making stream request
[Camera ffmpeg] ERROR: FFmpeg exited with code -2
My config is as follows:
                    "name": "Camera ffmpeg",
                    "cameras": [
                            {
                                "name": "Front yard",
                                "videoConfig": {
                                       "source": "-re -i rtsp://USER:PASSWORD@XXX.XXX.XXX.XXX:554//h264Preview_01_main",
                    "stillImageSource": "-i http://XXX.XXX.XXX:80/cgi-bin/api.cgi?cmd=Snap&channel=0&user=ADMIN&password=PASSWORD",
                    "maxStreams": 2,
                    "maxWidth": 2048,
                    "maxHeight": 1536,
                    "maxFPS": 30,
                    "maxBitrate": 8192,
                    "vcodec": "libx264",
                    "packetSize": 1316,
                    "audio": false,
                    "debug": false
                }
            }
        ],
        "platform": "Camera-ffmpeg"
    }
]

}

Here is what i got to work with my ReoLink camera that i use with Synology also. snapshot updates every minute audio in the stream works.
        {
            "name": "Front Drive",
            "cameras": [
                {
                    "name": "Front Drive",
                    "videoConfig": {
                        "source": "-re -i rtsp://admin:Aj0440925@<IP_ADDRESS>:554/h264Preview_01_sub",
                        "stillImageSource": "-i http://<IP_ADDRESS>/cgi-bin/api.cgi?cmd=Snap&channel=0&rs=wuuPhkmUCeI9WG7C&user=admin&password=Aj0440925",
                        "maxStreams": 5,
                        "maxWidth": 2560,
                        "maxHeight": 1920,
                        "maxFPS": 30,
                        "audio": true
                    }
                }
            ],
            "platform": "Camera-ffmpeg"
        }

Here is what I got to work with my reolink cameras I also use on synology. Snapshots update and the audio feature I was able to get to work also.
        {
            "name": "Front Drive",
            "cameras": [
                {
                    "name": "Front Drive",
                    "videoConfig": {
                        "source": "-re -i rtsp://myusername:mypassword@/ipaddress:554/h264Preview_01_sub",
                        "stillImageSource": "-i http://ipaddress/cgi-bin/api.cgi?cmd=Snap&channel=0&rs=wuuPhkmUCeI9WG7C&user=myusername&password=mypassword",
                        "maxStreams": 5,
                        "maxWidth": 2560,
                        "maxHeight": 1920,
                        "maxFPS": 30,
                        "audio": true
                    }
                }
            ],
            "platform": "Camera-ffmpeg"
        }

This does not solve my problem.
I have fresh installed the docker, installed homebridge and the ffmpeg plugin on my synology DiskStation.
My Config.json is:
{
"name": "Camera ffmpeg",
"cameras": [{
"name": "Reolink",
"manufacturer": "Reolink",
"model": "RLC-420-5MP",
"serialNumber": "123456",
"videoConfig": {
"source": "-re -i rtsp://myuser:mypassword@192.168.x.xxx:554/h264Preview_01_sub",
"stillImageSource": "-i http://192.168.x.xxx/cgi-bin/api.cgi?cmd=Snap&channel=0&rs=wuuPhkmUCeI9WG7C&user=myuser&password=mypassword",
"maxStreams": 5,
"maxWidth": 2560,
"maxHeight": 1920,
"maxFPS": 30,
"audio": true
}
},
],
"platform": "Camera-ffmpeg"
}
The protocoll shows:
[Camera ffmpeg] Snapshot from Reolink Flur at 480x270
[Camera ffmpeg] An error occurs while making snapshot request
[Camera ffmpeg] Start streaming video from Reolink Flur with 1280x720@299kBit
[Camera ffmpeg] An error occurs while making stream request
[Camera ffmpeg] ERROR: FFmpeg exited with code -2
I can launch the rtsp stream with VLC and it works perfectly.
Really dont know what is the issue right here.

I failed to get it working on Synology / Docker install as well.  I believe this is related somehow to Synology not liking the ffmpeg running.  I have gotten it running well on a Raspberry Pi 4.

I might note I am running this container of homebridge that has ffmpeg. https://registry.hub.docker.com/r/nicholasrobinson/ffmpeg-homebridge

try and use the exact code I used also just with replacing your IP and credentials and see what happens.
I messed around with it for an entire day before I got this config to work and just stuck with it.

I might note I am running this container of homebridge that has ffmpeg. https://registry.hub.docker.com/r/nicholasrobinson/ffmpeg-homebridge

Your suggested container does not create any files.
There is no config.json in my folder.. do i have to create them manually?

I might note I am running this container of homebridge that has ffmpeg. https://registry.hub.docker.com/r/nicholasrobinson/ffmpeg-homebridge

Your suggested container does not create any files.
There is no config.json in my folder.. do i have to create them manually?

I just set the mount path the same as my original homebridge docker container so I didnt have to change anything.
I just setup a new test container with it and It setup just like a normal homebridge docker container. I logged into its UI on the port i set it to and all the files were in the /homebridge path I set.

Just wanted to pass along that while I stopped trying to get this working via Docker / Synology, I did use your suggested config.json entries on my RaspberryPI install however and they work perfectly!  The only item I didn't add was the snapshot line item but will try that later as well.
Thank you for sharing, saved me more hours of fighting with this.


CLI Binary for template generation

See: https://github.com/Malax/rust-cnb-starter

@hone I don't think we need this anymore with #199 which makes most of the project boilerplate obsolete. Thoughts? cc: @edmorley

Yeah I think between now having cargo libcnb package, the new Layer API being simpler, and having things like the buildpack_main! macro, there is a lot less to set up/understand when creating a new buildpack. In addition we have the hello world example buildpack people could just copy.

I'll close this for now, we can always reopen :)

@Malax I actually would still like something in carge libcnb package like rails new or a template unless we want to defer to pack for buildpack.toml generation etc.

Another option would to move this work upstream that would time into RFC#77.


Facepunch is dead

It's been replaced by knockout.chat, would be awesome if the code could be updated to reflect that in time for results!

Resolved, please see #28


[SP:1] Reconstruct the grid for whole project.

The goal is to change the width for all models, grid and complex blocks on the projects.

This epic isn't relevant anymore


remove ancestors config option

resolves #994
I had been hitting some instances where the ancestors option was out of date. I believe it was related to the fact that prepend was not clearing the ancestors branches.

@kevgo tests are failing due to an issue with npm install. Can you please look into that. The error message is less than helpful in terms of how to resolve it.


Fix 2 typos

Fix a typo in the README example, and another in the import of websocket.

Thanks Solana-man !


[MWPW-158749] Enhance Interact call efficiency and defer non-essentia…

Enhance Interact call efficiency
defer non-essential JavaScript to post LCP
Introduce a tag personalization-v2, that currently enables this feature
Move some common methods to mertech/helpers

Resolves: MWPW-158749
Test URLs:
Before: https://main--milo--adobecom.aem.page/?martech=on
After: https://mwpw-158749-optimise-interact-call--milo--swamu.aem.page/?martech=on

Is this really running on https://mwpw-158749-optimise-interact-call--milo--swamu.aem.page/?martech=on ?
Do you have a few pages where we expect the gains?

Is this really running on https://mwpw-158749-optimise-interact-call--milo--swamu.aem.page/?martech=on ? Do you have a few pages where we expect the gains?

@mokimo you can try https://main--cc--adobecom.hlx.live/products/photoshop?milolibs=mwpw-158749-optimise-interact-call--milo--swamu -- override utils.js andchange line 1064 to return true (inside the enablePersonalizationV2 function).

As a preliminary result using the .live photoshop page
Before

**trace:**
[Trace-20241126T234951.json.zip](https://github.com/user-attachments/files/17924206/Trace-20241126T234951.json.zip)
After:

**trace:**
[Trace-20241126T234740.json.zip](https://github.com/user-attachments/files/17924186/Trace-20241126T234740.json.zip)


CA-248243 plus a c# modification

The python example scripts were removed from all xapi build output; since the sdk still need to ship them, it was deemed better to move them to this repo.
The C# change is a line that has now moved to the build configuration since it is closely associated with the snk key.

Note to the reviewer: please review each commit separately; the one with hash 633edb4 can be skipped because it's a simple copy-paste of the files from xapi,


How to re-enter credentials?

When I use git-tfs used the first time I got the login popup for VSTS online.
Now my password is changed. How can I re-enter my credentials?

That should be stored in the windows credential manager.
See:
https://support.microsoft.com/en-ca/help/4026814/windows-accessing-credential-manager

Thanks, now I am not sure if the problem exists in the credentials.
I get the error: TFS repository can not be root and must start with "$/".
You may be able to resolve this problem.
I am sure it worked a few weeks ago. The command I am using is:
git tfs clone https://example.visualstudio.com $/EXAMPLE/trunk --branches=none
Any ideas?

I use 'cmd'. It seems that copy/paste of a command with '$' gives the problem.
When I type in the $, it all works fine! Thanks


Verifier CLI: configure provider from environment variables

Currently, the verifier CLI takes the provider configuration from CLI switches:
Provider options:
  -h, --hostname <hostname>
          Provider hostname (defaults to localhost)
  -p, --port <port>
          Provider port (defaults to protocol default 80/443)
      --transport <transport>
          Provider protocol transport to use (http, https, grpc, etc.) [default: http]
      --transports <transports>
          Allows multiple protocol transports to be configured (http, https, grpc, etc.) with their associated port numbers separated by a colon. For example, use --transports http:8080 grpc:5555 to configure both.
  -n, --provider-name <provider-name>
          Provider name (defaults to provider)
      --base-path <base-path>
          Base path to add to all requests
      --request-timeout <request-timeout>
          Sets the HTTP request timeout in milliseconds for requests to the target API and for state change requests.
  -H, --header <custom-header>
          Add a custom header to be included in the calls to the provider. Values must be in the form KEY=VALUE, where KEY and VALUE contain ASCII characters (32-127) only. Can be repeated.
      --disable-ssl-verification
          Disables validation of SSL certificates

We have a docker-compose setup with the (instrumented) provider running in a container, and want to use the pactfoundation/pact-ref-verifier docker container to run the verification. Ideally, the pact broker configuration for this setup would come from environment variables, so I can just specify these in the docker-compose service.
E.g. the docker-compose could look like this:
services:
  # An instrumented version of the application API for Pact verification
  pact_provider:
     # service configuration ...
     # service accepts connections on port 5000

  pact_verifier:
    image: pactfoundation/pact-ref-verifier:latest
    depends_on:
      pact_provider:
        condition: service_healthy
    environment:
      # these are hypothetical environment names, to configure how to access the pact broker
      PACT_PROVIDER_HOSTNAME: pact_provider
      PACT_PROVIDER_PORT: 5000
      PACT_PROVIDER_NAME: SomeProviderName
      PACT_PROVIDER_STATE_CHANGE_URL: http://pact_provider:5000/_pact/provider_states
      PACT_PROVIDER_STATE_CHANGE_TEARDOWN: yes

The advantage is that this configuration now lives right next to the provider configuration that dictates these settings.

That is easily do-able
https://github.com/pact-foundation/pact-reference/blob/master/rust/pact_verifier_cli/src/args.rs#L151
here the envs are read in for the token, password or username
would you be up for a PR.

Yup, I can put up a PR for this.

awesome! thank you very much


add std out & err file to replace the temp read log file

add std out & err file to replace the temp read log file

log


Eliminate the natural ventilation tab

It is empty now, and I guess once we have the graphs for it we might revive it, maybe as a "passive comfort strategies" tab

808b118507a595c19fde62a63dc61da64dd0487f


docs: more info about supported base operating systems

This documents the supported base OS versions supported by Machine.  It also removes the changelog from being embedded in the docs.
Added note about changing SSH user for provider image.
Closes #1237
Refs #1204

@nathanleclaire @fredlf PTAL

Couple nits, mostly looks pretty good.

Any plans for the generic driver to also support Arch Linux and its the next most popular base distro? Related to the generic driver is not requiring nopasswd sudo by have an option to be prompted for the password like Ansible has with --ask-become-pass or --ask-sudo-pass.

@nathanleclaire updated.  also note: i took out the "experimental" note on Swarm provisioning.  That's been in for a while so I think we can mark it as stable as the rest :)
PTAL

\o/
LGTM

Oops. Missed this one. Sorry about that.
@ehazlett Mary now owns all OS docs. You'll need to ping her in the future. Technically, this needs two docs maintainers LGTMs before merging.

@fredlf ah thanks fred -- sorry about that.


Update system-date2dwy-method.md

Just found a small mistake while learning As I am a technical consultant of Microsoft Dynamics 365 Business central. Kindly accept that change so that nobody will get confused.

Thanks for your contribution, much appreciated!


[Docs] Documentation Unclear about theme directory for MacOS

Describe the bug
I had a rough time trying to install a custom theme for my MacOS computer.
I put the theme in my custom $XDG_DATA_HOME directory which I mapped to ~/.local/share.
I finally looked at the source code to see which possible folder it was trying to read, when I see that the Environment Variable is ignored on MacOS and it searches only in ~/Library/Application Support according to these docs.
https://docs.rs/dirs/1.0.4/dirs/fn.data_local_dir.html
I am new to MacOS homebrew coming from Linux, so maybe it's a dumb thing to map the variable to that folder like I did.
Personally I am fine with where the program attempts to actually read the file, I am not requesting a change to the code.
But the documentation makes no distinction between the fact that macchina ignores the variable on MacOS, while it will read the variable on at least Linux. I think this is a quick change worth someone's time.
To Reproduce
No Bugs, so omitting extra information onward.

I would appreciate it, if the theme files are also read from ~/.config/macchina/themes on macOS, as they are currently not. On the config file is read from ~/.config/macchina/macchina.toml.
If this is not the intended behaviour, I'll happily open a bug report.
Thanks and take care,
Mick

Macchina is the first terminal application, that requires me to use ~/Library/Application Support/. I (personally) prefer to have my terminal configurations in ~/.config/ (which I e. g. could share easily with my *nix systems).
From a product (ownerish) view, I would keep both (because there may be macOS users who think otherwise) and would just first look in ~/.config/macchina and if there is no config & themes, go to ~/Library/Application Support/, if that wouldn't lead to more complex code base. Only if the later is true, I would omit ~/Library/Application Support/ all together. Because IMHO the majority of terminal based applications do use ~/.config/, so that anyone into terminal will now about that anyway.
My main conundrum was, that the config was read from ~/.config/macchina/macchina.toml, but not the themes.
Thanks for the answer!

Thank you for your feedback :)
macchina starting from v6 will read themes and the main configuration file from both:

~/Library/Application Support/macchina/
~/.config/macchina/

We will update the documentation to reflect the new changes.
Although it's not a major issue for me to maintain the feature to read from both directories, it does contribute to unnecessary complexity. You may open another issue to discuss with other fellow macOS users the future of macchina in this regard.

Thank you very much. I will open an discussion issue later today..

Seems like this has since regressed in the other direction. On macOS, the standard ~/Library/Application Support/macchina/ location is no longer being used for whatever reason...


Grpc WaitForStateChange  API hung and did not timeout

What version of gRPC and what language are you using?
language : C++,  version: 1.30.0
What operating system (Linux, Windows,...) and version?
Linux 4.19.81
What runtime / compiler are you using (e.g. python version or version of gcc)
Python 2.7.16, gcc (Debian 8.3.0-6) 8.3.0
What did you do?
I call WaitForStateChange from 2 threads, I hope it is ok to call from 2 threads as it is thread safe.
//This functionality is introduced to notify the application about the channel state change.
while (1)
{
auto grpc_state = channel->GetState(true);
if(channel.get()->WaitForStateChange(
grpc_state, std::chrono::system_clock::now() + std::chrono::seconds(10)) == true)
{
         //Notify application about the state change
     }
    else
    {
         //
    }
}

//Before sending any requests to server, cleint checke if channel state is "UP", if not up it retries for connectivity for
example : sendRequestServer()
{
WaitForReady(10); -> The code is hung in WaitForReady
}
WaitForReady(int timeout_seconds) {
grpc_connectivity_state state;
while (((state = channel->GetState(true /* try_to_connect */)) !=
GRPC_CHANNEL_READY) && (timeout_seconds!= 0)) {
(void)channel->WaitForStateChange(state, std::chrono::system_clock::now() + std::chrono::seconds(1));
timeout_seconds--;
}
}
Please find below backtrace:
#0  0x00007f4e8d6ea7ef in epoll_wait () from /lib/x86_64-linux-gnu/libc.so.6
#1  0x00007f4e8e3dfd67 in pollable_epoll(pollable*, long) () from /lib/libgrpc.so.10
#2  0x00007f4e8e3e0859 in pollset_work () from /lib/libgrpc.so.10
#3  0x00007f4e8e3e80be in pollset_work () from /lib/libgrpc.so.10
#4  0x00007f4e8e3ef5a0 in grpc_pollset_work(grpc_pollset*, grpc_pollset_worker**, long) () from
#5  0x00007f4e8e44bc90 in cq_next(grpc_completion_queue*, gpr_timespec, void*) () from
#6  0x00007f4e8e44c155 in grpc_completion_queue_next () from /lib/libgrpc.so.10
#7  0x00007f4e8fcfa676 in grpc_impl::CompletionQueue::AsyncNextInternal(void**, bool*, gpr_timespec) () from
#8  0x00007f4e8fce8ff7 in grpc_impl::CompletionQueue::Next(void**, bool*) () from
#9  0x00007f4e8fce83b5 in grpc_impl::Channel::WaitForStateChangeImpl(grpc_connectivity_state, gpr_timespec) () from
#10 0x00007f4e8e6551e4 in bool grpc::ChannelInterface::WaitForStateChange<std::chrono::time_point<std::chrono::_V2::system_clock, std::chrono::duration<long, std::ratio<1l,<PHONE_NUMBER>l> > > >(grpc_connectivity_state, std::chrono::time_point<std::chrono::_V2::system_clock, std::chrono::duration<long, std::ratio<1l,<PHONE_NUMBER>l> > >)
What did you expect to see?
WaitForStateChange shoudl have timed out every 1 second  at
What did you see instead?
WaitForStateChange  was hung at epoll_wait ()

Thanks a lot for your response. wait_for_ready information is helpful, can you please poing me to an example how to set this for an RPC. What happens when the request is queued and the channel is not ready? will the application be blocked by till channel is ready?

If you're using the synchronous API, then yes, the application will block until the RPC completes.  If you're using the async API, then the RPC will simply not complete until the channel connects and the RPC is sent.
You don't actually have to do anything special to get the channel to connect.  The channel will connect automatically when you send the first RPC.  If you really want to force it to connect before you send the first RPC, you can call channel->GetState(true) to start it trying to connect, although that's generally not needed unless you're really concerned about connection-establishment time hurting RPC latency.  (Note that the channel's WaitForConnected() method is basically the equivalent of calling GetState(true) to trigger the connection attempt and then looping over WaitForStateChange() until it reports state READY.)
Also, note that any RPCs sent on the channel before it is connected will automatically wait for the channel to become connected, even without wait_for_ready.  The only reason you would need wait_for_ready is if the channel fails to connect and reports TRANSIENT_FAILURE; if that happens, any pending RPCs will fail by default, but you can use wait_for_ready to tell gRPC to keep those RPCs pending until the channel does succeed in connecting (it will automatically retry to connect).

WaitForStateChange is called by 2 threads and the hung issue mentioned above is not reproducible. The issue happened only  once.


feat: issue template

@sajjadmrx you can have a final review.
bug:
desc, os, title, checkbox
feature:
desc, os(optional), title, checkbox


Device limit changed

with the new tailscale pricing you can now have 100 devices on a free account  https://tailscale.com/pricing/

its a small change, i know, i just saw it when i was updating

Thank you! :) I didn't notice it is mentioned in the DOCS file.

And I've forwarded your commit into the official add-on, keeping your name in the commit. https://github.com/hassio-addons/addon-tailscale/pull/192


Version switcher displays Choose version instead of actual version

Related to: #1681, #1500, #1512
When opening any built page on RTD with this theme, the switcher menu shows "Choose version" instead of the selected version. I largely followed the conf.py for this repo, especially after reading #1512.
From what I can see, my tags and settings are fairly standard, the only difference is that I dynamically load
name, author, copyright, release and version from .pyproject.toml and .bump_my_version.toml. But I added print statements and the build log shows these values are initialized correctly (version == release == 0.0.7 on stable, for example).
My versioning scheme follows semver, and works as follows: x.y.z-alphaN, x.y.z-betaN, x.y.z-rcN, x.y.z
conf.py
# Configuration file for the Sphinx documentation builder.
#
# For the full list of built-in configuration values, see the documentation:
# https://www.sphinx-doc.org/en/master/usage/configuration.html
import datetime
import os
import sys
from pathlib import Path

import toml

sys.path.insert(0, os.path.abspath('../'))

# -- Project information -----------------------------------------------------

pyproject_toml_dict: dict = toml.load(
    Path(__file__).parents[1] / "pyproject.toml")
project_dict: dict = pyproject_toml_dict["project"]
project = project_dict["name"]
_authors: list[dict] = project_dict["authors"]
author: str = ", ".join([f"{a['name']} <{a['email']}>" for a in _authors])
copyright: str = f'{datetime.datetime.now().year}, {author}'

bumpversion_toml = toml.load(Path(__file__).parents[1] / ".bumpversion.toml")
release: str = bumpversion_toml["tool"]["bumpversion"]["current_version"] # e.g., this becomes 0.0.7 on stable
version: str = release

print(version)

# -- General configuration ---------------------------------------------------

extensions = [
    'sphinx.ext.autodoc',
    'sphinx.ext.viewcode',
    'sphinx.ext.coverage',
    'sphinx.ext.doctest',
    'sphinx.ext.mathjax',
]

templates_path = ['_templates']
exclude_patterns = ['_build', 'Thumbs.db', '.DS_Store']

# -- Options for HTML output -------------------------------------------------

version_match = os.environ.get("READTHEDOCS_VERSION")
if not version_match or version_match.isdigit() or version_match == "latest":
    # For local development, infer the version to match from the package.
    if '-' in release:
        version_match = "dev"
        json_url = "_static/switcher.json"
    else:
        version_match = release
elif version_match == "stable":
    version_match = release

html_theme = 'pydata_sphinx_theme'
html_static_path = ['_static']
json_url = "https://impart.readthedocs.io/en/latest/_static/switcher.json"

html_theme_options = {
    "switcher": {
        "version_match": version_match,
        "json_url": json_url,
    },
    "footer_start": [
        "version-switcher"
    ],
    "show_version_warning_banner": True,
}
html_show_sourcelink = False
html_title = f"{project} {release}"

And this is my switcher.json, located in docs/_static/:
[
    {
        "name": "dev",
        "version": "dev",
        "url": "https://impart.readthedocs.io/en/latest/"
    },
    {
        "name": "0.0.7 (stable)",
        "version": "0.0.7",
        "url": "https://impart.readthedocs.io/en/stable/",
        "preferred": true
    },
    {
        "name": "0.0.6",
        "version": "0.0.6",
        "url": "https://impart.readthedocs.io/en/0.0.6/"
    },
    {
        "name": "0.0.5",
        "version": "0.0.5",
        "url": "https://impart.readthedocs.io/en/0.0.5/"
    }
]

I see the following when opening the stable page the first time after building:

After reloading the page, or clicking on 0.0.7 (stable), it looks like this (and remains like this for consecutive page visits):

Any help is appreciated!

If I load https://impart.readthedocs.io/en/stable/ and view the browser devtools console, I see
Uncaught (in promise) TypeError: document.querySelector(...) is null
    T pydata-sphinx-theme.js:569
    r mixin.js:11
    937 pydata-sphinx-theme.js:584

I see your site was built with version 0.15.2 of the theme. I think this problem was fixed as a side-effect of #1780
Can you see if it still happens when building the docs with the current version of the theme (0.15.3)?

I've gone ahead and updated to 0.15.3, which fixed the issue! :partying_face:
The switcher now shows the correct version in the preview.
As a side note, the vertical cyan bar to indicate the currently selected version when the switcher is expanded seems to work only sometimes. For me, it will often disappear after reloading the page. But that's another issue.
Anyways, thanks for the update!
Closing as resolved.