Debugger not working with SQL Server 2017 and Python

Environment data

VS Code version: 1.21.1
Extension version (available under the Extensions sidebar): 2018.2.1- OS and version: XXX
Python version (& distribution if applicable, e.g. Anaconda): XXX
Type of virtual environment used (N/A | venv | virtualenv | conda | ...): XXX
Relevant/affected Python packages and their versions: XXX

Actual behavior

Debugger does not stop at breakpoints
Program crashes in Microsoft code
Program runs fine from the CLI

Expected  behavior

Debugger stops at defined breakpoints
Program does not crash

Steps to reproduce:

On an machine with SQL Server 2017 and Microsoft Machine learning with Python installed, run the attached program

Logs
Output for Python in the Output panel (View→Output, change the drop-down the upper-right of the Output panel to Python)
There is no output panel called Python

Output from Console under the Developer Tools panel (toggle Developer Tools on under Help)

console.ts:136 [Extension Host] (node:5764) DeprecationWarning: os.tmpDir() is deprecated. Use os.tmpdir() instead.
t.log @ console.ts:136

traceback
Express Edition will continue to be enforced.
Elapsed time: 00:00:01.6467579
Error: *** Exception: 'resourcePath'
Error in processing machine learning request.
TlcBridge error: Error: *** Exception: 'resourcePath'
Error in processing machine learning request.TlcBridge function failed. Please see the console output for more information.
Traceback (most recent call last):
File "c:\Users\Jerry\Documents\Pluralsight\Getting Started with Python on SQL Server\05 MicrosoftML\Demo\probably_pizza_featurize.py", line 23, in
get_sentiment(cols=dict(scores="comment"))
File "C:\Program Files\Microsoft SQL Server\MSSQL14.SQL_PYTHON\PYTHON_SERVICES\lib\site-packages\microsoftml\modules\featurize.py", line 209, in rx_featurize
telemetry_info=telemetry_info
File "C:\Program Files\Microsoft SQL Server\MSSQL14.SQL_PYTHON\PYTHON_SERVICES\lib\site-packages\microsoftml\utils\entrypoints.py", line 150, in run
ret = tlc_bridge(maml_code=maml_code, analysis_type="RunGraph", **kargs)
File "C:\Program Files\Microsoft SQL Server\MSSQL14.SQL_PYTHON\PYTHON_SERVICES\lib\site-packages\revoscalepy\utils\RxFunctionUtils.py", line 8, in wrapper
return func(*args,**kwds)
File "C:\Program Files\Microsoft SQL Server\MSSQL14.SQL_PYTHON\PYTHON_SERVICES\lib\site-packages\microsoftml\utils\tlc_bridge.py", line 171, in tlc_bridge
functionname=function_name, params=call_parameters)
File "C:\Program Files\Microsoft SQL Server\MSSQL14.SQL_PYTHON\PYTHON_SERVICES\lib\site-packages\revoscalepy\RxSerializable.py", line 358, in rx_native_call
ret = px_call(functionname, params)
RuntimeError: revoscalepy function failed.

probably_pizza_featurize.zip

Please could you try debugging a simple python script such as the following:

print("Hello World")

When debugging the python program, where does SQL Server 2017 come into the picture? Is this used just to pull information from?
Please could you upload the contents of your launch.json file

When running print("hello world"), SQL Server comes into the picture since it is the version of Python shipping with SQL server that I am running:

PS > & 'C:\Program Files\Microsoft SQL Server\140\PYTHON_SERVER\python.exe'
Python 3.5.2 |Continuum Analytics, Inc.| (default, Jul  5 2016, 11:41:13) [MSC v.1900 64 bit (AMD64)] on win32
Type "help", "copyright", "credits" or "license" for more information.
>>>

In the program that causes the errors, it is using the Microsoft packages revoscalepy and microsoftml. with local data.
Uploaded launch.json
launch.zip

Running a simple Hello World works fine
Run without debugging fails the same way, for the sample program (though it works fine when run from powershell or cmd.exe)

Hi @gbritton1, could you try using C:\Program Files\Microsoft SQL Server\MSSQL14.SQL_PYTHON\PYTHON_SERVICES\  as your Python path?   Thanks!

That is what I'm using.  That's how I got the error I reported

I'm using the first one.  I'll check the second one later.  Not sure it exists

So, that was interesting.  Running from C:\Program Files\Microsoft SQL Server\140\PYTHON_SERVER
works both from the command line and from VS Code (at least, at the moment).  Running from C:\Program Files\Microsoft SQL Server\MSSQL14.SQL_PYTHON\PYTHON_SERVICES fails in both cases
You, know I've been slavishly following the docs for Python in SQL Server and never noticed that it could be installed twice,  but it stands to reason.  One is for the instance (to be used for in-database processing), the other is for the standalone installation.  Which is interesting.  If you only install the standalone version, which is typical for a desktop (where you don't need an instance), you only get C:\Program Files\Microsoft SQL Server\140\PYTHON_SERVER which thankfully works.
I also compared the contents of the two directories.  They are identical.  So, that nails down the repro (though, looking at my OP (the traceback) I see it did show the instance directory, not the standalone one.

It looks like you cannot break at that line of code that creates the tuple if it's separated into multiple lines. I see this in both PyCharm and VSCode with your code. If you make it into a single line, it'd stop at the line Comments. This is likely just a debugger thing and does not have to do with microsoftml.
As for crashing. It sounds like with C:\Program Files\Microsoft SQL Server\140\PYTHON_SERVER, your script works in either cmd and VS Code. While using C:\Program Files\Microsoft SQL Server\MSSQL14.SQL_PYTHON\PYTHON_SERVICES , the code fails with errors you showed in the traceback.
It may be that microsoftml package in PYTHON_SERVER\ contains the pre-trained model needed for the sentiment analysis while PYTHON_SERVICES\ does not. The pre-trained model are in Lib\site-packages\microsoftml\mxlibs<modelname>_updated.model for Python and \mxlibs\x64<modelname>_updated.model for R. (See reference). Let me know if this is the case.

Sorry, the model you need would be microsoftml/mxLibx/pretrained.model.  You can obtained the pretrained models here by clicking on MLM_<IP_ADDRESS>_1033.cab

Brilliant!  The models are indeed installed in the standalone tree but not in the instance tree.  Copying those across made it work properly.  You know, if you hadn't jumped in, I would not have noticed that there were two directories at play her.  Thanks a ton!.


opengles?

does gl3w support loading core context from opengles as well?

feel free to use/ port: https://github.com/paroj/glXXw

at least for es2.0 contexts (on my stoneage intel gpu laptop; arch linux) i fount that removing the check in "parse_version()" works quite well. Maybe gl3w could remove that check or test for ES and call itself "ES compatible" from then on?

OpenGL ES isn't supported (and it's unlikely it'll ever be, since the project seems abandoned), but, as @paroj said, glXXw is an alternative. It's possible to port said generator to Python and merge it into the gl3w project.


Use sendfile in StaticHandler

see tornadoweb/tornado#1448

Can I try to make this task?

Please do. But keep in mind:

RANGE support #165 should be implemented first
sendfile should be optional, if the function is not supported by platform (e.g. Windows) fallback to conventional way.

I'm working on this in https://github.com/KeepSafe/aiohttp/pull/503. Feedback welcome.

Fixed by #503


Install-OSRabbitMQ should have a -Sources parameter for offline instalations

Rabbit and Erlang are now included in the OutSystems platform server installer.
thirdparty\erlang.exe and thirdparty\rabbitmq.exe.
So, we will not download the sources.


Advanced vehicle

This list contains some of the current problems for the proper operation of the vehicle. I decided to correct them, so this list is more for reference than for a call to action. I will try to fix it soon.
The list contains inaccuracies (sorry for my English) and will therefore change in the future.
Status: Waiting for PR.

[x] (Done)  There is no way to lock a particular wheel to make it brake.
This will allow you to implement features such as wheel jamming.

[x] (Done) The brake parameter affects only those wheels where engine_traction == false.
Where it would be more correct to make it a separate adjustable parameter for each wheel.

[x] (Done) Not possible to set the steering angle for each wheel.
This is important for vehicles where the rear wheels have to turn in the opposite direction. Also, due to this, you can make a more realistic steering angle of the wheels when one wheel turns stronger than the other.

[x] (Done) There is no custom center of mass.
To create a vehicle, it is an important element.

[x] (Done) There is practically no transverse friction of the wheel.
If you brake, the vehicle will continue to move sideways for some time.

[x] (Done) To correct inaccuracies in the documentation VehicleBody and VehicleWheel nodes.
Documentation has unreliable examples, following which you can get "porridge" instead of the vehicle.

@bojidar-bg
I decided to fix it myself and so far everything is fine. However, the code will change greatly due to the fact that the original code was written around 2006, and being in Godot has not changed since the very first Godot version.
Could you return the "enhancement" label and add more of the following labels:

"breaks compat" (since I had to break compatibility with previous versions)
"bug" (since half of the items - bugs)
"documentation" (since there are changes in the documentation)

Not all changes are listed. In fact, they are much more, but they will remain "icing on the cake".

Ah I made porridge with the previous documentation, LOL :)
I'm going to find some time to go over this, all sounds like awesome enhancements, great work!

Could you return the "enhancement" label

No, since there are some pretty large changes discussed, as opposed to small enchancements. Will add bug and tracker though.

@BastiaanOlij I'll fix it myself soon.

@Timofffee I found your new-vehicle branch and your bugfixes solved some pretty big problems I was having with tires at rest never gaining traction.  I also found your per-tire steering to be valuable, I have a monster truck that requires this feature.
I was wondering if you were still planning on a PR with your fixes and features.  Maybe this could be split into two PRs, one for the non-breaking fixes and one for the features that break the API.  I've got a handful of vehicles if you need any help with testing.

This looks promising.

@jonri That sounds good. Thanks. Maybe I'll do that. I stopped the development because I couldn't adjust the car properly and it still looks terrible. If you managed to make a good suspension and get rid of sinking tires - let me know.
Also thank you for appreciating the new features. Now they also work quite badly, because I did not pay them enough attention.
I will be very happy if you help me with testing the code. Can I contact you anywhere other than GitHub?

@Timofffee sorry to not get back to you, I've been travelling a lot lately but will be back to normal next weekend.  I've gone ahead and re-done the per-wheel forces in a way that keeps backwards compatibility for existing projects using the current acceleration, braking, and steering.  I'm going to do final testing and a PR next weekend, I think we can make a case for including it because it restores functionality that was already in the bullet raycast vehicle and was probably just taken out for simplicity.
If you want to discuss anything outside of GitHub, I have the same username on the Godot discord server and on IRC where I lurk sometimes as well.

Should this be considered fixed by #30685?

Yeah, but it doesn't fix everything.
@jonri informed me that he would take part of my code in his PR. However, he took only a part. The rest of the code only extends the capabilities of the vehicle and makes it more enjoyable to use.
I'm doing it now, but it takes too much time to test. The topic should not be closed. Remove the label "bug". The contents of the subject I will fix soon.

Any updates?

Some updates are already available in 3.2 alpha 1. The rest either works poorly or I can't do it at all now.

It seems that this is 100% complete. Is there more, or does this need to be closed?

I'm sorry, but I was busy with other things and completely forgot about it. Within a month, I will review the code and if everything works well, I will do PR and close the issue.

Okay. This thread seems to be finished, but it does not make any sense, since all the work will be done again. I'll close the issue so it doesn't get in the way.


