Allow re-rendering Markdown preview when updated extra configurations provided by extension

If Markdown extension provides some extra settings to affect the rendering of Markdown, it would want to re-render Markdown preview when changed configuration values. In fact, VS Code built-in configurations (e.g. markdown.preview.breaks) will behave like that. (Tested in 1.33.0-insider)

I have met with such a case while creating a Markdown extension that has configurable rendering, but I could not find a way to re-trigger Markdown rendering manually. markdown.preview.refresh command is meaningless because it just refresh WebView and not triggered a rendering process.
import { ExtensionContext, commands, workspace } from 'vscode'

export const activate = ({ subscriptions }: ExtensionContext) => {
  subscriptions.push(
    workspace.onDidChangeConfiguration(e => {
      if (e.affectsConfiguration('extension.configuration')) {
        // FIXME: `markdown.preview.refresh` won't re-trigger rendering
        commands.executeCommand('markdown.preview.refresh')
      }
    })
  )

  return {
    extendMarkdownIt: md => {
      md.use(() => {
        const conf = workspace.getConfiguration('extension').get('configuration')
        // ... and use configured value in markdown-it plugin ...
      })
    }
  }
}

Thus, user have to change Markdown buffer once for previewing with updated configurations. Probably many users would expect to reflect the changed configuration to preview immediately.

It would be awesome if there was a command (or API for extension) to re-trigger rendering of Markdown.

PRs welcome to make the refresh command re-render

Reproducible for me, can I work on the PR?

The problem happens with cached tokens here
https://github.com/Microsoft/vscode/blob/2f2085f3e1e3c75916ea950d5c8c2e1e4c28b3dd/extensions/markdown-language-features/src/markdownEngine.ts#L121
It makes a decision whether to recalculate tokens based on the document version and markdown extension configurations. But it doesn't consider changing configurations of markdownItPlugins.
There are two possible solutions I see:

Extract configurations of all plugins which extend markdown base plugin and recalculate tokens if something changed there.
Clean tokens cache on refresh command or pass some force flag.

@mjbvz , maybe you can help me with a decision.

IMO I prefer 2, a controllable way in per extension.
Some settings defined in Markdown extension would not require clearing cache. I'm afraid to degraded performance.
In addition, I have an extension that must trigger re-rendering Markdown by VSCode native configuration window.zoomLevel. It does not defined in the extension. (https://github.com/marp-team/marp-vscode/issues/8#issuecomment-474656886)

@vitaliymaz I'm in favor of the second approach as well: making the refresh command flush the cache. That way it's entirely up to the extension to say when the preview is refreshed

@mjbvz, thank you for the feedback. Working on the second approach.

Fixed by #71442

Thanks @mjbvz, and @vitaliymaz's awesome work! I'll start to test in the next insider version. Cheers :beer:


Fixes an issue with RHUI certificate errors during deploy

Which issue this PR addresses:
Fixes: RP and gateway deploys.
What this PR does / why we need it:
There is a known issue in the RHEL 8 VM image that requires this command to be run yum update -y --disablerepo='*' --enablerepo='rhui-microsoft-azure*' as highlighted in the IcM Ticket. This is currently causing deploy failures in eastus2euap, Public INT, and FairFax INT for RP deploys as well as transient-VM-deploys used fro ACI Fallback. The fix for the transient VM deploys is in this related PR.
Test plan for issue:
Tested the fix using the VM deploy ACI fallback mechanism, which was failing for the same reasons, and it fixed the issue.
Successfully tested a deploy to a dedicate RP instance as well.
Is there any documentation that needs to be updated for this PR?
No.

but we should have the same stuff all over the place, this seems to be just a problem with code duplication, this is a recipe for weird bugs to appear here


Upgrade acktest framework version in all ACK controllers

Some off the controllers are still using old version of acktest framework. To make easier to support new sdk versions and add new CRDs we should ideally update all the controllers to use latest acktest version
A list to keep track of the controllers using the latest version of acktest:

[ ] acmpca-controller
[ ] apigatewayv2-controller
[ ] applicationautoscaling-controller
[x] cloudtrail-controller
[ ] cloudwatch-controller
[ ] cloudwatchlogs-controller
[ ] dynamodb-controller
[x] ec2-controller
[ ] ecr-controller
[x] eks-controller
[ ] elasticache-controller
[x] emrcontainers-controller
[ ] iam-controller
[ ] kms-controller
[x] lambda-controller
[ ] memorydb-controller
[ ] mq-controller
[ ] opensearchservice-controller
[x] prometheusservice-controller
[x] rds-controller
[ ] route53-controller
[ ] s3-controller
[ ] sagemaker-controller
[x] sfn-controller
[ ] sns-controller

/cc @aws-controllers-k8s/ack-admin @aws-controllers-k8s/community-maintainers

/lifecycle frozen


Error, check username and password

I'm using the latest version of everything. I can not login, the data is correct. Writes "Error, check username and password". Tried on all servers.There are no errors in the logs

I think problem with two step authorization #504


Auto-format "select all"

Auto-format selects all the text and if the text is longer than the window vertically it also automatically scrolls to the end.  I think the expected behavior is to not select any text and keep the vertical scroll as is.

fixed for next release!


fix(Prefabs): specify correct component for rule

The InteractableFacade component had been deleted from the rule
due to the change in namespace and Unity did not automatically update
the component to the new script namespace. This has now been fixed.

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

npm package (@latest dist-tag)
v1.0.11
GitHub release

Your semantic-release bot :package::rocket:


Updated to 0.11.7, Harvesters broke again

Recorded a short video of the issue, frustrating to say the least. TLDR: Blacklists dont work...
https://youtu.be/KXBzpBzbl9A

Was the harvester in the video using a blacklist set before the update? The old key might have saved incorrect values to it. Try clearing the blacklist (sneak-right-click) and re-creating it. I just tested it and it seems to work.

Making new keys as well seemed to fix it, though the change I made to the config to add in new ores still doesnt work. However that's for a different issue ticket.


3DX - Fix broken text sizes on roll templates.

Fixed a line which broke text sizes after months.

Submission Checklist
Required

[X] The pull request title clearly contains the name of the sheet I am editing.
[X] The pull request title clearly states the type of change I am submitting (New Sheet/New Feature/Bugfix/etc.).
[X] The pull request makes changes to files in only one sub-folder.
[X] The pull request does not contain changes to any json files in the translations folder (translation.json is permitted)

New Sheet Details

The name of this game is: <   >  (i.e. Dungeons & Dragons 5th Edition, The Dresden Files RPG)

The publisher of this game is: <   > (i.e. Wizards of the Coast, Evil Hat)

The name of this game system/family is: <   > (i.e. Dungeons & Dragons, FATE)

[X] I have followed the Character Sheets Standards when building this sheet.

[X] I have authorization from the game's publisher to make this an official sheet on Roll20 with their name attached.
[X] This game is not a traditionally published game, but a copy of the game rules can be purchased/downloaded/found at: < https://www.sistema3dx.com.br  >
[ ] This sheet is for an unofficial fan game, modification to an existing game, or a homebrew system.

Changes / Description

Remove one line from the css reset that suddenly broke the entire roll template font-sizes.

Character Sheet Info Roll20 Internal Use only.


Update CODEOWNERS

Organize rules
Add new rule for SqlToolsService config file for @alanrenmsft @kburtram to ensure we're added to reviews if that's updated.

Pull Request Test Coverage Report for Build 421529584

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.001%) to 43.161%

Totals

Change from base Build 421419736:
-0.001%

Covered Lines:
24109

Relevant Lines:
51208

💛  - Coveralls


[AWSErrorMarshaller] Format AWSErrors without using newlines

When AWSErrors are logged, newlines are included in the output. This makes it awkward when logging to a file, since not all logging statements produce a single line. This simple change formats how error messages are reported to make them friendlier to systems that log those to a file.

Looks good, will merge it soon.

Ping.

Thank you!


Tracking issue: benchmarks for RPO

Goal(s)

We want to get good benchmarks of the new Rescue Prime Optimized hash function.

Details
Must have

[ ]

Nice to have

[ ]

Working group:
@Dominik1999, @Al-Kindi-0

Workflow

Discussion should happen here or in the related sub-issues.
PRs should only be merged by the coordinator, to ensure everyone is able to review.
Aim to complete reviews within 24 hours.
When a related sub-issue is opened:

add it to the list of sub-issues in this tracking issue

When opening a related PR:

request review from everyone in this working group

When a sub-issue is completed:

close the related issue with a comment that links to the PR where the work was completed

Coordinator: @Dominik1999

Workflow
The working group coordinator ensures scope & progress tracking are transparent and accurate. They will:

Merge approved PRs after all working group members have completed their reviews.

add the PR # to the relevant section of the current tracking PR.
close any completed sub-issue(s) with a comment that links to the PR where the work was completed

Monitor workflow items and complete anything that slips through the cracks.
Monitor scope to see if anything is untracked or unclear. Create missing sub-issues or initiate discussion as required.
Monitor progress to see if there's anything which isn't moving forward. Initiate discussion as required.
Identify PRs with especially significant changes and add @grjte and @bobbinth for review.

This should be a part of the crypto repo - so, I'll move it there.


Cleanup of krnl iterate loops

The code used a mixture of constant and non-constant DialectBuilder, and that caused some strange code pattern. Now all of the lambda functions used in the Dialect Builders used constant Dialect Builder parameters.
This PR also change trivial
loops = krnl.defineLoops(1);
krnl.iterateIE(loops, loops, {lb}, {ub}, ...);

into the new pattern
krnl.forLoopIE(lb, ub, 1, /*parallel*/ false, ...);

Sometimes, blocking>1 and parallel can be true, depending on what the code was.
None of the changes here are functional.

Jenkins Linux ppc64le Build #14729 [push] Cleanup of krnl iterate ... started at 08:53

Jenkins Linux amd64 Build #15699 [push] Cleanup of krnl iterate ... started at 07:41

Jenkins Linux s390x Build #15702 [push] Cleanup of krnl iterate ... started at 08:41

Jenkins Linux s390x Build #15702 [push] Cleanup of krnl iterate ... passed after 1 hr 32 min

Jenkins Linux ppc64le Build #14729 [push] Cleanup of krnl iterate ... passed after 2 hr 0 min


.Net Core 2.0.0-preview1-final Nuget cannot be installed on .Net Standard 2.0 class library

I have a class library created with VS 2017 and .Net Core Preview 1 and targeting .Net Standard 2.0 and it has the following NuGets in addition to the standard library
Microsoft.AspNetCore.Antiforgery v1.1.2
Microsoft.AspNetCore.Localization v1.1.2

Both cannot be upgraded to  .Net Core 2.0.0-preview1-final and I receive as VS output
Package Microsoft.AspNetCore.Antiforgery 2.0.0-preview1-final is not compatible with netstandard2.0 (.NETStandard,Version=v2.0). Package Microsoft.AspNetCore.Antiforgery 2.0.0-preview1-final supports: netcoreapp2.0 (.NETCoreApp,Version=v2.0)
The problem is that I have a web app using Microsoft.AspNetCore.Antiforgery 2.0.0-preview1-final
and referencing my library. This results in
TypeLoadException: Could not load type 'Microsoft.AspNetCore.Antiforgery.Internal.CryptographyAlgorithms' from assembly 'Microsoft.AspNetCore.Antiforgery, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=adb9793829ddae60'.
Is this expected and is there a workaround?
To make it work so far I had to copy/paste the raw code from  CryptographyAlgorithms  directly into my class library, but that is obviously not the best thing to do.

