Creative Commons text license handling

As of right now, I (James Schloss) have exclusive commercial rights over the text in the Algorithm Archive (AAA). This was done deliberatively at the start of the project because I thought I would be the only author; however, since we have chapter submissions, this will no longer be the case and we need more rigorous chapter contribution guidelines along while also providing a license that "makes sense for each chapter."
I will explain the problem in as much depth as I can, so please forgive the length of this post.
Long-term goal:
I would like to be able to create a Print-on-Demand version of the AAA. Users select the chapters they want published and the language, and we print them physical copies. We should also provide guides through different topics (Like a "Guide to Computational Physics" or something), which provide a pre-bundled selection of chapters and some auxiliary text exclusive to each guide.
This is the only way I can actually see the AAA being sold, but I put the Non-Commercial tag in the Creative-Commons license because I didn't want people printing unofficial AAA copies and selling them.
The main problem:
There has been some discussion about whether the Non-Commercial is actually necessary in this case because:

It inhibits the freedom of others to sell the work and is thus less free according to Free Software Foundation standards.
If we do sell the book in the way described above, all the funds would go to me, not the algorithm-archivists group, which is notably unfair.
I don't think printing the AAA is generally possible without some sort of Print-on-Demand service that we work on together with a specific publisher. It might not actually be possible for other people to print the book in a reasonable way without negotiating with us directly.

In addition, we have one license over the entire AAA text (similar to how we have one license over all the code). If someone submits a new chapter to the AAA, they should be attributed for their work on the chapter. If someone edits the chapter, they should also be attributed. This is not currently how our CC license works.
Possible solutions to Non-Commercial problem:
No matter the case, the Non-Commercial tag seems unnecessary in this case; however, the discussion also came up about using Share-Alike instead. Share-Alike just means that any forks / modifications of the AAA must use the same license. It's one of those "more free licenses that also enforces freedom," so it is somehow more restrictive in another sense. I am 50:50 on the Share-Alike right now.
Possible solutions to authorship problem:
The authorship problem is a bit harder to tackle. We could do a number of things:

Keep a global licence, but change it to be attributed to James Schloss et. al. and define the et. al. to be in a separate file called AUTHORS.md, similar to CONTRIBUTORS.md for the code. If someone modifies the text, they can just put their name there. This has the advantage of only being in one place, so we don't need to put the license in every chapter online (and in print eventually).
Have a separate license for each chapter. This allows us to be more specific about attribution for each chapter, but is a little harder to keep track of.

There are probably other solutions here that we discussed, but I missed. Please let me know and I'll update this.
Conclusion:
My current opinion is that we should remove the Non-Commercial tag in the license and share the license between everyone in a global AUTHORS.md file. If we want to sell bundled versions of the AAA later, this is perfectly fine without the Non-Commercial tag. As I mentioned, I am still 50:50 on the Share-Alike, but if you guys want it, I don't mind putting it in there.
Please let me know what you guys think about this. It's kinda messy, but we need to get it right before merging chapters into the AAA.

HappyLittleRat also had the idea of asking the CC folks for advice.
I think that's a good idea and the issue here can double as a preparation, so that we know, what our goal is and which questions we want to ask.

I agree. That was part of the reason I tried to explain everything in sufficient depth.

ShareAlike
I'm also 50/50 on ShareAlike. While it is a good tool to make sure, that every modification of the AAA and therefore the ideas and work of the AAA stay open, it might be a problem if, let's say, a teacher wants to use a text snippet of the AAA as part of a text in school, which they can't ShareAlike for some reason.
The question is here: Does including a text snippet trigger the ShareAlike or is it just triggered, if you release a modified version? If it's the first case, I think we shouldn't use ShareAlike, if it's the second however, ShareAlike seems to be a good option.
Global or Local License
I think we should license each chapter locally. By doing so, we keep the list of authors short and don't have one bloated contributors file.
Using the chapter text and giving attribution is also easier, because you just have to mention 4-5 people instead of 50-60.
Minor Changes
If I understand it correctly, each modification of the text would produce a new modified version of it, which requires attribution to the new author. While this is absolutely fine for text revisions and additions, this would also mean, that each fixed typo would result in a new contributor added to the authors.
This isn't too much of a problem, but it does treat the people, who did much work, equal as people, who just fixed a typo.
The Contribution Process
Another question is, how the contribution process would work with the license. Does stating that all texts are licensed under CC-BY-SA implicitly makes the author of a PR publishing their text / modification / typo under the license or do we need them to do this explicitly?
(If the second case is true, we would have to get every past contributor of the AAA to explicitly publish their modification under CC-BY-SA)

Share-Alike /  Non-Commercial:
I honestly think Share-Alike is predatory and shady, but I understand that it prevents people from taking the book, modifying the license, and then selling it. The Non-Commercial is much more straightforward and usable for this purpose. It just says, "Hey. You can do what you want with this, but you cannot sell it unless you are one of the people listed here." Share-Alike says, "You can use this, but only in the way we want you to." It boggles my mind that the FSF thinks the former is somehow less free than the latter, but I didn't draw the lines here.
Again, I really don't care that much... It's just that Share-Alike bugs me a bit.
I think the teaching question is only related to teachers that want to take the AAA and distribute lecture notes (or something) online. Even then, we cannot police local distribution. If they want to share it in a public forum (maybe a note website of sorts), then the modification would need to be CC.
Global or local license:
I think the local license is harder to maintain and it means that if we print a book, every single chapter will need a separate license to be printed. That's not that big of a deal, honestly. I think it's a good idea to attribute the algorithm-archivists and invite anyone who has a chapter submitted to the organization, itself. I don't know if this means we need an actual company or if a github organization would do.
A global author file would be easy enough to create a license for:
Terms of Use: This text is licensed under a Creative Commons Attribution License 4.0. It is attributed to all authors in [Link to file].

Alternatively, if we claim the algorithm-archivists organization, it's:
Terms of Use: This text is licensed under a Creative Commons Attribution License 4.0. It is attributed to the algorithm-archivists organization.

For individual chapters, this might look like:
Terms of Use: This text is licensed under a Creative Commons Attribution License 4.0. It is attributed to the James Schloss and the algorithm-archivists organization.

Again, that's fine to put on each webpage and isn't too annoying to print. We just need to remember to update the license for each chapter.
Minor changes:
I feel like people who make minor changes should be acknowledged, but we can do it like we said above:
Terms of Use: This text is licensed under a Creative Commons Attribution License 4.0. It is attributed to the James Schloss and additional authors in [Link to authors file].

Or even:
Terms of Use: This text is licensed under a Creative Commons Attribution License 4.0. It is attributed to the James Schloss and [other folks].

The problem is that if we update the license in the latter way (as more people come in), I am afraid we will need to get the permission of the original author, and that might be hard to do in some cases. By saying we have a global list of authors / reviewers, it is clear from the license initially that the initial author will allow modification from anyone on the changing list of authors.
Contribution process:
Clear guidelines are necessary, but having a in-build PR template for code submissions and chapter submissions with a box that says, "I accept that this contribution will be under this license." is a good idea going forward.

Share-Alike / Non-Commercial
I very much understand the thought process of the FSF behind SA and NC, but that doesn't really matter for this discussion.
I would love to see the AAA being fully free, but I know about the risk, that not-NC (and not-NC in combination with not-SA), introduces.
So we should consider using NC.
Global or local license
The algorithm-archivists organisation doesn't necessarily consists exclusively of all authors. If you want a global license a file with the contributors seems more reasonable.
If we want to print the book, local licenses aren't really a problem. I think a smart solution can be figured out. For more info, look here under "Offline Work?".
This page also suggests, that we don't need to include the CC license we're using. Linking to it seems fine.

It inhibits the freedom of others to sell the work and is thus less free according to Free Software Foundation standards.

I just want to point out that I do not have a problem with that. I think the NC license fits this project. During today's stream, all I tried to say is that that's part of the FSF's definition of free.

I don't want get into the SA debate again. I stand by what I said before. I am sketched out by it, but if everyone else wants to use it, we can use it.
I still think the NC works here, but if we do individual licenses with different authors on every chapter, it becomes difficult to create a physical copy eventually because when we do, we need to contact all the authors who might have moved on. This is why a global attribution to the algorithm-archivists makes sense. We can then funnel all the money to the organization, instead of individuals, and then pay the individuals for dev time.
I think it's alright to give attribution to everyone in the organization (for writing, reviewing and facilitating code contributions), but leave the small edits in CONTRIBUTORS.md. After all, the text and code go hand-in-hand.
No matter the case, we need to make attribution clear in the chapter contribution guide.
I have no idea how to deal with SA with non-SA. Honestly, I just want to do whatever causes us the least stress in the future.

ShareAlike
If ShareAlike prevents the AAA from being used in educational scenarios (being incompatible with text-snippets from other papers and so on), we shouldn't use it. One key-goal of the AAA is enhancing educational content.
NonCommercial
I also think NC makes sense. The risk of companies abusing the AAA is just too high and the AAA organisation is so small, that we can't prevent it.
I think tho, that we should somehow reserve the possibility of making the AAA not-NC at a later point.
Global or Local License
After thinking about it, a global license seems nicer, since the AAA is licensed as a whole. Just relicensing on every text change doesn't seem to bad. (We just need to make sure to preserve NC somehow (,if we want to use it).)

I think I have changed my mind here. The only way for us to have exclusive commercial rights is with NC, but it seems like the NC license does more than what the wording of the license implies (here's a graphic to explain what I mean)
I want the AAA to be used by as many people as possible. I just wanted to make sure people didn't sell a copy of the AAA to make a profit off the community without our explicit permission.
I figure that it is a lot of work to take the current AAA and sell it, so it's probably fine to make everything SA so we can sell our own version (and I guess other people can sell a version too, but I doubt other people will).
In order to keep the NC, we need to either cede commercial rights to me for the time being or create a company to have the commercial rights (which would be hard for me to do right now). It doesn't make sense to have more than one entity have exclusive commercial rights over different parts of the book.

I believe most people are now more or less okay with the idea of re-licensing all AAA content under CC BY-SA, which can be done because I have exclusive rights over all written content right now. The main reason I am no longer in favor of NC is that under NC, the AAA cannot be used in tuition-based courses... Which is a bit restriction.
After this change is made, we can begin attributing authors and editors at the bottom of each chapter.
In principle, the CC BY-SA license will allow anyone to print and sell copies of the AAA, but we will assume that they either will not do this or that we can provide a copy for less because we are not intending to make a profit off AAA sales.
I will make this change soon, but would like to hear any further discussion before then.

The only reason this issue is not closed is because the PR templates are not working right now. I will close this and open a new issue for PR templates


Question about auxiliary features (BAP)

Thank you for your great work!
One question related to your another paper, have you ever tried replace BAP(3-dim) with MAP(21-dim) in SiFiGAN experiments?
I wonder whether MAP outperforms BAP in the case of this model.

Thank you for your interest in my work!
To be honest, I have never compared BAP and MAP in SiFi-GAN. However, based on my experience, I think that switching from BAP to MAP can yield some improvement.


Document docker-based installation instructions

We should probably split the installation instructions into the Docker install (default mode) and the "Developer / from source" install (with the existing text).
@EntilZha, could you take a stab at some brief documentation of how to get up and running on Docker? I'm happy to test it out once you've got something written.

Just a quick ping that I'me working on this as I have free time

Ok, great. One thing I was thinking of doing soon is switching the github.io page over to Jekyll (https://help.github.com/articles/using-jekyll-with-pages/). If you don't feel like dealing with the gross HTML on the gh-pages branch right now, you might want to wait on this until I've done that.

I can wait for that. Was writing them offline using Markdown and planning on inserting them into the right place.

better yet. If you just write them on this issue, I can handle the insertion into the actual docs.

Ok, I've migrated the docs over to Jekyll. Now editing the docs should be as simple as adding Markdown to the appropriate file on the gh-pages branch of this repo. In this case, that will be install.md.


eq-153 accessibility testing

Changes

Adds a gulp task to interface with an a11y runner for automated crawling/testing of code against various accessibility standards.
The tool will output a report in ./tests/a11y

Testing

npm install to get latest deps
gulp test:a11ym
check report in ./tests/a11y

Who can test
Anyone

LGTM :+1:


Refunded product status color box is not appearing.

Terms

[X] Before you openning this issue, i have checked if the issue has already been reported.

Bagisto Version(s) affected
Master
Issue Description
After the product, if we refund the product  then the status of the sales order changes to Closed, here the color of closed box is not appearing.
Preconditions

Customer must have ordered a product from shop.
Admin must be logged in.

Steps to reproduce

Go to admin -> Sales -> Create Invoice.
Now Click on refund. -> Complete all the steps.
Check the status color box, it is appearing blank.

Expected Result
It should reflect color box so that it would be easy to understand the exact status.
Please find the screenshot here
Actual Result
Currently we can see that issue is appearing. Please check the screenshot here
Screenshots
No response
Additional context
No response

Fixed in this PR - https://github.com/bagisto/bagisto/pull/9656


Use mypy, the canonical Python type checker.

Note that mypy is an optional type checker.  Users can still modify their forks without worrying about whether their modifications type check. It is however recommended to run make check before submitting a PR back upstream.

Note that Ubuntu Bionic LTS is still supported and uses Python 3.6.5-3.
Ubuntu Xenial and Trusty are in extended security maintenance only.
So, this is probably OK, but a bit tight for my liking.

@shevek-google According to the README for the GCS client lib:

Python == 3.6: the last released version which supported Python 3.6 was google-cloud-storage 2.0.0, released 2022-01-12.
Python == 3.5: the last released version which supported Python 3.5 was google-cloud-storage 1.32.0, released 2020-10-16.
Python == 2.7: the last released version which supported Python 2.7 was google-cloud-storage 1.44.0, released 2022-01-05.

So if we're willing and able to downgrade the GCS client lib then we can support older versions of Python.#39
@xiashang0624 I think you upgraded the GCS client lib? Do we need 2.3.0 or can we downgrade?

Also, I believe the type annotation syntax has been supported since 3.0. The typing module has only been available in the stdlib since 3.5. But there is a backport if we want to support older 3.x versions. See here for more info. So I think we can support this PR for 3.0+.
I should have done this in a separate PR. I just noticed that pyproject.toml was out of sync with @xiashang0624 's change to README.md.  I can take it out and we can address it in a separate PR that includes a downgrade of the GCS client lib if @xiashang0624 deems a downgrade worthwhile.

Also, I believe the type annotation syntax has been supported since 3.0. The typing module has only been available in the stdlib since 3.5. But there is a backport if we want to support older 3.x versions. See here for more info. So I think we can support this PR for 3.0+ but ideally I'd rather not go beyond 3.6.
I should have done this in a separate PR. I just noticed that pyproject.toml was out of sync with @xiashang0624 's change to README.md. I can take it out and we can address it in a separate PR that includes a downgrade of the GCS client lib if @xiashang0624 deems a downgrade worthwhile.

@shevek-google According to the README for the GCS client lib:

Python == 3.6: the last released version which supported Python 3.6 was google-cloud-storage 2.0.0, released 2022-01-12.
Python == 3.5: the last released version which supported Python 3.5 was google-cloud-storage 1.32.0, released 2020-10-16.
Python == 2.7: the last released version which supported Python 2.7 was google-cloud-storage 1.44.0, released 2022-01-05.

So if we're willing and able to downgrade the GCS client lib then we can support older versions of Python.
@xiashang0624 I think you upgraded the GCS client lib? Do we need 2.3.0 or can we downgrade?

I only used the '2.3.0' version during development so I'm not sure. Actually I don't think we have tested the compatibility matrix of Python versions vs. versions of dependencies for this tool yet. And I think this should be a task before we can declare an older version as "supported".  For example, I think dataclasses is also a Python 3.7+ feature https://docs.python.org/3.7/library/dataclasses.html#

Also, I believe the type annotation syntax has been supported since 3.0. The typing module has only been available in the stdlib since 3.5. But there is a backport if we want to support older 3.x versions. See here for more info. So I think we can support this PR for 3.0+ but ideally I'd rather not go beyond 3.6.
I should have done this in a separate PR. I just noticed that pyproject.toml was out of sync with @xiashang0624 's change to README.md. I can take it out and we can address it in a separate PR that includes a downgrade of the GCS client lib if @xiashang0624 deems a downgrade worthwhile.

I think what we want to do here is to test the tool using different Python versions. The GCS usage is pretty standard here so I think we can probably use an earlier version.

This PR looks good given tool currently supports Python 3.7+, which is documented in the README already. It appears that we want to support an earlier Python version.  I created a separate task to follow-up: http://b/237219199


Better stretched mode for Mac

I get that nobody wants to make the GPU plugin work for Mac, I've come to terms with that :(
However, I'd really love to see the UI scaling settings ported over if possible.

The Stretched Mode plugin is not dependent on the GPU plugin to work. The GPU-powered UI scaling options do require GPU to work on the platform, so they will not work until we are able to integrate GPU functionality on MacOS.


Fix for issue #2: support for downloading specific boards/sections

This fixes issue #2 for the example commands:
pinterest henjesuestrella/fashion-men-style
pinterest henjesuestrella/fashion-men-style/formal

Fix download of specific boards
Add support for boards containing "sections"
linting (style changes), using pylint as linter

yeah, looks great, thank you! =)


Polyline documentation missing arctype field

    viewer.entities.add({
      name: route.name,
      polyline: {
        positions: Cesium.Cartesian3.fromDegreesArray(flattened),
        width: 5,
        arcType: Cesium.ArcType.GEODESIC,
        material: Cesium.Color.GREEN
      }
    })

for example will add a polyline to the display. However, when I view the documentation for polyline, it does not mention it accepts an arcType.
Perhaps this is mentioned elsewhere and should not be mentioned explicitly here. If that is the case, there should be some mention that other options for a polyline are available.
Are there other options other than arcType that can be supplied to a polyline?

Hi @eric-g-97477, the name of the class to use here would be PolylineGraphics, not Polyline. This is the case for all the different entity types, ie. BillboardGrahpics, LabelGraphics, etc.
I concede that this is non-obvious based on naming alone, but the relationship is documented in Entity.
Happy to hear suggestions on how we can improve our reference doc, Sandcastle examples, or website tutorials to help make this relationship clearer.

Ah, makes sense. Thank you for the clarification.


Agent messaging and execution abstraction

Centralize functions to AuthJanitorService, which operates on other loaded services

This ended up being a bigger change than originally intended. Long story short, previous iterations of AuthJanitor have been very piecemeal, making it a challenge to figure out how to integrate alternate consumers of the AuthJanitor Core.
Now, integrating this will be a matter of calling into the AuthJanitorService instance to invoke Providers, or to stash credentials for future use. The intent here is to allow the data layer to exist outside of the AuthJanitor core, which I think is the right move.
This also adds the start of Agent Messaging (but no integration to implement the interface) as a part of this abstraction. If a provider configuration comes in with an Agent ID which doesn't match the running Agent's ID, then it is dispatched with a signed message.

One more note -- this also groups all the AuthJanitor services (not integrations) as a part of AddAuthJanitorService, including passing loaded Providers into it instead of the ProviderManagerService. That means it should be the only thing that needs to be invoked outside of loading support integrations.

... and because I can't resist stuffing PRs, this last (oddly minor) change completely breaks DataStores away from AuthJanitor.Core. 🎉


Resx doesn't work on Azure Webjobs deployed with git

Hi!
I'm using an Azure Webjob to send localized emails (en, es, fr...). The webjob is deployed using git.
The problem is that the resx dll's aren't copied to the app_data/jobs/continuous/MyWebJob folder, so the neutral resx's are used.
If I deploy it by compressing the local bin directory and uploading the resulting zip everything works ok, the texts are in the correct language.
It seems that those culture folders (/en, /es, /fr) aren't copied for any reason from a git deployment.
Thanks!

Could you share a test repo that demonstrates the problem? That will help us investigate. Thanks!

That's a really good idea!!
Here is it: https://github.com/subgurim/WebJobLocalizationTests
I've added some screenshots ;)
Thanks!

When I build this on a local machine, I end up with an es folder under WebJobLocalizationTests\bin\Debug. And when I push your repo to Kudu, I end up with exactly the same thing under D:\home\site\wwwroot\app_data\jobs\continuous\deployedJob.
So I'm not seeing anything different in the Kudu behavior.

BTW, I'm not seeing the screenshots you mentioned.

Hi!!
I'm so sorry!!! I had a big confusion. I have two projects, a web project and a console project. I use the "add an existing project as a webjob" to add the console project to the website and this is what is failing.
I'm really sorry, please let me change the test repo so you only need to clone it.

One thing you should try first is a 'Publish To File System', as described here. That will help isolate. Thanks!

Ok I did it, I've added a website and added the console as a WebJob.
This time I published using "publish azure website" instead of publishing it using git. Same problem :(
Here are the screenshots: https://github.com/subgurim/WebJobLocalizationTests
Thanks!

Did you try the Publish To FIle System I mentioned in my previous mail? If the same issue happens there, then at least we know it's a pure VS issue not related to Kudu. Thanks!

Wow!
You were right!
Publishing on file system produces the same problem!!

:(
Any advice of what can I do?

@bradygaster: this looks like a VS issue in the WebJobs deployment logic. Where is the best place to track it? Thanks!

Please excuse my insistence but do you know somewhere to report this problem?

@subgurim you'd want to report it to my team, the Web Tools Extensions team. I'll take this GitHub conversation to my team so we can triage. I think there's a simple fix for this particular scenario. We are working on a larger, more long-term resolution to some of the CI stories with WJ deployments, but I think the team will be able to provide a quick solution for you in the meantime. Give me a day or two (max) to sync with the team and I'll update this post.

Great!! Thank you very much. I wait for an answer here.

Hi @bradygaster! How's the matter? Have you managed to solve the problem? Is there any better place to talk about this?
Thanks!

@subgurim I don't have more information yet. I'll continue to follow up. We do have a large swath of WebJobs improvements that are related to the VSO side (not the GitHub side) that we're going to be making, but it won't be until the summer as it will require some coordination on the VSO team's side; those conversations are continuing. I'm going to ask one of our devs to take a quick look at this thread for more insight.

@subgurim i just heard back from the dev team. Could you try updating your WebJobs publishing NuGet to the newly-released 1.0.3 version and let me know if that fixes the issue? The NuGet is here: https://www.nuget.org/packages/Microsoft.Web.WebJobs.Publish/

Hi @bradygaster,
First of all thanks for your help.
Unfortunately 1.0.3 version doesn't solve the problem.
Just to put on record I want to remind you that this is not a "git" problem. The same problem happens if I "publish" the website to the "file system".
On 1.0.3 some changes happens, let me explain it with images.
This was the publish result of a website that point to a webjob (1.0.2):

This is the current publish result of a website that point to a webjob (1.0.3):

As you can see a *.resources.dll appears now
And this is the compilation result of the webjob (NOT the website publish result):

Inside the "es" directory is the *.resources.dll file.
IMPORTANT
If I get the result of a website that point to a webjob (1.0.3, second image), and I manually create an "es" directory and I cut&paste the *.resources.dll file to the "es" directory... it works!!! But it's not a good solution because it will fail if I work with more than 2 languages
I've updated the repository with the new changes: https://github.com/subgurim/WebJobLocalizationTests
Do you think it would be possible to finish to fix the problem?
¡¡¡Thanks!!!

@subgurim we took a long look at this and have a fix. I don't want you to wait for the fix to be unblocked for for now, here's a workaround. Please let me know if this works/doesn't. Full fix should be out soon.
Replace these lines:

<ItemGroup>
  <WebJobProjectOutputs Include="%(FilesToPublish.FullPath)"/>
</ItemGroup>

With this:

Hi @bradygaster,
I am very grateful for your help, but still not working :(
Now it gives me this error: "Copying file bin\Release\es\WebJobLocalizationTests.resources.dll to obj\Release\Package\PackageTmp\app_data\jobs\triggered\WebJobLocalizationTests\WebJobLocalizationTests.resources.dll failed. Could not find file 'bin\Release\es\WebJobLocalizationTests.resources.dll'." on Project MvcApplication1
The same error occurs both publishing to file system and deploying to azure via git.
I've pushed this changes to the repo: https://github.com/subgurim/WebJobLocalizationTests
Thanks!!

We're investigating - the fix we suggested will work if you're publishing the WJ directly from VS, but we're investigating the fix for publishing it with the Web App. Will update shortly.

@subgurim could you try this and update the webjobs.console.targets file one more time?
Lines 199-205, replace:
<ItemGroup>
  <FilesToPublish Include="$(OutputPath)**\*.*" Exclude="$(OutputPath)$(AssemblyName).vshost.*"/>
</ItemGroup>

<ItemGroup>
  <WebJobProjectOutputs Include="%(FilesToPublish.FullPath)"/>
</ItemGroup>

WITH:
<ItemGroup>
  <FilesToPublish Include="$(OutputPath)**\*.*" Exclude="$(OutputPath)$(AssemblyName).vshost.*"/>
</ItemGroup>

<ItemGroup>
  <WebJobProjectOutputs Include="%(FilesToPublish.FullPath)">
      <TargetPath>%(RecursiveDir)%(Filename)%(Extension)</TargetPath>
  </WebJobProjectOutputs>
</ItemGroup>

Awesome!! It works for me on the test solution.
During the next days I´ll apply the update to my productions webapps.
I'm grateful to you and the development team for giving me the customized solution to the problem.
Thanks!

For others ending up here and trying to solve the same problem: The fix @bradygaster describes above involves patching the file webjobs.console.targets in the Microsoft.Web.WebJobs.Publish nuget package.

@bradygaster This is still not fixed in the nuget package. Patching it on the client side is a big hack when using automatic package restore. The fix seems simple, so it would be nice if you could update the nuget package!

I've forwarded the suggestion over to the devs, who are actually working on some improvements in this area for an upcoming release (timeline is "very soon" - sorry I can't be more articulate about release dates).

Any update on this issue? I agree with @thedmi, this blocks end users who does package restore (ME!) :(

@fakedevil do you mean update the project reference to pull down the updated 1.0.7 package? I thought, as you called out, that the 1.0.7 package has the fix for the initial issue. So is your comment that we should include 1.0.7 in the official templates, or is there something else going wrong?

Yup update package to the latest 1.0.7 and the issue is resolved. I think we should close this issue and say its resolved from whichever version onwards.


Camera not showing

Hi guys, amazing project.
I want to access the camera when flying the drone, but it seems this is not possible.
The only difference is that instead of iris is I ahve gazebo-classic_iris in my PX4 airframe.
Any suggestions? Rest works fine, I can fly the drone. I use Isaac Sim 4.2.0. In other "pre"

Turns out when I run the initial tutorial it doesn't work, but when running scenes etc. it does. Probably some settng somewhere.


Default export not found when exporting as object literal

I'm using the following syntax:
export {
    foo as default,
    bar
}

And later importing as:
import foo, { bar } from "foo";

But I'm getting a 'No default export found in module'. Code runs and compiles fine with babel.
Switching the foo module to:
export const bar = ..;
export default foo;

Works OK.
Using 0.8.0, with babel-eslint 1.4.3 as  parser.

Fascinating. I have never run into this syntax. Esprima is cool with it too.
I would have thought this was invalid, but given the spec on exports, seems like this is a normal case. Not without precedent, either.
I'm not totally clear on whether this is valid per the spec (Babel allows a lot of things), but it certainly seems useful.
I'm not sure what to do... seems like the pragmatic thing is to handle it as you've described.

Probing the net with this StackOverflow post, too.

I think it should be accepted according to http://www.ecma-international.org/ecma-262/6.0/#table-42.

Yeah, between that and table 40, I think you're right. I will handle. 😁👍

Let me vet the changes a little tomorrow at work, and then I will publish as 0.8.1. Thanks for the report!

Amazing!! Thanks a lot for that prompt reply :).
I used the new version and it worked great for me.


fix: prevent hero's some fields from not being strings

Summary
prevent field from empty or setting as number

Related Issue

Checklist

[x] I have added changeset via pnpm run change.
[x] I have updated the documentation.
[x] I have added tests to cover my changes.

Are these mistakes my fault？

Are these mistakes my fault？

No, i have merged main branch and it'ok now


Touch id screen localization

Is there any way to change localization for touch id screen texts?

Hi :)
Sorry for the delay.
It seems like something to add i18n in react-native-touch-id is on its way.
https://github.com/naoufal/react-native-touch-id/pull/99
You can maybe check if it will be fixed soon and I’ll update the version.
Cheers

Hi,
It seems fixed in the latest version of react-native-touch-id. The PR supposing to fix it was merged anyway.
I updated the version in v1.7.1. Tell me if it works for you now please.
Cheers,
Jeremy


Tech 5012 invoca metrics gaugecache should never call sleep 0

ticket
Added patch on GaugeCache that it should never call sleep 0.

@jebentier @yinonrousso I pushed a commit with throw/catch. I think it looks good. It's nice that it won't get confused with the ScriptError test that really is testing exception rescue in the code. https://github.com/Invoca/invoca-metrics/pull/41/commits/dee74b00be0a3f7e0d0913e2787638b5ca280218

That looks great, I'll go ahead and make the other suggested changes 👍

I went ahead and made most of the change suggested. Let me know if I missed anything or if the warn can have a better description.


[WIP] A Proposal For Testing Things

As we talked about in https://github.com/jlord/git-it-electron/pull/178#issuecomment-270530180, it'd be nice to be able to run the verify tests automagically. This PR is my attempt to tackle the problem, which required some refactoring of how the verify scripts work.
Current Limitations
A couple of issues that I've had to work through:

verify scripts touch the DOM as part of rendering the results - I've attempted to move this code back up to the challenge script, but it's only done for one verify script so far (the repository tests).
some verify scripts are written as if they are blocking but due to using exec they need to be asynchronous - we need a way to inspect the results, so I've introduced a callback that takes an array of results. Maybe you have other ideas in mind.

How This Works
We're using tape to run scripts (I'm more familiar with mocha, so I'm open to other ways to get the best out of the tape API), but here's a simple test:
test('verifies the hello world repository', function (t) {
  var folder = helper.extractFixture('hello-world')
  verify(folder, function (result) {
    var expected = 'This is a Git repository!'
    var first = result[0]

    t.ok(first.result)
    t.equal(expected, first.message)
    t.end()
  })
})

For scenarios where we need the repository to be in a certain state, we can splat the folders onto disk under the tests/fixtures folder - and then extract them to a temporary folder when running the tests. To get this working, you can just rename the .git folder to _git and the repository state is then version-controllable.
The other notable change here is that we have a callback to poke at the results from the verify script (because exec is asynchronous), to ensure we're getting the expected results. I'm using the same shape of "message is a string" and "result is a boolean" that the existing verify scripts use.
To get this working in a test harness, I needed to move the code for addToList and challengeIncomplete/markChallengeCompleted out from the verify script to where the verify method is invoked. A partial implementation of this in challenge.js works for the repository script, but it definitely needs more work and testing as I might not be understanding it correctly.
What's Left To Do?
A rough list:

[ ] get feedback on this approach
[ ] sketch out the changes to test a path-less script, update challenge.js to handle this
[ ] propose a way to test request invocations (nock? something else?)
[ ] ???

verify scripts touch the DOM as part of rendering the results

Yeah, this is something I've wanted to improve; to separate these concerns. Which is probably a big enough project in itself to be a different PR and a thing to figure out before this?
I haven't gone through all this with a fine tooth comb but in general I feel like it's what I've been thinking! 👍
I have a couple things left to clean up, cause I don't think master is quite stable right now but then I'll pick up from what you've started here in moving out the DOM touching stuff and I'll get a PR going for that.
✨

:cool:

Which is probably a big enough project in itself to be a different PR and a thing to figure out before this?

Up to you - if you feel confident enough to do that refactoring before introducing tests then :thumbsup:


Geecreit screen all white

Description
This it the screen
The board has no real markings for its maker but this is their pin definition

Seems like it would be similar to #40. They also use the same driver download site as the other 3 pin screen issue git clone https://github.com/goodtft/LCD-show.git.
Steps taken
Tried using -DKEDEI_V63_MPI3501=ON and -DSPI_BUS_CLOCK_DIVISOR=30 (all the way to 100) and it sometimes flickered (guessing that was the CLS) but nothing after that. Always a white screen.

Duplicate of #138


Playlist generator groups and reordering withing such groups

Hi! When going around the existing limitations of the app, I find I have to create some buffer playlists in Spotify in order to do some more advanced filtering on my playlists. Partly related to this, I would find it helpful if I could create groups of generators and run them. At the same time, it would be nice to reorder the execution within said groups.

I'm just a user, but I think perhaps it would help if you got into more detail what exactly you are trying to do and what exact problems have arisen. I'm also curious what you were trying to do...


New platform added in Learning Platforms

🛠 Fixes Issue

👨💻 Changes proposed

✔ Check List (Check all the applicable boxes)

[ ] My code follows the code style of this project.
[ ] This PR does not contain plagiarized content.
[ ] The title of my pull request is a short description of the requested changes.

📄 Note to reviewers

📷 Screenshots

we already have Buildspace so  closing this PR


Custom map with Props not loaded properly on docker image

CARLA version: 0.9.13
Platform/OS: Ubuntu 20.04.4 LTS
Problem you have experienced:
I used to create custom maps with Roadrunner and import and run them on Unreal (on Windows on my local machine) and they work fine.
Then I needed to run them on a server and run Carla in the background without rendering, this is why I proceed with the solution of building Carla image with docker with the new custom maps (Ingest the map files into packages and then import them in carla).
So I used to go through all this process with different maps that I made and they all work fine on the server. The problem now is, I made a new map with a couple of elements in it (Trees, Buildings, vehicles...), it works on Unreal (on Windows on my local machine), and the process of exporting and ingesting and importing the map files is done without any error, however, when I try to run the simulation (manual_control script for example) on the server after ingesting, the vehicle is spawned without a map, which keeps the vehicle falling down.
Screenshot of the problem: simulation of the map that is not working on server:

Some images of the map on Unreal:

Image of previous maps that works fine on server:

I'm not sure if the map is not working because of the number of elements in it?
(default Carla maps are usually more complex than this one)
(I think this map is not considered large since it is 360m x 390m)

Update: after reducing the number of objects on the map (reducing trees, big buildings) the map works fine on the server after ingesting. The question here is how many elements you can add to the map so the ingestion process can works fine. (maybe how many objects per square meter?)

Also, I notice one thing that there are quite differences between the graphics of the objects before and after map ingestion


Install docs: link to a fast way for multiple users to spin up a server

The specs for a file server are minimal (a RPI4/5 should work but is not tested yet). At this moment, I am using a desktop and it works for me. If there are multiple users, then consider using a proper server.

ref: https://github.com/grd/FreePDM/blob/main/ConceptOfDesign/FreePDM_Install/Install.md
Maybe we can add a link from 'consider using a proper server' to the tutorial that helps quickly spin up a server (on docker etc..)? Thoughts?

Pff, I don't know. That is a hard question to answer. I have this document https://github.com/grd/FreePDM/blob/main/ConceptOfDesign/FreePDM_Install/Docker-Compose.md
Maybe I can advertise it better.

Yes, you can link to that
Proposal:

The specs for a file server are minimal (a RPI4/5 should work but is not tested yet). At this moment, I am using a desktop and it works for me. If there are multiple users, then consider using a proper server.

Okay. I thought that this was about writing specs and that would have been impossible. I am writing that right now.


[BugFix] Fix primary key table vertical compaction bug

What type of PR is this：

[x] BugFix
[ ] Feature
[ ] Enhancement
[ ] Refactor
[ ] UT
[ ] Doc
[ ] Tool

Which issues of this PR fixes ：

Fixes https://github.com/StarRocks/starrocks/issues/12856
Problem Summary(Required) ：

In vertical compaction of PrimaryKey table, The rowsets_mask_buffer parameter should correspond to the rowsets because we use rowsets_mask_buffer to determine the order in which subsequent segments are read. And if rowset only has one segment, we should push a emtpy RowSourceMask into rowsets_mask_buffer but not skip it, otherwise the misalignment may cause some unexpected error.
Checklist:

[ ] I have added test cases for my bug fix or my new feature
[ ] I have added user document for my new feature or new function

BE Build Success

run starrocks_admit_test

run starrocks_be_unittest

run starrocks_be_unittest

run starrocks_admit_test

@mergify backport branch-2.4


Use a 'scripts' directory to follow a common Go project layout

What this PR does / why we need it:
Common practice in Go projects is to have build, install, analysis scripts in a /scripts directory. This diff moves most scripts into the new /scripts directory. It keeps update_codegen.sh in /hacks, as this is common for this script.
Also refactors the e2e test script to have less code duplication.

Closing, as the convention for most Kubernetes project is to use /hack instead of /scripts.


preliminary structure

Description

Related Issues

Checklist

[ ] Is this code covered by new or existing unit tests or integration tests?
[ ] Did you run make test successfully?
[ ] Do new classes, functions, methods and parameters all have docstrings?
[ ] Were existing docstrings updated, if necessary?
[ ] Was external documentation updated, if necessary?

Additional Notes or Comments

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

Comparison is base (34830a7) 80.33% compared to head (b1d0e3e) 64.22%.
Report is 75 commits behind head on main.

Files
Patch %
Lines

superduperdb/backends/ibis/query.py
21.73%
72 Missing :warning:

superduperdb/components/stack.py
0.00%
65 Missing :warning:

superduperdb/cdc/app.py
0.00%
20 Missing :warning:

superduperdb/cdc/cdc.py
25.00%
12 Missing :warning:

superduperdb/backends/sqlalchemy/metadata.py
33.33%
10 Missing :warning:

superduperdb/cli/stack.py
0.00%
9 Missing :warning:

superduperdb/base/build.py
50.00%
7 Missing :warning:

superduperdb/components/model.py
53.33%
7 Missing :warning:

superduperdb/base/datalayer.py
83.33%
5 Missing :warning:

superduperdb/cli/serve.py
54.54%
5 Missing :warning:

... and 10 more

Additional details and impacted files
@@             Coverage Diff             @@
##             main    #1341       +/-   ##
===========================================
- Coverage   80.33%   64.22%   -16.12%
===========================================
  Files          95      100        +5
  Lines        6602     6818      +216
===========================================
- Hits         5304     4379      -925
- Misses       1298     2439     +1141

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


Installation issues on windows

1). "kikaha setup" doesn't consider %MAVEN_HOME% or %M2_HOME% env variables but tries setting up its own maven (?).
I use git bash. When running "kikaha setup" then %HOME%.kikaha\conf\setup.conf is created (e.g. c:\Users\Administrator.kikaha\conf\setup.conf). But in this file there is a line "export M2_HOME="/c/Users/Administrator/.kikaha/work/mvn", however my maven is installed in another place.
2). "kikaha project create" fail,
After manually fixing the path to maven, I run:
kikaha project create 2.0 -e --name=first-project
But it fails with an error:
[ERROR] Failed to execute goal org.apache.maven.plugins:maven-archetype-plugin:2.4:generate (default-cli) on project standalone-pom: The desired archetype does not exist (io.skullabs.kikaha.archetypes:2.0-archetype:1.0.0) -> [Help 1]

Hi @plinyar thanks for share your (bad) experience with Kikaha on Windows.

1). "kikaha setup" doesn't consider %MAVEN_HOME% or %M2_HOME% env variables but tries setting up its own maven (?).
I use git bash. When running "kikaha setup" then %HOME%.kikaha\conf\setup.conf is created (e.g. c:\Users\Administrator.kikaha\conf\setup.conf). But in this file there is a line "export M2_HOME="/c/Users/Administrator/.kikaha/work/mvn", however my maven is installed in another place.

You are right about it. The Kikaha CLI does download another Maven to is own location. The reason it does that is because it customizes the Maven output printing a more friendly feedback. But, I just opened the ticket #12 to handle this: I'm considering to let user choose if Kikaha should use the already installed Maven or download and customize it as it already does now.

2). "kikaha project create" fail,
After manually fixing the path to maven, I run:
kikaha project create 2.0 -e --name=first-project

But it fails with an error:

[ERROR] Failed to execute goal org.apache.maven.plugins:maven-archetype-plugin:2.4:generate (default-cli) on project standalone-pom: The desired archetype does not exist (io.skullabs.kikaha.archetypes:2.0-archetype:1.0.0) -> [Help 1]

This is bad... It seems that it wasn't able to generate the archetypes on your machine.
I just borrow a Windows machine here at work and I'll take a deep look in this issue right this night.
Again, thank you for your feedback.


feat(proto): create endpoint to count tasks

More details here https://github.com/aneoconsulting/ArmoniK/issues/573
Fix https://github.com/aneoconsulting/ArmoniK/issues/573

@esoubiran-aneo @lemaitre-aneo what is the status of this PR ?


JSX Scope - Capture dynamic imports

An instrumented library can be imported using a dynamic import.
Example:
import React, { useState } from 'react';

export default function App() {
  const [bla, setBla] = useState('');
  import('@carbon/react').then((carbon) => {
    setBla(<carbon.Button>A Button</carbon.Button>);
  });

  return bla;
}

Currently, the JSX scope is not designed to capture these type of import metrics, the scope of this issue is to research and implement functionality in JSX scope to be able to capture metrics coming from dynamic imports.

@IgnacioBecerra we came across an example of this during that call yesterday. It looked something like:
function importTearsheet() {
  return dynamic(
    async () => {
      const mod = await import('@carbon/ibm-products');
      return mod.Tearsheet;
    },
    {
      ssr: false;
    }
  );
}

const Tearsheet = importTearsheet();

We don't currently support this during our static source code analysis. CC @ariellalgilmore


FR: Add address, city, state as fields pulled.

Could you also add address information as fields pulled from google contacts?

will track in #30


Localisation pipelines are broken

Describe the bug
Due to changes in restrictions on GitHub Action policies, automations that transferred the English file to the devx-content repository no longer work
Expected behavior

Update GE.json and have your work committed to dev
A commit should be made to the dev branch of the devx-content repository
Once the localisation team has made changes to the language files, a PR is made with the changes back to GE

Proposed fix
Use azure pipelines to move files between the repositories

There is a new process where localization is handled in the browser so there is no need to keep track of the strings in any of the repositories.
This PR makes the changes to effect the new changes


[FR] Extend support for Json Schema

Is your feature request related to a problem? Please describe.

As a brokerpack author
in order to propose generated forms to my users for parameters of type array
I need more complete support of JsonSchema syntax than the current on in CSB 0.2.1

Describe the solution you'd like
The Json schema for array (see https://json-schema.org/understanding-json-schema/reference/array.html#list-validation ) requires support for the items element
{
  "type": "array",
  "items": {
    "type": "number"
  }
}

Without this element, OSB client form generator such as https://github.com/cloudfoundry/stratos don't generate the entry for the parameter of type array

details of rendering without `items` keyword
Json schema generated by CSB, which is missing the itemselement
{
    "$schema": "http://json-schema.org/draft-04/schema#",
                                    "properties": {
                                        "endpoints": {
                                            "description": "The endpoints of all app instances",
                                            "title": "Endpoints",
                                            "type": "array"
                                        },
                                        "healthcheck_uri": {
                                            "default": "/",
                                            "description": "The healthcheck endpoint",
                                            "title": "Healthcheck URI",
                                            "type": "string"
                                        },
                                        "method": {
                                            "default": "GET",
                                            "description": "The HTTP method",
                                            "enum": [
                                                "GET",
                                                "HEAD",
                                                "POST",
                                                "PUT"
                                            ],
                                            "title": "Method",
                                            "type": "string"
                                        },
                                        "port": {
                                            "default": 443,
                                            "description": "The TCP port",
                                            "maximum": 65535,
                                            "minimum": 1,
                                            "title": "Port",
                                            "type": "number"
                                        },
                                        "scheme": {
                                            "default": "https",
                                            "description": "The scheme",
                                            "enum": [
                                                "http",
                                                "https"
                                            ],
                                            "title": "Scheme",
                                            "type": "string"
                                        },
                                        "tags": {
                                            "default": [],
                                            "description": "The tags",
                                            "title": "Tags",
                                            "type": "array"
                                        },
                                        "type": {
                                            "default": "http",
                                            "description": "The service type (http or tcp)",
                                            "enum": [
                                                "http",
                                                "tcp"
                                            ],
                                            "title": "Type",
                                            "type": "string"
                                        }
                                    }
}

Rendering without items json schema element

Rendering with items json schema element (note the yellow button to add new array entries)

Json schema fragment with items
                                       "endpoints": {
                                            "description": "The endpoints of all app instances",
                                            "title": "Endpoints",
                                            "type": "array",
                                            "items": { "type":"string"}
                                        },

See https://hamidihamza.com/ajsf/?set=ng-jsf&example=ng-jsf-nested-arrays&framework=material-design&language=en for live demo of Json schema form generation

Describe alternatives you've considered

add just support for items constraint element
modify CSB to blindly include any constraint element without validation
add alternative CSB support for providing full Json schema (i.e. independently of input variables and constraints)

Additional Context

Currently adding a constraint with the following produces no error message at build or serving, but the served v2/catalog does not include the items element in the JSON Schema.
provision:
  plan_inputs: [ ]
  user_inputs:
    - required: true
      field_name: endpoints
      type: array
      details: The endpoints of all app instances
      constraints:
        items:
          type: string

Fails at catalog serving with output:
{"timestamp":"1611160181.886161089","source":"cloud-service-broker","message":"cloud-service-broker.encoding response","log_level":2,"data":{"data_dump":"lager.Data{\"error\":\"json: unsupported type: map[interface {}]interface {}\", \"r
esponse\":brokerapi.CatalogResponse{Services:[]brokerapi.Service{brokerapi.Service{ID:\"357f0f4f-c905-46b8-b4ee-dfc559537c03\", Name:\"global-server-load-balancer\", Description:\"Global Server Load Balancing\", Bindable:true, InstancesR
etrievable:false, BindingsRetrievable:false, Tags:[]string{\"gslb\", \"gtm\", \"load-balancer\", \"dns\"}, PlanUpdatable:true, Plans:[]brokerapi.ServicePlan{brokerapi.ServicePlan{ID:\"50d8f38b-37d3-47b7-a4aa-0830eb3bb34d\", Name:\"standa
rd\", Description:\"Standard HA Plan\", Free:(*bool)(0xc000597680), Bindable:(*bool)(nil), Metadata:(*brokerapi.ServicePlanMetadata)(0xc0001eddb0), Schemas:(*brokerapi.ServiceSchemas)(0xc00059cde0), MaintenanceInfo:(*brokerapi.Maintenanc
eInfo)(nil)}}, Requires:[]brokerapi.RequiredPermission(nil), Metadata:(*brokerapi.ServiceMetadata)(0xc0001916c0), DashboardClient:(*brokerapi.ServiceDashboardClient)(nil)}}}, \"status\":200}","lager serialisation error":"json: unsupporte
d type: map[interface {}]interface {}"}}

Priority

Priority Context

Platform

Applicable Services

Thank you for your PR in support of this feature @gberche-orange!
This is not currently prioritized by the CSB team, so I have marked it as "help-wanted."
We will follow up if anything changes 💯

Thank you for your PR in support of this feature @gberche-orange!
This is not currently prioritized by the CSB team, so I have marked it as "help-wanted."
We will follow up if anything changes 💯

Hi @gberche-orange, is this something you still want? We don't still have bandwidth to tackle this request and it has been open for contributions for a long time now. Would you be able to submit a PR?


implement meeting notes icon

Feature Request
Problem description
I'm missing an icon for Meeting Notes button.
Proposed solution
Exactly the same icon as there already is in angular codebase

Added in this PR: https://github.com/stardust-ui/react/pull/1953
fyi/ @MartinPtrl


Update K8s version history link

Proposal to change the K8s version history link to referencing the Kubernetes GitHub project page dedicated to release history: https://github.com/kubernetes/kubernetes/releases
Now it is referencing https://en.wikipedia.org/wiki/Kubernetes#History

@palma21
Can you review the proposed changes?
When the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

#reassign:Nickomang

#sign-off


Cant build

undefined reference to pow
it seems to be a problem with the linking and the math lib

modify Makefile line 134,
from
@$(CC) -o $@ $(ALL_LDFLAGS) $(ALL_CFLAGS) $(OBJS_TESTS) $(BLDDIR)/libdablooms.a
to
@$(CC) -o $@ $(ALL_CFLAGS) $(OBJS_TESTS) $(BLDDIR)/libdablooms.a $(ALL_LDFLAGS)

fwiw, this was fixed in https://github.com/ploxiln/dablooms (https://github.com/ploxiln/dablooms/commit/a9cc8567be85ae88fdf361d9f351ff29dad122b2)


It should be possible to exclude an image from the ImageCache

A large sliver list or grid of images can cause serious memory issues when scrolling because of caching. By default, we'll cache every image child we can during scrolling, and we don't get to clean up until later.  A recent PR https://github.com/flutter/flutter/pull/46357 has tried to address this (and contains some very good visualizations of the memory graph in https://github.com/flutter/flutter/pull/46357#issuecomment-565259623), but it would probably be better to just offer some way to not cache such images at all.
If they are network images, we'll want to address https://github.com/flutter/flutter/issues/47378 as well, but separately.
As a bonus, we should provide some hint to developers about when they should use this, e.g. by putting a timer on cache entries and detecting cache entries that are only accessed once over some substantive period of time.
/cc @gaaclarke @goderbauer @jonahwilliams @liyuqian @zmtzawqlp @cbracken

Alternative: large sliver collections should not cache when scrolling rapidly.

I guess if https://github.com/flutter/flutter/issues/47383 is solved, the scrolling performance won't be affected by the lack of image cache because we'll use the same render object, and thus the same image in the GPU memory without image cache?

Yes - we should not do this until #47383 is resolved.

Per a suggestion from @jonahwilliams in another bug, I think this would be easily resolvable with a widget that would impact caching behavior.  For example,
return ImageCachingStrategy.none(
  child: ListView.builder(...),
);

Which would prevent all child images in that subtree from being cached.  We might have other variants like ImageCachingStrategy.always to re-introduce caching to part of a subtree (e.g. for an image or group of images that repeat frequently in the list), or other versions that can take different behaviors depending on the key data for the image.

The spirit of this would be helped by #48536

Repurposing this bug for a general way to prevent an image from ending up in the cache. Lists are no longer as big of a concern after https://github.com/flutter/flutter/pull/49389.

Any news?


Add all-rules tests which verify that none of the rules crashes when it parses a test file that's missing some feature properties.

Additionally:

Fix a couple of rules that broke when parsing such features
Remove individual rule tests checking against an empty feature file

Pull Request Test Coverage Report for Build 434

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+1.8%) to 95.802%

Totals

Change from base Build 431:
1.8%

Covered Lines:
599

Relevant Lines:
616

💛  - Coveralls

Pull Request Test Coverage Report for Build 434

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+1.8%) to 95.802%

Totals

Change from base Build 431:
1.8%

Covered Lines:
599

Relevant Lines:
616

💛  - Coveralls


Subsetting Lecture and Lab

Lecture

[ ] The gut check on slide 35 was a bit confusing. Might get rid of the "NOT" and just ask what would work well.
[ ] The gut check on slides 48 was also a bit tricky - some folks got it, but I have been trying to put the gut check immediately after the concept to be "just in time-y"
[ ] Also might consider reframing how pull() is discussed.. you mentioned convenience of viewing the values a lot, but I feel like the major reason you really need pull() is to make a vector, and I wonder if this confused them
[ ] Slide 65 gut check I would simplify to either testing knowledge of filter() or | vs & - maybe both is too much?
[ ] [BUG‼🐛] Typo at the beginning of part 3 - It should be https://daseh.org/data/CO_ER_heat_visits.csv

Lab

[ ] Remove library(dplyr) from the lab, redundant with loading tidyverse.
[ ] Key has the wrong number of columns for ces, showing 68 not 67 - probably just needs to rerender.
[ ] Could reorder the 2.5-2.6 questions so that asthma comes first.
[ ] had someone do filter(ces_sub, Asthma > 100 & Traffic < 500 | Traffic==500) -- how does order of operations work with filter?

alignment of the cheetsheets could be fixed for the lab2 slide

before section 2 would be helfpful to assign reading in the data to er - just like before section 3:
#er <- read_csv("https://daseh.org/data/Colorado_ER_heat_visits.csv")
set.seed(1234)
er_30 <-slice_sample(er, n = 30)

[ ] in summary of part 2 point out that there are extra slides on having multiple patterns for selection

[ ] add a mention of the new pipe

had someone do filter(ces_sub, Asthma > 100 & Traffic < 500 | Traffic==500) -- how does order of operations work with filter?

I added some extra slides about this


25% Chance of ERT with Special Nuke Song when 50% or more of crew is dead during Nukeops

About the PR

This PR introduces a 25% chance of an ERT (one commander, one engineer, one security, one medical, one janitor) spawning when 50% or more of the crew are dead during nukeops. They are all automatically raffled ghost roles, and the shuttle is preloaded.
If an ERT has been dispatched and the nuke is armed, it will play a custom nuke song just for this scenario, "Rescue Imminent" by mrjajkes.
https://www.youtube.com/watch?v=DjV0OMskwiE
Why / Balance

Technical details

Media

I took a video of it working but honestly I'm too lazy to chop it up and blablabla. The shuttle loads with no lag, the ghost roles work, you can ftl to the station, the nuke song plays, etc. "Trust me bro".
Requirements

[X] I have read and am following the Pull Request and Changelog Guidelines.
[X] I have added media to this PR or it does not require an ingame showcase.

Breaking changes

Changelog

:cl: Piras314, mrjajkes

add: Added a 25% chance for an ERT to spawn during nukeops if 50% or more of the crew is dead. If the nuke is armed when an ERT is spawned in, a special nuke song will play!

is this somthing we can add for when blob goes get to 50% aswell?

is this somthing we can add for when blob goes get to 50% /delta stage aswell?

Yeah just copy over the changes from NukeopsRuleSystem. A deathsquad might be more fitting for that though maybe? Not super familiar with blob.

I think I should reduce the percentage of crew that need to be dead to roll ERT to 35% or something lower like that.

We can mess with the numbers and stuff later, this is just to start. The ERT is pretty weak anyway.

The centcom dispatch a announcement when send ert?

minor copypaste ops aside, my computer is imploding rn so i cant cherrypick this thing and run it, but whats the worst that could happen :clueless:

I'm honestly too lazy to refactor this right now and would rather do it in a later PR where we want to use the functionality for blob or whatever.

Also there should be an announcement if ERT is called and maybe gamma code as well.


SharePoint Restore no longer working

Hi Niels,
after upgrading to the last build, I can get a false error message:
When I select any organization, I get the response "No SharePoint sites found for this organization." When I click on "Explore last backup", I get the results and can run restores.
Rebooting the VBO and the webserver did not help.
Thank you,
Caroline

Hi Caroline, did you run multiple backups for this organization already?

Yes, we backup these orgs for months.

Caroline Kiel
PingUs Solutions
CEO
Tel.:
Fax:
Mobil:
E-Mail:
+49 33056 401-111tel:+49 33056 401-111
+49 33056 401-190tel:+49 33056 401-190
+49 172 3232 800tel:+49 172 3232 800
Caroline.Kiel<EMAIL_ADDRESS>Marienstr. 1a
D-16552 Mühlenbecker Land
Ortsteil Schildow
https://www.pingus.de

PingUs Solutions GmbH & Co. KG
Geschäftsführung (Managing Partners)
Anja Kiel, Caroline Kiel
Steuernummer (Tax ID)
053/161/09049
UID-Nr. (VAT-No.)
DE 813 720 069
Amtsgericht (County Court)
Neuruppin HRA 1440 OPR

Important Note:
This e-mail may contain trade secrets or privileged, undisclosed or otherwise confidential information. If you have received this e-mail in error, you are hereby notified that any review, copying or distribution of it is strictly prohibited. Please inform us immediately and destroy the original transmittal. Thank you for your cooperation.
.
[Logo PingUs Solutions]https://docks.pingus.de/
Von: nielsengelen<EMAIL_ADDRESS>Gesendet: Donnerstag, 29. Oktober 2020 09:20
An: nielsengelen/vbo365-rest<EMAIL_ADDRESS>Cc: Caroline Kiel<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Betreff: [Newsletter] [Ext] Re: [nielsengelen/vbo365-rest] SharePoint Restore no longer working (#74)
Hi Caroline, did you run multiple backups for this organization already?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/nielsengelen/vbo365-rest/issues/74#issuecomment-718467872, or unsubscribehttps://github.com/notifications/unsubscribe-auth/APFRMYMYYIXMX7WUDFELQVDSNEQTTANCNFSM4TASYRAQ.


Issue with generator dataset Tensorflow

With python = 3.6, tensorflow = 2.3.1 got an issue if the size of the dataset is not a perfect multiplier of the batch size

This issue is raised typically for the validation dataset which is not resized

It seems to only concern the finetuning method


allow create message translation/customization

Hi,
Thanks for your work on dal.
This little change allow to translate the message displayed by dal to create new item.

Thanks a lot for your feedback @maisim, would it be possible for you to provide a translation file as well ?

Yes, for the moment my way to use it is to symlink dal directory in my project to get "manage.py makemessages" able to find strings to translate and add them to my project translations files. But it should be nice to get translation for everywhere it's needed in dal. I will look for it asap.

Thanks that would be awesome ! Where are you from Simon ?


[FEATURE]: Allow specifying constraint names

Describe want to want
Computed constraint names can sometimes get too long for Postgres, resulting in this warning:
psql:migrations/0000_brown_rage.sql:110: NOTICE:  identifier "super_duper_duper_long_name" will be truncated to "super_duper_du"

It would be great if one could specify the constraint name in foreignKey:
foreignKey('table_other_table_fk', {
  columns: [table.id],
  foreignColumns: [otherTable.id],
})

Or
pgTable('table', {
  id: uuid('id').notNull().references('table_other_table_fk', () => otherTable.id),
})

NOTE: This probably applies to constraints in general, not just foreign key constraints.

Make sense. We are working on adding constraints as separate entity, so you can specify any constraint you want. Will think on names for it
Thanks!

Sounds good! Thank you!

@AndriiSherman do you have any ETA on this ? PG may generate a warning, but it completely breaks on MySQL.

@AndriiSherman do you have any ETA on this ? PG may generate a warning, but it completely breaks on MySQL.

In the meantime you can manually truncate the migration SQL file to 64 characters.

Just a note that this feature would be good from the perspective of incremental migration.
Currently, drizzle-kit introspect:mysql get the current constraint names correctly, but the first drizzle-kit generate:mysql removes and re-adds the FK constraint names as they are incorrectly named.

Any update on this ?

I'm finally got to this feature.  Its causing some issues for us migrating to Drizzle.

This is also a blocker for us.
The initial schema migration works despite the truncation warnings for postgres, but because drizzle-kit seems to be unaware of the truncation, subsequent migrations end up failing, rendering drizzle-kit unusable.
Anyone aware of a workaround?

@AndriiSherman I'm not sure about what is more important than support a basic feature for one of the most popular sql engine, but this is clearly overdue. I get your business but understand that because of this issue I cannot recommend drizzle to anyone building something serious.
This is not a "nice to have", it's not even an "enhancement" (despite the tag). It's a critical bug fix for MySQL and a major one for Postgres. And it's 6 months old.

This has been implemented in v0.29

It seems that this has been just partially fixed, as of now drizzle-kit introspect won't get names for single-column foreign keys translated in code using the references() method, this causes drizzle-kit generate defining fancy and sometimes long names for those constraints leading again to this issue. The only workaround I found is to remove foreign key definitions manually on column definitions and add them as composites with the foreignKey operator which is impractical for large databases. It would be nice if the references() method also includes a name param like references(name, ref, action?).

It seems like this isn't supported in the references() method as stated above. Is it possible for it to support this as this is a major blocker for us when migrating to drizzle.


Refactor config.ServerType to provide a globally accessible source for server(s) enabled

Closes #501
Now you can use config.IsServerEnabled anywhere in the server code to test if a server is enabled. A private function setEnabledServer is called in initServer() to indicate which server(s) are enabled.
This would be a necessary step for UI to redirect server web UI user to correct server UI landing page.
This can also be used in #172
When browsing through places where we used ServerType, it seems that we had a XrootDServer interface that relies on implementation to tell which server is running. I didn't touch that part of the code in this refactoring but if we want I can remove GetServerType() from XrootDServer interface and replace existing ones to config.IsServerEnabled. I'm fine either way. @turetske

When browsing through places where we used ServerType, it seems that we had a XrootDServer interface that relies on implementation to tell which server is running. I didn't touch that part of the code in this refactoring but if we want I can remove GetServerType() from XrootDServer interface and replace existing ones to config.IsServerEnabled. I'm fine either way. @turetske

@haoming29 I'm hesitant to do this, because in this case it specifically needs to know if the server is the one of that "type", not whether the server is running. Because the "pelican serve" code can run multiple servers, we need to distinguish between "is  server type running" vs "is  the server type that is calling this function".


Add new policy writer role

This adds a new role:

policy_writer: which can only create, update and delete rule types and profiles.

These new role is meant for CI jobs or automation to populate all the
needed pieces for an organization to set up policy via minder.

@eleftherias addressed you feedback. Thanks for the review!


fix: 🐛 dedup comments

Close https://github.com/oxc-project/oxc/issues/4843

It seems I need to clone the original repo


Cloud API adaptor on Azure unbound variable

Following the steps for setting up CAA on Azure, https://confidentialcontainers.org/docs/cloud-api-adaptor/azure/, after running CLOUD_PROVIDER=azure make deploy the operator install daemon pod fails and the logs show an unbound variable:
$ kubectl describe pod cc-operator-daemon-install-tr6fx -n confidential-containers-system
  Events:
  Type     Reason     Age                    From               Message
  ----     ------     ----                   ----               -------
  Normal   Scheduled  5m58s                  default-scheduler  Successfully assigned confidential-containers-system/cc-operator-daemon-install-tr6fx to aks-nodepool1-10076017-vmss000000
  Normal   Pulled     5m20s                  kubelet            Successfully pulled image "quay.io/confidential-containers/runtime-payload-ci:kata-containers-8de1f8e19f858134ba455a7c04edcb21d8bcf6b1" in 31.094025085s (37.069779763s including waiting)
  Normal   Pulled     5m13s                  kubelet            Successfully pulled image "quay.io/confidential-containers/runtime-payload-ci:kata-containers-8de1f8e19f858134ba455a7c04edcb21d8bcf6b1" in 153.957989ms (153.964689ms including waiting)
  Normal   Pulled     5m1s                   kubelet            Successfully pulled image "quay.io/confidential-containers/runtime-payload-ci:kata-containers-8de1f8e19f858134ba455a7c04edcb21d8bcf6b1" in 123.53862ms (123.55092ms including waiting)
  Normal   Created    4m36s (x4 over 5m20s)  kubelet            Created container cc-runtime-install-pod
  Normal   Started    4m36s (x4 over 5m20s)  kubelet            Started container cc-runtime-install-pod
  Normal   Pulled     4m36s                  kubelet            Successfully pulled image "quay.io/confidential-containers/runtime-payload-ci:kata-containers-8de1f8e19f858134ba455a7c04edcb21d8bcf6b1" in 152.888065ms (152.894165ms including waiting)
  Normal   Pulling    3m44s (x5 over 5m57s)  kubelet            Pulling image "quay.io/confidential-containers/runtime-payload-ci:kata-containers-8de1f8e19f858134ba455a7c04edcb21d8bcf6b1"
  Normal   Pulled     3m44s                  kubelet            Successfully pulled image "quay.io/confidential-containers/runtime-payload-ci:kata-containers-8de1f8e19f858134ba455a7c04edcb21d8bcf6b1" in 167.848396ms (167.859396ms including waiting)
  Warning  BackOff    53s (x21 over 5m12s)   kubelet            Back-off restarting failed container cc-runtime-install-pod in pod cc-operator-daemon-install-tr6fx_confidential-containers-system(744a6ad7-0d5e-48c2-98b2-81d150c89213)

$ kubectl logs cc-operator-daemon-install-tr6fx -n confidential-containers-system
Environment variables passed to this script
* NODE_NAME: aks-nodepool1-10076017-vmss000000
* DEBUG: false
* SHIMS: remote
* DEFAULT_SHIM:
* CREATE_RUNTIMECLASSES: true
* CREATE_DEFAULT_RUNTIMECLASS: false
/opt/kata-artifacts/scripts/kata-deploy.sh: line 417: SNAPSHOTTER: unbound variable

Ah.. this is fixed in the main - https://github.com/confidential-containers/cloud-api-adaptor/blob/main/Makefile#L168
The version that you are using will most likely be using
kubectl apply -k "github.com/confidential-containers/operator/config/release?def=v0.8.0"

instead
of
kubectl apply -k "github.com/confidential-containers/operator/config/release?ref=v0.8.0"

Thanks @bpradipt. I was able to run a pod with an older version of the instructions. Are there plans to either update the documentation or fix this in the next release?

cloud-api-adaptor-0.8.0  is actually using ref=v0.8.0  and changing that to def doesnt fix it
where did you find the older instructions?

cc @stevenhorsman

Yeah, so the key fix to enable this (in https://github.com/confidential-containers/cloud-api-adaptor/commit/defb3f12e309c885e40126649fcee29a36876bd7) is default -> release, but the correct command that should resolve it is
kubectl apply -k "github.com/confidential-containers/operator/config/release?ref=v0.8.0" as you said.
Chelsea this is already fixed in CAA main and we're talking about doing interim 0.8.1 release to get it working in a 'stable' place, but need to line up a few dependencies first. As that point we can bump the version in the wider CoCo docs and it should work. In the short-term if you think it would help we could ask @surajssd to put a note in the
https://confidentialcontainers.org/docs/cloud-api-adaptor/azure/#deploy-caa-on-the-kubernetes-cluster section to start that the default->release edit needs doing before deploying?

Created a PR that could fix this problem and will need updates to the documentation in the upstream: https://github.com/confidential-containers/cloud-api-adaptor/pull/1691

But I have also updated the docs so that now user also has choice if they want to deploy latest stuff: https://github.com/confidential-containers/confidentialcontainers.org/pull/31

@aep the chance to kubectl apply -k "github.com/confidential-containers/operator/config/release?ref=v0.8.0" did resolve it for me. The instructions I used a while back also worked, I pulled up this commit: https://github.com/confidential-containers/confidentialcontainers.org/commit/fd26c800dc569bf7c22ccc281847f2a965750c5b.
Thanks @stevenhorsman and @surajssd, I will take a look at the documentation PR.

@cmaf I assume we can close this issue, then?


Error when trying to find carriageway and position from point coordinates

Used the pyramm example to retrieve the position:
centreline = conn.centreline()
point = Point((172.618567, -43.441594))  # Shapely Point object
position = centreline.position(point, point_crs=4326)

This resulted in this error:
    raise NotImplementedError(
NotImplementedError: Multi-part geometries do not provide a coordinate sequence

Attributed to the conn.centreline()

@Winda2 please try the latest version of pyramm (v1.22) and see if you still have this issue.

Closing as unable to reproduce. Please open a new issue if the problem persists.


Link to CLR Profiler broken

In the TraverseHeap documentation, the link to the CLR Profiler (https://go.microsoft.com/fwlink/?LinkID=67325) is broken.
I found this (https://www.microsoft.com/en-us/download/details.aspx?id=16273), but I don't know if it's the latest one.

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

ID: 8e01c075-7a07-5850-d462-706675b545f1
Version Independent ID: 0dd6c661-01a1-592c-ea53-8427412ff8d0
Content: SOS.dll (SOS Debugging Extension)
Content Source: docs/framework/tools/sos-dll-sos-debugging-extension.md
Product: dotnet-framework
GitHub Login: @mairaw
Microsoft Alias: mairaw

Thanks for your feedback @paulomorgado.
@vivmishra @terrajobst @tommcdon who could help me find out where is the latest version of CLR Profiler?
In addition to the one that Paulo linked to in the downloads site, there seems to be this one:
https://github.com/microsoftarchive/clrprofiler/releases

Adding @brianrob and @PeterSolMS who might have know where the latest version of CLR Profiler is now.

It has been now more than 1 year since this issue was open.
https://www.microsoft.com/en-us/download/details.aspx?id=16273 is now 404!


Conversant: add Publink Userid Module

Type of change

[ ] Bugfix
[ X] Feature
[ ] New bidder adapter
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] CI related changes
[ ] Does this change affect user-facing APIs or examples documented on http://prebid.org?
[ ] Other

Description of change
Create a Publink userid module
sample config
pbjs.setConfig({
userSync: {
userIds: [{
name: "publinkId",
storage: {
name: "pbjs_publink",
type: "cookie",
expires: 30
},
params: {
e: "e80b5017098950fc58aad83c8c14978e", // example hashed email (md5)
}
}],
auctionDelay: 50             // 50ms maximum auction delay, applies to all userId modules
}
});
Be sure to test the integration with your adserver using the Hello World sample page.

contact email of the adapter’s maintainer
[ ] official adapter submission

For any changes that affect user-facing APIs or example code documented on http://prebid.org, please provide:

A link to a PR on the docs repo at https://github.com/prebid/prebid.github.io/

Other information

Looks good.  Thanks.


Failed to pull local image

When I build a image locally, I try to use that image in virtualmachine spec, I got the error from pod events:

Failed to pull image "my-local-image:tag": rpc error: code = Unknown desc = Error response from daemon: pull access denied for rancher-ubuntu, repository does not exist or may require 'docker login'

Is this because of the ImagePullPolicy = Always of pod? If yes, can you provide a way to change that default value? Thx.
https://github.com/rancher/vm/blob/f40cd374902cacf17dee43950586d8af836bbb58/pkg/controller/vm/util.go#L173

@lxs137 How can you solve this problem?

@lxs137 How can you solve this problem?

edit code, deploy controller yourself

@ lxs137你怎么解决这个问题？

编辑代码，自己部署控制器

how to modify, please advise


minitube: new package

Minitube is a pretty wrapper around (Phonon/)VLC for watching YouTube videos without having to deal with the human junkyard that is youtube.com.
Unfortunately, it doesn't work:
gnutls tls client error: cannot load trusted Certificate Authorities: An unimplemented or disabled feature has been requested.
gnutls tls session error: Certificate verification failure (0x0042)
gnutls tls session error:  * Certificate not verified
gnutls tls session error:  * Signer not found
core tls session error: TLS client session handshake error
http access error: cannot establish HTTP/TLS session
core input error: open of `https://r2---sn-cxab5jvh-cg06.googlevideo.com/<hilarious_cat_video>' failed
core input error: Your input can't be opened
core input error: VLC is unable to open the MRL 'https://r2---sn-cxab5jvh-cg06.googlevideo.com/<hilarious_cat_video>'. Check the log for details.
gnutls tls client error: cannot load trusted Certificate Authorities: An unimplemented or disabled feature has been requested.
Phonon error: "No active input"  Phonon::NormalError

Any kind souls with knowledge?

Actually, VLC can also just open youtube URLs, which I do sometimes (also due to performance reasons), and it claims the cert isn't trustworthy. It can be clicked to ignore the problem, but it's annoying nevertheless.

Indeed, when I tried pasting the offending URL into VLC proper I could manually trust the cert. From then on, that particular video also plays fine in Minitube.
Then I open the next video, and it fails all over again...

I believe some package in the stack isn't directed to our cert location, $SSL_CERT_FILE (I think).

I'd think gnutls is the place to patch.

I'll try a solution and file a PR changing our gnutls (to discuss possible alternative solutions).

Awesome! Looking forward to it.

Done #8121. It's rather a mass rebuild, unfortunately.

Pushed to staging, as reasonable usage depends on 651e83a.

Thank you!

Thanks for a nice package :-) though I slightly miss the configurability of vlc.


set request->need_rounter to be false to prevent from infinite recursion

https://github.com/pinterest/rocksplicator/blob/9849fc310ab904710508a8abccfcfc0ae75a78f7/examples/counter_service/counter_handler.cpp#L131
otherwise reading  an undefined key leads to infinite recursion.

its set to false in line 118

Maybe my version of fbthrift is not consistent.
In my case, I need to run  request->set_need_routing(false); to make this flag to be false.
I tried both the fbthrift in the docker, and the latest fbthrift. not sure why...


Fix compatibility with Django 1.10+

SubfieldBase is deprecated since Django 1.10

It definitely mistake


Tree Borrows: do not create new tags as 'Active'

Cc @Vanille-N

@bors r+

:pushpin: Commit 4daf54a49098401d7ab7f277e8f01fb2f132aa24 has been approved by RalfJung
It is now in the queue for this repository.

:hourglass: Testing commit 4daf54a49098401d7ab7f277e8f01fb2f132aa24 with merge 6f9139afee45e161f846b7b44b9a7856fecaaf88...

:sunny: Test successful - checks-actions
Approved by: RalfJung
Pushing 6f9139afee45e161f846b7b44b9a7856fecaaf88 to master...


sort by folder first character missing

Please check that this issue hasn't been reported before.

[X] I searched previous Bug Reports didn't find any similar reports.

Expected Behavior
When i sort by folder i would expect that the complete folder name would be visible.
Current behaviour
The first letter/number of the folder name is missing
Steps to reproduce
Each time i create the following task list it does not display the first number.
`
not done
group by folder
(has start date) OR (has due date)
(starts before tomorrow) OR (due before tomorrow)
short mode

`
Which Operating Systems are you using?

[ ] Android
[ ] iPhone/iPad
[X] Linux
[X] macOS
[ ] Windows

Obsidian Version
v1.3.4
Tasks Plugin Version
1.25.0
Checks

[X] I have tried it with all other plugins disabled and the error still occurs

Possible solution

in this case all the titles should be "200. xxxx" for some reason the number 2 fals outside the task frame

Thanks for using Tasks.
There is content in your report that is not provided as text for me to copy, so for reasons explained in the following post, I am unable to investigate:
https://publish.obsidian.md/tasks/Support+and+Help/Report+a+Bug
If you can provide full text, and full numbered steps for a reproduction, as per the above, I will willingly have a look when I have time.
In your case, I will also need to know specific file and folder names to use to reproduce the problem, and you should remove lines from your Tasks query block until only the minimum lines needed to show the problem are present.
Thank you.

hey @claremacrae
Sorry, i missed the part about how to copy the task itself. I'll try again
my folder name/structure is as follows:
format is: vault/folder
mywork/100. Inbox
mywork/200. Customers
The task itself is:
```tasks
not done
group by folder
(has start date) OR (has due date)
(starts before tomorrow) OR (due before tomorrow)
short mode
```

I hope this helps you more

Thanks, that is progress, but to replicate your problem I have to create tasks that match your vault, and again, I just don't have time to do that.
Let's switch to a different track.
Since your problem requires specific filenames and folders, please create a new vault, install tasks, set up a few files, Zip the vault and attach it here.
So that I can download the vault, expand it, and see the problem.
https://publish.obsidian.md/tasks/Support+and+Help/Report+a+Bug#Give+us+your+minimal+vault
Give us your minimal vault
Once you have figured out the steps to reproduce the vault:

Quit Obsidian
Create a .zip file containing the minimal vault
Make sure you [Give steps to reproduce the bug](https://publish.obsidian.md/tasks/Support+and+Help/Report+a+Bug#Give steps to reproduce the bug)
Attach the zip file to this bug report, by dragging the file from your file browser in a new comment here

hey @claremacrae
here you can find a screenshot of what you asked me + the zip file in attach. I hope it helps.
task_issue.zip

The steps i took are very simple:

create new vault
install tasks plugin
created folder 100. Inbox
created folder meetings nested underneath 100. Inbox
created a note called test in the meeting folder
added a task called test task with a due date for tomorrow
created the task note with a copy/paste of what i used in my post above.

Et voila, the same happens again.
I can add that if i remove the dot the problem is solved but it looks kinda stupid 100 Inbox it also works when doing 100 - Inbox.
So my guess is that Obsidian (or tasks) trips over the dot.

Fantastic. Now I can investigate very quickly.
Observations
With the compressed vault I found that:

The display of the query is fine in Reading mode:

The problem you describe only happens in Live Preview:

As you said, removing the dot makes the problem go away.

It's curious that with the dot, the the 100 is displayed in grey and without the dot, it's displayed in black.
I believe the dot means that the 100. is interpreted as a numbered list item, like this:

I experimented with this markdown:

### 100. Hello

### 200 Hello

Reading:

Live Preview

I had thought that it would have displayed the 100. in grey, and was surprised that it did not.
Observations
So the bug is very specifically:

If you have a folder name that begins with some digits and then a full-stop/dot
And then you use Tasks to group your tasks by Folder (and maybe Root and Path)
And then you view it in Live Preview
Then the first digit of the folder name is missing.

Do you agree?

That's exactly the issue.
And while it's not the biggest problem of changing the dot to a dash or something else, it's strange that it behaves this way.
Based on the fact that you said digit i also did the check with xyz.test and that displays normal. So your idea about tasks thinking it's a numbered list is a valid idea it seems.

Just to clarify, Tasks has no idea that it is a numbered list. But something is making Obsidian render it differently in Reading than Live Preview. No idea what.

I have updated the description to describe what we worked out about the behaviour.

I just re-tested the sample vault in Obsidian 1.6.4, with the development version of Tasks, and in Live Preview the full heading is now shown:


Database Implementation

As a developer
I need a database to store all the customer information
So that we can build the customer APIs and store the data
Details and Assumptions

[document what you know]

Acceptance Criteria
Given [some context]
When [action taken]
Then [outcome observed]

duplicate issue to #9


enabling url and matching url parameters

Description
-enabling the url as crawler is pushed
and matching url -just another way
Attach Screenshot

Note 2 code reviewer approval needed. Approach in twitter group & discord channel.

Why this change?
@PuruVJ @johnjacobkenny @Mira-Alf

Why this change?
@PuruVJ @johnjacobkenny @Mira-Alf

I have the same qn

Why this change?
@PuruVJ @johnjacobkenny @Mira-Alf

when testing i was getting failed message Error: Cannot find module 'string.prototype.matchall' so i tried to do it  without the module itself

Why this change?
@PuruVJ @johnjacobkenny @Mira-Alf

when testing i was getting failed message Error: Cannot find module 'string.prototype.matchall' so i tried to do it without the module itself

Just run npm install!


✨unpacker: switch from google/go-containerregistry to containers/image

Description
See https://docs.google.com/document/d/1s-nfpJqecWGuriFIVXr2REbZLUB8N8QxLp1MQKLPhlE/edit#heading=h.x3tfh25grvnv

[x] This tilt-support PR is required to make the tilt test pass: https://github.com/operator-framework/tilt-support/pull/9

Reviewer Checklist

[ ] API Go Documentation
[ ] Tests: Unit Tests (and E2E Tests, if appropriate)
[ ] Comprehensive Commit Messages
[ ] Links to related GitHub Issue(s)

/hold cancel


Clusteralerts and podalerts don't get recreated in icinga when searchlight-operator restarts

I'm only test this using the following (on v7.0.0):

NodeAlert: node-status and node-volume checks.
ClusterAlert: component-status, event, and a webhook checks.
PodAlert: pod-status check.

Whenever searchlight-operator pod restarts, NodeAlert hosts and services will automatically be registered back in icinga, but not so for ClusterAlert and PodAlert objects. I had to use kubectl delete and kubectl apply again to force their registration.
Looking at the logs after restart, I noticed only plugin.go and nodes.go keep retrying to connect like so:
icinga/searchlight-operator-84878b6df-r95z7[operator]: I0719 18:04:58.386974       1 plugin.go:60] Sync/Add/Update for SearchlightPlugin cert
icinga/searchlight-operator-84878b6df-r95z7[operator]: E0719 18:04:58.452500       1 worker.go:76] Failed to process key cert. Reason: command terminated with exit code 1
icinga/searchlight-operator-84878b6df-r95z7[operator]: E0719 18:04:58.742345       1 nodes.go:64] failed to reconcile alert for node k8sworker1c. reason: [: Put https://<IP_ADDRESS>:5665/v1/objects/hosts/icinga@node@k8sworker1c: dial tcp <IP_ADDRESS>:5665: getsockopt: connection refused, : Put https://<IP_ADDRESS>:5665/v1/objects/hosts/icinga@node@k8sworker1c: dial tcp <IP_ADDRESS>:5665: getsockopt: connection refused, : Put https://<IP_ADDRESS>:5665/v1/objects/hosts/icinga@node@k8sworker1c: dial tcp <IP_ADDRESS>:5665: getsockopt: connection refused]
icinga/searchlight-operator-84878b6df-r95z7[operator]: E0719 18:04:58.742381       1 worker.go:76] Failed to process key k8sworker1c. Reason: [: Put https://<IP_ADDRESS>:5665/v1/objects/hosts/icinga@node@k8sworker1c: dial tcp <IP_ADDRESS>:5665: getsockopt: connection refused, : Put https://<IP_ADDRESS>:5665/v1/objects/hosts/icinga@node@k8sworker1c: dial tcp <IP_ADDRESS>:5665: getsockopt: connection refused, : Put https://<IP_ADDRESS>:5665/v1/objects/hosts/icinga@node@k8sworker1c: dial tcp <IP_ADDRESS>:5665: getsockopt: connection refused]

So I tried removing return from this line:
https://github.com/appscode/searchlight/blob/a90f4bc264099230a02ca329df1c2a30a9e28d51/pkg/operator/pods.go#L28
Also this if condition:
https://github.com/appscode/searchlight/blob/a90f4bc264099230a02ca329df1c2a30a9e28d51/pkg/operator/cluster_alerts.go#L21
And now all my clusteralerts and podalerts registered back automatically after restart.
But I'm not sure what's the negative impact of skipping those checks?

OK, removing op.isValid(alert) most of the time will result in invalid memory address or nil pointer dereference error in op.clusterHost.Apply(alert). Previously I wasn't testing with enough number of alerts to see this.
I still want all ClusterAlert objects to be enqueued and retried during pod start-up though, so I ended up moving op.isValid(alert) from initClusterAlertWatcher() to reconcileClusterAlert().
Also increase the default MaxNumRequeues from 5 to 20 just to be sure.
Now everything register back automatically after restart.


chromium dosn't show up....

Hello,
I just tried to install this on my raspberry pi 4 with a waveshare 7inch TouchScreen (C), and used the config suggested on their wiki and btw you need to add "=" between hdmi_ctv and 1024.
I does boot in the right size (1024x600), i get the splash screen (unicorn) but after waiting a long time it doesn't show chromium at all.
I created a file named chilipi_url.txt and put the url in the first line.
I also tried to set up wifi and it worked the first time (because i see it on my router webpage) but after trying again with another sd card to see if it was the problem (btw no succes) it didn't connect at all.
when i try to plug it back in my windows machine, it just says "F: might be broken" or something like that (idk exactly because my windows is in my native language)
I hope someone has help
Thanks
Croissant

Is this issue solved?
If not ...

What Version of Raspberry Pi do you have?

Raspberry Pi 4 Model B has two micro HDMI outputs and when I remember correctly only one works with chilipie-kiosk.

Did you try to connect another display e.g. TV?


Visualization Unsuccessful, error

Hi! I'm getting this error, while visualizing my dataset on dashboard. Everything raedy but the visualization doesn't work out

What should I do?

Do you have source field in your payload?

Thank you sir, solved worked on it now tryna creat my own visualization dash board, inspired by qdrant and graph knowledge visualization.

tryna creat my own visualization dash board, inspired by qdrant and graph knowledge visualization.

Cool! Good luck with that!


sync: synced file(s) with lumirlumir/lumirlumir-configs

synced local file(s) with lumirlumir/lumirlumir-configs.
check out 3003191 @lumirlumir :hatching_chick:.

Changed files

synced local  .editorconfig with remote  ./configs/.editorconfig

This PR was created automatically by the repo-file-sync-action workflow run #10259511317

wrong sync. Mistake was occured by wrong initial setting for sync repo.


Updated keymap for mac

This just maps the command key onto the left Control key so I can use the keyboard with my Mac.

Hi, this isn't the sort of change that should be merged into the main repo