Command-line-parameter for alternate Mime type mappings

Which version of the AzCopy was used?
8.10-netcore
Which platform are you using? (ex: Windows, Mac, Linux)
Windows
Azure DevOps has a predefined Build/Release task definition built on AzCopy to publish/copy files or complete folders to Azure Blob Storage.
https://github.com/Microsoft/azure-pipelines-tasks/tree/master/Tasks/AzureFileCopyV2
We found that we can't use this task since it does not allow us to specify the file extension to content-type mapping for the individual files ourselves, and even in v8.10 of AzCopy the mapping that comes in AzCopyConfig.json is not complete.
There is also no universal truth about what is the correct content-type for a file extension, this of course ideally should be decided by the owner of the storage account.
If AzCopy just had a command-line switch to specify path to AzCopyConfig.json, we could use our own mapping file and the Azure DevOps task would be working for us.
Would it be possible to add this?

Hi @josundt, thanks for reaching out!
If sticking to AzCopy V8 is a mandatory requirement, then @EmmaZhu could perhaps chime in to help with your issue.
However, if you don't mind switching to the V10, we have this working as of today. Here is the documentation:

Please note that AzCopy automatically detects the Content-Type of files when uploading from local disk, based on file extension or file content(if no extension).
The built-in lookup table is small but on unix it is augmented by the local system's mime.types file(s) if available under one or more of these names:

/etc/mime.types
/etc/apache2/mime.types
/etc/apache/mime.types

On Windows, MIME types are extracted from the registry. This feature can be turned off with the help of a flag. Please refer to the flag section.

Apologies for commenting on an old issue, but what happened to the quoted documentation? All documentation on MIME types/Content-Type for AzCopy appears to have been scrubbed.

Hi @IGx89, no worries. To clarify, are you talking about V8 documentation?

V10 -- your quote above appears to be a snippet of V10 documentation about setting content type/MIM type, but when Binging ;) for the document it only brings up this issue.

@normesta Did we lose the mime type docs in the recent refactoring of the docs?  Shall we reinstate it as a FAQ entry perhaps?

So there will no longer be a way to explicitly define the extension/mime-type mapping with AzCopy?
That is sad, because operating systems are no source of universal truth to this.

Hi @josundt, do you mean specifying mime-type for each individual file instead of for file extensions in general?

It is essential for us to provide our own mappings for extension/mime-type when batch uploading. Relying on the OS mappings will not be suficcient

@josundt Just to clarify, there are two ways that AzCopy v10 can currently get Mime Types: from the OS, or by a command line parameter that specifies one Content-Type for all the files in the upload job.
We don't currently have a command line argument that can specify a path to a json file with the mappings.

Also, the "docs" extract above is actually from the in-app help, that AzCopy v10 displays with the "azcopy copy --help" command.  It hasn't been scrubbed. It's still there.
@josundt, at this stage, given that we don't have a path with the mappings, the only workaround I can think of for you, with AzCopyV10, is to rely on the files noted in that doc: e.g. /etc/mime.types  If you're running on Linux you can use that directly. If you're running on Windows, you could possibly consider using Windows Subsystem for Linux (WSL) and running the Linux version of AzCopy v10 inside WSL.  Sorry we don't have a cleaner workaround for you at this stage.

Some more background to make myself clearer:
We have attempted to use AzCopy to recursively copy/upload a local folder to blob storage from an Azure DevOps Pipeline Task. The goal is to upload new static content to our Azure CDN's backing store.
The static files consist of style sheet libraries, font libraries, javascript libraries and json files organized into multiple levels of subfolders (including version folders).
File formats inlcude .css/.map/.scss for styles sheets, .otf/.ttf/.woff/.woff2/.eot/.svg for web fonts, .js/,map/.ts files for javascript libraries, and .json files.
We have faced limitations we can't get around with AzCopy; we can't get the content-type correctly set on the target blobs, since the mime mapping table in the build agent OS registry is incomplete/incorrect in the context of our copy job.
The main point that I tried to emphasize in my earlier comments is that there's no such thing as a universally correct mime-type for a file extension.
Example:
A .ts file could in some contexts mean a "Transport Stream" mulitmedia file format.
In other contexts it could mean "TypeScript source file".
For this reason I believe that a "true" mapping table is not logically scoped to:

The operating system where AzCopy runs (AzCopy v10).
Nor the AzCopy.exe working directory (AzCopy v8).

In some scenarios (like ours); full control over this mapping table is needed for the single execution of the AzCopy command.
AzCopy had something close to what I request in AzCopy version 8; it was possible to configure mappings in a JSON file, but this file needed the name "AzCopyConfig.json" and it needed to exist in the AzCopy working directory.
What I want is the possibility to have a file similar to "AzCopyConfig.json", but to tell AzCopy where to find it (the path). (In our case, it would make sense to keep this file in in our git repository which is checked out to a given path on the Azure DevOps build agent).
I think what I am trying to accomplish here is not at all an edge case usage of AzCopy, it is a rather straight-forward common copy job that I believe AzCopy needs to support. I would be surprised if not other people have the same need and would benefit from this.
This is why ask for a feature request for a new command line argument:
/ContentTypeMappings:<path-to-mappings-file>

Thanks for the clarification @josundt, it's useful. I have logged it as a feature request in our product backlog.  Unfortunately I can't give you any expected date by which we might look at it. I'm not trying to be difficult ... just aware of how much stuff is already there!
In the meantime, I wonder if there's any chance you can script modification or replacement of the v8 json file, at the time your build runs... Not ideal, I know...

That's actually what we are doing at the moment. We keep the full AzCopy folder (241 files) in our Git Repo so that we can use our custom AzCopyConfig.json file.
This works but is of course far from an ideal solution.
Hoping my real-life example make you realize that this will become a limitation for many others as well, and that you will prioritize the feature request thereafter.

Glad to hear you have a workaround at present. And thanks again for your input to our prioritization.

How can I follow the progress of the backlog item?

Our project management pipeline (for sprints etc) is in our internal DevOps instance, and that's where the backlog is so I can't give you a link to it, sorry.
We have work roughly planned out for the next 4 months and, I'm sorry to say, "your" backlog item isn't included in those 4 months.  Currently it's sitting in our "un-triaged" bucket, which means we have not yet committed that we will actually do it (ever), and if we do, it will be at least 4 months until we start.  I feel bad saying that to you, because obviously you'd like us to look at it sooner :-) But the reality is that we need to be fairly "brutal" about prioritization and it's hard to justify earlier consideration of an issue where there is actually a workaround. (Even though we agree that your workaround with v8 in your Git repo is less than ideal).
I have marked your issue with a tag that means "this may be more important than other 'untriaged' requests".  That will draw our attention to it when we start planning beyond the 4 month period I've already mentioned.  But even that tag does not guarantee any particular outcome.

Came here to vote for this issue.
I've got a simple proof-of-concept site consisting of one HTML file, one CSS file, one JS file, and one .svg image.
The site is deployed to Azure blob storage using BitBucket Pipelines, which uses the Azure Deploy Pipe.
The pipe had a habit of uploading the .svg file as "application/octet-stream", when it needed to be "image/svg+xml".
I was able to get the Azure Deploy pipe updated so that it now uses azcopy v10 -- up from azcopy v7.3 previously.
However, azcopy v10 has the same issue, transferring an .svg as "application/octet-stream".  Since the pipe is in a Docker container that's not defined by me, it's not really feasible for me to set .json in that container's /etc/ folder.
A way to manually set MIME types by passing parameters to a configuration would be great!
Most of the above is just to point out that I'm not doing anything especially arcane; I'm just deploying a simple static site, but azcopy is stomping on one of my "regular" files.

Is it just the svg's that you have the problem on?  If so, there's a slightly awkward workaround that you can probably use for now (until we fix it properly, one day).  It goes like this:

Upload all your files (it's OK if the SVGs are included here)
Now, run AzCopy v10 again with the --content-type parameter set to "image/svg+xml" and set the other parameters such that only the SVG files are uploaded.  (The upcoming v10.3 has a nice --include-file *.svg syntax for this.  If you can wait a small number of weeks, that's your best option. Otherwise experiment with wildcards in the source path in v10.2.1. If you are using v10.2.1, you'll also need to add --no-guess-mime-type, because otherwise there's a bug where inference incorrectly overrides --content-type).

Step 2 overwrites the SVGs of step 1, with the same file content, but the right content type.
Obviously this workaround only works if you need it for a very small number of files types.
As noted somewhere (far) above in this thread, the --content-type parameter applies one content type to all files in the azCopy job.  Hence the need to filter to SVG files when specifying that parameter.

BTW, I can't remember how source path wildcards interact with --recursive in v10.2.1.  But I suspect the answer is that they don't.  In which case you should probably wait for 10.3 if your SVGs are scattered across subdirectories.

Correction, the v10.3 parameter that I mentioned above will probably be called --include-pattern

It is indeed just the SVG that I've had an issue with -- and the whole point of this "toy" site being deployed to multiple places is to flush out issues like this and find out which cloud providers are responsive and helpful.  Thanks for this highly responsive and helpful, er... response!
What you are describing as a workaround makes sense: copy twice, with the second time JUST transferring the .svg and using a "set content type" on that copy action.  I'll give that a shot.
I'll also subscribe to the changes for azcopy.  Thanks!

I still think AzCopy v8 was the closest you have been to a good solution to this. In v8 you were allowed to create your custom extension/mime mappings by modifying AzCopyConfig.json (ref. my commented from Jun 17).
I want that v8 feature back but with the ability to specify the path to the JSON file through command line arguments.
I currently need to keep all files for AzCopy v8 in my source repo with a custom AzCopyConfig.json to make the blob publishing task work the way I want through my Azure DevOps release pipeline.
I am surprised that this is not prioritized as it must be a obstacle for all users of AzCopy that care at all about controlling the mime types in the blob responses after uploading them. And since you have already more or less implemented what I asked for in v8, it must be a quite small investment.

Hi @josundt.  We've had to make some tough decisions about prioritization, since there are lots of things that lots of people are asking for :-)  As you may know, v10 is a totally new codebase, so things that exist in v8 don't necessarily provide us with any shortcuts in v10.

OK. I'll keep patiently but expectantly waiting :-)

@JohnRusk Just wanted to add my vote to this -- I'm hitting this issue today and not sure there are any good workarounds.
I am also using an Azure DevOps pipeline to copy static resources to Blob Storage for our website. I actually started with the existing AzCopy task, however since it was using AzCopy v8, I ran into issues there because I need to set the Cache-Control header on my blobs when I upload them. So I wrote a custom PowerShell script to use AzCopy v10.
I'm finding that a bunch of files, including my minified .js files, are getting the wrong mime-type (text/plain). (Also, the JS/CSS .map files have the wrong type.)
The "workaround" of double-copying the files with the mime-type set doesn't work, as I use --overwrite false to avoid changing modification dates on files that haven't changed (we use 1yr+ cache expiration on our assets, which have a hash in the filename). I'd love to have some way to manually override the file extension to MIME type mapping used via a command line argument. Any possibility this is coming soon? Or is there a better workaround that works in a DevOps pipeline task without double copying?

Thanks @dpolivy . BTW, which OS are you on?

@JohnRusk My DevOps pipeline is on Windows (as is our build process and front-end web server). Not sure if linux would work for this, since at the moment it's just copying files, but is it even possible to edit /etc/mime.types in a DevOps pipeline shell task?

Good points. I don't know the answer re etc/mime.types,  sorry. Thanks for outlining how and why this is important.  At this stage, I don't have any update on likely scheduling of any change, sorry.

Being able to control MIME types would be really useful, well, actually essential.
I've discovered that AzCopy doesn't understand the .mjs extension is javascript which breaks my app (they get uploaded as text/plain and the browser won't accept that)

We would be interested in this feature as well purely because .js extension in windows doesn't default to the application/javascript (or x-javascript) mime type. (My guess is that it's a way to help prevent script execution somehow? shrug)
We migrated from linux -> windows with the same version of azcopy and ended up breaking a release because .js started uploading as text/plain and service workers won't load if the mime type isn't correct.
Once I dug into the docs as noted above the workaround I came up with was to do 2 passes: 1) just .js files with --content-type application/javascript and one for "the rest" which seems to map the same on windows as it did before.
The downside of this is that some builds don't have non-js files so we need to detect that and not run azcopy on the "other" pass when there are no non-js files because azcopy exits nonzero if no copies are scheduled.
This isn't optimal but it works!
All that said, I second a feature to provide a CLI option to map mime types :)