This is because in Preview1 of ASP.NET Core 2.0, didn't support .NET Standard because it used API's that weren't available. In Preview2, I believe it will.
/cc @davidfowl @DamianEdwards

@Petermarcu , I think it is still not in preview 2...

In Preview2 ASP.NET moved back to .NET Standard. I'm not sure if there were specific libraries that couldn't. @DamianEdwards do you know?

https://apisof.net/catalog/Microsoft.AspNetCore.Antiforgery.Internal

Antigforgery 2.0.0-preview2-final does not recognize yet Microsoft.AspNetCore.Antiforgery.Internal.CryptographyAlgorithms

I don't know much about those packages. @DamianEdwards do you?

@Ponant do you have a minimal repro that you can post to GitHub?

@davidfowl , I need to strip out my project; give me some time.

@davidfowl , see class1, CryptographyAlgorithms is not recognized.
https://github.com/Ponant/AntiforgeryCryptoInternalCore20Preview2

@Ponant Where's the project that fails? Can you write some instructions on how to reproduce the problem?

If you open the project and goto class1.cs, you should see the CultureCryptoService class that CryptographyAlgorithms "does not exist in the current context". Whereas I installed the preview 2 antiforgery nuget.

@Ponant the project you put on github doesn't have any package references:
https://github.com/Ponant/AntiforgeryCryptoInternalCore20Preview2/blob/100d00de42077180be7a7afc1d1526321ddc0991/AntiforgeryCryptoInternalCore20Preview2/AntiforgeryCryptoInternalCore20Preview2/AntiforgeryCryptoInternalCore20Preview2.csproj
I'm still not clear on what you're expecting. Did you mean to add some package reference?

OH that is strange, probably an upload issue. Here is my csproj on my desktop
<Project Sdk="Microsoft.NET.Sdk">

  <PropertyGroup>
    <TargetFramework>netstandard2.0</TargetFramework>
  </PropertyGroup>

  <ItemGroup>
    <PackageReference Include="Microsoft.AspNetCore.Antiforgery" Version="2.0.0-preview2-final" />
  </ItemGroup>

</Project>

@davidfowl , to answer your question, what I expect is CryptographyAlgorithms.cs to be public.
https://raw.githubusercontent.com/aspnet/Antiforgery/master/src/Microsoft.AspNetCore.Antiforgery/Internal/CryptographyAlgorithms.cs

That type isn't in the preview2 version of the Microsoft.AspNetCore.Antiforgery package. That's why it's not resolving.
It was in 1.1.x -
https://github.com/aspnet/Antiforgery/tree/rel/1.1.0/src/Microsoft.AspNetCore.Antiforgery/Internal
It's missing from preview2 - https://github.com/aspnet/Antiforgery/tree/rel/2.0.0-preview2/src/Microsoft.AspNetCore.Antiforgery/Internal
It's back in dev - https://github.com/aspnet/Antiforgery/tree/f292fa8db5fbf89ab80eaff7f4c8a469fbe6813b/src/Microsoft.AspNetCore.Antiforgery/Internal
These types are what we call "pubternal" they're not real public API, so they can go missing from version to version that's why they are in an "internal" namespace. However, they can be useful if you're on a particular version of the framework.

Are they expected to be in 2.0? thanks

@Ponant you might be better off calling SHA256.Create()


python311Packages.google-resumable-media: 2.6.0 -> 2.7.0

Changelog: https://github.com/googleapis/google-resumable-media-python/blob/v2.7.0/CHANGELOG.md
Description of changes

Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
24.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.

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

32 packages marked as broken and skipped:

python311Packages.vllm
python311Packages.vllm.dist
python312Packages.cloudpathlib
python312Packages.cloudpathlib.dist
python312Packages.django-storages
python312Packages.django-storages.dist
python312Packages.duckdb
python312Packages.duckdb.dist
python312Packages.dvc-gs
python312Packages.dvc-gs.dist
python312Packages.filedepot
python312Packages.filedepot.dist
python312Packages.gcsfs
python312Packages.gcsfs.dist
python312Packages.google-cloud-automl
python312Packages.google-cloud-automl.dist
python312Packages.google-cloud-bigquery
python312Packages.google-cloud-bigquery.dist
python312Packages.google-cloud-storage
python312Packages.google-cloud-storage.dist
python312Packages.google-resumable-media
python312Packages.google-resumable-media.dist
python312Packages.papermill
python312Packages.papermill.dist
python312Packages.pathy
python312Packages.pathy.dist
python312Packages.smart-open
python312Packages.smart-open.dist
python312Packages.sqlglot
python312Packages.sqlglot.dist
python312Packages.weasel
python312Packages.weasel.dist

26 packages failed to build:

python311Packages.dalle-mini
python311Packages.dalle-mini.dist
python311Packages.fastai
python311Packages.fastai.dist
python311Packages.ibis-framework
python311Packages.ibis-framework.dist
python311Packages.k-diffusion
python311Packages.k-diffusion.dist
python311Packages.skrl
python311Packages.skrl.dist
python311Packages.spacy
python311Packages.spacy-loggers
python311Packages.spacy-loggers.dist
python311Packages.spacy-lookups-data
python311Packages.spacy-lookups-data.dist
python311Packages.spacy-transformers
python311Packages.spacy-transformers.dist
python311Packages.spacy.dist
python311Packages.textacy
python311Packages.textacy.dist
python311Packages.textnets
python311Packages.textnets.dist
python311Packages.wandb
python311Packages.wandb.dist
tts
tts.dist

71 packages built:

barman
barman.dist
baserow
baserow.dist
browsr
browsr.dist
dvc-with-remotes
dvc-with-remotes.dist
mycli
mycli.dist
python311Packages.cloudpathlib
python311Packages.cloudpathlib.dist
python311Packages.dbt-bigquery
python311Packages.dbt-bigquery.dist
python311Packages.django-storages
python311Packages.django-storages.dist
python311Packages.duckdb
python311Packages.duckdb-engine
python311Packages.duckdb-engine.dist
python311Packages.duckdb.dist
python311Packages.dvc-gs
python311Packages.dvc-gs.dist
python311Packages.filedepot
python311Packages.filedepot.dist
python311Packages.fuzzytm
python311Packages.fuzzytm.dist
python311Packages.gcsfs
python311Packages.gcsfs.dist
python311Packages.gensim
python311Packages.gensim.dist
python311Packages.google-cloud-automl
python311Packages.google-cloud-automl.dist
python311Packages.google-cloud-bigquery
python311Packages.google-cloud-bigquery.dist
python311Packages.google-cloud-storage
python311Packages.google-cloud-storage.dist
python311Packages.google-resumable-media
python311Packages.google-resumable-media.dist
python311Packages.natasha
python311Packages.natasha.dist
python311Packages.navec
python311Packages.navec.dist
python311Packages.optuna
python311Packages.optuna.dist
python311Packages.papermill
python311Packages.papermill.dist
python311Packages.pathy
python311Packages.pathy.dist
python311Packages.pins
python311Packages.pins.dist
python311Packages.pyannote-audio
python311Packages.pyannote-audio.dist
python311Packages.pyannote-pipeline
python311Packages.pyannote-pipeline.dist
python311Packages.ray
python311Packages.ray.dist
python311Packages.slovnet
python311Packages.slovnet.dist
python311Packages.smart-open
python311Packages.smart-open.dist
python311Packages.sqlglot
python311Packages.sqlglot.dist
python311Packages.vector
python311Packages.vector.dist
python311Packages.weasel
python311Packages.weasel.dist
snakemake
snakemake.dist
tandoor-recipes
wal_e
wal_e.dist

Already on master.


Kafkaconsumer stops consuming after ZK connection lost/timeout

