Deeply nested capturing lambda sets

Over the past couple weeks we've increasingly observed pathological compiler performance in the presence of deeply nested, capturing lambda sets. See #3449 for one report, and this Zulip thread for a longer discussion.
The current pathological performance occurs when there is a large-enough chain of lambdas called in sequence, and called in such a sequence that each lambda captures the next lambda in sequence (and by extension, its transitive closure). This issue is opened in order to form a technical plan for addressing this problem, and for us to coordinate work on tackling it.
As an example, take the lazy definition of Effect.after:
Effect.after = \@Effect effect, toEffect ->
  @Effect \{} ->
    when toEffect (effect {}) is
      @Effect thunk -> thunk {}

Notice that Effect.after returns an arrow that captures both its parameters effect and toEffect. Thus the following program
Effect.after (getLine) \line ->
    Effect.after (putLine line) \{} ->
        Effect.always {}

has the elaboration
Effect.after (getLine) \line -[f2]->          # Effect {} [thunk2 e2 f2] := {} -[thunk e2 f2]-> {}
    Effect.after (putLine line) \{} -[f1]->   # e2 = Effect {} [thunk1 e1 f1] := {} -[thunk1 e1 f1]-> {}
        Effect.always {}                      # e1 = Effect {} [always] := {} -[always]-> {}

Notice that e2 contains everything e1 contains, and the top-level lambda contains everything e2 contains. So, the total size of the lambda sets types (across the whole program) is quadratic in the depth of the largest chain of captures that capture other capturing lambda sets. That's too many "capture"s in a row, so let's just call this behavior a "lambda capture-chain".
You can see that the total size of the capture-chains will grow to be exponential in the presence of branches. One branch would make the total size 4^d, two branches would make the total size 8^d, etc, where d is the longest depth of a capture-chain.
The performance of this really is quite poor. As of #2226, the False interpreter now takes 2 seconds to compile on my 2021 M1. @Qqwy has observed much worse performance in #3449, where a parser combinator takes hours to compile, if it compiles at all.
Observations
I'll enumerate some observations I've previously made in investigating this problem in the context of #2226.

While it would seem that the largest problem here would be in type unification, I do not think this is the case. Indeed during monomorphization, type unification appears to complete just fine, no matter the size of the lambda set. As a simple test, I checked out https://github.com/rtfeldman/roc/commit/27c26284aadfb2282a46bc722f8fe04f594ec3cb and ran a version of the monomorphizer's layout generator that simply generates an arbitrary lambda set whenever it encounters a lambda set, and does not descend into recursive layout generation for the lambda set. With this change, the exponential increase in compile time observed on 27c26284aadfb2282a46bc722f8fe04f594ec3cb nearly disappeared, decaying into something that looks like a sublinear increase in compile time. Note that in this test, the same amount of type unification work is being done - only the amount of time generating layouts decreases.
Marten has observed large, nested, capturing lambda sets due to parser combinators causing the layout generator to OOM. This is more evidence that most of the work (memory-wise, at least) is going into layout generation.
We can explain type unification not being a significant cost factor in this problem by the union-find construction of the unification data structures. This greatly eliminates identical trees in the type unification forest and decays many unification checks to root pointer equality, especially after types are unified.
Today, the layout generator performs no caching or de-duplication of layouts for any type variable. This means that the runtime of layout generation for a type is at least linear in the total size of the type, whereas for unification it often is sub-linear (for the reasons described above).
Today, layout generation places generated layouts in a bump-allocated arena buffer. While this does not explain the exponential slowdown of compile times, this does mean that there is some extraneous work we may be doing by not packing related layouts more densely next to each other.
This problem is further exacerbated by how functions are specialized (monomorphized) across modules. Suppose the initial example I gave lives in an App module. In order to specialize the Effect.after calls, their specialization must be done in the context of all of the Effect module's specializations. This is because (today) only the Effect module can see all of its types, definitions, etc. This is done for parallelization reasons.
The consequence of this is that in order to specialize the nested Effect.after calls in our App module, the App module must export types those calls' types for specialization in the Effect module. In the example above, we ask for two Effect.after external specializations, and we independently export the wanted specialization types. In particular this means that no type variables are reused between the wanted specializations, even though many type variables between the two calls are exactly the same in the App module! And so, that means that the Effect module will see them as totally separate types as well. We've now blown up the forest of types quadratically in the number of trees, when before it may very well have been just a single tree.
Moreoever, when the Effect module goes to import the exported types, it does not preserve type variables in the domain being imported from between imports. For example, suppose the Effect module imports a : t1 and b : t1 -> t2 from the external specializations store. The Effect module will import them as a : t11, b : t31 -> t32 - note the relationship between t11 and t31 is lost in the image, though it was present in the domain.

Solutions
Here are some options I think will mitigate or eliminate this problem, in what I consider to be in decreasing order of effectiveness.

Layout caching - properly cache layouts based on the root pointer of a type variable in the type forest. This will reflect the unification forest structure in the layout generation. A natural question is when to invalidate the cache. The simple answer is "whenever we finish specializing a particular type", and while this is correct, this may reduce the effectiveness of caching in the current model of specialization - see point (3) below. There was also a prior attempt to cache layouts based on type roots that was unsuccessful, but I do not have context on why this was - @folkertdev and @rtfeldman may have more insight.

Note that caching might break down due to let-generalization, but let-generalization (or rather let-specialization) also causes disjoint trees in the unification forest, and that so far has not appeared to be a problem for the type solver. In general my hope is that if we can make layout generation look just like type inference in terms of data structures, this problem will dissipate.

Variable reuse in specialization storage - as mentioned above, we don't reuse variables when they are exported into Storage Subs (the storage for external specializations), and nor do we reuse variables when they are imported from Storage Subs for specialization into a module. We should do this, as it will avoid fragmenting the unification forest. With point (3) I think this will be especially powerful.
Sequence specializations - figure out a way to specialize functions with related types in sequence. In our original example, we'd like the Effect module to specialize the outermost Effect.after call, then the inner Effect.after, then the last Effect.always call in sequence, without any layout cache invalidation, in order to maximize reuse of cached layouts and avoid rewalking type trees. This is tricky to get right, but the fact that the type of Effect.after is nested in the type of the inner Effect.always which is nested in the type of the outermost Effect.after call gives us a starting point - at the very least you can walk type trees to determine the desired specialization order. How to do this more efficiently remains unclear in my head, but there is probably a good way, especially since the syntax tells us how we should expect the types to be nested.
Layout struct-of-arrays - I don't think this will affect the exponential layout-generation problems as much as the other suggested options, but it is an important thing to have regardless. By making the data representation more dense we can reduce cache misses due to distant pointer chases, even in a dense arena.

Finally, I'd like to note two things:

Even if these attempts ultimately do not improve the compilation speed of capture-chains, they are evident gaps in our compiler, and will improve the compile times of other programs with similar characteristics (basically, specializations that are similar or arise because of chained calls).
I have not mentioned how this affects parallelization of specialization within a module, if it all. That is mostly because today we do not parallelize the creation of specializations within a module whatsoever. However, especially if we can get idea (3) working, it gives us a natural partition for parallelization - specializations that must be computed in sequence are the equivalence groups of the partition.

Kicking this off today. I'm going to try to track work and various investigations for this in https://github.com/orgs/roc-lang/projects/1. Please let me know if you're interested in helping out with the investigation and resolution of this!

I've archived https://github.com/orgs/roc-lang/projects/1/views/1. After https://github.com/roc-lang/roc/pull/3981 lands, I would like to mark this project complete.


Implemented toast notification when new version of PWA is available and 'update' button to refresh on settings page (#85zrhdgkh)

Related Issues

Closes #
Short Description and Why It's Useful

When a newer version of our app is available, a toast will be shown to notify the user and an update button will appear on the settings page to refresh the app.
Screenshots of Visual Changes before/after (If There Are Any)

IMPORTANT NOTICE - Remember to add changelog entry
Contribution and Currently Important Rules Acceptance

[ ] I read and followed contribution rules

The refreshing variable takes care of multiple refreshes. (Mentioned in the reference- https://dev.to/drbragg/handling-service-worker-updates-in-your-vue-pwa-1pip)


Moderation file upload: Files aren't deleted when item finishes moderating

The files remain in the "WorkflowMessage" filestore bucket.

Fixed: verified using manual tesing


Changing color of liquid tags

Just a random question, but is it possible to change the default syntax colors?

You have to change the syntax theme and the color definitions in it.

@puranjayjain this probably also closed via #30 ?

yes we can close it


Support API Keys

In my Twilio project (in the Twilio console), we have the Account SID and Auth Token by default, but we use API Keys to hit the endpoints (for the basic auth portion).  Unfortunately, you can't use the API Key SID as the Account SID; you still have to provide the Account SID in the API endpoints.  I tried letting the Account SID be set by ENV var and then passing in the account and token options for the API key, but ExTwilio seems to take the account option as the account sid when generating the Endpoint URLs.
Does that sound right?  And how much effort do you think it would take to modify the library to allow you to pass the account SID as a config option using ENVs and passing the auth "username" as an option to the command (I'm still using ENV, but doing via Application.get_env/3)?
We may end up using subaccounts (inside a Twilio Project) for our customers, but API Key support would be nice for a more secure devops configuration.  (Thinking more like AWS IAM vs using AWS root credentials.)

I don't think this would necessarily require a lot of effort. Probably, you'd just need to extend the library to accept a new option, similar to account. I'm open to PRs for this!

I was wondering about this as well but, at least for sending SMS, this seems to be supported just by passing the :Authorization value as an option. Something like:
credentials = Base.encode64("#{api_sid:api_secret}")
ExTwilio.Message.create(to: "+1XXXXXXXXXX, from: "+1XXXXXXXXXX", body: "Totally auth'd via ExTwilio", Authorization: "Basic #{credentials}")

So maybe not first class support, but possible.

@objectuser unfortunately it seems there's a slight obstacle preventing this method from working with resource creation calls (such as ExTwilio.Message.create/2) as of today:
When Api.create/3 is called, it will always call auth_header/1 as seen below:
https://github.com/danielberkompas/ex_twilio/blob/00276cb855574f2dfd89f72f260a1d56136d48aa/lib/ex_twilio/api.ex#L68-L75
This, in turn, makes it so that our headers list in auth_header/2 is always empty:
https://github.com/danielberkompas/ex_twilio/blob/00276cb855574f2dfd89f72f260a1d56136d48aa/lib/ex_twilio/api.ex#L141-L144
Because of this, the Authorization header never gets overridden in the request.. :frowning_face:


Theme is lacking a QT stylesheet

The theme is lacking a QT stylesheet in order to look more like the sample screenshot provided.
As seen here, we can theme the rest of the UI using this.
I've made a very basic stylesheet based on this one by @thatlittleboy, I'll attach it to the issue. Feel free to use it. The colors are still very rough and don't really combine that well. This makes the UI much more consistent as seen in the screenshot below.

There are some issues with his stylesheet that are mentioned in his repo, maybe we as a community can find solution for it.
stylesheet.txt - GitHub didn't allow the .qss file to be uploaded, so the extension must be changed. The file must be placed in C:\Users\<your user here>\AppData\Roaming\texstudio in Windows.
I'll be more than glad to help with what I can!

The inline checking is missing, sch as underlined spelling, and grammar. even when selected it does not appear in the editor interface. Any suggestion how to solve this issue?


Check prettier on ci

[x] I have followed (at least) the PR section of the contributing guide.

Lets do this


Want to change e.g. README file without rebuilding and update lock file

Describe the problem you are trying to solve

I have a repository https://github.com/montao/spooks
That uses cargo for a crate. But if I want to change just the README and correct a typo, then I must rebuild it locally because it requires the lock file to regenerate. This makes it not possible to change the files of my project without regenerating the lock file. Therefore I cannot update the documentation e.g. the README because that requires the version in Cargo.toml to change and that requires in its turn that the lock file must change and I must shave the Yak locally.
Describe the solution you'd like

Make it easy just to make a small update, bump the version so that I can publish a new version of my crate without regenerating the lock file.
Notes

This is unfortunately necessary due to the way README is tied to the published crate file.  I'm going to close this in favor of the request at https://github.com/rust-lang/crates.io/issues/1750 which covers this, as this is a repository issue.


Contravariance Security fix

Contravariance Security fix
Summary:
Please read the desc of https://jira.mesosphere.com/browse/MARATHON-7974
I don't know if we will keep the Contravariance [-R] on Auth... however this definitely showed 5 locations in Marathon where security just plain didn't work.   The typing wasn't changed and there were no tests to confirm.     This PR fixes all but 1 issue which requires someone with more context of the code to fix.
Needs Fix needed for TaskController.  Line 129 authorizer.isAuthorized(identity, UpdateRunSpec, id)   can NOT be correct.   The "resource" is a PathId and needs to be a RunSpec.    Also it appears that it should have the DeleteRunSpec permission to me.   @alenkacz you were the last to write this code... could you that a look and update this branch?
JIRA issues: MARATHON-7974

reviewing now...

@alenkacz lgtm


Processed data link is broken

Hi,
This link does not work:
https://drive.google.com/open?id=1lqQmKflLisi-JBzFY0DL4sxqKtzZMrxt

It has been fixed

Awesome. Thanks!


No warning if order size input has any digit after decimal point

According to AC 7002-SORD-016 as a user I should see warning when putting too many digits after the decimal point in order size field.
Current implementation does not support creating orders with non integer values.
Reproduction steps:

Open Console app (e.g. https://console.fairground.wtf/)
Add non integer value e.g. 1.1
Try to place order.
-> Nothing happens (no order placed nor warning displayed)

Similar thing is happening with price, but no matter how many decimal places we put, it always go through and try to create an order, without any warning mentioned in 7002-SORD-016

I just found out that order size thing works correctly when market status is "Active". Price decimal validation does not work. Summary updated.


Driverless printing doesn't work with Pantum BM5100ADW Series (investigated and fixed)

Cc: @michaelrsweet, @tillkamppeter, @zdohnal
Dear colleagues,
I'm writing from Fedora 38 with cups 2.4.7, but the same issue exist for ALT Linux and ROSA Linux, so it seems to be common.
When Pantum BM5100ADW connected driverlessly, it scans perfectly well, but doesn't print. Attempt to print wakes up the printer, but job ends with "Print job canceled at printer" state.
The behaviour is the same regardless of how the device is connected: via network or via USB and ipp-usb. But on USB it causes more problems, because ipp-usb conflicts  with the proprietary driver (which prints and scans), so causing users to remove ipp-usb and complain.
After some investigations I've noticed the following facts:

Device supports (document-format-supported) image/urf,image/pwg-raster,application/octet-stream and prefers (document-format-default) image/urf
Cups sends image/urf to the device
The resulting image is RGB
Device rejects RGB images, while Grayscale images are printed perfectly well

After some deeper investigation I've found that generated PPD files are not the same when generated by driverless cat or by cups itself (and saved to /etc/cups/ppd/):

PPD file, generated by cups: Pantum_BM5100ADW_Series_2DFFA2.ppd.txt
PPD file, generated by driverless cat: driverless.txt

The PPD file, generated by driverless cat command works correctly: Grayscale image is generated and prints perfectly well, while PPD file, generated by cups, doesn't work.
There are many differences between these two files, but essential difference is the one single line:
*cupsUrfSupported: "V1.4,W8,IS1,CP99,PQ4,OB10,RS600,DM1"

This line present on driverless cat output and missed in the file, generated by cups.
Most likely, this problem affects not only Pantum but many other devices with the common feature to require image/urf format and to reject RGB image.
The simple patch to the cups/ppd-cache.c fixes the problem -- PR comes soon.

FWIW, the CUPS-generated PPD file only reports grayscale support, so if the filter is producing RGB then that is a serious bug in the filter.

FWIW, the CUPS-generated PPD file only reports grayscale support, so if the filter is producing RGB then that is a serious bug in the filter.

Agree. Anyway, the supplied PR provides a hot fix for the problem.
I suggest to merge in this simple patch, as it quickly solves the real problem and doesn't seem to break anything, and I can look to the cups-filters too (I want to familiarize myself with the Linux printing stack anyway).


Error: No default engine was specified and no extension was provided.

I cloned the repo and go this error when going to http://localhost:3000, any thoughts?
Error: No default engine was specified and no extension was provided.
    at new View (...mern-stack-authentication\node_modules\express\lib\view.js:61:11)
    at Function.render (...mern-stack-authentication\node_modules\express\lib\application.js:570:12)
    at ServerResponse.render (...mern-stack-authentication\node_modules\express\lib\response.js:1008:7)
    at ...mern-stack-authentication\app.js:40:7
    at Layer.handle_error (...mern-stack-authentication\node_modules\express\lib\router\layer.js:71:5)
    at trim_prefix (...mern-stack-authentication\node_modules\express\lib\router\index.js:315:13)
    at ...mern-stack-authentication\node_modules\express\lib\router\index.js:284:7
    at Function.process_params (...mern-stack-authentication\node_modules\express\lib\router\index.js:335:12)
    at next (...mern-stack-authentication\node_modules\express\lib\router\index.js:275:10)
    at ...mern-stack-authentication\app.js:29:3
    at Layer.handle [as handle_request] (...mern-stack-authentication\node_modules\express\lib\router\layer.js:95:5)
    at trim_prefix (...mern-stack-authentication\node_modules\express\lib\router\index.js:317:13)
    at ...mern-stack-authentication\node_modules\express\lib\router\index.js:284:7
    at Function.process_params (...mern-stack-authentication\node_modules\express\lib\router\index.js:335:12)
    at next (...mern-stack-authentication\node_modules\express\lib\router\index.js:275:10)
    at SendStream.error (...mern-stack-authentication\node_modules\serve-static\index.js:121:7)
    at SendStream.emit (events.js:182:13)
    at SendStream.error (...mern-stack-authentication\node_modules\send\index.js:270:17)
    at SendStream.onStatError (...mern-stack-authentication\node_modules\send\index.js:421:12)
    at next (...mern-stack-authentication\node_modules\send\index.js:764:28)
    at ...mern-stack-authentication\node_modules\send\index.js:772:23
    at FSReqWrap.oncomplete (fs.js:158:21)

nevermind, rtfd :)


Update unimod_mapper.py

This is a work around for user-defined xml files that have no unimod record ID. Happy to see alternative approaches / suggestions

Hi Toby,
thanks for the pull request. In principle that looks fine to, but just to clarify what the use case would be:
this would allow people to write their own unimod xml files (without unimod IDs included) and then use these mods as "normal" unimods in the modifications parameter list?
Wouldn't it be easier to just define those mods in the modifications uparam?
Like this (from docu/description):

Userdefined modifications need to have a unique name instead of
the unimod name the chemical composition needs to be given as a
Hill notation on the fifth position in the string
Example:
['S,opt,any,New_mod,C2H5N1O3' ]',

Those mods get added into the user-defined unimod xml which can then also be sent around for others to use, etc.

Hi Stefan,
Thanks for the feedback and, yes, that is the use cases. And yes your
suggestion was  was my original idea. But then I saw it was possible to
have a user-defined xml, I copied my mascot unimod.xml file over. That
way, any
and all modifications added by all users of our system would be included
and I don't have to copy them individually. I figured that's really what a
user-defined unimod xml is. So I blindly added the mascot-derived unimod
xml file and it didn't work. It didn't work because the user-defined.xml
file needs to have a unimod id, which of course cannot exist for
modifications using an in-house compound that binds to Cysteines or a
company standard way of calling SILAC modifications that is more
descriptive than the cryptic 13(L) notation in Unimod, for example.
would be good to hear what you think.
best
Toby
On Mon, 4 May 2020 at 16:49, StSchulze<EMAIL_ADDRESS>wrote:

Hi Toby,
thanks for the pull request. In principle that looks fine to, but just to
clarify what the use case would be:
this would allow people to write their own unimod xml files (without
unimod IDs included) and then use these mods as "normal" unimods in the
modifications parameter list?
Wouldn't it be easier to just define those mods in the modifications
uparam?
Like this (from docu/description):
Userdefined modifications need to have a unique name instead of
the unimod name the chemical composition needs to be given as a
Hill notation on the fifth position in the string
Example:
['S,opt,any,New_mod,C2H5N1O3' ]',
Those mods get added into the user-defined unimod xml which can then also
be sent around for others to use, etc.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/ursgal/ursgal/pull/145#issuecomment-623510085, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/ACOLXBAVB2BSS7BFDT4Y3PTRP3IZJANCNFSM4MYGTSEQ
.

I see, that makes perfect sense.
The only that I would change is to use "u123" for the user-defined unimod id instead of a number 10001+, since that would keep it consistent with our way of numbering user-defined mods so far and it would prevent overlap with actual unimod IDs (once they get above 10000).
In the future, the requirement for that unimod ID could probably also be removed fairly easily, but I would have to look closer into that.


doc: Adds missing t in collectByKey example

found this missing t in example code block

Hi @JrPribs! Thank you very much!


Error: Unable to authorize with the server

I've been using Magnus since it was released without any problems, but recently it started giving me this error message whenever I try connecting to our Rock server in VSCode:

Unable to authorize with the server

I have already verified that the Server URL, username and password are all correct, and that my IP address has been added to the Magnus plugin in Rock. I even tried removing the server in VSCode and re-added it. My user account in Rock has the Rock Admin security role.
This issue is happening on both my work and home computers, but both were working flawlessly before.
As far as I know, nothing has changed as far as our Rock server environment. I think the only things that may have changed would be either a VSCode update or an update to the Magnus VSCode plugin.
We're on Rock v13.7 with the latest version of the Magnus Rock plugin. My VSCode is currently at v1.74.2 with Magnus v1.0.2 installed.

Found the issue... My profile in Rock was merged with a duplicate record, and apparently when that happens, existing user accounts have their IsConfirmed flag reset to false. I just had to re-confirm my user account.


Optionally allow java to not be managed via the java cookbook.

Allows for conditional exclusion of the java cookbook's default recipe for those of us who use other methods to configure our java environments.

Seconded.

+1

+1


Support influx 0.8

After discussion with @bkutil we determined that the adapter will only support 0.9+ for now.


getContacts returns only a dozen contacts

async function run() {
  const api: Api = await connect({credentials: {username: "", password: ""}});
  for (const contact of await api.getContacts()) {
  console.log(contact);
 }
}

Returns only perhaps a dozen contacts. Do you know how a "contact" is defined by Skype? I have hundreds of contacts and I'm curious if I need to use paging or something to work through them all?
Thanks for this.

Hi,
Sorry for the long response delay.
It may be related to paging (not implemented) but from what I remember Skype has a relatively high limit. It can return about 50 results for my account.
It would help if you knew if the official Skype web app uses paging or not. Could you inspect your network tab for requests with the contacts/v2/users/<yourId> part? Does it perform multiple requests?
I'll double-check if there is some paging or not and then add support for it if it's the issue.


Fix length check in goLiveVectorParty.Equals.

Codecov Report

Merging #83 into master will increase coverage by 0.01%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master      #83      +/-   ##
==========================================
+ Coverage   75.33%   75.35%   +0.01%
==========================================
  Files          86       86
  Lines       13331    13331
==========================================
+ Hits        10043    10045       +2
+ Misses       2510     2509       -1
+ Partials      778      777       -1

Impacted Files
Coverage Δ

memstore/live_vector_party.go
70.77% <100%> (+1.29%)
: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 fcaa320...1dd00b8. Read the comment docs.

Please delete this branch, thx


Connection issues

PLC: <IP_ADDRESS>
AMSnetID PLC: <IP_ADDRESS>.1.1
Twincat 2
Raspberry Pi : <IP_ADDRESS>
Route has been created on PLC. Settings should be according tot tutorials
Error : "Failed to connect <IP_ADDRESS>.1.1:801 at startup: ClientException: Connection failed: Device system manager state read failed - ADS error -1 (Timeout - no response in 6000 ms)"

Can you set debugging level to 3 and copy paste the console output here?

I mean from Node.js side (terminal).

Not quite sure how to pull these up. i'm using the function blocks in NodeRed

Did you start node-red from a command line or terminal? If yes, then you will see it there.
If not, it's most probably logged somewhere else.
See
https://stackoverflow.com/questions/39243198/where-to-find-console-in-node-red
https://nodered.org/docs/user-guide/runtime/logging
The latter has a note:
Note : if you are running as a service on the Raspberry Pi using our install script, you can use the node-red-log command to view the service log.

i see targetAdsPort : 1000. Should this not be 801 on TC2 ? is configured as 801 in settings.

Unfortunately I don't know why it isn't working.  Are you sure that you don't have multiple connections already? See https://github.com/jisotalo/ads-client/issues/47
10000 is the port for system manager so that is correct.

Any news on this?

No, I tried multiple other settings but no effect I'm afraid.


About the color of the cropping border line

I configured the color in config.cropViewConfig.cropBorderColor, but it seems to have no effect after setting it. I wonder if it is supported?
Because my config.cropViewConfig.backgroundColor is set to white, in this case, the crop line and the background color will be the same, making it impossible to see whether it exists, but in fact I need it to be displayed prominently.
I would appreciate any help in resolving this issue.

Sample video :
https://github.com/user-attachments/assets/833097e3-7794-453f-9fd4-633a3f08b70f

@ByLiangCheng
config.cropViewConfig.cropBorderColor is for the border of cropped image like below, it is not for the cropping box whose border is always white currently.

Will you consider supporting the configuration of a cropping frame that is always white in the future? Because when my background is white, the cropping frame is always white, which makes it invisible.

@ByLiangCheng
Yes, I will consider supporting this configuration in the next release.

Thanks, looking forward to the update

I'm also looking forward for this update, I have similar issue with white background...

@ByLiangCheng @mihapp
Can you try PR #403 to see if it meets your requirement?
You may need to modify color properties of CropViewConfig.cropAuxiliaryIndicatorConfig

@ByLiangCheng @mihapp Can you try PR #403 to see if it meets your requirement?
You may need to modify color properties of CropViewConfig.cropAuxiliaryIndicatorConfig

Thank you for your support, I successfully solved it with this configuration

@ByLiangCheng
I released Mantis 2.22.0 which includes PR #403


ci: move macos jobs to nightly and cleanup protobug install in the CI

Pull Request type
Please check the type of change your PR introduces:

[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting, renaming)
[ ] Refactoring (no functional changes, no API changes)
[ ] Build-related changes
[ ] Documentation content changes
[ ] Other (please describe):

What is the current behavior?

Issue Number: N/A
What is the new behavior?

Does this introduce a breaking change?

[ ] Yes
[ ] No

Other information

This change is

Current dependencies on/for this PR:

main

PR #1585   👈

This stack of pull requests is managed by Graphite.


Not able to do reply thread with attachments in WS chat

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Action Performed:

Go to Profile setting
Create a new workspace
Then go to WS Chat.
Send an attachment
Try to reply in the thread.

Expected Result:
User should be able to  send reply in thread even for attachment thread
Actual Result:
Error message appears and not able to reply in thread
Workaround:
Can the user still use Expensify without this being fixed? Have you informed them of the workaround?
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android / native
[ ] Android / Chrome
[ ] iOS / native
[ ] iOS / Safari
[x] MacOS / Safari
[ ] MacOS / Desktop

Version Number: 1.3.24-3
Reproducible in staging?: y
Reproducible in production?: y
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
https://github.com/Expensify/App/assets/43996225/00fb26c0-6dfe-441a-b0ff-9cea639fc0d5
https://github.com/Expensify/App/assets/43996225/5ae07355-f9db-4ea4-84cb-5ebde5530982
Expensify/Expensify Issue URL:
Issue reported by: @niravkakadiya25
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1685707954067339
View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~0119e5938b32c3e46d
Upwork Job ID:<PHONE_NUMBER>091922944
Last Price Increase: 2023-07-04

I just tried to reproduce this issue on Safari 16.5, and I wasn't able to reproduce the issue. Creating the thread in the workspace chat worked perfectly. I'm adding needs reproduction and will leave open for a bit before closing it out.

Now onwards, Not able to see WS chat except #announcement and #admin @JmillsExpensify
https://github.com/Expensify/App/assets/100456857/03337470-3da7-429b-8a1c-dec7c6d76b53

@JmillsExpensify Here is it reproduces Steps and video as well

Goto Profile setting
Create a new WS
then go to WS Chat.
Send an attechments
Try to reply in the thread.

https://github.com/Expensify/App/assets/100456857/5ba427b4-30c4-4afd-bbfe-1276bc8a5fff

Thank you! Got the error this time, and based on the Auth error, I'm going to first triage internally.

@puneetlath mind double checking my thoughts above? You might also need to re-assign since your headed OOO soon.

Yeah, definitely sounds like an internal. Maybe it makes sense for @stitesExpensify or @chiragsalian to take over since it relates to threads.

Raised in NewDot.

Would appreciate your thoughts on the above!

@JmillsExpensify where did you bring this up in newdot? I think I might have missed that one?

Still internal and waiting to be picked up.

I'm not actively working on this issue. @stitesExpensify are you?

Sounds good, switched!

Its on my plate but my plate is swamped so I'm unsure when i can get to this. Should i just post in engg-chat looking for volunteer or are you okay if i tackle this slightly later?

Found the issue, its not really related to threads. Created threads from attachments works just fine.
The bug was in the logic of notifying participants and policy reports. When the user threads an attachment with themselves we were attempting to notify accountID 0 which threw the error. If other participants were in the thread it would work just fine.
Either way, i fixed the code to address this bug. Placing the PR for review in a bit.

Looks like the web PR was deployed to production last week, so is this fixed @chiragsalian ?

I'm pretty sure its resolved. QA even tested it on staging. Closing this issue, feel free to reopen if its not resolved.


[FLINK-4673] [core] TypeInfoFactory for Either type

Removes from TypeExtractor the explicit parsing for Either and adds an EitherTypeInfoFactory.
A test was removed that was expected to fail but now looks to succeed.

I will review this PR soon.


Syntax error in main.py

There is a small syntax error in the main.py file, which I can handle.

Hii
Can u assign me this issue

I'd like to take on this issue. I'll investigate the syntax error in main.py and provide a fix. Please assign this issue to me.


File parser

@iceboss3d Thanks for your time and effort, however I can't accept your pull request due to the following reasons:

You added a lot of opinionated things such as custom pages (Loans, AdminUsers, etc), authentication logic, backend calls, context containers, and so on.
You added API calls which are hardcoded to http://localhost:8080/ endpoint, so when I run the project using npm run serve it does not work properly (Login form does not work, other pages like Customers return error); Consider to use something like mirrorjs to emulate calls to the backend;

I would prefer to keep this project as simple as possible.


$meteor.camera on Android 5.0 & Chrome not working properly

Hi,
On Android 5.0 & Chrome, the modal is auto taking the picture without giving the user the choice to take a snapshot at a desired time.  i.e. when the modal pops up, the snapshot is already taken and the modal is populated.
The same code base is working fine on Ubuntu Chrome.
$ find . | grep mdg_camera ./.meteor/local/build/programs/server/packages/mdg_camera.js ./.meteor/local/build/programs/server/packages/mdg_camera.js.map ./.meteor/local/build/programs/server/assets/packages/mdg_camera ./.meteor/local/build/programs/server/assets/packages/mdg_camera/photo.html ./.meteor/local/build/programs/web.browser/packages/mdg_camera.js ./.meteor/local/build/programs/web.browser/packages/mdg_camera.js.map
I tried to make sense of ./.meteor/local/build/programs/web.browser/packages/mdg_camera.js but the code is terse.  If you guys give me some pointers I can try to root cause.  My Meteor knowledge is minimal but I can try.
Thanks,
-pavel

After a little digging, my previous description is not quite right.  There is no auto taking of the picture, it just appears that way as the video stream is just a single frame as opposed to being a real video stream.
i.e. On my Galaxy S4, when the modal pops up, you see a single still frame as opposed to a live video stream.
It's as if the video.play(); in mdg_camera.js is a nop on my Android device.  In fact, if I comment the video.play() out then I will see the same exact behavior on my desktop as I see in my broken Android case.

Adding autoplay, see last line, appears to be a workaround or a real fix in my situation:
.meteor/local/build/programs/web.browser/packages/mdg_camera.js:
Template["viewfinder"] = new Template("Template.viewfinder", (function() {                                      // 64
var view = this;                                                                                // 65
return [ HTML.DIV({                                                                                // 66
"class": "viewfinder"                                                                                // 67
}, "\n    ", HTML.VIDEO({                                                                                // 68
id: "video",
autoplay: "true",// 69
To be quite frank, I really don't know what I am doing :)
It would be great if someone smarter than me can opine on whether this is a legitimate issue and deserves a PR.
thanks,
-pavel

This is an issue with the underlying mdg:camera package and not angular-meteor.  Sorry about the noise, closing bug.

@ppotoplyak thanks for the persistence and the updates!


Unable to pin python version of extra dependencies

Here's a minimal example that requires isodate==0.6.0, but the latest version (0.7.2) gets installed instead:
import shiny.express
from shiny import render

import isodate

@render.text
def version():
    return isodate.__version__

## file: requirements.txt
## isodate==0.6.0

Looks as though this commit introduced the problem https://github.com/posit-dev/py-shinylive/commit/6df837fe035969b84ab35ffa450961764b4a46e9


Missing tags during select in influxdb_0.9.3-rc1

Issue: Tags are missing in 0.9.3-rc1 release during select, where as it is working on 0.9.2
=====
Steps to reproduce:
================

insert :  curl -i -XPOST 'http://localhost:8086/write?db=mydb' --data-binary 'cpu_load_short,host=server01,region=us-west value=0.64'
select * from cpu_load_short

Output on 0.9.3-rc1:
{
    results: [
    {
        series: [
        {
             name: "cpu_load_short",
             columns: [
                   "time",
                   "host",
                   "region",
                   "value"
              ],
              values: [
                  [
                        "2015-08-20T11:33:06.521791348Z",
                        "server01",
                        "us-west",
                         0.64
                  ]
               ]
            }
         ]
      }
   ]
}

Output on 0.9.2:
{
      results: [
      {
             series: [
             {
                     name: "cpu_load_short",
                     tags: {
                             host: "server01",
                             region: "us-west"
                      },
                      columns: [
                              "time",
                               "value"
                      ],
                      values: [
                            [
                                   "2015-08-20T11:52:57.85384834Z",
                                    0.64
                            ]
                      ]
               }
              ]
         }
     ]
}

Am I missing something?

@corylanou -- was this part of the work you did?

This is the correct behavior.  In 0.9.2 there was an implicit GROUP BY * on every SELECT * that didn't have a specified GROUP BY.
To get the same behavior as 0.9.2 in 0.9.3 you can issue this query:
SELECT * from cpu_load_short GROUP BY *
and it will give you tags.
If you issue the following:
SELECT * FROM cpu_load_short
the tags come back as columns in your results just like fields.
@otoolep we need to make this very clear in our release notes for 0.9.3

Done @corylanou https://github.com/influxdb/influxdb/commit/fa01abf3c992cb6b3e048c8d855de51815b36171

Closing as correct behaviour for 0.9.3.

I should provide a bit more of an explanation on why we did this. It was causing a problem because there were many cases where people wanted to do a SELECT * but they didn't want to do a corresponding GROUP BY *. Having that rewrite was causing a bunch of weird behavior for people.
Sorry about the change, we're hoping that it will actually enable more query functionality than was possible before.

Problem will  be if your application is aligned with 0.9.2 then it will not work with 0.9.3. If anyone want to upgrade Influx 0.9.2 to 0.9.3 then they need to change application code too. Please provide a detail information how it will improve overall product performance.

@buro1983 yes, unfortunately, it's a breaking change to the API. We really didn't want to make any, but felt it was better to get this done sooner rather than later. I also made an assumption that most people issuing SELECT * queries, were actually doing so from the CLI since it would be more common in code to actually select the columns you want explicitly.
This change wasn't about performance, it was about usability. Also, there were bugs that were introduced by automatically rewriting the query to GROUP BY *.

Thanks for the update. I will keep you guys posted if I observe any more issues


Date Annotation

In the Annotation Processor Dates are not supported. What's the intended way to get date and time into the db?
Best,
Alex

This will be a part of #475!
JFYI: annotation processor is totally optional and you can always write custom Put, Get and Delete Resolvers with needed behavior, you can mix generated and manual code if you wish!
So, at the moment I'll recommend write custom Get and Put resolver or store the field as long milliseconds.

If you don't mind, I'll close this issue in favor of #475. Feel free to comment here or (better) there #475!


Fix for 114923 - HTML figcaption tag should be suggested when using the full word

Fix for Issue - HTML figcaption tag should be suggested when using the full word instead of only "figc"

Thanks! Can you add a unit test for this case in expand.ts?

Fixes microsoft/vscode#114923.

I just checked and realized your new unit test fails. I'll leave it as an exercise for you to add in the right output (hint: it has a $ in it).
The easiest way to test the repository is to open it in VS Code, open the integrated terminal, and run yarn; yarn compile; yarn test. You'll also want to install yarn on your device if you don't have it yet.

Thanks again! 🎉


blue box styling off

As of 0e1c763936d26fa7f027875bc622570cb968bf54
blue boxes are too narrow on tablet
blue boxes look misaligned vert on desktop

Vertical box layout may look better on low tablet sizes.


What should the allowed action/capability for querying an index be?

I would say that the expected root capability for querying would be the same as the one for the /documents endpoint; you must be able to read all documents to be permitted to perform a query. So, if you have a capability to read from /documents you can use that capability for querying. I don't think it needs to be more complicated than that. And, IIRC, this is how current implementations handle it.
I think this is only a little bit tricky because of HTTP mechanics and the ever present debate over patterns/anti-patterns in RESTful architectures. To perform a query, we need to be able to send our query somewhere. But the query may be too large, or it just too awkward, to include that query in the URL itself as query parameters. This means using an HTTP body. But we already use POST <edvId>/documents for creating documents, so we need another endpoint ... which is what POST <edvId>/query is for. All we are really doing, however, is trying to do a GET <edvId>/documents with some parameters that filter what we're reading. This is why having the ability to "read" from <edvId>/documents fits with querying.

Discussed on July 8, 2021 WG call:

Come back to this after the updates to the ZCAP roots in the spec are done. (Sept-Oct 2021)


Fix broken headings in Markdown files

GitHub changed the way Markdown headings are parsed, so this change fixes it.
See bryant1410/readmesfix for more information.
Tackles bryant1410/readmesfix#1

@bryant1410 👍  although it misses the line before the changes.

Sorry! This was a bug which is now already fixed! It should fix Markdown headings only.


Class constructors that early return another object still require fields to be assigned

🔎 Search Terms
"constructor flow analysis", "constructor early return"
🕗 Version & Regression Information

This is the behavior in every version I tried, and I reviewed the FAQ for entries about constructors and flow control.

⏯ Playground Link
https://www.typescriptlang.org/play/?ts=5.7.0-dev.20240922&ssl=15&ssc=1&pln=1&pc=1#code/MYGwhgzhAEBiD29oG8BQ0PQCZgC5gC5oJcAnASwDsBzAblXU2HkpNIFdhd5SAKHfAH4ibKtQCUKRpkzkAZtF4BCAWEloZmzAHpt0XAAtyEAHSrs8AKYxK8XNEqXLWfUgBGl6JAjlqjlwaWpJbSWjp6kNDkuADkMGAgwWBYAJ5eUL7+0CzQACIAorAAggCqADIAKgD6sADytaFh0MG47KSUeYWllTX19E0Avo2YhsZmeGDQALzYE-2YQ0OozKz2BcXl1XW10w6WAO5wiLwxuZZyYOwg9gjwMeL0QA
💻 Code
class Foo {
    data: string;

    constructor(data?: string) {
        if (!data) {
            // this.data does not need to be assigned here
            // as it's already assigned on DEFAULT_FOO
            return DEFAULT_FOO;
        }
        this.data = data;
    }
}

const DEFAULT_FOO = new Foo('Default Foo');

🙁 Actual behavior
Property 'data' has no initializer and is not definitely assigned in the constructor.

🙂 Expected behavior
No error messages
Additional information about the issue
No response

Duplicate #27555


Configuring multiple plugins via the Command Center leads to confusion

Describe the bug
When creating/configuring multiple plugins via the Command Center and not saving and reloading the page in between, the Command Center will confuse the second with the first one when you click on the second plugin for further configuration.
To Reproduce
Steps to reproduce the behavior:

Go to the Command Center
Go to the module tab
Choose a module that is able to add some plugins
Add the first plugin
Add a second  plugin
Click on the second plugin, recently added
Unfortunately the configuration mask of the first plugin opens

Expected behavior
The configuration mask of the second plugin opens.
Environment:

Moryx.CommandCenter.Web Version 6.2.5

Supplement (09.01.2024)
There is another phenomenon that is probably based on the same problem. If I have configured several plugins and add another one, the order in the Config of MORYX is destroyed. If you have plugins with multiple versions, MORYX suddenly assigns the plugin to another configuration. This not only creates fatal errors, but also requires the entire configuration to be restored.

@seveneleven can you check whether this issue still exists after your updates?


Internet header: add a11y labels to the config object

To improve Internet Header's accessibility we need the following a11y labels to be added to the configuration object:

title for the meta navigation link group (#1339)

title for the skip link group (#1339)

-title for the breadcrumbs middle drop down menu (#1339 & #1358)

label for the active navigation link (e.g. "Aktiver Navigationspunkt") (#1341)

Did you inspect the current config? Most of the information should allready be there:

@alizedebray The skiplinks are not wrapped with a nav element and the ul element will have an aria-label attribute as soon as the PR #1402 is merged, therefore we do not need to add an aria-label.


Link to chart.js goes to highlight.js in README

😱 an issue by "THE" @tillnagel !!1!!
🥂 cheers to Mannheim - hope you're fine

Doing fine, thanks!
And doing my part in writing super important and complex issues. You know, helping out where I can. 🦸♂
But for real though: Cheers to Technologiestiftung for all your wonderful work!


I can't install python-jobspy on my venv python3.12.1

I can't install python-jobspy on my venv python3.12.1.
I'm getting the
AttributeError: module 'pkgutil' has no attribute 'ImpImporter'. Did you mean: 'zipimporter'?
error. I've tried many different methods but it always comes back to this.

creating an environment with different python version could be an alternative solution

Hey, try this out: https://stackoverflow.com/a/77364602
We only use Python 3.10 over here, so we hadn't ran into this issue.


Saving tags

How can we add the tags to the subscribber while creating new subscriber
$post_params = [
'EMAIL'=> $email_id,
'FNAME'=>$customer_data->getData()['First_Name'],
"LNAME"=>$customer_data->getData()['Last_Name'],
"KLANTNAAM"=>,
"KLANTNR"=>,
"INDUSTRIE"=>,
"TOER"=>,
'status'=>'subscribed'
	];

Should i need to add tags as key in above array ?
$mailchimp
->lists(LIST_ID)
->members()
->post($post_params);

Hello!
Good question. As I understand it Mailchimp tags are actually segments
so to add a contact to a tag you would interact with Mailchimp's segments
API. Here is a link to the segments API documentation:
https://developer.mailchimp.com/documentation/mailchimp/reference/lists/segments/
Best wishes,
Hutch
On Fri, Feb 1, 2019 at 1:15 PM pks891618<EMAIL_ADDRESS>wrote:

How can we add the tags to the subscribber while creating new subscriber
$post_params = [
'EMAIL'=> $email_id,
'FNAME'=>$customer_data->getData()['First_Name'],
"LNAME"=>$customer_data->getData()['Last_Name'],
"KLANTNAAM"=>,
"KLANTNR"=>,
"INDUSTRIE"=>,
"TOER"=>,
'status'=>'subscribed'
];
Should i need to add tags as key in above array ?
$mailchimp
->lists(LIST_ID)
->members()
->post($post_params);
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/Jhut89/Mailchimp-API-3.0-PHP/issues/55, or mute the
thread
https://github.com/notifications/unsubscribe-auth/AMn20NSeBbxCLPAOrjBEs_-DGzZdMpL5ks5vJIRHgaJpZM4ae05d
.

is it will work like this $mailchimp
->lists(LIST_ID)
->members()
->tags()
->post($tags);

Ah I think I understand what you are looking for. You would like to add a tag to a new contact in a POST to lists/{list-id}/members?
According to:
https://developer.mailchimp.com/documentation/mailchimp/reference/lists/members/#create-post_lists_list_id_members
It does appear that you should be able to:
$mailchimp
    ->lists(1234)
    ->members()
    ->post(
         // other params
         'tags' => [ // tag ids ]
    )

and as you have indicated above you could also make a quick followup request to tag a subscriber using:
https://developer.mailchimp.com/documentation/mailchimp/reference/lists/members/tags/
I hope that helps out! 🏷 (tag) 😉

$data=['tags'=>array("tag1")];
$mailchimp
->lists(LIST_ID)
<EMAIL_ADDRESS>->post(
// other params
$data
);
sorry but its not working , its giving 405 errors

ailchimpAPI\Responses\FailureResponse Object
(
[headers:MailchimpAPI\Responses\MailchimpResponse:private] => Array
(
[Server] => openresty
[Content-Type] => application/problem+json; charset=utf-8
[Content-Length] => 305
[X-Request-Id] => ac9dce7f-1b08-4176-b998-c049eb63de94
[Link] => ; rel="describedBy"
[Allow] => GET, DELETE, PATCH, PUT
[Date] => Fri, 01 Feb 2019 19:05:42 GMT
[Connection] => close
[Set-Cookie] => _mcid=1.1a98298eb2e94a492ceac6db140e94fd; expires=Sat, 01-Feb-2020 19:05:42 GMT; Max-Age=31536000; path=/; domain=.mailchimp.com
)
[body:MailchimpAPI\Responses\MailchimpResponse:private] => {"type":"http://developer.mailchimp.com/documentation/mailchimp/guides/error-glossary/","title":"Method Not Allowed","status":405,"detail":"The requested method and resource are not compatible. See the Allow header for this resource's available methods.","instance":"ac9dce7f-1b08-4176-b998-c049eb63de94"}
[http_code:MailchimpAPI\Responses\MailchimpResponse:private] => 405

)

I will go ahead and run a test once I get in front of my work machine to see if I can successfully hit that endpoint. For now though it's worth noting that the code you have above would put 'tags'=>array("tag1") into its own array in the body params and the documentation makes it look like tags is a top level index within the body of the request.

Thanks , i will really appericate if you can help me in checking  the actual function call to add the tags

Also in the most recent example you posted it looks like you are trying to hit an already existing subscriber. Which POSTing wouldn't be allowed to. Try calling the put() method instead.

yes after trying to use the put following response
MailchimpAPI\Responses\SuccessResponse Object
(
[headers:MailchimpAPI\Responses\MailchimpResponse:private] => Array
(
[Server] => openresty
[Content-Type] => application/json; charset=utf-8
[Content-Length] => 2947
[Vary] => Accept-Encoding
[X-Request-Id] => 6ee7f45e-5df3-47ed-b026-aa6a2e09a809
[Link] => ; rel="describedBy", ; rel="dashboard"
[Date] => Fri, 01 Feb 2019 19:48:17 GMT
[Connection] => keep-alive
[Set-Cookie] => _mcid=1.eab47e854f54e5c2283ef319d0cd09fb; expires=Sat, 01-Feb-2020 19:48:17 GMT; Max-Age=31536000; path=/; domain=.mailchimp.com
)
[body:MailchimpAPI\Responses\MailchimpResponse:private] => {"id":"7a2965e2dcdb00418cdd6c8a804ebcb9","email_address":"logistiek@mols.be","unique_email_id":"01e1e66ee7","email_type":"html","status":"subscribed","merge_fields":{"FNAME":"TESSA","LNAME":"CLAES","ADDRESS":"","PHONE":"","TOER":"B - Bruin","KLANTNR":"MOLS","INDUSTRIE":"TRANSPORT","KLANTNAAM":"MOLS HUURWAGENS nv"},"stats":{"avg_open_rate":0,"avg_click_rate":0},"ip_signup":"","timestamp_signup":"","ip_opt":"","timestamp_opt":"2018-11-23T11:35:01+00:00","member_rating":2,"last_changed":"2019-02-01T19:48:17+00:00","language":"","vip":false,"email_client":"","location":{"latitude":0,"longitude":0,"gmtoff":0,"dstoff":0,"country_code":"","timezone":""},"tags_count":0,"tags":[],"list_id":"78225e5f30","_links":[{"rel":"self","href":"https://us12.api.mailchimp.com/3.0/lists/78225e5f30/members/7a2965e2dcdb00418cdd6c8a804ebcb9","method":"GET","targetSchema":"https://us12.api.mailchimp.com/schema/3.0/Definitions/Lists/Members/Response.json"},{"rel":"parent","href":"https://us12.api.mailchimp.com/3.0/lists/78225e5f30/members","method":"GET","targetSchema":"https://us12.api.mailchimp.com/schema/3.0/Definitions/Lists/Members/CollectionResponse.json","schema":"https://us12.api.mailchimp.com/schema/3.0/CollectionLinks/Lists/Members.json"},{"rel":"update","href":"https://us12.api.mailchimp.com/3.0/lists/78225e5f30/members/7a2965e2dcdb00418cdd6c8a804ebcb9","method":"PATCH","targetSchema":"https://us12.api.mailchimp.com/schema/3.0/Definitions/Lists/Members/Response.json","schema":"https://us12.api.mailchimp.com/schema/3.0/Definitions/Lists/Members/PATCH.json"},{"rel":"upsert","href":"https://us12.api.mailchimp.com/3.0/lists/78225e5f30/members/7a2965e2dcdb00418cdd6c8a804ebcb9","method":"PUT","targetSchema":"https://us12.api.mailchimp.com/schema/3.0/Definitions/Lists/Members/Response.json","schema":"https://us12.api.mailchimp.com/schema/3.0/Definitions/Lists/Members/PUT.json"},{"rel":"delete","href":"https://us12.api.mailchimp.com/3.0/lists/78225e5f30/members/7a2965e2dcdb00418cdd6c8a804ebcb9","method":"DELETE"},{"rel":"activity","href":"https://us12.api.mailchimp.com/3.0/lists/78225e5f30/members/7a2965e2dcdb00418cdd6c8a804ebcb9/activity","method":"GET","targetSchema":"https://us12.api.mailchimp.com/schema/3.0/Definitions/Lists/Members/Activity/Response.json"},{"rel":"goals","href":"https://us12.api.mailchimp.com/3.0/lists/78225e5f30/members/7a2965e2dcdb00418cdd6c8a804ebcb9/goals","method":"GET","targetSchema":"https://us12.api.mailchimp.com/schema/3.0/Definitions/Lists/Members/Goals/Response.json"},{"rel":"notes","href":"https://us12.api.mailchimp.com/3.0/lists/78225e5f30/members/7a2965e2dcdb00418cdd6c8a804ebcb9/notes","method":"GET","targetSchema":"https://us12.api.mailchimp.com/schema/3.0/Definitions/Lists/Members/Notes/CollectionResponse.json"},{"rel":"delete_permanent","href":"https://us12.api.mailchimp.com/3.0/lists/78225e5f30/members/7a2965e2dcdb00418cdd6c8a804ebcb9/actions/delete-permanent","method":"POST"}]}

Great! 🎉  It looks like it worked.

[body:MailchimpAPI\Responses\MailchimpResponse:private] => {"id":"7a2965e2dcdb00418cdd6c8a804ebcb9","email_address":"logistiek@mols.be","unique_email_id":"01e1e66ee7","email_type":"html","status":"subscribed","merge_fields":{"FNAME":"TESSA","LNAME":"CLAES","ADDRESS":"","PHONE":"","TOER":"B - Bruin","KLANTNR":"MOLS","INDUSTRIE":"TRANSPORT","KLANTNAAM":"MOLS HUURWAGENS nv"},"stats":{"avg_open_rate":0,"avg_click_rate":0},"ip_signup":"","timestamp_signup":"","ip_opt":"","timestamp_opt":"2018-11-23T11:35:01+00:00","member_rating":2,"last_changed":"2019-02-01T19:48:17+00:00","language":"","vip":false,"email_client":"","location":{"latitude":0,"longitude":0,"gmtoff":0,"dstoff":0,"country_code":"","timezone":""},"tags_count":0,"tags":[],"list_id":"78225e5f30","_links":[{"rel":"self","href":"https://us12.api.mailchimp.com/3.0/lists/78225e5f30/members/7a2965e2dcdb00418cdd6c8a804ebcb9","method":"GET","targetSchema":"https://us12.api.mailchimp.com/schema/3.0/Definitions/Lists/Members/Response.json"},{"rel":"parent","href":"https://us12.api.mailchimp.com/3.0/lists/78225e5f30/members","method":"GET","targetSchema":"https://us12.api.mailchimp.com/schema/3.0/Definitions/Lists/Members/CollectionResponse.json","schema":"https://us12.api.mailchimp.com/schema/3.0/CollectionLinks/Lists/Members.json"},{"rel":"update","href":"https://us12.api.mailchimp.com/3.0/lists/78225e5f30/members/7a2965e2dcdb00418cdd6c8a804ebcb9","method":"PATCH","targetSchema":"https://us12.api.mailchimp.com/schema/3.0/Definitions/Lists/Members/Response.json","schema":"https://us12.api.mailchimp.com/schema/3.0/Definitions/Lists/Members/PATCH.json"},{"rel":"upsert","href":"https://us12.api.mailchimp.com/3.0/lists/78225e5f30/members/7a2965e2dcdb00418cdd6c8a804ebcb9","method":"PUT","targetSchema":"https://us12.api.mailchimp.com/schema/3.0/Definitions/Lists/Members/Response.json","schema":"https://us12.api.mailchimp.com/schema/3.0/Definitions/Lists/Members/PUT.json"},{"rel":"delete","href":"https://us12.api.mailchimp.com/3.0/lists/78225e5f30/members/7a2965e2dcdb00418cdd6c8a804ebcb9","method":"DELETE"},{"rel":"activity","href":"https://us12.api.mailchimp.com/3.0/lists/78225e5f30/members/7a2965e2dcdb00418cdd6c8a804ebcb9/activity","method":"GET","targetSchema":"https://us12.api.mailchimp.com/schema/3.0/Definitions/Lists/Members/Activity/Response.json"},{"rel":"goals","href":"https://us12.api.mailchimp.com/3.0/lists/78225e5f30/members/7a2965e2dcdb00418cdd6c8a804ebcb9/goals","method":"GET","targetSchema":"https://us12.api.mailchimp.com/schema/3.0/Definitions/Lists/Members/Goals/Response.json"},{"rel":"notes","href":"https://us12.api.mailchimp.com/3.0/lists/78225e5f30/members/7a2965e2dcdb00418cdd6c8a804ebcb9/notes","method":"GET","targetSchema":"https://us12.api.mailchimp.com/schema/3.0/Definitions/Lists/Members/Notes/CollectionResponse.json"},{"rel":"delete_permanent","href":"https://us12.api.mailchimp.com/3.0/lists/78225e5f30/members/7a2965e2dcdb00418cdd6c8a804ebcb9/actions/delete-permanent","method":"POST"}]}
sorry but it dont add the tags

Ok cool I'll have to check this out once I'm in front of my machine.

sure , no problem .thanks for the help

Hello! I finally got home to my computer and here is what I found.I was sucessful in tagging a subscriber using this endpoint with the following code:
https://developer.mailchimp.com/documentation/mailchimp/reference/lists/members/tags/
$mailchimp
    ->lists('1a2b3c4d')
<EMAIL_ADDRESS>    ->tags()
    ->post([
        'tags' => [
            [
                'name' => 'tag1',
                'status' => 'active'
            ]
        ]
    ]);

I was also successfully able to post a new contact with a tag by doing:
$mailchimp
    ->lists('1a2b3c4d')
    ->members()
    ->post([
        'email_address' =><EMAIL_ADDRESS>        'status' => 'subscribed',
        'tags' => [
            'tag2'
        ]
    ]);

I tried to PUT tags to an existing subscriber but found it doesn't look like tag have been implemented for:
https://developer.mailchimp.com/documentation/mailchimp/reference/lists/members/#edit-patch_lists_list_id_members_subscriber_hash

great  it works

I have'nt found a way to do that yet. For now it looks like Mailchimp will require you to get all the existing tags on a contact and then post with those plus the new ones.

When we are trying to add new then existing tag remains there ,
basically i want to update the existing tag , the new tag must overwrite the existing .
e.g if a contact have "city" tag then if we want to add the new tag country and state then the final tag will be state and country , city need to be overwrite with the new once


pbx file to big

I followed this process but got this error - Message: File upload has timed out. Reduce the file size and try again.
The model is 2.5G. When I try a really small model it works fine. I'm on the B1 pricing tier. Are there any limits on that tier? Or how can I upload larger PBX files to Azure Analysis Services?

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

ID: c804e09a-5163-7f39-82ec-67734ffe9741
Version Independent ID: 52757fd4-4c69-2a53-5b9c-f32404efbf13
Content: Import a Power BI Desktop file into Azure Analysis Services
Content Source: articles/analysis-services/analysis-services-import-pbix.md
Service: azure-analysis-services
GitHub Login: @Minewiskan
Microsoft Alias: owend

@Thank you for the valuable feedback,we are investigating the issue.

@jackman0200 The requests will be timing out after 3 minutes. If your files are large or you are on a slow network, they may take longer to load. You could try removing most of the data to make the file smaller and hopefully quicker. Then you can always process the data back in after the model has been imported. We are looking at ways to extend the timeout. This is currently only supported on Standard or Developer tiers. It is not yet available on basic. We are planning to add it there very shortly.

Thanks so much! Would you mind pointing me to a post on how to increase that timeout on a standard or dev tier?
From: VairavanS-MSFT<EMAIL_ADDRESS>Sent: Wednesday, July 18, 2018 5:06 PM
To: MicrosoftDocs/azure-docs<EMAIL_ADDRESS>Cc: Justin Ball<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [MicrosoftDocs/azure-docs] pbx file to big (#11943)
@jackman0200https://github.com/jackman0200 The requests will be timing out after 3 minutes. If your files are large or you are on a slow network, they may take longer to load. You could try removing most of the data to make the file smaller and hopefully quicker. Then you can always process the data back in after the model has been imported. We are looking at ways to extend the timeout. This is currently only supported on Standard or Developer tiers. It is not yet available on basic. We are planning to add it there very shortly.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/MicrosoftDocs/azure-docs/issues/11943#issuecomment-406101316, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AnehqkDTnSBSKXaAtsqKPLNP3zsxOuT5ks5uH79xgaJpZM4VVTEW.

On the last reply I got this:
“We are looking at ways to extend the timeout. This is currently only supported on Standard or Developer tiers. It is not yet available on basic. We are planning to add it there very shortly.”
So does that mean that the ability to extend the timeout will be available on Standard or Developer shortly? I initially read that to mean we can change it on Standard and Developer but not Basic, but it will be available on Basic soon.
From: VairavanS-MSFT<EMAIL_ADDRESS>Sent: Thursday, July 19, 2018 11:29 AM
To: MicrosoftDocs/azure-docs<EMAIL_ADDRESS>Cc: Justin Ball<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [MicrosoftDocs/azure-docs] pbx file to big (#11943)
@jackman0200https://github.com/jackman0200 You cannot increase the timeout, that is a thing the product team is working on. But with standard or developer tier, you could try removing most of the data to make the file smaller and hopefully quicker. Then you can always process the data back in after the model has been imported.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/MicrosoftDocs/azure-docs/issues/11943#issuecomment-406354422, or mute the threadhttps://github.com/notifications/unsubscribe-auth/Anehqu6f0VDt8wUzGz3q3xnzBaLtFQT6ks5uIMHQgaJpZM4VVTEW.

@jackman0200 Sorry for the confusing statement. I mentioned the workaround which I specified is currently available in standard or developer tiers not the increasing timeout.

Ok, thank you, that makes sense. I moved over to the standard tier.
From: VairavanS-MSFT<EMAIL_ADDRESS>Sent: Thursday, July 19, 2018 1:36 PM
To: MicrosoftDocs/azure-docs<EMAIL_ADDRESS>Cc: Justin Ball<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [MicrosoftDocs/azure-docs] pbx file to big (#11943)
@jackman0200https://github.com/jackman0200 Sorry for the confusing statement. I mentioned the workaround which I specified is currently available in standard or developer tiers not the increasing timeout.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/MicrosoftDocs/azure-docs/issues/11943#issuecomment-406389873, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AnehqmV8QglzObKtLRKPdrVAkEQJaEk_ks5uIN-TgaJpZM4VVTEW.

@jackman0200 We will now proceed to close this thread. If there are further questions regarding this matter, please reopen it and we will gladly continue the discussion.


fix: Update helm Neo4j deployment

Summary of Changes
Use emptyDir for Neo4j deployment plugin volumes instead of hostPath.
With this change we get rid of the dependency on hostPath and thus the dependency on the hosts filesystem and pods permissions to the hostPath.
The custom Neo4j deployment can now also run on GKE (and other k8s clusters).
I also sorted the config options for for Neo4j and disabled auth. The disabling of Neo4j auth seems to be a consistent setting through this repo (docker examples, and ECS examples).

cc @javamonkey79


Allow image export to generate sboms

Allow quick sbom generation for the image export workflow, which can be followed up by a common documentation update for both disk_export and image_export workflows.

/cc dorileo
/cc zmarano


dotnet sdk 7.0.200 breaking change in latest runner

Description
A breaking change from the dotnet sdk has been released as a runner image causing azure pipeline builds to fail when using --output on a solution build
https://github.com/dotnet/sdk/issues/30625
Platforms affected

[X] Azure DevOps
[ ] GitHub Actions - Standard Runners
[ ] GitHub Actions - Larger Runners

Runner images affected

[ ] Ubuntu 18.04
[ ] Ubuntu 20.04
[ ] Ubuntu 22.04
[ ] macOS 11
[ ] macOS 12
[ ] Windows Server 2019
[X] Windows Server 2022

Image version and build link
20230226.1
Is it regression?
20230219.1
Expected behavior
build a solution file with a specific --output location succeeds
Actual behavior
build a solution file with a specific --output location errors
Repro steps
build a solution file with a specific --output location and run via pipelines
as not all are updated to the latest image this fails inconsistently

Hello @nt-evander, could you please provide minimum repro steps?

It will be a part of the next image release

I've made a solution with some dotnet new commands and included the yaml for the pipeline to confirm
Build-Validation-Testing.zip

Is the new image already being rolled out, cause my builds are already failing due to this. I can't see an update to the image documentation though, so I don't know what to expect. It looks like it's being rolled out presently, cause the builds are failing sometimes - depending on whether version 7.0.200 or 7.0.103 is on that particular box.
Docs are saying windows-2022 has these versions installed: .NET Core SDK: 3.1.120, 3.1.202, 3.1.302, 3.1.426, 6.0.114, 6.0.203, 6.0.309, 6.0.406, 7.0.103. https://github.com/actions/runner-images/blob/main/images/win/Windows2022-Readme.md
Honestly.. removing a cli option in a patch.. smh.

Hello, @gautelo! New image will be rolled out next week. Thank you!

Very strange that it's already broken then @ddobranic. I wonder why that is. Perhaps the agents aren't cleaned appropriately after somebody else installs it explicitly? If so, that seems like a bit of an issue as well. Or how would you explain why the Microsoft hosted Azure Pipeline agent pool has that version of dotnet on it without me installing it explicitly?

This is occurring in GitHub Actions - Standard Runners as well.

Can confirm also hit by this on GitHub Actions - Standard windows-latest runner.
Runner Info:

Pack Step:

They have released version 7.0.201 with a fix for this issue.
Is it possible to update images with this version?

For my specific build needs, I was able to just build with the Windows 2019 runner. 🤷♂

I had to force the 7.0.201:
- name: Setup dotnet
  uses: actions/setup-dotnet@v3
  with:
    dotnet-version: '7.0.201'

@Dreamescaper updated image will be released this week.

Deployed


Clarify docs for segregated property

Connects to https://github.com/mapzen/vector-datasource/pull/760/files/a32c8a6ee2a0eab89638d5a13a218fe30731e2ed#r61586659
@nvkelso this look ok? Or did you have something else in mind?

Looks like the most common values are yes and no. Do we strip the no values? If so, probably worth just saying something like "Present on paths allowing both pedestrian and bicycle traffic with the value yes when pedestrian traffic is segregated from bicycle traffic."

At the moment we're only just exposing the value on the feature. Should we only include it if its value is yes?

Yeah, agree we should be stripping the no value (the property should only be present when it's se to yes). That's more in line with what we do for oneway, etc.
Something like this for docs? (Riffing on Matt's proposal):
Set to `yes` when a path allows both pedestrian and bicycle traffic, but when
pedestrian traffic is segregated from bicycle traffic.

Is this property on more highway types than path?

Is this property on more highway types than path?

It's currently on all road types, since it could appear anywhere there's a segregated foot/cycle path, including on the sidewalk of another type of road. But in North America, it only appears on these highway values:
   highway    | num
--------------+------
 cycleway     | 1556
 path         |  244
 footway      |   84
 service      |   10
 tertiary     |    6
 construction |    5
 pedestrian   |    5
 secondary    |    4
 trunk        |    3
 residential  |    3
 track        |    3
              |    2
 steps        |    1

I am intrigued as to where the segregated steps are, though...

Haha :)
I think it's okay to say "path" in this context (instead of road or way), since we're already grouping (cycleway,  path, footway) into kind:path and that's where almost all the tags are.

Updated the logic in 3cdbc9a, and the docs in 02ce1d5.

Neat use of expression in the YAML!

Oh, I thought we were stripping out all the tags with the value no later on in the pipeline... I must have been mistaken.


Copy paste files from native explorer to vscode

Issue Type: Feature Request
its impossible for me to paste files(like fonts into my flutter app), from a folder in the windows explorer.
i can drag and drop it but i have to create a different file and then drag and drop it
VS Code version: Code 1.41.1 (26076a4de974ead31f97692a0d32f90d735645c0, 2019-12-18T14:58:56.166Z)
OS version: Windows_NT x64 10.0.17763

This is a fair feature request however I believe there is a Chrome limtation which is preventing us from achieving this.
Thus assigning this to the backlog milestone

Is this a duplicate of #130036 (subtask of #164)?


collapsed by default

Updated build tools to 1.0 RC1 defaults
Added xml attribute to set collapsed by default in the list view xml.
Also added to sample.

收到

Hello，我已收到邮件，待我仔细阅读后再回复。


Error - [spfx-serve] Error: Invalid glob argument:

Hi
Am getting the error while running gulp serve. I reinstalled it with latest version gulp 4.0.0. But still the error occurs.

Could you please help me to resolve this. .

Does it repro with the latest stable gulp (3.9.1)?

Close this issue for long time no response. Feel free to contact us to reopen it.


[FEATURE] - Install completion at build time

#45 have been merged for now, but ultimately this should be done at build time and installed when packaging the application :
see https://github.com/sharkdp/bat/blob/master/.github/workflows/CICD.yml

FYI in Arch I install them into the system like this: https://github.com/archlinux/svntogit-community/blob/1243f22ab98092f550c6a92015ecf12ef75d373b/trunk/PKGBUILD#L28


Hello. Is it necessary to install SLURM in order to run the experiments?

@jesuisalexjamet

Hello,
I reckon SLURM would speed-up the process of obtaining the results. However, manually running the experiments is also a possiblity. I guess to achieve that, you'd need to modify the SLURM scripts I have provided in the repo.
Best,
Alex.

Hello. I have installed SLURM, but when I run ./scripts/run_single_core.sh, I encounter the following error. Do you know how to solve it? Also, do you know how much memory is required for the experiment? @jesuisalexjamet

Hello, that problem seems to be due to the machines being used by SLURM to not have enough memory at their disposal. I would suggest consulting the man sbatch and looking up the arguments relative to the memory allocation.
I personally never encountered this issue, so I don't think I will be of great help here.


Fix the show rule effect for bibliography heading level

Fixes #4618.

Using the OP's example in #4618:
#show heading.where(level: 1): set text(fill: red)

#show outline: set heading(level: 2)
#show bibliography: set heading(level: 2)

#outline()

= Part One

= Part Two

#bibliography("/assets/bib/works_too.bib")

Before
After

Output

Thank you!


⚠ Secret Website 3 has degraded performance

In e94a5d5, Secret Website 3 ($SECRET_WEBSITE_3) experienced degraded performance:

HTTP code: 200
Response time: 6257 ms

Resolved: Secret Website 3 performance has improved in 75e790b.


Formatting Response Data

Fixes #122
@Rick-Anderson please review

@danroth27  can you assign a tech review? I'm doing an edit review.

@javiercn Please review

@ardalis  nice article :watch:

@javiercn can you give us a ETA on when you can review or suggest another dev to review?

:watch:

:watch:
@javiercn @danroth27

:up: :date:
@danroth27
(I added a :watch: 5 days ago to indicate I was watching to see if there were any more review comments - though I missed the build failure and didn't get that fixed until just now).

yo @danroth27 can I shipit and we can address any issues afterward?

@danroth27 ping

@kichalla Please review

:shipit:


One image (at least) produces a hash(64) value of "0" ...

First bug... :-)
var Jimp = require('jimp');

var imageName = 'ts.jpg';

Jimp.read(imageName, function(err, image) {
  if (err) {
    return console.error('Jimp can\'t read image:', err);
  }
  var hash64 = image.hash(64);
  if (hash64.length !== 11) {
    return console.error('hash value ('+hash64+') length is', hash64.length, 'instead of 11!');
  }
  console.log('hash64:', hash64);
});

file "ts.jpg" is: https://upload.wikimedia.org/wikipedia/commons/thumb/4/4d/Taylor_Swift_112_(18119055110).jpg/250px-Taylor_Swift_112_(18119055110).jpg
All other images I am testing all return a hash lenght of 11 (why 11? :-)

Caught the issue with Taylor. See latest in git. Will publish this evening after testing.
With regards to why the hash is 11 digits long, I suspect the reason is chance. Example: Pick random a number between 0 and 9999999. Nine out of ten times, it will be 7 digits long. One way of thinking about this is because the chance of the leading digit being zero is just 1 in 10 decimal. In base 64, the chance of the leading digit being zero is 1 in 64.

This fix is published to NPM now. Thanks for catching it.


CLI needs --help command

Would be great to have a command that prints help.
Right now you get this -
e2e-runner@df6fec3128be:~$ k8s-claimer --help
2016/05/16 17:24:57 Error getting server config (required key K8S-CLAIMER_AUTH_TOKEN missing value)
e2e-runner@df6fec3128be:~$ k8s-claimer help
2016/05/16 17:25:09 Error getting server config (required key K8S-CLAIMER_AUTH_TOKEN missing value)

@jchauncey it looks like you're trying to start the server. when I run the cli with --help, I get this:
ENG000656:k8s-claimer aaronschlesinger$ ./k8s-claimer-cli --help
NAME:
   k8s-claimer - This CLI can be used against a k8s-claimer server to acquire and release leases

USAGE:
   k8s-claimer-cli [global options] command [command options] [arguments...]

VERSION:
   0.0.0

COMMANDS:
    lease

GLOBAL OPTIONS:
   --server 		The k8s-claimer server to talk to
   --help, -h		show help
   --version, -v	print the version

hrm then wtf are we publishing to bintray


refresh mosn-tls dependcy to go1.12.7, add support for tls1.3-gm

refresh mosn-tls dependcy to go1.12.7, add support for tls1.3-gm(see https://datatracker.ietf.org/doc/draft-yang-tls-tls13-sm-suites/)
Issues associated with this PR
see issue #1041
Solutions
SM related ciphersuites are implemented by cgo

Codecov Report

Merging #1066 into master will increase coverage by 0.72%.
The diff coverage is 72.28%.

@@            Coverage Diff             @@
##           master    #1066      +/-   ##
==========================================
+ Coverage   56.07%   56.80%   +0.72%
==========================================
  Files         246      250       +4
  Lines       23073    24423    +1350
==========================================
+ Hits        12939    13873     +934
- Misses       8975     9214     +239
- Partials     1159     1336     +177

Impacted Files
Coverage Δ

pkg/mtls/crypto/tls/alert.go
85.71% <ø> (ø)

pkg/mtls/crypto/tls/ticket.go
91.93% <ø> (+7.45%)
:arrow_up:

pkg/mtls/crypto/tls/tls.go
58.39% <ø> (-0.31%)
:arrow_down:

pkg/mtls/crypto/tls/transfer_tls.go
8.33% <ø> (-0.58%)
:arrow_down:

pkg/mtls/crypto/tls/handshake_client_tls13.go
49.49% <49.49%> (ø)

pkg/mtls/crypto/tls/auth.go
54.16% <54.16%> (ø)

pkg/mtls/crypto/tls/key_agreement.go
66.47% <61.66%> (+14.11%)
:arrow_up:

pkg/mtls/crypto/tls/handshake_server_tls13.go
62.75% <62.75%> (ø)

pkg/mtls/crypto/tls/conn.go
74.96% <72.91%> (-1.93%)
:arrow_down:

pkg/mtls/crypto/tls/prf.go
90.38% <74.35%> (-0.02%)
:arrow_down:

... and 19 more

Continue to review full report at Codecov.

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


Fix 3rd party licenses updater (action)

Opened in favour of https://github.com/erlef/setup-beam/pull/201 (since I wanna use workflow_dispatch).

Thus we shall soon have a new (automated) pull request, if everything goes according to plan 😄  Thanks, @starbelly.

Still not everything working as it should: https://github.com/erlef/setup-beam/actions/runs/4936735135/jobs/8824633938 😢

Released in v1.16.0 / v1.16. Tag v1 moved up...


Convert workflow view to hooks

Recommendation for review (links to commit ranges for review):

First review changes to components other than the WorkflowView itself
Then, review the conversion of WorkflowView to hooks
Finally, the normalizing for the WorkflowView

@zarsky-broad Closing as there hasn't been any activity on this. Please re-open if it becomes applicable.


test sd

What this PR does / why we need it:
Which issue(s) this PR fixes (optional, using fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when the PR gets merged):
Fixes #
Special notes for your reviewer:

part of #225 now


[Error] Attempted to index a nil value (field 'proc')

same happens for me :(

What are the steps to reproduce the issue? Any custom LSP config option, like config.plugins.lsp.more_yielding = true? Does it happen every time?

My LSP related config:
local lsp = require "plugins.lsp"
local lspconfig = require "plugins.lsp.config"

-- Normally set to false, but now true for testing
config.plugins.lsp.mouse_hover = true

lspconfig.gopls.setup()
lspconfig.pylsp.setup()

Yeah, it happens everytime I hover or I close a file which is in use by a server (a .py or .go file)

And now it also occurs when opening a file

Traced the issue to my configuration, something kept crashing it but it wasn't appearing in any logs. I didn't install a key requirement. Fixed it by installing it.
But this should be solved by doing a check if proc is nil before doing anything, and perhaps restart the LSP server if neccesary?

What's the full backtrace? You can ctrl+click in the Log viewer to copy it.
Is it the same backtrace every time?

But this should be solved by doing a check if proc is nil before doing anything, and perhaps restart the LSP server if neccesary?

I'm mostly trying to understand why the server is still being used after it's stopped.
At the moment the only way I was able to reproduce the issue was with config.plugins.lsp.more_yielding = true, so I think I'll investigate that.

Traced the issue to my configuration, something kept crashing it but it wasn't appearing in any logs. I didn't install a key requirement. Fixed it by installing it.

What was it? Maybe we can improve logging if it isn't found.

What was it? Maybe we can improve logging if it isn't found.

PyLSP was missing jedi, all the logs said was pylsp exited, check your config and then pylsp restarted, but it didn't restart, which caused proc to be nil (as the process is dead)

This is the only config that I have:
lspconfig.tsserver.setup {
  command = {
    '/opt/homebrew/bin/typescript-language-server',
    '--stdio'
  }
}

config.plugins.lsp.log_server_stderr = true

and the error that I get is:
Tue Nov  7 20:52:46 2023 [ERROR] /Users/skolind/.config/lite-xl/plugins/lsp/server.lua:1162: attempt to index a nil value (field 'proc') at /Applications/Lite XL.app/Contents/Resources/core/init.lua:1226

stack traceback:
/Users/skolind/.config/lite-xl/plugins/lsp/server.lua:1162: in function 'plugins.lsp.server.read_responses'
/Users/skolind/.config/lite-xl/plugins/lsp/server.lua:721: in function 'plugins.lsp.server.process_responses'
/Users/skolind/.config/lite-xl/plugins/lsp/init.lua:2141: in function </Users/skolind/.config/lite-xl/plugins/lsp/init.lua:2115>
[C]: in function 'xpcall'
/Applications/Lite XL.app/Contents/Resources/core/init.lua:1225: in function 'core.try'
(...tail calls...)

Tue Nov  7 20:52:46 2023 [INFO] [LSP]: typescript-language-server was shutdown, revise your configuration at /Users/skolind/.config/lite-xl/plugins/lsp/init.lua:789

Tue Nov  7 20:52:35 2023 [INFO] [LSP/typescript-language-server]: Error:
'env: node: No such file or directory
'
 at /Users/skolind/.config/lite-xl/plugins/lsp/init.lua:780

I guess it's the env: node part, but not sure what that means? I am using the LSP in other editors like Zed, Nvim etc.. with no issues.

I guess it's the env: node part, but not sure what that means?

Looks like your PATH might not contain node. Looking online this seems to be a common issue on macOS, so it's probably not related to Lite XL and the LSP plugin.
Still, I'm working on solving the attempt to index a nil value part, but that won't solve your tsserver issue.

Add this to your user module then check the Logs:
core.log("%s", os.getenv("PATH"))
local np, n_err = process.start({"node", "--version"})
if not np then
  core.log("Unable to start node: %s", n_err)
else
  local buf = {}
  repeat
    local buffer = np:read_stdout()
    if buffer and #buffer > 0 then
      table.insert(buf, buffer)
    end
  until not buffer
  core.log("%s", table.concat(buf))
end

It's very possible that the PATH depends on how you open Lite XL.

core.log("%s", os.getenv("PATH"))
local np, n_err = process.start({"node", "--version"})
if not np then
  core.log("Unable to start node: %s", n_err)
else
  local buf = {}
  repeat
    local buffer = np:read_stdout()
    if buffer and #buffer > 0 then
      table.insert(buf, buffer)
    end
  until not buffer
  core.log("%s", table.concat(buf))
end

I think you are right! It printed out this:
Tue Nov  7 21:26:02 2023 [INFO] /usr/local/bin:/usr/bin:/bin:/usr/sbin:/sbin at /Users/skolind/.config/lite-xl/init.lua:68

Which is not my PATH specified in .zshrc.
I am opening lite from spotlight. Which seems to not load my PATH correcrtly.
If I run /Applications/Lite\ XL.app/Contents/MacOS/lite-xl in the terminal, it is working and the LSP is loading correctly.
Any ideas how to fix this?

I wonder if you could run typescript-language-server from zsh. So something like:
local lspconfig = require "plugins.lsp.config"

lspconfig.tsserver.setup {
  command = {
    'zsh', '-c',
    '/opt/homebrew/bin/typescript-language-server --stdio'
  }
}

I wonder if you could run typescript-language-server from zsh. So something like:
local lspconfig = require "plugins.lsp.config"

lspconfig.tsserver.setup {
  command = {
    'zsh', '-c',
    '/opt/homebrew/bin/typescript-language-server --stdio'
  }
}

Unfortunately, that doesn't seem to make a difference.

Same env: node: No such file or directory error?
I guess you could try forcing zsh to source your .zshrc, like:
local lspconfig = require "plugins.lsp.config"

lspconfig.tsserver.setup {
  command = {
    'zsh', '-c',
    'source ~/.zshrc; /opt/homebrew/bin/typescript-language-server --stdio'
  }
}

lspconfig.tsserver.setup {
  command = {
    'zsh', '-c',
    'source ~/.zshrc; /opt/homebrew/bin/typescript-language-server --stdio'
  }
}

Force sourcing .zshrc seems to do the trick. It is a bit hacky however, but I guess it's an issue with how MacOS opens the application?
It seems to work now though, thank you so much!

Force sourcing .zshrc seems to do the trick. It is a bit hacky however, but I guess it's an issue with how MacOS opens the application?

I don't know much about macOS but a similar thing happens in some Linux DEs.
Let me know if with #79 applied, you aren't getting attempt to index a nil value anymore with the "bad" version of the config.

It seems to work now though, thank you so much!

No worries!

Force sourcing .zshrc seems to do the trick. It is a bit hacky however, but I guess it's an issue with how MacOS opens the application?

I don't know much about macOS but a similar thing happens in some Linux DEs.
Let me know if with #79 applied, you aren't getting attempt to index a nil value anymore with the "bad" version of the config.

It seems to work now though, thank you so much!

No worries!

I can't reproduce the issue with a "bad config" anymore. Awesome!

So I guess this issue is now fixed aswell?


Fix some minor things

I make the variable name the same in implementation and definition. One of it is typo.
Prevent socket opening when failed.

Thank you :+1:
Merged as https://github.com/Genymobile/scrcpy/commit/fcf225049dc783677244f08700ed4c53e8dc5382 and https://github.com/Genymobile/scrcpy/commit/a56045dd8047f9b147e63be67c41a4423023437f.
I just edited the commit messages to respect the 50-72 rule.

(for info: 296047d82a4d6e8d87a955b9643f7538e79200ee)


chore: bump deps

Update:
K8s to 0.28.3
Github Actions deps
Talos 1.5.5
Pull Request

What? (description)
Why? (reasoning)
Acceptance
Please use the following checklist:

[ ] you linked an issue (if applicable)
[ ] you included tests (if applicable)
[ ] you ran conformance (make conformance)
[ ] you linted your code (make lint)
[x] you linted your code (make unit)

See make help for a description of the available targets.

/m


Permanent URL (Heroku "No such app" error)

I noticed that https://warehouse.python.org/ produces a Heroku | No such app error at the moment. Is this intentional? Are we permanently at https://pypi.io/ now?
If so, we should probably update the URL in a few places: https://github.com/pypa/warehouse/search?utf8=✓&q="warehouse.python.org"

#1176 should make warehouse.python.org redirect to https://pypi.io/ but we are permanently at https://pypi.io/ (and https://test.pypi.io/) so we should still update those references.


"View" is repeated in several Dendron command names

Please select the area your bug applies to. (Multiple selections are Possible. You can leave blank if you're not sure.)

[ ] Workspace.  VSCode workspace, vaults, Intellisense/autocomplete, Dendron settings
[ ] Lookup.  Dendron's Lookup Command
[ ] Views.  Dendron Preview, Tree View, Side Panels in the UI
[ ] Schema.  Dendron Schemas
[ ] Pod.  Data import from / export to Dendron
[ ] Publish.  External Site Publish
[ ] Markdown.  Markdown features, such as syntax support and features

Describe the bug
The word "view" is repeated twice in the text of the following commands:
Dendron: Focus on Calendar View View (dendron.calendar-view.focus)
Dendron: Focus on Lookup View View (dendron.lookup-view.focus)
Dendron Focus on Tree View View (dendron.treeView.focus)
Explorer: Focus on Sample View View (dendron.sample.focus)
To Reproduce
Open keyboard shortcuts, search for dendron focus, see the extraneous text
Expected behavior
View is not repeated
Screenshots
(Highlighting added)

Desktop (please complete the following information)

OS: [e.g. iOS]
Browser (if with published websites; include version): [e.g. chrome, safari]
Versions (all that apply to bug):

Dendron Extension:
VS Code version:
Dendron CLI:

Additional context
Add any other context about the problem here.
Dendron Log file
Please attach the output of > Dendron:Dev: Open Logs here

Thanks @jeep for the issue.
The second 'View' in pink is actually a VS Code string - those commands like Dendron: Focus On ... View are actually vscode commands that are auto-generated when an extension registers a panel.
So if we want to make 'view' only show up once, we'll need to convert the name of the panels from 'Calendar View' -> 'Calendar'.  This should probably be fine, except for 'Tree View', because renaming that panel to 'Tree' won't make too much sense.

Calendar is significantly better than calendar view, IMO.
Lookup View and Sample View only shows in the keymap, as far as I can see. When I try to execute them from the command palette, nothing shows
Tree View is odd. I would have called it dendrogram- esp. given the name of the program. ;)

@jeep - re: calendar vs calendar view - I agree.  If you want to give the code change a go, feel free to change 'calendar view' to 'calendar'.
For 'Tree View' - let's leave it for now.  There can probably be a better name for it, but we refer to it as 'tree view' extensively in the docs and in Discord so there's some obstacles to change.
'Sample View' is something that should be deleted - I don't think we realized it was getting exposed here - thanks for pointing it out!


PyLint checks

@DanielAvdar I mean what is the advantage of creating new job?
Why not just to add the linter to the existing container?
aka - new step with:
poetry run pylint src/shmessy

The primary challenge with separated jobs lies in the duplication of initial steps within the workflow file. Nonetheless, this concern can be addressed by implementing templates for these steps in future iterations.
The primary rationale for employing separate jobs is to enhance the clarity of PR checks, particularly where pylint is distinguished as a distinct evaluation. Typically, segregating checks and tests results in more effective feedback. Additionally, there is no necessity to execute pylint checks for every operating system and Python version.
@ohadmata please tell me what you think

I concur with your earlier point that those checks aren't crucial since pre-commit is handling formatting.

i have a better one.


[MM-27923] Turn on Cloud Onboarding for Cloud-only instances

Summary
Cloud Onboarding should now only be turned on for Cloud-only instances.
Ticket Link
https://mattermost.atlassian.net/browse/MM-27923

@devinbinnie Can you think of any way I might be able to test this?


Undocked IPython console gets focus after shift+enter in editor

It is user unfriendly if you like to run several cells one by one.
EDIT: same issue if we have undocked editor.

To turn off this behavior you need to go to the menu
Tools > Preferences > Editor > Run Code

and activate the option called
Maintain focus in the editor after running cells or selections

That's the default (by the way) so at some point you changed it.

Sorry, I just noticed that you were talking about the console when it's undocked. I can confirm we're moving the focus to console after evaluating a cell when we should respect the setting I mentioned above.
We'll take a look at this in a future release.

This issue is also present when using runfile or run selection


⚠ ClamAV has degraded performance

In 1205dff, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 70 ms

Resolved: ClamAV performance has improved in f830fd9.


:memo: Fix: Update getting-started docs

Description
Concise description of proposed changes
Testing
Explain the quality checks that have been done on the code changes
Additional Information

[ ] I read the contributing docs (if this is your first contribution)

Your ENS/address:

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

#1099  👈
#1098
#1097
main

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

Join @roninjin10 and the rest of your teammates on  Graphite


Provide API to determine to grab the currently-running XCTest.

Hello!
I'm maintainer of Nimble, a framework that provides an alternative assertion DSL for Swift and Objective-C. Earlier today, I noticed that Nimble's integration with XCTest on Linux & Windows is lacking, which is entirely because there isn't a good place for Nimble to register its XCTestObservation conformer - which is how Nimble currently keeps track of the current test case - in a non-objc environment. On Darwin platforms, Nimble registers its XCTestObservation conformer when it's loaded, but that only works when the objc runtime is available.
Naively, I would think that making XCTCurrentTestCase publicly readable would solve this issue for Nimble, such as this patch:
diff --git a/Sources/XCTest/Public/XCTestCase.swift b/Sources/XCTest/Public/XCTestCase.swift
index 2d899b9..6e229fe 100644
--- a/Sources/XCTest/Public/XCTestCase.swift
+++ b/Sources/XCTest/Public/XCTestCase.swift
@@ -25,7 +25,7 @@ public typealias XCTestCaseEntry = (testCaseClass: XCTestCase.Type, allTests: [(

 // A global pointer to the currently running test case. This is required in
 // order for XCTAssert functions to report failures.
-internal var XCTCurrentTestCase: XCTestCase?
+public private(set) var XCTCurrentTestCase: XCTestCase?

 /// An instance of this class represents an individual test case which can be
 /// run by the framework. This class is normally subclassed and extended with

Which would work, but I'm curious if that's actually the right approach that should be used here. There's probably a very good reason you haven't made XCTCurrentTestCase publicly readable, after all.

Would you mind also filing feedback against Apple's XCTest implementation and let us know the FB number here? To add API for this to corelibs-xctest, we generally would want to add it to XCTest.framework first. (Mmm, bureaucracy.)

FB14167426


Add event consolidation

Hello,
I use phx_copy in my projects and I access the project through a network mount. Long story short, every time I save a file, it generates a dozen of events, which in turn make live reload crazy.
I think this could be solved by delaying copy for a file for a few milliseconds (configurable). So if you get 10 events for a file in rapid succession, only one copy is done.
If you think this is a good idea, I'll make a PR.

Hi there!
That sounds like a great idea. Just a heads-up, I am away from home this week, so may not get to see any PRs until later. But please feel free to create a PR with what you have in mind.
Thank you!

Released as 0.1.2.