File modification date in items

Some providers regenerate files in-place several times, and storing file/asset modification date in the catalog file would speed up reads compares with querying each file individually. (Reading directory listings helps with speed, but directory listings are not standardized.)

As discussed on gitter a bit, I think there is a need for a catalog listing that provides as little information as possible that still allows bulk mirroring of repos. Example:
{
"type": "collection_manifest",
"collection_id": "some_collection",
"items": [
"id": "item_id1",
"assets":  {
"B1": {
"href": "https://site/some_collection/asset_id1_B1.tif",
"mtime": "2010-02-03T04:05:06Z"
},
]
}

I'm closing this as a duplicate of #351 , which suggest more fields but includes one for file modification. Feel free to discuss there.


Minor updates

Cleaned up a hyperlink and added documentation to update the bot endpoint when registering the bot

OPS Build status updates of commit 070c573:
:clock10: Preparing: average preparing time is 46 sec(s)

OPS Build status updates of commit 070c573:
:clock10: Incremental building: average incremental building time is 21 sec(s)

OPS Build status updates of commit 070c573:
:warning: Validation status: warnings

File
Status
Preview URL
Details

msteams-platform/get-started/get-started-nodejs-app-studio.md
:warning:Warning

Details

msteams-platform/includes/get-started/get-started-use-app-studio.md
:white_check_mark:Succeeded

msteams-platform/get-started/get-started-nodejs-app-studio.md

Line 12: [Warning] Invalid file link:(~/concepts/apps/apps-overview).
Line 18: [Warning] Invalid file link:(~/get-started/get-started-tenant).
Line 19: [Warning] Invalid file link:(~/get-started/get-started-configure-teams).
Line 126: [Warning] Invalid file link:(~/get-started/get-started-nodejs-in-azure).
Line 176: [Warning] Invalid file link:(~/get-started/get-started-nodejs-in-azure).
Line 176: [Warning] Invalid file link:(~/get-started/get-started-nodejs-in-azure#configureenvironmentvariables).

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


Multiple result set and chart controls bug

Issue Type: Bug
When I run a multi result set query and trying to visualize as a chart, in some cases the control with the dropdown list with the chart types are not available, ui has some discrapancy
Azure Data Studio version: azuredatastudio 1.5.1 (6a500715a7f2a9c5466d66b3cc312308bf59846d, 2019-03-16T01:14:53.392Z)
OS version: Windows_NT x64 10.0.17763

System Info

Item
Value

CPUs
Intel(R) Core(TM) i5-7300U CPU @ 2.60GHz (4 x 2712)

GPU Status
2d_canvas: enabledchecker_imaging: disabled_offflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onnative_gpu_memory_buffers: disabled_softwarerasterization: enabledvideo_decode: enabledvideo_encode: enabledwebgl: enabledwebgl2: enabled

Memory (System)
7.93GB (1.75GB free)

Process Argv

Screen Reader
no

VM
0%

Extensions: none

Thanks for submitting this issue. Please also check if it is already covered by an existing one, like:

Preserve chart settings when re-running query (#185)

Nope, it is a different issue.

I have this issue for simple queries also.

@jpomfret , Are you still seeing this issue with latest ADS?

Don't remember exactly what I was doing, but no it seems to work fine now. Thanks


chore: Type ignore on max_length_time_axis

In the trajectory buffers (normal and prioritised), mypy is unhappy with max_length_time_axis, because it thinks it may be a None. If you follow the control flow, it won't ever be None, but using any of the other mypy changes (e.g. a "cast") will make the code messier, I believe. Simpler, here, to use the type ignore.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.1 out of 2 committers have signed the CLA.:white_check_mark: callumtilbury:x: vikranth-tYou have signed the CLA already but the status is still pending? Let us recheck it.


how can I use this project to train "Chinese keyword model"?

How can I train a new model?
how can I use this project to train "Chinese keyword model"?

read the faq, please.

Thank you!


UNSAFE Life cycles

I'm not using any UNSAFE life cycles in my project and I'm getting warnings mentioning to update withGoogleMap component.

@yasithgamage
The repo of this project is unmaintained more than a year, and we had build new version https://www.npmjs.com/package/@react-google-maps/api
We had rewrite it to TypeScript, and updating it frequently: https://github.com/JustFly1984/react-google-maps-api/tree/master/packages/react-google-maps-api
You can enjoy autocomplete.
You can see our docs: https://react-google-maps-api-docs.netlify.com/
Also a lot of examples: https://react-google-maps-api-gatsby-demo.netlify.com/ https://github.com/JustFly1984/react-google-maps-api/tree/master/packages/react-google-maps-api-gatsby-example/src/examples
The bundle size is much smaller: https://bundlephobia.com/result?p=<EMAIL_ADDRESS>Our Spectrum community: https://spectrum.chat/react-google-maps
Enjoy!

@yasithgamage
The repo of this project is unmaintained more than a year, and we had build new version https://www.npmjs.com/package/@react-google-maps/api
We had rewrite it to TypeScript, and updating it frequently: https://github.com/JustFly1984/react-google-maps-api/tree/master/packages/react-google-maps-api
You can enjoy autocomplete.
You can see our docs: https://react-google-maps-api-docs.netlify.com/
Also a lot of examples: https://react-google-maps-api-gatsby-demo.netlify.com/ https://github.com/JustFly1984/react-google-maps-api/tree/master/packages/react-google-maps-api-gatsby-example/src/examples
The bundle size is much smaller: https://bundlephobia.com/result?p=<EMAIL_ADDRESS>Our Spectrum community: https://spectrum.chat/react-google-maps
Enjoy!

If I go with your new


[styled-engine][styled-engine-sc] Allow use of styled.div, not just styled('div')

[x] I have searched the issues of this repository and believe that this is not a duplicate.

The only other issue regarding this is #22147.
Summary 💡
When using Emotion's styled, it's possible to use the syntax:
const Flex = styled.div`
  display: flex;
`

This doesn't work when importing styled from @mui/materials/styles. Instead, this is required:
const Flex = styled('div')`
  display: flex;
`

This is fine, but Emotion's syntax is a little nicer.
Examples 🌈
Note Emotion's styled documentation examples:

Motivation 🔦
As a developer who's used to Emotion, I've found myself making this mistake repeatedly over the past few days. It'd be nice for MUI to have the same syntactic sugars that Emotion has.
Relevant Code
This is the code on Emotion's side that powers this feature:
https://github.com/emotion-js/emotion/blob/main/packages/styled/src/index.js
// @flow
import styled from './base'
import { tags } from './tags'

// bind it to avoid mutating the original function
const newStyled = styled.bind()

tags.forEach(tagName => {
  // $FlowFixMe: we can ignore this because its exposed type is defined by the CreateStyled type
  newStyled[tagName] = newStyled(tagName)
})

export default newStyled

Thanks for all your hard work and efforts, v5 is looking amazing.

@markalfred I am pretty sure you can you use emotion's styled.div syntax. Just make sure you have @mui/material @emotion/react  and @emotion/styled installed. Please run one of the following commands:-
// with npm
npm install @mui/material @emotion/react @emotion/styled
// with yarn
yarn add @mui/material @emotion/react @emotion/styled
And then import styled from @motion/styled
check this code sandbox https://codesandbox.io/s/modern-wildflower-hijbc?file=/src/App.js

@HyphenFrox you're totally right. I didn't think theme would be available unless I imported from @mui/material/styles but I see it there in your codesandbox. Thanks for the response!

Please don't import anything from @mui/material/styles unless absolutely necessary. I believe they are legacy imports. That's why they didn't have the functionality you were looking for. Try to import everything from @mui/system or @mui/material
You can use the theme in the styled function as demonstrated at https://mui.com/system/styled/#using-the-theme

Got it. Just want to note that this results in an error:
import { styled } from "@mui/system";

const MyDiv = styled.div`
  background-color: red;
`

// => TypeError: _system.styled.div is not a function

I'll use @emotion/styled from now on, thanks for the help.

How do I use @emotion style syntax when passing theme ?
This works:
import { styled, createTheme, ThemeProvider } from '@mui/system';
const MyTestButton = styled(Button)(({ theme }) => ({
color: theme.palette.mode === 'dark' ? '#fff' : '#000',
padding: '20px 10px',
borderRadius: 0,
fontWeight: 500,
fontSize: '0.95em',
'&:hover, &:focus': {
backgroundColor: theme.palette.mode === 'dark' ? '#eee' : '#222',
},
}));
I would really prefer using standard css inside the style block ( the emotion syntax / kebab-case)  vs. using the camelCase and values in single quotes.
I did try enclosing the function within template literals but that does not seem to work either when passing the theme. Any pointers or a simple example would be appreciated.
Thanks

@kdgit1 As far as I know you cannot use theme inside styled.div because the theme is part of MUI and styled.div is an emotion feature. They can't talk to each other. If you want to use the theme, you have to use styled() function. https://mui.com/system/styled/#using-the-theme

Thanks for your quick response.
I was able to make it work by wrapping the app with ThemeProvider from emotion and passing the same theme object that is passed into the Mui ThemeProvider
Works like a charm ( for now; haven't tested out everything but the basics sure seem to work just fine ). So glad to be able to write css as simple clean css.

@kdgit1 You don't need to import ThemeProvider from emotion. Just use the ThemeProvider from @mui/material. Emotion will still read your theme object from MUI. I updated the codesandbox if you want an example. https://codesandbox.io/s/modern-wildflower-hijbc?file=/src/App.js

@mnajdova has been any progress on this?
Docs say that styled-engine-sc is a wrapper, but in reality, it cuts off functionality described in this issue as well as #29723 (which looks like a duplicate), as well as it cuts off .attrs functionality as mentioned in #29492.
If so, styled-engine-sc should not be stated as a wrapper, but rather as a fork with intersectioned with @emotion/styled interface.

@h0tw4t3r I don't see why we shouldn't add the API proposed in my previous comment, feel free to grab it you want to work on it. styled-engine-sc is an adapter, not wrapper, agree we should clarify that. Could you please link where did you read that it is a wrapper?

@mnajdova Sure.
https://mui.com/guides/styled-engine/#how-to-switch-to-styled-components

@mui/styled-engine - a thin wrapper around emotion's styled() API, with the addition of few other required utilities, such as the  component, the css and keyframe helpers, etc. This is the default.
@mui/styled-engine-sc - a similar wrapper around styled-components.

styled.div,  styled.small, etc. are very useful shorthands for styled('div'), styled('small').
Those who used styled-components will probably agree that styled(...) notation is used often for React components and not default HTML tags.
I'm not familiar with @emotion/styled, but I personally don't see a reason to cut off such functionality if such exists only for upcoming developers switching to @mui to ease their migration.
Can you please tell me if the patch provided above just misses the TypeScript part of it?
I could make a PR if so.

@h0tw4t3r sorry for the delay, let's add the patch and some spec tests and we can see what's missing. Are you still interested to work on it?

@h0tw4t3r sorry for the delay, let's add the patch and some spec tests and we can see what's missing. Are you still interested to work on it?

I'll work in the free time, yes.

Perfect, will look out for the PR :)

mnajdova
Would like to make this PR. Can you guide me?

@geraldaddey sorry for the late response, the guidance was provided in a comment above - https://github.com/mui/material-ui/issues/28574#issuecomment-938568571

I am closing this issue. Better to have one way of building styled component.


[Bug]: coolify-testing-host doesn't rebase /data/coolio paths.

Description
The coolify install.sh script creates /data/coolify folders on the docker host system. However the install of collabsio:coolify-testing-host in docker-compose.windows.yml does not.
The /data/coolify folder is correctly mounted in the coolify-testing-host container, but as it is using /var/run/docker.sock to use the host's docker instance, the host docker can only these files at/home/user/coolify/services/sgso880
Minimal Reproduction (if possible, example repository)

Follow the instructions at Docker Desktop to install the Coolify compose with coolify-testing-host
Create a first user, and project and try to add and deploy the Docker Registry Service as a resource.

Exception or Error
Saved configuration files to /data/coolify/services/sgso880.
Creating Docker network.
Starting service.
Pulling images.
 registry Pulling
 registry Pulled
Starting containers.
 Container registry-sgso880  Creating
Error response from daemon: invalid mount config for type "bind": bind source path does not exist: /data/coolify/services/sgso880/auth/registry.password

Version
v4.0.0-beta.258

I also use Docker Desktop and have the exact same problem when I create MongoDB resources using a template.
Starting mongodb-database-ro4ws40.
Pulling mongo:7 image.
 kggc8ws Pulling
 kggc8ws Pulled
 Volume "mongodb-db-kggc8ws"  Creating
 Volume "mongodb-db-kggc8ws"  Created
 Volume "mongodb-configdb-kggc8ws"  Creating
 Volume "mongodb-configdb-kggc8ws"  Created
 Container kggc8ws  Creating
Error response from daemon: invalid mount config for type "bind": bind source path does not exist: /data/coolify/databases/kggc8ws/docker-entrypoint-initdb.d

Have you figured out a solution for this?

Hey guys, appreciate a look at PR #2710. New option to choose install DIR aka data/coolify... would solve this right?

It would solve it, but only if it works with windows paths. Currently coolify automatically transforms any path into unix path, it adds / if it is not first symbol.

I see, but we could fix that PR to change docker-compose.windows.yml too I guess

There is no issue with that particular file, program as whole works fine (mostly). Issues happen in compose files of individual services. For example, install_path/databases/vc40gwc/docker-compose.yml. Manually changing those is pointless, as they are overwritten to default state the moment service starts.

If you want to use Coolify for hosting you stuff, please use one of the supported Linux distors as Windows (windows server) is not production ready yet. If you are a developer, please follow this guide: https://github.com/coollabsio/coolify/blob/main/CONTRIBUTING.md


make this api-client compatible with umami 2.9.0

hi, the goal is to make this api-client compatible with umami 2.9.0
there is an helper with the following ticket
boly38/umami-api-client#15
Bonus: it could be over-mega-great to add a "CONTRIBUTE.md" helper in order to help (this) api-client users to install/build current source. I miss how to install requirement to play unit tests for example an confirm my hypothesis.
(anyway it seems that /api/websites data level in 2.x umami responses are not undertaken here.)
Nb: notice this related issue on umami doc side: umami-software/website#150
thanks 👍

Hey! I would love to add support for umami 2.x, but it's quite time-demanding due to their documentation being out of date. I usually have to reverse-engineer it by reading everything in the folders. Since I'm in my last university semester, I'm really busy lately. I also don't really use umami anymore, so it's a bit harder to test (usually I spin up a local docker container). If you could make a PR for it, it would be great! If you can't that's fine, but I can't guarantee when I'll be able to update it 😅

Hello! :wave: I no longer work in web development, and do not have a lot of free time or interest in maintaining this package anymore. I am closing all issues and PR for it and will archive the repository, but if anybody is interested in maintaining it, I am willing to transfer the repository and NPM package. I know you had been using this package for a while, sorry that I can't maintain it anymore!


Moe Sync

This code has been reviewed and submitted internally. Feel free to discuss on the PR and we can submit follow-up changes as necessary.
Commits:
 Rename PreferDurationOverload to PreferJavaTimeOverload, as it now covers Instant as well.
RELNOTES: Rename PreferDurationOverload to PreferJavaTimeOverload, as it now covers Instant as well.
ef6fe4236d6e4de0434940cf59ac8eedc5457563

 Add further caveat about VisitorState.getSourceForNode.
Per comment in #1316
47167894bf3e94ddc9c96a28d601d685a6f7eced

 Fix AutoValueFinalMethods warnings in refaster code
93af397f40351d71a440dcb00c883a1d3011ea22

 Fix FormatString Checker crash on Console.readLine()
e4361a4117e246e5639448dfdc9bf24fdbbc6602

 Consolidate stripParentheses implementations
84ed4f03d4b1f0181ce932caa6b30520acb8603e

 Upgrade to Caffeine version 2.7.0 as requested by Ben Maines: https://github.com/google/error-prone/pull/1316#commitcomment-34487169
1cb86f9f554243ecd008c2be9fc2dfb1d08ff45f

 Fix some AutoValueImmutableFields warning in refaster code
237f657fb50f56a96a6e5bb2a930459b1f789b63

Replaced by https://github.com/google/error-prone/pull/1330


Prefix container logging with service instance name

To make it easier to interpret system test logs, prefix each log line logged by a docker container with the name of the service instance running in the container.
Reviewer checklist

[ ] Ensure relevant issues are linked (description should include text like "Fixes #")
[ ] Ensure correct labels applied
[ ] Ensure any appropriate documentation has been added or amended

Coverage increased (+0.001%) to 99.309% when pulling a41e6e331d697615b6e159a927e4b0017b12e128 on prefix_logs into e0880ee47c3498074071236edc400bbddd1acdb4 on main.


Add support for AWS keys to check domain availability

check_domain_availability from boto3 seems like it should do the trick.
Gandi is their registrar associate so that's cool too.

Forgot to close this, #18 accomplished this.


Convert Bom to object from string

LGTM. @VithikaS, should we make the corresponding change in the API server first before merging this?

LGTM. @VithikaS, should we make the corresponding change in the API server first before merging this?

Accompanying PR
https://github.com/DependencyTrack/hyades-apiserver/pull/174


errors while installing requirements.txt

Hi, great repository, I'm facing issues while installing the requirements.txt on my device. please help me fix this

Same...

I managed to get it running by editing the requirements file up to date.
requirements.txt
Gunicorn
Werkzeug
numpy==1.18.5
pandas
opencv-python
cmake
argparse
scipy
dlib
tensorflow
imutils
tika
wave
nltk
gensim
seaborn
scikit-learn
dill
keras
wordcloud
pandas
argparse
scipy
dlib
tensorflow
imutils


Modify Docker Run to fix root issue

As discussed in a meeting today, we want to be agnostic about the origin of our docker containers, which means handling the Docker root ownership issue ourselves.
Here's a starting point @hannes-ucsc

Start container in the background and keep running

 docker run --name test -d ubuntu tail -f /dev/null

Run command (and block, although can be run without blocking)

docker exec test sleep 5

Run command to deal with ownership issues

Can either be a docker command to chown or something else?

Kill container

docker kill test

The problem is that for a given image you don't know what the entry point is, so you don't know what to pass to exec.
How about we docker exec … chmod using a different, minimal image (say, ubuntu) that -v-mounts the same data directory.

@hannes-ucsc and I discussed this and our solution is to run the tool image a second time, overwriting the entry point, to chown the mount directory where output is placed by the first tool.


[FIPS] Allow to deploy VA-HCI with FIPS enabled

This patch adds parameters to enabled FIPS in VA-HCI scenario. A new top level parameter was added to allow users to enable FIPS in scenarios that supports it.
As a pull request owner and reviewers, I checked that:

[x] Appropriate testing is done and actually running
[x] Appropriate documentation exists and/or is up-to-date:

[x] README in the role
[x] Content of the docs/source is reflecting the changes

recheck

/approve
/lgtm


[Gluten-161] The children of BaseJoinExec should insert WholeStageTransformerExec separately

Currently, the strategy of inserting 'WholeStageTransformer' only meet the shuffle exchange scenario, when there is a join operator, it also needs to insert 'WholeStageTransformer' for the children of the join operator.
Please refer to : WholeStageCodegenExec.scala
Close #161 .
What changes were proposed in this pull request?
(Please fill in changes proposed in this fix)
How was this patch tested?
(Please explain how this patch was tested. E.g. unit tests, integration tests, manual tests)
(If this patch involves UI changes, please attach a screenshot; otherwise, remove this)

Could you explain why we need to seperate the WholeStageTransformer for join and its children? If they are in the same transform context, they can be off-loaded to backend together.

Currently, the

Could you explain why we need to seperate the WholeStageTransformer for join and its children? If they are in the same transform context, they can be off-loaded to backend together.

Keep the same separation 'WholeStageXXX' strategy as the Spark, please see the code: WholeStageCodegenExec.scala
If keep the children of the join in one 'WholeStageTransformer', there are some errors happen:
1). in case there is one child of the join in the same 'WholeStageTransformer', for example, 'buildPlan', calling ShuffledHashJoinExecTransformer.columnarInputRDDs will throw NPE, because the 'buildPlan' will call 'columnarInputRDDs' recursively, the last one node is 'BatchScanExecTransformer' and will return null;
2). in case two children of the join are all in the same 'WholeStageTransformer', it will generate substrait plan in the first stage ( NativeWholeStageColumnarRDD ) but not the final stage ( WholeStageZippedPartitionsRDD ), on the other hand, the read rel node of two children will be set the same 'localFiles'.

I see. Indeed, separating the children into different substrait plan can fix this. Although I still have no idea why the WholeStageCodegenTransformer must be consistent with the behavior of Spark's codegen, but basically, I think:
1). is due to the getChild function in ShuffledHashjoinExecTransformer only returns streamed plan, which is unreasonable and if it's called in other code paths, it will also return unexpected results.
2). The children will generate substrait plan in the first stage ( NativeWholeStageColumnarRDD ) only when the they are in the first stage. But since NativeWholeStageColumnarRDD doen't support multiple scan nodes, seperating the children in first stages can be the solution.