I'd like to add to this as I've just hit the javascript file mime type issue. I originally used the latest storage explorer to upload a static website to the $web blob container but it sets the mime type of any javascript file to plain/text which then throws an error in Chrome as the mime type is incorrect (this is using azcopy v10.3.3). I get the same issue running azcopy v10.4 from the command line BUT if I upload the file through the Azure portal it does set the type correctly. The problem here is that the portal only allows you to upload one file at a time (from what I can see) which is useless for uploading an entire website. Doing a 2 pass copy is just a massive pain and given uploading and running a static web site to the blob store is an advertised feature I would expect it to work with Javascript files correctly. It's very frustrating.

@rickedwards2001 I am facing exactly the same problem as you, in addition my CDN rules are not being respected because the content type of the files are not setted properly.

Thanks folks for the updates.  We are planning the fix some of these issues (most importantly the JavaScript ones) in an upcoming release.
I'd suggest you subscribe to updates, by using the "watch" button at the top of this page, and choosing to be notified when there are new releases.

Please import the following reg items for temporary solution.
Windows Registry Editor Version 5.00

[HKEY_CLASSES_ROOT\.js]
"Content Type"="text/javascript"

[HKEY_LOCAL_MACHINE\SOFTWARE\Classes\.js]
"Content Type"="text/javascript"

@capraynor I can't do that inside an Azure DevOps pipeline

This is also an issue for UWP app deployment; specific MIME types are required for the various MSIX file extensions.

I'm running into the JS text/plain issue.  I'm deploying an Angular site via Azure DevOps pipelines and using the aforementioned Azure Copy Files task, which relies on AzCopy.  All JS files are uploading as text/plain and Chrome/Chromium Edge refuses to run it.

What's the status on this? This issue basically makes it impossible to use Azure File Copy to efficiently deploy SPAs to a Blob Storage (like proposed here https://docs.microsoft.com/en-us/azure/api-management/howto-protect-backend-frontend-azure-ad-b2c#upload-the-js-spa-sample).

It's not pretty but if you use the V4 of the Azure File Copy task in Azure DevOps you can use the --content-type="application/javascript;charset=utf-8" parameter. You just need to have another step to deploy other files or other MIME types.

I ended up creating a specific proxy rule through my Azure Function that I use to proxy my SPA.  Not stoked about it but it works.

It's not pretty but if you use the V4 of the Azure File Copy task in Azure DevOps you can use the --content-type="application/javascript;charset=utf-8" parameter. You just need to have another step to deploy other files or other MIME types.

Okay, thanks, that's the way I did it now as well. I also added a job for correctly uploading *.json files and made a task group of the three different jobs.
Important to note:  using path/*.js to select the JS files does not work correctly if JS files are in a subdirectory. It is better to use the parameters --content-type "application/javascript" --recursive --include-pattern '*.js' instead.

@javafrog Does the pattern path/**/*.js work?

@javafrog Does the pattern path/**/*.js work?
I thought about trying that as well. But it works now and I don't want to fiddle with it anymore ;)

I would like to direct the attention here to the original feature request.
To me it is bleeding obvious that a batch copy jobs should be able to specify their own file extensions to mime type mapping tables, because different batch copy jobs may need different mapping tables.
To me its really strange that this is not identified as a important missing feature and prioritized thereafter.
With AzCopy v8 this was possible using a json file. After that is was removed and has to my knowledge never come back.
I still need to use AzCopy v8 since this feature was requested close to 2 years ago.

This seems like an odd step back, since this has been working for me for quite some time.
For those on Azure Pipelines, I just tried downgrading the Azure File Copy task to v2, and can confirm that the .js mime types are set correctly. I hope this can help some people stuck on this issue.

@mrlund Switching to AzureFileCopy@2 resulted in the following error [ERROR] The TLS version of the connection is not permitted on this storage account.
We are using a service connection azureSubscription: SubscriptionName(guid) if that information helps.
Version 4 seems to identify *.css and *.html files just fine and applies the correct mime types but js files are just text/plain.

@mrlund Switching to AzureFileCopy@2 resulted in the following error [ERROR] The TLS version of the connection is not permitted on this storage account.
We are using a service connection azureSubscription: SubscriptionName(guid) if that information helps.
Version 4 seems to identify *.css and *.html files just fine and applies the correct mime types but js files are just text/plain.

Apparently azure static websites can set mime-types based on extension
There is a link to an example file that explains everything. Will test in the near future. Can't say I'm inspired to alter my pipeline and run it again at the end of my day. Will circle back with results.
If it works, that makes the AzCopy mime-type issue go away for my use-case.

Apparently azure static websites can set mime-types based on extension
There is a link to an example file that explains everything. Will test in the near future. Can't say I'm inspired to alter my pipeline and run it again at the end of my day. Will circle back with results.
If it works, that makes the AzCopy mime-type issue go away for my use-case.

Apparently azure static websites can serve files with mime-types based on extension
There is a link to an example file that explains everything. Will test in the near future. Can't say I'm inspired to alter my pipeline and run it again at the end of my day. Will circle back with results.
If it works, that makes the AzCopy mime-type issue go away for my use-case.

Good find! Actually, the "experience" for hosting SPA content on Azure has been suboptimal so far, so that new service really hits a need. As long as it is still preview, however, we won't be using it.
Regardless of any of that, this issue here still persists and should be fixed, for other use cases as well.

Apparently azure static websites can serve files with mime-types based on extension
There is a link to an example file that explains everything. Will test in the near future. Can't say I'm inspired to alter my pipeline and run it again at the end of my day. Will circle back with results.
If it works, that makes the AzCopy mime-type issue go away for my use-case.

Good find! Actually, the "experience" for hosting SPA content on Azure has been suboptimal so far, so that new service really hits a need. As long as it is still preview, however, we won't be using it.
Regardless of any of that, this issue here still persists and should be fixed, for other use cases as well.

