[MRG+1] Add possibility to plot a source space in head coordinates

currently, this is broken:
import mne
path = mne.datasets.sample.data_path()
fwd = mne.read_forward_solution(path + '/MEG/sample/sample_audvis-meg-oct-6-fwd.fif')
fwd['src'].plot(subjects_dir=path + '/subjects')

It is occasionally useful to plot the source space inside a forward operator, to see which vertices may have been discarded due to falling outside the skull. In this case, the source space is in head coordinates, so things don't line up. This PR allows the user to specify a head<->MRI transformation to make the plotting work:
import mne
path = mne.datasets.sample.data_path()
fwd = mne.read_forward_solution(path + '/MEG/sample/sample_audvis-meg-oct-6-fwd.fif')
fwd['src'].plot(subjects_dir=path + '/subjects',
                trans=path + '/MEG/sample/sample_audvis_raw-trans.fif')

LGTM
@larsoner merge if you're happy too

Thanks @wmvanvliet


chore(data): added queue visualizer

Hey @ming-tsai ,
Please review it
Thanks

Hi @UG-SEP,
Does the repo is on private?

Sorry for that Link


Allow to do some actions before container process stop

Background:
The OCI hooks mechanism looks like a good place to register container in discovery service or add it to load balancer pool - as we have Prestart and Poststart hooks, but there is a problem when we want to remove old containers after new version deployment. We only have Poststop hook which is fired during container deletion (so process inside container is already dead), which in practice makes impossible to gracefully remove old containers (e.g. by using load balancer connection draining feature). When a single container handles high traffic (thousands of requests per seconds) it ends with a lot of 502 HTTP errors for the site users. Currently each popular container management software provides some way to achieve graceful container removal (Kubernetes has PostStart/PreStop container lifecycle hooks, on Mesos we can write custom executors with that logic), but OCI spec is missing that feature.
Question:
Can the OCI spec be extended to allow to do some actions before container process stop in a standardised way?

Kubernetes has PostStart/PreStop container lifecycle hooks...

The spec doesn't have pre-stop hooks, because it doesn't have a stop operation.    You'll probably want logic like:

Take the container out of the load balancer.

Wait for a gracefull shutdown, which for servers may include queue draining.
Optionally use kill to send KILL, if gracefull shutdown takes longer then you want.
delete once the process exits.

That's pretty easy to do if you can tell the controller to fire 1 (and 3, if any monitoring is necessary there), and thrre's no need to involve the runtime.  But since the create / start split, there's no need for hooks at all (#483).  So if we wanted new hooks for this case, I think we'd want prekill and postkill entries taking signal keys, and inside those per-signal entries the usual hook array:
{
  ...,
  "prekill": {
    "TERM": [
      ...remove from load balancer...
    ]
  }
}

That gets a bit wiggly, because the spec says nothing about supported kill signals (although the in-fligh opencontainers/runtime-tools#321 would require support for at least TERM and KILL).  And there's no guarantee that the runtime (or even the controller) would be sending the signal (the container process could crash on its own, among many other possibilities).  So I'm not sure it's worth supporting this in hooks, but if we want to, that pre/postkill structure might be sufficient.

And there's no guarantee that the runtime (or even the controller) would be sending the signal (the container process could crash on its own, among many other possibilities).

In my opinion this is not a good argument for this use case. To be honest, as you go along this reasoning, no hook is guaranteed to be run because the runtime controller process can crash in any moment also (as any other component of your infrastructure can fail).
I understand that it is impossible to guarantee a 100% effectiveness of a graceful shutdown mechanism in all situations, but 99% effectiveness in every day use cases (new version deployments, host maintenance etc.) is a great business value, because it allows you to work almost freely (and safely) with the containers.

On Mon, Oct 02, 2017 at 09:16:14AM +0000, Adam Medziński wrote:

To be honest, as you go along this reasoning, no hook is guaranteed
to be run because the runtime controller process can crash in any
moment also (as any other component of your infrastructure can
fail).

Fair.  And the fact that the signals supported by ‘kill’ are
undocumented is clearly something that could be addressed by the spec
if the maintainers agree that it's useful (I think it is, regardless
of whether kill-related hooks are supported).
This is really a judgement call for maintainers about the scope of
hooks in the spec.  I don't know if we have maintainers speaking for
themselves somewhere on that topic, but based on my paraphrasing in
1, it's possible that the current hooks were just grandfathered into
the spec because they had existing users, and that the maintainers
would rather not add additional hooks because folks can always address
these issues in higher levels (e.g. in containerd or whatever
controller you use).  Or it's possible that they want pre- and
post-hooks for all operations 2.  Or it's possible that they want to
debate the utility of pre- and post-hooks for each case, and only
require runtime support for those which they deem sufficiently useful.
If we do add pre- and post-kill hooks, an alternative to my previous
suggestion 3 might be to drop the signal keys and add a token
(‘${SIGNAL}’?) to pass the chosen signal through to the hook:
$ cat config.json
{
…,
"hooks": {
"prekill": [
{
"path": "/usr/bin/my-pre-kill-hook"
"args": ["ab", "${SIGNAL}", "cd"],
}
]
}
}
$ cat /usr/bin/my-pre-kill-hook
#!/bin/sh
AB="${1}"
SIGNAL="${2}"
CD="${3}"
STATE="$(cat)"  # read the state JSON from stdin
case "${SIGNAL}" in
TERM)
…whatever…
;;
KILL)
…other stuff…
;;
USR1|USR2)
…more stuff…
;;
esac
…
That would also allow the spec to continue to dodge the issue of
signal names, although I don't see a need to do so.

@wking Should I make a pull request with proposed changes or more opinions is needed?

Should I make a pull request with proposed changes or more opinions is needed?

Working up a pull request is more work for you, and the maintainers may end up rejecting it if you don't happen to guess what they want.  But it might increase the chances of a maintainer chiming in on this point, and that would help.  The December meeting is just around the corner; I'll try to get some opinions then.  If you can make the meeting, you may wan't to argue for your preferred approach.  I don't really care if the spec gets kill hooks or not.  They are convenient (good) but feel like scope-creep (bad) and those balance out for me.  I'd be more enthusiastic if we had a separate runtime-controller layer with all our hooks, but that would be a bigger shift.
If we don't get maintainer opinions at the meeting and you're comfortable investing time in a PR that might be rejected, then yoh can work up a PR.  Or if you don't mind investing time and want something more concrete to discuss at the meeting, you can work up a PR now.

Anything new to report here?
Would like to use it as podman hook to retrieve ip information of the used interfaces in the container.
After the container stops, all that information is lost.


Implement class based websocket consumer

Codecov Report

Merging #7 into master will increase coverage by 1.17%.
The diff coverage is 100%.

@@           Coverage Diff            @@
##           master     #7      +/-   ##
========================================
+ Coverage   98.82%   100%   +1.17%
========================================
  Files           8      7       -1
  Lines          85     83       -2
========================================
- Hits           84     83       -1
+ Misses          1      0       -1

Impacted Files
Coverage Δ

...a_browser_client/alexa_browser_client/consumers.py
100% <100%> (ø)
:arrow_up:

alexa_browser_client/__init__.py
100% <100%> (ø)
:arrow_up:

...exa_browser_client/alexa_browser_client/helpers.py
100% <100%> (ø)
:arrow_up:

alexa_browser_client/config/routing.py
100% <100%> (ø)
:arrow_up:

alexa_browser_client/alexa_browser_client/views.py
100% <100%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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


Initial implementation

This change is

Что такое "botv"?

ещё не всё исправлено. Author name here, например

Reviewed 6 of 9 files at r1, 5 of 5 files at r2.
Review status: all files reviewed at latest revision, 25 unresolved discussions.

app/TelegramApi.hs, line 13 at r2 (raw file):

sendMessage token message = do
 manager <- newManager tlsManagerSettings
 request <- parseRequest ("https://api.telegram.org/bot"++ token ++"/sendMessage?text="++message++"&chat_id=193856114")

Нужны пробелы вокруг операторов (++).

app/TelegramApi.hs, line 16 at r2 (raw file):

 response <- httpLbs request manager
 putStrLn $ "The status code was: " ++ show (statusCode $ responseStatus response)
 pure(responseBody response)

Нужен пробел между функцией pure и её аргументом.

app/TelegramApi.hs, line 17 at r2 (raw file):

 putStrLn $ "The status code was: " ++ show (statusCode $ responseStatus response)
 pure(responseBody response)

Нужен перевод строки в конце файла.

Comments from Reviewable

Reviewed 7 of 7 files at r3.
Review status: all files reviewed at latest revision, 10 unresolved discussions.

app/Main.hs, line 8 at r3 (raw file):

main = do
    str <- getLine
    mes <- sendMessage "" str

может, token?

Comments from Reviewable

Review status: all files reviewed at latest revision, 11 unresolved discussions.

app/TelegramApi.hs, line 4 at r3 (raw file):

    ( sendMessage
    ) where
import Network.HTTP.Client       (httpLbs, responseStatus, responseBody, newManager, parseRequest)

нужна пустая строка между заголовком модуля и списком импортов

Comments from Reviewable

LICENSE, line 16 at r1 (raw file):
Previously, cblp (Yuriy Syrovetskiy) wrote…
> И здесь своё имя впиши.

Done.

Comments from Reviewable

stack.yaml, line 45 at r1 (raw file):
Previously, cblp (Yuriy Syrovetskiy) wrote…
> Этот параметр не нужен.

Done.

Comments from Reviewable

stack.yaml, line 6 at r2 (raw file):
Previously, cblp (Yuriy Syrovetskiy) wrote…
> Последние 2 строчки можно просто стереть.

Done.

Comments from Reviewable

stack.yaml, line 6 at r2 (raw file):
Previously, kemphack wrote…
> Done.

Done.

Comments from Reviewable

stack.yaml, line 6 at r2 (raw file):
Previously, kemphack wrote…
> Done.

Done...

Comments from Reviewable

TaskBot.cabal, line 3 at r2 (raw file):
Previously, cblp (Yuriy Syrovetskiy) wrote…
> Task bot

Done.

Comments from Reviewable

TaskBot.cabal, line 5 at r2 (raw file):
Previously, cblp (Yuriy Syrovetskiy) wrote…
> https://github.com/MCL1303/TaskBot

Done.

Comments from Reviewable

TaskBot.cabal, line 16 at r2 (raw file):
Previously, cblp (Yuriy Syrovetskiy) wrote…
> Стиль: должна быть 1 пустая строка, не 2.

Done.

Comments from Reviewable

app/Main.hs, line 3 at r1 (raw file):
Previously, cblp (Yuriy Syrovetskiy) wrote…
> Надо в скобках написать явно, что импортируется

Done.

Comments from Reviewable

app/Main.hs, line 9 at r1 (raw file):
Previously, cblp (Yuriy Syrovetskiy) wrote…
> Функция не используется

Done.

Comments from Reviewable

app/Main.hs, line 13 at r1 (raw file):
Previously, cblp (Yuriy Syrovetskiy) wrote…
> Надо сделать отступ в 4 пробела здесь и далее, в соответствии с https://github.com/tibbe/haskell-style-guide

Done.

Comments from Reviewable

app/Main.hs, line 7 at r2 (raw file):
Previously, cblp (Yuriy Syrovetskiy) wrote…
> Отступы

