Are you planning a disk-backed cache?

This question is related to https://github.com/microsoft/reverse-proxy/issues/109 and replacing ARR. Will reverse proxy caching be on the roadmap? Like ARR (or Varnish)?

Nothing specific in the plans yet.
AspNetCore already has an in-memory response cache that should be proxy compatible (it keys off response cache headers). Someone could probably adapt it to use disk storage, though the concurrency model would be trickier.
Why a disk-backed cache specifically? Capacity?

It’s fair to say it’s something on our mind though. Not on the plans for 1.0 but it should be possible to do it via the extensibility points in 1.0, and we can look at it after the first version is solid.

Thanks. Yes, it's capacity - apps that can potentially generate 100s of GBs of highly cacheable content, that today you would (if you didn't fancy a CDN) stick IIS+ARR+cache, or Varnish, or NGINX Content Caching in front of.
Would be lovely to just add app.UseSuperDiskCache() or whatever. Sounds like with reverse-proxy as a toolbox it should be easier to make a disk-backed cache, but it's the kind of thing I'd much rather use something ready-made.
Really excited by this project, btw!
(What's the etiquette here - I close the issue as answered?)

Let’s leave this open to track further caching work. We’ll put it in our post-1.0 backlog though.

Having the option to be disk-based would really open up a lot of use cases, I believe. E.g. an integrated, disk-based cache inside a .Net Core app means fewer moving parts for a container stack. Also means you can apply logic and smarts to it locally instead of being split between the app and whatever process is doing the caching. Stretch goal for 1.0?

Currently, this is not critical for 1.0. As mentioned above, once the core functionality is solid, we can prioritize it with other feature requests.

Nothing specific in the plans yet.
AspNetCore already has an in-memory response cache that should be proxy compatible (it keys off response cache headers). Someone could probably adapt it to use disk storage, though the concurrency model would be trickier.
Why a disk-backed cache specifically? Capacity?

@Tratcher  At a high level, would someone need to just wire up an implementation of IDistributedCache to make this work? (for experimentation anyway)

The response caching middleware linked above caches in-memory. You could certainly fork that to prototype caching to disk. AspNetCore already has FileBufferingWriteStream that would be useful here.
https://github.com/dotnet/aspnetcore/blob/47e21d4849cc6cb88fc9ca9c95054add1261e312/src/Http/WebUtilities/src/FileBufferingWriteStream.cs#L20

Hmm, I'm gonna try to carve a little time to spike an implementation using FASTER KV for small to medium size responses.
Notes for myself...

https://source.dot.net/#Microsoft.AspNetCore.ResponseCaching/ResponseCachingMiddleware.cs

(the aformentioned middleware)

https://source.dot.net/#Microsoft.AspNetCore.ResponseCaching/Interfaces/IResponseCache.cs

(important interface)

https://source.dot.net/#Microsoft.AspNetCore.ResponseCaching/ResponseCachingKeyProvider.cs,53

(stack alloc span instead of SB?)

https://source.dot.net/#Microsoft.AspNetCore.ResponseCaching/Interfaces/IResponseCachingKeyProvider.cs,8

(fixed length hash instead of varlen string keys?)

https://source.dot.net/#Microsoft.AspNetCore.ResponseCaching/ResponseCachingMiddleware.cs,225

uses marker interface to distinguish between a "vary" key placeholder and an actual cache entry. Vary keys are the slow path so thats why we optimize around them.

https://source.dot.net/#Microsoft.AspNetCore.ResponseCaching/ResponseCachingKeyProvider.cs,86

Vary key computation is somewhat complex... don't mess with it

RE: dotnet/aspnetcore#40229

Nooo, it appears its been pushed to dotnet 8 😢😢.
That's quite catastrophic to our tentative roadmap.
Back to IIS+ARR/nginx we go 😭.

Note that https://github.com/dotnet/aspnetcore/issues/40229 is for output caching, not response caching. These are two separate things - output caching ignores cache-control headers.
I do not see any plans to add data storage options for response caching.

Some motion here:
https://github.com/microsoft/FASTER/issues/902

Some movement here:
https://github.com/microsoft/FASTER/issues/902

@Xhanti probably a bit premature to get too excited there, but yes; I hope we can get a FASTER-backed local-machine larger-than-RAM cache engine suitable for distributed cache, output cache, and whatever else people might want

@mgravell understood. Early days, but even if you MS does not proceed and support it, the work you've contributed here is a good jumping off point for the next person. I am also inordinately excited for this body of work :) .This is a problem I have been thinking about for some time now

I've been doing a lot of research and testing recently.
Trying to shoehorn in my own disk cache solution using various caching libs including the new hybrid cache in dotnet 9.
I've had some success however none that meets our requirements.
The biggest issue with all these existing caching libs is that they are all basically byte[] under the covers.
We need to be able to handle files that are gigs in size and all the existing caching mechanisms end up storing this all in memory.
We could try and do things like split up the body into chunks etc (basically write a stream over the top of the http/cache requests) but at this stage we may as well just implement the caching from scratch. I'm also not sure how that could be implemented with YARP as you would end up taking a single get request from a client and turning it into many range requests as you "stream" it through the cache pipeline.
@mgravell do you know if FASTER/Garnet supports anything like streams that could fit this use case?


Can't Install macOS 15.2 Beta 1 on UTM 4.6.0

Describe the issue
What I tried to install macOS 15.2 Beta 1 with UTM 4.6.0 on macOS 15.2 Beta 1 host I got the following error message:

Configuration

UTM Version: 4.6.0
macOS Version: 15.2 Beta 1
Mac Chip (Intel, M1, ...): M1 Max

I have the samme problem. Install Xcode 16.2 beta not fix the problem.

It seems that new macOS update fixed the problem.
HOST
Version 15.2 Beta (24C5073e)
GUEST
Version 15.2 Beta (24C5073e)


Allow 0s in , unlink Y/Z coords

What's New?

Fix bug where entering 0 results in -1 on AxisInputBox
Prevent Z axis from "linking" to Y axis.

Coverage increased (+0.08%) to 21.14% when pulling f65003d631983340dc67d92ecd942391ef990b2a on RickCarlino:master into 9d0c9e4ca823b273bd6c50f81073b1f4c4b0d890 on FarmBot:master.


Issue/53 show user details

Pull Request
Description

adds sites toolbox page to the OCF Dashboard
adds feature Get user details which shows information for a selected user:

adds test rig and the pytest for get_user_details

Fixes #53
How Has This Been Tested?
Currently the tests won't run, and I keep getting an error that the files aren't named correctly. It would be useful to understand what I need to do so the tests will run:

Checklist:

[x] My code follows OCF's coding style guidelines
[x] I have performed a self-review of my own code
[ ] I have made corresponding changes to the documentation
[ ] I have added tests that prove my fix is effective or that my feature works
[x] I have checked my code and corrected any misspellings

The tests dont currently work because it doesnt know where that file is. You may need to change 'sites_toolbox.py to '.sites_toolbox.py. Or one other trick is do run export PYTHONPATH=${PYTHONPATH}:./src or something like that


Don't add patient to call list when working in data entry mode

Thanks for creating an issue! Please fill out this form so we can be sure to have all the information we need, and to minimize back and forth.

What are we trying to do?
Not add patients to a call list when entering them through data entry mode.

What feature or behavior is this required for?
Not bogging down call lists on dashboard when entering ~100+ patients into the system at a time

How could we solve this issue? (Not knowing is okay!)
Pop into PatientsController#data_entry_create and remove the line that adds them to the current user's call list

Anything else?
nope!

I went ahead with a PR for this as it looked like something in active development I could knock out fast.


Made a few methods in MessageImpl synchronized

This should fix issues with concurrent calls to the modified methods.

Wouldn't the setters also have to be synchronized for this to actually have any effect?
Because the get and set calls could still interleave.

A better solution would be to have a lock that is only locked and released if subcontent is null, otherwise just return subcontent without locking.

What setters are you talking about exactly?

setContent() mainly

setContent sets the raw content returned by getRawContent.
getContent however overrides  the raw content and provides a content without the discord formats like <@86699011792191488> for mentions and instead replaces them with @Minn.
That new content with stripped formats is then saved as subContent.
getStrippedContent does something similar for markdown formatting.

The setContent method is only used by internal builders and should never be used when the user accesses the Message.

Okay, sorry I misunderstood what the setter methods were for!


AzCopy and the case for adding a flag to stop version check

Which version of the AzCopy was used?
Note: The version is visible when running AzCopy without any argument
10.14.0
Which platform are you using? (ex: Windows, Mac, Linux)
Windows
What command did you run?
azcopy
What problem was encountered?
A delay of up to 1 second when starting azcopy, depending on location.
We have a large (18.5 Million) amount of files that we need to sync to blob storage. After struggling with too much CPU (Concurrency), too much memory (BUFFER) and following a variety of posts, I came to the approach of creating multiple jobs (https://docs.microsoft.com/en-us/azure/storage/common/storage-use-azcopy-optimize).
By dicing up the folders using a PowerShell script to generate a batch file, I ended up with around 4500 calls to AzCopy. What I continually noticed, however, was a pause at the beginning of the AzCopy call (much like the 2-second bug #1653)
A process watch showed a network call on every startup taking around 1 sec. The destination URL was for a txt file of the latest version number available of AzCopy
To prove this delay problem, at the start of beginDetectNewVersion in root.go, I added
		tStart := time.Now()
and at the end, I added
		t := time.Now()
		theMessage := fmt.Sprintf("Time to check new version %s",t.Sub(tStart))
		glcm.Info(theMessage)

Running on my local machine in Auckland, New Zealand
INFO: Time to check new version 1.0095371s
Running on a system in Chicago
INFO: Time to check new version 226.4649ms
Running on an Azure VM in Central US
INFO: Time to check new version 198.0786ms
Because the home location of the latest version.txt file is based at
https://azcopyvnextrelease.blob.core.windows.net/releasemetadata/latest_version.txt, every time you call AzCopy you will have a delay based on the latency between your location and this blob storage (latency+SSL negotiation)
That means on any job, you incur between 200ms and 1+ second delay before anything even starts copying.
All our transfers happen from Chicago, which means we have 4500 * 220ms = 990 seconds, or 16.5 minutes of the entire process just "looking for a new version"
What I did not test - if the system was locked down, how long before it timed out the connection?
After looking to see if there was a switch to turn this off, I was surprised to see other issues where this was requested and closed as won't do. Whilst I understand it may be important to show users that there is a newer version, it significantly impacts transfers.
Hopefully the above will be sufficient to convince that a switch to turn this behavior off is critical to large transfer jobs, as well as sites that have no open internet access to "check latest version"

See #1175 and #1430

Hey @supermistag ,
Thanks for reaching out!
As you know we don't support disabling version check. Please allow me to correct your understanding on this a bit. Version check is done by an independent goroutine(thread) in parallel and not by main thread (sequentially) so it doesn't consume the amount of time you're claiming. Check this code for your reference. We'll not disable it as it is part of Microsoft products to do version check.
If you still feel that it's still taking lot of time and wish to disable the version check yourself, you can comment out the invocation/occurrence of this function beginDetectNewVersion() and build your own binary by running GOARCH=amd64 GOOS=windows go build -o azcopy_windows_amd64.exe or GOARCH=386 GOOS=windows go build -o azcopy_windows_386.exe.
As for the issue 1653, we have fixed it and are planning to release the fix in next release.
I hope that answers your question. Please reach out for more information!

Hi,
I’m sorry, I disagree – every single time you run AzCopy, you can incur a 1-second delay. Perhaps there was confusion about where the delay is. When I referred to a job, I didn't mean AzCopy generating thousands of plans in one pass and each job running a plan takes that time.
I was referring to the 4500 times I call AzCopy in my batch file job as per the MS recommendations https://docs.microsoft.com/en-us/azure/storage/common/storage-use-azcopy-optimize
"One way to reduce the size of a job is to limit the number of files affected by a job. You can use command parameters to do that. For example, a job can copy only a subset of directories by using the include path parameter as part of the azcopy copy command."
e.g.
c:\az\azcopy.exe copy folder1 folder2
-> 1 second delay
c:\az\azcopy.exe copy folder3 folder4
-> 1 second delay
c:\az\azcopy.exe copy folder5 folder6
-> 1 second delay
Try repeating that 4500 times.
I’ve built my own copy already (to do the timing test, and fix that 2-second delay and yes, already removed the getversion) to overcome these arbitrarily enforced delays that ultimately cost me and my company time .. which is money.
Perhaps it doesn’t affect you sitting right near the version server, but the ability for a client to choose to stop this check via a flag is required for clients of azcopy to save a significant amount of time. It is their informed choice to turn off this check to save them time.

Hey @supermistag ,
I understand your problem.
As you know AzCopy is an open source product, so we really encourage/appreciate users contributions. I see you've already removed these changes in your copy of build. That solves the problem I hope. I'm also curious to know about your findings in 2-sec delays and other improvements you may have thought in the process. We'll try to merge the fix of 2-sec delay in next release.
But, when you are trying to transfer as much as 18M files, the center of gravity lies in reducing the runtime of AzCopy sync command. Have you tried moving your machine (or VM) in same region as your storage account? Moving/creating VM in the same region as your storage account can significantly improve the performance by reducing scanning time and network transfer latencies.
Also, you can run AzCopy Benchmark command to find the correct tuning parameters specific to your environment.
You can find all our documentation on publicly available pages or in the AzCopy Wiki Section
Please reach out if you have question about optimizations. I'll be more than happy to help 😃

The data is stored on a SAN, so moving the source machine(s) is not an option. Believe me, I've gone over as much documentation on improving performance as I can find. Some of it is inconsistent to the source code.
I commented on the other 2-second issue saying the proposed fix works well.

I am thinking of optimizing the version check so it can cache the value and doesn't have to make REST Calls multiple times. I'll keep you posted :)

Optimize AzCopy refers to check-length for both copy and sync. Check-length doesn't exist in sync (logged #1704, closed as ... i don't know why)
Tried using --include-after, but that was closed for azcopy in #1398 (didn't read the OP question, they asked for support in azcopy sync)

The data is stored on a SAN, so moving the source machine(s) is not an option. Believe me, I've gone over as much documentation on improving performance as I can find. Some of it is inconsistent to the source code.
I commented on the other 2-second issue saying the proposed fix works well.

You could block it on the source host(s) by adding a hosts file entry to <IP_ADDRESS> for host azcopyvnextrelease.blob.core.windows.net
This way the traffic would not even leave the vNIC.
Obviously it would prevent you easily updating AzCopy (could also be a positive thing to prevent the version being auto-updated).
Pros & cons I guess.

In our case the we see the initial request to aka.ms which is dropped by Azure Firewall because we are not going to allow blanket access to this domain. It does mean that the job that periodically runs azcopy runs, it spams the log (in our case this runs every 1 min as its sync'ing  files from a storage account) files with errors around the check failing. It makes the logs noisy and it means any alerts we want to create off the back of this have to exclude this from the logs.

@Darryl-G I tried blocking it by adding a hosts file entry for azcopyvnextrelease.blob.core.windows.net to point to <IP_ADDRESS> – rather than make startup faster, it now takes a flat 8 seconds.

@Darryl-G I tried blocking it by adding a hosts file entry for azcopyvnextrelease.blob.core.windows.net to point to <IP_ADDRESS> – rather than make startup faster, it now takes a flat 8 seconds (consistently).
$ time azcopy -v
real 0m8.014s

Previously, I get around 0.6s.

AzCopy must be trying and trying until it hits some sort of timeout.  Instead of <IP_ADDRESS>  put some random IP that does not exist on your internal network.  See if that changes the behavior.

But presumably, what the program would do is the equivalent of telnet <IP_ADDRESS> 443.
When I do that on the machine, I get:
$ telnet azcopyvnextrelease.blob.core.windows.net 443
Trying <IP_ADDRESS>...
telnet: Unable to connect to remote host: Connection refused

This happens immediately.
The point is, azcopy must be doing a round (or several) of retrying on its own account. I don't think that can be worked around by picking a different IP.

A "connection refused" error is different to a "cannot resolve" error.
Underlying TCP/IP, the network stack still resolves <IP_ADDRESS> to a valid
MAC address and will allow the connection attempt on port 443.
If it is unable to get a MAC, then you might find a different outcome.
I can't be sure without testing.  I'm just saying that it might produce a
different outcome because it is a different scenario.
Note: it needs to be either a local class C up address or a private one
"192.x" that you choose. To prevent it trying to resolve something on the
internet.
On Mon, 12 Sep 2022, 09:25 Malthe Borch, @.***> wrote:

But presumably, what the program would do is the equivalent of telnet
<IP_ADDRESS> 443.
When I do that on the machine, I get:
$ telnet azcopyvnextrelease.blob.core.windows.net 443
Trying <IP_ADDRESS>...
telnet: Unable to connect to remote host: Connection refused
This happens immediately.
The point is, azcopy must be doing a round (or several) of retrying on
its own account. I don't think that can be worked around by picking a
different IP.
—
Reply to this email directly, view it on GitHub
https://github.com/Azure/azure-storage-azcopy/issues/1705#issuecomment-1243388337,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AJOKVBKYVSWMCNZGXQ4D7DDV53SHTANCNFSM5QUFQDSQ
.
You are receiving this because you were mentioned.Message ID:
@.***>


Filtering based on an array projection sometimes seems to have no effect

Using this data on a MongoDB mount:
{ "a": 1, "b": { "ba": true, "bb": ["x", false, { "foo": "bar" }], "bc": 0.5 }, "c": [] }
{ "a": 2, "b": { "ba": false, "bb": ["y", true, { "foo": "baz" }], "bc": 60 }, "c": [true, false, "maybe"] }
{ "a": 3, "b": { "ba": true, "bb": ["z", { "foo": "quux" }] }, "c": null }

With this query:
SELECT t.* FROM `/test/structure_test.json` AS t WHERE t.b.bb[1] != false

The WHERE has no effect, whatever the condition I give it as long as the bb[] projection is there. I can select sensible values from the projection however:
SELECT t.*, t.b.bb[1] as bbb1 FROM `/test/structure_test.json` AS t

Gives:
[
{ "value": { "_id": { "$oid": "58f7fa86be6ea60adc03ee5a" }, "a": 1.0, "b": { "ba": true, "bb": [ "x", false, { "foo": "bar" } ], "bc": 0.5 }, "c": [], "bbb1": false } },
{ "value": { "_id": { "$oid": "58f7fa86be6ea60adc03ee5b" }, "a": 2.0, "b": { "ba": false, "bb": [ "y", true, { "foo": "baz" } ], "bc": 60 }, "c": [ true, false, "maybe" ], "bbb1": true } },
{ "value": { "_id": { "$oid": "58f7fa86be6ea60adc03ee5c" }, "a": 3.0, "b": { "ba": true, "bb": [ "z", { "foo": "quux" } ] }, "c": null, "bbb1": { "foo": "quux" } } }
]

Homing these under the "Mongo Connector Overhaul" Epic - since this issue was recorded against a Mongo Mount. @sellout -> Please re home under the right epic, if this isn't a connector issue, but rather an underlying problem elsewhere.

@zeeiyerWork Labels?  You're basically the only one using ZenHub, so the rest of us can't really re-home things in its system.  I thought we were going to apply the label encoding to our epics?

@djspiewak  -> Yes am in the process of creating github labels for each epic as well. Hopefully will be done by that by around next Monday.  Sorry for the delay (slamjam, few other pressing things this week).

This works on new Mongo
{ "value": { "_id": { "$oid": "594bedf5d1c9652f8c81632d" }, "a": 2, "b": { "ba": false, "bb": [ "y", true, { "foo": "baz" } ], "bc": 60 }, "c": [ true, false, "maybe" ] } }
{ "value": { "_id": { "$oid": "594bedfad1c9652f8c81632e" }, "a": 3, "b": { "ba": true, "bb": [ "z", { "foo": "quux" } ] }, "c": null } }

@rintcius -> Just have to verify on the newMaster branch (as is the case with all of these On hold issues) and then mark them closed if it has been verified as fixed.

Filtering works but it's not consistent wrt select value semantics:
💪 $ SELECT t.* FROM `/m34/test/2061` AS t

Query time: 0.1s
 a  | b.ba   | b.bb[0] | b.bb[1] | b.bb[2].foo | b.bc | c[0]  | c[1]   | c[2]   | b.bb[1].foo | c     |
----|--------|---------|---------|-------------|------|-------|--------|--------|-------------|-------|
  1 |   true | x       |   false | bar         |  0.5 |       |        |        |             |       |
  2 |  false | y       |    true | baz         |   60 |  true |  false | maybe  |             |       |
  3 |   true | z       |         |             |      |       |        |        | quux        |  null |

💪 $ SELECT t.* FROM `/m34/test/2061` AS t WHERE t.b.bb[1] != false

Query time: 0.2s
 value._id                 | value.a | value.b.ba | value.b.bb[0] | value.b.bb[1] | value.b.bb[2].foo | value.b.bc | value.c[0] | value.c[1] | value.c[2] | value.b.bb[1].foo | value.c |
---------------------------|---------|------------|---------------|---------------|-------------------|------------|------------|------------|------------|-------------------|---------|
  59b2dfa2e84096ee6e8b58e9 |       2 |      false | y             |          true | baz               |         60 |       true |      false | maybe      |                   |         |
  59b2dfa2e84096ee6e8b58ea |       3 |       true | z             |               |                   |            |            |            |            | quux              |    null |

I'm closing this one and but refer to this example in #2682


chore(readme): fix some typos and wording

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
300

Relevant Lines:
300

💛  - Coveralls


[Fighter] Maneuver DC

Change Maneuver DC from 10 + highest of Dex or Str to 8 + Prof + highest of Dex or Str

Fixed by Larian on Patch 5:


Fix issue with missing post headers on threads

Summary
At some times, we saw that either both the user and the "commented on someone's message" did not appear on some messages. This was due to the logic to get the previous and the next post was updating the post model adding the previous and the next post. This caused those values to be overwritten when the thread screen opens, messing up all the logic that deals with the previous post.
This ticket removes the changes to the post model to create a new object to handle this information.
A reliable way to reproduce this is:

Open a channel
Open a thread
Get back to the channel
Pull up to get refresh the channel

Ticket Link
Fix https://mattermost.atlassian.net/browse/MM-52798
Release Note
Fix issue where sometimes replies seemed attributed to the wrong person or the wrong thread.

nice find..


(typescript) connect has seemingly no type checking

bug
connect has inaccurate type checking.  Consider the following example:
interface IProps {
  a: Ta;
  b: Tb;
  c: Tc;
}

const mapStateToProps: (state: IState) => Pick<IProps, "a"> = ...
const mapDispatchToProps: (dispatch: Dispatch) => Pick<IProps, "b"> = ...

const MyComponent: (props: IProps) => JSX.Element = ...

export default connect(mapStateToProps, mapDispatchToProps)(MyComponent);

What is the current behavior?
There is no TypeScript compilation error, so the type system becomes inaccurate.
What is the expected behavior?
The last line should throw a typescript compilation error, because Pick<IProps, "a"> & Pick<IProps, "b"> is incompatible with IProps, as it is missing property c.
Which versions of React, ReactDOM/React Native, Redux, and React Redux are you using?  Which browser and OS are affected by this issue? Did this work in previous versions of React Redux?
<EMAIL_ADDRESS><EMAIL_ADDRESS>

We don't maintain the TS typings here in the repo - that's handled over in DefinitelyTyped.


Customogre

Rewrites Ogre build scripts to not use Sagase, and enables the use of ogre-safe-nocrashes branch on Windows and Mac OS X.

Requires the deps to be updated for vs2008 so that the new script builds against the deps as well, before merging is ok.


Feature request: pipe into lsd

Hi,
I would like to use fd in conjunction with lsd.
fd is very good at filtering files. It produces a list of paths.
However lsd is very good at rendering the paths.
I would like to have something like:
fd -d2 -e rs regex_pattern | lsd
Regards

Most of the time for this kind of behaviour you use xargs
 fd -d2 -e rs regex_pattern | xargs lsd
Does that work for you?

Yes, it works!
On Mon, Nov 13, 2023 at 6:07 PM Riccardo Attilio Galli <
@.***> wrote:

Most of the time for this kind of behaviour you use xargs
fd -d2 -e rs regex_pattern | xargs lsd
Does that work for you?
—
Reply to this email directly, view it on GitHub
https://github.com/lsd-rs/lsd/issues/950#issuecomment-1808727629, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAXGMAXV3J7P2C2GLJRDNS3YEJOUTAVCNFSM6AAAAAA7ENVYIOVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQMBYG4ZDONRSHE
.
You are receiving this because you authored the thread.Message ID:
@.***>


Add LJ Speech dataset

This PR adds the LJ Speech dataset (https://keithito.com/LJ-Speech-Dataset/)
As requested by #1841
The ASR format is based on #1767
There are a couple of quirks that should be addressed:

I tagged this dataset as other-other-automatic-speech-recognition and other-other-text-to-speech (as classified by paperswithcode). Since the number of speech datasets is about to grow, maybe these categories should be added to the main list?
Similarly to #1767 this dataset uses only a single dummy sample to reduce the zip size (wavs are quite heavy). Is there a plan to allow LFS or S3 usage for dummy data in the repo?
The dataset is distributed under the Public Domain license, which is not used anywhere else in the repo, AFAIK. Do you think Public Domain is worth adding to the tagger app as well?

Pinging @patrickvonplaten to review

Hey @anton-l,
Thanks a lot for the very clean integration!

I think we should now start having "automatic-speech-recognition" as a label in the dataset tagger (@yjernite is it easy to add?). But we can surely add this dataset with the tag you've added and then later change the label to asr

That's perfect! Yeah good question - we're currently thinking about a better design with @lhoestq

Again tagging @yjernite & @lhoestq here - guess we should add this license though!

Thanks @anton-l for adding this one :)
About the points you mentioned:

Sure as soon as we've updated the tag sets in https://github.com/huggingface/datasets-tagging/blob/main/task_set.json, we can update the tags in this dataset card and also in the other audio dataset card.
For now we jus try to have them as small as possible but we may switch to S3/LFS at one point indeed
If it's not part of the license set at https://github.com/huggingface/datasets-tagging/blob/main/license_set.json we can add it to this license set

For now it's ok to have the other-* tags but we'll update them very soon

Let's merge this one and then we'll update the tags for the audio datasets. We'll probably also add something like this:
type:
- text
- audio

Thank you so much for adding this one, good job !


Enhance test configurations and coverage

Updated pytest configuration in pyproject.toml:

Changed --cov option to cover the src/pyecotrend_ista/ directory.
Added -n auto to enable parallel test execution with pytest-xdist.

Modified requirements_test.txt:

Added pytest-xdist==3.6.1 for parallel test execution.

Cleaned up __init__.py:

Removed unused import BaseError.

Fixed error in login_helper.py:

Changed requests.codes.no_content to requests.codes["no_content"].

Added new snapshot test for get_consumption_unit_details method.

Enhanced conftest.py to mock additional API responses for comprehensive testing.

Refactored test_consumptions.py:

Added tests for consum_raw method with various year and month filters.

Added test_login.py:

Parametrized tests for login method with force_login flag.
Added a new test for the logout method.

Added test_other.py:

Included tests for utility methods and deprecated method warnings.

Kannst du schauen was kaputt ist?

Kannst du schauen was kaputt ist?

 Sobald ich zuhause bin, generier ich neue und pushe sie


Allow Position Independent Code for Static Library

Build the static library with the fPIC compiler flag, so that it can be linked into a shared library.  This will fix  #71

Hi @nkowdley,
Thanks for this pull request! We will test it and merge after we're done.
Thanks!
Varun


Build with ubuntu 22.04 in the dockerfiles

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

Comparison is base (d2486dc) 91.43% compared to head (98ec8a0) 91.49%.
 Consider uploading reports for the commit b867566 to get more accurate results

Files
Patch %
Lines

src/common_dims.cpp
98.33%
1 Missing :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@             Coverage Diff             @@
##           develop    #2017      +/-   ##
===========================================
+ Coverage    91.43%   91.49%   +0.05%
===========================================
  Files          422      424       +2
  Lines        15771    15881     +110
===========================================
+ Hits         14420    14530     +110
  Misses        1351     1351

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

So MLIR fails to build on 22.04:
Traceback (most recent call last):
  File "/usr/local/cget/build/tmp-25313f4698354e8987318405eb382ca9/rocMLIR-bf2911e5245d6452d194ee5dc41c9c2b2869e5c0/external/llvm-project/mlir/lib/Dialect/GPU/AmdDeviceLibsIncGen.py", line 53, in <module>
    generate(Path(sys.argv[1]), Path(sys.argv[2]), sys.argv[3:])
  File "/usr/local/cget/build/tmp-25313f4698354e8987318405eb382ca9/rocMLIR-bf2911e5245d6452d194ee5dc41c9c2b2869e5c0/external/llvm-project/mlir/lib/Dialect/GPU/AmdDeviceLibsIncGen.py", line 29, in generate
    with (bcPath / (lib + ".bc")).open("rb") as libFile:
  File "/usr/lib/python3.10/pathlib.py", line 1119, in open
    return self._accessor.open(self, mode, buffering, encoding, errors,
FileNotFoundError: [Errno 2] No such file or directory: '/usr/amdgcn/bitcode/ocml.bc'

I opened this issue:
https://github.com/ROCm/rocMLIR/issues/1415

So I was missing a pin for the hip-clang docker. It should be fixed now.

@causten This passes the CI, so I think this can be reviewed and merged.


auto answer - Field/header

Hello,
Can i configure auto answer?
I don't know what put in:
Field/Header:   Value:
trace SIP sends:
From: "mnunes"<EMAIL_ADDRESS>

Field/Header: => From
Value: => "mnunes"<EMAIL_ADDRESS>


Assign to remote_tmp a value based on username

This should resolve permission issues caused by 982827b when different users run plays on the same target host.

We need to merge #64 then rebase this branch in order to fix the lint errors.

Ready now.

/tmp/.ansible_$USER makes sense to me. Changed in 3eac0fc.


Show custom Label every time

HI ,
I am facing issue to show custom label my slider . I want to show my custom label every time not only on slide . means there should be step name in it,
I found issue with>  vertical={true}
that when it is true it label is showing in horizontal line . My code is as following
   <MultiSlider                      containerStyle={{marginBottom:100,}}                     vertical={true}                     values={ this.props.values}                     sliderLength={290}                     step={10}                     onValuesChangeFinish={this.multiSliderValuesChange}                     min={0}                     max={40}                     allowOverlap={true}                     customMarker={CustomMarker}                     snapped={true}                     labels={['00','01','02','03','04']}.  //not showing                     enableLabel={true}                     />

Add  toValue: 1 in Animated.timing.


refactor: one store per request

Nice, thanks P! This is a huge performance improvement. I'm wondering though, are we now writing to the same file descriptors concurrently, and is that safe?  I also ran wrk against this with 4 threads and 200 conns, and it worked fine, so I'm not sure. :shrug:
The test is broken for me, but I just had to clone the router for each request to fix it (like is done in the server). Axum has also auto-updated to 0.6.1 in tmp/axum-wasm, but the only change needed is to remove the into_service() call in app.

are we now writing to the same file descriptors concurrently, and is that safe?

I'm not 100% sure yet. But it seems the file descriptors are linked to the context and every request also has its own context. So I don't think they are the same file descriptors


invite: Check validity of invite_as value when creating multiuse link.

This PR updates code to check whether invite_as value is one of the valid role values when creating multiuse invite link.

Self-review checklist

[ ] Self-reviewed the changes for clarity and maintainability
(variable names, code reuse, readability, etc.).

Communicate decisions, questions, and potential concerns.

[ ] Explains differences from previous plans (e.g., issue description).
[ ] Highlights technical choices and bugs encountered.
[ ] Calls out remaining decisions and concerns.
[ ] Automated tests verify logic where appropriate.

Individual commits are ready for review (see commit discipline).

[ ] Each commit is a coherent idea.
[ ] Commit message(s) explain reasoning and motivation for changes.

Completed manual review and testing of the following:

[ ] Visual appearance of the changes.
[ ] Responsiveness and internationalization.
[ ] Strings and tooltips.
[ ] End-to-end functionality of buttons, interactions and flows.
[ ] Corner cases, error conditions, and easily imagined bugs.

This looks like it overlaps with #23930.

Ahh; indeed. I think we can plan to integrate #23930, since it's almost complete and covers this. Thanks for investigating @sahil839!


extraContactUriParams now working

extraContactUriParams now working and tested against OpenSIPS 3.1 that has RFC8599 support.

LGTM


Cant connect to kafka

my KAFKA_ADVERTISED_HOST_NAME is not being set correctly I believe. the problem us when going through the setup documentation and running:
export HOST_IP=$(ifconfig | grep -E "([0-9]{1,3}\.){3}[0-9]{1,3}" | grep -v <IP_ADDRESS> | awk '{ print $2 }' | cut -f2 -d: | head -n1)
Does seem to work on my ubuntu 20.04
and running: $(ifconfig | grep -E "([0-9]{1,3}\.){3}[0-9]{1,3}" | grep -v <IP_ADDRESS> | awk '{ print $2 }' | cut -f2 -d: | head -n1)
returns <IP_ADDRESS>: command not found
Help getting connected would be greatly appreciated
The errors:
"level":"ERROR","timestamp":"2020-08-14T15:49:53.961Z","logger":"kafkajs","message":"[Connection] Connection error: getaddrinfo EAI_AGAIN kafka2","broker":"kafka2:9092","clientId":"my-app","stack":"Error: getaddrinfo EAI_AGAIN kafka2\n    at GetAddrInfoReqWrap.onlookup [as oncomplete] (dns.js:66:26)"}
{"level":"ERROR","timestamp":"2020-08-14T15:49:53.962Z","logger":"kafkajs","message":"[BrokerPool] Failed to connect to seed broker, trying another broker from the list: Connection error: getaddrinfo EAI_AGAIN kafka2","retryCount":0,"retryTime":319}
{"level":"ERROR","timestamp":"2020-08-14T15:49:54.286Z","logger":"kafkajs","message":"[Connection] Connection error: getaddrinfo EAI_AGAIN kafka1","broker":"kafka1:9092","clientId":"my-app","stack":"Error: getaddrinfo EAI_AGAIN kafka1\n    at GetAddrInfoReqWrap.onlookup [as oncomplete] (dns.js:66:26)"}
{"level":"ERROR","timestamp":"2020-08-14T15:49:54.287Z","logger":"kafkajs","message":"[BrokerPool] Failed to connect to seed broker, trying another broker from the list: Connection error: getaddrinfo EAI_AGAIN kafka1","retryCount":1,"retryTime":676}
{"level":"ERROR","timestamp":"2020-08-14T15:49:54.967Z","logger":"kafkajs","message":"[Connection] Connection error: getaddrinfo EAI_AGAIN kafka2","broker":"kafka2:9092","clientId":"my-app","stack":"Error: getaddrinfo EAI_AGAIN kafka2\n    at GetAddrInfoReqWrap.onlookup [as oncomplete] (dns.js:66:26)"}
{"level":"ERROR","timestamp":"2020-08-14T15:49:54.968Z","logger":"kafkajs","message":"[BrokerPool] Failed to connect to seed broker, trying another broker from the list: Connection error: getaddrinfo EAI_AGAIN kafka2","retryCount":2,"retryTime":1426}

My docker looks like this:
version: '2'
services:
  zookeeper:
    image: wurstmeister/zookeeper:latest
    ports:
      - "2181:2181"
  kafka:
    image: wurstmeister/kafka:2.11-1.1.1
    ports:
      - "9092:9092"
    links:
      - zookeeper
    environment:
      KAFKA_ADVERTISED_HOST_NAME: <IP_ADDRESS>
      KAFKA_ZOOKEEPER_CONNECT: zookeeper:2181
      KAFKA_AUTO_CREATE_TOPICS_ENABLE: 'true'
      KAFKA_DELETE_TOPIC_ENABLE: 'true'
      KAFKA_CREATE_TOPICS: "topic-test:1:1"
    volumes:
      - /var/run/docker.sock:/var/run/docker.sock

  # the redis configuration
  redis:
    image: redis
    container_name: redis
    ports:
      - "6379:6379"
    restart: always

This isn't really related to KafkaJS. For obvious reasons, we can't support issues with Kafka itself. I suggest you reach out to maybe Confluent or the Kafka project for support instead. Feel free to use our docker-compose files as reference:
https://github.com/tulios/kafkajs/blob/3b3022e093e74f70c1e8c97a7d873716feb0e351/docker-compose.2_4.yml#L1-L58
You should pretty much just need to rip out the SSL and SASL stuff and it should work. No need to set HOST_IP or anything like that.


Typo in examples/v1.0-receiver-stage-scheduled-transportfile.json

I haven't looked at the rest of the embedded SDP file, but the lack of a "\r\n" between the initial "v=0" and "o=" line jumped out at me...
https://github.com/AMWA-TV/nmos-device-connection-management/blob/master/examples/v1.0-receiver-stage-scheduled-transportfile.json#L10

Fix in typo fix branch, awaiting merge into v1.0.x


322 add support for pydantic logfire for llm agent tracing

This makes a new weather forecast demo that has tracing with logfire set up

I still need to update readme and ensure that logfire key is a required environment variable, but all the configs are ready

who is passing LOGFIRE_API_KEY env var to otel-collector? I think you may need to add this var in docker-compose.
You can also ensure that env var is passed in correctly using env var interpolation,
${LOGFIRE_API_KEY:$?please supply logfire api key}
You can also check for this var in run_demo.sh script.


[docs] yarn command to add @material-ui/icons

Added yarn command to add @material-ui/icons as well.

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

@Inambe It's a great first pull request on Material-UI 👌🏻. Thank you for working on it!


Typo timeOccured in ADHybridHealthService

Should be "Occurred" in the following
https://github.com/Azure/azure-rest-api-specs/blob/bd2aca046b487c2ae2f13cc91d26c5f8aae93571/specification/adhybridhealthservice/resource-manager/Microsoft.ADHybridHealthService/stable/2014-01-01/ADHybridHealthService.json#L4146
https://github.com/Azure/azure-rest-api-specs/blob/bd2aca046b487c2ae2f13cc91d26c5f8aae93571/specification/adhybridhealthservice/resource-manager/Microsoft.ADHybridHealthService/stable/2014-01-01/ADHybridHealthService.json#L4565
https://github.com/Azure/azure-rest-api-specs/blob/bd2aca046b487c2ae2f13cc91d26c5f8aae93571/specification/adhybridhealthservice/resource-manager/Microsoft.ADHybridHealthService/stable/2014-01-01/ADHybridHealthService.json#L5413

@gamitra please take a look


F3 + ESC (Pause Game) crashes minecraft

the title says everything ...
here is the uploaded log
as stated on the log all the mods are disabled and only fabric api, mod menu and mod settings are loaded
https://mclo.gs/rI0WGgk

	java.lang.IndexOutOfBoundsException: Index: 0, Size: 0
	at net.fabricmc.fabric.impl.client.screen.ButtonList.translateIndex(ButtonList.java:130) ~[fabric-screen-api-v1-1.0.10 d882b91560-713a1bc331c59ce.jar:?]
	at net.fabricmc.fabric.impl.client.screen.ButtonList.get(ButtonList.java:48) ~[fabric-screen-api-v1-1.0.10 d882b91560-713a1bc331c59ce.jar:?]
	at net.fabricmc.fabric.impl.client.screen.ButtonList.get(ButtonList.java:33) ~[fabric-screen-api-v1-1.0.10 d882b91560-713a1bc331c59ce.jar:?]
	at se.icus.mag.modsettings.gui.MenuScreensChanger.injectModSettingsButton(MenuScreensChanger.java:62) ~[modsettings-1.1.0 1.18.1.jar:?]
	at se.icus.mag.modsettings.gui.MenuScreensChanger.postGameMenuScreenInit(MenuScreensChanger.java:27) ~[modsettings-1.1.0 1.18.1.jar:?]

I have failed to reproduce this.
I'm not sure how the game ever ended up in this state, but apparently there were no buttons in the in-game menu, which caused a crash when trying to get the first button. I have fixed so this situation will not result in a crash.

Ah,

Will be fixed in 2.0.0.


SA1009 false positive on -> operator

struct H
{
    public ushort c;
}

struct G
{
    public unsafe H* bar()
    {
        return (H*) 0;
    }
}

internal class Program
{
    private unsafe ushort Foo()
    {
        G* pG = (G*)0;
        return pG->bar()->c;
    }
}

It wants the -> operator to be preceded by a space.

Nice catch :+1:
This one should be an easy fix. We already have a few exceptions for other operators in the same analyzer.


Fixing pasteDeleagte crash

This library is ⚠ deprecated ⚠ and is only accepting pull requests for critical bug fixes. Consider using MessageKit for new projects.

Pull request checklist

[ ] I understand that this library is ⚠ deprecated ⚠ and is only accepting pull requests for critical bug fixes.
[ ] All tests pass.
[ ] Demo project builds and runs.
[ ] I have resolved merge conflicts.
[ ] I have followed the coding style.

Contributing guidelines confirmation: ____
This fixes issue
What's in this pull request?
Crash on iOS10 because of the pasteDeleagete property declaration.

is this pull request has been accepted/merged? Coz I can't see the updated code
@adematalay

I does not seem to be merged. You know this library is deprecated and i don't think it will be merged. If you need you may cherry-pick for now.

@adematalay I just heard from you about cherry picking. I googled and got what it is. but I really don't know how to do it in this specific case as  this is not my repo, its a public one. give me an idea plz


Add modified timestamps

Task: Front End: Add Modified Timestamp To Detail Pages
Acceptance Criteria: As a user, I want to see a timestamp indicating last time modified for entities displayed on the detail page.  The application should show the date when the information was retrieved.
Screenshots:
Topics Page

Applications Page

@idrissyed7 See the DCO action for signing the commit.  Then I'll merge.

@idrissyed7 Please update this branch so we can merge it.

@jrw972 It's good to merge.


Version Detection Character Code error

In version-detection/version-detection.md the following segment contains code 49 (letter I) where the Explanation text shows the numeral 1.
Version Detection
To detect the GEDCOM file version, perform the steps below, using the character width and byte order determined
above.  If the end of the file is reached before
all steps can be successfully completed, the file is not a valid GEDCOM file.

Read until one of the following byte sequences is detected ("1 GEDC"):

Width
Order
Byte sequence to look for
Explanation

2
LE
49 00 20 00 47 00 45 00 44 00 43 00
"1 GEDC"

2
BE
00 49 00 20 00 47 00 45 00 44 00 43
"1 GEDC"

1
(N/A)
49 20 47 45 44 43
"1 GEDC"

1
(N/A)
49 20 53 59 53 54
"1 SYST"

The bolded numbers 49 above should be 31.
In paragraph 3 the text "2 VERS" is rendered using the value 32 for the numeral 2.

Continue reading bytes until the following byte sequence is detected ("2 VERS "):

Width
Order
Byte sequence to look for

2
LE
32 00 20 00 56 00 45 00 52 00 53 00 20 00

2
BE
00 32 00 20 00 56 00 45 00 52 00 53 00 20

1
(N/A)
32 20 56 45 52 53 20

Could you advise me if I should provide a pull request for this change.

Thanks for catching this! A pull request would be welcome


fails if the parent directory contains spaces

hi
downstream problem: https://github.com/Klipper3d/klipper/issues/5706
diownstream project members (i guess) claims is problem of this project, wich is used to build components of downstream project
basically, fail build if the parent directory contains spaces in the name
greetings

"fixed" in downstream (the problem is not in this project) (see
greetings


how do i install?

like the says title I want to install this mod!

update I know now

minecraft forge (files.minecraftforge.net) choose your version, make sure its the same on both the mod and the forge version


Not working with multi domain settings

How to set up configuration options for multiple domains? I tried as an array list but it doesn't work. Can you tell how to set this plugin for several domains?
$config['org_email_regex'] = array('/@(.*\.|)domain1\.ab\.pl/i','/@(.*\.|)domain2\.ab\.pl/i');

This setting does not work. There is an error in the logs:
PHP Warning:  preg_match() expects parameter 1 to be string, array given in /path_to_roundcube/plugins/banner_warn/banner_warn.php on line 137

Use regex or
$config['org_email_regex'] = "/@(.*\.|)(domain1|domain2)\.ab\.pl/i";

Thx, it's working now like a charm.

My config dosn't want to work(((
$config['org_email_regex'] = "/@(.*\.|)(example1|example2|example3)\.com\.net\.xyz/i";
Is there anything that can go wrong?
How to implement more domains?

@AlexBeakes your regex is wrong. It will match only @example1.com.net.xyz and similar. Use https://regex101.com/


Ability to trigger upstream tests

We have many repos that have upstream dependencies, e.g. an ORM library that many different applications depend on. It'd be great to be able to trigger tests in upstream dependencies upon completion of testing a repo.
Sidenote: I think it might be possible to hack this with a bash deploy hook that hits the drone webhook endpoint, but it'd have to recreate the last payload somehow...

I wanted to add some comments here from another thread:

I'm thinking the first step is to create a Drone plugin for Drone. The plugin could trigger builds for other repositories using the Drone API. We already have an API library available, which should make this pretty easy to create.
notify:
  drone_trigger:
    repos:
      - octocat/Hello-World
      - octocat/Spoon-Knife
    token: $$DRONE_TOKEN

While a plugin may be less ideal than a build-in solution, I think it is a great starting point for us to spec out the behavior and get something working as soon as possible. A built-in solution will take time and isn't currently a priority with everything else going on, however, the plugin can be created today without any code changes to Drone whatsoever. Plugin are pretty simple to create if anyone is interested in giving it a try.

Would it be simply better to allow for triggering a downstream from a command run by drone-exec? One particularly hard usecase we had to implement with Jenkins (also triggers from post-build actions) was to support gerrit review verification step. Compilation happened in one environment and simple smoke testing in another. We had to resort to various inconveniences to transmit artifacts and synchronize them.

@RomanSaveljev a plugin is the recommended way to move forward. Before anything is built directly into Drone it should be a plugin first. This gives us a chance to test out different options and behavior without impacting the core Drone codebase.

Compilation happened in one environment and simple smoke testing in another

I believe this will be solved by parallel builds. Parallel builds will behave differently than matrix builds because they will run on the same machine and have access to the same /drone volume.

Would it be simply better to allow for triggering a downstream from a command run by drone-exec

If we did build this functionality into Drone I'm not sure drone-exec is the right place because we enforce a strict rule that drone-exec only sends information back to Drone via stdout and the exit code. This is because we can't necessarily count on inter-container communication.
While the inter-container communication limitation would also exist within a plugin, I believe it is acceptable to have a plugin with system-level or configuration requirements. For example, the drone-docker plugin requires aufs or overlay but that restriction isn't placed on Drone itself.

Okay, thanks for the feedback. I am afraid I did not made myself clear. Sorry for that. Our scenario looked like this:

Trigger a job from Jenkins gerrit plugin
Compile the artifacts
Transfer artifacts to another testing job, which runs in a different slave
Compilation job waits for the verdict from the testing job
Compilation job publishes the status for the "Verified" field in the review

Now as I think of it, perhaps something like this renowned Jenkins plugin would solve it gracefully.
I hope proposed parallel builds will be able to handle this kind of setting.
Anyways, apart from being flawed Jenkins has had quite some back and forth, so we need not copy it, but we can learn from it.

@RomanSaveljev thanks for the clarification. Hopefully parallel builds will handle this sort of thing. Once an initial implementation is available I'll let you know. Would love to get your feedback

since the plugin is pretty basic, I have something that is almost ready for testing:
https://github.com/drone-plugins/drone-downstream
it isn't published to the registry yet, but if anyone wants to build and test locally feel free

OK, here it is .... the first plugin for Drone for Drone ...
http://addons.drone.io/trigger/
This plugin will trigger builds for downstream dependencies. You can specify a list of repositories in your yaml, including your Drone token, and you should be good to go. The repository name should be in owner/name or owner/name@branch format.
The plugin will re-run the latest build for the default or specific branch. It will not create a new build entry. It also needs the ability to communicate the with Drone API. If you are running Drone on localhost or running Drone in a container with icc disabled you might hit some issues.

Nice! Does it possible to skip token when triggering own repos/jobs ?
29 окт. 2015 г. 2:38 пользователь "Brad Rydzewski"<EMAIL_ADDRESS>написал:

OK, here it is .... the first plugin for Drone for Drone ...
http://addons.drone.io/trigger/
This plugin will trigger builds for downstream dependencies. You can
specify a list of repositories in your yaml, including your Drone token,
and you should be good to go. The repository name should be in owner/name
or owner/name@branch format.
The plugin will re-run the latest build for the default or specific
branch. It will not create a new build entry. It also needs the ability to
communicate the with Drone API. If you are running Drone on localhost or
running Drone in a container with icc disabled you might hit some issues.
—
Reply to this email directly or view it on GitHub
https://github.com/drone/drone/issues/297#issuecomment-152031371.


fix: Add an option DB_AUTH for PGSQL

In this PR:

Added an option DB_AUTH for PGSQL
Fixed lint issues and typos in README files

Many thanks @alwynpan for your handful of contributions to this repository for the benefit of others. Tagged as 3.12.0

Thank you muchly @tiredofit  for your speedy action and providing such a great tool! Happy Halloween. 🎃


sql autocompletion doesn't work

System information:

Operating system (distribution) and version :  Windows 10 Enterprise
DBeaver version : 6.0.5 May 19th, 2019

Connection specification:

Database name and version : Oracle 12

Describe your question:
SQL autocompletion is not working when pressed ctrl + space, it doesn't list out the column name.
We have multiple schema in the DB
I tried setting ALTER SESSION SET CURRENT_SCHEMA = <SCHEMA_NAME>
but that doesn't work either.

I got the same problem on openSUSE, I posted about it sometime ago #5955

Still an issue for me

@serge-rider
Just wanted to check in and see if there was any update here :) This and no smooth scrolling (#6713, #6536 ) are the only two things causing me to look elsewhere (and then miss all the awesome things DBeaver has that they don't :) )

@serge-rider just following up here. I keep trying every new release of DBeaver. Still an issue for me that auto-complete often offers no suggestions when it should (without pressing ctrl + space and with trying ctrl + space)
macOS 10.15.3 (19D76)
DBeaver <IP_ADDRESS>002151353
Snowflake 4.3.1 using Snowflake JDBC Driver 3.12.0

seems to be obsolete. Cannot reproduce it for now.


[Patch 2.0.x] Downgrade cancellation logs

We've gotten several complaints that session overlogs cancellation exceptions caused by client disconnects. These can be downgraded in severity and verbosity.
This is a backport of https://github.com/aspnet/Session/issues/189.
@muratg @Eilon

Hi, we have a public test feed that you can use to try out the ASP.NET/EF Core 2.0.3 patch!
To try out the pre-release patch, please refer to the following guide:

https://github.com/aspnet/Announcements/issues/274 (info)
https://github.com/aspnet/Home/issues/2254 (discussions and questions)

We are looking for feedback on this patch. We'd like to know if you have any issues with this patch by updating your apps and libraries to the latest packages and seeing if it fixes the issues you've had, or if it introduces any new issues. If you have any issues or questions, please reply on this issue to let us know as soon as possible.
Thanks,
Eilon


Fix Verify responses without "from" fields throwing an exception

If it exists, we assume the first attribute without "from" is always the most recent. We also assume that there will only be one, though the code will probably function fine if there are more than one.

 Verify responses without "from" fields throw an exception

Closing this for now, we're going to refactor and try a slightly different approach to simplify this code.


Detach from fork

The GovITHub repo looks like it won't be maintained anytime soon. At this point the forks have already diverged quite a bit so we should try and switch this into a normal repo.
I'll contact support to see if they can do that for us.

Emailed support and they fixed it, great success. Closing issue.


Increasing Threads/Pools for the Shutil obj?

Hi there, i'm trying to copy some directories very quickly. I'm looking to copy about 1 directory, 1000 times, to separate folders. I'm able to do this with sync shutil in about 2 minutes, and aioshutil in about 1 minute.
My Code:
    async def make_port_files(self):
        """
        copy all the files from base to this directory
        copytree() makes the directories needed recursively.
        """
        while True:
            try:
                await copytree(
                    src=f'{get_project_root()}/base_3proxy/3proxy/',
                    dst=self.port_directory,
                    dirs_exist_ok=True
                )
                break
            except shutil.Error:
                await self.stop_port()
                continue

It's part of a class that does other things. The directory it's trying to copy is about 5MB. Is there a way to make this faster? I'll eventually need to copy thousands of files and it would help if I could go a bit quicker. Please let me know if there's anything I can do to expand on this question.

Hi,
You can change the default ThreadPoolExecutor to your requirement by adding this add the top of application entrypoint
import asyncio
from concurrent.futures import ThreadPoolExecutor

threads = 10
loop = asyncio.get_event_loop()

loop.set_default_executor(ThreadPoolExecutor(max_workers=threads))


As an App Admin, I want DokuWiki content moved to Gogs to minimize manual effort.

From @bspidel on January 21, 2016 22:19
Acceptance Criteria:

I can see structure as user/lang-project

All dokuwiki markup is converted to gogs markdown

Copied from original issue: unfoldingWord-dev/sysadmin#41

From @jag3773 on February 4, 2016 13:29
@richmahn Do we have a script somewhere that can convert our DW syntax to MarkDown format?  I'd like to try getting an OBS project moved over to see if it will look as we expect.

From @jag3773 on February 4, 2016 22:21
I think we'll want to take the following as our procedure for the move:

[ ] Split out project from language, as in comment above
[ ] Push/Create the repo in git.door43.org following the lang-project nomenclature
[ ] Convert to MarkDown (possibly create a tag for this commit as something like "md" ?)

The advantage of doing it this way is that we'll have all the previous revision history using the DW syntax and then we'll have a single commit for the switch to MarkDown.
For resources that are published already, like en-obs, I'd also like to create a tag with the currently published version (v 4 in this case).  That will help facilitate change log creation later on down the road.

From @jag3773 on February 4, 2016 18:8
Couple notes, we'll need to split the monolithic language repos into their language-project counterparts.  The following links give information on how to do that while maintaining the version history:
https://help.github.com/articles/splitting-a-subfolder-out-into-a-new-repository/
http://blogs.atlassian.com/2014/04/tear-apart-repository-git-way/

I think we want to match https://github.com/unfoldingWord-dev/ts-requirements/wiki/ts.Translator.TargetTranslation#possible-long-term-solution for formatting the names of the repos.  Important part is to use _ as the separator character.

This is now ready. Once we switch Dokuwiki to using postgres, thus users.auth.php no longer accepting new users, we need to run the script on production to get all users from that file into a SQL file. Then that SQL file needs to be imported into the postgres database that Gogs uses.


Consider setting working directory to project folder

So generated paths with ~/ will be project-relative rather than My Documents relative. If someone wants to take project to Mac they won't be placing it in My Documents. In case of multiple projects in the same solution we can set WD to the startup project.

+1 on this. Make sure that we get this done for both F5 and the Run Shiny App both call the Set Working Directory to Project Location command before they start running.

Working directory isn't set by startup project. Every time I open my solution I need to set working directory to startup project. I have multiple R projects in my solution.

We should stop saving working directory and rather apply it per project or use default.  This especially prominent in Linux since it sets directory to OS-specific value that does not apply to Windows


Merge pull request #4 from annesauve1/main

Merge pull request #3 from annesauve1/annesauve1-patch-1

merging to main


Lexer Performance optimizations 0.27.0

Optimized short strings and short regular Expressions to use
charCodeAt for faster matches.
This is relevant for many grammars, (Curly Braces, commas, semicolons, ...).
A JSON lexer showed ~15% performance improvement
and a CSS Lexer showed ~30% performance improvement.

fixed in #430


Grafana 8 with js | Yet to test completely

USE this branch only for emergency grafana8 request and do not merge it until the migration is complete

Fixed all api calls  other   than query response
No issues with golang code atm

Todo

Send over the test build to anyone possible and test.
Collect issues  and do the ts migration in parallel to this

Things to remember

The other plugin from OCI - grafana logs  cannot be run in parallel to this as it doesn't support grafana 8  unless it it run in a separate grafana installation or instance.

Known issues :

On creating a new panel and creating a variable, if a variable is unused, it gives metric request error.  (error occurs occasionally)
Once, the variable is used and then changed to a value that is not set to the variable, the error goes away.

Screenshots :

Sample build of grafana-8 with js
/Users/athmural/Go/src/oci-grafana-metrics/oci-metrics-datasource-g8.zip

@wbrowne  The backend now sends the timestamps perfectly. Appreciate your help

Thanks @ryantxu  for your inputs and help in the other PR for the same.

Nothing yet guys? It's been 3 months since this issue was open.


Update Makefile

Description
Changes proposed in this pull request:

...
...
...

Related issue(s)

@suleymanakbas91: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

pre-master-kyma-gke-integration
ce2af2567906adc0d959dc6c3d5346f476d82f6e
link
/test pre-master-kyma-gke-integration

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@suleymanakbas91: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

pre-master-kyma-gke-integration
ce2af2567906adc0d959dc6c3d5346f476d82f6e
link
/test pre-master-kyma-gke-integration

pre-master-kyma-gke-upgrade
ce2af2567906adc0d959dc6c3d5346f476d82f6e
link
/test pre-master-kyma-gke-upgrade

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@suleymanakbas91: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

pre-master-kyma-gke-integration
ce2af2567906adc0d959dc6c3d5346f476d82f6e
link
/test pre-master-kyma-gke-integration

pre-master-kyma-gke-upgrade
ce2af2567906adc0d959dc6c3d5346f476d82f6e
link
/test pre-master-kyma-gke-upgrade

pre-master-kyma-gke-central-connector
ce2af2567906adc0d959dc6c3d5346f476d82f6e
link
/test pre-master-kyma-gke-central-connector

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@suleymanakbas91: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

pre-master-kyma-gke-integration
ce2af2567906adc0d959dc6c3d5346f476d82f6e
link
/test pre-master-kyma-gke-integration

pre-master-kyma-gke-upgrade
ce2af2567906adc0d959dc6c3d5346f476d82f6e
link
/test pre-master-kyma-gke-upgrade

pre-master-kyma-gke-central-connector
ce2af2567906adc0d959dc6c3d5346f476d82f6e
link
/test pre-master-kyma-gke-central-connector

pre-master-kyma-integration
ce2af2567906adc0d959dc6c3d5346f476d82f6e
link
/test pre-master-kyma-integration

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.


Ability to pickle the Catalog

Feature Request / Improvement
This allows distribution of the Catalog object within Ray.

https://github.com/apache/iceberg-python/issues/513#issuecomment-2009875953

@dev-goyal I see that you've merged ray-project/ray#46889 curious if you think pickling the Catalog or Table is still valuable to the Ray project

More context:
https://github.com/ray-project/ray/pull/42235#discussion_r1520929199

@dev-goyal I see that you've merged ray-project/ray#46889 curious if you think pickling the Catalog or Table is still valuable to the Ray project

I think it could help, but for me - I did not find recreating the catalog each time to add much overhead.

So, it will likely just make the code cleaner is all

Makes sense, the difference is directly pickling the catalog object so that it can be distributed to ray workers versus the current implementation which pass the catalog_kwargs to reconstruct the catalog at the worker node


Allow customizing of route names

Currently, only the following routes ,'/live' and '/ready' are exposed.
It would be great to allow customization of the routes eg. '/liveness' or '/readiness' or '/super/site/readiness'.

Pull request created in #20


Nelson 0.4.8

[X] Have you signed the Contributor License Agreement?
[X] Have you checked that there aren't other open pull requests for the same manifest update/change?

[X] Have you tested your manifest locally with winget install -m <manifest>?

https://www.microsoft.com/en-us/wdsi/submission/c2da152f-5c1f-4e4f-b9e2-8ab69b16957b
Microsoft Reviewers: Open in CodeFlow

Already submitted
see https://www.microsoft.com/en-us/wdsi/submission/c2da152f-5c1f-4e4f-b9e2-8ab69b16957b

It will be interesting to not have to do this at each upload.
This binarie have no digital signature
Thanks for winget ;)


Find a good solution to tree_hash's getrandom dependency

One of the libraries we use in the 08-wasm ethereum light client, tree_hash, has a downstream dependency on the library getrandom. The dependency graph we are talking about is tree_hash -> ethereum_hashing -> ring -> getrandom.
The reason getrandom is problematic is because it doesn't compile to CosmWasm (error at the bottom of the issue). Enabling "js" somehow would not solve the problem since CosmWasm doesn't have support for that either.
The current solution solved the problem at the ´tree_hash` level by forking and removing the ethereum_hashing dependency, which just does Sha256 with either sha2 or ring, depending on the platform.
Some potential solutions include:

Forks (and maintain ourselves)

Keep a fork of tree_hash like we do today - potentially finding a less invasive solution than the current one that makes changes from upstream easier to integrate.
Forking the etheruem_hash library and remove the ring dependency (or add a feature flag that disables it entirely)

Upstream a solution (i.e. avoiding having a fork - at least long-term)

Find a way to add a feature flag in either tree_hash or etheruem_hashing that we can upstream and therefore avoid having a fork to maintain
Go really deep and try to find a solution to add a feature flag in ´ring` that enables it to compile to CosmWasm (and get it upstreamed)

Error from running: cargo build --release --lib --target wasm32-unknown-unknown -p cw-08-wasm-etheruem-light-client
   Compiling getrandom v0.2.15
error: the wasm*-unknown-unknown targets are not supported by default, you may need to enable the "js" feature. For more information see: https://docs.rs/getrandom/#webassembly-support
   --> /Users/gg/.cargo/registry/src/index.crates.io-6f17d22bba15001f/getrandom-0.2.15/src/lib.rs:342:9
    |
342 | /         compile_error!("the wasm*-unknown-unknown targets are not supported by \
343 | |                         default, you may need to enable the \"js\" feature. \
344 | |                         For more information see: \
345 | |                         https://docs.rs/getrandom/#webassembly-support");
    | |________________________________________________________________________^

error[E0433]: failed to resolve: use of undeclared crate or module `imp`
   --> /Users/gg/.cargo/registry/src/index.crates.io-6f17d22bba15001f/getrandom-0.2.15/src/lib.rs:398:9
    |
398 |         imp::getrandom_inner(dest)?;
    |         ^^^ use of undeclared crate or module `imp`

For more information about this error, try `rustc --explain E0433`.
error: could not compile `getrandom` (lib) due to 2 previous errors
warning: build failed, waiting for other jobs to finish...
The following warnings were emitted during compilation:

warning<EMAIL_ADDRESS>error: unable to create target: 'No available targets are compatible with triple "wasm32-unknown-unknown"'
warning<EMAIL_ADDRESS>1 error generated.

error: failed to run custom build command for `ring v0.17.8`

Potentially related issues: https://github.com/CosmWasm/cosmwasm/issues/1143
In this ring issue, the maintainer is mentioning being open to accepting PRs for enabling more wasm32-unknown-unknown environments: https://github.com/briansmith/ring/issues/1921

Here's where in ring it uses getrandom:
https://github.com/briansmith/ring/blob/8eee46910095cee0f9f6e07d7da6115f749bf52c/src/rand.rs#L121-L166
You see it's only included on certain OS's. Specifically for wasm32, only wasi applies, not wasm32-unknown-unnown unless a wasm32_unknown_unknown_js feature is activated.
However in Cargo.toml, getrandom is imported regardless of OS or feautre:
[dependencies]
getrandom = { version = "0.2.10" }

You should make an issue in ring to fix this... tell them to make a condition on importing getrandom:
[target.'cfg(...........)'.dependencies]
getrandom = { version = "0.2.10" }

Not much to add to Larry's answer.
Maybe some context: this is not CosmWasm specific. By default no Wasm supports randomness since it is a sandbox that cannot access any randomness API. The hint regarding the "js" feature is misleading unfortunately as it only helps for Wasm in JS environments like browsers. Usually in such cases it can be tricky to find the exact user of getrandom. But getting that out of the dependency tree is important and the way to go.


CSGO NEW UPDATE and Osiris Crash

FYI: CSGO NEW UPDATE and Osiris Crash when you enter the map
Regards

me too, even if i use the clean copy of Osiris.

It crashes so I have to inject while im in the match, and opening consle just straight up crashes the game

It crashes so I have to inject while im in the match, and opening consle just straight up crashes the game

Thx, it s seems to be a temporary solution. I think we can inject when player in the match and don not open the consle.

Should be "fixed".


[Bug] Additional args are not being passed to the action menu

I might have stumbled upon another problem: I set my specific rofi theme via the --selector-args '-theme /home/das-s/.config/rofi/appmenu.rasi' . Then, using the new action menu it appears to reset the theme when entering the selector menu. A possible cause might be here.
Is there a reason to pass an empty list to the additional_args parameter or is this just an oversight?

Thanks for finding this! It was not done on purpose, but now I wonder if there are any arguments that work for the main menu but not for the autotype menu... I guess we'll see 😉


Hi, the smartphone should have root for this?

I tried to install but stuck on

cd metasploit-framework
cd: no such file or directory: metasploit-framework

U have to wait when it's install bundle

No Root required

i have only this problem, it can't create the Gemfile.lock
1 gem installed
cat: Gemfile.lock: No such file or directory
Hacker X Phantom                            \HXP_META.sh: line 86: cd:

but i successfully installed and run this version , maybe you can correct the installation script with some tweaks from there


Stage chunks in git

It would be awesome if VSCode could allow to stage chunks in git and not only entire files.

Not very discoverable... I know...


drivers: lora: Add user_data to lora_recv_async

Hi,
Right now, lora_recv_async does not allow to pass any user_data to callback. This makes the usage harder for example when wrapping API in c++.
This commit addresses this issue but it breaks the API, so we should discuss it first. Let me know what you think

Hi @carlescufi,
Fixed the issues with commit message, hopefully it is correct now.

You're changing a public API so it needs to be documented in the migration guide what users need to do to update their code.

Done

@JordanYates note how CI doesn't seem to have run tests on the sample despite recent change?

@JordanYates note how CI doesn't seem to have run tests on the sample despite recent change?

I don't believe this PR was rebased since that change based on the file contents:
e.g. https://github.com/adejewski/zephyr/blob/add_user_data_to_lora_receive_callback/samples/drivers/lora/send/sample.yaml


chore: pulled in original sources

Contains the code donated by Accenture as part of the lab
Commit history could not be kept due to lack of signatures.
List of contributors were attached in the README as thanks to the original contributors.
READMEs were merged: first the proposal README, then the original readme from Accenture.
The contents in this PR are taken from the dev branch of the original internal Accenture code repository.

Abandoning this in favor of https://github.com/hyperledger-labs/blockchain-integration-framework/pull/3 as per our internal discussions.


Removed target: _blank and changed new_window to False in 'view live' and 'view draft' buttons #6123

Removed target: _blank and changed new_window=True to new_window=False for the 'view live' and 'view draft' buttons.
This prevents 'view live' and 'view draft' from opening in a new window.
@allcaps kindly review.
Closes #6123

@sagar-wal  Thank you!
Just wanted to let you know that this is on my todo list. I'd like to check it out and try it in a browser.

LGTM. Code looks good, but I haven't tested it in the browser yet. Let's user test it before merging it (@allcaps)

I will rebase locally and get this merged in, see notes & tests below.
Checked the following - all working as expected

page explorer listing - view live
page explorer listing - view draft
Save page message buttons on publish - view live / view draft (for workflow)
create page message buttons - view live / view draft (for workflow)

Note: The home (dashboard) page listing - most recent edits / awaiting review still open in a new tab.

I will rebase locally and get this merged in, see notes & tests below.
Checked the following - all working as expected

page explorer listing - view live
page explorer listing - view draft
Save page message buttons on publish - view live / view draft (for workflow)
create page message buttons - view live / view draft (for workflow)

Note: The home (dashboard) page listing - most recent edits / awaiting review still open in a new tab.

Merged in via 592408ac3de0dc1a180b961bcf0e5fb59447a695


Fix crash in GIProbe::bake

The problem happened when the passed from_node was null and the GIProbe node had no parent node.
Fixes #45978

Thanks!

Cherry-picked for 3.2.4.


QIDO performance improvements

Description
Revision the API version to 2, to support smaller default QIDO response tags that can be returned by SQL wide tables(Study and Series)
Results will be pulled from blob if more than the default indexed fields are requested.
Related issues
AB#98125
Testing

docs update

we need to update URSA scanning to test new version

I know we talked lot about this versioning, but I have a question around where we include the new changes that we are doing to the controller, should we add it to v1 or v2?
For e.g the new /rendered  endpoint should it work in v1 or v2? Don't we then add new version of the controller like StoreV2Controller to explicitly say it works in V2? Some features need a flag, or the feature needs to be reverted in case of issue with a customer so in that case, is all the features that are added under v2 gets blocked?


HDDS-4903. Add Ozone logo to the website

What changes were proposed in this pull request?
Add Ozone logo to the website. I prefer this version over #14 and #15.
https://issues.apache.org/jira/browse/HDDS-4903

@adoroszlai, please take a look, thanks.

Thanks @kaijchen for the patch.  Looks good in Firefox and Safari, but Chrome displays the logo in huge size.

Thanks @kaijchen for the patch. Looks good in Firefox and Safari, but Chrome displays the logo in huge size.

It worked for me on Chrome 99 Windows.

And also worked on Chrome 99 on macOS. Please check again @adoroszlai.

Does not render correctly on Google Chrome.

I'll revert the change. @kaijchen please try again.


Throw warning in Script-Editor on changing array you currently loop

Godot version:
3.1.1
OS/device including version:
Windows 10
Issue description:
When i (accidentally) remove an element from the array im currently looping over in a for loop, it just breaks the loop.
Here imo it would be better to throw a warning in the editor. Especially for new-comers, as this is something you might tend to do when you are not that experienced.
The breaking of the loop for a new-comer is just behaviour that is hardly visible and difficult to find out without Debugging.
I know its a No-No in Programming to do that, but nothing is preventing you here from doing so.
Steps to reproduce:
Make array with elements
iterate over array,
delete element from array while looping

I wonder if someone can implement something like ConcurrentModificationException for GDScript

@Anutrix Do you mean a compiler-exception or runtime-exception?

Duplicate of https://github.com/godotengine/godot/issues/32261 (same cause).

@Anutrix Do you mean a compiler-exception or runtime-exception?

Both or either.


⚠ Bofrost.be has degraded performance

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

HTTP code: 200
Response time: 1360 ms

Resolved: Bofrost.be performance has improved in 7edc629 after 10 minutes.


[Change Request] Created rules should always have at least 1 rule selected

Description
It doesn't really make sense to create a role with zero permissions. Goal of this issue is that the form to submit the new role is invalid unless at least one permission is selected.

Fixed in #194


type enter in title, cursor go to body editor

Issue Summary
in Japanese, enter key is used for Kana-to-Kanji conversion finalize in addition to insert a new line.
when I type some title in Japanese and type enter to finalize conversion, the cursor go to post body editor.
Steps to Reproduce

type some title in Japanese
convert to Kanji
type enter for conversion finalize
the cursor go to post body editor

Technical details:

Ghost Version: 1.8.1
Node Version: v6.11.2(according to https://hub.docker.com/_/ghost/)
Browser/OS: firefox 55.0.3 on MacOS
Database: SQLite

running on docker container on Rancher.

Pressing <kbd>Enter</kbd> to move from the title to the body is the expected behaviour.
Do you know if there's anything specific about the kana-to-kanji conversion that is detectable via JS?

Pressing Enter to move from the title to the body is the expected behaviour.

I thought it...😥

Do you know if there's anything specific about the kana-to-kanji conversion that is detectable via JS?

I don't know...
If i know and I can write JS well, I will fix it and send PR.
So I'd like to make a suggestion.
Can you make a toggle config for Enter binds to move cursor or not?

It sounds like it may be possible to switch the Enter key detection to the keydown event according to the discussions here:
https://stackoverflow.com/questions/22705698/distinguishing-between-enter-keypress-for-japanese-kanji-suggestion-and-other
https://meta.stackexchange.com/questions/66646/confirming-context-menu-entries-via-enter-when-entering-comments-triggers-commen/217498#217498
I haven't tested it but that's a good place for anyone who wants to tackle this to start.

Can you make a toggle config for Enter binds to move cursor or not?

No, we have a policy not have configs for anything like this because if we start doing that it never ends then there's a million and one config options which are impossible to support, debug, and develop around because there are so many possible variations.

this problem is solved with 2.0 new editor.
thanks.


feat(HelpText): take reactnode instead of string

Change the type of helptext children from string to ReactNode.

Hello!
Whats is the use-case for needing a ReactNode instead of string? Font styling?

We have som cases of our app-owners wanting some markdown functionality in their helptexts, often links and lists. Our old implementation of helptext supports this. Although i agree with your point that helptexts should be a short descriptive text, it would be more pain free to be backwards compatible with the forms that have been created using markdown in helptexts.
Each implementation of the HelpText component would still be free to enforce the use of a simple string for their solution.

We have som cases of our app-owners wanting some markdown functionality in their helptexts, often links and lists. Our old implementation of helptext supports this. Although i agree with your point that helptexts should be a short descriptive text, it would be more pain free (for us) to be backwards compatible with the forms that have been created using markdown in helptexts.
Each implementation of the HelpText component would still be free to enforce the use of a simple string for their solution.

Ah, yeah. Makes sense! Use-cases like; if you have some informative text, you might have a link to somewhere for more information?
Is it possible to get a screenshot with examples of how this looks? This would be important feedback for the design system as whole in terms of design and guidelines.
Can this wait until next week, so I can do a check with the designers? Once we open the possibility for sending ReactNode as children, its opens a whole new can of worms as users can put anything there. There is also the case of WCAG2.1 where having font styling, links or any interactive content inside a tooltip is not recommended. Good explanation here also.

@mimarz It could probably wait until next week. Here is an example of todays popover in an Altinn3 app containing a link in the format [displaytext](link) as well as linebreaking after the link using <br/>:


can't build mitmproxy 0.12.1 on pfSense 2.2.2/FreeBSD 10.1 -- can't build cffi dep, can't find limits.h

pfSense 2.2.x is now based on FreeBSD 10.1, so I have the luxury of using some more modern software. Unfortunately FreeBSD ports still includes mitmproxy-0.10.1, where Transparent mode isn't supported. I understand this was fixed in mitmproxy-0.11.
I'm a noob at massaging build dependencies manually so I may well be missing some obvious -devel library.
All attempts to either 'pip install mitmproxy' or to build from source fail at the same point.
Here's netlib failing to build.
running build_ext
building '_cffi__x88a8fd90x94e38fe2' extension
creating /tmp/pip-build-SrTHd4/netlib/netlib/__pycache__/netlib
creating /tmp/pip-build-SrTHd4/netlib/netlib/__pycache__/netlib/__pycache__
cc -fno-strict-aliasing -O2 -pipe -fstack-protector -fno-strict-aliasing -DNDEBUG -fPIC -I/usr/local/include/python2.7 -c netlib/__pycache__/_cffi__x88a8fd90x94e38fe2.c -o /tmp/pip-build-SrTHd4/netlib/netlib/__pycache__/netlib/__pycache__/_cffi__x88a8fd90x94e38fe2.o -w
In file included from /usr/local/lib/gcc5/gcc/x86_64-portbld-freebsd10.1/5.1.0/include-fixed/syslimits.h:7:0,
                 from /usr/local/lib/gcc5/gcc/x86_64-portbld-freebsd10.1/5.1.0/include-fixed/limits.h:34,
                 from /usr/local/include/python2.7/Python.h:19,
                 from netlib/__pycache__/_cffi__x88a8fd90x94e38fe2.c:2:
/usr/local/lib/gcc5/gcc/x86_64-portbld-freebsd10.1/5.1.0/include-fixed/limits.h:168:61: fatal error: limits.h: No such file or directory
compilation terminated.

Full log: http://pastebin.com/YYzz9qCX
(I've also tried with gcc-4.8.4 and gcc49-4.9.3, it didn't make a difference.)
Here's cffi failing to build:
http://pastebin.com/PRaDaFM1
Here's the dump of all the packages I currently have installed.
http://pastebin.com/Czt0F8cH
edit: here's what I have installed via pip  http://pastebin.com/yiZ9jWVx
Although I have py27-cffi-0.8.6_3 and libffi-3.2.1 installed, but neither seems to satisfy the dependency.
You can grab current pfSense here: https://www.pfsense.org/download/mirror.php?section=downloads
Thanks in advance for any pointers on getting 0.12 installed.

This is apparently answered in https://forum.pfsense.org/index.php?topic=94838.0. Once you get cffi installed successfully, mitmproxy should follow trivially.


Use "Truth Set" records in test cases and examples

Change
https://github.com/Senzing/g2-sdk-go/blob/928a26aac90af212bf34899184b8c53575bbc40a/testhelpers/main.go#L20-L44
to records that are in the Truth Set.  Then modify the *_test.go files to reflect the new data.

Implemented via https://github.com/Senzing/g2-sdk-go/issues/24


Enable Instance.shared to be determined by Configuration

Every Instance CRD has a shared property, which determines whether the device/capability can be shared by multiple nodes. If an instance is not shared, every node that discovers it will create a new Instance CRD, while if it is shared, that Instance CRD will be shared by all nodes that can discover the instance/capability.
Currently the sharability of an instance is defined by the protocol that was used to discover it. For example, currently, all instances (more specifically ip cameras in this protocol) discovered by the ONVIF protocol are marked as shared, as can be seen in the ONVIF DiscoveryHandler's implementation of are_shared(). Similarly, all instances discovered with udev are marked as unshared, since the udev protocol only discovers devices on a specific node.
Should the "is this shared?" decision be made elsewhere, such as in the Configuration, allowing an operator to specify sharability? Should the "is this shared?" decision be made on a device by device case?
One thing to note about the validity of the current solution of defining sharability by protocol: Right now, if ONVIF cameras were allowed to be unshared, it could result in multiple instances for a single camera, making it hard to regulate capacity and possibly leading to an overloaded device. Also, having two Instance CRDs for a single device seems counter-intuitive.

The new Discovery Handler interface puts the decision on the Discovery Handler as to whether a device is shared or not. When a Discovery Handler registers with the Agent, it specifies whether it is discovering shared or unshared devices. This may need to change if there was some reason a Discovery Handler could discover both shared and unshared devices.


Datepicker w/ Moment parseDate parsing strings results in good date object not Invalid Date

fixes #1906
Code made two attempts to parse but passed the failed result from the first attempt to the second attempt instead of the raw date string.

@futuremint please sanity check now that i moved the dist build before the test.


CogAgent lost Windows support

CogAgent now requires triton, which not supported on Windows.

Try to use huggingface version and it is not require xformers package and not need triton

@1049451037 thanks, it works on Windows now.


Not working with Jest

Hi,
I installed this using the vue cli plugin found here. Works great, except when running unit tests with Jest, I get this:
 FAIL  tests/unit/example.spec.ts
  ● Test suite failed to run

    Jest encountered an unexpected token

    This usually means that you are trying to import a file which Jest cannot parse, e.g. it's not plain JavaScript.

    By default, if Jest sees a Babel config, it will use that to transform your files, ignoring "node_modules".

    Here's what you can do:
     • To have some of your "node_modules" files transformed, you can specify a custom "transformIgnorePatterns" in your config.
     • If you need a custom transformation specify a "transform" option in your config.
     • If you simply want to mock your non-JS modules (e.g. binary assets) you can stub them out with the "moduleNameMapper" config option.

    You'll find more details and examples of these config options in the docs:
    https://jestjs.io/docs/en/configuration.html

    Details:

    /Users/lachlanmiller/javascript/vue/tsx/src/components/HelloWorld.tsx:7
            return (
Ok
);

any ideas?

vue-tsx-support itself is tested by jest.
see package.json and files under test/jest as an example.

Hey,
Thanks for the reply. I had a look, but I must be missing something. I was able to get it to work using babel-jest and @babel/preset-typescript. I also followed up here: https://github.com/vuejs/vue-cli/issues/3100.
I'll close this for now, since it looks like it was something I was doing incorrectly. Thanks again, this library is excellent.


Deployment issue on artifactory

Hello everyone,
We are trying to upgrade bamboo but we are facing an artifactory plugin issue.
The plugin doesn't deploy release in the selected artifactory repo but keep using the one defined by default in the plan job instead. Note that we use maven.
Bamboo version: 5.9.4 and 5.9.7
Plugin version: 1.8.0, 1.8.1, 1.8.2 and 1.9.0
Any help or comment appreciated.
Thank you.

@JGroselle, we are looking into this and will upstate you soon. Thanks for reporting this.

Hi,
just adding some info, the buildinfo while performing the release set correctly the bamboo variable to the selected repo in the Artifactory Maven3 Task.
Ex: bamboo_releaseRepoKey=aRepoOnlyAcceptingReleasedArtifact-releases-local
But the in the task we have by default snapshot repo :

And in the release page we select upon the release the release repo :

But when the task is launched the default "lib-snapshot-local" is taken instead of the one selected for performing the release "lib-releases-local".
I hope this will help to sort the issue.

@JGroselle and @33man,
I was unable to replicate this issue. I used version 1.9.0 of the plugin with Bamboo 5.9.2:

I create a maven and a gradle task, each in a separate plan.
I configured each of the tasks to publish to a snapshot repository in Artifactory.
After a successful build, in the "Artifactory Pro Release Staging" I selected a release Artifactory repositories and had my release artifacts published to the selected release repository.
Is there something I missed in your replication steps?
Did the "Artifactory Pro Release Staging" page render properly? Do you see any errors in the Bamboo logs? Do you see the "Release Value" and "Next Integration Value" fields with the suggested release and snapshot versions?
Please share what you see - we want to help you perform the upgrade.

OK - I managed to reproduce the issue now. Will update soon.

Ok, quite happy to read that you have pointed the problem too.
We are waiting on you, you will be our savior ;)
Thank you for your time.

@eyalbe4 Hi, thanks for the support, this is what we have upon a successful build. (The default target repo (Publishing Repository) is a snapshot repo (libs-snapshots-local) but upon a release we select manually the releases only repo (libs-releases-local) but this change was never taken in account and artifactory return a HTTP 409 error as the plugin try to deploy a Release dependency into a snapshot only repository)

I created a snapshot version on top of release 1.9.0 of the plugin with a fix for this issue.
I added the download link to this version as a comment on BAP-312
@JGroselle and @33man, please try it out and let me know if it fixes the issue for you.

Hi,
just tested the default task with snapshot repository as default for compilation and then i triggered the  release selecting a different repository and now it works fine !
I only see that some keys have changed in the buildinfo file
1.9.0
"buildInfo.env.bamboo_releaseRepoKey" : "libs-releases-local"
"buildInfo.env.releaseRepoKey" : "libs-releases-local"
1.9.X
"buildInfo.env.bamboo_release_management_repoKey" : "libs-releases-local"
"buildInfo.env.release.management.repoKey" : "libs-releases-local"
Thanks for your quick help !

Hello @eyalbe4, for what we have tested now it works !
A huge thank you for your time !

Anytime guys :)


refactor(UserProjectsWrapper): align projects title with project title

This PR just add a margin-top to the headline "Meine Projekte" so that it aligns well visually with the selected project's headline (see screenshot).

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


-basin_keys in PlotMOverNAnalysis.py

lsdviztools version: latest
Python version:
Operating System:

Description
Chi mapping worked - expected output. Visualising all the data with  PlotMOverNAnalysis.py & -all True worked.
Visualising the data for just three basins - with -basin-keys 7,11,13 - failed, although this particular argument is listed in the "usage" of  PlotMOverNAnalysis.py.
Is it not possible to only visualise/plot selected basins for this "Full visualisation routine"?
What I Did
The final basin keys are:
[7, 11, 13]
There are 3basins that I will plot
I am making a plot for basin: 7
Traceback (most recent call last):
File "PlotMOverNAnalysis.py", line 493, in
main(sys.argv[1:])
File "PlotMOverNAnalysis.py", line 430, in main
SA.SAPlotDriver(this_dir, args.fname_prefix, FigFormat = args.FigFormat,size_format=args.size_format,
File "/LSDTopoTools/LSDMappingTools/LSDPlottingTools/LSDMap_SAPlotting.py", line 393, in SAPlotDriver
BinnedWithRawSlopeAreaPlot(DataDirectory, DEM_prefix, FigFileName=FileName,
File "/LSDTopoTools/LSDMappingTools/LSDPlottingTools/LSDMap_SAPlotting.py", line 851, in BinnedWithRawSlopeAreaPlot
sources = np.unique(RawDF['source_key'].as_matrix())
File "/usr/local/lib/python3.8/site-packages/pandas/core/generic.py", line 5274, in getattr
return object.getattribute(self, name)
AttributeError: 'Series' object has no attribute 'as_matrix'

The basin_keys works on plotting the chi profiles but is not implemented for the raster plots at the moment. This will probably not be implemented any time soon since it involves adjusting several layers of the routines.


Create Tab Component

Hi.
Welcome Tab Component :tada:
Closes issue #21

👋 is this still WIP?

@mariolamacchia Yeah. Need tests and examples

@mariolamacchia sorry that this PR takes that long. I will have more time dedicated to it.

@mariolamacchia there are tow things left to implement and I need to write some tests too. Even though you approved it I think I will finish this PR in this week and then we can merge it.

@mariolamacchia all features that react-semantic-ui tabs has are implemented now. Plus disabled state for the sui-tab-pane as a bonus. All I need to do now is to write some tests. Feels like it's close to the finish :cake:

@mariolamacchia You can go ahead and merge this PR because I don't know how much time I will spend on these tests. However if you want tests then let me know.

@ArStah merge maybe?

Sorry I missed your message. Looks great, thanks a lot for your help! ❤


Refactor formatter

This PR needs the changes from PR https://github.com/doorgan/sourceror/pull/79 .

Pull Request Test Coverage Report for Build 8ee1189d1f7a6a01af82f4229bc56268110fa737-PR-13

15 of 15 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.7%) to 98.328%

Totals

Change from base Build a690c59868f10ab6cbbe460a092508b4db6b462d:
0.7%

Covered Lines:
294

Relevant Lines:
299

💛  - Coveralls


client: replace ByteString with ByteSequence

exposed APIs should take ByteSequence instead of ByteString as the argument type.
In this way, jetcd APIs doesn't depend on Guava ByteString.

lgtm after CI is fixed.


Indicator and disable for form submit

Buttons have great build-in functionality - when triggered, display loading indicator and set 'disabled' class
But if I have a form with submit button:
<form ic-post-to="/items/add" ic-target="#items">
..fields...
<button type="submit">Add <i class="fa fa-spinner fa-spin"></i></button>
</form>
And click 'Add' button - no indicator and no diasble. Becouse not submit was triggered , but the form itself.
Forms with submits is very common use-case, is there way to automatically display  indicator and disable submit-buttons then submitting form?
Currently, indicator can be displayed via ic-indicator attribute at form, a little clumsy, but ok.  But how to declaratively disable submit button?

I do it like this:
$(document).on('beforeSend.ic', function (event, el, data) {
  if (el.is('form')) {
    // disable button
    el.find('input[type="submit"], button[type="submit"]').prop('disabled', disable);

    // ensure we don't submit if already submitted
    if (el.hasClass('form-submitted')) {
        event.preventDefault();
    } else {
      // show spinner
      const spinner = el.find('.fa-spinner');
      if (spinner) {
          spinner.show();
      }
      el.addClass('form-submitted');
    }
  }
});

This should be part of the default behavior for forms.  I will look at adding it.

on the code disabled css class is added.
https://github.com/intercoolerjs/intercooler-js/blob/master/src/intercooler.js#L343
Must be a disabled property!

PR #284

some news about this ?

Hi @zodman I have looked into this.  It looks like the disabled attribute only apply to inputs:
https://www.w3schools.com/tags/att_disabled.asp
Not to forms.  Do you have a link specifying otherwise?

let me edit the PR to apply to all inputs.. wait for it.

patch apply!