[Supplier] Provide config/spec file when staking Supplier

Objective
Productionize the interface via which supplier will be staking for numerous complex services
Origin Document

Should be somewhat similar to how we enabled specifying requests via a flag in the v1 repo
Related to #138
The following snippet from https://github.com/pokt-network/poktroll/blob/main/x/supplier/client/cli/tx_stake_supplier.go captures it:

func CmdStakeSupplier() *cobra.Command {
	// fromAddress & signature is retrieved via `flags.FlagFrom` in the `clientCtx`
	cmd := &cobra.Command{
		// TODO_HACK: For now we are only specifying the service IDs as a list of of strings separated by commas.
		// This needs to be expand to specify the full SupplierServiceConfig. Furthermore, providing a flag to
		// a file where SupplierServiceConfig specifying full service configurations in the CLI by providing a flag that accepts a JSON string
		Use:   "stake-supplier [amount] [svcId1;url1,svcId2;url2,...,svcIdN;urlN]",
		Short: "Stake a supplier",
		Long: `Stake an supplier with the provided parameters. This is a broadcast operation that
will stake the tokens and associate them with the supplier specified by the 'from' address.
TODO_HACK: Until proper service configuration files are supported, suppliers must specify the services as a single string
of comma-separated values of the form 'service;url' where 'service' is the service ID and 'url' is the service URL.
For example, an application that stakes for 'anvil' could be matched with a supplier staking for 'anvil;http://anvil:8547'.

Goals

Set the foundation for how complex specification & configuration files for supplier staking will be provided
Enable node runners to stake for repeatedly staking for complex services

Deliverables

[ ] A PR that modifies how supplier stake functions by providing a path to a file on disk
[ ] An ability to provide the complete complex configuration via an inline json file
[ ] Cleanup of all downstream code (e.g. message preparation) when staking for the service
[ ] Unit tests & validation for all existing fields & parametesr

Non-goals / Non-deliverables

Adding/removing configurations to the SupplierServiceConfig

General deliverables

[ ] Comments: Add/update TODOs and comments alongside the source code so it is easier to follow.
[ ] Testing: Add new tests (unit and/or E2E) to the test suite.
[ ] Makefile: Add new targets to the Makefile to make the new functionality easier to use.
[ ] Documentation: Update architectural or development READMEs; use mermaid diagrams where appropriate.

Creator: @Olshansk
Co-Owners: @h5law

In #177, we introduced this flag: cmd.Flags().StringVar(&flagSmtStorePath, "smt-store", "smt", "Path to the SMT KV store"). This is an example of something that should be part of the supplier's config file so we don't end up with dozens of flags.
For reference, in v0, we introduced both config.json and chains.json files and belive that something similar would be appropriate in Shannon as well.

cc @red-0ne @h5law for context

@red-0ne As part of #159, please make sure not to forget to add documentation for this!


convert workflows to html for printing

convert workflows to html for printing #74
probably via pandoc

Replaced by #73 - source is html


Update purchase-recommendations.md

Added link to recommendations video and linked trade-in commitment topic to existing article

@bandersmsft
IMPORTANT: When the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team


Release/v.0.1.3

Codecov Report
Attention: 10 lines in your changes are missing coverage. Please review.

Comparison is base (6cff065) 70.03% compared to head (15ac637) 69.90%.

Files
Patch %
Lines

mango/dashboards/file_explorer_app.py
0.00%
10 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##           master      #84      +/-   ##
==========================================
- Coverage   70.03%   69.90%   -0.13%
==========================================
  Files          40       40
  Lines        2259     2263       +4
==========================================
  Hits         1582     1582
- Misses        677      681       +4

Flag
Coverage Δ

clients-tests
?

plots-tests
?

shared-tests
?

unit-tests
69.90% <0.00%> (-0.13%)
:arrow_down:

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

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


🐞 Bugfix for optimum conversion

Describe your changes
Bugfix for optimum conversion:

if we give the model name in hf_config, optimum do not work then.
path joint bug

Checklist before requesting a review

[ ] Add unit tests for this change.
[ ] Make sure all tests can pass.
[ ] Update documents if necessary.
[ ] Format your code by running pre-commit run --all-files
[ ] Is this a user-facing change? If yes, give a description of this change to be included in the release notes.

(Optional) Issue link

The failure will be fixed in #538 which is related with azureml-evaluate-mlflow version.


WEBSITE: Add import aliases for @themes of Docusaurus

For any questions - ping me

@azinit is this issue actual? currently listed files doesn't has eslint-errors

@frkam Yeap, I suppose that we can close this issue as outdated
Thx for your attention!


Update SiteConfig.js

Description of the change:
Reason for the change:
Link to original source:

Closes #

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

These changes do not make sense

Please check out our open issues for changes that we’re looking for in this repo! We’re adding new ones from easy to difficult every day.

@pushkyn - I've really appreciated all of your reviews!


How to achieve 3d heatmap

I saw a picture of the 3d heatmap on the documentation, but I didn't see the code.
Can someone tell me how this works?

That is made from polygons with a z coordinate (elevation). So your coordinates will have 3 entries, [X, Y, Z]
The original source of this came this comment

Where can i find this whole example.

This specific example was made in R here, in this comment.


ci: run tests in parallel

This pull request makes the Ant-based tests run in parallel (multi-thread).
Rough benchmark on my physical machine:

test
current master
this branch

run-xspec-tests-ant.cmd
203 seconds
104

run-e2e-tests.cmd
133
72

Scope of this change
This change pertains only to CI tests. Nothing is changed in XSpec itself.
This change parallelizes only the Ant-based tests. Bats test is not changed.
Thread count
By default, the tests run with one thread per logical processor. To override it and set the total number of threads, specify thread.count Ant property.
Multi-threading may intermingle the Ant log of different tests. To run the tests sequentially and get a straight log, specify thread.count=1.
run-xspec-tests-ant.cmd -Dthread.count=1

Test

8785c77f447a500521a826978bb7898f75156838 verifies that CI detects run-xspec-tests-ant.* failure

Azure
AppVeyor
Travis

afe9f1c809465cc5f041a67d6c565ed9be4413e6 verifies that CI detects run-e2e-tests.cmd failure

Azure
AppVeyor
Travis

Further work
Mention -Dthread.count=1 in Tips.

As the test failures are demonstrated, I'm merging this so that we can review/experiment changes more quickly.

@xspec/xspec

Further work
Mention -Dthread.count=1 in Tips.

Done


Minor cleanup of 1.9 release notes draft.

Make the release notes a bit more readable.
/cc @enisoc @Bradamant3

Closing unmerged bc merge conflicts and time differences. Changes manually made in #521, with apologies for the confusion.

@Bradamant3 no worries. Yeah I couldn't solve the conflicts due to the time differences. Thanks for making the changes in the other PR!


Subscription with header doesn't work with current null-safety

I've migrated to null-safety ( ^5.0.0-nullsafety.5 ) and query and mutate are working just fine. But when I try to call subscribe I'm getting following error message:
Unhandled Exception: OperationException(linkException: UnknownException({type: error, id: 887be54b-4be8-4bf5-8cb2-44795163abb3, payload: {extensions: {path: $, code: start-failed}, message: cannot start as connection_init failed with : x-hasura-admin-secret/x-hasura-access-key required, but not found}}), graphqlErrors: [])
I'm using header as recommended here few days ago
class GraphQLController {
  GraphQLController(this._url, this._authService);
  final String _url;
  final AuthService _authService;
  GraphQLClient? _client;

  GraphQLClient getClient() {
    if (_client == null) _createClient();
    return _client!;
  }

  void _createClient() {
    var httpLink = HttpLink(_url, defaultHeaders: {
      'x-hasura-admin-secret': 'secret',
    });

    final authLink = AuthLink(
      getToken: () async {
        final token = await _authService.getToken();
        return 'Bearer $token';
      },
    );

    final wsLink = WSLink(GRAPHQL_WS_ENDPOINT_URL,
        headers: {'x-hasura-admin-secret': 'secret'});
    final link = Link.split(
        (request) => request.isSubscription, wsLink, authLink.concat(httpLink));

    _client = GraphQLClient(
      cache: GraphQLCache(),
      link: link,
    );
  }
}

class WSLink extends Link {
  /// Creates a new [WebSocketLink] instance with the specified config.
  WSLink(
    this.url, {
    this.config = const SocketClientConfig(),
    this.headers = const {},
  });

  final String url;
  final SocketClientConfig config;

  final Map<String, dynamic> headers;

  // cannot be final because we're changing the instance upon a header change.
  SocketClient? _socketClient;

  @override
  Stream<Response> request(Request request, [forward]) async* {
    if (_socketClient == null) {
      connectOrReconnect();
    }

    yield* _socketClient!.subscribe(request, true);
  }

  /// Connects or reconnects to the server with the specified headers.
  void connectOrReconnect() {
    _socketClient?.dispose();
    _socketClient = SocketClient(
      url,
      config: config,
      connect: (uri, protocols) => IOWebSocketChannel.connect(
        uri.toString(),
        protocols: protocols,
        headers: headers,
      ),
    );
  }

  /// Disposes the underlying socket client explicitly. Only use this, if you want to disconnect from
  /// the current server in favour of another one. If that's the case, create a new [WebSocketLink] instance.
  Future<void> dispose() async {
    await _socketClient?.dispose();
    _socketClient = null;
  }
}

Any idea?  🙇
Edit: Eventually duplicate of #876

I had same problem like yours. You need to change how you pass connect. Pass it like
  void connectOrReconnect() {
    _socketClient?.dispose();
    _socketClient = SocketClient(
      url,
      config: SocketClientConfig(
        //inactivityTimeout: Duration(seconds: 30),
        delayBetweenReconnectionAttempts: Duration(seconds: 1),
        autoReconnect: !url.contains('9002'),
        connect: (url, protocols) => IOWebSocketChannel.connect(
          url,
          protocols: protocols,
          headers: headers,
        ),
      ),
    );
  }

connect is should be a child in config.
Here is my full implementation
import 'dart:async';

import 'package:graphql_flutter/graphql_flutter.dart';
import 'package:web_socket_channel/io.dart';

typedef HeadersHandler = FutureOr<Map<String, dynamic>> Function();

/// https://gist.github.com/micimize/04ee011b445d38ce34193a5225992504
/// https://github.com/zino-app/graphql-flutter/issues/815
/// Custom version of [WebSocketLink] that connects with the given headers
class WSLinkWithHeaders extends Link {
  /// Creates a new [WebSocketLink] instance with the specified config.
  WSLinkWithHeaders(
    this.url, {
    this.headersHandler,
  });

  final String url;
  final HeadersHandler headersHandler;

  // cannot be final because we're changing the instance upon a header change.
  SocketClient _socketClient;

  @override
  Stream<Response> request(Request request, [forward]) async* {
    if (_socketClient == null) {
      final headers = await headersHandler();
      connectOrReconnect(headers);
    }

    yield* _socketClient.subscribe(request, true);
  }

  /// Connects or reconnects to the server with the specified headers.
  void connectOrReconnect(Map<String, dynamic> headers) {
    _socketClient?.dispose();
    _socketClient = SocketClient(
      url,
      config: SocketClientConfig(
        //inactivityTimeout: Duration(seconds: 30),
        delayBetweenReconnectionAttempts: Duration(seconds: 1),
        autoReconnect: !url.contains('9002'),
        connect: (url, protocols) => IOWebSocketChannel.connect(
          url,
          protocols: protocols,
          headers: headers,
        ),
      ),
    );
  }

  /// Disposes the underlying socket client explicitly. Only use this, if you want to disconnect from
  /// the current server in favour of another one. If that's the case, create a new [WebSocketLink] instance.
  Future<void> dispose() async {
    await _socketClient?.dispose();
    _socketClient = null;
  }
}

@kateile Wow awesome, that worked ! Huge Thanks ! 🙇

With lastest version:
class CustomWebSocketLink extends Link {
  /// Creates a new [WebSocketLink] instance with the specified config.
  CustomWebSocketLink(
    this.url, {
    this.getConfig = _defaultGetConfig,
  });
  static SocketClientConfig _defaultGetConfig() => const SocketClientConfig();

  final String url;
  FutureOr<SocketClientConfig> Function() getConfig;

  // cannot be final because we're changing the instance upon a header change.
  SocketClient? _socketClient;

  @override
  Stream<Response> request(Request request, [forward]) async* {
    if (_socketClient == null) {
      await connectOrReconnect();
    }

    yield* _socketClient!.subscribe(request, true);
  }

  /// Connects or reconnects to the server with the specified headers.
  Future<void> connectOrReconnect() async {
    _socketClient?.dispose();
    _socketClient = SocketClient(
      url,
      config: await getConfig(),
    );
  }

  /// Disposes the underlying socket client explicitly. Only use this, if you want to disconnect from
  /// the current server in favour of another one. If that's the case, create a new [WebSocketLink] instance.
  Future<void> dispose() async {
    await _socketClient?.dispose();
    _socketClient = null;
  }
}

final webSocketLink = CustomWebSocketLink('ws://localhost:8080/graphql',
    getConfig: () => SocketClientConfig(initialPayload: {
      'headers': {
        if (userController.isLogined)
          'Authorization': 'Bearer ${userController.accessToken}'
      }
    }),
  );

// You should call connectOrReconnect every time access token is changed
//webSocketLink.connectOrReconnect();


Need synopsis flow

Get sample APR for PPA analysis, no need for VCS sim

Do not merge with main, codebase would likely require a serious design review to succeed, we don't have time.


Modify chart not working when there are non-visible empty rows in datasheet xml

In case pptx-automizer is throwing this error:
Error: Index 0 not found at "c"
   at ModifyXmlHelper.assertNode...

This is because Excel will keep "invisible" empty rows in datasheet xml after you may have deleted rows.
    <row r="2" spans="1:10" x14ac:dyDescent="0.25">
      <c r="A2" t="s">
        <v>
          2
        </v>
      </c>
      <c r="B2" s="1">
        <v>
          0
        </v>
      </c>
    </row>
    <row r="3" spans="1:10" customFormat="1" x14ac:dyDescent="0.25"/>

As automizer will only replicate what is already there, those rows are missing slots for columns at <row r="3" />, which is causing the error.
This issue is related to #4 and will be fixed by implementing a fallback function to ensure existence of a certain node before it is cloned.

The xlsx-datasheet will be sliced now. Existing rows will be cut by max number of rows.


[Bug] Inline Completion causes inability to type "[" on italian keyboard

Reproducible in vscode.dev or in VS Code Desktop?

[X] Not reproducible in vscode.dev or VS Code Desktop

Reproducible in the monaco editor playground?

[ ] Not reproducible in the monaco editor playground

Monaco Editor Playground Link
link
Monaco Editor Playground Code
<Anything that will trigger an inline completion>
Reproduction Steps

Add and enable italian keyboard layout as described in https://support.apple.com/en-mt/guide/mac-help/mchlp1406/mac
Trigger an inline completion (any one will do)
Try to type the character "[", which is mapped to option+[

Actual (Problematic) Behavior
Nothing is typed
Expected Behavior
[ to be typed
Additional Context
Issue on our side: go/colabx-bugs/4335

Any updates?


Support boost in appropriate queries

Description:
The boost is already supported in MatchAll, Term, and Wildcard queries, so based on that similar needs to be done in Range and Bool queries.

I would love to work on this one !

@kevchuang, thanks! 🙌🏻


Fill project id for stackdriver adapter config if it is specified

Add ability to stackdriver adapter to figure out project id if it is not specified.
Also replace out of date attributes in stackdriver test config.

Hi @bianpengyuan. Thanks for your PR.
I'm waiting for a istio member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.
I understand the commands that are listed here.

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/assign @douglas-reid

/ok-to-test

@bianpengyuan you need to accept the CLA.

@bianpengyuan: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-presubmit.sh
dd9c4149e9dcf84d2d97a514a426a182108cdbaa
link
/test istio-presubmit

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

there is something wrong with this PR (755 commits 2100 files changed)
can you fix it or close and redo ?


rancher-metadata runs at high CPU

Steps to Reproduce:
Create a test stack: testservicealiases
Create two services s1 and s2 in this stack.
Now add a service alias s12 and point to both these services s1 and s2.
Create one more service alias sz -> s12
create one more service alias sy -> sz
create one more service alias sx -> sy
Now edit sz and point it to both s12 and sx.
This will cause rancher-metadata to run at very high CPU.
Results:
The service alias loop shouldn't cause high CPU usage.

rancher/metadata:v0.7.0 hogs CPU. docker 1.10.3
all i see in rancher docker is this:
time="2017-03-01T20:35:11Z" level=info msg="Downloaded in 122.144ms"
time="2017-03-01T20:35:11Z" level=info msg="Loading answers"
time="2017-03-01T20:35:12Z" level=info msg="Loaded answers"
time="2017-03-01T20:35:12Z" level=info msg="Applied http://abf.openmandriva.org:8080/v1/configcontent/metadata-answers?version=166776-5cc6db6887beb3525be1bd696730805e"
time="2017-03-01T20:35:12Z" level=info msg="Downloaded in 152.465ms"
time="2017-03-01T20:35:12Z" level=info msg="Loading answers"
time="2017-03-01T20:35:12Z" level=info msg="Loaded answers"
time="2017-03-01T20:35:12Z" level=info msg="Applied http://abf.openmandriva.org:8080/v1/configcontent/metadata-answers?version=166776-5cc6db6887beb3525be1bd696730805e"
@fedya

@tpgxyz looking

This should be significantly better in Rancher 1.5.x w/ metadata >= v0.8.x

Should be fixed in 0.7.2, and above. Please file a new issue if you experience problems with metadata.

Having the same issue on rancher 1.6.2 / Ubuntu 16.04 / docker 17.03.1-ce, build c6d412e

Having the same issue on rancher 1.6.3 / Ubuntu 16.04 / docker 1.13.1   @superseb
I observer high cpu on the rancher service also. iotop on mysql shows high traffic on disk writes causes by rancher

@tuwid @kilmarnock Thanks for reporting, are your issues related to the scenario described on top? If not, please find other issue matching your case or create a new one.

sorry, you are right. my scenario is different.


There is a bug of zhipuai llm while using metagpt on ubuntu22.04 under the environment of WSL2 of windows11 Home edition

Bug description
In the system of ubuntu 22.04 under the environment of WSL2 of windows11 Home edition.
With the test cli $ metagpt "Write a cli snake game", the program reports a bug as shown:
| ERROR    | metagpt.utils.common:log_it:476 - Finished call to 'metagpt.actions.action_node.ActionNode._aask_v1' after 0.615(s), this was the 1st time calling it. exp: Request failed, msg: {"error":{"code":"1003","message":"Authorization Token已过期，请重新生成/获取。"}}, please ref to https://open.bigmodel.cn/dev/api#error-code-v3
But it reports no bug under the environment of installing ubuntu 22.04 directly!
Bug solved method
Installing the linux system directly.
Environment information

LLM type and model name: zhupuai, glm-4
System version: ubuntu 22.04 under WSL2 of windows 11 Home edition
Python version: 3.11.7

Dependent packagess：

packages version: metagpt v0.7-release
installation method: pip3 install -e .

can you re-generate a new api_key and try in the same environment first ?

can you re-generate a new api_key and try in the same environment first ?
I have tried to generate a new api_key, and funded to my account, but it comes the same bug. And I have tried the key with the zhipuai official api call example, and it succeeded.

It's strange, the error-message seems to be zhipu api auth problem. Can you send a wechat id to my email of the count and later take a look on your environment remotely ?

As discussed offline, the problem can't reproduce with same env. It's zhipu api auth problem probably.

updated reason.
zhipu use jwt to encode api-key and has a expired ttl of 3 minutes.
Due to the user's system localtime is not same with the Beijing time, so it cause a token expiration error.
So, check your system time if you meet the same problem.


GEODE-9710: Cleanup PRColocationDUnitTestHelper

For all changes:

[ ] Is there a JIRA ticket associated with this PR? Is it referenced in the commit message?

[ ] Has your PR been rebased against the latest commit within the target branch (typically develop)?

[ ] Is your initial contribution a single, squashed commit?

[ ] Does gradlew build run cleanly?

[ ] Have you written or updated unit tests to verify your changes?

[ ] If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion under ASF 2.0?

Changing the test name as per the new refactoring guidlines.


Generating Documentation Files for GPT on ChatGPT

I would like to generate a PDF or JSON file for the documentation to create a GPT on ChatGPT. I am aware of the htmlbook2pdf crawler, but I am looking for an easier method.
Thank you, @Sammyjo20, for this awesome library!

Hey @dangnhdev I'm a little confused to your request, do you want to convert Saloon's documentation into something that ChatGPT can import?

Hey @dangnhdev I think the closest to "raw" documentation would be what you can find on the Github repository in markdown files https://github.com/Sammyjo20/saloon-docs/


Api will work but cant handle alot of request

Describe the bug
Api will work but cant handle alot of request
Expected behavior
Api will work but cant handle alot of request
Actual behavior
Api will work but cant handle alot of request
Additional context
Api will work but cant handle alot of request

For the API to handle a lot of requests, you'd have to scale it. Is there anything else you wanted to ask/report?

Hello thanks but the api has limit I have at least 8000 users daily u tried
the api but it stopping when I'm adding my big domain with traffic, I think
the api got blocked with hianime.to?
On Tue, Dec 3, 2024, 16:19 Ritesh Ghosh @.***> wrote:

@janglloyd12 https://github.com/janglloyd12
For the API to handle a lot of requests, you'd have to scale it. Is there
anything else you wanted to ask/report?
—
Reply to this email directly, view it on GitHub
https://github.com/ghoshRitesh12/aniwatch-api/issues/78#issuecomment-2514858015,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AS4QDSH2U7JMKK3FSHSAQKL2DXDYDAVCNFSM6AAAAABS55WNYOVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDKMJUHA2TQMBRGU
.
You are receiving this because you were mentioned.Message ID:
@.***>

@janglloyd12
I tried it right now and the api is not blocked by hianime.to. Did your self hosted version get blocked? Have you scaled the API to handle 8k users?

This is my server I'm using gogoanime api before and scrape it and works
properly.. but when I tried your api it cannot handle a lot of request..
maybe you can help me? I will pay for the time if you can fix it? I have a
copy of front end of hianime
On Tue, Dec 3, 2024, 19:34 Ritesh Ghosh @.***> wrote:

@janglloyd12 https://github.com/janglloyd12
I tried it right now and the api is not blocked by hianime.to. Did your
self hosted version get blocked? Have you scaled the API to handle 8k users?
—
Reply to this email directly, view it on GitHub
https://github.com/ghoshRitesh12/aniwatch-api/issues/78#issuecomment-2515309470,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AS4QDSFGANMMQLP4TROAPT32DX2TZAVCNFSM6AAAAABS55WNYOVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDKMJVGMYDSNBXGA
.
You are receiving this because you were mentioned.Message ID:
@.***>

@janglloyd12
Could you share the details with me on discord?
My username: @riteshgsh

I sent u a friend request it's me deku.. pls accept me
On Tue, Dec 3, 2024, 20:19 Ritesh Ghosh @.***> wrote:

@janglloyd12 https://github.com/janglloyd12
Could you share the details with me on discord?
My username: riteshgsh
—
Reply to this email directly, view it on GitHub
https://github.com/ghoshRitesh12/aniwatch-api/issues/78#issuecomment-2515391120,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AS4QDSBOLRGVU4TTFXXY6DD2DX75ZAVCNFSM6AAAAABS55WNYOVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDKMJVGM4TCMJSGA
.
You are receiving this because you were mentioned.Message ID:
@.***>

I sent u a friend request it's me deku.. pls accept me
On Tue, Dec 3, 2024, 20:19 Ritesh Ghosh @.***> wrote:

@janglloyd12 https://github.com/janglloyd12
Could you share the details with me on discord?
My username: riteshgsh
—
Reply to this email directly, view it on GitHub
https://github.com/ghoshRitesh12/aniwatch-api/issues/78#issuecomment-2515391120,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AS4QDSBOLRGVU4TTFXXY6DD2DX75ZAVCNFSM6AAAAABS55WNYOVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDKMJVGM4TCMJSGA
.
You are receiving this because you were mentioned.Message ID:
@.***>

Yeah, I've sent you a reply


Unable to show loader in admin options.

I wanted to show a loader and remove the loader and reload the page once the custom bulk action is complete.
"admin-bro": "^4.0.1",
"admin-bro-expressjs": "^2.1.1",
"admin-bro-mongoose": "^0.5.2",

My Custom bulk action looks like this :-
shortlist: {
actionType: 'bulk',
icon: 'View',
isVisible: true,
component: false,
layout: layout,
showInDrawer: true,
handler: async (request, response, context) => {
const {
records,
resource,
h,
translateMessage
} = context;
          if (!records || !records.length) {
            throw new _notFoundError.default('no records were selected.', 'Action#handler');
          }

          if (request.method === 'get') {
            const recordsInJSON = records.map(record => record.toJSON(context.currentAdmin));
            const recordIds = records.map(record => record.params._id);
            await UserSelectedFeedback.updateMany(
              {_id: { $in: recordIds}},
              { $set: { shortlist: true }},
              {multi: true}
            );
            return {
              records: recordsInJSON,
              redirectUrl: '/admin/resources/user_selected_feedback',
            };
          }

          throw new Error('method should be either "post" or "get"');
        }
      }

OS: [Mac OS]
Browser [chrome]
Version [96.0]

Let me know if there is any way to show the loader while fetching data and to reload the page once we've got the data on the same page.

You'd most likely need to create a custom component or update existing default components to show a loader while the action is in progress


model_dump ignores sub class attributes

Initial Checks

[X] I confirm that I'm using Pydantic V2

Description
model_dump ignores sub class attributes. Consider a minimalistic code below.
Generared Output: {'item': {'key': '1'}}
Expected Output : {'item': {'key': '1', 'model': 'VW'}}

Example Code
from pydantic import BaseModel, version
from sys import version_info

print(version_info)
print(version.version_info())

class ItemBase(BaseModel):
    key : str
class ItemCar(ItemBase):
    model : str

class ItemHolder(BaseModel):
    item : ItemBase

h =     ItemHolder(item = ItemCar(key = "1", model = "VW"))
h.model_dump()

Python, Pydantic & OS Version
sys.version_info(major=3, minor=8, micro=10, releaselevel='final', serial=0)
             pydantic version: 2.4.2
        pydantic-core version: 2.10.1
          pydantic-core build: profile=release pgo=false
                 install path: xxxxx py3.8\Lib\site-packages\pydantic
               python version: 3.8.10 (tags/v3.8.10:3d8993a, May  3 2021, 11:48:03) [MSC v.1928 64 bit (AMD64)]
                     platform: Windows-10-10.0.19045-SP0
             related packages: mypy-1.6.0 typing_extensions-4.8.0

Hi @SergeyHein,
Thanks for reporting this. This is the expected behavior. You can check out this section of the docs to learn more about serialization with duck typing, which is the behavior I think you're looking for.
Here's an example of how you could get the results you're looking for using SerializeAsAny:
from pydantic import BaseModel, SerializeAsAny

class User(BaseModel):
    name: str

class UserLogin(User):
    password: str

class OuterModel(BaseModel):
    as_any: SerializeAsAny[User]
    as_user: User

user = UserLogin(name='pydantic', password='password')

print(OuterModel(as_any=user, as_user=user).model_dump())
"""
{
    'as_any': {'name': 'pydantic', 'password': 'password'},
    'as_user': {'name': 'pydantic'},
}
"""

@SergeyHein,
Sure thing! Glad I could be of help 😄


Implement default trait for TransactionExecutionInfo

Solves #108

Codecov Report

Merging #113 (6d6f978) into main (5c4eed4) will increase coverage by 0.12%.
The diff coverage is 100.00%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@            Coverage Diff             @@
##             main     #113      +/-   ##
==========================================
+ Coverage   63.55%   63.68%   +0.12%
==========================================
  Files          30       30
  Lines        1608     1608
==========================================
+ Hits         1022     1024       +2
+ Misses        586      584       -2

Impacted Files
Coverage Δ

src/business_logic/execution/objects.rs
65.35% <100.00%> (+1.30%)
:arrow_up:

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

Great job removing that empty method. It was useless 👍🏼. Regarding the implementation of the Default trait: that isn't necessary since it is exactly the same return value that the derived one. You can add back the Default derive and remove the method.

Thanks! Done 👍


error running sergate as init.d

when i execute "service sergate start" i get this error:
[root@chicago init.d]# service sergate start
/etc/init.d/sergate: line 12: /lib/lsb/init-functions: No such file or directory
that is fixed with
yum install redhat-lsb
and next error is
[root@chicago init.d]# service sergate start
/etc/init.d/sergate: line 29: log_daemon_msg: command not found
/etc/init.d/sergate: line 30: start-stop-daemon: command not found
/etc/init.d/sergate: line 32: log_end_msg: command not found

Which distribution are you running?

centos.
if i run directly the ./sergate
starts ok.

@BotoX
I also encountered this problem, because before the file a lot of commands only apply to Debian, not applicable to the CentOS6, so there will be the following error.
service sergate start
/etc/init.d/sergate: line 29: log_daemon_msg: command not found
/etc/init.d/sergate: line 30: start-stop-daemon: command not found
/etc/init.d/sergate: line 32: log_end_msg: command not

I tried using CentOS to compile these commands and failed. Can not like this site and then design a script, thank you!

I'm using this on CentOS 6
https://ngadmin.com/tutorials/20160213-botox-centos

@vdar This is just for the client, but the client only need to modify the ./clients/ client.py,
then add su -l $USERNAME -c "/path/to/client.py &> /dev/null &"in /etc/rc.local can be monitored. The most important is the server service“sergate.initd”  can not be used for CentOS


Could not load dynamic library 'cudart64_101.dll'

System information

Windows 10
Laptop: Asus gl553vw
TensorFlow installed with pip
tensorflow-gpu 2.2.0
Python 3.8.3
CUDA version 10.1 (update2)
cudnn-10.1-windows10-x64-v<IP_ADDRESS>
Nvidia GTX 960M

I want to use tensorflow with GPU but I keep receiving this error Could not load dynamic library 'cudart64_101.dll' when I import Tensorflow.
the output of `pip list`:
Package                  Version
------------------------ -----------
absl-py                  0.9.0
astunparse               1.6.3
cachetools               4.1.0
certifi                  2020.6.20
chardet                  3.0.4
gast                     0.3.3
google-auth              1.18.0
google-auth-oauthlib     0.4.1
google-pasta             0.2.0
grpcio                   1.30.0
h5py                     2.10.0
idna                     2.9
Keras-Preprocessing      1.1.2
Markdown                 3.2.2
numpy                    1.19.0
oauthlib                 3.1.0
opt-einsum               3.2.1
pip                      19.2.3
protobuf                 3.12.2
pyasn1                   0.4.8
pyasn1-modules           0.2.8
requests                 2.24.0
requests-oauthlib        1.3.0
rsa                      4.6
scipy                    1.4.1
setuptools               41.2.0
six                      1.15.0
tensorboard              2.2.2
tensorboard-plugin-wit   1.6.0.post3
tensorflow-gpu           2.2.0
tensorflow-gpu-estimator 2.2.0
termcolor                1.1.0
urllib3                  1.25.9
Werkzeug                 1.0.1
wheel                    0.34.2
wrapt                    1.12.1

the output of `pip debug --verbose`:
pip version: pip 19.2.3 from c:\users\ghassen\downloads\lisadetection\lisadetection\venv2\lib\site-packages\pip (python 3.8)
sys.version: 3.8.3 (tags/v3.8.3:6f8c832, May 13 2020, 22:37:02) [MSC v.1924 64 bit (AMD64)]
sys.executable: c:\users\ghassen\downloads\lisadetection\lisadetection\venv2\scripts\python.exe
sys.getdefaultencoding: utf-8
sys.getfilesystemencoding: utf-8
locale.getpreferredencoding: cp1252
sys.platform: win32
sys.implementation:
  name: cpython
Config variable 'Py_DEBUG' is unset, Python ABI tag may be incorrect
Compatible tags: 15
  cp38-cp38-win_amd64
  cp38-none-win_amd64
  py3-none-win_amd64
  cp38-none-any
  cp3-none-any
  py38-none-any
  py3-none-any
  py37-none-any
  py36-none-any
  py35-none-any
  py34-none-any
  py33-none-any
  py32-none-any
  py31-none-any
  py30-none-any

For import tensorflow as tf this is the output:
2020-06-24 14:50:11.230153: W tensorflow/stream_executor/platform/default/dso_loader.cc:55] Could not load dynamic library 'cudart64_101.dll'; dlerror: cudart64_101.dll not found
2020-06-24 14:50:11.236957: I tensorflow/stream_executor/cuda/cudart_stub.cc:29] Ignore above cudart dlerror if you do not have a GPU set up on your machine.

and for `tf.test.is_gpu_available()` the output is:
```
WARNING:tensorflow:From :1: is_gpu_available (from tensorflow.python.framework.test_util) is deprecated and will be removed in a future version.
Instructions for updating:
Use `tf.config.list_physical_devices('GPU')` instead.
2020-06-24 14:51:00.146205: I tensorflow/core/platform/cpu_feature_guard.cc:143] Your CPU supports instructions that this TensorFlow binary was not compiled to use: AVX2
2020-06-24 14:51:00.171655: I tensorflow/compiler/xla/service/service.cc:168] XLA service 0x21868deee70 initialized for platform Host (this does not guarantee that XLA will be used). Devices:
2020-06-24 14:51:00.182635: I tensorflow/compiler/xla/service/service.cc:176]   StreamExecutor device (0): Host, Default Version
2020-06-24 14:51:00.190956: I tensorflow/stream_executor/platform/default/dso_loader.cc:44] Successfully opened dynamic library nvcuda.dll
2020-06-24 14:51:01.031439: I tensorflow/core/common_runtime/gpu/gpu_device.cc:1561] Found device 0 with properties:
pciBusID: 0000:01:00.0 name: GeForce GTX 960M computeCapability: 5.0
coreClock: 1.176GHz coreCount: 5 deviceMemorySize: 4.00GiB deviceMemoryBandwidth: 74.65GiB/s
2020-06-24 14:51:01.045344: W tensorflow/stream_executor/platform/default/dso_loader.cc:55] Could not load dynamic library 'cudart64_101.dll'; dlerror: cudart64_101.dll not found
2020-06-24 14:51:01.051842: W tensorflow/stream_executor/platform/default/dso_loader.cc:55] Could not load dynamic library 'cublas64_10.dll'; dlerror: cublas64_10.dll not found
2020-06-24 14:51:01.058515: W tensorflow/stream_executor/platform/default/dso_loader.cc:55] Could not load dynamic library 'cufft64_10.dll'; dlerror: cufft64_10.dll not found
2020-06-24 14:51:01.065241: W tensorflow/stream_executor/platform/default/dso_loader.cc:55] Could not load dynamic library 'curand64_10.dll'; dlerror: curand64_10.dll not found
2020-06-24 14:51:01.072266: W tensorflow/stream_executor/platform/default/dso_loader.cc:55] Could not load dynamic library 'cusolver64_10.dll'; dlerror: cusolver64_10.dll not found
2020-06-24 14:51:01.080353: W tensorflow/stream_executor/platform/default/dso_loader.cc:55] Could not load dynamic library 'cusparse64_10.dll'; dlerror: cusparse64_10.dll not found
2020-06-24 14:51:01.087954: W tensorflow/stream_executor/platform/default/dso_loader.cc:55] Could not load dynamic library 'cudnn64_7.dll'; dlerror: cudnn64_7.dll not found
2020-06-24 14:51:01.095638: W tensorflow/core/common_runtime/gpu/gpu_device.cc:1598] Cannot dlopen some GPU libraries. Please make sure the missing libraries mentioned above are installed properly if you would like to use GPU. Follow the guide at https://www.tensorflow.org/install/gpu for how to download and setup the required libraries for your platform.
Skipping registering GPU devices...
2020-06-24 14:51:01.186835: I tensorflow/core/common_runtime/gpu/gpu_device.cc:1102] Device interconnect StreamExecutor with strength 1 edge matrix:
2020-06-24 14:51:01.194906: I tensorflow/core/common_runtime/gpu/gpu_device.cc:1108]      0
2020-06-24 14:51:01.198498: I tensorflow/core/common_runtime/gpu/gpu_device.cc:1121] 0:   N
2020-06-24 14:51:01.206694: I tensorflow/compiler/xla/service/service.cc:168] XLA service 0x21876e35be0 initialized for platform CUDA (this does not guarantee that XLA will be used). Devices:
2020-06-24 14:51:01.214517: I tensorflow/compiler/xla/service/service.cc:176]   StreamExecutor device (0): GeForce GTX 960M, Compute Capability 5.0
False
```

I solve this problem by cd into the folder of anaconda,and find cudart64_101.dll, copy it into c:/window/system32

For me also, it worked after I copied the dll file to the System32 folder. Every CUDA related directory is in the PATH but still didn't work until I copy it to the System32.
But is there any way of doing that without copying every dll file..?
I'm using Python3.8 from Windows Store.
CUDA 10.1 Update 2.
Tensorflow was installed using pip inside a virtualenv.

For me, the problem was the python installation.
Don't use the python from the windows. Download the python installation file from the official site.

@GhassenBenMakhlouf
Is this still an issue.

@Saduf2019
Yes, when I copy it into c:/window/system32, it solves the problem.
Actually I had to copy all these files into c:/window/system32, otherwise my script won't use GPU:

cublas64_10.dll
cublasLt64_10.dll
cudart64_101.dll
cudnn64_7.dll
cufft64_10.dll
curand64_10.dll
cusolver64_10.dll
cusparse64_10.dll
nvcuda.dll

This is a dirty solution, I hope you find a better one.

@GhassenBenMakhlouf
As confirmed this solves the problem, please feel free to move this issue to closed status.

@Saduf2019
Yes, when I copy it into c:/window/system32, it solves the problem.
Actually I had to copy all these files into c:/window/system32, otherwise my script won't use GPU:

cublas64_10.dll
cublasLt64_10.dll
cudart64_101.dll
cudnn64_7.dll
cufft64_10.dll
curand64_10.dll
cusolver64_10.dll
cusparse64_10.dll
nvcuda.dll

This is a dirty solution, I hope you find a better one.

Download CUDA and cuDNN from NVIDIA, and try again.

@Saduf2019
Yes, when I copy it into c:/window/system32, it solves the problem.
Actually I had to copy all these files into c:/window/system32, otherwise my script won't use GPU:

cublas64_10.dll
cublasLt64_10.dll
cudart64_101.dll
cudnn64_7.dll
cufft64_10.dll
curand64_10.dll
cusolver64_10.dll
cusparse64_10.dll
nvcuda.dll

This is a dirty solution, I hope you find a better one.

Download CUDA and cuDNN from NVIDIA, and try again.

I've installed CUDA Toolkit 11.2.135 already, but I'm having the same issue. I've tried downloading an other version, but NVIDIA driver says the installed is newer than the downloaded - so?!

I solve this problem by cd into the folder of anaconda,and find cudart64_101.dll, copy it into c:/window/system32

This worked for me, thank you!


Switch from NextJS -> Vite for demos

NextJS's server-side rendering mostly just conflicts with our local-first approach, with multiple workarounds required to make it functional. This attempts to simplify the supabase-todolist demo by not using NextJS.
The app is mostly unchanged. The main changes are:

Use Vite as a bundler.
Use react-router instead of NextJS navigation.
Shuffle the providers around a bit to better work with react-router.
Move lists out as a separate TodoListsWidget.

In the future, it will also allow:

Using react-router's loaders to load data before navigating to a new page - see #84

This similarly migrates the Yjs demo, adds a new "example-nextjs" to still have the basic boilerplate for NextJS available.
TODO:

[x] Check if more config or dependencies can be cleaned up.
[x] Rename demo to react-supabase-todolist.
[x] Do the same for the Yjs demo.
[x] Add a new basic NextJS demo, to still have a working example (similar to the current Vite demo).
[x] Fully offline-capable PWA.

IIRC we published those to showcase bundling with these frameworks; I think
showcasing the sync setup is also going to be helpful, but could also be a
future addition. If we implement this later I'd just say we add a note here
explaining how this field would be used, or remove it.
On Wed, Mar 6, 2024 at 2:05 PM Ralf Kistner @.***>
wrote:

@.**** commented on this pull request.
In demos/example-nextjs/.env.local.template
https://github.com/powersync-ja/powersync-js/pull/83#discussion_r1514353194
:

@@ -0,0 +1 @@
+NEXT_PUBLIC_POWERSYNC_URL=

Actually getting sync to work would also require setting up a connector.
The other two examples (example-vite and example-webpack) also don't have
anything for that.
Do you think it's worth expanding all those examples, perhaps with
instructions using temporary development tokens?
—
Reply to this email directly, view it on GitHub
https://github.com/powersync-ja/powersync-js/pull/83#discussion_r1514353194,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABYH5IFFYLF7TCI56ZGPHYLYW4BB5AVCNFSM6AAAAABD6D6DGSVHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMYTSMJZGU3DCOBWGA
.
You are receiving this because you commented.Message ID:
@.***>

IIRC we published those to showcase bundling with these frameworks; I think showcasing the sync setup is also going to be helpful, but could also be a future addition. If we implement this later I'd just say we add a note here explaining how this field would be used, or remove it.

@benitav I added minimal notes on configuring the example with a development token.


Godot Shader error check does not check if inout is const

Godot version:
3.2.1 stable
OS/device including version:
linux
Issue description, steps to reproduce:
Sending constant to inout does not trigger godot shader error check.
Shader code:
void func1(inout float val){
	val+=1.;
}

float func2(){
	return 1.;
}

and call anywhere func1(func2()); in shader

Cc @Chaosus

Seems like its fixed in the master but not in 3.2 branch

Fixed by #39820.


Run commands asynchronously

Run each command on its own thread instead of blocking the chat watcher thread. This way, commands that freeze/deadlock won't render Smokey unresponsive. If a command runs for more than 60 seconds without completing, a warning message is printed.
Also adds a !!/traceback/!!/backtrace command which prints a traceback for a given thread ID; intended to ease debugging of deadlocks.

I think the backtrack command should be privileged, plus, have you tested this?

@Daniil-M-beep I have tested it.
Making the backtrace command privileged would be easy enough, but why? There's not really a way to cause harm with it as it just retrieves debugging information; and similar commands like threads and errorlogs aren't privileged.

@NobodyNada Judging by the CI test, not sure you have.

@Daniil-M-beep @user12986714 is right, I most definitely tested but I forgot to update the unit tests. I'll push those as soon as I'm done banging my head against a wall at Python's mocking system...

Phew. Tests are fixed and flake is happy.

What about we just add a kill-thread so we can stop jobs without rebooting the whole thing?

@user12986714 In general, it's not a good idea to kill a thread, since it would interrupt whatever code is running without giving it a chance to clean up. It could even cause problems with in the Python interpreter itself -- some Googling says that it's possible to completely deadlock all threads by killing a thread which holds the global interpreter lock. There's really no drawbacks to doing a reboot (we even have logic to catch up on missed post IDs), so I believe it's best not to implement a !!/kill-thread.


One item remains on navbar line after collapse

Dear twbs dudes,
following bug:
If I create a collapsing menu (responsive nav) one item remains on the same line as the brand. It happens if I open the menu. Following workaround I found for myself:
`
Stay-linked

        ☰