https://github.com/oap-project/gluten/blob/ee362571ca5304c1cc01fb8ceacd1b8a9d8a2eb1/jvm/src/main/scala/org/apache/spark/sql/execution/ColumnarCollapseCodegenStages.scala#L136-L138
We need to modify this to:
case t: TransformSupport =>
  WholeStageTransformerExec(insertInputAdapter(t))(codegenStageCounter.incrementAndGet())

Or else in the case of two consecutive hash joins, the generated plan would be wrong.

https://github.com/oap-project/gluten/blob/ee362571ca5304c1cc01fb8ceacd1b8a9d8a2eb1/jvm/src/main/scala/org/apache/spark/sql/execution/ColumnarCollapseCodegenStages.scala#L136-L138
We need to modify this to:
case t: TransformSupport =>
  WholeStageTransformerExec(insertInputAdapter(t))(codegenStageCounter.incrementAndGet())

Or else in the case of two consecutive hash joins, the generated plan would be wrong.

Fixed.

Verified locally. LGTM.

Working on the verification. BTW, could you paste the picture of the query plan here to help readers understand the issue you are fixing?

After this pr, the new spark plans are below:
in case the children of the join are not ShuffleExchange:

in case one child of the join are ShuffleExchange:

in case the children of the join are ShuffleExchange:

Looks like we do not support the operators in the blue lines to be merged into a whole stage transformer, right?

Verified locally.

Looks like we do not support the operators in the blue lines to be merged into a whole stage transformer, right?

Yes, the Apache Spark also does like this.


Added hasFlash() for iOS

Added hasFlash for iOS #112

You don't need the boolean instantiation if you're gonna pass it right back out though. I'll merge this in a couple hours and correct it myself if you don't submit a fix before I get around to it. Thanks for your contribution!


fix: typos in InstructArgs comments

Corrected "Assisant" to "Assistant" in multiple places.
Corrected "loose" to "lose" in the comment explaining dynamic chunking.

thanks!


fix: long select won't scroll

A long select list, despite implementation, doesn't actually want to enable scrolling.
I don't know 100% certain if this will actually fix it, but by design the dialog scroll area should not be wrapped within a dialog content, see: https://callstack.github.io/react-native-paper/docs/components/Dialog/DialogScrollArea
I also removed the 2 properties from the ScrollView that controlled the display of the scrollbar.
I limited the dialog height as I always do using:
            dialogStyle={Object.assign(
                {
                    maxHeight: 0.9 * Dimensions.get("window").height,
                },
                props.dialogStyle,
            )}

and whilst it does limit the height, it pushes the buttons down and the scrolling isn't being applied at all.
All of my other dialogs function the same way, and this is the only problematic dialog.

Will test and merge it.

I tried testing for 1 Lakh = 0.1 Million records, but the app freezes and won't open the modal, and then the app crash.
used for loop to replicate 1 Lakh records:
for (let i = 0; i < 100000; i++) {
    _tempList.push({
        _id: Math.round(Math.random() * 1000) + 22 + 'TEST',
        value: 'OTHERS' + i,
    });
}

up to 2500 records the modal opens after that app freezes and then crashes
Emulator specs:
8GB ROM,  1.5GB RAM

I guess the specs are too low, which is why it keeps getting crashed above 2.5K records

That is one problem with the scrolling list, it's not very performant.
I only barely got more items that it just went out of the view, not that many.
For large amounts there is a library called https://github.com/Shopify/flash-list which is faster at rendering by only rendering on screen items aswell as rendering using a predefined height per item so that the list height is not constantly updating as much.
I used it within my project as I have other pages where it needs to render a large list of checkboxes, text inputs, etc, and it made my list perform much better. Potentially this is an idea for a future version?

Maybe for future version but I don't want the package to be dependent on any other 3rd party package

I have published the latest code on npm but I don't know why I can't see it there, the updated version is 1.0.2 you can see the release on GitHub, got a mail from npm for the same that your package is published. Any idea?


Update index.js

if target's proto has the "key", target self "key" will not changed reactive

What kind of change does this PR introduce? (check at least one)

[x] Bugfix
[ ] Feature
[ ] Code style update
[ ] Refactor
[ ] Build-related changes
[ ] Other, please describe:

Does this PR introduce a breaking change? (check one)

[ ] Yes
[ ] No

If yes, please describe the impact and migration path for existing applications:
The PR fulfills these requirements:

[ ] It's submitted to the dev branch for v2.x (or to a previous version branch), not the master branch
[ ] When resolving a specific issue, it's referenced in the PR's title (e.g. fix #xxx[,#xxx], where "xxx" is the issue number)
[ ] All tests are passing: https://github.com/vuejs/vue/blob/dev/.github/CONTRIBUTING.md#development-setup
[ ] New/updated tests are included

If adding a new feature, the PR's description includes:

[ ] A convincing reason for adding this feature (to avoid wasting your time, it's best to open a suggestion issue first and wait for approval before working on it)

Other information:

What bug are you trying to bug here. A boiled down reproduction would help


Simplify stratified CV

I thought of a way to somewhat simplify the stratified CV code. It's still basically the same algorithm, but a little easier to read. The performance is about the same:
scv = StratifiedCV(nfolds=5)

N = 10_000_000
rows = 1:(10N)

y = (
    vcat(fill(:a, N), fill(:b, 2N), fill(:c, 3N), fill(:d, 4N))
    |> shuffle
    |> categorical
);

julia> @btime MLJBase.train_test_pairs_current($scv, $rows, $y);
  9.728 s (146 allocations: 4.84 GiB)

julia> @btime MLJBase.train_test_pairs_simplified($scv, $rows, $y);
  9.478 s (243 allocations: 5.59 GiB)

Codecov Report

Merging #277 into breaking will decrease coverage by 0.85%.
The diff coverage is 100.00%.

@@             Coverage Diff              @@
##           breaking     #277      +/-   ##
============================================
- Coverage     82.16%   81.30%   -0.86%
============================================
  Files            30       30
  Lines          2114     2081      -33
============================================
- Hits           1737     1692      -45
- Misses          377      389      +12

Impacted Files
Coverage Δ

src/resampling.jl
88.01% <100.00%> (-0.15%)
:arrow_down:

src/data/datasets.jl
86.11% <0.00%> (-8.34%)
:arrow_down:

src/measures/loss_functions_interface.jl
94.73% <0.00%> (-5.27%)
:arrow_down:

src/measures/finite.jl
94.96% <0.00%> (-3.82%)
:arrow_down:

src/composition/networks.jl
64.19% <0.00%> (-2.27%)
:arrow_down:

src/measures/measures.jl
91.07% <0.00%> (-1.92%)
:arrow_down:

src/hyperparam/one_dimensional_range_methods.jl
96.38% <0.00%> (-0.29%)
:arrow_down:

src/composition/composites.jl
90.30% <0.00%> (-0.15%)
:arrow_down:

src/measures/continuous.jl
100.00% <0.00%> (ø)

src/composition/arrows.jl
27.27% <0.00%> (+2.27%)
:arrow_up:

Continue to review full report at Codecov.

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

@CameronBieganek FYI, I have updated your access. You should be able to push branches directly now.


Update to Generic Inventory to prevent casting errors.

I was implementing my own changes to the Generic Inventory when I kept running into a casting error. I managed to troubleshoot the issue and found that non-marketable items would return a market_fee_app_id of null which would cause errors when converting it to "long". The best solution for this was to set the market_fee to 0 when the item was not marketable. This issue could be fixed in many other ways such as seeing if the item is actually marketable but this is the fix I came up with.

Looks like a safe change, I can't imagine this breaking anything.


<EMAIL_ADDRESS>requires Node 14 which seems to be a breaking change

I am working on a project that requires Node 12 and when upgrading dependencies with patch and minor releases I got this warning:
error<EMAIL_ADDRESS>The engine "node" is incompatible with this module. Expected version ">=14.7". Got "12.21.0"

Shouldn't such a change be released as a major release?

Yes, that is a good point. That is a miss that we didn't trigger a breaking change. Even though we publish a new breaking change the published version will stay. What do you think we should do?
Would it solve your issue if we deprecate the latest published version and then publish a new version under a new major version?

I think a new major release would be a good approach. You could deprecate 2.1.2 and add to the release notes that there was an unintended breaking change. Then we wouldn't need to fix the version to 2.1.1 and others will know that there are breaking changes.

I have now deprecated version 2.1.2 and published a new major version (3.0.0).
Thank you for being patient with us, and please tell us if it solves your issue or not!

Thank you @juliuscc! That went quite fast, this should indeed fix our problem :)


Publish New FAQ and Register Pages (#2)

name: "Publish"
about: "Template for publishing changes"
title: "[PUBLISH] "
labels: ["publish"]
assignees: []

type: markdown
attributes:
value: |
Please ensure you have read the Code of Conduct before submitting your changes.

type: checkboxes
attributes:
label: Conduct
description: You need to have Read and Accepted the Code of Conduct
options:
- label: Did you Read the Code of Conduct?
required: true
- label: Will you follow the Code of Conduct?
required: true

type: input
id: title
attributes:
label: What are you publishing?
description: Provide a brief description of the changes you are publishing.
placeholder: Publishing the new feature for user registration
validations:
required: true

type: markdown
attributes:
value: |
# Thank you for your Contribution

#14
#13  👈 (View in Graphite)
main

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


V3: New Market overview + Invitation page

Proposed Changes

Change background of app to white
New market -> invitation design

Implementation

So... I'll merge this! :)


How can I achieve protected routes based on any value?

Hi, I'm trying to achieve something like this:
When user changes to a page if that page has the below value:
export const GUARD = "any_value";

I want to search in user's permissions. Is it possible to create something like this?

Hey @candogusyilmaz, I usually define protected routes in one place inside of a wrapper component to handle the redirections based on auth state.
Here's an example from Render template:
// src/config/redirects.tsx

import { Navigate, useLocation } from 'react-router-dom'

import { useAuth } from '@/context'
import { Path } from '@/router'

const PRIVATE: Path[] = ['/logout']
const PUBLIC: Path[] = ['/login']

export const Redirects = ({ children }: { children: JSX.Element }) => {
  const auth = useAuth()
  const location = useLocation()

  const authedOnPublicPath = auth.token && PUBLIC.includes(location.pathname as Path)
  const unAuthedOnPrivatePath = !auth.token && PRIVATE.includes(location.pathname as Path)

  if (authedOnPublicPath) return <Navigate to="/" replace />
  if (unAuthedOnPrivatePath) return <Navigate to="/login" replace />

  return children
}

The <Redirects> component can be used at src/pages/_app.tsx to wrap the <Outlet> component:
// src/pages/_app.tsx

import { Outlet } from 'react-router-dom'

import { Redirects } from '@/config'

export default function App() {
  return (

        ...

        <Redirects>
          <Outlet />
        </Redirects>

  )
}

If you want to move the definition of the protected routes to each file, I'm thinking it can be as following:

Exporting a capitalized variable, for example export const Private = true or export const Role = 'admin'
Adding a custom integration to pass this variable to each route meta/handle function
You'll likely need something similar to <Redirects> to redirect based on the route meta/handle and the auth state

Here are some related issue #28, #103
Hope that helps!


compiler issue after updating from master branch

Describe the bug
After merging our android sdk toolkit project with base master branch, we are getting compiler errors on ids
Expected Behaviour
compiler issues must not be there
Steps to Reproduce the Problem

take update from master branch
click run app in android studio
compiler error occurs

you don't have a proper version of the repository
See the original repository : https://github.com/voxeet/android-sdk-toolkit/blob/master/toolkit/src/main/java/com/voxeet/toolkit/controllers/ReplayMessageToolkitController.java#L169
no trace of the issue you're having here. Make sure to update using the proper git command

for information, update made 12 days ago


Panic with OffsetFetchRequest

I'm using franz-go in some unit tests to watch produced and committed offsets and I'm occasionally seeing the same panic in our CI environment in tests (it occurs maybe 10% of the time or so):
panic: interface conversion: kmsg.Request is *kmsg.OffsetFetchRequest, not *kgo.pinReq [recovered]
	panic: interface conversion: kmsg.Request is *kmsg.OffsetFetchRequest, not *kgo.pinReq
goroutine 34 [running]:
testing.tRunner.func1.2({0x13619e0, 0xc0000fe480})
	/usr/local/go/src/testing/testing.go:1396 +0x372
testing.tRunner.func1()
	/usr/local/go/src/testing/testing.go:1399 +0x5f0
panic({0x13619e0, 0xc0000fe480})
	/usr/local/go/src/runtime/panic.go:890 +0x262
github.com/twmb/franz-go/pkg/kgo.(*Client).handleShardedReq.func2({0x0, {0x1666848, 0xc0000a0540}, {0x0, 0x0}})
	/go/pkg/mod/github.com/twmb/franz-go@v1.11.2/pkg/kgo/client.go:1804 +0xd05
github.com/twmb/franz-go/pkg/kgo.(*Client).handleShardedReq(0xc0005d2000, {0x1660918?, 0xc000036380}, {0x1666848?, 0xc0000a0540})
	/go/pkg/mod/github.com/twmb/franz-go@v1.11.2/pkg/kgo/client.go:1859 +0xed1
github.com/twmb/franz-go/pkg/kgo.(*Client).shardedRequest(0xc0005d2000, {0x1660950, 0xc00013a008}, {0x1666848, 0xc0000a0540})
	/go/pkg/mod/github.com/twmb/franz-go@v1.11.2/pkg/kgo/client.go:994 +0x811
github.com/twmb/franz-go/pkg/kgo.(*Client).Request(0xc0000add90?, {0x1660950, 0xc00013a008}, {0x1666848, 0xc0000a0540})
	/go/pkg/mod/github.com/twmb/franz-go@v1.11.2/pkg/kgo/client.go:778 +0x71
github.com/twmb/franz-go/pkg/kmsg.(*OffsetFetchRequest).RequestWith(...)
	/go/pkg/mod/github.com/twmb/franz-go/pkg/kmsg@v1.2.0/generated.go:10894

The code that runs the RequestWith call looks like this:
var resp *kmsg.OffsetFetchResponse
for {
  req := kmsg.NewOffsetFetchRequest()
  req.Group = c.ConsumerGroup
  c.mu.Lock()
  for tp := range c.offsets {
    req.Topics = append(req.Topics, kmsg.OffsetFetchRequestTopic{
      Topic:      tp.Topic,
      Partitions: []int32{tp.Partition},
    })
  }
  c.mu.Unlock()

  var err error
  resp, err = req.RequestWith(context.Background(), client) // <--- THIS LINE LEADS TO THE PANIC
  var ke *kerr.Error
  if err != nil && errors.As(err, &ke) {
    if ke.Code == 15 {
      time.Sleep(100 * time.Millisecond)
      continue
    }
  }
  require.Nil(tb, err)
  break
}

I'm on the latest version (1.11.2) on Go 1.19.  I'm being careful to not reuse the request object between tries.  Is this a bug or just user error?
Something that might be significant is that the unit tests use a single-broker kafka deployment.

This is a bug -- I'll patch asap. I remember testing exactly that scenario though so I'm not sure where I undid my original fix ... I'll also see how easy this is to wire into an integration test to avoid a repeat in the future.

I've recreated this, it can occur when fetching offsets for a group that doesn't exist, I see the fix.

Ok thanks a ton.  What you described matches exactly the circumstances that can occur in the test environment.

Should be fixed in v1.11.3


Align with requirements for neuro workflows

Please plan for alignment with requirements for annotators related to the neuro workflows here

closing, as we'll make more specific issues


bump docker to 19.03.13 (linux+windows) and docker-compose to 1.27.4 (linux only)

v4.5.0 of the elastic stack was released with docker 19.03.12 and docker-compose 1.25.1 - the upcoming release should use at least the same version.
I've bumped them a little higher that what was in v4.5.0, but only to versions I consider relatively low risk.
Windows appears to install whatever version of compose is in the package manager - I'll take a look at what it is in the build output

Windows install of docker-compose happens here: https://github.com/buildkite/elastic-ci-stack-for-aws/blob/2ad801ed582e40c7a04126a23d874f0de10d5fd4/packer/windows/scripts/install-docker.ps1#L15


Load new content with script,  not click event

Hi,
i try manually load content:
<script> $("#content").trigger("click"); </script>
but without result.
How can i manually load new content?
Thanks in advance!

You can use pjax.loadUrl(url).


Add Protector for all of the chaos to prevent unintentional injection

What problem does this PR solve?
Fix #179
What is changed and how does it work?
See #345
Check List
Tests

Unit test
Manual test (add detailed scripts or steps below)

Code changes

Has Go code change

Related changes
Does this PR introduce a user-facing change?:

NONE

/test

@cwen0 Help me to take a look at

Codecov Report

Merging #363 into master will not change coverage by %.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #363   +/-   ##
=======================================
  Coverage   56.72%   56.72%
=======================================
  Files          61       61
  Lines        4250     4250
=======================================
  Hits         2411     2411
  Misses       1641     1641
  Partials      198      198

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 ba9fb0b...dfeb65f. Read the comment docs.

@AngleNet Is there any progress on this pr?

@cwen0 Sorry for the late reply. I think it's almost there. However, I do not have time working on it lately. Is there kind guys to take over this?

I can take this after June 1th ( after TiDB 4.0 release 😂 )

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


Cannot install imagededup==0.0.1, .. imagededup==0.1.0 because these package versions have conflicting dependencies.

Not sure if this is a recent issue, or related to my using an M1 Mac. Nevertheless, the tail of a very long traceback is below:
ERROR: Cannot install imagededup==0.0.1, imagededup==0.0.2, imagededup==0.0.3, imagededup==0.0.4 and imagededup==0.1.0 because these package versions have conflicting dependencies.

The conflict is caused by:
    imagededup 0.1.0 depends on tensorflow==2.0.0
    imagededup 0.0.4 depends on tensorflow==2.0.0
    imagededup 0.0.3 depends on tensorflow==1.13.1
    imagededup 0.0.2 depends on tensorflow==1.13.1
    imagededup 0.0.1 depends on tensorflow==1.13.1

To fix this you could try to:
1. loosen the range of package versions you've specified
2. remove package versions to allow pip attempt to solve the dependency conflict

I had this a while ago. I think I managed to fix it by uninstalling all the packages imagededup depends on and letting pip automatically re-install them.

I have the same exactly problem. Please someone could post the solution here? Thank you in advance.

I had the same problem but it would be so easy to solve. First of install the requirements in the requirements.txt file which is located in the source code of imagededup (https://github.com/idealo/imagededup) by "pip install -r requirements.txt" and then every thing would be ok. Hope it works

On Ubuntu there are no conflicting depedencies among package versions and it works all fine

on apple m1 I got it working by

installing tensorflow metal (see)
removing tensorflow line from requirements.txt and setup.py
update COMPILE_LINK_ARGS to COMPILE_LINK_ARGS = ['-O3', '-mcpu=apple-m1', '-mtune=native'] in setup.py

Hi, new release of imagededup is out that doesn't depend upon tensorflow anymore. That should address installation issues.


Assertion '(c & LIT_UTF8_3_BYTE_MASK) == LIT_UTF8_3_BYTE_MARKER' failed in lit_read_code_unit_from_utf8

Jerry version:
Checked revision: bdcd2d8
Build command: ./tools/build.py --clean --debug

OS:
Ubuntu 17.10

Test case:
Date.parse("2015-01-01T01߄'a': 1}:01F");

Download test case
Backtrace:
ICE: Assertion '(c & LIT_UTF8_3_BYTE_MASK) == LIT_UTF8_3_BYTE_MARKER' failed at jerryscript/jerry-core/lit/lit-strings.c(lit_read_code_unit_from_utf8):425.
Error: ERR_FAILED_INTERNAL_ASSERTION

Program received signal SIGABRT, Aborted.
0x00000000005ffda8 in raise ()
(gdb) bt
#0  0x00000000005ffda8 in raise ()
#1  0x00000000005fff5d in abort ()
#2  0x00000000005956c1 in jerry_port_fatal (code=ERR_FAILED_INTERNAL_ASSERTION)
    at jerryscript/jerry-port/default/default-fatal.c:71
#3  jerry_fatal (code=code@entry=ERR_FAILED_INTERNAL_ASSERTION) at jerryscript/jerry-core/jrt/jrt-fatals.c:58
#4  0x0000000000595db2 in jerry_assert_fail (assertion=assertion@entry=0x691be8 "(c & LIT_UTF8_3_BYTE_MASK) == LIT_UTF8_3_BYTE_MARKER",
    file=file@entry=0x67e9e0 "jerryscript/jerry-core/lit/lit-strings.c",
    function=function@entry=0x693770 <__func__.2487> "lit_read_code_unit_from_utf8", line=line@entry=425)
    at jerryscript/jerry-core/jrt/jrt-fatals.c:82
#5  0x0000000000596e9f in lit_read_code_unit_from_utf8 (buf_p=<optimized out>, code_point=<optimized out>)
    at jerryscript/jerry-core/lit/lit-strings.c:425
#6  0x00000000005aede8 in lit_utf8_read_next (buf_p=0x7fffffffda08) at jerryscript/jerry-core/lit/lit-strings.c:467
#7  ecma_date_parse_date_chars (str_p=0x7fffffffda08, str_end_p=0x9c5f9a <jerry_global_heap+506> "", num_of_chars=1)
    at jerryscript/jerry-core/ecma/builtin-objects/ecma-builtin-date.c:60
#8  0x00000000005b523f in ecma_builtin_date_parse.isra.1.lto_priv.312 (arg=<optimized out>)
    at jerryscript/jerry-core/ecma/builtin-objects/ecma-builtin-date.c:269
#9  0x0000000000457051 in ecma_builtin_date_dispatch_routine (this_arg_value=219, arguments_number=1, arguments_list=0x7fffffffdc2c,
    builtin_routine_id=<optimized out>) at jerryscript/jerry-core/ecma/builtin-objects/ecma-builtin-date.inc.h:33
#10 ecma_builtin_dispatch_routine (builtin_object_id=<optimized out>, builtin_routine_id=<optimized out>,
    this_arg_value=this_arg_value@entry=219, arguments_list=arguments_list@entry=0x7fffffffdc2c, arguments_number=arguments_number@entry=1)
    at jerryscript/jerry-core/ecma/builtin-objects/ecma-builtins.inc.h:153
#11 0x000000000045a95e in ecma_builtin_dispatch_call (obj_p=0x9c5e88 <jerry_global_heap+232>, this_arg_value=219,
    arguments_list_p=0x7fffffffdc2c, arguments_list_len=1)
    at jerryscript/jerry-core/ecma/builtin-objects/ecma-builtins.c:844
#12 0x00000000004ac44d in ecma_op_function_call (func_obj_p=0x9c5e88 <jerry_global_heap+232>, this_arg_value=<optimized out>,
    arguments_list_p=0x7fffffffdc2c, arguments_list_len=1)
    at jerryscript/jerry-core/ecma/operations/ecma-function-object.c:343
