Feature/376 capex per case not shown in project view

History:

Task "Capex per case not shown in project view #376" was created as the Barchart within ProjectView did no longer display the data, as this was set to [0,0] even if the fetched project has values, ex [2,2].
Debugging showed that this no longer happened when commenting out the functionality for StoreRecentProject, so the bug occured within the change of: "Update recent projects storage strategy #357"

What has been done:

Within Project.ts, no longer .Strip() (Emptying out the project data)

Squashing this in main

Deleted branch


upload invoice throw exception

Method name
Name of the method that doesn't work e.g. getOrders
addOrderInvoiceFile
Description or exception details (optional)
Describe what happens or paste exception details
                    AddOrderInvoiceFile addOrderInvoiceFile = new AddOrderInvoiceFile();
                    addOrderInvoiceFile.InvoiceId = invoiceId;
                    addOrderInvoiceFile.File = File.ReadAllBytes(fileName + ".pdf"); //file size 123KB
                    addOrderInvoiceFile.ExternalInvoiceNumber = docNum;
                    var addFileResponse = await baseLinkerClient.SendAsync(addOrderInvoiceFile);

System.UriFormatException: 'Nieprawidłowy identyfikator URI: ciąg URI jest zbyt długi.'
Probably caused by:
Exception: Uri.EscapeDataString data to long
Or
FormUrlEncodedContent
Solution:
        public static string EncodeString(string str)
        {
            //maxLengthAllowed .NET < 4.5 = 32765;
            //maxLengthAllowed .NET >= 4.5 = 65519;
            int maxLengthAllowed = 65519;
            StringBuilder sb = new StringBuilder();
            int loops = str.Length / maxLengthAllowed;

            for (int i = 0; i <= loops; i++)
            {
                sb.Append(Uri.EscapeDataString(i < loops
                    ? str.Substring(maxLengthAllowed * i, maxLengthAllowed)
                    : str.Substring(maxLengthAllowed * i)));
            }

            return sb.ToString();
        }

Response JSON
If possible include a response JSON to help identify the problem. You can capture it using Fiddler or Charles.

Hi. The exception you get doesn't add up because it's URI related and the file you attach will be a part of JSON content. I think the issue must be somewhere else in your code.
Could you post a full stacktrace? What target framework do you use the library with? With the info provided I can't see how it's library issue as the URI will always be https://api.baselinker.com/connector.php and it will never exceed max allowed length.

Please check published 0.3.4 version https://github.com/bugproof/BaseLinkerApi/commit/d4671b18557be3b85b795ec2c6d5a5af94779544. It should fix this issue but beware it might not be without it's own flaws. For better performance, stability and support you should consider upgrading your app to at least .NET 6. I think I might drop .NET Standard support altogether in future versions of this library and just keep .NET 6, 7 and 8.
You should probably avoid using WebRequest.Create in your code as well as it's obsolete, ancient stuff.


making setup script re-runnable

just a tiny bit of extra bash

I've changed the script to clean out the directories on failure and before exit. Haven't tested it so it's probably riddled with syntax errors

i think you will want to run cleanup at the beginning, regardless.
this will handle if the script fails or the user hits ctrl+c or a power outage etc.
there are still remaining files that could be a hazard, as-is.
even using a trap won't handle all scenarios. (trap may have been a bit more reliable or less tedious for you to setup than this way in some ways, but mostly equivalent with it's own issues)


Delete Glamorous classes

The classes inside AI-GT are extensions for the Glamorous inspector. Glamorous has been deleted from Pharo 9 >
As we will no longer mantain this project for Pharo 8 maybe we should delete that pacakge

Sure, feel free to delete them :)

Done :)


Capture union of a parameter type from function overloads

@cntkillme came up with a potentially interesting use case for typeof, which could be used to capture the type of the first parameter of a function. In particular, this is useful for a type like Instance.new
--!strict
local function captureParam<P1, A...>(f: (P1, ...any) -> A...): P1
	error("not callable")
end

type ClassName = typeof(captureParam(Instance.new))

local name: ClassName = "Frame"

This works correctly for something like the following:
type Example = (p: "A" | "B" | "C") -> ()
local x: Example
type Params = typeof(captureParam(x)) -- "A" | "B" | "C"

However, assuming Instance.new is typed something like (("Accoutrement") -> Accoutrement) & (("Frame") -> Frame) & ..., Luau only captures the parameter from the first overload, and ClassName is just the type "Accoutrement".
I'm interested to know if its possible to union the types of all the first parameters in the overload, and make that the type of P1, so that ClassName would be typed as "Accoutrement" | "Frame" | ...
This may potentially be a bit too complex for Luau to handle though, which is completely understandable

I'm going to put this and other issues suggesting features for type-level computation in the icebox for now - the use case makes sense, but we currently aren't ready to contemplate this class of features for the type checker, as we have a solid roadmap for type checker fundamentals and a large rework of much of type checking in progress. Once we're solid on that we will reevaluate issues from the icebox, but for now that makes tracking easier.


Stable dependency updates for Compose sample

Landing without review to build upon. Let me know if you want me to revert, or make further changes.


Help need conect arducam ov2640 with module Sd card

Hellow. my problem is. I have an arducam ov2640, i wanna save the photos in a module SD card but when i conect ov2640 and module sd in a arduino mega, arducam doesnt works but whe disconect module Sd arducam works. maybe 1 pin in the code affect the other pls help me thx

@AfroDjeff Please make sure if your sd card shield is well designed for SPI bus sharing, there are a lot of bad design for the sd card shield on the market.

I have the same but it'll be fixed when I use 2 power supply 4 two devices (one for SD and another for Arducam. GND have to connect together). but now comes the next problem. I load this program (ArduCAM_Mini_2MP_TimeElapse2SD) on my Arduino_ Uno. the program run well until this CODE comes (while (!myCAM.get_bit(ARDUCHIP_TRIG , CAP_DONE_MASK));) it stands still. what are ARDUCHIP_TRIG , CAP_DONE_MASK ?? can so help me thx

@Bulhcmvn
You have to use external power supply, because 4 camera consume a lot of power when running simultaneously.
CAP_DONE_MASK is not set because of the camera is not initialized properly. Please try the external power supply or make sure one camera works before adding more cameras.

at first. Thanks for your answer, but my problem still there.
when i connect Arducam like the first picture (with out cable except power supply) it works very well

but when i connect like the under picture (with external power supply 5V-1A)

then become

the message "Capture Done!    Open file failed" comes when i reset external power supply (OFF in 5s and ON again)
what is here problem ??? --> thank for your help

@Buihcmvn, from the error message it seems that you did the wrong connection.
Please check your wiring and make sure the wiring match with the pin  used in the example.


Broken Link Familiar

Also, the move is with the wrong text.

Also, there is no roll for intense


edit "Features" page

https://merginmaps.com/docs/howto/input_features/
Most of this page is covered by new tutorials (installation, registration, synchro). The rest could be a nice How to add/edit/delete features in Input after some minor changes.

we have also 2 "input" features pages, please sort out the second too


Display html response in iframe using separate ResposeType tab

Thanks @udivankin! There are a couple of things that should be fixed:
First, when you make a request to a page that has external resources like scripts, css or images, this line https://github.com/udivankin/restman/commit/ff58cd852577575e667da3ef082b06c7662f1106#diff-592106cea1bfc28f712ce4bd79b40495R80 will fail and it won't change the active button.
Second, perhaps it is a good idea to remove all scripts tags from the content of the returned html, I think it can be done with jquery with $(value).children('script').remove() and that result add it to the iframe.
One thing I don't like is the fact that when you load the iframe, you get lots of security warnings about an iframe trying to get resources from a different origin (images, scripts, css, etc), but as far as I know there is no way around that, so it's something we'll have to live with. What Postman for Chrome seems to be doing is to download the content to a temporary file and load that file using the src attribute of the iframe, these removes the security warning (because the src is local) but shows  a 404 error in the console for each resource.
If you want to fix the first two things, let me know, otherwise I can take them.

It's going to be included in v0.5 (to be released soon).


"quality theme": broken dependency

This issue concerns this file: https://github.com/Tropix126/BetterDiscordStuff/blob/master/bestthemeever/quality_theme.theme.css
The theme file tries to import "https://tropix126.github.io/BetterDiscordStuff/bestthemeever/index.css", which returns a 404 error.

Obviously the file was set up like this for auto updating from GitHub, but the original source was deleted. Therefore it is misleading to include this folder.

The theme is fixed, sorry about this issue.


C++ popup is formatted poorly in Sublime Text

Sublime Text shows a popup that is poorly formatted given the following C++ member variable declaration:
ns_event_t *m_eventMsg{nullptr};

The syntax above is C++11 and is (nearly) equivalent to m_eventMsg = nullptr;. Sublime shows that initialization but with extra line breaks… it thinks that it’s a block.

I think this data comes from clangd via LSP.

Yes, that data is coming from the server(clangd).
Feel free to open an issue at clangd,
I will choose this, because we cannot do anything regarding that on the client(LSP-clangd).


Make commands take a config pointer

This was found because the mode command stopped working. It broke because the config structure was passed in by copy instead of reference. I changed the API to be pointers and then tested it and it seems to work now.
I would normally just merge this small fix but because it's an API change would like review from @AbGuthrie

This should also fix any other bugs we may have with modifying config state. I don't know if there are any other ones currently but this will also let us write commands that enable or disable debug mode


TOMEE-2283 - Websocket tls basic auth example

This is still a work in progress.

Hi @rmannibucau ,
Thanks for your comments.
I've performed the following changes:

I've removed the NOTICE/LICENSE files.
Added missing headers to a few files.
I updated the package name... I had forgot to rename the package name... :)
removed value= when
correct, the deplyment name is not needed here. Removed.

In relation to the server.xml and the keystore files...
The setup is on purpose. My initial idea was to have an arquillian configuration that can be easily migrated to an actual server. This way, The user can get inspiration to update their configuration files.
The keystore can be generated, but I fill it will complicate the example's interpretation. I feel it's easier like this.


Updated validations.py python script.

Updated validations.py python script.
Fixed the behavior of the validate_user function in validations.py
Closes: #1
Fixes #<issue_number_goes_here>

It's a good idea to open an issue first for discussion.

[ ] Tests pass
[ ] Appropriate changes to README are included in PR

@googlebot I signed it!


Remove Square Brackets from filenames

