use a vec for ExprKind in expr.rs

this might sound weird but currently in
/// An expression.
///
/// Reference: <https://docs.soliditylang.org/en/latest/grammar.html#a4.SolidityParser.expression>
#[derive(Debug)]
pub struct Expr<'ast> {
    pub span: Span,
    pub kind: ExprKind<'ast>,
}

we currently have a 7+ padding in the Expr struct, now if we were to vectorize ExprKind we can eliminate the passing and optimize memory and this is not all, the way a vector is arranged in memory is continuous i.e. data is side by side which allows for optimum cache ability i.e. more cache hit since the compiler now has a better predictability rate, take for example this:
            match kind {
                ExprKind::Array(exprs) => {
                    for expr in exprs.iter #_mut() {
                        self.visit_expr #_mut(expr)?;
                    }
                }
                ExprKind::Assign(lhs, _op, rhs) => {
                    self.visit_expr #_mut(lhs)?;
                    self.visit_expr #_mut(rhs)?;
                }
             .....................................
          }

of course this uses a match, so it should be exhaustive already but in runtime this might matter less in runtime depending on how complex the computation is, now looking at it can also be a slice

we currently have a 7+ padding in the Expr struct, now if we were to vectorize ExprKind we can eliminate the passing

this is not too bad, and most of it is likely because the variants of ExprKind have unequal sizes

the way a vector is arranged in memory is continuous i.e. data is side by side which allows for optimum cache ability i.e. more cache hit

structs are also contigous in memory
I don't see what making ExprKind a Vec gives us at all in this case, so I am closing this issue. appreciate the thought regardless:)

we currently have a 7+ padding in the Expr struct, now if we were to vectorize ExprKind we can eliminate the passing

this is not too bad, and most of it is likely because the variants of ExprKind have unequal sizes

the way a vector is arranged in memory is continuous i.e. data is side by side which allows for optimum cache ability i.e. more cache hit

structs are also contigous in memory
I don't see what making ExprKind a Vec gives us at all in this case, so I am closing this issue. appreciate the thought regardless:)

I disagree :)

@onbjerg it's not about the struct it's about the ds of the member, think of this way:
Currently with the enum approach, ExprKind variants have different sizes leading to padding between them. When we access these in sequence, like in the visitor pattern I showed, we're jumping around memory(might be cache) with padding gaps.
With a vectorized/slice approach, the actual ExprKind data would be stored contiguously without padding between them. This means:
The cpu has a better chance of predicting where the values will be, this is possible because we should have a better cache utilization
The struct being contiguous in memory isn't the focus - it's about how we arrange the actual ExprKind data for optimal access patterns.
I mean a struct that has a hashmap as one of it's member can't be called continuous the only thing it will contain in that particular instance is a pointer to the hashmap

theres a simpler solution: box the big variant to reduce padding. using a vec here makes no sense

theres a simpler solution: box the big variant to reduce padding. using a vec here makes no sense

I'm not sure but that would make it redundant and make it less predictable, the main goal is to use the l1 or l2 cache

theres a simpler solution: box the big variant to reduce padding. using a vec here makes no sense

I would write a poc then compare both

@onbjerg it's not about the struct it's about the ds of the member, think of this way:
Currently with the enum approach, ExprKind variants have different sizes leading to padding between them. When we access these in sequence, like in the visitor pattern I showed, we're jumping around memory(might be cache) with padding gaps.
With a vectorized/slice approach, the actual ExprKind data would be stored contiguously without padding between them. This means: The cpu has a better chance of predicting where the values will be, this is possible because we should have a better cache utilization
The struct being contiguous in memory isn't the focus - it's about how we arrange the actual ExprKind data for optimal access patterns.
I mean a struct that has a hashmap as one of it's member can't be called continuous the only thing it will contain in that particular instance is a pointer to the hashmap

This is not true, both the AST and HIR are allocated using an arena, meaning all pointers in the entire tree are as close as possible to each other in memory. This is the optimization you're looking for, I appreciate the thought but this has already been done and there is not much else to optimize here.

@onbjerg it's not about the struct it's about the ds of the member, think of this way:
Currently with the enum approach, ExprKind variants have different sizes leading to padding between them. When we access these in sequence, like in the visitor pattern I showed, we're jumping around memory(might be cache) with padding gaps.
With a vectorized/slice approach, the actual ExprKind data would be stored contiguously without padding between them. This means: The cpu has a better chance of predicting where the values will be, this is possible because we should have a better cache utilization
The struct being contiguous in memory isn't the focus - it's about how we arrange the actual ExprKind data for optimal access patterns.
I mean a struct that has a hashmap as one of it's member can't be called continuous the only thing it will contain in that particular instance is a pointer to the hashmap

This is not true, both the AST and HIR are allocated using an arena, meaning all pointers in the entire tree are as close as possible to each other in memory. This is the optimization you're looking for, I appreciate the thought but this has already been done and there is not much else to optimize here.

I still disagree, the poc is already seeing improvement, will do a pr soon. there's a difference between allocation and layout though


maintenance cartridge

when bypass the rfid reader do we enter maintenance mode i dont think so. Can you help the coding entering the maintenance mode uploading the code arduino almost 50sec

I don't know what maintenance mode is.


docs: add docs for build.disabled

Description
This experimental feature is only documented in the migration docs of v2 to v3.  It seems like this should be included in the main docs.  I added the experimental bullet but not sure if it is this is still considered experimental or not.
https://v3.vitejs.dev/guide/migration.html#using-esbuild-deps-optimization-at-build-time
Question:
Would we want to add a link to it from here too? https://vitejs.dev/guide/dep-pre-bundling.html
What is the purpose of this pull request?

[ ] Bug fix
[ ] New Feature
[x] Documentation update
[ ] Other

Before submitting the PR, please make sure you do the following

