Allow webview panels to provide a custom icon

Allow webview panels to set an icon. This should probably be done declaratively in the package.json, something like:
"contributes": {
  "views": {
    "webviews": [
      {
        "viewType": "markdown.preview",
        "icon": {
          "light": "./media/Preview.svg",
          "dark": "./media/Preview_inverse.svg"
        }
      }
    ]
  }
}

/cc @jrieken

@mjbvz Since the title is provided dynamically in code, why not provide the icon there as well?

Yeah, it's little weird to now start mixing declarative and symbolic information... @mjbvz why isn't this a property like the title?

The declarative approach has a few advantages:

Makes it clearer that you need per theme icons
We always know where to get the icon resource file from
We don't have to save the old icon or old icon path when restoring a webview after vscode restarts (if an icons path changes, such as during an extension update, we also would handle the change properly)

Discussion ended in agreeing to disagree. There is the declarative vs in-code approach, arguments for the declarative are listed above, concerns against it are: (1) it uses package.json/contributes in a new way, not describing parts of the UI before loading code, but augmenting parts of the UI after loading & executing code. (2) it won't be possible to use a ThemeIcon or dynamic icons, (3) it feels asymmetric to provide properties like title et al in code but icon not. Last, the icon challenge (caching, fetching, transmitting) is something we already have today (activity bar icons, decorations with icons, tree items) and something we need to solve anyways.

Probably not the best idea (because of the extra work, etc), but maybe support both? Support the declarative as spec'd above, which will be the default unless overridden in-code, where more features (ThemeIcon, dynamically changing) can be used. Maybe even add title to the declarative syntax for the same, so they are symmetric?

I still think a static contribution is the correct design.

The icon is used before code is loaded, for example after restarting vscode and a webview is first restored.

I don't think we want dynamic icons. I'm also not sure ThemeIcon is a good fit for webviews, at least how ThemeIcon is being used currently. Rather, to support themable icons, we could add a top level "webviews" entry to the file theme json, something like:

"webviews": {
      "markdown.preview": "_markdown_preview"
}

In fact, I believe that this would be required even if we did use ThemeIcon for the webviews icons.

The proposed package.json contribution is for a webview type: markdown preview, azure explorer, gitlens, and so on. The title and webview content on the other hand are instance properties on the panel itself. I believe the icon should be the same for all instances of a webview type, as this is consistent with how we display other "editor" types.

Two follow up ideas:

Change the contribution point to webviewIcons to make it very clear what the intended use case of

Move the contribution into themeIcons:

{
  "contributes": {
      "themeIcons": [
          {
              // no label or id, just always apply these icons but allow other icons themes to override them
               "path": "./icons/my-webview-icon-theme.json"
          }
      ]
  }
}

with my-webview-icon-theme.json being something like:
{
  "iconDefinitions": {
     "_markdown_preview": { "iconPath": "..." }
  },
   "webviewPanels": {
        "markdown.preview": "_markdown.preview"
    }
}

/cc @sandy081

I agree that web-view icon contribution is not a good fit into views contribution point. Moving it out makes sense to me. I would wait for @jrieken 's opinion about static vs dynamic.

Any update on this?


Fix README typo

What is the current behavior?
only fix typo in the readme sample command
What is the new behavior?
Checklist
Please make sure the following requirements are complete:

[x] Tests for the changes have been added (for bug fixes / features)
[x] Docs have been reviewed and added / updated if needed (for bug fixes /
features)
[x] All automated checks pass (CI/CD)

Thanks @hotatekaoru :bowing_man:


Modal add max-height on the contents of the modal, for resolution above 768px

This behavior is normal and is what we (together with UX team) decided.
Closing this issue.


Updated README.md  (Images & Links)

Description
Updated README.md  (Images & Links)
This PR fixes #
Notes for Reviewers
Signed commits

[x] Yes, I signed my commits.

@leecalcote What about this?
Layer5's cloud native application and infrastructure management software enables organizations to expect more from their infrastructure. We embrace developer-defined infrastructure. We empower developers to change how they write applications, support operators in rethinking how they run modern infrastructure and enable product owners to regain full control over their product portfolio.

Sounds good. If you'll exchange "developer" for "engineer" in the final sentence that will be more inclusive. Let's leave "developer-defined", though.


Use % to partition

LGTM :smile:

Was wondering how @xinan can comment, then realized this is a public repo. :scream:

Yeah we made that public so that it can be easily installed as a gem :no_mouth:

I can update your comments :trollface:

@xinan git blame is you :trollface: bb5eae51

:crying_cat_face: I thought that was the requirement at that moment


Off-by-one in mean fold

Could be misreading (not sure how to actually run the thing), but
(v_, 1) : case s | (m, n) -> (m + (v_ - m) / (n + 1), n + 1)

Input: [5]
Initial case: (5, 1)
Output: 0 + (5 - 0) / (1 + 1), 1 + 1 = 2.5
/cc @jystic @amosr

I think the current code is right, just need to start at 0 rather than 1.

(see the online variance algorithm below this)

ah. I think this is right, but I agree it's confusing. the ":" in the fold means "followed by" or "then", so it will only be executed for the second input (or first input if it were fold instead of fold1)
So in this case the value of the fold s is (5,1), then it just extracts the first part afterwards.
For a two-element input
[5, 2]

fold1 s = (_v, 1) : case s | (m, n) -> (m + (v_ - m) / (n + 1), n + 1)
==>
fold1 s = (5, 1) : case s | (m, n) -> (m + (v_ - m) / (n + 1), n + 1)
==>
fold1 s = case (5,1) | (m, n) -> (m + (v_ - m) / (n + 1), n + 1)
==>
fold1 s = (2 + (5 - 2) / (1 + 1), 1 + 1)
==>
fold1 s = (3.5, 2)

Yep, figured out how to run stuff, current code is indeed correct. :+1:


please help: unclear interface of p_sad8x8_f32, p_sad16x16_f32

Hi, I would like to implement the mentioned functions, but the description of the arguments is unclear to me. My guess (for the 8x8 case; for 16x16 analog):
SAD works on an input source image x with dimensions rows x cols. There is a second reference image with the same dimensions as x. m is a pointer to the upper left corner of a 8x8 block within this second image. (Therefore m is not an 8x8 array.) The result vector r is a 2D array with dimensions (rows - 7) x (cols - 7) holding the absolute differences for the possible offsets of the 8x8 block within the source image.
Please tell me if these assumptions are correct. Thank you.
Greetings, Alexander

You are right, the interface made no sense. I have updated the file, should make more sense now.  Function returns a scalar.

Thank you, for clarification. The pseudo code did it.


Load entities and policies from json file

Reason: Unable to load a large amount of entities (30,000 - 4.5MB) from the /data endpoint
What PR does: Adds the possibility to load entities and policies from a json file at rocket startup
Use the -d or --data option and inform the path of the json file you want to load in cedar's memory, it is also possible to use a DATA environment variable
examples:
cargo run -- -d "./examples/data.json"

cargo run -- --data="./examples/data.json"

export DATA="./examples/data.json"
cargo run

Use the  --policies option and inform the path of the json file you want to load in cedar's memory, it is also possible to use a POLICIES environment variable
examples:
cargo run -- --policies="./examples/policies.json"

export POLICIES="./examples/policies.json"
cargo run

@omer9564 Thank you very much for the review, in addition to the changes you requested, I changed the word policy to policies in the config.

two suggestion comments ( might make the code simpler ) one missing new line again 😅 , Besides that it looks great 💪

😅 thank you very much for the suggestion.

@cleissonbarbosa PMed you in the community <3

@orweis @danielbass37 Thank you guys 💜


http dependency deprecated

Because unleash 0.4.1 depends on http ^0.13.0 and no versions of unleash match >0.4.1 <0.5.0, unleash ^0.4.1 requires http ^0.13.0.
So, because yap_client depends on both http ^1.1.0 and unleash ^0.4.1, version solving failed.

1


customProps always logs duplicate keys

customProps always logs duplicate keys.
I am working with google cloud logging and it doesn't play nicely with the duplicate keys:
    customProps(req) {
      // adds a custom object to all http logs that stackdriver
      // uses to make logs more informative and visible at a glance
      // see: https://cloud.google.com/logging/docs/reference/v2/rest/v2/LogEntry#httprequest
      // for the details of the structured log messages
      return {
        httpRequest: {
          requestMethod: req.method,
          requestUrl: req.url,
          userAgent: req.headers['user-agent'],
          remoteIp:
            req.headers['x-forwarded-for']?.split(',').shift() ||
            req.socket?.remoteAddress,
        },
      };
    },

For instance:
custom props cause the following log:
"{ "httpRequest": {"method":"GET"},"httpRequest": {"method":"GET"}}"
ends up in stackdriver/google cloud logging looking like:
{httpRequest: { method: 'GETGET' }}
related to https://github.com/pinojs/pino-http/pull/197

Can you include what you are trying to achieve? The snippet is not enough.

I want to use customProps to add the httpRequest object for google cloud logging
I want to avoid the duplicate keys problem
I do NOT want to use a transport to format or clean up the keys

Currently, any usage of customProps causes duplicate keys of whatever is added there because of this PR. https://github.com/pinojs/pino-http/pull/197
Right now, as a user, I am forced to deal with duplicate keys if I want to use the custom props option.

Your usecase is a bit far from mine.
Would you like to send a Pull Request to address this issue? Remember to add unit tests.

I took a look at this over the weekend, and unfortunately I couldn't come up with a quick solution that doesn't cause unit tests to break. This one may require some time/care to fix correctly and maintain all current behavior.
In order to pass the correct (final) res.statusCode to the customProps bindings and have the bindings in logs on req.log and res.log we do need to have the bindings applied in both places.

I have the same issue.
Couldn't we just remove the customPropBindings on the loggingMiddleware

const customPropBindings = (typeof customProps === 'function') ? customProps(req, res) : customProps
if (customPropBindings) {
fullReqLogger = fullReqLogger.child(customPropBindings)
}

This will be done on the onResFinished function. I tested it quickly and it work.
The only question i have is, does the function onResFinished called on 100% use cases ?
Like on network failure, timeout ...
What do you think ?

Same issue here, any solutions?

Yeah. I can reproduce in pino-http.
Here an example:
File: pino-config.ts
import { Options } from 'pino-http';
import { IncomingMessage } from 'http';

export const pinoHttpConfig: Options = {
  quietReqLogger: false,
  autoLogging: true,
  base: { hostname: os.hostname },
  level: 'info',
  formatters: {
    level: (label: string) => {
      return { level: label };
    },
  },
  timestamp: () => `,"time":"${new Date(Date.now()).toISOString()}"`,
  serializers: {
    req: () => {},
  },
  customProps: function (req: IncomingMessage) {
    // Dummy message
    return { hola: 'hola' };
  },
};