With refracciones we recently had an issue of square brackets in a filename breaking the build process of the site.
While the error could be fixed in the build step, it still feels like a good idea to extend the list of special characters to be removed from a filename a bit more.
This PR

adds square and curly brackets to the list of characters to be removed
adds a simple test testing the expected behavior of filename creation

Be more strict with removing funky stuff from filenames.


sdist provider fallback not always working?

I found this requirements.txt snippet at $dayjob (where I'm trying to introduce Nix):
gql==0.2.0

and mach-nix fails to build it:
mach-nix on master [$?]
$ nix-build -A mach-nix && ./result/bin/mach-nix env -r <(echo gql==0.2.0) env && nix-shell env --run true
[...]
Processing ./gql-0.2.0-py3-none-any.whl
ERROR: Could not find a version that satisfies the requirement graphql-core<2,>=0.5.0 (from gql==0.2.0) (from versions: none)
ERROR: No matching distribution found for graphql-core<2,>=0.5.0 (from gql==0.2.0)
builder for '/nix/store/yhrmdp6sl4k3pdp4ygxbpkpdkwzq75d6-python3.7-gql-0.2.0.drv' failed with exit code 1

I dug in https://github.com/DavHau/pypi-deps-db/ and found the missing graphql-core package both as a wheel and as sdist.
But the wheel entries lack the required version(s), only sdist has it. (pip3 itself uses graphql-core 1.1, which exists only in the sdist db.) With that info I was able to build the python env by forcing sdist provider (yay!):
  customPython = mach-nix.mkPython {
    // ...
    providers = {
      graphql-core = "sdist";
    };
  };

Questions:

Why didn't mach-nix abort with a "Requirements conflict" error? (I've seen it doing that several times during my experiments, but in this scenario it allowed the build to start without any graphql-core package in the build env).
Why didn't it use the sdist provider as fallback? (Is it because it found some wheel versions, but none of them matched and gave up?)

Why didn't mach-nix abort with a "Requirements conflict" error? (I've seen it doing that several times during my experiments, but in this scenario it allowed the build to start without any graphql-core package in the build env).

Seems related to https://github.com/DavHau/mach-nix/issues/68.

Interesting, I just ran gql==0.2.0 through mach-nix and it produces the following tree:
gql - 0.2.0 - sdist
├── requests - 2.25.1 - wheel
│   ├── urllib3 - 1.26.4 - wheel
│   ├── idna - 2.10 - wheel
│   ├── chardet - 4.0.0 - wheel
│   └── certifi - 2020.12.5 - wheel
├── graphql-core - 2.0.dev20171009101843 - wheel
│   ├── six - 1.16.0 - wheel
│   └── promise - 2.3 - sdist
│       └── six - 1.16.0 - wheel
├── six - 1.16.0 - wheel
└── promise - 2.3 - sdist -> ...

The issue seems to be that mach-nix considers the version 2.0.dev20171009101843 to be lower than 2.0 and therefore accepts it. I'm wondering how pip solves that problem, because I'm quite sure that the underlying libraries return the same result when comparing versions.
Need to dig deeper into this.

I think I found it:
https://pip.pypa.io/en/stable/cli/pip_install/:
"Starting with v1.4, pip will only install stable versions as specified by pre-releases by default. If a version cannot be parsed as a compliant PEP 440 version then it is assumed to be a pre-release.
If a Requirement specifier includes a pre-release or development version (e.g. >=0.0.dev0) then pip will allow pre-release and development versions for that requirement. This does not include the != flag.
The pip install command also supports a --pre flag that enables installation of pre-releases and development releases."
With this information it shouldn't be hard to implement.


feat(LinearAlgebra/Multilinear/Basic): derivative of a multilinear map

Define the derivative of a multilinear map f at x, i.e. the first-order term of f(x+h)-f(x+h'). It also gives a lemma showing that f(x+h)-f(x+h') is indeed the sum of the derivative at x evaluated on h-h' and of higher-order terms, as well as a lemma calculating the difference of the values of f at two points that will be useful to bound the higher-order terms.

[ ] depends on: #9137

