Fixes to isArray, handled when input is any

Handled the cases when the input is any[] or any and not unknown or unknown[]
https://github.com/total-typescript/ts-reset/issues/48#issuecomment-1440269813
You might wanna check if the test are correct.

@mattpocock Found a better solution that returns less complex type for Unarray<T>.
interface ArrayConstructor {
  // @ts-ignore
  isArray<T>(arg: T[] extends T ? T | T[] : never): arg is unknown[];
}

Basically, this ^ only works when the input is any[], any, unknown or unknown[], for other types, it fallbacks to the default isArray() definition.
I have one question, when the input is any[] or any, should it return unknown[] or any[]?
Code above returns unknown[] at any case.
interface ArrayConstructor {
  // @ts-ignore
  isArray<T>(arg: T[] extends T ? T | T[] : never): arg is T[];
}

This ^ would return:

unknown[] for unknown or unknown[]
any[] for any or any[]

@DeepDoge I'm not sure that we can have // @ts-ignore in the bundle - experience from working on XState is that those can often cause unexpected surprises when bundled.
Do you have a test case that doesn't pass with the current implementation?

value in inner(value); was returning a complex type, which wasn't that nice to look at.

Gotcha, I don't mind too much because especially inside a generic function, it's fairly important to see how T is altered.

@mattpocock Wasn't expecting to solve it this quick. Got rid of @ts-ignore
Current behavior:
Case => Result

any => any[]
any[] => any[]
unknown => unknown[]
unknown[] => unknown[]
other => fallbacks to the default Array.isArray() definition

isArray(arg: T[] extends T ? T | T[] : never): arg is unknown[];

@mjxl This PR is for narrowing the type correctly.
Not for returning true or false, based on argument type.
But, if you ask me if the type of the argument is already known to be not an array, it shouldn't even let you use it as an argument. But we can't do this because it will just fallback to the default isArray definition of typescript.

@DeepDoge Would it be possible to make this handle readonly arrays as well?
With the current implementation these tests fail.
doNotExecute(() => {
  const arrOrString = [] as readonly string[] | string;

  if (Array.isArray(arrOrString)) {
    type tests = [Expect<Equal<typeof arrOrString, readonly string[]>>];
  }
});

doNotExecute(() => {
  const arrOrString = [] as readonly string[] | string;

  if (Array.isArray(arrOrString)) return;
  type tests = [Expect<Equal<typeof arrOrString, string>>];
});

I have been testing a bit but can't quite figure it out.
With
interface ArrayConstructor {
  isArray<T>(arg: T): arg is T extends readonly any[] | any[] ? T : T[] extends T ? T[] : never;
}

these tests fail
doNotExecute(() => {
  const maybeArr = [1, 2, 3] as any;

  if (Array.isArray(maybeArr)) {
    type tests = [Expect<Equal<typeof maybeArr, any[]>>];
  }
});

doNotExecute(() => {
  function test<T>(value: T) {
    type Unarray<T> = T extends Array<infer U> ? U : T;
    const inner = <X extends Unarray<T>>(v: X[]) => {};

    if (Array.isArray(value)) {
      inner(value);
    }
  }
});

and with
interface ArrayConstructor {
  isArray<T>(arg: T[] extends T ? T | T[] : never): arg is T[] extends T ? T[] : never;
  isArray<T>(arg: T): arg is T extends readonly any[] | any[] ? T : T[] extends T ? T[] : never;
}

only
doNotExecute(() => {
  function test<T>(value: T) {
    type Unarray<T> = T extends Array<infer U> ? U : T;
    const inner = <X extends Unarray<T>>(v: X[]) => {};

    if (Array.isArray(value)) {
      inner(value);
    }
  }
});

fails but I can't figure out how to make all of them pass.

Hey @DeepDoge , not sure if this is related or not, but if you're trying to add support for readonly arrays, the easiest way to do that is to treat all arrays like readonly arrays.
That's because ReadonlyArray is the supertype -- which is kinda backwards from what most people expect.
Here's a sandbox about it: https://codesandbox.io/s/arrays-vs-readonlyarray-in-typescript-xdgsfj?file=/src/array-stuff.ts
That's something I wish someone had told me, because I found this behavior very confusing until I figured out why it worked that way.
Hopefully that's helpful!

Although Matt told me to open this PR, I don't think this is gonna get merged, it has been months. I don't wanna deal with this PR anymore. So closing it and deleting the fork.
For future reference, here's the code:
interface ArrayConstructor {
   isArray<T>(arg: T[] extends T ? T | T[] : never): arg is T[] extends T ? T[] : never;
 }

And tests:
import { doNotExecute, Equal, Expect } from "./utils";

 doNotExecute(() => {
   const maybeArr = [1, 2, 3] as unknown;

   if (Array.isArray(maybeArr)) {
     type tests = [Expect<Equal<typeof maybeArr, unknown[]>>];
   }
 });

 doNotExecute(() => {
   const maybeArr = [1, 2, 3] as unknown[];

   if (Array.isArray(maybeArr)) {
     type tests = [Expect<Equal<typeof maybeArr, unknown[]>>];
   }
 });

 doNotExecute(() => {
   const maybeArr = [1, 2, 3] as any;

   if (Array.isArray(maybeArr)) {
     type tests = [Expect<Equal<typeof maybeArr, any[]>>];
   }
 });

 doNotExecute(() => {
   const maybeArr = [1, 2, 3] as any[];

   if (Array.isArray(maybeArr)) {
     type tests = [Expect<Equal<typeof maybeArr, any[]>>];
   }
 });

 doNotExecute(() => {
   const arrOrString = [] as string[] | string;

   if (Array.isArray(arrOrString)) {
     type tests = [Expect<Equal<typeof arrOrString, string[]>>];
   }
 });

 doNotExecute(() => {
   const arrOrString = [] as string[] | string;

   if (Array.isArray(arrOrString)) return;
   type tests = [Expect<Equal<typeof arrOrString, string>>];
 });

 doNotExecute(() => {
   let arrOrString = "" as string | [1, 2];

   if (Array.isArray(arrOrString)) {
     type tests = [Expect<Equal<typeof arrOrString, [1, 2]>>];
   }
 });

 doNotExecute(() => {
   let path = [] as string | string[];

   const paths = Array.isArray(path) ? path : [path];

   type tests = [Expect<Equal<typeof paths, string[]>>];
 });

 doNotExecute(() => {
   function test<T>(value: T) {
     type Unarray<T> = T extends Array<infer U> ? U : T;
     const inner = <X extends Unarray<T>>(v: X[]) => {};

     if (Array.isArray(value)) {
       inner(value);
     }
   }
 });


Accounting metrics

The aim of this pull request is to add accounting related metrics to the metrics endpoint

Total debited amount
Total credited amount
Total credit events
Total debit events
Total accounting related disconnects
Total accounting related reserve blocks ( when temporarily skipping a peer for the safety of not going through thresholds before we could settle)

Unrelated to the PR: we do not have unit tests for metrics. I assume they are not a priority for the PR - are they required? If so, is there a good way to write unit tests for Prometheus Counters, Gauges and other Metrics collectors?


Better default column name from ML_PREDICT

The current implementation of ML_PREDICT requires three different JIT functions pre/post-processing and inference function. So currently it shows in Rikai that, for a query like
SELECT ML_PREDICT(ssd, image) FROM ...

The column is shown as
|-- ssd_dpjlkw_post(ssd_dpjlkw(ssd_dpjlkw_pre(image))): array (nullable = true)

It would be better that ML_PREDICT 's default column name to be just col or the model name , i.e., ssd(image)

Closed


Don't rander the image link which is not relative path

Currently we just update the image links when building the website, it doesn't work when the image link is start with "http". We need to skip the randering when the image link has the perfix "http".

Merged the PR into master branch.


libsodiumInit needs to be called

It would be nice if we didn't need to call libsodiumInit directly when using libsodium based crypto. For example in cardano-node: https://github.com/input-output-hk/cardano-node/pull/1958/files#diff-af911e411955f9dc3a3b1ba96132e67830dd2322ac98d6c90071c5b30b224b00R18

There is no way around not initializing libsodium, it must be done at least once before libsodium based functionality can be used. Same applies to the secp256k1 librray. That is why we now provide this function that can be called instead that will take care of all of the crypto initialization:
https://github.com/input-output-hk/cardano-base/blob/a344b32d97b0d0e83b773069f6a20f7e5fa67478/cardano-crypto-class/src/Cardano/Crypto/Init.hs#L25


NPM Start hangs with Node 7.72

Description
npm run start fails when using node version 7.72 (current but not stable version number) on windows.
Expected behavior
Executing the  npm run start  command in windows powershell should successful run the project.
Actual behavior
Node exits with the following error messsags:
(node:13564) UnhandledPromiseRejectionWarning: Unhandled promise rejection (rejection id: 2): TypeError: "listener" argument must be a function
(node:13564) DeprecationWarning: Unhandled promise rejections are deprecated. In the future, promise rejections that are not handled will terminate the Node.js process with a non-zero exit code

Environment
npm ls react-scripts<EMAIL_ADDRESS>node -v:  v7.7.2
npm -v :4.1.2
Windows 10, powershell terminal in VSCode
Shore youtube clip of the issue.
https://youtu.be/1zCGofBZy-I

Yes, this is a duplicate of https://github.com/facebookincubator/create-react-app/issues/1776.
Please downgrade Node to an earlier version until this is fixed.

Duplicate of https://github.com/facebookincubator/create-react-app/issues/1776.
Node 7.7.2 has a regression, you must use 7.7.1 or (preferably) Node 6.10.0.
This will be fixed in the next node version.

0.9.5 is out with the fix.
https://github.com/facebookincubator/create-react-app/releases/tag/v0.9.5


pbplot error while running robyn_onepagers()

While trying to get the one-pager report , i got pblot error, i search for the package but couldnt find :S I am getting this error on ubuntu 20.04 but also the same error happened on windows10
R 4.2.2
Robyn 3.9
Ubuntu 20.04

I think I know what the issue here is. Can you please try setting quiet = TRUE within your function?

Thx this fixed the problm for me.


Is it possible that subtitles and TTS are also generated by the OpenAI API?

I see that Whisper is used, but is it possible to use whisper from the OpenAI api, I have openai key and I like the quality of their TTS and .srt, is it possible to use the api to generate these 2 elements?

thank you for your feedback
i will optimize it


Road to EVM

same here https://github.com/kodadot/nft-gallery/pull/10322#issuecomment-2172608195

need failed log again..

need failed log again..

scary error I guess

need failed log again..

scary error I guess

@hassnian Would you like to take a look at it?

I have no idea why the build on Cloudflare would fail.
It builds and deploys successfully with the same branch when I use my private account on Cloudflare.


feat: Added loading method for PyTorch artefact detection models from HF Hub

Following up on #426, this PR introduces the following modifications:

refactored faster rcnn
added a from_hub function to load artefact detection architectures in PyTorch
added unittest
fixed header unittest

With PyTorch backend, the following snippets are completely equivalent:
# Pretrained
from doctr.models.obj_detection import fasterrcnn_mobilenet_v3_large_fpn
model = fasterrcnn_mobilenet_v3_large_fpn(pretrained=True)

# HF Hub
from doctr.models.obj_detection.factory import from_hub
model = from_hub("mindee/fasterrcnn_mobilenet_v3_large_fpn")

Any feedback is welcome!

@fg-mindee Can you also add a section into the docs for this feature ? 🤗 Maybe where we can add later links to some models on the hub (this would also fix the different vocab pretrained support i think)

@fg-mindee Can you also add a section into the docs for this feature ? hugs Maybe where we can add later links to some models on the hub (this would also fix the different vocab pretrained support i think)

Hey Felix :wave:
This is something we have to address but the documentation is built using the TF backend for now (since most high-level feature are identical) and this PR is only about PyTorch (we have no implementation of faster rcnn in TF in docTR for now) :sweat_smile:

Thanks, @SiddhantBahuguna could you check this threshold by class when you have the time to do so ?

Hi @fg-mindee @charlesmindee, actually, since we are using the bare output of the model it lacks post nms strategy.
After observing the output, along with other filter methods I did use thresholds for logos specifically and it did lead to a better recall. I will open a PR soon for the same :)


Cache map tiles for offline use

There a few different options for doing this:

Use the mapbox vector tile api to fetch all the needed tiles for a subset of basemaps and cache those in the client's browser.

How many tiles would this take and what would it cost?
Could we cache detailed zoom levels near shorelines, but less detail over sparse ocean space? How would mapbox-gl handle that if a tile isn't cached but one at a "parent" zoom level is?

Instead, we could create our own mbtiles files specifically for offline use. This would require dedicated "offline basemaps" complete with tile packages and styles to be uploaded. This would be less ideal as it would require a lot more work for each project.
We could have truly global basemaps with very sparse data in the worst case scenario so we could build just 1 "offline basemap" once.

Other questions...

How to handle satellite imagery. Is it much different?

This thread has some useful discussion about how mapbox-gl-js overzooms "sparse" tilesets. It appears we may be able to get away with caching different areas at different zoom levels but there may be some issues.

I have a partial answer to how many tiles would take and what it would cost, per-basemap.

Tiling French Polynesia at zoom levels 16-10 would generate something around 35k tiles and cost around $9.00 to download from MapBox. Unfortunately, it is explicitly against their terms of service to download and redistribute tiles for offline use, so each time a survey facilitator downloads a map it would incur this cost. We may need to look at discounts through their community program.
Imagery is handled via the raster tile API, which actually has a more generous free tier.


Is com.apollographql.apollo3:apollo-mockserver multiplatform and supports js?

Question
Is com.apollographql.apollo3:apollo-mockserver multiplatform and supports js?

we'd rather prefer to keep running our tests on browser as we support few clients on this platform

apollo-mockserver uses the nodejs net API to create a "real" server listening to a "real" port. I don't see how it is possible on the browser today, there's just no API to do that.
If you want to keep your tests on the browser:

Either run a server on your dev machine outside your tests (that server can be node, JVM, anything else)
Or use TestNetworkTransport

thanks for your response @martinbonnin, I've tried TestNetworkTransport but it doesn't support CustomScalarAdapters, hence ended up with mockserver