Then, with pino-http:
File: logger.middleware.ts
import pino from 'pino-http';

import { pinoHttpConfig } from './pino.config';

export function LoggerMiddleware() {
  return pino(pinoHttpConfig);
}

Injecting middleware in express app:
app.use(LoggerMiddleware());

Output:

As you can see: "hola", appears twice.

This is because customProps is called twice:

https://github.com/pinojs/pino-http/blob/3361ced517772b18087c49cb7f32b44e779179ce/logger.js#L107-L110
https://github.com/pinojs/pino-http/blob/3361ced517772b18087c49cb7f32b44e779179ce/logger.js#L144-L147

I think https://github.com/pinojs/pino-http/pull/197 had a bad bug we didn't see at the time.
I think we should have have https://github.com/pinojs/pino-http/blob/3361ced517772b18087c49cb7f32b44e779179ce/logger.js#L107-L110 removed.
Would you like to send a PR?

@mcollina I don't see a way to build an artifact and test all changes in my local. No information in the README.md or ci.yml, please could you provide the steps to test in my local and see the expected result? I am following these steps (https://dev.to/scooperdev/use-npm-pack-to-test-your-packages-locally-486e) to link the dependency and test.

Workaround: use a WeakSet to keep track of requests that have already been through customProps.

@mcollina I don't see a way to build an artifact and test all changes in my local. No information in the README.md or ci.yml, please could you provide the steps to test in my local and see the expected result? I am following these steps (https://dev.to/scooperdev/use-npm-pack-to-test-your-packages-locally-486e) to link the dependency and test.

https://www.freecodecamp.org/news/how-to-make-your-first-pull-request-on-github-3/#:~:text=Go to your repository on,Congratulations!
To make the changes, run npm install to install the dependencies, and npm test to run the tests.

https://jrfom.com/posts/2017/03/08/a-primer-on-contributing-to-projects-with-git/ if you need another resource.

@mcollina if I remember correctly, you can't just remove one of the bindings - it causes a lot of other things to break and then you don't have the custom bindings in normal calls to res.log.method()
@mpartel had an interesting idea of using a WeakSet to conditionally apply the custom bindings in both places, but ensure that they are only applied once per response cycle.

To be clear, I'm just suggesting WeakSet as a temporary workaround for users. It doesn't sound like a clean solution for the library, and it might even have some performance implications (I don't know how it affects the GC).

@mpartel thanks for clarifying. I'm not well versed in the application of WeakSet. Mind sharing how you implement the workaround to avoid the issue? It's not immediately obvious to me.

We should really split customProps into two functions, not one.

Hello, I just bumped into this and sadly don't have the time to do a PR but I have a case where the first call will happen with a Request object that has not yet fully gone through the pipeline. I wanted to log the matched expressjs route and the current user user id, and in order to do that I had to do something like:
customProps(req) {
    if (!req['SEEN-THIS-ONE']) {
       req['SEEN-THIS-ONE'] = true;
    } else if (req['SEEN-THIS-ONE']) {
       return {
           route: req.route.path,
           userId: req.user.id,
       };
    }
}

If I used the values from the first call they would always be route: "/* and userId: null. So whichever function onResFinished or loggingMiddlware is being called the first, is called too soon.
The stack I have is express, nestjs, nestjs-pino.

Noticing this here too.

is this issue fixed? If yes then in which version. I am trying to log 'x-request-id' in the outermost json and for response logging it is appending twice.

I tried to resolve it at https://github.com/pinojs/pino-http/pull/288.

Hey everyone, I think this issue is still happening. I'm using customProps and like everyone else it is still duplicating the values. Could anything be done to fix this?

Awesome, thank you @youngkiu

Has the fix in the latest release resolved this problem for everyone? I am still seeing it on the latest version unfortunately.

@rickihastings @cat-pierrecharles

https://github.com/youngkiu/nestjs-kakaologin/blob/main/src/app.module.ts#L24-L31  HTTPLoggerModule.forRoot({
    pinoHttp: {
      customProps: (req, res) => ({
        context: 'HTTP',
      }),
    },
  }),

https://github.com/youngkiu/nestjs-kakaologin/blob/main/package.json#L61
"pino-http": "^8.4.0"

This is the log generated from my repository code
[Nest] 69441  - 09/19/2023, 11:06:01 PM     LOG [NestApplication] Nest application successfully started...
{"level":30,"time":1695132407596,"pid":69441,"hostname":"youngkiu-MacBookPro.local","req":{"id":1,"method":"GET","url":"/","query":{},"params":{"0":""},"headers":{"host":"localhost:3000","user-agent":"curl/7.78.0","accept":"*/*"},"remoteAddress":"::ffff:<IP_ADDRESS>","remotePort":61640},"context":"HTTP","res":{"statusCode":200,"headers":{"x-powered-by":"Express","content-type":"text/html; charset=utf-8","content-length":"5537","etag":"W/\"15a1-GwH5K5Ocq58NdT/1K7JSlOJlfds\""}},"responseTime":61,"msg":"request completed"}

It seems that I can't recreate this problem locally but it's an issue when deployed to GCP Cloud Run. I'll do some further investigation to see if I can replicate it outside of GCP.

This issue hasn't been fixed in v8.5.0.
The workaround made by @ilari-makimattila looks good to me, and those who might be using in Typescript can try this.
import express from 'express';
import pino from 'pino-http';

const app = express();
const logger = pino({
  customProps: (req, res) => {
    if (req.isLogged) return {};

    req.isLogged = true;
    return  {
      auth: decodeToken(req.headers.authorization)
    }
  },
});

declare module 'http' {
  interface IncomingMessage {
    isLogged: boolean;
  }
}

related, #288

I stumbled into the same issue but the workaround proposed above didn't work for me since I want to log some custom context that is not yet available when the customProps function is first called.
This worked for me to log those props only when the request is finished by checking res.writableEnded property:
customProps: function (_req, res) {
  if (!res.writableEnded) {
    return {};
  }

  return {
    context: getRequestContext()
  };
},

@henripoikonen's solution works for me.
Here is more background on why pino doesn't de-duplicate: https://github.com/pinojs/pino/issues/625..

According to this: pinojs/pino#625
and this: https://getpino.io/#/docs/child-loggers?id=duplicate-keys-caveat
duplication is normal and it is better for performance to revert fix #288
@mcollina what do you think?

PRs are welcome.


Add restriction to selection range for Date and Time

I want restriction on inputs for date and time. Like we can give lower and upper limit for time picker to restrict user to choose time within specific range only. And same for date.
Thanks..!

I would like to see this also.

ok I implement it ;)

done  on 1.0.2
.minDateRange(minDate)
.maxDateRange(maxDate)

Thanks :)

Can you change the padding so that it displays the current range only(not the past dates)


Add 40+ different types of Sorting Algorithm in JAVA

Hey @MryashAditya007 please assign this issue to me

Hey @MryashAditya007 can you give the description of this issue?

I wish to contribute. Kindly assign me this issue.

I would like to contribute. Please assign this question to me

hey @MryashAditya007  Can you assign this issue to me ?


LVS error net mismatch

Description
While hardening the core, which has SRAM macro and another digital which is hardened separately without any error, I get the LVS error given below.

Environment
Kernel: Linux v5.11.0-49-generic
Distribution: ubuntu 21.04
Python: v3.9.5 (OK)
Container Engine: docker v20.10.7 (OK)
OpenLane Git Version: -dev
pip:click: INSTALLED
pip:pyyaml: INSTALLED
pip:venv: INSTALLED
---
PDK Version Verification Status: OK