Done.

Comments from Reviewable

app/Main.hs, line 8 at r3 (raw file):
Previously, cblp (Yuriy Syrovetskiy) wrote…
> может, token?

Тут мы получаем ответ, в виде JSON, типа message(Telegram Api), который содержит информацию об отправленном сообении

Comments from Reviewable

app/Main.hs, line 8 at r3 (raw file):
Previously, kemphack wrote…
> Тут мы получаем ответ, в виде JSON, типа message(Telegram Api), который содержит информацию об отправленном сообении

Аа.. этот токен.. сейчас..

Comments from Reviewable

app/Main.hs, line 14 at r1 (raw file):
Previously, cblp (Yuriy Syrovetskiy) wrote…
> Не хватает пробелов вокруг операторов.

Done.

Comments from Reviewable

app/Main.hs, line 18 at r1 (raw file):
Previously, cblp (Yuriy Syrovetskiy) wrote…
> Не хватает перевод строки в конце файла.

Done.

Comments from Reviewable

app/TelegramApi.hs, line 12 at r2 (raw file):
Previously, cblp (Yuriy Syrovetskiy) wrote…
> Отступы

Done.

Comments from Reviewable

app/TelegramApi.hs, line 9 at r2 (raw file):
Previously, cblp (Yuriy Syrovetskiy) wrote…
> Странный комментарий. Что имелось в виду?

Done.

Comments from Reviewable

app/TelegramApi.hs, line 13 at r2 (raw file):
Previously, cblp (Yuriy Syrovetskiy) wrote…
> Нужны пробелы вокруг операторов (++).

Done.

Comments from Reviewable

app/TelegramApi.hs, line 17 at r2 (raw file):
Previously, cblp (Yuriy Syrovetskiy) wrote…
> Нужен перевод строки в конце файла.

Done.

Comments from Reviewable

app/TelegramApi.hs, line 16 at r2 (raw file):
Previously, cblp (Yuriy Syrovetskiy) wrote…
> Нужен пробел между функцией `pure` и её аргументом.

Done.

Comments from Reviewable

SuzianBot.cabal, line 10 at r1 (raw file):
Previously, cblp (Yuriy Syrovetskiy) wrote…
> Здесь твоё имя

Done.

Comments from Reviewable

SuzianBot.cabal, line 16 at r1 (raw file):
Previously, cblp (Yuriy Syrovetskiy) wrote…
> Оставь одну пустую строку.

Done.

Comments from Reviewable

SuzianBot.cabal, line 3 at r1 (raw file):
Previously, cblp (Yuriy Syrovetskiy) wrote…
> "Task bot"

Done.

Comments from Reviewable

SuzianBot.cabal, line 34 at r1 (raw file):
Previously, cblp (Yuriy Syrovetskiy) wrote…
> https://github.com/MCL1303/TaskBot

Done.

Comments from Reviewable

SuzianBot.cabal, line 5 at r1 (raw file):
Previously, cblp (Yuriy Syrovetskiy) wrote…
> https://github.com/MCL1303/TaskBot

Done.

Comments from Reviewable

SuzianBot.cabal, line 8 at r1 (raw file):
Previously, cblp (Yuriy Syrovetskiy) wrote…
> И здесь твоё имя.

Done.

Comments from Reviewable

SuzianBot.cabal, line 9 at r1 (raw file):
Previously, cblp (Yuriy Syrovetskiy) wrote…
> Здесь твоя почта

Done.

Comments from Reviewable

SuzianBot.cabal, line 1 at r2 (raw file):
Previously, cblp (Yuriy Syrovetskiy) wrote…
> Один кабалфайл лишний.

Done.

Comments from Reviewable

src/Lib.hs, line 1 at r1 (raw file):
Previously, cblp (Yuriy Syrovetskiy) wrote…
> модуль не используется

Done.

Comments from Reviewable

src/TelegramApi.hs, line 1 at r1 (raw file):
Previously, cblp (Yuriy Syrovetskiy) wrote…
> модуль не используется

Done.

Comments from Reviewable

stack.yaml, line 17 at r1 (raw file):
Previously, cblp (Yuriy Syrovetskiy) wrote…
> А лучше просто все комментари стереть.

Done.

Comments from Reviewable

stack.yaml, line 48 at r1 (raw file):
Previously, cblp (Yuriy Syrovetskiy) wrote…
> Этот параметр не нужен.

Done.

Comments from Reviewable

app/TelegramApi.hs, line 4 at r3 (raw file):
Previously, cblp (Yuriy Syrovetskiy) wrote…
> нужна пустая строка между заголовком модуля и списком импортов

Done.

Comments from Reviewable

Reviewed 4 of 4 files at r4.
Review status: all files reviewed at latest revision, 2 unresolved discussions.

app/TelegramApi.hs, line 16 at r4 (raw file):

    response <- httpLbs request manager
    putStrLn $ "The status code was: " ++ show (statusCode $ responseStatus response)
    pure . responseBody $ response

pure $ responseBody response

Comments from Reviewable

app/Main.hs, line 8 at r3 (raw file):
Previously, kemphack wrote…
> Аа.. этот токен.. сейчас..

Done.

Comments from Reviewable

app/TelegramApi.hs, line 16 at r4 (raw file):
Previously, cblp (Yuriy Syrovetskiy) wrote…
> ```haskell
> pure $ responseBody response
> ```

Done.

Comments from Reviewable

Reviewed 1 of 1 files at r5.
Review status: all files reviewed at latest revision, 1 unresolved discussion.

Comments from Reviewable

Review status: all files reviewed at latest revision, 1 unresolved discussion.

app/Main.hs, line 8 at r3 (raw file):
Previously, kemphack wrote…
> Done.

а попробуй тут галочку поставить (круглая кнопка справа)

Comments from Reviewable


Containerized postgres

Currently, a local deployment of the gitplag application requires a postgres database to be available on the host machine. It makes a deployment of the application quite tedious.
To make things easier you can use containerized postgres database with a mounted directory to store the postgres data locally on host.
Moreover, while you have two containers in your application the use of of docker-compose can be of help. An example of docker-compose file for a similar project can be found here.

Thanks for the issue! I've implemented a docker-compose with Postgres in a container, so these features will be available in the next release.

This is great, thank you!


Unable to generate URL

Hi,
What do you mean you're unable to generate a url?
Could you please give more details?


Cannot access ambient const enums when 'isolatedModules' is enabled.

You currently cannot use RuleType in any conditions if you have isolatedModules enabled.

Hmm, what's the fix? I don't use isolatedModules in the tsconfig.json for this library

Hi there,
I'm also encountering this issue, both in my source code when e.g. checking the RuleType of a given node:
Cannot access ambient const enums when 'isolatedModules' is enabled. ts(2748)
But more importantly when running tests with Jest, tests of code involving RuleType fail with the following error:
TypeError: Cannot read properties of undefined (reading 'htmlBlock') Jest
FYI, I'm using rsbuild & SWC.
From what I could gather, the official recommendation from TypeScript is to completely avoid the use of const enums, including publishing ambient const enums: https://blog.beraliv.dev/2022-09-10-with-or-without-enums#ambient-const-enum-pitfalls.
Perhaps it would be worth considering converting RuleType to a simple object instead of an enum?
Thanks for this library, it's really great that it's so small!

Perhaps it would be worth considering converting RuleType to a simple object instead of an enum?

Yeah, I think that's the way to go. Would you be interested in submitting a PR? Have a bunch of life stuff going on atm so not sure when I will be able to work on this.

Hello @quantizor, sure I can take a look, also quite busy but I think it could be worth it! I'll try to tackle it in the coming weeks and keep you updated on my progress.

Hi there @quantizor, sorry for the long wait, but I finally had some time to look into refactoring the const enum for the RuleType into an object. I created a PR here : https://github.com/quantizor/markdown-to-jsx/pull/594
It would be great if you could have a look, the changes are fairly minor and should not impact the usage of the library at all.
Let me know what you think, I hope this approach makes sense and can make the library easier to integrate with different bundling setups.


Updated readme for instructions how to test on kuva infra

Description :sparkles:
Issues :bug:
Testing :alembic:
Screenshots :camera_flash:
Additional notes :spiral_notepad:

review fixes done, but Nizar has left the building, so I'm merging now.


feat: added a new sepolia faucet link

Added a new faucet link to the document
Updated the document to include a total of 5 faucet links

This new faucet link is known for its reliability and ease of use compared to others.

As we already have the link to directly receive zkSync Sepolia ETH, there's no need for a Sepolia ETH faucet, as that would add extra steps to getting funds on the testnet


Remove Image Alt Text in Story Mode

even if the image fails to load the text is there anyway and it would show duplicated text. this duplicated text also shows when you try to copy paste. example:
"Radiant won a teamfight by trading NecrophosNecrophos and Death ProphetDeath Prophet for RubickRubick, Shadow ShamanShadow Shaman, and UnderlordUnderlord, resulting in a net worth increase of 314314 gold."
where it should be:
"Radiant won a teamfight by trading Necrophos and Death Prophet for Rubick, Shadow Shaman, and Underlord, resulting in a net worth increase of 314 gold."

I think lint rules require image elements to have alt text.
The images shouldn't be failing to load either, is this happening consistently?

html wants an alt text but it doesn't care if its empty so alt=""
also they don't fail to load but if they would then you would see the alt text + the manual text so its a duplicate like when you try to copy paste

This should be closed.


Context Menu "Open with Code - OSS" ?

VSCode Version: 1.10.1
OS Version: Windows 7

Is that normal ?
The person who respond me on twitter tell me not and I should post a screenshot

Fixed in #21659 .

Dup of https://github.com/Microsoft/vscode/issues/21659


Canvas ui is very basic

It is just black blue buttons looks very bad assign me and I will make a better design for this

I would like to work on this issue pls assign me this @vishanurag


Create extensions for LSP-compliant editors

In alphabetical order:
Atom
Eclipse
Sublime
Vim (?)

Closing. Will be created as required, with separate issues.

I'm afraid we haven't gotten to creating an official extension for vim (neovim, etc).
I do know that the vim ecosystem has several projects that act as adapters for LSP based language plugins (like ours).
Someone in our company (for their personal use) has previously successfully cloned this repo and wrapped it with such an adapter (coc.nvim),.
We might get around to providing this, but it probably won't be in the immediate future - if you try to set it up for yourself, we'd be happy to try and help if you come across any issues.


Highlight decorator name

In this code
@my_decorator
def foo():
  return 1

module [0, 0] - [3, 0]
  decorated_definition [0, 0] - [2, 10]
    decorator [0, 0] - [0, 13]
      identifier [0, 1] - [0, 13]
    definition: function_definition [1, 0] - [2, 10]
      name: identifier [1, 4] - [1, 7]
      parameters: parameters [1, 7] - [1, 9]
      body: block [2, 2] - [2, 10]
        return_statement [2, 2] - [2, 10]
          integer [2, 9] - [2, 10]

with the current rules, only the @ is highlighted as a function in the Zed editor. This change is to make the my_decorator part also get highlighted as a function.
The reason this special case is needed is because decorators can be expressions, you can write something like this
decorators = [dec_a, dec_b, dec_c]

@(decorators[1])
def foo():
  return 1