Using banch refactor (#72) and after applying PR @ samuel/go-zookeeper#84 to my code tree, my kafkaconsumer still fails to consume data when an IO timeout occurs on the connection to Zookeeper.
To reproduce you can prevent the consumer from accessing zookeeper for a few seconds:
root@root# iptables -A  OUTPUT -p tcp -m tcp --dport 2181 -j DROP # Add rule to block outgoing traffic to zookeeper
... Wait a few seconds...
root@root# iptables -D  OUTPUT -p tcp -m tcp --dport 2181 -j DROP # Remove rule

Log output (I changed consumerManager.run() so it runs every 10s):
15:19:51 Recv loop terminated: err=read tcp <IP_ADDRESS>:2181: i/o timeout
15:19:51 Send loop terminated: err=<nil>
15:19:51 [instance=f8565f67ec4a] Failed to watch subscription: zk: connection closed. Trying again in 1 second...
15:19:53 Failed to connect to <IP_ADDRESS>:2181: dial tcp <IP_ADDRESS>:2181: i/o timeout
15:19:53 [instance=f8565f67ec4a] Failed to watch subscription: zk: could not connect to a server. Trying again in 1 second...
..
15:20:26 Connected to <IP_ADDRESS>:2181
15:20:26 Authenticated: id=94641789739226871, timeout=4000
15:20:26 [instance=f8565f67ec4a] Currently, 0 instances are registered, to consume 2 partitions in total.
15:20:26 [instance=f8565f67ec4a] This instance is assigned to consume 0 partitions, and is currently consuming 2 partitions.
[Sarama] 2015/10/08 15:20:26 consumer/broker/0 closed dead subscription to rqueue.out.bs_msg_in/0
[Sarama] 2015/10/08 15:20:26 consumer/broker/0 closed dead subscription to rqueue.out.bs_msg_in/1
15:20:26 [instance=f8565f67ec4a partition=rqueue.out.bs_msg_in/0] Offset 579 has been processed. Continuing shutdown...
15:20:26 [instance=f8565f67ec4a partition=rqueue.out.bs_msg_in/1] FAILED to release partition: Cannot release partition: it is not claimed by this instance
15:20:26 [instance=f8565f67ec4a partition=rqueue.out.bs_msg_in/0] FAILED to release partition: Cannot release partition: it is not claimed by this instance
15:20:36 [instance=f8565f67ec4a] Currently, 0 instances are registered, to consume 2 partitions in total.
15:20:36 [instance=f8565f67ec4a] This instance is assigned to consume 0 partitions, and is currently consuming 0 partitions.
15:20:46 [instance=f8565f67ec4a] Currently, 0 instances are registered, to consume 2 partitions in total.
15:20:46 [instance=f8565f67ec4a] This instance is assigned to consume 0 partitions, and is currently consuming 0 partitions.

i have the same problem.

@yejingx Does your commit fix the issue?
@wvanbergen Is his PR going to be merged?

@jee1mr yes, my commit fixed the issue on my side.

@wvanbergen please check @yejingx 's PR to fix the i/o timeout issue.

@yejingx @wvanbergen  yes. That commit fixed for me too. Thanks.

is this bug fixed? Why is there such a problem

still have this problem

Here are our cases.

Case: After Zookeeper i/o timeout, if we get the error: zk: connection closed from consumer.CG.Errors(), and it reconnects to Zookeeper.
Result: Consuming continues.
Case: After Zookeeper i/o timeout, if we get the error: error while consuming ${topic}/${partition}: Cannot release partition: it is not claimed by this instance from consumer.CG.Errors(), and it reconnects to Zookeeper.
Result: Consuming stops.

@wvanbergen Hi. Any thoughts on this?

In our case:
After Zookeeper i/o timeout, we get the error: error while consuming ${topic}/${partition}: Cannot release partition: it is not claimed by this instance from ConsumerGroup.Errors(), and it reconnects to Zookeeper.
Sometimes consuming stops, but it doesn't always happen this way, it may continue to consume.
@wvanbergen Hi. Any thoughts on this?

@XuejiaoZhang I meet the same issue with you,  do you solve it?

any updates for this case? i have same problem here.
my temporary solution is re-run / rebuild the application.

Hello. I am no longer actively working on this library myself, but if somebody has a fix I will gladly merge a PR.


How to test a video using a pretrained model?

Hi,
I want to use your pretrained model to predict my video. How can I make a video-level prediction?
Can you show me a pipeline for the testing?
Thanks

Hey,
I am not sure what do you mean. Can you be more specific? A pre-trained spatial-stream ResNet-101, temporal-stream ResNet-101, TS-LSTM, or Temporal-Inception?

Does your model support the video-level prediction? I mean the input is a video. I want to use your total pretrained model to predict my own video. For example, "th test.lua --input_video test.avi".
Thanks.

We don't have the code for doing that, but you can certainly local both the two-stream ResNet and combine with either TS-LSTM or Temporal-Inception model for inference. Reading a video can be done by FFmpeg.


Fix slicing decorator

Changes
(Note: Take 2 on https://github.com/HazyResearch/snorkel/pull/1381)

Fixes problem where initializing with @slicing_function() would actually initialize a LabelingFunction
Removes @nlp_slicing_function; currently cannot be initialized after separate @nlp_labeling_function (because of parameters collision). This was a nice-to-have — can still achieve the functionality with preprocessors.

Test Plan

passes tox

@henryre can you have another look here? added some tests + made some additional changes to simplify


Quick Suggestions with loading message

Hello,
I am trying to create an intellisense / code completion using CompletionItemProvider and I would like to display the "loading..." message when I press the triggered character which is dot ('.') just like what happened on the GIF below after I type FROM and then ctr+space the loading message is displayed then the completion items afterwards. I added a timeout there that's why it shows the loading message before the completion items and I would like to to the same thing when I pressed dot ('.') after a word.

I think that suggestions have two big modes of working.
One is that they are triggered manually, by the user. So the user presses ctrl+space to see suggestions. In that case, the suggestion widget renders Loading... while the results are computed. This is to give the user assurance that their command has been interpreted and the results are computing.
The automatic triggers, like pressing . are not indicative of a user explicitly asking for suggestions. It is possible that the user keeps on typing after . and does not want to see a Loading... message in their face. So in this case the suggestions render directly when the result came in.


use standard language

I spoke with greg about this.
for example:
"Misaligned loads and stores may not be supported"
shouldn't this be supported optional (assuming the word may implies it may be supported too)?
if we use the language you have already defined, it becomes clearer: m, so, uo, i

"may not" should be avoided since it could be interpreted as a prohibition.  Either "might not be supported" (keeping similar language) or "may be supported" (using RFC 2119 language) or "supported/unsupported optional" (using the proper terms from the profile spec).

Definitely agreed we should favor "might not" over "may not" in general.  In this case, "Misaligned stores are supported optional" seems like the right phrasing.


Made datetime handling in plotting code more general

Improves datetime handling for datetime.datetime and pd.tslib.Timestamp types.

Very happy with this refactor the code is no longer duplicated and now handles datetime types more consistently. Also added a bunch of unit tests just now. Ready to merge when tests pass.

Ok, the tests have now passed. The changes look good and I am very happy to see the code simplified and the addition of new tests.
Merging.

hi small remark. testing with holoviews latest build, I get this error:
# Holoviews
# for more example, see http://holoviews.org/Tutorials/index.html
import holoviews as hv
%load_ext holoviews.ipython
fractal = hv.Image(image)

((fractal * hv.HLine(y=0.16)).hist() + fractal.sample(y=0.16))

C:\WinPython\basedir35\buildQt5\winpython-64bit-3.5.x.2\python-3.5.3.amd64\lib\site-packages\holoviews\core\util.py:26: FutureWarning: pandas.tslib is deprecated and will be removed in a future version.
You can access Timestamp as pandas.Timestamp
  datetime_types = datetime_types + (pd.tslib.Timestamp,)

Thanks for the note, I'll make a PR with the fix now.

further down, i see errors in statsmodels "The pandas.core.datetools module is deprecated and will be removed in a future version. Please use the pandas.tseries module instead.
from pandas.core import datetools"... maybe a full check would be nice

further down, i see errors in statsmodels "The pandas.core.datetools module is deprecated and will be removed in a future version. Please use the pandas.tseries module instead.
from pandas.core import datetools"... maybe a full check would be nice

Could this be triggered by some other dependency? I think it's probably seaborn, because holoviews does not import statsmodels directly.

it's not triggered by holloviews, just meant that there may be a lot more deprecation warnings to chaze per holoviews  with pandas-0.20.0rc1

I see, yes that makes sense, I'll try it out.

oups ! it's a new kind of deprecation for me "FutureWarning". I new "DeprecationWarning" and "UserWarning" up to now.


Running optimized graph with two output nodes on android gives “Session was not created with a graph before Run()” error.

Hi, please would you please be so kind and help me with one issue that prevents me from moving forward. I have graph with two output layers (final_result_orig – which is basically coming form retraining example,final_result_added – my custom layer) and I am unable to strip/optimize_on_inference it in order to run on android device (on pc it runs fine)
When I run:
bazel-bin/tensorflow/python/tools/optimize_for_inference
–input=/tmp/output.pb
–output=/tmp/optimized.pb
–input_names=Mul
–output_names=”final_result_orig,final_result_added”
Then in my android application, I get “Session was not created with a graph before Run()” error, and both final_result_orig and final_result_added are not found.
When I run:
bazel-bin/tensorflow/python/tools/optimize_for_inference
–input=/tmp/output.pb
–output=/tmp/optimized.pb
–input_names=Mul
–output_names=”final_result_orig”
It works fine, final_result_orig is available and works correctly, however final_result_added is obviously not found and not available for my app to use.
And when I run:
bazel-bin/tensorflow/python/tools/optimize_for_inference
–input=/tmp/output.pb
–output=/tmp/optimized.pb
–input_names=Mul
–output_names=”final_result_added”
It does not work as well with “Session was not created with a graph before Run()” error, and both final_result_orig and final_result_added are not found.
I do not understand what I am doing wrong – what could be wrong with “final_result_added”, as it works fine on PC and not android? Thank you very much.
What related GitHub issues or StackOverflow threads have you found by searching the web for your problem?
https://github.com/tensorflow/tensorflow/issues/5553
Environment info
Ubuntu 16.04 + Android 6.0

@bazinac Are you using a recent version of TensorFlow? The loading code has been improved recently to give more descriptive errors when something goes wrong. I'd suggest syncing first and then seeing what it says.

Unfortunatelly, after installing latest Tensorflow, I am strangely unable to even build optimize_for_inference script with
`Traceback (most recent call last):
File "/home/poborak/SW/tensorflow-master/bazel-bin/tensorflow/python/tools/optimize_for_inference.runfiles/org_tensorflow/tensorflow/python/tools/optimize_for_inference.py", line 65, in
from tensorflow.python.framework import dtypes
File "/home/poborak/SW/tensorflow-master/bazel-bin/tensorflow/python/tools/optimize_for_inference.runfiles/org_tensorflow/tensorflow/python/init.py", line 72, in
raise ImportError(msg)
ImportError: Traceback (most recent call last):
File "/home/poborak/SW/tensorflow-master/bazel-bin/tensorflow/python/tools/optimize_for_inference.runfiles/org_tensorflow/tensorflow/python/init.py", line 61, in
from tensorflow.python import pywrap_tensorflow
File "/home/poborak/SW/tensorflow-master/bazel-bin/tensorflow/python/tools/optimize_for_inference.runfiles/org_tensorflow/tensorflow/python/pywrap_tensorflow.py", line 28, in
_pywrap_tensorflow = swig_import_helper()
File "/home/poborak/SW/tensorflow-master/bazel-bin/tensorflow/python/tools/optimize_for_inference.runfiles/org_tensorflow/tensorflow/python/pywrap_tensorflow.py", line 20, in swig_import_helper
import _pywrap_tensorflow
ImportError: No module named _pywrap_tensorflow
Failed to load the native TensorFlow runtime.
`
Note that otherwise tensorflow works properly, (i can even run "import tensorflow as tf" in python shell, everywhere else than in /tensorflow-master dir). I fought this whole evening and now I gave up and will hope that you somehow will be able to help me.  Otherwise, thanks for very good job with this project. It is fantastic that you make tool like tensorflow available to us.

Perhaps #97 or #1013 are relevant here?  What Python version are you using?
My assumption is that you have a different version of tf installed that Python is finding when you import tf elsewhere, so that is probably misleading.

I am using Python 3.5. I have tried to build tf from source, but it has failed to build Python wheel. I therefore opted to install latest (12.1) release using pip.
I then ran ./configure in downloaded copy od latest tensorflow-master and built optimize_for_inference (I am sorry to provide misleading info on my previous comment). Then this abovementioned error arises when I try to run built optimize_for_inference...

It sounds like @andrewharp is right. There are two versions in your system. I'm guessing that the best at this point is to go back and pip uninstall tensorflow, then work through the issues building from source.

I followed your hint and somehow managed to build from source. When I run it know, tensorflow is much more verbose and I can see that -
 E/native: tensorflow_inference_jni.cc:146 Could not create TensorFlow graph: Invalid argument: No OpKernel was registered to support Op 'Pow' with these attrs.
I google out somewhere that this can be surpassed by adding necessary OpKernel to Android operator sets in BUILD file. But I don't understand what OpKernel I should add to Android operator set in case of Op 'Pow' and how do to so. Could you please tell me.
Sorry for being so annoying lately :D

On behalf of our users, thanks for helping us clarify things!
You should be able to add ops from there:
https://www.github.com/tensorflow/tensorflow/blob/master/tensorflow/core/kernels/BUILD#L3378

Could you try that and let us know?

It looks like cwise_op_pow.cc needs to be added to android_extended_ops_group1.
AFAIK there's no reason not to make this change -- we've just been adding ops as needed, so long as they actually compile for Android without adding undue size.
I think it would also be informative to list registered ops in the case one isn't found; is there an easy way to do that?

It's actually already included in the Makefile build, so not including it for Bazel seems to be an oversight. Sending a CL to fix.

Yay after adding "cwise_op_pow.cc", it works. Thank you guys!


Sliders do not respond correctly under DPI-scaled environments

The examples/ui/slider example works fine at 100% DPI, but behaves incorrectly if the scaling is more than 100%. It's likely that the scaling is either not being applied or being double-applied somewhere.

This is still an issue - this is using a MacBook Pro running at the "More Space" setting (but any setting should do, as macOS doesn't let you run unscaled for the Retina Displays):

@philpax Thanks, turns out it was an issue for more stuff too (debugger) so fixed all of it

Can confirm that fixes it, thanks!


Invite Button in Settings

Description

Embed invite form into Settings
Motivation and Context

Invite functionality was previously only accessible during signup
How Has This Been Tested?

manually using many fake email addresses
Screenshots (if appropriate):
Types of changes

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

Checklist:

[x] My code follows the code style of this project (run $ yarn lint).

house cleaning complete


Spanner: Tests for empty array lists and array lists of various types such as BOOL

No system test required ("empty read" tests are marked "unit").


added getLastUsedTransport in Swift_Transport_LoadBalancedTransport

When using LoadBalancedTransport, I need to know (for tracking) which Transport it actually used. So I added this method to retrieve it.

Travis is failing (Build #337) because of a segfault in phpunit, which is unrelated to this commit. Can it be restarted?

Looks good to me. Can you add some tests?

Thank you @poison.


Only update apt-cache when needed

Instead of always updating the apt cache, leverage the 'update_cache'
method during the apt install task.  Prior to this patch, every run of
this role resulted in a "changed" state.  This patch will clean up that
report to only be "changed" when this actually did indeed change (as
updating the apt-cache isn't a meaningful "change").

Was about to submit the same thing - thanks for beating me to it!

Awesome. LGTM!


Update guess_format_id with new formats

We're getting some new formats in dataone, https://gist.github.com/amoeba/d4771fc01d4f8f66c44202856d078e8e

Update guess_format_id's extension<->formatId map
Make sure ipynb -> json is in there as it's non-obvious
Can we add MATLAB version detection to the algo? I already did this for netcdf
What can we do about RAW files? Are there a few common RAW extensions?

May be difficult to find MATLAB version.
Could potentially use foo <- R.matlab::readMat("file_path")
and attr(foo, "header")$version, but on the few tests I've run all return "5" which is not? a MATLAB file type version. Also v7.3 won't open with the above R package (could use a try call and possibly the h5 package to confirm v7.3 files). Other approaches may be more efficient.

@dmullen17 i can look into this if it hasn't been resolved yet!


Install rsyslog configuration file keeping Flocker logs out of /var/log

To review you need to install generated RPM somewhere and run the services. Things you will need to verify:

Flocker logs are not in /var/log/messages. Maybe... ssh into running acceptance test node?
Logs are still available via journald. You can validate this by checking that the remote logs for the CentOS acceptance builders still contains Flocker logs.

Packages for both CentOS and Ubuntu cannot be built due to missing ignores in packaging.py.
See https://github.com/ClusterHQ/flocker/blob/67d9f2aaab0ddcdb3a099d8d24422bfa2e054f5b/admin/packaging.py#L585 for an example of what is needed.

Could not reproduce locally, so did a cargo-cult change. Hopefully this will fix it. Thanks for the exact pointer to what to do!

I installed the packages from:

http://build.clusterhq.com/results/omnibus/release-maintenance/flocker-1.0.1/suppress-log-messages-FLOC-2541/centos-7/

On some centos-7 acceptance test nodes.
Verified that the new config files were present.
Restarted rsyslog
systemctl restart rsyslog

Rotated the logs
logrotate --force /etc/logrotate.d/syslog

Verified that the new log files no longer contain any flocker agent or control service logs
There are some warnings however:
Jun 30 17:26:11 acceptance-test-richardw-0 rsyslogd-2307: warning: ~ action is deprecated, consider using the 'stop' statement instead [try http://www.rsyslog.com/e/2307 ]
Jun 30 17:26:11 acceptance-test-richardw-0 rsyslogd-2307: warning: ~ action is deprecated, consider using the 'stop' statement instead [try http://www.rsyslog.com/e/2307 ]
Jun 30 17:26:11 acceptance-test-richardw-0 rsyslogd-2307: warning: ~ action is deprecated, consider using the 'stop' statement instead [try http://www.rsyslog.com/e/2307 ]
Jun 30 17:26:11 acceptance-test-richardw-0 rsyslogd-2307: warning: ~ action is deprecated, consider using the 'stop' statement instead [try http://www.rsyslog.com/e/2307 ]

I think we either need to figure out how to trigger a reconfiguration of the rsyslog daemon, or document that rsyslog needs restarting / reconfiguring after installing flocker.
Create a followup issue for that if you think it can be postponed for a later release.


chore: remove oidc warning about double parsing

using urlencoded on / parse the params even for /oauth oidc's routes which uses its own parser.

TODO make some tests on admin routes


Need to be able to specify checks required for merge.

It would be useful if we could specify the minimum required checks before a merge happens.
We use Jenkins for unit test and our platform team have been experimenting with snyk.
The problem is if snyk responds first then a build can be merged that has failing tests.
We either need to be able to do this in the UI or in the config file.
All I really need to to be able to see that the unit tests have passed.
Snyk should also be able to fail the build.

Closing this as we don't expect to implement anything in Dependabot outside the GitHub notion of required checks.

Closing this as we don't expect to implement anything in Dependabot outside the GitHub notion of required checks.

@infin8x Was there an existing issue that tracked this? I can only see https://github.com/dependabot/dependabot-core/issues/2661 which was opened after you closed this issue here.


Read 2 phixs files for hybrid atomic data simulations

Added switch to allow choice of reading one or two phixsdata files depending on the atomic data configuration being used

Should I go ahead and squash and merge this pull request?

Should I go ahead and squash and merge this pull request?

The last thing is that fstream header, then go ahead and merge.


preparation of "ckpts/sparsebev_resnet.pth"

Thank you for sharing. I noticed that the pretrained weight for ResNet50 is "ckpts/sparsebev_resnet.pth." How can I prepare this weight? What is the difference between this and the default "ckpts/resnet50-19c8e357.pth" used in MapTRv2?

Hi, first of all, thank you for taking an interest in my code. The weight is a checkpoint used in the SparseBEV code, and as far as I know, it was trained using nuimage. You can find more detailed information on the GitHub(https://github.com/MCG-NJU/SparseBEV). The method to create this checkpoint was simply to modify the SparseBEV's resnet50 weight to have the same format as "ckpts/resnet50-19c8e357.pth" used in MapTRv2.

Hi, first of all, thank you for taking an interest in my code. The weight is a checkpoint used in the SparseBEV code, and as far as I know, it was trained using nuimage. You can find more detailed information on the GitHub(https://github.com/MCG-NJU/SparseBEV). The method to create this checkpoint was simply to modify the SparseBEV's resnet50 weight to have the same format as "ckpts/resnet50-19c8e357.pth" used in MapTRv2.

ok, thank u!


fix for LangChainDeprecationWarning

Summary of the changes / Why this is an improvement
In the original code you get the following warning:
LangChainDeprecationWarning: The class langchain_community.embeddings.openai.OpenAIEmbeddings was deprecated in langchain-community 0.1.0 and will be removed in 0.2.0.
An updated version of the class exists in the langchain-openai package and should be used instead.
To use it run pip install -U langchain-openai and import as from langchain_openai import OpenAIEmbeddings.
Checklist

[ ] Link to issue this PR refers to (if applicable):

To use it run pip install -U langchain-openai...

Should already be part of the requirements.txt:
https://github.com/crate/cratedb-examples/blob/f8637a0134cb7188c89521f54e97869fc1066bbf/topic/machine-learning/llm-langchain/requirements.txt#L8


[Flow] Allow number as route param

Current Behavior
Flow complains about setting params other that string type.
Failing use cases:
// at call site
navigation.navigate('UserProfile', { userId: 1 })

// Navigator config
{
  initialRouteName: 'UserProfile',
  initiarRouteParams: {
    userId: 1,
  },
}

and probably others.
When reading those params, it is often necessary to convert params back to appropriate type, e.g string to number.
Expected Behavior
Being able to pass number, or any other serialisable data type as route param.
Your Environment

software
version

react-navigation
1.0.0-beta.11

react-native
*

node
*

npm or yarn
*

Notes
I am not sure, whether params of type other that string type will work, or break anything. Maybe it feasible to adjust type definitions.

I think this is fixed via #1574

I think this is closed via #1574


Add bash script update_script.sh

Implemented a bash script update_script.sh that creates files on nearby sftp servers (server 1 sends log file to server 2 and 3, ...) with a record of the date, time and name of the server that created it. Use apt-get install sshpass to install sshpass package on the system that will run the script.

Updated the bash script to be more optimized and without the need for installing the sshpass package.


Parallelize contig assembly on multiple samples

(Not urgent, but an easy optimization)
Right now this creates a single celery task which runs the assemblies in series:
https://github.com/churchlab/millstone/blob/d75ca087fa8cf51c34622d1648961f89ede9c36b/genome_designer/genome_finish/assembly.py#L73
Should be a matter of just kicking off a separate celery task for each.
Is there any concurrency issue that might arise here?

Originally I implemented it so that it kicked off a separate celery task for each, but there were concurrency issues.  I believe the issue was two threads could make the same variant at the same time.  I tried using existing atomicity features of the django database wrapper, but I couldn't get it to work.  I'll try again, because this would obviously speed up SV discovery quite a bit.


STOP minification

Theres really no point to doing it, and the size of the project really isn't a big deal at all. As none of this code gets transferred to the client, theres no bandwidth issue.

Bump.

I think this can be closed?

Its still being minified

I must be missing it - where is it minified?

http://medoo.in/download

No links to minification on the main page, or in source control. Closing this issue


Add e2e tests/#249

Description
This PR fixes #
Notes for Reviewers
Signed commits

[ ] Yes, I signed my commits.

some lint issues coming up.


Screens Business Logic implementation

[ ] Splash Screen
[ ] #401
[ ] Year Screen
[ ] #403
[ ] Home Screen Tab
[ ] #405
[ ] Settings Screen
[ ] License Screen
[x] #394
[ ] #397
[ ] #399
[ ] Date Picker Screen
[ ] Create Mood Picker Screen

Baaqi Screens p to cubit ki zarort ni paregi

Resources:

https://bloclibrary.dev/#/
https://www.youtube.com/watch?v=y564ETOCog8
https://bloclibrary.dev/#/fluttertodostutorial


Handle Non-Serializable Exception Payload

Set non-serializable event's payload field to None and wrapped non-serializable throwable as NotSerializableWrapperException when failed to serialize them, and print out a warning message on both the client and server side so that user can follow-up to make the related throwable or payload type serializable, if they needs them to.  This approach avoid the non-serializable payload or throwable from crashing the test run.

@bvenners I have pushed the adjustments, do you mind to review and consider again this PR for merging?
Thanks.

@bvenners I have added the pattern guard according to your comment, do yo mind to review this again?
Thanks!


Checking LSF log if bjobs fails

This PR checks LSF logs if bjobs fails 3 times (by default).
In the current code, the pipeline will fail with the error message that it got an unknown job status.
In this event, this PR will look at LSF logs to check if the pipeline failed or succeeded.
However, if the logs are stored in a slow filesystem, the submission rate can degrade (my experience is that 1 job was submitted every 10-20 seconds, while when not looking at the logs in the filesystem to check the job status, the rate was several jobs per second).
Test-wise, the coverage is close to 100%. CookieCutter.py and OSLayer.py are not supposed to be tested however, they are just an abstraction on getting values from CookieCutter and interacting with the OS so that they can be mocked in the testing code.
Still searching for a way to do this without degrading the submission rate.

Solved the simplest changes now, will continue later

I think now we have two pending items. I will now run some pipelines with the current version of the code. I wonder if we should merge into a branch while we make sure I did not introduce any bugs, and you can also work on this branch to add rule-specific cluster configs.

I think we are good to merge! Will keep using this profile for the next pipelines to spot any issue.

I think we are good to merge! Will keep using this profile for the next pipelines to spot any issue.

Not quite. There is a conflict still. I think this is because the origin of this branch is before a change to lsf-status.py was made and now that file is deleted in this PR. We can't solve this on github, it has to be done locally.

So one way to be able to merge is to add the old lsf-status and lsf-submit files. We can merge and then delete them.


Prototype: Bug 1920699: clusterresourcequota: retry quota object updates on conflict

/hold

the flow I'm concerned about

CRQ has used count of 1
kas-1 receives create
kas-2 receives create
kas-1 updates quota to say 2 instances used
kas-2 has also counted and wants to say 2 instances used
kas-2 conflicts (this should result in a full re-evaluate after the update from 4 has happened)
in this PR, kas-2 forces the update through and says 2 instances used
the API now has incorrect data in place because there are 3 instances used

A full re-evaluation may work, but it would just be hoping it observed the change in 4. I'm not sure whether this does a full re-evaluation or not.

/close


Makes FacebookAuthenticationHandler respect Options.UserInformationEndpoint

Fixes #365

Hi @bchavez, I'm your friendly neighborhood .NET Foundation Pull Request Bot (You can call me DNFBOT). Thanks for your contribution!
You've already signed the contribution license agreement. Thanks!
The agreement was validated by .NET Foundation and real humans are currently evaluating your PR.
TTYL, DNFBOT;

Do you think you could add a test verifying your fix to this class?
https://github.com/aspnet/Security/blob/dev/test/Microsoft.AspNet.Authentication.Test/Facebook/FacebookMiddlewareTests.cs

Sure, no problem, be back in a few. :+1:

@HaoK @PinpointTownes Should be ok now. My new Unit Test is passing.

1f8368e2efeada70c00d24bbb104b1dde4173c44 now voids double URL encoding with QueryHelpers. Thanks @PinpointTownes .

You're welcome, @bchavez! :smile:
On a side note, it seems that Facebook now supports the Authorization header with the Bearer scheme, just like the other OAuth2 providers.I just tested it and it works with the latest graph API.
Sadly, I can't find any reference :cry:
That's really unfortunate, since flowing the access token in the headers is safer (intermediate proxies won't cache it)... but without an official reference, it's hard to know if it will be supported in the long term.

@PinpointTownes Sounds great, hoping they publish a reference soon rather than passing access_token around the URL string.
Slightly off topic, but I tried to run the unit test again just to make sure everything was okay. But ran into a problem. Everything was working prior to opening the project today. After opening this project in VS2015 RTM, waited for everything to load, then saw the "package manager" console go crazy like it was doing some updates to dnx automatically. I'd prefer VS not do this without permission because I suspect the recent auto-update actions by VS broke my unit testing environment with this project.
------ Discover test started ------
------ Test started: Project: Microsoft.AspNet.Authentication.Test ------
Starting  Microsoft.Framework.TestHost [C:\Users\Profile\.dnx\runtimes\dnx-clr-win-x<IP_ADDRESS>-beta5\bin\dnx.exe --appbase "C:\Code\Projects\Public\Security\test\Microsoft.AspNet.Authentication.Test" Microsoft.Framework.ApplicationHost --port 47363 Microsoft.Framework.TestHost --port 47424]
System.MissingMethodException: Method not found: 'Boolean Microsoft.Framework.Runtime.Project.TryGetProject(System.String, Microsoft.Framework.Runtime.Project ByRef, System.Collections.Generic.ICollection`1<Microsoft.Framework.Runtime.DiagnosticMessage>)'.
   at Microsoft.Framework.TestHost.Program.<>c__DisplayClass2_0.<<Main>b__0>d.MoveNext()
   at System.Runtime.CompilerServices.AsyncTaskMethodBuilder`1.Start[TStateMachine](TStateMachine& stateMachine)
   at Microsoft.Framework.TestHost.Program.<>c__DisplayClass2_0.<Main>b__0()
   at Microsoft.Framework.Runtime.Common.CommandLine.CommandLineApplication.<>c__DisplayClass68_0.<OnExecute>b__0()
   at Microsoft.Framework.Runtime.Common.CommandLine.CommandLineApplication.Execute(String[] args)
   at Microsoft.Framework.TestHost.Program.Main(String[] args)
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
   at Microsoft.Framework.Runtime.Common.EntryPointExecutor.Execute(Assembly assembly, String[] args, IServiceProvider serviceProvider)
   at Microsoft.Framework.ApplicationHost.Program.ExecuteMain(DefaultHost host, String applicationName, String[] args)
   at Microsoft.Framework.ApplicationHost.Program.Main(String[] args)
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
   at Microsoft.Framework.Runtime.Common.EntryPointExecutor.Execute(Assembly assembly, String[] args, IServiceProvider serviceProvider)
   at dnx.host.Bootstrapper.RunAsync(List`1 args, IRuntimeEnvironment env)
   at dnx.host.RuntimeBootstrapper.ExecuteAsync(String[] args)
   at dnx.host.RuntimeBootstrapper.Execute(String[] args)

Unable to start Microsoft.Framework.TestHost
========== Discover test finished: 0 found (0:00:03.4269352) ==========

:(

I guess that's due to the fact you're using a beta5 DNX runtime while contributing on beta7 packages :smile:
You can upgrade to a recent DNX version but sadly, there's currently a bug that prevents the latest one from working correctly. You can either wait for the next nightly build or use the volatile packages:
SET DNX_FEED=https://www.myget.org/F/aspnetvolatile/api/v2
dnvm upgrade -r clr

I see. Thanks @PinpointTownes. My pull-request for Facebook here will eventually propagate to ASP.NET 4 too right? Ultimately, v4 is what I'm looking to fix at the moment. Thanks for all your help. :+1:

If by "ASP.NET 4" you mean Katana (aka Microsoft.Owin), nope, changes introduced in ASP.NET 5 are never backported. Actually, Katana is not under development anymore :smile:

Oh noes, so is no way to fix this Nuget package?
https://www.nuget.org/packages/Microsoft.Owin.Security.Facebook/
:( :( :(

Absolutely no way, I'm afraid. I guess the only reason the ASP.NET team would update a Katana package would be to fix a major security flaw.

:shipit:

Merged.

Looks like the Authorization header support is not new (https://jira.spring.io/browse/SOCIAL-96), but the official page announcing it has since disappeared: http://developers.facebook.com/docs/authentication/oauth2_updates
:facepunch:


doc: Cleanup readme

Codecov Report

Merging #160 into master will increase coverage by 0.32%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #160      +/-   ##
==========================================
+ Coverage   82.38%   82.70%   +0.32%
==========================================
  Files          12       12
  Lines        1226     1226
==========================================
+ Hits         1010     1014       +4
+ Misses        181      178       -3
+ Partials       35       34       -1

Impacted Files
Coverage Δ

pkg/threescale/authorizer/authorizer.go
73.52% <0.00%> (+2.35%)
:arrow_up:

Continue to review full report at Codecov.

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


Refactor cors handling into dedicated middleware

Changes proposed in this PR
Creates a middleware that handles CORS requests and preflights.
Adds unit tests.
Why are we making these changes?
Consolidate CORS handling into one middleware.

It should be but a previous merge related to service description constraints broke the internal build.  We should probably fix that before we merge anything else.


[uwsgi] [Failed Installation] [python:3.4-alpine] - Unable to pip install uwsgi  in python:3.4-alpine

I am trying to install uwsgi along with other pypy packages using a requirements.txt file. I am able to install everything except uwsgi. Its giving me the following error:
# requirements.txt
Flask == 0.12
uwsgi == 2.0.14
requests == 2.13.0
redis == 2.10.5

#Dockerfile
FROM python:3.4-alpine

RUN apk add --update --no-cache \
		pcre-dev
# 		build-base \
#		python3-dev

RUN addgroup -S uwsgi && adduser -S -g uwsgi uwsgi

WORKDIR /ci_testing_python

COPY requirements.txt /ci_testing_python/
RUN pip install -r requirements.txt
# RUN pip install --verbose uwsgi

COPY ci_testing_python /ci_testing_python/ci_testing_python/
COPY setup.py /ci_testing_python/
COPY setup.cfg /ci_testing_python/

COPY tests /ci_testing_python/tests/

RUN pip install --no-deps .

EXPOSE 5000 9090 9191
USER uwsgi

CMD ["./docker-entrypoint.sh"]

So now when I try to run the build command, this is the following error that I get.
$ docker build -t myapp:latest .
Sending build context to Docker daemon 92.67 kB
Step 1/15 : FROM python:3.4-alpine
 ---> 9ac5db25a0ca
Step 2/15 : RUN apk add --update --no-cache             pcre-dev
 ---> Using cache
 ---> 97037b3e67f5
Step 3/15 : RUN addgroup -S uwsgi && adduser -S -g uwsgi uwsgi
 ---> Using cache
 ---> e4647269bef5
Step 4/15 : WORKDIR /ci_testing_python
 ---> Using cache
 ---> dbd5cbf04260
Step 5/15 : COPY requirements/requirements-main.txt /ci_testing_python/
 ---> 5b1ff8a4ba15
Removing intermediate container 163a45c9e6f1
Step 6/15 : RUN pip install -r requirements-main.txt
 ---> Running in 78ab52d5ef57
Collecting Flask==0.12 (from -r requirements-main.txt (line 1))
  Downloading Flask-0.12-py2.py3-none-any.whl (82kB)
Collecting uwsgi==2.0.14 (from -r requirements-main.txt (line 2))
  Downloading uwsgi-2.0.14.tar.gz (788kB)
Collecting requests==2.13.0 (from -r requirements-main.txt (line 3))
  Downloading requests-2.13.0-py2.py3-none-any.whl (584kB)
Collecting redis==2.10.5 (from -r requirements-main.txt (line 4))
  Downloading redis-2.10.5-py2.py3-none-any.whl (60kB)
Collecting Jinja2>=2.4 (from Flask==0.12->-r requirements-main.txt (line 1))
  Downloading Jinja2-2.9.5-py2.py3-none-any.whl (340kB)
Collecting itsdangerous>=0.21 (from Flask==0.12->-r requirements-main.txt (line 1))
  Downloading itsdangerous-0.24.tar.gz (46kB)
Collecting Werkzeug>=0.7 (from Flask==0.12->-r requirements-main.txt (line 1))
  Downloading Werkzeug-0.12.1-py2.py3-none-any.whl (312kB)
Collecting click>=2.0 (from Flask==0.12->-r requirements-main.txt (line 1))
  Downloading click-6.7-py2.py3-none-any.whl (71kB)
Collecting MarkupSafe>=0.23 (from Jinja2>=2.4->Flask==0.12->-r requirements-main.txt (line 1))
  Downloading MarkupSafe-1.0.tar.gz
Installing collected packages: MarkupSafe, Jinja2, itsdangerous, Werkzeug, click, Flask, uwsgi, requests, redis
  Running setup.py install for MarkupSafe: started
    Running setup.py install for MarkupSafe: finished with status 'done'
  Running setup.py install for itsdangerous: started
    Running setup.py install for itsdangerous: finished with status 'done'
  Running setup.py install for uwsgi: started
    Running setup.py install for uwsgi: finished with status 'error'
    Complete output from command /usr/local/bin/python3.4 -u -c "import setuptools, tokenize;__file__='/tmp/pip-build-zvqmmzqp/uwsgi/setup.py';f=getattr(tokenize, 'open', open)(__file__);code=f.read().replace('\r\n', '\n');f.close();exec(compile(code, __file__, 'exec'))" install --record /tmp/pip-1l6suehh-record/install-record.txt --single-version-externally-managed --compile:
    running install
    using profile: buildconf/default.ini
    detected include path: ['/usr/include', '/usr/local/include']
    Traceback (most recent call last):
      File "/tmp/pip-build-zvqmmzqp/uwsgi/uwsgiconfig.py", line 734, in __init__
        gcc_version_components = gcc_version.split('.')
    AttributeError: 'NoneType' object has no attribute 'split'

    During handling of the above exception, another exception occurred:

    Traceback (most recent call last):
      File "<string>", line 1, in <module>
      File "/tmp/pip-build-zvqmmzqp/uwsgi/setup.py", line 124, in <module>
        distclass=uWSGIDistribution,
      File "/usr/local/lib/python3.4/distutils/core.py", line 148, in setup
        dist.run_commands()
      File "/usr/local/lib/python3.4/distutils/dist.py", line 955, in run_commands
        self.run_command(cmd)
      File "/usr/local/lib/python3.4/distutils/dist.py", line 974, in run_command
        cmd_obj.run()
      File "/tmp/pip-build-zvqmmzqp/uwsgi/setup.py", line 77, in run
        conf = uc.uConf(get_profile())
      File "/tmp/pip-build-zvqmmzqp/uwsgi/uwsgiconfig.py", line 742, in __init__
        raise Exception("you need a C compiler to build uWSGI")
    Exception: you need a C compiler to build uWSGI

    ----------------------------------------
Command "/usr/local/bin/python3.4 -u -c "import setuptools, tokenize;__file__='/tmp/pip-build-zvqmmzqp/uwsgi/setup.py';f=getattr(tokenize, 'open', open)(__file__);code=f.read().replace('\r\n', '\n');f.close();exec(compile(code, __file__, 'exec'))" install --record /tmp/pip-1l6suehh-record/install-record.txt --single-version-externally-managed --compile" failed with error code 1 in /tmp/pip-build-zvqmmzqp/uwsgi/
The command '/bin/sh -c pip install -r requirements-main.txt' returned a non-zero code: 1

What is wrong here?
I have also looked into unbit/uwsgi/issues/1318 and gliderlabs/docker-alpine/issues/158 but both of them use alpine:3.4 as the base image and hence when trying to apk add  the required packages it throws some error in installing those mentioned packages in mentioned in those issues.

You need to also install the build tools: gcc libc-dev linux-headers. You can add these to your apk add line or if you don't want to persist gcc in the final image you can install and remove in the line they are used:
RUN set -e; \
	apk add --no-cache --virtual .build-deps \
		gcc \
		libc-dev \
		linux-headers \
	; \
	pip install -r requirements-main.txt; \
	apk del .build-deps;

@yosifkit Thanks. I made the above changes to my Dockerfile and it worked successfully. So my fundamental issue is solved.
But, there is a slight other problem that I noticed. I also tested with a different base image, python:3.6-alpine. But in both the variations, 3.4-alpine and 3.6-alpine. The size of the images after being pulled are quite larger than what is mentioned in the docker hub. I pulled it in an aws ec2 instance with ubuntu 16.04 as host OS. What is the reason for this?
Here is the screenshot.

@yosifkit  Any update on this?

The size mentioned on the Docker Hub is the compressed size, so the amount that would be downloaded when pulled, while the images list is the on disk size.  Layers can shared between images (like everything up to line 18) can be shared between all the python alpine versions.  It is not a huge saving here, since ca-certificates is fairly small, but can definitely help on the Debian based images that are from buildpack-deps.

@yosifkit  Thanks. Its clear now.

@yosifkit is there any world where you would share your terminal theme? Here is mine.. It's similar, but not as awesome with the branches


Add a "clear" button / element for search field

Would be especially helpful for clearing tags.
Maybe it would be nice to enhance the search field even further (like adding filters as UI elements etc.), but this would be a separate request.

Thanks for all of the feedback. But just to inform you, I will be fixing bugs, but I will not be adding new features in a while. Probably until next year. There are reasons. I'm not asking you to stop, but just so you know. 😅


feat : version update notification in settings and update button for downloading latest update

Version update notification in settings and update button for downloading latest update of ollama-webui

here we can also have a drop down menu to jump to particular version or nightly version that uses latest github code for latest update by pulling latest github repo code .


Query ArtifactHub to hint the missing repository

For the chart missing the repo, we can query https://artifacthub.io/api/v1/packages/search?offset=0&limit=1&facets=false&ts_query_web=grafana&kind=0&deprecated=false&sort=relevance and then hint the possible repo.
Maybe instead of limit=1 do ~5.
Think of right presentation of it.

newSearchHubCmd actually does query ArtifactHub, but older version of API


Curse API changed

uploading jars via CurseGradle does no longer work, because curse changed their API.
quoting Technostar:
requiredLibrary and optionalLibrary are going to become requiredDependency and optionalDependency

I'm working on a PR for it.

closed by #17

https://plugins.gradle.org/plugin/com.matthewprenger.cursegradle
Version 1.1.0 has the fix.


Show the hospital bed capacity line differently... currently looks just like cases... maybe label it or even consider removing

This will be closed/updated with the move to High-charts #74


Feature Request: NuGet package containing native libraries

Having a package with libraries for every platform would be nice. If referenced as a dependency the .NET runtime will find native libs automatically.
Just need to name them correctly  runtimes/<platform rid>/native/duckdb.<dll/so/dylib>, for example runtimes/win-x86/native/duckdb.dll

I'll close this issue once I publish the packages.

Packages published as https://www.nuget.org/packages/DuckDB.NET.Data.Full/ and https://www.nuget.org/packages/DuckDB.NET.Bindings.Full/


Connection [] not configured.

i am trying to use the GrahamCampbell/Laravel-Dropbox, i have done all the configurations and also set the dropbox token and app in dropbox config file, but when i run Dropbox::createFolder('foo'); just to test if it is working, i get an error message saying "Connection [] not configured" and the default connection is set to main. I am using Laravel 5.

Have you correctly installed the orchestral config component, and published your config, and added your api key?

Ok i think i skipped the Orchestral config component during installation, lemme install it...

Hello Graham,
I was able to get the installation right but i have a problem using the uploadFile(string $path, WriteMode $writeMode, string $data) method, the problem is i don't know the WriteMode, please help me list out the WriteMode. Thanks

Got what i was looking for...Thanks


Fix switching selections between an in-focus schema and another schema in the UI.

This appears to be due to Selecto somehow inhibiting onClick events under certain conditions (including when a textarea is in focus)
The solution was to move the logic currently in top-level onClick into Selecto.onSelect, as this is definitely called.
https://github.com/pdfme/pdfme/assets/7068515/c975887f-9453-4c83-ab0c-35d3cb7af23d

LGTM!

@peteward
On a PullRequest, if you press ".", the VSCode editor of github.dev comes up,
but do you think codesandbox is necessary?
https://github.com/pdfme/pdfme/assets/24843808/b76ffc1b-ba9f-4c6b-b31d-071f941f1881
Also I came up with an idea, but what do you think about putting the code of the playground repository into pdfme?
I thought it could be easily tested in codespaces.

I'm using Webstorm not VSCode. I didn't check the codesandbox links, but I will next time!
I think it would be a good idea to include the playground in pdfme - it makes the project more accessible to other collaborators.


Flaky test: Microsoft.AspNetCore.SignalR.Client.Tests.HubConnectionTests.CannotCallInvokeOnClosedHubConnection

This test is flaky and it's because we don't cancel the _connectionActive CancellationTokenSource until the underlying connection's Closed Task is triggered. This means DisposeAsync can complete but the connection hasn't been fully closed.
Two ways to fix this:

await hubConnection.Closed in the test, this solves the race in the test code but customers may still encounter it.
Remove the race entirely by either triggering the CTS in DisposeAsync or having DisposeAsync await Closed itself.

What can happen here is that even if I await DisposeAsync(), calls to InvokeAsync after that await ends can proceed all the way to the underlying Connection. They'll fail there, but the exception won't be as clear.

You mean https://github.com/aspnet/SignalR/pull/1116 :D

Brennan... are you watching me code?


Feature Request - Is it possible to extend the api  token-target functionality to token-token ?

I was updating my Mount UP module, but I noticed some strange behavior when using your macros here.
It is almost certainly due to the fact that you expect a "UserTarget" element and I instead provide a "Token" for the second argument.
Before getting back to my code I wanted to know if on your side it was complicated to add the following methods to your api for accept the token element or simply accept the string id of the token

tokenAttacher.attachElementToToken(element, otherTokenOrStringId, suppresNotification=false)
tokenAttacher.attachElementsToToken(element_array, otherTokenOrStringId, suppresNotification=false)
tokenAttacher.detachElementFromToken(element, otherTokenOrStringId, suppressNotification=false)
tokenAttacher.detachElementsFromToken(element_array, otherTokenOrStringId, suppressNotification=false)
tokenAttacher.detachAllElementsFromToken(otherTokenOrStringId, suppressNotification=false)
tokenAttacher.getAllAttachedElementsOfToken(otherTokenOrStringId, suppressNotification=false)
tokenAttacher.getAllAttachedElementsByTypeOfToken(otherTokenOrStringId, type, suppressNotification=false)

The api doesn't have any assumptions about anything being a target or not. In fact there's no difference between a target from Array.from(getGame().user.targets) and a token.
In the example macro I just used targets because that's how a user is able to select anything beside his own token. In your scripts you can actually do for example await window['tokenAttacher'].attachElementToToken(riderToken, mountToken, true); no problem.
Also what you're suggesting for the api to accept an id instead of a token is actually how it works. You can either pass a token or it's id.
So if some code of yours doesn't work, show me which part doesn't and I can maybe clear up your misunderstanding.

Ty fort the clarification. I must be some mistake on  my side


[BUG] duplicate download with Real Debrid

Describe the bug
Hello guys, i dont know why but when i download a file with Real-Debrid its download in final is duplicate
Steps to Reproduce
Just change to real debrid in settings
input my link to donwload and when make a download, this happen

Expected behavior
Just change to real debrid in settings
input my link to donwload and when make a download, this happen

Screenshots
No response
Operating System
Windows 10
Hydra Version
2.0.3
Additional Information
No response

Did you download it using only Real Debrid, or did you first download with torrent and then later changed to real debrid?

Same problem!

We identified what is happening. It will be fixed on next release

Sorry, i dont see this notification, but @zamitto i just download to "real debrid".  Ty


fix: move entrypoints to inside /src, fix require paths

Description
Moves the three entrypoint files into /src whilst fixing the require paths inside to match.
Related Issue
#15
Motivation and Context
This will make it a bit easier to find the entrypoint files - instead of cluttering the root.
How Has This Been Tested?
With the dev command only
Types of changes

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

Checklist

[x] My code follows the code style of this project.
[x] My change requires a change to the documentation.
[x] I have updated the documentation accordingly.
[x] I have read the CONTRIBUTING document.
[x] I have added tests to cover my changes.
[x] All new and existing tests passed.

@all-contributors please add @davidzwa for code


[documentation]: Change folder generated with .g.dart

Could be good to add in the documentation, how to change the generated folder with the .g.dart included, this is my build.yaml to generate all .grapqhql.dart AND .g.dart in the generated folder:
targets:
  $default:
    sources:
      - graphql/**
      - $package$
      - lib/**
      - pubspec.*
      - generated/**
    builders:
      source_gen:combining_builder:
        options:
          build_extensions: {
            'lib/features/{{}}.dart': 'lib/generated/features/{{}}.g.dart',
            'lib/core/{{}}.dart': 'lib/generated/core/{{}}.g.dart',
            'lib/generated/{{}}.dart': 'lib/generated/{{}}.g.dart'
          }
      graphql_codegen:
        options:
          outputDirectory: /lib/generated
          clients:
            - graphql
            - graphql_flutter

Now every files generated is in the /lib/generated/ folder. And got a clean lib/features and lib/core with only .graphql files.

A pr is always appreciated. Also, the newest version doesn't generate .g.dart files.

Ok new version rocks ! Well done. No need the .g.dart part to move the files now. Only outputDirectory: /lib/generated is necessary. I think we don't need a change in the documentation for that. I close it.


Crash On iOS 12 and above

Hi @iosdroid, could you please provide me with more details? I am running it on iOS 13.1, Xcode 11.1 and it seems to be running fine without any crashes.

Merging issue with #32 since they seem to be the related.


Data race in veterans page.

When viewing a veteran, we want to mark the cemetery they're buried in as "selected" in the menu bar. As of this writing, the menu bar is filled in asynchronously from a JSON fetch. So is the veteran's information. If the veteran info arrives before the cemetary list, then the menu item is never marked selected. (The selection code can't find the correct menu item, because it doesn't exist yet.)

Fixed in the move to JSP.


chore(bullmq-task-queue): upgrade bullmq to latest

This PR contains some module updates and code changes as follows.
BullMQ changes deprecated the QueueScheduler alltogether for new, simplified APIs. Since this example is a simple queue/worker setup (without Flows), the scheduler setup has been removed.
Major versions:

BullMQ: 1.7 -> 2 (Breaking change, see queue.server.ts fix)
React 17 -> React 18 (Breaking change, see entry.client.tsx fix)

Minor versions:

IORedis
eslint
Typescript

Thanks for working on this! 🙏
Could you please revert all React 18 changes? We're going to update all examples to React 18 once our default templates are updated as well, but we're still figuring some things out there.

Sure, no problem. Makes sense to do the React upgrade on one move.


Allow to copy SSH public/private keys to image

How do i add ssh public/private keys to this image to help with connecting to remote servers? I know there is SSH agent plugin, but that is TOO much work. If i can add SSH keys to this container, will make like SO much easier.
I understand jenkins runs as jenkins user, so when i try to copy anything to /var/jenkins_home it never gets copied.
Please HELP

The SSH Agent plugin is difficult to setup and use? I guess I have been ramping up on Jenkins so hard over the last year that I forget such things but this seems like a dubious assertion to me. That aside, have you tried extending the official Jenkins image and ADDing your id_rsa and id_rsa.pub files to /usr/share/jenkins/ref/.ssh/? You will want to make sure your permissions are correct of course. another issues is you will want to setup your default, system ssh client with a proper non-interactive /usr/share/jenkins/ref/.ssh/config, something like:
Host *
    PasswordAuthentication  no
    StrictHostKeyChecking   no
    UserKnownHostsFile      /dev/null

Also, you will want to make sure your permissions are all in line for ssh to be happy (there are likely config options to override that as well but I do not know these off the top of my head).
The significance of /usr/share/jenkins/ref as per the README.md is that, unless overridden, at startup the jenkins.sh script will copy all content from /usr/share/jenkins/ref to $JENKINS_HOME provided there there is not content of the same name already in place.
Hope this helps.

P.S. The reason you are losing content you COPY to $JENKINS_HOME aka /var/jenkins_home is that it is declared as a VOLUME in the Dockerfile. This is why the /usr/share/jenkins/ref copying facility exists.

Also: don't publish an image with keys in it to Docker Hub if you do choose this route!

Actually i just found out jenkins jobs are run as root user actually, so i just copied the SSH keys under /root/.ssh/* so all is working well now.
So i have no more issues.
P.S. I didn't mean the SSH Agent plugin is hard to use, but i meant its not the best way to approach login to remote servers, because one has to add credentials for all servers to get into, which is one less secured and second too much stuff to manage.
Using the SSH key inside container means i manage just one file. So i do not need the SSH Agent plugin anymore.
Thanks.


[BugFix] SlotDescriptor might be nullptr

Signed-off-by: dorianzheng<EMAIL_ADDRESS>What type of PR is this：

[x] BugFix
[ ] Feature
[ ] Enhancement
[ ] Refactor
[ ] UT
[ ] Doc
[ ] Tool

Which issues of this PR fixes ：

Fixes #
Problem Summary(Required) ：

Checklist:

[ ] I have added test cases for my bug fix or my new feature
[ ] This pr will affect users' behaviors
[ ] This pr needs user documentation (for new or modified features or behaviors)

[ ] I have added documentation for my new feature or new function

run starrocks_admit_test

run starrocks_admit_test

run starrocks_admit_test

run starrocks_admit_test

run starrocks_admit_test

run starrocks_admit_test

run starrocks_admit_test

run pr-title-check

run pr-title-check

@mergify backport 2.5


Update README.md

That you for your contributions and sorry for my late. Papers from NeurIPS'22 have been added to the list :)


Update what-is-cloud-sync.md

Line 82 was referred to Active directory domain services, on checking this wiki, it should be Azure Active Directory Domain Services, Due to Entra we have changed the terminology of Azure ADDS to Microsoft Entra Domain Services.
Hence updated the same and created a PR, please review the same and approve it.
https://supportability.visualstudio.com/AzureAD/_wiki/wikis/AzureAD/236881/Azure-AD-Cloud-Sync?anchor=comparison-between-azure-ad-connect-and-cloud-sync

#sign-off

Merging now.


Auto-refresh queue

I'm often working with a single queue or queues that do not have subscriptions or event hubs (yet or ever) - I know that F5 will trigger a refresh of an queue however for those monitoring mesasge/DLQ numbers (note I am not suggesting wider use as queue monitoring application!) would auto-refresh be an option?

@Asura5 could you walk through how this would work? I imagine it would be some toggling to have a queue refreshed. But how often? Is it something that should be configured? Etc. You see where I'm leaning with this. Thanks.

This is not the first time that I receive this request. We should introduce two options in the application configuration and options dialog:

Enable autorefesh
Refresh interval in seconds
At each refresh, we should basically refresh the entire TreeView making sure to avoid flickering.

That's pretty much what I had in mind. The refresh rates are global to all queues, so:

Add Queue refresh time (seconds) option to OptionForm.
0 in this box disables auto-refresh, and is the default
Non-zero input will enable auto-refresh with the given number of seconds:

GetEntities called asynchronously (I am not sure of exact implementation details - awaitable Task?)

If it's in the UI I think we should provide a slider with reasonable values to avoid hammering SB because somebody wants to refresh every second, but that's my personal opinion.

@tomkerkhove  I couldn't agree more

Refreshing entities to get message counts is a bad idea.
First, we're on the old client and the new client has removed that functionality for a good reason.
Second, message count is not as accurate as you'd think. So pinging it every second for all entities will be just counterproductive.
Is F5 that hard? SBE is for managing entities and messages, not monitoring. If you need monitoring, I'd argue that's a different tool. My 2c 🙂

As suggested in the initial post I was in not suggesting use for monitoring - not nearly in the order of every second - more along the lines of every x minutes. However - thanks for the very clear feedback. I'll implement locally and close the issue.


Add "Performance Mode"

Provide library wide constexpr flags to enable high performance API options.
For example, currently local objects will explicitly release themselves which is required for tests to guarantee their correctness.  It also prevents memory leaks (e.g. if a local object was created in a tight loop).  This should be configurable.

I played with this a little but without more bandwidth I can't make this happen. JNI Bind is working fine as is for our projects so I sadly am going to close this with the hope some future collaborator can help in the future.


TemplateCacheWarmer uses kernel.secret to build a reproducible template map, but the secret is not known at build time

In https://github.com/symfony/ux/pull/1497 the kernel.secret  was introduced as a way to make the template map reproducible.
This is fine, except the kernel.secret value used is a runtime value (meant to be injected via env() or similar) and not necessarily known at build time. The secret being requested started breaking my build (in Docker) suddenly when upgrading to Symfony 7.2.
Since the usage of the secret is just to provide a stable seed, it doesn't need to use the actual secret, it can use the container checksum, just as similar uses in Symfony proper done recently by @nicolas-grekas, one example is https://github.com/symfony/symfony/pull/56831
It would make sense to swap out the use of the secret (runtime variable) with the container checksum (build time variable).
cc @stof

Hm... i don't like using "secrets" for this. We "just" need a reproductible way to hash template names... maybe can we use Twig class names, that are reproductble now i think ?

Agreed, IMO using the secret here was a mistake, doubly so now since 7.2 requires env vars to be populated and this is using it during build time while it's a runtime var, as noted earlier, even ignoring the fact we shouldn't be injecting the secret in non-security related scenarios to reduce chances of exposure.
Container hash seems like it would be the "most correct" solution here, but I'm not sure why we can't inject it, it's probably a chicken or egg situation? In that case, your suggestion with class names should also work. :+1:

Do you have a small reproducer so I can try to understand the problem ? I replaced on my machine and it seems to work well 😞

@smnandre if I replace the hash, I get
/app $ bin/console cache:clear -e prod --no-debug -vvv

 // Clearing the cache for the prod environment with debug false

 // Clearing outdated warmup directory...

 // Warming up cache...

{"@timestamp":"2024-12-16T09:06:58.218358+00:00","@version":1,"host":"b61d644ee550","message":"User Deprecated: Since doctrine/doctrine-bundle 2.13: Enabling the controller resolver automapping feature has been deprecated. Symfony Mapped Route Parameters should be used as replacement.","type":"app","channel":"deprecation","level":"INFO","monolog_level":200,"context":{"exception":{"class":"ErrorException","message":"User Deprecated: Since doctrine/doctrine-bundle 2.13: Enabling the controller resolver automapping feature has been deprecated. Symfony Mapped Route Parameters should be used as replacement.","code":0,"file":"/app/vendor/doctrine/doctrine-bundle/src/DependencyInjection/DoctrineExtension.php:541","trace":["/app/vendor/doctrine/doctrine-bundle/src/DependencyInjection/DoctrineExtension.php:129","/app/vendor/symfony/dependency-injection/Compiler/MergeExtensionConfigurationPass.php:81","/app/vendor/symfony/http-kernel/DependencyInjection/MergeExtensionConfigurationPass.php:40","/app/vendor/symfony/dependency-injection/Compiler/Compiler.php:73","/app/vendor/symfony/dependency-injection/ContainerBuilder.php:814","/app/vendor/symfony/http-kernel/Kernel.php:498","/app/vendor/symfony/http-kernel/Kernel.php:743","/app/vendor/symfony/http-kernel/Kernel.php:120","/app/src/Infrastructure/Symfony/Kernel.php:55","/app/vendor/symfony/http-kernel/Kernel.php:135","/app/vendor/symfony/framework-bundle/Command/CacheClearCommand.php:229","/app/vendor/symfony/framework-bundle/Command/CacheClearCommand.php:141","/app/vendor/symfony/console/Command/Command.php:279","/app/vendor/symfony/console/Application.php:1094","/app/vendor/symfony/framework-bundle/Console/Application.php:123","/app/vendor/symfony/console/Application.php:342","/app/vendor/symfony/framework-bundle/Console/Application.php:77","/app/vendor/symfony/console/Application.php:193","/app/vendor/symfony/runtime/Runner/Symfony/ConsoleApplicationRunner.php:49","/app/vendor/autoload_runtime.php:29","/app/bin/console:15"]}}}
{"@timestamp":"2024-12-16T09:06:58.218358+00:00","@version":1,"host":"b61d644ee550","message":"User Deprecated: Since doctrine/doctrine-bundle 2.13: Enabling the controller resolver automapping feature has been deprecated. Symfony Mapped Route Parameters should be used as replacement.","type":"app","channel":"deprecation","level":"INFO","monolog_level":200,"context":{"exception":{"class":"ErrorException","message":"User Deprecated: Since doctrine/doctrine-bundle 2.13: Enabling the controller resolver automapping feature has been deprecated. Symfony Mapped Route Parameters should be used as replacement.","code":0,"file":"/app/vendor/doctrine/doctrine-bundle/src/DependencyInjection/DoctrineExtension.php:541","trace":["/app/vendor/doctrine/doctrine-bundle/src/DependencyInjection/DoctrineExtension.php:129","/app/vendor/symfony/dependency-injection/Compiler/MergeExtensionConfigurationPass.php:81","/app/vendor/symfony/http-kernel/DependencyInjection/MergeExtensionConfigurationPass.php:40","/app/vendor/symfony/dependency-injection/Compiler/Compiler.php:73","/app/vendor/symfony/dependency-injection/ContainerBuilder.php:814","/app/vendor/symfony/http-kernel/Kernel.php:498","/app/vendor/symfony/http-kernel/Kernel.php:743","/app/vendor/symfony/http-kernel/Kernel.php:120","/app/src/Infrastructure/Symfony/Kernel.php:55","/app/vendor/symfony/http-kernel/Kernel.php:135","/app/vendor/symfony/framework-bundle/Command/CacheClearCommand.php:229","/app/vendor/symfony/framework-bundle/Command/CacheClearCommand.php:141","/app/vendor/symfony/console/Command/Command.php:279","/app/vendor/symfony/console/Application.php:1094","/app/vendor/symfony/framework-bundle/Console/Application.php:123","/app/vendor/symfony/console/Application.php:342","/app/vendor/symfony/framework-bundle/Console/Application.php:77","/app/vendor/symfony/console/Application.php:193","/app/vendor/symfony/runtime/Runner/Symfony/ConsoleApplicationRunner.php:49","/app/vendor/autoload_runtime.php:29","/app/bin/console:15"]}}}

In DefinitionErrorExceptionPass.php line 48:

  [Symfony\Component\DependencyInjection\Exception\RuntimeException]
  You have requested a non-existent parameter "container.build_hash".

Exception trace:
  at /app/vendor/symfony/dependency-injection/Compiler/DefinitionErrorExceptionPass.php:48
 Symfony\Component\DependencyInjection\Compiler\DefinitionErrorExceptionPass->process() at /app/vendor/symfony/dependency-injection/Compiler/Compiler.php:73
 Symfony\Component\DependencyInjection\Compiler\Compiler->compile() at /app/vendor/symfony/dependency-injection/ContainerBuilder.php:814
 Symfony\Component\DependencyInjection\ContainerBuilder->compile() at /app/vendor/symfony/http-kernel/Kernel.php:498
 Symfony\Component\HttpKernel\Kernel->initializeContainer() at /app/vendor/symfony/http-kernel/Kernel.php:743
 Symfony\Component\HttpKernel\Kernel->preBoot() at /app/vendor/symfony/http-kernel/Kernel.php:120
 Symfony\Component\HttpKernel\Kernel->boot() at /app/src/Infrastructure/Symfony/Kernel.php:55
 App\Infrastructure\Symfony\Kernel->boot() at /app/vendor/symfony/http-kernel/Kernel.php:135
 Symfony\Component\HttpKernel\Kernel->reboot() at /app/vendor/symfony/framework-bundle/Command/CacheClearCommand.php:229
 Symfony\Bundle\FrameworkBundle\Command\CacheClearCommand->warmup() at /app/vendor/symfony/framework-bundle/Command/CacheClearCommand.php:141
 Symfony\Bundle\FrameworkBundle\Command\CacheClearCommand->execute() at /app/vendor/symfony/console/Command/Command.php:279
 Symfony\Component\Console\Command\Command->run() at /app/vendor/symfony/console/Application.php:1094
 Symfony\Component\Console\Application->doRunCommand() at /app/vendor/symfony/framework-bundle/Console/Application.php:123
 Symfony\Bundle\FrameworkBundle\Console\Application->doRunCommand() at /app/vendor/symfony/console/Application.php:342
 Symfony\Component\Console\Application->doRun() at /app/vendor/symfony/framework-bundle/Console/Application.php:77
 Symfony\Bundle\FrameworkBundle\Console\Application->doRun() at /app/vendor/symfony/console/Application.php:193
 Symfony\Component\Console\Application->run() at /app/vendor/symfony/runtime/Runner/Symfony/ConsoleApplicationRunner.php:49
 Symfony\Component\Runtime\Runner\Symfony\ConsoleApplicationRunner->run() at /app/vendor/autoload_runtime.php:29
 require_once() at /app/bin/console:15

cache:clear [--no-warmup] [--no-optional-warmers]

Not sure how I'd make this into a small reproducer honestly, if it works for you, I guess the fix is OK and the issue is specific to my app in some unexpected way.

Changing this in LiveComponent dependencyi injection class, everything works locally... (linking the component locally etc)

        $container->register('ux.live_component.twig.cache_warmer', TemplateCacheWarmer::class)
            ->setArguments([
                new Reference('twig.template_iterator'),
                self::TEMPLATES_MAP_FILENAME,
-                '%kernel.secret%',
+                new Parameter('container.build_hash'),
            ])
            ->addTag('kernel.cache_warmer');

But i'd really need more people to test this...

Interesting enough, if I do it like '%container.build_hash%', it doesn't work, but as new Parameter('container.build_hash'), it does work here too.

Time for a PR :)
'%container.build_hash%' is a compile-time reference while new Parameter('container.build_hash') is a runtime one.

@nicolas-grekas @smnandre PR linked.


Adapt org-wide linter

See https://github.com/nspcc-dev/neofs-node/issues/2940. It's gonna be easier here, just some exhaustive to fix.

This will also fix https://github.com/nspcc-dev/neo-go/issues/3417.


ContractId not usable for allowance approval via AccountAllowanceApproveTransaction

Description
Using an AccountAllowanceApproveTransaction() and passing in a ContractId object throws an error.
Forcing conversion via solidity Address  -> AccountId.fromSolidityAddress(_contractId.toSolidityAddress())
(or fromString()/toString())
In the past, I never had to do this, but in recent SDK versions, I have to make this change; unsure which version it changed.
Possibly it was intentional
Steps to reproduce

deploy a contract, _contractId returned as a ContractId object
create a AccountAllowanceApproveTransaction() to grant an approval (NFT or FT) to _contractId
fails on execution
retry with hard translation to AccountId (e.g. AccountId.fromSolidityAddress(_contractId.toSolidityAddress()) )

Additional context
No response
Hedera network
mainnet, testnet
Version
<EMAIL_ADDRESS>Operating system
None

Hello @Burstall and than you for your issue,
I've created this example based on your explanations:

I am creating a fungible token
I am creating a file to store the bytecode
I am deploying a contract
Create AccountAllowanceApproveTransaction

I am getting the error INVALID_ALLOWANCE_SPENDER_ID
When I cast it to the solidity address and then again back to the account Id (as you described), the transaction is successful.
I am going to dig it.
  //Grab your Hedera testnet account ID and private key from your .env file
  const myAccountId = AccountId.fromString(process.env.MY_ACCOUNT_ID);
  const myPrivateKey = PrivateKey.fromString(process.env.MY_PRIVATE_KEY);

  // If we weren't able to grab it, we should throw a new error
  if (myAccountId == null || myPrivateKey == null) {
    throw new Error(
      "Environment variables myAccountId and myPrivateKey must be present"
    );
  }

  // Create Hedera Testnet Client

  //The client has a default max transaction fee of 100,000,000 tinybars (1 hbar) and default max query payment of 100,000,000 tinybars (1 hbar).
  // If you need to change these values, you can use.setMaxDefaultTransactionFee() for a transaction and .setDefaultMaxQueryPayment() for queries.
  // So the max transaction fee is 1 hbar and the max query fee is 1 hbar, but those values can be changed
  const client = Client.forTestnet();

  // The operator is the account that will pay for the transaction query fees in HBAR
  client.setOperator(myAccountId, myPrivateKey);

  //Create Fungible Token
  const tokenCreateTransaction = await new TokenCreateTransaction()
    .setTokenName("Black Sea LimeChain Token")
    .setTokenSymbol("BSL")
    .setTreasuryAccountId(myAccountId)
    .setInitialSupply(10000) // Total supply = 10000 / 10 ^ 2
    .setDecimals(2)
    .setAutoRenewAccountId(myAccountId)
    .freezeWith(client);

  const txResponse = await tokenCreateTransaction.execute(client);
  const tokenCreateReceipt = await txResponse.getReceipt(client);
  console.log("TokenID: ", tokenCreateReceipt.tokenId.toString());
  console.log(
    "Solidity Token Address",
    tokenCreateReceipt.tokenId.toSolidityAddress()
  );
  const tokenId = tokenCreateReceipt.tokenId;

  // Create a file on Hedera and store the bytecode
  const fileCreateTx = new FileCreateTransaction()
    .setKeys([myPrivateKey])
    .freezeWith(client);
  const fileCreateSign = await fileCreateTx.sign(myPrivateKey);
  const fileCreateSubmit = await fileCreateSign.execute(client);
  const fileCreateRx = await fileCreateSubmit.getReceipt(client);
  const bytecodeFileId = fileCreateRx.fileId;
  console.log(`- The bytecode file ID is: ${bytecodeFileId} \n`);

  //Append contents to the file
  const fileAppendTx = new FileAppendTransaction()
    .setFileId(bytecodeFileId)
    .setContents(contract.bytecode)
    .setMaxChunks(10)
    .freezeWith(client);
  const fileAppendSign = await fileAppendTx.sign(myPrivateKey);
  const fileAppendSubmit = await fileAppendSign.execute(client);
  const fileAppendRx = await fileAppendSubmit.getReceipt(client);
  console.log("Status of file append is", fileAppendRx.status.toString());

  // Instantiate the contract instance
  const contractTx = await new ContractCreateTransaction()
    //Set the file ID of the Hedera file storing the bytecode
    .setAdminKey(myPrivateKey)
    .setBytecodeFileId(bytecodeFileId)
    //Set the gas to instantiate the contract
    .setGas(100000)
    //Provide the constructor parameters for the contract
    .setConstructorParameters();

  //Submit the transaction to the Hedera test network
  const contractResponse = await contractTx.execute(client);

  //Get the receipt of the file create transaction
  const contractReceipt = await contractResponse.getReceipt(client);

  //Get the smart contract ID
  const newContractId = contractReceipt.contractId;

  //Log the smart contract ID
  console.log("The smart contract ID is " + newContractId);

  // Give `spender` allowance for fungible token
  const receiverApproveTx =
    new AccountAllowanceApproveTransaction().approveTokenAllowance(
      tokenId,
      myAccountId,
      newContractId
    );

  const approveRx = await receiverApproveTx.execute(client);

  const approveReceipt = await approveRx.getReceipt(client);
  console.log(
    // eslint-disable-next-line @typescript-eslint/restrict-template-expressions
    `Approve spender allowance - status: ${approveReceipt.status}`
  );


feat: certificate revocation

What would you like to be added?
Certificate revocation is a process in which a certificate is deemed invalid before the end of its lifecycle. Here are some reasons:

The private key is compromised.
The certificate authority that issued this certificate was compromised.
User information had to be changed.

Ratify should support certificate revocation to distinguish invalid and untrusted certificates from valid trusted ones.
Anything else you would like to add?
No response
Are you willing to submit PRs to contribute to this feature?

[ ] Yes, I am willing to implement it.

@yizha1 as we discussed offline, this issue is probably just for tracking the notation cert revocation support. Could you update the issue and create new issues if we want to add revocation to Ratify as well?

@binbin-li Sorry for not responding this issue in time. The cert revocation feature is supported by Notation as the verifier. I want to make sure that Ratify can log the correct reason that the signature verification failure is due to certificate revocation, then users can take proper actions based on it.


Redraw prompt indicator after history search

The prompt indicator gets drawn over when searching the history so we
need to restore it when done searching.

Merged into #30