Stay-linked

Home

Features

Pricing

About

`
It works for me.
Further information:

OS: Windows 10 (Build: 10.0.10586)
Browser: Chrome Version 53.0.2785.143 m (64-bit)

Thanks
DeveloperLen

Duplicate of #20749. Please search for pre-existing issues in the future.


\middle preceded by primed symbol gives an error.

The following code gives me an error "\middle must be within \left and \right" when I render it in my browser (Firefox 21.0, MathJax v2.4.0):

\left\{q\in Q \middle| P(q)\right\} works;
\left\{q\in Q' \middle| P(q)\right\} doesn't, nor \left\{q\in Q'' \middle| P(q)\right\} incidentally.
This also happens when Q' is the result of macro expansion:
\newcommand{\Q}{Q} \left\{q\in \Q \middle| P(q)\right\} works;
\newcommand{\Q}{Q'} \left\{q\in \Q \middle| P(q)\right\} doesn't.

The error is reproducible (though no error message is displayed) on the dynamic rendering page [1], yet I did not try it on the development branch for lack of time to install it (can anyone try?).
[1] http://cdn.mathjax.org/mathjax/latest/test/sample-dynamic.html

Thank you for reporting the issue! (I've modified your posting to escape the TeX). We'll look into it.

For now you can use
\left\{q\in {Q'} \middle| P(q)\right\}

to get the result you are looking for.  There is a problem with how \middle detects whether it is within \left...\right (obviously).

The issue1023 branch of my fork of MathJax resolves this issue (as well as a potential spacing problem with \middle).

==> Merged.

==> In testsuite
LaTeXToMathML/delimiters/issue1023.html


Allow Ffi calls to be marked as potentially blocking / exiting the isolate.

Some users are running into an issue where many isolates are calling out to C code that will then block. This can cause the dart app to no longer work due to our limitation on maximum number of threads that can be active in an isolate group at a given point in time.
The limitation is there to avoid too many threads executing Dart code at same time. This can lead to situations where X threads all have TLABs which may contain unallocated memory, but the X+1 thread tries to obtain TLAB and fails, which will cause it to trigger GC (despite other thread's TLAB still having unallocated memory)
=> Allowing unbounded number of threads to enter an isolate group can lead to excessive triggering of GCs (despite free memory in other thread's TLAB)
See runtime/vm/heap/scavenger.h for the current calculation of the limit:
  // The maximum number of Dart mutator threads we allow to execute at the same
  // time.
  static intptr_t MaxMutatorThreadCount() {
    // With a max new-space of 16 MB and 512kb TLABs we would allow up to 8
    // mutator threads to run at the same time.
    const intptr_t max_parallel_tlab_usage =
        (FLAG_new_gen_semi_max_size * MB) / Scavenger::kTLABSize;
    const intptr_t max_pool_size = max_parallel_tlab_usage / 4;
    return max_pool_size > 0 ? max_pool_size : 1;
  }

We may consider adding a boolean flag to specify that a FFI call may be blocking / should exit the isolate.
// Static binding
@Native("sleep", exitIsolate: true)
external void sleep(int seconds);

// Dynamic binding
dylib.lookup().asFunction<...>(exitIsolate: true);

to automatically exit and re-enter the isolate to avoid custom C code like this:
auto isolate = Dart_CurrentIsolate();
Dart_ExitIsolate();
<... run blocking C Code, e.g. sleep() ...>
Dart_EnterIsolate(isolate);

See motivating use case: https://github.com/dart-lang/sdk/issues/51254

The underlying issue is that new space (which we require for bump-allocation) doesn't scale with number of threads. The fact that this limitation would surface to the FFI API does seem a bit iffy.
We could device a scheme where FFI calls will give up their TLAB on transitions to C and re-acquire on the way back and limit the number of outstanding TLABs instead of number of active isolates. Though that would make transitions more heavyweight, would make returning from C (as well as Dart C API calls) possibly blocking for arbitrary amount of time. Seems less than ideal.
/cc @rmacnak-google

I think this can be for free for the uncontented case: What we could do is when a new mutator wants to enter the isolate and the limit has been reached, we can check if any existing mutators are in an ffi-exited safepoint state, CAS its safepoint state to one meaning it has been kicked out, causing the safepoint transition on the ffi-return to hit the slow path, and take its TLAB away. The safepoint transition slow path then has a new check if it needs to wait on the mutator count to re-enter as a mutator.

I think this can be for free for the uncontented case: What we could do is when a new mutator wants to enter the isolate and the limit has been reached, we can check if any existing mutators are in an ffi-exited safepoint state, CAS its safepoint state to one meaning it has been kicked out, causing the safepoint transition on the ffi-return to hit the slow path, and take its TLAB away. The safepoint transition slow path then has a new check if it needs to wait on the mutator count to re-enter as a mutator.

That's an interesting idea.
I'm a little worried that doing this blindly can lead to situations where e.g. Flutter UI isolate does a FFI call, then another thread kicking the UI isolate out. When the FFI call on UI isolate returns it will take the slow path and block (which could freeze flutter UI).
This can also happen to some extent today as well - but only at event loop boundary (e.g. Flutter UI isolate is idle, N threads enter isolate group and then flutter UI isolate cannot enter anymore but has to wait).
If one mutator has been kicked out and returns from ffi call then in the slow path it should be allowed to kick out another thread if it's in a ffi call. That would mean the system would work flawlessly irrespective of number of threads - as long as there are not more than N threads executing Dart code concurrently (which may be an ok restriction as that seems rather unlikely). Though it will require some synchronization on both sides:

Calling native via ffi: If there's another thread waiting to execute Dart we need to notify it (could use similar mechanism as our existing "gc-safepoint-requested" bit which forces GeneratedToNative to slowpath)
Return from ffi call: Safepoint may have been stolen from the side, so we have to take slowpath and wait for mutator count (or kick another thread out if there's any in ffi-exited state)

Hi. I have a complaint about this. If we're going to expose to FFI developers to those kinds of Isolate details, why we can't have some way for native code to at least return Dart objects that can be created through the Dart_CObject struct?


Fixing Typo

Fixing typo.

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

skype/skype-ps/skype/Stop-CsWindowsService.md
:white_check_mark:Succeeded
View (skype-ps)

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

skype/skype-ps/skype/New-CsEdgeAllowList.md
:white_check_mark:Succeeded
View (skype-ps)

skype/skype-ps/skype/Remove-CsCertificate.md
:white_check_mark:Succeeded
View (skype-ps)

skype/skype-ps/skype/Stop-CsWindowsService.md
:white_check_mark:Succeeded
View (skype-ps)

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Playback bug on setting play range start and end

Have you read the FAQ and checked for duplicate open issues?
YES
What version of Shaka Player are you using?
3.3.2
Can you reproduce the issue with our latest release version?
No
Can you reproduce the issue with the latest code from main?
No
Are you using the demo app or your own custom app?
Demo APP
If custom app, can you reproduce the issue using our demo app?
What browser and OS are you using?
Chrome, Firefox, safari edge on iOS, iPadOS, macOS, Linux, windows
For embedded devices (smart TVs, etc.), what model and firmware version are you using?
What are the manifest and license server URIs?

https://dash.akamaized.net/akamai/bbb_30fps/bbb_with_tiled_thumbnails.mpd
https://multiplatform-f.akamaihd.net/i/multi/will/bunny/big_buck_bunny_,640x360_400,640x360_700,640x360_1000,950x540_1500,.f4v.csmil/master.m3u8
What configuration are you using?  What is the output of player.getConfiguration()?

playRangeStart: 10,
playRangeEnd: 100
What did you do?

I try to set the start and end range for a VOD DASH and HLS
What did you expect to happen?
To have a timeline according and a playback
What actually happened?

As the previous problem related to the timeline has been resolved by #4026 of #4068
But after setting the range the playback is paused at the start edge of the timeline.
When I click ahead of the start point it plays the video but after some time for both DASH and HLS
When the HLS VOD gets ended the player is not able to play it again
Every time when I click on the timeline or try to seek the video is played very delayed and sometimes it stops the playback, especially in the case of  HLS VOD.

One more bug:
When we seek the timeline it changes the playRangeEnd === duration === endtime in the player

Hello @joeyparrish Can you please comment on this bug?
Thanks

Hello, did you have time to see about this bug?

Hello, We are in urgent need of this feature can you please respond to this bug?


Contrast problem with the activity source icon on the very left.

The github icon on top of white on top of a really pale blue. My eyes hurt!
Possible solutions?
1 Add border
In that case, increase the size of the icon too.

2 Add a shadow
In this case too, just increase the size of the icon. Increase it anyway. it's too tight right now.

Personally I like the border one. Thanks.

we can decrease the size of the font icon and i like the shadow effect

In the screenshot, it just looks too big because "Retina" but it's bumped up from 36px to 44px which kinda looked nice.
Shadow looks good but it just somehow doesn't go with the existing overall design.


fix and update some typings for fe2

Copied some interfaces from fe2 (as they were better) to replace the previous ones, functionality wise they are the same, although more restricting.
I had to rename .d.ts files to.ts as typescript wouldn't correctly import otherwise.
Some typings have also been corrected.
I am not sure whether this should be considered a minor or a major update, please advice.

@SimonSchick, thanks for your PR! By analyzing the history of the files in this pull request, we identified @ProbablePrime to be a potential reviewer.

Also looping in @connor4312


Preprocessor macro for targets

In some cases, it would be useful to have a preprocessor macro for targets to allow custom handling for the target's platform.
For example, when compiling to SPIRV, it would be useful to have __spirv__ to add attribute decorations to resource declarations:
struct ParamDesc {
    int Value;
};

#if defined(__spirv__)
[[vk::push_constant]]
#endif
ConstantBuffer<ParamDesc> Params : register(b1);

float4 main(float4 P : SV_POSITION) : SV_TARGET
{
    return float4(Params.Value, 0, 0, 1);
}

For the above case, not all compilers can handle attribute decorations and ignore unknown ones. FXC fails with a syntax error if the surrounding #if defined is removed:
C:\local\Temp\4e959031-9817-4f20-9573-ef6712bed9c6.hlsl(6,2): error X3000: syntax error: unexpected token '['

Another example is when using mesh shaders, there's some functionality differences between Apple GPUs and PC GPUs that users may wish to custom handle:
struct Payload {
    uint MeshletIndices[32];
};

struct MeshOutput {
    float4               PositionCS : SV_POSITION;
#if defined(__metal__)
    // Metal doesn't support support barycentric interpolation using
    // vertex id from mesh shader to pixel data, so pass attribute
    /// data.
    float3               PositionWS : POSITIONWS;
    float3               Normal     : NORMAL;
    float2               TexCoord   : TEXCOORD;
#else
    // Use barycentric coords in pixel shader to interpolate attributes
    nointerpolation uint VertexIndex : VERTEX_INDEX;
#endif
};

[outputtopology("triangle")]
[numthreads(128, 1, 1)]
[shader("mesh")]
void main(
                 uint       gtid : SV_GroupThreadID,
                 uint       gid  : SV_GroupID,
     in payload  Payload    payload,
    out indices  uint3      triangles[128],
    out vertices MeshOutput vertices[64])
{
	SetMeshOutputCounts(4, 16);

    vertices[gtid].PositionCS = float4(1, 0, 0, 1);

#if defined(__metal__)
    vertices[gtid].PositionWS = float3(0, 0, 0);
    vertices[gtid].Normal     = float3(0, 1, 0);
    vertices[gtid].TexCoord   = float2(0, 0);
#else
	vertices[gtid].VertexIndex = gtid;
#endif
}

Besides having __spirv__ to maintain compatibility with DXC, I don't have recommendations on what the preprocessor macros should be for the targets.

The way Slang compiler is architected is that we will always compile user code to platform-neutral Slang IR modules in-dependent of the final code generation target, and then specialize the IR to each target after linking.
This means that our front-end has no assumption or knowledge of the code gen target, and it cannot evaluate anything at parsing/type checking time that is target-dependent.
In function bodies, we already have __target_switch that can deal with target-dependent code.
What is being asked is what should we do for the global scope.
I think this should be handled by the capability system. We can allow the user to do:
struct MyType
{
     [require(metal)] int metalOnlyField;
}

And the compiler can implement the behavior that such fields declared with require is only available and defined for metal.

In the meanwhile, the user can always choose to define those macros themselves if they know what target they are compiling for. It is just that we cannot provide a builtin __spirv__ macro due to the compiler design.

Got it! Thanks for the explanation, Yong.
Just looked at __target_switch and it makes sense for any shader that's authored as Slang source. The syntax style is new to me and will take a bit to get used to, but from first reading it seems able to achieve similar things to the platform macro blocks. I do think the documentation could use some additional wording for anyone coming to look for a solution on how to customize behavior in function bodies for different targets.
For the case statements, do the values just correspond to the parameters that arg value passed to -target or is there a different mappings that users should be aware of?
Since the compiler can't support the macros due to architecture, we can address the issue through guidance per your suggestion of user defined macros.

For the very first example, you can use the command line argument to control the macro.
struct ParamDesc {
    int Value;
};

#if defined(__spirv__)
[[vk::push_constant]]
#endif
ConstantBuffer<ParamDesc> Params : register(b1);

float4 main(float4 P : SV_POSITION) : SV_TARGET
{
    return float4(Params.Value, 0, 0, 1);
}

In the command-line argument, you can use -D__spirv__ in your example above.
slangc.exe source.slang -target spirv -D__spirv__ -entry main -stage fragment

For the case statements, do the values just correspond to the arg values passed to -target or is there a different mappings that users should be aware of?

For the case statements you may pass almost any capabilities from the Slang capabilities system to target specific feature sets

case GLSL_450: will only be targeted if a user compiles code with equal to or greater than GLSL450 capabilities.

For most users, case glsl: case spirv: case hlsl: default: will be enough.
For examples, hlsl.meta.slang and glsl.meta.slang is a good reference for how to use __target_switch.

@ArielG-NV Just for completeness, will there also be a Metal target as well? Something like metal?

@chaoticbob  Yes, the metal capability is already defined.

Thanks. Closing this issue, will follow with @bmillsNV and @swoods-nv about documentation.


Add HRIS assertions in GraphQL.

Coverage remained the same at 89.534% when pulling 19874430cd7473acff4c7b1cf2d223f6d9f285e8 on akatsoulas:hris-graphql-schema into 359530c54e79b948b58f07708c48e8b761e49693 on mozilla:master.

@johngian can you have a look at thsi?

Closing in favor of #2716


Setup post database schema with mongoose

Before a post can be created, a schema must be created in order to set up a template for what is stored in the database.

Complete


Is tweetinvi designed for all three tiers of search APIs with twitter?

Or just the standard level that I have been using it for up to now?

It is designed to the free tier as I never managed to have access to upper tiers.
If you are happy to share yours, I will be more than happy to add to the library.

Thanks for your quick response. Looks like I have to build one and it will be part of https://github.com/BHoM - public so would be interesting to discuss how to combine with tweetinvi. What you have created is really very cool. 👍


Intl.select "other" case requirement inconsistency

Docs at https://api.flutter.dev/flutter/intl/Intl/select.html say:

We look up the value of choice in cases and return the result, or an empty string if it is not found.

However in fact the following exception is thrown when value not matching any case is passed:
Invalid argument(s): The 'other' case must be specified
When the exception was thrown, this was the stack:
#0      Intl.selectLogic (package:intl/intl.dart:552:7)
#1      Intl._select (package:intl/intl.dart:533:26)
#2      Intl.select (package:intl/intl.dart:517:12)

Code reproducing this issue:
print(Intl.select(
      'foo',
      {
        'bar': 'bar',
        'baz': 'baz',
      },
      args: ['foo'],
    ));

$ flutter --version
Flutter 1.22.3 • channel stable • https://github.com/flutter/flutter.git
Framework • revision 8874f21e79 (2 weeks ago) • 2020-10-29 14:14:35 -0700
Engine • revision a1440ca392
Tools • Dart 2.10.3

Hi @koral--
This ist just documentation in flutter docs, for errors, please file a new issue on intl repository
I will label this for documentation
Thank you

My intent was to first try here to determine where exactly the problem is located:

According to the https://github.com/dart-lang/intl/issues/343#issuecomment-1284998469 it's a bug in the documentation.


not a git repository (or any of the parent directories): .git

I initially tried to git clone this repo, but then realised that I would have to run cmake if I did that, so I instead downloaded the source bundle from the releases page, and followed the instructions. However once I got to the make stage, it threw this error:
$ make
fatal: not a git repository (or any of the parent directories): .git
fatal: not a git repository (or any of the parent directories): .git
mkdir -p obj
>> src/pdb2cif.cpp
rm -f version-info-*.txt
fatal: not a git repository (or any of the parent directories): .git

download or git, no matter what you choose, you still have to use cmake.


Add back method which was removed in #3969

Summary
The public method Mail.ConvertToText was accidentally removed in #3969, this PR adds it back.

I think it's a helpful method (we discovered it was missing because @donker was using it).  However, we are duplicating the functionality (#3969 moved this to be a private method of CoreMailProvider).  It would be nice to adjust one of the call sites to use the other (which, in this case, would probably mean adjusting CoreMailProvider to call Mail.ConvertToText).
This isn't functionality I would plan to directly use, so I can't comment on whether it's still a fit solution, so perhaps there's an alternative we should promote instead.

I think it's a helpful method (we discovered it was missing because @donker was using it).  However, we are duplicating the functionality (#3969 moved this to be a private method of CoreMailProvider).  It would be nice to adjust one of the call sites to use the other (which, in this case, would probably mean adjusting CoreMailProvider to call Mail.ConvertToText).
This isn't functionality I would plan to directly use, so I can't comment on whether it's still a fit solution, so perhaps there's an alternative we should promote instead.

@bdukes shouldn't this be targeted to the 9.8.1 release branch?

@bdukes shouldn't this be targeted to the 9.8.1 release branch?

Not necessarily.  We can decide that's what we want to do.  However, this is not a regression introduced by 9.8.1 (it was introduced in 9.8.0), so it doesn't meet the standard criteria for hot-fix adding to an active RC.

Not necessarily.  We can decide that's what we want to do.  However, this is not a regression introduced by 9.8.1 (it was introduced in 9.8.0), so it doesn't meet the standard criteria for hot-fix adding to an active RC.


Opus-cat not on the list of TM:s under Use

Hello,
I think I have managed to download and install Opus-cat correctly, as the translation test worked fine. But when I want to add the Opus-cat to a translation project, Trados Studio 2021 apparently does not find Opus-cat. Any ideas about what went wrong and what I should do?
Carmatia

Hi,
This probably means that the Trados plugin has not been loaded for some reason. Have you installed the plugin yet, either via the Trados 2021 AppStore integration or by downloading it from here https://appstore.sdl.com/language/app/opus-cat-nmt/1160/?
-Tommi

Hi,
You were right, even though i thought I hade loaded the Trados pugin correctly. Clearly, the down loading had not been completed. Now it has and the Opus-Cat is listed in the Use menu. Thank you so much!
Carola


Other correlation time estimators

Sokal's notes (page 16), citing an earlier paper use an procedure for choosing the window for summing the empirical ACF where the length of the window, M is chosen to be the smallest value such that M is at least c times the estimated autocorrelation time, where c is something like 4, 6, or 10.
Jenke also uses the same scheme, with c=6.
Evertz doesn't seem to like this procedure (page 59)
LaplacesDemon's in R uses the Geyer IPS estimator.
Geyer's ICS estimator.

In case you haven't seen Geyer's chapter from the Handbook of Markov chain Monte Carlo, it's has some nice discussion of this as well in Section 1.10 "Variance Estimation".
I'd love to see the initial convex sequence (ICS) estimator be implemented as well (discussed in Section 1.10.2).   The canonical reference for this and related methods seems to be:

Geyer, C. J. 1992. Practical Markov chain Monte Carlo (with discussion). Statistical Science, 7:473–511.
PDF available here.

Yes, I've read the paper and cited it in the IPS docs. Adding the ICS is a good idea too.

Also there's another ACF from http://www.sciencedirect.com/science/article/pii/0370269389915633 that I just saw, where you truncate the summation after a fixed number of terms (e.g. 20), and then add in a remainder based on the ratio of the final two values included in the sum, which looks like an analytical correction based on the assumption that the terms are decaying exponentially.


Added custom shadow inside search view.

Extracted drawable from Google´s IO app
Also added some tools params to visualize the layout with the design tab in Android Studio.

Need to remove the formattings and properly put the shadow.


Documentation display and link issues

Environment
Docker
Version
0.15.0
Describe the problem
when i browse to this url:  https://homarr.dev/docs/management/integrations/  i noticed that the headings seem to be not in the right place.

also when i click the bottom two integrations, i get a page not found error.

Additional info
No response
Please tick the boxes

[X] You've read the docs
[X] You've checked for duplicate issues
[X] You've tried to debug yourself

Should get solved by #38

Got solved by #38


[szip:x64-mingw-dynamic] Reason for Could not find proper second linker member

I am playing with my version of the vcpkg, as per this https://github.com/microsoft/vcpkg/issues/10349 issue, to make it bootstrap under MSYS2-MINGW64 without using any Visual Studio related toolchain. I changed the relevant parts of the bootstrap scripts and made the C++ codes compile with g++ etc. Now I have a vcpkg version bootstrapped under MSYS2 running fine. Most of the ports I am interested in are also building just fine, keep in mind that I am using community triplet x64-mingw-dynamic and it is not officially supported in the first place.
But while building szip as part of opencv:

./vcpkg.exe install szip:x64-mingw-dynamic

Your feedback is important to improve Vcpkg! Please take 3 minutes to complete our survey by running: vcpkg contact --survey
Computing installation plan...
The following packages will be built and installed:
szip[core]:x64-mingw-dynamic
Detecting compiler hash for triplet x64-mingw-dynamic...
Starting package 1/1: szip:x64-mingw-dynamic
Building package szip[core]:x64-mingw-dynamic...
Could not locate cached archive: C:\Users\burak\AppData\Local\vcpkg\archives\78\789f979763e4fb257ce37380862d2c721a65fb74.zip
-- Using community triplet x64-mingw-dynamic. This triplet configuration is not guaranteed to succeed.
-- [COMMUNITY] Loading triplet configuration from: C:\dev\bin\msys64\opt\vcpkg\triplets\community\x64-mingw-dynamic.cmake
-- Using cached C:/dev/bin/msys64/opt/vcpkg/downloads/szip-2.1.1.tar.gz
-- Cleaning sources at C:/dev/bin/msys64/opt/vcpkg/buildtrees/szip/src/szip-2.1.1-6b767acfa7.clean. Use --editable to skip cleaning for the packages you specify.
-- Extracting source C:/dev/bin/msys64/opt/vcpkg/downloads/szip-2.1.1.tar.gz
-- Applying patch fix-szip-config-to-set-szip-found.patch
-- Using source at C:/dev/bin/msys64/opt/vcpkg/buildtrees/szip/src/szip-2.1.1-6b767acfa7.clean
-- Configuring x64-mingw-dynamic
-- Building x64-mingw-dynamic-dbg
-- Building x64-mingw-dynamic-rel
-- Performing post-build validation
Could not find proper second linker member

I want to know what could be the reason for Could not find proper second linker member message since actual build proceeds without errors. I didn't modify cofffilereader.cpp to make it bootstrap under MSYS2 and it was validating other ports just fine. If someone could point me towards the correct direction as to cause of that error msg, I would be glad.
I found a ticket on a similar issue:
https://github.com/microsoft/vcpkg/pull/10398
but I don't know how to make use of it.
I also used dumpbin myself on the generated lib file. It indeed shows:

36C size
correct header end
34 public symbols

for dumpbin /linkermember:1 szip_D.lib but not for dumpbin /linkermember:2 szip_D.lib. The other lib files I generated with my vcpkg don't have such problems. Is it maybe szip is not viable to building as a dynamic library?

@thumpro Have you contacted us recently about you having issues with other ports?
You should refrain yourself from mixing environments and/or compilers! You are looking for trouble.
Just built szip (dynamic) from source on Windows 10 without issues.

02/11/2020  09:16 PM            87,999 szip.dll

It is a bit hard to follow your description of this issue! Can you further elaborate on it?

Note that this issue has nothing to do with the previous one I opened about a build failure on x64-mingw-dynamic. [https://github.com/microsoft/vcpkg/issues/14012]
This time, I am trying the investigate a feature request found on following open issue: https://github.com/microsoft/vcpkg/issues/10349. There they suggest making vcpkg bootstrappable using MinGW64 toolchain and outline the stage where they failed to compile vcpkg itself,
To that end, I modified the vcpkg source code and finally bootstrapped it under MinGW64. After, I wanted to check by building some ports to see if it is correctly working. I was able to build many but szip failed during post-build check with the error Could not find proper second linker member. I am not keen on modifying the source file cofffilereader.cpp myself, so I was asking for explanation on what this error message could mean.

The issue #10349 is from March and it remains open! Wow!
I didn't have/need to modify any file in order to build szip successfully.
Probably your environment is broken somehow.

set(VCPKG_POLICY_SKIP_DUMPBIN_CHECKS enabled) skips the dumpbin checks.  second linker member are only generated if VS link.exe is used

@Neumann-A That's strange. I will try it as soon as I have the time. It is strange because I am sure I am not using link.exe at all. I bootstrapped vcpkg under MSYS2's Mingw64 shell, using g++ to compile vcpkg itself etc. And I am sure Visual C++ toolchain is not in the PATH for Mingw64 shell.
So, if second linker member is only generated if VS link.exe is used, how then I was able to build other ports without complain? They didn't use link.exe either but they passed the post-build checks just fine.

because there is a policy setting in the triplet. But this policy setting only works for import libraries/dll not for static libraries.

@thumpro Do not mix environments and/or compilers and/or linkers (e.g. link, cl, g++, etc).
You are looking for trouble!

@sfhacker I can assure you, I am not mixing toolchains in this issue. The whole idea of the issue I first linked, about bootstrapping vcpkg with Mingw64, is to be able to bootstrap and use vcpkg without even installing Visual Studio.
@Neumann-A I tried downloading the szip source and build it manually through their configure and Makefile. configure recognized the system as mingw64 and ended up building a static library. There was a warning error msg, somewhere around the lines Undefined symbols are not supported by Mingw dlls, building static library instead. So, it seems there is an issue with natively building this library as shared library under Mingw. And CMake somehow manages it but it does not pass the dumpbin checks in the end.
To that end, I added you policy, first directly to x64-mingw-dynamic triplet cmake file, set(VCPKG_POLICY_SKIP_DUMPBIN_CHECKS enabled). This didn't help, second linker member check is still performed and it failed.
Then I tried using the policy in szip's portfile.cmake, as follows:
if(${TARGET_TRIPLET}  MATCHES "x*-mingw")
set(VCPKG_POLICY_SKIP_DUMPBIN_CHECKS enabled)
endif()
This does not help either. It looks like it should definitely bypass dumpbin checks but I am still getting the Cannot find proper second linker member. And surely, my software is recent enough to include the commit where you introduced that policy.
https://github.com/microsoft/vcpkg/pull/9901/files/9298dec358d145b0feb0d7336f755c639fbacde8
I guess the other policy you were talking about, the one in the triplet file was set(VCPKG_POLICY_DLLS_WITHOUT_LIBS enabled). For szip this triplet creates the .lib file, .a file and .dll. But as we said, .lib's dumpbin check fails. All the other ports that I built successfully for Mingw64 created only a .dll file and a .dll.a file. I have no idea why this one creates a .lib file as well which makes the whole check fail.

try
set(VCPKG_POLICY_SKIP_ARCHITECTURE_CHECK enabled)
set(VCPKG_POLICY_SKIP_DUMPBIN_CHECKS enabled)

it should work since the same problem arises if compiling with clang-cl and using ldd

@thumpro It seems we are unable to make any progress here! We are stuck, going round in circles!
You lost me here!

@sfhacker Don't worry, we are actually making quite the progress 👍
set(VCPKG_POLICY_SKIP_ARCHITECTURE_CHECK enabled)
set(VCPKG_POLICY_SKIP_DUMPBIN_CHECKS enabled)

with this, it actually worked. Could you explain the machinery a little bit @Neumann-A and what is actually going on here? Since I bypassed the check essentially, I am not 100% sure in my mind that the build was correct and I won't be lacking any required smybols etc. inside the built binaries.
The CMake targets that were suggested to use at the end was:
find_package(szip CONFIG REQUIRED)
    target_link_libraries(main PRIVATE szip-shared szip-static)

Mingw building both targets? you should only link against one of them.
the checks within vcpkg are only performed on windows and under the assumption that MSVC is used. As such disabling them for other things is fine. The linker will warn you otherwise if the architecture is wrong.

Then how other people using this x64-mingw-dynamic triplet are able to build this port without adding that policy to their portfile.cmake? For example, @sfhacker shows above a screenshot showing a successful build of szip.
If I use

.\bootstrap-vcpkg.bat

to bootstrap my vcpkg and then use vcpkg.exe install some_port:x64-mingw-dynamic, will it still use Visual C++ toolchain to build stuff? Because then you would not be able to link those with mingw64 distributed g++ since they are built with MSVC. In a sense, I am uncertain how people are using this x64-mingw-dynamic triplet on a Windows machine, bootstrapped using the .bat bootstrap script.

./bootstrap-vcpkg.bat only uses MSVC to build vcpkg itself. Libraries will be built with mingw if you use the mingw triplets (see https://github.com/microsoft/vcpkg/blob/master/scripts/toolchains/mingw.cmake)

@longnguyen2004 Just one more thing that I really wanted to know, thanks. I guess since this is a community triplet, there is not much documentation.

@thumpro It all depends on the community to provide support for it. I've worked on a lot of other ports (openssl, libffi, freetype, just to name a few) to get it to build on mingw. Anyway I'll take a look at szip.

@thumpro Please test #14460

Did a clean checkout of this branch and verified that szip builds. Thanks a lot!
Then I moved on to building the package that I actually needed szip for, hdf5, to see if it actually correctly links with other packages. There I was not able to move further because zlib dependency started failing. I am not sure if your changes can cause zlib port to fail and I can not directly say that is the reason but zlib was building fine for me previously. And this is just a clean checkout of your branch.

Can you upload the build log? It's building fine for me

Log

./vcpkg --triplet x64-mingw-dynamic install zlib
Your feedback is important to improve Vcpkg! Please take 3 minutes to complete our survey by running: vcpkg contact --survey
Computing installation plan...
The following packages will be built and installed:
    zlib[core]:x64-mingw-dynamic
Detecting compiler hash for triplet x64-mingw-dynamic...
Starting package 1/1: zlib:x64-mingw-dynamic
Building package zlib[core]:x64-mingw-dynamic...
Could not locate cached archive: C:\Users\[redacted]\AppData\Local\vcpkg\archives\00\00221dc6ee21e0aee1145088a50007e348759eb7.zip
-- Using community triplet x64-mingw-dynamic. This triplet configuration is not guaranteed to succeed.
-- [COMMUNITY] Loading triplet configuration from: C:\vcpkg\triplets\community\x64-mingw-dynamic.cmake
-- Using cached C:/vcpkg/downloads/zlib1211.tar.gz
-- Cleaning sources at C:/vcpkg/buildtrees/zlib/src/1.2.11-63309e48e5.clean. Use --editable to skip cleaning for the packages you specify.
-- Extracting source C:/vcpkg/downloads/zlib1211.tar.gz
-- Applying patch cmake_dont_build_more_than_needed.patch
-- Applying patch 0001-Prevent-invalid-inclusions-when-HAVE_-is-set-to-0.patch
-- Applying patch add_debug_postfix_on_mingw.patch
-- Using source at C:/vcpkg/buildtrees/zlib/src/1.2.11-63309e48e5.clean
-- Configuring x64-mingw-dynamic
-- Building x64-mingw-dynamic-dbg
-- Building x64-mingw-dynamic-rel
-- Using msys root at C:/vcpkg/downloads/tools/msys2/c809757c94447846
-- Fixing pkgconfig file: C:/vcpkg/packages/zlib_x64-mingw-dynamic/lib/pkgconfig/zlib.pc
-- Fixing pkgconfig file: C:/vcpkg/packages/zlib_x64-mingw-dynamic/debug/lib/pkgconfig/zlib.pc
-- Installing: C:/vcpkg/packages/zlib_x64-mingw-dynamic/share/zlib/copyright
-- Performing post-build validation
-- Performing post-build validation done
Stored binary cache: C:\Users\[redacted]\AppData\Local\vcpkg\archives\00\00221dc6ee21e0aee1145088a50007e348759eb7.zip
Building package zlib[core]:x64-mingw-dynamic... done
Installing package zlib[core]:x64-mingw-dynamic...
Installing package zlib[core]:x64-mingw-dynamic... done
Elapsed time for package zlib:x64-mingw-dynamic: 2.258 min

Total elapsed time: 3.618 min

The package zlib is compatible with built-in CMake targets:

    find_package(ZLIB REQUIRED)
    target_link_libraries(main PRIVATE ZLIB::ZLIB)

I have seen several issues regarding zlib!

About zlib, it is definitely not related to this issue, but something else. During ninja build, it looks for a program called x86_64-w64-mingw32-windres.exe due to some CMake config or something I guess. The problem is, this binary was named just windres.exe on my MinGW installation which I installed through MSYS2. I created a copy of it with the correct name and was able to build zlib fine. Still, this might use some streamlining.

I wasn't aware that prefixed windres isn't always available, I'll add a fallback into the toolchain file.

Cool, I am looking forward to merging of this PR.

Maybe we should close this ticket once the other PR is merged but since we caught some traction on this community triplet, I felt like adding this to here.
ffmpeg with x64-mingw-dynamic community triplet was failing for me (which is apart from hdf5, the other big dependency for opencv4 if you want to do video capture etc.)
I had to change the portfile as:
`$ git diff ports/ffmpeg/portfile.cmake
diff --git a/ports/ffmpeg/portfile.cmake b/ports/ffmpeg/portfile.cmake
index cbe8df318..f09e57df4 100644
--- a/ports/ffmpeg/portfile.cmake
+++ b/ports/ffmpeg/portfile.cmake
@@ -25,7 +25,7 @@ endif()
vcpkg_find_acquire_program(YASM)
get_filename_component(YASM_EXE_PATH ${YASM} DIRECTORY)
-if(VCPKG_TARGET_IS_WINDOWS)
+if(VCPKG_TARGET_IS_WINDOWS AND NOT VCPKG_TARGET_IS_MINGW)
#We're assuming that if we're building for Windows we're using MSVC
set(INCLUDE_VAR "INCLUDE")
set(LIB_PATH_VAR "LIB")
@@ -37,6 +37,10 @@ endif()
set(ENV{PATH} "$ENV{PATH}${VCPKG_HOST_PATH_SEPARATOR}${YASM_EXE_PATH}")
set(OPTIONS "--enable-asm --enable-yasm --disable-doc --enable-debug --enable-runtime-cpudetect")
+if(VCPKG_TARGET_IS_MINGW)

  set(VCPKG_TARGET_IS_WINDOWS false)