Okay, thanks, that's the way I did it now as well. I also added a job for correctly uploading *.json files and made a task group of the three different jobs.
Important to note: using path/*.js to select the JS files does not work correctly if JS files are in a subdirectory. It is better to use the parameters --content-type "application/javascript" --recursive --include-pattern '*.js' instead.

Im a bit late to the conversation but I also went with this strategy for uploading my SPA to get the service workers well, working. I think with spas becoming more popular, giving us the opportunity to declare our own mime types is a worthwhile feature

This feature was added in v10.11.

@zezha-msft Is there any documentation of how to use the new AZCOPY_CONTENT_TYPE_MAP environment variable? Is the value supposed to be the complete map, or just the path to a mapping file?

@josundt sorry my bad, here is the wiki link: https://github.com/Azure/azure-storage-azcopy/wiki/Custom-mime-mapping


On Windows 8 the light box overlaps the status bar - clearer if there was no overlap

On Windows at the default opening size the light box overlaps (by about 30%) the status bar.
It is useful to be able to see the synchronising info when the light box is present so I think it would be worth either increasing the default height slightly.

Agreed. Should be a trivial fix that could go into 0.1.3. Can you supply a screen shot for context?

Screenshot at default (smallest) size:

(908px x 552px)
And then longer to keep the status bar unoccluded:

(912px x 603px)
So needs an extra 50px of height

Minimum / default size is now set to be 900px x 600px.
Screenshot:

Awaiting review and closing

And this is what the minimum size looks like on Windows 8:

Looks good to me.
Closing.


Add deviceId as an message input parameter

Hi,
first of all, thanks for your great job.
I'd like to specify the deviceId inside the message sent to the node and not always inside the node itself, like msg.deviceId= .....
It should is a simple fix, in ewelink-connect.js, just replaced the row:
    evaluatedParams.unshift(deviceId);
    // First parameter should be always the device ID

with following rows:
    let device_id = '';
    if (deviceId) {
      device_id = deviceId;
    } else if (typeof msg.deviceId !== "undefined") {
      device_id = msg.deviceId || '';
    }
    // First parameter should be always the device ID
    evaluatedParams.unshift(device_id);

In this way it the deviceId is not specified inside the node configuration, it will use the one specified inside the message.
Thanks
Claudio

Pull Request Test Coverage Report for Build 87

4 of 6 (66.67%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-5.2%) to 92.308%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/utils/ewelink-connect.js
4
6
66.67%

Totals

Change from base Build 84:
-5.2%

Covered Lines:
102

Relevant Lines:
106

💛  - Coveralls

Pull Request Test Coverage Report for Build 87

4 of 6 (66.67%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-5.2%) to 92.308%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/utils/ewelink-connect.js
4
6
66.67%

Totals

Change from base Build 84:
-5.2%

Covered Lines:
102

Relevant Lines:
106

💛  - Coveralls

Don't use additional variables, use the same deviceId (you can modify declaration to let instead of const or and you can do all of this inline at line 48).

Done. Is it ok now?


Fixes inconsistent initialization in compileBrTable test

Also fixes some residual use of the word "vm"

okie dokie. I'll leave it at an inconsistency fix and untag the other issue. I was hoping it would have fixed the crash which has been more consistently failing recently.


Slider is displayed but thumb is not moving

Hello,
I want to use a range slider for iOS but I did not get how the thumb is working as it is displayed on the screen based on the value set but the thumb is not moving when I slide it.
Please provide the help so that I can get a callback when the slider move and the value updated
Thanks

Please provide the way to move thumbs based on touch


Interest Calculator

4/13/21 IBM class final

Not sure if I needed to create an actual calculator?


MN-589: utility app

- What I did
A CLI app for creating an additional deposit in the member's wallet
- How I did it
- How to verify it
- Description for the changelog

Codecov Report

Merging #63 into master will decrease coverage by 0.35%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master      #63      +/-   ##
==========================================
- Coverage   15.30%   14.95%   -0.36%
==========================================
  Files           7        7
  Lines        1006     1070      +64
==========================================
+ Hits          154      160       +6
- Misses        844      897      +53
- Partials        8       13       +5

Impacted Files
Coverage Δ

...cation/builtin/contract/deposit/deposit.wrapper.go
0.00% <0.00%> (ø)

...et/application/builtin/contract/deposit/deposit.go
2.51% <0.00%> (+2.51%)
:arrow_up:

Continue to review full report at Codecov.

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


Recognise RxObservable.of and others

Hi, Sam
I found that although Listr claims that it supports RxObservable but in fact, it only recognises the plainest RxObervable comes from new Observable(). Neither RxObservable.of() nor RxObservable.combineLatest() and more will be recognised.
I add the instance check base on this SO answer. https://stackoverflow.com/a/41452513/1016407
Please let me know if there is any problem and please kindly accept the PR.

It seems that the issue, which is linked in the code (https://github.com/sindresorhus/is-observable/issues/1), was resolved. I'm looking into it wheter it's really the case, but if it is, we could use is-observable again. I tested Observable.of with is-observable and it works fine.
So hold on a brief moment until I'm sure that the issue was resolved. Thanks for the PR!

Thank you. Please let me know if there is anything I can help.

If it's possible to test your code in the meantime by using is-observable, that would be great :).

I just added is-observable to detect Observables. I'm releasing the next version today.


import error

Python 2.7.6 on ubuntu-trusty-64 3.13.0-87-generic
>>> import atws
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/usr/local/lib/python2.7/dist-packages/atws/__init__.py", line 4, in <module>
    from .wrapper import connect
  File "/usr/local/lib/python2.7/dist-packages/atws/wrapper.py", line 32, in <module>
    from . import connection
  File "/usr/local/lib/python2.7/dist-packages/atws/connection.py", line 19, in <module>
    from requests.exceptions import ConnectTimeout, Timeout, ReadTimeout, SSLError
ImportError: cannot import name ConnectTimeout

I had an old version of requests, 2.2.1. I updated requests to 2.18.4 and it fixed that issue, but now when I import atws I get:
No handlers could be found for logger "atws.connection"

It's just because atws makes logging calls, but you have not configured logging.
https://docs.python.org/2/howto/logging.html#configuring-logging-for-a-library
from the Requests code itself, we can find a suitable solution.  I'd accept a pull request to get rid of this message:
# Set default logging handler to avoid "No handler found" warnings.
import logging
try:  # Python 2.7+
    from logging import NullHandler
except ImportError:
    class NullHandler(logging.Handler):
        def emit(self, record):
            pass

logging.getLogger(__name__).addHandler(NullHandler())


Do not require component usage to type check in react-redux

Fixes issue where non-used components does not type check, see https://github.com/flowtype/flow-typed/issues/1946

@AndrewSouthpaw @maxsalven Do you mind to try this out? Note that this does not fix the potential problem with options, but only the issue where the component is not used.

I'm curious, how does this address the issue? It looks like you just inlined a few things, did that change the behavior of flow somehow?

@AndrewSouthpaw That's a very good question! This is one of those complicated parts of Flow, and I don't have very solid answer for it. I think it shouldn't matter, but in practise it does. It apparently "stores" the type information to type parameter (CP in this case) and "executes" the type check at that time, when previously it "flowed" to outbound component definition when the type is only applied when the type/component is used. So the type check is actually kind of made where it's needed, and when assigning type to type parameter it is considered to be the place to do the check when in case of component it is the usage of component. I haven't experimented, but changing the place of type definition might in some cases have an effect on the place where the error is shown. Sorry if my explanation is not understandable, it's difficult concept which I haven't grasped well enough to explain it properly :D

Nah, that makes plenty sense. :)
Looks like the build is failing, is that a regression or a fluke?

Build failure seems to be blast from the past: It's again the previous libdefs. Which is weird as this PR does not have anymore anything to do with them. Apparently the code picking the libs based on changes is a bit loose.

ping @AndrewSouthpaw

I've tested this on a relatively complex component/container combination in our code base and it worked just fine :+1:
Would be great to see this go in.

Seems fine to me then, we just need to get the merge conflicts resolved. Sorry for the slow movement.

@villesau If you fix the conflicts I'll merge this in!

@GAntoine @AndrewSouthpaw conflicts resolved. Also did some minor cleanup.


renovatebot(deps): update dependency com.nexmo:client to v5.1.0

This PR contains the following updates:

Package
Update
Change

com.nexmo:client (source)
minor
5.0.0 -> 5.1.0

Release Notes

Nexmo/nexmo-java
v5.1.0
Compare Source
Added

Added support for setting an application id as a messages callback value in updating a number.
Added support for specifying a number type when searching for a number to purchase.

Renovate configuration
:date: Schedule: At any time (no schedule defined).
:vertical_traffic_light: Automerge: Disabled by config. Please merge this manually once you are satisfied.
:recycle: Rebasing: Whenever PR is stale, or if you modify the PR title to begin with "rebase!".
:no_bell: Ignore: Close this PR and you won't be reminded about this update again.

[ ] If you want to rebase/retry this PR, check this box

Available now for Enterprise: Renovate Pro with real-time webhook handling and priority job queue.

PR has been edited
:construction_worker: This PR has received other commits, so Renovate will stop updating it to avoid conflicts or other problems. If you wish to abandon your changes and have Renovate start over you may click the "rebase" checkbox in the PR body/description.


NFC API for Smartphone

NFC extension created by Martin Wooley and John Murray from the BlackBerry developer relations team. Initial code reviewed by Jeff Heifetz, Sergey Gold and Adam Stanley.
Feedback received during review process indicates that v1 is sound (no concerns about memory leaks) however recommend refactoring is performed in v2 to provide support for future Ndef tags.

Need to add the Apache header to all the files.


Remove p-event dependency

@danez: could have a quick look please? Or let me know if there's somebody else from netlify I could ping for a quick review. Thanks!

I'm not at netlify anymore and not sure who could take a look at this.

Thanks for the reply, I didn't know that :)
@eduardoboucas: maybe you could have a quick look? You are using my fork in https://github.com/netlify/gh-release-fetch and I could use a pair of eyes. So far I've landed https://github.com/XhmikosR/download/compare/v13.0.1...master and I wanted to land this PR too for the next major version.
Thanks in advance!


Avoid using the working directory

Fixes https://github.com/sbt/setup-sbt/issues/4

Thanks 👍
Are going to update the tags ?

I'll do that now


[DataFactory]Added support for more frequency types for TumblingWindowTrigger

Swagger PR: https://github.com/Azure/azure-rest-api-specs/pull/11254

@m-nash  Could you please review this PR?

@m-nash  Could you please review this PR?

/check-enforcer evaluate


added password meter

Description
I have added a password meter to the password field.

Related Issues
#340

Screenshots / videos (if applicable)
Note: I have made it visible only for example screenshot case. I  made the display none later.
Initially,

Later

@PavanTeja2005 resolve the issue


Add flex 0 to container styles

This can totally be done in a stylesheet, but how do you feel about adding flex: 0 into the containerContainer inline styles? https://github.com/wnr/element-resize-detector/blob/master/dist/element-resize-detector.js#L793

That seems reasonable :) I would accept such PR.

Awesome :) I'll get it together as soon as I can.

Sounds great!

This is now fixed in version 1.1.10

Thanks!


dependency incompatibility with rails 3.2.1 and sass-rails 3.2.3

Bundler could not find compatible versions for gem "railties":
  In Gemfile:
    rails (= 3.2.1) ruby depends on
      railties (= 3.2.1) ruby

    sass-rails (~> 3.2.3) ruby depends on
      railties (3.2.0)

Scratch that. This is an issue with minitest-rails.


Flaky test reports

[x] I've read and followed the Contribution Guidelines
[ ] step.yml and README.md is updated with the changes (if needed)

Version
Requires a MINOR version update
Context
It is fixing an issue where flaky tests weren't reported properly in the Test Reports Add On or in Bitrise Checks. Reporting flaky tests as failures on Bitrise Checks would block merging.

Changes
This change is mapping each group of flaky tests into one successful test with duration that is the sum of all flaky runs.
Investigation details
The alternative solution is to extend the junit format to account for flakiness and update all the systems that will use the new format.
Decisions
The status of a flaky test group is "success". The duration of the test is reported as the sum of all runs of the flaky test.

I am closing this PR because in the meantime we have introduced a solution to this:
https://github.com/bitrise-steplib/steps-deploy-to-bitrise-io/pull/190


Typed createSelector function-solution-1

Hey folks.
So I was digging through the codebase and I thought I could provide some help, the first thing I thought I could do is providing an option to create a pre-typed createSelector function that is typed for the RootState much like TypedUseSelectorHook. I came up with a couple of potential solutions. This is the first solution:
export interface CreateSelectorFunction<
  State,
  MemoizeFunction extends (
    func: (...args: unknown[]) => unknown,
    ...options: any[]
  ) => (...args: unknown[]) => unknown,
  MemoizeOptions extends unknown[] = DropFirst<Parameters<MemoizeFunction>>,
  Keys = Expand<
    Pick<ReturnType<MemoizeFunction>, keyof ReturnType<MemoizeFunction>>
  >
> {
  /** Input selectors as separate inline arguments */
  <Selectors extends readonly Selector<State>[], Result>(
    ...items: [
      ...Selectors,
      (...args: SelectorResultArray<Selectors>) => Result
    ]
  ): OutputSelector<
    Selectors,
    Result,
    (...args: SelectorResultArray<Selectors>) => Result,
    GetParamsFromSelectors<Selectors>,
    Keys
  > &
    Keys

  /** Input selectors as separate inline arguments with memoizeOptions passed */
  <Selectors extends readonly Selector<State>[], Result>(
    ...items: [
      ...Selectors,
      (...args: SelectorResultArray<Selectors>) => Result,
      CreateSelectorOptions<MemoizeOptions>
    ]
  ): OutputSelector<
    Selectors,
    Result,
    (...args: SelectorResultArray<Selectors>) => Result,
    GetParamsFromSelectors<Selectors>,
    Keys
  > &
    Keys

  /** Input selectors as a separate array */
  <Selectors extends readonly Selector<State>[], Result>(
    selectors: [...Selectors],
    combiner: (...args: SelectorResultArray<Selectors>) => Result,
    options?: CreateSelectorOptions<MemoizeOptions>
  ): OutputSelector<
    Selectors,
    Result,
    (...args: SelectorResultArray<Selectors>) => Result,
    GetParamsFromSelectors<Selectors>,
    Keys
  > &
    Keys
}