Reproduction Material
in the repository, I run the following command
flow.tcl -design user_project_wrapper -tag with_user_proj_0.5 -overwrite -disable_output
Expected behavior
No lvs error
Logs
Please find the attached log file.
[29-user_project_wrapper.lef.log](https://github.com/The-OpenROAD-Project/OpenLane/files/8302756/29-user_project_wrapper.lef.log)

@gncemre23 Looks like you have a power short. Can you post your gds?

Sure! Please find the attached
user_project_wrapper.gds.zip
d

@gncemre23 Looks like your user_proj_example has met5 routing layers that are shorting with the user_project_wrapper power grid. Should be easy to see if you just display met5. Maybe set your max routing layer to met4.

I don't believe this is an openroad issue and suggest the issue be closed.

Also you have typo here:
set ::env(VDD_NET) {vcdd1}
set ::env(GND_NET) {vssd1}

@gncemre23 Looks like your user_proj_example has met5 routing layers that are shorting with the user_project_wrapper power grid. Should be easy to see if you just display met5. Maybe set your max routing layer to met4.
I don't believe this is an openroad issue and suggest the issue be closed.

Is this klayout?

Yes, klayout.

Can we observe that on the Openroad -gui ? I cannot import gds or lef files in openroad.

I realized that I did not set DESIG_IS_CORE parameter as 0 while hardening the user_proj_example macro. I guess that causes the tool is using met5 even if I set rt_max_layer as met4. Now, with this adjustment, I don't have any LVS errors.
Thank you @d-m-bailey


feature support insert only transactional hive table on FE side

Proposed changes
Issue Number: close #xxx
Problem summary
Describe your changes.
Checklist(Required)

[ ] Does it affect the original behavior
[ ] Has unit tests been added
[ ] Has document been added or modified
[ ] Does it need to update dependencies
[ ] Is this PR support rollback (If NO, please explain WHY)

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

run buildall

TeamCity pipeline, clickbench performance test result:
the sum of best hot time: 33.43 seconds
stream load tsv:          415 seconds loaded<PHONE_NUMBER>9 Bytes, about 171 MB/s
stream load json:         22 seconds loaded<PHONE_NUMBER> Bytes, about 102 MB/s
stream load orc:          59 seconds loaded<PHONE_NUMBER> Bytes, about 17 MB/s
stream load parquet:          31 seconds loaded 861443392 Bytes, about 26 MB/s
https://doris-community-test-1308700295.cos.ap-hongkong.myqcloud.com/tmp/20230511065421_clickbench_pr_142144.html


Add spec for NameError#name

Looks great, thank you for this spec!
Could you address my two comments?

Merged, thank you for your contribution!
I am sure it will help other rubies implement this :smile:


requirements.txt file generated by poetry export causes conflicting dependencies in pip

[x] I am on the latest Poetry version.
[x] I have searched the issues of this repo and believe that this is not a duplicate.
[x] If an exception occurs when executing a command, I executed it again in debug mode (-vvv option).

OS version and name: Windows 10
Poetry version: 1.1.4
Link of a Gist with the contents of your pyproject.toml file:

Issue

With the linked pyproject.toml file, running
poetry export -f requirements.txt --output requirements.txt --without-hashes

generates this requirements.txt file.
These lines cause dependency conflict on Python 3.7 when installing this requirements file with pip.
ipython==7.16.1; python_version < "3.7" or python_version >= "4.0" or python_version >= "3.6" and implementation_name == "cpython"
ipython==7.20.0; python_version >= "3.7" and python_version < "4.0"

It tries to install both 7.16.1 (because python_version >= "3.6" and implementation_name == "cpython" matches) and 7.20.0 (because python_version >= "3.7" and python_version < "4.0" matches).

@dimbleby Is this resolved now via #94?

should be


[full-node] Fix flaky smoke test test_full_node_basic_flow

Motivation
Fixing flaky test failure that happened in https://circleci.com/gh/libra/libra/10259#tests/containers/3
Test was failing because the second full node was not synced up with the most recent transaction. Adding an explicit wait until the sequence is synced up will remove this flakiness
Test Plan
cargo test -- test_full_node_basic_flow

@bors-libra: r+

:pushpin: Commit 5fe5f1e has been approved by phoenix-antigravity

:hourglass: Testing commit 5fe5f1e1262fe5dee876ca000982456193214d71 with merge ddebe9048ea91bda6257d9f8c45c75020cdede60...

:sunny: Test successful - checks-circle_commit_workflow
Approved by: phoenix-antigravity
Pushing ddebe9048ea91bda6257d9f8c45c75020cdede60 to master...


Use Fill.Color for colored svg

I am working on improving the some of the samples that I think users are most interested in. I think this is mostly simple pins and feature info on those pins. One other thing that I think is relevant is selecting colors for the pins that are placed. This PR allow the user of SymbolStyle.Brush.Color to determine the color of an SVG pin.
A problem with this PR is that it changes the default behavior of adding a pin because the SymbolStyle.Brush.Color is default set to White. So If you do not remove this color the svg will be drawn in white. In most cases this is not what people want. Options:

Explain that this is a breaking change and users need to set the Brush.Color to null (current solutions)
Make the SymbolStyle.Brush default null. This is also a breaking change.
Introduce a new kind of style for this. This may be confusing for starting users. Which style to use?
Add a new field SymbolStyle.ImageColor
Generate or modify svg images. Perhaps we can set the color in the svg itself.

Decided to go with a new field ColorBlendMode, so there are no breaking changes. Not sure if it is a great name. Just based on the named in skia. Perhaps we need to add a BlendMode type as well at some point. In general I don't think it is bad to stay close to skia.
I was also thinking about experimenting with a SkiaStyleStyle where you could use the skia SKPaint so there is no translation whatsoever, and users could be pointed to the skia documentation to know learn how it works. May also be an advantage for performance.


[BUG] Validation of URLs Fails for Certain Status Codes

Bug Description:
The URL validation logic in our project currently only considers a URL to be valid if it returns a 200 OK status code. However, some valid URLs, such as LinkedIn's homepage, return other status codes like 999, which causes our validation to incorrectly mark them as invalid. This issue affects the accuracy and reliability of the URL validation feature.
Steps to Reproduce:
Use the current URL validation logic to validate https://www.linkedin.com/in/srikarmellachervu/
Observe that the validation fails due to the status code 999 returned by LinkedIn.
Expected Behavior:
The URL validation should account for valid status codes other than 200 OK, including but not limited to 301 Moved Permanently, 302 Found, and specific custom status codes like LinkedIn's 999.
Actual Behavior:
The URL validation fails for valid URLs that return status codes other than 200 OK which gives an output "Invalid URL"

Possible Solution:
Update the validation logic to handle a broader range of valid HTTP status codes, such as 301 Moved Permanently, 302 Found, etc.
Implement handling for specific known custom status codes, like LinkedIn's 999.
Consider implementing a retry mechanism for temporary issues or status codes indicating the need for a second attempt.
Please issue this to come under gssoc 24

OS: Linux
Browser Brave

Thanks for pointing it you this issue.
Hey @RoshanAnsy, look into this issue.

I have a working solution to this please assign this to me so I can create a pull request

Hey @Srikar04, could you explain your approach. If this is the case then we can't add every response code for every website in our code.

This is fine but are you sure it will work for all have you tested it because in later time what if someone else reports that some links are not working. Although I am assigning this issue to you.
Hey @RoshanAnsy, i think we missed these cases.

Closing this issue as done.


Add receive file rejection functionality

Currently, if the receiver tries to cancel the transfer on the ReceiveConsentScreen, the sender is not notified that the transfer was cancelled. This PR fixes that
Also needs https://github.com/LeastAuthority/wormhole-william/pull/100 to be merged

Assuming fix of w-w part is correct, can we adjust behavior on frontend side, because now only Receiver side initiate rejections, also after rejection on Receiver side, it still stays in the same page (also Sender stays in the same page).

@donpui I updated this pr so that the sender and receiver resets back to the begin pages when a rejection happens. This pr as well as https://github.com/LeastAuthority/wormhole-william/pull/100 should be ok now.

@donpui I updated this pr so that the sender and receiver resets back to the begin pages when a rejection happens. This pr as well as LeastAuthority/wormhole-william#100 should be ok now.

CI and for me fails to build:
#48 31.18     [tsl] ERROR in /usr/src/app/client/src/app/wormhole/client_worker.ts(243,19)
#48 31.18       TS2345: Argument of type '{ code: any; cancel: () => Promise<void>; done: Promise<void>; }' is not assignable to parameter of type 'TransferProgress | PromiseLike<TransferProgress>'.
#48 31.18   Property 'reject' is missing in type '{ code: any; cancel: () => Promise<void>; done: Promise<void>; }' but required in type 'TransferProgress'.

@donpui fixed the type error, it should be ok now


Redux in react components

Wrapped <App> with the <Provider> component from react-redux
Passed configured store into the <Provider>
Created an initialState variable for  booksSlice, which included the following:// Initial state:
[
  {
    "item_id": "item1",
    "title": "The Great Gatsby",
    "author": "John Smith",
    "category": "Fiction"
  },
  {
    "item_id": "item2",
    "title": "Anna Karenina",
    "author": "Leo Tolstoy",
    "category": "Fiction"
  },
  {
    "item_id": "item3",
    "title": "The Selfish Gene",
    "author": "Richard Dawkins",
    "category": "Nonfiction"
  }
]

Displayed books, received from the slice, in a reusable component
Dispatched actions using useDispatch
Added a <Button> component, which includes:

An event handler that adds a book to the books array (with attributes id, title and author)

Added a <Button> component, which includes:

An event handler that removes a book from the books array (by id)

Hello @fatmahussein could you check your slack?


Make input file filter rely on user inputs

Description
The previous filter definition only included recent days (the last ref_lag + training_days days before "today"), regardless of user settings. However, a user can request models trained on older data or predictions made for older dates. Those cases require data entirely not in that recent range.
Changelog

io.R

So now the order of the steps are:

Check training_end_date and training_start_date according to params
Together with test_date, modify the issue_start_date and issue_end_date

However, if we really want to train the model for older test_dates, isn't the order to be:

Check the training_end_date according to test_date and training_end_date from the model we trained already
Modify the issue_end_date and issue_start_date accordingly

Hm, I don't understand your proposal

Check the training_end_date according to test_date and training_end_date from the model we trained already
Modify the issue_end_date and issue_start_date accordingly
Hm, I don't understand this bit.

The point of this function, subset_valid_files, is to reduce the full list of input files to only those needed for the current task. Either we are training models (and don't need testing input data), making predictions (we need both testing and training input data. The training data is in case some models are missing from the cache and we need to recreate them), or both training and predicting (we need both testing and training input data).
The two threshold dates, issue_end_date and issue_start_date, are the first and last issue dates we want to keep to get the correct input data.
training_start_date - ref_lag and training_end_date are the first and last issue dates we would want to keep if we were only training models and not making any predictions.
test_dates are used in an issue date filter here to create testing data. So min(test_dates - ref_lag) and min(test_dates) are the first and last issue dates for input data we would want to keep for making predictions.
In cases where we need input data for both training and predicting, I'm just taking the most extreme values for the start and end issue dates (min of training_start_date - ref_lag and min(test_dates - ref_lag), etc).
It sounds like maybe you're concerned that the defined training_end_date isn't appropriate in some cases.

Check the training_end_date according to test_date and training_end_date from the model we trained already

When train_models is not turned on, get_training_date_range does check dates from cached models. If there are no cached models, training_end_date is set to today; if there are some cached models, training_end_date is set to match those and any models missing from the cache for that particular training_end_date are regenerated.
Unless I'm misunderstanding some bit of code logic, training_end_date should be entirely independent of the test_dates. For example, we can train a model on the last month of data and use it to make predictions for test dates from a year ago.

Check the training_end_date according to test_date and training_end_date from the model we trained already
Modify the issue_end_date and issue_start_date accordingly

Hm, I don't understand this bit.

Backfill correction is highly time sensitive. It's inappropriate to make prediction of historical data based on a model trained using future data.
My understanding is that, if we only keep the most recent model saved, once we want to make prediction of an old test date t0 (t0<T, T is today), we always need to train the model based on data (t<t0).

It's inappropriate to make prediction of historical data based on a model trained using future data.

For example, we can train a model on the last month of data and use it to make predictions for test dates from a year ago.

I agree that in reality we don't want to do this, but I thought it could be useful to have the option. It sounds like you're saying that the extra flexibility is dangerous (besides just being unnecessary). So we need to add a check that min(test_date) > training_end_date to verify that this is true:

make prediction of an old test date t0 (t0<T, T is today), we always need to train the model based on data (t<t0).

I agree that in reality we don't want to do this, but I thought it could be useful to have the option. It sounds like you're saying that the extra flexibility is dangerous (besides just being unnecessary). So we need to add a check that min(test_date) > training_end_date to verify that this is true:

Exactly.


fix(clients): support dict in helpers

🧭 What and Why
🎟 JIRA Ticket: https://algolia.atlassian.net/browse/DI-3335
Changes included:
we have runtime errors if we use dict/hash instead of models, but the clients support both, we should align in the helpers as well

Seems good but I don't read Ruby :-D

me neither dw


🐛 Bug Report — Runtime APIs - FTS5 authorization needed

According to https://github.com/cloudflare/workerd/pull/619/files#diff-3009a3da17d76a94673aa828ce6c364fc24e34d889b4995adfc249a0e2a3c754 we're looking for the exact string fts5 when working out if a command is allow-listed.
It should probably be case-insensitive?

Should be as simple as making this comparison case-insensitive: https://github.com/cloudflare/workerd/blob/d8ffd3de075ad5dba5b5d072165c9e1830586695/src/workerd/util/sqlite.c++#L658
I'll jump in and fix it next time I'm in there, unless someone gets to it first.


generate OWL output

As part of makefile, generate OWL artefact.
cc @cmungall @turbomam

On it. Should be good for visualizing issues about slot inheritance, like #402 and #410
@sujaypatil96 I'm going to put a free-standing OWL generation step in the Makefile. We should integrate that into the existing generation patterns in the Makefile, or maybe switching to gen-project could be part of an overall repo refactor like we were discussing yesterday.

Start over after having closed messy PR

#425

owl is now generated in the cookiecutter integration branch issue-658-team-cookiecutter


.svg files cannot be loaded as "Image" resource

Godot version
4.1.1 stable
System information
Godot v4.1.1.stable - Windows 10.0.19045 - Vulkan (Compatibility) - NVIDIA GeForce GTX 750 Ti (NVIDIA; <IP_ADDRESS>79) - 12th Gen Intel(R) Core(TM) i5-12400F (12 Threads)
Issue description
When trying to import an svg as an "Image", it does not allow it to be loaded.

Deleting .godot does not fix the issue.
Steps to reproduce
See above
Minimal reproduction project
n/a

This is probably because that import mode means it's unchanged, but there's no way to open a SVG file directly, it has to be rasterised, so all it does is allow you to access the original file
This might be a case where the option should be disabled for SVG, as you need additional options to open it

This is probably because that import mode means it's unchanged, but there's no way to open a SVG file directly, it has to be rasterised, so all it does is allow you to access the original file
This might be a case where the option should be disabled for SVG, as you need additional options to open it
In 4.2 we have the ability to load svg more dynamically, can you test 4.2-dev1?

Can confirm that it doesn't work in 4.2 dev-1.
I would say like what you said to either disable the option, or if it's really wanted, to import it as a rasterized image resource


Crack dependency

Version 0.7.5 available on gemcutter.org seems to be depended on the gem crack, but that gem wasn't installed at runtime. I had to manually install crack and also require 'crack' in order to get the irb example to run past the Linode object creation.

I just realized that HTTParty already parses JSON and XML in the response, so why do you need Crack for?...

I'm guessing that dependencies and functionality in httparty from the time when the linode API was written have drifted.  At least for now I will keep the internal usage of Crack in place, but patches accepted.
This should be closed with 0.7.6 -- thanks for the help!


inputText command automatically focuses on any initial field and begins typing

Describe the bug
We use Maestro's web driver to also run our ETE tests in the browser or web (chrome) as well. Upon running the test flow, the inputText automatically focuses on the email field or any initial field and begins typing.
To Reproduce
Steps to reproduce the behavior, e.g.:

Run web localhost in frontend folder (yarn start:web)
Run maestro test ete_web.yml in maestro folder
The Flow fails at password inputText that automatically focuses on the email field or any initial field and begins typing.

Expected behavior
The user should input email and password to login correctly.
Screenshots
Here is the screenrecord for further details:
https://www.loom.com/share/bb30f31e599745cb840b0b1d7b2f93c4
Here are the other methods I tried:
[1] Use tapOn for each field you want to inputText
[2] Use testId
tapOn:
id: passwordFieldId
[3] Tried maestro studio suggestion commands
Environment information (please complete the following information):

Maestro version [v1.27.0]
Platform: [Chrome]
Framework: [React Native]
ETE tests in the browser or web (chrome)
Host [Linux: Ubuntu]

i have the same problem, but when i open the web in android emulator the problem does not exist and it only happens on chromium

Having same issue. on my login screen email is entered, the password field is tapped but typing continues in email field.

Hello, having the same issue too


Adds to the gitignore and removes .log files

Gets rid of .log files from assets folder and moves saveFile.json to other gitignore.

Everything is a bit lag on my laptop but our features are not affected.

just add you since you added a file I am removing seems like it was not meant to be committed though

I wonder if there's a point to not having a blanket *.log gitignore rule, but I don't think it really matters either way.

Was thinking about adding that but it seems to only put log files in the assets folder so should hopefully stop all of them


Add exp and log for SoftFloat64

This is a draft for now since it's only implemented for f64, and doesn't have any tests.

@zRedShift Are you planning to continue working on this, or posting it so that someone else can finish it?

@knickish I plan to continue working on this sometime next week, shouldn't be much left. Of course, if you want to take over, I have no objections, I'm already using this code as is, I just want to upstream it, and make sure the precision is tested and there's f32 support too.

@zRedShift No problem, not trying to take it if you're still planning to do things with it. Appreciate the upstream.

@knickish hey, sorry, this PR this completely slipped my mind. Thanks for finishing it up and merging it 🙏

@zRedShift It's all good! thanks for implementing those


Angular 17 support - cannot drop steps on empty canvas

Please add compatibility to Angular 17. After upgrading project from Angular 16 to 17, we no longer can drop steps onto empty canvas which is a breaking issue.

+1, not working in NG17 using the most basic configuration (outlined in Wiki), can't drop to canvas
HTML

    Item {{item.name}}

<ng-template #stepContent>

      heading

      content

</ng-template>

CSS
.canvas-wrap {
  min-height: 400px;
  display: flex;
  background-color: #000;
}

.goal-canvas {
  position: relative;
  flex-grow: 1;
  background-size: 30px 30px;
  background-image: linear-gradient(90deg, #343434 1px, transparent 0), linear-gradient(180deg, #343434 1px, transparent 0);
}

.palette-item {
  width: 250px;
  height: 40px;
  border: solid 2px #334799;
}

TS
@NgModule({
  declarations: [
    MyCustomComponent
  ],
  imports: [
    NgFlowchartModule
  ],
  exports: [
    MyCustomComponent
  ]
})
export class MyCustomAppModule{ }
...

@Component({
  selector: 'app-my-custom-component',
  templateUrl: './my-custom-component.html',
  styleUrl: './my-custom-component.scss'
})
export class MyCustomComponent {
      items = [
          {
            name: 'Logger',
            type: 'log',
            data: {
              name: 'Log',
              icon: { name: 'log-icon', color: 'blue' },
              config: {
                message: null,
                severity: null
              }
            }
          }
        ]
}

I've probably found the solution. At least it works for me on v17 project.
In the ng-flowchart-canvas.directive.ts file I've replaced lines 38-40 with the following code:
let closestCanvasId = event.target.closest('.ngflowchart-canvas-content')?.id;
if (closestCanvasId === undefined) {
  closestCanvasId = event.target.closest('div').querySelector('.ngflowchart-canvas-content')?.id;
}

I know it probably could have been done better but at least it works in my use case. It would be great if somebody smarter than me checked it out.

Angular 17+ note: Requires use of { provide: REMOVE_STYLES_ON_COMPONENT_DESTROY, useValue: false } OR @import<EMAIL_ADDRESS>to apply base step styles as Angular breaks base Component style inheritance with the new default REMOVE_STYLES_ON_COMPONENT_DESTROY: true


How to use flashAll function?

i have been face some issues to use the flashAll function, i was looking at documentation but dont find nothing about.
can someone give me an example of how to use it?
 await redisClient.flushAll();

It's flushAll, not flashAll
client.flushAll():


Feature Request - Remove junk Characters from Text

Hi,
Your plugin works great!
Any possibility we could have a feature where in it removed junk characters from paragraph/pasted text, just leaving "."/periods where they are.
Thank you!

Sorry, I don't quite understand what junk characters are. Can you give an example, like input is blablabla and expected output(result) is blablabla.

Sure please,
Following is input text"
The quick *~~~Brown f%%ox jumps) right~ over$ the lazy! dog).
Expected output (Clean) :
The quick brown fox jumps right over the lazy dog.

It typically happens when I use OCR for old documents (hand typed documents)  or corrupted PDF's or Old dbase/Foxpro Memo files (corrupted). I think its mainly because old documents are yellow and smudgy and are hard to scan where OCR inserts characters on its own :)

Here is one example I picked from Google, Some documents have close resemblance to example :

)