but the vast majority of the time you're just doing @my_decorator.

I haven't tried this because
  CXX(target) Release/obj.target/tree_sitter_python_binding/bindings/node/binding.o
In file included from ../bindings/node/binding.cc:1:
../node_modules/nan/nan.h:2548:8: error: no matching member function for call to 'SetAccessor'
  tpl->SetAccessor(
  ~~~~~^~~~~~~~~~~

It seems like on GitHub, the current rule makes everything in the expression highlight as a function, but in Zed the opposite, only the @ is highlighted.


Fix brent bugs v2

Fixes the routes console command to handle formatting correctly and no longer fail tests.
See the action for passing tests.


Fix broken anchor links

This PR fixes two broken anchor links.

Thanks for these fixes @deining and apologies we didn't get to them in a timely way.
I have picked them up in https://review.gerrithub.io/c/cue-lang/cuelang.org/+/1194116 now that we have moved to the new https://cuelang.org site.


How can i get OnError???

if my video 404 how can i get onError report??

You can add a method in the class of 'JCVideoPlayer'.Such as 'setOnErrorListener(xxx)'.See method fm.jiecao.jcvideoplayer_lib.JCMediaManager.MediaHandler#handleMessage.Catch is here.


Enable FontAwesome for IconImageSource in TabItems

Currently you can only enter a string in IconImageSource that is related to an actual resource that has been added to either the forms project or the individual platform projects. It would be nice if you could use something like FontAwesome for Icons in the TabItems
Proposal (allow for backward capability)
Add an additional property that is something like "FontIconImageSource" and a corresponding IValueConvertor to get the icon from the fontawesome library. It would look like the following
<tabs:BottomTabItem FontIconImageSource="{x:Static fontawesome:FontAwesomeIcons.Users}" FontFamily="{StaticResource FontAwesomeSolid}" />
Here is a great article on using fontawesome fonts in a xamarin.forms library. User would have to the the same for this.
https://medium.com/@tsjdevapps/use-fontawesome-in-a-xamarin-forms-app-2edf25311db4
Describe alternatives you've considered
Tried to the following the above article and tried this
<tabs:BottomTabItem IconImageSource="{x:Static fontawesome:FontAwesomeIcons.Users}" FontFamily="{StaticResource FontAwesomeSolid}" />

TabItems already support font icons. Maybe you are using an old version of Tabs?

What is the recommended way to use the font icons in the tab item?

You can define icon source like this in your styles:
ˋˋˋxml

ˋˋˋ

Interesting,  and you would set the FontIconImageSource property to "IconSuccess" in the above example? It also sounds like you would need multiple FontImageSource for the icons.

Well one font image source for one icon obviously. It's a good practice. And you just set it with classic StaticResource


Class VoyagerAuth does not exist (View: **/vendor/tcg/voyager/resources/views/master.blade.php)

Laravel: v5.6.*
Voyager: v1.3.*
PHP: 7.2.11
Database: MySQL 5.0.12

Description

After updating voyager from 1.2 to 1.3 I am getting getting an error. Here is exception
Class VoyagerAuth does not exist (View: /var/www/html/****/vendor/tcg/voyager/resources/views/master.blade.php
Steps To Reproduce
Steps to reproduce the behavior:

Update voyager 1.2 to 1.3
Create Admin through command
Go To /admin/login
Put email and password
See errors

Expected behavior
You will get an exception as following
Class VoyagerAuth does not exist (View: /var/www/html/****/vendor/tcg/voyager/resources/views/master.blade.php)
Screenshots
Here is screenshot
https://imgur.com/RbX48ER

There is no VoyagerAuth in master.blade.php for v1.3.0:
https://github.com/the-control-group/voyager/blob/v1.3.0/resources/views/master.blade.php
It looks like some of your files are not updated.
Can you try to remove vendor/tcg folder and run composer install?

@MrCrayon I solved it by using below command
php artisan view:clear

oh yes, I forgot about cache!
Good, we can close here.


[CUDA] Improve default kernel config

Try to pick a smaller tile size based on the shape dimension to prevent
vectorization failure.

Code LGTM! Can we have vectorSize=2? If not, isn't the name vectorSize slightly misleading now? Since IIUC, in this loop, we only change tile size and won't really change the size of vector?

Code LGTM! Can we have vectorSize=2? If not, isn't the name vectorSize slightly misleading now? Since IIUC, in this loop, we only change tile size and won't really change the size of vector?

Yes we can have vectorSize=2 we are changing the vector size we will get after tiling to threads and vectorizing


Not able to use android example in adnroid studio

Describe the bug
A clear and concise description of what the bug is.
I get this error
error: cannot find symbol
public static  T plan(Message input, CoinType coin, Parser parser) throws Exception {
^
symbol:   class CoinType
location: class AnySigner
To Reproduce
Steps to reproduce the behavior:
Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
Additional context
Add any other context about the problem here.

Thanks for the report, will fix it soon


Add tags to default template

I suggest to also add "dotnet", "visual-programming" and "creative-coding".
The topic versions with dashes seem to be far more popular than the tags in which the space is left out, see here:
https://github.com/topics/creativecoding
https://github.com/topics/creative-coding
https://github.com/topics/visualprogramming
https://github.com/topics/visual-programming

where exactly do you suggest to add these?

In the tag section of the GitHub repository.

Alright!


Unable to import Graph crate from git due to error

Hello,
I am trying to use the ensmallen graph crate in my project:
ensmallen = { git = "https://github.com/AnacletoLAB/ensmallen.git"}

Howerver I get the following error while trying to compile the code:
error[E0283]: type annotations needed
     --> C:\Users\Barak\.cargo\git\checkouts\ensmallen-ebfa24490134db1a\73e715b\bindings\python\src\auto_generated_bindings.rs:17509:42
      |
17509 |         .map(|x| x.into_iter().map(|x| x.into()).collect::<Vec<_>>())
      |                                          ^^^^
      |
      = note: cannot satisfy `_: From<u16>`
      = note: required for `u16` to implement `Into<_>`
help: try using a fully qualified path to specify the expected types
      |
17509 |         .map(|x| x.into_iter().map(|x| <u16 as Into<T>>::into(x)).collect::<Vec<_>>())
      |                                        +++++++++++++++++++++++ ~

I actually don't need the binding sub-folder, but unfortunately, rust does not allow to compile only sub-folder like stated here.
Thanks in advance.

I suggest you clone the repository and import these dependencies via path.

Thanks, That's what I actually ended up doing.


Read from config syncUrl param

when from react native sends configure, app ignore syncUrl

Ok I can merge this. FYI configuration was moved to separate class ConfigMapper, but till then...
https://github.com/mauron85/react-native-background-geolocation/blob/android/android/lib/src/main/java/com/marianhello/react/ConfigMapper.java

merged thank you


Token should be an enum

Currently Poiinter::tokens returns an iterator of Token, but in doing so omits the root. This is not ideal.
Token should be an enum consisting of two possible options, Root and Normal.
A new wrapper type will be needed for the contents of Normal.

In retrospect, I don't like the name Normal. I also dont know if the Token should be an enum or if there should be a separate enum specifically for iteration. Having it be Cow based would be neat, regardless, though.
@asmello mentioning you as you said you may look at Token.

I'm not sure I understand the motivation behind emitting a root token. Can you add some examples showing why that is useful? (I don't think I've run into any, or maybe I just forgot)
With regards to the enum variant names, I think Trunk (or Stem) might be an appropriate name for the internal components of the pointer. Or alternatively the type could be renamed to something like Component or Segment and that frees Token as the variant name.

I'm going to dedicate some time to it myself but if you happen to work on it and have ideas or make progress, I'll happily concede it over or work with you on it.

Yeah, I'm going to play around a bit, it's an interesting design puzzle. Feel free to make your own attempt, regardless. Whether or not my experiments turn into a PR, they'll make me able to better to form an opinion over the final design.

Can you add some examples showing why that is useful?

This is from my previous attempt at a json schema compiler. This method is invoked when a non-root schema (one that has a json pointer fragment) is referenced from a schema currently being compiled but the root schema (uri sans fragment) has not. In order to properly compile the target schema, I have to walk up the path and queue ancestors of the target schema.
As the current iterator does not emit a token for the root, I had to use a while loop to work around it.
    fn queue_ancestors(
        &mut self,
        target_uri: &AbsoluteUri,
        q: &mut VecDeque<SchemaToCompile<K>>,
    ) -> Result<(), CompileError<L, K>> {
        let mut path = Pointer::parse(&target_uri.fragment_decoded_lossy().unwrap())?;

        q.push_front(SchemaToCompile {
            key: None,
            uri: target_uri.clone(),
            parent: None,
            continue_on_err: true,
            ref_: None,
        });
        while !path.is_root() {
            path.pop_back();
            let mut uri = target_uri.clone();
            if path.is_empty() {
                uri.set_fragment(None)?;
            } else {
                uri.set_fragment(Some(&path))?;
            }
            if let Some(key) = self.schemas.get_key(&uri) {
                let is_not_compiled = !self.schemas.is_compiled(key);

                ensure!(is_not_compiled, SchemaNotFoundSnafu { uri });
                continue;
            }

            q.push_front(SchemaToCompile {
                key: None,
                uri,
                parent: None,
                continue_on_err: true,
                ref_: None,
            });
        }
        let mut uri = target_uri.clone();
        uri.set_fragment(None).unwrap();

        ensure!(
            !self.schemas.is_compiled_by_uri(&uri),
            SchemaNotFoundSnafu { uri: target_uri }
        );

        q.push_front(SchemaToCompile {
            uri,
            key: None,
            parent: None,
            continue_on_err: true,
            ref_: None,
        });
        Ok(())
    }

It should have been modeled after std::path::Component which has a RootDir variant.

While the example I posted turned out not to be so great, I do think there's still need and perhaps even an expectation that when traversing a path, you have an opportunity to execute logic at the root / base.
It was an annoyance to me as a user to discover my loop body wasn't executing at the base path.

std::path::Component is a great analog here, but I think the motivation for it may not apply to JSON Pointers, precisely. That's because, as defined in RFC 6901, pointers are always absolute. I believe std::path::Component was introduced because filesystem paths sometimes are absolute, sometime not, and there was a need to make a distinction when decomposing them into components.
That doesn't mean we shouldn't provide a way to iterate over the components that treats the root as one. I can definitely see how that would be useful in the example you've shown. I'm just not convinced it should be the default iterator, because it does add some complexity in cases where the root token is not used (which I think may be relatively often?).
It's an interesting design question. There is sometimes a need to special case the first (or last) item in a sequence, and I've started a discussion once before with the Rust community about whether we should do something to help reduce related off-by-one mistakes. There were some interesting ideas proposed, but nothing that seemed like we should obviously add to std.
So my instinct here is to keep Pointer::tokens with the current semantics, and maybe introduce a separate Pointer::components that produces the described enum.

Sold.
This doesn't add complexity to Token and circumvents a major breaking change.

I'll handle that. Although it's not needed for your PR to get pushed out since it wont be breaking.
Thanks.

This was addressed in one of my grab-bag-o-change PRs.
I'll track it down if anyone ever needs to know precisely where.


problem when committing with annex hook

homebrew git and git-annex on osx.
here is the error log.
`/usr/local/bin/git commit -q -F -` failed with following output:

WARNING: You called a Git command named 'annex', which does not exist.
Continuing under the assumption that you meant 'amend'
in 0.1 seconds automatically...
error: pathspec 'pre-commit' did not match any file(s) known to git.

Sublime Version?
Git version?
Annex version if it exist?

Sublime: v3 build 3103
Git: v2.6.1
Annex:v 6.20160126
weird, now it is working. sorry for your time.

Glad to hear it,. Than this is not an issue any longer.


Revert "openssl: 1.1.1k -> 1.1.1l"

Should go to staging
Reverts NixOS/nixpkgs#135611

Let's revert until https://hydra.nixos.org/eval/1700096 is rebuilt. I hope that's an acceptable compromise.
cc @mweinelt

fwiw patches like openssl bug fixes with HIGH security impact should go to master directly.


Expose Flic button press up/down events to Flutter listeners

Implements Feature Request #8

That's some great stuff Matthias!
I've pushed that to pub.dev as a new minor version 1.2.0 as more of a
change with the added function to the interface.
On Wed, 9 Aug 2023 at 18:36, Matthias Schicker @.***>
wrote:

Implements Feature Request #8
https://github.com/DarkerWaters/flic_button/issues/8
You can view, comment on, or merge this pull request online at:
https://github.com/DarkerWaters/flic_button/pull/9
Commit Summary

6531259
https://github.com/DarkerWaters/flic_button/pull/9/commits/65312593ae50d623e4399cfcb9d7c40419ac2003
Updated plugin dependencied
69f76ff
https://github.com/DarkerWaters/flic_button/pull/9/commits/69f76ff4b520e8927dccf316169a016aa0887ee9
Started exposing more callbacks from the Flic SDK to the Flutter plugin
e5249df
https://github.com/DarkerWaters/flic_button/pull/9/commits/e5249dfd6e8b957c25f1335dcacfaebed75e3ef7
Merge branch 'main' into feature/emit_button_down_and_up_events
702df60
https://github.com/DarkerWaters/flic_button/pull/9/commits/702df605498f6c0a379086290b4cbfd10bdb82e8
Added onConnect callback from FlicSDK
968aa16
https://github.com/DarkerWaters/flic_button/pull/9/commits/968aa1643594b161f915cca5149d4cde212ae192
Removed some unused callbacks
52bce28
https://github.com/DarkerWaters/flic_button/pull/9/commits/52bce28aebb83e3fd2448d286651738cea9367a9
Merge pull request #1 from komaxx/feature/emit_button_down_and_up_events
ee336dc
https://github.com/DarkerWaters/flic_button/pull/9/commits/ee336dcf11eb1f2a7aeecf6afa3a72f8452c65cc
Removed unnecessary log output
7a81fcb
https://github.com/DarkerWaters/flic_button/pull/9/commits/7a81fcb0daa12284d40f40568521914d12f102c8
Implemented button up/down event processing on iOS

File Changes
(7 files https://github.com/DarkerWaters/flic_button/pull/9/files)

M
android/src/main/java/uk/co/darkerwaters/flic_button/Flic2Controller.java
https://github.com/DarkerWaters/flic_button/pull/9/files#diff-1840e51c3f2f89ebb9b080d31cf280bd466c81680ab746d678b9a17208058494
(34)
M
android/src/main/java/uk/co/darkerwaters/flic_button/FlicButtonPlugin.java
https://github.com/DarkerWaters/flic_button/pull/9/files#diff-1329ddcad50ca132c9b5ef6f7562a5fc23319ef1cf7a8d5c29168b64ad9acd26
(7)
M ios/Classes/Flic2Controller.m
https://github.com/DarkerWaters/flic_button/pull/9/files#diff-73753559af22188efe309f2da6082d779726738282f84be7dc83161560702d2e
(15)
M ios/Classes/Flic2ControllerListener.h
https://github.com/DarkerWaters/flic_button/pull/9/files#diff-8df3396bdb17b0845b210340a6f04d0aa924f449e4c833a36ad04926b15574ec
(1)
M ios/Classes/FlicButtonPlugin.m
https://github.com/DarkerWaters/flic_button/pull/9/files#diff-6caed095d800e19d10dbbe39790e9355a753cb005a64ad0c15c5ad953e77dfd8
(15)
M lib/flic_button.dart
https://github.com/DarkerWaters/flic_button/pull/9/files#diff-858080569f26e5100e2078ca3f94e692849e63e441dcd6e65a3d66e48fd52ee2
(61)
M pubspec.yaml
https://github.com/DarkerWaters/flic_button/pull/9/files#diff-8b7e9df87668ffa6a04b32e1769a33434999e54ae081c52e5d943c541d4c0d25
(4)

Patch Links:

https://github.com/DarkerWaters/flic_button/pull/9.patch
https://github.com/DarkerWaters/flic_button/pull/9.diff

—
Reply to this email directly, view it on GitHub
https://github.com/DarkerWaters/flic_button/pull/9, or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABKMPQHL3WOWUVZHRPVX5PDXUPDBDANCNFSM6AAAAAA3KKYT6E
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

Thank you for the merge Douglas :)

I can make you an admin to the repository if you want? You write better
Dart than I did when I wrote it... I keep seeing things I would do better
now, especially now there's null-safety and Dart 3 (O:
On Fri, 11 Aug 2023 at 09:28, Matthias Schicker @.***>
wrote:

Thank you for the merge Douglas :)
—
Reply to this email directly, view it on GitHub
https://github.com/DarkerWaters/flic_button/pull/9#issuecomment-1674384211,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABKMPQDTVM56TB7BMXX22M3XUXUMTANCNFSM6AAAAAA3KKYT6E
.
You are receiving this because you modified the open/close state.Message
ID: @.***>

Sure, I'm happy to help!

I keep seeing things I would do better now, especially now there's null-safety and Dart 3 (O:
We never stop learning :D

There you go - I think giving you write access means you can push to the
main branch without having to wait for me (O:
On Fri, 11 Aug 2023 at 09:37, Matthias Schicker @.***>
wrote:

Sure, I'm happy to help!
I keep seeing things I would do better now, especially now there's
null-safety and Dart 3 (O:
We never stop learning :D
—
Reply to this email directly, view it on GitHub
https://github.com/DarkerWaters/flic_button/pull/9#issuecomment-1674394339,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABKMPQCUELD5LMSEN3YEG43XUXVMTANCNFSM6AAAAAA3KKYT6E
.
You are receiving this because you modified the open/close state.Message
ID: @.***>


Cannot open programs from another display

Suppose I have a wlc compositor with WAYLAND_DISPLAY=wayland-0 and I am on another display. I want to open an application on wayland-0. So I do
WAYLAND_DISPLAY=wayland-0 gnome-calculator

I expect gnome-calculator to run on the wlc compositor on wayland-0, but nothing shows up.
This works great in Weston. This does not work in Orbment or Sway.
I need this feature for testing.

This issue is related to vt switching.
When I do
sleep 5; WAYLAND_DISPLAY=wayland-0 gnome-calculator

with enough time to switch back to the display Sway is running on, it works correctly.
So when windows show up when the vt is not active, they do not get mapped.


[ci] sync template files

This PR syncs the specified GitHub template files from the central repository.
Changes:

add special handling for deployment file - (60e9260)

The branch has been updated. No changes detected for trueberryless-org/blog.


Vim extension is not working

code-server version: code-server v1.1119-vsc1.33.1
OS Version: DigitalOcean Ubuntu 18.04
Extension: VSCodeVim

Description
After installing the VSCodeVim extension, I'm unable to use vim keymappings. I tried toggling vim mode as well, to find the error command 'toggleVim' not found.
I've tried installing extension versions 1.4.0, 1.8.0, and 1.0.8 as per indicated in previous similar issues but am not able to get it to work at all. Any version of code-server in particular I should try downgrading to?
Steps to Reproduce

Install code-server v1.1119-vsc1.33.1
Install VSCodeVim (1.4.0)
Try Vim Keybindings / Toggle Vim Mode

Could not reproduce with same code-server and VSCodeVim versions. I was able to navigate around the file with vim keybindings and toggle vim mode through the command palette.

Try run the browser in incognito mode.
Probably some chrome plugins mess up the keybindings.

I've got the same issue, similar setup. An additional piece of info: whenever I try to use Vim with : or / I get an "Activating Extensions" briefly loading on the bottom bar, then disappears. Also, I can't see other extensions loaded, like pretty either. Though they are all there on the extensions tab showing as Enabled (tried to disbale en re-enable, no luck).

Used --skipsync and installed Vim extension fresh, on a new install, then it worked. It was syncing my extensions through sshcode before.

I wasn’t able to get this working, but am also not going to be using the project in the near future as it was part of a lost battle in discussion at work.
Thanks for the awesome work here, it’s really impressive.

Any chance this could re-opened? Or should I open a new issue?
Running 2.1692-vsc1.39.2 in a docker container (Ubuntu 18.04) with VSCodeVim 1.12.2

This is the issue I get when trying to launch VSCodeVim:

@fwcd Is that Firefox? The vim extension uses regular expressions Firefox doesn't support yet, so for now the only workaround is to use Chrome.

Let's add this to the FAQ.

Quick question, if the issue is with Safari, why in online.microsoft.com vim works quite well? Do they use a different one?

We run it in the browser while they probably run it on the server. The issue we had with running it on the server is that depending on your connection it can get rather laggy because it has to communicate to the server and back for every keystroke. Perhaps we should have a way to force it to load on the server though? Or keep it on the server by default and have a way to make it run in the browser?

We run it in the browser while they probably run it on the server. The issue we had with running it on the server is that depending on your connection it can get rather laggy because it has to communicate to the server and back for every keystroke. Perhaps we should have a way to force it to load on the server though? Or keep it on the server by default and have a way to make it run in the browser?

Yup I noticed is a little slow, but I agree that we running it on the server can be a good choice if you have a good connection or better, code-server is in your LAN.
Do you have a document or instructions to force it on the server?

Nope, unfortunately at the moment it's hard-coded so we'd need to make that customizable.

This is being fixed upstream https://github.com/VSCodeVim/Vim/pull/3525#issuecomment-662529074


Runtime state filtering

Description
Changes proposed in this pull request:

Implement filtering of runtimes on the /runtimes API by state
By default, if no state filters param(s) are provided, filter out suspended runtimes.
Create indexes to speed up lookup and sorting of operations based on the most frequently used attributes: by instance ID, and by Orchestration ID.

Related issue(s)
https://github.tools.sap/kyma/backlog/issues/931

/retest

/retest


Migrate from jennifer to text/template

Pull Request Test Coverage Report for Build 255240314

323 of 491 (65.78%)  changed or added relevant lines in 15 files are covered.
6 unchanged lines in 5 files lost coverage.
Overall coverage decreased (-11.4%) to 83.63%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

gen/aggregates.go
20
22
90.91%

gen/repository.go
10
12
83.33%

gen/predicates.go
33
36
91.67%

test/integration/repository/repository.go
20
23
86.96%

gen/meta.go
8
12
66.67%

gen/sorts.go
15
19
78.95%

gen/postgres.go
43
48
89.58%

gen/generate.go
29
42
69.05%

gen/file.go
40
57
70.18%

gen/internal/col.go
0
17
0.0%

Files with Coverage Reduction
New Missed Lines
%

gen/file.go
1
71.43%

gen/generate.go
1
77.05%

gen/predicates.go
1
89.47%

test/integration/repository/repository.go
1
96.64%

gen/aggregates.go
2
83.33%

Totals

Change from base Build 249409624:
-11.4%

Covered Lines:
1553

Relevant Lines:
1857

💛  - Coveralls


Update button.types.ts: support button type 'normal'

Closes IBM/carbon-components-angular#
Button type 'normal' is not supported in the last version but present in the published storybook version of the library
Changelog
Changed

include button type 'normal'

@MEKNI-D the button size is "normal" by default, no classes need to be applied. If size is set to anything other than sm | field | lg | xl then it will be regular size. You can set it to nothing if it is causing type issues in your build. Please feel free to reopen if I'm missing anything!


Author Block

Add new block for Profile-type  and Content-type modules, which shows basic profile overview - image, info and action buttons, as well as potentially some extra data from other modules, like Skills, Groups, etc....
This is useful for creating profiles where profile cover can be turned off and only a block is available. Also for posts/products/classes where extended info about the author is useful.

Author Block was realized as system service method and can be placed on any view content page.
The service method is based on 'get profile cover as block' service method which is available in profile modules.


ADDS multiserver support

Hello!
Is it possible to connect to Active Directory via FQDN?
Used this TLS Options var
var ldap_client = ldap.createClient({url: ldap_bind_url, tlsOptions: {'rejectUnauthorized': false}});

I have tested connection to FQDN of single Active Directory controller (like ldaps://ctrl01.domain.tld:636/) - success
I have tested connection to FQDN of Active Directory (like ldaps://domain.tld:636/) - failed with error:
LdapErr: DSID-0C0605EB, comment: Error decrypting ldap message

I have never seen that error and I have used FQDN, can you check if you get the error while using ldap.exe?
Also I'd double check that the certificates on the controllers have the FQDN of the domain, if they are set to their personal FQDN it will be an invalid certificate.
For example ctrl01.domain.tld should have domain.tld in the certificate, otherwise it will not be valid for connections to domain.tld.

An alternative nasty hack is to use a list of DCs for your authentication, so if 1 is down try 2.
In my experience there was something weird the DNS holding onto a controller until it was flushed on the server, so when a DC was offline the app couldn't authenticate anyway. This may be something easy to fix I'm unaware of, but I always had to clear the DNS on the server that LDAPjs was working on.
That is why I switched to defining multiple servers, so if one was offline the app would automatically try the next server.

I have not tried ldap.exe. I use "tlsOptions: {'rejectUnauthorized': false}" also but do not know will that option suppress all TLS warnings
Would like to add that LDAP modification with the error above completes successfully

Reject unauthorized will not validate the certificate, so it shouldn't suppress any other warnings.
Think of it like when you visit a site signed with a privet key and you click continue anyway. It technically leaves you open to man in the middle attacks, but depending on your network security this may not be a big threat internally.
That's interesting that the modification still works, I'd test it with ldap.exe and see if you get the same error to help narrow it down to your AD configuration or the application / ldapjs.

@akhmarov I think @tastypackets has given sufficient information to aid you in figuring out your issue. If you are still having this problem, feel free to re-open with more information.

Hi all,
@akhmarov I am encountering the same issue (LdapErr: DSID-0C0605EB, comment: Error decrypting ldap message) I will be interested if you have found a solution for this particular case. thanks!

⚠ This issue has been locked due to age. If you have encountered a recent
problem that seems to be covered by this issue, please open a new issue.
Please include a minimal reproducible example
when opening a new issue.


Master 822 missing cherry pick apiserver

Pick up missing 1.18 changes from https://github.com/kubernetes/kubernetes/pull/84692
. PR 822 has one change from 84692 (Create a shim for Codecs that handles client duties)

dependencies for 84692 & arktos adoption

1/25 density test data looks better than 1/15 & 1/21. apiserver_current_inflight_request is much smooth and closer to 1.18 (see https://github.com/CentaurusInfra/arktos/pull/946). This is a good cherry-pick

/approve

/approve

/lgtm

/lgtm


cmake: west: invoke west using same python as rest of build system

When running CMake, then Python3 will be used.
This is detected through FindPython3, with a preference for using the
python or python3 in path, if any of those matches the required Python
minimal version in Zephyr.
It is also possible for users to specify a different Python, as example
by using:
cmake -DPYTHON_PREFER=/usr/bin/python3.x
However, when running west as native command, then west will be
invoked on linux based on the python defined in:
west launcher, which could be: #!/usr/bin/python3.y
Thus there could be mismatch in Pythons used for west and the python
used for other scripts.
This is even worse on windows, where a user might experience:
>.\opt\bin\Scripts\west.exe --version
Traceback (most recent call last):
  File "C:\Python37\lib\runpy.py", line 193, in _run_module_as_main
    "__main__", mod_spec)
  ...
  File "C:\Python37\lib\socket.py", line 49, in <module>
    import _socket
ImportError: Module use of python38.dll conflicts with this version of
Python.

when testing out a newer Python, but the python in path is still a 3.7.
By importing west into zephyr_module.py and by using
python -c "from west.app.main import main; main(['<command>'])"
we ensure the same python is used in all python scripts.
Also it allows the user to control the python to use for west.
It also ensures that the west version being tested,m is also the version
being used, where old code would test the version imported by python,
but using the west in path (which could be a different version)
If the west version installed in the current Python, and west invocation
is using a different Python interpreter, then an additional help text
is printed, to easier assist users with debugging.
Signed-off-by: Torsten Rasmussen<EMAIL_ADDRESS>

Example of improved help text, if wanting to build with python3.7 when python3 points to another version.
west build -bnrf52840dk_nrf52840 -dbuild_west -- -DPYTHON_PREFER=/usr/bin/python3.7
...
CMake Error at /projects/github/ncs/zephyr/cmake/host-tools.cmake:38 (message):
  The detected west version is unsupported.

    The version was found to be 0.7.0:

    But the minimum supported version is 0.7.1
    Please upgrade with:
        /usr/bin/python3.7 -mpip install --upgrade west

    Note:
    The Python version used by west is:  /usr/bin/python3
    The Python version used by CMake is: /usr/bin/python3.7
    This might be correct, but please verify your installation.

@mbolivar due to compliance test, I discovered that I wasn't testing the error message properly, when not in a west workspace.
Code now redirects sys.stderr to be able to correctly check the message, so please re-review.

@mbolivar-nordic please re-review

I was testing this out before adding my +1, but this breaks west build for me.

Fixed, see also: https://github.com/zephyrproject-rtos/zephyr/pull/26060#discussion_r444777952

@nashif could you take a look ?


Drop unused dependencies

@mattcieslak is there anything we can drop from QSIPrep?

I think we got most of it


surface max slopes to make it easier to interpret

On appearances introduction.jl allows me to set a full immediate deployment of mitigation (the kind of thing I do immediately to interpret a model).
However, the controls are clamped behind the scenes in enforce_maxslope.
To make the user aware of this clamping behavior the control plots could indicate the clamped deployments.
Alternatively we could remove the clamps altogether and/or describe them with a toggle button - I think justifying them is a good idea anyways? The notebook is otherwise very informative about the mechanics!
Removing the call to enforce_maxslope in forward_controls_temp gives me sight of a cost analysis for complete mitigation and there is always a net benefit. This causes issues in the optimization step for a temperature target (higher temperatures unreachable). Clearly, this is a really significant feature in the model to surface visually and make the user fully aware.

There are comments about the slopes in this example:
https://github.com/ClimateMARGO/ClimateMARGO.jl/blob/fbbe619f21cc50f8cdf137d6243773edcf3381f2/examples/default_configuration.jl#L280C3-L280C3

Second, we set upper limits on how quickly each control can be ramped up or down.

We can modify a similarly constructed max_slopes param by adding it to the call to optimize_controls inside opt_controls_temp definition.

Supplementary information in the margo paper has the following:

Hey @declann! You suggested that the plots could visually indicate the max slope. Can you draw a picture of what you have in mind?

Sure @fonsp !
I'm really looking at the interactive visuals under 'Mitigating emissions'.
The initial view is probably ok, because the ramp up to full deployment takes ~40 years.

However, if I move the lever to the left it appears like full immediate deployment of mitigation but it isn't:

Further, we can't really know whats going on without inferring it backwards from the Emissions chart below it.
So, I think an overlay like this helps:

And, I think a note should explain that due to inertia and practicalities there is a limit to how fast mitigation can actually be ramped up.
Trying to keep changes low-key with that approach, what do you think?
Ps: I'm reimplementing bits of the ClimateMARGO model in a new language calculang and developing different visuals and interactions - your notebooks are very helpful for inspiration/understanding the model and to figure about interacting with the Julia codebase, thanks!

Hi @declann, thanks for pointing out this inconsistency between the apparent controls shown in the notebook and those actually employed in the underlying model instance.
I like both of your ideas of visualizing the max-deployment clamps to satisfy initial condition controls and also for allowing the clamps to be toggled on or off. I won't have the capacity to solve this for a while, but you're more than welcome to make a PR here if you think you know how to fix it.

Cool @hdrake
Right now it isn't feasible I do a PR on Julia/Plots.jl code but maybe I'll be able to do it eventually.
Sorry for going offtopic: I'm replicating some climatemargo numbers in a calculang model.
Approach for my own interactives will be to derive mitigation control values from emissions projections. Those controls look like https://twitter.com/calculang/status/1716211017875706013 (source in vega editor but I will also add constant emissions controls)
I'll be going a bit outside climatemargo model intended scope (where I permit changing past emissions for instance), and will be sure to highlight that, and maybe not even report some economics outputs in new scenarios.
I don't think I'll get optimization into this model in the immediate future, but hopefully will be able to add to interesting climate interactives - for simpler scenarios.
Thanks for the nice straightforward model to work and learn with!

Thanks! I fixed it in https://github.com/ClimateMARGO/interactive/commit/36331561243dc5d46e0d1bb97e5106ce3fc31f23 by applying the enforce_maxslope! function to the controls before plotting them.

Thanks! I fixed it in 3633156 by applying the enforce_maxslope! function to the controls before plotting them.

Thanks, @fonsp, but it looks like the changes unfortunately broke https://margo.plutojl.org/introduction.html!

Sorry! Fixed again yesterday


How to deploy Wagtail?

Reading the deployment section here is not very helpful.
http://docs.wagtail.io/en/v1.13/advanced_topics/deploying.html
I can get a site running on my local system but how to now transfer it to a live server (I have a ssh connection to a droplet on digitalocean running ubuntu/nginx)?
I searched on the internet for deployment information but no luck. Also, I don't have any background in django so it would be great if there is a tutorial of some sorts.
Goal is to develop the site locally and push it to a live server.

I found a good guide for Django deployement on Nginx server:
http://uwsgi-docs.readthedocs.io/en/latest/tutorials/Django_and_nginx.html
Perhaps we can link it to the deployment section in the docs? I am not familiar with Wagtail at all so if someone could point out the specifics on deployment that are different from the tutorial above, that'd make an excellent guide to get non-familiar people onboard!

Thanks for the report, @neilpanchal.
As the page says, the base requirements for Wagtail deployment are the same as for Django itself (and any optional extras are covered in the Performance docs). Giving detailed deployment instructions here would duplicate the information that already exists for Django, and probably end up not being kept up-to-date. (Keeping the minimal details on this page up-to-date is already enough of a task - see #3898 for a recent example.)
To be honest, I think the real problem newcomers face here is not a lack of documentation, but too much of it. There are any number of different platforms you can deploy on - the Django documentation alone covers three - and actually understanding the pros and cons of each one involves reading up on things like WSGI and proxy servers. Answering the question "which deployment setup is best for me?" is out of scope for the Wagtail documentation - we do the next best thing, which is to identify a setup that works well for us (namely, nginx + gunicorn on Debian). The one you've identified (nginx + uwsgi on Ubuntu) is probably equally good, but we don't have the resources to document every possible setup, or even to recommend specific tutorials for setups other than the one we use.


Add exec-all-actions config setting

This PR adds the exec-all-actions boolean setting. When enabled, all actions are executed (even if some of them fail). If any of these actions has failed, the hook finally finishes with a non-zero exit code.
Furthermore, I adjusted the console output to better connect action logging + action errors. For CLI commands, it is now clearer which output came from which command.

There is too much going on in this pull request. Please split it up into multiple requests.


Release: hwc-buildpack (May)

Release reminder for hwc-buildpack

See diff from latest version and validate if a release is required.
Make sure the latest commit on  has passed tests on the CI
Refer release instructions. (private link)

v3.1.31 Released.


Github Action for ctests

This is a first working workflow for running radae ctests on Github Actions  ( #1 ) .
It will be triggered on "push" and should be working in every branch.
There is possible support for multiple operating systems which could be added easily in future.
I also decided testing against several latest python versions. Sometimes there are incompatibilities with newer versions, so the test might "warn" early enough if there is an possible problem with a specific python version. Also its common, users have different python versions installed. However, if not needed, this could be disabled easily.
All tests need about 12minutes to complete.
Since all tests are passing, #22 can be closed as well

Hi @DJ2LS - thank you so much for working on this one :slightly_smiling_face:  Interesting that it passed with the standard torch package, I have been installing from the Torch site to date.  I'm a bit busy with the day job for the next few days but will take a closer look and merge later in the week.

You're welcome, @drowe67 ,but it was like every GitHub action work - a pain 😀
https://pypi.org/project/torch/ provides the latest stable version.

@drowe67 meanwhile 24.04.1 LTS is available. I think we should add it to the test as well, even if its not part of the project scope, yet.

No.  The project scope is there to stop exactly this sort of distraction, so that we can maintain focus on what really matters.

Understood 👍
I try to remove the remaining debugging code the next days.

Understood 👍
I try to remove the remaining debugging code the next days.

Sure, just tell me when it's ready to test & merge.  Hmm, how will I test if I pull from your repo?  If I git push I'm not sure if it will run the tests :thinking:

Understood 👍
I try to remove the remaining debugging code the next days.

Sure, just tell me when it's ready to test & merge. Hmm, how will I test if I pull from your repo? If I git push I'm not sure if it will run the tests 🤔

you could push to my fork of radae, then having a look at actions and check if they are failing:
https://github.com/DJ2LS/radae/tree/ls-github-action
Another option would be, merging, and if you have stability concerns, you could disable it via Github Actions. This PR shouldnt affect the code itself, as its a separated task.

Thanks @DJ2LS - I'll merge and do any residual debugging in main


about data

Hello, is the dataset of this paper completely provided? I encountered data processing problems when reproducing the code of this paper. Are the data sets such as trainset.jsonl, validset.jsonl, testset.jsonl and feature.hdf5 not provided in the data set? I would be grateful if the corresponding data can be provided.

Hi, please see release for the processed data.


Spenceral/versioned buckets

This change attempts to remove Redis flushing entirely by expiring old keys based on a configurable TTL and adding the config version to each key to ensure that operations use current keys.

Pull Request Test Coverage Report for Build 767

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 68.097%

Totals

Change from base Build 764:
0.0%

Covered Lines:
476

Relevant Lines:
699

💛  - Coveralls

Pull Request Test Coverage Report for Build 767

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 68.097%

Totals

Change from base Build 764:
0.0%

Covered Lines:
476

Relevant Lines:
699

💛  - Coveralls

Pull Request Test Coverage Report for Build 767

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 68.097%

Totals

Change from base Build 764:
0.0%

Covered Lines:
476

Relevant Lines:
699

💛  - Coveralls

Dynamic buckets are already created with a TTL equal to the maxIdleTimeMillis bucket attribute. I overlooked that before and need to clean this up accordingly.


Blockhash on tokens.html unaffected by changes to first transaction.

Observed behavior:
The blockhash of any block on the tokens.html page is unaffected by changes to the first transaction (the top most transaction) in any block.
To reproduce:
Load https://anders.com/blockchain/tokens.html
Modify the tx value or sender or receiver, for the first transaction (the top most transaction) in the block.
Expected result:
Any modification to any data in the block should cause the Blockhash to change.
Block color with a modified tx, and the dependent blocks, should turn red until mined again.
P.S.
Thanks so much for this wonderful demonstration.

Thanks - should be fixed now.

Expected result now occurs.
You're fast, Anders!  Thanks again for the demonstration tool


Update Documentation for Release

[ ] proofread documentation
[ ] merge info from comprehensive README.md to elsewhere
[ ] figure out how to "deploy" docs, including api docs

Merged into separate branches and #161


Add different if generation to support skipping in tests

Don't merge until it's decided this should be an approach we take.

We decided not to go down this route for Image Ordering, so closing.


Server-side Blazor 0.5.1 doesn't update active class on NavLink element

I have just published "default" server side application to Azure. I have made only one change in WeatherForecastService.cs
return Task.FromResult(Enumerable.Range(1, 500).Select(index => new WeatherForecast

As you can see service now returns 500 instead of 5 forecasts.
We can click on Home and Counter menu links and everything works as expected. Try to click on Fetch data. At first everything works, but active class on NavLink (strictly speaking on <a> element) is not removed from Home or Counter and not added to Fetch data.
http://blazorserwer.azurewebsites.net/

@MarkStega it is not related to #1211 at all. This is standard project created by VS. It doesn't require any JavaScript.
Problem is visible when I try to return 500 forecasts. When I return 5 or 50 forecasts it works correctly. Probably there is subtle bug somewhere in async code on the server or on the client.

Seeing this as well...  It's not limited to just the navlink.  Other simple bindings can fail to update as well, and it's fairly easy to reproduce.
Seems to be some kind of timing issue.

This problem still exists in 0.6.0

Any Error message in the console?

@vertonghenb I don't see any errors in web browser console.

@Andrzej-W, did you try to place the NavLink for FetchData between the NavLink for Home and that of the Counter, and see if the issue remains ?


spurious validity errors

From @cmaeder on June 23, 2015 18:26
on my machine (Ubuntu 15.04) I get validity error messages on stderr:
maeder@lati:/var/data/localmh/MathHub/smglom/mv$ run-latexml.sh source/piecewise.de.tex
element XMApp: validity error : ID cvar.6 already defined
element XMTok: validity error : ID piecewise.de.definition3.CMP1.p3.m3.1 already defined
element XMApp: validity error : ID cvar.5 already defined
element XMApp: validity error : ID cvar.4 already defined
element XMTok: validity error : ID S0.Ex1.m1.1 already defined
element XMApp: validity error : ID cvar.0 already defined
element XMApp: validity error : ID cvar.1 already defined
element XMApp: validity error : ID cvar.2 already defined
element XMApp: validity error : ID cvar.3 already defined
No obvious problems
Wrote /var/data/localmh/MathHub/smglom/mv/source/piecewise.de.omdoc

On ash (Debian 8 (jessie)), there are no errors shown. The output created is identical on both machines, so the messages could be simply ignored, but they are still irritating.
(An old related issue seems to be http://www.cpantesters.org/cpan/report/ba10553c-8029-11e4-b744-f4345ef060ad )
The script to run latexml is https://github.com/KWARC/MMT/blob/svn-mirror/src/mmt-stex/run-latexml.sh.
Copied from original issue: KWARC/sTeX#100

From @dginev on June 23, 2015 18:36
Oh wow, 15.04 is out?
If this is another breaking change of libxml2 I will have to find a place to channel anger. It took forever to resolve a breaking valdiation problem with the newer version of libxml2 for the 0.8 release of LaTeXML back in November 2014, hopefully we are not hitting that again.

From @cmaeder on June 24, 2015 8:24
I had to add deb http://de.archive.ubuntu.com/ubuntu/ utopic main to /etc/apt/sources.list to get the older libxml2. However, the version for libxml-libxslt-perl is identical for utopic and vivid, namely  1.92-1build1.

From @cmaeder on June 24, 2015 7:53
Yes, I could apt-get install libxml2=2.9.1+dfsg1-4ubuntu1 and got:
maeder@lati:/var/data/localmh/MathHub/smglom/mv$ run-latexml.sh source/piecewise.de.tex
Warning: XML::LibXML compiled against libxml2 20902, but runtime libxml2 is older 20901
Warning: XML::LibXML compiled against libxml2 20902, but runtime libxml2 is older 20901
No obvious problems
Wrote /var/data/localmh/MathHub/smglom/mv/source/piecewise.de.omdoc

From @cmaeder on June 24, 2015 7:33
My machine has libxml2:amd64: /usr/lib/x86_64-linux-gnu/libxml2.so.2.9.2 whereas ash has
libxml2.so.2.9.1. I'll try, if I can replace my libxml2.

From @dginev on June 26, 2015 2:8
And so I took the plunge and upgraded to 15.04. Good news for @cmaeder : my setup 1) can't run TAP at all and 2) segfaults when running latexmlc :>
I think my libxml just got completely disconnected from XML::LibXML during the upgrades. Will hunt down more tomorrow, will borrow this issue. @brucemiller : I am calling from the future, it ain't good :>

From @dginev on June 24, 2015 12:28
Well this will bring back horrible memories. CC-ing @brucemiller here, so that he knows to expect a native LaTeXML issue soon. I am happy I am yet to update my Ubuntu installation.

From @cmaeder on June 26, 2015 7:47
@kohlhase @m-iancu How was pine.eecs.jacobs-university.de:/var/data/localmh/sty created? (I don't see a corresponding repository). Why is this directory needed (if it is at all)?

From @cmaeder on June 26, 2015 7:40
My software setup is done using https://gl.mathhub.info/meta/inf/tree/master/config/MMT
(see setupLocalMH.sh)
./setupSoftware.sh
./setupCpanm.sh
./setupLaTeXML.sh

It has a couple of packages I did not know where to put otherwise.

From @dginev on June 27, 2015 16:54
For now I am assuming the validity errors come from the libxml changes we already addressed for the last release.

From @dginev on June 27, 2015 16:53
CC-ing @brucemiller here since it's again related to the LaTeXML installation.
What had happened is that my cpanm-installed JSON::XS was out of sync with a system library, and it was segfaulting. I reverted to using the JSON::XS from the package manager, which worked like a charm.
I am also successfully running make test under libxml 2.9.2, which means I can't reproduce the validity errors reported by @cmaeder, everything seems fine.

From @cmaeder on June 29, 2015 8:46

It has a couple of packages I did not know where to put otherwise.

see #102

From @cmaeder on June 29, 2015 7:50
make test does not produce the validity errors that I reported above. They are reported for our example piecewise.de.tex from smglom/mv (and for other examples).

From @brucemiller on June 27, 2015 17:10
Any chance that this is related to a few random failures on cpantesters? (i doubt it, but...)

For \inliedef if we have more than two inlinedefs in one omtext or axiom alike, we also get this error.

I have separated out the inlinedefs error and am closing this one.


--fork-deployments removed without proper documentation

What is the replacement for:
npx hardhat node --hostname <IP_ADDRESS> --fork-deployments mumbai --as-network localhost --watch"

I don't quite understand this commit. to get it working again:
https://github.com/wighawag/hardhat-deploy/commit/8f1fbab9a39f2bcc8c64dc3943137c46f81743bc

@wighawag Any help?

There is also a change in behaviour.
Previously when npx hardhat node with config:
hardhat: {
      tags: ['test', 'local'],
      chainId: 80001,
      forking: {
        url: `https://speedy-nodes-nyc.moralis.io/${SPEEDY_NODES_API_KEY}/polygon/mumbai/archive`,
        blockNumber: 16260000, // See: https://mumbai.polygonscan.com/block/16260000
      },
    },
    localhost: {
      tags: ['local'],
    },

hre.network.tags.local will be set in deploy scripts.
The new behaviour is that it will not be set.

Hi @heri16 thanks for the report, the reason for the change was that it was hard to maintain the previous mechanism and that modifying the network name raised potential issue with other plugins. The proper solution should be handled in hardhat in my opinion, see https://github.com/nomiclabs/hardhat/issues/1164
Re tags, I ll see what I can do, maybe I need to reinstantiate the equivalent of the previous cmd options

Closing this and I created a new issue for the tags issue here : https://github.com/wighawag/hardhat-deploy/issues/202


Doubled "%"

SW details (please complete the following information):

IDE version: Android Studio 4.0.1
Plug-in Version: 1.0.0
System: Windows 10

Summary and background of the bug
All the occurrences of "%" are doubled.
Eg.

Authentication error: %s! -> Authentication error: %%s!
%1$s - %2$s -> %%1$s - %%2$s

Another problem: all the string have been put into quotation marks
Eg.

Profile -> "Profile"

Steps to reproduce
Just execute: gradlew importPoEditorStrings
Expected behavior
I expect that these parameters ("%") string stay unattached
Current behavior
All the occurrences of "%" are doubled.

Hi @prof0021! First of all, thanks for using the plugin!
About the issue with the ", that's how PoEditor exports the strings, so we can't do much about it. The raw exported exported files from PoEditor already append the quotation marks in all strings. Anyway, this shouldn't pose any problem when developing since that format is accepted by Android strings.
Regarding the double %, for now that's how the plug-in behaves, as it assumes that you're using the custom syntax we developed for text variables.
This syntax is explained in https://github.com/bq/poeditor-android-gradle-plugin#handle-placeholders. It allows us to be more explicit about the variables we define and helps translators by giving them more context about the variables. If you have any questions about the syntax, don't hesitate to ask :)
I hope this clears up the questions!

I close the issue due to inactivity.
Feel free to open it again if needed 😉


cc site not loading

see attached error

It's loading OK here.
The Ray ID isn't in our dashboard. This has happened a few times recently, I'm contacting CloudFlare support.

@robmyers okay if I close this issue?

A closed issue is a happy issue. :-)
I think this was the same problem that was causing VPN and payment issues.


Fix GET return no error when key type is not string

fixed #1969

@jihuayu Could you add a test case to make sure this issue was resolved?

@git-hulk No problem, I will add test cases for each type.

@git-hulk No problem, I will add test cases for each type.

Thank you!

@jihuayu This bug should be resolved after https://github.com/apache/kvrocks/pull/1971, but can add test cases to make sure it works well if you like.

@git-hulk All right, I'm a bit busy these days, I will come back later to solve it.

Many commands have this kind of constraint (cannot operate on keys of other types), and we may need a universal method to test them.
I will find a better way that covers this issue.
Close this PR.


Fix warnings detected in upcoming toolchains

Future Rust toolchains will warn about unknown cfgs and introduce more clippy warnings. This PR eagerly fixes those warnings. See commits for more information.

neat feature, for checking features :-D


Added support for refs.tags

Fixes #56
Fixtures added in https://github.com/philschatz/octokat.js-fixtures/issues/1

Seems to fail on travis because it uses cache mode when it runs. I used my own access when recording. But that is deleted now (for security)
It works locally when I have linked in my branch with the new fixtures. So will be resolved when the PR for the fixtures is merged. (if I understand it correctly)


(QENG-4823) AWS EC2 Reaper

This is the initial PR for the AWS EC2 Reaper.

@tvpartytonight I'll wait to review the Python once you've made the changes you were discussing IRL.
Nice work though!

OK, I believe I have added changes that address all the comments so far; ready for further review!


Formatting and rounding of very small numbers

Hi,
This is a continuation of Issue 80 - I'm afraid it won't let me re-open that one.
Formatting of very small numbers doesn't seem to follow the right path. I've updated my test program from the previous issue to show you what I mean.
package main

import (
	"fmt"

	"github.com/ericlagergren/decimal"
)

func testPrint(floatValue float64, decimalValue *decimal.Big) {
	fmt.Printf("F: %.10f\n", floatValue)
	fmt.Printf("D: %.10f\n", decimalValue)
}

func main() {
	testDecimal, _ := new(decimal.Big).SetString("0.1234567891")
	testPrint(0.1234567891, testDecimal)
	testDecimal, _ = new(decimal.Big).SetString("0.123456789")
	testPrint(0.123456789, testDecimal)
	testDecimal, _ = new(decimal.Big).SetString("0.123")
	testPrint(0.123, testDecimal)
	testDecimal, _ = new(decimal.Big).SetString("0.0000000000000000000000000000000000000000000000000000000000001")
	testPrint(0.0000000000000000000000000000000000000000000000000000000000001, testDecimal)
}

This outputs:
F: 0.1234567891
D: 0.1234567891
F: 0.1234567890
D: 0.1234567890
F: 0.1230000000
D: 0.1230000000
F: 0.0000000000
D: 0.0000000000000000000000000000000000000000000000000000000000001

As you can see, the last line doesn't follow what I'd expect for a %.10f format. I'd expect the number to be rounded to 10 decimal places, which in this case would round to zero.
Have I misunderstood something?
Many thanks,
Geoff

Yeah, sorry about that. Turns out I had to rework more of Format than I thought. It needs to be tested better. There aren't many tests for Format because generating them is a bit difficult. Should put that on my TODO list :)


docs: update Global Filter description

Description
Rename global (task) filter to Global Filter everywhere - comment, test, docs.
Motivation and Context
Have a unified description of Global Filter across the project.
How has this been tested?
Unit tests.
Types of changes
Changes visible to users:

[x] Documentation (prefix: docs - improvements to any documentation content for users)
[x] Sample vault (prefix: vault - improvements to the Tasks-Demo sample vault)
[x] Contributing Guidelines (prefix: contrib - any improvements to documentation content for contributors - see Contributing to Tasks)

Internal changes:

[x] Refactor (prefix: refactor - non-breaking change which only improves the design or structure of existing code, and making no changes to its external behaviour)
[x] Tests (prefix: test - additions and improvements to unit tests and the smoke tests)

Checklist

[x] My code follows the code style of this project and passes yarn run lint.
[x] My change requires a change to the documentation.
[x] I have updated the documentation accordingly.
[x] My change has adequate Unit Test coverage.

Terms

[x] My contribution follow this project's contributing guide
[x] I agree to follow this project's Code of Conduct

I need to think about this.
I haven't argued against it in code, but I find the capitalisation of concepts in the middle of sentences interrupts the flow of reading.

I need to think about this.
I haven't argued against it in code, but I find the capitalisation of concepts in the middle of sentences interrupts the flow of reading.

Closing...


throw error if service.id is missing

Fixes #19
I implemented this as an error instead of a warning, because the plugin just won’t work as expected if it’s not set.  This makes it more obvious that it needs to be fixed right away.

💥  :shipit: nice work @marshallswain

Released as<EMAIL_ADDRESS>


iOS bundle ID output is incorrect when using flavors

I am observing an issue when building flavors. Building flavors picks up the defaultConfigurationName instead of the -configuration when looking at BuildSettings. This causes wrong messaging on console when printing the bundle id being built . In my case, I have two configs - Release-Dev and Release-prod. When showing the output for the prod build I see the dev BUNDLE_IDENTIFIER printed to the console. Below is a selective grep for -scheme in the verbose mode.
 If no build configuration is specified and -scheme is not passed then "Release-dev" is used.
[   +4 ms] executing: [/myapp/ios/Runner.xcodeproj/] /usr/bin/xcodebuild -project /myapp/ios/Runner.xcodeproj -scheme prod -showBuildSettings
[        ] executing: [/myapp/ios/Runner.xcodeproj/] /usr/bin/xcodebuild -project /myapp/ios/Runner.xcodeproj -scheme prod -showBuildSettings
               /Applications/Xcode.app/Contents/Developer/usr/bin/xcodebuild -project /myapp/ios/Runner.xcodeproj -scheme prod -showBuildSettings
[        ] executing: [/myapp/ios/] xcrun xcodebuild -configuration Release-prod VERBOSE_SCRIPT_LOGGING=YES -workspace Runner.xcworkspace -scheme prod BUILD_DIR=/myapp/build/ios -sdk iphoneos CODE_SIGNING_ALLOWED=NO CODE_SIGNING_REQUIRED=NO CODE_SIGNING_IDENTITY="" SCRIPT_OUTPUT_STREAM_FILE=/var/folders/wh/fhcq39c902q526w__rn697340000gn/T/flutter_tools.TlDLsq/flutter_build_log_pipe.p6V1GC/pipe_to_stdout FLUTTER_SUPPRESS_ANALYTICS=true COMPILER_INDEX_STORE_ENABLE=NO

As you can see the showBuildSettings do not include the -configuration.
Running /Applications/Xcode.app/Contents/Developer/usr/bin/xcodebuild -project /myapp/ios/Runner.xcodeproj -scheme prod -showBuildSettings -configuration Release-prod | grep BUNDLE_IDENTIFIER does pick up the correct identifier as expected. I think adding the -configuration to the showBuildSettings command would be an improvement. The build seems to be working as expected bar the console messaging.

[✓] Flutter (Channel stable, 1.22.4, on Mac OS X 10.15.7 19H15 darwin-x64, locale en-CA)
    • Flutter version 1.22.4 at /somepath/s/flutter
    • Framework revision 1aafb3a8b9 (2 weeks ago), 2020-11-13 09:59:28 -0800
    • Engine revision 2c956a31c0
    • Dart version 2.10.4

[✓] Android toolchain - develop for Android devices (Android SDK version 30.0.2)
    • Android SDK at /somepath/Library/Android/sdk
    • Platform android-30, build-tools 30.0.2
    • Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b3-6222593)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 12.2)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 12.2, Build version 12B45b
    • CocoaPods version 1.10.0

[!] Android Studio (version 4.0)
    • Android Studio at /Applications/Android Studio.app/Contents
    ✗ Flutter plugin not installed; this adds Flutter specific functionality.
    ✗ Dart plugin not installed; this adds Dart specific functionality.
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b3-6222593)

[✓] VS Code (version 1.51.1)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.16.0

[✓] Connected device (2 available)
    • sdk gphone x86 arm (mobile) • emulator-5554                        • android-x86 • Android 11 (API 30) (emulator)
    • iPhone 12 Pro Max (mobile)  • 7C5E6E60-FCD0-4E02-85E0-37311B1E2D47 • ios         • com.apple.CoreSimulator.SimRuntime.iOS-14-2 (simulator)

! Doctor found issues in 1 category.

Issue extracted from https://github.com/flutter/flutter/issues/70576. @jmagman

Hi @harsimranmaan
Thanks for filing the issue
Can you please provide your flutter run --verbose > flutter_run.log logs (please provide this log file), and a minimal complete reproducible code sample in a GitHub repo?
Thank you

@harsimranmaan already provided in https://github.com/flutter/flutter/issues/70576#issuecomment-736212117.  I know what's going on here, and I have a PR that should fix it at https://github.com/flutter/flutter/pull/72309

@harsimranmaan Can you give us the flutter run output without the --verbose flag?  It doesn't really matter that the "wrong" one shows up in a verbose output--all kinds of things show up there that Flutter then parses and tweaks and passes back in.

@jmagman Since observing the original behaviour on fluter stable last month, I have moved onto flutter beta for web support. The ios release build shows the correct bundle id. Previously it showed the prod bundle id when releasing the dev app. As far as I am concerned, this issue is resolved and can be closed.
$ flutter build ios --flavor dev --release --no-sound-null-safety
Building ca.bigpanther.shipanthertest for device (ios-release)...
Automatically signing iOS for device deployment using specified development team in Xcode project: 5RZ2K4F4F5
Running pod install...                                           2,635ms
Running Xcode build...
 └─Compiling, linking and signing...                        12.3s
Xcode build done.                                           54.6s
Built /Users/harsimranmaan/dev/h/shipanther/build/ios/iphoneos/Runner.app.

@harsimranmaan Thanks for the update!

Does this works in Flutter 2.0?

@wheel1992 if it doesn't you should let us know.
If you do see it we want to fix it, but don't worry too much about it, the build was using the right bundle ID, it was just logging the wrong one.


⚠ Bofrost.at has degraded performance

In 7e129fe, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 2721 ms

Resolved: Bofrost.at performance has improved in 54c8219 after 12 minutes.


Transform3D#SetRotation() around arbitrary axis scales coordinates

The method Transform3D#SetRotation(double, Point3D) scales the co-ordinates if the axis vector Point3D is not a unit vector:
https://github.com/rdkit/rdkit/blob/7e575db3c495d5546832be2317291861fa7b1030/Code/Geometry/Transform3D.h#L64-L69
Either the documentation should state this requirement, or the method implement it.
This can be demonstrated in the Java wrapper as follows:
String molBlock = "Single Atom should be invariate\n     RDKit          3D\n\n  1  0  0  0  0  0  0  0  0  0999 V2000\n"+
"    1.0000    1.0000    1.0000 C   0  0  0  0  0  0  0  0  0  0  0  0\nM  END\n\n$$$$";
ROMol mol = RWMol.MolFromMolBlock(molBlock);

// An axis around which the molecule should be invariate:
Point3D axis = new Point3D(1.0, 1.0, 1.0);

//The transform
Transform3D t3d = new Transform3D();
t3d.SetRotation(Math.PI, axis);

//Apply it...
mol.transformMolsAtoms(t3d);

System.out.println(mol.MolToMolBlock());

Gives:
Single Atom should be invariate
     RDKit          3D

  1  0  0  0  0  0  0  0  0  0999 V2000
    3.0000    3.0000    3.0000 C   0  0  0  0  0  0  0  0  0  0  0  0
M  END

The result can be correct in the calling code, but this is not obviously required from the documentation, e.g.:
...
// An axis around which the molecule should be invariate:
Point3D axis = new Point3D(1.0, 1.0, 1.0);
axis.normalize();
...

I agree that the behavior should be properly documented.
There's an argument for changing it, but that would remove functionality and potentially break a lot of existing code.

Hi @greglandrum - I think we are agreed - the behaviour change would be 'ideal' but the breakages could be... ouch.
I will add a sentence to the doc comment in the header file and submit a pull request if you think that would cover it?

@sroughley

I will add a sentence to the doc comment in the header file and submit a pull request if you think that would cover it?
That would be perfect. Thanks!


Linear Algebra Error

Received the following error when the code was "checking for generalized symmetry" and "trying (0, 1)":

LinAlgError: Array must not contain infs or NaNs

Is there any way to fix this?

I'm having the same problem.
Initially I was only getting it when trying to apply AIF to data covering big numbers (numbers going from 1 to 10^5 in the same column), so I was able to avoid it by "normalizing" the data to numbers of order 1 (just divided every column by the average of that column).
But when I increased the number o NN training Epochs from 500 to 5000 it came back, even on the "normalized" data.

Could you please share the input file so I can reproduce the error?

Same error with input file being this one:
rocketData.txt

One example were I had this problem was (when using NN training Epochs of 5000):
exampleM2uB.txt
The function used to generate the table is of the form:
const-x0x1/x2-x1x2/x0-x0*x2/x1

One example were I had this problem was (when using NN training Epochs of 5000):
exampleM2uB.txt
The function used to generate the table is of the form:
const-x0x1/x2-x1x2/x0-x0*x2/x1

I am running into the same problem.

Exactly the same problem for me.
a-  @SJ001 did you manage to reproduce the error with the dataset given by @patrickfleith ? is it generic in every OS or specific to some installations of BLAS (as suggested here :  https://github.com/numpy/numpy/issues/4291)  ?
b- I got a results.dat file but no solution files in 'results' folder. Is is because the early stopping due to the LinAlgError ?
By the way, I like very much this "understandable" (and physically usable) ML work; Congrats !

Ran into the same issue. I'm inclined to think this might be a problem due to vanishing gradients, especially because you guys mentioned that playing with epochs or normalization appeared to prolong the inevitable. Depending on what causes the issue, np.nan_to_num might be a quick fix. Another quick fix might be to add some batch normalization layers.

anyone got it working after facing the error?


A error in Spring's new website

Spring Initializr misspelling in projects dropdown menu.
Correct spelling：Spring Initializr
Current spelling：Spring Initialzr

@zbq2019 thanks a lot for reporting it. The spring.io website is managed by the Sagan project, I've just moved your report to the appropriate issue tracker.


Unable to use the docker image

Running into this error when I do cd OpenSeq2Seq; sudo docker build -t openseq2seq .
Sending build context to Docker daemon  83.85MB
Step 1/40 : FROM gitlab-dl.nvidia.com:5005/dgx/tensorflow:18.04-py2-devel
Get https://gitlab-dl.nvidia.com:5005/v2/: dial tcp: lookup gitlab-dl.nvidia.com on <IP_ADDRESS>:53: no such host

Please try official NVIDIA TensorFlow Docker container: https://www.nvidia.com/en-us/gpu-cloud/deep-learning-containers/
It contains OpenSeq2Seq in /workspace/nvidia-examples/OpenSeq2Seq directory.


Update Craco For Release

This pull request will contain (once its done):

Removal of author tags (done)
Removal of deprecated and unused classes
Removal of schemes that we will archive
Renaming of lib in code

I have merged this into #33.

I have merged this into #33.


DM members when securing the bag 🤑

Issue Reserved for Bloomberg Mentorship Program
Description
We would like more members to update their work experiences in the member profile, and so we can prompt users to do so after they secure a new job and post about it in the channel #career-secured-the-bag. Whenever a member makes a post in #career-secured-the-bag, we want the ColorStack Bot to send them a private message with a brief message suggesting they should update their work experiences in the member profile, and a link to the work experiences page on the member profile.
Acceptance Criteria

Use a job that watches the channel #career-secured-the-bag for new messages
Extract the userId  to fetch a member’s slackId
Use the job notification.slack.send  to send a message to the user

Additional Context
Check out the function addSlackMessage which utilizes the job slack.message.add (which monitors for new slack messages) and then extracts certain values from the message and adds it to the database. For this issue, we don’t need to add anything to the database, but we do need certain values from the message.
Search the codebase to find various usages of the job notification.slack.send!
Later on we’d like to integrate AI to verify that a new post in the channel is actually announcing a new job (and not some other kind of post), but this is out of the scope for this issue.

I'll take this one!


eosjs-ecc methods and migration

Change Description
Following feedback and requests, the ecc helper functions have been renamed to match the eosjs-ecc methods where possible.  A private key generation method has also been added, as well as a PublicKey.toLegacyString() method for backwards compatibility.
Lastly, an ecc exported object is available with all the functions eosjs-ecc has that utilizes the new functions.
API Changes

[ ] API Changes

Documentation Additions

[ ] Documentation Additions

Re: Jon's comment:
If this is the code going forward, then ideally this change should be merged before v21.0.0 since these method changes are breaking.
During v21.0.0, developers will need to change their code regardless to migrate from eosjs-ecc. Adding pass-through methods in v21.1.0 will cause an unnecessary change to code again when v22.0.0 is released and those pass-through methods are removed.
I think the discussion is either change the names now to be the same as eosjs-ecc or keep them what they were before this PR.


More info in README.md

From @jjaone on October 17, 2015 9:54
Describe in the projects  GitHub Readme.md -file what the APIOps portal is, what is it used for, who are the intented users and how does it link to other national (Finland) API-communities and sites, like API:Suomi etc.
Copied from original issue: APIOps/portal#28

This issue was moved to APIOps/apiops-meetup#3


Fix maas template references after common->helm-toolkit rename

What is the purpose of this pull request?:
This fixes template rendering errors in MaaS after the common->helm-toolkit refactor: https://github.com/att-comdev/openstack-helm/pull/167
Specific reviewers for pull request:
@larryrensing

@alanmeadows i think we need to determine how MaaS will be handled once it is outside of the Openstack-Helm repository. are we requiring community operators to deploy via helm-toolkit? if so, how would this work, and what is their path? @intlabs @larryrensing

@alanmeadows LGTM -- These are the exact changes that @larryrensing  walked me through to get the MaaS pods running the in the ATLF.   Thanks!

@alanmeadows thanks for catching this, totally my bad, sorry!

this looks good @alanmeadows

@v1k0d3n We would have to discuss how we're going to handle inter-dependencies (such as helm-toolkit chart) in the issue about what to move and where

@alanmeadows i agree. we'll merge this in and then pick up the discussion in #209.
changes LGTM.


Edit html-forms for clarity.

Changed wording and deleted repetitive information to improve clarity of information.

[ x] I have read freeCodeCamp's contribution guidelines.
[ x] My pull request has a descriptive title (not a vague title like Update index.md)
[x ] My pull request targets the master branch of freeCodeCamp.
[ x] None of my changes are plagiarized from another source without proper attribution.
[ x] My article does not contain shortened URLs or affiliate links.

If your pull request closes a GitHub issue, replace the XXXXX below with the issue number.
Closes #XXXXX

LGTM

Looks good @MechanikerGA, congratulations on your first contribution to freeCodeCamp!