Reindexing scheduled on every "Go to Definition"

👓 What did you see?
Every time I "go to definition" of a step, a reindexing is scheduled which completely blocks VS code features, making it unusable.
✅ What did you expect to see?
Reindexing should happen when a step definition file changes, or on window reload or on demand. For larger projects I would prefer the indexation happening once when the editor starts and it can be reindexed manually with "Reload window".
📦 Which tool/library version are you using?
Visual Studio Code: 1.85.2
<EMAIL_ADDRESS><EMAIL_ADDRESS>

🔬 How could we reproduce it?
Open output channel for "Cucumber Language Server"
Right-click over a feature step and select "Go to Definition", alternatively click the step while pressing down cmd key.

Thanks for the report @josemigallas!
Could you advise:

In the attached gif, are you holding the command key and just hovering over the lines? Or also clicking?
The reindexing is scheduled via the Language Server on file changes, are there any other extensions you are using or extensions that may be triggering file saves or formatting while carrying out this action?

    // The content of a text document has changed. This event is emitted
    // when the text document is first opened or when its content has changed.
    documents.onDidChangeContent(async (change) => {
      this.scheduleReindexing()
      if (change.document.uri.match(/\.feature$/)) {
        await this.sendDiagnostics(change.document)
      }
    })

In the attached gif, are you holding the command key and just hovering over the lines? Or also clicking?

Holding the command key while hovering.

The reindexing is scheduled via the Language Server on file changes, are there any other extensions you are using or extensions that may be triggering file saves or formatting while carrying out this action?

I don't think so but here's the list just in case:
bierner.emojisense
bierner.github-markdown-preview
bierner.markdown-checkbox
bierner.markdown-emoji
bierner.markdown-footnotes
bierner.markdown-mermaid
bierner.markdown-preview-github-styles
bierner.markdown-yaml-preamble
castwide.solargraph
codezombiech.gitignore
CucumberOpen.cucumber-official
dbaeumer.vscode-eslint
denoland.vscode-deno
donjayamanne.githistory
eamodio.gitlens
elia.erb-formatter
fabiospampinato.vscode-open-in-github
firsttris.vscode-jest-runner
formulahendry.auto-close-tag
formulahendry.auto-rename-tag
hbenl.vscode-test-explorer
janisdd.vscode-edit-csv
kaiwood.endwise
LinusU.auto-dark-mode
LoranKloeze.ruby-rubocop-revived
ms-vscode.test-adapter-converter
ms-vsliveshare.vsliveshare
ms-vsliveshare.vsliveshare-pack
RobbOwen.synthwave-vscode
seatonjiang.gitmoji-vscode
shd101wyy.markdown-preview-enhanced
Shopify.ruby-lsp
sianglim.slim
sibiraj-s.vscode-scss-formatter
sissel.shopify-liquid
stevejpurves.cucumber
streetsidesoftware.code-spell-checker
stylelint.vscode-stylelint
SUPERTSY5.line-length-checker-vscode
Tyriar.lorem-ipsum
wix.vscode-import-cost

Thanks for detail @josemigallas - genuine bug!
Reason I could not initially reproduce, is once you click 'Go to Definition' and repeat the behaviour, it no longer continuously reindexes.
https://github.com/cucumber/vscode/assets/5904340/4652d4ee-902c-4dac-b880-582655d73924
We will at least need to reassess the event triggering the reindexing in the Cucumber Language Server to understand whether we can eliminate unnecessary reindexing calls. Feel free to take a look at the code as well in case you spot a fix - all inputs welcome.
Thanks again!

My pleasure!

Any update on this issue please? I have the same issue and reindexing takes about 45 seconds in my project, so it really would help a lot of this issue could be fixed. Thanks
Am using VSCode version 1.93, Cucumber 1.10, Cucumber with TypeScript, disabled all other extensions

Any update on this issue please? I have the same issue and reindexing takes about 45 seconds in my project, so it really would help a lot of this issue could be fixed. Thanks
Am using VSCode version 1.93, Cucumber 1.10, Cucumber with TypeScript, disabled all other extensions

Meanwhile this bug is fixed, I am using Cucumber (Gherkin) Full Support:
https://marketplace.visualstudio.com/items?itemName=alexkrechik.cucumberautocomplete
It has its own issues but it may work better for you.


fix : UI improvement for Tab Tracker

This is the current UI, from your creativity improve its colors or design or anything it's totally up to you!
Ps: You can change its icon also

While creating a PR, add a preview picture to view the changes.
Happy contributing 🎊

@harshita214 Can I work on this?

I want to work on this issue under JWOC

I want to work on this issue under JWOC

hey, @Lavanyabisht11 assigning issue to you. All the best 👍
@varunwho Please look for another issue

I want to work on this issue under JWOC

Okay, @DarkFalc0n assigning you. All the best 👍

Completed - #35


extend emitter scaleData Array

I easily extend the scaleData array by adding an other equation at the end/start of it. Seems to work pretty well as long as the live span of a particle is long enough to play trough the scaleData array.
Double checked the Phaser code to make sure it will not cause any errors. Seems good to me, can anyone please let me know if I'm right?? Rich?
P.S.
The emitter docs on the http://docs.phaser.io/Phaser.Emitter.html is broken, please fix. Thanks

What you are doing sounds fine to me @inkfood


bug: unnecessary pagination at the end of the tracks

Describe the bug
The pagination at the end of scrolling all the tracks do not mean any sense, also the pagination buttons do not work.
To Reproduce
Steps to reproduce the behavior:
https://www.loom.com/share/27a6f93c4ae142ceadf161323f51fc0b?sid=a4704d42-f1e7-4d93-a7cc-be505e9b8ae1
Expected behavior
Pagination should be removed
Info (please complete the following information):

Browser [chrome]
Version [129.0.6668.58]

Hey, i have done the pagination for tracks but when i commit those changes to github it shows husky -deprecated how can i solve this and make a PR

Pagination is already implemented here - #659 you can work on something else


Add calculate tokens helper

Add method that will calculate tokens a request will use for text completion and chat API.
This is useful when consuming the library in an application and needing to truncate prompt.
Help avoid each implementation using a different calculation.

Bit late to the party, but you can use TikTokenSharp for this (port of the tiktoken python library)


UCI Vehicle Charging Translation Update 2

Pull Request doc
Purpose
What is the larger goal of this change?
To add the file that contains code for the immediate charging scenario.
What the code is doing
How is the purpose executed?
This code generates charging profiles by iterating through NHTS census data and doing calculations with them.
Testing
How did you test this change (unit/functional testing, manual testing, etc.)?
The code has not yet been tested but we will test it by running it with the same inputs as the original MATLAB code and comparing outputs.
Where to look

It's helpful to clarify where your new code lives if you moved files around or there could be confusion/
The new code is located at PreREISE/prereise/gather/demanddata/transportation_electrification/immediate.py

What files are most important?
The file named "immediate.py" (this is the only new file in this PR)

Usage Example/Visuals
How the code can be used and/or images of any graphs, tables or other visuals (not always applicable).
Time estimate
How long will it take for reviewers and observers to understand this code change?
Should not take more than a week or two.

Could you please post a minimum-working-example of how you would use the main user-facing function(s), and what they return?

Could you please post a minimum-working-example of how you would use the main user-facing function(s), and what they return?

@josephxu1, you can edit the PR description and write fill out the section (Puropose, What is the code is doing, etc) and, in particular under the Testing section, give a code snippet and corresponding outputs as @danielolsen mentioned

Could you please post a minimum-working-example of how you would use the main user-facing function(s), and what they return?

@josephxu1, you can edit the PR description and write fill out the section (Puropose, What is the code is doing, etc) and, in particular under the Testing section, give a code snippet and corresponding outputs as @danielolsen mentioned

@danielolsen @rouille I will ask Kate for an example input/output case and edit the Pull Request to have that as soon as possible.

Sorry for all the weird target-changing; I think the target needs to be develop in order for new commits to trigger the automated workflow run, even though this will eventually get merged to the transportation-electrification feature branch.

Sorry for all the weird target-changing; I think the target needs to be develop in order for new commits to trigger the automated workflow run, even though this will eventually get merged to the transportation_electrification feature branch.

Sorry, have I not been doing that? I was under the impression that all the changes I'm committing were to the develop branch

@josephxu1 this isn't anything you need to worry about. Your commits have been to the branch named develop in your fork, but ultimately we will be merging the commits into the transportation_electrification branch in our repo, since this feature will likely have multiple PRs and we don't want to merge to our develop until everything is done. However, it seems like Github doesn't run the automated checks on code formatting, testing, etc. unless the PR target is our develop branch, so I've changed it back to that for the purpose of reviewing.

Closing for now following conversations with @isond, since we're expecting a new PR for the rewritten module.


Something familiar

https://github.com/AZ-X/WPF-GO-dnscrypt-proxy-md/blob/25af8f06346ca3ee677ba00a430d57d125714e44/stammel/stammel/minisign.go#L106-L165
:thought_balloon:

It almost costs more than 1k-LOCs other than go...
https://github.com/bitbeans/minisign-net/blob/7df7d878d615e46dfc88fbe30f7fe9b12f457c85/Minisign.Net/Core.cs#L724
https://github.com/bitbeans/minisign-net/tree/master/Minisign.Net/Models
sighing...
:hourglass: :hourglass_flowing_sand: :hourglass: :hourglass_flowing_sand: :hourglass: :hourglass_flowing_sand: :hourglass: :hourglass_flowing_sand: :hourglass: :hourglass_flowing_sand: :hourglass: :hourglass_flowing_sand: :hourglass: :hourglass_flowing_sand: :hourglass: :hourglass_flowing_sand: :hourglass: :hourglass_flowing_sand: :hourglass: :hourglass_flowing_sand: :hourglass: :hourglass_flowing_sand: :hourglass: :hourglass_flowing_sand: :hourglass: :hourglass_flowing_sand: :hourglass: :hourglass_flowing_sand: :hourglass: :hourglass_flowing_sand: :hourglass: :hourglass_flowing_sand: :hourglass: :hourglass_flowing_sand: :hourglass: :hourglass_flowing_sand: :hourglass: :hourglass_flowing_sand: :hourglass: :hourglass_flowing_sand: :hourglass: :hourglass_flowing_sand:


ra_syntax: can't parse weird-exprs.rs correctly

rust-analyzer reports lots of syntax errors on this code (extracted from the compiler + added ktulhu).

Excellent, thank you so much for pointing out this test!


[test] Use actual element over document.activeElement

While it's definitely a good idea to make sure the keyboard event target is the active element, it hurts readability in many cases. It's oftentimes not immediately clear what element is currently focused.
We now lint against fireEvent.key*(document.activeElement) while also ensuring that the target is focused. If it's not we throw and include the currently focused element.

I was wondering about it at some point. I liked how using document.activeElement fits closer to real-life usage. We get the guarantee that if an event moves the focus, we stay on the new element but don't like that we don't really know "where we are". No preference, it feels equality ok-ish in both cases. Maybe there is an opportunity for a fireEvent.activeElement method?