The pictures you provide are illegible even by myself, what is the copy text like? My OCR result is below
The preser.t rerort is Oric of 玉 numbr wiich dr：prr4lrex duri上i anJ 1945 fpr the Frreign poncnic 永ministration Lmembevs:st t.

the unitedi St:tes Tarift' Cornissint:. Orine to the desire of thr 我 、” Econoniy Aininistration to obt ir this matcri.1. !.xs prompt1y as pces1.o, the reports yere Yot revievvd by the Trri:: Connissien. A11 st:tenont.s o1 fagt or opinion in tese renorts CI &ttributithlp t； the. irciyilei Etaef nembers tho prraredi th.em. Th.:Y. 1l，洲以：rieinlt：itsed f conf idential u: of Goverrnent xgencivs, ut .•r（〉 noR brin.: Hdpnniis with the consent oi the For(imr. Eocnnic iuiri:.istrtior:.

If the copy text is similar to this, I don't think directly deleting junk characters can get the expected text.

Thank you for responding! :)
I am noobie and I posted same issue on Obsidian forum, requesting for help. ( https://forum.obsidian.md/t/replace-all-asterisks-in-a-given-file/35238/25?u=looney.apache )
Solution was not as elegant as yours - But it works for me (at least for now )
I paste my text to be scrubbed in to  https://textcleaner.net/ and get back cleaned text as need be.
I value your assistance. Thank you!

True!
Also , I've been using "Obsidian Text Format" and I've discovered that it's fantastic at fixing broken paragraphs.
Unquestionably one of the GOOD plugins.


chore(master): release 0.8.0

:robot: I have created a release beep boop
0.8.0 (2023-12-26)
Features

updating from newer svd2rust (#65) (db2a5f5)

Bug Fixes

moving to 2021 edition (efcdc16)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/xmc-rs/xmc4300/releases/tag/v0.8.0 :sunflower:


When will be the next release ?

Does anyone know an estimate time when the next release ? Waiting for the md-select and md-input : )

@johnqiuwan See https://github.com/angular/material2/issues/2045

lol, ok, you are right. I close it myself : )


Add the ability to use local files as a notification sound.

Implements the feature suggested in #6.
NOTE: This is terrible because, unless there is magic I don't know about, every time a sound is needed from a file, device storage will be read. I just didn't want to make too drastic of a change to already existing code; I feel that that'd be pretty rude.