We can also rename it to TypedCreateSelectorFunction.
export function createSelectorCreator<
  /** A memoizer such as defaultMemoize that accepts a function + some possible options */
  MemoizeFunction extends (
    func: (...args: unknown[]) => unknown,
    ...options: any[]
  ) => (...args: unknown[]) => unknown,
  /** The additional options arguments to the memoizer */
  MemoizeOptions extends unknown[] = DropFirst<Parameters<MemoizeFunction>>
>(
  memoize: MemoizeFunction,
  ...memoizeOptionsFromArgs: DropFirst<Parameters<MemoizeFunction>>
) {
  const createSelector = (...funcs: Function[]) => {
    let recomputations = 0
    let lastResult: unknown

    // Due to the intricacies of rest params, we can't do an optional arg after `...funcs`.
    // So, start by declaring the default value here.
    // (And yes, the words 'memoize' and 'options' appear too many times in this next sequence.)
    let directlyPassedOptions: CreateSelectorOptions<MemoizeOptions> = {}

    // Normally, the result func or "output selector" is the last arg
    let resultFunc = funcs.pop()

    // If the result func is actually an _object_, assume it's our options object
    if (typeof resultFunc === 'object') {
      directlyPassedOptions = resultFunc as any
      // and pop the real result func off
      resultFunc = funcs.pop()
    }

    if (typeof resultFunc !== 'function') {
      throw new Error(
        `createSelector expects an output function after the inputs, but received: [${typeof resultFunc}]`
      )
    }

    // Determine which set of options we're using. Prefer options passed directly,
    // but fall back to options given to createSelectorCreator.
    const { memoizeOptions = memoizeOptionsFromArgs, inputStabilityCheck } =
      directlyPassedOptions

    // Simplifying assumption: it's unlikely that the first options arg of the provided memoizer
    // is an array. In most libs I've looked at, it's an equality function or options object.
    // Based on that, if `memoizeOptions` _is_ an array, we assume it's a full
    // user-provided array of options. Otherwise, it must be just the _first_ arg, and so
    // we wrap it in an array so we can apply it.
    const finalMemoizeOptions = Array.isArray(memoizeOptions)
      ? memoizeOptions
      : ([memoizeOptions] as MemoizeOptions)

    const dependencies = getDependencies(funcs)

    const memoizedResultFunc = memoize(
      function recomputationWrapper() {
        recomputations++
        // apply arguments instead of spreading for performance.
        return resultFunc!.apply(null, arguments)
      } as Parameters<MemoizeFunction>[0],
      ...finalMemoizeOptions
    )

    // @ts-ignore
    const makeAnObject: (...args: unknown[]) => object = memoize(
      // @ts-ignore
      () => ({}),
      ...finalMemoizeOptions
    )

    let firstRun = true

    // If a selector is called with the exact same arguments we don't need to traverse our dependencies again.
    // TODO This was changed to `memoize` in 4.0.0 ( #297 ), but I changed it back.

    // But, there's multiple issues:
    // - We don't pass in `memoizeOptions`
    // Arguments change all the time, but input values change less often.
    // Most of the time shallow equality _is_ what we really want here.
    // TODO Rethink this change, or find a way to expose more options?
    const selector = defaultMemoize(function dependenciesChecker() {
      const params = []
      const { length } = dependencies

      for (let i = 0; i < length; i++) {
        // apply arguments instead of spreading and mutate a local list of params for performance.
        // @ts-ignore
        params.push(dependencies[i].apply(null, arguments))
      }

      const finalStabilityCheck = inputStabilityCheck ?? globalStabilityCheck

      if (
        process.env.NODE_ENV !== 'production' &&
        (finalStabilityCheck === 'always' ||
          (finalStabilityCheck === 'once' && firstRun))
      ) {
        const paramsCopy = []

        for (let i = 0; i < length; i++) {
          // make a second copy of the params, to check if we got the same results
          // @ts-ignore
          paramsCopy.push(dependencies[i].apply(null, arguments))
        }

        // if the memoize method thinks the parameters are equal, these *should* be the same reference
        const equal =
          makeAnObject.apply(null, params) ===
          makeAnObject.apply(null, paramsCopy)
        if (!equal) {
          // do we want to log more information about the selector?
          console.warn(
            'An input selector returned a different result when passed same arguments.' +
              '\nThis means your output selector will likely run more frequently than intended.' +
              '\nAvoid returning a new reference inside your input selector, e.g.' +
              '\n`createSelector([(arg1, arg2) => ({ arg1, arg2 })],(arg1, arg2) => {})`',
            {
              arguments,
              firstInputs: params,
              secondInputs: paramsCopy
            }
          )
        }

        if (firstRun) firstRun = false
      }

      // apply arguments instead of spreading for performance.
      lastResult = memoizedResultFunc.apply(null, params)

      return lastResult
    } as Parameters<MemoizeFunction>[0])

    Object.assign(selector, {
      resultFunc,
      memoizedResultFunc,
      dependencies,
      lastResult: () => lastResult,
      recomputations: () => recomputations,
      resetRecomputations: () => (recomputations = 0)
    })

    return selector
  }
  // @ts-ignore
  return createSelector as CreateSelectorFunction<
    any,
    MemoizeFunction,
    MemoizeOptions
  >
}

I will make a separate PR for the second solution.

@aryaemami59 : now that #626 is merged, do we still need either of these PRs?

@aryaemami59 : now that #626 is merged, do we still need either of these PRs?

I will go ahead and close this one, and #625 since they are no longer applicable, will probably come up with a new solution on a type level or include a copy paste pattern in the docs.


Release 0.1.40

Update base image in Dockerfile
Update base image in Dockerfile
Update base image in Dockerfile
Update base image in Dockerfile
Update  Dockerfile
Update  Dockerfile
Update Dockerfile
Update Dockerfile
Rebuild
Update Dockerfile

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

#179  👈
#174
main

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

Join @maximearmstrong and the rest of your teammates on  Graphite


Azeru

[Enter feedback here]

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

ID: 224b73dd-c59d-a833-0d4d-a778af6086a9
Version Independent ID: c6b167c9-9589-45d0-ee6a-7a47a4aec41e
Content: Tutorial: Map existing custom DNS name - Azure App Service
Content Source: articles/app-service/app-service-web-tutorial-custom-domain.md
Service: app-service
GitHub Login: @cephalin
Microsoft Alias: cephalin

@TheLostkingg Just wanted to check and see if you had a question or if this was created by mistake. We look forward to your reply.

@TheLostkingg Since we have not heard back from you we will now proceed to close this thread. If there are further questions regarding this matter, please tag me in your reply. We will gladly reopen the issue and continue the discussion.


Complex object with KnownType attribute C#

I have my response structure with inherited concept
    [Schema]
    [KnownType(typeof(CustomResponse))]
    public class ICustomResponse
    {
        [Id(0)]
        public int Id { get; set; }
    }

    [Schema]
    public class CustomResponse : ICustomResponse
    {
        [Id(1)]
        public int Payload { get; set; }

        [Id(2)]
        public int Output { get; set; }
    }