I think in almost all cases you shouldn't use document.activeElement because the code that tells you what's focused could move away (or isn't there in the first place). I know it feels like a chore to repeat querying the focused elements but when choosing between readability and writability you should prefer readability since most of the time programming is spent on reading.


No releases anymore?

Hi, are there no GH releases anymore? There were a lot of tags created during the last hours, but no corresponding GH Releases + Changelog. Is this intended?

I created the tags just for reference. Im waiting on @meatballhat to make the releases. Im not aware of how to create the change log for a release automatically
Get Outlook for iOShttps://aka.ms/o0ukef

From: Robert Kaussow @.>
Sent: Thursday, September 1, 2022 11:04:23 AM
To: urfave/cli @.>
Cc: Subscribed @.***>
Subject: [urfave/cli] No releases anymore? (Issue #1468)
Hi, are there no GH releases anymore? There were a lot of tags created during the last hours, but no corresponding GH Releases + Changelog. Is this intended?
—
Reply to this email directly, view it on GitHubhttps://github.com/urfave/cli/issues/1468, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AAYNLZWO5NPE3VBV3OT2NJTV4DAXPANCNFSM6AAAAAAQCOLIYI.
You are receiving this because you are subscribed to this thread.Message ID: @.***>

The problem is, tags will create a proper release of the package on gopkg and update mechanisms like dependabot, renovate or even go mod update will fetch those versions already. That's why creating a lot of tags "just for reference" without a real release is IMO not a good way in this context.

The v2.14.0 tag was removed?
https://github.com/urfave/cli/releases/tag/v2.14.0 is now 404

@dearchap hey I'm afraid I made the situation worse by not catching up on notifications before pushing a completely different tag 😐  I'm happy to try to clean this up, and I think I could also do a better job of describing the norms and process in docs/RELEASING.md.

The process I'm using goes like:
$ curl -s<EMAIL_ADDRESS>| jq -r .Origin.Hash
dac1c40ee53c6d210a69ec9b98b1e21f342af667
$ git tag -sam 'Release 2.11.5' v2.11.5 dac1c40ee53c6d210a69ec9b98b1e21f342af667
$ git push origin v2.11.5

I'm done backfilling annotated tags that match what's in the go package cache and creating github-level releases 👍🏼 Thanks your your patience, @xoxys!

Feel free to close the issue. Everything clarified from my side.


Meena Individual Review Ticket

Pranavi I.
Meena A.
Madhumita N.
Saumya P.
Natalie B.

Search Bar + Update Item
Filter
Create Account + Update Item
Display Marketplace
Profile + Purchase Item

Link to Search Bar, Link to Update Item
Link to Filter
Link to Create Account, Link to Update Item
Link to Marketplace
Link to Profile, Link to Purchase Item

Item
Self
Alice Tang
Rebecca
Avg

NATM Attendance
1
1
1
1

NATM Capture
1
1
1
1

Project quality
1
1
1
1

UI contains inputs
1
1
1
1

UI Create and Read
1
1
1
1

UI Update and Delete
1
1
1
1

JWT
0
0
0
0

Project Quality
1
1
1
1

API w/ error conditions
1
1
1
1

API GET and POST
1
1
1
1

Create and Read
1
1
1
1

Update and Delete
1
1
1
1

Individual Score for Runtime (binary criteria below, 5 points)

[ ] Project quality, running cross-domain and no crashes on FE to BE with these reviews
[ ] UI contains inputs that are validated for garbage, data cleaned (ie validate DOB or Password)
[ ] UI contains actions for Create, Read,
[ ] UI contains actions for Update, Delete
[ ] JWT is guarding a feature, demonstration of cookies

Individual Score for Code Review (binary criteria below,  5 points)

[ ] Project quality, observation of comments in every function, class, or method reviewed.  Be sure to review POJO, API Controller, and JPA.
[ ] BE contains and API that shows that it handles error conditions (ie duplicate user id)
[ ] BE contains support for GET and POST methods
[ ] BE contains a database that supports Create, Read
[ ] BE contains a database that supports Update, Delete

Link to Frontend Repo
Link to Backend Repo

Score: 18.5/20
(2/2)
(3/3)
(4/5) - No JWT protection
(5/5)
(2/3) - Video and blogs
(2.5/2) - Blogs, guides for review

18.5/20
Nice website with well planned out features
2/2
3/3
4/5   no JWT, can login even with incorrect password
5/5
2/3
2.5/2


VM loses generic function type after an await

void main() {
  doStuff<String>();
}

doStuff<T>() async {
  print(T);
  await null;
  print(T);
}

prints:
String
dynamic

This is with --preview-dart-2

out/DebugX64/dart --preview-dart-2 ~/bug.dart
String
dynamic
out/DebugX64/dart --reify-generic-functions ~/bug.dart
String
String
So this is an issue when using fasta/kernel, and not an issue in the VM runtime.
I suspect that the local variable holding the type arguments in the async closure of function doStuff is not captured as it should.
I am reassigning this issue to area-kernel.

It feels to me that there's something wrong in the VM.  The compiled Kernel for doStuff looks as I expected:
static method doStuff<T extends core::Object>() → dynamic /* originally async */ {
  final asy::Completer<dynamic> :completer = asy::Completer::sync<dynamic>();
  asy::FutureOr<dynamic> :return_value;
  dynamic :async_stack_trace;
  dynamic :async_op_then;
  dynamic :async_op_error;
  dynamic :await_jump_var = 0;
  dynamic :await_ctx_var;
  dynamic :saved_try_context_var0;
  function :async_op([dynamic :result, dynamic :exception, dynamic :stack_trace]) → dynamic yielding
    try {
      #L1:
      {
        core::print(self::doStuff::T);
        [yield] let dynamic #t1 = asy::_awaitHelper(null, :async_op_then, :async_op_error, :async_op) in null;
        :result;
        core::print(self::doStuff::T);
      }
      :completer.{asy::Completer::complete}(:return_value);
      return;
    }
    on dynamic catch(dynamic :exception, dynamic :stack_trace) {
      :completer.{asy::Completer::completeError}(:exception, :stack_trace);
    }
  :async_stack_trace = asy::_asyncStackTraceHelper(:async_op);
  :async_op_then = asy::_asyncThenWrapperHelper(:async_op);
  :async_op_error = asy::_asyncErrorWrapperHelper(:async_op);
  asy::Future::microtask<dynamic>(:async_op);
  return :completer.{asy::Completer::future};
}

Note two occurrences of self::doStuff::T.    I suspect that it just needs to be added to the list of variables that are always captured in StreamingScopeBuilder::VisitFunctionNode`` in runtime/vm/compiler/frontend/kernel_binary_flowgraph.cc`.

By not being an issue in the VM runtime, I meant that everything works if you do not use kernel.
But since flow graph building from kernel is now part of the VM (not runtime per se), yes, it is a VM bug.
However, I am not familiar at all with how kernel scopes are being processed and how variables referred to by local functions get marked as captured. Note that the reference to function_type_arguments_var in this case is somewhat hidden, because it is accessed to instantiate the type parameter. This case is handled in the non-CFE parser by the call CaptureAllInstantiators() or sometimes just CaptureFunctionTypeArguments().
I see code in StreamingScopeBuilder::BuildScopes() creating the :function_type_arguments_var variable. The question is how does this variable get marked as captured when accessed from a local function (from the async closure of doStuff)?
  // Add function type arguments variable before current context variable.
  if (I->reify_generic_functions() &&
      (function.IsGeneric() || function.HasGenericParent())) {
    LocalVariable* type_args_var = MakeVariable(
        TokenPosition::kNoSource, TokenPosition::kNoSource,
        Symbols::FunctionTypeArgumentsVar(), AbstractType::dynamic_type());
    scope_->AddVariable(type_args_var);
    parsed_function_->set_function_type_arguments(type_args_var);
  }

The hacky function LocalScope::CaptureLocalVariables mentioned by Slava is probably irrelevant here, since it is only called in particular cases (this should be cleaned up) and the variable should get captured by a call to LocalScope::CaptureVariable.
If I understand correctly, StreamingScopeBuilder::BuildScopes() visits all nodes of the graph, but fails to notice that the :function_type_arguments_var is accessed to instantiate a type.
I can continue digging, but once again, this code is totally new to me. Let me know.

@crelier sorry, I should have mentioned explicitly that I have reassigned the bug to myself and I have a fix.
https://dart-review.googlesource.com/c/sdk/+/53664

Ah, I see. It is not that visiting the graph failed to capture the variable, but the graph was just wrong.
Thanks for fixing.

@crelier there are two ways to fix it - I actually first implemented a different approach with capturing it, but it seems wasteful because the only case when it can be captured is the case where it can be simply reloaded from the closure field.

it looks like in more complicated case error is still happening, at least in dev 53 version:
import 'dart:async';
import 'dart:convert';

void main() async {
  ServerResponse<SigninResult> result = await post<SigninResult>(
    "/signin",
    SigninResult.fromJson,
  );
  print(result);
}

typedef Converter<S> = S Function(Map<String, dynamic>);

class SigninResult {
  SigninResult();

  static SigninResult fromJson(Map<String, dynamic> json) {
    return null;
  }
}

class ServerResponse<S> {
  final S result;
  final dynamic exception;

  ServerResponse.success(this.result) : exception = null;

  ServerResponse.fromException(this.exception) : result = null;
}

Future<ServerResponse<S>> post<S>(
  String url,
  Converter<S> converter,
) async {
  ServerResponse<S> response;
  try {
    //when this line is inlined (comment this out and uncomment two following lines)
    //everything works fine
    String body = await doServerCall();

//    await openUrl();
//    String body = "some response";

    response = ServerResponse.success(
      converter != null ? converter(jsonDecode(body)) : null,
    );
  } catch (e) {
    response = ServerResponse.fromException(e);
  }
  return response;
}

Future<String> doServerCall() async {
  await openUrl();
  return "some response";
}

Future<Null> openUrl() {
  throw Error();
}

The fix is not in dev 53.

Nope. You can open the commit (
https://github.com/dart-lang/sdk/commit/f35bb0379d249e197c45b94fbbf52c5d5e8bf019)
and look at the top – you'll see the tags the commit is in...
On Mon, May 7, 2018 at 4:59 PM Keerti Parthasarathy <
<EMAIL_ADDRESS>wrote:

The fix is not in dev 53.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/dart-lang/sdk/issues/33025#issuecomment-387242173,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AABCip9pgohDYE2e5yYOWd4w-sTbESfOks5twN_JgaJpZM4TwDBl
.

if the fix is in dev 53, then what I have is different issue, right?

@radzish – the fix is not in -dev.53. Likely in -dev.54

@radzish could you also describe what you think is a bug? I run your code and I get Instance of 'ServerResponse<SigninResult> in both cases. Is this what you expect or not?

@radzish perfect. that means that the issue is fixed on master. the fix should reach Flutter next time we roll Dart into it, which usually happens pretty regularly. the schedule is not tied to Dart SDK dev releases at the moment.


Uninstalling Rocket Mod.

Hello. So the other night, I downloaded Rocket Mod. I opened up Unturned and it was saying that I couldn't play servers that didn't have Rocket Mod and it only gave me 2 options. Go to the wiki for help or to close the game. How do I go back to as if I never downloaded Rocket Mod in the first place?
I deleted the Rocket folder in my server, but I can't join any server now, not even my own. How do I fix this? I'm assuming a get a clean Managed folder without the Rocket files but I don't know how I would go about doing that. Please help?
Thanks for reading and have a nice day/night

right click unturned in your steam library > click propperties > click local files > click verify integrity of game cache.
also close unturned before doing this


update: upgraded to oras-go v2.0.0-rc.5

This PR upgrades notation-go to use oras-go v2.0.0-rc.5.
Main change: ReferrerFinder is added into registry.Repository interface.
Signed-off-by: Patrick Zheng<EMAIL_ADDRESS>

Codecov Report

Merging #209 (fa7b181) into main (2573c88) will increase coverage by 0.05%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main     #209      +/-   ##
==========================================
+ Coverage   65.29%   65.35%   +0.05%
==========================================
  Files          23       23
  Lines        1576     1570       -6
==========================================
- Hits         1029     1026       -3
+ Misses        477      475       -2
+ Partials       70       69       -1

Impacted Files
Coverage Δ

registry/repository.go
32.20% <100.00%> (-1.65%)
:arrow_down:

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


Fix - Handle Routing Event - Don't Unmount If Navigation is Cancelled

This PR addresses a bug brought up in Issue 151 and Issue 180.  The bug is around the current app mounted on the screen disappearing when navigation is cancelled while using the single-spa-layout engine.
This issue can be reproduced in this forked repository by trying to navigate from application2 to application1.
The fix was to add a small logical check to the handleRoutingEvent function in the constructLayoutEngine file.  Previously this function was always unmounting the application regardless of navigation state. This change only unmounts the application if navigation is not cancelled.  If navigation is cancelled, the app currently mounted on the screen will remain mounted.
Here is a small gif demonstrating the fix while tracking the window events that the single-spa-layout engine subscribes to:

Released in https://github.com/single-spa/single-spa-layout/releases/tag/v2.1.1


Local search with noIndex: true [feature request], was: No search results on local build

I installed the plugin per the instructions.
I do not have a blog directory on my site.
My config looks like this:
      [
        require.resolve("@easyops-cn/docusaurus-search-local"),
        ({
          hashed: true,
          highlightSearchTermsOnTargetPage: true,
          explicitSearchResultPath: true,
          indexBlog: false,
          indexPages: true
        })
      ]
    ],

Then I ran the build & started the server with npm run serve to serve from the build directory. Every search query returns empty ("no results").
I also ran the build with debug but saw no errors.

The cause was the noIndex option in my config. The site is not for public consumption so I was discouraging search engines from crawling the site. So I'm turning this into a feature request: can we have an option to use noIndex: true but also have internal search work?

V0.38.0 introduced the feature: ignore unlisted context (the noIndex option has the similar effect).  Your can try previous versions to see it it works for you.
Maybe we can add an option to force enable local search even if noIndex is set.

Maybe we can add an option to force enable local search even if noIndex is set.

Please do!


Trying to use cargo chef with ekidd/rust-musl-builder

Hello,
Thank you for the great work..
I have an issue I am not sure how to debug to find what is the problem.
I have the following docker file:
FROM lukemathwalker/cargo-chef as planner
WORKDIR app
COPY . .
RUN cargo chef prepare --recipe-path recipe.json

FROM lukemathwalker/cargo-chef as cacher
WORKDIR app
COPY --from=planner /app/recipe.json recipe.json
RUN cargo chef cook --release --recipe-path recipe.json

FROM ekidd/rust-musl-builder:latest AS builder
COPY --chown=rust:rust . .
# Copy over the cached dependencies
COPY --chown=rust:rust --from=cacher /app/target target
COPY --chown=rust:rust --from=cacher /usr/local/cargo /home/rust/.cargo
# RUN cargo build --release --bin back-office

FROM alpine:latest as runtime
WORKDIR app
COPY --from=builder /home/rust/src/target/x86_64-unknown-linux-musl/release/back-office /usr/local/bin
EXPOSE 8585
ENTRYPOINT ["/usr/local/bin/back-office"]

Here, back-office is the name of my package in cargo.toml.
This kind of works ... in the sense that, once the first docker build occurs, if cargo.toml isn't changed, the dependencies aren't downloaded again (the updating crate index ... does not happen at all).
However, the dependencies are compiled at every docker build. I tried many things, because from my point of view what is possibly happening is that the cargo on ekidd/rust-musl-builder takes in consideration stuff on cargo_home but does not recognize stuff coming from targedt from the cacher.
Do you have any hint on how I can fix that so that it works just like if I was using the rust image as builder?
Thanks,
Marlon

Hi @marlon-sousa!
To help you troubleshoot I need to have a minimal reproducible example of the issue - can you upload a repository with code that runs into the problem you mentioned when building the Docker container?

Without a minimal reproducible example I am afraid I can't move forward here.
Closing for the time being.


[qtdeclarative] Build error on x64-windows

To reproduce:
vcpkg x-set-installed qtwebengine[core]
You get:
Installing 38/40 qtdeclarative:x64-windows...
error: The following files are already installed in G:/v/vcpkg1/vcpkg_installed/x64-windows and are in conflict with qtdeclarative:x64-windows
Installed by qttools:x64-windows
tools/Qt6/bin/libcrypto-3-x64.dll

Elapsed time to handle qtdeclarative:x64-windows: 22.4 ms
Please ensure you're using the latest port files with `git pull` and `vcpkg update`.
Then check for known issues at:
  https://github.com/microsoft/vcpkg/issues?q=is%3Aissue+is%3Aopen+in%3Atitle+qtdeclarative
You can submit a new issue at:
  https://github.com/microsoft/vcpkg/issues/new?template=report-package-build-failure.md&title=[qtdeclarative]+Build+error+on+x64-windows
Include '[qtdeclarative] Build error' in your bug report title, the following version information in your bug description, and attach any relevant failure logs from above.
    vcpkg-tool version: 2999-12-31-unknownhash
    vcpkg-scripts version: 7039df020 2023-11-21 (8 hours ago)

@autoantwort I'm not able to repro this issue, I tried to install qttools through qttools: x64 Windows and qttools [*]: x64 Windows, but I couldn't find the conflict file tools/Qt6/bin/libcrypto-3-x64.dll.

strange :)

Not so strange: Installation order and artifact caching. Note that tools/Qt6/bin is a shared folder, not a per-port folder.
I would assume that libcrypto-3-x64.dll is a transitive dependency via qtbase runtime. All runtime in tools/Qt6/bin related to qtbase should probably be installed and owned by qtbase - even if not used by qtbase directly. Otherwise the ownership is taken over by reverse dependencies of qtbase, subject to installation order. And then you will get clashes it packages are restored from artifact cache.

But in the binary cache is only the content of the packages folder which is not shared. So the clash should be reproducible or there should be none in the first place.


Remove app packaging services text

Before:

After:

I decided to merge this. @kentonv may have an opinion fwiw!

(Deploying now)


for loops benchmarks

Looks great!
Could you remove the real_logs folder and the all the package.json?

There are more big files,  e.g., evaluation/benchmarks/for-loops/pMir/other/10000.txt and some files in pacaur.

There is one more big file:  evaluation/benchmarks/for-loops/pacaur/packages

There is one more big file: evaluation/benchmarks/for-loops/pacaur/packages
Fixed

I haven't included any of the datasets for nginx and pcap analysis.
@angelhof Could we upload the datasets for those two (and potentially other big files) to http://ndr.md/ server (since I don't have the original links to the data). This doesn't need to be done right now (I have the files in all the machines, so I can run the benchmarks for now).

I say we put all non-negligible files on ndr. I don't see why we should have any of them in the PR.
Furthermore, there is a 1.5 million line file that you added with your last commit. Could you remove it? We have gone through this PR 3 times and there are still big files left. Let's just remove all big files, add them to ndr, and then merge this PR.

I think this is ready. We will add the datasets from the ndr in a different PR.
Merging to future


Multi-chart slider and colors for individual bars

RangeSlider now works with multiple charts and a single slider, by using an array with the graph variables:
var slider_three = new Rickshaw.Graph.RangeSlider({
    element: document.querySelector('#slider-range'),
    graph: [graph_one, graph_two, graph_three]
});
The bar chart renderer now looks for a proposed color for each individual bar, it will default to the chart-specific color if no color is declared:
var data = [{
  data: [{
    x: -1893456000, y: 29389330}, {
    x: -1577923200, y: 33125803}, {
    x: -1262304000, y: 37857633, color:'#06f'}, {
    x: -946771200, y: 41665901, color:'#0cf'}
  ],
    color: '#222'
  }]

@bozdoz thanks! i moved your changes from the built file rickshaw.js into the src and tested them out you can now use them if you open the shared slider example:  https://github.com/shutterstock/rickshaw/blob/master/examples/shared_slider.html
closing this PR :)


Dependence on cosmiconfig for @kubb/cli throws breaking error in es module envrionments

What version of kubb is running?
kubb/1.14.6 darwin-arm64 node-v18.16.0
What platform is your computer?
MacOS
What version of external packages are you using(@tanstack-query, MSW, React, Vue, ...)
"@kubb/cli": "^1.14.6",
"@kubb/swagger": "^1.14.6",
"@kubb/swagger-client": "^1.14.6",
"@kubb/swagger-faker": "^1.14.6",
"@kubb/swagger-ts": "^1.14.6",
"@kubb/swagger-zod": "^1.14.6",

What steps can reproduce the bug?
The @kubb/cli relies on cosmicconfig to resolve config files, cosmic config doesn't yet support es modules (afaik), so you get the [ERR_REQUIRE_ESM]: require() of ES Module not supported error when it tries to require your config.
Steps to repro:

Create a kubb.config.js file in a project with type: module defined in package.json
Try and run kubb
See error

How often does this bug happen?
Every time
What is the expected behavior?
Kubb CLI should be able to run in es modules
Swagger/OpenAPI file?
No response
Additional information
No response

Maybe this could help you out: https://www.kubb.dev/configuration/configure#typescript-javascript.
If type: module is being used you need to use an import syntax, you can rename the file to kubb.config.cjs and that should then use CJS instead.

@stijnvanhulle I am using es imports everywhere, but cosmic uses require when loading the config file, so if it uses es import/export syntax (like the docs suggest) it throws

@madi-tracksuit Tried it with the following codesandbox but seems to work fine here, can you provide me a codesandbox/fork with the issue?
codesandbox example

Hmm okay sorry for the false alarm! Thought it could have been our monorepo setup but forked that codesandbox and still couldn't repro. Assuming gremlins our side, will close this one out

Just a quick follow up, turns out I was hitting this issue https://github.com/cosmiconfig/cosmiconfig/issues/312
There's an error somewhere in my config (still figuring that one out...) and cosmic was throwing the generic require error


Example for create-react-app

Hi,
Great project. I would like to use itk.js with my create-react-app project.
I am having problems configuring the react-scripts webpack.config.js to get things working.
Would it be possible to also add an example that works when using create-react-app ?
This would be huge !
Thanks so much,
Lorenz

Hi Lorenz,
Thanks for the kind words.
Good idea -- yes, I will add a create-react-app example.
Thanks,
Matt

This would be absolutely great Matt, thanks so much for putting this together! After trying so many clunky things in JS and with WebGL, this would be a real game-changer.

Love this request, a create-react-app example would be amazing!!
Keep up the great work!

CC @floryst

Work in progress is here:
https://github.com/InsightSoftwareConsortium/create-itk-app
It still needs an example pipeline.


Fix typos in CellMeasurer example modules

Hi there,
First off - great work on this library, and thanks for the excellent documentation and examples!
I noticed a few minor typos when looking at the examples, and figured I'd ping you with a quick fix.
Thanks 🙃

Thanks


Request for support iOS 16.3.1

Is it possible to raise the platform support to iOS 16.3.1? So we can have this useful tool on this version as well?
p.s. how can I make the deb file by myself to test if is it working on ios 16.3.1 or not?

Hey! I have the version capped at 14.x.x as the jailbreaks for versions greater than that are largely rootless, which I have yet to add support for. If you are using a rootful jailbreak on iOS 16.3.1, this version cap can likely be removed, at your own risk, but your jailbreak's bootstrap repo(s) will have to be added to the filter list in IALBackupManager.m. Additionally, the method for running (un)packaging commands with setuid 0 may not work on newer versions, in which case that implementation will also have to be updated.
I plan on adding support for rootless jailbreaks and higher versions once the iOS 15 jailbreaks are in a stable state. My only device is currently sitting on iOS 14 and I don't plan on updating it until the jailbreaks' issues have been sorted and I can trust that they won't change too drastically. Said device is capped at iOS 15.x.x, but I don't foresee that posing too many issues for adding higher versions support as IAmLazy isn't exactly version dependent.
If you want to give packaging it a shot, you'll need Theos and an NSTask.h header.
Closing this in favor of #30. Thanks for your interest and support!

Thanks for your effort. IAmLazy is an excellent tool indeed.
I'm not a specialist at tweaking can you help me with the testing procedure?
I tried to make a deb from your code. I had Theos but it gives me the NSTask error:
`Managers/IALGeneralManager.m:13:9: fatal error: 'NSTask.h' file not found
How should I import the NSTask package?
(I tried modifying the 'control' file in the 2.1.2 deb and raising the firmware depend from 15.0 to 16.4! and after installing it with Filza doesn't give me an error but nothing appeared on the Home Screen.)
[  I've iPhone X at 16.3.1 with Palera1n rootfull jailbreak.]

You'll have to get the NSTask header from a header site such as the one available at headers.cynder.me or developer.limneos.net and place it in $THEOS/include.
If you're not familiar with development, I would advise waiting for the official update.


Update adding project docs

What this PR does / why we need it:
Which issue(s) this PR fixes:
Fixes #
Does this PR introduce a user-facing change?:

NONE

/lgtm


Comparison of String and dynamic (function) fails with uncaught error in core_ajax

This issue was originally filed by<EMAIL_ADDRESS>
What steps will reproduce the problem?
Please include any steps we need to take to reproduce the problem. The
best way to help us investigate the issue is to provide a small example
(preferably a file or two) that we can copy & paste into an empty polymer
project and try it out.
What is the expected output? What do you see instead?
Please include the following information:
- Dart SDK version:
- polymer package version:
- Operating system:
- Browser (if any):

This issue has been moved to dart-lang/polymer-dart#300.


Reload PGN button doesn't have translation

The Analysis Reload PGN button has text and title 'reloadPGNText' and 'reloadPGNTitle' respectively.

Will be fixed with V2.0.6


Fix#2635, mitmdump does not apply filter to saved data

This bug is caused by a missing condition. Now we only write the stream to file when save_stream_file is updated, therefore the save_stream_filter can not be applied to the output file.

Thanks, @MatthewShao! 😃


TestCafe tests fails while upgrading to v1.14.2

What is your Test Scenario?
While trying to upgrade TestCafe dependency to the latest version v1.14.2, tests are consistently failing with Too long with no output (exceeded 10m0s): context deadline exceeded.
Firstly, I tried to up the resource class for CI assuming it could be due CPU/memory allocation and later discovered that tests are failing on my local machine too with same timed-out issue. Tried further by upgrading to even newer version v1.15.0-rc.1 and got same failing result. 🤷♀
I can not find anything in the release notes that could make these tests fail. The only question is: why are they failing? Is there anything else I can try?
What is the Current behavior?

Tests fails with timeout-related issue.

What is the Expected behavior?

Tests should pass as they are in v1.14.0

What is your web application and your TestCafe test code?

Your complete configuration file (if any):
{
	"src": "tests/e2e/tests/*.tsx",
	"browsers": ["chrome:headless"],
	"disableMultipleWindows": true,
	"concurrency": 5,
	"skipJsErrors": true,
	"color": true,
	"pageLoadTimeout": 20000,
	"selectorTimeout": 20000,
	"assertionTimeout": 20000,
	"skipUncaughtErrors": true,
	"screenshots": {
		"takeOnFails": true,
		"fullPage": true,
		"path": "tests/e2e/screenshots",
		"pathPattern": "${DATE}/${TEST}.png"
	},
	"reporter": [
		"spec",
		{
			"name": "xunit",
			"output": "reports/testcafe/test-results.xml"
		}
	]
}

Screenshots:

Your Environment details:

testcafe version:                   v1.14.2
node.js version:                    v14.16.0
command-line arguments:  testcafe -q -F Smoke tests/e2e/tests/*
browser name and version: HeadlessChrome 91.0.4472.77 / Linux 0.0
platform and version:          locally on Mac, and Circle CI v2.1

You can try to loop through following sample code.

import { Selector } from 'testcafe';

fixture(`Getting Started`).page(`http://devexpress.github.io/testcafe/example`);

test('My first test', async (t) => {
    await t.typeText('#developer-name', 'John Smith').click('#submit-button');
});

I was not able to reproduce this behavior.
Maybe I misunderstood your suggestion regarding a loop:
import { Selector } from 'testcafe';

for (let i = 0; i < 80; i++) {
    fixture(`Getting Started`).page(`http://devexpress.github.io/testcafe/example`);

    test(`Test #${i}`, async (t) => {
        await t.typeText('#developer-name', 'John Smith').click('#submit-button');
    });
}

Would you please send an example where the behavior you are talking about is reproduced?

Same thing -- Have a fairly large test suite that was running fine. Upgraded to 1.14.2 (from 1.14.0) and started getting a bunch of these failures. Thought it was on our end, but then downgraded to 1.14.0 and all was well again.

We didn't get any reproducible example from @vrunda-s. So, I'll close this issue.
@duhseekoh
If this issue can be reproduced with the latest TestCafe v1.15.0, please open a new issue with an example that we can run on our side.


client: implement the Store type

A Store wraps an API client to give the caller a transparent way to get secret
values that automatically stay up-to-date as values change on the server.

Added tests for the store (though not for the underlying Client—I'd prefer to do that separately if that's OK).


Remove iOS/Android from SauceLabs testing

iOS/Android is heavy in SauceLabs.
These are always waited for available.

closed by https://github.com/almin/almin/commit/47adcd21b4de64710f14650c72e2f36c480f4f73


MetaPath

If I click Run Pathway analysis, there will be an error message.
Parameters:
log transformed: FALSE
filtering: 0.6, 0.6
Others: default

I guess this is again the issue with the version of MetaPath you installed, it seems there is an error reported when I tried to install MetaPath from MetaOmics homepage. Let me open another issue for that.

It seems I am able to install MetaPath fromMetaOmics homepage after removing it.
I will further test and if there is no problem, I will close this issue.

Oops, the following small bug about create working directory:
After I click "Run Pathway analysis"

I am trying to resolve it now since I encounter the same problem (seem not a problem before I reinstall MetaPath), it seems previously it requires "/", but now adding a "/" will give you double slash ...

I just fixed the bug on feature/metaQC branch, please try it out.


Newcomer documentation improvements

Several things can be done to improve the experience for newcomers to the OmniBox website.

Add a few, small pictures near the top of the landing page to help with reader visualization. The only current pic is at the bottom and a render.
Add a version history/release log page for an at-a-glance update.
Explain stability guarantees between versions.

Oops. Didn't auto-close.


Checkbox consistency detailed design PR

@YuliKl and @chrisglein FYI. I've updated the spec with latest visual agreement with the design.

Will the Unselected Outline styles be made consistent with the Outline Button styles, if the proposed Button styles are implemented? It appears as if the Filled styles will match the Primary/Accent button styles, including pressed and hover styles.
microsoft/microsoft-ui-xaml#1054

@mdtauk , I do not believe they will be made exactly the same, though they should feel coherent. That said, the actual outline style is not being looked at right now so I'm unable to answer with confidence. If you have a passion, why don't you propose checkbox/button outline style that aligns so we can take a look, perhaps?

@mdtauk , I do not believe they will be made exactly the same, though they should feel coherent. That said, the actual outline style is not being looked at right now so I'm unable to answer with confidence. If you have a passion, why don't you propose checkbox/button outline style that aligns so we can take a look, perhaps?

I posted some thoughts in the Proposal I mentioned
Should there be Outlined style and Filled style for all controls including CheckBoxes?  I really don't know to be honest.

Windows Phone 7 compared to Windows Phone 8
Both these styles have existed in the past, but were there good reasons one was swapped for the other before?
Android has Outlined and Default text fields.

Fabric/Fast DNA has Outlined, Default, and Primary buttons.

It strikes me that outlined buttons, with a 1px border, would feel a lot more light weight on a form or page.  But Filled buttons may stand out a bit better.  I don't have a firm preference for one or the other, only that Fast DNA includes all three styles, and Windows UIs have moved between filled and outlined control designs in the past, so there is scope to include both styles - if there can be a simple way to switch for devs.
And what the Windows Shell's pick, will probably influence developers choices - but until the modern Shell is in dev's or user's hands - its all just discussion and hope that the design vision comes together.


blip-2 training freeze image encoder, but finetune does not freeze it, why is that?

Can you please help understand the difference here?
AFAIK, finetune shouldn't be much different with training, it is kind of training using specific distributed data, right?

Finetuning the image encoder generally leads to improved performance on a particular downstream task.


Check the validity of a preimage

When new pre-image is revealed, it must be checked if it's the right pre-image for a current pre-image stored in the validator set. This PR adds the isInvalidPreimageReason function checking the validity of a preimage and makes the validator set use the function in the addPreimage function.
Relates #694

Ready for review.
I applied the suggestion to the code and dealt with the problem related to the height of an preimage.


Analysis and retraining the model via the Visualization Dashboard

@yongtang @greenpau
I tried integrating model retraining in the visualization dashboard app, however, the application becomes very slow. Hence I think a suitable workaround can be:

Creating a python program that runs in the background waiting for instructions from the visualization dashboard and once it receives the go-ahead from the user, the model retrains in the background and then the data regarding training such as the accuracy as well as loss can be communicated back to the dashboard. Although this communication can be done via https requests, a faster way would be to communicate via the elasticsearch database, since both the programs would be run on the same local machine.

Please let me know if this sounds good :)

@cekbote , I think sending "jobs" to elasticsearch and having separate app reading the "jobs" from elasticsearch for re-training works.

Cool I'll work on that then :)

The frontend of the model training via the dash app can be seen below:
Added the ML Bridge logo (To make it look professional 😄 ):

The Accuracy Graph:

The Loss Graph (The graphs can be toggled via a toggle switch [surrounded by a blue box]):

Confusion Matrices for Training, Testing and Validation:

Confusion Metrics for Training Testing and Validation:

A large part of the backend is yet to be done (The front end and logo design took more time than I anticipated)


joint IC and SF few-shot dataset

Hi Yutai,
first of all, thank you for the great tools for a few-shot generation. when I went through the code I noticed there are three types of task (sl, sc  and slu). that means if I want to have a few-shot dataset for a joint IC and SF task I should use the task type slu.
but when I ran the code with slu I got the error that  "--task: invalid choice: 'slu' ".

my question is that if your tool is applicable for such a task or not ? If yes what else should I consider.

another question is regarding the cross validation for splitting the test/dev/training set.  my understanding from your approach was that for training, you use support-set of each episode to train and query-set to you evaluate your model( calculating loss between the correct label and the predicted label). this process will be done for 100 episodes and at the end you will choose the best model( according to score) from the 100 trained model. for testing on target domain (unseen data) you will again evaluate according the query set in test set and average the result of 100 episodes.

I want to know, if there is a reason you didn't use at least one of the seen domain in the test or dev set to evaluate both seen and unseen data.

If I understood the concept of the episodic training --> support set for training and query set for evaluation and choosing the best model . then why do we need the dev set? In ML approached we use train set to train and dev to evaluate the model while training.

I would be really glad if you can correct me if I missed a part.
thank you in advance

For question 1, we had provided a example script scripts/gen_meta_data.sh, you can run it after changing the value of two parameters in that script called input_dir & output_dir. I can run it successfully, so I suggest you can refer this scirpt for your error.

For question 1, we had provided a example script scripts/gen_meta_data.sh, you can run it after changing the value of two parameters in that script called input_dir & output_dir. I can run it successfully, so I suggest you can refer this scirpt for your error.

thank you for your fast reply, I have followed the steps in https://github.com/AtmaHou/MetaDialog/issues/8#issuecomment-717841510.
but there are some changes to be considered:

the path to config file are missing in scripts/gen_meta_data.sh.  so I added a for loop for the configuration in the  scripts/gen_meta_data.sh and also add the argument --eval_config  to refer the config path and --eval_config_id for reading the configs.

then I got the "raise RuntimeError('Error: test_labels and dev_labels are  when split_basis is label')
RuntimeError: Error: test_labels and dev_labels are  when split_basis is label"
to solve it : line 107  for old_domain_name, old_domain in raw_data ---> for old_domain_name, old_domain in raw_data.items() and
" with open(opt.eval_config, 'r') " --->  "with open(opt.eval_labels, 'r') "

For question 1, we had provided a example script scripts/gen_meta_data.sh, you can run it after changing the value of two parameters in that script called input_dir & output_dir. I can run it successfully, so I suggest you can refer this scirpt for your error.

thank you for your fast reply, I have followed the steps in #8 (comment). but there are some changes to be considered:

the path to config file are missing in scripts/gen_meta_data.sh.  so I added a for loop for the configuration in the  scripts/gen_meta_data.sh and also add the argument --eval_config  to refer the config path and --eval_config_id for reading the configs.
then I got the "raise RuntimeError('Error: test_labels and dev_labels are  when split_basis is label')
RuntimeError: Error: test_labels and dev_labels are  when split_basis is label"
to solve it : line 107  for old_domain_name, old_domain in raw_data ---> for old_domain_name, old_domain in raw_data.items() and
" with open(opt.eval_config, 'r') " --->  "with open(opt.eval_labels, 'r') "

Good point~ Can you provide a pull request for the changes? Then we can add you as contributors :)

For question 2.1, it's a great question and a popular research direction called Catastrophic Forgetting which is not the kernel problem we want to solve in this work. It's interesting and important to overcome the catastrophic forgetting in few-shot learning, and we hope we can do some meaningful work with this direction in the future.
For question 2.2, it's useful to split dev & test datasets, dev dataset is mainly used for selecting the best model (or called checkpoint) with the domain information of dev dataset, the dev score cannot represent the performance in other domains, so we need a random test dataset to test the true performance.

If we only consider one intent for test split, then predicting the correct intent wouldn't be difficult as it has only seen that specific intent in the support set of test split. but if we have at least 2-3 different intents there is a fair evaluation. don't you think this is a kind of bias?

'''
question2. I want to know, if there is a reason you didn't use at least one of the seen domain in the test or dev set to evaluate both seen and unseen data.
question3. If I understood the concept of the episodic training --> support set for training and query set for evaluation and choosing the best model . then why do we need the dev set? In ML approached we use train set to train and dev to evaluate the model while training.

For question 2: We follow the classical setting of few-shot learning, where models are tested on domains with unseen labels. We just choose a prevailing setting and your mentioned setting may be an interesting extension.

For question 3: Similar to question 2, using dev domains is also a common choice. This can help to select a model that performs well on unseen domains, which is consistent with the evaluation settings mentioned in question 2.


Auto discover unix linker path and setup Android NDK linker

Change default behavior when IREE_LLVMAOT_LINKER_PATH isn't set for unix host and target systems:

For unix host & target search for system linker path.
For android target set up linker path using ANDROID_NDK variable.

This should close issues #2391 and #2645


[Bug]: Regular expression with toJson method panics

Description
The toJson method is allowed in anydata types where a non-json types are converted to a respective json compatible types according to the method definition:
# This does a deep copy of parameter `v` converting values that do
# not belong to json into values that do.
# A value of type `xml` is converted into a string as if
# by the `toString` function.
# A value of type `table` is converted into a list of
# mappings one for each row.
# The inherent type of arrays in the return value will be
# `json[]` and of mappings will be `map<json>`.
# A new copy is made of all structural values, including
# immutable values.

But the regexp:RegExp with toJson method panics at runtime:
error: {ballerina/lang.value}ConversionError {"message":"'anydata & readonly' value cannot be converted to 'json'"}
        at ballerina.lang.value.0:toJson(value.bal:262)
           sample:main(sample.bal:7)

The method description should be updated if this is the correct behaviour or the method should return a json value
Steps to Reproduce

Simple regular expression with toJson

import ballerina/io;
import ballerina/lang.regexp;

regexp:RegExp regExp = re `a`;

public function main() {
    io:println(regExp.toJson());
}

Regular expression as a record field and record with toJson

import ballerina/io;
import ballerina/lang.regexp;

regexp:RegExp regExp = re `a`;

type Rec record {
    xml xmlValue;
    table<map<string>> tableValue;
};

public function main() {
    Rec rec = {
        xmlValue: xml `<name>John</name>`,
        tableValue: table [{"key": "value"}]
    };
    io:println(rec.toJson());

    rec["regExpValue"] = regExp;
    io:println(rec.toJson());
}

Affected Version(s)
Ballerina SwanLake Update 10(2201.10.x)
OS, DB, other environment details and versions
No response
Related area
-> Runtime
Related issue(s) (optional)
No response
Suggested label(s) (optional)
No response
Suggested assignee(s) (optional)
No response

Related to https://github.com/ballerina-platform/ballerina-lang/issues/42070.


Lack of underrepresented minorities on the contributors team.

https://github.com/todogroup/opencodeofconduct/graphs/contributors
Of the 11 contributors there seems to be a severe lack of diversity on the team. Only two females and everyone appears to be white males. There does not appear to be any persons of color and only cisgendered (as best I can tell from an avatar). Also are any non-heterosexuals represented on the team.

Why is that the case? If the purpose is to amplify underrepresented voices then why has the core team not included underrepresented minorities as part of their core team?

The Open Code of Conduct purpose is fairly well defined in the FAQ:
https://github.com/todogroup/opencodeofconduct#why-a-code-of-conduct
Contributions are welcome from everyone as stated in the FAQ: https://github.com/todogroup/opencodeofconduct#can-i-contribute-and-improve-this-code-of-conduct

How are you actively seeking to amplify underrepresented voices?

@just3ws There is nothing in the Open Code of Conduct FAQ or content that hints at positive discrimination. I think the goal is simply to avoid discrimination, negative or positive.

@lleaff

Our open source community prioritizes marginalized people's safety over privileged people's comfort.

I have read many calls for white, male, heteronormative people to stop speaking on behalf of people of color, women and other minorities. Yet, here we have a group of seemingly all white, mostly male people whose response to asking how they are helping to amplify minority voices and the response is basically, "Read the FAQ" and that says they accept Pull Requests. That is putting the onus on the minorities to submit their requests for consideration by a panel of white (mostly) men. Either they are building a safe place where minorities are represented or they are looking to show how altruistic and progressive they are for their own purpose.


Added TypeScript pre-requisite to getting started

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license.
Docs: Added TypeScript pre-requisite (>= 2.7) to Getting Started topic; how to update TS to latest version.

Why is this required? I don't think typescript is needed unless you are programming with typescript, in which case it should be added to your package.json.

What's going on here?


Danger crashes when running on PR between feature branches

Report
Danger crashes when running on PR between feature branches. It works like expected when PR is from feature-branch to master.
What did you do?
Run Danger from CI on PR from one feature-branch to another (not to master branch).
What did you expect to happen?
Not to crash.
What happened instead?
Danger crashed with error:
fatal: --unshallow on a complete repository does not make sense
bundler: failed to load command: danger (/usr/local/bin/danger)
RuntimeError: Commit 98f1d936 doesn't exist. Are you running `danger local/pr` against the correct repository? Also this usually happens when you rebase/reset and force-pushed.
  /usr/local/lib/ruby/gems/2.3.0/gems/danger-4.0.0/lib/danger/scm_source/git_repo.rb:58:in `raise_if_we_cannot_find_the_commit'
  /usr/local/lib/ruby/gems/2.3.0/gems/danger-4.0.0/lib/danger/scm_source/git_repo.rb:43:in `ensure_commitish_exists!'
  /usr/local/lib/ruby/gems/2.3.0/gems/danger-4.0.0/lib/danger/request_sources/github.rb:77:in `setup_danger_branches'
  /usr/local/lib/ruby/gems/2.3.0/gems/danger-4.0.0/lib/danger/danger_core/environment_manager.rb:57:in `ensure_danger_branches_are_setup'
  /usr/local/lib/ruby/gems/2.3.0/gems/danger-4.0.0/lib/danger/danger_core/dangerfile.rb:258:in `setup_for_running'
  /usr/local/lib/ruby/gems/2.3.0/gems/danger-4.0.0/lib/danger/danger_core/dangerfile.rb:269:in `run'
  /usr/local/lib/ruby/gems/2.3.0/gems/danger-4.0.0/lib/danger/danger_core/executor.rb:27:in `run'
  /usr/local/lib/ruby/gems/2.3.0/gems/danger-4.0.0/lib/danger/commands/runner.rb:66:in `run'
  /usr/local/lib/ruby/gems/2.3.0/gems/claide-1.0.1/lib/claide/command.rb:334:in `run'
  /usr/local/lib/ruby/gems/2.3.0/gems/danger-4.0.0/bin/danger:5:in `<top (required)>'
  /usr/local/bin/danger:23:in `load'
  /usr/local/bin/danger:23:in `<top (required)>'

Your Environment

Bitrise CI
macOS 10.12 Sierra
Ruby 2.3.1p112
Danger 4.0.0
Dangerfile:# Make it more obvious that a PR is a work in progress and shouldn't be merged yet
warn("PR is classed as Work in Progress", sticky: true) if github.pr_title.include? "[WIP]"

# Warn when there is a big PR
warn("Big PR", sticky: true) if git.lines_of_code > 500

# Only accept prs to the master branch
fail "Please re-submit this PR to master, we may have already fixed your issue." if github.branch_for_base != "master"

# Ensure that all prs have and assignee
warn "This PR does not have any assignees yet." unless github.pr_json["assignee"]

# Don't let testing shortcuts get into master by accident
test_directory = "MyAppTests"

fail("fit left in tests", sticky: true) if `grep -rI "fit(" #{test_directory}/`.length > 1
fail("fdescribe left in tests", sticky: true) if `grep -r fdescribe #{test_directory}/`.length > 1
fail("fcontext left in tests", sticky: true) if `grep -r fcontext #{test_directory}/`.length > 1

fail("xit left in tests", sticky: true) if `grep -rI "xit(" #{test_directory}/`.length > 1
fail("xdescribe left in tests", sticky: true) if `grep -r xdescribe #{test_directory}/`.length > 1
fail("xcontext left in tests", sticky: true) if `grep -r xcontext #{test_directory}/`.length > 1

# Stop skipping some manual testing
warn("Needs testing on a Phone if change is non-trivial", sticky: true) if git.lines_of_code > 50 && !github.pr_title.include?("📱")

# Run SwiftLint
swiftlint.directory = 'MyApp'
swiftlint.lint_files
swiftlint.directory = 'MyAppTests'
swiftlint.lint_files

Got any clues?
Off-hand I'm not really sure, the commits Danger is looking for doesn't seem to be there, could it be specific to Bitrise?

It could be Bitrise issue. From my observations, Danger crashes on each PR from feature branch to another feature branch, but the same crash happened once on PR from feature branch to master. More than 50  builds on PR from feature branch to master passed without Danger crashing in the meantime.
It's hard for me to give more details right now. @orta if you have some hints what could be helpful in solving the issue, like logs etc - let me know, but I am not sure if I will be able to get those from Bitrise.

@darrarski - want to give #662 a shot, this might fix it 👯♂

Great to find this PR! We've been bumping into this error on and off a number of times. We'll run with #662 for a week or so and report 👍 .

Any update on this? We're still running into the issue after doing a rebase and force push. (I work on the same app as Gio).

Would recommend looking at the PR changes in #662 and seeing what could be still causing problems, we don't have a reproducible case to be able to test it on I'm afraid

@orta @JuanitoFatas @darrarski the @iflix-letsplay iOS team has been rolling with the version from #622 for the past 25 days (see my comment above), and haven't seen the fatal: --unshallow on a complete repository does not make sense ever since. 👍

Scratch that 😩. My team mate @mattdelves pointed out that he did see an ---unshallow failure, which didn't make sense to me because the version of Danger in #662 does not make an unshallow clone.
Well... Turns out I don't know how to Ruby Gem, and that to actually fetch the PR branch I had to do this:
-gem 'danger', branch: 'jf/deep-fetch'
+gem 'danger', git: 'git@github.com:danger/danger.git', branch: 'jf/deep-fetch'

🤦♂
Back to square one. We'll report back in a few weeks.

Hi, just an update on this. We've been using the changes in https://github.com/danger/danger/pull/662, for real this time, for the last two months and haven't had any issues and with more team members doing frequent rebases on PR branches.
😄 .

hah, nice, OK, rebased it and if it greens it can be merged


react-toolbox v1 continued support, react 16 ?

Has v1 been abandoned?
Has there been any conversation about continuing to support v1 while v2 is in beta ?   V1 is falling behind on those essential peerDependencies -- specifically the current version of react.

Right now we're focusing on finishing the v3 (agnostic-typescript branch), PRs solving issues in v1 and v2 are much appreciated, but there is no 'official' work in v1 and v2 anymore.

So, can we already jump to agnostic-typescript? If so, could you publish it to npm?
Any consideration to be taken to port our stuff from beta2?
Thanks!

@rubenmoya Is there any status update that we can follow for v3?

Updated v1 to be react16 compatible.
https://github.com/react-toolbox/react-toolbox/compare/1.3.4...bionikspoon:mp_react_16
I don't know your release process.  It has green tests.  And I'm going to test it in a private app at work.
Note:  Had to update react-css-themr.  Should probably wait until they merge my changes.  Can follow that conversation here: https://github.com/javivelasco/react-css-themr/issues/75

Updates:
I published a built version of this here: https://github.com/bionikspoon/react-toolbox/tree/react_16_build
In the short term, my team will be using this.  The app is on a private repo, but I'll keep this thread posted on it's journey to be used in production:

[x] open pr
[x] review app open, properly builds, tests pass, and passes a basic click test
[x] team signoff
[ ] qa signs off
[ ] production release 🥇

//package.json
{
  "dependencies": {
    "react-toolbox": "git://github.com/bionikspoon/react-toolbox.git#react_16_build"
  }
}

List of components we use:

Button
Checkbox
Dialog
Dropdown
Input
ProgressBar
Radio
Tabs
TimePicker
Tooltip
also, we use a ton of themr features.

@bionikspoon
ERROR in ./node_modules/react-toolbox/lib/date_picker/Calendar.js
Module not found: Error: Can't resolve 'react-transition-group/CSSTransitionGroup' in '/Users/aupeterson/Projects/ansible-playbooks/marketplace/git-repos/marketplace/Apps/v2/node_modules/react-toolbox/lib/date_picker'
 @ ./node_modules/react-toolbox/lib/date_picker/Calendar.js 17:26-78
 @ ./node_modules/react-toolbox/lib/date_picker/index.js
 @ ./node_modules/react-toolbox/lib/form/index.js
 @ ./node_modules/react-toolbox/lib/index.js
 @ ./app/components/WpLive/view.js
 @ ./app/components/WpLive/index.js
 @ ./app/builds/ssr/Wplive.js

ERROR in ./node_modules/react-toolbox/lib/time_picker/Clock.js
Module not found: Error: Can't resolve 'react-transition-group/CSSTransitionGroup' in '/Users/aupeterson/Projects/ansible-playbooks/marketplace/git-repos/marketplace/Apps/v2/node_modules/react-toolbox/lib/time_picker'
 @ ./node_modules/react-toolbox/lib/time_picker/Clock.js 17:26-78
 @ ./node_modules/react-toolbox/lib/time_picker/TimePickerDialog.js
 @ ./node_modules/react-toolbox/lib/time_picker/TimePicker.js
 @ ./node_modules/react-toolbox/lib/form/Form.js
 @ ./node_modules/react-toolbox/lib/form/index.js
 @ ./node_modules/react-toolbox/lib/index.js
 @ ./app/components/WpLive/view.js
 @ ./app/components/WpLive/index.js
 @ ./app/builds/ssr/Wplive.js

@bionikspoon
https://github.com/reactjs/react-transition-group/blob/master/Migration.md
could you update to add this?

guys, any activity here? Does anyone know, is react toolbox team going to make v1 branch compatible with  react 16? Or may be release v2 version with react 16?
@bionikspoon @rubenmoya

@bionikspoon Thank you for that! I've forked your v16 branch, could you give any insight into how you setup your webpack or do style imports?
I tried to just render a basic AppBar, but it's always completely unstyled. It's inlining some styles, but react-toolbox has undefined styles

  <button class=" undefined" type="button" data-react-toolbox="button">
    menu
  </button>

Test Application

There is a PR open for React 16 in v1 https://github.com/react-toolbox/react-toolbox/pull/1719 just waiting for the merge. Also, keep in mind that v3 with agnostic components is around the corner.

v3 with agnostic components how close to release?


Error on Integration of Ariston Velis Wi-Fi

Hey there, first of all many thanks for this integration. It looks like that you put a lot of efforts in it to make it work. I recently installed it and it returned this error when adding integration:
Logger: ariston
Source: /usr/local/lib/python3.10/site-packages/ariston/init.py:114
First occurred: 11:23:06 (2 occurrences)
Last logged: 11:24:03
Unsupported whe type 1
NoneType: None
I am running following HA version:
Home Assistant 2023.1.1
Supervisor 2022.12.1
Operating System 9.4
Frontend 20230104.0 - latest
I wonder if you could support or advice here.  Many thanks in advance.
Cempaa

Same issue here (Velis Evo 50)

Hey kajarit,
Thanks for looking into it. So did it work for you after changing the constant to 1 in HA ? I do not see the same content as you pasted on the link. No change to get to see the constant list, I searched everywhere.
Cempaa

Hi @Cempaa,
I've created a fork, and it's working perfectly on my local HA instance.
The link refers to the original fustom/python-ariston-api (main) branch, with the problematic enum value.

Hello Kajarit,
Thanks for your prompt reply ! I reinstall the api and tried out an older version of the integration v0.10.1. I did not change anything and now all works all of a sudden. I am beginner with HA and have not attempted programming in Python yet. So I am not sure what worked in the end but I am happy that it works.


uses a spintest to avoid task switches in finalizers

Hopefully fixes #161

This fixes #161 for my use case.


[UWP]Getting "A device attached to the system is not functioning" when trying to get PosPrinter

Getting "A device attached to the system is not functioning" when trying to get PosPrinter via
printer = await PosPrinter.FromIdAsync(deviceInfo.Id);
Printer details: Sewoo SLK-TS400 Direct Thermal POS Printer(USB(B type)+Serial(RS-232C)).

This feels not so much like feedback specifically about the documentation, but rather like a support request. Please contact the following support channel.

For developer issues, or suspected product bugs, there's Microsoft Support; a support engineer will look at the repro and see whether the issue is a Microsoft product bug, or something else. If it's a bug with a Microsoft product then the ticket will be free of charge.

Here are all of the developer support resources, for completeness.

The Feedback Hub app is the right place to lobby for a change, a feature, or to make a suggestion.
For community assistance with development questions, there are our forums, and StackOverflow.
The Windows developer support site explains other support options.
The Visual Studio developer community is available, and responsive. The former User Voice feature is now there on the Suggest a feature tab, if you want to lobby for a change, a feature, or to make a suggestion.

If you receive assistance via any support resource, and you believe that information would help us improve the docs, then please come back and let us know.


Bash and Zsh completion for flags

Fixes #128
Fixes #26
Downsides:

Current version supports only ./_state or using environmental variables. This should not be a big thing, even if it stays.
Profiles are typed in as constant. We will fix this once we better know how we will manage profiles. We will probably need an command to list all profiles.

Available for --name and --profile flags.

LGTM thanks


Fedora package doesn't install required dependency

After installing, I needed to manually install python3-evdev in order to run ratbagctl

fixed in f30, f31 and rawhide now with 0.10-2. ftr, please file distribution-specific bugs in their bug tracker, we don't always have a direct contact to the maintainers so there's often little we can do.


Make UnrolledLoop a shared header file

Existing Sample Changes
Description
Several DPC++ FPGA code samples use C++ metaprogramming techniques to create unrolled loops. This change introduces a shared header file, DirectProgramming/DPC++FPGA/include/unrolled_loop.hpp, that all samples needing this functionality should include. This change also updates all DPC++ FPGA code samples that use static loop unrolling to use this header file, instead of their own copies.
Fixes Issue#
N/A
External Dependencies
N/A
Type of change
Please delete options that are not relevant. Add a 'X' to the one that is applicable.

[ ] Bug fix (non-breaking change which fixes an issue)
[X] New feature (non-breaking change which adds functionality)
[ ] Implement fixes for ONSAM Jiras

How Has This Been Tested?
Emulation compiles and runs on Linux all succeed for the affected code samples.
'make report' target succeeds on Linux for all affected code samples.

[X] Command Line
[ ] oneapi-cli
[ ] Visual Studio
[ ] Eclipse IDE
[ ] VSCode
[ ] When compiling the compliler flag "-Wall -Wformat-security -Werror=format-security" was used

@ByrSinc @mdbtucker @tyoungsc please review.


Duplicates in windows discovery

Hello, i've installed simpleDLNA 1.0 and it's been working like a charm. The only thing that seems strange is that there are many duplicates of this DLNA Server in windows -> settings -> bluetooth and other devices -> other devices.

Is there a way to disable such behavior? In program or in windows itself.

i confirm. after Start\stop duplicate appear.


CNV-44306: Kubevirt to KUBEVIRT_DYNAMIC

📝 Description
The console changed the flag from KUBEVIRT to KUBEVIRT_DYNAMIC, which caused many things to not load/see/work.
This will fix it to work the same and load all our extensions as it should
🎥 Demo

Please add a video or an image of the behavior/changes

/retest


$and condition and paginate()

Hi! Thank you for this wonderful package! It made it easier to create filters!!!
But there are points that are not specified in the documentation.

How do I use a filter inside my livewire component with the condition $and? I have an array of identifiers by which to filter with the condition that they are all present in the output. With the use of $eq, $contains, everything is clear, but here I can't figure out how to use it.
I'll clarify. I don't use the query string, everything happens inside the livewire component.

How to apply pagination after filtering and sorting? If I now apply ->paginate(), then it causes an error.

hello @abbasudo ,
I had a problem with pagination for example:
$dataProducts = Product::filter()->sort()->whereHas('theme', function ($query) use ($idT) {
$query->where('id', $idT);
})->with('theme')->paginate(36);
in blade by clicking on the pagination page the filter is deleted

Hi @romatexdev
I don't know how you're doing it, but I used paginate() with no problems just like this: Model::filter()->paginate()
I don't use sorting on my project, but it works too


500: Opportunities for better play in NT

Some notes based on a recent review of the code:

In NT, if both opponents are void in the suit but we don't have boss, lead it anyway (either partner has boss or the boss card is buried)

In NT, if we don't have enough sure winners to make our contract, lead low from a suit where we don't yet have boss

If partner led non-boss from a suit in NT, prefer leading that suit back to them (#50)

When discarding in NT, keep cards needed to fill gaps unless we can discard all losers in a suit

Every once in a blue moon, you make some sort of adjustment to your 500 “Bot AI”, which causes Bots to bid 7 or 8 No Trump WAY more frequently that it should. The Bots lose far more than frequently than usual as a result, which has a negative effect on the game play, regardless of whether you have a Bot as a teammate, an opponent, or both. Clearly that is the case right now—Bots are recently bidding 7 or 8 No Trump far more frequently than previously, and are losing those bids far more frequently as a result.
Please investigate, and then switch your “Bot AI” back to its previous settings. Thanks!!
P.s. I typically play American, 5-card blind, no nullo. (Might you have made the change trying to improve play in a different game type, only to have unintentionally negatively impacted this particular style?)
Still a great game—but it was recently better, so please change it back.
Thanks!!!


How to enter "Charting Mode" without the "7" key in PsychEngine 7+?

Describe your problem here.
Before, there were scripts that allowed you to have the "Charting Mode" active (such as to advance the level time, activate/deactivate botplay, activate/deactivate practice mode, etc.). Does anyone have a script or something similar to keep the "Charting Mode" always active without entering the level editor? (For PsychEngine 7+). Thanks ;c
Are you modding a build from source or with Lua?
Lua
What is your build target?
Windows x64
Did you edit anything in this build? If so, mention or summarize your changes.
I tried to use scripts that worked before PsychEngine 7, but they no longer work, like this one for example: function onPause()     setPropertyFromClass("PlayState", "chartingMode", true)     return Function_Continue end

I tried, but it doesn't work. It's the only script, so there wouldn't be any interference. I even tried with all the variables of the function onCreate():
setProperty(“chartingMode”, true)
setProperty('chartingMode', true)
setProperty(chartingMode, true)
Even just putting:
chartingMode = true
My exact version is v0.7.3 with Friday Night Funkin v0.2.8. I don't know why it still doesn't work

i found this in playstate code, you can do something with this?
	function openChartEditor()
	{
		FlxG.camera.followLerp = 0;
		persistentUpdate = false;
		paused = true;
		if(FlxG.sound.music != null)
			FlxG.sound.music.stop();
		chartingMode = true;

		#if DISCORD_ALLOWED
		DiscordClient.changePresence("Chart Editor", null, null, true);
		DiscordClient.resetClientID();
		#end

		MusicBeatState.switchState(new ChartingState());
	}

Thank you, but I tried for several days with the little I know about "Haxe" but I can't fully understand it. Still, thanks again for your help :)


New refresh token is not gathered after refreshing tokens

As noted in aws/aws-sdk#454, the InitiateAuth method fails to return a new refresh token when using the REFRESH_TOKEN_AUTH auth flow. Currently the same refresh token may be used multiple times, but once this token becomes invalid it will no longer regenerate an id token. Some additional commentary is in this StackOverflow thread.
This is likely to cause an awkward authentication experience after the refresh token is invalidated and will require that new tokens be generated from credentials (which are not to be stored locally).

I'm not sure if this is causing problems - authentication seems fine - so going to close for now!


compile_protos for multi-lang project

My project is structured such that the .proto files live in a different directory from the rust code and build.rs script. The main driver for this is to allow both the go and rust codebases to share a common set of protos across the project.
|—>go
|	|—>file.go
|
|—>proto
|	|—>my.proto
|
|—>rust
	|—>Cargo.toml
	|—>build.rs
	|—>src
		|—>lib.rs

I haven't figured out the right way to specify the proto path to compile_protos and get the following error when providing the absolute path. What is the correct way to do this?
_File does not reside within any path specified using --proto_path (or -I).  You must specify a --proto_path which encompasses this file.    Note that you have to pass the protoc path, just like you would when calling protoc from the command line, it should probably look something like
    prost_build::compile_protos(&["../proto/my.proto"]),
                                &["../proto"]).unwrap();

Ahh got it. It's working when I use the correct protoc path 👍


⚠ Bofrost.es has degraded performance

In 5a1b924, Bofrost.es (https://www.bofrost.es) experienced degraded performance:

HTTP code: 200
Response time: 1738 ms

Resolved: Bofrost.es performance has improved in 777c125 after 6 minutes.


Support USASCII encoding as UTF8

Fixes https://github.com/mailhog/MailHog/issues/282
USASCII is a subset of UTF8 so lets support it using the UTF8 methods which already exist.

@khag7 thanks for turning my patch into a PR - though crediting me for it would have been nice.
@tyndyll or anyone from the @mailhog team, any chance this could be merged, and a new release published ?

Thank you !


arm64 build error: unknown mnemonic lock' -- lock xaddl x0,[x2]'

Compiling error occurred when we build the nacos-sdk-cpp on arm64:
/tmp/ccbrPsXn.s: Assembler messages:
/tmp/ccbrPsXn.s:875: Error: unknown mnemonic lock' -- lock xaddl x0,[x2]'
Makefile:42: recipe for target '/home/megaium/nacos-sdk-cpp/obj/EventDispatcher.o' failed
make: *** [/home/megaium/nacos-sdk-cpp/obj/EventDispatcher.o] Error 1
How can I solve this problem?

Could you please provide the platform information and compiler/os information to solve this?
This error is due to that a x86-specific instruction is used to implement the atomic int operation, we need addition information to adapt it to ARM hardware

The platform and compiler/os information as follows:
Linux version 5.4.0-47-generic (buildd@bos02-arm64-073) (gcc version 7.5.0 (Ubuntu/Linaro 7.5.0-3ubuntu1~18.04)) #51~18.04.1-Ubuntu SMP Sat Sep 5 14:36:25 UTC 2020

Hi @Xiongyou-Peng ,
Could you try this:
https://github.com/nacos-group/nacos-sdk-cpp/tree/develop-atomics-no-asm
to see if the problem is solved?
The solution is to replace the inline asm atomic operation with the gcc built-in one.

If the problem still persists, please reopen this issue :)


Add check for IP overlap in support.sh

Add a simple check and a summary report for the support script.
Report:
==SUMMARY==
         Processed 3 networks
         IP overlap found: 1
         Processed 167 containers

Overlap found:
*** OVERLAP on Network 0ewr5iqraa8zv9l4qskp93wxo ***
      2  "<IP_ADDRESS>",

Signed-off-by: Flavio Crisciani<EMAIL_ADDRESS>

Codecov Report

Merging #2099 into master will increase coverage by <.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #2099      +/-   ##
==========================================
+ Coverage   40.38%   40.39%   +<.01%
==========================================
  Files         139      139
  Lines       22356    22356
==========================================
+ Hits         9029     9031       +2
+ Misses      12000    11998       -2
  Partials     1327     1327

Impacted Files
Coverage Δ

cmd/proxy/tcp_proxy.go
82.97% <0%> (-2.13%)
:arrow_down:

networkdb/cluster.go
62.06% <0%> (+0.73%)
:arrow_up:

Continue to review full report at Codecov.

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


[docs][joy-ui] Revise the Drawer page

[x] I have followed (at least) the PR section of the contributing guide.

Mostly a few increments and corrections here and there, as well as some demo tweaks.
Got some questions along the way that I'll drop as comments!
https://deploy-preview-38988--material-ui.netlify.app/joy-ui/react-drawer/

Woo, thanks a ton for the reviews @samuelsycamore! I'll need either @mnajdova or @siriwatknp's help in adding these components to the API section ⎯ couldn't figure out how to do it 😅

Just realized we forgot to add an Anatomy section to this page! 🤦♂ We can come back to it in another PR.


Tighten Up Account Resolution Methods

This PR swaps methods from ExtraAccountMetaList to ExtraAccountMeta and tightens up implementation.
This sets the stage for:

Removal of AccountInfo helpers
Updated integration with spl-transfer-hook-example

Summary:

Moved de_escalate_account_meta and resolve_pda into account.rs as private helpers
Added resolve to ExtraAccountMeta
Trimmed implementation of add_to_instruction for ExtraAccountMetaList

Note: As it relates to #4963:

An is_pda() or enum is no longer required
An AccountKeyable trait is no longer required

Just the question about the match statement. In general, if you're just moving functions around, it's easier to review if there's one commit that just moves everything, and then a separate commit with functional changes.

Cool sounds good, I can do that next time


output = model(features, adj) Are test features involved in the trainning process?

Hello,tkipf !
Thanks for your share! In the trainng process,features are 2708 dims, does it involve test samples? thank you very much!

Although the model uses all data as input, it calculates loss only from idx_train which defined in load_data.
Hope this would help.

@ChrisZhangcx
Thanks for your answer !
In trainning procedure,if testing samples  have edges with  trainning  samples ,the corresponding position in the adjacency matrix will become 1. the testing sample's feature will be involved in the trainning sample's output  and in the training sample's loss.
Thank you very much!

@houhouhouhou11
You are right. It does seem like we combine the information of both train and test data.
I try to think about this issue in a different way: we do need these edges in the adjacency matrix since our task is to give node representations according to each node's features as well as topology info of its neighbors.
By removing the relation edges between the train and test nodes (by manually masking their edges in adjacency matrix), there might occur some of the following issues:

Change the distribution of data and features. Intuitively, we assume that both train and test data share the same distribution of the whole dataset.
The graph is no longer fully connected. Especially we set train nodes only to 140 and the test to 1000 in this case. Some of the train nodes might even have no neighbors.
Affect the information of the graph structure, from which we are able to extract features to further strengthen our model performance.

Thanks for your reply! I'm a starter in the graph network. Please let me know if you have any comments.

@ChrisZhangcx
Thanks for your reply ,I'm a  starter too.
Your answer is very reasonable .
Whether we remove the relation edges between the train and test nodes or not,can I think it's a problem of transductive reasoning or inductive reasoning?
In fact,I'm chinese too!^^^^^_^All in all,thank you very much!


Questions about default run settings

Hi, I have been running on the basic celebs dataset you guys include in this repo. I have a few related questions:

The ffhq_64_512 training dataset appears to only contain four images. Why are there so few images being trained on?
I completed a run in about 8 hours on these four images. Why did it take so long to train on them? Does the noising/denoising process really add that much extra time to training?
I got good results when testing on the validation set. How is this possible, given that there were only four images in the training set? I had assumed that the model would have severely overfit.

Four images are just for formatting reference, and if you don't have your own training data, you can refer to the guidelines.
What do you mean a run? I think it will not take so long in a epoch.
I think so....

Four images are just for formatting reference, and if you don't have your own training data, you can refer to the guidelines.
What do you mean a run? I think it will not take so long in a epoch.
I think so....

Thanks, didn't see that! Here are results from the validation set when I trained with just those four images. I was confused as to why the results were so good.

by "a run", I meant running sr.py with the command %run sr.py -p train -c config/sr_sr4_64_512.json -enable_wandb -log_wandb_ckpt -log_eval until the script stopped (I am assuming when the loss converged).

Meaning, you think it overfit?

I see,  and this is indeed a very interesting phenomenon, have you tested the other images of the validation set? Because of the random noise, I don't think the training will overfit, but the diversity of the generation will be limited.


How to set up redirect uri for oauth authorization using on docker when running containers behind IIS reverse proxy

From @yamac-kurtulus on Monday, 10 December 2018 17:41:47
Hello everyone,
We are building a web application with dotnet core running inside docker containers. We have put the app behind IIS on WS2016 which is used as reverse proxy. The requests incoming to our URL (let's call it www.myurl.com) are forwarded to the related docker container using the container name as the hostname with the help of url rewrite module(in our case, let's call it "web"). This setup works up to here.
However, a few days ago we needed to add Facebook oauth authorization support to register to the system and a problem appeared. We setup our authorization as the following:
    ...
    services.AddAuthentication().AddFacebook(o =>
    {
        o.CallbackPath = new PathString("/account/facebookreturn");
        o.AppId = appSettings.FacebookLogin.AppId;
        o.AppSecret = appSettings.FacebookLogin.AppSecret;
    })
    ...

Our facebook authentication challenge then redirects user to facebook oauth page with the following redirect uri:
https://www.facebook.com/v2.12/dialog/oauth......redirect_uri=http%3A%2F%2Fweb%2Faccount%2Ffacebookreturn&state=....
As you see the redirect_uri has the container name / hostname of the docker container whereas it should actually be "www.myurl.com".
We tried several solutions such as using x-forwarded headers. Using UseIISIntegration() and UseUrls did not seem to work either. What are we missing here? I am not much experienced with IIS and web headers, so it could be fairly obvious, and I appreciate the help.
Some issues I have stumbled upon and related, but had no luck applying the fixes: #2043,  https://github.com/aspnet/Security/issues/1508
https://github.com/aspnet/Docs/issues/2384
Copied from original issue: dotnet/core#2127

Hi @yamac-kurtulus - can you try the troubleshooting instructions at https://docs.microsoft.com/en-us/aspnet/core/host-and-deploy/proxy-load-balancer?view=aspnetcore-2.2#troubleshoot ?

@Eilon Tried but did not help much, we were able to log the headers, which seemed fine, however didn't know what to do with that info, and opted to use ajax. Issue could be closed, or left open in case someone else has the same problem. I guess it is more likely to be related to how docker works.

FYI. I ran into a similar issue with Open ID Connect. My server had ForwardedHeadersOptions configured like this when I was seeing the redirect_url protocol written as http when there was a another device doing the SSL termination.
services.Configure<ForwardedHeadersOptions>(options =>
{
  options.ForwardedHeaders = ForwardedHeaders.XForwardedFor | ForwardedHeaders.XForwardedProto;
});

When I changed them to the following, the issue went away and the protocol for the redirect_url was https
services.Configure<ForwardedHeadersOptions>(options =>
{
  options.ForwardedHeaders = ForwardedHeaders.XForwardedProto;
});


on_mount calls a fn twice and on_load calls a fn tree times on debug mode.

Problem:
https://discord.com/channels/1029853095527727165/1239586151178703020
when i use my AsyncClient to fech data on my API (on Debug Mode):
The INFO LOG at the API
on_load -> call/fetch tree times
on_mount -> call/fetch two times
When i use with a button, just call/fetch once.

This is my Async client: (i remove the async with cuz reflex said: client was close)
# -------------------------------------------------------------------
# QuickBio API Async client
# -------------------------------------------------------------------

async_client = httpx.AsyncClient(base_url=URL_BASE, timeout=None)

@dataclass
class QbAsyncClient:
    """Sync client for QuickBio API"""

    user_id: str | None = None
    endpoint: str | None = None
    client: httpx.AsyncClient = async_client

    async def get(self) -> Any:
        headers = {"user_id": self.user_id} if self.user_id else {}

        # async with self.client as client:
        response = await self.client.get(url=self.endpoint, headers=headers)
        print(response)
        # response = await self.client.get(url=self.endpoint, headers=headers)

        if response.status_code == 200:
            return response.json()

    async def post(self, data: str) -> Any:
        """Every data should be a model dump json"""

        headers = {"user_id": self.user_id} if self.user_id else {}

        # async with self.client as client:
        response = await self.client.post(url=self.endpoint, headers=headers, data=data)

        if response.status_code == 200:
            return response.json()

This is when i call
import reflex as rx

from ....api.client import QbAsyncClient

class MyTask(rx.State):
    bars: list[str]

    @rx.background
    async def get_bars(self):
        bars = await QbAsyncClient(
            endpoint="/analysis/richness/histogram/barmode"
        ).get()
        # The API returns a list ['group', 'stack']

        print(bars)
        async with self:
            self.bars = bars

def render() -> rx.Component:
    return rx.box(
        rx.vstack(
            # rx.button(
            #     "connect to client",
            #     on_click=MyTask.get_bars,
            # ),
            rx.select(
                MyTask.bars,
                on_mount=MyTask.get_bars,
                placeholder="Select a bar mode",
            ),
        )
    )

I fix this with a if statement
class MyTask(rx.State):
    bars: list[str] = []

    @rx.background
    async def get_bars(self):
        async with self:
            print(self.bars)

            if not self.bars:
                bars = await QbAsyncClient(
                    endpoint="/analysis/richness/histogram/barmode"
                ).get()

                self.bars = bars

On the INFO LOG at the API just call once
But still print twice

with 'reflex run --env prod' call once  without the 'if not self.bars'

Confirming that I have also ran into this problem. There also might be an issue with on_load. I basically had to pull all of the on_loads out of my code and change them to on_mount. Sometimes it would stop the new page from loading at all.


systemd-networkd: add DHCPServer test

Motivation for this change

Demonstrate https://github.com/NixOS/nixpkgs/issues/73504
Stop such regressions happening in future.

Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[x] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nix-review --run "nix-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[ ] Fits CONTRIBUTING.md.

Notify maintainers
cc @

IIRC, there were some problems with useNetworkd in nixos vm tests, due to some assumptions in the testing infrastructure.

I did spend an unreasonable amount of time battling with the per-VLAN networks that the test infrastructure adds by default, but eventually found the solution of picking a lexiographically smaller unit file name, which:

works, by allowing me to override the default networks
is documented in the test
is small/simple enough to not impose a large maintenance burden.

This is blocked on https://github.com/NixOS/nixpkgs/pull/74207

Now that systemd >=243.4 has been included in master, and thus https://github.com/NixOS/nixpkgs/issues/73504 is fixed, this test no longer needs the workaround for https://github.com/NixOS/nixpkgs/issues/73504, which had previously blocked this PR's approval and submission. I think this PR is ready for submission.

Thanks for picking this up!
I also did check this against staging/staging-next (which ships with systemd 245.3) to ensure we don't immediately break it again ;-)
LGTM, merged :-)


Revised notebook and add delete to MyScale vector store

Description:

Add .delete to myscale vector store.
Revised vector store notebooks

Tag maintainer: @baskaryan
Twitter handle: @myscaledb @mpsk_liu

thank you @mpskex!


Output of segmentation network

Hi,
Thanks for the great contribution and for releasing the code.
I'm not having an issue, but a question, before investing time into integrating this network into my code.
For the segmentation case, does the output features are per vertex, or per face of the input shape?
Thank you in advance!

Hi,
In the segmentation scenario, the network outputs a feature per edge to predict the class of the edge. Is there a way to aggregate these feature to produce a point based features?
Thank you in advance!