[x] Read the Contributing Guidelines.
[x] Read the Pull Request Guidelines and follow the PR Title Convention.
[x] Check that there isn't already a PR that solves the problem the same way to avoid creating a duplicate.
[x] Provide a description in this PR that addresses what the PR is solving, or reference the issue that it solves (e.g. fixes #123).
[x] Ideally, include relevant tests that fail without this PR but pass with it.

Ah I didn't see this PR. I just merge https://github.com/vitejs/vite/pull/12066 which covers the same thing. Yeah it should be optimizeDeps.disabled instead of build.disabled as Ben mentioned. Closing this for now, but thanks for the PR!


Linear Search | C#

Linear Search -
A linear search algorithm for C#
Type of Issue -

[x] Adding New Code

Programming Language

[x] C#

Self Check

Ask for issue assignment before making Pull Request.
Add your file in the proper folder
Clean Code and Documentation for better readability
Add Title and Description of the program in the file

:star2: Star it :fork_and_knife:Fork it :handshake: Contribute to it!
Happy Coding,

I'd like to work on this please as I'm teaching myself C#

Interested to work on this.

I want to contribute to this issue, Please assign this to me.


Title Page on /app or Home is still displaying "Frappe"

Hi
i noticed that everytime you access the Home Page or hostname_url/app, the Page Title still displays " Frappe" even after updating the app name

Can you try to run bench migrate?

Hi i have tried to run the command bench migrate several times but it seems that it still shows that Title on the Home Page whenever you refresh the Home Page

same with me, also the favicon doesnt get changed aswell


Tapo Smart Platform Error 371

Hi there ,thanks for your great plugin, i have 2 Tapo colorlight bulbs L530E  and it works just fine. I can see my 2 bulbs and i can see a third one with the name Tapo Smart Plaform, do i have to keep that one to ON ?
With Tapo Smart Platform to On i am getting this error to Homebridge Event Log. Any clue what's this ?
Thanks in advance
P.S: Sorry my English are really bad because i am not a native speaker.

@iceage2609 if the third one doesn't correspond to an actual bulb then it is probably an configuration error.


ci: fix version and e2e

Codecov Report

Merging #63 (94edf6a) into master (e2edd63) will increase coverage by 1.33%.
The diff coverage is n/a.

:exclamation: Current head 94edf6a differs from pull request most recent head 967679b. Consider uploading reports for the commit 967679b to get more accurate results

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
@@            Coverage Diff             @@
##           master      #63      +/-   ##
==========================================
+ Coverage   88.79%   90.12%   +1.33%
==========================================
  Files           7        7
  Lines         116       81      -35
  Branches       13        4       -9
==========================================
- Hits          103       73      -30
+ Misses         10        8       -2
+ Partials        3        0       -3

Flag
Coverage Δ

adapters.nestjs
90.12% <ø> (+1.33%)
:arrow_up:

core
90.12% <ø> (+1.33%)
:arrow_up:

jest
90.12% <ø> (+1.33%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.
see 4 files with indirect coverage changes


Create a content writer content card

One need to make this component using HTML and Tailwind CSS. I would suggest to use Tailwind Playgrounds to make things faster and quicker.
Here is a reference to the component.

After building the component please raise a PR with a screenshot of the component and add the component in path of the file.
If you need to use any icon please use it from Hero Icons
Good luck.

@Rajdip019 I would like to work on this issue

Go ahead @nishit3134

@Rajdip019 I was doing the same thing i.e Issue No #70 and I have created a PR as well so it will create a conflict I guess.
Please look at this.

@Rajdip019  please solve it out and close this issue. I will work on something else.


Dynamic Imports don't respect custom BUILD_ID, the uuid used at build time is hardcoded

Based on the guide in the wiki I assumed it would be safe to override the build id by changing that file. However, any use of dynamic imports import('.../....') will still use the old build id, which is hardcoded in the generated webpack bundle.
This results in a 404.
One solution might be to allow overriding the buildId at build time instead of afterwards:
https://github.com/zeit/next.js/blob/52ccc14059673508803f96ef1c74eecdf27fe096/server/build/index.js#L10-L31

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

Your Environment

Tech
Version

next
3.0.6

node
7.10

OS
Mac OS X

browser
All

Closing https://github.com/zeit/next.js/issues/3402 because this issue is more up-to-date

Dynamic imports now use file hashing


:feat exclude wallet RDNS

Changes
feat: Added a new excludeWalletRDNS param. This array contains RDNS entries for wallet domain names. Previously, developers could not exclude wallets injected from EIP6963. Now, they can add the RDNS names of wallets they wish to filter out.
fixes https://linear.app/walletconnect/issue/APKT-473/customers-cannot-override-6963-to-exclude-non-compatible-wallets

Closing as this is not the ideal solution


ckpt file not found for image classification

Hi, I download checkpoints from 'Image Classification on ImageNet-1K',  which are data in fact after decompressed. So is there something wrong here?

Hi,
The file extension is ‘pth.tar’. You can directly load the model by the 'torch.load' function. Feel free to post here if there are other questions.
Regards,
Mingyu


PTFE-1144 gcloud instances support of fixed labels

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

Comparison is base (195e920) 85.44% compared to head (923ed66) 66.12%.

Files
Patch %
Lines

runner_manager/backend/gcloud.py
0.00%
1 Missing :warning:

Additional details and impacted files
@@             Coverage Diff             @@
##             main     #496       +/-   ##
===========================================
- Coverage   85.44%   66.12%   -19.33%
===========================================
  Files          31       31
  Lines        1168     1169        +1
===========================================
- Hits          998      773      -225
- Misses        170      396      +226

Flag
Coverage Δ

api
66.12% <50.00%> (+0.02%)
:arrow_up:

unit
?

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

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


IBM.com Style Model Grid Guideline (part 2)

Continuation of #1075
Main goal for this sprint is make the PR

Defined style models and applied them to existing pages to check if they work well

Applied style models to My IBM's new design to continue testing if the hypothesis stands
Interesting to note, My IBM's content doesn't work best suit Dashboard model, but Documentation style model works well

When testing on marketing pages:

Boxfolder
https://ibm.ent.box.com/folder/100790817827?s=q8s7umnxe4rq52bvym9app0te0yot2tp


Color in github actions

It would be nice if colors are enabled in GitHub Actions . It would make reading and understanding actions easier.

@sidhu1012 can you tell me where is the file that needs to be changed?

@sidhu1012 can you tell me where is the file that needs to be changed?

.github/workflows/testing.yml


URL safe encoding

Reference Issues/PRs
#48
What does this implement/fix? Explain your changes.

Add an option for URL-safe encoding (using Base64 format) in indirect sharing mode, and handle it with the encoded query parameter.

Any other comments?

Great job dear Fatemeh, I talked with Sepand and I think it would be better to have additional (cause we already apply percent encoding (which is an encoding) to the given url) encoding selection similar to a radio group with the exact encoding name for example Base64 (more robust) for now.
This radio group (or single select checkboxes) could appear next to the direct indirect box after selecting indirect case, or maybe under the direct | indirect box.
Also there should be None checkbox to remove additional encoding selection (or the checkbox could be de-selectable).
Thanks.

Thanks for your good suggestion, yeah it sounds better.
Ok, I'll do the same.


Donation link not working

Hey I would donate you because I will be using this project, but I can't click the link. It redirects me to an image.

That would be awesome :), but I don't think we have enabled any donation link.
We are more than happy just with you using the project.


pod EaseUI失败

[!] Unable to satisfy the following requirements:

Hyphenate required by Podfile
Hyphenate (= 3.2.3) required by Podfile.lock
Hyphenate (~> 3.3.1) required by `EaseUI (3.3.1)

忘记加版本控制了


Add Duration::{ZERO, MAX, is_zero} methods

They are based on the same methods in the standard library that stabilized on Rust 1.53.

bors r+

bors r+


Split by APP_TYPE Only Returns 1 Row Per Day

I'm using the v5 API in Python to pull Account Analytics by day.
analytics_user = Analytics(0, api_config, access_token).last_30_days().split_field("APP_TYPE").get(advertiser_id)

This returns the summary data, plus 30 READY days and 1 PROCESSING day (today, so that makes sense). But the data is not "SPLIT" by APP_TYPE.  Inside of the daily data there is just a single Metric (I thought maybe there would be 3 entries under metrics, one for each day)
Looking at the log, here is the HTTP request made:
GET https://api.pinterest.com/v5/user_account/analytics?start_date=2021-12-12&end_date=2022-01-11&ad_account_id={AccountID}&split_field=APP_TYPE
<Response [200]>

I'd like to get the split by APP_TYPE, what am I missing here?

Thanks for the feedback! Honestly, I implemented to the spec without understanding the semantics really well, so I'm going to have to spend a bit of time learning about our product to answer your question.
If the problem that you're seeing with split_field is due to a big in this quickstart, I'll fix it. If the problem is due to the implementation in Pinterest's API or back-end, I'll file an appropriate bug (and then maybe fix it myself if that makes sense). If it turns out that the analytics API is working as designed, I'll figure out how to let our product management team know that split-by-app-type isn't working as expected.

Hi, @patricker. It turns out that split_field is a missing feature of the API. The feature is specified (so that it's in the API documentation) but not yet implemented. I'm double checking the status of the feature, and will reach out to you when it's ready to use. In the meantime, there's nothing that we can do in your code or this quickstart repo to get split_field to work.

@davidchaiken Do you know if the app_types filter is working? I could make... 3 calls to the API, one for each app_type.

Sounds like a clever work-around, @patricker. The tech lead for Pinterest's API team just double-checked to make sure that the app_types filter is working, and we think that you're good to go. Please let us know if it works for you.

@patricker Just wondering... Did the app_types filter work for you?

I just had time to test this today.
I ran three calls, and then compared the summary metrics with the v3 metrics available on the Analytics Web UI. While the numbers weren't identical, they were very close by device type.
    analytics_user_web = Analytics(0, api_config, access_token).last_30_days().app_types("web").get(advertiser_id)
    analytics_user_mobile = Analytics(0, api_config, access_token).last_30_days().app_types("mobile").get(advertiser_id)
    analytics_user_tablet = Analytics(0, api_config, access_token).last_30_days().app_types("tablet").get(advertiser_id)

I think this will be a good work around for now.

Thanks for the update! It's good to hear that using the app_types filter works for you. I'll chat with folks on our API and analytics teams about whether:

We should investigate why v3 and v5 metrics by device type aren't identical.
How to document the filter workaround for split_field as part of the quickstart code in this repo.

Pull request #40 captured this conversation in code.

Hi, @patricker.
Circling back here... I just checked, and it looks like split_field is working now. Just wanted to let you know.
Best,
David


Added support for comments

Please have a look at my changes for comment support.  You will need to create a comment "Hello World!" on your test page. I've never used Kotlin before, so please let me know if there are any issues.

This pull request may resolve #64

Please review my changes.

The change should be fine but just in case, I will look into it on my end this weekend. Thanks for being patient here!

@eblanchette Thanks a lot for taking the time to make this change! I've merged your change https://github.com/seratch/notion-sdk-jvm/commit/b5513556eef493de00d341fe8f517bf3685f7d34


paragraph in README file needs some changes (punctuations and delivery)

Hi @DeveloperAshish8 , sorry if the PR came in first (new to contributing to open source), but I noticed some lapses on the delivery and punctuation of the repo description in the README file. I did some changes in PR #31 . Let me know if you need more or okay with the changes :)).

As this is a minor edit, this won't be counted in hacktoberfest @chaaals

closed with PR #31


Fix: backdrop-filter not supported in Firefox

The backdrop-filter property which is used to blur the background of note dialogues is not supported in Firefox.
ToDo:

[ ] Implement a workaround, e.g. as in Glassmorphism for Firefox or SVG filters

Firefox:

Chrome:

They are working on bringing backdrop-filter back to Firefox though, and it should be available "soon". Maybe not worth the effort to implement a workaround, as it is not really critical for us?

Maybe it's worth mentioning that backdrop-filter does work on Firefox but you must explicitly enable the feature.

Yes, I think we can put this in the "spare time" section ;)


Update scss in components to use & instead of repeating containing class name

Description
New styles follow the & notation to include the containing class and follow a nested structure.
Styles that were in the past were using a different flat style.
This causes an inconsistency on the templates
Acceptance criteria

All styles follow the nested structure where the sub styles start with &

Tasks

[ ]

Additional

Check if there is a way to enforce that automatically with some tool

I think we should go with https://github.com/sasstools/sass-lint because it seems to use packages that we already use. The following rules sound interesting:

force-element-nesting
no-color-hex
bem-depth (so we don't go overboard with nesting, probably a depth of 3 or 4 is sufficient?)

Sounds good :) thanks for taking time to look at it.

Seems sass-lint is unmaintained!
I decided to go with stylelint plus stylelint-scss instead. They have rules to enforce nesting, which I'm working on.

Weird I guess I pasted the wrong link, meant to post something other but sass-lint. I think it was stylelint :)


[SPARK-11813] [MLlib] Avoid serialization of vocab in Word2Vec

jira: https://issues.apache.org/jira/browse/SPARK-11813
I found the problem during training a large corpus. Avoid serialization of vocab in Word2Vec has 2 benefits.

Performance improvement for less serialization.
Increase the capacity of Word2Vec a lot.
Currently in the fit of word2vec, the closure mainly includes serialization of Word2Vec and 2 global table.
the main part of Word2vec is the vocab of size: vocab * 40 * 2 * 4 = 320 vocab
2 global table: vocab * vectorSize * 8. If vectorSize = 20, that's 160 vocab.

Their sum cannot exceed Int.max due to the restriction of ByteArrayOutputStream. In any case, avoiding serialization of vocab helps decrease the size of the closure serialization, especially when vectorSize is small, thus to allow larger vocabulary.
Actually there's another possible fix, make local copy of fields to avoid including Word2Vec in the closure. Let me know if that's preferred.

Test build #46204 has started for PR 9803 at commit 028138a.

Test build #46204 has finished for PR 9803 at commit 028138a.

This patch passes all tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):\n  * public class JavaGradientBoostedTreeClassifierExample \n  * public class JavaGradientBoostedTreeRegressorExample \n  * public class JavaRandomForestClassifierExample \n  * public class JavaRandomForestRegressorExample \n

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/46204/
Test PASSed.

To make this change, you'd need to argue that some of these fields are only needed on the driver. Is that true? Or else, why is this object being serialized if it's mostly operating on the driver? I recognize it's Serializable. That is to say, I think just marking things @transient is suspicious as it at least indicates suboptimal design somewhere.

Yes. Word2Vec as an object does not need to be serialized. The mapPartitions in method fit uses some members of the outer class (Word2Vec) and causes the serialization. And before mapPartitions, vocab is wrapped as an broadcast variable already.

Can we avoid getting Word2Vec in a closure entirely then by modifying its usage?

Yes, by making quite a few local copy of the member variables in the method.

It is a good practice in Spark to mark variable transient if we know they are not used remotely. It is tricky to avoid serialize the entire class into a closure because it is not easy to tell which method/variable pulls in the parent object. We can create local variables, but it might not worth the complexity to do so.
LGTM. Merged into all branches since 1.1.


Error running syncdb

Hi!
I have python 3.3 and django 1.6. When I'm running syncdb this error occurs;
File "C:\Python33\lib\site-packages\django_smart_selects-1.0.9-py3.3.egg\smart_selects\db_fields.py", line 19, in init
NameError: global name 'basestring' is not defined
I'm following this example:
http://nickbewley.com/development/chained-select-boxes-in-django/
In other machine, with python 2.7 and django 1.6 this runs fine.

I have a same error, could you solved it?

Sadly, no. I switched to python 2.7.

Ah, i can solve it using a fork: here https://github.com/digi604/django-smart-selects/issues/63

fixed in new release 1.1.1


Send email with team info to elected officials

Send after City Camp to drive DataPalooza registration.

Couldn't end up doing this


MasterDetailPage and TabbedPage Support

Currently the navigation service does not support MasterDetailPage and TabbedPage. We should add support for it.

Implemented with #78


Pixelit makes NodeRED crashing with latest versions

Hi,
Since latest update from HassIO, Node-Red is crashing while trying to launch Pixelit.core
See bellow log output
PS: If I remove this specific lib from "package.json", then NodeRed starts
Can you please have a look.
Thank you
` Add-on version: 13.3.1
You are running the latest version of this add-on.
System: Home Assistant OS 8.4  (armv7 / raspberrypi4)
Home Assistant Core: 2022.8.4
Home Assistant Supervisor: 2022.08.3
Welcome to Node-RED
15 Aug 20:26:34 - [info] Node-RED version: v3.0.2
15 Aug 20:26:34 - [info] Node.js  version: v16.16.0
15 Aug 20:26:34 - [info] Linux 5.15.32-v7l arm LE
15 Aug 20:26:35 - [info] Loading palette nodes
rpi_gpio_s32.node driver V1.23 (c) Ocean<EMAIL_ADDRESS>15 Aug 20:26:42 - [info] Dashboard version 3.1.7 started at /endpoint/ui
15 Aug 20:26:43 - [info] Settings file  : /etc/node-red/config.js
15 Aug 20:26:43 - [info] Context store  : 'default' [module=memory]
15 Aug 20:26:43 - [info] User directory : /config/node-red/
15 Aug 20:26:43 - [warn] Projects disabled : editorTheme.projects.enabled=false
15 Aug 20:26:43 - [info] Flows file     : /config/node-red/flows.json
15 Aug 20:26:43 - [info] Server now running at http://<IP_ADDRESS>:46836/
15 Aug 20:26:43 - [info] Starting flows
15 Aug 20:26:44 - [info] Started flows
15 Aug 20:26:44 - [info] [mqtt-broker:HASSIO] Connected to broker: mqtt://<IP_ADDRESS>:1883
15 Aug 20:26:45 - [red] Uncaught Exception:
15 Aug 20:26:45 - [error] SyntaxError: Unexpected token , in JSON at position 183
at JSON.parse ()
at createScreenJson (/config/node-red/node_modules/node-red-contrib-pixelit/pixelit/core.js:243:48)
at processTicksAndRejections (node:internal/process/task_queues:96:5)
at core._inputCallback (/config/node-red/node_modules/node-red-contrib-pixelit/pixelit/core.js:145:37)
[20:26:45] WARNING: Node-RED crashed, halting add-on
[20:26:45] INFO: Node-RED stoped, restarting...
s6-rc: info: service legacy-services: stopping
[20:26:45] INFO: Node-RED stoped, restarting...
s6-svwait: fatal: supervisor died
s6-rc: info: service legacy-services successfully stopped
s6-rc: info: service legacy-cont-init: stopping
s6-rc: info: service legacy-cont-init successfully stopped
s6-rc: info: service fix-attrs: stopping
s6-rc: info: service fix-attrs successfully stopped
s6-rc: info: service s6rc-oneshot-runner: stopping
s6-rc: info: service s6rc-oneshot-runner successfully stopped
`

There is a bitmap array error in your flow, did you enter it manually or by API ID?

To start playing with this nice project, I used the example flow, and added  (copy/past) some nodes to play with.
The "strange" thing, it worked until last update.
Attached the Flow
Pixelit-tst.json.txt

I found the error, you used the bitmap node instead of the required bitmap animation.

But I will still install something that does not immediately support everything with this error

Hi Dennis,
Thank you for your fast response.
Indeed, I removed the bitmap node, and it worked fine
Thank you
Best regards
Gregory
Le mar. 16 août 2022 à 09:45, Dennis Rathjen @.***> a
écrit :

I found the error, you used the bitmap node instead of the required bitmap
animation.
[image: image]
https://user-images.githubusercontent.com/19333515/184825419-e857538b-caf1-4c47-96de-87c81306ee96.png
But I will still install something that does not immediately support
everything with this error
—
Reply to this email directly, view it on GitHub
https://github.com/pixelit-project/node-red-contrib-pixelit/issues/25#issuecomment-1216268409,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AKEUY336IRCB6D5HXRYFVODVZNBJ7ANCNFSM56TDBUEQ
.
You are receiving this because you authored the thread.Message ID:
@.***>


chore(cicd): fix update_dict timing

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Nest item under others above it by dragging right.

Hi! Checking this out.
I noticed that if an item is already below other items in a group. Then dragging it right does not nest it under the item above it. Possible/How? Thx!
Example:
item1
item2
item3
drag item3 right
item1
item2
-> item3
dragging item3 right does not nest it under the above item, item2
Currently you have to drag it on top of a group of items above it.
And if it's already the bottom item, then it's not something a user might quickly understand to do. They may just try to drag it to the right. Since it's already below that item.
How can I do? Thx!

I dont understand you. If I drag any item to the right it emmediately creates a nested item.

Hi, thanks for the quick response and the fine software. Will try to explain.
On the demo page at http://camohub.github.io/jquery-sortable-lists/
Go to the demo of Insert zone plus lower down the page:
Drag Item 2 directly to the right it does not become a nested item of Item 1.  It bounces back.
Drag Item 4 directly to the right it does not become a nested item of Item 3.  It bounces back.
For Item 2 to nest in Item 1.

You have to drag Item 2 up on top of Item 1 until an area opens up for it.
For Item 4 to nest in Item 3.
You have to drag Item 4 up on top of Item 3 until an area opens up for it.

Give it a try, drag item 4 only to the right.
Is there a way to have it nest with the item above it on just right drag only? Much Thx.
Same with left drag only?

Hi, thanks for the quick response and the fine software. Will try to explain.
On the demo page at http://camohub.github.io/jquery-sortable-lists/
Go to the demo of Insert zone plus lower down the page:
Drag Item 2 directly to the right it does not become a nested item of Item 1. It bounces back.
Drag Item 4 directly to the right it does not become a nested item of Item 3. It bounces back.
For Item 2 to nest in Item 1.
You have to drag Item 2 up on top of Item 1 until an area opens up for it.

For Item 4 to nest in Item 3.
You have to drag Item 4 up on top of Item 3 until an area opens up for it.
Give it a try, drag item 4 only to the right.
Is there a way to have it nest with the item above it on just right drag only? Much Thx.
Same with left drag only?

Hi, thanks for the quick response and the fine software. Will try to explain.
On the demo page at http://camohub.github.io/jquery-sortable-lists/
Go to the demo of Insert zone plus lower down the page:
Drag Item 2 directly to the right it does not become a nested item of Item 1. It bounces back.
Drag Item 4 directly to the right it does not become a nested item of Item 3. It bounces back.
For Item 2 to nest in Item 1.
You have to drag Item 2 up on top of Item 1 until an area opens up for it.
For Item 4 to nest in Item 3.
You have to drag Item 4 up on top of Item 3 until an area opens up for it.
Give it a try, drag item 4 only to the right.
Is there a way to have it nest with the item above it on just right drag only? Much Thx.
Same with left drag only?

Hi, thanks for the quick response and the fine software. Will try to explain.
On the demo page at http://camohub.github.io/jquery-sortable-lists/
Go to the demo of Insert zone plus lower down the page:
Drag Item 2 directly to the right it does not become a nested item of Item 1. It bounces back.
Drag Item 4 directly to the right it does not become a nested item of Item 3. It bounces back.
For Item 2 to nest in Item 1.

You have to drag Item 2 up on top of Item 1 until an area opens up for it.

For Item 4 to nest in Item 3.

You have to drag Item 4 up on top of Item 3 until an area opens up for it.

Give it a try, drag item 4 only to the right.
Is there a way to have it nest with the item above it on just right drag only? Much Thx.
Same with left drag only?

Yes it is correct. It is sensitive to the mouse cursor. Its position determines what happens. I know how it works. So deal with it.

But you didn't answer my question.

Dragging it to the right as I said doesn't nest it, is there a way that it could? thx.

Dragging it to the right as I said doesn't nest it, is there a way that it could? thx.

Example: when an item is dragged right. Nest it in the item just above it.
Dragging right means you want to nest it in the item above it as a sub.
Dragging left means you want to remove it from the current parent item and nest it in the next above parent of that item, if there is a parent.
Unless there isn't a parent and then it becomes parent-less and is placed below the parent item it was taken from.  Or above it if you move the cursor above the item you just removed it from.
Possible? thx.

Dragging right means you want to nest it in the item above it as a sub?
And what if you want to nest it in the item below instead of above item?
Dragging left means you want to remove it from the current parent item and nest it in the next above parent of that item, if there is a parent.
Still the same problem. What about below items? There is above and below. And also nest or not nest. Left/right is not enough to handle all the states.

Both bullet points and file trees always nest natively to the one above as their default when you tab the item right and when you drag right. It's part of their nature how they are designed.
To nest in something below your item, you always have to mouse drag the item below something, because well it's below it.
So just dragging right should tab it right and under for the item above it. It's just another case to add to the others.
No need to remove other cases.
Dragging left is also another case to be added. Much like dragging right is.  It's just another case where it instead removes the item from the current parent and places it in the parent above it, above the item it was in. Or if no parent then it is parent-less. Moved above the item it was in.
The default placing is always up on drag right or left. Unless you drag it to a more specific spot in the list. Dragging right places it up under the item above it. Dragging left places it up under the parent above it..
Anyhow it sounds like there isn't a current way. I just needed to know if there was a way currently.
Still fine software.

Take a look at:
https://frontend-collective.github.io/react-sortable-tree/
Drag an item either right or left to see. Anyhow I figured it would be an add.
I don't currently use React, but the right or left dragging of items is the above case. Just another case to be added.

So just dragging right should tab it right and under for the item above it

I don't agree with this premises.
There is a way. But nobody want to pay for it.

Many bullet lists/draggable tree lists have this. Could be a good shortcut addition for the bullet lists you show in the demo.
Better to say no it doesn't do that, instead of yes. Then we as a dev can decide to add to the code or not. In this case for now you can't drag an item directly to the right to nest it under the item directly above it. Or drag it directly to the left to move it from the current bullet list parent.
The only reason to ever drag an item to the right is to nest it as a sub item to the one above it.
Knowing helps us decide if we want to add to it or use it.
Best,

Ok have a nice day.


No ability to merge maps

I'd like to be able to provide a map and have it merged into the existing map (on a shallow level).
I'm prepared to write a PR assuming it seems like a good idea and there isn't some other way to do this that I am missing.

Sounds interesting.  Can you provide an example of what exactly you're wanting to do?

Basically I want a map that is being used to list a bunch of items that can be updated by either adding items to the list of modifying existing ones. At some point I'll want removal but I'm not there yet.
initialMap = {
  1: { name: 'a' },
  2: { name: 'b' },
  3: { name: 'c' }
}

updates = {
  1: { name: 'new a' },
  4: { name: 'd' }
}

desiredResult = {
  1: { name: 'new a'},
  2: { name: 'b' },
  3: { name: 'c' },
  4: { name: 'd' }
}

According to this merging of maps can be done like this:
 new Map([...map1, ...map2, ...map3])

but that doesn't do what I expected. I could iterate and do it manually, but i figured there was a cleaner way to do it with deconstruction... somehow.


Libuuid destdir

With this change, the PR works locally for me, although this appears to be breaking the tests.

Hum, I came across a thread where @ax3l was posting about this.
(https://github.com/spack/spack/issues/7632)

I would like to have a cleaner fix.


Thread: port to Windows

This restructures Thread to have a more generic representation of a
thread handle.  Using that it is possible to have an implementation of
the threading APIs for Windows.
Similarly, introduce an abstraction over the Thread Specific Key.  This
allows us to abstract away the key type and the usage.
[One line description of your change]
Motivation:
[Explain here the context, and why you're making that change. What is the problem you're trying to solve.]
Modifications:
[Describe the modifications you've done.]
Result:
[After your change, what will change.]

@swift-nio-bot test this please


(TWILL-144) No need to rename resource name when localization to YARN

Why we no longer rename resources anymore? I could not fully understand from TWILL-144 description

The rename is already handled by Yarn localization, hence there is no point to rename it again.

LGTM
+1


Separate shadow and shady dom logic

Clarify the separation of style scoping logic that gets run on shadow / shady DOM. Currently, some of the code gets run on both modes making the logic harder to follow & maintain.

This change is

Reviewed 1 of 1 files at r1.
Review status: all files reviewed at latest revision, all discussions resolved.

Comments from Reviewable

Review status: :shipit: all files reviewed at latest revision, all discussions resolved, all commit checks successful.

Comments from Reviewable

 even though, there's one more if nesting level, the code readability is better now :+1:

Reviewed 1 of 1 files at r1.
Review status: :shipit: all files reviewed at latest revision, all discussions resolved, all commit checks successful.

Comments from Reviewable


Add check in build_deploy.sh to tag with security-compliance when that branch is built

cc @tisnik :eyes:
Description
This change to the build_deploy.sh script is for the new security-compliance branch. It just adds a check if the current branch is security-compliance, and if it is it tags the image and pushes it up.
This will only be called on merge to the branch from app-interface. I am going to create that job here shortly.
Fixes # (issue)
Type of change
Please delete options that are not relevant.

New feature (non-breaking change which adds functionality)

Testing steps
Please describe how the change was tested locally. If, for some reason, the testing was not done or not done fully, please describe what are the testing steps.
Checklist

[x] make before_commit passes
[x] updated documentation wherever necessary
[x] added or modified tests if necessary
[x] updated schemas and validators in insights-data-schemas in case of input/output change

MR Adding the build here: https://gitlab.cee.redhat.com/service/app-interface/-/merge_requests/66156


'upload file to vm' script fails without error message for invalid VM uuid

python upload_file_to_vm.py --host vcenter_name -u root -p XXXX --vm_uuid 6977ced-d0ed801c6 --vm_user root --vm_pwd XXXX -l / -f some_file.txt
Traceback (most recent call last):
File "upload_file_to_vm.py", line 116, in
main()
File "upload_file_to_vm.py", line 76, in main
tools_status = vm.guest.toolsStatus
AttributeError: 'NoneType' object has no attribute 'guest'

This seems to happen if the user provides the wrong the VM uuid on the command line. The "FindByUuid" function returns None when it doesn't find a valid VM corresponding to the UUID. And trying to get any vm info on "None" fails as expected.

The fix for this would be exit when the VM is not found. submitting a patch shortly.


Alice created categoryData.js with details in seeds folder

Hello everyone,
I have just created a new file in seeds folder - categoryData.js, and also made some adjustments in recipeData.js: added const and then module.exports

You're on a ROLL, Alice. Nice work!


tsm1: disallow concurrent writes and deletes

There is a race described in the comments about how a delete
concurrent with a write can cause the index to miss the series.
The severity of that issue isn't described in the comment, though.
If a series is missing from the index, then it is invisible to future
deletes, meaning that you can have entries in the fields.idx for
series that don't exist, and data in the tsm file that is unreachable.
That is, until another write comes in for that series for that shard.
In that case, all of that ghost data and field constraints come back
causing great confusion for users, engineers, and the database.
There may be cheaper ways to fix the race, but this definitely
fixes the race, and we can iterate on making it faster while keeping
it correct. Additionally, leaving these races in makes it harder
to diagnose and reproduce other possible issues with deletes.

@zeebo this is a fix for https://github.com/influxdata/influxdb/issues/9694#issuecomment-435488714 isn't it?

I've been able to reproduce the effects described in that issue concurrently writing and deleting some points, so it may indeed be a fix.

Closed in favor of #10516 which will do this with less contention.


Prevent potential crash cases reported by Sentry

and update deps

@Pmant it is asyncronous stuff ... and https://github.com/Pmant/ioBroker.mqtt-client/blob/0d3c52a16460679d686986c7ec7e2e1e0fc31bb9/main.js#L648 ... So case is that stuff happens and "In parallel" the custom config is removed and so the object gets deleted and then it can happen that custom[id] was defined before an async methodcall started but not after it


⚠ Bofrost.be has degraded performance

In cedc40f, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 4386 ms

Resolved: Bofrost.be performance has improved in 9f4069a.


render/display area shinked vertically

Windows Terminal version
1.13.10336.0
Windows build number
10.0.22000.0
Other Software
No response
Steps to reproduce

copy a bunch of text from another where
paste into vim in Terminal
a paste prompt pop out, asking me really going to pasting these bunch texts, and bug happens now.

Expected Behavior
No response
Actual Behavior
about one quarter parts above there in the window "client area" disappeared, and that area becomes totally transparent , but mouse actions cannot go through, i.e. cannot click items back there.
the active area of terminal also shrinked, if I'm trying to cls, the prompt line will be in the top of the shrinked area, not disappeared instead.
the screenshot:

what I've tried but no help:

"restore window" and "maximize window"
resizing Terminal window
create split pane (e.g. Alt+Shift+'+'/'-')
create new tabs, all tabs suffering same situation.

but I feel this is pretty cool BTW🤣, can we possible making this as a feature?

Ah jeez, another one.
I'm gonna track this in #12447 since there's clearly something wrong with all these dialogs.
/dup #12447


Proper auth, simplify defaulttemplate vs per template

The options object should take in an Auth object where credentials are stored.

Simplify templates by having the same key. Overrides can still happen on a per-email basis.

Unsure. It's actually not standard to use an auth object, only the SMTP and SendGrid transport uses it. For example, another API transport, the official Amazon SES transport (https://github.com/andris9/nodemailer-ses-transport) doesn't.
Also defaultTemplate is clearer than just template. It's implied that it's overridable when calling sendMail.
I don't really care that much, though, but it should be functionally equivalent. For example, it allows an empty apiKey/apiSecret right now, in which case you'll get a nice error from the Sailthru client.

Pull request will also blow up if keys don't exist in auth object.
Given that the default transports use the auth object, I'd say nodemailer itself pushes that to the standard. (SMTP is part of core)

Sorry, services are part of core. And that uses the auth block.
Plus SMTP.

For my thoughts:

I like the use of defaultTemplate here - especially since you can override template [1]
I personally prefer the mechanism of the auth object. I like it's pattern and added readability.

[1] https://github.com/RebelMail/nodemailer-sailthru-transport/blob/master/lib/sailthru-transport.js#L20

@motdotla the PR allows you to override the defaultTemplate also.
You instantiate your transport with template making it the default one. On a per email basis, you can pass template and it will override the default one for that given email. Same functionality, just simplified by having just 1 key.

This is related to this: https://github.com/RebelMail/nodemailer-sailthru-transport/issues/2
Closing that relation, and just having this live here.
What should we settle on for this?
I am personally a fan of using the auth approach - some of that is personal preference and some of that is contentment with running tests on QA through the smtp transport.

Regardless of what we do, we will have to create a factory in order to keep adding Transports so it works with those which don't use auth block etc. But +1 for auth block.
For the keys, I still vouch for a single one.


ADR: Use flipt for feature flags

This builds off the work that @pbtruss did exploring feature flag options.
This PR supercedes #355

Thank you for your comments and questions!

Can we get examples of how the workflow would look like if we use an option like Flipt? For example, would it change how I merge things into the base branch.

I think the workflow for all feature flag options will be approximately the same: Add the code behind the feature flag that defaults to disabled and then use the feature flag system to enable it appropriately in the various environments. We'll probably have to explore how best to do that but we can possibly/probably have ways to do things like have all users that match a certain pattern have all flags enabled / disabled. I was guessing we'd probably have most things enabled in demo and use staging as a way to test out our production rollout strategy.

Also, I'm curious to why no frontend solutions were explored. I've work on large software that just hid the UI using feature flags managed in a database.

How we expose this to the frontend is another discussion, I think. I was imagining we would use a pattern similar to our Restricted component. There are pros and cons to that approach, but again, I believe that would be shared by all feature flag approaches.
I did add a section about rolling our own feature flags and why I believe that is not as optimal.

@ahobson I merged ADR 0081 (expecting yours to be merged first since it got approval first). Just noting in case you're worried about needing to bump your ADR number, you won't need to, you can stick to 0080 without conflict.


Plug ESLint plug-in into unified pipelines.

This PR normalizes the ESLint release process to the new unified pipeline process.

/cc @Azure/azure-sdk-eng


Add USD values if possible

Apparently CoinGecko has a year of data for free: https://x.com/sooneggg/status/1850523195662901442
There's also GeckoTerminal API, which is 6 months data
They think they cover most Jupiter tokens: https://x.com/sooneggg/status/1850527507445055532
Worth trying this! It'd be great to be able to add input/output/fee USD values

@mcintyre94 attempted a fix at #9 please review it!


Browser randomly crashes - only solution is wipe data

I have already experienced this twice this week alone - one before the latest version and one after.
Clearing cache does nothing and I've only been able to get things working again by wiping out the whole app, which if course isn't very helpful when it comes to losing my data.
Currently, I'm still in the crash state and am trying to see if there is another solution to this problem outside wiping.  Some times when launching it will be a white screen and I can see data is being transmitted - unsure if it is reloading my opened tabs or what.
Are there perhaps certain files that can be deleted that control the tabs cache of whatnot?

It should also be noted that I can get into the program by holding down on the icon and clicking settings - but nothing I have tried so far allows it to fully open.

i have the problem sometimes


us-fl-duval

Duval County FL just brought online a new ArcGIS server with address points.
http://maps.coj.net/coj/rest/services/DuvalMaps/duvalProperty_2012/MapServer/34

I'm having trouble opening the data in QGIS right now, it appears to stall about halfway thru, and I can't tell for certain what the fields are without seeing the source data.  I'll keep an eye on it.

I scraped 1000 records into shapefile format with all of the fields included - you can download the sample here:
https://www.mapwise.com/data/address_points_duval_test.zip

Thanks!  I'll check it out in a few minutes.


Adding maintained field to schema

I did the first steps of this in #419. There is now a maintained field, pretty much at the top level of the schema. I didn't see an existing category in which it seemed to belong.
maintained is currently defined as a nullable boolean. i.e., it should be true or false, but it can optionally be null, at least for the time being, while we work to update existing entries.
I've also added this field to every current database entry. And it's null everywhere—but this is just a starting point. All tests are passing, and we can improve our data over time. I do have some ideas for detecting entries for which maintained must be true. For example, we can search for projects that have defined funding periods that are still active. I would be comfortable saying that currently funded == maintained. (The entire problem is that the inverse does not hold…)
Anyway, if this field doesn't work out for one reason or another, or if it proves more hassle than it's worth, we can always remove it. @XeniaMonika, we should probably bump the schema version (to 0.2.2?). @Jomula, this may become an occasion for publishing new releases on GitHub and Zenodo.

In #421, I was able to set maintained to true for 56 entries (or some number close to that). I just wrote a script to identify projects that have defined funding periods that continued at least into 2023. Those are now marked as maintained as a matter of course. Separately, it occurred to me that most organizations—though not all—remain active over a longer period of time. So I looked at those as well, and felt comfortable setting maintained to true in all but a few cases.

I have bumped the schema and set the maintained value to false for 11  projects I have already marked as such during my research for the taxonomy.

closed with #423

Ok. We can leave the issue "closed," while continuing to update our records. Should just remember to follow up with Jonas about publishing another release of our schema and database.

Reopened for release task

Reopened because I still need to do the release :P

Sure! My bad

Made a release: https://zenodo.org/records/10391227


Fast Clear

Not really an issue, more of a question.
What is the fastest way to clear the led lights?
Tried 2 things so far:

strip.fill( (  0,   0,   0) ) - very slow
strip.pixels = array.array("I", [0 for _ in range(strip.num_leds)]) - much faster, but seems hacky

Yeah, first of all fill function uses set_pixel function, which does quite a bit of shifting to set R, G and B components to right position in 32 bits, so this already makes it a bit slower. Then at the end it waits for 100 microseconds, which is used since neopixel protocol needs so called "latch time" at the end of transmission for color to "latch", but I realized that it's really useless here since no real "sending" is done, it's just filling the array, meaning that delay should only be present in show function.
That "hacky" solution, would probably be the best way to do this, so if you want you can open a PR and implement clear function that sets array with zeros directly this way. If you for any reason can't do that I'll fix this redundant delay and implement clear function once I have a bit of time.
Thanks you for the question/issue :)

Cool, cool. Let me know if looks good to you https://github.com/blaz-r/pi_pico_neopixel/pull/5

Nice, thank you.


New widget: Heart rate alarm

This invisible widget vibrates whenever the heart rate gets close to the upper limit or goes over or under the configured limits.

Looks great - thank you!


Convert modifiers from hex to decimal

See similar PR done on Showdown main server; https://github.com/smogon/pokemon-showdown/pull/8003
This helps for maintenance so you don't have to pull out a calculator to convert back and forth. I don't claim that all modifiers here are accurate, but all conversions should be accurate. I did make one change beyond conversions, since I noticed it was low-hanging fruit: Auras in Gen 6 should also be 5448.

thx


Release notes

There is any release notes somewhere?
I use 0.3.0 and saw the version bumped to 0.3.3, I would like to know what changed (bug fixes and/or new features).

I'm not generating release notes yet, but I'll try to work them in at one point.
Since I use [AngularJS commit convention] (https://docs.google.com/document/d/1QrDFcIiPjSLDn3EL15IJygNPiHORgU1_OOAqWjiDU5Y/edit), this shouldn't be hard.
Right now, you can look at the commits and see which one is marked as a "fix" or a "feature".
It's actually even easier to know if there were any new features, because I use [semantic versioning] (http://semver.org/), so if the last number changes it's only a fix, new features change the middle one.

Can probably use https://pypi.python.org/pypi/gitchangelog for that...


feat(articles): implements articles index page

Sitting under /articles2 for now.

TODO

[ ] Add featured and published arguments to articlesConnection field.
[ ] Migrate RSS feed code

Warnings

:warning:

Routes added to routes.tsx should have a corresponding cypress.js smoke test. See the cypress/integration folder for examples.

  Generated by :no_entry_sign: dangerJS against ce4e30744fc4d6eaf4123caabcf7cc8a07ba2bfe


Revive and improve initial deploy script to simplify first time setup

Revive and improve initial deploy script to simplify first time setup
script/init_dev_host.sh is an example of how to setup bublik. It should be fixed to work fine now.
Kerberos ticket must be optional.
TE is not strictly required for now and should be unspecified and skipped by default.

LGTM now, all other problems, if any, should be filed separately


formatDate causes empty input box on mobile

Repro link: https://jsfiddle.net/qj1uk2xr/ (when on mobile only)
Your Environment

flatpickr version used: ^4.5.1 (ember-flatpickr)
Browser name and version: Android Chrome & iOS Safari
OS and version: iOS 12 & Android (i dunno)

If you specify a custom formatDate function and are running on mobile the formatted date will not be displayed and the the text box will be blank when a date is picked.
Simple fix on my end is to wrap the assignment of the formatDate function in an isMobile check.
            // The format date doesn't work on mobile as it uses native date handlers.
            if (!this._isMobile) {
                this._flatPickrInstance.set('formatDate', bind(this, this._formatDate));
            }

This was confirmed on both iOS and Android.

I'm experiencing the same thing in iOS when I try to attach flatpickr to a non-input field, then using onChange event to display the selected datetime in that field. However, I may not be doing it the correct way. Since I couldn't figure out how to make 'altInput' work in this use case, this is what I did:
html

Javascript
$( ".time-button" ).flatpickr({
...
onChange: function(selectedDates, dateStr, instance) {
// display date
$(this.element).html(dateStr);
	// put date in hidden input field
	$(this.element).next('input').eq(0).val(flatpickr.formatDate(selectedDates[0], "Y-m-d h:i K"));
}

});
It works great in a browser, but in mobile (at least iOS), it inserts a new input field just below my div element with class "flatpickr-mobile". I tried to just hide it, but the picker doesn't appear when I tap the div. (It only appears when the new input field is tapped.) I even tried to add the "flatpickr-mobile" class to the div but that didn't help. Thanks for any suggestions.

Same issue. Had to remove input, formatDate, parseDate for mobile.

I've also just encountered this issue. An interim fix could be to simply ignore these properties if on mobile and 'disableMobile' is false. It would be preferable to have the field working with the wrong format instead of blanking out and not updating at all while this bug exists.


Undefined symbol: mfem::MemoryManager::Delete_(void*, mfem::MemoryType, unsigned int)

As I said in previous posts I am new to MFEM and C++
I have followed the serial tutorial and have built and run several examples and produced the results with GLVis.
I am attempting to use (mess around and learn) the MFEM libraries with Xcode using this stub as a test (after adding my entire mfem-4.2 directory to the header search paths):
#include "mfem.hpp"
#include <fstream>
#include <iostream>

using namespace std;
using namespace mfem;

and it fails to build with this error:
Undefined symbol: mfem::MemoryManager::Delete_(void*, mfem::MemoryType, unsigned int)
I don't know what this means or how to fix it. Is using Xcode the wrong approach? My limited understanding is that any C++ code will ned to be run through a compiler and Xcode is the only one I'm currently familiar with. Thanks

UPDATE: I had neglected to actually link the libmfem.a library in my Xcode project. Apparently adding my mfem-4.2 directory to the search paths was not sufficient.

Note that it is possible to create an Xcode project for mfem using cmake, e.g.:
cd mfem
mkdir build
cd build
cmake .. -G "Xcode"
open mfem.xcodeproj


The Typescript language service died 5 times

VSCode Version: 1.25.1
OS Version: macOS 10.13.6

Dupe of https://github.com/Microsoft/vscode/issues/43103
TS Just keeps dying. God knows why.

Steps to Reproduce:

Does this issue occur when all extensions are disabled?: Yes/No

Please try collecting the TS Server log:

Set "typescript.tsserver.log": "verbose",
Restart VS Code and reproduce the problem
In VS Code, run the TypeScript: Open TS Server log command
This should open a folder with a tsserver.log file in it

Look through that log file for errors or stack traces. If you can share the log, I can also take a look to see if anything stands out

⚠Warning: The TypeScript log may include information from your workspace, including file paths and source code. If you have any concerns about posting this publicly on Github, just let me know and we can arrange something else. On our side, we only use these logs to investigate issues like this


[Enhancement] Spring Framework 身份认证绕过漏洞(CVE-2023-20860)

Search before asking

[X] I had searched in the issues and found no similar issues.

Description
Spring Framework 身份认证绕过漏洞(CVE-2023-20860) in master branch
Solution
Upgrade Spring Framework to 5.3.26、6.0.7 or higher level
Are you willing to submit PR?

[ ] Yes I am willing to submit a PR!

Code of Conduct

[X] I agree to follow this project's Code of Conduct

I checked the code of Doris, and using Doris does not trigger such a situation. Of course, we should upgrade to the appropriate version as soon as possible.
BTW, any questions about security vulnerabilities should not be discussed in public,


[Issue-651] Upgrade Flink to 1.15

Signed-off-by: Fan, Yang<EMAIL_ADDRESS>Change log description
Upgrade Flink version to 1.15.0 on master branch
Purpose of the change
Fix #651
What the code does
Upgrade connector to be compatible with these changes.
NOTE THAT SINK IS UPGRADED TO V2. THE CURRENT V1 TEST WILL FAIL AND IS COMMENTED OUT.
FLINK-23986 Add support for opting-out of Scala
FLINK-24427 Hide Scala in flink-table-planner from API
FLINK-25276 FLIP-203: Support native and incremental savepoints
FLINK-25555 FLIP-191: Extend unified Sink interface to support small file compaction
FLINK-24427 Hide Scala in flink-table-planner from API
FLINK-24857 Upgrade SourceReaderTestBase to Use Junit 5
How to verify it
./gradlew clean build should pass

Amend the comment and merged in b4b0015


add a new mol draw option to draw wedge bonds with a single color

In general I do like the practice of coloring bond lines to match the atoms at either end, but I find it rather distracting on wedged bonds.  For example I prefer the version on the right,

The new option simply uses the symbolColour to draw the wedges, alternatively the new option could be a color itself.
If it is kept as a boolean, perhaps the name blackWedgeBonds is inappropriate?  I am always keepint symbolColour as the default black so it works for me.
I'm not sure I added the hook into python correctly, as I don't build the python extensions  myself (or if I do then I don't know how to use them)
I haven't added a test.  I think a test would create an SVG on a simple molecule and test for the presence of a single triangle or three, but I have no experience with the SVG exporter.
Looking at the code a bit closer, the logic to apply black to both colors could be done here instead. Then there is only one spot in the code that overrides the bond colors to make it black.

after making this PR I realized I could achieve the same goal with a suitable override to drawBond.  So if this isn't a feature for general rdkit audience I have a local workaround.


parse and validate server config

This introduces a new function parse_server which parses and validates a server configuration. Also, is_client_config is introduced, which checks whether Remote is present or not.
This superseeds #50 #38

Thanks! I wonder how you are supposed to use is_client_config: We have Config.parse which does not try to validate the client- or server specific details. But then if we discover it is a client config (server config) we don't get the client (server) defaults merged. Maybe we need to expose some more functions?

Thanks! I wonder how you are supposed to use is_client_config: We have Config.parse which does not try to validate the client- or server specific details. But then if we discover it is a client config (server config) we don't get the client (server) defaults merged. Maybe we need to expose some more functions?

Yes, I guess we will need some more functionality (and figure out the exact use case for it). So, as a first step I removed the is_client_config -- we can easily re-add it at a later point when we're clear what we need.

CI failure is due to the fact that cirrus tests all the unikernels -- which each pins miragevpn to the main branch of the repository -- and here the Miragevpn.Config.parse_server is not yet bound.
We should adapt the CI scripts so that the miragevpn of the specific branch / checkout is used. But that's something that can be addressed in a separate PR -- I opened issue #106 for tracking this.


Conceptual approach to developing new functionality

Me and my colleagues are implementing the bot in an app that is an "interviewer" for the user (opposed to the usual bot implementation, where the bot mostly replies to the user questions). There are a few areas where we'd love to hear your thoughts or suggestions on how to approach this conceptually:

Self-initiated replies, meaning that at the beginning of a topic (for example), the bot will start the conversation without the user writing anything. It has been referred (I think) also here: https://github.com/silentrob/superscript/issues/24. Maybe a generic trigger such as *:1 (or whatever) that is greedier than *?
Self-learning loops: if a user asks the bot something, the bot will say "Sorry, I don't know what to reply. Can you teach me what to reply to this?". Then the user offers an answer, which the user records. Not sure if the best way is to keep this as user facts, or auto-generate topics/gambits based on the initial question from user and the subsequent reply.
Recalling user replies. Initially the bot is an interviewer, and the user the interviewee. Later on, we want to switch the context (and the bot to "impersonate" the user). How could we store initial user replies and recall them when the bot has to "impersonate" the user? Any suggestions? User facts? Auto-generated topics?

:-)

Aweome! This is a great use-case for Superscript. I would like to work through these points with you.
Would you be interested in jumping into a chat or want to setup a call sometime to discuss these in more detail?


extend host:port to support multiple agents

Currently only one host:port is supported.
It would be great if a list of host:ports could be entered if Consul is running in cluster mode.
In that case, the next host:port from the list should be used if the current node is or goes down.

Looking at the Go API client it seems like they're against this:

https://github.com/hashicorp/consul/issues/1232
https://discuss.hashicorp.com/t/how-to-make-a-client-for-multiple-hosts/29617

If they end up adding this to the Go client I'd consider replicating the functionality here, but I think it's better to just follow their lead around how API clients should work.
It seems like you're best bet is to 1) put consul behind a load balancer or 2) handle the failover logic in your application.

Sorry, I probably didn’t make myself clear.
I was proposing for the client to have one connection to one Consul agent from the list.
Only if that agent goes down or is unavailable, the client should select another agent from the list.

Yeah, that's what I understood you to mean.
What "down or ... unavailable" means is pretty complicated, and the experience is going to be bad in all but the most trivial cases (e.g. DNS doesn't resolve, DNS times out, HTTP port not open, HTTP times out, TLS certificate invalid, Consul no longer a member of the cluster, etc...).
If you have a definition of what down or unavailable means to you it should be pretty easy to create a wrapper for the client to handle those cases.

Just read the comment on the Consul thread in detail, and I think I understand better now.
Seems like the recommendation is to connect the the local Consul agent by default. Makes sense and should cover my use-case.
Closing this issue.


Add conditions that don't depend on web element

There are few cases when conditions don't depend on web element.
E.g. conditions for asserting clipboard\localstorage\etc
Seems it should be like Selenide.clipboard().shouldHave(value("foobar")

Should it be some kinda common interface that others like Clipboard or LocalStorage should extend?

@bereg2k seems yes

@asolntsev @BorisOsipov
What if we'll make check by predicate?)
Create interface Conditional<T> with method void should(Predicate<T> predicate, String message);
And all who wants support conditional wait need implement it with own type.
e.g. -  LocalStorage implements Conditional<LocalStorage> and overrides method with this type
public void should(Predicate<LocalStorage> predicate, String message)  where we can use some check utils method which will check predicate until timeout.
In test it will looks like
 Selenide.localStorage().should(storage -> storage.getItem("cat").equals("Tom"), "cat attribute in local storage doesn't match");

And one more question )
Do we need make own await implementation or can use some lib like Awaitility?)

Create interface Conditional<T> with method void should(Predicate<T> predicate, String message);

Yes, I like the idea with localStorage().should. Though, I would like to have some most used predicates/conditions in Selenide out of the box. Say,

localStorage().shouldHave(item("cat", "Tom")),
localStorage().shouldBe(empty) etc.

How could look like this API for the browser?  Something like

BLAH().shouldHave(url("/bank/login/"))
BLAH().shouldHave(openedWindows(5)) etc.

Do we need make own await implementation or can use some lib like Awaitility?)

We can use Awaitility if needed. I just didn't see a need for it until now.

@dbudim

Create interface Conditional<T> with method void should(Predicate<T> predicate, String message);

Yes, I like the idea with localStorage().should. Though, I would like to have some most used predicates/conditions in Selenide out of the box. Say,

localStorage().shouldHave(item("cat", "Tom")),
localStorage().shouldBe(empty) etc.

I though about flexibility for user to define any predicate as need)
We can save flexibility and add built-in predicates via creating conditions for appropriate functionality.
E.g. for local storage - create LocalStorageConditions with static methods that returns built in predicate
public static Predicate<LocalStorage> itemHasValue(String item, String value) {
 return localStorage -> localStorage.getItem(item).equals(value);
  }
And in test will be
Selenide.localStorage().should(LocalStorageConditions.itemHasValue("cat","Tom"), "item value doesn't match");
or with static import
Selenide.localStorage().should(itemHasValue("cat","Tom"), "item value doesn't match");

How could look like this API for the browser?  Something like

BLAH().shouldHave(url("/bank/login/"))
BLAH().shouldHave(openedWindows(5)) etc.

Actions with browser will be the same because it's just retry operation with ignoring exceptions until timeout)

Do we need make own await implementation or can use some lib like Awaitility?)

We can use Awaitility if needed. I just didn't see a need for it until now.

@asolntsev @BorisOsipov  I've made WIP: PR 1478 just to show implementation)
For now didn't add docks and not prettified PR)
Please look and if all is ok, i'll add docs and make some pretty for PR)

will be released as Selenide 5.23.0 soon


Need docker

Really need a docker to run this on linux. Taichi is hard to run naturally

@Kashu7100

I will create PR

is there any problem with pip install taichi?
@ Kashu7100 could you help with this?

Basically my offline enviroment is messy since many people work on it. So I want to use it on cloud. A docker file will be best.

https://github.com/ETSTribology/Genesis
Still working on it

I make it work with following Dockerfile. It does not build genesis from source. But install genesis through pip and install all dependencies.
FROM pytorch/pytorch:2.5.1-cuda12.4-cudnn9-devel
ENV DEBIAN_FRONTEND=noninteractive
# install basic tools
RUN apt-get update && apt-get install -y git wget curl \
        vim tar xz-utils

# install python package genesis
RUN pip install genesis-world
# install pillow for image processing
RUN pip install pillow

# install OpenGL
RUN apt-get update && apt-get install -y \
       libglvnd0 libgl1 libglx0 libegl1 libgles2

# fix missing libraries
RUN apt-get update && apt-get install -y libxrender1 libglib2.0-0

# fix pyrender
# install ffmpeg
# https://github.com/Genesis-Embodied-AI/Genesis/pull/27
RUN apt-get update && apt-get install -y ffmpeg
# install pyrender
RUN pip install pyrender
RUN pip install PyOpenGL==3.1.7
# note that there would be a conflict between PyOpenGL and pyrender.
# but we still need PyOpenGL 3.1.7 refer to https://github.com/Genesis-Embodied-AI/Genesis/issues/74
# PyOpenGL 3.1.7 raise error when using pyrender even if i build pyrender from source
# fix glGetError refer to: https://github.com/MPI-IS/mesh/issues/23
RUN apt-get install -y python3-opengl

Sorry for taking some time. Finally created a PR with Luisa support.
https://github.com/Genesis-Embodied-AI/Genesis/pull/303


Controller support

Currently the snake reacts to controller button presses, but menus don't. Probably an easy fix that would please a bunch of people.

Actually not that easy. You'll notice that the controller does work on the menus if you restart the entire game completely. I'll see what I can do about this.


[backport]Remove Project Monitoring In System Project

What kind of request is this (question/bug/enhancement/feature request):
Enhancement
Steps to reproduce (least amount of steps as possible):
None
Result:
None
Other details that may be helpful:
The project monitoring in system project make no senses. The cluster monitoring should do its job. It has been removed from system project in v2.3.x. So we should remove it in v2.2.x also.

rancher/rancher:v2.2-head d3dfcff0
UI: latest-2.2

Monitoring can not be enabled in System project, and provides a message: 	 If you want to collect custom metrics for pods in System Project, please enable cluster level monitoring directly.


Search PATHEXT instead of 3 hardcoded values

Better to search the PATHEXT variable instead of hard-coding an incomplete list for Windows executables. A default fallback consisting of XP's default value is also introduced just in case. The fallback value is hardcoded as a constant at the top of the file.

Hi- thanks for the patch. Alas I no longer have a Windows machine to test this on and Travis doesn't support Windows, can someone please confirm the tests pass? Thanks!

@isiahmeadows would you be able to rebase this off master? This might be a valuable addition, since we ran into that issue in #239 (and the PATHEXT variable was brought to my attention). @BYK could you review?

Closed this and redid it in #290.
Problem was that I have changed my username since I made the original PR.


made enum fromString() less error-prone

This is no major issue, but when I saw it, i could not leave it as it was. So many hard coded duplicate strings. Error-prone on new enum values.

Hi, this is a read only mirror, you should create an issue in JIRA. However… Although I agree with you that these duplicate strings are a risk, I don't like that your approach uses a loop. It may be slower, and also other people who see this code may come up with an optimization that maybe brings us back to the old code. So unless you can come up with a quote by some java guru who recommends this approach, I think an alternative solution would be to create a test that uses this loop strategy to verify that the strings are equal.

Oh man - start think for yourself. Or wait for a java guru. "It may be slower" - there are lots of quotes from java gurus out there that warn you from "premature optimization" - And other tons of quotes that state "maintainable code" is the first duty of every programmer.
So here's my link to the quotes: https://www.google.com/search?q=premature+optimization
I wait for your measurement how slow my code is if you - let's say - call this method 1 million times per second - oh, wait - how often is it called?
Sorry, I won't follow up this contribution - it's your code and it was just an offer to make it better. Please do follow my link.

I committed your patch, thank you. You can close the PR.

sorry guys, but why do you not use the static java.lang.Enum.valueOf()-method ?

Heh heh, I tested it and the tests failed. That one works only with the capital names.


Notify webos timeout error fix

Description:
Fix for timeout error caused by big image icon for notification
Default image changed to smaller size.
Added timeout option for connection.
Related issue (if applicable): fixes #
Pull request in home-assistant.github.io with documentation (if applicable): home-assistant/home-assistant.github.io#<home-assistant.github.io PR number goes here>
Example entry for configuration.yaml (if applicable):
notify:
  - platform: webostv
    host: <IP_ADDRESS>
    name: livingroom_tv
    filename: webostv.conf

Checklist:
If user exposed functionality or configuration variables are added/changed:

[ ] Documentation added/updated in home-assistant.github.io

If the code communicates with devices, web services, or third-party tools:

[X] Local tests with tox run successfully. Your PR cannot be merged unless tests pass

If the code does not interact with devices:

[X] Local tests with tox run successfully. Your PR cannot be merged unless tests pass

It seems fair enough to remove the default use of the hass logo as it's no longer available in this repo, but to remove the config for the default icon does not seem right. After this change if you want to use an icon, you must specify it everywhere that sends a notification to the TV.
For almost all my use cases, I want the home assistant logo to be used, so would have to make config changes to lots of automations to set a file. In the other use cases that I don't want the same logo to be used, the previous implementation worked nicely as the default icon could be overridden with individual notifications.

Yes. My mistake. I misunderstood @balloob and I removed the option instead of set default to None


Use XZ logic to specify Pauli terms

This PR cleans up some unnecessary creation of data structures to specify a Pauli operator's properties in the qubit module. These are already baked in to the x and z fields of Pauli

I think we should leave the diag, sign, imag language in the notebooks as it helps illustrate the idea. We'll just change this internally for efficiency


Feature: Add/remove DVD iso to/from a VM

This will allow you to get a boot ready system to be built

ready for 0.3.0


x/tools/gopls: add the unusedwrite analysis

CL 287034 added the new unusedwrite analysis pass.  I think it would be nice to have in gopls.
I'm not sure if it should be among the default analysers, but either way, if there are no objections to this, I am ready to send a CL.

We'd be happy to accept a CL--if you think it should be off by default, we can start with that first, and then enable it later on?

Ah, looks like it uses SSA, so let's keep it off by default to start.

Thanks for reviewing!

SSA is now enabled by default (for nilness), so the marginal cost of enabling unusedwrite is very small. I just ran a very quick experiment and 10 out of 10 findings I randomly sampled in kubernetes were genuine opportunities for simplification. So let's do a proper evaluation and perhaps enable it in gopls.

I should run a job over the Module Mirror corpus to get results that we can share, but in the meantime Googlers can inspect a list of findings of this analyzer on Google source code at go/unusedwrite-diagnostics-b1c6ea0fd8b46c. The false positive rate is zero in my manual sampling so far. I think this might be a good feature for gopls.

I ran unusedwrite over the Module Mirror corpus of 22867 modules. It reported 2976 unique diagnostics, of which 100 picked at random are shown below. I analyzed a dozen or so by hand, and all were true positives, indicating that the cost was unnecessarily complicated and confusing (though potentially still correct). In three cases, the complication was clear evidence of a bug, for example updating a field of the receiver struct before returning from a method with a non-pointer receiver.
I think we should enable this analyzer by default in gopls.
 Most of the reports in non-test code look really good.
<EMAIL_ADDRESS>FYI there are some false positives due to struct literals:

<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>
The common pattern is x := struct{f, g int}{f: 0, g: 1}; println(x.f). The implicit x.g = 1 write is being reported as it is never used. The solution is likely just better representation of struct literal initialization at the SSA level and not reporting these. This could be something like what staticcheck does in its ir, which is to have a struct literal Instruction, or better 'debug info' on the write instructions.
Just something to keep in mind. Unclear if it is urgent.

FYI there are some false positives due to struct literals:

They're not bugs, but they do indicate that a field of a struct is assigned but never used, which is a sign that the code is building a bigger data structure than it actually needs.


Created Blog speed-up-python-code

Checklist

Please note that in case of below checklist not updated accordingly, the maintainers have the right to immediately close your pull request.

[x] You created a branch from dev branch.
[x] You PR is raised on dev branch and not onmaster.
[x] Blog follows Blog Guidelines.
[x] You have read the Contribution Guidlines before creating this PR.
[x] Checked for any duplicate blog entry.
[x] Checked for all grammatical errors in the blog.
[x] You have not published the blog content anywhere else on the internet on platforms like medium, dev.to etc.
[x] The content is not copy-pasted from anywhere else on the Internet.

Description of change
Added blog on how to speed up (decrease runtime) python programme.

Author Details for Blog Post Entry

id: Md. Tahmid Hossain
bio: I am a programmer, developer and digital-marketer.
github: https://github.com/tahmid02016

Hello @tahmid02016
Thanks for writing this blog and we appreciate your effort while writing this. However, as per our project contribution guidelines, we need the blog of 600-900 words minimum.
Can you please add more information.

I have added more content. Please check it now.

Thank you very much @puneetsingh24 for fixing the typos.
I have uploaded a new cover image.  Please have a look at that. Thank you.


Created package.

Made it into a package. Can one of you install pip3 and test this?

Coverage decreased (-0.2%) to 99.491% when pulling ba71b9ff10c050cdc691a5845ec6b3df222d1838 on hsim13372:make_package into f0bc8c9801d43dc2c2d14bd46c02a8091f683968 on cs207-group11:master.

Coverage increased (+0.008%) to 99.747% when pulling 87c15be7964640a10579bc9af67d27d58123a51d on hsim13372:make_package into f0bc8c9801d43dc2c2d14bd46c02a8091f683968 on cs207-group11:master.


playing the iTunes Asset URL.

II dealt with for iTunes asset URL and checked.

There was already a PR for handeling ipod-library urls, but I think this one is better.
The other one is: https://github.com/zmxv/react-native-sound/pull/180

Hi, I checked #180.
But it was not possible playing sound on my iPhone.

Why is this still not working? I am unable to play anything from my itunes library on iphone.

Is it not DRM file?
Did you set "NSAppleMusicUsageDescription" key in info.plist?
And, Did you check Line 189 to 192 of the latest file "RNSound.m"?
https://github.com/zmxv/react-native-sound/blob/master/RNSound/RNSound.m
It's not latest If you find the difference.
My project is correct working that using RNSound on iPhone.


[BUG]: Build breaks with ENOBUFS error after successful Java unit tests

What happened?
Build breaks with ENOBUFS error after successful Java unit tests. The Java build step works fine on a local machine. Tests are also executed sucessfully on the agent VM. The VM has 8 Cores and 16 GB RAM which should be more than enough, memory/CPU limites are not reached at all.
Versions
3.238.0
Environment type (Please select at least one enviroment where you face this issue)

[X] Self-Hosted
[ ] Microsoft Hosted
[ ] VMSS Pool
[ ] Container

Azure DevOps Server type
dev.azure.com (formerly visualstudio.com)
Azure DevOps Server Version (if applicable)
No response
Operation system
Ubuntu 22.04
Version controll system
git
Relevant log output
[...]

nsctestsupport_jacocoreport:
[jacoco:report] Loading execution data file /home/azureuser/myagent/_work/1/s/core-customize/hybris/log/junit/jacoco.exec
[jacoco:report] Writing bundle 'JaCoCo' with 1134 classes

BUILD SUCCESSFUL
Total time: 3 seconds

BUILD SUCCESSFUL in 1m 31s
2 actionable tasks: 2 executed

##[error]Unhandled: write ENOBUFS
##[error]Error: write ENOBUFS
    at afterWriteDispatched (node:internal/stream_base_commons:160:15)
    at writevGeneric (node:internal/stream_base_commons:143:3)
    at Socket._writeGeneric (node:net:950:11)
    at Socket._writev (node:net:959:8)
    at doWrite (node:internal/streams/writable:588:12)
    at clearBuffer (node:internal/streams/writable:765:5)
    at onwrite (node:internal/streams/writable:644:7)
    at WriteWrap.onWriteComplete [as oncomplete] (node:internal/stream_base_commons:106:10)
Finishing:  Unittest

The build setup:
  - task: Gradle@3
    displayName: ' Unittest'
    inputs:
      gradleWrapperFile: 'gradlew'
      tasks: 'unittest'
      options: '-PbuildConfigHybris=$(hybris.config.environment)'
      publishJUnitResults: true
      testResultsFiles: '**/TEST-*.xml'
      javaHomeOption: 'JDKVersion'
      sonarQubeRunAnalysis: false
      spotBugsAnalysis: false

Hi, @DenisRumyantsev and team, any ideas what we can to to mitigate this issue? Our pipeline does not work on Microsoft hosted agents because we need some special software, and it also does not work on self-hosted agents (Azure VM) because of this issue.


Multiqc metrics

PR checklist

[ ] This comment contains a description of changes (with reason).
[ ] If you've fixed a bug or added code that should be tested, add tests!
[ ] If you've added a new tool - have you followed the pipeline conventions in the contribution docs
[ ] Make sure your code lints (nf-core lint).
[ ] Ensure the test suite passes (nextflow run . -profile test,docker --outdir <OUTDIR>).
[ ] Usage Documentation in docs/usage.md is updated.
[ ] Output Documentation in docs/output.md is updated.
[ ] CHANGELOG.md is updated.
[ ] README.md is updated (including new tool citations and authors/contributors).

Thanks the test fail though oops!

@mvheetve can you take another look please :)

Thanks!


cqinclude namespace should affect more properties

Currently the servlet only modifies properties named "name". In the case of the smartimage widget, and perhaps others, there are multiple properties that have the same behavior as "name" typically does (eg "fileName"), and these should be modified as well.
I think 99% of the time it would be safe to say that any property with a value that starts with "./" should be modified. I could also see implementing this as an OsgiConfig property so a list of property names could be configured, if the first idea seems too broad.

@jtriemstra please check out #560 and let me know your thoughts


Separate src files and test files

mkdir /test

mv test files to /test/*

dub setting

I decided to leave them as they are because some members are not accessible from separated files.


Part 2 of RFC2906 -- allow inheriting from a different Cargo.toml

Tracking issue: #8415
RFC: rust-lang/rfcs#2906
Part 1
This PR focuses on inheriting from a root workspace:

Allow inheriting from a different Cargo.toml
Add in searching for a workspace root in to_real_manifest as needed
Fixed problem where a package would try to pull a dependency from a workspace and specify { workspace = true, optional = true } and it would not respect the optional
Added tests to verify everything is in working order

Remaining implementation work for the RFC

Correctly inherit fields that are relative paths

Including adding support for inheriting license_file,  readme, and path-dependencies

Path dependencies infer version directive
Lock workspace dependencies and warn when unused
Optimizations, as needed
Evaluate any new fields for being inheritable (e.g. rust-version)

Problems:

There is duplication of code that can't be removed without significant refactoring
Potential to parse the same manifest many times when searching for a root

This should not happen when a [package] specifies its workspace
This should only happen if the workspace root is greater than one folder above

r? @ehuss
(rust-highfive has picked a reviewer for you, use r? to override)

r? @epage

@bors r+

:hourglass: Testing commit 8b3ce981caa65d48e91436f78dcad26b8263fbc6 with merge e2e2dddebe66dfc1403a312653557e332445308b...

:sunny: Test successful - checks-actions
Approved by: epage
Pushing e2e2dddebe66dfc1403a312653557e332445308b to master...


Machine should not update/install packages if Docker is already installed

Hi,
I saw that actually for Ubuntu machines the provisionner does apt-get update, install curl, then after check if docker exists.
According to me, it should check before if docker is already installed, and then if it's the case do not install/update packages (if it's possible, I'm not very familiar with the code).
It would make machine faster and for example if we give a local repo for the swarm image, it would make machine works on a non-internet environment.
What do you think about that ?

It will not install Docker if it detects docker binary on the system, but the other package installation happens anyway.

Yup exactly what I understood =) and I'm asking why ?

So, any answer =) ? As I said, it implies that the machine have to has an internet connection, which is not could be the case.

Sorry, just dropped the ball on this one.  I think checking for the existence of Docker and then skipping updates as well as install if it is present makes sense, and I'd be happy to review and possibly merge a PR doing so, however I don't really have capacity at the moment to implement the feature myself.


recieving multiple parameters from server

I can't seem to find how to do this socket.On("foo",(foo,bar)=>{...}). Is it possible to receive something from the server who did socket.Emit("foo",foo,bar)?
I checked https://github.com/Quobject/SocketIoClientDotNet/issues/111 but I did sleep well still couldn't find the answer....

How about implementing custom IListener..
let's say, your server says:
socket.emit('custom_test',"this is from ack2",new Date());
then to receive such params, we can write:

using System;
using System.Collections.Generic;
using System.Linq;
using System.Text;
using System.Threading.Tasks;
using Xamarin.Forms;
using Quobject.SocketIoClientDotNet.Client;
using System.Threading;
using Newtonsoft.Json.Linq;
using Quobject.EngineIoClientDotNet.ComponentEmitter;

namespace temp6
{
    public class CustomListener : IListener
    {
        private static int id_counter = 0;
        private int Id;
        private readonly Action<object[]> fn;

        public Acks2ListenerImpl(Action<object[]> fn)
        {

            this.fn = fn;
            this.Id = id_counter++;
        }

        public void Call(params object[] args)
        {
            var arg1 = args.Length > 0 ? args[0] : null;
            var arg2 = args.Length > 1 ? args[1] : null;

            //fn(arg1, arg2);
            fn(args);
        }

        public int CompareTo(IListener other)
        {
            return this.GetId().CompareTo(other.GetId());
        }

        public int GetId()
        {
            return Id;
        }
    }

    public partial class MainPage : ContentPage
    {
        public MainPage()
        {
            InitializeComponent();
        }

        void Handle_Clicked(object sender, System.EventArgs e)
        {
            var socket = IO.Socket("http://<IP_ADDRESS>:3000");

            socket.On(Socket.EVENT_CONNECT, () =>
            {
                //socket.Emit("server_receive","Hello Server");
            });

            socket.On("custom_test", new CustomListener((object[] server_params) =>
            {
                Console.WriteLine("I got " + server_params[0]);
                Console.WriteLine("I got " + server_params[1]);
            }));
        }
    }
}

This is a xamarin example, though it's easy to understand!