I took the liberty to label this PR by topic area, and tweak title and description to better match the standard format.
(Github's defaults are not always ideal here.)

Thanks !

I took the liberty to [...] tweak the title [...] to better match the standard format

A very minor point: we don't put a space before the first (.

I pushed another file to my branch and it got added to the PR, even though that was not my intention (this other file needs some work before it can become a PR). Does anybody know how to reverse this ?

I've discarded that file. Regarding what you attempt to do in the file: we have AnalyticAt.contDiffAt and AnalyticAt is defined in terms of ContinuousMultilinearMap. So I guess it shouldn't be hard to get what you want through that route?

I've discarded that file. Regarding what you attempt to do in the file: we have AnalyticAt.contDiffAt and AnalyticAt is defined in terms of ContinuousMultilinearMap. So I guess it shouldn't be hard to get what you want through that route?

It depends what you mean by "not hard"; mathematically it's very easy, but in Lean it might be a pain. Certainly if I had known that analytic functions were in mathlib, I would have tried to go that route, because this is the best way.

I've discarded that file. Regarding what you attempt to do in the file: we have AnalyticAt.contDiffAt and AnalyticAt is defined in terms of ContinuousMultilinearMap. So I guess it shouldn't be hard to get what you want through that route?

It depends what you mean by "not hard"; mathematically it's very easy, but in Lean it might be a pain. Certainly if I had known that analytic functions were in mathlib, I would have tried to go that route, because this is the best way. (And yes, I did ask about what I wanted to do on Zulip before starting.)

Okay, I know what to do, I hope it won't be too painful.

I found your Zulip thread (cc @ocfnash) and I do recall seeing it before. However, I had no idea Analytic is defined in terms of multilinear maps, even though I did know it's in mathlib. I only became aware 2 weeks ago due to my involvement in the Hairer challenge, when I looked up the definition of Analytic when proving evaluations of multivariate polynomials are analytic. Sorry for being unable to help earlier and for the extra work!

I found your Zulip thread (cc @ocfnash) and I do recall seeing it before. However, I had no idea Analytic is defined in terms of multilinear maps, even though I did know it's in mathlib. I only became aware 2 weeks ago due to my involvement in the Hairer challenge, when I looked up the definition of Analytic when proving evaluations of multivariate polynomials are analytic. Sorry for being unable to connect the dots earlier and for the extra work!

Eh, it's not your fault. So anyway, I proved that continuous multilinear maps are analytic, however, to deduce that they are differentiable, I need the target F to be complete ! At least that's a hypothesis in AnalyticAt.contDiffAt, I haven't checked whether it's possible to remove.
But my old proof works without that extra hypothesis on F ! So I guess I should still PR it, and also PR the proof that continuous multilinear maps are analytic, now that it's written ?

That's nice! I only just worked out the informal argument why a multilinear form f (of finitely many factors) is analytic; you need to compose it with the (linear) projections to obtain f(p1, ..., pn) and put that as the nth term in the power series. As for the completeness assumption, I think it's removable and FormalMultilinearSeries.changeOrigin_eval seems to be the key lemma to generalize. I'll try later if you don't beat me to it!

I now realized that the derivative of an analytic function may not even exist if the space isn't complete, let alone continuously differentiable; the original series converging can't guarantee the derivative doesn't fall into a hole. So I think your result for continuous linear maps will be a valuable addition to mathlib! Maybe we could generalize it to analytic functions with finitely many nonzero terms ("polynomial functions"?). As a different project, the derivative's power series is in mathlib as FormalMultilinearSeries.shift, but apparently no one has shown it evaluates to the derivative yet ...

Maybe we can replace the CompleteSpace conditions by the necessary Summable conditions. Then they'll apply to both CompleteSpaces (where we can use Summable.of_norm) and our situation ("polynomial functions"), where we have summability by virtue of being finite sums.

Yes, the completeness is only really used to prove the convergence of infinite power series. So maybe the simplest would be to have some HasFiniteFPowerSeriesOnBall structure which would cover both polynomial and multilinear functions. This should be stable by taking derivatives, and so we would get both analyticity and differentiability results with any completeness condition. I won't get to it today as I have a pretty full schedule.
By the way, the "continuous multilinear functions are analytic" code is there:

Ok, I added some more stuff to prepare for the proof that continuous multilinear maps are analytic: the definition of what will be the formal multilinear series at x and a proof that it does calculate f (x + y) (with a finite sum). Also, the comparison of the degree 1 term with the previously defined derivative. I kept the previous derivative, because it definition doesn't use a linear order on ι, while the definition of the formal multilinear series does.
Also, I kept running into merge problems that for some reason duplicated the new code, so I hope that those are resolved now. It seems to be okay. And I broke mathlib in my branch when trying to update it (again :-(((( ), so nothing compiles locally for me, but I did check the new code in another lean project, and again hopefully it's okay. Now I'll try to fix my local mathlib.

Yeah, it's not okay, the thing didn't build. Why do I break stuff every time I update mathlib ? I only wanted the new lemma from PR #8867 9137. :-(

(the base branches above are to work around a github quirk caused by a weird (but ultimately harmless) merge that you did)

Okay, after last night's git drama, and after a lot of help from Eric, and after I accidentally pushed the whole of mathlib4 to a mathlib3 branch, it seems that I have finally been able to successfully update this. So I added some code to prepare for the proof that continuous multilinear maps are analytic: the definition of what will be the formal multilinear series at x, i.e. the higher derivatives, and a proof that it does calculate f (x + y) (with a finite sum). Also, the comparison of the degree 1 term with the previously defined derivative. I kept the previous derivative, because it definition doesn't use a linear order on ι, while the definition of the formal multilinear series does.
As the new code uses things like MultilinearMap.domDomCongr, I moved it down in the file so it would come after that, and put it in its own section called 'Derivative`.

bors r+

bors r+

This branch is what I consider some unfinished work in this PR. I'm done for today but feel free to fill the sorries and submit another PR.

This branch is what I consider some unfinished work from this PR. I'm done for today but feel free to fill the sorries and submit another PR.

This has been PR'd as #9846. Maybe reviewers here could take a look at this follow-up PR?


hugging face demo error

when I used this demo show error

Hello! I've only had this issue if I didn't have both pictures uploaded, otherwise it seems to work for me.


dirfs.OpenFile can return ENAMETOOLONG even if that is not defined FS.OpenFile in the interface documentation

This is similar to  #2101 but it's more general and it basically contains all other errnos.
Basically, dirFS can return a lot more other errnos - not just the few defined here
https://github.com/tetratelabs/wazero/blob/ba569623fcb7b32bb8fc64dadc842672befd6819/imports/wasi_snapshot_preview1/fs.go#L1534...L1540
when the os returns them as a syscall error, and it's in the list here
https://github.com/tetratelabs/wazero/blob/cb3dc29c08f8f27f668daff1ce3fb852156b07a5/experimental/sys/syscall_errno.go#L7
A trivial repro on macOS (need to run from somewhere where you have access to internal/sysfs/dirfs)
rootFS := dirfs.DirFS("/")
res, err := rootFS.OpenFile("tmp/"+strings.Repeat("a", 400), sys.O_CREAT, 0)
fmt.Println(res, err)

This should not be possible according to FS.OpenFile documentation:
https://github.com/tetratelabs/wazero/blob/ba569623fcb7b32bb8fc64dadc842672befd6819/experimental/sys/fs.go#L34...L39
Again I hit this when I implement my own FS implementation and I am not sure if I should adhere to the doc with the errnos, or just return whatever.
(Sorry if this is duplicate of #2101 and should not be a different issue)

The same repro should work on Linux, max filename length is 255.

I'd say it is not necessary to open an issue for each new error that is not documented :D let's keep it tidy, as I think I have mentioned in https://github.com/tetratelabs/wazero/issues/2076 this feature is still experimental so you might find more of these idiosyncrasies. We can continue on this or https://github.com/tetratelabs/wazero/issues/2101, as essentially the issue boils down to limited documentation and/or unspecified behavior.
Do follow up with your questions and feedback, all it's welcome.


Mempool bug:

This bug found a few times (after running for about 24 hours or if mempool is really full)
fail: Mempool[0]
      System.ArgumentException: An item with the same key has already been added. Key: e492cffb55da94e2e4f5ecb29cb47a47365a1572867f8db523e7e323c315d52f
         at System.ThrowHelper.ThrowAddingDuplicateWithKeyArgumentException(Object key)
         at System.Collections.Generic.Dictionary`2.Insert(TKey key, TValue value, Boolean add)
         at Stratis.Bitcoin.Consensus.UnspentOutputSet.SetCoins(FetchCoinsResponse coins) in C:\Users\Dan\Documents\GitHub\StratisBitcoinFullNode\Stratis.Bitcoin\Consensus\UnspentOutputSet.cs:line 52
         at Stratis.Bitcoin.MemoryPool.MempoolCoinView.<LoadView>d__10.MoveNext() in C:\Users\Dan\Documents\GitHub\StratisBitcoinFullNode\Stratis.Bitcoin\MemoryPool\MemPoolCoinView.cs:line 41
      --- End of stack trace from previous location where exception was thrown ---
         at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
         at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
         at Stratis.Bitcoin.MemoryPool.MempoolValidator.<AcceptToMemoryPoolWorker>d__27.MoveNext() in C:\Users\Dan\Documents\GitHub\StratisBitcoinFullNode\Stratis.Bitcoin\MemoryPool\MempoolValidator.cs:line 107
      --- End of stack trace from previous location where exception was thrown ---
         at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
         at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
         at Stratis.Bitcoin.MemoryPool.MempoolValidator.<AcceptToMemoryPoolWithTime>d__25.MoveNext() in C:\Users\Dan\Documents\GitHub\StratisBitcoinFullNode\Stratis.Bitcoin\MemoryPool\MempoolValidator.cs:line 71
      --- End of stack trace from previous location where exception was thrown ---
         at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
         at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
         at Stratis.Bitcoin.MemoryPool.MempoolBehavior.<ProcessTxPayloadAsync>d__24.MoveNext() in C:\Users\Dan\Documents\GitHub\StratisBitcoinFullNode\Stratis.Bitcoin\MemoryPool\MempoolBehavior.cs:line 184
      --- End of stack trace from previous location where exception was thrown ---
         at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
         at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
         at Stratis.Bitcoin.MemoryPool.MempoolBehavior.<AttachedNode_MessageReceived>d__20.MoveNext() in C:\Users\Dan\Documents\GitHub\StratisBitcoinFullNode\Stratis.Bitcoin\MemoryPool\MempoolBehavior.cs:line 70

Unhandled Exception: System.ArgumentException: An item with the same key has already been added. Key: e492cffb55da94e2e4f5ecb29cb47a47365a1572867f8db523e7e323c315d52f
   at System.ThrowHelper.ThrowAddingDuplicateWithKeyArgumentException(Object key)
   at System.Collections.Generic.Dictionary`2.Insert(TKey key, TValue value, Boolean add)
   at Stratis.Bitcoin.Consensus.UnspentOutputSet.SetCoins(FetchCoinsResponse coins) in C:\Users\Dan\Documents\GitHub\StratisBitcoinFullNode\Stratis.Bitcoin\Consensus\UnspentOutputSet.cs:line 52
   at Stratis.Bitcoin.MemoryPool.MempoolCoinView.<LoadView>d__10.MoveNext() in C:\Users\Dan\Documents\GitHub\StratisBitcoinFullNode\Stratis.Bitcoin\MemoryPool\MemPoolCoinView.cs:line 41
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at Stratis.Bitcoin.MemoryPool.MempoolValidator.<AcceptToMemoryPoolWorker>d__27.MoveNext() in C:\Users\Dan\Documents\GitHub\StratisBitcoinFullNode\Stratis.Bitcoin\MemoryPool\MempoolValidator.cs:line 107
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at Stratis.Bitcoin.MemoryPool.MempoolValidator.<AcceptToMemoryPoolWithTime>d__25.MoveNext() in C:\Users\Dan\Documents\GitHub\StratisBitcoinFullNode\Stratis.Bitcoin\MemoryPool\MempoolValidator.cs:line 71
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at Stratis.Bitcoin.MemoryPool.MempoolBehavior.<ProcessTxPayloadAsync>d__24.MoveNext() in C:\Users\Dan\Documents\GitHub\StratisBitcoinFullNode\Stratis.Bitcoin\MemoryPool\MempoolBehavior.cs:line 184
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at Stratis.Bitcoin.MemoryPool.MempoolBehavior.<AttachedNode_MessageReceived>d__20.MoveNext() in C:\Users\Dan\Documents\GitHub\StratisBitcoinFullNode\Stratis.Bitcoin\MemoryPool\MempoolBehavior.cs:line 86
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
   at System.Threading.ExecutionContext.Run(ExecutionContext executionContext, ContextCallback callback, Object state)
   at System.Threading.QueueUserWorkItemCallbackDefaultContext.System.Threading.IThreadPoolWorkItem.ExecuteWorkItem()
   at System.Threading.ThreadPoolWorkQueue.Dispatch()

I bet this is similar to https://github.com/stratisproject/StratisBitcoinFullNode/issues/102
Also rooted in the MessageReceived event.
The NodeConnections are running on other threads, and occasionally there's going to be the same message coming in at the same time. I'm looking at adding semaphores around the collections, but in case of TxMempool, the inner dictionary isn't concurrent as it is with the BlockPuller problem.

yeha with the mempool I took a different approach of a lock free implementation, all the thread synchronization happens at the scheduler pair level (i.e asyncLock.Read and asynclock.Write)

Happend to me. this probably doesn't add any additional useful info, but here it is:
info: Stratis.Bitcoin.MemoryPool[0]
      accepted orphan tx b2bd50f96457c25e62a8f13e66bc1122d384ff9430c0f86c8c42960bcda14224
info: Stratis.Bitcoin.MemoryPool[0]
      accepted orphan tx 8dcdbb062488a8e3d0bc775dd8b6b947205cee14b57a07c26509cc27c375b7ba
fail: Stratis.Bitcoin.MemoryPool[0]
      System.ArgumentException: An item with the same key has already been added. Key: 412f49eb4f70d08af9adb1cacf2454aec7499b83eb9740d71f5f16dc64476dfc
         at System.ThrowHelper.ThrowAddingDuplicateWithKeyArgumentException(Object key)
         at System.Collections.Generic.Dictionary`2.Insert(TKey key, TValue value, Boolean add)
         at Stratis.Bitcoin.Consensus.UnspentOutputSet.SetCoins(FetchCoinsResponse coins) in D:\Dev\StratisBitcoinFullNode\Stratis.Bitcoin\Consensus\UnspentOutputSet.cs:line 52
         at Stratis.Bitcoin.MemoryPool.MempoolCoinView.<LoadView>d__10.MoveNext() in D:\Dev\StratisBitcoinFullNode\Stratis.Bitcoin\MemoryPool\MemPoolCoinView.cs:line 41
      --- End of stack trace from previous location where exception was thrown ---
         at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
         at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
         at System.Runtime.CompilerServices.ConfiguredTaskAwaitable.ConfiguredTaskAwaiter.GetResult()
         at Stratis.Bitcoin.MemoryPool.MempoolValidator.<AcceptToMemoryPoolWorker>d__27.MoveNext() in D:\Dev\StratisBitcoinFullNode\Stratis.Bitcoin\MemoryPool\MempoolValidator.cs:line 107
      --- End of stack trace from previous location where exception was thrown ---
         at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
         at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
         at System.Runtime.CompilerServices.TaskAwaiter.GetResult()
         at Stratis.Bitcoin.MemoryPool.MempoolValidator.<AcceptToMemoryPoolWithTime>d__25.MoveNext() in D:\Dev\StratisBitcoinFullNode\Stratis.Bitcoin\MemoryPool\MempoolValidator.cs:line 71
      --- End of stack trace from previous location where exception was thrown ---
         at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
         at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
         at System.Runtime.CompilerServices.TaskAwaiter`1.GetResult()
         at Stratis.Bitcoin.MemoryPool.MempoolBehavior.<ProcessTxPayloadAsync>d__24.MoveNext() in D:\Dev\StratisBitcoinFullNode\Stratis.Bitcoin\MemoryPool\MempoolBehavior.cs:line 184
      --- End of stack trace from previous location where exception was thrown ---
         at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
         at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
         at System.Runtime.CompilerServices.ConfiguredTaskAwaitable.ConfiguredTaskAwaiter.GetResult()
         at Stratis.Bitcoin.MemoryPool.MempoolBehavior.<AttachedNode_MessageReceived>d__20.MoveNext() in D:\Dev\StratisBitcoinFullNode\Stratis.Bitcoin\MemoryPool\MempoolBehavior.cs:line 70

Unhandled Exception: info: Stratis.Bitcoin.MemoryPool[0]
      accepted orphan tx fb1644ca0174367fb58707af1856bfc4671b485b351fcc8498740350724604e4
System.ArgumentException: An item with the same key has already been added. Key: 412f49eb4f70d08af9adb1cacf2454aec7499b83eb9740d71f5f16dc64476dfc
   at System.ThrowHelper.ThrowAddingDuplicateWithKeyArgumentException(Object key)
   at System.Collections.Generic.Dictionary`2.Insert(TKey key, TValue value, Boolean add)
   at Stratis.Bitcoin.Consensus.UnspentOutputSet.SetCoins(FetchCoinsResponse coins) in D:\Dev\StratisBitcoinFullNode\Stratis.Bitcoin\Consensus\UnspentOutputSet.cs:line 52
   at Stratis.Bitcoin.MemoryPool.MempoolCoinView.<LoadView>d__10.MoveNext() in D:\Dev\StratisBitcoinFullNode\Stratis.Bitcoin\MemoryPool\MemPoolCoinView.cs:line 41
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at System.Runtime.CompilerServices.ConfiguredTaskAwaitable.ConfiguredTaskAwaiter.GetResult()
   at Stratis.Bitcoin.MemoryPool.MempoolValidator.<AcceptToMemoryPoolWorker>d__27.MoveNext() in D:\Dev\StratisBitcoinFullNode\Stratis.Bitcoin\MemoryPool\MempoolValidator.cs:line 107
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at System.Runtime.CompilerServices.TaskAwaiter.GetResult()
   at Stratis.Bitcoin.MemoryPool.MempoolValidator.<AcceptToMemoryPoolWithTime>d__25.MoveNext() in D:\Dev\StratisBitcoinFullNode\Stratis.Bitcoin\MemoryPool\MempoolValidator.cs:line 71
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at System.Runtime.CompilerServices.TaskAwaiter`1.GetResult()
   at Stratis.Bitcoin.MemoryPool.MempoolBehavior.<ProcessTxPayloadAsync>d__24.MoveNext() in D:\Dev\StratisBitcoinFullNode\Stratis.Bitcoin\MemoryPool\MempoolBehavior.cs:line 184
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at System.Runtime.CompilerServices.ConfiguredTaskAwaitable.ConfiguredTaskAwaiter.GetResult()
   at Stratis.Bitcoin.MemoryPool.MempoolBehavior.<AttachedNode_MessageReceived>d__20.MoveNext() in D:\Dev\StratisBitcoinFullNode\Stratis.Bitcoin\MemoryPool\MempoolBehavior.cs:line 86
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
   at System.Threading.ExecutionContext.Run(ExecutionContext executionContext, ContextCallback callback, Object state)
   at System.Threading.QueueUserWorkItemCallbackDefaultContext.System.Threading.IThreadPoolWorkItem.ExecuteWorkItem()
   at System.Threading.ThreadPoolWorkQueue.Dispatch()

The fix is probably easy, when we come from the mempool we can check if a trx is already in the utxo set.
But I want to be sure this is the case.


Any specific guidelines for IME icons (i.e. icons on the language bar)

There was a Windows 8 guideline for IME icons (including branding and mode icons)  https://docs.microsoft.com/en-us/previous-versions/windows/apps/hh967425(v=win.10)#ime-icons  But that is outdated. Are there any new guidelines specifically for IME icons? Or can I design IME icons in whatever way I like?

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

ID: ac89746a-7629-6a95-6c5c-9498a9d7989f
Version Independent ID: 044558cb-9f69-42df-362c-6c35317c3411
Content: App icons and logos - UWP app developer
Content Source: windows-apps-src/design/style/app-icons-and-logos.md
Product: uwp
GitHub Login: @mijacobs
Microsoft Alias: mijacobs

Hi @ztl8702 - Apologies for the numerous delays in addressing this issue. I confirmed with various IME-related teams that this guidance is not out of date, even though the topics have been archived.
With that information, I pulled the topics out of the archive, rewrote to some degree, and incorporated them into the current Windows guidance.
https://docs.microsoft.com/en-us/previous-versions/windows/apps/hh967426(v=win.10) ==> https://docs.microsoft.com/en-us/windows/uwp/design/input/input-method-editors
and
https://docs.microsoft.com/en-us/previous-versions/windows/apps/hh967425(v=win.10) ==> https://docs.microsoft.com/en-us/windows/uwp/design/input/input-method-editor-requirements
If still interested, please review and file a new Issue for any feedback/questions you might have. Reactivating this issue might not move it to the top of the more current issues we're trying to address.
Thanks.


Need ASC REST API and SDK for .Net/Python/NodeJS for ASC Secure Score and Recommendations

Will ASC REST API and SDK for .Net/Python/NodeJS be available for developers to query ASC REST API or use Azure SDK for .Net/Python/NodeJS to programmatically query the Azure Security Center (ASC) Secure Score and list the ASC Recommendations?
https://feedback.azure.com/forums/347535-azure-security-center/suggestions/39133543-asc-securescore-available-through-graph-api-or-asc

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

ID: b9d7741b-cd5a-29b0-c360-e054f7686c87
Version Independent ID: f03434f5-504a-d87f-b9ce-3ed695db9227
Content: Enhanced Secure Score (preview) in Azure Security Center
Content Source: articles/security-center/secure-score-security-controls.md
Service: security-center
GitHub Login: @memildin
Microsoft Alias: memildin

@kiazhi
Thanks for your feedback! We will investigate and update as appropriate.

@kiazhi Exposing Secure Score through REST APIs is something the team is actively working.  Refer to the UserVoice on latest update on the same.

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


[GOBBLIN-468] Fix bug with enum for json to avro converter

Dear Gobblin maintainers,
Please accept this PR. I understand that it will not be reviewed until I have checked off all the steps below!
JIRA

[x] My PR addresses the following Gobblin JIRA issues and references them in the PR title. For example, "[GOBBLIN-XXX] My Gobblin PR"

https://issues.apache.org/jira/browse/GOBBLIN-468

Description

[x] Here are some details about my PR, including screenshots (if applicable):

Schema was incorrectly null for enums in json to avro conversion
Tests

[x] My PR adds the following unit tests OR does not need testing for this extremely good reason:

Updated unit test
Commits

[x] My commits all reference JIRA issues in their subject lines, and I have squashed multiple commits if they address the same issue. In addition, my commits follow the guidelines from "How to write a good git commit message":

Subject is separated from body by a blank line
Subject is limited to 50 characters
Subject does not end with a period
Subject uses the imperative mood ("add", not "adding")
Body wraps at 72 characters
Body explains "what" and "why", not "how"

@eogren please review

+1 - thanks!


get.sh fails with unknown flag: --version error

What happened:
+ curl -sL https://raw.githubusercontent.com/ppc64le-cloud/pvsadm/master/get.sh
+ VERSION=v0.1.8 FORCE=1 bash
#=#=#
##O#-#

                                                                           0.1%
##############################################                            64.1%
######################################################################## 100.0%
Error: unknown flag: --version
Usage:
  pvsadm [command]

Available Commands:
  completion  Generate the autocompletion script for the specified shell
  create      Create the resources
  delete      Delete the resources
  dhcp-sync   dhcp-sync command
  dhcpserver  dhcpserver command
  get         Get the resources
  help        Help about any command
  image       PowerVS Image management
  purge       Purge the powervs resources
  version     Print the version number

Flags:
  -k, --api-key string                   IBMCLOUD API Key(env name: IBMCLOUD_API_KEY)
      --env string                       IBM Cloud Environments, supported are: [prod, test] (default "prod")
      --debug                            Enable PowerVS debug option(ATTENTION: dev only option, may print sensitive data from APIs)
      --audit-file string                Audit logs for the tool (default "pvsadm.log")
      --add_dir_header                   If true, adds the file directory to the header of the log messages
      --alsologtostderr                  log to standard error as well as files (no effect when -logtostderr=true)
      --log_backtrace_at traceLocation   when logging hits line file:N, emit a stack trace (default :0)
      --log_dir string                   If non-empty, write log files in this directory (no effect when -logtostderr=true)
      --log_file string                  If non-empty, use this log file (no effect when -logtostderr=true)
      --log_file_max_size uint           Defines the maximum size a log file can grow to (no effect when -logtostderr=true). Unit is megabytes. If the value is 0, the maximum file size is unlimited. (default 1800)
      --logtostderr                      log to standard error instead of files (default true)
      --one_output                       If true, only write logs to their native severity level (vs also writing to each lower severity level; no effect when -logtostderr=true)
      --skip_headers                     If true, avoid header prefixes in the log messages
      --skip_log_headers                 If true, avoid headers when opening log files (no effect when -logtostderr=true)
      --stderrthreshold severity         logs at or above this threshold go to stderr when writing to files and stderr (no effect when -logtostderr=true or -alsologtostderr=false) (default 2)
  -v, --v Level                          number for the log level verbosity
      --vmodule moduleSpec               comma-separated list of pattern=N settings for file-filtered logging
  -h, --help                             help for pvsadm

Use "pvsadm [command] --help" for more information about a command.

job ref: https://jenkins.ppc64le-cloud.cis.ibm.net/job/powervs/job/ipi/job/4.16/job/daily-ipi4.16-powervs-frankfurt1/116/execution/node/36/log/
What you expected to happen:
How to reproduce it (as minimally and precisely as possible):
Anything else we need to know?:
Environment:

pvsadm version (use pvsadm --version):
Environment - On-Prem or IBM PowerVS:
OS (e.g: cat /etc/os-release):
Kernel (e.g. uname -a):
Others:

The changes are made to use the --version flag for >=0.1.18, and version subcommand for the versions priors to the release. This ensures that the appropriate command is executed based on the version.
However, with 0.17.0, as the version command is broken due to #664. An option to delete and re-release the changes can help to fix the breaking change.
With future versions, if we try to run get.sh, while a version > 0.1.18 if it is already installed on the host, the subcommand option will tend to execute, which is bound for deprecation as the locals vars would default to 0(s).
Ps: As discussed, we can plan to deprecate the version sub-command in a major release.
Thanks!

Fixed through #674
/close


Readme updates

A PR that adds the contents of the readme updated in the meantime on the main branch to the develop branch.

Since this is already in the main branch, we'll include it in develop as well.


Anerdw: Blacklist (+52) 55-8526-4013

Anerdw requests the blacklist of the number (+52) 55-8526-4013. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

Approved by Fastnlight in Charcoal HQ


Upgrade to Karma v6.3.20

This PR upgrades our Karma dependency to version 6.3.20. The key breaking change was how the server was initialized with the configuration. Fixes https://github.com/CesiumGS/cesium/issues/9635

Thanks for the pull request @sanjeetsuhag!

:heavy_check_mark: Signed CLA found.
:grey_question: CHANGES.md was not updated.

If this change updates the public API in any way, please add a bullet point to CHANGES.md.

:grey_question: Unit tests were not updated.

Make sure you've updated tests to reflect your changes, added tests for any new code, and ran the code coverage tool.

Reviewers, don't forget to make sure that:

[ ] Cesium Viewer works.
[ ] Works in 2D/CV.


Why GPL3 ? (Edit: Switch to Apache 2.0)

It is common conception that gpl3 is pretty restrictive when it comes to linking the gpled libraries with other software, so for example if one would link this repo to their game it could mean that this game will be gpled too. Not everybody wants to give out all their work considering the virality of said gpl license.
Wouldnt it be wiser to use LGPL3 that allows using libraries linked to the closed source without infecting it with license?

hi hi, so sorry I haven't have time to change this repo's license! It is supposed to be Apache 2.0 XD, please feel free to use it on any projects!

Dont need to be sorry, just was curious if its deliberate or not.
Apache 2 is the warm fuzzy license everybody loves .

hi, I have switched this repo to Apache 2.0 alongside with a alpha release of v0.0.1 :)

Awesome thank You.


Integrate 2x speed viewing to pump retention times with half the effort (threads) ;)

If you make the threads view the video at 2x, you can speed things up quite a bit.. I highly recommend finding the switch that turns 'fast viewing' on and setting it for all worker threads.

@Airkek is this Next Release including the lower speed ?

@Airkek is this Next Release including the lower speed ?

yes

@Airkek is this Next Release including the lower speed ?

yes

thanks!!!!! 💯 %


As a user I want to switch to the prev/next result and bookmark it from the Quickview modal

@jwassenaarbg commented on Wed Jan 09 2019
Acceptance criteria:

[x] The user sees buttons to go to the next or previous searchresult when previewing it in Quickview
[x] The user is able to select the current result with a visible checkbox in Quickview
[x] The user is able to use the arrow keys and S (for selection) to do the above actions with his keyboard

I guess I put the buttons in the modal etc and that Eduardo possibly need to move them around with css or something if the location is not right?

@jwassenaarbg I see that this is functionality works also while testing branch iss_250_quickviewhighlights. I tested in that branch. All OK, nice idea/feature!
If you still need to send a pull request, please do it ;)

@jwassenaarbg @avlisodraude I've finally merged this code and made major changes to the single search recipe code (and it's related components). The changes were really needed to make the code more readable & cleaner again. Also this way Frontwise will have an easier time doing the restyling.
@lilimelgar could you test this issue and the other one (#170 ) in the master? When there are bugs, simply report them in new issues. I never want to see these two issues anymore ;-)


Workload description

Added description field in workloads

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


easyocr is useless in demo, why use it?

Do you have an alternative recommendation?

I agree that it is not the best OCR solution, and also often doesn't get the text right for me. I recommend PaddleOCR https://github.com/PaddlePaddle/PaddleOCR/blob/main/README_en.md

Thanks @aliencaocao . Looks like it only needs modest changes to integrate a different OCR into utils.py, eg here:
https://github.com/microsoft/OmniParser/blob/d7708e830a2f16a40a8970211a9911fa776d4aa2/utils.py#L375
Is that correct and was it straightforward when you switched out EasyOCR for the demo above?

Yes, I can make a PR if you wish, I already have it impl in my fork: https://github.com/aliencaocao/OmniParser/commit/a3fe0e11c5b67a6ed8de916cdd77ddffba4d135f

Thanks for sharing that!
I'll try it out.
Incidentally I agree switching to a better OCR seems best but for anyone after a quick fix in the Gradio demo, I found that bumping the value for text_threshold in easyocr_args down from 0.9 to 0.7 or so helped reduce a fair amount of missing or prematurely split text.
https://github.com/microsoft/OmniParser/blob/d7708e830a2f16a40a8970211a9911fa776d4aa2/gradio_demo.py#L68

Setting to 0.7 or 0.5 both did not make things better for this particular example:

I still think paddleocr is a clear better alternative.

Yes, I was agreeing with you 🙂
The quick fix isn't going to work for everything, but it did slightly improve several of my images where the full width of a text span wasn't being captured.
In your image above this happens with the timestamps: in your earlier version item 31 is missing the 2022 plus five entries under the Solved and Points columns were missed and above only one is missed. However it's still not handling the entries in the Rank column well enough.

Yes, I can make a PR if you wish, I already have it impl in my fork: aliencaocao@a3fe0e1

Feel free to make a PR

Yes, I can make a PR if you wish, I already have it impl in my fork: aliencaocao@a3fe0e1

Feel free to make a PR

PR made


No example for copyOut

I have tried:
uploadfs.copyOut(ufsUrl, tmpFilename, function(e, info) { ...my handler... });

where ufsUrl is a self.apos.attachments.url(attachment) from Apostrophe related code.
But I have no output at expected tmpFilename path.
What am I doing wrong? I see lack of documentation about copyOut, but I need this method for post-processing of my images.

One suggestion would be to take a look at the test/ folder which has working copyOut examples for the various storage backends.

True! Wouldn't hurt us to include a sample as such, though.
On Fri, Jun 1, 2018 at 8:54 AM, Paul Walker<EMAIL_ADDRESS>wrote:

One suggestion would be to take a look at the test/ folder which has
working copyOut examples for the various storage backends.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/punkave/uploadfs/issues/41#issuecomment-393871650,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAB9fUGcLUbwUKUoSFyN-X7mAAZHLwQzks5t4TmWgaJpZM4UV2x7
.

--
THOMAS BOUTELL, CHIEF SOFTWARE ARCHITECT
P'UNK AVENUE |<PHONE_NUMBER>  |  punkave.com

I'm trying self.apos.attachments.url(attachment, {"uploadfsPath": true}); and only difference in behaviour I get is that the returned url contains prefix of upload path (because I use Apostrophe with local backend for storing uploaded files). Anyway, copyOut doesn't run my callback handler at all :(

This difference in behaviour is from local.js, luckily it works as expected, but somehow copyFile isn't invoked or exceptions handled and omitted somewhere. Investigating...

Finally got it. I was invoking copyOut within queue (better-queue) task. What I get now is that I don't know how to get uploadfs instance from Apostrophe. Any help? Please!..

@boutell Thanks for pointing it! Love you, bro, you saved my life and pocket! No need coffee today 💃
I've tried to get uploadfs settings from apo's attachments module, but newly created instance behave stranger.


Implement StaticType and GValue traits for boxed/shared types too

As long as a get_type() function is provided to the macro.

This should be ready too now, and is needed for gtk-rs/gir#410
CI is unhappy though, which looks like out of memory or timeouts... before this PR already.

Restarted CI.

Value impls for enums and flags don't have #cfg (see GdkGLError  and GdkAxisFlags)

@GuillaumeGomez 👍 for merge this PR despite CI internal problem
@sdroege, thanks.

Thanks!


Sollet is not raising disconnect event on closing Sollet.io window

Reproduce:

connect using Sollet adapter
close the Sollet child window

Expected:

disconnect callback is invoked

Actual:

disconnect callback is not invoked

Serum Sollet adapter is handling this using window beforeunload event

Thanks for the report! I'll check this out today.

This is handled now by https://github.com/solana-labs/wallet-adapter/blob/45013f04ea3be6a877e899f16c33d23884d147ec/packages/sollet/src/adapter.ts#L94 https://github.com/solana-labs/wallet-adapter/blob/45013f04ea3be6a877e899f16c33d23884d147ec/packages/sollet/src/adapter.ts#L156-L166
The same fix was made to the Ledger adapter. Thanks again!

I met two issues:

When I select Sollet wallet, it will open a new window instead of a new tab. Expect: open new tab
When I close the Sollet window, the connection status will be "disconnect". Then, I click connect button, It will open a new tab to sollet.io, I connect to my wallet but the connection status still is "disconnect"

@Sotatek-CongNguyen I'm just seeing this. In the future, if you create a new issue rather than comment on a closed issue, it will have better visibility.

1. When I select Sollet wallet, it will open a new window instead of a new tab. Expect: open new tab

To my knowledge, JavaScript cannot control or detect whether a popup opens in a new window or tab. That's a browser setting.

2. When I close the Sollet window, the connection status will be "disconnect". Then, I click connect button, It will open a new tab to sollet.io, I connect to my wallet but the connection status still is "disconnect"

Can you provide more context about the browser and device you're using that experiences this behavior?


Publish CF CLI to WinGet Repository

Please provide details on the following items. Failure to do so may result in deletion of your feature request.
What's the user value of this feature request?
Easy, command line installation and updating for CF CLI
Who is the functionality for?
All Windows users
How often will this functionality be used by the user?
Every installation and update, hopefully
Who else is affected by the change?
No
Is your feature request related to a problem? Please describe.
No
Describe the solution you'd like
Please add the necessary downstream deployment configuration to automatically publish new builds to WinGet.  See https://docs.microsoft.com/en-us/windows/package-manager/package/
Describe alternatives you've considered
Current alternative is to manually keep binaries updated via downloading and installing.
Additional context
n/a
Note:
As of the GA of the v7 CLI, the v6 line is no longer under active development and will be updated only to patch CVEs or fix severe blocking defects with no workarounds.

Just stumbled across this chocolatey package: https://community.chocolatey.org/packages/cloudfoundry-cli
Maybe this can help you?

Thank you @codepwner for your submission. As @wanddynosios mentioned, we have chocolatey package for CLI v8 for windows installation. So will close this issue.


Add skypeForConsumer as conferencing provider.

Free Outlook accounts supports creating  skype as conferencing while creating events.
Currently this is not listed as supported conferencing provider in JAVA sdk.
When user fetches event and an event has skype as conferencing, the request fails due to unsupported conferencing.
Add skypeForConsumer as supported conferencing provider.
Reproduced the error locally and tested the fix locally.

License

I confirm that this contribution is made under the terms of the MIT license and that I have the authority necessary to make this contribution on behalf of its copyright owner.

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 79.82%. Comparing base (2dd721f) to head (98c7c6f).

Additional details and impacted files
@@            Coverage Diff            @@
##               main     #255   +/-   ##
=========================================
  Coverage     79.82%   79.82%
  Complexity      287      287
=========================================
  Files            31       31
  Lines           897      897
  Branches         88       88
=========================================
  Hits            716      716
  Misses          130      130
  Partials         51       51

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


Define a Simple Data Model and OData Service with CDS

Tutorial URL: https://developers.sap.com/tutorials/odata-05-data-model-service.html
Please specify the step you are referring to
Step 3: Set CF and CDS preferences in SAP Web IDE
The screen I get when I check the Cloud Foundry preferences looks different to what is shown in the tutorial as there isn't anything pre-filled in the input fields and when I select the shown API endpoint I get a pop-up to login:

Is this step still needed? The informational message at the top seems to indicate that it's no longer needed.
I had worked through the tutorial shown as a prerequisite.
Thanks and Cheers
Bärbel

Not too surprisingly, not being able to complete Step 3 to select the CF Space, I'm now not able to complete Step 11 to create the database. When I try to build the DB I get an error message:

Being a complete beginner and mostly not really knowing what I'm doing where and why in the course of the tutorial, I'm not really sure how the mentioned prerequisite for the Cloud Foundry mentioned in the linked official help documentation applies to the trial setup. How could I e.g. "ask your administrator to create a subaccount for you" in this type of setting as mentioned in the help documentation?
What could I try/do?
Thanks & Cheers
Bärbel

Hi @BaerbelW  - I would suggest a two level approach. If we can get you going quickly here, let's do that; otherwise I'd suggest you take this to answers.sap.com as it's not specifically related to the tutorial content itself, and others may benefit from the question and the answer.
Let's see if we can get you going quickly - go to https://cockpit.hanatrial.ondemand.com/cockpit/#/home/trial and hit the "Enter Your Trial Account" button. Follow the process for creating the subaccount, CF org and space, and see where you get to. Then retry this step in the tutorial. If you get stuck with this, come back to me here first :-)
In the meantime, I'll be modifying the step (3) anyway thanks to an issue that @IObert pointed out #3730

Hi DJ - thanks for bearing with me. I'm really confused by all of this (I'm sure you can tell!) and it doesn't help that I don't really know what I'm doing or supposed to be doing here. So, I think I now have a CF trial account:

Do I now have to click on "New Subaccount" and if yes, what - exactly - do I have to enter/select in the next pop-up?

Thanks much & Cheers
Bärbel

Just click on the big while oblong (with "trial" in the top left of it) - that's your existing subaccount. That should take you to where you can check / create your CF org and space. I think at this stage it's better if you take this over to answers.sap.com as we're moving away from specifics for this particular tutorial - hope that's OK! See you over at answers!

Hi DJ,
all sorted (I think): while capturing screenshots for a question to ask on answers, I tried to login with another combination of email & pw I saw in Chrome's list of stored credentials which was accepted so I could then proceed. While in the list I also deleted some entries utilising my S-accounts instead of the P-account I had set up for the trial. It's really tricky to keep track of all these different logins needed to work through just a simple tuturoial!
Thanks for helping!
Cheers
Bärbel

Great! And thanks for letting me know too. Cheers and have fun.


added error ratio to metrics

Added Error ratio to list of available metrics in Locust

Why add this here instead of just making it part of the metric query on the experiment?
I kind of like that these images expose metrics 1-1 with the underlying tool.

Why add this here instead of just making it part of the metric query on the experiment?
I kind of like that these images expose metrics 1-1 with the underlying tool.

I agree that it's nice to have only the metrics from the underlying tool. However that means that you cannot optimize for the error_ratio from the app.yaml you would have to edit the generated experiment.yaml.

I agree that it's nice to have only the metrics from the underlying tool. However that means that you cannot optimize for the error_ratio from the app.yaml you would have to edit the generated experiment.yaml.

You cannot optimize on arbitrary metrics from the application definition (you are limited to "objectives" which are currently resource requests and latency) so no matter what it is going to require manual edits.
We could add an "error rate" objective that is smart enough to use the StormForger error_ratio metric or generates the failure_count/request_count query for you if you are using Locust.

You cannot optimize on arbitrary metrics from the application definition (you are limited to "objectives" which are currently resource requests and latency) so no matter what it is going to require manual edits.
We could add an "error rate" objective that is smart enough to use the StormForger error_ratio metric or generates the failure_count/request_count query for you if you are using Locust.

I did not realize that we only had access to cost and latency in the objectives but i see that here: https://github.com/thestormforge/optimize-controller/blob/1f9ccf2ba4eea94a4d748d19323d925cb0176563/api/apps/v1alpha1/application_defaults.go#L105
Well i think it's worth adding error_rate or error_rate and pull it directly from stormforger metrics or do the ratio there for locust as you said. I am closing that PR then and we can do it on the controller side.

I did not realize that we only had access to cost and latency in the objectives but i see that here: https://github.com/thestormforge/optimize-controller/blob/1f9ccf2ba4eea94a4d748d19323d925cb0176563/api/apps/v1alpha1/application_defaults.go#L105
Well i think it's worth adding error_rate or error_rate and pull it directly from stormforger metrics or do the ratio there for locust as you said. I am closing that PR then and we can do it on the controller side.


x/tools/cmd/oracle: Support recursive search of packages to include in scope

Oracle takes a list of arguments that set the analysis scope. If a given argument (e.g. github.com/example/project) does not have any immediate source files, it is excluded from the analysis. However, many projects have sub-packages that do have main or test files, and enumerating these manually to oracle is difficult to manage. The user should have some way of indicating to oracle that a given path should be treated recursively and searched for any sub-packages that may have main or test files. In other words, given github.com/example/project which only has two sub-projects github.com/example/project/subproject-one and github.com/example/project/subproject-two which have main and/or test files, both should be included in the analysis scope.
This could be indicated with a command-line parameter ("includes packages recursively in scope") or as part of the path itself (github.com/example/project/...).

This has been implemented in https://github.com/golang/tools/commit/3f95c6634578c08ac4fc93560638031f114d076b


Car Unsupported

Describe the bug
Hey all, just bought my first comma 3x, however im getting Car Unrecognized: Dashcam Mode during setup. I switched to the master-ci branch and verified my car fingerprints are present on the branch, however im still getting the same error. Any Idea what can be done ?
Which car does this affect?
Honda CR-V 2017
Provide a route where the issue occurs
a3367c53243b7b6a|2024-01-06--15-34-40
openpilot version
0af7c3ed803676f2023996a962dd2448610e3d0a
Additional info
No response

"carFingerprint": "mock"
is this an issue?

Reconnecting all cables seems to have fixed the issue


[Ziro] Selecting 'gray' color scheme selects 'blue' scheme instead

Both gray and blue color schemes are identical.

OS -> Windows 10
Spotify version -> <IP_ADDRESS>9.gf1bb1e36
Spicetify version -> <IP_ADDRESS>
Ziro

Can you send screenshots? The themes look different to me.
gray-dark

blue-dark

Ahh, I forgot to add '-dark'.
For some reason all of the three color scheme names:
'gray', 'blue' and 'blue-dark'
select 'blue-dark' scheme for me which got me confused.


Addition: keychain parser

I've written a small perl script that parses the output of the security command on OSX.
I've only tested it on a couple of keychains that I have available but it seems to provide a slightly more readable version that Apple does.
It can be found here; https://github.com/randomaccess3/MacForensics

Hi Phill,
with "slight" delay (really sorry for the long on-hold period of the project):  maybe I have missed it, but is there any new artifact location you wanted to add from this? Or simply to notify/add the tool to the project page?
If the latter, I think a better location could be to include it in https://github.com/meirwah/awesome-incident-response#osx-evidence-collection

I only vaguely remember what I used this script for - probably doesn't even work on todays systems


The extrinsic params used in the project

Hi, dear author, @Qingq-li
Nice work! Thanks for your nice working.
I'm wonder what is the extrinsic param between the vlp16 and the livox lidar (In the data set you provided )?
Can you give me some help?
Thanks!

The identity matrix is used in the picture, clearly wrong.

calib .


Build fails with both shaded OTP and non-shaded OTP

At first analyst-server would not build because it was using two conflicting versions of JTS. OTP depends on geotools 10.5, which depends on JTS 1.13, while gtfs-lib depends on JTS 1.8. PreparedPolygon from JTS 1.13 does not like Polygon from JTS 1.8.
We were only making one JAR of OTP, the “shaded” one that includes all the dependencies. That was being deployed to the Maven repo. It’s pure luck that this ever worked at all, since the transitive dependencies of OTP can’t be seen/managed by build tools that include OTP via a Maven repo.
This has been fixed via ticket https://github.com/opentripplanner/OpenTripPlanner/issues/1986 .
Now, depending on the non-shaded OTP JAR, we get another error from SBT/Play:
java.lang.IllegalStateException: impossible to get artifacts when data has not been loaded. IvyNode = org.apache.httpcomponents#httpclient;4.3.1
All references I can find to this error on the web indicate that it's a bug in Ivy but if so it hasn't been fixed after many years. As a temporary fix, you can add this problematic transitive dependency directly to the SBT build (see http://stackoverflow.com/a/25126544/778449). In this case, I added  "org.apache.httpcomponents" % "httpclient" % "4.3.1", and the build completes successfully. Why, I have no idea. Might it have something to do with circular dependencies?
Increasing the logging level to Debug (add logLevel := Level.Debug to build.sbt) does not reveal anything particularly helpful.

Fixed in master as well.


How to install and use wav2letter in jupyter notebook

Hello guys, I'm start learn this approach, I try some approach for install wav2letter (e.g. install flashlight using vcpkg), but unfortunately when the installation is done this command from wav2letter.decoder import CriterionType doesn't work.

I changed the status of this issue to close because I find a way to use wav2letter. In summary, wav2letter was moved to flashlight project, and for install in Jupyter Notebook, just need to follow this steps: this link. Or if you want to test using Google Colab, the author also provide a guide in here. I hope this tip help other in the future.


Pagination List of List

Our documentation says that any request involving pagination will return a list of lists. Each list including one page of results.  See AuthenticatedClient.get_orders() for an example. Do we actually want this behavior? My first thought while using that method would be to return a single list including all results.
I have made some changes to make these return a single list, and also to remove all of the current pagination helpers and replace them with while loops. You can see the changes on my "auth-client-cleanup" branch
I think that this matches the API spec better, and conforms to our new guiding principles. If you guys agree I will include those changes in my next PR which includes some other fixes and docstrings for AuthenticatedClient.

Oh and in get_fills(...) we were using the limit param incorrectly so I modified that to comply with the API spec.

limit	100	Number of results per request. Maximum 100. (default 100)

I agree, it makes sense to abstract away pagination through the API. My thought for paginated requests was to allow before, after, and limit parameters to be specified through **kwargs (and to describe these parameters in the docstring so the user knows they're there :) ). That way they're available if needed but don't clutter up the function signature too much.
I actually made very similar changes on my authorized-client-cleanup branch. Maybe we can align our changes between our branches to present a unified PR here?

Nice! That is exactly where I was leaning with thebefore after and limit.
Also I think that we should abstract out all of the other calls to requests into a private method. Something along the lines of_make_request(self, http_verb, url, params). That could internally use a version of _get_paginated_response().
How can we collaborate on a branch prior to submitting a PR?

@Jigsaw-Jams That idea sounds good to me. If you don't mind working on my authorized-client-cleanup branch, we can collectively push our changes there and then one of us can push the final PR here. Just sent you an invite.

Shouldn't we use a generator for pagination? There should be no need to make multiple requests unless the client calls next().

This makes a lot of sense. I was sort of trying to go along with the philosophy of making the methods do exactly what the API docs say, but if we want to go the generator route, I'm not opposed.

Using a generator means letting the user handle multiple pages if they want more than 100 results into the past, right? Or is your idea for the generator to grab results added since the last method call?
I like the idea of abstracting away pagination so the user can simply ask for 300 results and get them, regardless of how many HTTP requests it takes. We can still expose enough options so results can be returned a single page at a time in case that behavior is desired (the use case I can think of is getting as close to real-time processing as possible, but since
I do like the idea of exposing a way to simply grab the results added since the previous method call. That could be a generator or simply saving a page state for each paginated function in the class object.

While I see the usefulness of limits, I'm not convinced that we oughta include it in the pagination logic. If we return a generator, and the client wants only 300 results, they can simply say:
from itertools import takewhile
stuff = takewhile(lambda i: i < 300, client.get_stuff())

…although I now realize that would be 300 pages of stuff. Maybe it would be good to provide help.

Well, no, it wouldn't have to be. We should provide a generator that simply yields each next item in the page, and then fetches the next page as needed. The user would have no visibility into that process, so takewhile(lambda i: i < 300, client.get_stuff()) should work fine.

I've never used takewhile before, pretty cool! I also realized that one important use case might be that you have 100,000 fills in your account history and you want to dump them to a DB without holding the entire history in memory - a generator would make that easy.
I'll try to sketch out the use cases for the two proposed APIs here. I haven't used generators very much, so you should correct whatever mess I write :)

Yep, you're exactly right about that use case.
Re: sketched out use cases: Sounds good! Thanks @acontry

I should point out though that it would be pretty easy to do even without takewhile.
i = 0
g = client.get_stuff()
while i < 300:
    try:
        do_something_with(next(g))
    except StopIteration:
        break

The question of whether limits are worth including in the library is, I think, ultimately a question of the trade-off between how much complexity it adds versus how much it helps users who may not know how to limit generators. Generally I prefer to give users credit, but if it's just another line or two of code, it really wouldn't bother me.

I've spent some time figuring out the dirty details of how this pagination works by playing in the sandbox API. The logic to manage pagination for the user isn't too complicated, but it was tricky to figure out some of the edge cases and understand exactly how before/after work. Ideally the user shouldn't have to dig into these API details, so I think managing pagination (through generators or otherwise) is worth it.
I have three different scenarios and three different APIs. The first API doesn't manage the cursors at all, so it's up to the user. the second and third ones do manage cursors by maintaining state, but present different interfaces.
Dump entire fill history (which doesn't fit in memory) to a DB.
First option: function returns result and dictionary page_id with the current cursor locations. To grab data that happened further in the past, pass after=page_ids['after'], and for data that has happened more recently since the last call, use before=page_ids['before'].
result, page_ids = client.get_orders(limit_pages=10)
while page_ids['after'] is not None:
    result, page_ids = client.get_orders(limit_pages=10, after=page_ids['after'])
    process_result(result)

Second option: client object manages the cursors for the beginning and end of data received. Use which_orders='older' to grab data older than the last received data, or which_orders='newer' to grab more recent data since last call.
while True:
    result = client.get_orders(limit_pages = 10, which_orders='older')
    if not result:
        break
    process_results(result)

Third option: get_orders_gen() returns a generator to get older and older data, and also a cursor which can later be used to grab more recent data.
order_gen, before = client.get_orders_gen()  # order_gen yields one result at a time
while True:
    try:
        process_result(next(order_gen))
    except StopIteration:
        break

Get 300 most recent orders.
result, page_ids = client.get_orders(limit_pages=3)

result = client.get_orders(limit_pages=3)

from itertools import takewhile
results = takewhile(lambda i: i < 300, client.get_orders())

Get recent orders. Sometime later, return any orders that have occurred since last call.
result, page_ids = client.get_orders(limit_pages=1) # limit pages so we don't dump full history
# sometime later
result, page_ids = client.get_orders(before=page_ids['before'])

result = client.get_orders(limit_pages=1)
# sometime later
result = client.get_orders(which_orders='newer')

from itertools import takewhile
order_gen, before = client.get_orders_gen()
results = takewhile(lambda i: i < 100, client.get_orders())
# sometime later
order_gen, before = client.get_orders_gen(before) # feed in `before` so generator stops at data already received
results = [x for x in order_gen]

What an impressive write-up.
My personal vote is for Option 3, because:

it makes the elimination of unneeded requests effortless
It does not attempt to maintain state between calls, which could be problematic if the function is called from more than one place

Does the third option need to yield (results, before)? Why not just results?
One additional use-case that I think should be documented here is "I just want all of the results." I think ideally client.get_orders() should work for all three options. Would it?

Thanks for the comments! Now that I've figured out how to use a generator to work cleanly with all our scenarios, I'm on board with going that route. No shared state for the client is a good thing! I added your scenario to the list below,
Dump entire fill history (which doesn't fit in memory) to a DB.
order_gen = client.get_orders()  # get_orders() is a generator, yielding one order at a time
while True:
    try:
        process_result(next(order_gen))
    except StopIteration:
        break

Get 300 most recent orders.
from itertools import takewhile
results = takewhile(lambda i: i < 300, client.get_orders())

Get recent orders. Sometime later, return any orders that have occurred since last call.
from itertools import takewhile
order_gen = client.get_orders()
results = takewhile(lambda i: i < 100, client.get_orders())
# For orders, the cursor used is order id number. We save id of the most recent order here.
cursor = results[0]['id']
# sometime later
order_gen = client.get_orders(before=cursor) # feed in `before` so generator stops at data already received
results = [x for x in order_gen]

Just give me everything!
results = [x for x in client.get_orders()

@acontry This is exceptional work. I'm totally on board with it, and I look forward to reading other people's thoughts.
The only thing I have to add is that for the last two scenarios, those list comprehensions can be replaced with list(), as in list(client.get_orders()). But in most cases it's preferable to avoid reading them all into a list, because that defeats the advantages of using a generator in the first place. If you wanted to process them iteratively, you could say something like:
for order in client.get_orders():
    do_something_with(order)

No list required!

@bryankaplan  Thanks for the syntax tips! Updated my above post to reflect those suggestions.

I really like this.  Are there any changes to the repo that may help to make this better for the end user?


Bundle of patches for WMAgent 1.4.5.patch1 release

Backported version of the following 3 PRs (test/pylint changes skipped):

Trap SSL_connect error and Service Unavailable in DBSUploadPoller #10252
[py2py3] pycurl_manager - better handling of curl.setopts() input arguments #10250
Make WMAgent job wrapper slightly more verbose; fix one error message #10261
Check if DID exists before creating a replication rule #10265
Update production DBS URL to cmsweb-prod #10269

 Fix #10263 - Bundle of patches for WMAgent 1.4.5.patch1 release

This has all been tested as well, but there are so many changes that another quick test with a brand new release (and clean!) is required. Merging everything that has been backported.


small changes to the Makefile.linux

Might be an idea to put the BLANKER stuff into the linux Makefile too. Took me a while to work that one out :)

Fixed: d121eaf69459f3c54f1988c3a0dfbfe82b7272b8


Create a property file to set some defaults for shibby

A property file containing the default download location would be useful. This can be expanded to include other things like default libraries for searching and other properties for new functionality.

With this should be functionality to view the properties with a command like --show-properties or something.


Create an SAP Cloud Platform ABAP Environment Trial User

Tutorial URL: https://developers.sap.com/tutorials/abap-environment-trial-onboarding.html
Please specify the step you are referring to
When trying a create an instance in cockpit at step 2.7, it gives the following error -

Unable to create service instance. We are sorry, but we have encountered an internal error. If the problem persists, please create a support ticket.
Service broker error: Internal Server Error

Since it's a trial account, i cannot log a support ticket.

@mathan91 Please try it out again. Otherwise delete your subaccount and your dev space and create it again with Europe (Frankfurt) or US as region and try it out again.

Hi @mathan91,
do you need further help? Or can we close this issue?
Best regards,
Merve

@mathan91 Thank you for your feedback. We haven't heard from you in the past 30 days, so we are closing the issue.
If you still have questions, feel free to reopen the issue.

I have de same problem.
My  subaccount was created with Europe (Frankfurt) - AWS.


nmigen.build.res: Ensure all pins are available in a DiffPairs

Fixes #456

Breaks here:
Traceback (most recent call last):
  File "soc.py", line 108, in <module>
    platform.build(soc, do_program=True)
  File "/home/jeanthomas/Documents/nmigen/nmigen/build/plat.py", line 90, in build
    plan = self.prepare(elaboratable, name, **kwargs)
  File "/home/jeanthomas/Documents/nmigen/nmigen/build/plat.py", line 151, in prepare
    for pin, p_port, n_port, attrs, invert in self.iter_differential_pins():
  File "/home/jeanthomas/Documents/nmigen/nmigen/build/res.py", line 182, in iter_differential_pins
    yield pin, port.p, port.n, attrs, res.ios[0].invert
  File "/home/jeanthomas/Documents/nmigen/nmigen/hdl/rec.py", line 149, in __getattr__
    return self[name]
  File "/home/jeanthomas/Documents/nmigen/nmigen/hdl/rec.py", line 160, in __getitem__
    raise AttributeError("{} does not have a field '{}'. Did you mean one of: {}?"
AttributeError: Record 'ddr3_0__clk' does not have a field 'n'. Did you mean one of: p?


i18n(es): Update typescript.md & manual.md

What kind of changes does this PR include?

New or updated content
Translated content

Description

What does this PR change? Update typescript.md & manual.md based on this commit 5721d825a04a8a3d51292a12a53254aa84b84204

Thanks @kevinzunigacuellar


Proposal: "Beef Up" docs, issue templates, etc.

As media-chrome starts moving to a more "official" project, we should probably start making the non-code aspects of the repository a bit more "polished" as well. By way of example, the mux-elements has added things like:

Contributing Docs and Badges

Including a Code of Conduct

npm package badges
Issue Templates
Additional Labels
husky pre-commit hooks
standard .gitignore

fyi: for now, npm badges are in pending PR here: https://github.com/muxinc/elements/pull/35/commits/793d48919f184854c596564b448b7b7ed142193f

💯  I'm in favor of this

Sounds good to me. I'm not familiar with husky pre-commit hooks.

Husky pre-commit hooks are most often used to run yarn lint anytime you do git commit -- it prevents developers from pushing up commits that will fail at the linting step


Authorization V2: One bugfix and some optimizations

This PR contains 3 topics:

Bugfix: There was a server error when a staff-user without a global role views another user with a global role
Optimization: There where duplicated selects to read the Role in dialogues for Product, Product Type, User and Group
Maintenance: There was still functionality to get a API key for API V1, which caused a server error

Thanks @valentijnscholten, I think I found all occurences


Read set download link seems broken

Hi,
Thanks for hosting and updating this great evaluation. We're interesting in looking at how Canu 2.1 is doing on this data and seeing if we can improve Canu performance. It seems the read set download link is not working right now (https://bridges.monash.edu/articles/dataset/Read_sets/11369952). Both files report:
<Error>
NoSuchKey
<BucketName>fpfiles</BucketName>
<RequestId>
tx000000000000000001ad1-005f6e0807-148c5119-au-east
</RequestId>
<HostId>148c5119-au-east-au</HostId>
</Error>

Using download all aborts after 80 bytes (I assume because the files aren't found). No issues downloading the reference sets. Could you please check if the read links are correct and let us know how to download the data?
Thanks

Hi, have you solved this problem yet?  I got the same error.

So sorry about this - it's a problem with Figshare/Bridges. I've reported the problem to them (many times) and they assure me they are on it.
In the meantime, somebody else ran into this problem and I scped the files to them, and they kindly made them available on their Dropbox:
https://www.dropbox.com/sh/cfwabn5cywmuq6e/AAC24OU48F-omvj0aHi5KbeBa?dl=0
So give that a try! And I will continue to nag about the broken links 😄


ariaLabelOverride not working

The ariaLabelOverride property is not working
Requested enhancement: ability to use the index within a custom aria label through the override/new callback to get the current index that can then be used however needed

The property ariaLabelOverride is not working. I have managed to get a workaround using this:
inputProps={{ "aria-label": "Custom aria label" }}
But it would be great to get the dedicated parameter working.
Also what would be amazing is if there was a way to get access to the index to be able to add it into the aria label. The default aria label says which digit you are on for each input which is a better experience for users that require a screen reader, would be great to be able to add a custom aria label (we  translate it based on locale) but still have that index information included.

@Alison-Sarah-P-C Could you share me your implementation as well as the DOM screenshot to check if it is not working.

@Alison-Sarah-P-C Any update on your implementation, this issue would be closed due to non-response after 7 days.


Allow a different extracted file naming scheme

Going to vote for preserving the original file extension for the file naming scheme. When coming up with formats that need signatures created having the original file extension is a good way to do a first pass in splitting the files up into groups of similar files.
This is particularly useful when pulling application/octet-stream files.

Thinking about this some more, the rest of the original file name also has value for figuring out the file format. File names like 1-butanol_PROTON-1-1.jdf and Comfortaa-Regular-LGR-Slanted.vf give hints about the contents of the file, and allow us to trace an original file back to a specific website (which we can also do through the index file or the extracted-urls.csv file).
However, appending something to the original file names so as to avoid duplicate file names brings the possibility of running into filesystem limits. Tricky.

It is tricky. After dealing with this again and again, I’ve defaulted to
current setup: file hashes as names and a sidecar table with source urls.

On Fri, Jun 2, 2023 at 3:28 PM gleporeNARA @.***> wrote:

Thinking about this some more, the rest of the original file name also has
value for figuring out the file format. File names like
1-butanol_PROTON-1-1.jdf and Comfortaa-Regular-LGR-Slanted.vf give hints
about the contents of the file, and allow us to trace an original file back
to a specific website (which we can also do through the index file or the
extracted-urls.csv file).
However, appending something to the original file names so as to avoid
duplicate file names brings the possibility of running into filesystem
limits. Tricky.
—
Reply to this email directly, view it on GitHub
https://github.com/tballison/commoncrawl-fetcher-lite/issues/29#issuecomment-1574214445,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABTNNPWZVAHHF262MOVVRYTXJI5HBANCNFSM6AAAAAAYYODT6E
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

Agreed, I've got some ugly hacks to rename the files based on the values in extracted-urls. I guess that will work.

Yes.


install the package that installs python2 and install docutils for rst support

This makes restructredtext rendering work.
Fixes #3

@exarkun
Thank you!


Golden tests can fail due to relative paths

I'm assuming this failure is due to relative paths and the process's working directory. Ideally the test wouldn't be sensitive to that.
$ cd ouroboros-network
$ cabal v2-run ouroboros-consensus:test:test-consensus
...
    Golden tests
      ChainState:                                         FAIL
        Exception: test-consensus/golden/cbor/byron/ChainState: openBinaryFile: does not exist (No such file or directory)
      LedgerState:                                        FAIL
        Exception: test-consensus/golden/cbor/byron/LedgerState: openBinaryFile: does not exist (No such file or directory)
      GenTxId:                                            FAIL
        Exception: test-consensus/golden/cbor/byron/GenTxId: openBinaryFile: does not exist (No such file or directory)
...
$ ls -1 ouroboros-consensus/test-consensus/golden/cbor/byron/
ChainState
GenTxId
LedgerState

After running ln -s ouroboros-consensus/test-consensus, the tests pass.
cc: @mrBliss

I see. With stack, I don't have this problem, I can successfully run the tests from any directory within the project.
Surely there must be a solution for this, as we can't be the first to run into this.

shakes head sadly


[New Segment] TaskWarrior

Description
This is a new segment created to display data from TaskWarrior.
The segment was named tw.

Data is obtained with:
today=$(task status:pending +DUETODAY count)
over=$(task status:pending +OVERDUE count)
pending=$(task status:pending count)
The icon used is: TODO_ICON

The test-in-docker script is for manual testing only (and they are defunct ATM; see #884). To write automated tests you need shunit, which is a git module of p9k. Here is a short introduction (which I should copy over to TESTS.md..):
Installation
You need to install the submodule first to be able to run the unit tests. You can do that with git submodule init && git submodule update.
Test location
The tests live under test/. Segment specific tests under test/segments/.
Executing tests
The tests are shell scripts on their own. So you can execute them right away.
General Information
The tests usually have a setUp() function which is executed before every test function. Speaking of, test functions must be prefixed with test. In the tests, you can do different Assertions. It is always a good idea to mock the program you want to test (just have a look at other tests).
Travis
We use Travis for Continuous Integration. This service executes our tests after every push. For now, we need to tell travis where to find the tests, which is what happens in the .travis.yml file.
Hope that helps.

Thanks @dritter !
That was really helpful.
I managed to create some tests and everything seems to be working now.

Hey @nzachow - Thanks so much for filing this PR! It looks like it's kind of gone into a holding pattern since July, but I would really like to get this one finished up and merged into our next branch! It's a great contribution - sorry it's been a bit of a pain to tie up.
@nzachow - The next branch is a new codebase, and is organized a bit differently (much more cleanly). The only major changes for you will be moving your segment code to its own file in the segments/ directory, and shortening any env names from POWERLEVEL9K_ to P9K_.
@dritter @onaforeignshore - Was there anything else you wanted to see done, here, before it gets merged? Obviously the failing tests & conflicts need to be resolved, but I suspect once @nzachow updates the code that will be easy / resolve on its own.

Hi @bhilburn !
From what I recall the biggest hold up was something
related to the tests. I have the impression that they were
passing locally but Travis wasn't able to run them correctly,
and I could not find why.
After that I kinda forgot about this PR ...
I'll take a look into the next branch to see how easy would
be to merge this code.

@bhilburn @nzachow The new codebase doesn't make use of segment states in the same way as the original code. You now register a complete state with its own icon and colours. This will require some rewriting to use the new way.
There is still a call to gawk which is definitely not available on all systems that needs to be resolved.

Agreed, the external call to gawk needs to be replaced with pure ZSH. @nzachow - let us know if you need help with that.
@nzachow - Check out some of the other segment code in the segments/ directory in the next branch, it should serve as a good example of how to make things work, now, and please let us know if you have any questions. We need to update the developer docs for the new codebase, so you going through this process would actually be really helpful for us!


a test case for providers that should run as profiles other than default

in this case, eunit should run as test

I believe #226 includes this PR, right?

@talentdeficit yea, this is in #226 right? So should we close this?


Better error handling of gh api failures

Description

In https://github.com/sigstore/root-signing/blob/main/.github/workflows/scripts/review-pull-request.sh, we should handle whenever gh api fails and pass this status to https://github.com/sigstore/root-signing/blob/main/.github/workflows/review-snapshot-timestamp.yml

closing -- I'm sure there are still cases where error handling is not stellar but these particular workflows are no longer an issue