+endif()
+
if(VCPKG_TARGET_IS_WINDOWS)
if(VCPKG_TARGET_ARCHITECTURE STREQUAL "arm" OR VCPKG_TARGET_ARCHITECTURE STREQUAL "arm64")
vcpkg_acquire_msys(MSYS_ROOT
@@ -61,8 +65,15 @@ if(VCPKG_TARGET_IS_WINDOWS)
 set(SHELL ${MSYS_ROOT}/usr/bin/bash.exe)
 set(OPTIONS "--toolchain=msvc ${OPTIONS}")

-else()

set(SHELL /bin/sh)
+elseif(VCPKG_TARGET_IS_MINGW)

  set(SHELL sh)

  if(VCPKG_TARGET_ARCHITECTURE STREQUAL "x64")

          set(OPTIONS "${OPTIONS} --target-os=mingw64")

  elseif(VCPKG_TARGET_ARCHITECTURE STREQUAL "x86")

          set(OPTIONS "${OPTIONS} --target-os=mingw32")

  endif()

+else()

  set(SHELL /bin/sh)

endif()
set(ENV{${INCLUDE_VAR}} "${CURRENT_INSTALLED_DIR}/include${VCPKG_HOST_PATH_SEPARATOR}$ENV{${INCLUDE_VAR}}")`
to finally be able to build. Also, that port is not printing usage information once the build is done. That might be due to my changes.

Hmm weird, I remember it working a while ago, I'll take a look at some point
Also, do not unset VCPKG_TARGET_IS_WINDOWS, instead use if(VCPKG_TARGET_IS_WINDOWS AND NOT VCPKG_TARGET_IS_MINGW)

So, if second linker member is only generated if VS link.exe is used, how then I was able to build other ports without complain? They didn't use link.exe either but they passed the post-build checks just fine.

For the record, the difference is whether import library generated by the mingw linker is named "....dll.a" (the common case with mingw) or "....lib" (the szip case). The failing check in vcpkg-tool is applied only to ".lib" files.
#14460 changes the policy in the mingw triplet just for the szip. I wonder how this is meant to scale. I run into this error with hdf5 today. I can submit another if/endif, or the policy is generally toggled in the mingw triplets, due to "second linker member are only generated if VS link.exe is used".

After more investigation, I realized the .lib file isn't a proper VS import library, but a mingw import library with the extension changed to .lib. We can either ask upstream to change the behavior, or patch it out by ourselves, which is what MSYS2 has done (libszip: here and here, hdf5: here and here)

I'll pull the patch from MSYS2 and make a pull request when I have time.


Remove getAuxQueryResolvers method from keystone

This method was a bit of a weird outlier that didn't serve much purpose. I've moved it inline to reduce the public API surface area.
(Also brings in the result of yarn format on some changelogs).

Good code-health clean up 👍


[Improvement] Add check mechanism to disable assert in Gravitino

What would you like to be improved?
We should explicctly disable using assert in Gravition and throw an exception if it violate the rules
How should we improve?
No response

Hi @yuqi1129, is this issue about adding AssertFalse errorprone? If yes, I would like to take it. Thank you.

Hi @yuqi1129, is this issue about adding AssertFalse errorprone? If yes, I would like to take it. Thank you.

@FrankYang0529  I think it is related to https://github.com/datastrato/gravitino/pull/2375.

Hi @yuqi1129, is this issue about adding AssertFalse errorprone? If yes, I would like to take it. Thank you.

I don't think so,  what we are going to do is to forbid the syntax assert condition.


Add documentation for make targets

We should list the actual make targets that we suggest using on a regular basis.

This is already partially documented here.

This is already partially documented here.


SCRD-3501  Added disable compute service and network agent page

This commit includes:

Added a page to disable compute service, delete aggregates,
migrate instances and disable network agents.
When results contain partial failed, it will pop up for user confirmation
to abort or continue.
When old compute host is reachable, starts instance live migration,
otherwise will skip.
Added a dialog to allow user monitor instances migration
progress
Added a complete page for replacing a compute host.
Removed the complete page for adding a compute host

merged with latest master ...the replace detail seems broken...can not continue testing. need investigate

It is odd that it shows that convert CompleteAddCompute to migrationmonitor.less. What I did was removing CompleteAddCompute.js

I noticed while testing this that you can't reuse the IP address of the server you're replacing... is that intentional?

I must have done something wrong: There are no valid server(s) with hostname(s) to run deployement  (also, deployment only has 2 e's)

@jeremy-moffitt  Replacing a compute is to add a new compute first, migrate instances from old compute to new compute, and then disable and delete the old compute,  therefore can not use the existing IP on the newly added compute.

the problem is not reproducing for me, I'm not sure what I did differently, but its working fine now

The changes based on the comments look good... I'm taking another look at the error handling... these probably should've been submitted as a separate commit.

I should be more familiar with this, but is the "logger" thats passed in providing a means to log additional data to the log window where the playbook results are being streamed?

In this case the logger is a function, and when called, it outputs a line into the log that the user is viewing on screen (the one that shows the output of the playbook)


Question: Hope we have silent installation for wiltonDB, can you please confirm?

Question: Hope we have silent installation for wiltonDB, can you please confirm?

Yes, silent installation can be run like this (under admin user):
msiexec /i c:\tmp\wiltondb3.3_10.15.0.msi /qn /l*v c:\tmp\wiltondb_install_log.txt

thanks Alex


🚀 Feature: push Entity updates to an external System like Kafka

🔖 Feature description
Entity updates should get pushed to Kafka or SQS, etc.

I think a better approach and to separate the concerns would be to send Entity updates to Kafka, where these entities could be processed.
🎤 Context
We enriched the Component with a custom property to specify security relevant information.
Our CSO wan't this information in Jira and we wrote a cronjob in backstage which export the data. But now Backstage needs to know a lot of stuff like, mapping between Jira fields and the Entity, credentials and URLs.
✌ Possible Implementation
Not sure if a simple CatalogProcessor would be enough, or if the Sticher could implement a similar interface where only updated entities get emitted.
👀 Have you spent some time to check if this feature request has been raised before?

[X] I checked and didn't find similar issue

🏢 Have you read the Code of Conduct?

[X] I have read the Code of Conduct

Are you willing to submit PR?
Yes I am willing to submit a PR!

No, you understand me wrong. We have some recesses that, for my opinion, don't belong into backstage (e.g. export service information into Jira). I just want the final entity to get pushed to Kafka, so that external systems could do things with other external systems.

@benkeil perhaps https://github.com/backstage/backstage/issues/8219 and https://github.com/backstage/backstage/pull/17174 would be of interest to you?


Make load_store_chunk args from store_chunk's args

Reuse the name generated for load_store_chunk's keys based off the keys used for store_chunk. Also tack on the arguments used for load_store_chunk after those used for store_chunk. Should avoid deviations between the two cases as the load_store_chunk case is merely an extension of the store_chunk case.

I may be confused, but neither of these changes seem to have any effect?

How do you mean? return_stored and load_stored can both be True.

I'm just looking at these lines that surround the changes here
Before
    name = 'store-%s' % arr.name
    func = store_chunk
    args = ()
    if return_stored and load_stored:
        name = 'load-store-%s' % arr.name
        func = load_store_chunk
        args = (load_stored,)

After
    name = 'store-%s' % arr.name
    func = store_chunk
    args = ()
    if return_stored and load_stored:
        name = 'load-%s' % name
        func = load_store_chunk
        args = args + (load_stored,)

I believe (though I could be mistaken) that they are semantically equivalent

It is more of a style change if that is what you are meaning. Was pulled from PR ( https://github.com/dask/dask/pull/3325 ), which I haven't had time to complete yet.

Planning on merging tomorrow if no comments.


Update Spring Boot Gradle plugin smoke test to 3.x

Currently, we are smoke testing the 2.x stream.
Also update documentation snippets and samples.

Note that this will require to run these tests with Java 17 as it is the minimal supported version.

Also note that there are snippets/samples currently using boot 2.x that should be updated too.

We're currently smoke testing version 3.2.1.


Add chainerx.hinge

Takeover #7616.

LGTM
Jenkins, test this please

Jenkins CI test (for commit 2e10e97bdf3e3358fee5b7f1b8d980d4f533b8e3, target branch master) failed with status FAILURE.

Jenkins, test this please

Jenkins CI test (for commit 2e10e97bdf3e3358fee5b7f1b8d980d4f533b8e3, target branch master) succeeded!


pull request for e2e test at 01/16/2017 23:26:14

:white_check_mark: Validation status: passed
For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Docs: duplicated requirements

Hey guys! Was looking at your wiki and noticed that on section Building From Source Code, while listing the prerequisites you've listed twice the following items:

Java 8
MongoDB

Since by default only collaborator's can contribute to a project's wiki, I can't make a PR about It. Keep on the good work!

Hey, thank you for pointing that, so we could fix the doc.
We appreciate the contact!


CopyDirectory - Missing Log information

When using the CopyDirectory would be nice to have a log of the Files copied.
This is being done in CopyFile for instance.

I will submit a Pull Request on this.

Fixed by #1447


Create a REST API with common commands

[ ] #26
[ ] #28
[ ] #29
[ ] #30

Closing in favor of API milestone.


Website - Use navbar instead of hotdog menu

Navbar menu can stick to the bottom of screen and can be opened all the times, even when scrolled to the bottom of a page.

lui_phone-navbar implemented; Site content structure not ideal tho


Implements display and prompt chunk options

This implements what was discussed in #39.
If you think this is the right way to go I can try adding tests and a doc entry.
P.S.: I think there is some code replication, but I like to have the flexibility to change how the display is implemented when display=true.

Coverage decreased (-2.7%) to 87.12% when pulling cffbb7ff5ca842e65a932dc7bbdeef51558f829d on gragusa:master into 9caf014c07bb35a3b963b8f1e83bedd65376f01a on mpastell:master.

Coverage decreased (-0.1%) to 89.725% when pulling 4a2062644b4c245c04648ee12848d191a2e7d76a on gragusa:master into 9caf014c07bb35a3b963b8f1e83bedd65376f01a on mpastell:master.

Thanks! Generally it looks good and thanks for adding the test as well. See my line comment above about adding another formatter.

I have updated run.jl as you pointed out in the inline comment.

Coverage decreased (-0.1%) to 89.725% when pulling 1d723099598363269d53bbf0dd9ca8c6b29940d9 on gragusa:master into 9caf014c07bb35a3b963b8f1e83bedd65376f01a on mpastell:master.

Thanks,
Can you explain why you'd like to have a separate  collect_results(chunk::CodeChunk, fmt::DispResult) method instead of just using the existing ScriptResult/CollectResult ?

This is what I mean when I said "there is some code replications". I was thinking about having more flexibility. After thinking about it, it seems that this flexibility will never be used. I will take care of it.

Coverage remained the same at 89.836% when pulling 1834b93cb350779537735a68f3dcce5010174c56 on gragusa:master into 9caf014c07bb35a3b963b8f1e83bedd65376f01a on mpastell:master.

Looks good, you can always add the flexibility back if there is a use case. I will merge this now, it would be great if you could also had the time to add an entry to the docs but I can also do it myself at some point.
Thanks again!


Enable scaling machinedeployments subresource

enable scale subresource so that `kubectl scale machinedeployment` would work

/lgtm
/approve
/retest

Ah, cool! Thanks for enabling this. I already tried to scale a machinedeployment and it didn't work.


Rename ID Docs "Scheduling: concept as "Scheduling and Eviction"

Rename docs/concepts/scheduling to docs/concepts/scheduling-eviction
Retitle concept header to "Penjadwalan dan Pengusiran"
Update redirects
Update internal links (id only)

Part of proposal #19081 and fix #26955
/assign habibrosyad

Hi @danninov.... this PR automatically closed #26955  :)