#13 0x00000000005ca5ab in opfunc_call.isra.1 () at jerryscript/jerry-core/vm/vm.c:424
#14 vm_execute (frame_ctx_p=frame_ctx_p@entry=0x7fffffffdc40, arg_p=<optimized out>, arg_list_len=<optimized out>)
    at jerryscript/jerry-core/vm/vm.c:2803
#15 0x00000000005dc396 in vm_run (bytecode_header_p=<optimized out>, this_binding_value=<optimized out>, lex_env_p=<optimized out>,
    is_eval_code=<optimized out>, arg_list_p=<optimized out>, arg_list_len=<optimized out>)
    at jerryscript/jerry-core/vm/vm.c:2883
#16 0x000000000040b5eb in vm_run_global (bytecode_p=<optimized out>) at jerryscript/jerry-core/vm/vm.c:231
#17 jerry_run (func_val=203) at jerryscript/jerry-core/api/jerry.c:558
#18 main (argc=<optimized out>, argv=<optimized out>) at jerryscript/jerry-main/main-unix.c:758

Found by Fuzzinator with grammarinator.

I think this issue can be closed now.


Circles ngOnChanges do not handles Clickable

Circles ngOnChanges do not handles Clickable

The solution is simple:
AgmCircle._mapOptions = [
    'fillColor', 'fillOpacity', 'strokeColor', 'strokeOpacity', 'strokePosition', 'strokeWeight',
    'visible', 'zIndex','clickable'

Pull request Ready #1079

https://github.com/SebastianM/angular-google-maps/pull/1079 is merged, so closing this! Thanks for the contribution.

 doesn't work with  only  i just tried it with latest code.


JSON To Redshift schema

Can I use schema guru to convert JSON schema to Redshift. Is that supported?

Technically yes, but that functionality has really migrated to the Iglu project, specifically:
https://github.com/snowplow/iglu/tree/master/0-common/igluctl


neil adds trailing whitespace

After neil add build the added entry in deps.edn contains trailing whitespace.

Thanks, will take a look soon, if you don't beat me to it.

Should be fixed.

Thanks :)


Slime hair isn't transculent

Description

Title
Reproduction

Make a slime
Add hair
Oh no my hair

Screenshots

uhhhh


Broken URL: https://aka.ms/copilotchat/memorypipeline/latest

Describe the bug
https://aka.ms/copilotchat/memorypipeline/latest points to https://github.com/microsoft/chat-copilot/releases/download/0.4/memorypipeline-0.4.0.zip which doesn't exist.
To Reproduce
Steps to reproduce the behavior:

Go to https://learn.microsoft.com/en-us/semantic-kernel/chat-copilot/deploy-to-azure
Click on 'Deploy to Azure'
See value of Memory Pipeline Package Uri default to https://aka.ms/copilotchat/memorypipeline/latest
Deploying this will cause an error because the URL is broken

Expected behavior
Successful IAC deployment
Screenshot

@raffertyuy Thanks for reporting this.
The problem was due to a change in process on our side.
This has been fixed now.
Cheers.

Thanks for resolving quickly!


Poll method for write operation in Stream

I've been working with the Stream structure and noticed that there's currently a poll method for reading operations only. While this works for most cases, in some specific network situations, I need to check if a socket is ready for a write operation. This is especially crucial when dealing with scenarios where write buffers might be full, or when there's a need to ensure non-blocking writes.
read_timeout and write_timeout options. These would allow users to define timeout durations for reading from and writing to websockets, like this in Client.handshake

Hard to test this, when you get the chance, can you checkout master?
There's a new write_timeout_ms which should error.Timeout on a timeout. For handshake, the timeout_ms that already existed for reads is being re-used. Hence, for handshake timeout_ms now represents the read+write timeout.

@yarreg I'm considering an alternative approach, which is to replace os.poll with os.setsockopt for better compatibility with Windows.
There's a branch, nopoll with the changes.
Essentially, it would be up to the application to call client.stream.writeTimeout(MILLISECONDS) and client.stream.writeTimeout(0) to remove any previous set timeout.
Do you think that would work for you? I'd really like to have a single codepath that works for all platforms.

Sorry for delay. Yes it is suitable for me.


"Smallest Common Multiple" troubles

Challenge Name

Smallest Common Multiple
Issue Description

I don't know why my code passed the first four test, but not the last one --smallestCommons([23, 18]).
Due to "检测完毕" in console, it should have tested all numbers from 23 to 78681840. But if it tested number 6056820, it should echo "检测6056820" in console. I'm really confused.
Browser Information

win7, chrome

Browser Name, Version:
Operating System:
Mobile, Desktop, or Tablet:

Your Code

function smallestCommons(arr) {
  function rangeOf(arr1){
    var low,high;
    if(arr1[0]<arr[1]){
      low = arr1[0];
      high = arr1[1];
    }else{
      low = arr1[1];
      high = arr1[0];
    }
    var result = [];
    for(var i = low;i<=high;i++){
      result.push(i);
    }
    return result;
  }
  function productOfArr(arr){
    var result = 1;
    for(var i = 0;i<arr.length;i++){
      result *= arr[i];
    }
    return result;
  }
  function isMultipleOfArr(number,array){
    for(var i = 0;i<array.length;i++){
      if(number%array[i] !==0){
        return false;
      }
    }
    return true;
  }
  var betArr = rangeOf(arr);
  var max = productOfArr(betArr);
  var min = Math.max(arr[0],arr[1]);
  console.log("betArr: ");
  console.log(betArr);
  console.log("min: "+min+" max: "+max);

  for(var i = min;i<=max;i++){
    if(i == 6056820){
      console.log("检测6056820");
      console.log(isMultipleOfArr(i,betArr));
    }
    if(isMultipleOfArr(i,betArr)){
      console.log("arr: ");
      console.log(arr);
      console.log("i: "+i);
      return i;
    }
  }
  console.log("检测完毕");
//   return max;
}

Screenshot

@stskyblade Can you please help us with that challenge link? Could be a Unicode issue.

Please clarify with information as requested. Some screenshots might help as well. Closing for now, we can re-open this when you are ready.

I had issue with the last test.
Later discovered that it is due to infinite loop protection enabled in browser.
Add // noprotect on the first line of your code to pass the test.

@Jez55 Thank you so much. I knew my code was right. thank you so much


docs(README) Point all docs links to azure.github.io/brigade

Point all README.md documentation links to the site https://azure.github.io/brigade.
Also fix markup in the first link. This was initially the main point, but I went a bit overboard with this: please let me know if I did.
This is related to #510

Ah, I just noticed the formatting of docs] and was about to do the same.
Good to have the other links point to the same site too for continuity IMO

Thanks! And I must say I like Brigade so far: it's been a pleasure working with it.


cache results of GetInboxRemote CORE-4186

So this patch is very large because I refactored inboxsource.go a bunch (which spilled over into chat_local.go and gregor.go). I've outlined my strategy below:
1.) We store the server result from GetInboxRemote on the disk. Unboxing of MaxMessages uses the bodies cache, so we don't need to store them twice.
2.) In order to know we can serve a given query+pagination request for reading the inbox, we track which queries we've seen before. IFF we have seen the query+pagination before, we will serve results from the cache.
3.) inbox.go implements running query conditions and performing pagination in a way identical to the server. It also handles finalize mechanics properly.
4.) Add way more logging, and label it better so chat activity is more easily pulled out of the logs with grep.
5.) Add the Syncer type, which now runs the show on alerting the frontend about changes in the validity of both the inbox and bodies cache. This will be triggered either from traditional reasons (disconnect, bad Gregor new message), but as well if the inbox version mismatches the stored copy. On reconnect, we load the inbox version and compare to the disk.
6.) Trigger Syncer notifications if we detect an inbox version mismatch on any Gregor message from HybridInboxSource.
7.) Put InboxSource on the global context, so it can be easily found and configured from the environment.
8.) Add a bunch of tests for inbox.go, and use HybridInboxSource in all the rest of the chatand service package tests.
Anyway, apologies for the length of this, but I wasn't sure how else to get this up here.

@maxtaco did you want to take a look? If not I am going to merge tomorrow afternoon when I get in, I'd for this to be in use for awhile before we do the alpha release.


Warn on MaximumReceiveMessageSize

Fixes #27013

Let's hold off on review. We're discussing this on the PU issue. We might do something different ... an INCLUDE used in the Blazor doc and the SignalR doc, for example. We might do this, but we might only do it in the SignalR doc.

Ok ... well ... the PU issue was closed right in the middle of the discussion on this.
Let's go with Rick's Gambit™ ... i.e. ... Call for review! 😆
Brennan, which way would you like to go?

Just note this on the Blazor-side, as this PR does.
Create a duplicate NOTE for the SignalR config topic ... I'll add it to this PR.
Do something else.

I can place it in the Description in the table at ...

Sure, similar to how ApplicationMaxBufferSize in a later table mentions memory usage can increase if the setting is changed.

@BrennanConroy ... Added a remark to the table. How's it look?

I will ... and btw ... I've talked myself out of using explicit byte conventions, as you recommended, on the issue that I opened ...
https://github.com/dotnet/AspNetCore.Docs/issues/27056
I wish MS/.NET foundation would adopt the standard. What I can do is place a remark on it in the Blazor Fundamentals topic and just use "KB"/"MB"/"GB" everywhere.

@BrennanConroy ... Are you OOF? No rush ... just checking.

@BrennanConroy ... Ok ... ready for another 👁.


"Running the contributed command:'markdown.showPreviewToSide' failed." on file open from explorer

VSCode Version: 1.12.2
OS Version: Windows 10 Pro

Steps to Reproduce:

"Open Folder ..." containing an .md file
Open .md file from explorer
Preview CTRL-K v
I get "Running the contributed command:'markdown.showPreviewToSide' failed."

If I drag and drop the same file from windows explorer into vscode, the preview works.

Isn't this the same as https://github.com/Microsoft/vscode/issues/26464 ?

No, the .md file is located on disk (c:). The other issue talks about network files.

Can you please run the Toggle Developer Tools command and see if the error printed in the console provides any more context?

@mjbvz I've uploaded the console output.
console.txt

@popra Can you please give the latest insiders builds a try to see if the issue has been fixed: https://code.visualstudio.com/insiders


A transaction through offline token stucks in waiting for receiver status while the receiver is offline

Description:
Any transaction through offline token stucks in waiting for receiver status while the receiver is offline. The transaction is started and changes status to "in progress" when the receiver becomes online.
Pre-conditions:
There are two running Wallets A and B
Steps to reproduce:

In wallet A open receiver window.
In Transaction info section switch to max privacy
Press on Copy and close button in Offline token section
Paste Offline token in the sender screen from Wallet B
Enter some amount (for example, 45)
Push to send and confirm it in the popup

Expected: Transaction is started and after few seconds changes status from "waiting for receiver" to "in progress".
Actual: The transaction stucks in waiting for receiver status.
Attachments:

beam_ui_20_08_19_11_31_35.log
Tx_id: 4ec57a10ca1548b99f43ddb6bb5d4a5e