My Return type of the method is ICostomResponse
public static Method<CustomRequest, ICustomResponse> Method =
new Method<CustomRequest, ICustomResponse>(.....

Implementation would be
private static ICustomResponse AddTwoNumbers(CustomRequest additionRequest)
{
return new CustomResponse { Output = additionRequest.X + additionRequest.Y };
}

I wants to retrieve actual data returned from the server. But while I receive the response at client end it contains only the ICustomResponse object. Can't convert ICustomRespnse to CustomResponse object.
Is there any way to include [KnownType(typeof(CustomResponse))] while serializing and deserializing the object?
I refereed sample project and made few changes- https://github.com/Horusiath/GrpcSample

Looks like you are using Microsoft Bond as the serialization layer (while the default and recommended serialization format is Google Protobuf) and it also seems that this question is about Bond serialization specifics, not really about gRPC itself, so this is not the right repository to ask the question.  (Perhaps you can ask here https://github.com/Microsoft/bond ?)


Make the whole library to be platform-agnostic

This library currently can only work on Node.js environment. Replacing the lower-level HTTP library is a good start for this. Some considerations are:

axios (may be bad for streaming API)
cross-fetch

:wave: It sounds like you and I are on the same page! Do you want to help out with:
https://github.com/Preposterous/twitter-lite
Some features:

client & server support
591 bytes of source code
very flexible
promise driven async / await support
will be maintained for the forseeable future

I would really appreciate to hear what you think!
Thanks 😄

Hello there,
Looks like a nice tiny library. I'll give it a try sometime and see if I can help something out!

@Frizz925: We've done that with twitter-lite, so feel free to join efforts!


Neon.Kube.Resources: remove dependencies on ASP.NET and .NET 6

While this issue is pending over at the dotnet-operator-sdk repo, let's investigate a workaround:
https://github.com/buehler/dotnet-operator-sdk/issues/362
The problem is essentially this:

We use the dotnet-operator-sdk to generate the CRDs for our custom resources from .NET classes defined in the Neon.Kube.Resources library,

dotnet-operator-sdk is currently a single library that depends on ASP.NET so we're pulling all of that into Neon.Kube.Resources just to decorate our custom classes with the necessary metadata attributes.

This is a problem because ASP.NET ends up also being pulled into Neon.Kube.Setup which is also referenced by neon-cli and neon-desktop, bloating these apps significantly (for no useful reason).

dotnet-operator-sdk targets .NET 6 only but we'd prefer our custom resources to target .NET Standard 2.1.

The workaround involves creating a separate library Neon.Kube.ResourceDefinitions where the custom resources are defined and the CRDs are generated and then have the Neon.Kube.Resources reference and build the resource classes from the definitions project.
We'd use conditional compilation to ignore the dotnet-operator-sdk metadata attributes in the dotnet-operator-sdk project.
This solution is actually pretty good and I think we could live with this over the long term.  It effectively decouples our custom resource library from the dotnet-operator-sdk and .NET 6.

DONE


sqlalchemy.exc.CompileError: MSSQL requires an order_by when using an OFFSET or a non-simple LIMIT clause

When using MS SQL as the database, running the fake.posts(100) command from the shell results in the following error message:
sqlalchemy.exc.CompileError: MSSQL requires an order_by when using an OFFSET or a non-simple LIMIT clause
I am drawing a blank on how to solve the issue. Any advice from anyone watching this?

I have fixed this by ammending the code to be as follows, though I havent tested this with any other backend:

def posts(count=100):
    fake = Faker()
    user_count = User.query.order_by(User.id).count()

    for i in range(count):
        u = User.query.order_by(User.id).offset(randint(0, user_count - 1)).first()
        p = Post(body=fake.text(),
                 timestamp=fake.past_date(),
                 author=u)
        db.session.add(p)
    db.session.commit()

I have tested this using an sqlite backend and the code works.


Change prefix to use "=" instead of "/"

Closes #118
Note: Not sure if Index will have its own "i=" prefix? It doesnt have for edit and delete command. Updated the UG accordingly
DG diagrams for the original addressbook will also have to change

Not sure if we care about javadocs comments, but there are some comments in seedu.address.logic.parser.ArgumentTokenizer seedu.address.logic.parser.Prefix that reference the / also.

Not sure if we care about javadocs comments, but there are some comments in seedu.address.logic.parser.ArgumentTokenizer seedu.address.logic.parser.Prefix that reference the / also.

Okay, edited to make it clearer. Thanks for the catch.

I think some text in the DG still use the "/" command.

I think some text in the DG still use the "/" command.

Fixed, sorry for missing it.


Washer not added to smartthings

I have one washer in my lg connector but the washer is not showing up in the smartthings.
I Have the app and handlers installed (picures bellow).
Any ideas?

fix it. Maybe update the docs about the ST Settings
Need to be updated with the info in smarttapp -> config

fix it. Maybe update the docs about the ST Settings
Need to be updated with the info in smarttapp -> config

Hi João... could you give me a quick help? I have the same problem with my washer.
I can't find the problem. Valeu!!

you need to add the token in the connector page. you also need to add the config on the connector.
But to be honest this never worked very well. I moved to homeassistant.

you need to add the token in the connector page. you also need to add the config on the connector.
But to be honest this never worked very well. I moved to homeassistant.

when you say add the token in the connector page and the config on the connector, how did you do that?
I went to the 'External Management' tab and it opens the connector - but i'm not sure what i do from there -- my devices show up on the dashboard, but not on the devices - and when i try to register them the only option is for a TV.
The 'Internal Management' tab doesn't open anything....did it for you?
Did your washer get added as a device then?

Agreed, bite the bullet and move to home assistant.  I did it a year ago
with tons of customizations in smart things and it was the right decision
for me.
Craig
On Mon, Jun 28, 2021, 12:32 AM Alberto V M @.***> wrote:

fix it. Maybe update the docs about the ST Settings
Need to be updated with the info in smarttapp -> config
Hi João... could you give me a quick help? I have the same problem with my
washer.
I can't find the problem. Valeu!!
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/fison67/LG-Connector/issues/16#issuecomment-869345870,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AB6LN7NEEZGC77KA43ZQZKLTU73MDANCNFSM43BRTCXQ
.


MNT: Re-rendered with conda-build 3.26.1, conda-smithy 3.26.1, and conda-forge-pinning 2<IP_ADDRESS>.15.04

…to make sure the current build still works with the current infra and nothing has broken in last four years.

Checklist

[x] Used a personal fork of the feedstock to propose changes
[x] Bumped the build number (if the version is unchanged)
[x] Reset the build number to 0 (if the version changed)
[x] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[x] Ensured the license file is being packaged.

@fhoehle Ready for review.


Fix TensorboardBinaryClassificationExporter that gives a zero-division like error

The code I am using:
import torch.nn as nn
import h5py
import torch
from deeprankcore.DataSet import HDF5DataSet
from deeprankcore.NeuralNet import NeuralNet
import numpy as np
from deeprankcore.ginet import GINet
from deeprankcore.models.metrics import OutputExporter, TensorboardBinaryClassificationExporter

# Number of cores the program is using
torch.set_num_threads(20)

database = '/Users/giuliacrocioni/remote_snellius/data/pMHCI/gnn_hdf5/rd.hdf5'
node_features = ["bsa", "depth", "hb_acceptors", "hb_donors", "hse", "ic", "pssm", "polarity", "sasa", "size", "type"]
edge_features = ["coulomb", "covalent", "dist", "vanderwaals"]

metrics_output_directory = "reports/"
metrics_exporters = [OutputExporter(metrics_output_directory),                TensorboardBinaryClassificationExporter(metrics_output_directory)]

if __name__ == '__main__':

    # creating HDF5DataSet object and selecting molecules, node/edge features, target, and clustering method:
    dataset = HDF5DataSet(
        root = "./",
        database = database,
        node_feature = node_features,
        edge_feature = edge_features,
        target = "binary",
        clustering_method = "mcl"
    )

    device = torch.device('cuda' if torch.cuda.is_available() else 'cpu')

    nn = NeuralNet(
        dataset,
        GINet,
        task = "class",
        batch_size = 16,
        percent = [0.8, 0.2],
        metrics_exporters = metrics_exporters
    )

    nn.train(nepoch = 10, validate = True)

Data can be found on Snellius at /projects/0/einf2380/data/pMHCI/gnn_hdf5/rd.hdf5.
Error I get:

Traceback (most recent call last):
  File "/Users/giuliacrocioni/Desktop/eScience/projects_2022/3D-vac/3D-Vac/src/models/gnn_training.py", line 100, in <module>
    nn.train(nepoch = 10, validate = True)
  File "/Users/giuliacrocioni/Desktop/eScience/projects_2022/3D-vac/deeprank-core/deeprankcore/NeuralNet.py", line 301, in train
    self.eval(self.train_loader, 0, "training")
  File "/Users/giuliacrocioni/Desktop/eScience/projects_2022/3D-vac/deeprank-core/deeprankcore/NeuralNet.py", line 441, in eval
    self._metrics_exporters.process(
  File "/Users/giuliacrocioni/Desktop/eScience/projects_2022/3D-vac/deeprank-core/deeprankcore/models/metrics.py", line 54, in process
    metrics_exporter.process(pass_name, epoch_number, entry_names, output_values, target_values)
  File "/Users/giuliacrocioni/Desktop/eScience/projects_2022/3D-vac/deeprank-core/deeprankcore/models/metrics.py", line 122, in process
    roc_auc = roc_auc_score(target_values, probabilities)
  File "/Users/giuliacrocioni/miniforge3/envs/3d_vac/lib/python3.10/site-packages/sklearn/metrics/_ranking.py", line 546, in roc_auc_score
    y_score = check_array(y_score, ensure_2d=False)
  File "/Users/giuliacrocioni/miniforge3/envs/3d_vac/lib/python3.10/site-packages/sklearn/utils/validation.py", line 800, in check_array
    _assert_all_finite(array, allow_nan=force_all_finite == "allow-nan")
  File "/Users/giuliacrocioni/miniforge3/envs/3d_vac/lib/python3.10/site-packages/sklearn/utils/validation.py", line 114, in _assert_all_finite
    raise ValueError(
ValueError: Input contains NaN, infinity or a value too large for dtype('float64').

Something went wrong it the preprocessing. This change should fix it: https://github.com/DeepRank/deeprank-core/pull/113


add init command

Closes #35

I'd love it if someone could give this a review. Notice that I'm merging into next. I want to implement #33 before this gets released.
As far as I'm concerned though this LGTM :)

[Current coverage][cc-pull] is 100% (diff: 100%)

Merging [#36][cc-pull] into [next][cc-base-branch] will not change coverage

@@           next   #36   diff @@
=================================
  Files         7    10     +3
  Lines       137   177    +40
  Methods       0     0
  Messages      0     0
  Branches      0     0
=================================
+ Hits        137   177    +40
  Misses        0     0
  Partials      0     0

Powered by Codecov. Last update [ac0c1f0...cea3474][cc-compare]
[cc-base-branch]: https://codecov.io/gh/kentcdodds/p-s/branch/next?src=pr
[cc-compare]: https://codecov.io/gh/kentcdodds/p-s/compare/ac0c1f0c9b20042eb729fa1ad8906db8eef2855f...cea347475f2f6bd9433c4bd4e8c6706e9765703a?src=pr
[cc-pull]: https://codecov.io/gh/kentcdodds/p-s/pull/36?src=pr


OSS-Fuzz: Floating-point-exception in spvtools::opt::InstructionFolder::BinaryOperate

https://bugs.chromium.org/p/oss-fuzz/issues/detail?id=49570
@s-perron for info.

The oss fuzz issue was closed as verified. Not sure what fixed it, but I will go with it.


Add Control Right to Mac keycodes

According to https://eastmanreference.com/complete-list-of-applescript-key-codes and https://github.com/servo/core-foundation-rs/blob/8c2444066aac42c20443b696270f5b1afff18993/core-graphics/src/event.rs#LL62C45-L62C45 the Control Right key actually does exist, although many keyboards don't have it. I was experiencing a crash in kanata (a Rust keyboard mapper a la kmonad) related to this, so it would be nice to have it fixed :)

thanks for this. TIL ! :D


How do I prevent the app mounting twice?

I'm submitting a...

[x] Performance issue

Browser:

[ ] Chrome version XX
[x] Firefox version Developer Edition
[ ] IE version XX
[ ] Edge version XX
[ ] Safari version XX

Hi, I'm using  msal.js to write an AAD2.0 app in Javascript (React16, ES6) and after performing a loginPopup the entire app reloads (see attached images). What can I do to prevent this if the browser doesnt need to redirect anywhere for the login? (the entire login process is successful) As you can see in the images the app class is initiated twice.

@didair Please don't delete the questions in template. You have removed the version information. Can you please tell me what version of msal.js you are using?

@didair  you shouldn't see a reload for loginpopup. It can happen for loginRedirect. Is it possible for you to share your code snippet here or repo link(if possible). Also is this issue only reproducible in firefox and works fine in chrome?

@nehaagrawal Sorry. Im running the latest version of msal.js. I also forgot to link the repo i've set up. https://github.com/didair/aadv2-msal-example. Seems to be the same issue in Chrome as well.


fix(deps): update images to latest

This PR contains the following updates:

Package
Type
Update
Change

icr.io/ibm-iac-charts/sysdig-deploy
helm_release
minor
1.49.8 -> 1.51.0

icr.io/ibm-iac/agent-kmodule

minor
13.0.4 -> 13.1.0

icr.io/ibm-iac/agent-slim

minor
13.0.4 -> 13.1.0

Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
👻 Immortal: This PR will be recreated if closed unmerged. Get config help if that's undesired.

[ ] If you want to rebase/retry this PR, check this box

This PR has been generated by Renovate Bot.

/run pipeline

/run pipeline

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


F12/Go to Definition not working on upgrade from 1.26 to 2.6.24

Type: Bug
Issue Description
Go to definition via right click or F12 does not work when upgrading v 1.26 to 2.6.24
Video clip: https://www.dropbox.com/scl/fi/b3h4f7whrwin7t6y2be2x/Screen-Recording-2023-10-18-at-11.04.52-AM.mov?rlkey=7ie4mp8tqdud26eghdem9riou&dl=0
Steps to Reproduce

With version 1.26 installed, open a folder containing many projects. Note: there is no solution file.
Open a class file that references another class/interface located in another directory and/or project
Click on the referenced class/interface and hit F12 or right click/Go to Definition to navigate to the file.

Expected Behavior
You should be taken to the definition/file
Actual Behavior

Now upgrade to v2.6.24, reload vs code, and repeat the repro steps

You will not be taken to the file and a message is displayed:
"No definition found for XXX".
Logs

C# log
Post the output from Output-->C# here
C# LSP Trace Logs
Post the output from Output-->C# LSP Trace Logs here.    This might be a known issue

Here is the output as-is:
Using dotnet configured on PATH Dotnet path: /usr/local/share/dotnet/dotnet Activating C# standalone... waiting for named pipe information from server... [stdout] {"pipeName":"/var/folders/92/bxy3961s0c39s5d2r5mfs8cw0000gp/T/eae4447e-c631-456c-8e5a-16b847e9c823.sock"} received named pipe information from server attempting to connect client to server... client has connected to server [Info  - 1:02:36 PM] [Program] Language server initialized
Where would i set dotnet.server.trace?

@ahanusa in vscode settings -

@jasonmalinowski looks like everything is in misc and theres no project loading lines.  Maybe something going wrong on the client side with finding all projects to load?

@dibarbet I updated my last message.  I figured out how to trace and re-ran before your reply.  Sorry for the confusion.

@ahanusa What types of projects do you have in this workspace? Are the .csproj files actually present in the workspace?

@ahanusa Also, if you go through your options prefixed with either 'dotnet' or 'csharp', what is set?

@ahanusa What types of projects do you have in this workspace? Are the .csproj files actually present in the workspace?

I've never setup a workspace.  The way I've always worked is by opening a directory containing all of the projects directly

@ahanusa Also, if you go through your options prefixed with either 'dotnet' or 'csharp', what is set?

What "options" are you referring to?

Ah yes, remove the "dotnet.defaultSolution" thing -- if that's set to disable that's going to cause us to disable loading.

Ok, so I received a message when reopening that I thought was in-error, but clicking no re-added that option to my workspace settings:

Turns out there are a couple solution files in some of the folders.  However, they belong to another team and have no use to me.

@jasonmalinowski any other thoughts?

@ahanusa At this point this is a limitation of the new extension; we don't have a way to say "I don't want to load a solution, just load all the projects". The workaround for now would be to check in a new solution file with whatever projects you do want. You can use dotnet sln to make the solution file, there's some examples of how to pass in globbing if you just wanted to add all .csproj files to your solution in one single go.
Can you share what your Omnisharp settings were? Did you have load on demand enabled? Or if not, where you manually picking projects before?

@ahanusa At this point this is a limitation of the new extension; we don't have a way to say "I don't want to load a solution, just load all the projects". The workaround for now would be to check in a new solution file with whatever projects you do want. You can use dotnet sln to make the solution file, there's some examples of how to pass in globbing if you just wanted to add all .csproj files to your solution in one single go.
Can you share what your Omnisharp settings were? Did you have load on demand enabled? Or if not, where you manually picking projects before?

Ok, that seemed to work.  Thanks for the help and timely correspondence!

@ahanusa Happy to hear you're unblocked! We'll still leave this open since there is an implicit feature request to make this scenario a bit easier.

@jasonmalinowski I just noticed after getting everything resolved that the run and debug links that used to hover above unit test methods are no longer present?

@jasonmalinowski After upgrading to latest C# extension the run and debug links no longer appear above unit test methods?

Closing - feature for loading all projects w/out sln is tracked by https://github.com/dotnet/vscode-csharp/issues/5722 and https://github.com/dotnet/vscode-csharp/issues/6008

After upgrading to latest C# extension the run and debug links no longer appear above unit test methods?

Should have been resolved in latest versions, see https://github.com/dotnet/vscode-csharp/issues/5719.  If you're having issues, please file a new issue.


auth file, using alternative bcmp implementation instead to

check the token. less optimised than the usual memcmp especially
it goes through the whole buffers but more resilient against possible
attacks.
While at it, constifying a var which should have been.

thanks!
avoiding side channels but not timing?

well, this should be time constant flavour.

Yeah; it checks each byte it's still checking the plen instead of the bufferlen right? think you're supposed to zerofill and use constant size so you can't guess the length of the password as well

true. here it is more bcmp than anything else as well. just reasonable approach for a start :-)


README: Update PR dependency section

Short description:
README: Update PR dependency section
More details:
What this PR does / why we need it:
Which issue(s) this PR fixes:
Special notes for reviewer:
Bug:
Release notes:

NONE

/approve


Transaction pool & "dirty" transactions

Hi! I had a look at your project and had a question about the following:
https://github.com/neozo-software/jblockchain/blob/0232c6475e5240b0cd99a9b3699ac831fba22e91/node/src/main/java/de/neozo/jblockchain/node/service/BlockService.java#L48-L62
After appending the block, the transactions are removed from the nodes' transaction pool. Is there anyway that the other nodes are informed that these transactions are processed? And if so, what if another node is currently processing those transactions as well?
Thanks!

Hi. The removal from the transaction pool happens in the method you mentioned. The key is that this method is invoked on each node, because the successful miner will inform all other nodes about it's newly found block.

And if so, what if another node is currently processing those transactions as well?

Yes this might be the case. The current implementation won't handle this case and let the MiningService trying to mine a block with transactions which are already processed - all for nothing. A fix would be to restart the current mining operation so it would pull new transactions from the pool.

Thanks for the quick reply!

The key is that this method is invoked on each node, because the successful miner will inform all other nodes about it's newly found block.

Ofcourse, I completely missed that!

A fix would be to restart the current mining operation so it would pull new transactions from the pool.

That would work! Seems like there are a lot of ways to tackle this issue.
Your implementation is very useful to get to know some of the blockchain principles, thanks!


[Bug][Java]: Bridge dynamic library gar-jni build fatal with Apache Arrow 14.0.0

Is there an existing issue for this?

[X] I have searched the existing issues

Current Behavior
When link arrow as static:
[INFO]      [exec] -- Found re2Alt: /usr/lib/x86_64-linux-gnu/libre2.so
[INFO]      [exec] -- Found BZip2: /usr/lib/x86_64-linux-gnu/libbz2.so (found version "1.0.8")
[INFO]      [exec] -- Looking for BZ2_bzCompressInit
[INFO]      [exec] -- Looking for BZ2_bzCompressInit - found
[INFO]      [exec] -- Found utf8proc: /usr/lib/x86_64-linux-gnu/libutf8proc.so (found version "2.7.0")
[INFO]      [exec] -- Found nlohmann_json: /usr/lib/cmake/nlohmann_json/nlohmann_jsonConfig.cmake (found version "3.10.5")
[INFO]      [exec] -- Found CURL: /usr/lib/x86_64-linux-gnu/libcurl.so (found version "7.81.0")
[INFO]      [exec] -- Arrow version: 14.0.0
[INFO]      [exec] -- Found the Arrow shared library: /usr/lib/x86_64-linux-gnu/libarrow.so.1400.0.0
[INFO]      [exec] -- Found the Arrow import library: ARROW_IMPORT_LIB-NOTFOUND
[INFO]      [exec] -- Found the Arrow static library: /usr/lib/x86_64-linux-gnu/libarrow.a
[INFO]      [exec] -- Configuring done (2.6s)
[INFO]      [exec] -- Generating done (0.1s)
[INFO]      [exec] CMake Error at /usr/lib/x86_64-linux-gnu/cmake/Arrow/ArrowTargets.cmake:74 (set_target_properties):
[INFO]      [exec]   The link interface of target "Arrow::arrow_static" contains:
[INFO]      [exec]
[INFO]      [exec]     protobuf::libprotobuf
[INFO]      [exec]
[INFO]      [exec]   but the target was not found.  Possible reasons include:
[INFO]      [exec]
[INFO]      [exec]     * There is a typo in the target name.
[INFO]      [exec]     * A find_package call is missing for an IMPORTED target.
[INFO]      [exec]     * An ALIAS target is missing.
[INFO]      [exec]
[INFO]      [exec] Call Stack (most recent call first):
[INFO]      [exec]   /usr/lib/x86_64-linux-gnu/cmake/Arrow/ArrowConfig.cmake:135 (include)
[INFO]      [exec]   CMakeLists.txt:23 (find_package)
[INFO]      [exec]
[INFO]      [exec]
[INFO]      [exec] CMake Generate step failed.  Build files cannot be regenerated correctly.
[INFO] ------------------------------------------------------------------------
[INFO] BUILD FAILURE
[INFO] ------------------------------------------------------------------------
[INFO] Total time:  21.156 s
[INFO] Finished at: 2023-11-02T13:55:00Z
[INFO] ------------------------------------------------------------------------
Error:  Failed to execute goal org.apache.maven.plugins:maven-antrun-plugin:3.0.0:run (make-and-link-bitcode) on project gar-java: An Ant BuildException has occured: The following error occurred while executing this line:
Error:  /home/runner/work/GraphAr/GraphAr/java/build.xml:20: exec returned: 1
Error:  around Ant part ...<ant antfile="/home/runner/work/GraphAr/GraphAr/java/build.xml">... @ 5:69 in /home/runner/work/GraphAr/GraphAr/java/target/antrun/build-main.xml
Error:  -> [Help 1]
Error:
Error:  To see the full stack trace of the errors, re-run Maven with the -e switch.
Error:  Re-run Maven using the -X switch to enable full debug logging.
Error:
Error:  For more information about the errors and possible solutions, please read the following articles:
Error:  [Help 1] http://cwiki.apache.org/confluence/display/MAVEN/MojoExecutionException
Error: Process completed with exit code 1.

When link arrow as shared:
[INFO]      [exec] /usr/bin/c++ -fPIC  -std=c++17 -Wall -O3 -DNDEBUG -O3 -g -shared -Wl,-soname,libgar.so -o libgar.so CMakeFiles/gar.dir/src/arrow_chunk_reader.cc.o CMakeFiles/gar.dir/src/arrow_chunk_writer.cc.o CMakeFiles/gar.dir/src/chunk_info_reader.cc.o CMakeFiles/gar.dir/src/data_type.cc.o CMakeFiles/gar.dir/src/edges_builder.cc.o CMakeFiles/gar.dir/src/expression.cc.o CMakeFiles/gar.dir/src/filesystem.cc.o CMakeFiles/gar.dir/src/graph.cc.o CMakeFiles/gar.dir/src/graph_info.cc.o CMakeFiles/gar.dir/src/reader_util.cc.o CMakeFiles/gar.dir/src/util.cc.o CMakeFiles/gar.dir/src/version_parser.cc.o CMakeFiles/gar.dir/src/vertices_builder.cc.o CMakeFiles/gar.dir/src/yaml.cc.o "CMakeFiles/gar.dir/thirdparty/mini-yaml/yaml/Yaml.cpp.o"  -ldl -Wl,--exclude-libs,ALL -Wl,--whole-archive /usr/lib/x86_64-linux-gnu/libarrow.a /usr/lib/x86_64-linux-gnu/libparquet.a /usr/lib/x86_64-linux-gnu/libarrow_dataset.a /usr/lib/x86_64-linux-gnu/libarrow_acero.a -Wl,--no-whole-archive /usr/lib/x86_64-linux-gnu/libssl.so -lcurl /usr/lib/x86_64-linux-gnu/libparquet.a /usr/lib/x86_64-linux-gnu/libthrift.so /usr/lib/x86_64-linux-gnu/libarrow.a /usr/lib/x86_64-linux-gnu/libssl.so /usr/lib/x86_64-linux-gnu/libarrow_bundled_dependencies.a /usr/lib/x86_64-linux-gnu/libcrypto.so /usr/lib/x86_64-linux-gnu/libbrotlienc.so /usr/lib/x86_64-linux-gnu/libbrotlidec.so /usr/lib/x86_64-linux-gnu/libbrotlicommon.so /usr/lib/x86_64-linux-gnu/libbz2.so /usr/lib/x86_64-linux-gnu/liblz4.so /usr/lib/x86_64-linux-gnu/libsnappy.so.1.1.8 /usr/lib/x86_64-linux-gnu/libz.so /usr/lib/x86_64-linux-gnu/libzstd.so /usr/lib/x86_64-linux-gnu/libcurl.so /usr/lib/x86_64-linux-gnu/libutf8proc.so /usr/lib/x86_64-linux-gnu/libre2.so -lprotobuf::libprotobuf -ldl -lrt
[INFO]      [exec] /usr/bin/ld: cannot find -lprotobuf::libprotobuf: No such file or directory
[INFO]      [exec] make[5]: Leaving directory '/home/runner/work/GraphAr/GraphAr/java/target/native/graphar_ep-build'
[INFO]      [exec] make[4]: Leaving directory '/home/runner/work/GraphAr/GraphAr/java/target/native/graphar_ep-build'
[INFO]      [exec] make[3]: Leaving directory '/home/runner/work/GraphAr/GraphAr/java/target/native/graphar_ep-build'
[INFO]      [exec] make[2]: Leaving directory '/home/runner/work/GraphAr/GraphAr/java/target/native'
[INFO]      [exec] make[1]: Leaving directory '/home/runner/work/GraphAr/GraphAr/java/target/native'
[INFO]      [exec] collect2: error: ld returned 1 exit status
[INFO]      [exec] make[5]: *** [CMakeFiles/gar.dir/build.make:344: libgar.so] Error 1
[INFO]      [exec] make[4]: *** [CMakeFiles/Makefile2:112: CMakeFiles/gar.dir/all] Error 2
[INFO]      [exec] make[3]: *** [Makefile:136: all] Error 2
[INFO]      [exec] make[2]: *** [CMakeFiles/graphar_ep.dir/build.make:86: graphar_ep-prefix/src/graphar_ep-stamp/graphar_ep-build] Error 2
[INFO]      [exec] make[1]: *** [CMakeFiles/Makefile2:85: CMakeFiles/graphar_ep.dir/all] Error 2
[INFO]      [exec] make: *** [Makefile:91: all] Error 2
[INFO] ------------------------------------------------------------------------
[INFO] BUILD FAILURE
[INFO] ------------------------------------------------------------------------
[INFO] Total time:  01:44 min
[INFO] Finished at: 2023-11-02T13:13:18Z
[INFO] ------------------------------------------------------------------------
Error:  Failed to execute goal org.apache.maven.plugins:maven-antrun-plugin:3.0.0:run (make-and-link-bitcode) on project gar-java: An Ant BuildException has occured: The following error occurred while executing this line:
Error:  /home/runner/work/GraphAr/GraphAr/java/build.xml:24: exec returned: 2
Error:  around Ant part ...<ant antfile="/home/runner/work/GraphAr/GraphAr/java/build.xml">... @ 5:69 in /home/runner/work/GraphAr/GraphAr/java/target/antrun/build-main.xml
Error:  -> [Help 1]
Error:
Error:  To see the full stack trace of the errors, re-run Maven with the -e switch.
Error:  Re-run Maven using the -X switch to enable full debug logging.
Error:
Error:  For more information about the errors and possible solutions, please read the following articles:
Error:  [Help 1] http://cwiki.apache.org/confluence/display/MAVEN/MojoExecutionException
Error: Process completed with exit code 1.

Expected Behavior
Build libgar-jni.so successfully.
Minimal Reproducible Example
See CI.
Environment

Operating system: ubuntu latest
GraphAr version: 0.10.0
System installed Arrow 14.0.0

Link to GraphAr Logs
No response
Further Information
Refer to https://cmake.org/cmake/help/latest/command/target_link_libraries.html and Arrow to find more information.
Consider the difference between linking arrow as static and linking arrow as shared, and choose the better one.
Further features:

[ ] When arrow is not installed, let gar-jni link to graphar_ep's arrow library(shared or static? why?).
[ ] Keep source code and gar library generated by CMake after running mvn clean

If arrow 14.0.0 is not fit to JAVA SDK, you change the CI install dependency:
https://github.com/alibaba/GraphAr/blob/1d714605fadf37f4a311b0f158535e9415a1599c/.github/workflows/java.yml#L39
to
    sudo apt install -y libarrow-dev=13.0.0-1 libarrow-dataset-dev=13.0.0-1 libarrow-acero-dev=13.0.0-1 libparquet-dev=13.0.0-1

as a temporary solution to fix CI.


Fix publication date.

:+1:

:tada: This has been pushed live to https://developers.google.com/web/


Multiple Login Dialogs

Hi,
Again, not sure if this is a BUG at all, but I am in the final stages of testing in-app purchases (this is an auto-renewing subscription).  All the code works in terms of checking for existing subscription and buying a new one (and remotely checking these values etc via IInAppBillingVerifyPurchase).
So, really great, but the reason for this polite help request ...
When testing via TestFlight (and my real IOS ITunes account), and I am now getting MANY MANY (5-7+) login requests when trying to check / restore my initial purchase (when calling GetPurchasesAsync) and when I finally get to Buy a Subscription (via PurchaseAsync).
If I type passwords for them all in, things eventually seem to get to the end of the process, and I am fairly sure I only had to do it the once on the first even install for that user.
But if this was the case in production, it would break my app, and was just wondering if anyone has seen anything similar in TestFlight (I had issue with sandbox accounts during development but this is slightly different)?
I get the feeling that the number of dialogs in increasing as I test the app and re-purchase the subscriptions, which makes me feel it "could" be something to do with similar to this -
https://stackoverflow.com/questions/4988663/skpaymentqueue-addtransactionobserver-asking-for-app-store-password-on-startup-a
Although I can see the code specifically -
https://github.com/jamesmontemagno/InAppBillingPlugin/blob/master/src/Plugin.InAppBilling.iOS/InAppBillingImplementation.cs
calls to -
SKPaymentQueue.DefaultQueue.FinishTransaction(transaction);
I am not familiar with the IOS workflow for all of this (hence was trying to get by with using the plugin and not digging in), but will download the source and link into my app and debug the workflow to see if can get it to repeat during development builds, and if anyone has any ideas how one would test this at the testflight stage, very welcome.
Thanks for your help.
p.s. got about 12-13 dialogs on my latest single purchase test :( can send screenshots internally if that helps at all.
Version Number of Plugin: 1.2.1 (and just tried latest <IP_ADDRESS>)
Device Tested On: IOS
Version of VS: Latest
Version of Xamarin: Latest

With regards to getting this debugging in my local project, I have the following set up (as this is an old project) -
Xamarin.IOS.Library (Core Logic)
Xamarin.IOS Platform Project
I have downloaded source from github and included (and referenced instead of Nuget)
Plugin.InAppBilling.Abstractions
Plugin.InAppBilling
Plugin.InAppBilling.IOS
Builds fine, but not sure how to get InAppBillingImplementation constructor breakpoint to fire on device testing.
Does this sound correct? or am I missing something.
Thanks
Matt

Yeah for debuggign, you should add Abstractions/iOS to your iOS project and then InAppBilling and Abstractions to your PCL/NetStandard

Thanks James, I will persevere until I get a breakpoint hit :)

As I do not have a PCL in this project, just a Xamarin.IOS Library (which referenced the InAppBilling Nuget etc originally), will this stop the debugging working (i.e. effects the bait and switch somehow?)
Thanks
Matt

Nope, just reference the Abstractions and iOS project

Right, pasted the code in and got it debugging, "interestingly" on app start I got 12 transactions in the UpdateTransactions call (now this is using the Debug build NOT the test flight build, so not 100% sure whose transactions these where) -
Resolved pending breakpoint at 'InAppBillingImplementation.cs:747,1' to void WTMXAM.Core.BL.Class.PaymentObserver.UpdatedTransactions (StoreKit.SKPaymentQueue queue, StoreKit.SKPaymentTransaction[] transactions) [0x00027].
Updated Transaction | ProductId:SUBSCRIBE_YEAR | AutoRenewing:False | State:Purchased | Id:1000000316908988
Thread finished:  #6
Thread finished:  #7
Thread finished:  #4
Thread started:  #9
Thread started:  #10
Thread started:  #12
Thread started:  #11
Updated Transaction | ProductId:SUBSCRIBE_YEAR | AutoRenewing:False | State:Purchased | Id:1000000316908988
Updated Transaction | ProductId:SUBSCRIBE_YEAR | AutoRenewing:False | State:Purchased | Id:1000000316908988
Resolved pending breakpoint at '/Users/matthewwaring/Dropbox/Xamarin2/WTMXam/WTMXAM.Core/BL/Class/InAppBillingImplementation.cs:771,1' to void WTMXAM.Core.BL.Class.PaymentObserver.UpdatedTransactions (StoreKit.SKPaymentQueue queue, StoreKit.SKPaymentTransaction[] transactions) [0x000fa].
Thread finished:  #10
Updated Transaction | ProductId:SUBSCRIBE_YEAR | AutoRenewing:False | State:Purchased | Id:1000000316908988
Updated Transaction | ProductId:SUBSCRIBE_YEAR | AutoRenewing:False | State:Purchased | Id:1000000316908988
Updated Transaction | ProductId:SUBSCRIBE_YEAR | AutoRenewing:False | State:Purchased | Id:1000000316908988
Updated Transaction | ProductId:SUBSCRIBE_YEAR | AutoRenewing:False | State:Purchased | Id:1000000316908988
Updated Transaction | ProductId:SUBSCRIBE_YEAR | AutoRenewing:False | State:Purchased | Id:1000000316908988
Updated Transaction | ProductId:SUBSCRIBE_YEAR | AutoRenewing:False | State:Purchased | Id:1000000316908988
Updated Transaction | ProductId:SUBSCRIBE_YEAR | AutoRenewing:False | State:Purchased | Id:1000000316908988
Updated Transaction | ProductId:SUBSCRIBE_YEAR | AutoRenewing:False | State:Purchased | Id:1000000316908988
Updated Transaction | ProductId:SUBSCRIBE_YEAR | AutoRenewing:False | State:Purchased | Id:1000000316908988
2017-11-02 10:27:19.967 WTMIOS[403:32853]  ** formatter.StringFromNumber(4.49) = £4.49 for locale en_GB@currency
Which the for loop seemed to then clean up with FinishTransaction (they where gone on next restart) ...
On searching some more, I think its something to do with the auto-renew system in sandbox mentioned here -
https://stackoverflow.com/questions/10722630/skpaymentqueue-finishtransaction-within-network-completion-block
"After weeks of "phantom" purchases, I've finally realized what is going on. It wasn't the fault of finishTransaction: or AFNetworking.
The transactions that were occurring at app launch were actually part of the auto-renewal process that Apple was performing. The life of a 1 month subscription in the Apple Sandbox is 5 minutes, so I would have 1, 2, or 3 transactions in the queue when launching my app depending upon whether it has been 5, 10, or 15 minutes since I launched it last.
I had to tweak some server-side settings to make sure the renewals weren't being treated the same as fresh subscriptions."
Still not sure exactly what to do next, as not sure why they weren't getting cleared, but will continue with some console.writelines in the TestFlight output around the Trans Q ...

Just an update to close this thread, the app has been live on the app store for about a week and no horrible dialogs, hurrah!
Although whatever apple is doing with sandbox is still horrible for all the testflight beta testers as sandbox login dialog pops-up loads still, if I have any support incidents at the end of the year I will raise with apple support and get it sorted.
Thanks again for library.


Update promgen deps

What this PR does / why we need it:
This PR updates the servicegraph promgen deps to the common v1 version of the prometheus API in client_go. This standardizes the entire istio/istio repo on a single verison of hte API code.
Which issue this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged): fixes #
Fixes #1307
Special notes for your reviewer:
Release note:

@douglas-reid: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/e2e-cluster_wide-auth.sh
06b71b9a16dd701a2aef10b65cca003a4499d50b
link
/test e2e-cluster_wide-auth

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


Adding find_id method to collection

PERL-518

After you simplify the docs, it will LGTM.

@xdg Cut out most of the duplicate documentation. Is there a way to link to the documentation for find_one instead of just mentioning it?

Yes: L<find_one|/find_one> (first part before pipe is the display text, second part is the link)

Ok, should be good now.


Create QuickSort on Doubly Linked List.cpp

any issues?

any issues?

Please add details to CONTRIBUTING.md to get your PR approved


Video without any sound

After rendering, I view the finished video and watch it with sound in the web interface, but when I download it, it is completely without sound. What could be the problem?

It's a codec issue, I'm currently working on a fix. Should be fixed within the next 24 hours.

@egkodin meanwhile audio will actually play on another android/windows device or uploaded to youtube.

Is the problem still persisting?

Is the problem still persisting?

Now it works! Thank you!


Update toplevel ARCHITECTURE.adoc links

TBH these never turned out to be very useful, maybe we should just get rid of them....


5.x planning

(not to be confused with restify's 5.x)
Remove a lot of the logic around the context property and simply use VError's info property. This will necessitate a reworking of the variadic args parsing, since an options object as a first parameter is what we used to differentiate custom restify-errors logic vs straight pass through to VError. This will be a breaking change to the API. It might be worth re-evaluating what this module brings to the table on top of VError now that VError is able to do most of what this module does. In my mind, those things are:

Pre-defined constructors for all http 4xx/5xx errors
Ability to create pre-defined constructors. VError allows you to create typed errors in-situ, but does not expose sugar APIs for creating a constructor itself, which can be a lot of boilerplate if you have many custom constructors.
Bunyan serializers for err.context and err.info. This, arguably, should be an independent module.

Would appreciate any feedback from folks who are using this module.

I'm just now starting to use this package, however all of your suggested points sound fantastic! I'm especially interested in sticking with VError since everything I do lately seems to result in an array of promises, and VError can be converted to WError for production. Additionally the serializers sounds fantastic since I'm using restify-errors with express-pino-logger and the bunyan serializer already helpful. It would also be useful if this could expose VError and WError directly (limits potential version incompatibility issues)

@DonutEspresso mind breaking this out into feature requests? No rush, this module is stable as is, but we have had problems with staging a lot of changes for releases in the past :smile:
We can just leverage conventional changelog to cut major releases as bits of this land.

This really only boils down to a single feature - opened a new ticket.


combine EuclideanVector & EuclideanVectorN, and rename them to Vector

This is pretty much renaming with no functionality change. Now we have a single protocol Vector with all the requirements that are needed for vectors.
I needed to remove the dynamically-sized Vector and Matrix types to free up the names and also because the Vector protocol does not support dynamically-sized vectors.
In a subsequent PR, I want to change Vector to allow dynamically-sized vectors so that I can bring dynamically-sized vectors back. I want to make this change in a separate PR so that it is not overwhelmed by all the renaming changes.
Resolves #142.

Seems like if a fixed size is encoded into the protocol, now I can't make AnyVectorArrayBuffer conform to Vector, which is probably gonna make everything I'm doing more difficult.

Seems like if a fixed size is encoded into the protocol, now I can't make AnyVectorArrayBuffer conform to Vector, which is probably gonna make everything I'm doing more difficult.

I am working on removing the fixed size requirement. I wasn't able to remove it quite as quickly as I wanted, because a lot of things accidentally depend on the fixed size requirement, but I think all those things can be made to work without it.


Theme Uses Grey Instead of White in Mac

I am a mac user and use the default mac terminal. When I installed the theme, and when using the light theme, the background becomes grey instead of white.

Some info that might help finding out the root cause:

I use nvim
I use mac terminal (the default one)
echo $TERM returns xterm-256color
I tested my vimrc file without anything but vim-one and I am still getting grey background. So it is not related to any other configuration.
I am simply using colorscheme one and set background=light
I ran :highlight Normal ctermfg=white ctermbg=white and suprisingly it used grey instead of white background (which probably indicates the problem is nothing to do with the theme?)
If I use vim without theme, the background is white.

Any help is appreciated.

The default background in light mode is a light grey #fafafa.
I think you ran into a limitation of Vim where it is not possible to change the color of Normal background.


after unicorn restart, some zombie processes stay

so many processes like:

what is it?
Seems i need to write a script to detect and kill these processes.

So what was it?

@karmi sorry, my bad, it's [spring] (https://github.com/rails/spring)

@Sen Cool.


Selection Options

This issue was imported from CodePlex
jirikadlec2 wrote 2011-05-10 at 23:38
The selection tool should include the following capabilities (similar to ArcGIS):
-set selectable layers/ make only selectable layer
-add to/ replace/ remove from current selection (from Home tab, not just from attribute table)
-clear selected features (from Home tab, not just from attribute table)
-select by attributes/ select by location.
(http://hydrodesktop.codeplex.com/workitem/7370)

was just fixed

Can be done by using either shift (add), nothing (replace) or ctrl (remove) key while using the select tool.
@mogikanin Do we need extra tool buttons for this?

already works

Select by Attributes can be done via Find tool
@mogikanin Do you know what is meant by select by location?

No need
Probably it means: give me a way to select features which located inside another feature. E.g. select all counties inside given state.

Select by location sounds like a not really simple feature. Not sure we need it but if we do I'd like to create a new issue for this and close this one.

OK


fix header file include errors when compile integration_tests/main.cpp

I'm willing to merge it.
But I wonder why the old state does not build in your setup.
cd build && cmake .. && make dummy_executable works for me.

I didn't use an IDE, but cmake.
I think maybe you install adobe/forest.hpp in your include path, so you can make it on your own computer.
It seems there is no question that something is wrong. The solution may be we modify cmake to put adobe/ on the root include path (this may also avoid file conflicts if we have anohter project use adobe/forest.hpp), or adopt the patch what I submitted.


Fix MachinePool poll interval for sharded controller

Previous work (#2310 / 4117577) added a knob to control the poll interval for the MachinePool controller.
Then a subsequent effort (#2341 / 35ce9c6) split the MachinePool controller into its own StatefulSet.
But the latter forgot to carry the poll interval plumbing into the sharded version of the controller.
Fix.
HIVE-2536

/assign @suhanime

/lgtm


Add Icons to ui_dashboard.html

Describe the Issue
In the "ui_dashbaord.html" page, we have the redirect icons by the left when you hover on "Pages" on the top bar. We would love to include another set of icons by the right. This new icons should be unique to that Page category and it should provide some  clue on where the user is heading to. For instance:

Render Page(This describes a live view of your project): so an icon describing activity or live changes can be used.
Dashboard: An icon describing design or creativity should be added to the right.
Profile Page: An Icon describing a user or person should be added to the right.

File/Files you will work on
dashboard.html
Possible Lines of Code
You just need to adjust three sections of the code in dashboard.html file. All necessary imports have been already made.
Result or Final Looks
The redirect icon should be on the left and a new icon describing the new page should be added by the side. A user icon can be used by the right of the "Profile page".

Hi! I want to work on this issue.

It's alright. Done

Can I add new colored icons or just black and white matching with the theme

thank you.


fix: Consolidate requirements and use compatible release syntax

Resolves #7
This is a follow up PR to PR #6
By consolidating requirements that are dependencies of other libraries into those libraries this reduces the amount of work for the dependency resolver as it places more responsibility on the dependent libraries. By using compatible release syntax it also helpfully restrict the ranges that can be tried which is fine as this course should be viewed as more of an application than a library.

@cranmer This is ready for review.

Thanks again @matthewfeickert


CI test failure

The test "Wem file can be converted to vorbis file" is failing with no apparent reason.
revorb process failed with exit code: 127 (command not found?)
Running the test script works locally on Windows and Linux.

ubuntu-latest is being changed to Ubuntu 22.04. Still strange that revorb would stop working.

Now works again on ubuntu-latest. 😕


GetOutputDirectory changed

https://github.com/gammasim/gammasim-tools/blob/669da4824fc31b9aef1df23b34e127812dabb1fa/simtools/simtel_runner.py#L107
This function does not exist anymore. Noticed when trying to fix #7 when test failed.

Sorry, my bad. Fixed with https://github.com/gammasim/gammasim-tools/commit/7685926bf84e72a6940f1edf7107b391f3a7b831 on the master branch. Please, merge it to your branch.

Thanks, but now I get a different error for "isRayTracingMode" which doesn't exist anymore.
For the record, I am not using my branch, I am using the master just so that I can fix and close #7.

Give me a minute

Take as much time as you want, this is not urgent at all!

Done.
pytest all PASSED at the master now.
Thanks for spotting these bugs.