please don't do this via data urls and inputting paths, a much better way would be to use DiscordNative.fileManager, fs, and blob urls
blob urls > data urls because you don't need to specify or hardcode mime type

I was in a VC with a buddy while I was doing this and he told me exactly that, I just wanted to get something done fast.
I will ammend my code to be closer to that which you've described as soon as I get a second.

Two questions:
Where can I read about DiscordNative? I've never had to deal with it, and I'm a bit too inexperienced to open the console and screw around until I find out.
Using this way of file access, would supplying a file path still be fine or would a file dialogue opening actually be necessary?

Where can I read about DiscordNative? I've never had to deal with it, and I'm a bit too inexperienced to open the console and screw around until I find out.
fileManager of discordnative has stuff to use the OS file picker to get a path


Moved to proper repo.


[r2d2clk] Add screenshot to metadata.json

The screenshot was missing from the metadata in the app store initially, so this just adds the screenshot in so it will show up.

Great - thanks!


Add README.md and package.json for example packages

This pull request introduces new documentation and configuration files for example packages in the repository.

Thanks a lot for this very thorough update and documentation. My one minor concern is that duplicating an entire SQRL script and GraphQL schema into the documentation will be hard to maintain. If we need to update the source files, we would have to remember to update the README. I propose that we instead link to those files. That's slightly less convenient for the user, but much easier to maintain moving forward. WDYT?

I agree with your point and had the same thought. However, I believe it’s crucial to prioritize user convenience. Sometimes, saving a click can significantly enhance their experience, especially when navigating a document like this one. What if we embedded other README files into this document, similar to using an  in HTML. Fortunately docusaurus allows for similar functionality: https://docusaurus.io/docs/markdown-features/react#importing-markdown
One downside is that, in a few cases (maybe just one), I had to adjust the script slightly to fit the example packages. I’m unsure how to address this, but it might be lower on our priority list.
What do you think?

I agree with you that having the full example in the documentation (without having to link elsewhere) is what we want. I think the iframe idea is a neat why to accomplish that without duplicating code.
However, this PR is about the documentation of the packages in the repository. That's where these readme's are embedded and there we want to link to the repository or datasqrl-examples. I don't think the repository should turn into another source of documentation beyond what is in the actual packages.


Add more comprehensive inject/extract example to API docs

Related: https://discuss.elastic.co/t/java-agent-how-to-manually-instrument-an-rpc-framework-which-is-not-already-supported/172730


Implement ExceptionSpec

r @jcorbin @malandrew @leizha

needs a rebase, and please keep the fixtures consistent, it can be easy:

install https://github.com/godlygeek/tabular plugin
with cursor on any one of the fixtures lines, :Tab/\/\/
add normal mappings to taste for #productivity

oh but: lgtm otherwise


Replace load channel cleaning thread with bthread.

There are too many threads in our BE process, try to reduce the number of threads by
running some lightweight tasks in bthread instread of pthread.
What type of PR is this：

[ ] bug
[ ] feature
[ ] enhancement
[ ] refactor
[x] others

run starrocks_admit_test


Added recipe exclusions

TODO: add fixes.
TODO: update changes

Closing the draft, I will update the open issues with info in a moment


Question / issue with Update Email functionality?

Hi guys!
First thing is this is an awesome repo! Great work everyone!
Disclaimer:
Forgive me if submitting an issue is not the right place for a question.
I'm new to programming and am going through the code and working on understanding all the logic and actions that are taking place.
These may not be issues and I haven't actually cloned this repo and tested things yet.  Just trying to understand it all.
My questions

What exactly happens when the function in the below code is called (what happens in Supabase and Stripe)?
Does this also trigger an update to update their email address on the customer Object in Stripe? Should it or does it not create issues if there is a different email address on the 'users' table in Supabase vs the email address on the customer object in Stripe?
What is the impact on future logins for the user if you have social providers setup (say Google and Facebook) and they try to login again but choose to use Google instead of say email+password and each of those methods have different email addresses?
What is the impact on future checkout sessions if there is a different email address on the 'users' table in Supabase vs the email address on the customer object in Stripe?

The Code: Found under app/account/page.tsx
const updateEmail = async (formData: FormData) => {
    'use server';

    const newEmail = formData.get('email') as string;
    const supabase = createServerActionClient<Database>({ cookies });
    const { error } = await supabase.auth.updateUser({ email: newEmail });
    if (error) {
      console.log(error);
    }
    revalidatePath('/account');
  };

Thanks!
Nate

@nate-oo Great questions Nate, did you get the answers to these?

@scottklein7 I did not.
I figured I would just Close it was sitting as Open for such a long time and make it one less Open Issue/Question in the repo. I know the contributors are busy building awesome stuff at Supabase, Stripe, and Vercel/Nextjs.
I also haven't looked into this deeply on my own yet. But here are my initial thoughts.
Friendly Disclaimer: This could be wrong as I haven't spent the time yet and am not at that stage of my app.

I believe that the updateEmail Server Action will only update the email address on the Supabase managed table auth.user.
Unless I am missing something, I did not see code anywhere else that updates the email on the user's related record on the Stripe Customer object. In my opinion, I think it probably should so those stay in sync with each other.
If the email address on their social login is the same as the email address on their email+password login, I believe Supabase might automagically link those together.  If the email addresses are different, I believe it will create a different (i.e. duplicate) user in Supabase. I don't know if there really is a way you can prevent that and I don't know how to merge those users in Supabase if that happens.  My initial thoughts are that I probably will just add some additional text to my sign in form below the sign in button that says something to the user about this.
I believe there is no technical impact if the email addresses differ from the Supabase auth.users email value and the email value of their related Stripe Customer record.  I believe the Stripe Customer record's email is what will show as greyed out in checkout.  So at most I think this would create confusion for the user.  Say for instance, on the Account page of your app they update their email. The Server Action updates it. They now have a different email than what is on their related Stripe Customer record because the code doesn't update the user's related Stripe Customer record's email value. NOW, they then make it to checkout and the email greyed out in Checkout show's their old email because their related Stripe Customer record's email value was never updated.  This may create some confusion for the user.

Again could be missing things or wrong on some things here. Hopefully this is some food for further thought.


Using AF3 weights with Protenix

Is there a way to use the recently released AF3 weights using Protenix so that we can do a head-to-head comparison on specific use cases? OpenFold has a similar implementation to use AF2 weights with their PyTorch OpenFold model.

Hi @abhinavb22, our model is released before the AF3 inference code is available, currently we do not support do a head-to-head comparison with released AF3. Additionally, due to the limitations of the AF3-related license, this functionality will not be supported in the future.


Turn questionnaire into a series of inquiries

The current questionnaire exists of one large inquirer.prompt()-call.
Turn this into a series of prompts instead to streamline the user experience.

[x] Turn questionnaire into a series of prompts
[ ] ~Substitute assets/screenshot.svg with a video~

Video demo scrapped. Will show website screenshot when #23 is done


Add toggle for profile

Just as the title says. Is there a way you could add a toggle for profile?  So if someone isn't signed in I can have that whole class dissapear.
Thanks

Hi, as soon as possible I will add this feature. Thanks for your offer.


Replace tabix/bgzip dependencies with seqminer

Having separate command-line installation steps outside of R is a pain for users (e.g. htslib).
Instead, I've rewritten all the tabix-related functioned to be entirely R-based, using seqminer and Rsamtools for bgzip.

Now implemented in dev branch

There's actually several places where tabix is used:

DONE✅ : Converting fullSS to tabix-indexed file
DONE✅ : Querying tabix-indexed fullSS
Querying 1KG VCF to compute LD

Still working on this last part, since queryingf a remote VCF seems to be a bit more tricky using the available packages. Here's some of my latest attempts:
vcf_url <- "ftp://ftp-trace.ncbi.nih.gov/1000genomes/ftp/release/20110521//ALL.chr18.phase1_release_v3.20101123.snps_indels_svs.genotypes.vcf.gz"
 subset_DT <- echolocatoR::BST1
 gr <-  DT_to_GRanges(subset_DT)

  #### Rsamtools ####
  tab <- Rsamtools::scanTabix(file = vcf_url, gr)

  #### seqminer ####
  tab <- seqminer::tabix.read(tabixFile = vcf_url, tabixRange = coord_range )

  #### VariantAnnotation::scanVcf ####
  vcf <- VariantAnnotation::VcfFile(file = vcf_url, index = paste0(vcf_url,".tbi"), yieldSize = 5000)
  line <- VariantAnnotation::scanVcf(file=vcf)
  #### VariantAnnotation::readVcf ####
  vcf <- VariantAnnotation::VcfFile(file = vcf_url, index = paste0(vcf_url,".tbi"))
  line2 <- VariantAnnotation::readVcf(file=vcf, genome = "hg19", param = gr)

seqminer seems to work best for the fullSS tabix indexing and querying, but I've submitted an Issue to try to get it working for remote VCFs:
https://github.com/Bioconductor/VariantAnnotation/issues/29

All tabix-related functions are now handled by echotabix:
https://github.com/RajLabMSSM/echotabix


Benchmarks

Hi,
Do you have benchmarks of the iteration time of the same model in both caffe and caffe64
Thanks

My bet would be that caffe would be faster since those folks know what they're doing. But we have more numbers appended after our name (64 vs "" = 0), compile much faster, and save a ton on disk space!


translation of Tui.calendar into german

I use the tui.calendar as part of my TYPO3-extension timer.
How can i translate it into german?
The issue https://github.com/nhn/toast-ui.vue-calendar/issues/49 won't work for .
The renaming of the weekday needs dayNames instead of daynames. Then it works.
I did not get to work the renaming of the template-part. ('Milestone, Task, Edit[Button], ... ) with the issue from above.
How can I translate tui.calendar into german?

Version
tui.calendar 2.1.3

I think, it was my mistake. the version with the template works.
Close this ticket.


Project needs to fail to load gracefully when it's missing targets

This is potentially an MsBuild bug. When the project is missing targets it should fail to load (in the same way as a missing import). Currently it's loading but due to it missing targets there are many issues down the road (#921 is an example of this)

This isn't targets per se, but "SDKs". I think missing with regards to missing targets - there is a more graceful errors (and then the Installer query runs, to check for missing workloads)

I think this is 2 issues.

SDK is missing. In this case MSBuild should throw an exception and it will be displayed to the user.
Props/Targets is missing critical XAML Rule files. This will be a CPS fix. Right now CPS throws an obscure error along the lines of "ConfigurationGeneral BrowseObject could not be found" when critical rule files that must be present for CPS to run are missing. We need to display a clearer message to the user.

2 is tracked by https://github.com/dotnet/roslyn-project-system/issues/793

Changing this issue to track the work to show some UI around missing SDK and offering some acquisition experience.


Treat lists as big paragraphs

Assembla: https://www.assembla.com/spaces/idio-platform/tickets/5199
Reporter: @keynmol

Given that the number of pages that have missing elements in lists is increasing, it's worth revisiting the algorithm vanilla jusText uses to classify list elements(it does it separately).
The idea was briefly described here: https://github.com/idio/jusText/issues/1 and successfully tested in a similar concept in Mal's JS scraper.

/cc @keynmol, @pierslowe

At 2015-04-13T08:35:53.000+00:00, @keynmol said:

On Hold because a simple solution greatly reduces the precision of paragraph classification due to various list-based navigations and other website elements.
Clumping list elements into one big super-element also affects classification of surrounding paragraphs, which screws up everything.


MPI Stress Test Suite

A nice to have:It would be great to have a set of scripts that we can use as standard VisIt MPI Stress test on various platforms: This may include tests using the following features:PosCMFE, CCL, PICS, IceT, Volume Rendering, GhostZone Comm
-----------------------REDMINE MIGRATION-----------------------
This ticket was migrated from Redmine. As such, not all
information was able to be captured in the transition. Below is
a complete record of the original redmine ticket.
Ticket number: 1482
Status: Pending
Project: VisIt
Tracker: Feature
Priority: Normal
Subject: MPI Stress Test Suite
Assigned to: -
Category: -
Target version: -
Author: Cyrus Harrison
Start: 05/31/2013
Due date:
% Done: 0%
Estimated time:
Created: 05/31/2013 07:36 pm
Updated: 06/19/2013 02:52 pm
Likelihood:
Severity:
Found in version: 2.12.3
Impact: 4 - High
Expected Use: 1 - Exceptional
OS: All
Support Group: Any
Description:
A nice to have:It would be great to have a set of scripts that we can use as standard VisIt MPI Stress test on various platforms: This may include tests using the following features:PosCMFE, CCL, PICS, IceT, Volume Rendering, GhostZone Comm
Comments:

Our test suite is now split into serial, parallel, and IceT versions.


Add 5.0 compiler options

It would be great if all the errors were reported to me at one time instead of waiting 20 minutes to find out the next one

Yeah, I would always update locally and so going through errors was trivial - I imagine that change might need to happen inside the shiki-twoslash project as I think we throw in there?

These errors don’t appear for me locally when I run any build command I can find

The one that’s failing now is one I’ve already updated. Stale cache or something?

Yeah, sounds like a stale cache - I wonder why I never hit this locally
If you run rm -rf node_modules/.cache/twoslash it should empty the cache

Regarding verbatimModuleSyntax, I think the docs could be slightly more clear, in terms of how it behaves differently when it's true vs. false. Right now it just says "by default, TypeScript does import ellision..." - how does this translate to the default value for verbatimModuleSyntax? It's not clear from the docs, AFAICT, but I might be missing something. Additionally, the type hint of "do not transform..." is confusing because - I'm assuming - it means "if set to true, do not transform..."
Maybe this won't be so confusing once all the docs are pushed out and there is more information about this on the web, but I just thought I'd mention I found it confusing while upgrading tonight.
Thanks for all the hard work on 5.0!

Error codes are scattered throughout https://github.com/microsoft/TypeScript-Website-Localizations and have to be changed there first. 🤬


added support for specifying user data for the worker launch config t…

In reference to #42

Hey @jimbeck -
This looks good to me. The one change I'd request is aligning the userdata variable names. Let's make what you have here conventional and alter the other to be post_userdata. There's a spot in the test fixture where this should be altered as well.
Also, be sure to add your change to the CHANGELOG to be released under v1.3.0.


rustup, and now we also need to install llvm-tools

Fixes fallout from https://github.com/rust-lang/rust/pull/72000

@bors r+

:pushpin: Commit 42a3e8783861e82a58a769df8e2e718b38cd25cd has been approved by RalfJung

:hourglass: Testing commit 42a3e8783861e82a58a769df8e2e718b38cd25cd with merge 427c8a6bb876ec7856d69ef7037294a1f1d2a1db...

:sunny: Test successful - checks-travis, status-appveyor
Approved by: RalfJung
Pushing 427c8a6bb876ec7856d69ef7037294a1f1d2a1db to master...


Camera not opening on Android 4.0.3

Description of the problem:
Using the sample project, the camera is not opening on "CONTINUES SCANNING", "ACTIVITY WITH TOOLBAR" and "CUSTOM SCANNER ACTIVITY'.
However, the scanner is working on the rest of the options. As I need to use the continues scanning, I couldn't find the reason why the camera is not opening on these particular activities.
**Which library version are you using?
3.3.0
**Which phone/tablet are you using, and which Android version does it run?
Samsung Galaxy S II GT-I9100
Android: 4.0.3
**Does the same happen on other devices or an emulator?
No
Can you reproduce the issue in the sample project included with the library? If not, can you
provide your own sample project or sample code that produces this error?
Yes, this issue produced using the sample project.
In the case of an error do you have a stack trace or adb logs?
Not showing any error.
Thanks.

Seems like a problem with that specific device - see the workaround in #188.


Fix #3054, paste bin doesn't close unexpectedly anymore

This fixes #3054.
Basically by detecting if we are entering the paste bin panel after leaving the paste bin trigger.

Coverage remained the same at 94.444% when pulling 945e14b9d50a515480b0d4e75b354b7a570db446 on johnraz:Issue3054-paste-bin-hard-to-reach into 77304e2cc67752735241439d6069a51bd996a29c on divio:develop.


pkg/build: don't take trailing arguments in runMakeImpl

The call has quite a lot of fixed arguments, which makes it prone to errors that do not manifest themselves during compilation (see #3441).

lgtm


Alfred 2.4+ detected, but local preferences file not found

I'm having the same error as #7216 - see my comment there.
As requested, I've run the link command with the debug flag:
❯ brew cask alfred link --debug
==> Creating directories
==> Reading Alfred primary preferences
==> Executing: ["/usr/bin/defaults", "read", "com.runningwithcrayons.Alfred-Preferences", "version"]
==> Reading Alfred primary preferences
==> Executing: ["/usr/bin/defaults", "read", "com.runningwithcrayons.Alfred-Preferences", "syncfolder"]
==> 2014-12-12 14:02:17.573 defaults[38188:333772]
==> The domain/default pair of (/Users/alex/Library/Preferences/com.runningwithcrayons.Alf
Error: Alfred 2.4+ detected, but local preferences file not found. Try running Alfred first.
/usr/local/Cellar/brew-cask/0.50.0/rubylib/cask/cli/alfred.rb:50:in `assert_installed'
/usr/local/Cellar/brew-cask/0.50.0/rubylib/cask/cli/alfred.rb:56:in `link'
/usr/local/Cellar/brew-cask/0.50.0/rubylib/cask/cli/alfred.rb:33:in `run'
/usr/local/Cellar/brew-cask/0.50.0/rubylib/cask/cli.rb:81:in `run_command'
/usr/local/Cellar/brew-cask/0.50.0/rubylib/cask/cli.rb:121:in `process'
/usr/local/bin/brew-cask.rb:42:in `<top (required)>'
/System/Library/Frameworks/Ruby.framework/Versions/2.0/usr/lib/ruby/2.0.0/rubygems/core_ext/kernel_require.rb:55:in `require'
/System/Library/Frameworks/Ruby.framework/Versions/2.0/usr/lib/ruby/2.0.0/rubygems/core_ext/kernel_require.rb:55:in `require'
/usr/local/Library/brew.rb:59:in `require?'
/usr/local/Library/brew.rb:142:in `<main>'

Here are my alfred preferences - there seems to be no syncfolder?
~/Library/Preferences
❯ defaults read com.runningwithcrayons.Alfred-Preferences
{
    NSNavLastRootDirectory = "~/Applications";
    NSNavPanelExpandedSizeForOpenMode = "{750, 480}";
    "NSToolbar Configuration com.apple.NSColorPanel" =     {
        "TB Is Shown" = 1;
    };
    "NSWindow Frame alfredpreferences" = "79 115 1100 662 0 0 1280 777 ";
    lastTab = appearance;
    lastTabBuild = 308;
    lastTabTS = "439999406.510222";
    version = "2.5.1";
}

Perhaps because I haven't synced my Alfred:

Yes, adding that users can manually add /opt/homebrew-cask/Caskroom to the searched folders would help. This fixed it for me but it took some time to find.

FWIW, I had a short twitter dialog with @alfredapp suggesting Alfred find symlinked apps in their search scope, which naturally includes /Applications by default.
I think this is a failing on Alfred's part not Cask's when out of the box Spotlight (at least on Yosemite) has the right behavior and Alfred doesn't (where "right behavior" is finding installed apps). In Spotlight's case, it finds the original, not the symlink. Since one of Alfred's core features is reduced search scope, I suggested the symlink thing. Were Alfred a little smarter wrt symlinked apps, then brew cask alfred could go away entirely. It also wouldn't matter which installer I used as long as it created a symlink.
In terms of error message for non-paid users, perhaps reference the link to Alfred's own kb article (which sparked my Twitter snark).

Hey there, just thought I'd pop a little response in here...
The syncfolder is only ever set if the user has set a sync folder, which may be never. If a syncfolder has never been set (or the user is a non Powerpack user), you can just assume that Alfred.alfredpreferences will be located in /Library/Application Support/Alfred 2/
Alfred, by design, has a more limited search scope than Spotlight to give more relevant and focused results with higher performance, as such, things like homebrew have unfortunately been left out of this (as the homebrew folder may be different per user). [Deep] Indexing symlinks is a huge headache for various reasons... So to flip this around a little bit, is there an easy way Alfred can find where a user's homebrew Cellar folder is? This way, I could add an option (enabled by default) which automatically includes the homebrew Cellar folder as part of Alfred's default search scope if installed.
Cheers,
Andrew

Thank you for stopping by, Andrew.
Adding the default directories for the Cellar (/usr/local/Cellar) and the Caskroom (/opt/homebrew-cask/Caskroom) would already be a great step. It would solve the issue for most people with the least amount of work, and if the directories don’t exist at those locations, no harm done1.
As for detecting the location of those directories, that could be a pain since you couldn’t just brew --cellar or /usr/local/bin/brew --cellar, since the idea is exactly that you don’t know where brew is, if installed to a custom location. It could require you to read the user’s PATH, but to do that you’d need to know what their shell is (I do it as part of an Alfred Workflow, but that’s not entirely ideal for this case). Automatically firing a terminal won’t really work because we couldn’t be sure even then2.
All that said, to find homebrew’s Cellar you can brew --cellar, while to find the Caskroom you’d (currently) have to do a bit of trick like brew cask doctor | grep -A 1 'Staging Location' | tail -1.
I’ll reiterate, however, that just including their default directories in Alfred’s results by default might be the best solution for the amount of work it requires.

1Yes, there could still exist user-created directories in those exact locations with those exact highly-specific names, but that is extremely unlikely.
2My default terminal application is iTerm, while I keep Terminal pristine on purpose for when I need to check if something is working/failing due to my custom configurations or not. Yes, Alfred does have the option to pick a default terminal, but it seems to be limited to those two and that assumes the user had actually changed that setting.

Yes, cheers @preppeller.

I’ll reiterate, however, that just including their default directories in Alfred’s results by default might be the best solution for the amount of work it requires.

This would probably solve it for 80% or more of homebrew and cask users. The odds that non-homebrew /usr/local/Cellar & /opt/homebrew-cask/Caskroom exist and contain apps and users don't wish them found are indeed pretty low.
This would have solved it for all but one of my problem apps. Only Emacs.app wouldn't be found by default and only because I install it with vanilla brew instead of brew cask and only because my Homebrew is at a non-standard location (via @boxen). An edge case that I'm willing to accept as my responsibility.
One potential snag is that Alfred's default search scope applies to more than just applications. The out-of-the-box defaults are pretty safe, but users that include Extras like Archives, Images, or Text Files may dislike that internal files from tools like git, node, or python are in their Alfred search results. Ideally it would only search Cellar for apps, but that's not how Alfred's search scope works today. That said, users would be no worse off than when they manually add Cellar today; if they would be unhappy with the new defaults they could remove Cellar from their search scope.

Cool, thanks for the info! I'll just add these two folders to Alfred's default search scope in 2.6... people can always remove these if they don't want Alfred finding stuff in there.
It's worth noting that this will only affect Alfred's default search scope... if somebody has modified their search scope (added or removed items), they won't see these paths added. It will help for all new Alfred users, and all users who haven't changed Alfred's default scope.
Hope this helps :)
Cheers!
Andrew

Definitely :) Thanks Andrew!
On 12 January 2015 at 00:05, Andrew Pepperrell<EMAIL_ADDRESS>wrote:

Cool, thanks for the info! I'll just add these two folders to Alfred's
default search scope in 2.6... people can always remove these if they don't
want Alfred finding stuff in there.
It's worth noting that this will only affect Alfred's default search
scope... if somebody has modified their search scope (added or removed
items), they won't see these paths added. It will help for all new Alfred
users, and all users who haven't changed Alfred's default scope.
Hope this helps :)
Cheers!
Andrew
—
Reply to this email directly or view it on GitHub
https://github.com/caskroom/homebrew-cask/issues/8052#issuecomment-69518811
.