checked


fix news wall on appinventor.mit.edu home page

This was all working except it seems to require a new version of the (???) library and upgrading the library broke several things.See Jeff for details.

Fixed


Update fastcrypto in sui (including changing the serde of EdDSA & BLS sigantures)

Includes a few non breaking changes & the new binary serializations of EdDSA and BLS signatures.
BLS/EdDSA signatures that were stored using the old serialization might not be deserialized successfully after this change.
See https://github.com/MystenLabs/fastcrypto/issues/401 for why we use insecure_default().

landing to reduce my PR complexity


feat: cli tool register install

Screenshots

minikube not installed

minikube installed

minikube update available

Related issues
Fixes https://github.com/containers/podman-desktop-extension-minikube/issues/149
Testing

[x] unit tests has been provided

go to CLITool
assert minikube cli tool card is visible
uninstall if already installed
assert no minikube provider in the resource page
install through cli tool page minikube
assert minikube provider in resource page

pnpm format:fix ?

pnpm format:fix ?

@gastoner My bad! Missed it :)

Also but maybe not related to this PR, I found this strange that the tooltip are different on the Update button on the left and the Update link on the rigth

@jeffmaury The minikube cannot change the UI, therefore I would argue the problem is on the podman-desktop side 🤔 .

@gastoner I updated the code, the problem was I wasn't checking for update after the install, only in the activate. Should be fixed now, thanks for the catch 👍

@gastoner the issue you encountered is related to https://github.com/containers/podman-desktop/pull/9813


:on-create is not working

Following the Example 12 from the Doc, the :on-create is not working. @tag works, but still push new value to the options even return false.

Are you sure you are using the latest 2.4.2 version @jackywongboy ?


The mouse should show up in top-down view of Panda3D visualizer

Can't remember what this issue means. Maybe it means we should not force mouse to disappear with Panda3d renderer?


Data to EVCC

I flashed the precompiled wifix.bin. All is working but i only have one topic in mqtt with all its data in it.
I want to send the data to evcc.
Anyone has an example how he sends his data to evcc?

Shine X

Which Inverter are you using?

MOD-4000TLXH

Thx

EVCC need to parse the JSON. I needed to do it in iobroker by script for example.

Yes I get to work with this configuration in my evcc.yaml:

name: solar
type: custom
power:
source: http
uri: http://<IP_ADDRESS>/status
method: GET # default HTTP method
headers:
- content-type: application/json
jq: .OutputPower
timeout: 5s

@nacho26 If you are using the latest master you can query individual values directly: http://<IP_ADDRESS>/value/OutputPower
Unfortunately no release binary yet.


Move previous Golang organizer

https://golang-companies-organizer.readytotouch.com/ukraine

Pages:

https://readytotouch.com/organizers/golang/companies
https://readytotouch.com/organizers/golang/companies/ukraine


Previous position missing from jumplist when navigating by symbol

Steps

Open a file type known by VSCode with symbols such as methods (e.g. .ts)
G
gg
"Go To Symbol in File" (e.g. selecting a method a couple of lines down)
ctrl+o

Actual

Cursor is at end of file

Expected

Cursor is at beginning of file (where it was before jump with "Go To Symbol in File")

Workaround

Use VSCode provided ctrl--

I guess the issue here is that vscode-neovim needs to be notified that a jump was initiated by other means. As a counter example, gd and ctrl-o works fine.