fix fix ccr test cases (#28664)

Proposed changes
cherry-pick from #28664

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall


Provide Route informations (ReRoute Key, input request context) in User defined aggregators.

Expected Behavior / New Feature
Provide Route informations (ReRoute Key, input request context) in User defined aggregators.
Task Aggregate(List responses, List rerouteKeys, List inputRequests)
Actual Behavior / Motivation for New Feature
I want to enrich the results of a fowarded request with the results of some other requests (Ex : The 1st request (Route A) returns an entity A, which contains the key of an entity B. I want to add the label of the entity B to the DownstreamReponse).
I would like to be able to do it by config (having some json like :
  "ResultEnrichers": [
    {
      "ReRouteKey": "get_A",
      "JoinResults": [
        {
          "SourceKey": "B_key",
          "UrlPattern": "http://B_entity_api/api/B_entity/{B_key}",
          "ProjectedFields": [
            {
              "SourceName": "property_name_of_label_in_B_object",
              "TargetName": "property_name_of_label_in_A_object"
            }
          ]
        }
      ]
    }
  ],

I though i could achieve this in a custom aggregator but :

i don't have access to the headers of the first request to use them for the second request
as the config is in a json file (so done at runtime), i need to know the ReRoute Key to know which config i should use.

Linked to issue  1069

Provide Route informations (ReRoute Key, input request context) in User defined aggregators.

Route can be obtained from context like this:
    public async Task<DownstreamResponse> Aggregate(List<HttpContext> responseHttpContexts)
    {
        var route = responseHttpContexts[0].Items.DownstreamRoute();
        // ...
    }

I would like to be able to do it by config (having some json like :
  "ResultEnrichers": [
    {
      "ReRouteKey": "get_A",
      "JoinResults": [
        {
          "SourceKey": "B_key",
          "UrlPattern": "http://B_entity_api/api/B_entity/{B_key}",
          "ProjectedFields": [
            {
              "SourceName": "property_name_of_label_in_B_object",
              "TargetName": "property_name_of_label_in_A_object"
            }
          ]
        }
      ]
    }
  ],

You want customs properties for the route. It is not implemented right now, but we have nice PR for that

#1843

And this feature will be delivered in version 24.0


Jupyter notebook does not save

Issue Type: Bug
I have a large notebook (~10MB) (not created from but edited in the native interface) that cannot be saved .
Repro:

Ctrl + S

VS Code version: Code 1.58.2 (c3f126316369cd610563c75b1b1725e0679adfb3, 2021-07-14T22:09:06.581Z)
OS version: Darwin x64 20.4.0
Restricted Mode: No
Remote OS version: Linux x64 5.8.0-59-generic
Remote OS version: Linux x64 5.8.0-36-generic

System Info

Item
Value

CPUs
Intel(R) Core(TM) i9-8950HK CPU @ 2.90GHz (12 x 2900)

GPU Status
2d_canvas: enabledgpu_compositing: enabledmetal: disabled_offmultiple_raster_threads: enabled_onoop_rasterization: enabledopengl: enabled_onrasterization: enabledskia_renderer: disabled_off_okvideo_decode: enabledwebgl: enabledwebgl2: enabled

Load (avg)
4, 3, 3

Memory (System)
32.00GB (0.20GB free)

Process Argv
--crash-reporter-id 1572910a-3d68-41d5-8b2d-a8224b00f95b

Screen Reader
no

VM
0%

Item
Value

Remote
SSH: torino

OS
Linux x64 5.8.0-59-generic

CPUs
Intel(R) Xeon(R) CPU E5-2650 v2 @ 2.60GHz (32 x 2591)

Memory (System)
125.80GB (9.36GB free)

VM
0%

Item
Value

Remote
SSH: stelvio

OS
Linux x64 5.8.0-36-generic

CPUs
Intel(R) Core(TM) i9-10900X CPU @ 3.70GHz (20 x 1200)

Memory (System)
125.50GB (13.95GB free)

VM
0%

Extensions (9)

Extension
Author (truncated)
Version

remote-containers
ms-
0.187.1

remote-ssh
ms-
0.65.7

remote-ssh-edit
ms-
0.65.7

remote-wsl
ms-
0.58.2

vscode-remote-extensionpack
ms-
0.21.0

gc-excelviewer
Gra
3.0.42

python
ms-
2021.7.1053846006

vscode-pylance
ms-
2021.7.4

jupyter
ms-
2021.8.1054968649

@ad12 can you share a bit more about what happened after you run ctrl+s? Do you get any UI feedback?

@rebornix Thanks for your response! There was no UI feedback after pressing ctrl+s - the interface hung for several minutes, after which I restarted VSCode. I wasn't able to run any additional operations on the Jupyter Notebook, so it appeared that the save was blocking any further interaction with the notebook.
I got around this issue by reducing the size of the notebook by removing all cell outputs, but as i run the notebook again, I would like to have the cell outputs persist in the save state to avoid having to recompute them every time i open the notebook.

@rebornix Might this be the notebook output buffer perf issue you fixed recently?

Also @ad12 any chance you can share the notebook so we can repro?


Route to different page

I am using react router v4 for routing. There is a scenario that when products is clicked i want to show products just beneath banner but when add restaurant is clicked, instead of showing it in the same page beneath banner, i want to show it in different page. How can i do it on react router v4?
Here is my code (Right now when add restaurant is clicked, form box is shown on same page under banner)
const routes = [
  {
      pattern: '/restaurant',
      component: () => <Content />
  },
  { pattern: '/addrestaurant',
    component: () => <AddRestaurant />
  }
];

class HomeScreen extends Component {
    render() {
        return (
          <Router>

                    <Link to="/restaurant" className="brand item">Restaura</Link>
                    <Link to='/addrestaurant' className="item tab">Add Restaurant</Link>
                    <Link to="/products" className="item tab">Products</Link>

              {routes.map((route, index) => (
                   <Match
                       key={index}
                       pattern={route.pattern}
                       component={route.component}
                       exactly={route.exactly}
                   />
              ))}

          </Router>
        );
    }
}

I want to show addrestaurant inside this component instead of HomeScreen Component( but the menu of add restaurant is in there )
const GlobalLayout = (props) => (

			code for navigation

			{props.children}

);
export default GlobalLayout;

This is a usage question, not a bug report. Please post this kind of thing on Stack Overflow or Reactiflux.


Use GolantCI-lint instead of Gometalinter

Gometalinter is officially deprecated in favor of golangci-lint, which has many advantages.
I am enabling many checks here since they encapsulate community best practices, it seems advisable to follow them since I don't feel strongly the other way.
This PR uses dep that is introduced in #1195.

Nice, I also switched cucumber-engine over to this recently

@charlierudolph Whoops I hadn't realized that this didn't pass on CI. I updated Travis to Go 1.12 and now it works. Please take another look. 🙂


fix #1521

Codecov Report

Merging #2194 into master will decrease coverage by 0.01%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master    #2194      +/-   ##
==========================================
- Coverage   92.06%   92.05%   -0.02%
==========================================
  Files         191      191
  Lines       19581    19581
==========================================
- Hits        18027    18025       -2
- Misses       1554     1556       +2

Impacted Files
Coverage Δ

qiita_db/reference.py
96.61% <ø> (ø)
:arrow_up:

qiita_db/analysis.py
95.96% <ø> (ø)
:arrow_up:

qiita_db/commands.py
97.08% <ø> (ø)
:arrow_up:

...iita_db/support_files/patches/python_patches/36.py
38.7% <ø> (ø)
:arrow_up:

qiita_db/artifact.py
98.54% <ø> (ø)
:arrow_up:

...ita_db/metadata_template/base_metadata_template.py
95.05% <ø> (ø)
:arrow_up:

qiita_db/util.py
96.51% <100%> (ø)
:arrow_up:

qiita_db/test/test_util.py
99.78% <100%> (ø)
:arrow_up:

...t/handlers/api_proxy/tests/test_sample_template.py
97.77% <0%> (-0.89%)
:arrow_down:

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 06f40cf...79504a5. Read the comment docs.


TileCatalog and Pagination RTL fixes

Closes #451
Summary

Fixed arrow direction and padding in RTL.
Fixed tile padding and checkmark location in RTL
The search icon moving also appears to be a non-issue, but a change can be added if wanted

Change List (commits, features, bugs, etc)

added dir="ltr" to arrow components in pagination so in RTL they're flipped
added padding in pagination so the text is the right distance away in RTL
added styles that override RadioTile defaults in RTL
snapshots updated

Acceptance Test (how to verify the PR)

Verify both in Pagination and TileCatalog the bottom bar has the correct padding and arrow direction in RTL
Verify that when a tile is selected in TileCatalog, the checkmark switches sides in RTL and the padding is correct

There are still some padding issues in RTL mode around the plus icon and the checkmark does not move to the opposite side.

Fixed checkmark location and padding in RTL

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


make comparision in StrictInArraySniff unaware of uppercase (Solves #1061)

With using strtolower the check does not implicitly rely on constants being lowercase.
Solves #1061

Thanks - I feel this needs a unit test entry though, to show that we're covered for upper and lowercase.

Thank you for your PR. Looks good to me!
FYI: It will be a little while before your PR gets merged as we're working on finishing a dedicated release which will make WPCS compatible with PHPCS 3.x.
Ref: https://github.com/WordPress-Coding-Standards/WordPress-Coding-Standards/issues/1040

Hi,
thanks for the quick feedback. I hope, this is what is needed for the test.

@websupporter You will need to append the line number and no. of error in StrictInArrayUnitTest.php file.
https://github.com/WordPress-Coding-Standards/WordPress-Coding-Standards/blob/develop/WordPress/Tests/PHP/StrictInArrayUnitTest.php#L40-L54

Hi @emgk,
I want in_array( 1, array( '1', 1 ), TRUE ); // Ok. not to throw any warnings, like e.g. https://github.com/WordPress-Coding-Standards/WordPress-Coding-Standards/blob/develop/WordPress/Tests/PHP/StrictInArrayUnitTest.inc#L3

You will need to append the line number and no. of error in StrictInArrayUnitTest.php file.

@emgk In this case that is actually not needed as the unit test is for a code sample which used to give an error where there shouldn't have been one.


[Bug] PropertyPane validations are not working as expected

Description
I tried to pass a random non-URL value to Video Widget's URL and it is not showing any error.
In the code, I could see that we are validating this field using Regex.
Steps to reproduce the behaviour:
Add a video widget and pass in any non-URL to its URL field.
https://user-images.githubusercontent.com/10436935/133488535-225af348-359b-4ae4-96a7-fa67eba8b407.mov
Important Details

Version: [Cloud]
OS: [MacOS]
Browser [chrome]
Environment [production]

@aswathkk can you add note on what changes are required here to fix this validation issue here, This is a good issue to solve for a first time contributor

Can I contribute to this issue?

Thank you @errorinc0de for showing interest. I've assigned it to you now.
Please take some time to go through the Contributing Guidelines.
You could see the configuration for VideoWidget's property pane here.
The validations are evaluated on validate function on validations.ts file.
In order to fix this issue, Take a look at how the Regex that we pass from VideoWidget's property pane is being evaluated here.

Thanks for assigning me the issue, I will work and try to resolve it as soon as possible and also thanks for sharing the files concerned with the bug.
@aswathkk

@aswathkk : Got it.
If I use npm for running project and packages - its failing and works with yarn. Although both does the same work, so I used npm even doc says yarn to use.
I reverted my TS lint fixes as well from my previous commit that gives error with yarn start. Added fix for this issue and this time it works as well. For ref:

Fix: RegExp was not used on regex and regex was compared with string, so failing that returns always true of url.

This is no more an issue in current release


Update workflow.yml

Added CodeCov to GitHub Action

Welcome to Codecov :tada:
Once merged to your default branch, Codecov will compare your coverage reports and display the results in this comment.
Thanks for integrating Codecov - We've got you covered :open_umbrella:


Help: Reset to bootloader by software or hardware button

Hello,
Is possible to reset the MCU to bootloader starts?
The reset button, reset by software or watchdog only resets it to run the code flash, and not to bootloader.
EX software reset: PFIC->CFGR    = PFIC_RESETSYS | PFIC_KEY3
may miss some pointer to define the start reset vector flash if exits ?
this:
actual = Code flash start is 0x0800
need = Bootloader flash start is 0x1FFF
Thank  you,
VS

you can use this to reset into bootloader
FLASH->BOOT_MODEKEYR = FLASH_KEY1;
FLASH->BOOT_MODEKEYR = FLASH_KEY2;
FLASH->STATR = 1<<14; // boot into bootloader
PFIC->SCTLR = 1<<31; // Reset

but be aware that there is still a condition required for the bootloader to stay active or it will just boot the user-code again.

USB_DM needs to be low for few µs before you reset to ensure a USB re-enumeration. Depending on your schematic either disable the pull-up on D- or force D- low (output low).
You confirmed that the bootloader works otherwise on power-up, right?

USB_PIN_DM needs to be low for few µs before you reset to ensure a USB re-enumeration. Depending on your schematic either disable the pull-up on D- or force D- low (output low). You confirmed that the bootloader works otherwise on power-up, right?

MY D- pull-up is connected to the MCU GPIO.
Yes in power up the bootloader works by USB and wait 5 seconds, but with your code just wait 2/3 seconds.
so it I set the GPIO pull-up resistor before the code reset will work ?

I the the "USB_PIN_DM needs to be low for few µs before you reset to ensure a USB re-enumeration. Depending on your schematic either disable the pull-up on D- or force D- low (output low)." with no sucess.
So I see the "void boot_usercode()"  in bootloader example and try...
Solved by add the FLASH->CTLR = CR_LOCK_Set:
procedure:
FLASH->BOOT_MODEKEYR = FLASH_KEY1;
FLASH->BOOT_MODEKEYR = FLASH_KEY2;
FLASH->STATR = 0; // 1<<14 is zero, so, boot user code.
FLASH->CTLR = CR_LOCK_Set;
FLASH->STATR = 1<<14; // boot into bootloader
PFIC->SCTLR = 1<<31; // Reset
reset the device correctly, and goes to bootloader with USB detected just fine!
Thanks for the help @xsrf


fix: trim whitespace from signer key before parsing

#32  👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @L8D and the rest of your teammates on  Graphite

Merge activity

Sep 25, 8:02 PM EDT: @L8D merged this pull request with Graphite.


Bug: Cannot sell tokens

Describe the bug
Going to the view for a specific token and attempting to place a sell order does not work. It appears that the validation is acting as if the user is trying to replace a buy order.
To Reproduce
Steps to reproduce the behavior:

Go to the exchange view for a token you have a balance of greater than 0
Click, "Sell"
Attempt to place a sell order

Expected behavior
Validation should not be assuming I am trying to buy tokens, it should allow me to place a sell order.
Screenshots

This was caused by the token balance not being stored, thus breaking validation. Fixed.


Grafana Dashboard needs updated for newer APIs

Description
Observed Behavior:
Dashboard still uses older metrics and types (e.g. provisioner vs nodepool)
Expected Behavior:
Dashboard to be compatible with latest version
Reproduction Steps (Please include YAML):
Versions:

Chart Version:
Kubernetes Version (kubectl version):

Please vote on this issue by adding a 👍 reaction to the original issue to help the community and maintainers prioritize this request
Please do not leave "+1" or "me too" comments, they generate extra noise for issue followers and do not help prioritize the request
If you are interested in working on this issue or have submitted a pull request, please leave a comment

Looks like the json source has been updated in the repo but the dashboard needs updated in grafana.com
https://github.com/aws/karpenter-provider-aws/blob/d678b0d7907118b141b31767e6c72aaa90becd90/designs/metrics.md?plain=1#L186

I've reached out to the person who made the dashboard to see if they can update the dashboard.

Updating further, we don't have plans to maintain and upkeep this dashboard. Thoughts on if we should delete it or just keep it around knowing it's stale?

I'm going to take the opinion that this is something we don't need to maintain, but will keep up to not break anyone who's still relying on it. Feel free to cut an issue if you feel strongly the contrary


more info / Chef recipe about what is include in various AMI's (GPU drivers, etc)

Bug description and how to reproduce:
There is no easy way to know if the bundled AMI's contain cuda drivers, and which version, as well as other software.  Is there some chef recipe that I can refer to that describes how each AMI (alinux, alinux2, etc) is built
Thanks!

Hi @habemus-papadum
in the aws-parallelcluster-cookbook- default.rb file you can see the version of the software associated with the released with the 2.6.1.
You can use the Tag selector to switch between ParallelCluster versions and verify the software installed within that specific release.
Another way to retrieve this information is by checking the changelog or the releases page.
Let us know if it helps.

Brilliant! Thank you.


fix: mappings in Dockerfile

README.md uses mappings.txt, says it reads from a single file, not a directory update Dockerfile to reflect that

Superseded by 1a8a47


ci: add winget releaser workflow

Resolves #735

This action automatically generates manifests for Winget Community Repository (microsoft/winget-pkgs) and submits them.
lsd has been added to Winget in https://github.com/microsoft/winget-pkgs/pull/118210, and this workflow will be used to update it.
Before merging this:

Add a classic PAT with public_repo scope as a repository secret named WINGET_TOKEN. You can go to this URL to create it: https://github.com/settings/tokens/new?scopes=public_repo

Fork https://github.com/microsoft/winget-pkgs under @lsd-rs. The action will use that fork for making a branch and creating a PR with the upstream winget-pkgs repository on every release.
Install Pull on the winget-pkgs fork to ensure that it is constantly updated.

If you want to see an example of a PR created using this action, see microsoft/winget-pkgs/pulls (Pull request has been created with WinGet Releaser).

TODO

[ ] Use cargo fmt
[ ] Add necessary tests
[ ] Add changelog entry
[ ] Update default config/theme in README (if applicable)
[ ] Update man page at lsd/doc/lsd.md (if applicable)

Bump @zwpaper

Sorry for the late reply, I am not a windows user. the PR seems detailed enough, thanks for the help and let's merge it to see whether magic will happen

@zwpaper Windows build failed https://github.com/lsd-rs/lsd/actions/runs/10475485501? (Winget releaser depends on it)

@sitiom it seems CI has some issue and gnu build for windows is totally broken, I have to spend some time to look into it


Voted back in Producer Watermark fix

#3835
Moved watermark code to on_block and to check if the future block would result in the schedule changing to prevent causing a producer from dropping a block when it was put back in the production schedule after being removed a long time in the past.
Added initial long running test for voting that exposed this problem.

@wanderingbort can you verify the change in producer_plugin.cpp?  I am using generate_next and maybe_promote_schedule because when I used next(...) passing in a fabricated header, there were odd problems, so I only did the minimal parts to get the schedule to promote.
@heifner can you review the test changes and new test?  I will do more cleanup and additions later, just wanted to get in a test that would cause this problem to exhibit if it were still present.

Something went wrong, closing and resubmitting


Fix aspnet home git link

The existing form of the git link for aspnet home returns an "access denied" error when you try to use it for cloning. It is replaced with the public repository link that GitHub suggests.

Hi @drinkbird, I'm your friendly neighborhood .NET Foundation Pull Request Bot (You can call me DNFBOT). Thanks for your contribution!

This seems like a small (but important) contribution, so no Contribution License Agreement is required at this point. Real humans will now evaluate your PR.

TTYL, DNFBOT;

@drinkbird Thanks for this, but I am going to retire this document in favor of docs.asp.net. So I am going to close this one, I will stop linking to this shortly and then delete it at some point in the future.
Keeping this and the docs site up-to-date doesn't seem very valuable. I don't think docs.asp.net gives instructions on cloning the repo, so they probably don't have this issue.


DevContainer move to single dockerfile from compose

Hi,
This is a quick set of tweaks to make the devcontainer a bit more friendly and easy to get started with on different platforms (linux, mac etc).
Some of them are quite opinionated so feel free to push back on changes if they don't fit general workflow the team is currently using.
Changes

Add alias for Kubectl
Add kubectx to allow easy switch between kind cluster and remote clusters
Install git prompt and bash completion for kubectl, kubectx etc

@Azadehkhojandi Thanks for the review, I've fixed those ones up now. Sorry for the delayed response my daughter has been unwell so I've been away.


Crash reports are very delayed.

What App Center service does this affect?
Website -> Diagnostics -> Issue
appcenter-crashes.Crashes
Describe the bug
Crashes in react-native apps take a long time (10 minutes) to show up on the  Diagnostics -> Issue tab on the website.
To Reproduce
Steps to reproduce the behavior:

Generate a test crash using Crashes.generateTestCrash()
Visit  Diagnostics -> Issue tab on the app webpage.

Expected behavior
Issues should be displayed there instantaneously.
Desktop (please complete the following information):

OS: Windows
Browser Chrome
Version 74

Smartphone (please complete the following information):
Android  9 Emulator
Additional context
   <string name="appCenterCrashes_whenToSendCrashes" moduleConfig="true" translatable="false">ASK_JAVASCRIPT</string>
has been written to strings.xml
and
 Crashes.setListener({

      shouldProcess: function (report) {
          return true; // return true if the crash report should be processed, otherwise false.
      },

      // Other callbacks must also be defined at the same time if used.
      // Default values are used if a method with return parameter is not defined.
  });

has been added to the constructor of the default class.

Also, errors generated by throw new Error('blah') do not appear at all (even after a day). Is there something that I am missing?

@frags51 thanks for reporting this. Are you still experiencing these delays? Can you start a conversation in the blue chat bubble on the bottom right? This will put you in touch with our support and engineering teams so they can investigate for you.
Thanks!

Me too, Crash report for React Native is not working in Android...

@jinyang1994 sorry to hear. Can you also please start a support ticket in App Center? Either the blue chat bubble on the bottom right or the via the question mark icon on the top. Thanks!


fix: Xcode 12 compatibility

Does any other open PR do the same thing?
No.

What issue is this PR fixing?
The latest Xcode 12 fails to build when a module does not depend on React-Core directly. This change is necessary for all native modules on iOS. For details please see: https://github.com/facebook/react-native/issues/29633#issuecomment-694187116
How did you test this PR?

Build an App with Xcode 11.x. Result: The app builds.
Build an App with Xcode 12. Result: The app will not build when depending on React.
Build an App with Xcode 12 with the PR changes applied. Result: The app builds.

LGTM @christopherdro 🐽

@Fetten could you please sync and one travis passes I'll merge ? 😻

😻


How to get the decoded frame data (yuv) in CicadaPlayer

Thanks

Thanks

Hi, there's no existing API to get yuv, but the core has implemented, you can reference the bridge example
between  CicadaPlayer.java:
public interface OnVideoRenderedListener{
void onVideoRendered(long timeMs , long pts);
}
and MediaPlayer.cpp:
MediaPlayer::SetAudioRenderingCallback.
, to build your own OnRenderFrameCallback and this callback has yuv data.


Dd/improve input validation

Codecov Report

Merging #413 into master will decrease coverage by 0.11%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #413      +/-   ##
==========================================
- Coverage   99.89%   99.78%   -0.12%
==========================================
  Files          25       25
  Lines        1832     1822      -10
  Branches      293      293
==========================================
- Hits         1830     1818      -12
  Misses          2        2
- Partials        0        2       +2

Flag
Coverage Δ

#unittests
99.78% <ø> (-0.12%)
:arrow_down:

Impacted Files
Coverage Δ

src/schemathesis/cli/__init__.py
98.33% <ø> (-1.67%)
:arrow_down:

src/schemathesis/runner/__init__.py
99.15% <0%> (-0.85%)
:arrow_down:

src/schemathesis/extra/pytest_plugin.py
100% <0%> (ø)
:arrow_up:

src/schemathesis/models.py
100% <0%> (ø)
:arrow_up:

src/schemathesis/runner/events.py
100% <0%> (ø)
:arrow_up:

src/schemathesis/lazy.py
100% <0%> (ø)
:arrow_up:

src/schemathesis/_compat.py
100% <0%> (+12.5%)
: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 9a385f5...86c9a52. Read the comment docs.


Copyright information

Can you please add your copyright information in the license file?
If possible, we also need a tag for 1.0.5 version in the git repository.

@nerminator , he-he, I hardly understand what you're talking about :) I'm not into law, so, if you guide me with more details, I'll appreciate a lot :)

Your first line in your License file starts with "The MIT License (MIT)..."
License File:
https://github.com/egslava/edittext-mask/blob/master/LICENSE
Just after that line put something like:
Copyright (c) 2018 YourName YourSurname
You can put company name or any other name if you do not want to give your name. Some people give their website name for example.
You can have a look at the following license file for reference:
https://github.com/afollestad/material-dialogs/blob/master/LICENSE.txt
And we need a tag for 1.0.5 please, if it is possible

Sorry, it took me quite long to response :) But:

The file you're pointing to is no longer available.
If it's still actual, version 1.0.5 now is available.


docs_dir prefixed incorrectly for linked documents

I would like to generate public API documentation. I have configured the following:
RspecApiDocumentation.configure do |config|
  config.docs_dir = Rails.root.join("public", "api")
end

Documentation is now written to public/api. I can access index.html via path /api.
The problem is that the template is rendering the hyperlinks incorrectly: orders/returns_all_orders.html
I would expect this to be: api/orders/returns_all_orders.html which is also the actual path and works when entered in the browser.

I found the solution. In development simply going to /api would result in the /api/index.html page being displayed.
When going to /api/index.html directly the other links are okay. I now added this to my routes:
get "/api", to: redirect("/api/index.html")
This results in everything working as desired.


operator project-quay (3.10.0-nightly.20230929)

Add 3.10.0-nightly.20230929 to community operators

/merge possible

/merge possible


Chess Games Analysis

Can I add Some Chess Games with full Information about Games,Players and even the Moves the Player Played the Match with the Results?

Hi @guptavaibhav159 Quite interesting idea! 🔥  Thank you for your message. You may if that the content does not violate any copyrights & please make sure how about GDPR in this case.
If everything is fine, please create a new Folder for the items (similar like it is with media section) & link them to the main file (about.md).
Thanks! Best wishes 🦄

Yeah Sure.
On Sun 11 Oct, 2020, 7:35 PM Michal<EMAIL_ADDRESS>wrote:

Hi @guptavaibhav159 https://github.com/guptavaibhav159 Quite
interesting idea! 🔥 Thank you for your message. You may if that the
content does not violate any copyrights & please make sure how about GDPR
in this case.
If everything is fine, please create a new Folder for the items (similar
like it is with media
https://github.com/mbiesiad/awesome-chess/tree/master/media section) &
link them to the main file (about.md).
Thanks! Best wishes 🦄
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/mbiesiad/awesome-chess/issues/16#issuecomment-706709694,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ARFK5MG6TISAGEWBDVSXCFDSKG3RJANCNFSM4SLWAQIA
.

If you have any questions, feel free to ask. Can I close this issue?

Yup You May Close It. I will Post My PR soon.
On Sun 11 Oct, 2020, 8:19 PM Michal<EMAIL_ADDRESS>wrote:

If you have any questions, feel free to ask. Can I close this issue?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/mbiesiad/awesome-chess/issues/16#issuecomment-706716036,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ARFK5MHCN7JWPIIKA6FAHI3SKHAVZANCNFSM4SLWAQIA
.

Cool, thanks for quick answer. Keep in touch!


feat: track logs in git

also optional tracking of log files in a git repo.
daylog init -p project will create a new git repo for the specified project. if a project has a repo, log updates are committed automatically.

meh


Arkansas 2023 Joint 81K wages one dep

Arkansas 2023 Joint 81K wages one dep
http://taxsim.nber.org/out2psl/test33
The emulator provides no intermediate data, such as standard deduction or taxable income, so I can't guess as to the problem.

closed with PR #88


There are  no changelog and licence button in the addon setting page

Check that again. I have both those buttons and the 'view code' button

now working.
Thanks


Markdown setup

In this PR I’ve altered the setup of the repository to allow for markdown partials to be injected into the html source files.
This allows the content of the project to be separated into smaller easier to read markdown files.

Should probably create a build task 🤔


Styles are broken if hook names are not valid CSS identifiers

I initially tried configuring my hooks like this because I wanted to remove any aliasing and make it feel closer to raw CSS:
createHooks({
  "@media (prefers-color-scheme: dark)": "@media (prefers-color-scheme: dark)",
  "@media (min-width: 640px)": "@media (min-width: 640px)",
  "@media (min-width: 768px)": "@media (min-width: 768px)",
  "@media (min-width: 1024px)": "@media (min-width: 1024px)",
});

However this leads to invalid CSS:
*{--@media (prefers-color-scheme: dark)-0:initial; /* etc. */ }

What do you think about mapping hook names to short identifiers internally so that you're free to name them whatever you want? This would also help make the generated styles a bit shorter too.

Thanks for the report @markdalgleish. I think this could be tackled in two phases:

Quick fix: Update the type definition for the createHooks function to prevent this (currently) invalid configuration.
Review and implement the internal mapping you described.

Regarding the second point, the main trade-off that comes to mind is that a little bit of "readability" would be lost in the generated style sheet and style attribute value. For me, that's not much of a problem, but it's just something that might pop up in another issue later.

Just stumbled over this repo and have to say this is exactly what I was looking for!
FWIW I'd also like to author my styles without aliases if possible. Less magic and more familiar I'd assume.


Hide GPG signature auth method in Login form

We dont need GPG signature auth method in SS Login form

Done


Reload protection grade cell when updaing UI

Task/Issue URL: https://app.asana.com/0/414709148257752/1198191970712393
CC:
Description:
Grade section on Privacy Protection screen is not reloaded properly.
Steps to reproduce: Open privacy Practices screen, toggle protection.
Steps to test this PR:

Open privacy Practices screen
Toggle protection.

Protection grade and description should be updated.

Copy Testing:

[x] Use of correct apostrophes in new copy, ie ’ rather than '

Orientation Testing:

[x] Portrait
[x] Landscape

Device Testing:

[x] iPhone SE (1st Gen)
[x] iPhone 8
[x] iPhone X
[x] iPad

OS Testing:

[x] iOS 11
[x] iOS 12
[x] iOS 13

Theme Testing:

[x] Light theme
[x] Dark theme

Internal references:
Software Engineering Expectations
Technical Design Template

Thanks @brindy


Modifying response after .toJSON()

I use .toJSON(...) in my models to define how they should appear in JSON responses - however I'm running into some trouble.
It seems that I cannot modify this object after it has been created. This is a problem because in many cases I'll have a property that I do not wish to be available under most circumstances, however there are a few cases where I do need to expose that property.
One example is a User's email - I don't want it to show up in the /users list, but I want it to show up when I query /me.
The current code I have for /me is something like this:
User.findOne({ id: req.user.id })
        .exec(function(err, user) {
            if(err) return res.negotiate(err);
            if(!user) return res.notFound();
            Alias.find({ owner: user.id }).populate('platform').exec(function(err, aliases) {
              if(err) return res.serverError(err);
              user.aliases = aliases;
              var json = user.toJSON();
              json.settings.passwordDetails = user.passwordDetails;
              json.email = user.email;
              res.json(json);
            });
        });

I would expect that the JSON response should include an email property, however it is removed. I checked to make sure that the json.email property does in fact exist, and it does if I console.log the json object. What gives? (Note: The property json.settings.passwordDetails DOES work as expected (settings is a property of type json on User) - which I don't fully understand though I have a few ideas.)

Is this resolved?


[Chart] enable to setup more then one hostname (fqdn)

What kind of request is this (question/bug/enhancement/feature request):
feature request
Steps to reproduce (least amount of steps as possible):
NA
Result:
acces service by more then one name
Other details that may be helpful:
Nowadays we usually can deploy Rancher like this:
helm install rancher rancher-latest/rancher \
  --namespace cattle-system \
  --set hostname=rancher.my.org

It would be good if could do set two hostnames for it, e.g.:
helm install rancher rancher-latest/rancher \
  --namespace cattle-system \
  --set hostname=rancher.my.org,rancher.local

Environment information

Rancher version (rancher/rancher/rancher/server image tag or shown bottom left in the UI): 2.x
Installation option (single install/HA): HA

Cluster information

Cluster type (Hosted/Infrastructure Provider/Custom/Imported): NA
Machine type (cloud/VM/metal) and specifications (CPU/memory): NA
Kubernetes version (use kubectl version): NA

NA

Docker version (use docker version):

NA

Ops!

Hi


Potentially outdated manifest.json.elevenlabs.example

I couldn't get ASTRA to work using the elevenlabs example manifest.json. When I used the manifest.json.example and manually added replaced the azure_tts to the elevenlabs_tts it functioned properly.
In the diff I notice two differences:

It uses version 0.1.0 instead of 0.2.0
Doesn't contain the pcm_frame.

pcm_frame will be removed from extension agora_rtc in the upcoming version, consider it fixed. :)
BTW, nice found. @tadaspetra

With the latest code, elevenlabs config is in agents/property.json, fill out the keys, once that's done. Swap the value in 	graphNameDefault = "" from config.go, then you should be good with elevenlabs. :)


Guardian.Plug.EnsureAuthenticated on failure using wrong controller

Hi there,
Quite new to elixir and phoenix so apologies if this isn't a bug but a mistake in my code, I had a look at the code and couldn't see what was wrong with it to be able contribute a fix or confirm if it is a bug in Guardian or I'm just doing something wrong.
Basically when I use
plug Guardian.Plug.EnsureAuthenticated, on_failure: { SessionController, :new }

In a controller and the on_failure triggers rather than send me to the :new action of the SessionController it seems like it's trying to send me to the :new action of the current controller. I've attached a screenshot of the error and the code the code I'm getting this error with is here: controller where EnsureAuthenticated is being called, https://github.com/oscarduignan/phoenix-time-tracker/blob/master/web/controllers/user_controller.ex#L7; and the action it's trying to use for on_failure, https://github.com/oscarduignan/phoenix-time-tracker/blob/master/web/controllers/session_controller.ex#L6-L8.

Hi @oscarduignan. I think this is because in the SessionController#new function, when you call render you'll need to supply it with the view.
render conn, Session.View, 'new.html', stuff: 'and things'

Unfortunately by the time we get to the view the connection has already cached what view it's going to use by default so you'll need to be explicit.
HTH
Daniel

Thank you Daniel that did it! Still getting my head around Phoenix so I'm very greatful for your help solving that and thank you for all the work you've put into Guardian, it's proving really useful :+1: hopefully I'll be able to contribute something back soon as I get to know it and Phoenix better

Great to hear :) I always love help with Guardian. Good luck with Phoenix.


README Webpage tutorial link is broken

Apparently it should now point to: http://codigogenerativo.com/code/kinectpv2-k4w2-processing-library/

thank you!, fixed 3bda24ba8b7c62155bf308c0d86c961ca89dbfa3


Page description

Duplicates

[X] I have searched the existing issues

Latest version

[X] I have tested the latest version

Summary 💡
Make it possible to describe what the page is about. It's similar to https://github.com/mui/mui-toolpad/issues/417 but about the page specifically
Examples 🌈

I have built https://master--toolpad.mui.com/_toolpad/app/cl9hns2z30002pr9kzw4dqx76/pages/k103mb4

to help with https://www.notion.so/mui-org/KPIs-1ce9658b85ce4628a2a2ed2ae74ff69c#f3c0c606c62148f1bd06784f367bd8c4. But it's not clear what this Toolpad page is about when you open it.

Metabase

https://mui.metabaseapp.com/question/5-mui-store-missing-transfer-number

Retool has it as a paid feature

Motivation 🔦
No response

Relates to https://github.com/mui/mui-toolpad/issues/754.
In the above issue, I proposed to rename from the page menu. Should we rather support inline editing for page name and description?


instance_set_transform: Condition ' Math::is_nan(v.x) ' is true

Godot version
3.3.2 Stable Flathub
System information
GLES3  ||  Windows 7 NVIDIA GTS 450 / Windows 10 NVIDIA 2070 SUPER / Fedora 34 AMD Radeon R7 M340
Issue description
Sometimes my game hangs and starts spamming instance_set_transform: Condition ' Math::is_nan(v.x) ' is true to the console. When that happens, the game stops rendering and outputs just a plain color to the screen only.

Windows 7 NVIDIA GTS 450
Happened very often. Happened even when the game was idling. Not only the game hang, it also crashed.
Windows 10 NVIDIA 2070 SUPER
Happened somewhat often. Happened when the game removed and created the camera again on a scene change.
Fedora 34 AMD Radeon R7 M340
Happened rarely, when the game removed and created the camera again on a scene change.

Additionally:

I used Godot 360 in this project.
Bug happened more frequently when I enabled multiple threads in the project settings.
Never happened on Windows 7 with Intel HD Graphics 5500

I strongly suspect that it's related to the camera translation code. I think it stopped occurring when I removed everything that was related to the translation in it. Also, there is code that is translating the camera on level changes that may trigger the bug. (not the code itself, but the act of translation)
I apologize for uploading a complete 180MB project instead of a minimal reproduction one, I can't find time to make a minimal project...
Could be related to some of these: #32611, #39738, #35913, #50384
Steps to reproduce

Skip the intro by pressing Z multiple times.
After that either just wait 10-20 minutes or...
2.1 Make the character enter the other location. (just go up and you will stumble upon a door) You may need to re-enter that location multiple times (five, ten or maybe more)

Eventually the bug will occur. If it does not occur, enable multiple threads in the project settings and try again.
Minimal reproduction project
https://drive.google.com/file/d/1Co-h-YtVqOETfqg5HVUlPmYC-4Buqj8O/view?usp=sharing

Also experiencing this bug and it's really hard to debug. It would be a great help if the error log message would contain the path of the problematic node or any other info which could help identify what object or part of GDScript caused the problem.
Currently, it's not possible to say if this happens due to a bug in the physics engine or is caused by something in gdscript.

We are having the same issue with this small amateur project, whatever the platform (though we target mainly HTML5).
The player can "take" items in their hand, in which case the item will be tracking the hand position and try to move there through velocity tweaking (similar to how steering behavior would have been, but we don´t have a general solution for that so went for the simplest). When moving close to another block (as in the "box" provided by default by godot) we end up with the item being stuck in some corner of blocks and the logs showing the same is_nan failure.
The player moves through move_and_slide, I don't know if thatś important though.
The player movement code: https://github.com/gamkedo-la/the-house/blob/main/player/player.gd#L66-L87
The code moving the item in the hand, which tries to track a specific spatial node: https://github.com/gamkedo-la/the-house/blob/main/items/item.gd#L110-L124
We have been seeing the issue since the beginning of the project as taking an object was implemented very early. Even when taking was implemented by just moving the item's node into the hierarchy of the player, we would hit the issue. So I suspect it have to do with how move_and_slide is implemented or used?

I cannot give you my code at this point, and this quite hard to reproduce with my project anyway… But I assure you move_and_slide_with_snap can return Vector3(nan, nan, nan), and I believe move_and_slide can too.
You can check like this:
var new_velocity := move_and_slide(velocity)
if new_velocity != new_velocity:
    breakpoint

velocity = new_velocity

This breaks on nan since nan != nan is true.

Was a solution for this ever found? My project which worked fine beforehand spontaneously started throwing this error every time I start it, making it completely unusable.

This is still among the issues I keep having in my project (source above).

So far the only workaround that worked for me was to release the game in Debug :/

So far the only workaround that worked for me was to release the game in Debug :/
I suspect that there are uninitialized variables at play in the C++ code of Godot.

I doubt it's a bug in Godots C++ code, it's more likely that you are passing NaN to it somewhere from your code. In my case, there was a part of the code that recursively increased the x coordinate until it exceeded the max int value. From the error, it was not clear which part of our gd script code caused the error, I found it by analyzing all the recent changes, and it was still not obvious. I 'v been lucky to find it and fix it.
The message for this type of error should be more precise.

I doubt it's a bug in Godots C++ code, it's more likely that you are passing NaN to it somewhere from your code.

While that would be my initial thought too, does this reasoning still holds if Debug mode does not expose the issue at all though? Because this is what I'm observing in my project.
I dont know enough yet of how Godot is implemented to understand the issue either. I intend to maybe try to build a debug version of godot and see for myself (I'm more versed in C++ than Godot/GDScript) If Godot does something to floats calculations in Debug, that might mean you are correct, I'll try to check more potential sources.
Also note that my project's code is readable there: https://github.com/gamkedo-la/the-house
See the code using the move functions there: https://github.com/gamkedo-la/the-house/blob/main/player/player.gd#L200-L233 (current version also have the workaround specified few messages before, but it does not work).
In this code speed is basically a runtime constant (depending on walking, crouching etc.), movement_translation is built using vector3 constants normalized, oriented movement is just movement_translation rotated through a valid quaternion (because it comes from Godot), gravity is litterally the value from the editor 9.x something, and we pass oriented_movement directly to the move functions. So in this speicfic code I'm not seeing how NaN could come in, there is no additions. But I might be indeed missing something.
To reproduce the issue:

build a Release version of the game
"Start" the game, walk around
If you take an object (left click on one of the mushrooms for example) you might hit the issue faster
When the issue hits, you will get major performance drop, 1fps slideshow.

I have many mitigation in the code of the movement of the items: https://github.com/gamkedo-la/the-house/blob/main/items/interactive_item.gd#L159-L201
Basically when held by the player, they just follow the hand's position, in the same fashion as simple "arrive" steering behavior would do.
The issue seems to be detected in this code more often, but basically this is just reducing the distance between the object's position and the hand's position, so not sure how NaN get introduced here. Here base_linear_velocity comes from the code previously linked.
I'll have to investigate more I guess.
I'm interested if you find anything in this code that would point the issue.


[Handles] integration tests

Lets try and just copy the happy path we already have and expand in then future.

Puneet wrote some end-to-end integration tests.  We need to determine what others, if any, are needed.


Naming of example Delegate types doesn't conform with Naming Guidelines

The naming of example delegates in this document doesn't conform to rules listed in Design Guidelines - Names of Common Types - Delegates which state:

√ DO add the suffix "EventHandler" to names of delegates that are used
in events.
√ DO add the suffix "Callback" to names of delegates other
than those used as event handlers.
X DO NOT add the suffix "Delegate" to a delegate.

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

ID: c6b07199-b747-9596-1d38-3257f43b4833
Version Independent ID: d162375c-ef5a-41c1-fe26-2e6394bf1df1
Content: How to: Declare, Instantiate, and Use a Delegate - C# Programming Guide
Content Source: docs/csharp/programming-guide/delegates/how-to-declare-instantiate-and-use-a-delegate.md
Product: dotnet-csharp
GitHub Login: @BillWagner
Microsoft Alias: wiwagn

Thank you for opening this issue. I'll add it to the backlog. Good catch.
Thanks again!

@BillWagner I wrote a pull request to at least partially resolve this issue. #22497

@BillWagner I wrote a pull request to at least partially resolve this issue. #22497


Manager.GetCircuit should work on an empty manager

On an empty manager object, it should return nil

I was looking at issues in this repo to contribute to. Either this isn't an issue any longer or this needs more info. There appears to be a test for this:
func TestManager_Empty(t *testing.T) {
	h := Manager{}
	if h.GetCircuit("does_not_exist") != nil {
		t.Error("found a circuit that does not exist")
	}
}

If I'm misunderstanding the issue and this needs work please let me know.

Hmmm perfect!  Thanks for catching this.


Support EntitySpec ConfigKeys type.

Adding support for creating a EntitySpec from a ConfigKey

@sjcorbett  and @robertgmoss,
does it make sense for you?
Thx

I think this PR is the wrong approach. It leaks Brooklyn into TOSCA:
    properties:
      dynamiccluster.memberspec:
        $brooklyn:entitySpec:
          type: org.apache.brooklyn.entity.webapp.jboss.JBoss7Server

Someone writing TOSCA shouldn't be thinking about $brooklyn:entitySpec.
I'd prefer to explore some different approaches.
One is to use the scalable capability. I imagine that the transformer would recognise the capability and turn such specs into cluster+memberSpec.
Another idea is to represent both clusters and their members in the blueprint and to link them with a relationship. For example, something like:
    relationship_types:
      brooklyn.relationship.ClusteredOn:
        derived_from: tosca.relationships.HostedOn

Along with a node type using the relationship in a requirement with lower_bound and upper_bound both one. The transformer would recognise the relationship and configure the source as a member spec on the target.
My preference is the second approach.
@kiuby88 What do you think?
@robertgmoss (and @ahgittin) do you have any additional comments?

Completely agree.  While it might be useful to permit this a tosca user would probably expect to define the entity spec as a tosca element which this node references somehow.
As an aside I'm not sure this is supported by the tosca spec.  Does it allow freeform maps?  I've only seen it allowing maps where the keys and values are completely described.
Returning to the thread, i think tosca assumes that nodes are all deployed -- so you can't really include a spec as a node.  It has node_templates for that.  But node_templates have a couple problems I think:

can we target a relationship at a node_template?
how would we customize it, e.g. to make it aware of a database?

maybe we could define a new "NodeTemplateUsedAsASpecInANode" type consisting of:
type: <string>  # a node_template
properties: <map>  # permitted keys and values are dependent on "type"

and then we could have
properties:
  dynamiccluster.memberspec: <NodeTemplateUsedAsASpecInANode>

or maybe it could even be part of a relationship, USES_TEMPLATE and dynamiccluster.memberspec comes from that relationship ?

Thanks for your replies so interesting, @ahgittin and @sjcorbett.
Really, my proposal does not make sense with TOSCA philosophy. I am trying to model a cluster configkey and I used an property directly, I forgot the relationships.
Then, I agree with use a relationship to configure the property. I think you propose something like the following:
cluster:
      type: org.apache.brooklyn.entity.webapp.ControlledDynamicWebAppCluster
      properties:
        ...
nodeSpec:
  type: NodeTemplateUsedAsASpecInANode
      requirements:
        - childOf:
            node: cluster
            relationship: tosca.relationships.ConfigureSpecAsProperty
            properties:
              prop: dynamiccluster.memberspec

However, I think the requirement should be from cluster side (relation source). I think the requirement should be defined by the node which needs the entitySpec. This may allow to avoid new type definitions as NodeTemplateUsedAsASpecInANode. When the topology is processed each node is transformed in a EntitySpec. Then, when the relations are managed the target node is added to the source node as a property.
cluster:
      type: org.apache.brooklyn.entity.webapp.ControlledDynamicWebAppCluster
      properties:

      requirements:
        - memberSpec.req:
            node: nodeSpec
            relationship: tosca.relationships.ConfigureSpecAsProperty
            properties:
              prop: dynamiccluster.memberspec

nodeSpec:
  type: org.apache.brooklyn.entity.webapp.jboss.JBoss7Server

We do not need to define new types, we only need to add the requirements to the types which define  ConfigKey<EntitySpec>.
WDYT?

Like this direction.
But how is it known that cluster is to be deployed but nodeSpec is not (it
is just a template)?
On 29 Jan 2016 18:18, "Jose"<EMAIL_ADDRESS>wrote:

Thanks for your replies so interesting, @ahgittin
https://github.com/ahgittin and @sjcorbett
https://github.com/sjcorbett.
Really, my proposal does not make sense with TOSCA philosophy. I am trying
to model a cluster configkey and I used an property directly, I forgot the
relationships.
Then, I agree with use a relationship to configure the property. I think
you propose something like the following:
cluster:
type: org.apache.brooklyn.entity.webapp.ControlledDynamicWebAppCluster
properties:
...
nodeSpec:
type: NodeTemplateUsedAsASpecInANode
requirements:
- childOf:
node: cluster
relationship: tosca.relationships.ConfigureSpecAsProperty
properties:
prop: dynamiccluster.memberspec
However, I think the requirement should be from cluster side (relation
source). I think the requirement should be defined by the node which needs
the entitySpec. This may allow to avoid new type definitions as
NodeTemplateUsedAsASpecInANode. When the topology is processed each node
is transformed in a EntitySpec. Then, when the relations are managed the
target node is added to the source node as a property.
cluster:
type: org.apache.brooklyn.entity.webapp.ControlledDynamicWebAppCluster
properties:
  requirements:
    - memberSpec.req:
        node: nodeSpec
        relationship: tosca.relationships.ConfigureSpecAsProperty
        properties:
          prop: dynamiccluster.memberspec

nodeSpec:
type: org.apache.brooklyn.entity.webapp.jboss.JBoss7Server
We do not need to define new types, we only need to add the requirements
to the types which define ConfigKey.
WDYT?
—
Reply to this email directly or view it on GitHub
https://github.com/cloudsoft/brooklyn-tosca/pull/71#issuecomment-176894540
.

@kiuby88 do you want to do anything with this or can it be closed?

@robertgmoss I have closed this PR, I will be back here ASAP


Slider

Is the team working on any slider component?
I was expecting a slider in the Forms part in the storybook but couldn't find. Surprisingly, project hasn't needed it till now.
I require it for this feature:
Pausing notifications for a few minutes.
I want the user to be able to choose using a slider, although I can use a number input as well for the time being.
Expected behavior

Able to pass min and max value.
If possible, the component should handle checkpoints/marks/step-size.

Image

@umakantv We didn't have a case where we needed a more "advanced" range component. For our needs, we use the approach below, maybe it can help you.
<Box display='flex'>
    <Box is='input' type='range' value={100} min='0' max='100'/>
  <Tooltip placement='right-top' mis='x8'>{100}</Tooltip>
</Box>;

Thanks for your suggestion!


plasma membrane PIP level PMID:29975157

[quality] plasma membrane [substance] level for:
decreased phosphatidylinositol-3,4,5-trisphosphate
increased phosphatidylinositol-3,4,5-trisphosphate
decreased phosphatidylinositol-4,5-bisphosphate
increased phosphatidylinositol-4,5-bisphosphate
normal phosphatidylinositol-4,5-bisphosphate
decreased phosphatidylinositol-4-phosphate
normal phosphatidylinositol-4-phosphate
also see fypo:6222

decreased phosphatidylinositol-3,4,5-trisphosphate level in plasma membrane FYPO:0006623
increased phosphatidylinositol-3,4,5-trisphosphate level in plasma membrane FYPO:0006624
decreased phosphatidylinositol-4,5-bisphosphate level in plasma membrane FYPO:0006625
increased phosphatidylinositol-4,5-bisphosphate level in plasma membrane FYPO:0006626
normal phosphatidylinositol-4,5-bisphosphate level in plasma membrane FYPO:0006627
decreased phosphatidylinositol-4-phosphate level in plasma membrane FYPO:0006628
normal phosphatidylinositol-4-phosphate level in plasma membrane FYPO:0006629
edit file: 0ac528b14f75e5f8dab2c368f6361786fdf7327d
release: ed34ce74d1e2c18cf32e12deaa9efa80eed0d566


Cannot search APIs by custom properties in publisher and devPortal

Description:
$Subject

Steps to reproduce:

Create an API via publiser
Go to properties and add custom properties and save
Go to search bar and search by property ( syntax: property_name:property_value

Affected Product Version:

Environment details (with versions):

OS:
Client:
Env (Docker/K8s):

Optional Fields
Related Issues:

Suggested Labels:

Suggested Assignees:

Closed as not reproduced in the latest pack


not receiving the email

hi! please help me, I followed the instructions and ran the example, but upon checking the receiving gmail account. there is no email.

Hi darwinganata. I am not sure which example you are referring to but I think you are mentioning sending-a-csv-file-through-email-using-smtp. I tested the example and it works fine, so if this is the example you had trouble with, please check the SPAM folder of your e-mail account.


Add border to droparea

It can be kinda hard to see the target for dragging and dropping files. Add something like this maybe to .droparea:
border: dashed 3px gray;
border-radius: 5px;

The entire window is a valid target for dropping files, actually. Also, an older design had the dashed line (archive.org), but we dropped it later.
I'm open to re-adding the dashed line, though I liked the original look+feel more than the grey dashed line. Anyone else have thoughts? @jdiez17?

-1 on the dashed line, never liked it


fix to avoid nodus swallowing errors

sometimes an error doesn't have a code :feelsgood:

good work Jose finding that out!

Great improvement! thanks @josedahlquist

Nice :)


Add person image import

This PR adds the avatar image to the PersonalData model, and extends the import script so that images can be imported from the external data source.

I decided to start the simplest way.  Normally systems allow self-hosted avatars, and that should be a default, to my opinion.  Gravatars are a nice option but using them means depending on a third party service, and the user has to create one.
We can add support for gravatars later if we want.  I will file an issue for that.


test checkpoints via curl

Moved our test checkpoints around so they can be fetched directly via curl with no requirement of S3 setup for recipe tests.
This PR is from my fork so if CI is green then we're a-ok

Can you promise to block creation of new branches for everyone but pytorch org after this?


"Parallel" option for training? Parallel adapter outputs required (without interacting with each other).

Hello,
Thanks for this nice framework 👍 . I might be asking something that isn't yet possible but wanted to at least try asking!
I am trying to feed two BERT-based model's outputs to subsequent NN.
This requires having two BERT models to be loaded, however, the memory consumption becomes too high if I load two BERT models. To remedy this, I was wondering if I could do something like "Parallel" in training time.
(FYI, I am not trying to dynamically drop the first few layers and simply trying to create two BERT forward paths with lesser memory consumption)
I understand that active adapters can be switched by set_active_adapters().
(Actually, could you confirm if my understanding is correct?)
But, this doesn't seem to fit my purpose as, in my case, I need both adapters to output independent representation based on respective adapters.
Is there anyways that I can make adapters not interact with each other on the forward path while not loading original BERT parameters twice?

Making this question even more complex, I also need to make one adapter's parameters to be non-differentiable while requiring them in the forward loop.
Any ideas perhaps? :)

Hey @leejayyoon,
I think you should be able to implement all of what you are trying to achieve out-of-the box.
Just to clarify: what do you mean with "non-differentiable"? Do you want to freeze the parameters of one adapters and train  the parameters of another adapter?
In general I think you will want to use the Parallel functionality. If you add two adapters, the Parallel functionality will loop through the "grouped" batch, and only pass the examples through their respective adapters. The output representations will correspondingly be completely independent of the respective other adapter.
Given that you want a slightly more complicated setup you probably want something like this:
model.active_adapters = ac.Parallel(adapter1, adapter2)
during training you can iterated through
# only finetune adapter1, freezing adapter 2
model.train_adapter(adapter1)
...
# only finetune adapter2, freezing adapter 1
model.train_adapter(adapter2)

to only train the parameters of one adapter at a time.
@calpt and @hSterz will be able to help you more on the implementation side.

Hi  @JoPfeiff,
Thank you for your prompt response! It's impressive.
By non-differentiable, I meant freezing just as you have interpreted.
I see. I just wasn't sure whether Parallel functionality should be used as in the last line of this link, it says

Note that the Parallel block is only intended for inference, not for training adapters.

Is this not the case anymore?

I think the only reason is that it hasn't been tested in training scenarios yet. It should work though.

OK, good to hear that from you. However, I would suggest having a unit test for this in near future to provide some assurance!
What's the best way to do this? The straightforward way I can think of is training two adapters independently and check whether  Parallel produces the same adapters?

@JoPfeiff  One last clarification question!
In your example of freezing adapters 1 & 2, does model.train_adapter(adapter_name) only freeze pretrained model's (e.g. BERT) parameters and do not affect the rest of the computation graph at all?
Reading the code makes it clear, but wanted a confirmation to make it sure.
In train_adapter()--> self.freeeze_model(True) this line of code,
https://github.com/Adapter-Hub/adapter-transformers/blob/82a3d80a98d33610f38746ff72344c9d2fd66336/src/transformers/adapters/model_mixin.py#L515
, I am assuming self.base_model refers to the pretrained model part.

checking in again to get answers to this question! Thank you for your time in advance! @JoPfeiff

@JoPfeiff One last clarification question!
In your example of freezing adapters 1 & 2, does model.train_adapter(adapter_name) only freeze pretrained model's (e.g. BERT) parameters and do not affect the rest of the computation graph at all?
Reading the code makes it clear, but wanted a confirmation to make it sure.
In train_adapter()--> self.freeeze_model(True) this line of code,
https://github.com/Adapter-Hub/adapter-transformers/blob/82a3d80a98d33610f38746ff72344c9d2fd66336/src/transformers/adapters/model_mixin.py#L515
, I am assuming self.base_model refers to the pretrained model part.

model.train_adapter([ada1, ada2]) first freezes all parameters in the module base_model and then reactivates all adapters in the list, in this case ada1 and ada2.
base_model includes all transformer weights as well as the embedding layer. It does not include the prediction head.

I looked into it and the Parallel block should work for training out-of-the-box.  Only if you want to use the Trainer class it does not work in the current version. The output with parallel adapters is currently just a list of the outputs of the parallel heads without a combined loss, but the trainer class requires such a loss.

@hSterz Thanks for the reply.
I am not sure if I fully understood what you meant.
Are you saying there needs to be an extra modification on the Parallel block for it to have a backpropagatable loss function?
For my usage, I don't need to update two adapters simultaneously, rather, I need to update them in alternating fashion.

I looked into it and the Parallel block should work for training out-of-the-box. Only if you want to use the Trainer class it does not work in the current version. The output with parallel adapters is currently just a list of the outputs of the parallel heads without a combined loss, but the trainer class requires such a loss.

Yes, currently there is only a separate backpropagatable loss for each adapter in the parallel block (which can't be handled by the Trainer class). But from what I understand this could be sufficient for your case.

@hSterz  I see. thanks for prompt response! (I somehow missed this notification on my email)
It's nice that there isn't any expected problem.
I asked this earlier, but are you guys planning any unit test on the training with parallel block perhaps?
For additional background: I would be using it in the allenNLP framework, I don't know whether that would change the trainer story.

Yes, we are planning to add unit tests for parallel blocks.
I am not that familiar with the allenNLP framework. From a quick look, the trainer (like the GradientDescentTrainer) seems to have a similar problem to the adapter trainer because the model output has no aggregated loss attribute.

@hSterz Thanks for patienly answering my questions. 👍
I'll keep an eye for unit tests. Even better if you can notify here.
cheers!

The unit tests are now merged into the master branch.

Thank you @hSterz 👍

@hSterz  I actually skimmed through some of the commits you made. When I look at test_adapter_compositions.py, it seems not only Parallel has gone through unit tests, now you can backpropagate to multiple adapters together, is this correct?

Yes, a new MultiHeadOutput class is added which contains the sum of the individual losses of the heads. That allows us to backpropagate multiple parallel adapters together.


Importing CSS from node_modules no longer works

Reproduction
I've been testing this with Mantine and used their starter with upgraded dependencies:
https://github.com/ahallicks/remix-template
Clone this, npm i and then npm run dev and when you load the page you'll see messages in the console like:
Could not load<EMAIL_ADDRESS>
and no styles load.
System Info
System:
    OS: Windows 11 10.0.22631
    CPU: (16) x64 12th Gen Intel(R) Core(TM) i7-1260P
    Memory: 18.56 GB / 47.70 GB
  Binaries:
    Node: 22.11.0 - C:\Program Files\nodejs\node.EXE
    npm: 10.8.1 - ~\AppData\Roaming\npm\npm.CMD
    bun: 1.1.30 - ~\.bun\bin\bun.EXE
  Browsers:
    Edge: Chromium (130.0.2849.68)

Used Package Manager
npm
Expected Behavior
These imported styles should be automatically bundled and loaded on the front end.
Note that I have also tried importing them as paths and using the LinksFunction but this still doesn't work.
Actual Behavior
Any styles imported this way do not load as Remix can't find them:
Could not load<EMAIL_ADDRESS>
For the example repo this means that almost no styles load at all.

Remix does not support Vite 6 yet, so probably downgrading Vite to v5 would solve your issue.
I made two PRs that fixes the incompatibilities last week (https://github.com/remix-run/remix/pull/10254, https://github.com/remix-run/remix/pull/10255), hoping to get them merged. 🫡

Remix does not support Vite 6 yet, so probably downgrading Vite to v5 would solve your issue. I made two PRs that fixes the incompatibilities last week (#10254, #10255), hoping to get them merged. 🫡

Thank you! This fixed it. Not sure how I missed that Vite had sneakily upgraded to 6.

Same issue. For me all css files like tailwind, fontsource are not working. I came across your question and the answers. At least I found a workable solution. Thanks.

Same issue. Downgrading vite to v5 fixed it for me. Thank you!


updates for adding pangenesets column and linkouts for gene families …

…+ pangenesets columns.
also updated to use webcomponents 1.4.0

as pair-programmed if not peer-programmed

works for me too, thanks


Fix build script always using python3 in OSX builds

Hi!
I noticed that when compiling pyo3 on OSX for a non-standard Python interpreter (such as PyPy), the build script would at some point assume python or python3 to be the name of the interpreter, despite PYTHON_SYS_EXECUTABLE being set. This was caused by get_macos_linkmodel using the default PYTHON_INTERPRETER eventually.

Thank you for contributing to pyo3!

Run cargo fmt (This is checked by travis ci) :white_check_mark:
Run cargo clippy and check there are no hard errors (There are a bunch of existing warnings; This is also checked by travis) :white_check_mark:
If applicable, add an entry in the changelog. :white_check_mark:

the build script would at some point assume python or python3 to be the name of the interpreter, despite PYTHON_SYS_EXECUTABLE being set.

Ah, it's because of #596.
Sorry, I overlooked the problem when reviewing, and thank you for fixing it!


Ui changes

I noticed beehive_url was actually pointing to honeycomb, so I added honeycomb_url to correct that.
This adds an icon for DEC with a link to the DEC homepage.

:+1:

:+1:


Cuda Memory Exception

Hi,
First of all, thank you so much for this fantastic library!
I encountered CUDA memory exceptions when using free camera views that were different from the original cameras in the test dataset (e.g., counter).
My setup is as follows:

CUDA version: 12.4
GPU: NVIDIA RTX A2000 Laptop GPU with 4GB memory
OS: KDE Neon 22.04
Development environment: C++ project set up with CMake, using LibTorch 2.5.1 with CUDA and the cxx11 ABI.

The crashes occurred when the camera viewport was partially or fully outside the bounds of the Gaussian splats.
I managed to resolve the issue, but I must admit that I don’t fully understand your code yet. Here are the changes I made:

After Preprocess

If there are no Gaussian splats to draw, abort the sorting and rendering processes. Otherwise, memory exceptions occur.

renderCUDA

I identified an uninitialized variable in certain cases and an array out-of-bounds access when a Gaussian splat was skipped. I did attach a patch.
crashFixes.zip

Your GPU memory is small, so maybe there's an oom? Set MAX_NUM_RENDERED = 2 ** 26 and try a small scene.

Yes, your right, i already did this:
_rasterizer = std::make_shared(*_scene, 1 << 24, 1 << 18);

Hi, @xohm.
I can reproduce this issue. How do you fix this problem exactly?
After Preprocess
If there are no Gaussian splats to draw, abort the sorting and rendering processes. Otherwise, memory exceptions occur.


about mots_eval/visualize_mots.py

There is no “visualize_mots.py” file in the code。How do I get it？Thank you！

it's in the mots_vis subfolder:
https://github.com/VisualComputingInstitute/mots_tools/blob/master/mots_vis/visualize_mots.py

thank you very much！


chore: re-enable backend migrations

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#325
#324  👈
#318
#317
#316
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @NathanFlurry and the rest of your teammates on  Graphite


Add a CladeTime method to return an "as_of" reference tree

To simplify the process of creating target data, add a method to CladeTime that gets a reference tree based on its tree_as_of attribute.
Definition of Done
Once complete, users should be able to do something like this:
from virus_clade_utils.cladetime import CladeTime

ct = CladeTime(sequence_as_of="2024-09-01", tree_as_of="2024-07-31")

# get reference tree that was in use as of 2024-07-31:
ct.reference_tree()

Getting the correct reference tree
The original plan was to use the process described in #7 to get a SARS-CoV-2 reference tree at a specific point in time (@elray1 and I worked out that process in consultation with some nextclade folks)
However, those steps rely on the nextclade cli to download a dataset package and extract a reference tree.
We can make that work for pipeline-based processes (e.g., generating target data for the variant nowcast hub). However, now that we've also re-purposed cladetime as a way to support more interactive work cases, relying on the cli becomes an impediment to project setup.

The nextclade cli is not on pypi (it can be installed via conda, which I'd prefer to avoid, since it doesn't fit into our current python workflow)
Using the standalone download for cladetime setup would adds complexity around determining the user's environment (and even then, the macos version is unsigned and requires jumping through hoops to make it run)
There's a docker image for the nextclade cli, but it's preferable not to require Docker as a dependency for cladetime users

New proposal
As an alternative to using the nextclade cli, some sleuthing reveals the pattern for accessing nextclade datasets via the following URL pattern:
https://data.clades.nextstrain.org/v3/<dataset name>/<dataset version>/<dataset file>
In other words, to get the reference tree for the 2024-09-25--21-50-30Z version of the nextstrain/sars-cov-2/wuhan-hu-1/orfs dataset:
https://data.clades.nextstrain.org/v3/nextstrain/sars-cov-2/wuhan-hu-1/orfs/2024-09-25--21-50-30Z/tree.json
(we get the correct dataset version from the metadata for nextclade's ingest workflow, for example: https://nextstrain-data.s3.amazonaws.com/files/ncov/open/metadata_version.json?versionId=uYgLvuF9AKtkbyAUz95KmCTojm1QQ7BX)
@elray1 what do you think? Using direct https requests rather than going through the CLI presumably puts us at risk if nextclade makes breaking changes on their backend. But it would make the cladetime package much more usable.
The other, longer-term, option might be asking cladetime to make their CLI available via pypi?


Create Webpage for the REST Endpoints

The REST commands should be executable over a webpage

Implemented by https://github.com/floriankammermann/rest-error-simulator/pull/3


can we get Tweet on the base of tweet id like we get tweet on the base of screenName?

Is it the same question as #266?

no its different for example if we have tweet id like<PHONE_NUMBER>63093760
how i am able to get tweet which contain that status id?

Same answer as #266
// For single tweet
var tweet = Tweet.GetTweet(738621958863093760 );

ok sir thank you
how can i get all followers of a specific user by using
var usr = Tweetinvi.User.GetUserFromScreenName(ScreenName);
var followers = usr.GetFollowers();

Tweetinvi cursor queries are not yet easy to use for complex cases related to RateLimits. I would suggest you read the following wiki to get ALL the followers.
https://github.com/linvi/tweetinvi/wiki/Get-All-Followers-Code

sir how to get screenname of those users who retweets the specific tweet?

tweet.CreatedBy.ScreenName

Tweetinvi has been designed to be simple to navigate via intellisense. Please try to use it!
var tweetId = 42;
var retweeterIds = Tweet.GetRetweetersIds(tweetId);
var retweeters = User.GetUsersFromIds(retweeterIds);

sir how can i get tweet video?

Before asking could you please try to use intellisense?

sir actually i am using tweetinvi from past three months and i have write down all questions and i am looking for a plateform where i can get answer and i found you thanks to you.

var tweet = Tweet.GetTweet(tweetId);
var medias = tweet.Entities.Medias[0];
but it will return  image.

It will return images or videos. You will have to check the tweet.MediaType or tweet.VideoDetails properties.

how to get replies against specific tweet?

If you could open such questions on stackoverflow it would be great.
It will give some answer easy to access to people looking for the same questions as you do.

i want to get get retweet count and favcount of specific tweet on the base of Tweet id but it will return "Null reference" or "Object reference not set to an instance of an object."
long id = Convert.ToInt64(TweetId);
var refreshedTweet = Tweet.GetTweet(id);
                var retweetCount = refreshedTweet.RetweetCount.ToString();
                var favcount = refreshedTweet.FavouriteCount.ToString();

how to get replies against specific tweet?

Stackoverflow

i want to get get retweet count and favcount of specific tweet on the base of Tweet id but it will return "Null reference" or "Object reference not set to an instance of an object."

I will need more information regarding your NullReferenceException. I think that the refreshedTweet received is null. This happen when the request to Twitter has failed.


Backport of runtime transaction converter

Closes #508

Is this duplicate of #559?

I believe this is dup. But let me know if it's not!

It is, but this PR was first :/


Resolve dependency versions

Trying to resolve issues that have come up on this PR: https://github.com/yeatmanlab/AFQ-Insight/pull/138

Coverage: 97.028%. Remained the same when pulling 82c7e3af531b42ae489c5cd7262bdcf87e0aecae on arokem:update_deps into db56156cd7b05a959a2555bdd68d1890971a4733 on richford:main.

@richford : Maybe we can make a release after merging this, and see whether we can resolve the issues in https://github.com/yeatmanlab/AFQ-Insight/pull/138 ?


Adding completion support for client-decl-stmt and module-client-decl

Purpose

Describe the problems, issues, or needs driving this feature/fix and include links to related issues.

Fixes #37688
Approach

Describe how you are implementing the solutions along with the design details.

Samples

Provide high-level details about the samples related to this feature.

Remarks
Need to provide completions for following cases after #37629 improves the clientDeclSymbol to include module details.
client "https://postman-echo1.com/get?name=projectapiclientplugin" as testPrefixMod;
testPrefixMod:<cur>

client "https://postman-echo1.com/get?name=projectapiclientplugin" as testPrefixMod;

function testModuleClientDecl() returns string {
    testPrefixMod config = {specVersion : "3.0.0"};
    testPrefixMod cl = new (config);
    cl-><cur>
}

Update: above two cases are addressed
Check List

[x] Read the Contributing Guide
[ ] Updated Change Log
[ ] Checked Tooling Support (#)
[x] Added necessary tests

[x] Unit Tests
[ ] Spec Conformance Tests
[ ] Integration Tests
[ ] Ballerina By Example Tests

[ ] Increased Test Coverage
[ ] Added necessary documentation

[ ] API documentation
[ ] Module documentation in Module.md files
[ ] Ballerina By Examples

Blocked by #37629

Project API issue tracking https://github.com/ballerina-platform/ballerina-lang/issues/37734

Skipped failing test module_client_declaration_client_keyword.json. https://github.com/ballerina-platform/ballerina-lang/issues/37920 was created to track this.


Create Math Section and Add Newton API

In this Pull Request, I intend to add a mathematics section to this API list. This is because the API that I am adding is not accurately described as a Science API (but more of a Math API).

Thanks @aunyks ! :)


cpu_core_count and cpu_threads_per_core attributes deprecation in favor of the cpu_options block

Description
Terraform AWS Provider v4.66 deprecates the use of cpu_core_count and cpu_threads_per_core attributes at the aws_instance level which displays deprecation warnings when running the examples delivered with this module.
If your request is for a new feature, please use the Feature request template.

[X] ✋ I have searched the open/closed issues and my issue is not listed.

Versions

Module version [Required]: v5.0.0

Terraform version: v1.4.6

Provider version(s): v4.66.1

Reproduction Code [Required]
Run the complete example as per this TF module.

Steps to reproduce the behavior:

Expected behavior
Terraform plan and apply execute without deprecation warnings.

Actual behavior

│ Warning: Argument is deprecated
│
│   with module.ec2_complete.aws_instance.this[0],
│   on ../../main.tf line 24, in resource "aws_instance" "this":
│   24:   cpu_core_count       = var.cpu_core_count
│
│ use 'cpu_options' argument instead
│
Terminal Output Screenshot(s)

Additional context

This issue has been resolved in version 5.1.0 :tada: