Non-swift data, indicated by :NS:

Hi there,
thanks for making and supporting this!
I have a mt940 file with (what apparently is) non-swift, indicated by :NS:, data. In my case it holds all the transaction details. I have no clue if my bank handles this correctly. But here I am having to deal with it.
:61:1908090809D000000000825,00FMSCNONREF
:NS:01My Street 17
02City and Region
03Invoice Nr. # 20190808/1 for Rent August 2019
04
05
06
07
08
09
10
11
12
13
14
15
16
17E-Banking Order for Foo Corp

My javascript is a bit rusty but could you maybe point me into a direction were I would start If I wanted to parse this with your lib?
Cheers!

Hi @dommmel , sorry for late response, quite busy these days.
Have a look at the pull request above. If OK, I'll merge.
In particular there is an example (can be called by ./cli test/test-with-NS.mt940x) and the unit test with the expected results
Briefly - if NS happens inside the transaction context it is added to nonSwift attribute. From there you can parse it further

should be solved, closing


Otel alignment

Description

Type of change:

Non-functional change (e.g. documentation changes, removing unused using directives, renaming local variables, etc)
Bug fix (non-breaking change that fixes an issue)
New feature (non-breaking change that adds functionality)
Breaking change (fix or feature that could cause existing functionality to not work as expected)

Checklist:

Have you reviewed your own code? Do you understand every change?
Are you following the contributing guidelines?
Have you added tests that prove your fix is effective or that this feature works?
New and existing unit tests pass locally with these changes?
Have you made corresponding changes to the documentation?
Will this change require an update to an example project? (if so, create an issue and link to it)

Reviewer guidelines

Should we rename the library in this PR? Or should we open another PR to do that separately? @NeilSchiefer @JasonBock

I don't think it needs to be renamed yet.  The HttpContext which is extended in this library, is under ASPNetCore in the Microsoft documentation - https://learn.microsoft.com/en-us/aspnet/core/fundamentals/http-context?view=aspnetcore-6.0.

Although @jjmckain can you update the description in the pr with a bit of detail on the changes?

@NeilSchiefer  description completed.


Is there a way to keep the local dump up to date?

Thank you for the guide. It would be helpful to include a way to keep the setup up to date periodically. I wish there was a way to to just download the 'diff' every month instead of complete dump.

I think unfortunately this is something that would have to be provided by Kiwix or the content creators themselves and is out of my control, I don't know of any download sources for only the diffs off the top of my head.  If you find something feel free to post back here and I'll reopen this!

@balki Incremental update has always be a topic at Kiwix. Unfortunately we are not ready yet for that. We are even not sure about the technology stack to provide this feature. We have work to do around Mirrorbrain to get it work properly with zsync... we would then check if zsync would be a good candidate. See https://github.com/kiwix/maintenance/issues/37


DRILL-7267: Add Slack Link in Documentation

LGTM +1

@cgivre , Is there anything else you want me to add to this or we can merge this to master ?

One more thing... Can you please change the commit message to:
DRILL-7267: Add Slack Link in Documentation

@cgivre ,
I changed the commit message as well. Thank you for strictly enforcing standards. I will keep this in mind for future PRs ( :) )

@cgivre, could you please resolve corresponding Jira ticket after the PR is merged?

@vvysotskyi , I resolved the issue. Thank you


[TACHYON-651] Cleanup code of concatPath

https://tachyon.atlassian.net/browse/TACHYON-651

+1 LGTM


Licencing options incorrect

According to this page, Microsoft 365 for business provides Teams Access.
However, when I go to: How do I get access to Microsoft Teams? link in this page, the following information is presented:
I have an Office 365 subscription. Why can’t I access Microsoft Teams?
If you have a personal Office 365 subscription, you won't be able to access Microsoft Teams. To access the app, you need one of the following Office 365 license plans:
Business Essentials
Business Premium
Enterprise E1, E3, or E5
Enterprise E4 (for anyone who purchased this plan prior to its retirement)
365 for business is not included.
Which information is correct??

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

ID: c59eff85-a7f1-764b-4716-b79d70c18d18
Version Independent ID: 49ecb4f7-3e80-c81a-32b6-4f3fda8826dd
Content: Office 365 licensing for Microsoft Teams - Microsoft Teams
Content Source: Teams/Office-365-licensing.md
Service: msteams
GitHub Login: @LolaJacobsen
Microsoft Alias: lolaj

@kimiseeley Thank you for submitting feedback and contributing to the docs. We are currently investigating this.

Hi @kimiseeley
Thank you for reaching out!
Both the documents are correct. This document does not say that "Office 365 Personal" subscription enable users for team. Kindly let us know if you find it in the document anywhere. We will correct it.
Looking forward to your response. Thank you!

Hiya
I think you got the wrong end of the stick. I stated that 365 for business
is not included. I don't recall mentioning anything about 365 personal.
I have a 365 for business account & I don't have access to teams.
Thanks
Kim
On Thu, 9 Jan 2020 at 17:46, sowjanya365<EMAIL_ADDRESS>wrote:

Hi @kimiseeley https://github.com/kimiseeley
Thank you for reaching out!
Both the documents are correct. This document does not say that "Office
365 Personal" subscription enable users for team. Kindly let us know if you
find it in the document anywhere. We will correct it.
Looking forward to your response. Thank you!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/MicrosoftDocs/OfficeDocs-SkypeForBusiness/issues/3446?email_source=notifications&email_token=ALBQXPMNODLZNGNXSX7AKJTQ43B4BA5CNFSM4JO3F7O2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEIPGBVA#issuecomment-572416212,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ALBQXPMU6LSP5W6RJBW3DPTQ43B4BANCNFSM4JO3F7OQ
.

--
Kim Seeley
Director | Senior IT Consultant |
Syntegrity Solutions[image: mobile.vsmall.jpg] +61 4 2487 7065  [image:
email.vsmall.jpg]<EMAIL_ADDRESS><EMAIL_ADDRESS>Syntegrity Solutions Pty Ltd http://www.syntegrity.com.au/ | Level 24 |
570 Bourke Street | Melbourne | VIC 3000 | 1800 235 542
ADVANCE LEAVE NOTICE: 3 February - 22 February
My Digital Badges https://www.youracclaim.com/users/kimiseeley/badges
Analyse  >>  Integrate  >>  Secure  >>  Educate
[image: Cert_all_color.jpg]

IMPORTANT: The contents of this email message and any attachments are
intended solely for the addressee(s)and may contain confidential and/or
privileged information and may be legally protected from disclosure. If you
are not the intended recipient of this message or their agent, or if this
message has been addressed to you in error, please immediately alert the
sender by reply email and then delete this message and any attachments. If
you are not the intended recipient, you are hereby notified that any use,
dissemination, copying, or storage of this message or its attachments is
strictly prohibited

Hi @kimiseeley
Thank you for submitting feedback. I think the best way forward is if you open a service ticket in your tenant so this can get addressed ASAP. Based on the outcome let me know if it is something that can
be called out in the docs.
Please follow this link to contact support for business products: https://docs.microsoft.com/office365/admin/contact-support-for-business-products
Please keep us posted here on the resolution so we can feed whatever information you discover into the content.
Thank you!

@kimiseeley Thank you for submitting feedback. We understand that this issue has been resolved.
Please feel free to re-open this issue if there is a specific area of the docs that we can improve or make better. Thank you.


In cluster mode jobs are scheduled multiple times

Hello, I have been testing Quantum 3.3 and I noticed an issue when running in cluster mode. I defined a function which was basically just a log entry "[time] Hello from #{Node.self()}" and made it run every minute. Then I run my tiny test project as 3 separate nodes on my computer.
Results were unexpected.
What I expected:

when running as single nodes each node should run the job once
when running as cluster, random node should run the job once

What I got:

when running as single nodes each node run the job once (as expected)
after joining nodes, random node would run the job n times, with n being number of nodes in the cluster.

Basically, in cluster mode, using default strategy, jobs are executed on random node, but they are scheduled once per each node. That seems to be defying the purpose and I'd like to know how to solve this.

@rradz Quantum 3.0 removed that mode:
https://github.com/quantum-elixir/quantum-core/blob/master/CHANGELOG.md#changed-1
See https://github.com/quantum-elixir/quantum-swarm

Hi I ran into this exact same problem.  I think the design is probably good, but the documentation could really be improved.  The quantum-core documentation kind of leads me to expect that this will "just work" when I set the run policy to {Quantum.RunStrategy.Random, :cluster}.  It's possible that I'm thrown off a little because I haven't used this lib in a while and last time I did the global feature was still supported.  The README for quantum-swarm could be a lot more useful - I had to dial in to the source code for quantum-swarm to see how to use it.
I'd be glad to submit PRs to help with the docs (and maybe quantum-core could/should emit a warning if you have global: true), but I'm super busy :(  In about a month I might have some time, so if it's still an issue then maybe I can help.

@dantswain I'll happily accept PRs :)


License MIT or Apache?

I'm wondering if license of this great plugin is MIT or Apache? In README.MD it states MIT, but license file says Apache. So which one is correct?

Thanks, Apache Lic.  Confusion fixed.
If you like this, the Premium Version is much better.


Switch back to upstream versions

Marker bug, depending on:
https://github.com/bendavis78/paper-time-picker/issues/28
https://github.com/bendavis78/paper-date-picker/issues/74
https://github.com/bendavis78/moment-element/pull/1

Fixed by https://github.com/Collaborne/paper-date-picker-item/commit/789289e130bb3bba5f4a7492924d5b9519f86f76


Create Introduction/Onboarding documentation.

Documentation pages should include:

Contact details

Discord links only for now

Introduction to code structure. Break project into different sections:

Level Editing
Game play
Navigation/scene managment
Level storage

Any key concepts explained regarding dependency management/code communication

Events
Singletons
Data storage / player preferences

This is all handled now by the wiki (https://github.com/LPGameDevs/EditarrrPublic/wiki) and contributor guide (https://github.com/LPGameDevs/EditarrrPublic/blob/develop/CONTRIBUTING.md)


created outquote module but need seeded data to test it out

I added seeded Outquote data in the latest develop commit of stuy-spec-api.


MSUnmerged: Failing to retrieve correct RSE list for different site tiers.

Impact of the bug
MSUnmerged
Describe the bug
While working on this issue https://github.com/dmwm/WMCore/issues/11548 to confirm the validity of a new robot certificate we are about to use in all WMCore service, we have figured out the different instances of MSUnmerged are not fetching the correct RSE list for their work. In the past we have chosen a static scalability model, where  we use one MSUnmerged instance per a set of RSEs, chosen by a mixed approach based the site's  tier level and geographical creteria. Meaning we are expected to have the following MSUnmerged instance to sites mapping:

msunmerged_t1 - all T1 level Disk RSEs
msunmerged_t2t3 - all T2 && T3 level Disk RSEs outside US
msunmerged_t2t3us -  all T2 && T3 level Disk RSEs inside US

Currently in all three instances the service is running on every RSE returned by the generic RSE expression: rse_type=DISK&cms_type=real'. This causes repetitive actions on every site, delay in service run and maybe even preventing the service polling cycle for completing.
How to reproduce it
Restarting one of the instance (e.g. msunmerged_t1) and looking at the initial lines of the logs for a single pooling cycle, gives us [1]. Which is not the expected RSE list for sure.
Expected behavior
Every MSUnmerged instace to take care only of the set of RSEs it is supposed to work on.
Additional context and error message
[1]
2023-07-03 13:34:14,180:INFO:MSUnmerged: Retrieved list of 89 RSEs: ['T3_IR_IPM',
 'T2_RU_ITEP',
 'T3_US_NERSC',
 'T3_US_Rutgers',
 'T2_US_Nebraska',
 'T3_KR_UOS',
 'T2_IT_Pisa',
 'T2_US_Wisconsin',
 'T3_HR_IRB',
 'T3_US_CMU',
 'T2_IN_TIFR',
 'T1_IT_CNAF_Disk',
 'T3_US_NotreDame',
 'T2_US_MIT_Buffer',
 'T3_CH_CERN_OpenData',
 'T3_IT_Trieste',
 'T3_US_Princeton_ICSE',
 'T2_RU_JINR',
 'T2_DE_DESY',
 'T2_TW_NCHC',
 'T3_US_OSU',
 'T3_TW_TIDC',
 'T3_CH_CERNBOX',
 'T3_US_UMiss',
 'T2_BR_UERJ',
 'T2_ES_IFCA',
 'T2_US_Purdue',
 'T1_RU_JINR_Disk',
 'T3_US_PuertoRico',
 'T1_UK_RAL_Disk',
 'T2_CH_CERN',
 'T0_CH_CERN_Disk',
 'T2_RU_INR',
 'T1_ES_PIC_Disk',
 'T2_RU_IHEP',
 'T3_BG_UNI_SOFIA',
 'T3_CY_UCY',
 'T2_IT_Bari',
 'T2_PT_NCG_Lisbon',
 'T3_FR_IPNL',
 'T2_PK_NCP',
 'T2_TR_METU',
...
 'T2_UK_London_Brunel',
 'T2_BE_IIHE'] Service set to process up to 200 RSEs per instance.

the things that mattered the most from the previous version of run.sh, were the exportation of the $X509_USER_PROXY variable and the substitution of the rseExpr inside the config file before running the service:
https://github.com/dmwm/CMSKubernetes/blob/9910d982b9b9b0f7ff57d80969b9560cacb315b1/docker/reqmgr2ms-unmerged/run.sh#L154
While the rseExpr itself was passed as an external parameter to the run.sh script:
https://github.com/dmwm/CMSKubernetes/blob/9910d982b9b9b0f7ff57d80969b9560cacb315b1/docker/reqmgr2ms-unmerged/run.sh#L34-L39

And here is the fix:
https://github.com/dmwm/CMSKubernetes/pull/1400
@amaltaro Please go ahead and review the changes so we can deploy this tonight and then I can continue tomorrow with monitoring the errors and resolving https://github.com/dmwm/WMCore/issues/11658 as well

Thanks @amaltaro for building and deploying an image  based on the PR from here https://github.com/dmwm/CMSKubernetes/pull/1400
And it solves both problems

The wrong RSE lists are fixed now. From the pod's log:

[13/Jul/2023:21:07:55]  MicroService entire configuration:
from WMCore.Configuration import Configuration
config = Configuration()
...
config.views.data.rseExpr = '(cms_type=real|cms_type=int)&rse_type=DISK&tier=1'
...

2023-07-13 21:07:56,276:INFO:MSUnmerged: Retrieved list of 1 RSEs: ['T1_US_FNAL_Disk'] Service set to process up to 200 RSEs per instance.

And it also exports the proper variable for the proxy:

_reqmgr2ms@ms-unmer-t1-5bbf49574d-vfj87:/data$ ps auxf
USER         PID %CPU %MEM    VSZ   RSS TTY      STAT START   TIME COMMAND

_reqmgr+      10  0.0  0.0   5924  3372 ?        S    21:06   0:00 /bin/bash /data/run.sh -e (cms_type=real|cms_type=int)&rse_type=DISK&tier=1

_reqmgr+      71  0.0  0.0  36340 26532 ?        S    21:07   0:00 python3 /usr/local/bin/wmc-httpd -r -d /data/srv/state/reqmgr2ms -l |rotatelogs /data/srv/logs/reqmgr2ms/reqmgr2ms-%Y%m%d-m
_reqmgr+      76 70.8  0.9 2994364 281988 ?      Sl   21:07   0:11  \_ python3 /usr/local/bin/wmc-httpd -r -d /data/srv/state/reqmgr2ms -l |rotatelogs /data/srv/logs/reqmgr2ms/reqmgr2ms-%Y%m

_reqmgr2ms@ms-unmer-t1-5bbf49574d-vfj87:/data$ xargs -0 -L1 -a /proc/76/environ |grep 509
X509_USER_CERT=/data/srv/current/auth/reqmgr2ms/dmwm-service-cert.pem
X509_USER_KEY=/data/srv/current/auth/reqmgr2ms/dmwm-service-key.pem
X509_USER_PROXY=/etc/proxy/proxy

I think we are all good here and we can proceed with merging the PR in CMSkubernetes and deploying in production.

Thanks Todor.
As mentioned over Slack, I have rebuilt MSUnmerged with the very same latest tag 2.2.2, the docker push overwrites the previous image.
In order to pull the new image though, the service yaml manifests had to be updated, the kubernetes deployment removed and then we could redeploy ms-unmerged with the very same 2.2.2 tag. Here is the relevant change made for the current deployment (thus, not persisted):
--- a/kubernetes/cmsweb/services/reqmgr2ms-unmerged-t2t3.yaml
+++ b/kubernetes/cmsweb/services/reqmgr2ms-unmerged-t2t3.yaml
@@ -85,6 +85,7 @@ spec:
       containers:
       - image: registry.cern.ch/cmsweb/reqmgr2ms-unmerged #imagetag
         name: ms-unmer-t2t3
+        imagePullPolicy: Always
         lifecycle:
           postStart:
             exec:

Thanks @amaltaro
And just to confirm - All MSUnmerged instances in production are now correctly fetching their RSE lists:

ms-unmerged-t1:

$ head -n 1000 reqmgr2ms-20230713-ms-unmer-t1-6b6886cf8f-86lqw.log |grep -A 7  "MSUnmerged: Retrieved list of"
2023-07-13 21:52:36,898:INFO:MSUnmerged: Retrieved list of 7 RSEs: ['T1_UK_RAL_Disk',
 'T1_IT_CNAF_Disk',
 'T1_FR_CCIN2P3_Disk',
 'T1_RU_JINR_Disk',
 'T1_DE_KIT_Disk',
 'T1_ES_PIC_Disk',
 'T1_US_FNAL_Disk'] Service set to process up to 200 RSEs per instance.

ms-unmer-t2t3:

$ head -n 1000 reqmgr2ms-20230713-ms-unmer-t2t3-5d7d88f97c-mh2mc.log |grep -A 57  "MSUnmerged: Retrieved list of"
2023-07-13 21:52:32,925:INFO:MSUnmerged: Retrieved list of 57 RSEs: ['T2_IT_Legnaro',
 'T2_BE_IIHE',
 'T2_UK_SGrid_RALPP',
 'T3_KR_UOS',
 'T2_FR_GRIF_IRFU',
 'T3_BG_UNI_SOFIA',
 'T2_BR_SPRACE',
 'T2_RU_INR',
 'T2_FR_GRIF_LLR',
 'T2_FI_HIP',
 'T2_AT_Vienna',
 'T3_MX_Cinvestav',
 'T2_IT_Pisa',
 'T2_IT_Rome',
 'T2_FR_IPHC',
 'T2_EE_Estonia',
 'T2_CH_CERN',
 'T3_CH_PSI',
 'T2_TR_METU',
 'T2_CN_Beijing',
 'T3_IT_Trieste',
 'T3_FR_IPNL',
 'T2_PT_NCG_Lisbon',
 'T2_HU_Budapest',
 'T3_CH_CERNBOX',
 'T2_BE_UCL',
 'T3_HR_IRB',
 'T3_TW_NTU_HEP',
 'T2_UK_London_Brunel',
 'T2_UK_SGrid_Bristol',
 'T3_KR_KNU',
 'T3_IT_MIB',
 'T2_TW_NCHC',
 'T2_ES_IFCA',
 'T3_CH_CERN_OpenData',
 'T3_KR_KISTI',
 'T2_UA_KIPT',
 'T2_UK_London_IC',
 'T2_PK_NCP',
 'T2_ES_CIEMAT',
 'T3_CY_UCY',
 'T2_RU_IHEP',
 'T2_RU_JINR',
 'T3_IR_IPM',
 'T2_FR_GRIF',
 'T2_PL_Cyfronet',
 'T2_BR_UERJ',
 'T3_TW_NCU',
 'T2_DE_RWTH',
 'T2_RU_ITEP',
 'T2_PL_Swierk',
 'T2_CH_CSCS',
 'T2_DE_DESY',
 'T2_KR_KISTI',
 'T2_IN_TIFR',
 'T3_TW_TIDC',
 'T2_IT_Bari'] Service set to process up to 200 RSEs per instance.

ms-unmer-t2t3us:

$ head -n 1000 reqmgr2ms-20230713-ms-unmer-t2t3us-7977bdc4ff-lggdw.log |grep -A 24  "MSUnmerged: Retrieved list of"
2023-07-13 21:52:32,907:INFO:MSUnmerged: Retrieved list of 24 RSEs: ['T3_US_MIT',
 'T3_US_NotreDame',
 'T3_US_NERSC',
 'T3_US_OSU',
 'T2_US_MIT',
 'T3_US_UMiss',
 'T2_US_Purdue',
 'T2_US_Vanderbilt',
 'T2_US_Caltech',
 'T3_US_PuertoRico',
 'T2_US_Nebraska',
 'T2_US_Florida',
 'T2_US_UCSD',
 'T3_US_Rutgers',
 'T3_US_UMD',
 'T3_US_Brown',
 'T3_US_FNALLPC',
 'T2_US_Wisconsin',
 'T3_US_Princeton_ICSE',
 'T3_US_Baylor',
 'T3_US_Colorado',
 'T3_US_Rice',
 'T2_US_MIT_Buffer',
 'T3_US_CMU'] Service set to process up to 200 RSEs per instance.
2023-07-13 21:52:32,907:INFO:MSUnmerged: RSE: T3_US_UMiss Reading rse data from MongoDB.


chore: use latest go in builds and CI

In the Cosmos SDK we already updated to Go v1.17. Let's update here as well.

Can you run go mod tidy?

updated

Codecov Report

Merging #497 (3e50e19) into main (a05d119) will increase coverage by 0.13%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main     #497      +/-   ##
==========================================
+ Coverage   20.06%   20.20%   +0.13%
==========================================
  Files         163      163
  Lines       23326    23358      +32
==========================================
+ Hits         4681     4720      +39
+ Misses      17888    17878      -10
- Partials      757      760       +3

Impacted Files
Coverage Δ

osmomath/math.go
65.90% <0.00%> (ø)

x/lockup/keeper/store.go
94.73% <0.00%> (+0.14%)
:arrow_up:

x/lockup/keeper/lock.go
76.73% <0.00%> (+1.27%)
:arrow_up:

app/app.go
86.09% <0.00%> (+1.45%)
:arrow_up:

x/incentives/keeper/gauge.go
62.41% <0.00%> (+2.70%)
:arrow_up:

x/lockup/genesis.go
75.00% <0.00%> (+11.36%)
:arrow_up:

Continue to review full report at Codecov.

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


no example method init_websocket, no doced driver methods

According to the README, it is not very clear exactly how to use websocket. Can we add this example to the README and documentation? I think it will be difficult for developers to delve into the sources. The documentation contains information on endpoints, but I did not see the driver methods. Could I have missed something?

Well, I'm not attentive. I apologize for my stupidity.
https://embl-bio-it.github.io/python-mattermost-autodriver/#mattermostautodriver.Driver.init_websocket

Hi @ArtemIsmagilov
Our documentation is currently rather limited and lacking some simple examples.
If you have suggestions they are very much welcome.


fix(bottom-navigation): Xcode 13 background-color correction

This will solve the issue discussed in Discord recently: https://discord.com/channels/603595811204366337/751068755206864916/892445879129210920
Tabstrip items would end up with completely transparent background colors and exhibit strange color behavior when switching tabs
I contemplated using a majorVersion >= 15 to only apply for iOS 15 in the fallback however it doesn't matter. The default behavior was always to be white when no background-color was set so this just ensures standard behavior stays same as always but also ensure building with Xcode 13 or greater retains default behavior
You can also add a class to each MDTabStripItem with a background-color set however I would not rely on developers doing this or leaving them to figure that out in this case

@NathanWalker thanks a lot for finding this one!

@farfromrefug @NathanWalker it seems that with this fix if you set a background to MDTabStrip it'll be hidden behind the white background from the MDTabStripItem. Perhaps we should set a default white background for MDTabStrip instead?

@edusperoni I'll post 1 followup PR to this hopefully this week - It actually needs one last tweak with appearance apis similar to this:
https://github.com/NativeScript/NativeScript/pull/9617/files


Add sbt-dependency-check

About this change - What it does
This PR adds sbt-dependency-check which provides an automatic way of transitively scanning your dependencies against https://nvd.nist.gov/
Resolves: #39
Why this way
Some notes

I had to move the various akka-http-* dependencies from Test to normal compile scope because an older version of akka-http was being resolved in the dependency scanning which happened to have a security vulnerability
I needed to add a suppression for akka-stream-json_2.13-0.8.2.jar because an old akka vulnerability was being incorrectly applied to akka-streams-json due to matching the same akka-* regex
I had to update akkaStreamsJson to 0.8.2 because the previous versions had a vulnerable version of akka/akka-http

@mdedetrich I wouldn't do it unless there is a way to mark things as false positives, or "not affected by". Otherwise, we might be block by vulnerabilities that are not really affecting us

So the problem regarding the false positives has been resolved. Now we have an issue with sbt-coveralls wrt to a dependency conflict
[error] com.fasterxml.jackson.databind.JsonMappingException: Scala module 2.12.3 requires Jackson Databind version >= 2.12.0 and < 2.13.0
[error] 	at com.fasterxml.jackson.module.scala.JacksonModule.setupModule(JacksonModule.scala:61)
[error] 	at com.fasterxml.jackson.module.scala.JacksonModule.setupModule$(JacksonModule.scala:46)
[error] 	at com.fasterxml.jackson.module.scala.DefaultScalaModule.setupModule(DefaultScalaModule.scala:17)
[error] 	at com.fasterxml.jackson.databind.ObjectMapper.registerModule(ObjectMapper.java:853)
[error] 	at org.scoverage.coveralls.CoverallsClient.newMapper(CoverallsClient.scala:21)
[error] 	at org.scoverage.coveralls.CoverallsClient.<init>(CoverallsClient.scala:16)
[error] 	at org.scoverage.coveralls.CoverallsPlugin$.$anonfun$coverallsTask$2(CoverallsPlugin.scala:80)
[error] 	at org.scoverage.coveralls.CoverallsPlugin$.$anonfun$coverallsTask$2$adapted(CoverallsPlugin.scala:58)
[error] 	at scala.Function1.$anonfun$compose$1(Function1.scala:49)
[error] 	at sbt.internal.util.$tilde$greater.$anonfun$$u2[219](https://github.com/aiven/guardian-for-apache-kafka/runs/5429575010?check_suite_focus=true#step:8:219)$1(TypeFunctions.scala:62)
[error] 	at sbt.std.Transform$$anon$4.work(Transform.scala:68)
[error] 	at sbt.Execute.$anonfun$submit$2(Execute.scala:282)
[error] 	at sbt.internal.util.ErrorHandling$.wideConvert(ErrorHandling.scala:23)
[error] 	at sbt.Execute.work(Execute.scala:291)
[error] 	at sbt.Execute.$anonfun$submit$1(Execute.scala:282)
[error] 	at sbt.ConcurrentRestrictions$$anon$4.$anonfun$submitValid$1(ConcurrentRestrictions.scala:265)
[error] 	at sbt.CompletionService$$anon$2.call(CompletionService.scala:64)
[error] 	at java.base/java.util.concurrent.FutureTask.run(FutureTask.java:264)
[error] 	at java.base/java.util.concurrent.Executors$RunnableAdapter.call(Executors.java:515)
[error] 	at java.base/java.util.concurrent.FutureTask.run(FutureTask.java:264)
[error] 	at java.base/java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1128)
[error] 	at java.base/java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:628)
[error] 	at java.base/java.lang.Thread.run(Thread.java:829)

It appears that sbt-coveralls is using a different binary incompatible version of jackson compared to sbt-dependency-check

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 76.676%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
263

Relevant Lines:
343

💛  - Coveralls


Suggestion: Type Inference.

I have a suggestion for a feature.
I was trying to implement something that would be simple using C++ templates, but is effectively impossible with generics, as generic wildcards cannot be used as full types.
I tried to get around this using the (deprecated) @Delegate, both in a method-local class and in an anonymous class, hoping that I could expand the interfaces of T  to the new class in order to support rudimentary metaprogramming and type inference. This did not work, as @Delegate cannot used in methods with arguments (presumably to prevent the user from passing Arg.class to the annotations type list).
If something like this could be supported, it would make Java far more powerful and easier to use, and would gain some of the compile-time dynamicism that C++ has had since CFront.
Effectively, I would like some way to expand a generic wildcard into a meaningful type.

I have no idea what this would precisely look like, but I sounds like something that just doesn't mesh with what generics are.

Sorry for the late response (and at home, so using personal account).
@rzwitserloot
It doesn't mesh with what Java generics are, which is why I'm wondering if they could be implemented as a post-compilation AST step (similar to Lombok).
Scala has a somewhat better level of functionality with generics, not quite on par with C++ but closer. I suspect it isn't practical to implement in Java itself using annotations, just wasn't sure and wanted to strike up a discussion about it.
@victorwss - Sorry. It is difficult to make posts on my phone (as I was). I'd use @lombok.experimental.Delegate, but as it is deprecated and marked to be removed in the future, I am concerned that it will cause a maintenance issue - it will lock us into the last version of lombok that supports it.


cc: add newer gcc lib dir to linker flags when gcc is a dependency

[ ] Have you followed the guidelines in our Contributing document?
[ ] Have you checked to ensure there aren't other open Pull Requests for the same change?
[ ] Have you added an explanation of what your changes do and why you'd like us to include them?
[ ] Have you written new tests for your changes? Here's an example.
[ ] Have you successfully run brew style with your changes locally?
[ ] Have you successfully run brew typecheck with your changes locally?
[ ] Have you successfully run brew tests with your changes locally?
[ ] Have you successfully run brew man locally and committed any changes?

This pull request should fix a bug on Linux when attempting to use a newer gcc when the brewed gcc@5 is installed and is linked into $HOMEBREW_PREFIX/lib.  The determine_library_paths method in extend/ENV/super.rb adds $HOMEBREW_PREFIX/lib to the list of the library paths, which results in -L$HOMEBREW_PREFIX/lib being add as a linker flag to the superenv.  This creates a problem when using newer of versions of gcc, because this flag causes the linker to use $HOMEBREW_PREFIX/lib/libstdc++.so.6 and the other run time libraries from gcc-5 instead of the versions of those libraries that come with newer version of gcc which we are trying to use.
This solution resolves this by inserting an additional linker flag for the directory which contains the runtime libraries for the newer gcc before $HOMEBREW_PREFIX/lib, so that the linker will always find the newer gcc runtime libraries before finding the older copies of those libraries in $HOMEBREW_PREFIX/lib.
This fix should not be merged until gcc on Linux points to gcc@10.  Once that is completed, we will migrate all formulae which need a newer gcc from whatever version they are currently using to depends_on gcc.  In the unlikely event that we have formulae which need a gcc version between 5 and 10, we'd have to make equivalent changes for those versions of gcc as well.  This is because the directory with the run time libraries includes the gcc version, and therefore differs for each version.
The other solution that was mentioned was to remove $HOMEBREW_PREFIX/lib from the list of library paths added by determine_library_paths, but I don't know what kind of breakage this could create, so this solution is safer.  However feedback on this decision would be great as there may be other opinions about how best to handle this.

Review period will end on 2021-02-15 at 07:38:15 UTC.

gcc formulae provide a spec file, did you check if this is used by the linker, and if we could add this flag to the respective spec files?
If not, this code needs more logic to work for any version of gcc, not gcc@10 only. Because if someone updates gcc to gcc@11 in homebrew-core, this will break this part of the code.

I agree with both of you that we should not hardcode the GCC version here.  The easiest way to avoid that would be to programmatically determine the GCC version and then replace "10" with the value of the variable containing the version.
Regarding GCC specs, I think the way those specs are currently used is that they are passed in as args, "cleaned" through refurbish_args, and then added before ldflags.  If we can get the path<EMAIL_ADDRESS>into the specs when linking, that should fix the problem.  The GCC specs files are rather cryptic so I'll have to look into how to add this.
Regarding the removal of the GCC files in $HOMEBREW_PREFIX/lib, this was the solution I was originally exploring, but it wouldn't be as simple as removing them.  I think the reason those files are there is so that binaries built with the CI host gcc-5 can find libstdc++ and other GCC runtime libraries on systems which have brewed gcc-5.  If you run brew unlink gcc@5 on a system with brewed gcc-5 (which will delete the symlinks to the GCC files in $HOMEBREW_PREFIX/lib), binaries built with gcc-5 will not be able to find libstdc++.
I think we could circumvent that by moving the symlinks for the GCC files to a subdirectory like $HOMEBREW_PREFIX/lib/gcc (-L$HOMBREW_PREFIX/lib is not recursive).  That directory could then be added during keg relocation if the bottle does not depend on a newer version of gcc.  On systems that don't use brewed gcc-5 this wouldn't have any affect as $HOMEBREW_PREFIX/lib/gcc would be non-existent or empty.
I think if I can get the linker flags into the GCC spec files, that would be the "cleanest" solution.  If it's okay I'd like to leave this open since it's a WIP, but I'll definitely close it if I am successful with the other two solutions.

Review period ended.

@iMichka @sjackman I was digging around the specs for gcc-10 trying to see where I should try to add the linker flag for GCC runtime libs, and I found this very interesting line:
*link_libgcc:
-nostdlib<EMAIL_ADDRESS>-L/home/dnachun/.linuxbrew/lib/gcc/10 -L/home/dnachun/.linuxbrew/lib

As you can see, -L$HOMEBREW_PREFIX/lib is already being added here, and is after -L$HOMEBREW_PREFIX/lib/gcc/10, exactly as we would want it if were were trying to avoid linking to the gcc-5 GC runtime libraries.
I believe the reason we are getting these linker errors is because we are adding -L$HOMEBREW_PREFIX/lib in the superenv and this preempts the linker flags added in the specs.  However, we don't actually need to add -L$HOMEBREW_PREFIX/lib in the superenv when building with gcc in Linux, because the specs already add it for us!  I'm guessing it was added to the superenv for building with clang on macOS (it might still be needed when building with clang under Linux too).
To test this, I tried three different scenarios:

Remove -L$HOMEBREW_PREFIX/lib from superenv and the specs file
Result: gcc breaks completely
Remove -L$HOMEBREW_PREFIX/lib from specs file, but not the superenv
Result: gcc compiles the formula successfully, but I get the same linker error as before because it links to the gcc-5 runtime libs instead of gcc-10
Remove -L$HOMEBREW_PREFIX/lib from the superenv, but not the specs file
Result: gcc compiles the formula successfully, and there is no linker error!

The solution here is very simple - we just remove -L$HOMEBREW_PREFIX/lib from the ldflags on Linux unless we are using clang.  I don't think clang will have the same issue because they bump the soname of their runtime libraries with every new version, unlike GCC.  And it doesn't affect GCC on macOS either, because we don't need to link the GCC runtime libs to $HOMEBREW_PREFIX/lib like we do on Linux.
I've renamed the issue since the change for this PR is different now, but I think this is a much simpler solution than what I've tried before.

Makes sense to me. The spec files are currently only installed on Linux (we still need to backport that code to homebrew-core but it's coming soon). It's a good way to handle this, as each formula provides their linker flags and it avoids conflicts.
Can you maybe just check one final thing: what happens with mixed dependency trees? (I mean with for example gcc@6 and gcc@7 being dependencies of a formula). Are both paths added, and in which order?
Also, are the lib flags for system gcc5 still added? Or does it also use the spec file from system gcc?

If you run brew unlink gcc@5 on a system with brewed gcc-5 (which will delete the symlinks to the GCC files in $HOMEBREW_PREFIX/lib), binaries built with gcc-5 will not be able to find libstdc++.

I appreciate this means that in the short-term we can't just unlink this formula but long-term it feels like relying on opt/gcc@5/lib is going to be a better strategy that's more resistant to user behaviour (as, presumably, a user can still run brew unlink gcc@5 and have everything start to explode which is explicitly something we try to avoid with linking).

I think we could circumvent that by moving the symlinks for the GCC files to a subdirectory like $HOMEBREW_PREFIX/lib/gcc (-L$HOMBREW_PREFIX/lib is not recursive).

If we're going to move them anyway (in a way that doesn't break existing binary packages which I'm not sure is possible): I'd suggest using the existing optlink path.

I'm guessing it was added to the superenv for building with clang on macOS (it might still be needed when building with clang under Linux too).

Yes, most probably.

3. but not the specs file

Stupid question: what's the "specs file" reference here?

Makes sense to me. The spec files are currently only installed on Linux (we still need to backport that code to homebrew-core but it's coming soon). It's a good way to handle this, as each formula provides their linker flags and it avoids conflicts.

Would there be any value/benefit in doing it here before we ship this PR?

@MikeMcQuaid see https://github.com/Homebrew/linuxbrew-core/blob/master/Formula/gcc.rb#L224-L270 for the specs file. This code is in the post-install step of each gcc formula provided by linuxbrew-core. It allows to defined the different include and lib paths for each gcc version.

@MikeMcQuaid see https://github.com/Homebrew/linuxbrew-core/blob/master/Formula/gcc.rb#L224-L270 for the specs file. This code is in the post-install step of each gcc formula provided by linuxbrew-core. It allows to defined the different include and lib paths for each gcc version.

That looks pretty heavy for formulae code. Is it just use in a single GCC or multiple? Why does it have to be in post-install rather than install? Would be good to get something (tested, too!) into Homebrew/brew if it's used on more than a single formula.

There are some steps to keep a backup of the original specs file, but we might just drop that. And it's in the post install step as the content depends on the version of glibc that is installed (though I do not know the details here). I agree that we could move that code to Homebrew/brew as it is used by each gcc formula in the same ways.

I agree that we could move that code to Homebrew/brew as it is used by each gcc formula in the same ways.

Sounds good in that case, yeh.

If you run brew unlink gcc@5 on a system with brewed gcc-5 (which will delete the symlinks to the GCC files in $HOMEBREW_PREFIX/lib), binaries built with gcc-5 will not be able to find libstdc++.

I appreciate this means that in the short-term we can't just unlink this formula but long-term it feels like relying on opt/gcc@5/lib is going to be a better strategy that's more resistant to user behaviour (as, presumably, a user can still run brew unlink gcc@5 and have everything start to explode which is explicitly something we try to avoid with linking).

I think we could circumvent that by moving the symlinks for the GCC files to a subdirectory like $HOMEBREW_PREFIX/lib/gcc (-L$HOMBREW_PREFIX/lib is not recursive).

If we're going to move them anyway (in a way that doesn't break existing binary packages which I'm not sure is possible): I'd suggest using the existing optlink path.

Using the opt_lib path sounds like the best long term solution to me as well.  Ideally brew unlink shouldn't be able to break anything, especially not anything poured from bottles.  We could add the opt_lib for gcc-5 (or whatever the host gcc version is in CI) to the RPATH when pouring bottles if the user has the brewed gcc-5 installed and the formula does not depend, even indirectly, on a newer version of GCC.  We'd have to do a lot of checking to make sure nothing breaks by doing this but I believe it would improve the user experience if we remove the possibility of breaking things with brew unlink.

Makes sense to me. The spec files are currently only installed on Linux (we still need to backport that code to homebrew-core but it's coming soon). It's a good way to handle this, as each formula provides their linker flags and it avoids conflicts.
Can you maybe just check one final thing: what happens with mixed dependency trees? (I mean with for example gcc@6 and gcc@7 being dependencies of a formula). Are both paths added, and in which order?

I should clarify that when I look at the build logs for a formula using a newer version of GCC, I don't actually see any of those linker flags in the GCC call.  I think the arguments from the specs file are "invisibly" added to the GCC call after all the arguments provided to it on the command line.
When you say mixed dependency do you mean a situation with depends_on gcc@6 and depends_on gcc@7 in the same formula?  Or one where formula A depends on formula B, and formula A also has a runtime dependency on gcc-7, while formula B has a runtime dependency on gcc-6?
If you're talking about the latter case, I think the arguments from the specs are added based on whichever version of gcc is actually called by the formula.  If the formula uses gcc-7 and depends on a formula which has a runtime dependency on gcc-6, this shouldn't be an issue - the linker will use the libstdc++.so.6 from the gcc-7 specs which is backwards compatible with gcc-6 (and gcc-5).
Where we'd run into problems is if we had the opposite situation - the formula uses gcc-6 but depends on a formula which has a runtime dependency gcc-7.  The libstdc++.so.6 from gcc-6 is not forwards compatible with the one from gcc-7 (I think GCC should bump the soname of libstdc++ with each major release like LLVM does, but we don't control that).
I think the only solution there is to forbid formulae on Linux from using a version of gcc that is older than the newest version   used by anything in the formula's dependency tree.  The easiest way to achieve this is by just using the latest version of GCC (currently gcc-10) whenever the CI host gcc (currently gcc-5) is too old.  However we could add an of audit to check for this automatically in PRs.
Sorry for such a long answer, but this is a rather complex problem!  I'll try to find some test cases to verify what I've said here.

Also, are the lib flags for system gcc5 still added? Or does it also use the spec file from system gcc?

I think the lib flags from the system GCC spec are only used when we actually call the system GCC.  We use the system GCC in CI or whenever the host GCC is version 5 or greater.  The first argument in the specs is -nostdlib, which should remove the standard lib paths fro the linker search path, effectively "hiding" the system GCC from brewed GCC.


Can't change Default Typescript Component PropsState Prefix

settings.json

@MuttakinHasib Fixed in v4.1.0 as I forgot to implement this feature 😅

I just installed v4.1.0, but still I don't get interfaces in favor of types.
Also reactSnippets.settings.prettierEnabled doesn't seem to have an impact.


Example 'Cluster-Aware Service Routing' fails due to timing

Describe the bug
When installing cluster 2 as described here, step 3 mentions:
$ kubectl create --context=$CTX_CLUSTER2 ns istio-system
$ kubectl create --context=$CTX_CLUSTER2 secret generic cacerts -n istio-system --from-file=samples/certs/ca-cert.pem --from-file=samples/certs/ca-key.pem --from-file=samples/certs/root-cert.pem --from-file=samples/certs/cert-chain.pem
$ kubectl create --context=$CTX_CLUSTER2 -f istio-remote-auth.yaml

This results in the egress-gateway to fail with an error about missing certs. After restarting the pod, it works.
Reason: The secret is generated in the same step as the egress-gateway that mounts it as optional mount.
Expected behavior
To either works smoothly or at least mention that the egress-gateway has to be restarted. Seperating it into 2 steps should be enough.
Steps to reproduce the bug
Follow the example description until 2.2.
Version
Client Version: version.Info{Major:"1", Minor:"13", GitVersion:"v1.13.3", GitCommit:"721bfa751924da8d1680787490c54b9179b1fed0", GitTreeState:"clean", BuildDate:"2019-02-04T04:49:22Z", GoVersion:"go1.11.5", Compiler:"gc", Platform:"darwin/amd64"}
Server Version: version.Info{Major:"1", Minor:"13", GitVersion:"v1.13.3", GitCommit:"721bfa751924da8d1680787490c54b9179b1fed0", GitTreeState:"clean", BuildDate:"2019-02-01T20:00:57Z", GoVersion:"go1.11.5", Compiler:"gc", Platform:"linux/amd64"}
Istio 1.1.0-rc.0
Installation
As described in the example.
Environment
AWS cluster.

This came up during community testing for 1.1, unfortunately I am not able to set the milestone.

@c0d1ngm0nk3y seems sufficient doc would help in this case... cc @ymesika

@c0d1ngm0nk3y with the next RC (or final) release of 1.1 the egress won't be installed by default.
Because the steps in the Cluster-aware doc are based on the default Istio values there won't be egress on both primary and secondary clusters. Which kind of indirectly solves this issue.

@ymesika Thanks for the information, but the ingress has the same problem. So this this is probably not solved by this alone. But @linsun is of course right, better documentation should be enough. We will push a PR.

@c0d1ngm0nk3y It finally catches up with the generated secrets/certs. It just takes time until the proxy is picking the newly created secrets from the volume. You can tail the logs from the ingress and see that after 30-60sec it loads the certs and connect to pilot with those.
 It was somehow hidden by issue #12021. So I will close this one.


Slider onChangeEnd is never called when arrow keys are used.

Steps to Reproduce

Execute flutter run on the code sample
Click on the slider so it gains focus
Use left and right arrow keys to change slider position
Note that onChangeEnd is never called.  Only tested on Windows 10.

Expected results: onChangeEnd should be called at the end of every slider change, regardless how it is triggered.  With arrow keys, I would expect onChangeEnd to be called when the arrow key is released.
Actual results: onChangeEnd is never called when the arrow keys are used.

Code sample

import 'package:flutter/material.dart';

void main() => runApp(const SliderApp());

class SliderApp extends StatelessWidget {
  const SliderApp({super.key});

  @override
  Widget build(BuildContext context) {
    return const MaterialApp(
      home: SliderExample(),
    );
  }
}

class SliderExample extends StatefulWidget {
  const SliderExample({super.key});

  @override
  State<SliderExample> createState() => _SliderExampleState();
}

class _SliderExampleState extends State<SliderExample> {
  double _currentSliderValue = 20;

  @override
  Widget build(BuildContext context) {
    return Scaffold(
      appBar: AppBar(title: const Text('Slider')),
      body: Slider(
        value: _currentSliderValue,
        max: 100,
        divisions: 5,
        label: _currentSliderValue.round().toString(),
        onChanged: (double value) {
          setState(() {
            _currentSliderValue = value;
          });
        },
        onChangeEnd: (double value) {
          print('onChangeEnd');
        },
      ),
    );
  }
}

@git-n-pissed
Looking at the code sample, I see the onChangeEnd is not called anywhere:

Also, take a look at this SO link and see if it helps in your case.

@git-n-pissed Looking at the code sample, I see the onChangeEnd is not called anywhere:

Also, take a look at this SO link and see if it helps in your case.

Apologizes, the code I provided was missing the closing bracket for onChanged.  I have corrected the code and the issue is still as described in the OP.
Thank you for the SO link, but I do not believe adding a delay will be helpful in this case since the onChangeEnd callback is not being called at all when the arrow keys are used on Windows.

Thanks for the update. I was able to replicate this on mobile platform (iOS). Upon running code sample and clicking on slider, the onChangeEnd is triggered, but then using arrow keys although the slider moves, but the said method isn't called.

 stable, master flutter doctor -v
[!] Flutter (Channel stable, 3.7.7, on macOS 12.2.1 21D62 darwin-x64, locale
    en-GB)
    • Flutter version 3.7.7 on channel stable at
      /Users/dhs/documents/fluttersdk/flutter
    ! Warning: `flutter` on your path resolves to
      /Users/dhs/Documents/Fluttersdk/flutter/bin/flutter, which is not inside
      your current Flutter SDK checkout at
      /Users/dhs/documents/fluttersdk/flutter. Consider adding
      /Users/dhs/documents/fluttersdk/flutter/bin to the front of your path.
    ! Warning: `dart` on your path resolves to
      /Users/dhs/Documents/Fluttersdk/flutter/bin/dart, which is not inside your
      current Flutter SDK checkout at /Users/dhs/documents/fluttersdk/flutter.
      Consider adding /Users/dhs/documents/fluttersdk/flutter/bin to the front
      of your path.
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 2ad6cd72c0 (12 days ago), 2023-03-08 09:41:59 -0800
    • Engine revision 1837b5be5f
    • Dart version 2.19.4
    • DevTools version 2.20.1
    • If those were intentional, you can disregard the above warnings; however
      it is recommended to use "git" directly to perform update checks and
      upgrades.

[!] Xcode - develop for iOS and macOS (Xcode 12.3)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    ! Flutter recommends a minimum Xcode version of 13.
      Download the latest version or update via the Mac App Store.
    • CocoaPods version 1.11.2

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] VS Code (version 1.62.0)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.21.0

[✓] Connected device (5 available)
    • SM G975F (mobile)       • RZ8M802WY0X • android-arm64   • Android 11 (API 30)
    • Darshan's iphone (mobile)  • 21150b119064aecc249dfcfe05e259197461ce23 •
      ios            • iOS 14.4.1 18D61
    • iPhone 12 Pro Max (mobile) • A5473606-0213-4FD8-BA16-553433949729     •
      ios            • com.apple.CoreSimulator.SimRuntime.iOS-14-3 (simulator)
    • macOS (desktop)            • macos                                    •
      darwin-x64     • Mac OS X 10.15.4 19E2269 darwin-x64
    • Chrome (web)               • chrome                                   •
      web-javascript • Google Chrome 98.0.4758.80

[✓] HTTP Host Availability
    • All required HTTP hosts are available

! Doctor found issues in 1 category.

[!] Flutter (Channel master, 3.9.0-16.0.pre.42, on macOS 12.2.1 21D62
    darwin-x64, locale en-GB)
    • Flutter version 3.9.0-16.0.pre.42 on channel master at
      /Users/dhs/documents/fluttersdk/flutter
    ! Warning: `flutter` on your path resolves to
      /Users/dhs/Documents/Fluttersdk/flutter/bin/flutter, which is not inside
      your current Flutter SDK checkout at
      /Users/dhs/documents/fluttersdk/flutter. Consider adding
      /Users/dhs/documents/fluttersdk/flutter/bin to the front of your path.
    ! Warning: `dart` on your path resolves to
      /Users/dhs/Documents/Fluttersdk/flutter/bin/dart, which is not inside your
      current Flutter SDK checkout at /Users/dhs/documents/fluttersdk/flutter.
      Consider adding /Users/dhs/documents/fluttersdk/flutter/bin to the front
      of your path.
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision a8afbfa22e (47 minutes ago), 2023-03-24 00:00:20 -0400
    • Engine revision 1da070c57c
    • Dart version 3.0.0 (build 3.0.0-362.0.dev)
    • DevTools version 2.22.2
    • If those were intentional, you can disregard the above warnings; however
      it is recommended to use "git" directly to perform update checks and
      upgrades.

[!] Android toolchain - develop for Android devices (Android SDK version 30.0.3)
    • Android SDK at /Users/dhs/Library/Android/sdk
    ✗ cmdline-tools component is missing
      Run `path/to/sdkmanager --install "cmdline-tools;latest"`
      See https://developer.android.com/studio/command-line for more details.
    ✗ Android license status unknown.
      Run `flutter doctor --android-licenses` to accept the SDK licenses.
      See https://flutter.dev/docs/get-started/install/macos#android-setup for
      more details.

[✓] Xcode - develop for iOS and macOS (Xcode 13.2.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Build 13C100
    • CocoaPods version 1.11.2

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] IntelliJ IDEA Ultimate Edition (version 2021.3.2)
    • IntelliJ at /Applications/IntelliJ IDEA.app
    • Flutter plugin version 65.1.4
    • Dart plugin version 213.7228

[✓] VS Code (version 1.62.0)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.29.0

[✓] Connected device (3 available)
    • Darshan's iphone (mobile) • 21150b119064aecc249dfcfe05e259197461ce23 • ios
      • iOS 15.3.1 19D52
    • macOS (desktop)           • macos                                    •
      darwin-x64     • macOS 12.2.1 21D62 darwin-x64
    • Chrome (web)              • chrome                                   •
      web-javascript • Google Chrome 109.0.5414.119

[✓] Network resources
    • All expected network resources are available.

! Doctor found issues in 1 category.

[!] Xcode - develop for iOS and macOS (Xcode 12.3)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    ! Flutter recommends a minimum Xcode version of 13.
      Download the latest version or update via the Mac App Store.
    • CocoaPods version 1.11.2

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] VS Code (version 1.62.0)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.21.0

[✓] Connected device (5 available)
    • SM G975F (mobile)       • RZ8M802WY0X • android-arm64   • Android 11 (API 30)
    • Darshan's iphone (mobile)  • 21150b119064aecc249dfcfe05e259197461ce23 •
      ios            • iOS 14.4.1 18D61
    • iPhone 12 Pro Max (mobile) • A5473606-0213-4FD8-BA16-553433949729     •
      ios            • com.apple.CoreSimulator.SimRuntime.iOS-14-3 (simulator)
    • macOS (desktop)            • macos                                    •
      darwin-x64     • Mac OS X 10.15.4 19E2269 darwin-x64
    • Chrome (web)               • chrome                                   •
      web-javascript • Google Chrome 98.0.4758.80

[✓] HTTP Host Availability
    • All required HTTP hosts are available

! Doctor found issues in 1 category.

Hi, I was looking into this issue and saw that using the arrow key only fires the onChanged callback and neither onChangeStart or onChangeEnd callback. I am brand new at trying to contribute, any tips on where I could look to resolve this? I am wondering if its somewhere in this slider class I am looking over or if it might be in the gestures package. Thanks for any guidance you can give.

I have something that may work as a fix? I'm wondering what people think. here is a gif of it. Basically every time you press a key there is a call  to onChangeStart, then  onChanged is called like  before, then onChangeEnd is called with the value of the spot you landed at. I did not implement something like a drag however, if you hold a key it will call all three callbacks per position switch.

@darshankawar, good morning, I was wondering if you could check out my comments on this issue and give feedback. Also, was wondering you had the ability to assign me this issue. Thanks for your time.

@TahaTesser Hi, I noticed that you were assigned the umbrella of Slider issues. Is there a way you could assign this issue in particular to me?

@TahaTesser Hi, I noticed that you were assigned the umbrella of Slider issues. Is there a way you could assign this issue in particular to me?

@dleyba042
Hi, I recommend taking a look at another issue. This will be fixed part as part of a bigger project under https://github.com/flutter/flutter/issues/125329

Thanks @TahaTesser just doing this for education purposes. I have begun to look and start working on fixes for other issues as well.


Relation extraction on pretagged candidates

Hi,
I want to use snorkel to extract 5 types of relationships between pretagged entities. I already have a trained NER model that detects the desired entities pretty well. I just need snorkel to extract relations between preannoted candidates.
Is it possible to use my own pretagged entities as snorkel candidates? I have found a class named PretaggedCandidateExtractor but I'm not sure that it suits me.
Do you have any documentation or examples?
Thank you very much !

@stephenbach @ajratner
Any help please :-)
Thank you !

Hi @amineabdaoui , the PreTaggedCandidateExtractor is indeed the class we currently have for this setting. You can see it used in the CDR tutorial we have.
Another option (which might end up being ultimately cleaner, depending on your NER tagger) is to create a new Parser subclass for it(for example, see the Spacy subclass as an example)
Hope this helps!

Thank you very much @ajratner
Indeed, your answer is pretty helpful :-)
Right now, I'm trying to load labeled categorical relations as a gold standard to evaluate my models. However, it seems that the Categorical_Classes Tutorial do not use any specific load_external_labels method. Is this means that we can load categorial labels in the same way as we did this on binary labels ?

Yep!
On Wed, Jun 13, 2018 at 4:14 AM Amine Abdaoui<EMAIL_ADDRESS>wrote:

Thank you very much @ajratner https://github.com/ajratner
Indeed, your answer is pretty helpful :-)
Right now, I'm trying to load labeled categorical relations as a gold
standard to evaluate my models. However, it seems that the Categorical_Classes
Tutorial
https://github.com/HazyResearch/snorkel/blob/master/tutorials/advanced/Categorical_Classes.ipynb
do not use any specific load_external_labels method. Is this means that
we can load categorial labels in the same way as we did this on binary
labels ?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/HazyResearch/snorkel/issues/947#issuecomment-396853852,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABgw_TW50PuusmUDvZFCw2xOk0H0H0cGks5t8MoCgaJpZM4Uf1P-
.


Emoji

Syntax update
Added more bytes to work with emojis
Updated test and added emoji test

IE nooooo!!! hahaha...
I will send a new PR with only emoji compatibility.

Well, it's ES5 intentionally so that it runs nearly everywhere. Until JS ecosystem wakes up a bit, it's the best bet... especially with tiny modules like this.
I went ahead & edited your fork so that this PR only has the relevant changes. It's still not complete (padStart) if you want to pick it up from there?
Also, I need to research the u RegExp flag (TIL, or at least, today I re-learned haha) and see if it can make the cut.
Thanks!

Ok,
We're here to help or try to help! 😆
I will use the lib to make an app to send secret messages and is must necessary have emojis.
You can close the PR if you want.
Thanks and Cheers

No worries, thanks! I'll accept it how it is (no more style changes please) – I'm just figuring out scheduling & timing with existing uses.
PS: If you're looking for security, salting isn't very safe. This is a quick-n-dirty approach for very light masking. True encryption should rely on a time-tested algo with private keys.


allcontributors-bot test

@all-contributors please add @PandaLIU-1111   for  contributions

@all-contributors please add @her-cat  for contributions

@all-contributors please add @her-cat for infrastructure, tests and code

https://github.com/all-contributors please add @Rock-liyi  for infrastructure, tests and code

@all-contributors please add @her-cat for infrastructure, tests and code

@all-contributors please add @CbYip for infrastructure, tests and code

@all-contributors please add @PandaLIU-1111 for infrastructure, tests and code

@all-contributors please add @Rock-liyi  for infrastructure, tests and code

我已收到你的文件


Generated OpenAPI doesn't include enum query parameter  details when it defines in separate module

Description:
Query parameter details have skipped when generating the OpenAPI specification when it is denied in a separate module
Steps to reproduce:
service file
import ballerina/http;
import enum_openapi.types as tp;

service / on new http:Listener(9090) {
    resource function get pet(tp:STATUS status) returns error? {
        //logic to get pet
        return error("Not implemented");
    }
}

type.bal which is in different module
public enum STATUS {
    IN_PROGRESS,
    DONE
}

generated yaml
....
paths:
  /pet:
    get:
      operationId: getPet
      responses:
        "500":
          description: Internal server error
          content:
            application/json:
              schema:
                $ref: '#/components/schemas/ErrorPayload'
components:
  schemas:
    ErrorPayload:
      type: object
      properties:
        reason:
          type: string
          description: Reason phrase
        path:
          type: string
          description: Request path
        method:
          type: string
          description: Method type of the request
        message:
          type: string
          description: Error message
        timestamp:
          type: string
          description: Timestamp of the error
        status:
          type: integer
          description: Relevant HTTP status code
          format: int32

Affected Versions:
2201.8.0 swanlake
OS, DB, other environment details and versions:
Related Issues (optional):

Suggested Labels (optional):

Suggested Assignees (optional):

Fixed via https://github.com/ballerina-platform/openapi-tools/pull/1564


Example layout is broken on iPhone 5S simulator

Hi @Gargo! Can you post a screenshot? I know Demo layout might look clipped on any screens smaller than iPhone Plus, because optimizing this was my lowest priority.

@vladaverin24 the issue is very obvious to post screenshots. For example look at your own screenshots - in iphone 5s simulator the top white labels overlap cards

Got it. Thanks for reporting.


kernel too old on Synology

I'm getting this log output when running this project through Docker on my Synology DS916+ (DSM 7.1.1-42962 Update 6)
Ensuring user with uid:[1045] gid:[65539] exists ...
Group with gid [65539] name [spotc] already exists.
user with uid [1045] name [spotc] already exists.
Using SpotConnect raop version [0.4.1]
Command Line: [/app/bin/spotraop-linux-static -r 320 -x /config/config.xml -I -j -k -Z]
Segmentation fault (core dumped)

When I attach to the console and try to run the command myself, I get this output:
root@SpotConnect:/# /app/bin/spotraop-linux-static -r 320 -x /config/config.xml -I -j -k -Z
FATAL: kernel too old
Segmentation fault (core dumped)

My kernel version is 3.10.108
Any hope to fix this with some config or change to my Docker env?

What happens with the non-static version?

# /app/bin/spotraop-linux -r 320 -x /config/config.xml -I -j -k -Z
[22:31:30.872] main:1221 Starting spotraop version: v0.5.0 (Nov 10 2023 @ 22:13:28)
[22:31:30.872] main:1228

!!!!!!!!!!!!!!!!!! ERROR LOADING CONFIG FILE !!!!!!!!!!!!!!!!!!!!!

[22:31:30.873] Start:945 Cannot load SSL libraries
[22:31:30.873] main:1325 Cannot start, exiting

I didn't have a config, but not sure if it needs it.
I tried creating an empty config file, but still errors out.
What about the SSL libraries?

The config file is not a problem. I was not aware of that work from @GioF71. Normally, if you can find the ssl libraries, you could install them in the same directory. They are libssl.so and libcrypto.so. If you find such libraries with a slightly different name, let me know so I can include them.

Hi @GioF71, just did and it started as expected!
Thanks, I was afraid my vintage Synology Linux kernel was blocking me again, but it's working.
Works as advertised. My AirPlay (1 & 2) devices now show as Spotify Connect speakers, and also keep playing even without the initiating device.
Great stuff!

Biggest thank you should go to @philippe44 who put me in the right direction.
Consider starring the repos if that doesn't bother you :-)
Cheers


bug: new library path auto added to .libPaths() upon mounting a volume

Describe the bug
Mounting a package library path to host-site-library leads to the addition of "/home/rstudio/R/x86_64-pc-linux-gnu-library/4.1" as the first search term in the library path.
> .libPaths()
[1] "/home/rstudio/R/x86_64-pc-linux-gnu-library/4.1"
[2] "/usr/local/lib/R/host-site-library"
[3] "/usr/local/lib/R/site-library"
[4] "/usr/local/lib/R/library"

To Reproduce
Steps to reproduce the behavior:

Start container, and devel-lib is the location where I've been installing all my packages so far

docker run \
       --rm \
       -v /home/niteshturaga/devel-lib/:/usr/local/lib/R/host-site-library  \
       -p 8787:8787 \
       -e DISABLE_AUTH=true \
       bioconductor/bioconductor_docker:devel

check .libPaths() on R session

Expected behavior
The library search path should have only 3 entries.
> .libPaths()
[1] "/usr/local/lib/R/host-site-library"
[2] "/usr/local/lib/R/site-library"
[3] "/usr/local/lib/R/library"

Desktop (please complete the following information):

OS: Ubuntu
Browser: Chrome

This seems to happen when the directory is not available on the host machine. It needs to be created first.


[WIP] Don't use mock user in tests

This is a proof-of-concept of updating tests to use a real IndividualUser rather
than a MockK test double.

#2588  👈
main

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

Join @sgrimm and the rest of your teammates on  Graphite


Startup option -u is ignored in GUI mode

To reproduce:

Replace the contents of your ~/.vimrc with (or add to it somewhere) asdfsafsdfs.
Launch MacVim with mvim -u NONE -N.
Expected: MacVim launches and nothing happens.
Actual: MacVim launches and about 0.5-1s later this error is output to the launching terminal:

$ Error detected while processing /Users/andy/.vimrc:
line    1:
E492: Not an editor command: asdfsfsf

I get the same behaviour with mvim -u NORC -N.
However there is no error with non-GUI mode, i.e. mvim -v -u NORC -N – which the mvim script converts to /path/to/MacVim.app/Contents/MacOS/Vim -g -v -u NORC -N.
I'm using MacVim 7.4.1795 compiled via Homebrew.

Can't reproduce on latest snapshot from GitHub.
Custom Version 7.4 (101)

I can reproduce with the latest snapshot (101) from GitHub.

I can't reproduce either on a snapshot or the latest HEAD.

Wait, I get it, but only if I append the argument --help.
However, this is not a MacVim problem.  It happens in regular vim too, so it was introduced on their end and should be reported there.

Wait, scratch that.  I can't yet pin it on regular vim.
And I only get the issue intermittently.

Arrrgh, mistakenly introducing a custom wrapper into the picture.  When I use "stock" mvim, I cannot reproduce this issue at all.

I rebooted my computer and tried snapshot 101 again:
$ mvim -u NONE -U NONE -i NONE -N
<macvim gui launches...>

$ Error detected while processing /Users/andy/.vimrc:
line    1:
E492: Not an editor command: asdfasdf

The error message appears in the terminal about 1 second after the gui launches.
There's no error if I add the -v flag:
$ mvim -v -u NONE -U NONE -i NONE -N

Here are my MacVim's details:
$ mvim --version
VIM - Vi IMproved 7.4 (2013 Aug 10, compiled Apr  4 2016 02:34:27)
MacOS X (unix) version
Included patches: 1-1707
Compiled by<EMAIL_ADDRESS>Huge version with MacVim GUI.  Features included (+) or not (-):
+acl             +file_in_path    +mouse_sgr       +tag_old_static
+arabic          +find_in_path    -mouse_sysmouse  -tag_any_white
+autocmd         +float           +mouse_urxvt     -tcl
+balloon_eval    +folding         +mouse_xterm     +terminfo
+browse          -footer          +multi_byte      +termresponse
++builtin_terms  +fork()          +multi_lang      +textobjects
+byte_offset     +fullscreen      -mzscheme        +timers
+channel         -gettext         +netbeans_intg   +title
+cindent         -hangul_input    +odbeditor       +toolbar
+clientserver    +iconv           +packages        +transparency
+clipboard       +insert_expand   +path_extra      +user_commands
+cmdline_compl   +job             +perl/dyn        +vertsplit
+cmdline_hist    +jumplist        +persistent_undo +virtualedit
+cmdline_info    +keymap          +postscript      +visual
+comments        +langmap         +printer         +visualextra
+conceal         +libcall         +profile         +viminfo
+cryptv          +linebreak       +python/dyn      +vreplace
+cscope          +lispindent      +python3/dyn     +wildignore
+cursorbind      +listcmds        +quickfix        +wildmenu
+cursorshape     +localmap        +reltime         +windows
+dialog_con_gui  +lua/dyn         +rightleft       +writebackup
+diff            +menu            +ruby/dyn        -X11
+digraphs        +mksession       +scrollbind      -xfontset
+dnd             +modify_fname    +signs           +xim
-ebcdic          +mouse           +smartindent     -xsmp
+emacs_tags      +mouseshape      +startuptime     -xterm_clipboard
+eval            +mouse_dec       +statusline      -xterm_save
+ex_extra        -mouse_gpm       -sun_workshop    -xpm
+extra_search    -mouse_jsbterm   +syntax
+farsi           +mouse_netterm   +tag_binary
   system vimrc file: "$VIM/vimrc"
     user vimrc file: "$HOME/.vimrc"
 2nd user vimrc file: "~/.vim/vimrc"
      user exrc file: "$HOME/.exrc"
  system gvimrc file: "$VIM/gvimrc"
    user gvimrc file: "$HOME/.gvimrc"
2nd user gvimrc file: "~/.vim/gvimrc"
    system menu file: "$VIMRUNTIME/menu.vim"
  fall-back for $VIM: "/Applications/MacVim.app/Contents/Resources/vim"
Compilation: clang -c -I. -Iproto -DHAVE_CONFIG_H -DFEAT_GUI_MACVIM -Wall -Wno-unknown-pragmas -pipe  -DMACOS_X_UNIX  -g -O2 -U_FORTIFY_SOURCE -D_FORTIFY_SOURCE=1
Linking: clang   -L. -L/usr/local/lib -L. -L/usr/local/lib  -L/usr/local/lib -o Vim -framework Cocoa -framework Carbon       -lm  -lncurses -liconv -framework Cocoa   -fstack-protector  -L/System/Library/Perl/5.16/darwin-thread-multi-2level/CORE

Download snapshot-103, and put MacVim.app into /Applications.
$ rm -rf ~/.vim* ~/.gvimrc
$ echo "asdfasdf" > ~/.vimrc
$ /Applications/MacVim.app/Contents/MacOS/Vim -g -u NONE -U NONE -i NONE -N

still happing?

Thank you.  It's still happening...

@airblade you did exactly what I wrote? really?
$ rm -rf ~/.vim* ~/.gvimrc
$ echo "asdfasdf" > ~/.vimrc
$ /Applications/MacVim.app/Contents/MacOS/Vim -g -u NONE -U NONE -i NONE -N

At the risk of derailing this thread I would like to remark I have observed some very odd behaviour when running MacVim with the -u option, albeit not a quite the same as @airblade: when testing for a plugin conflict with the above option, I did get an uninitialised instance of MacVim *and, at times a second instance that had processed my regular vimrc. As it does not happen reproducibly I dismissed that as an annoyance, but this thread makes me wonder if MacVim somehow runs afoul of recent OS X versions’ rather aggressive process caching, maybe only in some configurations?

@splhack Yes, I did exactly that.  I just did it all over again with the same result: MacVim launches and 2s later this gets written out in the terminal:
$ /Applications/MacVim.app/Contents/MacOS/Vim -g -u NONE -U NONE -i NONE -N

<...2 seconds...>

$ Error detected while processing /Users/andy/.vimrc:
line    1:
E492: Not an editor command: asdf

@kopischke When I launch MacVim as per my previous comment, I see one MacVim process and two Vim processes.  Here's a screenshot from Activity Monitor:

@airblade yeah, I had up to three Vim processes running when experiencing the issue I described. I think we might be experiencing aspects of the same problem, and if is an OS X version / configuration / machine specific caching issue, it’s not surprising @splhack cannot reproduce.

You can use opensnoop what process opened and read /Users/andy/.vimrc.

let me know if it's still happening.


Drain: retain leadership setting of source broker

@lucaluo @alexism

LGTM

@nnalam @lucaluo
thanks for the PR. Change looks good, except for the Gemfile.lock: it is better to commit that file.
Please revert that part.
there are few posts on the web on that topic:
http://ryanbigg.com/2010/12/commit-it-or-else/
http://thelazylog.com/you-should-always-commit-gemfile-lock-and-schema-rb/
thanks
Alexis


Update HorizontalDiscreteColor example to show off stroke style properties

I thought that adding another legend might crowd the examples, so I changed the horizontal legend example to use the new stroke style properties. Let me know if you think this is overloaded and I can break it out!

happy to! :raised_hands:


Added tests for send and receive

I have added initial tests prior to changing/adding any new features. I added a test framework that allows tests for being run with "cargo nextest run" and show test result in a more concise manner compared to "cargo test".

I left a few comments. When I run the tests, all 5 are failing:
---- tests::connect_disconnect_test stdout ----
thread 'tests::connect_disconnect_test' panicked at 'Can't connect: Url(UnableToConnect("ws://<IP_ADDRESS>:9000/"))', tests/tests.rs:16:82

---- tests::send_binary_message_test stdout ----
thread 'tests::send_binary_message_test' panicked at 'Can't connect: Url(UnableToConnect("ws://<IP_ADDRESS>:9004/"))', tests/tests.rs:146:65

---- tests::receive_binary_message_test stdout ----
thread 'tests::receive_binary_message_test' panicked at 'Can't connect: Url(UnableToConnect("ws://<IP_ADDRESS>:9002/"))', tests/tests.rs:84:69
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace

---- tests::receive_text_message_test stdout ----
thread 'tests::receive_text_message_test' panicked at 'Can't connect: Url(UnableToConnect("ws://<IP_ADDRESS>:9001/"))', tests/tests.rs:55:69

---- tests::send_text_message_test stdout ----
thread 'tests::send_text_message_test' panicked at 'Can't connect: Url(UnableToConnect("ws://<IP_ADDRESS>:9003/"))', tests/tests.rs:114:65

Ohhhh, what environment do you run it in? As it is the connect step, I guess the bind went fine, so my guess it is a local firewall not allowing for connecting to localhost? Maybe using "localhost" may work over "<IP_ADDRESS>" - could you give that a try?

Hmm, doesn't seem to be the problem. localhost doesn't work in the test either. The server does get started though, I can confirm I can netcat localhost {UNIQUE_TEST_PORT}.

What OS/Build environment do you execute the cargo test command in?
As a narrow-down process, does it work if you disable your firewall?

Ah, the issue is that the tests are trying to connect before the thread running the server has had time to start up. For now, adding a short delay after all the calls to launch() in the tests will fix it - things seem to be working for me with a thread::sleep(500ms) after launch().

Ohhh, great you found the reason! Even though sleeping is never 100% stable, as one day the bind may take longer than that, for 99.9% of the time it will be sufficient, I guess. And with a low number of tests, I hope the total run time is acceptable?
Should I add the sleep to the PR?

Yeah, lets add the sleep to the PR. I've been planning some rework on simple-websockets' API for a while that would remove a lot of these rough edges, but until then let's just leave the delay :)

Looks good now. Thanks :)


npm install for 4.1.0 not working on macOS

Installing via npm i @breejs/later on macOS Monterey, after some time, it completes without error, but I get a node_modules/@breejs/later folder which only has three files - licence, package.json and readme.
If I install with npm i<EMAIL_ADDRESS>then I get a dist and lib folder created as well.
Is this something awry with packaging of latest on the npmjs repo, or is this something client-side that I can provide more info to debug? I'm running npm 8.2.0 at the moment.

For info, I've tried it on a Pi 3 running npm 6.11.3 as well as npm 8.2.0, and latest installed correctly.

I'm unable to reproduce and I'm guessing something was cached on your machine?  What version only had 3 files?
❯ npm i @breejs/later

+<EMAIL_ADDRESS>added 1 package from 2 contributors and audited 47 packages in 1.388s

4 packages are looking for funding
  run `npm fund` for details

found 0 vulnerabilities

❯ cd node_modules/@breejs/later
❯ ls
dist  lib  LICENSE  package.json  README.md

Sorry, I've changed jobs since I raised this, and don't have access to that Mac any more, so I can't test. I think it's unlikely to have been down to caching though, as it was a first-time install.


The panel:breakpoint event is not fired at init

It seems that the panel:breakpoint event is fired only after a resize of the window and not after app has initially loaded.
I'm using Framework7 Vue with param
panel: { 					rightBreakpoint: 1280 				}

I found the issue: I put display: none on  until it was loaded


Animation on elements' load

Add animations for elements that will always be changing or loading in.
This will make the overall user experience better and (more) polished.

Re-factor code that regards changing content of the DOM.
Push everything into the rightful places, then make the respective DOM content visible.
Instead of making it visible and then pushing items into place.

Loading sequence has been refined and is adequate for production version


Add pages on how to contribute a new data source

Will include pages on 3 different methods:

biothings SDK (done)
plover (in progress)
plater/automat (in progress)

each will probably have linkouts to team/tool-specific info

Convo from Slack:
Amy Glen (Expander Agent):

added initial version for Plover; it links to some placeholder sections in our repo, will get those updated soon… https://github.com/NCATSTranslator/TranslatorTechnicalDocumentation/blob/data-contributor-guide/docs/data-contributor-guide/plover.md

Evan Morris:

Plater needed quite a bit of updating, and we are very close to releasing a new version that would affect the docs as well, but I will get this done within the next week.

Hi @colleenXu  (cc: @andrewsu), I guess we were starting work on this objective concurrently. There's now a draft landing page: https://github.com/NCATSTranslator/TranslatorTechnicalDocumentation/blob/master/docs/development-guide/tutorials/component_builders_roadmap.md.  Please take a look and see how to cross-link the two

Update
Kelyne asked me to resolve the issue Richard brought up above (cross-linking). So I did this with my recent commits.
There are some pending updates to pages:

@amykglen Is your page ready?
@EvanDietzMorris Your page isn't filled out yet.

I imagine if this PR ends up merged before your updates are done, you'll have to make a new PR.

Sorry, I am still not finished with releasing our new Plater version which will have substantial changes to the Plater docs, so this got delayed. I should be able to get something in tomorrow, but if you want you could go ahead and merge this with a placeholder for Plater and I will make a new PR.

I dunno how urgent this is either :P, just wanted to check in on the status of things!
I imagine if you can get something done this week, that'd be great.

Hi @colleenXu , thanks for the contributions. I tweaked the commit a tiny bit - page locations, links, merging landing page (into 'Tutorials") but left the contributed markdown files intact. Merged the result.  Content now live at https://ncatstranslator.github.io/TranslatorTechnicalDocumentation/development-guide/tutorials/component_builders_roadmap/


FEATURE: Add Multi Modal Capabilities to Flowise

Update after touchup:

Hi, it seems like only GPT-4 Vision will be added in this pull. Will Ollama+LLaVA be added in the future? Ollama itself already supports LLaVA, but the chain does not. I wish this can be added along.

Hi, it seems like only GPT-4 Vision will be added in this pull. Will Ollama+LLaVA be added in the future? Ollama itself already supports LLaVA, but the chain does not. I wish this can be added along.

we'll first roll out chatopenai first, and move on to ollama

@vinodkiran pushed a couple of UI fixes:

error message when audio recording is not supported
messages not autoscrolling to the bottom

@vinodkiran @HenryHengZJ Made a couple of changes:

Change the UI for Speech to text configuration
Made messages in view message dialog consistent with internal chat

related chat embedded PR

@HenryHengZJ @chungyau97 Issues have all been fixed. We can review and merge this. I think everything's good to go on @vinodkiran's end too.

@0xi4o, thanks for the fix for invalid characters in file 2024-02-21 20_43_24-Elon Musk - Elon Musk.pdf — Mozilla Firefox.png

Another error where importing flows then turn on Speech To Text will remove file and mic logo.

Steps to reproduce error:

Click Add New
Import MultiModal chatflow
Save chatflow
Turn on Speech To Text

Another issue:
1.) Open OpenAI Whisper, put in credential:

2.) switch to assembly ai, you can see the openai credential there:

Another error where importing flows then turn on Speech To Text will remove file and mic logo.
Steps to reproduce error:

Click Add New
Import MultiModal chatflow
Save chatflow
Turn on Speech To Text

solved

Another issue:
1.) Open OpenAI Whisper, put in credential:
2.) switch to assembly ai, you can see the openai credential there:
I think that's because of the credentialNames in the useEffect, I've removed that as it caused infinite loop, but you were trying to put it there to prevent this scenario right

solved

Could there be an option to configure another audio or image model (selfhosted)?

Awesome feature. It would be great to add it to the chat embed as well.


Docs mention appender dateFile but doesn't exist

Informations

Type
Version

Question
5.4.1

Description
I tried to use the dateFile appender as the docs showed but got the error below.
Error
UNKNOW_APPENDER: Appender dateFile doesn't exists. Check your configuration:
{"type":"dateFile","filename":"log.log","pattern":".dd-MM-yyyy","compress":true}

Hello @SyntleLmao
It's a typo. Here the right example:
logger.appenders
  .set('file', {
    type: 'file',
    filename: `${__dirname}/../logs/myfile.log`,
    pattern: '.yyyy-MM-dd'
 });
```
I'll fix the documentation :)

Just tested it, that works, thank you.


Fix Fusion Source Creation

The fusionEventTypeId needs to be fetched and compared.
We also need to resolve all the right config values in the frontend client

Cool, meaning from now all the sources will  necessarily have fusionEvnetTypeId. Shall we also need to add the filter to other ensureSourceType in forntendClient


Rename Copy.Item.Tests.ps1 to match naming convention

PR Summary
Rename Copy.Item.Tests.ps1 -> Copy-Item.Tests.ps1
in order to match naming convention inside \test\powershell\Modules\Microsoft.PowerShell.Management
PR Context
All filenames in \test\powershell\Modules\Microsoft.PowerShell.Management use Verb-Command.Tests when naming their files. This one however does not match that standard.
PR Checklist

[X] PR has a meaningful title

Use the present tense and imperative mood when describing your changes

[X] Summarized changes
[X] Make sure all .h, .cpp, .cs, .ps1 and .psm1 files have the correct copyright header
[X] This PR is ready to merge and is not Work in Progress.

If the PR is work in progress, please add the prefix WIP: or [ WIP ] to the beginning of the title (the WIP bot will keep its status check at Pending while the prefix is present) and remove the prefix when the PR is ready.

Breaking changes

[X] None
OR
[ ] Experimental feature(s) needed

[ ] Experimental feature name(s):

User-facing changes

[X] Not Applicable
OR
[ ] Documentation needed

[ ] Issue filed:

Testing - New and feature

[X] N/A or can only be tested interactively
OR
[ ] Make sure you've added a new test if existing tests do not effectively test the code changed

Tooling

[X] I have considered the user experience from a tooling perspective and don't believe tooling will be impacted.
OR
[ ] I have considered the user experience from a tooling perspective and enumerated concerns in the summary. This may include:

Impact on PowerShell Editor Services which is used in the PowerShell extension for VSCode (which runs in a different PS Host).
Impact on Completions (both in the console and in editors) - one of PowerShell's most powerful features.
Impact on PSScriptAnalyzer (which provides linting & formatting in the editor extensions).
Impact on EditorSyntax (which provides syntax highlighting with in VSCode, GitHub, and many other editors).

We have many files not matching naming convention.
If we want have consistency we must fix everything or fix nothing.

I agree completely. However we cant just fix everything all at once.. That's why we need the approach fix it when you see it. :)

Mm true. I could see fixing the naming on each file in a given folder, doing it one folder at a time, perhaps?

I don't see a value from the change because we use modern IDEs with power and fast search, c# plugins allows us to follow by references, also sometimes we have many classes in one file - as result file name lost value but in the same time renaming complicates commit history.

... in the same time renaming complicates commit history.

I kinda agree with @iSazonov on that. Renaming a file will cause the file history to be "lost" (or to be more accurate, harder to retrieve). So if it's not necessary, maybe we should avoid renaming files.

Is it so frequent that we need to dig up file history?
The additional time needed to dig up file history is usually not immense, and not needed very often. I would be surprised if that's necessary anywhere near as much as folx need to go looking for the file. Time lost finding the file as it is now v.s. time lost finding the file as it was before the rename weighted depending on frequency of need.
I would argue that having a consistent naming convention will make maintenance easier both for the PS team and folx in the community, especially folx just starting to look at contributing. In my opinion the consistency is valuable here, but y'all may have different ideas. 🙂

Is it so frequent that we need to dig up file history?

If you want to get better understanding how a code works you will look history frequently.

The change in this PR has nothing to do with the CI failure, so I will force merge it.


kafkaClient.refreshMetadata callback never called

We are using KafkaClient refreshMetadata/send like below . But some times the callback is not getting called., due to which our consumer remains paused [we are calling async queue callback within] forever.
producerClient.refreshMetadata([producerTopic], function (error, response) {
        producer.send(payloads, function (err, data) {
             // Code here
           }
       );
    });

I can see that there is a return statement inside KafkaClient.refreshMetadata which might have caused this.
if (operation.retry(err)) {           return;   }
Can you please suggest what could be done in this situation

We're using the retry module and operation.retry is a way to check the operation will be retried so a return is necessary there. Have you turned on debugging to see if there's any helpful logs there?

This is happening once in a while .. i will enable debug logs today, but with the application logs it is verified that the callback is never called .. it is either refreshMetadata callback or send callback.

@hyperlink : I am getting below logs in debug mode in loop . Any idea ?
`Mon, 30 Oct 2017 23:37:27 GMT kafka-node:KafkaClient clearing test.com:9092 callback queue without error
Mon, 30 Oct 2017 23:37:28 GMT kafka-node:KafkaClient kafka-node-client reconnecting to test.com:9092
Mon, 30 Oct 2017 23:37:28 GMT kafka-node:KafkaClient Sending versions request to test.com:9092
Mon, 30 Oct 2017 23:37:28 GMT kafka-node:KafkaClient Received versions response from test.com:9092
Mon, 30 Oct 2017 23:39:17 GMT kafka-node:KafkaClient clearing test.com:9092 callback queue without error
Mon, 30 Oct 2017 23:39:18 GMT kafka-node:KafkaClient kafka-node-client reconnecting to test.com:9092
Mon, 30 Oct 2017 23:39:18 GMT kafka-node:KafkaClient Sending versions request to test.com:9092
Mon, 30 Oct 2017 23:39:18 GMT kafka-node:KafkaClient Received versions response from test.com:9092
Mon, 30 Oct 2017 23:39:28 GMT kafka-node:KafkaClient clearing test.com:9092 callback queue without error
Mon, 30 Oct 2017 23:39:29 GMT kafka-node:KafkaClient kafka-node-client reconnecting to test.com:9092
Mon, 30 Oct 2017 23:39:29 GMT kafka-node:KafkaClient Sending versions request to test.com:9092
Mon, 30 Oct 2017 23:39:29 GMT kafka-node:KafkaClient Received versions response from test.com:9092
Mon, 30 Oct 2017 23:41:18 GMT kafka-node:KafkaClient clearing test.com:9092 callback queue without error
Mon, 30 Oct 2017 23:41:19 GMT kafka-node:KafkaClient kafka-node-client reconnecting to test.com:9092
Mon, 30 Oct 2017 23:41:19 GMT kafka-node:KafkaClient Sending versions request to test.com:9092
Mon, 30 Oct 2017 23:41:19 GMT kafka-node:KafkaClient Received versions response from test.com:9092
Mon, 30 Oct 2017 23:41:29 GMT kafka-node:KafkaClient clearing test.com:9092 callback queue without error
Mon, 30 Oct 2017 23:41:30 GMT kafka-node:KafkaClient kafka-node-client reconnecting to test.com:9092
Mon, 30 Oct 2017 23:41:30 GMT kafka-node:KafkaClient Sending versions request to test.com:9092
Mon, 30 Oct 2017 23:41:30 GMT kafka-node:KafkaClient Received versions response from test.com:9092
`

What version of Kafka are you using?

We are using  kafka <IP_ADDRESS>. And the version of kafka-node we are using is 2.2.3

Could you checkout the code, setup docker (description in the README) and create sample code that replicates this issue?

@hyperlink : Thanks. I will try to do so. Can you please confirm, with the kafka-node 2.2.3 is it a good idea to call refresh metadata before producer.send ?

the callback queues are cleared when the socket is closed. There maybe a period of time before node notices that the socket is closed (TCP/IP sockets seems forgiving).

@hyperlink  : So i believe i need to figure out a work around to fix this.  Thanks for the response.
I can understand the need to refresh metadata (broker down, rebalancing, etc) , do i need to explicitly call refresh metadata before every send request ? I can see that in case of 'brokerNotAvailable' refresh metadata is called, but are other cases catered as well (like increase in the number of partitions ?)

@iamsahil1989 refreshing periodically seems reasonable and would catch the case when the number of partitions has been increased. Calling refresh before every send request seems overkill.

@hyperlink  : Thanks a lot :)


⚠ ClamAV has degraded performance

In 9bb0dfb, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 109 ms

Resolved: ClamAV performance has improved in 7bbe7b0.


Performance of methylation pipeline

Users reported slowdowns of methylation pipeline on trim_galore and bismark steps.

trim_galore uses a non-straightforward threading scheme:

https://github.com/FelixKrueger/TrimGalore/blob/master/Docs/Trim_Galore_User_Guide.md
https://github.com/FelixKrueger/TrimGalore/issues/16
https://github.com/bcbio/bcbio-nextgen/blob/master/bcbio/wgbsseq/trimming.py#L44

solution: with bcbio_nextgen.py -n 4 trim_galore runs cutadapt with 1 thread and it is very slow. Increasing bcbio treads speeds up trim_galore.

We tested bismark step with two threading parameters:
https://bcbio-nextgen.readthedocs.io/en/latest/contents/methylation.html#benchmarking
The currently recommended combination is 16/2/100G.

https://github.com/bcbio/bcbio-nextgen/pull/3303
https://github.com/bcbio/bcbio-nextgen/pull/3301
S

Some samples failed then extractor step:
bismark \
--bowtie2 \
--temp_dir . \
--gzip \
--parallel 16 \
-o . \
--unmapped /bcbio/genomes/Hsapiens/hg38/bismark/ \
-1 /path/bcbio/work/trimmed/val_1.fq.gz \
-2 /path/bcbio/work/trimmed/val_2.fq.gz \
-p 2

[FATAL ERROR:] The IDs of Read 1  and Read 2 are not the same.
This might be the result of sorting the BAM files by chromosomal position or merging several files with Samtools sort, and this is not compatible with correct methylation extraction. Please use an unsorted file instead or sort the file by name using the command 'samtools sort -n'. Paired-end files may be merged properly (without risking this error) using either 'samtools merge -n' or 'samtools cat'.

it seems to happen in some samples because we do
1. bismark alignment
2. sorting
3. deduplication
4. extraction

step 4 fails with sorted reads, but for 3 we have to sort. the solution is to skip sorting and deduplication or do step4 in a single-end mode (-s).

https://github.com/FelixKrueger/Bismark/issues/360

Lambda phage discussion: https://github.com/FelixKrueger/Bismark/issues/361

Finished the test cohort + vs Lambda genome.
4/2/100G (-n 16) passes without errors.
For the fastest processing use 16/2/192G (-n 32)


Fix typo in libsqlite flags variable

The build.rs script searches for LIBSQLITE3_FLAGS, but the README mentions LIBSQLITE_FLAGS (missing 3).

Sorry, opened from the wrong account.


Add issuer support as url

Description
Add issuer support as url, support also projectID alone for backward.
Must

[x] Tests
[ ] Documentation (if applicable)

you're assuming issuer format of descope.com/xxx/projectid
does that mean we will not be using subdomain?
and the issuer needs to be the whole URL, not just the project ID


monkey-patch deferred.run

... such that it logs the function and it's parameters.
See https://github.com/rogerthat-platform/rogerthat-backend/blob/master/src/add_1_monkey_patches.py#L257

3fa4973840c0fef64237bed0931d8a61ab519af7


Upstream packages version mismatch when using inertiajs + vue3 + SSR

Problem
When using inertiajs + vue3 + SSR, versions of the following package must be the same:
{
  ...
  "@vue/server-renderer": "^3.2.20",
  "vue": "^3.2.20",
  "@vue/compiler-sfc": "^3.2.20",
  ...
}

Suppose laravel-mix uses "vue": "^3.2.20",
user of laravel-mix can not use "@vue/server-renderer": "^3.2.29",
otherwise, some type errors will occur.
Workaround
Use the same version of vue and @vue/server-renderer as laravel-mix.
Solutions

(A) Maybe laravel-mix should use vue in peerDependencies?

This might be a bad solution, because it will limit laravel-mix users' vue version.

(B) Please help!

Note
In my case, if I use <slot> in vue,
I will meet the following type errors on mix watch --mix-config=webpack.ssr.mix.js:

The errors are due to version mismatch between vue and @vue/server-renderer.

✖ Mix
  Compiled with some errors in 4.43s

ERROR in /home/xyh/readonlylink/readonlylink-next/resources/js/pages/home/Index.vue.ts
6:4-11
[tsl] ERROR in /home/xyh/readonlylink/readonlylink-next/resources/js/pages/home/Index.vue.ts(6,5)
      TS2322: Type 'Function' is not assignable to type 'SSRSlot'.
  Type 'Function' provides no match for the signature '(props: Props, push: PushFn, parentComponent: ComponentInternalInstance | null, scopeId: string | null): void'.

ERROR in /home/xyh/readonlylink/readonlylink-next/resources/js/pages/home/Index.vue.ts
6:23-24
[tsl] ERROR in /home/xyh/readonlylink/readonlylink-next/resources/js/pages/home/Index.vue.ts(6,24)
      TS7006: Parameter '_' implicitly has an 'any' type.

... many more ...

webpack compiled with 6 errors

After more tests, I found the issue is not about "upstream packages version mismatch".
Sorry to open a wrong issue.


Failing test cases while setup

Checklist

[x] I've searched for similar issues.
[x] I'm using the latest version of HTTPie.

Minimal reproduction code and steps

gh repo clone httpie/httpie (in codespaces)
make install
make test-cover

Current result
282 failed, 736 passed, 4 skipped, 3 xfailed, 1 xpassed, 352 warnings in 125.12s (0:02:05) ==================================================
make: *** [Makefile:107: test] Error 1
…
Expected result
all test cases should pass successfully on the master branch
…

Debug output
Please re-run the command with --debug, then copy the entire command & output and paste both below:
$ http --debug <COMPLETE ARGUMENT LIST THAT TRIGGERS THE ERROR>
<COMPLETE OUTPUT>

Additional information, screenshots, or code examples

…

@jakubroztocil
can you pls help me on the subject
Best Regards,
Nishant

How which subject do you want
On Sat, 7 Jan 2023 8:54 am Nishant Sikarwar @.***>
wrote:

@jakubroztocil https://github.com/jakubroztocil
can you pls help me on the subject
Best Regards,
Nishant
—
Reply to this email directly, view it on GitHub
https://github.com/httpie/httpie/issues/1467#issuecomment-1374408967,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AWQ5KYT36UYWXGBSCCMZPVLWREOM3ANCNFSM6AAAAAATRQFFDU
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

@Chukwudum12
I meant the setup is failing in codespaces with the instruction mentioned in the README,  I have described above.
can you help in fixing the setup?
best Regards,
Nishant

Yes, I can help.
…

@Chukwudum12, can you check the above steps i followed and tell me where i made the mistake or what the issue is exaclty
Best Regards,
Nishant.

@Chukwudum12 and @jakubroztocil can you help on the setup pls
best regards,
Nishant

Yes, I can.
On Fri, 13 Jan 2023 6:01 am Nishant Sikarwar @.***>
wrote:

@Chukwudum12 https://github.com/Chukwudum12 and @jakubroztocil
https://github.com/jakubroztocil can you help on the setup pls
best regards,
Nishant
—
Reply to this email directly, view it on GitHub
https://github.com/httpie/httpie/issues/1467#issuecomment-1381317674,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AWQ5KYUVKBXHZOD222HJKL3WSDOULANCNFSM6AAAAAATRQFFDU
.
You are receiving this because you were mentioned.Message ID:
@.***>

@Chukwudum12 | @jakubroztocil
can you pls describe the steps to setup the project or tell me the mistake in the above steps that I might be making
best regards,
Nishant


Update disks-shared-enable.md

adding --resource-group per https://github.com/MicrosoftDocs/azure-docs/issues/89775

@DerekHerman-MSFT : Thanks for your contribution! The author(s) have been notified to review your proposed change.

#label:"aq-pr-triaged"

#sign-off


Continuous integration failing

Some tests do not pass, likely related to pandas updating to 2.0.0 (breaking changes).
Warning says that we are passing set as indexer to dataframe, which is no longer supported: https://github.com/pandas-dev/pandas/issues/42825
Try fixing it by requiring older pandas version in requirements.txt.
If this does not work, manually either convert the sets to lists in our code or patch Loc as suggested here: https://github.com/pandas-dev/pandas/issues/42825#issuecomment-1334633012

solved by requiring pandas<2.0


Removed clean-up step due to Jenkins sequencing

@mhines-usgs merging this without review because it does less than before, and I'm in a hurry to test it.

Coverage remained the same at 91.222% when pulling 21ee0e4624a1d64c8be6e6dbf15aeef56ae07e25 on ashalper-usgs:AQCU-936 into 97784cc04ba085e527d24e091c4f650fcc966c47 on USGS-R:master.