Easy workaround - execute go to symbol command from neovim, prepending by manual mark creation (normal! m')

Clever idea! For reference, how exactly would I accomplish this? Though I assume it won't catch other jumps e.g. from breadcrumbs, outline ,or by mouse. I vaguely remember folks at vscode-vim discussing this and hoping for an event emitted by VSCode.

keybindings.json
// Delegate <cmd+r> to vim
{
  "key": "cmd+r",
  "command": "vscode-neovim.send",
  "args": "<d-r>",
  "when": "editorTextFocus"
}

.vimrc
# Handle cmd+r (received as <d-r>), create a mark manually then jump to symbol
nnoremap <silent> <d-r> m':<c-u>call VSCodeNotify('workbench.action.gotoSymbol')<cr>

Using vscode jumplist now

Using vscode jumplist now


Troubleshoot Applications - POD Restart due to OOM

Hi Team,
I am facing an issue in one of the pod where the pod is being killed when the memory limit reached maximum and no heap dump is being generated even though the Heap dump parameters are configured.
I understand that the pod is getting killed by kubernetes instead of its own. Can you please suggest a way to extract heap dump for such pods. So that it will be helpful for us to debug the issue.
Thanks in advance.

Hmm I was wondering if the content is better addressed in the slack channel kubernetes-users or stackoverflow since this involves debugging applications instead of documentation related matter.

ok. Closing the issue.

/kind support

@kkrhdd if you can identify a gap in the documentation and can describe that, please do feel welcome to file an issue about that gap.


CFFL

CFFL

Linter check found the following problems:
The following problems have been found:
ERROR: /tmp/abs_bd4_kc1ent/clone/recipe/meta.yaml:16: pip_install_args: pip install should be run with --no-deps.
ERROR: /tmp/abs_bd4_kc1ent/clone/recipe/meta.yaml:21: missing_wheel: For pypi packages, wheel should be present in the host section
ERROR: /tmp/abs_bd4_kc1ent/clone/recipe/meta.yaml:30: missing_imports_or_run_test_py: Python packages require imports or a python test file in tests.
ERROR: /tmp/abs_bd4_kc1ent/clone/recipe/meta.yaml:37: missing_documentation: The recipe is missing a doc_url or doc_source_url
WARNING: /tmp/abs_bd4_kc1ent/clone/recipe/meta.yaml:37: missing_description: The recipe is missing a description
ERROR: /tmp/abs_bd4_kc1ent/clone/recipe/meta.yaml:37: missing_license_family: The recipe is missing the about/license_family key.
ERROR: /tmp/abs_bd4_kc1ent/clone/recipe/meta.yaml:37: http_url: http://www.sqlalchemy.org is not https
Errors were found


Raspberry PI4 - invalid capacity 0 on image filesystem

inspection-report-20200823_022916.tar.gz
I'm following this guide on how to setup microk8s on a raspberry pi 4. https://ubuntu.com/tutorials/how-to-kubernetes-cluster-on-raspberry-pi#4-installing-microk8s I've managed to get it up and running and can connect with kubectl, but when I try to install dns and dashboard those services don't come online. The containers are stuck in Pending state and the node status is tainted with Not Ready and giving this error:
Warning  InvalidDiskCapacity      7s     kubelet, ubuntu     invalid capacity 0 on image filesystem

Looking at similar threads on the issue seem to indicate zfs needs some special configuration, but that's not the case here for me, as it's running on an ext4 mounted microSD card.
I did try similar recommendations for the ZFS issue to set my snapshotter = "native" in  /var/snap/microk8s/current/args/containerd-template.toml but that had no effect, same error.
I'm not sure where to look next, being new to k8s. Any suggestions would be helpful. Thank you!

Hi @kensykora
Thank you for reporting this.
I just updated the docs [1] to point out the kernel command line parameters are distribution specific. We also have a pending PR [2] against microk8s inspect that could detect missing cgroups and will point to the docs.
[1] https://discuss.kubernetes.io/t/alternative-installs-macos-windows-10-multipass/11257
[2] https://github.com/ubuntu/microk8s/pull/1501

Wonderful, thank you @ktsakalozos!


[Bug Report]  Huge lag caused by CrazyEnchantments::Event: m.b.c.e.Helmets (PlayerMoveEvent)

Huge lag caused by CrazyEnchantments::Event: m.b.c.e.Helmets (PlayerMoveEvent)
Ver: 1.16.4
Pl Ver: CrazyEnchantments version 1.8-Dev-Build-v8.1-SNAPSHOT-Build#156
Timings: https://timings.aikar.co/?id=4fbbae59e41f4e55ab0cc8fca6fb39b9

I ended removing the Helmet Listener from the Source Code, since only contains one enchantment: Commander.


handle complicated declaration maps in tsconfig.json

having these 3 leads to sadness
declaration: true, sourceMap:true, declarationMap: true
there's just a vague error emitting code error with no context. we need better errors here
digging deeper seems to be this error: https://github.com/microsoft/TypeScript/issues/6307#issuecomment-168321011

        console.debug(emitResult)

was helpful. diagnostics property has more info that needs to be parsed


Bt classic auto pts setup for GAP part

This PR is used to add BR/EDR PTS support for GAP part to bt_tester.
Including following changes,

include: bluetooth: Add definition for HCI_Write_Current_IAC_LAP

Add definition and structure for write current IAC LAP command

Bluetooth: tester: Enable set_connectable for classic

Enable/disable connectable for classic when BTP set_connectable command is received.

Bluetooth: tester: Update set_discoverable for classic

Enable/disable discoverable for classic when BTP set_discoverable command is received.

Bluetooth: tester: Update start_discovery for classic

Start descovery for classic when BTP start_discovery command is received.

Bluetooth: tester: Update stop_discovery for classic

Stop discovery for classic when BTP set_discoverable command is received.

Bluetooth: tester: Add gap_connect_br for classic

Add a command gap_connect_br for classic to create br connection.
Add a structure btp_gap_connect_br_cmd for this command.

Bluetooth: tester: Get conn from address for classic

Get the conn handle from the peer address for classic when the following BTP command received, including disconnect, pair, unpair, passkey_entry, passkey_confirm, and conn_param_update.

Bluetooth: tester: Support l2cap BREDR

Support L2CAP BREDR for BTP listen and connect command.

Bluetooth: tester: Fix conn cannot be free issue

After the btp l2cap command connect called, the conn cannot be free when the connect broken.
The function bt_conn_unref should be called to free conn before the btp l2cap command connect done.

Bluetooth: tester: Add device name to ext inquiry rsp

Add device name to extended inquiry response for classic.

Bluetooth: tester: Correct peer addr for classic/spp.c

The peer address in BTP event is incorrect if the conn type is classic. Correct the peer address.

Bluetooth: tester: Support pincode request

Add pincode entry if the IOCAP is KeyboardDisplay or KeyboardOnly.

Bluetooth: Host: SSP: Fix bonding type issue

Currently, the bonding type of Authentication_Requirements parameter is always Dedicated Bonding if the device is pairing initiator.
But if the bonding is performed during connection setup or channel establishment as a precursor to accessing a service, the
bonding type should be General bonding.
Add a flag BT_CONN_BR_GENERAL_BONDING. Set the flag if the bonding is performed in the L2CAP_BR/RFCOMM channel establishment.
Set bonding type depends on the flag when receiving IO cap request.

Bluetooth: Host: smp: Add function to get bonding setting

Add a function bt_get_bondable to get the bonding setting.

Bluetooth: SSP: Support non-binding case

If the return value of function bt_get_bondable is false, clear the bonding flag when controller requiring Authentication_Requirements.

Bluetooth: SSP: No MITM if required level is less than L3

Regardless IO capabilities, clear MITM flag for pairing initiator if the required security level is less than BT_SECURITY_L3.

Bluetooth: tester: Add a GAP command pair_with_sec_level

In current implementation, the security level is a hard code BT_SECURITY_L2.
Add GAP command pair_with_sec_level to set the security level when starting pairing procedure.

Bluetooth: tester: Add a L2CAP command connect_with_sec_level

For case GAP/SEC/SEM/BV-09-C, the first L2CAP connect requires encryption with security level 2. And for secondary L2CAP connect, it requires encryption with security level 3.
For current implementation, the required security level is always 2. It cannot meet the requirement.
Add a L2CAP command connect_with_sec_level to set the security level when creating L2CAP connection.
In the command, pass the required security level to field required_sec_level of L2CAP channel.

Bluetooth: tester: Custom ad flags

If the ad flags is set by auto-pts client, overwrite the ad flags by using custom ad flags.

Bluetooth: BREDR: L2CAP: Recovery ident when conn is pending

The ident of L2CAP BR connection req will be cleared if function l2cap_br_conn_req_reply called to send L2CAP BR
connection rsp with result BT_L2CAP_BR_PENDING.
Then the invalid ident (it is zero) will be filled in the L2CAP BR connection rsp after the ACL connection is encrypted.
Recovery ident if the result of the connection rsp is BT_L2CAP_BR_PENDING.

Bluetooth: SSP: Support security level 4

Currently, error code -ENOTSUP will be returned if start security with security level 4.
For SC supported case, level 4 for ssp should be supported.
Remove the code limitation to support security level 4.

Bluetooth: SSP: Improve BR SC only mode

Actively disconnect the connection with error code BT_HCI_ERR_AUTH_FAIL when the notified link key type is not BT_LK_AUTH_COMBINATION_P256 in BR SC only mode.

Bluetooth: BR: improve br_sufficient_key_size

Currently, the minimum value of encryption key size is BT_HCI_ENCRYPTION_KEY_SIZE_MIN.
There is a Kconfig BT_SMP_MIN_ENC_KEY_SIZE. It is used to set the specific minimum encryption key size.
Use CONFIG_BT_SMP_MIN_ENC_KEY_SIZE as minimum encryption key size in br_sufficient_key_size.

The changes for auto-pts script is here

@lylezhu2012 not directly related to this PR, but I'm pointing it out since I didn't realize it earlier: we really shouldn't have both l2cap_br_interface.h and l2cap_br_internal.h those should probably be merged into one, just like we have a single l2cap_internal.h in the main host directory. We can discuss the possibility of factoring out e.g. protocol structs to a separate header, but for now I'd just follow the same design as LE L2CAP uses.

@jori-nordic ,
Branch of my PR is deleted now. It causes the PR is closed. And the repo used to create this PR cannot be used anymore.
Is there a way to change the source branch of my PR? Than I can reopen it. Or I can only recreate a PR.

I don't know about a way to do that. You can make a new PR :+1:


will you release the supernet training scheme?

what's your plan?
Thanks.

Hi,
We have released the supernet training scheme in this project. Thanks for your interest in our project!
Best,
Hao.


Safari isn't working

On both Safari desktop and iOS playback doesn't work.
Possibly something different about Safari rules for starting playback: Autoplay is not allowed by the browser autoplay rules
A click handler needs to correlate with the play interaction

https://developer.spotify.com/documentation/web-playback-sdk/reference#spotifyplayeractivateelement

Some browsers prevent autoplay of media by ensuring that all playback is triggered by synchronous event-paths originating from user interaction such as a click. In the autoplay disabled browser, to be able to keep the playing state during transfer from other applications to yours, this function needs to be called in advance. Otherwise it will be in pause state once it’s transferred.

Safari does prevent autoplay and is seemingly more aggro about it for security or power savings compared to Chrome.
It seems like the reference docs are wrong. You need to call activateElement in the synchronous event-path from a click, not before. Interesting that in Svelte it didn't work putting it in the playPause effect, but did work in its own effect.


[UI] Increase font weight of meshery welcome screen

Current Behavior
The "Please choose a provider to continue" font weight is currently 500
Desired Behavior
Proposing the increase of the font weight of the "Please choose a provider to continue" from 500 to 700
Implementation
below is the design we currently have and how it should look like

I'd like to take up this issue.

@Jagrati24 Thank you

@Jagrati24 how is this coming along? Can we offer any help?

@leecalcote Can I work on this? 👀
Ig I can finish it off with another issue #2448  as this is a very simple fix


Update to Angular 16

Migrating to 16 should be done and merged into dev first before tackling a couple of tasks that need to be completed.
AC:
Everything should look and behave unchanged in frontend after moving from 15 to 16

@bcb37 reviewing


Allow user to select icons to add to gallery on gallery#new

Also has tests to ensure has_gallery is set appropriately. (Unfortunately with just the has_many_and_belongs_to relation, saving without a title but with icon IDs ends up setting them to has_gallery true, despite not having a gallery for them to be part of. The workaround in this prevents that being an issue.)

Fixes #89, I think.

With no galleryless icons

With galleryless icons

(The width is fixed at 100%, as the default for formtables is 400px so this overrides it forcefully. Unfortunately, this means that if there's only one galleryless icon it might look weird on large screens. Screenshot below.)

This no longer passes as set_has_gallery skips the icons when they're set in new (as the gallery has no name and hence isn't valid) and then doesn't get triggered when the gallery is saved.
I think it might be worth making the GalleriesIcons model or whatever and having an after_create callback there – after_add on the relation is called even when the model isn't saved, due to how Rails handles these callbacks (ie weirdly).

Transactions seem to work well for this but the new test ensuring set_has_gallery only occurs when the gallery is valid fails (as that line has been removed here). I put a skip in for it but we should maybe revisit that later? (Transactions could be a better way of dealing with things when they have associations, though, to prevent this sort of problem occurring elsewhere.)

I've rebased this with the fact we now use set_has_gallery on a separate model, as well as Rails 4 strong params, and it seems to still pass tests.


Untranslated "played for" lines in user page

As you can see above not all play times are translated, I just saw that by accident I never go to the user site from the home menu, but I was looking for the "user page" theme from NXThemes, and it wasn't in the user page in the system settings, so that is how I found those missing lines by chance, so I wanted you to know. Have a great day, bye.

We've had problems with these from the beginning. Editing the lines corrupts the file so we won't fix this until we find a better way.

Oh, that's a bummer. But hardly anyone uses this page so it's not a biggie, thanks for the quick respond, bye.

Hallo again, I had a similar problem when translating my 3ds eShop, and here's what I did:
I took those bad lines from a different language that didn't have the "day/days" format and just "d.".
I found it in the "russian.msbt" file, so I translated all missing lines not just the "played for" lines in "myPage.msbt.szs".
And voila:

I used "MSBT Editor Reloaded v0.9.8" because "Kuriimu" displayed the lines incorrectly, and to compress it back to a Yaz0.szs file.
So how can I sent the files to you? I don't want any credit I just wanted to help my fallow translator, so your welcome:D

The easiest way would be to do a pull request here on GitHub,

I don't know how, I only started GitHub, to download files from here, just give me your email, and after you get my mail, just delete it from the comment, ok?

Sure. Send me an email at<EMAIL_ADDRESS>

I've just sent you "fix.rar" file, good day.


ci: fix docker image name

:tada: This PR is included in version 0.5.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Why did you use the relationship belongsTo in the model emailconfirmation for userId

Correct me if im wrong , maybe im not understanding correctly the use of relationship , but im
wondering why in this scenario its not more appropiate the  --- hasOne --- relationship because you only need to registered and validated just one user, and this will appear only once in the table, right?

What table/model doe you mean?

Hi sergey  the table is email_confirmation / EmailConfirmations.php

There is two use cases.
User can't change email
hasOne should be defined in the parent model (Users) and belongsTo should be defined in the child model EmailConfirmations. In this case a user has one email confirmation and that email confirmation belongs to one user.
User can change email:
hasMany should be defined in the parent model (Users) and belongsTo should be defined in the child model EmailConfirmations. In this case a user has many email confirmations and each email confirmation belongs to one user.

Hi sergery thanks, i really need to study and understand well the relationships uses, thanks a lot for your answer


CSS in custom renderer messes up markdown rendering

VS Code Version: 1.55.0-insiders
OS Version:  Darwin x64 19.6.0

Steps to Reproduce:

Use the yeoman generator to create a Notebook or use the Notebook here: https://github.com/tanhakabir/vscode-notebooks-demo at branch complete
Verify/Make the renderer render a list
Verify/Make the css of the list have a weird font
Run the Notebook in Extension Host
Create a markdown cell with a list
See that your css change for lists in the renderer was also applied to the list in the markdown

Output css bleeds to markdown ;(

The markdown output or the renderer output likely need to live inside a Shadow DOM. This is not specific to markdown either since an I believe an output renderer could set: body { background: red; } and effect the entire page

/verified


fix(spinner): add the refresh spinner to the select-multiple template

The multiple mode would not show a refresh spinner even with spinner-enabled set to true. Add the same spinner html from the single selection template to the multiple selection template.
Closes #1985

Thanks for the PR could you add some test for this?

@Jefiozie, I added tests.


Verify asp PKGBUILD against pkgbuild_sha256sum in BUILDINFO

BUILDINFO contains the hash of the PKGBULD which was use to build the package, compare this two the fetched PKGBUILD by asp and stop the build if they don't match up.

This one is fixed with #40!


SMTP server request

With current setup, you have no way to use Contact Form or forgot password link because there is no mail server being setup.
Actually, there are bunch of SMTP plugins so this could be delegated to some other service which I think is the best option which rewrites PHPMailer function

We haven't installed any SMTP clients because we're in favour is using a 3rd party service like Mandrill, Sendgrid, Mailgun etc.
But I do think we should still provide a simple client by default that's only for sending. msmtp and ssmtp both seem simple so I'll leave this issue open until I check them out.

Do you mind to explain briefly how do you integrate it? Do you using any plugin? How do you reset password from wp-admin?

All of those services are very similar and usually work in 2 ways: either via their API (raw HTTP calls or a PHP client library), or via SMTP (which you'd need a client like you mentioned).
To hook into WP mail there's various plugins. Here's an example of one: https://wordpress.org/plugins/wpmandrill/
This one uses curl in the background for their API so not SMTP needed at all.
In general you should always use an email service for sending external email. It's very hard and not really worth the trouble to setup your own. If it's just for "internal" emails like signups/resets for developers/admin then you don't need to worry.

but it doesn't solve the problem for reset password link

What doesn't? The Mandrill plugin would work for example.
But as I said I'm leaving this issue open and we'll add an SMPT client which will fix this problem as well. Then emails will work regardless of what solution people use.

I love the Mandrill plugin and service, but :+1: for for SMPT client just in case.

Can I vote against it? SMTP would feel more like an awkward dongle that will probably fall behind in maintenance. Reason I say this is it appears everyone uses an external service, so why bake it in?

@nathanielks SMTP should be opt-out and not opt-in. We should definitely recommend people use an external service but email not working out of the box by default is unacceptable.

Yeah, you need something. I think there was one instance where I added a user, not realizing there was no mail system installed, and the adding process timed out (it worked, the user was added, but it was a very odd experience).


tests for lib http

Before creating a pull request make sure that:

[ ] commit messages are meaningful and follow good commit message guidelines
[ ] README and other documentation has been updated / added (if needed)
[ ] tests have been updated / new tests has been added (if needed)

Please remove this line and everything above and fill the following sections:
JIRA link (if applicable)
Change description
Does this PR introduce a breaking change? (check one with "x")
[ ] Yes
[ ] No

would suggest keeping the file same consitency? errorConstants.spec.ts instead of error-const.spec.ts?


npm repo issues

In the version you get from npm

some of the files are marked as executable
there are some duplicate files ending in ~ (backups from some text editor I think)
there is a test.js file included which is not needed?

Fixed in 0.1.0. (By default, npm will publish everything in a directory except the node_modules folder, so that's probably how the ~ files snuck in.)


chore: release java-compute v1.1.1-alpha

This pull request was generated using releasetool.
05-17-2021 16:08 PDT

fix: fix fields serialization for cases when the field name start with a capital letter (#417)

Release is at https://github.com/googleapis/java-compute/releases/tag/v1.1.1


Add Dynamo 🇨🇦

Please review our Contribution Guidelines to ensure your changes abides by it.
Review your changes and complete the checklist below.

[x] Company name
[x] Company description
[x] Company location
[x] GitHub link
[x] How Elixir is used
[x] Sorted alphabetically

Thank you @bokay 👍


jQuery - GetProperty

Hello,
I found a bug with the jQuery Wrapper.
The method jQueryObject.GetProperty in the C# DLL returns always a string, this seems correct because in the documentation of jQuery, the return value of the "prop" method is a String (http://api.jquery.com/prop/)
But, it seems the documentation of jQuery is not up to date because if I get the "checked" property of a CheckboxElement, jQuery returns a Boolean.
Script.Alert(jQuery.Select("... a checkbox element...").GetProperty("checked").GetType().ToString());

displays
function Boolean() {
    [native code]
}

Thank you Nikhilk.

Fix included in <IP_ADDRESS>


[WIP] Mongod config and mongos config reading

adding the ability to use config files instead of pure command line args for more advanced mongodb functionality.

now needs testing for mongod config, and creation of new unit tests.

All existing unit tests have been built, and everything works. Next will be testing the new features but since they only add to existing without breaking anything I am happy to pull this in.


DOIs

It would be really great if we could have DOIs for the releases to make citing this repo easier.
This should not take too long to set up according to the documentation but needs to be done by the repo owner, as it involves setting up a repo-hook.

I haven't been able to get this thing to respond to our build webhooks. I've got the project added and enabled within Zenodo, but it hasn't carried out the build a few weeks after I tagged and released to PyPi/github.

Closing due to inactivity. Feel free to re-open!


More touchscreen issues

Hey again
this behaves much better, but I'm still having touchscreen issues
ISSUE 1
open your demo file and scroll down to "Numeric special"
touch RGB color pick "..." > dont change anything in the popup picker  > try to close it by touching outside. If it closes open the color picker again and try to close it ...
At some point (rather sooner than later) I get stuck in the opened popup picker and cant close it. If that happens , the only way to close it is to select a new color (or move the mouse)
ISSUE 2:
open "RGB" picker by touching the blue "..."
changeit to red
touch outside to close color picker
touch "RGBA" ".."
-> color picker opens with the correct color (orange) but if you click in the big square to pick a new orange tone the hue will jump to the hue of the previou(RGB) par (red)

Ahh yes, I was able to recreate this. the trouble comes when dragging the finger ever so slightly after tapping on a picker.
There's another error/bug I found associated with this behavior. I'll get this patched soon.
Thanks again.

thank you!


Consider switching from LightGraphs.jl to Graphs.jl

Randomly stumbled upon this project. The development of LightGraphs has stopped, but all its code has moved to Graphs.jl, so maybe you should consider switching to that one.

Thanks for making us aware of this! We'll look into it.


VS crashes when you try to fix an issue

Create a sarif  file with a result and a fix.
double click the result
right side, window, click preview or apply
vs will crash

sample_sarif.txt

temp.txt
sample_sarif.txt
rename sample_sarif.txt to sample_sarif.sarif and try again

temp.txt
sample_sarif.txt
rename sample_sarif.txt to sample_sarif.sarif and try again

This is because the sarif log result.fixes.replacements.deletedRegion doesn't have charOffset, the default value is -1. There is a check in fix.CanBeAppliedToFile only returns fix which replacement.Offset >=0

This is because the sarif log result.fixes.replacements.deletedRegion doesn't have charOffset, the default value is -1. There is a check in fix.CanBeAppliedToFile only returns fix which replacement.Offset >=0

Since this is related to an old version, let's try to publish the new version.


Vulnarability issue in package got-9.6.0.tgz which testrail-integration package  is using

Hi Team
Please update  got-9.6.0.tgz  to it's latest version in  testrail-integration package there is vulnerability issue for the package which testrail is using

Thanks
Suyog

Sure, I will update and publish new version

Probably it will take 1 month to publish new package


CFE reports an error as a crash when using part and import statements

// File main.dart
library test_lib;
import 'test.dart';
part "part_of_the_lib.dart";

main() {
  test();
}

// File test.dart
library test_lib;
part "part_of_the_lib.dart";

test() {
  foo();
}

// File part_of_the_lib.dart
part of test_lib;

bar() {}

foo() {
  bar();
}

Now run dart --preview-dart-2 main.dart
Output:
Crash when compiling file://.../test.dart,
at character offset null:
part_of_the_lib.dart:8:1: Internal problem: Couldn't find 'bar'.
bar() {}

Now run dart main.dart and there will be no issues.
Also no issue in analizer even with --preview-dart-2 option
Dart VM version: 2.0.0-dev.50.0 (Wed Apr 25 11:29:49 2018 +0200) on "windows_x64"

@leafpetersen
Is it a strong-mode/Dart 2 error to import two libraries with the same name? It's a warning in Dart 1, so consistency would suggest that it's an error. It's not a very important error, though.

Of course, it's an issue that dart --preview-dart-2 (that is, the common front end) says Crash when compiling ..., so that should be fixed no matter which treatment the situation gets thereafter.
We could say that (1) it's an error if a program contains two libraries with the same, non-empty name, or that (2) it's an error if a part has an ambiguous part of directive because multiple libraries in the program has the indicated name, or we could say that (3) it's an error if a part P has an ambiguous part of directive because multiple libraries in the program has the indicated name and they specify part U where U denotes P, and we could even (4) allow the latter.
(4) can be perfectly well-defined, but it conflicts with separate compilation, so we probably don't want that; (3) is possible (though confusing, because you could be looking at library L and part of L, and the latter would still not necessarily be a part of the former). You could say that (1) is overly strict, because there is simply nothing that goes wrong because of that type of library name conflict (which is already sort of implied by the fact that we accept having several libraries with the empty name).
So, to me, (2) seems to be a reasonable level of strictness: If you say part of L, there can't be two libraries named L in the current program.
But what does the spec say now? I remember that it used to be a name clash, and Gilad made an adjustment specifically to make it OK to have two or more libraries whose name is empty (to allow library directives to be omitted entirely). Here's the relevant update to dartLangSpec.tex (commit a0e4f281609de0e4c66676c299f630d570e9e276):
-It is a static warning to import two different libraries with the same name.
+It is a static warning to import two different libraries with the same name unless their name is the empty string.
...
-... It is a static warning to export two different libraries with the same name.
+... It is a static warning to export two different libraries with the same name unless their name is the empty string.

So that's the wording that makes it a warning (in Dart 2 presumably: error) for direct imports to have this type of name clash, and the update eliminates the conflict for the empty name, but it is not obvious to me that it would be a name clash to have two libraries with the same name in the same program, as long as no library imports/exports both of them directly.
This means that we currently have something which is not any of the levels (1)..(4) that I mentioned (and I really don't think it's a problem to import or, especially!, export two libraries with the same name), so I'd recommend that we adjust the specification to make it (2).

cc @vsmenon @sigmundch @bwilkerson @jmesserly
I'm interested in implications (if any) for tools.  Library names are mostly pointless from the user perspective in Dart, right?  So it doesn't seem terrible to import the same library names.  But it doesn't seem especially useful either.

I'm interested in implications (if any) for tools. Library names are mostly pointless from the user perspective in Dart, right?

DDC ignores library names. They're often not well maintained (e.g. still use an old file name or path). The style guide encourages using the part of syntax that accepts strings: https://www.dartlang.org/guides/language/effective-dart/usage#do-use-strings-in-part-of-directives
So in this case, you'd have to pick part of "main.dart" or part of "test.dart".  The old part of lib_name syntax is deprecated right?

@lrhn Any thoughts?
I'm somewhat disinclined to make it an error for two libraries with the same name to exist in a program, since it feels like causing breakage for no really good reason, and in ways that users can't necessarily control (e.g. they use two packages, both of which have an internal library named "internal_lib").
I would prefer to not allow the same files to be included in different libraries at all, but I seem to remember that we have code which uses this "feature".

My suggestion is to do what DDC does: Ignore library names.
Then allow library; with no name and even part; with no of name, and forget that library names ever existed.
I don't care if someone wants to include the same part in multiple libraries. It's textual inclusion, so you get what you ask for (separate statics, potentially completely different declarations because type names mean different things). I am worried about someone using parts as an API for abstracting ("import this part, declare the Foo and Bar types, and then you have a customized class for your types"), but if that's an issue, maybe we should just provide a better feature for doing that kind of modularity.
In practice, all imports are package imports. The case that duplicate library name checking was about was to avoid importing the same library via different paths. That just doesn't happen for imports (it happens for entry-points, but we should fix that instead).
If two distinct package names refer to libraries with the same name, you probably did it on purpose (e.g., to have two different versions of the same library at the same time).
Complaining about a conflict between self-appointed names in a global name-space is just not a good idea.

Sounds like you'd prefer to declare that there is no conflict, even when part of foo.bar; in a part P is resolved to multiple distinct libraries L1 .. Lk named foo.bar in the same program (just like part; would be happily matched up with multiple nameless libraries, as long as they all specify part _P_;).
I can see that it should be a non-problem for compilers to handle this, as long as modular compilation schemes do not generate code for a part, but it would be somewhat confusing if IntelliJ is showing a part file (which is embedded in several distinct semantic contexts in the same program) and it doesn't really know which typing environment to use for completion etc.

We are going for option (3) for now: make it an error to include the same file as a part in more than one library in the same program. https://github.com/dart-lang/sdk/issues/33225 tracks the necessary spec change.

Before the fix in https://dart-review.googlesource.com/c/sdk/+/56521 can be landed, we need duplicate parts cleaned out from the co19 test framework code: https://github.com/dart-lang/co19/issues/149


Update .NET Core SDK

Update to .NET Core SDK 2.1.400.

Codecov Report

Merging #41 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #41   +/-   ##
=======================================
  Coverage   99.86%   99.86%
=======================================
  Files           8        8
  Lines         738      738
  Branches       99       99
=======================================
  Hits          737      737
  Partials        1        1

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 e03ccb8...7a08a1a. Read the comment docs.