Sitemap priority.  Setting a value for the homepage that's different to the default

According to the sitemap documentation https://gohugo.io/templates/sitemap/ you can set defaults in the site's config file, and override individual page priority levels within the frontmatter.  But you can't set the homepage separately?
It seems that this commit https://github.com/spf13/hugo/commit/be3519fac06d828192bfd7942e9ff7daff787cc2 would prevent someone from setting their priorities like this:

example.com - 1.0
example.com/about -  0.9
example.com/anyotherpage - 0.5

It's possible I may be misunderstanding how sitemap priority works; I can't seem to find anything concrete that explains whether search engines automatically assume 1.0 priority for homepages.

I suspect this will be solved by #2297

This issue has been automatically marked as stale because it has not been commented on for at least four months.
The resources of the Hugo team are limited, and so we are asking for your help.
If this is a bug and you can still reproduce this error on the master branch, please reply with all of the information you have about it in order to keep the issue open.
If this is a feature request, and you feel that it is still valuable, please open a proposal at https://discuss.gohugo.io/.
This issue will automatically be closed in four months if no further activity occurs. Thank you for all your contributions.

Note/Update: This issue is marked as stale, and I may have said something earlier about "opening a thread on the discussion forum". Please don't.
If this is a bug and you can still reproduce this error on the latest release or the master branch, please reply with all of the information you have about it in order to keep the issue open.
If this is a feature request, and you feel that it is still relevant and valuable, please tell us why.

I guess it's more of a feature request in that it seems like the arrangement is semantically incomplete without explicitly being able to set a distinct priority for the homepage (effectively because it lacks its own frontmatter).  Otherwise what?  Set the priority to 1.0 and override every single other page explicitly?
I'm a bit out of my depth in terms of my knowledge in this area to be frank (not a developer, just some hobbyist neophyte).  So I'm open to someone telling me if I've got something wrong here.
I don't consider this high priority, regardless!  🙂

@pedrodude, can you confirm that this issue still exists with the latest Hugo release?

I'm still having issues with this (v0.25). My sitemap.xml ignores any attempt to override the default values on a page by page basis. Adding something like —
changefreq: "monthly"
priority: 0.5

— to a page's frontmatter has no effect, whether it's an _index.md or otherwise. Can anyone reproduce this? I'm curious to hear.


✨ [REQUEST] calibre-web

Which addon?
calibre-web
Is your feature request related to a problem? Please describe.
I want to convert books from this library to mobi format, to send them to kindle, but the addon need the calibre-convert binaries
Describe the solution you'd like
It would be gratefull to have this feature in the addon, as this web app is a very good way to manage the library
Describe alternatives you've considered
I also have the calibre addon, but I can't select the binaries from there, maybe it is not possible as they are different docker containers. If it is possible, will be a good idea to wirte how to in the readme.
In the docker container description says something about another image, that comes with this feature enabled:
https://github.com/janeczku/calibre-web:

Github - (Optional Calibre layer) - https://github.com/linuxserver/docker-calibre-web/tree/calibre
This image has the option to pull in an extra docker manifest layer to include the Calibre ebook-convert binary. Just include the >environmental variable DOCKER_MODS=linuxserver/calibre-web:calibre in your docker run/docker compose file. (x64 only)

Thank you very much in advance.

Hi, thanks for having done the preliminary search work. Indeed I've found the active linuxserver mod and have added it to the latest version of the addon. There is no readme on how to use it but I hope this will be enough for you ?

Hi, thanks for having done the preliminary search work. Indeed I've found the active linuxserver mod and have added it to the latest version of the addon. There is no readme on how to use it but I hope this will be enough for you ?

Its working perfect. Only needed config it is to set the converter external binary to "/usr/bin/ebook-convert".
Thnal you very much for your quick response nada solution.

Thanks for the feedback! Glad it works!


[Documentation] A website with Detailed Documentation

Request Statement
Kindly provide a full Dio documentation page with simple explanations, usage examples, and API references. Dio's acceptance and popularity in the community will be powered by a user-friendly interface that will make it easy for developers to obtain the required information.
Solution Brainstorm
No response

https://pub.dev/documentation/dio/latest/ should be enough?

Well, having enough is a good starting point, but let's take to the next level. Picture a complete website contains abundance of examples and explanations like how to use dio how to use it with dio cookies and stuff like that it will cater to various skill levels, ensuring everyone can grasp the concepts easily.

The idea is good, though we don't have enough resources to implement such improvements. Any PRs are welcome for the idea.

According to dartdocs documentation, we can include Markdown files in the generated docs. See https://pub.dev/packages/dartdoc#advanced-features. I guess that would be a good starting point.
That would make it easy to add, easy to maintain, and we don't have to host an extra webpage.

I guess it will be great it's an amazing effort from you @ueman  🙏


Describe global request hooks

What is your Scenario?
Use the new "Global Request Hooks" feature.
What are you suggesting?
We should describe the feature that allows assigning request hooks through a JS config: https://github.com/DevExpress/testcafe/pull/6649
What alternatives have you considered?
No response
Additional context
No response

Test

Done


Windows Latest on Python 3.8 Constantly Fails