Use API to change color function?

I know you can use the API to change the mapping of colors to series (e.g. http://c3js.org/samples/api_data_color.html).
Is it possible to use the API to change the color function, like in the example below except after the chart has been generated?
http://c3js.org/samples/data_color.html

I think you can do that like this:
// var chart = c3.generate({ ... });
chart.internal.config.data_color = function (color, d) { return '#000000'; };
chart.internal.color = chart.internal.generateColor();
chart.flush();

This can be an API, but I'm not sure there is so much use case for that, so I didn't do that.
And can you ask this kind of question in Google Group first? Then, you could get an answer more quickly. Thanks. https://groups.google.com/forum/#!forum/c3js

Please let me close at the moment and please reopen if it should be implemented as a new feature.


contentContainerStyle retro-compatibility ?

Hello !
Thank you first for this package, I'm really thrilled to use it !
I tried to migrate from FlatList to FlashList and found that my paddingHorizontal is not working when passed to contentContainerStyle props. Is there something to do to add paddings to the list ?

@billouboq Add padding to the wrapper View component.

<View style={{ paddingHorizontal:12 }} >
<FlashList/>
</View

@billouboq Add padding to the wrapper View component.

<View style={{ paddingHorizontal:12 }} >
<FlashList/>
</View

Yep already created some container around FlashLists but I wanted to know if it will be possible in the future to use styles props the same way in FlatList and FlashList or if we need to think differently when using FlashList and it will stay that way ?

Thanks for the reply ! And thank you for the hard work !
Le sam. 2 juil. 2022 à 15:33, Talha Naqvi @.***> a
écrit :

Closed #491 https://github.com/Shopify/flash-list/issues/491 as not
planned.
—
Reply to this email directly, view it on GitHub
https://github.com/Shopify/flash-list/issues/491#event-6922175158, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AB3HZTCDLN2TNRWY2LQSGATVSBAJNANCNFSM52M3DQGQ
.
You are receiving this because you were mentioned.Message ID:
@.***>

@billouboq Add padding to the wrapper View component.
<View style={{ paddingHorizontal:12 }} >
<FlashList/>
</View

I think this isn't a good solution.

You will see a vertical scroll indicator also padded which is not expected.

Put padding on your items not the list wrapper


Initial MkDocs Skeleton

This sets up:

A bare bones, MkDocs generated website
The website uses the mkdocs-material theme
Github Actions is used to deploy the website

The website will be built for any Pull Request (PR) targeting the main branch
The website will be built and deployed for any push (e.g., PR merge) to main

Once merged, the website will be hosted at  https://ESIPFed.github.io/esds-community-guide/

It also:

Adds a Code of Conduct
Updates the  README.md describing how to work with MkDocs (likely could be more fleshed out)
Adds a .gitignore for Python, Vim, PyCharm, and VS Code (should cover the most popular workflows)

TODOs
There is some repository setup that needs to be done for this to work

[ ] Someone with push access (can be me if I'm added as a maintainer) to this repository needs to setup a GitHub Personal Access Token and add it to this repository as a repository secret called ESIPFED_ACCESS_TOKEN with 'repo' scope
(will need to be able to push to the repository with this token)

[ ] Confirm after deployment (I believe mkdocs will do this setup when initially deployed) that this repository is configured to host GitHub Pages for a project site. In the Repository Settings you should see section that looks simmilar to:

You can see what this PR will look like once deployed at https://jhkennedy.org/esds-community-guide/
Note: It appears as a sub-site of my personal website because my personal website is hosted as a Github Pages User/Org site (with a CNAME) -- all projects websites for my repositories (at https://github.com/jhkennedy) will appear as subpages of my User/Org sites without giving them their own domain and setting up the CNAME.
Unfortunately ESIP also hosts a Organization size https://esipfed.github.io/. For the most part, this won't affect our ability to host this website, except

404 and other errors will appear as part of the organization site.
The organizational site is actually a project site for LakePy and would be better hosted as a project site at https://esipfed.github.io/LakePy. "Fixing" that would likely require

deploying the LakePy website as a project site
Either

deleting the Org site (will break url with no help for people looking for the LakePy website), or
Creating a simple ESIP landing page that describes ESIP and points to the project sites it hosts (will help people get to the LakePy website)


(void-variable callback): look into why lexical bindings is not enabled

Despite
https://github.com/prometheansacrifice/esy-mode/blob/26f80310f718310f48070346fad4d0163e2f5954/esy-mode.el#L32
This causes a failure at the end of (compile 'esy')

This happens when the current buffer changes. closed by commit


[Bug] [Improvement] Improve the import of Postman variables

Describe the bug
In my new job they're using Postman and personally I prefer Insomnia. So I tried to import postman environnement variables to Insomnia and it didn't work for because the variables were either nested (openapi.tokenId) or with a dash
To Reproduce
Steps to reproduce the behavior:

Go to 'Preferences'
Click on 'Data'
Import data using this example :

{
  "id": "65851a98-bb7b-4132-a0b4-5175edc63f01",
  "name": "env",
  "values": [
    {
      "key": "dev_or_prod_vpc",
      "value": "dev",
      "enabled": true
    },
    {
      "key": "color",
      "value": "abidjan",
      "enabled": true
    },
    {
      "key": "service-gateway-host",
      "value": "service-gateway-{{dev_or_prod_vpc}}.eu-west-1.elasticbeanstalk.com:8080",
      "enabled": true
    },
    {
      "key": "openAM.tokenId",
      "value": "AQIC5wM2LY4SfcztVCmR1P5o2QlyQQpJuapG3IZgT6E3AZc.*AAJTSQACMDIAAlNLABQtNDIwNjIxMDAxNTIyODQyMDAxNwACUzEAAjAx*",
      "enabled": true
    }
  ],
  "_postman_variable_scope": "environment",
  "_postman_exported_at": "2020-04-20T09:40:11.697Z",
  "_postman_exported_using": "Postman/7.22.1"
}

You'll get an error.
Solution
So I wrote some code to handle this for my own purpose and then I decided to add it to the Insomnia source code and it work like a charm.
So I can make a PR if this issue is relevant.
Thank you for your great work.

Hi @LamineMbn! Thanks for using Insomnia! ♥
This is a commonly reported issue: #1666 and #456, and the limitations for why it's a tricky one are outlined in the linked there too.
However! If you have something working, feel free to submit a PR and we'll take a look. We were talking about the dash restriction just yesterday!

Closing this issue to keep discussing in #1666 :)

I'm going to close this issue in favor of other issues reported in the comment above.


upgrade Csproj tests, remove old nuget restore

fixes #2

Unfortunately all current tests has  [Ignore("Ignore")] :(

anyway, this works:


Service exited with abnormal code: 1

Apr 23 15:40:19 mbp com.apple.xpc.launchd[1] (homebrew.mxcl.skhd[4026]): Service exited with abnormal code: 1
Apr 23 15:40:19 mbp com.apple.xpc.launchd[1] (homebrew.mxcl.skhd): Service only ran for 0 seconds. Pushing respawn out by 10 seconds.

I can see that in /var/log/syslog constantly
here is my skhdrc https://github.com/anildigital/dotfiles/blob/master/.config/skhd/skhdrc
What could be the issue?

This started happening for me as well today. Surprisingly, the error turned out to be caused by Bitwarden, which I had just installed to try out. After closing and uninstalling Bitwarden, skhd is running again.

@jryom I don't use bitwarden though. Error still occurs.


Changes: Updating BASEIMAGE for s390x from s390x/busybox to s390x/alp…

I have create this PR to resolve below listed issue:
https://github.com/kubernetes/contrib/issues/2661
Description: Updated Exechealthz s390x BASEIMAGE to s390x/alpine which has nobody user in nobody group. s390x/busybox image has nobody user in nogroup group.

This change is

@MrHohn @bowei  Actually I think nobody:nobody in the yaml/dockerfile specs is a bug. I think it should be nobody:nogroup, isn't that pretty much a standard?

I seem to recall that there were some base images that required nobody:nobody, but am not sure which image or whether that is actually true.

Humm, seems like we are using nobody:nobody in for the latest kube-dns s390x images (rule.mk#L30-L51), would that be broken as well?

From https://github.com/kubernetes/dns/issues/61#issuecomment-288391559 looks like this specific issue only happens on exechealthz.
cc @gajju26

@MrHohn
I think for dns we did not face this issue because in the Dockerfile they are not explicitly switching to user "nobody" as they are doing in exechealthz

@gajju26 Thanks for the reply. Though from https://github.com/kubernetes/dns/blob/master/Dockerfile.sidecar#L21 we could see at least sidecar image is switching to nobody:ARG_NOBODY, for s390x that corresponding to nobody:nobody.

Agreed that we shouldn't hardcode nobody:nobody in Dockfile. Sent https://github.com/kubernetes/contrib/pull/2675 to change the user:group configurations instead of switching base image.

@MrHohn referring to the link which you had mentioned earlier https://github.com/kubernetes/dns/blob/master/rules.mk#L30-L51
for kube-dns too we will have to either update all the BASEIMAGE's to respective alpine images for all ARCH or set
NOBODY?=nogroup if we continuing with respective busybox images for PowerPC and s390x ARCH.
I had encountered the nobody user specific issue when I had executed e2e-test of kube-dns which  spawns sidecar container.
CC: @gajju26 @bowei @luxas

@MrHohn as my suggested changes has gone in a new PR https://github.com/kubernetes/contrib/pull/2675 marking this PR as closed. Kindly let us know what the plan for suggested Kube-dns changes in rules.mk files.
Thanks.
cc: @gajju26

@guirish Thanks for the reminder, I will send PR to fix the kube-dns images.


Feature/add protect to entry model

We wanted to use the protect feature that the MD files have but this doesn't work by default in the Eloquent solution.
With these changes it does work.

Thanks! I'd probably be inclined to just use the data column for it, much like how blueprint works. Saves a migration?
If you want to continue with the migration thats fine, but it would need an update script and the base migration would need updated (currently your update migration doesn't run).

I agree on that. I moved this way considering this is how the MD files work. I removed the migration and updated the model to use data protect

Can you supply a test for this where it doesn't work without this change? For me adding protect to the data column already works without this change.

Your correct i cant believe i missed that it already worked with the data field


Update CORS doc

Fixes #10623, Fixes #9705, Fixes #10839
Internal review URL

@blowdart 	please assign someone to review this doc - perhaps @javiercn

@pranavkm Could you please take a look at this?

Here is my feedback as a whole
// Shows UseCors with CorsPolicyBuilder.
app.UseCors(builder =>
{
builder.WithOrigins("http://example.com",
"http://www.contoso.com");
});
The above sample is an advanced case. It's more common to configure CORS through MVC attributes or through
services.AddCors and then have the call be app.UseCors() (There's a default CORS policy if I remember well
and you can configure named policies)
Razor Page action method -> Unless this has changed, this statement is false. Have you tried it out?
model/action method, and CORS is enabled in middleware, the attribute takes precedence. -> Are you sure about this?
I would have swore that it applies both. We usually recommend people to use one or the other AFAIK.
$.ajax({
type: 'get',
url: 'https://www.example.com/home',
xhrFields: {
withCredentials: true
}
This is very 2000s. I would simply use the fetch API which is the modern way to do XHR requests from the browser.
#########################
How CORS works
This section describes what happens in a CORS request at the level of the HTTP messages. It's important to understand how CORS works so that the CORS policy can be configured correctly and debugged when unexpected behaviors occur.
The CORS specification introduces several new HTTP headers that enable cross-origin requests. If a browser supports CORS, it sets these headers automatically for cross-origin requests. Custom JavaScript code isn't required to enable CORS.
The following is an example of a cross-origin request. The Origin header provides the domain of the site that's making the request:
Copy
GET https://myservice.azurewebsites.net/api/test HTTP/1.1
Referer: https://myclient.azurewebsites.net/
Accept: /
Accept-Language: en-US
Origin: https://myclient.azurewebsites.net
Accept-Encoding: gzip, deflate
User-Agent: Mozilla/5.0 (compatible; MSIE 10.0; Windows NT 6.2; WOW64; Trident/6.0)
Host: myservice.azurewebsites.net
If the server allows the request, it sets the Access-Control-Allow-Origin header in the response. The value of this header either matches the Origin header from the request or is the wildcard value "*", meaning that any origin is allowed:
Copy
HTTP/1.1 200 OK
Cache-Control: no-cache
Pragma: no-cache
Content-Type: text/plain; charset=utf-8
Access-Control-Allow-Origin: https://myclient.azurewebsites.net
Date: Wed, 20 May 2015 06:27:30 GMT
Content-Length: 12
Test message
If the response doesn't include the Access-Control-Allow-Origin header, the cross-origin request fails. Specifically, the browser disallows the request. Even if the server returns a successful response, the browser doesn't make the response available to the client app.
#########################
I would reword this section to make emphasis on a couple of things

CORS is NOT a security feature.

For example, an evil actor could XSS your site and do a Cross-site request to their CORS enabled site to steal information.

Your API is not safer by allowing CORS.

It's up to the client to enforce CORS, the server is going to continue executing the request no matter what, so for example
a .NET HTTP Client can keep calling your API. It's completely up to the browser to enforce it on the client and prevent the page
from accessing the response.

It's a way for a given server to allow browsers to do something (cross-origin xhr request) that otherwise would be forbidden.

Browsers couldn't tipically do cross-origin requests, that's why they resorted to techniques like jsonp wich receives the response
as the contents of a script that get injected into the page (scripts are allowed to be loaded cross-origin).

@Rick-Anderson Looks like Index2.cshtml in the sample app is never used. You can probably delete that too. I was going to in my PR, but I forgot.

That's the actual snippet with the following change:
    `fetch('https://<web app>.azurewebsites.net/api/values/1')`

I copy/pasted your code.

@pranavkm please review. I think I address all of @javiercn 	comments.

@javiercn 	@pranavkm 	can one of you finish this for the Doc-a-thon?

@javiercn @pranavkm can one of you review this?

MOved to #11141 as I picked up some unwanted commits.


feature: incremental star retrival

This PR enables incremental star retrieval!
Here's a quick demo:
https://user-images.githubusercontent.com/1811651/103342693-11fb2980-4a3f-11eb-8ca6-bea7dca2086e.mov
I think you'll notice a few things in the demo.

As soon as there are updates in retrieved data, it's shown as a chart.
It shows the percentage of data that's already been retrieved.
When you refresh the page and come back to the same repo, the data retrieval progress is not lost.

I achieved this by creating a new table called stars_incremental2, which has three additional columns - cursor, is_complete, and total_star_count.
cursor is the cursor of the last stargazer that we've seen.
is_complete is false when all of the data haven't been retrieved yet.
total_star_count shows the total star count of the repo even when all of the data haven't been retrieved yet.
With this, I'm able to store intermediate data retrieval results in this table so we don't have to get all the data every time.
If this seems okay, I would suggest renaming this new table to stars, and the current stars table to stars_legacy2 or something like that.
Also, I noticed that the UI changed a bit when I merged master. I hope I didn't break anything major when I resolved the merge conflicts. I really liked how it used to show the total number of stars - did it disappear because of one of @joshenlim's commits or because of how I resolved the merge conflicts? I wasn't sure.

NOTE: this fixes #23, and it makes #11 much easier to implement.

This is amazing stuff @ykdojo - yeah! let's swap over the tables, i'll rename them once these changes are in master.


Can't load shared library when run SampleApp

IMPORTANT: Before you create an issue, please take a look at our Issue Reporting Guide.
Briefly summarize your issue:
Can't run SampleApp successfully because of the shared library path.
I built SampleApp on the PC then placed it in my target platform which uses MIPS CPU; also, all the libraries which SampleApp needs are in the lib folder in my target platform.
However, when I tried to run the SampleApp, it shows that SampleApp cannot load shared library, because the shared library path is based on the PC platform.
Could you tell me how to set up the shared library path for SampleApp in target platform?

The error as below:
Error loading shared library ../../ApplicationUtilities/DefaultClient/src/libDefaultClient.so: No such file or directory (needed by ./SampleApp)
Error loading shared library ../../CapabilitiesDelegate/src/libCapabilitiesDelegate.so: No such file or directory (needed by ./SampleApp)
Error loading shared library ../Authorization/CBLAuthDelegate/src/libCBLAuthDelegate.so: No such file or directory (needed by ./SampleApp)
Error loading shared library ../../ESP/src/libESP.so: No such file or directory (needed by ./SampleApp)
Error loading shared library ../../MediaPlayer/GStreamerMediaPlayer/src/libMediaPlayer.so: No such file or directory (needed by ./SampleApp)
Error loading shared library ../../CapabilityAgents/Alerts/src/libAlerts.so: No such file or directory (needed by ./SampleApp)
Provide the steps to reproduce the issue, if applicable:

Tell us about your environment:
Cross compile the AVS Device SDK v1.12.1 for MIPS linux
Tell us what hardware you're using:

[ ] Desktop / Laptop
[ ] Raspberry Pi
[V] Other - tell us more:

Tell us about your OS (Type & version):

[V] Linux
[ ] MacOS
[ ] Raspbian Stretch
[ ] Raspbian Jessy
[ ] Other - tell us more:
mips_24kc+nomips16_gcc-6.3.0_musl-1.1.16

Hi @radinet-zoe thanks for reaching out. When you mentioned you compiled the code on PC and put it on the MIPS platform did you mean you cross compiled for that platform specifically?
Also, since the SampleApp is built using cmake it expects specific path where the libraries are located and hence you are seeing this issue. Why not move the whole BUILD folder over to the platform to try this?

I am closing this issue due to inactivity. Please feel free to re-open it if it has been closed in error.


Preliminary end-to-end stuff

Action items/todos

seed all datastore stuff
make models swappable, config and all that
add tests (lmao)
make it easy to run

@JonahSussman feel free to merge this. we can work on separate PRs for the rest of the todo items


Update README.md

description added

In IT sector usually you have 5 Hours working day and 8 or 9 hour working day, is your question similar to that, because I don't get what shift means


Entity names should not be allowed to include () bracket openings and closings

In Xerxes, we have a YAML and a simple text interface to read in statistics. The simple text interface looks like
F3(Pop1, Pop2, Pop3)
because of this structure, the names of the groups cannot contain bracket openings and closings, and that is something that has to part of the parser. Right now, poseidon-hs does not include that constraints, which unfortunately leads to Xerxes having a redundant parser (https://github.com/poseidon-framework/poseidon-analysis-hs/blob/fedf433d10b5fb9f7bd9847928193c8fbc7f4318/src/Poseidon/Analysis/FStatsConfig.hs#L162)
It would be nice if we could simply add the two symbols ( and ) to the forbidden symbols in entity names here (https://github.com/poseidon-framework/poseidon-hs/blob/e7c1ce6813be3369b545d6f5834128fe1a74ceac/src/Poseidon/EntitiesList.hs#L126) so we can avoid this redundancy.

Hmm... good point. I don't really know, to be honest. Technically, it is unnecessary to forbid brackets altogether, since I could in principle parse things like "MyFavoriteGroup(take 1)" by matching opening and closing brackets. The only thing that wouldn't work is "LetsTryAndHackThis)".
Finally, I could also change the interface to accept group names in quotation marks, optionally. Perhaps that would even be better...
I would not act on the schema for now, I will think more about it.


dom-change event does not fire when HTML is moved to modal

I have a Polymer element that has a dom-if template with a on-dom-change attribute, and inside is a div that calls a function to set the id:
<template is="dom-if" if="{{!isCombined(item)}}" on-dom-change="fieldChanged">

</template>

When I first load my page, this builds and fires the dom-change event handler "fieldChanged".
However, I am using this template within a nanoModal and it seems the dom-repeat and dom-if templates are executing as I can see the internalId function being called, but the fieldChanged event handler is not being fired.
Many thanks.

Are you expecting the dom-change event to be fired when the isCombined(item) value changes? Or the internalId(item.id) value? It should only be fired for the former.
Not familiar with nanoModal, but perhaps you can include a little more code to clarify the issue.

That would explain it. Is there a way to know when a
It may be that the use of Polymer is incorrect as the  tags are being converted into other DOM elements, like input and select, depending on some configuration values supplied. It may be correct to use dom-if, but the there are multiple conditions and this section of code is used twice in the Polymer component. It may be that the component is too complex.

If you depend on the element existing then you can remove the dom-if and just use hidden, as it would be effectively the same.

The issue is with the dom-repeat too, and not just the dom-if. Both are asynchronous calls.


hope comming version will have custom drag action

May I know whether comming version will have custom drag action that I can handle graphic in "dragmove" event function instead of svg.draggy.js?

This is already implemented: https://github.com/jillix/svg.draggy.js#dragmove
The dragmove event is emitted when dragging the element.

Sorry for my unclear description.
Actually, I wanna one feature that is without any drag effect, and set custom effect like changing svg's path instead of moving the whole svg's path in dragmove function.

@CyrusNG I'm not sure, but maybe this animate function is what you want?

@CyrusNG this is implemented in the official draggable plugin from svg.js.
You can cancel the default dragmove and do anything else instead.


JAMES-3487 Java property: MimeMessageInputStreamSource THRESHOLD

Docmentation available
Added but commented within jvm.properties file (cf #744)

This is a rework of #285 using #744 to improve discoverability

Cc @ottoka @mbaechler @jeantil

Nice to see there is actually a good use for jvm.properties. Did not expect anyone to find one so soon :-)

Did not expect anyone to find one so soon :-)

That's why I was enthousiastic about your proposal ;-)

I am glad these settings are more readily available, I can't help but wonder if introducing a parallel (and poorer) configuration file system is really a good move though

Checkstyle errors:
04:42:46,299 [ERROR] src/main/java/org/apache/james/server/core/MimeMessageInputStreamSource.java:[52] (whitespace) EmptyLineSeparator: 'METHOD_DEF' should be separated from previous statement.

04:42:46,299 [ERROR] src/main/java/org/apache/james/server/core/MimeMessageInputStreamSource.java:[58] (whitespace) EmptyLineSeparator: 'VARIABLE_DEF' should be separated from previous statement.


Major improvement to this package

Removed most of the buggy code

When will this be merged? Urgently needed.


During the training process, the f1 score did not improve

Thank you for sharing the code, it helped me a lot. However, I used the java-small dataset and your parameters in run_small.sh to train more than a dozen epochs, and the F1 score did not improve. Do you have this situation? Because the GPU of my machine is not very good, an epoch will take a long time. I would like to ask if there will be changes if I continue to train.

At least in my environment, F1 score goes up in training. However, the result is lower than the score in the paper. I'm looking for the cause, but I'm still not quite sure what it is.
If you want to quickly replicate the score of the paper, I think you should refer to another implementation.

Thanks for your reply, I am not in a hurry to replicate the F1 score in the paper. I just started and only learned pytorch, so your code has been very helpful to me.
Did you use "python main.py --trainpath train.h5 -g 0 --validpath valid.h5 --datapath java-small.dict.c2s -e 40 -b 256   --savename output --trainnum 691974 --validnum 23844"?
I used this to train, and F1 score never exceeds 0.5.


NEW VERSION DESTROYED HARD DRIVE CONTENTS - DO NOT USE

Tried new Rend3.
Cube test worked.
Rebuilt "render-bench" with new Rend3.
Converted code where necessary for new Winit and new Rend3-Framework, but no other changes.
Rand "render-bench".
Try #1: Skybox was drawn, but hung at "Drawing permanent buildings". Exited OK.
Try #2: Segmentation fault.
Try #3: Running under GDB. Many threads seen to exit. GDB hung. Window manager hung. System completely unresponsive to keyboard and mouse. Continuous disk I/O in progress. After several minutes, finally had to press RESET.
System failed to restart. See screenshot.
This is very bad. Is there hostile code in this build?
Ubuntu 22.04 LTS, unmodified.

Recovery underway. File journal recovery reported about 50 lost files. Evaluating damage.

So far, it looks like nothing important was lost. Whew.
This is very strange. render-bench doesn't write anything to disk. It just reads some images at startup. How could it possibly go into frantic disk activity for several minutes? That machine has 32GB of RAM and a small swap file only because Ubuntu creates one. No idea what it was doing.
I don't dare run it again. It's scary that it does more damage when run under gdb.

Closing as this was due to OOM after request_redraw wasn't called in the main loop.


Start time at midnight issue

If you set the Start of a SendCalendarEvent using a date that's at exactly midnight like the following it doesn't render out the DTSTART correctly.
calEvent.Start = new CalDateTime(DateTime.UtcNow.Date);
It renders out like the following:
DTSTART;VALUE=DATE:20210217
When it should probably render out as:
DTSTART:20210217T000000Z

It look like, that this issues also causes that the result of GetOccurrences does not provide the time of day for an event defined like this:
`        // Every Day at hour 11 and hour 14, for 14 occurrences
var tomorrow = DateTime.Now.Date.AddDays(1);
var rrule = new RecurrencePattern(FrequencyType.Daily, 1)
{
Count = 14,
ByHour = new List { 11, 14}
};
    var calendarEvent = new CalendarEvent
    {
        Start = new CalDateTime(tomorrow),
        RecurrenceRules = new List<RecurrencePattern> { rrule },
    };

    var calendar = new Calendar();
    calendar.Events.Add(calendarEvent);
    // calendar.RecurrenceEvaluationMode = RecurrenceEvaluationModeType.AdjustAutomatically;
    var occurrences = calendar.GetOccurrences(new CalDateTime(DateTime.Now.Date), new CalDateTime(DateTime.Now.Date.AddDays(50)));

`

Just ran into this myself, is there a workaround currently?

FYI this is what I am doing and it seems to fix the issue
if (!cdtStart.HasTime) cdtStart.HasTime = true;
if (!cdtEnd.HasTime) cdtEnd.HasTime = true;

Thanks, a cleaner solution to the bug than mine.

FYI this is what I am doing and it seems to fix the issue if (!cdtStart.HasTime) cdtStart.HasTime = true; if (!cdtEnd.HasTime) cdtEnd.HasTime = true;

In my case, all of my dates (even those at midnight) are meant to have a time, so I just set HasTime = true on every record now. This is exactly the right solution. Thanks, @kellcomnet2


PCL version of current release not supported ?

The database is awesome .. unfortunately I need to use a PCL version with my Xamarin app.
It appears that you dropped support for the PCL version.
All of the NuGet packages I use are PCL's.......
As far as I know I cannot use a PCL version of NuGet packages at the same time I use the .NET Standard library versions in a single project.
So is there an option for a current release of LiteDB as a PCL ?
Thanks

So I did a little more digging and found that you de-supported the PCL version....
https://github.com/mbdavid/LiteDB/wiki/Portable-Version
I can't be the only one who needs a PCL version .. I would be happy to make a "donation" to have a PCL version kept current ..... anyone else with me ?

Hi, netstandard cann't br used? V3 are build over new netstandard format, that are new pcl

As far as I know you cannot mix profile 259 (PCL) and .NET Standard is 111 ....
For instance.. I am using this ...
https://www.nuget.org/packages/Acr.UserDialogs/
It supports profile 259.... as far as I know .NET Standard is profile 111 (?) and cannot be mixed ... anyone else know if this is true or not ?

I guess the big issue is this .. I use about 25 or so  NuGet packages in any given project and they are all PCL (259) ....
I would like to use LiteDB as a 259 because the 25 or so packages I do use aren't going to be converted to .NET Standard very quickly...... if at all ?
I "wish" they were all .NET Standard but alas that is not the case :)

Hi! With the objective of organizing our issues, we are closing old unsolved issues. Please check the latest version of LiteDB and open a new issue if your problem/question/suggestion still applies. Thanks!


Support for file pattern in the advanced option

Hi, Thanks for this. I was looking at this package to add to my build workflow for nuget creation. As a part of my workflow i get my processed file to some folders say features/A, features/B under a dist folder. But in the nuget package i was looking to have the content folder not to include "dist" hierarchy instead just content/features/A/*.* for package A, content/features/A/*.* for package B etc. But it seems like if i provide the pattern "." within the advanced file option files: [{src: "dist/features/A/*.*", dest: "content/features/A/*.*"}," it creates package with invalid file names and the package is corrupted. I see i cannot use the simple option of src: ["dist/features/A/*.*"] because it will include "dist" in the content folder as well. Is there a configuration i am missing possibly? Appreciate your help on this.

Just saw the source code. you are deriving relative path based on the baseDir in conjunction with the simple source syntax. And it worked. So closing the issue.


AuthenticateJWTLocal does not return custom claims

When we use AuthenticateJWT, which calls out to Stytch's APIs, a custom Session is used, which supports custom claims.
In the local version, the intermediary Claims type is used, which does not support custom claims.

This should be fixed by #164, which just released as v12.2.1.
Thank you for reporting this!


Group exercises are now returned even from parent groups

Coverage increased (+0.01%) to 68.873% when pulling 201edb9344a8e46d42df3c34bd59f6d7d182c47c on group-exercises-fix into 8faf8beadfcdf895f31a00ea5cac16086028f6a7 on master.


Lane after receive breaks the publisher

Hey, Marin!
I might be totally wrong here, but I'm almost sure this used to work in an older version, but now it also breaks the whole flow, sink doesn't even get called:
myObject.objectWillChange
	.lane(myObject.date) // This works
	.subscribe(on: DispatchQueue.global(qos: .utility))
	.receive(on: RunLoop.main)
	.lane("Received for \(myObject.date)") // This breaks
	.sink { ... }

I do have a feeling I'm missing something obvious, or don't fully understand how this should work.
Have a great evening!

Switching queues shouldn't affect the subscription but I'll give it a try - if it's a bug - need to fix it 👍🏼

@rolandleth thanks for catching this - it was a very subtle bug. This release fixes it https://github.com/icanzilb/TimelaneCombine/releases/tag/1.0.10

Nice, thanks for looking so quickly! 🎉

Well, I had a bit of free time today 🤷🏽♂ glad it's fixed


Added dataframe skew method

In this PR I am addressing #6880 by adding a skew-method to the DataFrame class. I pretty much followed the var implementation, for now focusing on the numeric part.
In order to make the output consistent with methods like var, min and max I had to slightly edit the 0d output for the stats skew-method such that it returns a float and not a 0d array. I don't think it should be considered a full fix and more of a temporary fix, so I kept the TODO there to properly fix that. I couldn't figure out how to properly extract the float without item, but using min or max worked decently well. When changing the output, I noticed it didn't break any test, so I added one for the case I changed.
I noticed, one of the import tests failed when I added the array.stats import to the top of the core.py file because scipy is needed for array.stats. Given the conversation in #6890, I moved the import into the method so as to not break the overall module.
Futher, I checked the pandas behaviour for timedelta columns and generally pandas skew() fails with TypeError: reduction operation 'skew' not allowed for this dtype when using the method on such a column. This translates through to dask via meta = self._meta_nonempty.skew(), so the behaviour is consistent. I added a test for that as well.
One thing to note is that the dask version does not handle nan's (returns nan for a column that inclued nan), but the pandas version does.
Lastly, I also added the dependency entry to the docs for scipy to address #6890.
Please let me know if there are more things that should be changed or adjusted and I'm happy to do it!

Regarding the output type, it looks like there is a TODO in da.stats that mentions returning a scalar rather than an array when the output has 0 dims. I imagine that is where you'd want to look: https://github.com/dask/dask/blob/b3707ffac2563c2fa925bc9247564c1c8929e6dd/dask/array/stats.py#L205-L208

Regarding the output type, it looks like there is a TODO in da.stats that mentions returning a scalar rather than an array when the output has 0 dims. I imagine that is where you'd want to look:
https://github.com/dask/dask/blob/b3707ffac2563c2fa925bc9247564c1c8929e6dd/dask/array/stats.py#L205-L208

Ah nice! Now I just need to figure out how the other methods implemented that (or find another way) such that the tests pass.

As far as I can tell from a brief scan, this looks pretty good.
There are quite a few caveats in your description, so I would certainly encourage adding this information to the docstring, else we are certain to get "results not what I expected" from users.

As far as I can tell from a brief scan, this looks pretty good.
There are quite a few caveats in your description, so I would certainly encourage adding this information to the docstring, else we are certain to get "results not what I expected" from users.

I added a note explaining the two differences to pandas.

@jsignell , I can merge on Monday, if you have nothing more to add.

Thanks for this @jborchma!


Compute image usage

Fixed docopt setup for compute image commands

Options for the image update command were not part of the usage help
Be more consistent what --name is used for
Be more clear and consistent for the --label option

:+1:


[lapack-reference] build failure

Package: lapack-reference[blas-select,core,noblas]:x64-linux -> 3.11.0
Host Environment

Host: x64-linux
Compiler: GNU 12.2.0
vcpkg-tool version: 2022-12-14-7ae0d8527fb488fde10a89c2813802dc9b03b6f9
vcpkg-scripts version: f14984af3 2023-01-09 (4 hours ago)

To Reproduce
vcpkg install armadillo
Failure logs
-- Using cached Reference-LAPACK-lapack-v3.11.0.tar.gz.
-- Cleaning sources at /home/eugene/tools/vcpkg/buildtrees/lapack-reference/src/v3.11.0-5e6de67bfb.clean. Use --editable to skip cleaning for the packages you specify.
-- Extracting source /home/eugene/tools/vcpkg/downloads/Reference-LAPACK-lapack-v3.11.0.tar.gz
-- Applying patch cmake-config.patch
-- Applying patch lapacke.patch
-- Using source at /home/eugene/tools/vcpkg/buildtrees/lapack-reference/src/v3.11.0-5e6de67bfb.clean
-- The Fortran compiler identification is unknown
CMake Error at scripts/cmake/vcpkg_find_fortran.cmake:120 (message):
  Unable to find a Fortran compiler using 'CMakeDetermineFortranCompiler'.
  Please install one (e.g.  gfortran) and make it available on the PATH!
Call Stack (most recent call first):
  ports/lapack-reference/portfile.cmake:53 (vcpkg_find_fortran)
  scripts/ports.cmake:147 (include)

Additional context
Seems to fail finding fortran compiler. Installing this package as a dependency to armadillo.

Installing gcc-fortran locally makes the installation proceed, but still, as I understand, the package should bring portable compiler with it.

the package should bring portable compiler with it.

No it is your task to provide the compilers to build a package :)

Wait, really?

Yes, you also have to provide the c/c++ compiler yourself

@autoantwort  Okay, thanks for the info! Windows build finished without the need for compilers (maybe it used binaries?), so I thought the same would be on Linux...

Should I close the issue then? Everything works for me now.

Should I close the issue then?

Imho yes :)


Lots of icons will disappear in Windows High Contrast Black mode/ White mode  (Accessiblity issue)

Detailed description

Describe in detail the issue you're having.
Lots of icons will disappear in Windows High Contrast Black mode or Windows High Contrast White mode
Is this a feature request (new component, new icon), a bug, or a general issue?
Bug
Is this issue related to a specific component?
No
What browser are you working in?
FireFox
What version of the Carbon Design System are you using?
"4.27.7"

Steps to reproduce the issue
In Windows OS,  change the theme to Windows High Contrast Black mode or Windows High Contrast White mode
Launch a Web APP that using Carbon-Angular
Additional information

Screenshots or code
High Contrast White mode issues:

High Contrast Black mode issues:

Notes

@cal-smith , could you please help to investigate this issue?

It looks like there's a new CCS Media queries that are supposed to be dealing with high contrast: https://developer.mozilla.org/en-US/docs/Web/CSS/@media/forced-colors
I did try it in FF 83 but it doesn't seem to be working

A search query for related issues in Carbon, there's quite a few unfortunately:
https://github.com/carbon-design-system/carbon/issues?q=is%3Aissue+is%3Aopen+high+contrast

This is something that would need to be resolved at the carbon-components level ... We don't control the styling of the icons at all.


fix: Support optional scm host domain for ASE

Description
Added field to the publish profile for customers with ASE deployments to use to set the custom ASE SCM host domain.
Task Item
fixes: #8400
Screenshots

Coverage decreased (-0.002%) to 55.758% when pulling db7094b717dc092068f15ed1b29d21f7c478e48d on gcox/custom-scm-url into 6761f1615ddd589b7d814b2515ec1d3b43fc88a2 on main.


Alaskafixd

okay it's two lines of code and a comment.
Example api call to edit and return image is below:
(defn test-api []
  (let [response (api/create-image-edit {:image (io/file "")
                                         :prompt "Make it cuter"
                                         :n 1
                                         :size "1024x1024"}
                                        {:api-key})]
    (println "Response:")
    (clojure.pprint/pprint response)

    ;; Extract the URL from the response
    (let [url (get-in response [:data 0 :url])]
      (println "Image URL:" url)

      ;; Save the image locally
      (let [image-response (client/get url {:as :byte-array})]
        (with-open [out (io/output-stream "output.png")]
          (.write out (:body image-response)))
        (println "Image saved as output.png")))))

Reference for the api for clojure openai here:
https://cljdoc.org/d/net.clojars.wkok/openai-clojure/0.18.1/api/wkok.openai-clojure.api#create-image-edit

lgtm. that file cant get shittier lol


Issue with music playing, possibly due to Callbacks no-longer being supported.

Hello all,
I would just like to let you know I am aware of an issue with music playing, possibly due to Callbacks no-longer being supported.
I will work on an immediate fix for this and will update as soon as I have fixed this issue.

I believe this issue is fixed, I am still currently monitoring so please let me know if this issue happens again.


Negentropy [WIP]

A few questions here:

do I need to cut a release on the go-negentropy lib? I feel like there's something weird with the versioning
currently we read all events into memory "Vector" to do the syncing, then push them all back to db. It seems more efficient to make the graviton db conform to the expected interface here: https://github.com/illuzen/go-negentropy/blob/main/base.go but it's not obvious how to implement some of the methods
is my event deserialize method really necessary here?

@f7f376a1fcd0d0e11a10ed1b6577c9 could you comment?
TODO:

use syncing method to sync repo roots (?)
decide and implement trigger conditions for both kinds of sync

@illuzen
Do you mind explaining what you mean by "I feel like there's something weird with the versioning" as I'm not sure what you're referring to.
I see what you mean with the deserialize event method and making graviton conform to the storage interface for negentropy, realistically we can add some additional functions for taking the event into the database as raw bytes but verifying that the event is correct is important so we are probably going to end up deserializing it and verifying them all anyway. If more data is required to be stored for the negentropy sync then it's probably best we cache it as I would think this additional data is always generated from the event data itself anyway.
Let me know if I have misunderstood anything.

the go.mod has this line
github.com/illuzen/go-negentropy v0.0.0-20240505133407-a298222fce41
it just looks kind weird, the other imports don't have long hex strings after the semver

Ah yeah I wouldn't worry about that at the moment, probably a way to specify the actual version so it doesn't just v0.0.0 with the time stamp and potentially the commit id?
It's been on my list of things to look into.

ok @f7f376a1fcd0d0e11a10ed1b6577c9 , made some serious progress, the main thing left to do is upload and download raw events that are identified as being missing. There are two TODOs in the code for this. Can you advise on how to do this using existing machinery if possible? Probably should use the same mechanism clients use to upload events...
also, I was thinking, should we make git commits a special kind of event? create a NIP for it? maybe you've already discussed this

Ok @f7f376a1fcd0d0e11a10ed1b6577c9 I think this is done, the test passes.

@Maphikza maybe you can review and merge? Seems like Adam is busy

I'm adding dht code + tests, fyi

waiting for review from @f7f376a1fcd0d0e11a10ed1b6577c9


Minimum depth of binary tree using c++. (Leetcode)

Given a binary tree, find its minimum depth.
The minimum depth is the number of nodes along the shortest path from the root node down to the nearest leaf node.
Note: A leaf is a node with no children.

Please merge this request for hacktober fest.. Thank you in advance


zulip_bots, zulip_botserver: Add PEP 561 type annotation markers

Given that we can't usefully use typing at runtime on Python 2.7 anyway, I guess this is a reasonable solution.

Oh this is still supposed to support Python 2.7? We do have typing imports at top level, so we should put back 'typing;python_version<"3.5"' in that case.


Update en_GB.yml

SELECT_NO_PARENT is missing from the new localisation.

Thanks, but it should be fixed in other way.


Remove "private" static members

We have a lot of things like Foo.bar_.  If a property is private, it should not be exported.  So instead of tacking these properties onto the constructor, they can be declared separately and not exported.
It looks like these modules need work (based on grep -E '(^|\s)[A-Z]\w+\.\w+_ =' -lr src):

[ ] ol/control/FullScreen.js
[ ] ol/control/MousePosition.js
[ ] ol/control/ScaleLine.js
[ ] ol/control/ZoomSlider.js
[ ] ol/format/EsriJSON.js
[ ] ol/format/GML2.js
[ ] ol/format/GML3.js
[ ] ol/format/GMLBase.js
[ ] ol/format/GPX.js
[ ] ol/format/KML.js
[ ] ol/format/MVT.js
[ ] ol/format/OSMXML.js
[ ] ol/format/OWS.js
[ ] ol/format/WFS.js
[ ] ol/format/WKT.js
[ ] ol/format/WMSCapabilities.js
[ ] ol/format/WMSGetFeatureInfo.js
[ ] ol/format/WMTSCapabilities.js
[ ] ol/geom/GeometryCollection.js
[ ] ol/interaction/DragAndDrop.js
[ ] ol/interaction/DragBox.js
[ ] ol/interaction/DragPan.js
[ ] ol/interaction/DragRotate.js
[ ] ol/interaction/DragRotateAndZoom.js
[ ] ol/interaction/Draw.js
[ ] ol/interaction/Extent.js
[ ] ol/interaction/Modify.js
[ ] ol/interaction/MouseWheelZoom.js
[ ] ol/interaction/PinchRotate.js
[ ] ol/interaction/PinchZoom.js
[ ] ol/interaction/Select.js
[ ] ol/interaction/Snap.js
[ ] ol/interaction/Translate.js
[ ] ol/Object.js
[ ] ol/render/canvas/ReplayGroup.js
[ ] ol/render/webgl/LineStringReplay.js
[ ] ol/render/webgl/ReplayGroup.js
[ ] ol/renderer/Map.js
[ ] ol/source/Image.js
[ ] ol/source/ImageWMS.js
[ ] ol/source/Raster.js
[ ] ol/source/TileDebug.js
[ ] ol/source/TileUTFGrid.js
[ ] ol/source/Zoomify.js
[ ] ol/style/Style.js
[ ] ol/style/Text.js
[ ] ol/tilegrid/TileGrid.js
[ ] ol/View.js
[ ] ol/webgl/Context.js

I'll work on the controls now.


Radial gradient disappears or renders incorrectly in html5 target

When a shape created by code as a graphics or obtained as swflite asset uses a radial gradient with a transform matrix that has zero for a and d values, it looks like the shape is not displayed at all.
package;

import openfl.display.GradientType;
import openfl.display.Sprite;
import openfl.display.StageAlign;
import openfl.display.StageScaleMode;
import openfl.geom.Matrix;

class Main extends Sprite {

	private static inline var SIZE = 200;
	private static inline var GAP = 30;
	private var mPosX = GAP;
	private var mPosY = GAP;

	public function new() {
		super();

		stage.scaleMode = StageScaleMode.NO_SCALE;
		stage.align = StageAlign.TOP_LEFT;

		// works in flash and html5
		drawGradient(GradientType.RADIAL, new Matrix(0.25, 0, 0, 0.25, SIZE / 2, SIZE / 2));

		// works well in flash but disappears in html5
		drawGradient(GradientType.RADIAL, new Matrix(0, 0.25, 0.25, 0, SIZE / 2, SIZE / 2));

		mPosX = GAP;
		mPosY += SIZE + GAP;

		// both linear gradients work in flash and html5
		drawGradient(GradientType.LINEAR, new Matrix(0.25, 0, 0, 0.25, SIZE / 2, SIZE / 2));

		drawGradient(GradientType.LINEAR, new Matrix(0, 0.25, 0.25, 0, SIZE / 2, SIZE / 2));
	}

	private function drawGradient(type:GradientType, matrix:Matrix) {
		var spr = new Sprite();
		spr.x = mPosX;
		spr.y = mPosY;
		this.addChild(spr);
		spr.graphics.beginGradientFill(type, [0xCCDDEE, 0x334455], [1, 1], [0, 255], matrix);
		spr.graphics.drawRect(0, 0, SIZE, SIZE);
		mPosX += SIZE + GAP;
	}

}

This code should show 4 squares with different gradient types. But when I target html5 the second radial gradient with the "rotated" matrix disappears.

Project file:
<?xml version="1.0" encoding="utf-8"?>
<project>
	<meta title="openfl-gradient-test" package="openfl-gradient-test" version="1.0.0" company="meps" />
	<app main="Main" file="openfl-gradient-test" path="bin" />
	<window background="#CCCCCC" fps="60" />
	<window orientation="auto" vsync="false" antialiasing="0" />
	<source path="src" />
	<haxelib name="openfl" />
	<haxeflag name="-dce full" />
</project>

Libraries versions:
lime: 7.8.0
openfl: 9.0.2

I ran into even more incorrect gradient behavior. Making the radial gradient elliptical (with different scales along the axes) does not affect the target html5 rendering, the gradient is rendered at a large scale factor (a) and still disappears when it is set to zero.
		// shows elliptical gradient in flash that became radial with larger scale in html5
		drawGradient(GradientType.RADIAL, new Matrix(0.5, 0, 0, 0.125, SIZE / 2, SIZE / 2));

		// shows elliptical gradient in flash but still disappears in html5
		drawGradient(GradientType.RADIAL, new Matrix(0, 0.5, 0.125, 0, SIZE / 2, SIZE / 2));

And even more:
		// both shows skewed elliptical gradient in flash
		// that became radial with scaling from (a) matrix coefficient in html5
		drawGradient(GradientType.RADIAL, new Matrix(0.5, 0.1, 0, 0.125, SIZE / 2, SIZE / 2));
		drawGradient(GradientType.RADIAL, new Matrix(0.1, 0.5, 0.125, 0, SIZE / 2, SIZE / 2));

Looks like radial gradient in html5 is getting both scales rom (a) coefficient and ignores (d) and both skews (b, c).

I've found that used in OpenFL method cairo_pattern_create_radial of the external Cairo Graphics library is absolutely incompatible with flash radial gradient format. So OpenFL creating quick and dirty concentric gradients that lost all transformation matrix data. And there are not even TODO or FIXME hints at the code or documentation!

Cairo is the native software renderer, not html5. I'll take a look at this.

Cairo is the native renderer, not html5. I'll take a look at this.

Whatever, there is the same implementation at createGradientPattern.

I've done small workaround for radial gradient in canvas:
<!DOCTYPE html><html><body>
<canvas id="myCanvas" width="500" height="500"/>
<script>

let SIZE = 200;
let c = document.getElementById("myCanvas");
let ctx = c.getContext("2d");
let grd = ctx.createRadialGradient(0, 0, 0, 0, 0, SIZE * 2);
grd.addColorStop(0, "#CCDDEE");
grd.addColorStop(1, "#334455");

ctx.save();
ctx.fillStyle = grd;
ctx.rect(0, 0, SIZE, SIZE);
ctx.setTransform(0.5, 0.1, 0, 0.25, SIZE / 2, SIZE / 2);
ctx.fill();
ctx.restore();

</script>
</body></html>

I believe it helps a little.


MyLocationNewOverlay drawable not anchored correctly

Please note: issues related to or caused by osmbonuspack will be closed with a
suggestion to open an issue at https://github.com/MKergall/osmbonuspack
(feel free to delete whatever doesn't apply)
Issue Type
[ ] Question
[x] Bug
[ ] Improvement
[ ] Build system related
[ ] Performance
[ ] Documentation
Description and/or steps/code to reproduce the problem
I am not 100% sure, but it seems that the drawables of current location are not correctly anchored. I'd expect that person should be anchored in bottom center and arrow in center. Btw the current location seems to be positioned correctly in the center of MapView. Please check it.
Environment
If it's a bug, version(s) of android this affects:
All
Version of osmdroid the issue relates to:
6.1.0

can you provide code and/or steps to reproduce?

I am providing two screen shots. On the first one there is my location
correctly (responding the reality) in the center given by the cross. On the
second one there is the same location with the person icon, where is
visible that the person is not positioned correctly..
Corresponding code:
mMyLocationOverlay = new MyLocationNewOverlay(mMapView);
BitmapDrawable stop = (BitmapDrawable)
ResourcesCompat.getDrawable(getResources(), R.drawable.location_stop,
null);
BitmapDrawable arrow = (BitmapDrawable)
ResourcesCompat.getDrawable(getResources(), R.drawable.location_arrow,
null);
mMyLocationOverlay.setDirectionArrow(stop.getBitmap(), arrow.getBitmap());
mMyLocationOverlay.setDrawAccuracyEnabled(true);
mMyLocationOverlay.runOnFirstFix(() -> {
TrackUtils.playTone(ToneGenerator.TONE_CDMA_EMERGENCY_RINGBACK);
GeoPoint myLocation = mMyLocationOverlay.getMyLocation();
if (myLocation != null) {
runOnUiThread(() -> mMapView.getController().animateTo(myLocation));
}
});
}
On Sat, Dec 14, 2019 at 7:33 PM spyhunter99<EMAIL_ADDRESS>wrote:

can you provide code and/or steps to reproduce?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/osmdroid/osmdroid/issues/1360?email_source=notifications&email_token=ACIONDOKK4FUBOUOLJZ2YUDQYURHDA5CNFSM4HYTNR3KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEG4IMIQ#issuecomment-565741090,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACIONDMT7FVGR26VALRRO2LQYURHDANCNFSM4HYTNR3A
.

the screenshots:

try calling setPersonHotspot....that's the anchor point for the icon. i.e. the position on the marker that you want to represent the location on the map

not very clear how to set the x, y params, probably a complex calculation
needed according to icon resolution and map scale
couldn't it be resolved somehow defaultly, because it must be common
problem regardless the customized icons
On Tue, Dec 17, 2019 at 3:32 PM spyhunter99<EMAIL_ADDRESS>wrote:

try calling setPersonHotspot....that's the anchor point for the icon. i.e.
the position on the marker that you want to represent the location on the
map
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/osmdroid/osmdroid/issues/1360?email_source=notifications&email_token=ACIONDK5PNZJKMJ5KO4F3CDQZDPGTA5CNFSM4HYTNR3KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEHCRZFY#issuecomment-566566039,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACIONDIJVYPDISJTD7UOMYDQZDPGTANCNFSM4HYTNR3A
.

@mlendl Indeed, line 113 of MyLocationNewOverlay stinks:
mPersonHotspot = new PointF(24.0f * mScale + 0.5f, 39.0f * mScale + 0.5f);

It should be more like what happens in Marker, where the anchor has a horizontal and a vertical [0,1] factor, to be multiplied by the icon's width and height.

sorry guys, still not clear the magic numbers, but probably I would be able to implement some correction, but I am not sure if it helps, because the icon is positioned completly out of the location gray circle, I think the solution should be within Osmdroid, because the problem is probably general concernig all users

@mlendl I've just PR'ed #1460 - can you try again with this version?

sorry I don't know to checkout the branch, but actually I have customized
only the direction icon and everything's OK
On Mon, Dec 23, 2019 at 11:43 AM monsieurtanuki<EMAIL_ADDRESS>wrote:

@mlendl https://github.com/mlendl I've just PR'ed #1460
https://github.com/osmdroid/osmdroid/pull/1460 - can you try again with
this version?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/osmdroid/osmdroid/issues/1360?email_source=notifications&email_token=ACIONDMBV3FQQONMWXN237LQ2CI4FA5CNFSM4HYTNR3KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEHQ25FQ#issuecomment-568438422,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACIONDOFTSPRXGJKM2WX3MTQ2CI4FANCNFSM4HYTNR3A
.

Not sure if it's the same issue but when I use a custom drawable (Google Maps-like circle) with setPersonIcon(), the position of the icon changes in relation to the map when I zoom out placing the user in the incorrect location. This does not happen when I use the default yellow guy.

I had exactly the same observation, this is the same issue.

@zamojski @AndroidPat Does #1460 fix your issue? Even without connecting to that branch and before a merge and a release, you can just duplicate and rename class MyLocationNewOverlay (from #1460) and test it in your code.
I hope this works. Please tell us if it does.

@monsieurtanuki  I think it does what we need. The change in commit 53bd75267b368f18971b4dc6e25eea83987b3679 looks similar to what I have in re-implemented setDirectionArrow method. I've also checked the class itself in my project as you suggested and it looks ok.

@spyhunter99 I guess #1460 could be part of the next release.

i had the same issue.
        myLocationOverlay.setPersonAnchor(.5f,.5f);
adding this after setPersonIcon() function call, fixed the problem.
in the documentation it states
_

Anchors for the person icon Expected values between 0 and 1, 0 being top/left, .5 center and 1 bottom/right
Since:
6.2.0

_

Yes, order of calling methods is important:
val mLocationOverlay = MyLocationNewOverlay(GpsMyLocationProvider(context), binding.mapView)
mLocationOverlay.setPersonIcon(userIcon.toBitmap())
mLocationOverlay.setPersonAnchor(Marker.ANCHOR_CENTER, Marker.ANCHOR_CENTER)
mLocationOverlay.setDirectionIcon(userArrow.toBitmap())
mLocationOverlay.setDirectionAnchor(Marker.ANCHOR_CENTER, Marker.ANCHOR_CENTER)
binding.mapView.overlays.add(mLocationOverlay)


Support getting bound local port from Application and avoid using fixed port in unit tests

https://github.com/confluentinc/schema-registry/pull/758 and https://github.com/confluentinc/kafka-rest/pull/414 are downstream changes that will rely on this being merged first.

LGTM


student not found

The problem
Trying to add a new student. Error with "student not found". Log in on webuntis.com is working though. When I type a wrong password on purpose it says wrong username and/or password. So I guess the connection is working.
Version of Home Assistant?
11.4
What type of installation are you running?
Home Assistant OS
Version of WebUntis
v1.04
Last working WebUntis version
No response
Traceback/Error logs
No log entry for the error message: student not found.
Additional information
No response

Try entering the name in different ways, such as "First_name, Last_name" or "Last_name, First_name." Normally, the name displayed on the WebUntis app should work.
If that doesn't resolve the issue, attempt to add the timetable by class, as described here:
https://github.com/JonasJoKuJonas/homeassistant-WebUntis/issues/92#issuecomment-1791731970

Thank you Jonas. Works fine. Two kids on the same school but different settings. Weird.


Razor Pages authorization conventions topic

Addresses #4281
Internal review topic
Notes

Covers:

AllowAnonymousToPage
AllowAnonymousToFolder
AuthorizePage
AuthorizeFolder
No go for requiring auth on a page inside a folder with anonymous access allowed

The Lenny Nero version ... i.e. ... The methods are fairly straightforward to understand and explain, so I was economical with the language used.
Razor Pages sample follows the template fairly closely. The most significant delta is using cookie auth without identity (imo full Identity is overkill just to show a few RP features). However, the sample README calls it out clearly.

@guardrex,
Thanks for having already signed the Contribution License Agreement. Your agreement was validated by .NET Foundation. We will now review your pull request.
Thanks,
.NET Foundation Pull Request Bot

@guardrex  is this ready for review?

Yes, ready for a look. :eyes:

@scottaddie kk ... ready for :eyes:
We're moving ...

AddPageRoute
WithRazorPagesAtContentRoot
WithRazorPagesRoot

... out. It leaves this topic auth convention-focused.
cc/ @pranavkm Note that if we go with Razor Pages authorization conventions in ASP.NET Core, it's a little bit constraining for later to have the word "conventions" in there (i.e., making auth attributes work with page models and later having some kind of feature to implement the filters with something in the .cshtml files). Are u sure we shouldn't just go with Razor Pages authorization in ASP.NET Core? I could even add a section at the bottom of this that explains that the [Authorize] and [AllowAnonymous] attributes can be added to the page model today and that the .cshtml feature is  in-the-works.

Thanks @pranavkm ... I corrected the first one, too ...
.AuthorizeFolder("/Private").AllowAnonymousToPage("/Private/Public")

Thx @scottaddie Changes made.
Although most CSS minifiers clean up the excess semi-colons in the CSS file, I went ahead and made them consistent.

@guardrex,
Thanks for having already signed the Contribution License Agreement. Your agreement was validated by .NET Foundation. We will now review your pull request.
Thanks,
.NET Foundation Pull Request Bot

Stand-by ... I'll resolve these bad boys.

@guardrex,
Thanks for having already signed the Contribution License Agreement. Your agreement was validated by .NET Foundation. We will now review your pull request.
Thanks,
.NET Foundation Pull Request Bot

🦎 Green Dinosaur: 2
:beetle: Bugs: 0
@scottaddie If that last commit looks good, this one is ready to :guitar:.

@Rick-Anderson @guardrex I think the security/authorization section of the TOC should also contain a link to this doc. Thoughts?

Other possibilities would be ...

Add it to the topic links in the security/authorization/index page.
Create a very short topic there analogous to the security/authorization/views.md topic called pages.md, or perhaps even better ...
Change the title of that one to Page and View authorization and make the RP bits just a section there. Link that out to this topic. I could that in short order on this PR.

I think the security/authorization section of the TOC should also contain a link to this doc.

@danroth27  see
Internal review topic
For sure it needs to go under Security/Authorization, but think having in also in MVC/Razor Pages is inconsistent.

Since this is an authorization specific topic I lean towards putting it under Security/Authorization instead of under the Razor pages node. If we decide we also want to duplicate security related content for MVC and Razor pages under the MVC node, we will need to do that in a holistic fashion.

Thx ... I'll take care of it shortly.

@danroth27 Lick'd it. 😋

I shortened the uid by chopping "conventions" get us down to "security/authorization/razor-pages-authorization" and under the 59 char limit.
I added a couple of topic cross-links to make sure visibility is good when landing into either area of the docs (RP or security).
I positioned this at the bottom of the security/authorization TOC entries because the rest (or most) are MVC-specific. Otherwise, let me know if you prefer it in the 2nd spot under the Introduction.

I positioned this at the bottom of the security/authorization TOC entries because the rest (or most) are MVC-specific. Otherwise, let me know if you prefer it in the 2nd spot under the Introduction.

I'd say 2nd or better yet 3rd. I'll be updating user data to Razor Pages.
@danroth27  @scottaddie  cast your vote.

I went ahead b/c I wager that they'll agree ... easy enough to move if not tho.

My vote is for putting this TOC link immediately below "Introduction", since the team is really trying to promote Razor Pages.

If @Rick-Anderson converts the user data topic (the 2nd one) to RP, then the top two will be RP topics.

@scottaddie @Rick-Anderson ... are we good here? If this is ok in the 2nd TOC slot under Authorization, then it's ready to :rocket:.


Error after identifying connected components

Hi! I have an error after the step "identifying connected components".
I installed SuperPang using conda and I am working with 4 bins (>90% compl <5% contamination) from the same species (ANI > 97%). test-SuperPang.py runs with out problems.
The error occurs after running
$SuperPang.py -f genomes/*fa -t 20 -o SuperPang --force-overwrite --checkm checkm_sox.txt
Traceback (most recent call last):
File "/home/jcifuentes/miniconda3/envs/SuperPang/bin/SuperPang.py", line 291, in
main(parse_args())
File "/home/jcifuentes/miniconda3/envs/SuperPang/bin/SuperPang.py", line 144, in main
contigs = Assembler(input_minimap2, args.ksize, args.threads).run(args.minlen, args.mincov, args.bubble_identity_threshold, args.genome_assignment_threshold, args.threads)
File "/home/jcifuentes/miniconda3/envs/SuperPang/lib/python3.8/site-packages/SuperPang/lib/Assembler.py", line 322, in run
psets = get_psets(comp2nvs[nc1] | comp2nvs[nc2])
File "/home/jcifuentes/miniconda3/envs/SuperPang/lib/python3.8/site-packages/SuperPang/lib/Assembler.py", line 292, in get_psets
assert len(nvs_) == 2
AssertionError
The checkM file has the format Bin_Id Completeness Contamination, but I have the same error after running SuperPang with the --assume-complete flag.

Could you share the input bins with me here in a zip file?

I'm sorry, but I can't share them. Is there any other information I can provide?
I tried running it with another set of bins (5 bins medium- to high-quality >95% ANI) and I have the same error.

Hm, I wonder what the problem may be. Does test-SuperPang.py work for you?
Are these "standard" bacterial/archaeal genomes? something "strange" about them?
I have an idea of a couple of things that might be throwing that error, but it's hard to know what's really happening without being able to reproduce it myself (of course it is understandable if you can't share your data).

Let's test a thing
Edit the
/home/jcifuentes/miniconda3/envs/SuperPang/lib/python3.8/site-packages/SuperPang/lib/Assembler.py
In line 292 change
assert len(nvs_) == 2
for
if len(nvs_) > 2: print(nvs); assert False
This will still die but will print some info about what's going wrong. You can share the output (a horrible bunch of numbers) with me.

I only obtained this output after the "Extracting forward component"
[0, 1, 2, 3]
I changed the order of the bins and SuperPang run until comp 246/718 and then it fails also after:
Extracting forward component
[0, 1, 2, 3]
Traceback (most recent call last):
File "/home/jcifuentes/miniconda3/envs/SuperPang/bin/SuperPang.py", line 291, in
main(parse_args())
File "/home/jcifuentes/miniconda3/envs/SuperPang/bin/SuperPang.py", line 144, in main
contigs = Assembler(input_minimap2, args.ksize, args.threads).run(args.minlen, args.mincov, args.bubble_identity_threshold, args.genome_assignment_threshold, args.threads)
File "/home/jcifuentes/miniconda3/envs/SuperPang/lib/python3.8/site-packages/SuperPang/lib/Assembler.py", line 322, in run
psets = get_psets(comp2nvs[nc1] | comp2nvs[nc2])
File "/home/jcifuentes/miniconda3/envs/SuperPang/lib/python3.8/site-packages/SuperPang/lib/Assembler.py", line 292, in get_psets
if len(nvs_) != 2: print(nvs_); assert False
AssertionError
This is how it look lines 291-293
 for nvs_ in psets.values():
    if len(nvs_) != 2: print(nvs_); assert False
 return psets

Thanks for trying this. It seems like a different case of a bug I'd fixed before, but I would need more info
Can you now try changing the whole get_psets function to this one?
            ### Extract forward component
            def get_psets(nvs):
                """Get a dict with a frozenset of vertices as keys, and the two complementary non-branching paths sharing those vertices as values"""
                psets = defaultdict(list)
                for nv in nvs:
                    pset = frozenset(vertex2NBP[nv])
                    psets[pset].append(nv)
                for fs, nvs_ in psets.items():
                    if len(nvs_) != 2:
                        print('\nERROR!', len(nvs_), fs)
                        print()
                        for nv_ in nvs_:
                            print(nv_, vertex2NBP[nv_])
                        print('\nDONE\n')
                    assert len(nvs_) == 2
                return psets

This is the output after modifiying Assembler.py
SuperPang_error_cifuj_20220407.txt

Thanks! This gives me something to work with... I'll come back to you if I need some extra stuff

Ok, try replacing the function with this
            ### Extract forward component
            def get_psets(nvs):
                """Get a dict with a frozenset of vertices as keys, and the two complementary non-branching paths sharing those vertices as values"""
                psets = defaultdict(list)
                for nv in nvs:
                    pset = frozenset(vertex2NBP[nv])
                    psets[pset].append(nv)
                for ps, nvs_ in psets.items():
                    if len(nvs_) > 2: # If more than two NBPs share the same pset, select the first two that are RC of each other
                        for nv1, nv2 in combinations(nvs_): # this is a temporary patch, I should look into why this happens when I get a proper example
                            NBP1, NBP2 = vertex2NBP[nv1], vertex2NBP[nv2]
                            if NBP1 == NBP2[::-1]:
                                psets[ps] = [nv1, nv2]
                                break
                        else:
                            assert False # if there is no RC pair, die anyways
                for nvs_ in psets.values():
                    assert len(nvs_) == 2
                return psets

Hopefully this'll work

I obtained the following error.
Traceback (most recent call last):
File "/home/jcifuentes/miniconda3/envs/SuperPang/bin/SuperPang.py", line 291, in
main(parse_args())
File "/home/jcifuentes/miniconda3/envs/SuperPang/bin/SuperPang.py", line 144, in main
contigs = Assembler(input_minimap2, args.ksize, args.threads).run(args.minlen, args.mincov, args.bubble_identity_threshold, args.genome_assignment_threshold, args.threads)
File "/home/jcifuentes/miniconda3/envs/SuperPang/lib/python3.8/site-packages/SuperPang/lib/Assembler.py", line 331, in run
psets = get_psets(comp2nvs[nc1] | comp2nvs[nc2])
File "/home/jcifuentes/miniconda3/envs/SuperPang/lib/python3.8/site-packages/SuperPang/lib/Assembler.py", line 293, in get_psets
for nv1, nv2 in combinations(nvs_): # this is a temporary patch, I should look into why this happens when I get a proper example
TypeError: combinations() missing required argument 'r' (pos 2)
I tried with 2 different set of bins from 2 different species and I obtained the same error after a different number of "Working on comp XX/YY"

My fault, was missing an argument
            ### Extract forward component
            def get_psets(nvs):
                """Get a dict with a frozenset of vertices as keys, and the two complementary non-branching paths sharing those vertices as values"""
                psets = defaultdict(list)
                for nv in nvs:
                    pset = frozenset(vertex2NBP[nv])
                    psets[pset].append(nv)
                for ps, nvs_ in psets.items():
                    if len(nvs_) > 2: # If more than two NBPs share the same pset, select the first two that are RC of each other
                        for nv1, nv2 in combinations(nvs_, 2): # this is a temporary patch, I should look into why this happens when I get a proper example
                            NBP1, NBP2 = vertex2NBP[nv1], vertex2NBP[nv2]
                            if NBP1 == NBP2[::-1]:
                                psets[ps] = [nv1, nv2]
                                break
                        else:
                            assert False # if there is no RC pair, die anyways
                for nvs_ in psets.values():
                    assert len(nvs_) == 2
                return psets

Hi again!
I managed to reproduce the issue with data from another user. Version 0.9.4beta1 (available in conda) has a temptative fix for this problem. Let me know if it works for you!

Hi! Thank you very much!
It works with the 2 datasets I had problems before.

Glad to hear. Thanks for testing it!


Training your own dataset ....

To train my own dataset, you mentioned that :
 " for each video make a separate folder with all the frames in '.png' format.

Does this mean that I have to create  .png images for each ,mp4 video in train directory?
FrameDataset appears to access only .mp4 videos in train directory (not .png images) ???

You have to create a folder with all png images for each mp4. It accept both, but mp4 will be slower.


MiKo_2082 is now aware of 'None', 'On', 'Off' and 'Undefined'

@CodiumAI-Agent /describe

Title
MiKo_2082 is now aware of 'None', 'On', 'Off' and 'Undefined'

User description
@CodiumAI-Agent /describe

PR Type
Enhancement, Tests

Description

Enhanced the MiKo_2082_CodeFixProvider to handle specific enum member phrases: 'None', 'On', 'Off', and 'Undefined'.
Implemented string replacements to ensure consistent phrasing in documentation comments.
Added new test cases to verify the correct application of these replacements in MiKo_2082_EnumMemberAnalyzerTests.

Changes walkthrough 📝
Relevant filesEnhancement

MiKo_2082_CodeFixProvider.csAdd handling for specific enum member phrases in code fix

MiKo.Analyzer.Shared/Rules/Documentation/MiKo_2082_CodeFixProvider.cs
Added replacements for specific enum member phrases.  Enhanced string manipulation for enum member descriptions.

+4/-0

Tests

MiKo_2082_EnumMemberAnalyzerTests.csAdd tests for enum member phrase replacements

MiKo.Analyzer.Tests/Rules/Documentation/MiKo_2082_EnumMemberAnalyzerTests.cs
Added test cases for new enum member phrase replacements.  Verified code fixes for 'On', 'Off', 'Undefined', and 'None'.

+29/-0

💡 PR-Agent usage: Comment /help "your question" on any pull request to receive relevant information


Enable parallel build

Measurement: gradlew clean check
Current master: 16s (https://scans.gradle.com/s/im3x27mkieat6)
Gradle parallel: 10s (https://scans.gradle.com/s/xbixn2tzo6va4)
I like playing with builds, so I tried to go further:
Move APT classes to modules: 7s (https://scans.gradle.com/s/vutaiobangdts)
Move IntegrationTest next to classes: 6s (https://scans.gradle.com/s/5wiwkyulfkuy2)
The last two steps are not included in this PR, but the commits are here:
https://github.com/TWiStErRob/dagger-reflect/commits/abandoned/parallel
if you think it would be useful, happy to clean it up and raise a PR
Mind you I have a brand new PC with 14 cores, dagger-reflect builds use 4-8 cores only, so YMMV significantly.
Also tried https://guides.gradle.org/performance/#parallel_test_execution, but makes no difference with current amount of tests, compilation is the slowest.
Also tried -Adagger.formatGeneratedSource=disabled but makes no difference

So we can't check this in because it allows the uploadArchives task to run in parallel. Sonatype, the target for the artifacts, sees concurrent uploads as separate and creates two separate staging repositories. This will break the snapshot and release builds as a result. Those tasks need restricted to run serially before something like this can be merged.

Do you mean something like this?
val uploads = allprojects.mapNotNull { it.tasks.uploadArchives }
uploads.zip(uploads.skip(1)) { task1, task2 -> task1.mustRunAfter(task2) }

but Groovy and hopefully using lazy task getters :)
Assuming order of them don't matter, just that they're serial.

That might work. There's some other options from the feature request on Gradle: https://github.com/gradle/gradle/issues/7047#issuecomment-430139316

Speed is not a concern on CI

Agree, the point here was to utilize Gradle features to speed up local development as much as possible. I guess I can try some global override from my home directory, since the uploadArchives is not an issue locally.


METRON-2155 Cache Maven in Travis CI Builds

This PR attempts to resolve the recent spat of failed Travis CI builds. These builds seem to fail when attempting to download Maven from the Apache Mirrors.
This PR attempts to cache Maven so it does not need to be repeatedly downloaded from the Apache Mirrors.  This would mitigate the issue if there are transient problems with Travis connectivity to the Apache Mirrors or if the Apache Mirrors themselves are having sporadic outages.
Pull Request Checklist

[x] Is there a JIRA ticket associated with this PR? If not one needs to be created at Metron Jira.
[x] Does your PR title start with METRON-XXXX where XXXX is the JIRA number you are trying to resolve? Pay particular attention to the hyphen "-" character.
[x] Has your PR been rebased against the latest commit within the target branch (typically master)?

+1 by inspection. Hrm, I don't see a Travis status for this PR. Any idea why that is?

I think Travis is having some operational issues today.  I have noticed when trying to test this change on my personal account that Travis is not pulling in new commits since maybe 5 or 6 hours ago.  Of course, this will not be merged until Travis is healthy and we see some good CI builds


add interactive charts sample

@juliantoledo  PTAL

@kul3r4 PTAL

Please change:
embedd -> embed
wil be using-> will be using
in an mobile apps -> in mobile apps or in a mobile app

Thanks. Fixed it. Any comments regarding the content? Is it clear or anything missing?

Content is clear!
Sorry, I missed one minor comment:
On thing is important -> One thing is important

PTAL


Add FindIndex operator

Part of #109

Codecov Report
Base: 87.81% // Head: 87.96% // Increases project coverage by +0.14% :tada:

Coverage data is based on head (50bbc25) compared to base (916f31e).
Patch coverage: 97.36% of modified lines in pull request are covered.

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #115      +/-   ##
==========================================
+ Coverage   87.81%   87.96%   +0.14%
==========================================
  Files         186      188       +2
  Lines        4925     5001      +76
  Branches     1121     1145      +24
==========================================
+ Hits         4325     4399      +74
- Misses        315      316       +1
- Partials      285      286       +1

Impacted Files
Coverage Δ

Source/SuperLinq.Async/FindIndex.cs
94.73% <94.73%> (ø)

Source/SuperLinq/FindIndex.cs
100.00% <100.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Rewrite in Rust

For the good of the Internet, and in fact humanity, we need to migrate our software from Go to Rust (or something better) as quickly as possible. Here are some steps we need to take:

Use Rust. Using Rust instead of Go makes code safer. Using Rust instead of any other language grows the Rust community, making it easier for other people to use Rust.
Rewrite code in Rust. Starting with our most critical infrastructure, rewrite Go code to use Rust instead.
Extend Rust's guarantees. We can extend the classes of bugs that Rust prevents. For example, we should try to make Rust release builds check for integer overflow by default.
Verify "unsafe" Rust code. Sometimes the Rust type system is not strong enough to let you prove to the compiler that your code is safe, so you have to mark code blocks "unsafe". With tool support, you could instead generate a mathematical proof that your code is safe, checked by the compiler.

I wouldn't be opposed to a Rust-themed fork of this project, however at this time I'm only interested in personally providing a solution for golang developing gophers around the world.
If you would like, I'm happy to consult on the project as a thought-leading architectural expert on the use of generics for small fee.
Thanks for interest in the project!


Connect to PC running Cyber outside local segment

When referring to #7050, I came across a note saying that the IP of two systems must be in the same local segment.
However, is it possible to connect to an IP which is just outside the local segment, for example connecting a <IP_ADDRESS> to <IP_ADDRESS>. Is there any changes that I can do to the code so that the pc outside the local segment can be discovered as well?
I've also confirmed that both tcp and udp communication can be established using netstat.

If you want to support service auto-discovery, it must be on the same network segment, or you can set the subnet mask and try


Tutorial + docs needs to be updated for removal of .forget()

At the moment, there is no clear reference (that I could find) to explain how to go about implementing the functionality of .forget().  In addition, the Tutorial still has references to .forget() in it although they have been removed from the documentation.
EDIT: I read in other issues that Executors seem to be the new accepted way of consuming a future in the background.  However, their documentation is rather sparse and there are no examples showing how to go about utilizing that functionality.

Indeed! At this point unfortunately the tutorial basically needs to be rewritten probably from scratch :(

Ok, updated!

Woohoo!! :D


Epson ET-7700 does not support image/pwg-raster

Hi,
I have report in Fedora  - https://bugzilla.redhat.com/show_bug.cgi?id=1756726 - which is about Epson ET-7700 reports bad document format when image/pwg-raster format is used.
It seems the model does not support pwg-raster, but image/urf format works fine.
The patch is about crossing out that specific model from pwg-raster document format, so output ppd does not have the line for pwg-raster.
Would you mind adding it to the project?

Thank you very much for your patch.
This patch is at best a temporary solution, but not what should be permanent part of cups-filters. It works around a firmware bug of one single printer model (the bug being either the printer reporting it does PWG Raster but PWG Raster support not implemented in the printer, or the printer's PWG Raster support simply being buggy).
As there can be more printers with firmware bugs it would be more useful to have a quirk rule file, like CUPS has for the USB communication with USB printers:
/usr/share/cups/usb/*.quirks

cups-filters should also get quirk rule files, one for IPP firmware bugs, telling for each buggy printer model which advertised printer features are not usable due to a bug.
Quirk rule files give a nice overview of firmware bugs and workarounds, and they are text files so a sys admin, an experienced user, or a reporter of a bug guided by a developer/package maintainer can add rules and try out what makes his printer working again. In case of success one only needs to report an issue or pull request here to get the new rule included.

Ok, that sounds good, I'll rewrite it into the way similar to CUPS quirks.
Do you have any suggestion about structure of quirk file structure?
I would propose the scheme "manufacturer  - model - quirk", so for our case would it be:
Epson - ET-7700 Series - no-pwg-raster
The quirks will be loaded at the start of ppdCreateFromIPP2() function, and they will be checked only when there will be image/pwg-raster in document-format-supported attribute.
I would not go into defining other feature names or putting the quirk check into other parts of code unless we find other firmware bug.

I suggest something like this for the quirk file:
make model-regexp wrong-attr attr=value attr=value1,value2,... ...
make model-regexp get-printer-attributes arg1 arg2 ...

The first form is to tell about wrong attributes in the response to the get-printer-attributes request, in our case it would be image/pwg-raster under document-format-supported so the quirk rule line would be
"Epson" "ET-7700 Series" wrong-attr document-format-supported=image/pwg-raster

before generating the PPD these would be "substracted" from the attribute in the get-printer-attributes IPP response, so in this case the PPD would be built based on the other listed formats.
This syntax would also allow to describe broken arguments in more than one attribute. Imagine our printer reports to do 1200 dpi and 2400 dpi but trying to print in these resolutions gives broken output. So you extend the line to
"Epson" "ET-7700 Series" wrong-attr document-format-supported=image/pwg-raster printer-resolution-supported=1200dpi,2400dpi

If a printer misbehaves (or even crashes) on the standard get-printer-attributes IPP request with the arguments "all" and "media-col-database" one could use the second form to let the get-printer-attributes IPP request be done with other arguments. For example
"Canon" ".*" get-printer-attributes all

would let the request be done with only the "all" argument on all Canon printers.
More than one line should be allowed for one and the same printer, to allow both forms for the same printer if needed and also for example to tell about a wrong argument on a series of printers of one manufacturer and another wrong argument on one individual printer in the series. So after a line with matches manufacturer and model the rest of the quirk file should be read to find further matches.
I think this should cover most possible firmware bugs when requesting printer capabilities via the get-printer-attributes IPP request.

Also the Epson WF-2630 shows this problem. See issue #171.

The third printer known to show this problem is the Canon Pixma Pro 10, according to apple/cups#5238.

As the only known quirk in driverless printing is PWG Raster support reported by printers which are not able to actually print it, we follow CUPS as in apple/cups#5238 for now letting the generated PPD make printing done in Apple Raster when both PWG Raster and Apple Raster are supported.
This way we can get away without quirk rule file for now, as this would also be somewhat against the principles of driverless printing. Therefore I am closing this issue for now.
@zdohnal, thank you anyway for offering to add quirk rule support.

By the way, the fix is commit 80f87b0a.

Ok, thank you for the info.

FYI, same issue with Epson L3160 (perhaps common for all EcoTank's?). BTW, the CUPS debug showed
D [04/Jan/2020:21:34:07 +0200] [Job 91] Send-Document: client-error-document-format-error (client-error-document-format-error)
which, ideally, should have been an error message.

@fnevgeny, did you try the fix of the bug presented here (or a newer cups-filters version which contains it)?

Yes, sure, removing the "pwg-raster ..." string helped; sorry for not being clear about it.

Sorry again - if you mean the patch of cups-filters, no I didn't do it. I just fixed the ppd file.

@fnevgeny, your edit on the PPD file has the same effect as the cups-filters fix, so all is OK for you and with the new cups-filters you do not need to edit your PPD any more.