--
Maple Design Ltd
http://www.mapledesign.co.uk
+44 (0)845 123 8008
Reg. in England no. 05920531

I've now added these folders into Alfred's 2.6 default search scope. 2.6 should be out in a few weeks or so...

Cheers,
Andrew :)

🍻

Adding to the list isn't quite easy.
Until 2.6 comes out... this works well in the standard Mac terminal if you only have one UID.
nano "~/Library/Application Support/Alfred 2/Alfred.alfredpreferences/preferences/local/$(ls ~/Library/Application\ Support/Alfred\ 2/Alfred.alfredpreferences/preferences/local)features/defaultresults/prefs.plist"

add these to the scope:
		<string>/usr/local/Cellar/</string>
		<string>/opt/homebrew-cask/Caskroom</string>

You can also manually drag folders onto that list.

yes, thanks for mentioning that.
however it isn't a command line tool (which i need for scripting) AND not everyone knows how to easily access those folders in finder. =)


WIP: Adding a simple wallet for Miden in MASM

Closes https://github.com/0xPolygonMiden/miden-base/issues/22
This PR implements a basic wallet interface. For now we only have receive_asset, remove_asset, and auth_tx.

Tests are missing, we need to build the correct mock data (blocked by https://github.com/0xPolygonMiden/miden-base/issues/93). Test structure is already there
auth_tx doesn't work yet. Falcon signature verification is missing

closing in favor of https://github.com/0xPolygonMiden/miden-base/pull/233


Add the first target and test

stdx has no dependencies, so it's a good choice for our first target.
We make an INTERFACE target, using FILE_SET HEADERS for more modern
handling.  By trial and error, it seems like the right move is to set
the BASE_DIRS to the main project folder.  The following other choices
are supported by reading the docs (i.e., Professional CMake, 18th
Edition), specifically, section 16.2.7 on File Sets:

When we say FILE_SET HEADERS, we do not need to provide TYPE.
Providing relative paths is correct, because they will be taken to be
relative to CMAKE_CURRENT_SOURCE_DIR.

We also take advantage of section 32.6, "File Sets Header Verification".
The CMAKE_VERIFY_INTERFACE_HEADER_SET variable (which we also reflect
in our docs) will automatically generate synthetic targets that make
sure we got our include paths correct.  We can "run" this "test" by
building the target all_verify_interface_header_sets.  This is how I
figured out what to set BASE_DIRS to.
On the googletest side, I followed the docs.  We use FetchContent, but
fall back to find_package in case there's a system-installed version.
This latter behaviour seems highly suspect to me, but the docs were
adamant that this is typically the right move, especially for open
source projects.  See Section 39.8 ("Recommended Practices").  It feels
like it would be nice to at least set a minimum version that we would
accept from the find_package fallback, but we can probably defer that
until we get an actual problem report.  (And, honestly, maybe there's
simply no use case for this anyway?  People who want to develop the
tests can use bazel, which for Au supports a zero-install setup.  And
people who just want to depend on Au via CMake don't really need to run
the tests.)
Another consideration is conditionally enabling the GoogleTest
dependency, as was done for the branch with the initial CMake support. I
would like to defer adding that complexity until we encounter a concrete
use case that's blocked by it, so I can be sure I understand why we
would want to do that.
It turns out that we can "build-and-run-all-tests, incrementally" with a
single CMake command that builds three named targets: all, "all header
verifications", and test.  I updated the instructions to include this.
Finally, it appears that running the tests can sometimes create a folder
called Testing, so I added that to .gitignore.
Future PRs will add a bunch more targets.
Helps #215.

Another consideration is conditionally enabling the GoogleTest
dependency, as was done for the branch with the initial CMake support. I
would like to defer adding that complexity until we encounter a concrete
use case that's blocked by it, so I can be sure I understand why we
would want to do that.

We could add if statements with BUILD_TESTING. It might be a good option to provide users an escape hatch to drop some dependencies and targets. But it would be even better to get everything to work. 😄 I agree that can be considered in a future PR.

On the googletest side, I followed the docs. We use FetchContent, but
fall back to find_package in case there's a system-installed version.
This latter behaviour seems highly suspect to me, but the docs were
adamant that this is typically the right move, especially for open
source projects. See Section 39.8 ("Recommended Practices"). It feels
like it would be nice to at least set a minimum version that we would
accept from the find_package fallback, but we can probably defer that
until we get an actual problem report. (And, honestly, maybe there's
simply no use case for this anyway? People who want to develop the
tests can use bazel, which for Au supports a zero-install setup. And
people who just want to depend on Au via CMake don't really need to run
the tests.)

Yeah, in general Ubuntu and cmake users are pretty loose on the dependency version requirements. This enables fewer downloaded libraries. But there will be less reproducibility. Also, it requires more testing. gtest like all software is under development 😄
There is a PACKAGE_FIND_VERSION_MIN_MAJOR parameter in find_package.

CMAKE_VERIFY_INTERFACE_HEADER_SET didn't seem to work as expected.

Well, I guess the docs might explain that:

New in version 3.24.

If you replace 3.23 with 3.24 in the min version command, does that help it work as intended?
If it does, then I'll make that change.

If you replace 3.23 with 3.24 in the min version command, does that help it work as intended?

I'm seeing the same issue. Here are the repro instructions.
docker run --rm \
ubuntu@sha256:2e863c44b718727c860746568e1d54afd13b2fa71b160f5cd9058fc436217b30 \
/bin/bash -c \
'apt update && apt install -y clang cmake git && git clone -b chiphogg/stdx-cmake#215 https://github.com/aurora-opensource/au.git && cd au && sed -i 's/23/24/g' CMakeLists.txt && cat CMakeLists.txt && cmake -S . -B cmake/build -DCMAKE_VERIFY_INTERFACE_HEADER_SET=TRUE && cmake --build cmake/build --target all all_verify_interface_header_sets  test'

If you replace 3.23 with 3.24 in the min version command, does that help it work as intended?

I'm seeing the same issue. Here are the repro instructions.
docker run --rm \
ubuntu@sha256:2e863c44b718727c860746568e1d54afd13b2fa71b160f5cd9058fc436217b30 \
/bin/bash -c \
'apt update && apt install -y clang cmake git && git clone -b chiphogg/stdx-cmake#215 https://github.com/aurora-opensource/au.git && cd au && sed -i 's/23/24/g' CMakeLists.txt && cat CMakeLists.txt && cmake -S . -B cmake/build -DCMAKE_VERIFY_INTERFACE_HEADER_SET=TRUE && cmake --build cmake/build --target all all_verify_interface_header_sets  test'

This would appear to be a PEBKAC issue.  (Both the K and C belonged to me.)  See 67738aa for the fix.

Yep, it works. 🎉
docker run --rm \
ubuntu@sha256:2e863c44b718727c860746568e1d54afd13b2fa71b160f5cd9058fc436217b30 \
/bin/bash -c \
'apt update && apt install -y clang cmake git && git clone -b chiphogg/stdx-cmake#215 https://github.com/aurora-opensource/au.git && cd au && cmake -S . -B cmake/build -DCMAKE_VERIFY_INTERFACE_HEADER_SETS=TRUE && cmake --build cmake/build --target all all_verify_interface_header_sets test'


Poor performance/errant behavior on iOS 13

I've confirmed that this bug can be reproduced with the examples at https://visjs.github.io/vis-network/examples/
I'm toying with a minimal implementation (4 nodes, 3 edges - one image node, shadows enabled, default config otherwise) and I've noticed that the performance when dragging nodes is significantly poorer on iOS (both Safari and Firefox, iPad Pro 11 and iPhone XS Max) than on any other platform I've tested. It performs beautifully on a cheap Android phone from several years ago, so hardware is certainly not the limiting factor.
Specifically, I notice that when I drag nodes around, the animation is quite choppy. The framerate does not appear to recover, as dragging the view around remains slow long after. Additionally, I notice node shadows flickering as I drag, and node/label text does not appear to become bolder when selected (unlike the edges) as it does on Android/macOS.
Update
I've just discovered that performance is fine on both the iPhone and iPad until I either:

tap anywhere on the canvas, or
zoom in close enough to fade labels

That is, dragging nodes around is as smooth as on any other platform until the moment I tap/zoom, at which point the animation suddenly becomes jarringly choppy (persisting even after I tap outside of the canvas/zoom out).
The other issues I described (flickering shadow, failure of node/edge labels to become bold) are always evident, however.

Hi @ptrckqnln,
sadly I have no device I could use to test this. The only thing that may help right now are logs. Could you check if it logs some errors into the console, please?

@Thomaash Unfortunately no errors are logged to the console. I'm trying to see what I can glean from the debugger with my iPhone on https://visjs.github.io/vis-network/examples/network/nodeStyles/circularImages.html, and immediately noticed a difference in the timeline (which at least confirms that I'm not simply losing my mind 😅).
Here's a profile of the application running as I drag a node around before tapping to select a node/zooming in:

And after tapping a node then dragging it around:

The green bars in the middle are composite/paint calls, and the purple bars are the js events (hammer events/timers). The green at the bottom shows CPU utilization (which is about 10% higher on average when the animation is choppy).
Notably, in the first (pre-select) case, the paint call follows composite immediately, whereas in the second case, there's an average delay of ~30ms between composite and then a doubled paint call. This is a bit outside my expertise, so I'm grasping at straws at the moment, but I'll see what else I can dig up. Let me know if there's anything in particular you'd like me to test/retrieve.

I've also learned that I don't need to tap on a node specifically to cause this issue, but rather anywhere on the canvas. There seems to be something about that tap that seriously disrupts rendering.

I also see this on a m1 Mac book pro, Monterey 12.0.1, safari v15.1.   Having the canvas selected causes the network to be very very slow.  Unselecting the canvas speeds things back to normal.  The same network works fine on chrome.      I can't see any errors in the console.  If I can help more please let me know..  my current work around is just to use chrome.
Thanks for the awesome package.

Same here, MBP M1 Pro / Safari 15.3.


ARROW-4066: [Doc] Instructions to create Sphinx documentation

This begins the documentation part about how to build the documentation itself

Can you open a JIRA issue about this and add it to the PR title? (see https://cwiki.apache.org/confluence/display/ARROW/Contributing+to+Apache+Arrow)

Codecov Report

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

@@            Coverage Diff             @@
##           master    #3198      +/-   ##
==========================================
+ Coverage   86.43%   86.46%   +0.03%
==========================================
  Files         504      504
  Lines       69628    69959     +331
==========================================
+ Hits        60181    60489     +308
- Misses       9350     9367      +17
- Partials       97      103       +6

Impacted Files
Coverage Δ

go/arrow/datatype_numeric.gen.go
41.66% <0%> (-8.34%)
:arrow_down:

cpp/src/arrow/python/helpers.cc
77.54% <0%> (-2.12%)
:arrow_down:

go/arrow/type_traits_numeric.gen.go
13% <0%> (-2%)
:arrow_down:

cpp/src/arrow/csv/column-builder.cc
95.45% <0%> (-1.95%)
:arrow_down:

python/pyarrow/tests/conftest.py
73.91% <0%> (-0.78%)
:arrow_down:

cpp/src/arrow/python/common.h
100% <0%> (ø)
:arrow_up:

go/arrow/datatype_fixedwidth.go
12.5% <0%> (ø)
:arrow_up:

cpp/src/arrow/util/variant.h

cpp/src/arrow/util/xxhash/xxhash.c

cpp/src/arrow/util/string_view/string_view.hpp

... and 25 more

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 5d1934f...c132dff. Read the comment docs.

I've pushed some improvements to the wording and made instructions more detailed.


Documents need files

The documents section should allow for folders/sub-folders.  Folder names are not static or pre-defined.
Example:
/portalroot
|
|- 2019 ConCom
|- 2018 ConCom
|- ConCom Archive
| |-
|- ConCom Common
|- How to brush a lama

This  brings up some important design questions.

Do you want to load the whole directory structure on initial load and have a tree view sort of control?  For small directory trees  this will be fast and easy to navigate. However if the total  documents gets really large then it start to take a long time to load everything and that becomes a problem.

Do you want the UI to be browsing based where when you click on folders they somehow refresh the web page with that folders contents? We would likely need '..' or 'parent folder' or something similar as links as well to allow back up the folder tree.   This should have faster initial load times but then you have additional  load times with each tree navigation up and down. Also an easier time getting "lost"

something else? Maybe a tree view that populates branches on first open?

I think I vote #2 honestly.  Though I do not think it would be the easiest to use I think it is likely to be the most robust.

I agree with you and also prefer #2, with a link "jump to top folder" so people don't get lost.


Raise Jackson version to 2.8.4 with bugfixes

fixes #389

Coverage remained the same at 82.601% when pulling 1338955f3d61ad5752f1ac0ddb206775c32eceea on jackson into 57f1db891dce133bf32a02c4d0a774a8d84a7247 on master.


Unit Test Coverage Threshold

Description
Setting unit test coverage target to 80% with a 2% threshold. Builds will fail if coverage hits 78%.
Issues Resolved
Closes #98
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Codecov Report

Merging #101 (026c9f7) into main (8fc68ae) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #101   +/-   ##
=======================================
  Coverage   81.42%   81.42%
=======================================
  Files          13       13
  Lines         140      140
  Branches       27       27
=======================================
  Hits          114      114
  Misses         24       24
  Partials        2        2

Flag
Coverage Δ

dashboards-search-relevance
81.42% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


how i create blink bar like this

how i create blink bar like this
issue.zip

You need to keep the time value for the bar you want to blink, and look up the bar index based on that time value. See the example at https://jsfiddle.net/nagix/gyojxv31.

@nagix
Thk for this way but
in your example i want chart not slide
Is it possible?
i disabled type: 'realtime', in barOptions  then chart not display
how i do chart Blink and chart not slide;
please help me
var barOptions = {
animation : { duration: 0 },
scales: {
xAxes: [{
type: 'realtime',
ticks: {
fontSize: 6
}
}],
yAxes: [{
ticks: {
fontSize: 7,
beginAtZero: true,
max: 10,
stepSize: 2,
fixedStepSize: 2
}
}]

If you don’t want to slide bars, why are you using this plugin? Your example in issue.zip seems to be working well.

@nagix  i think your plugin will be solving my  Problem well more than my way. thank you for your recomendation.


ci: update opbeans branch to main

:green_heart: Build Succeeded

the below badges are clickable and redirect to their specific view in the CI or DOCS

Expand to view the summary

Build stats

Start Time: 2021-12-16T16:47:44.378+0000

Duration: 68 min 13 sec

Commit: 2db0e7e13de5a483e841534671c6c7271279796c

Test stats :test_tube:

Test
Results

Failed
0

Passed
2639

Skipped
21

Total
2660

:green_heart: Flaky test report
Tests succeeded.
:robot: GitHub comments
To re-run your PR in the CI, just comment with:

/test : Re-trigger the build.

run benchmark tests : Run the benchmark test.

run compatibility tests : Run the JDK Compatibility test.

run integration tests : Run the APM-ITs.

run elasticsearch-ci/docs : Re-trigger the docs validation. (use unformatted text in the comment!)

:green_heart: Build Succeeded

the below badges are clickable and redirect to their specific view in the CI or DOCS

Expand to view the summary

Build stats

Start Time: 2021-12-17T08:39:16.356+0000

Duration: 67 min 55 sec

Commit: 2db0e7e13de5a483e841534671c6c7271279796c

Test stats :test_tube:

Test
Results

Failed
0

Passed
2639

Skipped
21

Total
2660

:green_heart: Flaky test report
Tests succeeded.
:robot: GitHub comments
To re-run your PR in the CI, just comment with:

/test : Re-trigger the build.

run benchmark tests : Run the benchmark test.

run compatibility tests : Run the JDK Compatibility test.

run integration tests : Run the APM-ITs.

run elasticsearch-ci/docs : Re-trigger the docs validation. (use unformatted text in the comment!)


Resolve Web Dependabot Vulnerabilities

Title of ticket: Resolve Web Dependabot Vulnerabilities
Description
Resolve as many dependabot issues for the web/ folder as possible.  This does not include the api/ issues, which is a separate task.
Dependabot issues can be found here: https://github.com/bcgov/foi-requests/security/dependabot?page=3&q=is%3Aopen
Most issues should be resolvable by updating package.json.  If any are particularly tricky, and might require coordination with third party package maintainers, I'm happy to help.  However, the first-pass of fixes should be done by a developer with domain expertise for the web section.
Dependencies
Are there any dependencies?
DOD

[ ] All web dependapot vulnerabilities are either resolved or split into other tickets

closing as duplicate


Update the examples

utilizes the updated elm-style-animation lib
utilizes the updated elm-scroll lib
includes changes required by the update to 0.17

I just realized that @pietro99 has opened #4 for this.  Closing.


Fix bug with ChartingState when backspacing out

The title should say everything

so this basically fixes #10368 ????

so this basically fixes #10368 ????

yes


add 32bit architecture/pythons for windows

Running into problems with py27/32b:
https://github.com/conda-forge/staged-recipes/pull/702
Adding them here!

Coverage remained the same at 47.196% when pulling 17a7ed8b3934c0f8f625feb911c786d6ceba7189 on appveyor-32 into d6dd56bdc15ef90d0c3735f83ad62ace24b28792 on master.


Fix typo and add property

asyuc -> async
Tail -> Tall
Add WebViewHeightRatio and MessengerExtensions in UrlButton

Thank you 👍


Modularize and clean up Categories.Category.Construction.Graphs

The module Categories.Category.Construction.Graphs should be refactored.
The module currently contains
* basic definitions (graphs and their morphisms),
* the definition of the category of graphs,
* the functor mapping graphs to their free categories,
* an adjunction between the forgetful functor and the free functor,
* various helper/utility constructs and lemmas (e.g. path equality, etc.).
These should probably all go in separate modules. Some of the code should be moved to the standard library.
(See comments in the code for more details.)

Note that there is currently a pending PR (#42) that involves changes to the module. Any further changes should probably be made after that PR has been closed.

I entirely agree, that module is a mess.


[DOCFIX] Fix typo in Code-Conventions.md

Fix https://github.com/Alluxio/alluxio/issues/12131

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-PR-Builder/11301/
Test PASSed.

alluxio-bot, merge this please

alluxio-bot, cherry-pick this to branch-2.3 please