For some reason, Windows-2022 is now failing for me on Python 3.8 only:
Run snok/install-poetry@v[1](https://github.com/adam-grant-hendry/qtpygraph/actions/runs/3202205327/jobs/5230946943#step:6:1).3.2
Run $GITHUB_ACTION_PATH/main.sh

Setting Poetry installation path as C:/Users/runneradmin/AppData/Roaming/Python/Scripts

Installing Poetry 👷

Retrieving Poetry metadata

# Welcome to Poetry!

This will download and install the latest version of Poetry,
a dependency and package manager for Python.

It will add the `poetry` command to Poetry's bin directory, located at:

C:\Users\runneradmin\AppData\Roaming\Python\Scripts\bin

You can uninstall at any time by executing this script with the --uninstall option,
and these changes will be reverted.

Installing Poetry (1.2.1)
Installing Poetry (1.2.1): Creating environment
Installing Poetry (1.2.1): An error occurred. Removing partial environment.
Traceback (most recent call last):
  File "C:/Users/RUNNER~1/AppData/Local/Temp/tmp.0IGyZ1qQC7", line 9[40](https://github.com/adam-grant-hendry/qtpygraph/actions/runs/3202205327/jobs/5230946943#step:6:42), in <module>
    sys.exit(main())
  File "C:/Users/RUNNER~1/AppData/Local/Temp/tmp.0IGyZ1qQC7", line 919, in main
    return installer.run()
  File "C:/Users/RUNNER~1/AppData/Local/Temp/tmp.0IGyZ1qQC7", line 550, in run
    self.install(version)
  File "C:/Users/RUNNER~1/AppData/Local/Temp/tmp.0IGyZ1qQC7", line 571, in install
    with self.make_env(version) as env:
  File "C:\hostedtoolcache\windows\Python\3.8.10\x64\lib\contextlib.py", line 113, in __enter__
    return next(self.gen)
  File "C:/Users/RUNNER~1/AppData/Local/Temp/tmp.0IGyZ1qQC7", line 643, in make_env
    raise e
  File "C:/Users/RUNNER~1/AppData/Local/Temp/tmp.0IGyZ1qQC7", line 629, in make_env
    yield VirtualEnvironment.make(env_path)
  File "C:/Users/RUNNER~1/AppData/Local/Temp/tmp.0IGyZ1qQC7", line 319, in make
    builder.create(target)
  File "C:\hostedtoolcache\windows\Python\3.8.10\x64\lib\venv\__init__.py", line 68, in create
    self._setup_pip(context)
  File "C:\hostedtoolcache\windows\Python\3.8.10\x64\lib\venv\__init__.py", line 289, in _setup_pip
    subprocess.check_output(cmd, stderr=subprocess.STDOUT)
  File "C:\hostedtoolcache\windows\Python\3.8.10\x64\lib\subprocess.py", line [41](https://github.com/adam-grant-hendry/qtpygraph/actions/runs/3202205327/jobs/5230946943#step:6:43)5, in check_output
    return run(*popenargs, stdout=PIPE, timeout=timeout, check=True,
  File "C:\hostedtoolcache\windows\Python\3.8.10\x64\lib\subprocess.py", line [49](https://github.com/adam-grant-hendry/qtpygraph/actions/runs/3202205327/jobs/5230946943#step:6:51)3, in run
    with Popen(*popenargs, **kwargs) as process:
  File "C:\hostedtoolcache\windows\Python\3.8.10\x64\lib\subprocess.py", line 8[58](https://github.com/adam-grant-hendry/qtpygraph/actions/runs/3202205327/jobs/5230946943#step:6:60), in __init__
    self._execute_child(args, executable, preexec_fn, close_fds,
  File "C:\hostedtoolcache\windows\Python\3.8.10\x[64](https://github.com/adam-grant-hendry/qtpygraph/actions/runs/3202205327/jobs/5230946943#step:6:66)\lib\subprocess.py", line 1311, in _execute_child
    hp, ht, pid, tid = _winapi.CreateProcess(executable, args,
FileNotFoundError: [WinError 2] The system cannot find the file specified
Error: Process completed with exit code 1.

Thanks for the report. Does changing v1 to v1.3.1 (the previous release) fix the issue for you?

@sondrelg How strange. Yes! That completely fixed it!
...What's happened from v1.3.1 to v1.3.2?

Specifically, the error only happened for Windows and version 3.8. MacOS and Linux on 3.8 worked and 3.9 and 3.10 on all OS's worked.

We mostly just quoted env vars in the latest release. I'll add a test case for Windows later to see if I can reproduce.
Would you be able to share the relevant parts of your workflow to make that easier?

@sondrelg It's a public repo, which makes it easier. Here's the link: https://github.com/adam-grant-hendry/qtpygraph

@sondrelg Feel free to look at my workflows

@sondrelg
Probably should add python 3.8 back to the test matrix?
https://github.com/snok/install-poetry/compare/v1.3.1..v1.3.2?diff=unified#diff-faff1af3d8ff408964a57b2e475f69a6b7c7b71c9978cccc8f471798caac2c88R12
Also, there is an open bug about this script not working with windows:
https://github.com/snok/install-poetry/compare/v1.3.1..v1.3.2?diff=unified#diff-f121df8f1f148ef72415f98dd27089a0e6cec3ec939840286358170e5e20c990R6
Ill try to find it. I think if you change this back
++curl -sSL https://raw.githubusercontent.com/python-poetry/poetry/48339106eb0d403a3c66519317488c8185844b32/install-poetry.py --output "$installation_script"
--curl -sSL https://install.python-poetry.org/ --output "$installation_script"

The problem will be fixed/

Removing -sSL and adding --ssl-no-revoke  to the installer arguments supposedly fixes it.
https://github.com/python-poetry/poetry/issues/6117

Here it is: https://github.com/python-poetry/install.python-poetry.org/issues/46

If you remove the shell default it will work

Interesting...that's been part of what was recommended previously for this action (i.e. to use bash and the default shell). @sondrelg isn't that true? Does this need to change going forward?

Of course there's a regression specifically for 3.8 just as we remove it from the matrix 😅 It's a bit of a pain to have 20+ permutations of the basic install test, but I suppose this is a prime example of why it's worth having.
Yes @adam-grant-hendry, IIRC, removing the bash shell default on windows causes other problems. Not sure if they've gone away yet, but would be good to know.
Either way, I suppose we could just switch to --ssl-no-revoke if that fixes it for all platforms. Don't think certificate revocation checks are relevant when curling the hardcoded official installation script, like we're doing here. Do you agree?

Surprisingly, CI doesn't fail when including windows 3.8 (here) 🤔

Surprisingly, CI doesn't fail when including windows 3.8 (here) thinking

If you add this to the test.yml it will fail
https://github.com/SickChill/sickchill/blob/test-deploy/.github/workflows/pythonpackage.yml#L27L29

There is this already
https://github.com/snok/install-poetry/blob/main/.github/workflows/test.yml#L58:L60
Are you saying the location of the default specification matters?

Either way, I suppose we could just switch to --ssl-no-revoke if that fixes it for all platforms. Don't think certificate revocation checks are relevant when curling the hardcoded official installation script, like we're doing here. Do you agree?

I'm perfectly fine sticking with v1.3.1 right now. I don't have an immediate need to upgrade. I personally used default: bash only because it was easier to stick to one scripting language instead of having to switch back-and-forth between that and powershell.
Not sure I can say I know much about the --ssl-no-revoke option or how it works, but if it works I'm fine with it! 😆

Are you saying the location of the default specification matters?

Looks like yours is specific to a job, whereas mine and @miigotu have ours for the entire workflow. Is that correct?

There is this already
https://github.com/snok/install-poetry/blob/main/.github/workflows/test.yml#L58:L60
Are you saying the location of the default specification matters?

I'm guessing because that job isnt creating a venv (https://github.com/snok/install-poetry/blob/main/.github/workflows/test.yml#L117) it skips the bug

In that issue they said its only with msys bash on windows, I have to check OP logs but here it is git bash
https://github.com/snok/install-poetry/actions/runs/3202507531/jobs/5231559465#step:3:104

I wonder if it is him setting: SHELLOPTS: errexit:pipefail and we do that also, causing some weirdness? I dont see anything else really different with the environment.

So far I've tried:

Setting Python to 3.8.10 which is the version specifically used in the failing run above
Added SHELLOPTS to env
Tried setting windows-2022 instead of windows-latest

but so far nothing reproduces

In the latest push to #95 I've added the --ssl-no-revoke command to main.sh.
@adam-grant-hendry, would you mind testing with @94 instead of v1.3.1 to see if that works for you?

I was having with the same issue than @adam-grant-hendry in my workflow using:
      - name: Install and configure Poetry
        uses: snok/install-poetry@v1
        with:
          version: 1.1.10
          virtualenvs-create: false

and (after trying to update to the latest version) with:
      - name: Install and configure Poetry
        uses<EMAIL_ADDRESS>        with:
          version: 1.2.1
          virtualenvs-create: false

After reading this thread and seeing the tests workflows @sondrelg shared, I used instead:
      - name: Install and configure Poetry
        uses<EMAIL_ADDRESS>        with:
          version: 1.1.14
          virtualenvs-create: false

And the error didn't occur anymore 🤷♂

We need to find the cause though. Can you try updating just one or the other of those? Aka is the problem here, or in poetry itself?

@miigotu: It worked on the windows-latest runner with the action version 1.3.1, but not with the version 1.3.2 using poetry version 1.1.14

Then perhaps the changes made to https://github.com/snok/install-poetry/blob/main/main.sh#L38:L40 is the reason.
The thing we need to get to the bottom of this is a reproducible example of a failing workflow for v1.3.2. So far we haven't been able to reproduce the issue. Any ideas as to why not @GuillaumeFalourd?

I've got a workflow reproducing the error here using this workflow implementation.

After making some tests in a fork, it seems the problem is coming from this commit [https://github.com/snok/install-poetry/commit/1218f0b98ae913ee2f2ad64868af8a3ca66511b9], by adding the set -eo pipefail line.
Just removing this line from the main.sh made the previous workflow which was failing working again: evidence

Great, thanks for that! I'll do a little bit more testing, then see if we can release a fix in a few hours 👍

https://askubuntu.com/questions/1044280/set-eo-pipefail-not-working-in-windows-subsystem-for-linux-ubuntu-16-04

Having had time to look at https://github.com/python-poetry/install.python-poetry.org/issues/46 @miigotu, I think they're saying git bash on windows runners are affected, so I think this is an upstream bug. The script target was changed a while back, but I hadn't released a new version until v1.3.2, so that's why this is happening.
Perhaps we should do a simple
if windows on python 3.8 or less:
    install using old script
else:
    install using new script

What do you think?


Added the ability to limit the size of a collection

As the title says, this patch adds the ability to limit the size of a collection by using a limit option in its definition.
My use case: in the home page of my website, I want to display the 10 last blog posts (and I don't want to write a template filter/function/operator ;-))

I think this type of feature has some merit, so I will work on getting it merged in. @dpobel if you have the time/interest to refactor the branch, it would help me a lot.

great news! I'm still interested as I'm using my custom branch in my projects.
I rebased the branch and fixed the conflict. The unit tests are passing. I hope that's ok.


Setting device for SapienRenderer doesn't work

System:

OS version: Ubuntu 22.04
Python version (if applicable): Python 3.10
SAPIEN version (pip freeze | grep sapien): 2.2.2
Environment: Server with xvfb

Describe the bug
#139 mentions setting device for SapienRenderer like this:
sapien_renderer = SapienRenderer(..., device="pci:0")

However, it seems the device somehow couldn't be found—
[2024-08-05 16:43:29.704] [svulkan2] [error] GLFW error: X11: The DISPLAY environment variable is missing
[2024-08-05 16:43:29.704] [svulkan2] [warning] Continue without GLFW.
[2024-08-05 16:43:29.944] [svulkan2] [info] Vulkan instance initialized
[2024-08-05 16:43:29.944] [svulkan2] [info] Devices visible to Vulkan
 Id                                    name   Present Supported    PciBus    CudaId     RayTracing
  0                   NVIDIA H100 80GB HBM3         0         1         0         0              1
  1                   NVIDIA H100 80GB HBM3         0         1         0         0              1

[2024-08-05 16:43:29.944] [svulkan2] [info] Devices visible to Cuda
    CudaId    PciBus             PciBusString
         0         0             000A:00:00.0
         1         0             000B:00:00.0

[2024-08-05 16:43:29.944] [svulkan2] [info] Vulkan finished
0it [00:22, ?it/s]
Traceback:
...
File "[...]/ManiSkill2_real2sim/mani_skill2_real2sim/envs/sapien_env.py", line 107, in __init__
    self._renderer = sapien.SapienRenderer(**renderer_kwargs)
RuntimeError: Cannot find cuda device suitable for rendering cuda:1

P.S. [error] GLFW error: X11: The DISPLAY environment variable is missing isn't a real error, since I can still run the code, if I don't specify device for SapienRenderer.
I tried both "cuda:1" and "pci:1" but neither worked.
However, my issue shouldn't be the same as #115 because I can run the code without specifying the device.
Could you tell me what the device format should be?

I am recently getting many different types of issues related to H100, probably because this GPU does not even include rendering cores to run graphics workloads. Your issue seems like a new one.
First, you can try using SAPIEN 3.0.0b1, SAPIEN 2 is a bit too old. Next you can try adding environment variable SAPIEN_DISABLE_RAY_TRACING=1, somehow simply enabling ray tracing can break H100 completely, even if the driver decides to report it can support ray tracing. For now my own workaround is just to avoid H100 altogether as it is not a good choice for rendering anyway.

Regarding the X11 error, your observation is correct. SAPIEN logs an "error" when it can successfully workaround it (in this case, SAPIEN simply disables on-screen display), otherwise it throws an exception.


[Feature] Refactor the code of consoles

Is your feature request related to a problem? Please describe.
Now there are several files for Consoles: Console.py, CnosoleMC.py, ConsolePlus.py, Interface.py
They should be merged into a single file to implement Console.

Maybe they can be aggregated into a new module, such as "IO" or "GUI".
I also considered to make a websocket-based IO channel for PyNARS based on websockets recently. Hope this will not make things more messy.

@ARCJ137442 I am working on GUI and adopt websocket to do communication among processes. Websocket is a low-level protocal, so I wrote a RPC package based on websocket.
See  https://github.com/bowen-xu/PyNARS/issues/7#issuecomment-1722370004, the rpc repo and branch gui


Support Gradle 5+

Is your feature request related to a problem? Please describe.
Gradle 5 adds support for Java 11 LTS.
Getting Gradle 5+ supported will unblock uPortal's path toward adopting Java 11 LTS.
Describe the solution you'd like
Build system that remains as close to the current setup as possible, but with Gradle 5.
Describe alternatives you've considered
Trying to run use Java 8 to run Gradle, and have Gradle try to compile with a separate JDK 11.
Additional context
This could then be merged into the ongoing #1007 work.

https://docs.gradle.org/5.1-rc-1/userguide/upgrading_version_4.html

resolved by #1501


Document (and setup integration test) for service account white/blacklisting

See: istio/issues#23.
We should add a few details to our default config and set up an integration test and a Task that references those files to demonstrate how to use Mixer with service accounts to limit service access.

cc: @mandarjog
This is something we need ASAP. Once the e2e transition to the new adapters, etc., is done, let's add this.


String Literal quotes

Why double quotes on string literals are preferable?

@lazycoder9 I don't exactly remember. This is just more useful for most datarockers (mostly me and @AleksSenkou ).
Also, in this case, we don't need to change the type of quotes on interpolation (which is really annoying) and can use ' symbol (which is really useful on printing test descriptions)

can use ' symbol (which is really useful on printing test descriptions)

e.g.
context "when the user's an admin"

Heard different opinion from @AleksSenkou 😄 But, anyway, if everyone agree with this, had no questions)

I also was the one who advocated for double-quotes for the reasons @roman-dubrovsky described:

no need to change them later when we need to add a ' symbol or interpolation;
all the string literals look the same.

I don't like double-quotes.

They take space and makes text unnecessary larger, for example: "apple" instead of 'apple'. Really? 😄
They show that there is no interpolation or ' symbol

@AleksSenkou in your case better to use :apple 🙂

I don't like double-quotes.
They show that there is no interpolation or ' symbol

@AleksSenkou didn't get your point?

me too. I don't like double-quotes cause some performance issue. They need more time (need to click two buttons instead of one 😄 )

They show that there is no interpolation or ' symbol

@roman-dubrovsky I mean single-quotes show that the string is just string without overloading

@AleksSenkou in your case better to use :apple 🙂

JFYI @DDKatch symbols are not equal strings

@lazycoder9 can we discuss anything else here? Or we can close this issue?

@roman-dubrovsky think everything is clear. Can be closed

@roman-dubrovsky does that mean that we go with double-quotes strings as default?

yes, It does

@AleksSenkou also I'm wondering. Is They show that there is no interpolation or ' symbol a joke?

Was "They show that there is no interpolation or ' symbol" a joke?

No, it's not a joke. single quotes show that the string is just a string.

does that mean that we go with double-quotes strings as default?
yes, It does

I don't like the fact that we didn't make voting or whatever to have results on which we can agree with one of the choices

No, it's not a joke. single quotes show that the string is just a string.

and which reason for this. I can understand it for interpolated string (but also don't understand which benefit we have if the string with interpolation will have another type of quotes. Also, that is really annoying that we need to make some additional changes just for adding interpolation).
But why should we make some separation between string with and w/o  ' symbol ? (they are just a string). + in this case, robocop does not allow us to use double quotes too and we should use something like
context 'when the user\'s permissions have admin access`

I don't like the fact that we didn't make voting or whatever to have results on which we can agree with one of the choices

@AleksSenkou why should we vote here? It was a question issue (no feature request) and @lazycoder9 accepted that we can close it after answering. Right?

why should we vote here?

@roman-dubrovsky I'd like clearly understand that most of us are on double-quotes or single-quotes side. I'd like not to have a situation when someone disagrees and just change cop on own project to use not what we agreed on.
Anyway, I'd go with voting on most of cops where we have polemic between people

@lazycoder9 accepted that we can close it after answering. Right?

@roman-dubrovsky it's true. I'm re-raising the question 😄
For me question is opened

@AleksSenkou ok, we can try to make voting but it does not resolve this point for me

I'd like not to have a situation when someone disagrees and just change cop on own project to use not what we agreed on.

And happens cause there aren't any comments about points why I prefer double-quotes and no points why we prefer single points from your side (only that this looks pretty (what is really personal) + point about interpolation which I didn't understand and don't get any answers on my question)
So if we really make voting, for me it'll look like Oh, man, you have some reasons for this, but it doesn't matter for us. Let's just make voting because of democracy 🎉

Anyway, let's I repeat my points by this question:
Why I prefer double quotes:

We don't need to change the type of quotes when we want to use interpolation or symbols such as \t, \n,  etc.

irb(main):009:0> puts "sd\n\:"
sd
:
=> nil
irb(main):010:0> puts 'sd\n\:'
sd\n\:
=> nil

@AleksSenkou  You wrote above that this is cons for you that we can recognize if the string literal has or doesn't have interpolation or special symbols.

They show that there is no interpolation or ' symbol

So can you explain to me which benefits we get in this case? Cause for me it looks really annoying. Let's see a typical working case:

you have some code

def say_hello
  puts 'Hello, Roma'
end

you need to add interpolation here. So you go to the string and adds # symbol

def say_hello
  puts 'Hello, #Roma'
end

sublime does not make interpolation by default here. So in this case, I just say Blyat', kak eto zaebalo, remove symbol # and go to change type of quotes

def say_hello
  puts "Hello, Roma"
end

now you can add interpolation

def say_hello
  puts "Hello, #{name}"
end

So my question: which benefits I can get so that I'll be agreed to make such things?
Also, we can get similar annoying things with removing interpolation:
E.g. you have some quick fix where you need to remove interpolation. And in these cases, you can skip checking the code locally via rubocop and make a PR for this. And then, when you see the red CI, you'll say Blyat', kak eto zaebalo and go running rubocop with autocorrection (and usually need to interrupt your current story)

we can use ' symbol with double quotes (but in this case, we have an issue with " symbol)

First of all, how can we resolve this issue:

use\ symbol for making all string consistent

'test \''
"test \""

does not make consistent string in this case (rubocop allows to do it)

# when style is single quotes
'test'
"test '"

# when style is double quotes
"test"
'test "'

The second one, let's discuss when we're using ' and " symbols in the string literals
' is our friend on writing good test messages (in the code we should use string literals and prefer locales) cause this symbol is an important part of English language. Yes, some cases we can fix via full form it 'does not create object' do instead of it 'doesn\'t create object' do but not all cases.
For " I know only one case (thank you @atatarenko ) when we need to copy some json or some HTML from a browser (usually in this case it contains " symbol)
And the last one: what I prefer to do in this case?
So coping json or HTML to the code is very uncommon case, so I prefer not to worry about it and just use ' symbol in this case (cause changing " to \" in html or json is very annoying).
Another case with ' case.  I'm facing with this comment every day, I don't really want to see \' or ton of string in Rspecs unconsistent

They take space and makes text unnecessary larger, for example: "apple" instead of 'apple'. Really?

I don't think so but this is really something which we need to get via voting. But in the current case, for me, this is a minor point cause practical thing more useful than so cosmetical. Does it make sense?

The single point, why I prefer to use a single quote that we use it in the eslint config so we don't need to worry about a different type of quotes when we're writing Rails backend with react app (like a habit issue). But this case we can fix via autocorrection

So, it's looks like holy war for now, but both variants is about style preference.
And single quotes have also a some preferences. For me its:

I understand when it just a word, so it's more easy to find some metamagic
It's takes less place, so i better see it's on my laptop's screen and don't need to scroll
It's looks pretty clean(my own opinion)
Less buttons need to be clicked
@roman-dubrovsky , for sure, if we want use \t, \n', 'we can use in this cases double quotes default rubocop rule allows you to do this And i totally disagree with yoursBlyat', kak eto zaebalo` points, because almost any code editor or ide can be setuped for your rubocop lint and show you errors or in other case you there are many around-commit tools, which will run autocorrect without your manipulations
So, my vote goes to single quotes

I understand when it just a word, so it's more easy to find some metamagic
It's takes less place, so i better see it's on my laptop's screen and don't need to scroll

@TheBlackArroVV can you explain it more? I don't really understand it.

, for sure, if we want use \t, \n, ' we can use in this cases double quotes default rubocop rule allows you to do this

yes, and we have the same issues as for interpolation https://github.com/datarockets/datarockets-style/issues/40#issuecomment-514593556

almost any code editor or ide can be setuped for your rubocop lint and show you errors or in other case you there are many around-commit tools, which will run autocorrect without your manipulations

no, there aren't good tools for Sublime editor. (some of them looks good by documentation but have a bad performance)
And, if we would have this tool, there isn't a tool which will change quotes for interpolation automagically, is there?

@TheBlackArroVV can you explain it more? I don't really understand it.

When i see 'word', and "#{word}" i understand where is just a string from first symbol

yes, and we have the same issues as for interpolation #40 (comment)

Sure, and i believe, that it's what can be changed for thing, that i described above in this answer

no, there aren't good tools for Sublime editor. (some of them looks good by documentation but have a bad performance)

So, there many around-commit tools, which works without editor

And, if we would have this tool, there isn't a tool which will change quotes for interpolation automagically, is there?

Sure, but will it save so much time to make it matter?

@TheBlackArroVV you forgot about this one

It's takes less place, so i better see it's on my laptop's screen and don't need to scroll

@roman-dubrovsky , o, it's easy
Just 'abc' takes less place then "abc", less place - more characters i can see

@TheBlackArroVV really? any screenshot with it?
cause the editor should render it with the same width

Type 20 symbols

@roman-dubrovsky , hmm, sure, it was my mistake, it's really don't work like this

@TheBlackArroVV as summary. 1-3 are just cosmetical things

It doesn't fix the issue by adding interpolation or special symbols.

About removing interpolation issue: yes, you're right but this is a very optimistic position (funny to see it from a guy who pushes broken code to github time to time 😉 ). Usually, we have this check on CI and nobody set up it locally. So agreed with your point, only when you set up needed git hook and shared it with other (so that everybody can use it) e.g. via https://github.com/datarockets/dotfiles Otherwise, I don't feel that this point resolved
Does it make sense?

Guysб looks like single/double quotes isn't something that can be discussed with arguments usage. I think that the root is just a personal taste. Sorry, I don't have the energy to leave my thoughts cause it's started to be not productive at all.
My personal taste is one-quote because it's esthetic to me, I like how it looks.
If most of us are for double-quotes - I will use them too.

@AleksSenkou I'm sorry but I'm really becoming angry when seeing your comment above.
First of all, you said that the question is opened and want to discuss it. Ok, I spent some hours and shared you my pain which I faced on my working experience (and I think I left good arguments here). And after that, you want to say that this does not matter and do not want to discuss it.
maybe this isn't a really important issue and we can live with any of styles but why did you want to discuss in this key?
P.S. I'm really grateful @TheBlackArroVV for his comment cause he also left his pain (I can understand Less buttons need to be clicked).

@roman-dubrovsky I didn't want to offend you somehow.
I just don't agree that your points are valuable.  I don't want another "разбор полетов" over here.
Look how much they said about it:

Those who agreed to use double quotes (thoughtbot) - https://github.com/thoughtbot/guides/pull/174
Those who agreed to use single quotes (rubocop) - https://github.com/rubocop-hq/ruby-style-guide/issues/96

As you can see none of them had full agreement ^. A lot of people gave arguments to use double quotes and not less to use single quotes

Guysб looks like single/double quotes isn't something that can be discussed with arguments usage. I think that the root is just a personal taste.

This is what I'm trying to say to you:

I don't know how to deal with it in our case, one option is just to disable this rule, see - https://github.com/rubocop-hq/ruby-style-guide/issues/96#issuecomment-141834351

ok, as a summary of this issue:

we won't force to use some required rule for this cop. The team or developers use their own decision which rule they will use in the project.

For legacy projects, we can suggest the next suggestion: call rubocop with both rules and choose which has fewer offenses.

Ping pls if you don't agree with that

@roman-dubrovsky agree 👍


Add missing durations

Sorry, accidentally deleted the branch

@th-we could you please resolve the conflicts?

still conflicting

@th-we would you be so kind to remove the conflicts? Otherwise we should close this PR.

@th-we Any chance that you would be able to resolve the conflicts? It seems that these are only connected to different formatting of the files.

Instead of resolving the conflicts, I created PR #71.


Django rest swagger views.py only uses first default renderer class

My django rest framework application is using a custom content-type application/vnd.api+json with the following default renderer classes:
'DEFAULT_RENDERER_CLASSES': (
        'api.base.renderers.JSONAPIRenderer',
        'rest_framework.renderers.JSONRenderer',
        'rest_framework.renderers.BrowsableAPIRenderer',
    ),

With this configuration, I get a 406 : NOT ACCEPTABLE error, but if I switch the order of the 'DEFAULT_RENDERER_CLASSES' so that rest_framework.renderers.JSONRenderer is first, then the swagger documentation will render.
I think this might be the cause: https://github.com/marcgibbons/django-rest-swagger/blob/master/rest_framework_swagger/views.py#L22-28
It looks like the view expects rest_framework.renderers.JSONRenderer to be first when ideally, the order should not matter.

This has been fixed in version 2.0; the SwaggerUI and the OpenAPI specs are now their own renderer classes.


Remove vcs package since its behaviour diverges from cmd/go import path resolution behaviour.

As stated in the issue https://github.com/golang/go/issues/11490
the package 'vcs' diverges significantly from cmd/go import path resolution
behaviour and thus needs to be removed for goreportcard to support
modules and versioning.
See also: https://golang.org/cl/159818.

I was having troubles generating the go report for my package https://github.com/NicoNex/echotron/v3 and I noticed that goreportcard didn't select the latest version (v3.17.0) even appending the /v3 at the end of the URL.


Pass terraform exit code through

Tested by stopping the default storage pool:
libvirt_network.backend: Creation complete (ID: d4689bf4-46a5-493e-8196-3864b15dc281)
Error applying plan:

4 error(s) occurred:

* libvirt_cloudinit.minion_ci[2]: 1 error(s) occurred:

* libvirt_cloudinit.minion_ci.2: can't find storage pool 'default'
* libvirt_volume.base_volume: 1 error(s) occurred:

* libvirt_volume.base_volume: can't find storage pool 'default'
* libvirt_cloudinit.minion_ci[0]: 1 error(s) occurred:

* libvirt_cloudinit.minion_ci.0: can't find storage pool 'default'
* libvirt_cloudinit.minion_ci[1]: 1 error(s) occurred:

* libvirt_cloudinit.minion_ci.1: can't find storage pool 'default'

Terraform does not automatically rollback in the face of errors.
Instead, your Terraform state file has been partially updated with
any resources that successfully completed. Please address the error
above and apply again to incrementally change your infrastructure.
kiall@kiall-z620:~/SUSE/caasp/terraform> echo $?
1

I'll merge this using "admin powers" without CI, as it's been tested + working, and CI is currently having TF issues and wasting time trying to deploy even when TF fails...


RCS-UI International payment consent details process implementation

Description

[ ] Add support to present the international payment consent details on RCS user interface Here
[ ] Add mock for international payment consent details here

Acceptance criteria

[ ] The consent page mock render properly with the consent details mocked.
[ ] The consent details page is render properly with all components need it.
[ ] The consent details (response from RCS-backend) is populated properly in the consent details page with no errors.

Hey team! Please add your planning poker estimate with ZenHub @andra-racovita @mariantiris


Create a single post page

As a user, I can view an article's details on a separate page.

As a logged-in user, I can add a comment to an article.

As a logged-in user, I can upvote an article.

single post page by clicking on comments


Two Virus Scanners On Virustotal Report "Malicious" For The Installer EXE File

Two Virus Scanners On Virustotal Report "Malicious" For The Installer EXE File

You already posted this in https://github.com/dvingerh/tb-vol-scroll/issues/49


Question. Memory and cpu limits

How can i configure asp dot net core to have memory and/or cpu usage limits like IIS app pools settings. We would like to use kestrel without IIS but we need to be able to setup that kind of resources usage limits to our applications.

Duplicate of https://github.com/aspnet/Home/issues/2300


fix: codeditor scroll cursor into view

Description
Was able to repro the issue on an isolated project where codemirror has auto height and has a scrollable container which does not take the full height of the viewport. Apparently scrolling for this scenario isn't handled in codemirror, related issue https://github.com/codemirror/codemirror5/issues/1104. https://github.com/codemirror/codemirror5/issues/2912#issuecomment-62161976 led me to figure out a solution using scrollCursorIntoView event.
The original source of the solution is from https://github.com/WebKit/WebKit-http/blob/817c46e152af795d735678386db68805d0aa505e/Source/WebInspectorUI/UserInterface/Views/CodeMirrorAdditions.js#L354 .
The solution here partially works in the property pane while scrolling down, but the issue remains while scrolling up. The property pane is wrapped with a position static wrapper and has some sticky headers which might be causing some issues. The fix for the property pane will have to be figured out separately in https://github.com/appsmithorg/appsmith/issues/19293
The solution involves scrolling to the cursor(which is an html element). We use the scroll library to only scroll when the target element is out of view.
Fixes https://github.com/appsmithorg/appsmith/issues/6325
DP: https://appsmith-7fo141hgf-get-appsmith.vercel.app/
Media
https://user-images.githubusercontent.com/67054171/233363358-8622b59d-d573-451b-ab84-22181395f394.mov
Type of change

Please delete options that are not relevant.

Bug fix (non-breaking change which fixes an issue)

How Has This Been Tested?

Manual

Test Plan

Add Testsmith test cases links that relate to this PR

Issues raised during DP testing

Link issues raised during DP testing for better visiblity and tracking (copy link from comments dropped on this PR)

Checklist:
Dev activity

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes
[ ] PR is being merged under a feature flag

QA activity:

[ ] Test plan has been approved by relevant developers
[ ] Test plan has been peer reviewed by QA
[ ] Cypress test cases have been added and approved by either SDET or manual QA
[ ] Organized project review call with relevant stakeholders after Round 1/2 of QA
[ ] Added Test Plan Approved label after reveiwing all Cypress test

/build-deploy-preview env=release

Tested the bug fix on the DP url and it looks good to me.

/ok-to-test

/ok-to-test

/build-deploy-preview env=release

/ok-to-test


As a billing owner on Cody Pro who is in dunning, me and my team should have 7 day grace period of using the product until me and my team will have access cut off if I do not have a successful payment transaction within that time period

Details being hashed out here: https://docs.google.com/spreadsheets/d/1QXnrHkgGLXdbfHEhI_VjtDTy-mCtnK1qri3IMQu7rNk/edit#gid=1512994667

Need to test, blocked on not being able to edit the payment info in subscription management form.

Moving this post-Feb because we won't have dunning in the first month

Confirmed this manually. The specifics can be configured within the Stripe console.
https://github.com/sourcegraph/cody/issues/1600
https://sourcegraph.slack.com/archives/C05SZB829D0/p1711646863422739?thread_ts=1711398277.413729&cid=C05SZB829D0


✨(api) improve team accesses endpoint

Purpose
Improve /teams/<id>/accesses/ endpoint to return user info
Relies on #60.
Fixes #64
Currently, serializer returns :
{
'id' : access.id
'user': access.user.id
'role': access.role
'abilitites': access.abilities
}

We want to

add user main identity's email
add user main identity's first and last name
remove user id ?

Proposal
Description...

[x] small commit to separate team accesses tests into diferent files.
[x] return serialized userinfo in /teams/<id>/accesses/
[x] tests
[x] fix tests suite
[x] optimize the number of queries ⚡

Improvement

[ ] improve permissions : have queries from unrelated users return 403

@AntoLC name and email can be null, ex: when creating a superuser, they don't have any identity at all, then these fields are null. We have to handle it in the frontend, because user login through SSO are only forced to have a valid sub. Email and Name can be null for whatever reason


Adding AWS X-Ray tracer with 5% default sampling rate.

Signed-off-by: Lulu Zhao<EMAIL_ADDRESS>Description: The implementation of X-Ray to be one of the tracing extension inside Envoy proxy, which with 5% sampling rate for spans.
Testing: Unit test, Integration test
Docs Changes: Need a separate section for X-Ray among with other tracers.
[#Issue] Related to: 5644
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Make sure you have done format fix as described here: https://github.com/envoyproxy/envoy/blob/master/support/README.md#fixing-format-problems
The format check is failing.

Once the formatting fixes are in (in particular using the Envoy abstractions for time, mutex, and condvar) I wonder if one of the people involved in Hystrix would be an appropriate initial reviewer, maybe @trabetti or @rgs1 ?

I will address the formatting issue later and thank you so much for taking a look at it.

cool; note that most of the formatting issues can be fixed automatically, though not the mutex/timer abstractions.

Agreed let's please break this up and make sure we can get a small PR in with the correct style/tests, etc. Thank you.

Thank you all for the suggestions. I will break it up to several small PRs. In order to start implementing X-Ray tracer, I was looking at how other tracers got implemented like Zikpin and try to do the same style like it. For example, I didn't use "= default" for the copy constructors is because of this. It got published so I thought I should do similar style. I will read the link about Envoy style coding and correct the style.


not working on nix os

[user@nixos:~/Documents/Source/hello-world]$ cargo zigbuild --target x86_64-pc-windows-gnu
Error: failed to copy file from /nix/store/wsi9bc39az42zmclk7fchx8vwssivmp0-zig-0.10.1/lib/zig/libc/mingw/lib-common/api-ms-win-core-synch-l1-2-0.def to /nix/store/wsi9bc39az42zmclk7fchx8vwssivmp0-zig-0.10.1/lib/zig/libc/mingw/lib-common/synchronization.def
Caused by:
Read-only file system (os error 30)

I don't use NixOS so please feel free to send PRs to fix it if you want to use it.


Subkey derived SR25519 keys do not display seeds.

Nope.  There are 32 byte seeds for sr25519 private keys but we call them MiniSecretKeys while ed25519 calls them SecretKeys, so maybe someone is only storing the 64 byte SecretKey somewhere.
Just fyi, I changed the naming because HDKD only works with the 64 byte version, so after doing HDKD then seed no longer makes sense, but that should not be happening here.

I testest this scenario, it seems this issue has been resolved.
@ ./target/debug/subkey generate
Secret phrase `honey initial impulse rebel arm visit lab easily physical sponsor power brisk` is account:
  Secret seed: 0xf85e5060ba7f646e6510afbd3331b77d1a3f5e3f59e7be794af7f1b6f21e509a
  Public key (hex): 0xc4f50bf1f331fa15a454d93a9459c7b3b4d4f67985c68868b8c7efdf0e8c8632
  Address (SS58): 5GWx35Tn7eKitR14UzrDmYt9jZnBzrXNjbV8X2zrLR356yp6

@ ./target/debug/subkey -p "" -e inspect "honey initial impulse rebel arm visit lab easily physical sponsor power brisk"
Secret phrase `honey initial impulse rebel arm visit lab easily physical sponsor power brisk` is account:
  Secret seed: 0xf85e5060ba7f646e6510afbd3331b77d1a3f5e3f59e7be794af7f1b6f21e509a
  Public key (hex): 0xee1171254f3b3f58c46bb5e64dafb89174ae45c769800cad0db20228474d4b6d
  Address (SS58): 5HSrSBprbvKv4vFXVabgPxmNuRENKbGFgd1ChAwPytFSPHDL

@ ./target/debug/subkey -p "" -s inspect "honey initial impulse rebel arm visit lab easily physical sponsor power brisk"
Secret phrase `honey initial impulse rebel arm visit lab easily physical sponsor power brisk` is account:
  Secret seed: 0xf85e5060ba7f646e6510afbd3331b77d1a3f5e3f59e7be794af7f1b6f21e509a
  Public key (hex): 0xc4f50bf1f331fa15a454d93a9459c7b3b4d4f67985c68868b8c7efdf0e8c8632
  Address (SS58): 5GWx35Tn7eKitR14UzrDmYt9jZnBzrXNjbV8X2zrLR356yp6

If it is solved, we can close this :)


Use .Values.kubecostModel.extraEnv

Fixes #20
The documentation found here https://docs.kubecost.com/troubleshooting/troubleshoot-install#configuring-log-levels is not correct so this should fix that!

Hi Any update on this?!

Hi Any update on this?!

@polarn thank you for the contribution! and @tomodachi for the ping here. We'll get this tested and merged soon and will update you here.

see https://github.com/kubecost/cost-analyzer-helm-chart/pull/3731

This should be added regardless of the logLevel fix, but future readers of this should set log levels of cost-model with
--set 'kubecostModel.logLevel=debug

The kubecost cloud docs are here: https://docs.kubecost.com/kubecost-cloud
This repo is for that platform.
The doc referenced in the PR is for kubecost self-hosted platform.
That said, there's no harm in this PR.


fix(title): Updating title

:tada: This PR is included in version 1.0.1 :tada:
Your semantic-release bot :package::rocket:


Feature.resource docstrings 205

@pjbreaux Please see the commit message for deets.
Fixes #205

Before you merge this I'd like to see what the generated docs look like. I don't think they will be what we had in mind for user facing docs.

Ok, can you walk me through how to generate today?

@pjbreaux OK, now it's correct...  -er.   I can show you how to render the docstrings for review.

@pjbreaux I addressed your typo fixes

Add :inherited-members: for each module which inherits from resource.Resource so we can see the inherited attributes and methods in the documentation.

The generated docs look good. I don't require any additional fixups.


Fix for move response_file and response_file_variables out of base package resource

Signed-off-by: Kapil Chouhan<EMAIL_ADDRESS>Description

Moved response_file and response_file_variables properties from package resource to apt_package and dpkg_package resources
Created a Chef::Provider::Package::Debian module for apt and dpkg providers and included it into Apt and Dpkg classes which are the sub-classes of the Package class, and also moved common methods of apt and dpkg providers in Chef::Provider::Package::Debian
Added test cases
Ensured chef-style on the code changes made

Issues Resolved
Fixes #7599 MSYS-986
Check List

[x] New functionality includes tests
[x] All tests pass
[ ] RELEASE_NOTES.md has been updated if required (not required for bugfixes, required for API changes)
[x] All commits have been signed-off for the Developer Certificate of Origin. See https://github.com/chef/chef/blob/master/CONTRIBUTING.md#developer-certification-of-origin-dco

I have verified all the test cases are working fine on locally but its failing on Travis build and AppVeyor build.

I have fixed Travis build and AppVeyor build failure, and Please let me know if you have any required changes.

hmm chefstyle seems annoyed about something though


TypeError in index.android.bundle:733

Error in mSupply Mobile
TypeError in index.android.bundle:733
undefined is not a function (evaluating 'this.openModal(b.ITEM_SELECT)')
View on Bugsnag
Stacktrace
index.android.bundle:733 - value

View full stacktrace
Created automatically via Bugsnag

An error linked to this issue has been marked as fixed in Bugsnag
TypeError in index.android.bundle:733


Fix #131: add own e2e tsconfig

Coverage remained the same at 100.0% when pulling bdd3567ed7c60347d0f8df96c641e4cf33d2c06b on fix-e2e into 67a8cdcb0d797ad7159c5b39c0c8a71dfbe46738 on master.


ODPI-97. HADOOP_GETCONF test should be using the hdfs command from us…

removing the absolute path prefix from the call to hdfs. Instead the one from $PATH will be called now (which is /usr/bin/hdfs normally)

::thumbsup::


Simpler dockerfile

What does this PR do
This PR simplifies the Dockerfile.builder to avoid some things were unnecessary. There's no need for both the deps and builder stages as they can just be combined into one, caching still works right. Also the previous Dockerfiles was copying ./pkg to /go/pkg (go toolchain's pkg...) unnecessarily and probably a footgun waiting to fire...?

Here's some output of me running docker buildx build a few times to show that caches are still used correctly:
First a build of what's in git
[19:12:38]-[~/p/g/m/conditionorc]-[manny@zennix]
docker buildx build --build-arg GIT_COMMIT=somecommit --build-arg GIT_BRANCH=manny-branch --build-arg GIT_SUMMARY=my-summary --build-arg VERSION=42 --build-arg BUILD_DATE=not-today --progress=plain  -f Dockerfile.builder .
#0 building with "default" instance using docker driver

#1 [internal] load build definition from Dockerfile.builder
#1 transferring dockerfile: 792B done
#1 DONE 0.0s

#2 [internal] load .dockerignore
#2 transferring context: 34B done
#2 DONE 0.1s

#3 [internal] load metadata for docker.io/library/alpine:3.18.0
#3 DONE 0.0s

#4 [internal] load metadata for docker.io/library/golang:1.21.1-alpine3.18
#4 DONE 0.0s

#5 [stage-1 1/3] FROM docker.io/library/alpine:3.18.0
#5 DONE 0.0s

#6 [build 1/6] FROM docker.io/library/golang:1.21.1-alpine3.18
#6 DONE 0.0s

#7 [internal] load build context
#7 transferring context: 2.69kB done
#7 DONE 0.0s

#8 [stage-1 2/3] RUN apk -U add curl
#8 CACHED

#9 [build 4/6] RUN go mod download
#9 CACHED

#10 [build 2/6] WORKDIR /go/src/github.com/metal-toolbox/conditionorc
#10 CACHED

#11 [build 3/6] COPY go.mod go.sum ./
#11 CACHED

#12 [build 5/6] COPY . ./
#12 CACHED

#13 [build 6/6] RUN GOOS=linux GOARCH=amd64 go build -o /usr/sbin/conditionorc -ldflags "-X github.com/metal-toolbox/conditionorc/internal/version.GitCommit=somecommit -X github.com/metal-toolbox/conditionorc/internal/version.GitBranch=manny-branch -X github.com/metal-toolbox/conditionorc/internal/version.GitSummary=my-summary -X github.com/metal-toolbox/conditionorc/internal/version.AppVersion=42 -X github.com/metal-toolbox/conditionorc/internal/version.BuildDate=not-today"
#13 CACHED

#14 [stage-1 3/3] COPY --from=build /usr/sbin/conditionorc /usr/sbin/
#14 CACHED

#15 exporting to image
#15 exporting layers done
#15 writing image sha256:ae6757b09f778f326b2ecb1d748d6a105c4b76fc3c60947faadf011ffe7d658f done
#15 DONE 0.0s

Now lets modify right at the end of the Dockerfile to make sure everything is cached as I expect:
[19:12:41]-[~/p/g/m/conditionorc]-[manny@zennix]
hx Dockerfile.builder
[19:12:53]-[~/p/g/m/conditionorc]-[manny@zennix]
git diff | command cat
diff --git a/Dockerfile.builder b/Dockerfile.builder
index e4c1adf..ae68069 100644
--- a/Dockerfile.builder
+++ b/Dockerfile.builder
@@ -26,4 +26,5 @@ RUN apk -U add curl

 COPY --from=build /usr/sbin/conditionorc /usr/sbin/

+RUN echo 43
 ENTRYPOINT ["/usr/sbin/conditionorc"]

and we re-run docker build:
[19:12:56]-[~/p/g/m/conditionorc]-[manny@zennix]
docker buildx build --build-arg GIT_COMMIT=somecommit --build-arg GIT_BRANCH=manny-branch --build-arg GIT_SUMMARY=my-summary --build-arg VERSION=42 --build-arg BUILD_DATE=not-today --progress=plain  -f Dockerfile.builder .
#0 building with "default" instance using docker driver

#1 [internal] load build definition from Dockerfile.builder
#1 DONE 0.0s

#1 [internal] load build definition from Dockerfile.builder
#1 transferring dockerfile: 804B done
#1 DONE 0.0s

#2 [internal] load .dockerignore
#2 transferring context: 34B done
#2 DONE 0.1s

#3 [internal] load metadata for docker.io/library/alpine:3.18.0
#3 DONE 0.0s

#4 [internal] load metadata for docker.io/library/golang:1.21.1-alpine3.18
#4 DONE 0.0s

#5 [build 1/6] FROM docker.io/library/golang:1.21.1-alpine3.18
#5 DONE 0.0s

#6 [stage-1 1/4] FROM docker.io/library/alpine:3.18.0
#6 DONE 0.0s

#7 [internal] load build context
#7 transferring context: 2.69kB done
#7 DONE 0.0s

#8 [build 2/6] WORKDIR /go/src/github.com/metal-toolbox/conditionorc
#8 CACHED

#9 [build 3/6] COPY go.mod go.sum ./
#9 CACHED

#10 [build 4/6] RUN go mod download
#10 CACHED

#11 [build 5/6] COPY . ./
#11 CACHED

#12 [build 6/6] RUN GOOS=linux GOARCH=amd64 go build -o /usr/sbin/conditionorc -ldflags "-X github.com/metal-toolbox/conditionorc/internal/version.GitCommit=somecommit -X github.com/metal-toolbox/conditionorc/internal/version.GitBranch=manny-branch -X github.com/metal-toolbox/conditionorc/internal/version.GitSummary=my-summary -X github.com/metal-toolbox/conditionorc/internal/version.AppVersion=42 -X github.com/metal-toolbox/conditionorc/internal/version.BuildDate=not-today"
#12 CACHED

#13 [stage-1 2/4] RUN apk -U add curl
#13 CACHED

#14 [stage-1 3/4] COPY --from=build /usr/sbin/conditionorc /usr/sbin/
#14 CACHED

#15 [stage-1 4/4] RUN echo 42
#15 0.221 42
#15 DONE 0.2s

#16 exporting to image
#16 exporting layers
#16 exporting layers 0.5s done
#16 writing image sha256:fd793972a2ed1b5ada58e845a916b374eeaaca39995bd48a5259a2dacebe7600
#16 writing image sha256:fd793972a2ed1b5ada58e845a916b374eeaaca39995bd48a5259a2dacebe7600 done
#16 DONE 0.5s

Note every layer says CACHED until [stage-1 4/4] RUN echo 42. Now we re-run and everything should be cached:
[19:13:02]-[~/p/g/m/conditionorc]-[manny@zennix]
docker buildx build --build-arg GIT_COMMIT=somecommit --build-arg GIT_BRANCH=manny-branch --build-arg GIT_SUMMARY=my-summary --build-arg VERSION=42 --build-arg BUILD_DATE=not-today --progress=plain  -f Dockerfile.builder .
#0 building with "default" instance using docker driver

#1 [internal] load build definition from Dockerfile.builder
#1 transferring dockerfile: 40B done
#1 DONE 0.1s

#2 [internal] load .dockerignore
#2 transferring context: 34B done
#2 DONE 0.1s

#3 [internal] load metadata for docker.io/library/golang:1.21.1-alpine3.18
#3 DONE 0.0s

#4 [internal] load metadata for docker.io/library/alpine:3.18.0
#4 DONE 0.0s

#5 [stage-1 1/4] FROM docker.io/library/alpine:3.18.0
#5 DONE 0.0s

#6 [build 1/6] FROM docker.io/library/golang:1.21.1-alpine3.18
#6 DONE 0.0s

#7 [internal] load build context
#7 transferring context: 2.69kB done
#7 DONE 0.0s

#8 [build 5/6] COPY . ./
#8 CACHED

#9 [stage-1 2/4] RUN apk -U add curl
#9 CACHED

#10 [build 6/6] RUN GOOS=linux GOARCH=amd64 go build -o /usr/sbin/conditionorc -ldflags "-X github.com/metal-toolbox/conditionorc/internal/version.GitCommit=somecommit -X github.com/metal-toolbox/conditionorc/internal/version.GitBranch=manny-branch -X github.com/metal-toolbox/conditionorc/internal/version.GitSummary=my-summary -X github.com/metal-toolbox/conditionorc/internal/version.AppVersion=42 -X github.com/metal-toolbox/conditionorc/internal/version.BuildDate=not-today"
#10 CACHED

#11 [stage-1 3/4] COPY --from=build /usr/sbin/conditionorc /usr/sbin/
#11 CACHED

#12 [build 4/6] RUN go mod download
#12 CACHED

#13 [build 3/6] COPY go.mod go.sum ./
#13 CACHED

#14 [build 2/6] WORKDIR /go/src/github.com/metal-toolbox/conditionorc
#14 CACHED

#15 [stage-1 4/4] RUN echo 42
#15 CACHED

#16 exporting to image
#16 exporting layers done
#16 writing image sha256:fd793972a2ed1b5ada58e845a916b374eeaaca39995bd48a5259a2dacebe7600 done
#16 DONE 0.0s

Re-edit and re-run, now we echo 43 not 42:
[19:13:07]-[~/p/g/m/conditionorc]-[manny@zennix]
hx Dockerfile.builder
[19:13:10]-[~/p/g/m/conditionorc]-[manny@zennix]
docker buildx build --build-arg GIT_COMMIT=somecommit --build-arg GIT_BRANCH=manny-branch --build-arg GIT_SUMMARY=my-summary --build-arg VERSION=42 --build-arg BUILD_DATE=not-today --progress=plain  -f Dockerfile.builder .
#0 building with "default" instance using docker driver

#1 [internal] load build definition from Dockerfile.builder
#1 transferring dockerfile: 804B done
#1 DONE 0.1s

#2 [internal] load .dockerignore
#2 transferring context: 34B done
#2 DONE 0.1s

#3 [internal] load metadata for docker.io/library/golang:1.21.1-alpine3.18
#3 DONE 0.0s

#4 [internal] load metadata for docker.io/library/alpine:3.18.0
#4 DONE 0.0s

#5 [stage-1 1/4] FROM docker.io/library/alpine:3.18.0
#5 DONE 0.0s

#6 [build 1/6] FROM docker.io/library/golang:1.21.1-alpine3.18
#6 DONE 0.0s

#7 [internal] load build context
#7 transferring context: 2.69kB done
#7 DONE 0.0s

#8 [build 3/6] COPY go.mod go.sum ./
#8 CACHED

#9 [build 4/6] RUN go mod download
#9 CACHED

#10 [build 5/6] COPY . ./
#10 CACHED

#11 [build 2/6] WORKDIR /go/src/github.com/metal-toolbox/conditionorc
#11 CACHED

#12 [build 6/6] RUN GOOS=linux GOARCH=amd64 go build -o /usr/sbin/conditionorc -ldflags "-X github.com/metal-toolbox/conditionorc/internal/version.GitCommit=somecommit -X github.com/metal-toolbox/conditionorc/internal/version.GitBranch=manny-branch -X github.com/metal-toolbox/conditionorc/internal/version.GitSummary=my-summary -X github.com/metal-toolbox/conditionorc/internal/version.AppVersion=42 -X github.com/metal-toolbox/conditionorc/internal/version.BuildDate=not-today"
#12 CACHED

#13 [stage-1 2/4] RUN apk -U add curl
#13 CACHED

#14 [stage-1 3/4] COPY --from=build /usr/sbin/conditionorc /usr/sbin/
#14 CACHED

#15 [stage-1 4/4] RUN echo 43
#15 0.211 43
#15 DONE 0.2s

#16 exporting to image
#16 exporting layers
#16 exporting layers 0.8s done
#16 writing image sha256:1366d5953033e151f30caa9dd058abaf66367a81933c9bbee9353683bc56e235
#16 writing image sha256:1366d5953033e151f30caa9dd058abaf66367a81933c9bbee9353683bc56e235 done
#16 DONE 0.8s

Lets re-write the commit message so stuff in git changes but that shouldn't cause any cache layers to be busted because we don't include git stuff:
[19:13:31]-[~/p/g/m/conditionorc]-[manny@zennix]
git commit --amend -C HEAD
[simpler-dockerfile 1db7e9f] docker: Simplify Dockerfile.builder
 Date: Wed Sep 20 19:07:50 2023 -0400
 2 files changed, 9 insertions(+), 13 deletions(-)
 create mode 100644 .dockerignore
[19:13:32]-[~/p/g/m/conditionorc]-[manny@zennix]
docker buildx build --build-arg GIT_COMMIT=somecommit --build-arg GIT_BRANCH=manny-branch --build-arg GIT_SUMMARY=my-summary --build-arg VERSION=42 --build-arg BUILD_DATE=not-today --progress=plain  -f Dockerfile.builder .
#0 building with "default" instance using docker driver

#1 [internal] load build definition from Dockerfile.builder
#1 transferring dockerfile: 40B done
#1 DONE 0.1s

#2 [internal] load .dockerignore
#2 transferring context: 34B done
#2 DONE 0.1s

#3 [internal] load metadata for docker.io/library/golang:1.21.1-alpine3.18
#3 DONE 0.0s

#4 [internal] load metadata for docker.io/library/alpine:3.18.0
#4 DONE 0.0s

#5 [stage-1 1/4] FROM docker.io/library/alpine:3.18.0
#5 DONE 0.0s

#6 [build 1/6] FROM docker.io/library/golang:1.21.1-alpine3.18
#6 DONE 0.0s

#7 [internal] load build context
#7 transferring context: 2.69kB done
#7 DONE 0.0s

#8 [stage-1 3/4] COPY --from=build /usr/sbin/conditionorc /usr/sbin/
#8 CACHED

#9 [build 2/6] WORKDIR /go/src/github.com/metal-toolbox/conditionorc
#9 CACHED

#10 [build 5/6] COPY . ./
#10 CACHED

#11 [build 4/6] RUN go mod download
#11 CACHED

#12 [stage-1 2/4] RUN apk -U add curl
#12 CACHED

#13 [build 3/6] COPY go.mod go.sum ./
#13 CACHED

#14 [build 6/6] RUN GOOS=linux GOARCH=amd64 go build -o /usr/sbin/conditionorc -ldflags "-X github.com/metal-toolbox/conditionorc/internal/version.GitCommit=somecommit -X github.com/metal-toolbox/conditionorc/internal/version.GitBranch=manny-branch -X github.com/metal-toolbox/conditionorc/internal/version.GitSummary=my-summary -X github.com/metal-toolbox/conditionorc/internal/version.AppVersion=42 -X github.com/metal-toolbox/conditionorc/internal/version.BuildDate=not-today"
#14 CACHED

#15 [stage-1 4/4] RUN echo 43
#15 CACHED

#16 exporting to image
#16 exporting layers done
#16 writing image sha256:1366d5953033e151f30caa9dd058abaf66367a81933c9bbee9353683bc56e235 done
#16 DONE 0.0s

and now lets change one of the build args everything should be cached until we actually make use of it (changing BUILD_DATE=not-today -> BUILD_DATE=yesterday):
[19:22:51]-[~/p/g/m/conditionorc]-[manny@zennix]
docker buildx build --build-arg GIT_COMMIT=somecommit --build-arg GIT_BRANCH=manny-branch --build-arg GIT_SUMMARY=my-summary --build-arg VERSION=42 --build-arg BUILD_DATE=yesterday --progress=plain  -f Dockerfile.builder .
#0 building with "default" instance using docker driver

#1 [internal] load build definition from Dockerfile.builder
#1 transferring dockerfile: 40B done
#1 DONE 0.1s

#2 [internal] load .dockerignore
#2 transferring context: 34B done
#2 DONE 0.1s

#3 [internal] load metadata for docker.io/library/golang:1.21.1-alpine3.18
#3 DONE 0.0s

#4 [internal] load metadata for docker.io/library/alpine:3.18.0
#4 DONE 0.0s

#5 [stage-1 1/4] FROM docker.io/library/alpine:3.18.0
#5 DONE 0.0s

#6 [build 1/6] FROM docker.io/library/golang:1.21.1-alpine3.18
#6 DONE 0.0s

#7 [internal] load build context
#7 transferring context: 2.69kB done
#7 DONE 0.0s

#8 [build 2/6] WORKDIR /go/src/github.com/metal-toolbox/conditionorc
#8 CACHED

#9 [build 3/6] COPY go.mod go.sum ./
#9 CACHED

#10 [build 4/6] RUN go mod download
#10 CACHED

#11 [build 5/6] COPY . ./
#11 CACHED

#12 [build 6/6] RUN GOOS=linux GOARCH=amd64 go build -o /usr/sbin/conditionorc -ldflags "-X github.com/metal-toolbox/conditionorc/internal/version.GitCommit=somecommit -X github.com/metal-toolbox/conditionorc/internal/version.GitBranch=manny-branch -X github.com/metal-toolbox/conditionorc/internal/version.GitSummary=my-summary -X github.com/metal-toolbox/conditionorc/internal/version.AppVersion=42 -X github.com/metal-toolbox/conditionorc/internal/version.BuildDate=yesterday"
#12 DONE 8.1s

#13 [stage-1 2/4] RUN apk -U add curl
#13 CACHED

#14 [stage-1 3/4] COPY --from=build /usr/sbin/conditionorc /usr/sbin/
#14 DONE 0.1s

#15 [stage-1 4/4] RUN echo 43
#15 0.249 43
#15 DONE 0.3s

#16 exporting to image
#16 exporting layers
#16 exporting layers 0.3s done
#16 writing image sha256:2ccaf16dedadc8c8bdddd113ca92a5535b084a18443620c9495990e5b3cf5f15 done
#16 DONE 0.3s

Everything is cached until go build :tada:

@DoctorVin its all green now so good to merge if you want


Build against multiple versions of ElasticSearch

Thank you @scorphus, very nice pull request!

GJ @scorphus

I'm glad to contribute!


TwitterSystemProvider ..  deprecated?

I have implemented code and see this when clicking the button in my UI that would trigger the code
    @IBAction func systemButtonPressed(_ sender: Any) {
        provider = TwitterSystemProvider()
        authorizeMe()
    }

AuthorizeMe] The identifiers for system account types were deprecated
Problem with provider

TwitterSystemProvider is deprecated because Apple removed supporting Facebook, Twitter, Vimeo, Flickr system accounts in iOS 11. You can use this type of accounts with iOS 10.
More details here - http://appleinsider.com/articles/17/06/05/apple-drops-twitter-facebook-flickr-vimeo-integration-in-ios-11

So that means you don't support Twitter now?

It's deprecated because iOS 11 doesn't support system accounts. But you can use it in iOS < 11.


Confusing \help statements

\\dt, \\di, \\help would be \dt, \di, \help.

Already fixed in the internal version but not updated.


Can't launch pyleecan: Could not load the Qt platform plugin "windows"

Hello,
I have installed pyleecan in conda in two laptops. In the first one I have no problem, but in the second one, which has a better processor, when I start with the tutorial of the Webminar of October the 16th in the second "block" of the Jupyter Notebook, when it launchs pyleecan I obtain the next error in my terminal:
qt.qpa.plugin: Could not load the Qt platform plugin "windows" in "" even though it was found.
This application failed to start because no Qt platform plugin could be initialized. Reinstalling the application may fix this problem.
The same if I launch pyleecan from terminal with python command.
I would thank any clue about why it is hapenning me in the 2ndd laptop.
Thanks in advance.
Juanjo Ugartemendia

Hello Juanjo,
If you use Windows, have you tried to setup the following path variable in your system path ?
"QT_QPA_PLATFORM_PLUGIN_PATH : path\to\anaconda3\Lib\site-packages\PySide2\plugins\platforms"
Best regards,
Emile

Hello Juanjo,
Please note that we have changed pyleecan during the webinar. The plot command in the first notebook are not used any more and will raise error if you are using pyleecan 1.0.3. The more efficient command that we introduced are detailed in the second webinar.
Best regards,
Pierre

Thank you!
That was the problem. I was trying with the first webminar's notebook, where I saw that the line where pyleecan was called was commented:
%run -m pyleecan
But I removed the # symbol.
Thank you!
Juanjo

Hello again,
sorry, but if I try to laucn pyleecan from a jupyter notebook or from my terminal, the kernel is restarted and it reports the error that I told to you. I have added that variable to the system path, but, it didn't fix the error. I can't launch pyleecan.
Juanjo Ugartemendia

Hello,
Which version of pyleecan are you using ? If it is not 1.0.3 please run "pip install -U pyleecan".
The error occurs when running the line to open the GUI ? Are you able to plot the machine at the beginning of the notebook ?
Best regards,
Pierre

Hello,
yes I have 1.03 version installed. And yes the error occurs when try to run the GUI: %run -m pyleecan.
The machine is plotted at the beginning of the notebook.
To say the truth, I usually work with Linux and I am not used to work with Windows. Perhaps I didn't set up correctly the variable you told me to the system path, thus:
"QT_QPA_PLATFORM_PLUGIN_PATH : path\to\anaconda3\Lib\site-packages\PySide2\plugins\platforms"
I added the path (path\to\anaconda3\Lib\site-packages\PySide2\plugins\platforms), to the system path. I didn't add any variable (I mean: QT_QPA_PLATFORM_PLUGIN_PATH)
Another question. I suppose it is not possible to work with Linux, because FEMM doesn't work in Linux, isn't it?
Thanks in advance for your help!
Juanjo Ugartemendia

Hello,
By waiting that the GUI issue is solved, you can start to use pyleecan in the notebook. The GUI only enables to define the "simple machine" in a more efficient way.
Just to be sure, did you update "path\to\anaconda3" to your own path (on my computer it is C:\Anaconda3)? We need to make a proper documentation on this issue, I will soon update the page on pyleecan.org.
I don't think that adding the path should work (but I may be wrong), you need to create the variable. This post seems to explain how: https://superuser.com/questions/949560/how-do-i-set-system-environment-variables-in-windows-10
Yes for now the only Magnetic model we have is FEMM so we are sadly windows only. If you are interested to help us on this topic, there is an issue to use FEMM on Linux #104 and one to add a new magnetic model with Elmer that should work on Linux #202.
Best regards,
Pierre

Hello Pierre,
Fixed! The problem was the path variable. I followed the instructions of your link and now it works! Thank you very much, now I must train with the help of your tutorials.
Regarding your propostion, in this moment what I want is to use your software. I knew FEMM, and I think it is a good software, but, I have been looking Elmer in the web and it looks great also and it is open soucde, it would be wonderful to integrate it in pyleecan. This way it would be 100% open source and by the way it would be possible to have a Linux version. In any case, I will continue using your software, we'll be in touch.
Congratulations to the whole the team!
Juanjo

Hello @jonjox,
I'm reviewing the opened issue and I wanted to tell you that since our last exchange there were several PR/discussions regarding Elmer #272 #291 #202 ;)
Can I close this issue or do you have further questions ?
Best regards,
Pierre

Hello Pierre, yes, you can close it, thank you very much!
Regards,
Juanjo
Pierre Bonneel<EMAIL_ADDRESS>igorleak hau idatzi zuen (2021
ots. 5, or. 22:37):

Hello @jonjox https://github.com/jonjox,
I'm reviewing the opened issue and I wanted to tell you that since our
last exchange there were several PR/discussions regarding Elmer #272
https://github.com/Eomys/pyleecan/pull/272 #291
https://github.com/Eomys/pyleecan/pull/291 #202
https://github.com/Eomys/pyleecan/issues/202 ;)
Can I close this issue or do you have further questions ?
Best regards,
Pierre
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/Eomys/pyleecan/issues/227#issuecomment-774301820, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/ARVOACGJAG4DRCS2V5W4FCDS5RQIHANCNFSM4TN2AV4A
.


Fixed M121 Lab2.1

Problem:
"For movies released in the USA..."
Solution (match added):
"countries": { $in: ["USA"] },
Problem:
"What is the title of the 25th film in the aggregation result?"
Solution:
// { $skip : 25 } wrong!
{ $skip : 24 }

Best regards,
Boris Traljic

Hey @btraljic ! Thanks for the fix! Is the solution still "The Heat"?

Hi @aurasphere
Yes, answer is "The Heat"...

My mistake, thanks for your help!


emc_insert: fix the bug of wrong replacement

In original implementation, when the first entry is alive, the second entry is not alive and the hash value of the second entry is less than the first, the replacement will occur on the second entry. This commit fix the problem.

Thanks for the patch, but it looks like your change is not passing the check patch script. For details see the following document in the repo https://github.com/openvswitch/ovs/blob/master/Documentation/internals/contributing/submitting-patches.rst.
In addition, it would be preferred to submit patches through email, as this will leverage various automation through patchwork, and you will get the robot to send you messages when some tests fail.
But more important this way other people working on this project will see your patch, as most of them are not monitoring the GitHub pull requests.
Cheers,
Eelco

Hi,
Thanks for your contribution.
I am closing this PR as it appears to have gone stale.
If you believe this change is still relevant then please do feel welcome to resubmit it using the preferred mechanism of the Open vSwitch project, the email workflow described in Submitting Patches.


the GUI isn't very intuitive?

I mean, how to use it? select and load the model -> upload file -> ?
after I did the model thing and clicked load, there's a processing number running, but I have no idea what it means

Thank you for the comment, the UI made with Gradio is not the best one. For now, you can just load in a model and chat with it, if you want to chat with a file, then upload a file and hit "start indexing"


displayed data reverts after completing a task

after completing a meal task the display will often revert back to a previous state.

[x] make sure that the local collection is refreshing after pushing data to server
[ ] make sure that the local server data is saving correctly
[ ] make sure that the local server data isn't getting overwritten by stale hub data (will need to add modified field to meal_plan model)

i think i may have fixed this... so far it hasn't reverted back after completing a task... and i wasn't refreshing the local cache so it's probably that it was just updating the display from the stale cache.... and that's fixed now.


Aug23 issues

Overall Review of Changes:
Several issues
new workflow files added
new badge layout on readme
Issue Fixes:
Issues:
Thanks to @zac90

#53
#54
#55

PR:
thanks to @rspataru

#50

Enhancements:
Updates to workflow now using centralised config
lint file updates
readme updates to badges and layout
added secrets-baseline in preperation for pre-commit
readme tidy up
How has this been tested?:
Manually

expect workflow failure once files are removed and tidied up


PCB reference

I have many open source PCB circuit diagrams here, which may be helpful for your design.
https://oshwhub.com/q12888873/mppt9635a
https://oshwhub.com/muyan2020/wei-yan-zheng-esp32-guang-fu-pv-mppt-ina226_copy_copy_copy_copy_copy

Thank you for sharing.
Interesting to see a buck-boost topology . Have you tried to use it with a wind generator?
I see you use the CSD19505KCS for high side (Q5, Q13). Did you measure the voltage wave form at switching nodes?
These switches are not designed for HS switching due to high Qrr and can cause a lot of ringing and EMI (also see my https://github.com/fl4p/fugu-mppt-firmware/issues/2#issuecomment-1701801832)
Maybe you are interested in seeing my PCB design here: https://cadlab.io/project/27217/master/files

You can try these measures, but unfortunately they can only be alleviated, not completely eliminated.
[img]https://i.imgs.ovh/2023/12/14/mt0Dt.jpeg[/img]


Clarified section Sample custom parameters file

Added verbiage on using the default arm-template-parameters-definition.json file and making additions/removals from it
Added example on how to add a databricks cluster id to parameters file

This stemmed from a service ticket w/ MS Support. This should help others.
I would also recommend splitting this page into a section about ADF DevOps and breaking this page down a bit. Too lengthy

@patpicos : Thanks for your contribution! The author, @douglaslMS, has been notified to review your proposed change.

@Jak-MS please assign to @craigg-msft

@patpicos apologies for the delay.  The changes look good to me.  Would you have time to please resolve the merge conflict and at me in the updated PR?  Thank you.

@patpicos : Thanks for your contribution! The author, @douglaslMS, has been notified to review your proposed change.

resolved the conflict. looks like the page was heavily modified since when i made the changes. looks like a lot of clarifications that are similar to mine ;)

#sign-off

@craigg-msft: I'm sorry - only the author of this article, @douglaslMS, can sign off on your changes. But we do have an exception process - if you are on the Microsoft content or product team for this product area, you can ask the PR review team to review and merge it by sending mail to the techdocprs alias.


Prettify all the type declarations

Closes #1106

Should @iov/cli's package.json match the same pattern for build/format-types?

I don't think it makes sense to prettify the build/ folder, which is not checked-in. We run prettier consistenty after mocing typed into the ./types folder, which has the advantage that we don't run prettier in spec files. I'd leave it as it is unless we have a good reason to change.


XGBoostClassifier fails to train on a large (>6TB) distributed dataset in a cluster

Hi, I have tried tweaking the number of executors, executor memory for spark-submit as well as the num_workers in XGBoost params set. However, I keep running into the following error:
21/10/06 02:25:47 INFO XGBoostClassifier: ml.dmlc.xgboost4j.scala.spark.XGBoostClassifier inferred 2 classes for labelCol=xgbc_1a1a7b72ecd0__labelCol since numClasses was not specified in the column metadata.
21/10/06 02:25:47 INFO XGBoostSpark: Running XGBoost 1.1.0 with parameters:

alpha -> 0.0
min_child_weight -> 1.0
sample_type -> uniform
base_score -> 0.5
rabit_timeout -> -1
colsample_bylevel -> 1.0
grow_policy -> depthwise
skip_drop -> 0.0
lambda_bias -> 0.0
silent -> 0
scale_pos_weight -> 104.0
seed -> 0
cache_training_set -> false
features_col -> features
num_early_stopping_rounds -> 5
label_col -> isPossiblySensitive
num_workers -> 27
subsample -> 1.0
lambda -> 1.0
max_depth -> 2
probability_col -> probability
raw_prediction_col -> rawPrediction
tree_limit -> 0
custom_eval -> null
dmlc_worker_connect_retry -> 5
rate_drop -> 0.0
max_bin -> 16
train_test_ratio -> 1.0
use_external_memory -> false
objective -> binary:logistic
eval_metric -> error
num_round -> 10
maximize_evaluation_metrics -> true
timeout_request_workers -> 1800000
missing -> 0.0
rabit_ring_reduce_threshold -> 32768
checkpoint_path ->
tracker_conf -> TrackerConf(0,python)
tree_method -> hist
max_delta_step -> 0.0
eta -> 0.1
verbosity -> 1
colsample_bytree -> 1.0
normalize_type -> tree
allow_non_zero_for_missing -> false
custom_obj -> null
gamma -> 0.0
sketch_eps -> 0.03
nthread -> 1
prediction_col -> prediction
checkpoint_interval -> -1
21/10/06 02:25:47 WARN XGBoostSpark: train_test_ratio is deprecated since XGBoost 0.82, we recommend to explicitly pass a training and multiple evaluation datasets by passing 'eval_sets' and 'eval_set_names'
Tracker started, with env={DMLC_NUM_SERVER=0, DMLC_TRACKER_URI=<IP_ADDRESS>, DMLC_TRACKER_PORT=9091, DMLC_NUM_WORKER=27}
21/10/06 02:25:47 INFO RabitTracker$TrackerProcessLogger: 2021-10-06 04:25:47,864 INFO start listen on <IP_ADDRESS>:9091
21/10/06 02:25:47 INFO XGBoostSpark: starting training with timeout set as 1800000 ms for waiting for resources
21/10/06 02:38:09 ERROR XGBoostTaskFailedListener: Training Task Failed during XGBoost Training: ExecutorLostFailure(13,true,Some(Remote RPC client disassociated. Likely due to containers exceeding thresholds, or network issues. Check driver logs for WARN messages.)), stopping SparkContext
21/10/06 02:38:09 ERROR RabitTracker: Uncaught exception thrown by worker:
org.apache.spark.SparkException: Job 12 cancelled because SparkContext was shut down
	at org.apache.spark.scheduler.DAGScheduler.$anonfun$cleanUpAfterSchedulerStop$1(DAGScheduler.scala:1085)
	at org.apache.spark.scheduler.DAGScheduler.$anonfun$cleanUpAfterSchedulerStop$1$adapted(DAGScheduler.scala:1083)
	at scala.collection.mutable.HashSet.foreach(HashSet.scala:79)
	at org.apache.spark.scheduler.DAGScheduler.cleanUpAfterSchedulerStop(DAGScheduler.scala:1083)
	at org.apache.spark.scheduler.DAGSchedulerEventProcessLoop.onStop(DAGScheduler.scala:2463)
	at org.apache.spark.util.EventLoop.stop(EventLoop.scala:84)
	at org.apache.spark.scheduler.DAGScheduler.stop(DAGScheduler.scala:2369)
	at org.apache.spark.SparkContext.$anonfun$stop$12(SparkContext.scala:2069)
	at org.apache.spark.util.Utils$.tryLogNonFatalError(Utils.scala:1419)
	at org.apache.spark.SparkContext.stop(SparkContext.scala:2069)
	at org.apache.spark.TaskFailedListener$$anon$2.$anonfun$run$1(SparkParallelismTracker.scala:119)
	at scala.runtime.java8.JFunction0$mcV$sp.apply(JFunction0$mcV$sp.java:23)
	at scala.util.DynamicVariable.withValue(DynamicVariable.scala:62)
	at org.apache.spark.TaskFailedListener$$anon$2.run(SparkParallelismTracker.scala:119)
	at org.apache.spark.scheduler.DAGScheduler.runJob(DAGScheduler.scala:868)
	at org.apache.spark.SparkContext.runJob(SparkContext.scala:2196)
	at org.apache.spark.SparkContext.runJob(SparkContext.scala:2217)
	at org.apache.spark.SparkContext.runJob(SparkContext.scala:2236)
	at org.apache.spark.SparkContext.runJob(SparkContext.scala:2261)
	at org.apache.spark.rdd.RDD.$anonfun$foreachPartition$1(RDD.scala:1020)
	at org.apache.spark.rdd.RDDOperationScope$.withScope(RDDOperationScope.scala:151)
	at org.apache.spark.rdd.RDDOperationScope$.withScope(RDDOperationScope.scala:112)
	at org.apache.spark.rdd.RDD.withScope(RDD.scala:414)
	at org.apache.spark.rdd.RDD.foreachPartition(RDD.scala:1018)
	at ml.dmlc.xgboost4j.scala.spark.XGBoost$$anon$1.run(XGBoost.scala:565)
java.util.concurrent.RejectedExecutionException: Task scala.concurrent.impl.CallbackRunnable@7b8a5cb4 rejected from java.util.concurrent.ThreadPoolExecutor@7bcd6cfe[Terminated, pool size = 0, active threads = 0, queued tasks = 0, completed tasks = 4925]

I would sincerely appreciate any help in figuring out what could potentially cause this or how I can address this issue. Thank you in advance!

Your system doesn't have enough memory for spark.

Hi @trivialfis ! Thank you for the quick response. Could you please give me some additional information on what does it mean when "system doesn't have enough memory for spark"?

terminate called after throwing an instance of 'std::bad_alloc'

This is an error that indicates memory allocation failure.


Add inline support for a range of useful text files

I added support for rendering the content of text files ... but I didn't stop at .txt files. This should be useful for web developers who want to share html, css, javascript, and markdown files!

Thanks for the PR! Merged and deployed.


Gradle setup

While setting up the project for development I noticed that the setup includes a number of manual steps and settings. This could increase the risk for errors and unnecessary troubleshooting. As I also fell into some traps my motivation was to automate some of these setup tasks with Gradle - which is also the preferred way recommended by JetBrains.
This PR provides a basic setup with Gradle for further evaluation and maybe it's useful for the project.

Wow!
I must admit I've never worked with Gradle (I'm mainly a web dev and not a Java dev) - but it seems a very interesting direction
I have a few question I hope you can answer:

What are the benefits of building with Gradle?
Can we use Gradle to do automated GUI testing (#205)? If so that would be amazing!
Why is it necessary to change the file structure from src/main/gitlfow to src/main/java/gitflow?

Thank you. As I'm also a novice using Gradle I'll try my best to answer your questions...

What are the benefits of building with Gradle?

I would say the main benefit is the ability to automate tasks from the perspective of building the project.
Example:
You have defined the following in the build.gradle file.
intellij {
    version '2018.2.1'
    plugins 'git4idea', 'tasks'
}

Gradle will take care about downloading this exact version of IntelliJ and the corresponding plugins. You don't need to manually download releases, set paths which could differ from machine to machine, etc.

Can we use Gradle to do automated GUI testing?

Gradle itself cannot be used for UI automation but it could trigger/handle it once some automated checks are implemented. I'm also checking the testing guides from Jet

Why is it necessary to change the file structure?

I'm not sure if it is actually necessary to change the structure. However it is the default used by Gradle Java projects.
Please see this link for more details:
https://docs.gradle.org/current/userguide/organizing_gradle_projects.html

@straurobNote that since I've merged the bugfix feature PR this PR can't be merged - address this and I'll merge this too, then I'll release a new version of the plugin to the jetbrains repo

I apologize for the delay - looks like there's one more thing to handle before the merge, that is that the BugFix files are in src/gitflow and the rest are in main/java.gitflow/
We need to be consistent with one or the other :)

Glad youd found this inconsistency. I corrected the structure so that all sources reside in the java directory.

Hi @OpherV, Is there any update on this pull request? Or are you busy? In the latter case, please excuse me :-)

Merged. Again sorry for the delay, I have a 3 month old baby which takes up pretty much all of my free time recently!
I have some more incoming PRs to review, then I'll upload a new version to the intellij repo so all of your hard work will finally be out :)

Sorry... I forgot about that but will remember in the future. Your baby is way more important than this :-) A very Merry Christmas to you and your family.


phantom run start on Docker (Linux / Ubuntu) call Error: spawn Unknown system error -8

Expected behavior
When I start on Docker, phantom must be called : spawn Unknown system error -8
Actual behavior
Error description：
{ Error: spawn Unknown system error -8
at _errnoException (util.js:1022:11)
at ChildProcess.spawn (internal/child_process.js:323:11)
at exports.spawn (child_process.js:502:9)
at new Phantom (/app/ws/node_modules/_phantom@4.0.12@phantom/lib/phantom.js:93:45)
at resolve (/app/ws/node_modules/_phantom@4.0.12@phantom/lib/index.js:15:41)
at new Promise ()
at Object.create (/app/ws/node_modules/_phantom@4.0.12@phantom/lib/index.js:15:10)
at createPdf (/app/ws/src/model/file/index.js:17:34)
at Object.exports.exportPdfFiles (/app/ws/src/model/file/index.js:71:13)
at /app/ws/src/controller/file/index.js:67:37
code: 'Unknown system error -8',
errno: 'Unknown system error -8',
syscall: 'spawn' }
Environment

OS: Alpine Linux running in a Docker container (node:8:11.1-alpine)
Node version (node -v):8.11.1
package version: 0.0.1

Could you give me solution please.

My Dockerfile:
FROM node:8.11.1-alpine
RUN apk update && apk add --no-cache fontconfig curl curl-dev &&
mkdir -p /usr/share &&
cd /usr/share
&& curl -L https://github.com/Overbryd/docker-phantomjs-alpine/releases/download/2.11/phantomjs-alpine-x86_64.tar.bz2 | tar xj
&& ln -s /usr/share/phantomjs/phantomjs /usr/bin/phantomjs
&& phantomjs --version

There is a lot of dependeices you need to install. You can use this file as an example. I don't know how to do it in Alpine.

Thx I will try it.


[AAP-4436] Combine the same playbook on multiple hosts

Run the same playbook only once based on multiple events in a
configurable time window through environmental variable
RUN_PLAYBOOK_DELAY (seconds). Combine hosts collected from all
related events.
Resolves AAP-4436: Combine the same action together on multiple hosts
based on multiple events in a certain period of time
This work only places run_playbook actions into the combinable queue. Technically all other actions can be treated the same way, but I feel there is no need since they are all quick actions. The only exception is the run_module action which can be easily handled the same way after the review.
depends on #202
replaces #159

@bzwei Can you add some real test cases to verify your logic, playbooks combined, hosts updated?

Added a test case to show playbook is run only once with combined hosts.


Bug: Weaviate panics on filtered vector search

The weaviate logs of the crashed weaviate:
panic: runtime error: index out of range [360640] with length 325000

goroutine 8531 [running]:
github.com/semi-technologies/weaviate/adapters/repos/db/vector/hnsw.(*hnsw).flatSearch(0xc00014b2c0, 0xc0bec78000, 0x200, 0x200, 0x320, 0xc12bb8ff50, 0x4450f2f57d02, 0x2f9fda9e, 0x2f9fda9e00000000, 0x626ba9ba, ...)
        /go/src/github.com/semi-technologies/weaviate/adapters/repos/db/vector/hnsw/flat_search.go:25 +0x465
github.com/semi-technologies/weaviate/adapters/repos/db/vector/hnsw.(*hnsw).SearchByVector(0xc00014b2c0, 0xc0bec78000, 0x200, 0x200, 0x320, 0xc12bb8ff50, 0xc12bb8ff50, 0x0, 0x0, 0xc1152ab0a0, ...)
        /go/src/github.com/semi-technologies/weaviate/adapters/repos/db/vector/hnsw/search.go:71 +0x1dd
github.com/semi-technologies/weaviate/adapters/repos/db.(*Shard).objectVectorSearch(0xc0001d0dc0, 0x14dd360, 0xc128cd4600, 0xc128f1ec00, 0x200, 0x2c7, 0x320, 0xc120eecf50, 0x1000000, 0x0, ...)
        /go/src/github.com/semi-technologies/weaviate/adapters/repos/db/shard_read.go:311 +0x106
github.com/semi-technologies/weaviate/adapters/repos/db.(*Shard).localObjectVectorSearch(0xc0001d0dc0, 0x14dd360, 0xc128cd4600, 0xc128f1ec00, 0x200, 0x2c7, 0x0, 0x320, 0xc120eecf50, 0x1000000, ...)
        /go/src/github.com/semi-technologies/weaviate/adapters/repos/db/shard_read.go:198 +0x1ca
github.com/semi-technologies/weaviate/adapters/repos/db.(*Index).objectVectorSearch.func1(0xffffffffffffffff, 0xc128e5ebc0)
        /go/src/github.com/semi-technologies/weaviate/adapters/repos/db/index.go:724 +0x236
golang.org/x/sync/errgroup.(*Group).Go.func1(0xc128563ef0, 0xc114874fc0)
        /go/pkg/mod/golang.org/x/sync@v0.0.0-20210220032951-036812b2e83c/errgroup/errgroup.go:57 +0x59
created by golang.org/x/sync/errgroup.(*Group).Go
        /go/pkg/mod/golang.org/x/sync@v0.0.0-20210220032951-036812b2e83c/errgroup/errgroup.go:54 +0x66

Query:
{
  Get {
    ProductImage (where: {
      operator: Equal
      path: ["ofProduct", "Product", "ofCategory", "Category", "category_filter"]
      valueString: "Men - Accessories - Bags - Briefcases"
    }, nearText: {
      concepts: ["brown leather briefcase"]
      certainty: 0.0
    }, limit: 800)
    {
      _additional {
        certainty
      }
      ofProduct {
        ... on Product {
          name
          productId
          ofCategory {
            ... on Category {
              category
            }
          }
          ofBrand {
            ... on Brand {
              name
              hasBadges {
                ... on Badge {
                  name
                }
              }
            }
          }
        }
      }
    }
  }
}

Weaviate has a CLIP, t2v and spellchecker module. There are 317628 objects in the ProductImage class and 349000 in the Product class. 339 of those match the where filter.

Note (to the assignee) that there is also a video with some more tech details around this issue on our internal Jira.


Improve /Page validation for linearized documents (issue 18138)

The referenced PDF document contains corrupt linearization-data, that doesn't point to the first page as intended.

/botio test

/botio-linux preview

Thank you for improving this!
/botio makeref


Map does not show up

Sorry for this might seem as a trivial question but I cannot get mapkit-react to show a proper map.
Token is valid and has been tested on the storybook page.
React is version 18.2.0.
My map page is just a simple Map tag with a proper token.
No special warniing when looking at the Chrome console apart from the following two warnings:
vTools failed to load source map: Could not load content for chrome-extension://lmbopdiikkamfphhgcckcjhojnokgfeo/src/tierces_parties/browser-polyfill.min.js.map: System error: net::ERR_BLOCKED_BY_CLIENT
vTools failed to load source map: Could not load content for https://mw-ci1-mapkitjs.geo.apple.com/admin/source-maps/5.75.95/source-maps/mapkit.min.js.map: Connection error: net::ERR_NAME_NOT_RESOLVED

Hello, thank you for your message! The first warning seems to be caused by the Antidote Chrome extension, but I don’t think that it is related to the issue you’re experiencing. The second warning is normal, it seems to appear in every project that uses MapKit JS.
The map will have the same height as its parent element. If the parent element has no height (e.g. it is an unstyled div), the map will also have zero height and will not appear on the screen. Could that be the origin of your problem?

Hi Nicolas,
I did try to set the height on the Map tag but that did not solve the problem.
You are absolutely right about the height of the containing div. This did solve the problem.
Thanks,
Jean-Marc


Summary printed twice.

Thanks for this theme that I'm trying to use.
I can't see where I'm wrong, but on each article the summary is treated twice.
Once in smaller type and then a second time with all the rest of the article text.
Can you please tell me how to avoid this?
Thank you for your help.

El artículo que estoy probando
Lorem ipsum dolor sit amet, consectetur adipiscing elit. …
12-03-24 15:34
Lorem ipsum dolor sit amet, consectetur adipiscing elit.
Sed convallis venenatis vehicula. Morbi ultrices finibus neque, ac vestibulum ante pharetra cursus. Nullam faucibus leo non dui porttitor, a sodales quam pharetra. Aliquam viverra, purus sit amet dictum rhoncus, sapien arcu vestibulum lacus, ac lobortis nunc dolor nec dui. Nullam cursus ligula at mi semper interdum. Donec id sapien ac justo pretium fringilla. Integer in sollicitudin turpis, a dignissim tellus. Nam ligula tortor, sagittis sit amet convallis eu, lacinia ut urna. Mauris volutpat metus at arcu ornare, vel sollicitudin sapien pulvinar. Ut efficitur ultricies ultricies. In efficitur felis enim, non mollis diam gravida nec. Phasellus odio risus, bibendum

I use TinyMCE Editor Integration v1.3.3 and the editor's window is only available in expert mode.
I also tried Telephasic. In the latter, TinyMCE Editor is available in standard mode and the summary is shown once.

The purpose of this summary is to be displayed after the title and subtitle, and before the image, date of the article and authors.
The article will be shown later, but obviously, like everything in text, summary included. However, you can disable the summary on each blog page, in the blog article tab.
In this article you can find information on how to exclude the summary of the content of the page.
Maybe in the next version I will implement this option in the theme.

Thank you for these explanations.
If I understand the logic of this feature, the summary is not a part of the article text as in the Telephasic theme, and in many others, but a separate text like a preface.

Please, where can I past the code ?
In the article's text ?

Near line 68, replace {% page.content|raw %} with:
{% set content = page.content|slice(page.summary|length) %}
{{ content|raw }}

However, I am giving some thought to this behavior of the summary, because it does not convince me.
In the next version, I will make some modifications to this.

In the next version, I will make some modifications to this.
Thanks, I'll keep an eye on it.

In the meantime, I consider that the summary is quite a sub-title.

Please, test the latest version 1.1.0 to view new option to show summary on pages, indepent of summary in blog list items.

Thank-you for that.
I don't have time this morning, but i tested the new summary function.
I see the parameter in the admin, but it has no effect in the display of the page. There is the summary at the top.
I will redo tomorrow or tonight.

Now the summary can bee the start of the article.

Now the summary can bee the start of the article.


Disable suffocation optimization for withers

The enableSuffocationOptimization from pufferfish usually does a great job reducing entity lag on huge servers.
It reduces the rate of checking for suffocation 10 times (which is collision box calculation and getBlock)
However, it also breaks suffocation based wither cages.
I feel like purpur goes mostly against tnt duping, so wither cages seem like the vanilla way to go around block farms.
This simple change would fix suffocation based wither cages on purpur with the optimization enabled, while still removing most of the lag from the suffocation check.
Given that there are usually only a few withers and thousands of other entities on the server, this option should be the best of both worlds on most of the servers.
Also from a practical standpoint, wither cages are the only useful contraptions that are broken by this patch.

This needs to either be in it's own patch or PR'd to Pufferfish. Inline your imports as well

I inlined the imports for now. Which option do you think would be the best if I were to make it into a separate patch? Patch it the way it is (withers are always not optimized as they're very rare mobs) or add a separate switch for less aggressive optimization so that some servers may still want to optimize all mobs?
If you could propose the name of that additional switch, I can make a PR with that

Maybe something like mobs.wither.ignore-suffocation-optimization? Although I'm inclined to include "pufferfish" somewhere in the config name since it's otherwise implied that Purpur adds the optimization, not Pufferfish. But the name might be confusing if it's ignore-pufferfish-suffocation-optimization... i'm not too sure. I'd rather it be PR'd to Pufferfish, tbh, but I don't think it will get merged anytime soon.

looks like the PR you made for pufferfish, https://github.com/pufferfish-gg/Pufferfish/pull/80, is almost ready to be merged into pufferfish. I'll go ahead and close this PR in favor of that one. With the assumption that it will take a while to merge, i've gone ahead and pulled it downstream into it's own patch.