working deb with icon for debian/ubuntu

do you have the icon in 128x128? it took that from wikipedia
—————
Mauricio Vargas Sepúlveda 帕夏
Do you like Data Science? visit pacha.hk
你爱科学数据专吗？你走pacha.hk
2018-07-13 16:47 GMT-04:00 Thad Guidry<EMAIL_ADDRESS>
@thadguidry requested changes on this pull request.
That's our old icon...use our new one, please.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/OpenRefine/OpenRefine/pull/1679#pullrequestreview-137184681,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AJn6OZkSTBacjNtuKyL2Rd2NLf4Nx0Qsks5uGQdIgaJpZM4VPdX8
.

@pachamaltese Hack away on those as needed. :-)

do you have an svg version? the transparency is quite tricky for the icon
—————
Mauricio Vargas Sepúlveda 帕夏
Do you like Data Science? visit pacha.hk
你爱科学数据专吗？你走pacha.hk
2018-07-13 17:26 GMT-04:00 Codacy Bot<EMAIL_ADDRESS>
[image: Codacy]
https://camo.githubusercontent.com/b72992711a855bd9a03bfbdfc5ef7fcf528482f5/68747470733a2f2f6170702e636f646163792e636f6d2f6173736574732f696d616765732f66617669636f6e2e706e67
Here is an overview of what got changed by this pull request:
Issues
======- Added 1
See the complete overview on Codacy
https://app.codacy.com/app/OpenRefine/OpenRefine/pullRequest?prid=1902034&bid=8008957
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/OpenRefine/OpenRefine/pull/1679#issuecomment-404958215,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AJn6OY_czk26S2Lww4_o8jCV84cJrQ62ks5uGRB0gaJpZM4VPdX8
.

@pachamaltese Here's what we were given for SVG files
_SVG.zip

@thadguidry Hi. I was thinking about moving the bin to /opt just like chrome and other popular software. Give me a few minutes to edit the PR.

@thadguidry I had another idea that I just pushed. Now the package integrates better with Debian, it creates ~/.openrefine containing with default settings and the tools are stored there too.
Also, the full package is stored in /opt/openrefine and a link is added to /usr/bin in order to avoid touching the user PATH

@pachamaltese awesome.  Let us know when you think its fairly "ready to ship" and we'll review again.

IMO now it's ready to be reviewed
—————
Mauricio Vargas Sepúlveda 帕夏
Do you like Data Science? visit pacha.hk
你爱科学数据专吗？你走pacha.hk
2018-07-18 15:03 GMT-04:00 Thad Guidry<EMAIL_ADDRESS>
@pachamaltese https://github.com/pachamaltese awesome. Let us know when
you think its fairly "ready to ship" and we'll review again.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/OpenRefine/OpenRefine/pull/1679#issuecomment-406040035,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AJn6OfINdDsTib221tYUdhnWhbK15fmoks5uH4Z5gaJpZM4VPdX8
.

no worries
cool changes btw
and yes, on debian java version > 8 works but sometimes creates problems
because recent versions do not provide a javah path
—————
Mauricio Vargas Sepúlveda 帕夏
Do you like Data Science? visit pacha.hk
你爱科学数据专吗？你走pacha.hk
2018-08-07 5:40 GMT-04:00 Antonin Delpeuch<EMAIL_ADDRESS>
@wetneb requested changes on this pull request.
Hi @pachamaltese https://github.com/pachamaltese, sorry for the delay
in reviewing this. Because Travis build is failing, I was not expecting you
to be waiting for feedback. The priority should be to fix the Travis build

I think we need to reconsider your changes to the refine script.

I have left a couple of comments on the changes.
In DEBIAN/postrm
https://github.com/OpenRefine/OpenRefine/pull/1679#discussion_r208162082
:

@@ -0,0 +1,11 @@
+#!/bin/bash

+case "$1" in (remove)

rm /usr/bin/openrefine /usr/share/applications/openrefine.desktop
rm -rf /opt/openrefine
rm -rf $HOME/.openrefine

Should we really be deleting user data here? I think this should only
happen with the --purge option.
In refine
https://github.com/OpenRefine/OpenRefine/pull/1679#discussion_r208162434
:

@@ -4,6 +4,18 @@

OpenRefine Control System
##########################################################
+# ---------------- user settings ------------------------
+
+if [ ! -d $HOME/.openrefine ]; then

echo "Creating user directory"
mkdir $HOME/.openrefine

We currently use .local/share/openrefine by default - why change to
.openrefine?
In deb-exclude
https://github.com/OpenRefine/OpenRefine/pull/1679#discussion_r208163059
:

@@ -0,0 +1,10 @@
+.git
+.github
+.gitignore
+.gitattributes
+.travis.yml
+appveyor.yml
+DEBIAN
+refine.bat
+deb-exclude
+deb-creation.sh

Could we move this file to .deb-exclude maybe, so that it is hidden by
default, just like .gitignore?
In DEBIAN/control
https://github.com/OpenRefine/OpenRefine/pull/1679#discussion_r208163466
:

@@ -0,0 +1,12 @@
+Package: openrefine
+Version: 2.8
+Maintainer: Mauricio Vargas<EMAIL_ADDRESS>+Section: utils
+Source: OpenRefine
+Homepage: http://openrefine.org/
+Architecture: amd64

I think the architecture should not be restricted to amd64, as it should
run independently anywhere with a JVM, no?
In DEBIAN/control
https://github.com/OpenRefine/OpenRefine/pull/1679#discussion_r208163555
:

@@ -0,0 +1,12 @@
+Package: openrefine
+Version: 2.8
+Maintainer: Mauricio Vargas<EMAIL_ADDRESS>+Section: utils
+Source: OpenRefine
+Homepage: http://openrefine.org/
+Architecture: amd64
+Priority: optional
+Depends: default-jre

Should we enforce Java 8 in the dependencies?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/OpenRefine/OpenRefine/pull/1679#pullrequestreview-143913816,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AJn6Obq-mNrTqI8tTejcuuxPPzhvHxjYks5uOWCLgaJpZM4VPdX8
.

and about the architecture I am not sure because that is more openrefine
related
let me know your thoughts
—————
Mauricio Vargas Sepúlveda 帕夏
Do you like Data Science? visit pacha.hk
你爱科学数据专吗？你走pacha.hk
2018-08-07 9:10 GMT-04:00 Mauricio Vargas<EMAIL_ADDRESS>
no worries
cool changes btw
and yes, on debian java version > 8 works but sometimes creates problems
because recent versions do not provide a javah path
—————
Mauricio Vargas Sepúlveda 帕夏
Do you like Data Science? visit pacha.hk
你爱科学数据专吗？你走pacha.hk
2018-08-07 5:40 GMT-04:00 Antonin Delpeuch<EMAIL_ADDRESS>
@wetneb requested changes on this pull request.
Hi @pachamaltese https://github.com/pachamaltese, sorry for the delay
in reviewing this. Because Travis build is failing, I was not expecting you
to be waiting for feedback. The priority should be to fix the Travis build

I think we need to reconsider your changes to the refine script.

I have left a couple of comments on the changes.
In DEBIAN/postrm
https://github.com/OpenRefine/OpenRefine/pull/1679#discussion_r208162082
:

@@ -0,0 +1,11 @@
+#!/bin/bash

+case "$1" in (remove)

rm /usr/bin/openrefine /usr/share/applications/openrefine.desktop
rm -rf /opt/openrefine
rm -rf $HOME/.openrefine

Should we really be deleting user data here? I think this should only
happen with the --purge option.
In refine
https://github.com/OpenRefine/OpenRefine/pull/1679#discussion_r208162434
:

@@ -4,6 +4,18 @@

OpenRefine Control System
##########################################################
+# ---------------- user settings ------------------------
+
+if [ ! -d $HOME/.openrefine ]; then

echo "Creating user directory"
mkdir $HOME/.openrefine

We currently use .local/share/openrefine by default - why change to
.openrefine?
In deb-exclude
https://github.com/OpenRefine/OpenRefine/pull/1679#discussion_r208163059
:

@@ -0,0 +1,10 @@
+.git
+.github
+.gitignore
+.gitattributes
+.travis.yml
+appveyor.yml
+DEBIAN
+refine.bat
+deb-exclude
+deb-creation.sh

Could we move this file to .deb-exclude maybe, so that it is hidden by
default, just like .gitignore?
In DEBIAN/control
https://github.com/OpenRefine/OpenRefine/pull/1679#discussion_r208163466
:

@@ -0,0 +1,12 @@
+Package: openrefine
+Version: 2.8
+Maintainer: Mauricio Vargas<EMAIL_ADDRESS>+Section: utils
+Source: OpenRefine
+Homepage: http://openrefine.org/
+Architecture: amd64

I think the architecture should not be restricted to amd64, as it should
run independently anywhere with a JVM, no?
In DEBIAN/control
https://github.com/OpenRefine/OpenRefine/pull/1679#discussion_r208163555
:

@@ -0,0 +1,12 @@
+Package: openrefine
+Version: 2.8
+Maintainer: Mauricio Vargas<EMAIL_ADDRESS>+Section: utils
+Source: OpenRefine
+Homepage: http://openrefine.org/
+Architecture: amd64
+Priority: optional
+Depends: default-jre

Should we enforce Java 8 in the dependencies?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/OpenRefine/OpenRefine/pull/1679#pullrequestreview-143913816,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AJn6Obq-mNrTqI8tTejcuuxPPzhvHxjYks5uOWCLgaJpZM4VPdX8
.

@pachamaltese yes I think the current storage directory should be kept identical. In any case we cannot merge a PR that breaks travis build.

@wetneb Hi. Sorry about the long delay, do you think is ok to put the DEB scripts in a separate repo?

@pachamaltese I think that would be a good idea actually!

@wetneb I moved everything to a new repo with standard deb packaging https://github.com/pachamaltese/openrefine-deb

@pachamaltese awesome, thanks a lot! For now I am working on migrating the build system to Maven, I will then have a look at this and see if we can integrate your .deb workflow in the packaging process, so that we could distribute the .deb along with the other artifacts.

@wetneb in any case the deb is here: https://github.com/pachamaltese/openrefine-deb/releases

@wetneb Hi. Pls let me know if I can help retaking the DEB package.

@pachamaltese it would be great! Ideally I think we should aim to get in the Debian/Ubuntu repositories, so it would be fantistic to package OR with this goal in mind:
https://wiki.debian.org/Java/Packaging


Error: Unexpected response format - {"error": "option "temperature" must be of type float32"}

Try to change "temperature" in the "IF Chat Prompt" node. In the image you have it at 0.0. Eg. change it to 0.2.
From the Instructions:
"temperature control the randomness of the response  goes from 0-2 a lower value like 0.2 will give very strict output for creative prompts 0.7-1.2 might work well above that starts to become wild but experiment your own settings"

Try changing it to 0.2.

Thank you, at least I moved from a dead point))

You probably didn't do step 3 move the IF_AI folder from ComfyUI/custom_nodes/ComfyUI-IF_AI_tools to the root input with the rest of the presets.
When you select seed the temperature does not affect the generation only the seed number that is why there is a ramdomness mode button when you chose off the button it takes temperature mode. Also there is NaN value on the seed that should be a random number or fixed depending on your selections.
I think for that model you need a Llama 2 or 3 stopping string and probably 256 as max tokens. There is one wf that has help notes about each parameter included in the workflow folder

Also the prompt you using is too large for an SD prompt I mean you can use a long prompt but it defeats the purpose of the tool the idea is that you give it something like small two or three words phrase and let the Ai create something with it


problems happened when I  use"create a 2048 game"

Bug description

2024-04-29 11:55:25.443 | ERROR    | metagpt.utils.common:log_it:554 - Finished call to 'metagpt.actions.action_node.ActionNode._aask_v1' after 11.990(s), this was the 1st time calling it. exp: RetryError[<Future at 0x7f8ce04056d0 state=finished raised JSONDecodeError>]
2024-04-29 11:56:40.379 | INFO     | metagpt.utils.repair_llm_raw_output:repair_invalid_json:237 - repair_invalid_json, raw error: Expecting ',' delimiter: line 4 column 31 (char 89)
2024-04-29 11:56:40.380 | ERROR    | metagpt.utils.common:log_it:554 - Finished call to 'metagpt.actions.action_node.ActionNode._aask_v1' after 86.928(s), this was the 6th time calling it. exp: RetryError[<Future at 0x7f8ce04c8250 state=finished raised JSONDecodeError>]
2024-04-29 11:56:40.380 | WARNING  | metagpt.utils.common:wrapper:649 - There is a exception in role's execution, in order to resume, we delete the newest role communication message in the role's memory.
2024-04-29 11:56:40.392 | ERROR    | metagpt.utils.common:wrapper:631 - Exception occurs, start to serialize the project, exp:
Traceback (most recent call last):
File "/Users/moonmovewhilecloudchange/opt/anaconda3/envs/metagpt/lib/python3.9/site-packages/tenacity/init.py", line 382, in call
result = fn(*args, **kwargs)
File "/Users/moonmovewhilecloudchange/opt/anaconda3/envs/metagpt/lib/python3.9/site-packages/metagpt/utils/repair_llm_raw_output.py", line 296, in retry_parse_json_text
parsed_data = CustomDecoder(strict=False).decode(output)
json.decoder.JSONDecodeError: Expecting ',' delimiter: line 4 column 31 (char 89)
The above exception was the direct cause of the following exception:
Traceback (most recent call last):
File "/Users/moonmovewhilecloudchange/opt/anaconda3/envs/metagpt/lib/python3.9/site-packages/tenacity/_asyncio.py", line 50, in call
result = await fn(*args, **kwargs)
File "/Users/moonmovewhilecloudchange/opt/anaconda3/envs/metagpt/lib/python3.9/site-packages/metagpt/actions/action_node.py", line 425, in _aask_v1
parsed_data = llm_output_postprocess(
tenacity.RetryError: RetryError[<Future at 0x7f8ce04c8250 state=finished raised JSONDecodeError>]
The above exception was the direct cause of the following exception:
Traceback (most recent call last):
File "/Users/moonmovewhilecloudchange/opt/anaconda3/envs/metagpt/lib/python3.9/site-packages/metagpt/utils/common.py", line 640, in wrapper
return await func(self, *args, **kwargs)
File "/Users/moonmovewhilecloudchange/opt/anaconda3/envs/metagpt/lib/python3.9/site-packages/metagpt/roles/role.py", line 550, in run
rsp = await self.react()
tenacity.RetryError: RetryError[<Future at 0x7f8ce04c8dc0 state=finished raised RetryError>]
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "/Users/moonmovewhilecloudchange/opt/anaconda3/envs/metagpt/lib/python3.9/site-packages/metagpt/utils/common.py", line 626, in wrapper
result = await func(self, *args, **kwargs)
File "/Users/moonmovewhilecloudchange/opt/anaconda3/envs/metagpt/lib/python3.9/site-packages/metagpt/team.py", line 134, in run
await self.env.run()
Exception: Traceback (most recent call last):
File "/Users/moonmovewhilecloudchange/opt/anaconda3/envs/metagpt/lib/python3.9/site-packages/tenacity/init.py", line 382, in call
result = fn(*args, **kwargs)
File "/Users/moonmovewhilecloudchange/opt/anaconda3/envs/metagpt/lib/python3.9/site-packages/metagpt/utils/repair_llm_raw_output.py", line 296, in retry_parse_json_text
parsed_data = CustomDecoder(strict=False).decode(output)
File "/Users/moonmovewhilecloudchange/opt/anaconda3/envs/metagpt/lib/python3.9/site-packages/metagpt/utils/custom_decoder.py", line 297, in decode
return super().decode(s)
File "/Users/moonmovewhilecloudchange/opt/anaconda3/envs/metagpt/lib/python3.9/json/decoder.py", line 337, in decode
obj, end = self.raw_decode(s, idx=_w(s, 0).end())
File "/Users/moonmovewhilecloudchange/opt/anaconda3/envs/metagpt/lib/python3.9/json/decoder.py", line 353, in raw_decode
obj, end = self.scan_once(s, idx)
File "/Users/moonmovewhilecloudchange/opt/anaconda3/envs/metagpt/lib/python3.9/site-packages/metagpt/utils/custom_decoder.py", line 65, in scan_once
return _scan_once(string, idx)
File "/Users/moonmovewhilecloudchange/opt/anaconda3/envs/metagpt/lib/python3.9/site-packages/metagpt/utils/custom_decoder.py", line 36, in _scan_once
return parse_object((string, idx + 1), strict, _scan_once, object_hook, object_pairs_hook, memo)
File "/Users/moonmovewhilecloudchange/opt/anaconda3/envs/metagpt/lib/python3.9/site-packages/metagpt/utils/custom_decoder.py", line 164, in JSONObject
value, end = scan_once(s, end)
File "/Users/moonmovewhilecloudchange/opt/anaconda3/envs/metagpt/lib/python3.9/site-packages/metagpt/utils/custom_decoder.py", line 38, in _scan_once
return parse_array((string, idx + 1), _scan_once)
File "/Users/moonmovewhilecloudchange/opt/anaconda3/envs/metagpt/lib/python3.9/json/decoder.py", line 242, in JSONArray
raise JSONDecodeError("Expecting ',' delimiter", s, end - 1)
json.decoder.JSONDecodeError: Expecting ',' delimiter: line 4 column 31 (char 89)
The above exception was the direct cause of the following exception:
Traceback (most recent call last):
File "/Users/moonmovewhilecloudchange/opt/anaconda3/envs/metagpt/lib/python3.9/site-packages/tenacity/_asyncio.py", line 50, in call
result = await fn(*args, **kwargs)
File "/Users/moonmovewhilecloudchange/opt/anaconda3/envs/metagpt/lib/python3.9/site-packages/metagpt/actions/action_node.py", line 425, in _aask_v1
parsed_data = llm_output_postprocess(
File "/Users/moonmovewhilecloudchange/opt/anaconda3/envs/metagpt/lib/python3.9/site-packages/metagpt/provider/postprocess/llm_output_postprocess.py", line 19, in llm_output_postprocess
result = postprocess_plugin.run(output=output, schema=schema, req_key=req_key)
File "/Users/moonmovewhilecloudchange/opt/anaconda3/envs/metagpt/lib/python3.9/site-packages/metagpt/provider/postprocess/base_postprocess_plugin.py", line 68, in run
new_output = self.run_repair_llm_output(output=output, schema=schema, req_key=req_key)
File "/Users/moonmovewhilecloudchange/opt/anaconda3/envs/metagpt/lib/python3.9/site-packages/metagpt/provider/postprocess/base_postprocess_plugin.py", line 32, in run_repair_llm_output
parsed_data = self.run_retry_parse_json_text(content)
File "/Users/moonmovewhilecloudchange/opt/anaconda3/envs/metagpt/lib/python3.9/site-packages/metagpt/provider/postprocess/base_postprocess_plugin.py", line 47, in run_retry_parse_json_text
parsed_data = retry_parse_json_text(output=content)  # should use output=content
File "/Users/moonmovewhilecloudchange/opt/anaconda3/envs/metagpt/lib/python3.9/site-packages/tenacity/init.py", line 289, in wrapped_f
return self(f, *args, **kw)
File "/Users/moonmovewhilecloudchange/opt/anaconda3/envs/metagpt/lib/python3.9/site-packages/tenacity/init.py", line 379, in call
do = self.iter(retry_state=retry_state)
File "/Users/moonmovewhilecloudchange/opt/anaconda3/envs/metagpt/lib/python3.9/site-packages/tenacity/init.py", line 326, in iter
raise retry_exc from fut.exception()
tenacity.RetryError: RetryError[<Future at 0x7f8ce04c8250 state=finished raised JSONDecodeError>]
The above exception was the direct cause of the following exception:
Traceback (most recent call last):
File "/Users/moonmovewhilecloudchange/opt/anaconda3/envs/metagpt/lib/python3.9/site-packages/metagpt/utils/common.py", line 640, in wrapper
return await func(self, *args, **kwargs)
File "/Users/moonmovewhilecloudchange/opt/anaconda3/envs/metagpt/lib/python3.9/site-packages/metagpt/roles/role.py", line 550, in run
rsp = await self.react()
File "/Users/moonmovewhilecloudchange/opt/anaconda3/envs/metagpt/lib/python3.9/site-packages/metagpt/roles/role.py", line 517, in react
rsp = await self._react()
File "/Users/moonmovewhilecloudchange/opt/anaconda3/envs/metagpt/lib/python3.9/site-packages/metagpt/roles/role.py", line 463, in _react
rsp = await self._act()
File "/Users/moonmovewhilecloudchange/opt/anaconda3/envs/metagpt/lib/python3.9/site-packages/metagpt/roles/role.py", line 392, in _act
response = await self.rc.todo.run(self.rc.history)
File "/Users/moonmovewhilecloudchange/opt/anaconda3/envs/metagpt/lib/python3.9/site-packages/metagpt/actions/project_management.py", line 43, in run
task_doc = await self._update_tasks(filename=filename)
File "/Users/moonmovewhilecloudchange/opt/anaconda3/envs/metagpt/lib/python3.9/site-packages/metagpt/actions/project_management.py", line 66, in _update_tasks
rsp = await self._run_new_tasks(context=system_design_doc.content)
File "/Users/moonmovewhilecloudchange/opt/anaconda3/envs/metagpt/lib/python3.9/site-packages/metagpt/actions/project_management.py", line 76, in _run_new_tasks
node = await PM_NODE.fill(context, self.llm, schema=self.prompt_schema)
File "/Users/moonmovewhilecloudchange/opt/anaconda3/envs/metagpt/lib/python3.9/site-packages/metagpt/actions/action_node.py", line 505, in fill
return await self.simple_fill(schema=schema, mode=mode, images=images, timeout=timeout, exclude=exclude)
File "/Users/moonmovewhilecloudchange/opt/anaconda3/envs/metagpt/lib/python3.9/site-packages/metagpt/actions/action_node.py", line 457, in simple_fill
content, scontent = await self._aask_v1(
File "/Users/moonmovewhilecloudchange/opt/anaconda3/envs/metagpt/lib/python3.9/site-packages/tenacity/_asyncio.py", line 88, in async_wrapped
return await fn(*args, **kwargs)
File "/Users/moonmovewhilecloudchange/opt/anaconda3/envs/metagpt/lib/python3.9/site-packages/tenacity/_asyncio.py", line 47, in call
do = self.iter(retry_state=retry_state)
File "/Users/moonmovewhilecloudchange/opt/anaconda3/envs/metagpt/lib/python3.9/site-packages/tenacity/init.py", line 326, in iter
raise retry_exc from fut.exception()
tenacity.RetryError: RetryError[<Future at 0x7f8ce04c8dc0 state=finished raised RetryError>]

LLM type and model name:
System version:glm-4
Python version:3.9
MetaGPT version or branch:

packages version:
installation method:

Screenshots or logs

Seems get wrong answer from LLM, prefer gpt-4-turbo


Not working iPhone 7 plus with IOS 11

Not working properly with iPhone browsers such as Chrome, Safari and Mozilla

You'll need to install the w3c Intersection Observer polyfill in case you're targeting a browser which doesn't support it.
Intersection Observer is supported from iOS v12.2

@mornir How did you implement this fix?

@mornir How did you implement this fix?

https://github.com/w3c/IntersectionObserver/tree/master/polyfill

@mornir I've already done it, but thank you anyway

https://github.com/Timkor/nuxt-polyfill (there's an example for intersection-observer)

@EnzoBatista Then please close this issue :-)

@jigeshraval Can you please close the issue?

@mornir How did you implement this fix?

https://github.com/w3c/IntersectionObserver/tree/master/polyfill

@webistomin I installed polyfill as mention in docs, but the problem persist.
According docs, just need to install via NPM and use require('intersection-observer').
I xcode console the error reported before is not showing anymore, but the image that´s suposeed to be load, is not showing.
There is any other configuration to be done?
tks

@mornir How did you implement this fix?

https://github.com/w3c/IntersectionObserver/tree/master/polyfill

@webistomin I installed polyfill as mention in docs, but the problem persist.
According docs, just need to install via NPM and use require('intersection-observer').
I xcode console the error reported before is not showing anymore, but the image that´s suposeed to be load, is not showing.
There is any other configuration to be done?
tks

I'm usin nuxt.js, so I can only help you with that framework.
Alternatively, you can install pollyfill through Polyfill.io.
Just insert this
<script src="https://polyfill.io/v3/polyfill.min.js?features=IntersectionObserver%2CIntersectionObserverEntry"></script>


ENH: Update maintenance script

Fixes #682 and #685.
This updated version of UpdateRemoteModules.sh uses more constraining parsing rules to avoid issues and removes version specific syntax to be more portable.
It also adds the option -ongreen to update only if the latest build was successful.

May be the PR subject could be reformatted to ENH: Update remote module maintenance script so that it becomes clearer.

@maekclena Can you run the script and make a new PR with updated remotes?

I've just tested the PR and it works overall:

Nice output of the taken actions. Could be improved by grouping the updated vs. non-updated ones vs. those not update because they are failing (thus avoiding the (...) is already up to date and (...) was updated to latest commit).  But this is not a show stopper.
We still have the tag issue with the remotes (i.e. RTK would be updated to the master commit), but I did not address that either and is closely related to the way remotes can be handled concerning versions (i.e. not an easy thing to be systematical), so for the time being, this is correct to me.
Checked that the PR does fix #682. Great.
The ancient version was introducing some undesired CR/LFs (my bad) to all files that were later filtered by git (ITK hooks may be) so as to avoid unnecessary commits. The new version does not introduce them. Which is simply great. Thanks @maekclena.

Unfortunately I checked that:

The script updated the ITKWikiExamples remote, which shows a failing Azure Pipeline badge, and I trust it is failing. The script updated the remote anyway, though, so the CI check in the script seems not to be working correctly.

@maekclena whenever you find the time, could you have a look?
Note that all remote CIs will eventually be migrated to Azure Pipelines, if this is of any interest for having the status answer.

I've re-tested the script:

The output message now shows only the updated modules. Nice.
Looks like the status is now taken into account. Great.

Thanks @maekclena  !
However, there is still at least one more thing that I've detected: for remotes with upstream URLs in comment (e.g. SmoothingRecursiveYvvGaussianFilter.remote.cmake), it looks like the script takes the URL that first matches the regex, even if it lives on a commented line. Thus, in the above case, instead of updating the commit hash from the expected repository, it will do so from the upstream repository, and it will be printed as such in the output:
Modules updated:
Inria-Asclepios/SmoothingRecursiveYvvGaussianFilter

The commit to which the remote module cmake file has been updated correspond's to the upstream latest, which, in this case, is well behind ISC's latest. Even if ISC's was behind (which may happen if the upstream has a merge commit from ISC), we'd like to stick to the appropriate repository (which may not necessarily be the upstream).
So I'd dare to say that the fix boils down to discarding the regex results that lie on a comment line.

So I'd dare to say that the fix boils down to discarding the regex results that lie on a comment line.

Equally valid fix is to remove the commented lines, or put them later so they don't throw of this script.

@maekclena  Thanks ! Checked and now it is working as expected !!
:rocket: :tada: :fireworks:
This is an awesome improvement !

I will merge when Windows and CDash builds are completed successfully.


Readme windows update

I've added the steps needed to install this for development on Windows.
I've also reordered a couple of sections to make more clear the difference between quick install and full install.

Thank you Lucy!


fix(cdc-backfill): limit the number of rows for each table snapshot

I hereby agree to the terms of the RisingWave Labs, Inc. Contributor License Agreement.
What's changed and what's your intention?

Limit each table snapshot to 10k rows to avoid consuming a large number of rows when dropping a result set
Ensure the result stream has been consumed once

Tested with 6M rows lineitem table locally.
Checklist

[ ] I have written necessary rustdoc comments
[ ] I have added necessary unit tests and integration tests
[ ] I have added test labels as necessary. See details.
[ ] I have added fuzzing tests or opened an issue to track them. (Optional, recommended for new SQL features #7934).
[ ] My PR contains breaking changes. (If it deprecates some features, please create a tracking issue to remove them in the future).
[ ] All checks passed in ./risedev check (or alias, ./risedev c)
[ ] My PR changes performance-critical code. (Please run macro/micro-benchmarks and show the results.)

[ ] My PR contains critical fixes that are necessary to be merged into the latest release. (Please check out the details)

Documentation

[ ] My PR needs documentation updates. (Please use the Release note section below to summarize the impact on users)

Release note
If this PR includes changes that directly affect users or other significant modifications relevant to the community, kindly draft a release note to provide a concise summary of these changes. Please prioritize highlighting the impact these changes will have on users.

I am testing this PR with test cases of our test pipeline, expect to merge it in the early of next week.


[Question]: Why is there no longer an option to register an account on the login page?

Contact Details
not important
What is your question?
For some reason, the login interface for Librechat no longer has the option to register an account. When I enter my previously registered account, it says it cannot find the user information, resulting in being unable to use it.
More Details
no more
What is the main subject of your question?
User System/OAuth
Code of Conduct

[X] I agree to follow this project's Code of Conduct

You need to explicitly allow registration in your .env file like this:
ALLOW_REGISTRATION=true

Most who copy .env.example will have it set as such since it's listed as needed

However, I am accessing LibreChat on the website and cannot make changes to the site.

However, I am accessing LibreChat on the website and cannot make changes to the site.

are you hosting it yourself? I can't help you if you're not.


How do I get the citation key in postscript?

Is it possible to get access to the generated or pinned citekey value in the postscript JS? I'd like to add an explicit key string to BibTeX exports since the citeproc I'm using doesn't think to use the regular key when generating citation text.

You can get it from item.citekey. But I don't understand how you're trying to use it -- citeproc doesn't do BibTeX.

pandoc-citeproc can take in Bib(La)TeX as one of its input formats for handling citations in Markdown documents. However, I've found a slightly better workflow that lets me use CSL YAML with it instead.

If you export using Better CSL YAML, you'll get the citekeys as the reference IDs.

@retorquere yeah, that's what I did, although I still need citation-label explicitly set. Fortunately a simple sed command in my toolchain takes care of that.

You can set the citation-label in the postscript. Postscripts work for the CSL exports too.

On 5.0.105 You'd use a postscript such as
if (Translator.BetterCSL) { // covers both Translator.BetterCSLJSON and Translator.BetterCSLYAML
  reference['citation-label'] = item.citekey;
}

(on pre-.105, you'd have to test against Translator.BetterCSLJSON and Translator.BetterCSLYAML separately)

@retorquere having done that, I don't get citation-label in the exported YAML even with .105. For now I'm going to stick with my sed regex to duplicate id as citation-label.

:robot: this is your friendly neighborhood build bot announcing test build 6016 ("fixes #933").

My bad, I had forgotten to merge a branch. The postscript works on 6016.


feat(api): Introduce Error Codes for Node RPC

Implementation ideas
The JSON-RPC specification mandates the return of an error object with specific codes for failed requests (see JSON-RPC Error Specification). Our current node API implementation only provides data or a generic error message, omitting these specific error codes. As we continue to develop a comprehensive canonical API that will address this in the future, we can start improving our existing API by introducing error codes for common error scenarios. Initially, we don’t need a full set of refined error codes; we can implement a basic set and expand and refine these codes gradually based on user feedback and system requirements.
To streamline this process, we should establish an infrastructure to wrap all errors in a generic "Internal error" code, which can later be detailed into more specific codes as we refine our error handling strategy. This initial step will help standardize our error responses and make them more predictable for API users.
Implementation ideas
Node may utilise go-jsonrpc existing functionality for error codes, by providing global map of error codes to server and client via WithErrors option: https://github.com/filecoin-project/go-jsonrpc/blob/e75dcdc8133848b9b60335d5779e3af1ac3cb447/options.go#L88
It will allow go clients to use golang native errors.Is call to identify which error is returned. Other language clients would need to parse error code from response json object and map it to proper error value.
Some basic error codes that were requested include:

[ ] blob.GetAll: Error codes needed for scenarios such as [list specific errors here].](https://github.com/celestiaorg/celestia-node/issues/3292)

Cross posted from https://github.com/celestiaorg/celestia-node/issues/3335#issuecomment-2079748320 :

[ ] "rpc error: code = Unknown desc = timed out waiting for tx to be included in a block" error is returned frequently possibly due to 1) mempool congestion 2) big blobs 3) too small timeout_broadcast_tx_commit
[ ]  ": tx already in mempool" always happens immediately after the above error and requires waiting out until the tx drops from the mempool
[ ] ": incorrect account sequence" occasionally happens in place of the above error but also requires waiting out until the tx drops from the mempool
[ ] rpc error: code = Unknown desc = error on broadcastTxCommit: tx size is too big: 1962442, max: 1962441 happens when submitting big blobs due to padding shares even though the supposed limit is 1974272 so this breaks the contract with node.
[ ]  rpc error: code = Unknown desc = error on broadcastTxCommit: tx too large occasionally big blobs return this in place of the above. I see that there's also Tx too large. Max size is %d, but got %d in the code.

Adding error codes will enhance error traceability and debugging, making our API more robust and user-friendly in the interim period before the canonical API rollout.

related comment: https://github.com/celestiaorg/celestia-node/issues/3335#issuecomment-2079748320


super(type, instance) in classmethod is not bound

 Environment data

Language Server version: 2022.1.0
OS and version: win32 x64
Python version: 3.10.1
python.analysis.indexing: undefined
python.analysis.typeCheckingMode: strict

 Expected behaviour
super(type, instance) should always give access to bound methods.
 Actual behaviour
super(type, instance), when invoked within a classmethod or staticmethod, is given a type of Type[Class] rather than Class, and therefore methods of it are considered unbound and require one extra positional argument.
 Logs

Python Language Server Log
[Info  - 23:41:19] Pylance language server 2022.1.0 (pyright 5829dcbb) starting
[Info  - 23:41:19] Server root directory: c:\Users\henry\.vscode\extensions\ms-python.vscode-pylance-2022.1.0\dist
[Info  - 23:41:19] No configuration file found.
[Info  - 23:41:19] No pyproject.toml file found.
[Info  - 23:41:19] Setting pythonPath for service "(workspace)": "C:\Users\henry\AppData\Local\Programs\Python\Python310\python.exe"
[Warn  - 23:41:19] stubPath (workspace)\typings is not a valid directory.
[Info  - 23:41:19] Assuming Python version 3.10
[Info  - 23:41:19] Assuming Python platform Windows
Search paths for (workspace)
  c:\Users\henry\.vscode\extensions\ms-python.vscode-pylance-2022.1.0\dist\typeshed-fallback\stdlib
  (workspace)
  (workspace)\typings
  c:\Users\henry\.vscode\extensions\ms-python.vscode-pylance-2022.1.0\dist\typeshed-fallback\stubs\...
  c:\Users\henry\.vscode\extensions\ms-python.vscode-pylance-2022.1.0\dist\bundled\stubs
  c:\Users\henry\AppData\Local\Programs\Python\Python310\DLLs
  c:\Users\henry\AppData\Local\Programs\Python\Python310\Lib
  c:\Users\henry\AppData\Local\Programs\Python\Python310
  c:\Users\henry\AppData\Local\Programs\Python\Python310\Lib\site-packages
  (...)
  c:\Users\henry\AppData\Local\Programs\Python\Python310\Lib\site-packages\win32
  c:\Users\henry\AppData\Local\Programs\Python\Python310\Lib\site-packages\win32\lib
  c:\Users\henry\AppData\Local\Programs\Python\Python310\Lib\site-packages\pythonwin
[Info  - 23:41:20] Searching for source files
[Info  - 23:41:20] Found 9 source files
[Info  - 23:41:20] Background analysis(1) root directory: c:\Users\henry\.vscode\extensions\ms-python.vscode-pylance-2022.1.0\dist
[Info  - 23:41:20] Background analysis(1) started
Background analysis message: setConfigOptions
Background analysis message: setImportResolver
Background analysis message: ensurePartialStubPackages
[FG] parsing: (workspace)\example.py (48ms)
[FG] parsing: c:\Users\henry\.vscode\extensions\ms-python.vscode-pylance-2022.1.0\dist\typeshed-fallback\stdlib\builtins.pyi [fs read 5ms] (188ms)
[FG] binding: c:\Users\henry\.vscode\extensions\ms-python.vscode-pylance-2022.1.0\dist\typeshed-fallback\stdlib\builtins.pyi (78ms)
[FG] binding: (workspace)\example.py (2ms)
Background analysis message: setTrackedFiles
Background analysis message: markAllFilesDirty
Background analysis message: setFileOpened
Background analysis message: analyze
IntelliCode model c:\Users\henry\.vscode\extensions\visualstudioexptteam.vscodeintellicode-1.2.15\cache\E61945A9A512ED5E1A3EE3F1A2365B88F8FE_E4E9EADA96734F01970E616FAB2FAC19
[BG(1)] analyzing: (workspace)\example.py ...
[BG(1)]   parsing: (workspace)\example.py (79ms)
[BG(1)]   parsing: c:\Users\henry\.vscode\extensions\ms-python.vscode-pylance-2022.1.0\dist\typeshed-fallback\stdlib\builtins.pyi [fs read 6ms] (327ms)
[BG(1)]   binding: c:\Users\henry\.vscode\extensions\ms-python.vscode-pylance-2022.1.0\dist\typeshed-fallback\stdlib\builtins.pyi (131ms)
[BG(1)]   binding: (workspace)\example.py (3ms)
[BG(1)]   checking: (workspace)\example.py ...
[BG(1)]     parsing: c:\Users\henry\.vscode\extensions\ms-python.vscode-pylance-2022.1.0\dist\typeshed-fallback\stdlib\typing_extensions.pyi [fs read 6ms] (40ms)
[BG(1)]     binding: c:\Users\henry\.vscode\extensions\ms-python.vscode-pylance-2022.1.0\dist\typeshed-fallback\stdlib\typing_extensions.pyi (3ms)
[BG(1)]     parsing: c:\Users\henry\.vscode\extensions\ms-python.vscode-pylance-2022.1.0\dist\typeshed-fallback\stdlib\typing.pyi [fs read 1ms] (98ms)
[BG(1)]     binding: c:\Users\henry\.vscode\extensions\ms-python.vscode-pylance-2022.1.0\dist\typeshed-fallback\stdlib\typing.pyi (34ms)
[BG(1)]     parsing: c:\Users\henry\.vscode\extensions\ms-python.vscode-pylance-2022.1.0\dist\typeshed-fallback\stdlib\_typeshed\__init__.pyi [fs read 1ms] (30ms)
[BG(1)]     binding: c:\Users\henry\.vscode\extensions\ms-python.vscode-pylance-2022.1.0\dist\typeshed-fallback\stdlib\_typeshed\__init__.pyi (4ms)
[BG(1)]     parsing: c:\Users\henry\.vscode\extensions\ms-python.vscode-pylance-2022.1.0\dist\typeshed-fallback\stdlib\types.pyi [fs read 1ms] (29ms)
[BG(1)]     binding: c:\Users\henry\.vscode\extensions\ms-python.vscode-pylance-2022.1.0\dist\typeshed-fallback\stdlib\types.pyi (22ms)
[BG(1)]     parsing: c:\Users\henry\.vscode\extensions\ms-python.vscode-pylance-2022.1.0\dist\typeshed-fallback\stdlib\abc.pyi [fs read 1ms] (2ms)
[BG(1)]     binding: c:\Users\henry\.vscode\extensions\ms-python.vscode-pylance-2022.1.0\dist\typeshed-fallback\stdlib\abc.pyi (1ms)
[BG(1)]     parsing: c:\Users\henry\.vscode\extensions\ms-python.vscode-pylance-2022.1.0\dist\typeshed-fallback\stdlib\traceback.pyi [fs read 1ms] (11ms)
[BG(1)]     binding: c:\Users\henry\.vscode\extensions\ms-python.vscode-pylance-2022.1.0\dist\typeshed-fallback\stdlib\traceback.pyi (4ms)
[BG(1)]   checking: (workspace)\example.py (494ms)
[BG(1)] analyzing: (workspace)\example.py (1041ms)
Background analysis message: getSemanticTokens full
[BG(1)] getSemanticTokens full at (workspace)\example.py (42ms)
Background analysis message: getSemanticTokens range
[BG(1)] getSemanticTokens range 39:0 - 90:0 at (workspace)\example.py (41ms)
Background analysis message: resumeAnalysis
Loading ONNX runtime...
Loaded ONNX runtime. Creating IntelliCode session...
Created IntelliCode session.
Initialize deeplearning succeeded
[FG] parsing: c:\Users\henry\.vscode\extensions\ms-python.vscode-pylance-2022.1.0\dist\typeshed-fallback\stdlib\typing_extensions.pyi [fs read 2ms] (18ms)
[FG] binding: c:\Users\henry\.vscode\extensions\ms-python.vscode-pylance-2022.1.0\dist\typeshed-fallback\stdlib\typing_extensions.pyi (6ms)
[FG] parsing: c:\Users\henry\.vscode\extensions\ms-python.vscode-pylance-2022.1.0\dist\typeshed-fallback\stdlib\typing.pyi [fs read 3ms] (56ms)
[FG] binding: c:\Users\henry\.vscode\extensions\ms-python.vscode-pylance-2022.1.0\dist\typeshed-fallback\stdlib\typing.pyi (12ms)
[FG] parsing: c:\Users\henry\.vscode\extensions\ms-python.vscode-pylance-2022.1.0\dist\typeshed-fallback\stdlib\_typeshed\__init__.pyi [fs read 2ms] (15ms)
[FG] binding: c:\Users\henry\.vscode\extensions\ms-python.vscode-pylance-2022.1.0\dist\typeshed-fallback\stdlib\_typeshed\__init__.pyi (3ms)
[FG] parsing: c:\Users\henry\.vscode\extensions\ms-python.vscode-pylance-2022.1.0\dist\typeshed-fallback\stdlib\types.pyi [fs read 1ms] (25ms)
[FG] binding: c:\Users\henry\.vscode\extensions\ms-python.vscode-pylance-2022.1.0\dist\typeshed-fallback\stdlib\types.pyi (5ms)
[FG] parsing: c:\Users\henry\.vscode\extensions\ms-python.vscode-pylance-2022.1.0\dist\typeshed-fallback\stdlib\abc.pyi [fs read 1ms] (3ms)
[FG] binding: c:\Users\henry\.vscode\extensions\ms-python.vscode-pylance-2022.1.0\dist\typeshed-fallback\stdlib\abc.pyi (1ms)
[FG] parsing: C:\Users\henry\AppData\Local\Temp\pyright-15380-4yyQnIjee5jL\builtins-15380-GMgPIxJ7rLBH-.py [fs read 17ms] (185ms)
[FG] binding: C:\Users\henry\AppData\Local\Temp\pyright-15380-4yyQnIjee5jL\builtins-15380-GMgPIxJ7rLBH-.py (45ms)
Background analysis message: getDiagnosticsForRange
Background analysis message: getDiagnosticsForRange
Background analysis message: getDiagnosticsForRange
(etc.)

 Code Snippet / Additional information
from traceback import print_exc
from typing import TYPE_CHECKING, Any

if not TYPE_CHECKING:
    def reveal_type(obj: Any) -> None:
        pass

class BaseClass:
    def my_method(self, value: int) -> int:
        return value

class SubClass(BaseClass):
    def method_plain_super(self, value: int) -> int:
        reveal_type(super())  # => BaseClass
        return super().my_method(value)

    # correctly accepted
    def method_super(self, value: int) -> int:
        reveal_type(super(__class__, self))  # => BaseClass
        return super(__class__, self).my_method(value)

    # correctly erroneous
    def method_super_extra_arg(self, value: int) -> int:
        reveal_type(super(__class__, self))  # => BaseClass
        # Return type is unknown Pylance(reportUnknownVariableType)
        # Expected 1 positional argument Pylance(reportGeneralTypeIssues)
        return super(__class__, self).my_method(self, value)

    # incorrectly erroneous
    @classmethod
    def classmethod_super(cls, value: int) -> int:
        self = cls()
        reveal_type(super(__class__, self))  # => Type[BaseClass]
        # Argument missing for parameter "value" Pylance(reportGeneralTypeIssues)
        # Return type is unknown Pylance(reportUnknownVariableType)
        return super(__class__, self).my_method(value)

    # incorrectly accepted
    @classmethod
    def classmethod_super_extra_arg(cls, value: int) -> int:
        self = cls()
        reveal_type(super(__class__, self))  # => Type[BaseClass]
        return super(__class__, self).my_method(self, value)

    # same problem with staticmethod as with classmethod

    @staticmethod
    def staticmethod_super(value: int) -> int:
        self = SubClass()
        reveal_type(super(__class__, self))  # => Type[BaseClass]
        # Argument missing for parameter "value" Pylance(reportGeneralTypeIssues)
        # Return type is unknown Pylance(reportUnknownVariableType)
        return super(__class__, self).my_method(value)

    @staticmethod
    def staticmethod_super_extra_arg(value: int) -> int:
        self = SubClass()
        reveal_type(super(__class__, self))  # => Type[BaseClass]
        # accepted
        return super(__class__, self).my_method(self, value)

print(SubClass().method_plain_super(0))  # => 0
print(SubClass().method_super(0))  # => 0

try:
    # TypeError: Test.method1() takes 2 positional arguments but 3 were given
    print(SubClass().method_super_extra_arg(0))
except:
    print_exc()

print(SubClass().classmethod_super(0))  # => 0

try:
    # TypeError: Test.method1() takes 2 positional arguments but 3 were given
    print(SubClass().classmethod_super_extra_arg(0))
except:
    print_exc()

print(SubClass().staticmethod_super(0))  # => 0

try:
    # TypeError: Test.method1() takes 2 positional arguments but 3 were given
    print(SubClass().staticmethod_super_extra_arg(0))
except:
    print_exc()

Thanks for the bug report. I agree this is a problem. I'll work on a fix.

This will be fixed in the next release.

This issue has been fixed in version 2022.1.1, which we've just released. You can find the changelog here:  CHANGELOG.md


Handle character codes in JSON Responses (ie. ''' should be apostrophe)

Fields in Jasonette need to handle character codes in JSON by translating them to their appropriate characters.
When the server returns JSON containing these characters, they display in the Jasonette app as character codes rather than being replaced.
Currently, adding this to each field does the trick, but a solution is needed app-wide for Jasonette.
{{$get.moderator_profile.name.replace(/&#([0-9]+);/g, function(match, dec) { return String.fromCharCode(dec);})}}

It would a be a hit to performance, but you could do the same replace in the project for all values individually or you could transform the entire JSON response with a replace one time. Possibly in the JSONResponseSerializer some encoding option or replace/processing could be done on the entire response if its JSON...
Any way its done, its obviously going to need native implementations for iOS/Android individually.

I think the server should handle the html chars.
For example in php using html_entities_decode function.
A Rest API should response with the right character encoding.
😉

I think the crux of the issue is that it's cumbersome to write this function every time. Which means maybe there's a way to solve this by making it easy to reuse template expressions. Mixins kind of work but I can see how it's still not optimal if you want to use the same function but with different variables every time.
Maybe if we can build a way to reuse Javascript functions and objects.....

How about in the head's actions.
"head" : {
  "actions" : {
    "decode" : {
      "type" : "$js.set",
      "options" : {
        "name" : "decode(name)",
        "description" : "{{name.replace(/&#([0-9]+);/g, function(match, dec) { return     String.fromCharCode(dec);})}}"
      }
    }
   }
}

Usage
$js.fn.decode($get.moderator_profile.name)

I like this idea, @clsource. But, unfortunately for us it still means that we will have to deal with this decoding all over the place in our app. Every single editable field of ours in potentially hundreds of views will need this same decoding. I believe this decoding should be handled transparently for Jasonette in the JSONResponseSerializer.
I'm not an expert in string encoding, but this is how I understand things. Our server returns JSON that contains HTML entity encoded characters.
Here's a link to the ASCII translation table.
http://www.asciitable.com
Under the HTML column you can see &#charcode values. We need all of these values decoded in all of our response strings for our entire app essentially.
When I use Jasonette and log the JSON response to console it contains the encoded string:
shortDescription = "this has a \U2018 in it";
When I use POSTMAN ( a tool for playing around with Web APIs), the raw response has been decoded:
"shortDescription": "this has a ‘ in it"
I use this as an example of how this should be transparently handled for Jasonette users as they will expect this decoding.
@gliechtenstein There's a slew of discussion about decoding HTMLEntities in strings in obj-c. (Here's one place to get started.) I believe this needs to be implemented natively for both iOS and Android.
https://stackoverflow.com/questions/7602693/ios-html-unicode-to-nsstring?noredirect=1&lq=1

I think then this functionality could be added to the core
like blade using.
{{!! !!}}
in that way will have the same behaviour as using {{ }} but pasing a html_entity_decode function.
I think jasonette should not have a global html_entity_decode mainly because it will have problems with certain components like html.
I think it's better to have a special brackets for that use case.

it will have problems with certain components like html.
That is true.

I think the special brackets like blade aren't a bad idea. It'll mean a lot of edits for us, but it'll be very intentional and clear. I'll just add that maybe different types of decoding should be added to this capability and thought of first before making this syntax specifically for decoding HTML character character or entities only.

What about this.
{% {{value}}, html_decode | trim | uppercase | base64_encode | md5 | sha1 | etc %}
{% <jasonette expression> <comma> <post processing functions chainable by | > %}
in that way you can have single param functions that could be called after evaluating a {{ }}  expression.

I'm closing this issue since it will be solved with the Jasonette's Javascript imports
"head" : {
   "scripts"
}"

Please if you have any comments go to https://github.com/Jasonette/Jasonette/issues/4
👍
Thanks for helping Jasonette 😄


Faulty chat links

Link doesn't take you to page when clicking chat links you've joined:

Link above is the url of the soup group chat

Link above is the url i clicked, which in this case is "https://mybearworld.github.io/roarer/#/chats/25ecd679-e81b-4cc0-a388-0652a8e2abc2", "🦖 Dinonuggie's EMPIRE 🦖", but it doesn't take you there. Reloading the tab will get you to the group chat link you've clicked.

Can't replicate this - what browser/OS are you on?

Can't replicate this - what browser/OS are you on?

ChromeOS Google Chrome
Just go to a group chat and click a different group chat link.

You can go to my group chat here:
https://mybearworld.github.io/roarer/#/chats/61e6f424-a738-4cbb-9525-d61d686b6eac
If you look below, i did a command with my @SoupBot that allowed it to posts all empire group chat links. Click one of the links.

This only happens in group chats, I can replicate it now

This only happens in group chats, I can replicate it now

Good to hear

This seems to have the same root cause as #102, I'll use that to have everything in one place


Create dnf-sort.cpp

I have added a sorting algorithm popularly known as “Dutch National Flag algorithm” or dnf-sort used to sort an array of 0's,1's and 2's in one go.
Language used is C++.

[x] My submission is formatted according to the guidelines in the contributing guide

[x] My addition is on refer on the language README.md file

[x] My addition does not have a spelling problem

[x] My submission has a proper and user-friendly description of the algorithm

[x] My submission has the time complexity of the algorithm

[x] My submission has sample input-output of the program (NOT FOR PYTHON)

[ ] Bugfix

[x] New algorithm

[ ] Optimization in previous algorithms

[ ] Code style update

[ ] Refactor

[ ] Documentation

[ ] Other, please describe:

I have added a sorting algorithm popularly known as “Dutch National Flag algorithm” or dnf-sort used to sort an array of 0's,1's and 2's in one go.
Language used is C++.


Add TechDocs plugin

[ ] I added a CHANGELOG entry for this change.
[ ] I adjusted the corresponding documentation for this change.
[ ] I adjusted the values.yaml file and the documentation for all values.

Codecov Report

Merging #333 (275bfb9) into rework-kobs-architecture (a958e23) will decrease coverage by 2.06%.
The diff coverage is 16.13%.

@@                     Coverage Diff                      @@
##           rework-kobs-architecture     #333      +/-   ##
============================================================
- Coverage                     50.25%   48.19%   -2.07%
============================================================
  Files                            77       83       +6
  Lines                          4911     5227     +316
============================================================
+ Hits                           2468     2519      +51
- Misses                         2383     2647     +264
- Partials                         60       61       +1

Impacted Files
Coverage Δ

packages/plugin-sql/cmd/sql.go
49.05% <ø> (ø)

packages/plugin-techdocs/pkg/providers/s3/s3.go
0.00% <0.00%> (ø)

packages/plugin-techdocs/pkg/shared/shared.go
0.00% <0.00%> (ø)

pkg/hub/api/plugins/plugins.go
0.00% <0.00%> (ø)

...kages/plugin-techdocs/pkg/providers/local/local.go
9.30% <9.30%> (ø)

packages/plugin-techdocs/cmd/techdocs.go
26.60% <26.60%> (ø)

packages/plugin-techdocs/pkg/instance/instance.go
33.33% <33.33%> (ø)

...ackages/plugin-techdocs/pkg/providers/providers.go
77.77% <77.77%> (ø)

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 a958e23...275bfb9. Read the comment docs.


App Svc Resiliency Report: Enable Health Check

Todo

Create health check endpoint
Update App Svc to send requests to Health Check endpoint
Update Azure Front Door to use Health Check endpoint

We have a manual failover process and there are no metrics being reported in this health check so it doesn't add value to describe it in this phase. In the next phase we'll have metrics to measure.


Move override support for max tx capacities into the ProtocolInfo constructors

Fixes #3235 (again).

@Jasagredo This is ready for review now. The first 5 commits are as before, the remaining commits are a bit of a departure. It's something Pawel and I had discussed before, and once I got my hands in here, it seemed worthwhile. The commit message on consensus: implement TxLimits.pointwiseMin via a meet-semilattice explains my reasoning.
I would understand if this feels like too much. The reason I've left it in one PR is because the downstream interface is different as of the last commit. I don't like merging work that is known to be incomplete, so I'd like to review these commits together.
On that note, I've left it as a Draft PR after all, since I'll want to squish these commits down once we're happy with them; the first commit goes one way (introduces Overrides = Measure -> Measure1), but we end up somewhere else by the end of the PR (Overrides = Measure).
Also, if the Generics stuff is unfamiliar, let me know and I can ping Clarke/Edsko/etc for review there 👍

We discussed this in the Planning Meeting; Consensus Team + @nc6

The lattice based approach is fine, has enough minor benefits that they add up.
Same for the generic definitions -- they're very simple and allow for deriving everything for AlonzoMeasure (and future such types).
We should define the lattice stuff in cardano-base, however. (Spoiler: I'm going to send FromGeneric with it in my PR there.)

We also spent a couple hours (by now) discussing what the right interface is for these lattice concepts we're invoking. This question remains unanswered. But here are some remarks.

We know we have a ("full") lattice: both meet and join and both bottom and top. EG in the concrete cases (like AlonzoMeasure), we want https://en.wikipedia.org/wiki/Product_order , which gives a full lattice.
However, we do not want a join operation. Whereas meet lets the user's override tighten the bound (ie restrict it), join would loosen it (ie relax it). Which would create a block that other nodes not overriding the ledger limit would then consider invalid. On the other hand, meet can't create invalid blocks, just smaller-than-necessary blocks.
So what we want is the full lattice structure, but also to explicitly prevent any (direct?) use of join.

Then we questioned if we definitely want to disallow relaxing the ledger limit -- eg maybe benchmarking team would do that to every node on a testnet. But in that case they should just change the limit in the ledger state so that the largest block they'd want to create is valid on every node. They could still use the restriction (ie meet) in order to have some nodes create smaller-than-necessary blocks, if they're going for a mixture.
Ultimately, though, we're wondering who really needs this --- maybe we should rip it out, the ultimate simplification :D

Ultimately, though, we're wondering who really needs this --- maybe we should rip it out, the ultimate simplification :D I've asked on the Node Team slack channel for clarification regarding this, included pinging Benchmarking Team lead.

@njd42 and I discussed it. He pointed out that honest SPOs being able to quickly restrict the size of blocks could help us avoid potential disaster scenarios. And some honest SPOs may (currently?) need this to make-up for limited CPU/Mem/etc resources. So we should include it. (Even if we excluded it, it'd be easy enough for some one dedicated to patch into the code.)

@EncodePanda -- as has become par for the course, I'm surprised that I didn't get this merged during your vacation week. I'll Slack you some details, but please focus on Genesis. I expect we can review this on a quick phone call sometime early this week.

@EncodePanda I'll resolve the two open Comments today (comments on Overrides and the change in type on noOverrides and mkOverrides to make it simpler for users to understand and use).
Would you review the rest of the PR around those? Those two will be straight forward for me, so please feel free to Approve before I upload fixes for those.

Thank you @EncodePanda.
I rebased just now; automatic, no conflicts 👍

bors r+


--restrict-lexicon

I find --restrict-lexicon at arguments.py. Has this argument beem implemented and is there any instructions for it?

There is some documentation about this here (https://github.com/awslabs/sockeye/tree/master/sockeye_contrib/fast_align), describing how to create a lexicon from fast_align models. If used in decoding, the model can only generate words that are part of this lexicon.

Thanks, and I close this issue.

When I got the result of lex.out from fast_align, I deleted most of the lines, leaving only some words that I focused on.  The model exactly only generated words that are part of this lexicon in decoding, as you said above. But the model can never gererate words that I deleted.
I have two questions:
1> As the results show, it seems that it is not the combination of discrete probability and continuous probability, but only discrete probability is used. Why can't words out of the lexicon be generated?
2> I added --restrict-lexicon key1:path1 key2:path2 ... when starting the model, and used  {"text": "some input string", "restrict_lexicon": "key"} when tanslating a sentence.  Is there any way to dynamically add any vocabularies when translating?  Because I want the same source word to be translated into different target words in different sentences depending on different lexicon.

As the results show, it seems that it is not the combination of discrete probability and continuous probability, but only discrete probability is used.

I don't understand what you mean by a "combination of discrete and continuous probability". Could you perhaps explain this a bit more?

Why can't words out of the lexicon be generated?

Because that is precisely what --restrict-lexicon does: it limits possible translations to the target tokens present in a lexicon.
What is perhaps confusing is that for a given source sentence, the list of allowed target tokens that can be generated is the set of all target tokens that appear in the lexicon as the translation of at least one of the tokens in the source sentence. Only this subset of target vocabulary entries is used to make predictions, in the sense that the softmax used to create a distribution over the target vocabulary is fed only the probabilities of this subset of the vocabulary.
You perhaps assumed that if you delete certain source tokens from the lexicons, the model is free to choose a translation for them among all possible target words. But there is no token-level alignment in a standard NMT model, it cannot be determined which source token "causes" which target token.

The paper being referenced is: https://arxiv.org/pdf/1705.01991.pdf

Thanks from you detail reply.
Maybe it’s appropriate to call it "combination of probability coming from discrete mapping and probability coming from continuous representation (the NMT probability)" instead of "combination of discrete and continuous probability". That come from this paper Incorporating Discrete Translation Lexicons into Neural Machine Translation. Has sockey2 implemented this paper? I find only one function quotes this paper and one issues mentions it.
My 1> question was that if I delete certain source tokens from the lexicons, the model will use NMT probability only , instead of the combination of two.

Sockeye2 has not implemented this paper. It was part of an old version of Sockeye 1 (as the issue you link to states), but was removed in the meantime.
The current --restrict-lexicon functionality simply causes the final output layer to be defined over a subset of tokens (the ones that are part of the lexicon). There is no combination of probabilities at all, in fact the lexicon that you would provide, does not even store any probabilities, it simply stores source to target word mappings that should define the set of target words the beam search during inference can search over.

Could you tell me the reason why removed this implementation, is it not as good as claimed in the paper? And is there any plan to reimplement it in the future?

I honestly don't remember, this is a few years back. From our side there are no plans to add this back, but feel free to open a PR.

Thank you very much，I close it.

Using word alignment probabilities introduces a bias towards pre-trained word level mappings.  This was a great way to boost early NMT models that had some known weaknesses and didn't learn as well as modern models.  As NMT models got stronger, we passed a point where adding a word alignment bias actually degraded quality compared to letting the model learn everything on its own.

@mjdenkowski Thank you for your explanation. My original idea is to use sockeye to do paraphrase generation(monolingual), and I want to use the idea of this paper to integrate synonyms,  which I think maybe achieve the goal of diverse paraphrase. Anyway, thank you for your explanation. I'll look for some other ways.


Fatal Error: Install linuxbrew bash completion stuff

Hi,
I got the below error while executing sudo make docker under Mac OS X.
TASK [galaxyprojectdotorg.devbox : Install linuxbrew bash completion stuff] ****
fatal: [localhost]: FAILED! => {"changed": true, "cmd": "/home/ubuntu/.linuxbrew/bin/brew install -v bash-completion", "delta": "0:00:00.371178", "end": "2016-11-22 03:44:50.997213", "failed": true, "rc": 1, "start": "2016-11-22 03:44:50.626035", "stderr": "/home/ubuntu/.linuxbrew/Library/brew.sh: line 26: warning: setlocale: LC_ALL: cannot change locale (en_US.UTF-8): No such file or directory\nError: undefined method `rebuild' for #<BottleSpecification:0x00000002dbac20>\nPlease report this bug:\n    https://github.com/Linuxbrew/linuxbrew/blob/master/share/doc/homebrew/Troubleshooting.md#troubleshooting\n/home/ubuntu/.linuxbrew/Library/Taps/homebrew/homebrew-core/Formula/bash-completion.rb:13:in `block in <class:BashCompletion>'\n/home/ubuntu/.linuxbrew/Library/Homebrew/software_spec.rb:91:in `instance_eval'\n/home/ubuntu/.linuxbrew/Library/Homebrew/software_spec.rb:91:in `bottle'\n/home/ubuntu/.linuxbrew/Library/Homebrew/formula.rb:1729:in `bottle'\n/home/ubuntu/.linuxbrew/Library/Taps/homebrew/homebrew-core/Formula/bash-completion.rb:11:in `<class:BashCompletion>'\n/home/ubuntu/.linuxbrew/Library/Taps/homebrew/homebrew-core/Formula/bash-completion.rb:3:in `load_formula'\n/home/ubuntu/.linuxbrew/Library/Homebrew/formulary.rb:21:in `module_eval'\n/home/ubuntu/.linuxbrew/Library/Homebrew/formulary.rb:21:in `load_formula'\n/home/ubuntu/.linuxbrew/Library/Homebrew/formulary.rb:34:in `load_formula_from_path'\n/home/ubuntu/.linuxbrew/Library/Homebrew/formulary.rb:83:in `load_file'\n/home/ubuntu/.linuxbrew/Library/Homebrew/formulary.rb:74:in `klass'\n/home/ubuntu/.linuxbrew/Library/Homebrew/formulary.rb:70:in `get_formula'\n/home/ubuntu/.linuxbrew/Library/Homebrew/formulary.rb:207:in `factory'\n/home/ubuntu/.linuxbrew/Library/Homebrew/formulary.rb:352:in `find_with_priority'\n/home/ubuntu/.linuxbrew/Library/Homebrew/extend/ARGV.rb:20:in `block in formulae'\n/home/ubuntu/.linuxbrew/Library/Homebrew/extend/ARGV.rb:16:in `map'\n/home/ubuntu/.linuxbrew/Library/Homebrew/extend/ARGV.rb:16:in `formulae'\n/home/ubuntu/.linuxbrew/Library/Homebrew/cmd/install.rb:44:in `install'\n/home/ubuntu/.linuxbrew/Library/brew.rb:84:in `<main>'", "stdout": "", "stdout_lines": [], "warnings": []}

	to retry, use: --limit @/tmp/ansible/provision.retry

PLAY RECAP *********************************************************************
localhost                  : ok=9    changed=8    unreachable=0    failed=1

The command '/bin/sh -c ANSIBLE_FORCE_COLOR=1 PYTHONUNBUFFERED=1 ansible-playbook /tmp/ansible/provision.yml --tags=image        -c local -e<EMAIL_ADDRESS>$ANSIBLE_EXTRA_VARS &&     ANSIBLE_FORCE_COLOR=1 PYTHONUNBUFFERED=1 ansible-playbook /tmp/ansible/provision.yml --tags=database     -c local -e<EMAIL_ADDRESS>$ANSIBLE_EXTRA_VARS &&     ANSIBLE_FORCE_COLOR=1 PYTHONUNBUFFERED=1 ansible-playbook /tmp/ansible/provision.yml --tags=galaxy       -c local -e<EMAIL_ADDRESS>$ANSIBLE_EXTRA_VARS &&     ANSIBLE_FORCE_COLOR=1 PYTHONUNBUFFERED=1 ansible-playbook /tmp/ansible/provision.yml --tags=toolshed     -c local -e<EMAIL_ADDRESS>$ANSIBLE_EXTRA_VARS &&     ANSIBLE_FORCE_COLOR=1 PYTHONUNBUFFERED=1 ansible-playbook /tmp/ansible/provision.yml --tags=galaxyextras -c local -e<EMAIL_ADDRESS>$ANSIBLE_EXTRA_VARS &&     ANSIBLE_FORCE_COLOR=1 PYTHONUNBUFFERED=1 ansible-playbook /tmp/ansible/provision.yml --tags=devbox       -c local -e<EMAIL_ADDRESS>$ANSIBLE_EXTRA_VARS &&      sh /tmp/cleanup.sh &&     apt-get autoremove -y && apt-get clean && rm -rf /var/lib/apt/lists/* /tmp/* /var/tmp/*' returned a non-zero code: 2
make: *** [docker] Error 2

What did I do wrong?
Thank you in advance.
Mic

I believe this https://github.com/galaxyproject/planemo-machine/commit/de1fdab104c97cca20a2d5c6eff8985186534ccc fixed that - sorry for the delay in responding to this. Thanks for the report and for trying to get this to work.


Update UG/DG, optimized misc. code

Coverage increased (+0.09%) to 88.367% when pulling 028b72d361d999d270932f643868769dfec636cd on icehawker:master into a1dfea3f692cbdb674b69e7bae8a74b6bec89a68 on CS2103AUG2017-T15-B2:master.


Deploying framework-dependent .NET apps will fail if service doesn't have right version

When publishing to app service or functions, a user may be using an framework-dependent deployment option (FDD) (the default).  Upon publishing (especially with newer versions, go-live RCs, etc.) there is a possibility that the resulting service doesn't have FDD support yet.  While self-contained deployment (SCD) is an option it isn't the default in our flows.
Publishing from the extension should do a pre-check with all known information to see if the resulting deployment will succeed in a runtime start (sans user-delivered errors).

I new to FDD and SCD but read about them on https://learn.microsoft.com/en-us/dotnet/core/deploying/. Let me know if there are any other resources.

Publishing from the extension should do a pre-check with all known information to see if the resulting deployment will succeed in a runtime start (sans user-delivered errors).

Is there more information available so we can implement this pre-check? How can we know if a service supports FDD or SCD?

@alexweininger -- I don't think a service needs to know it supports SCD because the nature of SCD is it shouldn't care mostly.  But FDD requires that it will need to support whatever framework will need to be deployed.  The source can look at the source target framework version and the source (app service) should have ability to check to see if the target is available on that instance (probably need to work with App Service team to see what info is available to interrogate)


det-1130: add purge option to delete-test and delete-threat

I know I am late, but is purge used to tombstone a test/threat?

I know I am late, but is purge used to tombstone a test/threat?

purge is used to completely delete a test/threat. Tombstone marks it as deleted but doesn't actually delete anything. If you set purge=true, it will delete the row out of the database and delete the files off S3.


feat: implement Ord for Date

fix #8

I am closing this PR due to lack of test.
I implemented the date comparison myself in 43b1a2e (available in version 1.6.0)


Receive and send max_distance value.

Receive and send max_distance value.
Resolve #2767
Motivation and context

How has this been tested?

Checklist

[x] I submit my changes into the develop branch
[ ] I have added description of my changes into CHANGELOG file
[ ] I have updated the documentation accordingly
[ ] I have added tests to cover my changes
[ ] I have linked related issues (read github docs)
[ ] I have increased versions of npm packages if it is necessary (cvat-canvas,
cvat-core, cvat-data and cvat-ui)

License

[x] I submit my code changes under the same MIT License that covers the project.
Feel free to contact the maintainers if that's a concern.
[x] I have updated the license header for each file (see an example below)

# Copyright (C) 2021 Intel Corporation
#
# SPDX-License-Identifier: MIT

Coverage decreased (-0.04%) to 70.992% when pulling 3fa5766f99d838914d1f8aa5aab336e20a2ff8ed on DmitriyOparin:upstream/do/fixed_issue_2767 into 268abcbe51fdc6b699e9964fadfd8cb271667cbc on openvinotoolkit:develop.

@dvkruchinin, please review

@dvkruchinin, please review

LGTM


Django19

This adds support for django 1.9, as well as dropping some other deprecated features.
Closes #166.
Closes #164.
This changeset has backwards-incompatible changes. I suggest a pushing the major in next release after merging this.
Noticably  import path for models, managers, etc has been moved (both to stay in line with django standards, and to avoid crashes).

There are some issues with the admin (a redirection one with the test app, and another which I can only repro on a local project).
Feel free to poke around though.

Do we need the migration file?

I made a few adjustments here: https://github.com/jonashaag/django_polymorphic/commit/e09a59b69c73aaed36cfe225108a09cc3559cf35

Do we need the migration file?

Without it, we can't create a clean database to run the example file, so, yes. There's no harm in it being present for older versions of django anyway.

Oops, this breaks the build for Django < 1.9, I'll fix that tomorrow.

Should work now.

Added them to the PR, thanks. There's still an admin redirection issue (as seen in the example app), which I'll try to look at during the weekend.

I just added an example (via 4f0a9de) for a scenario that reproduces the last pending issue:

Create a ModelC instance.
Relate it to one or more ModelBs.
Open the change page in the admin.

Crash.
I can't quite wrap my head around this one, so I'll appreciate any help I can get.

I just added an example (via 4f0a9de) for a scenario that reproduces the last pending issue:

There doesn't seem to be a unit test covering this case. Is that correct? How did you discover the issue then?

There doesn't seem to be a unit test covering this case. Is that correct? How did you discover the issue then?

By attempting to port an existing app to django1.9. Using it for a while and triggered this issue pretty fast (since I have two models with basically that same relationship in it). I left a big TODO to add a unittest to cover that at some point for that exact reason.

Note: The migrations don't work on Django < 1.8.

I added a fix in my branch, please pull.

@jonashaag Thanks for that last fix. You clearly have a better understand of both django_polymorphic and django's QuerySet than I do. 😁
I'd consider this no longer WIP.

@jonashaag Thanks for that last fix. You clearly have a better understand of both django_polymorphic and django's QuerySet than I do. 😁

Not really; at best you can say I looked at the right places. I wondered why the super iterator returned integers on Django 1.9 (whereas it returns objects on Django < 1.9), so I compared the iterator implementations of Django 1.8 and 1.9 and found the behaviour change.

so I compared the iterator implementations of Django 1.8 and 1.9 and found the behaviour change.

I wonder why I didn't think of that.
Well, this is ready to merge IMHO. I'll include another PR improving test case running and adding some missing scenarios (I'd like to do quite a bit of cleanup there separately).

bump?

bump again? Any news?

Hello? Any feedback on this?

+1
keeps me from upgrading to 1.9 and pull request seems to work perfectly so far.

It's been about two months since the last activity by the author, so I've made a fork of this, with dj19 support.
Hopefully we can have that merged back in here at some point, but since I need an upstream package to depend on for a project of mine, I've no choice for now:
django-polymorphic-ng:

github
pypi

@hobarrera thanks for this!
@chrisglass ??

Thanks for the great work on this pull request! Sorry for not getting back to this earlier.
I've added some comments that need to be looked at. When that's done, I'm happy to merge it!

I've fixed everything mentioned in the comments (note that I've rewritten history to keep this PR tidy).

Thanks!
I've a few more changes on my fork; I'll send PRs for those ASAP and then get rid of it once we have a new release. :)

Thanks for your work on this!
I noticed a few issues with this change, perhaps you can look into those?

The support for fields starting with an underscore broke. so a query of translations___field (3 underscores), is recognized as translations class, whereas the old code also tried a join on translations._field as second option if that didn't work.
The removed imports from __init__.py break other packages, as from polymorphic.models import .. does not work with older versions. They have to implement an try.. import except flow now. Would it make sense to keep the imports in __init__.py for Django <= 1.8 ?

The support for fields starting with an underscore broke. so a query of translations___field (3 underscores), is recognized as translations class, whereas the old code also tried a join on translations._field as second option if that didn't work.

I'll try and have a look at that, I'm not too familiar with that feature (yet?).

The removed imports from init.py break other packages, as from polymorphic.models import .. does not work with older versions. They have to implement an try.. import except flow now. Would it make sense to keep the imports in init.py for Django < 1.9 ?

I don't really see the sense of keeping the imports, maybe documenting the API change for the latest release would be best. Note that this even fired a warning on django 1.8, since models were imported before app initialization, so even if it works, it's still not the recommended way.


Add Spawner form page

cc @zonca, @ellisonbg
Spawners can define an options_form class attribute. This is an HTML snippet, inserted into a form page. If defined, spawning a server loads this form instead of spawning directly.
The form data is passed to the Spawner.options_from_form classmethod, in order to turn the form structure (always a dict of lists of strings) into the intended shape (handling numbers, single values, defaults, etc.).
With this, Spawners can expose arbitrary parameters for users to override,
such as resources, hosts, etc.
Specifying user-configurable parameters is completely orthogonal to the admin-configured values from config=True traits, since there's ~no relationship between what is configrable in general and what should be exposed to the user.
Possible future ideas:

declarative form data instead of pasting raw HTML. There are roughly one million ways to turn a small spec into a webform. IPython widgets are one, but roughly none of the work there is reusable here. We could build a similar, simpler traits->form control, but we probably shouldn't. This would be a convenience, but it doesn't preclude using an arbitary form, so I think we can start with a fully arbitary form, and come up with a more convenient form-building spec at a later point.

I like this idea and think it is a good idea to keep it simple (as it is here) starting out.
I see why these form options have to be handled separately from our traitlets config stuff. I am fine with that but if you can clean it up that is also fine too.
I think for now, going with raw HTML makes the most sense - eventually if we have completely standardize UI controls/layout, we could explore the declarative form generation - but I wouldn't worry about it now.
Great work!

great, thanks, I'll try to test and provide feedback next week! @minrk do you have a gist of a simple working example of this? I see you have unit tests but I don't understand much the mocking machinery.

I moved Spawner initialization earlier, so that options_form can be a configurable, and accessed on the instance.
@zonca I added an example which allows specifying env and args for the single-user notebook server.

Thanks @minrk, works great:

Just a question, I noticed formdata.get for an input field returns a list like ["12:00:00"], is there a specific reason for that? it would be more convenient to just get a string back.

Just a question, I noticed formdata.get for an input field returns a list like ["12:00:00"], is there a specific reason for that? it would be more convenient to just get a string back.

Yes, since HTTP forms are an extremely primitive format where it's impossible to tell the difference between a single value and a list of length one. In order to be generally correct, tornado always returns a list of strings. This is precisely what the options_from_form method is for: to turn the HTTP form data into the right structure.


include Utils.Shell in Formula

[x] Have you followed the guidelines in our Contributing document?
[x] Have you checked to ensure there aren't other open Pull Requests for the same change?
[x] Have you added an explanation of what your changes do and why you'd like us to include them?
[ ] Have you written new tests for your changes? Here's an example.
[x] Have you successfully run brew tests with your changes locally?

This is a patch that makes the shell_profile method accessible from the Formula class again.

So I tested this locally and was able to install nvm with the two-line change in the PR. Adding a test is proving to be more difficult. Since shell_profile is part of the public API for a formula, I wanted to add a test to make sure that it's there.
I've tried everything I can think of to write a test for the presence of the shell_profile method.
Formula.respond_to? :shell_profile does returns false. Calling Formula.shell_profile and checking to make sure it returns a string throws an exception. I tried creating a Dummy class and inheriting from Formula, but apparently you can't have a class local to a method in Ruby.

apparently you can't have a class local to a method in Ruby.

You can make anonymous classes using Class.new:
klass = Class.new(Formula) do
  # ...
end

Okay I am completely lost on how to write a test for this.
With the patch above installation of nvm worked locally and I can refer to shell_profile in the body of a class inheriting from Formula.
I'm having trouble understanding exactly how including a module from a class works and "scaling down" the logic so I can understand it. I'm not able to reproduce what's happening inside Homebrew in a toy example.
I have a small toy example of a class include-ing a module local to another module in another file. If I use def shell_profile then shell_profile is accessible from the my_shell_profile method and does not need to be qualified with a namespace. However, if I define it as self.shell_profile then I cannot use the unqualified method and must explicit qualify it with its module.
# util.rb
module Wrapper
    module Utils
        # shell_profile (no error)
        # self.shell_profile (NameError)
        def shell_profile()
            return "4"
        end
    end
end

# test.rb
require_relative "util"

class Formula
    include Wrapper::Utils

    def initialize()
    end

    def my_shell_profile()
        shell_profile
        # must use Wrapper::Utils::shell_profile if
        # shell_profile was defined using "def self.shell_profile"
    end
end

a = Formula.new()

puts a.my_shell_profile()

Here is an excerpt from utils/shell.rb
 71     # return the shell profile file based on users' preferred shell
 72     def self.shell_profile
 73       SHELL_PROFILE_MAP.fetch(preferred_shell, "~/.bash_profile")
 74     end

I'm going to go ahead and merge this as-is to stem the bleeding.

sounds good.

Don't merge this! It didn't actually fix the issue. It only appeared to because brew executes git checkout master in the course of installing a module.

Okay I think I got it now. Formulas relying on shell_profile should work without modification.
I added a test making sure that shell_profile is in scope inside the body of a method on a class inheriting from Formula.


Add Accessibility Features

This includes features like Changing font size and adding Screen Reader

@Yashshukla11 i would like to contribute to this issue... kindly assign me

@Yashshukla11 please assign this to me i am GSSOC'24 contributor and have 2+ years of experience in flutter


Add command purview retentioneventtype add

Closes #4341

Well done, we'll try to review it ASAP!

@MathijsVerbeeck, could you please fix the merge conflicts on this PR? Also I'm seeing an odd commit about eslint and eventreceiver, which should not be part of this PR, I'm thinking.

Resolving them as we speak. The eslint file is needed however, due to mee adding event to the eslintrc.js file, the eventreceiver commands failed due to naming policies (as it splitted event and receiver in two words). This is why this is necessary.

Thanks for the feedback @martinlingstuyl . The issues should have been resolved.

Ok, the code looks OK. I just tried to execute it and I get 401's. Being a Global admin and having the correct permissions. Also: running on Graph Explorer works fine. Could be a fluke, Adam had the same on another endpoint.
Does it work OK for you?

Ok, the code looks OK. I just tried to execute it and I get 401's. Being a Global admin and having the correct permissions. Also: running on Graph Explorer works fine. Could be a fluke, Adam had the same on another endpoint.
Does it work OK for you?

It did work for me, but I'm currently getting the following error running any of the purview commands:

Running it with --debug shows that I get the 401 Unauthorized response, however when decoding my access token, it shows that it's valid and it has the correct scope RecordsManagement.ReadWrite.All. Weird...

Yeah, same here. It's a beta endpoint. Must be a glitch on the MS side.

Yeah, same here. It's a beta endpoint. Must be a glitch on the MS side.

It's very strange, as I've copied the exact API call from Graph Explorer into postman and just changed the Bearer token and I still get the exact same error. I've also compared the bearer tokens in jwt.ms and there doesn't seem anything strange going on there.

I found out that, I can get it to work if I get an accesstoken using m365 util accesstoken get --resource '00000003-0000-0000-c000-000000000000' --new and then use this accesstoken in Postman. The audience of the accesstoken that we currently retrieve is 'https://graph.microsoft.com'. When decoding the accesstoken from the graph explorer. The audience also shows as 00000003-0000-0000-c000-000000000000. Maybe it does have something to do with that?

That truly is odd. But you may just leave it alone. It's a MS issue I believe. It will probably start working again soon 😁


Bump rustc nightly-2022-09-20

Pull Request Description
Bump rustc nightly-2022-08-30 and fix new errors and lints.
Important Notes

removes DEPRECATED_Tween from the animation framework, which failed to compile with a newer version of rustc. All uses were replaced with the newer Easing instead.
Removed some potential UB indicated by Clippy in lib/rust/data-structures/src/diet.rs @mwu-tow, please double-check this.

Checklist
Please include the following checklist in your PR:

[x] The documentation has been updated if necessary.
[x] All code conforms to the
Scala,
Java,
and
Rust
style guides.
All code has been tested:

[x] Unit tests have been written where possible.
[ ] If GUI codebase was changed: Enso GUI was tested when built using BOTH
./run ide build and ./run ide watch.

Superseded by #3743.


Process invalid UTF-8 without crashing

On encountering invalid UTF8 values Text.decodeUtf8 will throw an exception that can't be caught from pure code. Result doesn't have the right type to make a MonadCatch, so by process of elimination we're using Text.decodeUtf8' to get an Either UnicodeException Text we can turn into a Result.

Thanks, good catch!


Example input file out of date

The example input file hasn’t been updated when the support for attachment positioning was being implemented and still bears a statement that the contextual kerning subtable is the only type currently available.
Moreover, there’s a comment saying that a “list of values is applied to the kern stack by popping glyphs from the stack one by one…” This is unfortunate wording as the stack isn’t actually being popped: kern values are applied in order of popping the stack, but the stack itself stays untouched and can only be pushed down with new glyphs coming in.

Updated, see 59d7dd6c777dffe399c47554913852aa3bbcbb6b


Problem with version 2.6 with Anaconda 3 21.05 on Windows

Hi,
I did install it with
pip install pyriemann
But it gives me this error:
ImportError: cannot import name '_init_centroids' from 'sklearn.cluster._kmeans
So one needs to fix this or download the code for pyriemann 2.7dev and replace it in Anaconda.
pip install pyriemann from github does not work good because it can mess up with versions of the existing libraries.
So I think you need to release version 2.7 to the public.

I push for testing in https://test.pypi.org/project/pyriemann/
can you try if it works for you?
if so I'll push one release to pypi

You did not change the version. It still says 2.6 and thus it does not want to install. Please update.

did you try to run
pip install -i https://test.pypi.org/simple/ pyriemann
?

yes, it says already installed.
But here https://test.pypi.org/project/pyriemann/ it says pypi package 2,6

don't look at the badge
can you do:
pip install -U -i https://test.pypi.org/simple/ pyriemann

How it gives another thing:
ERROR: Command errored out with exit status 1:
     command: 'C:\Anaconda\python.exe' -c 'import sys, setuptools, tokenize; sys.argv[0] = '"'"\\Local\\Temp\\pip-install-6dp0u7ei\\pyriemann_2679cd1d615b4c51adc63480f01d4229\\setup.py'"'"'; __file__='"'"'C:\\Users\\AppData\\Local\\Temp\\pip-install-6dp0u7ei\\pyriemann_2679cd1d615b4c51adc63480f01d4229\\setup.py'"'"';f=getattr(tokenize, '"'"'open'"'"', open)(__file__);code=f.read().replace('"'"'\r\n'"'"', '"'"'\n'"'"');f.close();exec(compile(code, __file__, '"'"'exec'"'"'))' egg_info --egg-base 'C:\Users\AppData\Local\Temp\pip-pip-egg-info-56tbqbub'
         cwd: C:\Users\AppData\Local\Temp\pip-install-6dp0u7ei\pyriemann_2679cd1d615b4c51adc63480f01d4229\
    Complete output (7 lines):
    Traceback (most recent call last):
      File "<string>", line 1, in <module>
      File "C:\Users\AppData\Local\Temp\pip-install-6dp0u7ei\pyriemann_2679cd1d615b4c51adc63480f01d4229\setup.py", line 17, in <module>
        long_description = fid.read()
      File "C:\Anaconda\lib\encodings\cp1252.py", line 23, in decode
        return codecs.charmap_decode(input,self.errors,decoding_table)[0]
    UnicodeDecodeError: 'charmap' codec can't decode byte 0x9d in position 4325: character maps to <undefined>
    ----------------------------------------
WARNING: Discarding https://test-files.pythonhosted.org/packages/50/11/5f530fe7d865d96a9939ad55c6cb4710ed23955395bcdbc877b6ef7024e3/pyriemann-0.2.7rc0.tar.gz#sha256=d8150b99c7f419ebbdc846a3e8e86426d9b197a5fc20a84a041345d66a5e4f11 (from https://test.pypi.org/simple/pyriemann/). Command errored out with exit status 1: python setup.py egg_info Check the logs for full command output.
Requirement already satisfied: scikit-learn in c:\andreevi\anaconda\lib\site-packages (from pyriemann) (0.24.1)
Requirement already satisfied: pandas in c:\andreevi\anaconda\lib\site-packages (from pyriemann) (1.2.4)
Requirement already satisfied: scipy in c:\andreevi\anaconda\lib\site-packages (from pyriemann) (1.6.2)
Requirement already satisfied: numpy in c:\andreevi\anaconda\lib\site-packages (from pyriemann) (1.20.1)
Requirement already satisfied: joblib in c:\andreevi\anaconda\lib\site-packages (from pyriemann) (1.0.1)
Requirement already satisfied: pytz>=2017.3 in c:\andreevi\anaconda\lib\site-packages (from pandas->pyriemann) (2021.1)
Requirement already satisfied: python-dateutil>=2.7.3 in c:\anaconda\lib\site-packages (from pandas->pyriemann) (2.8.1)
Requirement already satisfied: six>=1.5 in c:\anaconda\lib\site-packages (from python-dateutil>=2.7.3->pandas->pyriemann) (1.15.0)
Requirement already satisfied: threadpoolctl>=2.0.0 in c:\anaconda\lib\site-packages (from scikit-learn->pyriemann) (2.1.0)

can you try again now?

Yes. it works.

you're welcome ...


Please add a license

No license is mentioned in the README or included in the repository, which makes the project fully copyright by its authors by default.

I agree. @mojodna ^^^

Does a 3-clause BSD license work for you all?

Works for me


subctl should detect OLM if installed and use that instead of bare operator

This is still relevant.

This is still relevant.

bump


Jira extension - Add ability to paste pictures to markdown description

Extension – Jira

Author: @raycastbot @imthath-m

Describe the feature and the current behavior/state.
Add ability to paste pictures to markdown description input. It should work like github issue description input.
Who will benefit from this feature?
Agile teams.

I would be willing to collaborate on developing this functionality.


AddCounterSigners bug fix in SignerInformation

The old method was functionnal for only one counter signer. It added only one CounterSigner attribute for all counterSigners present in the input store. It adds one for each in this version.

Verification code for counter signatures is now recursive.


Overhaul csv loading

Add DataReader which will read each csv file row by row.
Add DataWriter which will maintain and write rows to dirty and clean csv files from the cleaned source.
Add a Cleaner which will clean the csv files based on known type.
Began to create a Postgres utility file which will handle the interaction with postgres.

Merging as-is. I will delete the /mockups/d3 stuff post-merge, and will leave the /scripts folder in place until I hear back from @blw9u2012 re: 'correct' version of the /scripts/demo_load....  vs. /python/demo_load.... files.


ExtensionsParser

When I want to create more than one instance of app. So I need include App.h in more than one xxx.cpp,
But that cause ExtensionsParser::ExtensionsParser(...) redefined, and I find class ExtensionsParser's member function declare and definition are both in WebSocketExtensions.h file. Does that because of new branch was designed for head only library, so do like this ?

This should be fixed now


added twoSum problem

The problem has been added to Array subfolder and question has been added to readme.md of the same folder.

Also added remove duplicates from sorted array problem


Improve configuration for search paths

Parts of this question was already answered several times, so be patient when reading. :)
Our usual workspace structure is:
rootDir/:
    - project.vscode-workspace
    - sources/:
        - main.py
        - someSrcFile.py
        - ...
        - common/:
            - commFileOne.py
            - ...

Nothing unusual here.
Than, we have another folder with common python modules in remote location - this location is added by modules (inserting to sys.path). I will call this location remoteDir.
<some remote location>
remoteDir/:
    - remoteModule.py
    - ...

When running main.py from VS Code, we specify launch configuration with cwd changed to rootDir/sources. It runs OK, files from common and remoteDir are accessible (since sys.path is manually modified).
To implement autocomplete (either Intellisense Python Language Server or JEDI) and Pylint, this directories are not seen by default - .vscode-workspace file is not in sources folder. I am aware of PYTHONPATH variable in .env file - I added sources and remoteDir to PYTHONPATH variable and VS Code autocomplete and pylint works as expected.
The problem is that this PYTHONPATH variable affects the code and is bound to this IDE, which I don't like - one could easily develop code with VS Code (which has specified PYTHONPATH) and forget to handle sys.path, not knowing that the code now can't be executed without this IDE and it's .env settings.
Example: A newbie come to our department, take prepared VS Code workspace and create a new module in common folder, which must be executed also in standalone mode. In this module, he wants to use functions from remoteDir. Since PYTHONPATH is already set in .env, he doesn't know that he actually have to manually handle sys.path - autocomplete and pylint just works out of the box. Also, using VS Code, he can run main.py or his new script without problems. But, if he (or someone else) run this script from a command line: cd ./sources/common; python hisCommonFile.py, this will fail. Why - because nothing specify PYTHONPATH variable, and he doesn't have sys.path handled properly.
The question is: how to properly configure folder paths for autocomplete and pylint in VS Code, that wouldn't have any effect on how code is executed? I don't expect autocomplete and linter would interactively resolve modified sys.path - seems like there should be something similar to PYTHONPATH, but would be only applied to autocomplete/pylint VS Code tasks, not launch configurations.
Just for a clarification: I know this might be a silly example. Our case is a little more complex than described above, but the problem is somewhat simillar - I miss includeFolders=[] in both autocomplete and pylint settings section, or better, some common settings. autocomplete.extraPaths only solve problems for autocomplete (if it is working, last time I was checking It didn't work properly), not pylint - again, duplicated settings.

Thank you for the suggestion! We have marked this issue as "needs decision" to make sure we have a conversation about your idea. We plan to leave this feature request open for at least a month to see how many 👍 votes your idea gets to help us make our decision.

@karthiknadig  correct me if I'm wrong but I believe this may be possible now with the following extensions and settings:

Pylance extension

Using "python.analysis.include"

Pylint extension

Using pylint.args and passing [--init='pythonpath=<your_extra_path>']

Pylint extension automatically adds paths from python.analysis.extraPaths to PYTHONPATH when running pylint.

Awesome, thanks!
I think this can be closed now but let us know if you still have any issues and we can reopen :)


Assertion Error: world_size % group_num == 0 When finetune cogv1.5-i2v on distributed environment

System Info / 系統信息

Information / 问题信息

[ ] The official example scripts / 官方的示例脚本
[X] My own modified scripts / 我自己修改的脚本和任务

Reproduction / 复现过程

run code on 2*8 GPU
find assertion error world_size % group_num != 0

Expected behavior / 期待表现
I found group_num in loss_fn_config in yaml is 40.
Should the group_num be modified according to the machine amount I am using?

we have not conducted experiments on multi-machine multi-card inference, all are single-machine. I'm sorry we couldn't solve the problem.


BUG-398: add fastly to the cli

This will make it available to veritech for execution

bors merge

bors retry

bors merge

bors merge

bors merge


Upload profile image in local register

I'd love to get an example of a library that will allow it with next.js

It's an interesting use case. For now I'm using a gravatar based solution. See https://sorters.club/users which uses staart. The source code is here: https://github.com/nmaro/sorters


Chrome 131: "This extension was turned off because it's no longer supported"

Per the below screenshot, Chrome automatically disables the KeeWeb extension now due to not supporting latest Chrome standards (<-- this is the link in the screenshot):

I imagine this might be related to Manifest v3 or something similar, but didn't look into it in-depth at this time.
My specific Chrome version for reference:
Version 131.0.6778.205 (Official Build) (x86_64) on macOS Sonoma 14.5

Likely related to https://github.com/keeweb/keeweb/issues/2188 in the main repo; not sure if the Manifest v3 changes got a release?
Also the extension's page (using the direct link from the README) displays this:

not sure if the Manifest v3 changes got a release?

Seems like no as far as I can tell; 0.3.7 from Aug 2021 is still the latest GitHub release and matches my extension's version


Additions

Issue #8343

Docs Build status updates of commit c331d36:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

exchange/exchange-ps/exchange/New-MobileDeviceMailboxPolicy.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/Set-MobileDeviceMailboxPolicy.md
:white_check_mark:Succeeded
View (exchange-ps)

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


[Google Translate] ...

Extension
https://www.raycast.com/gebeto/translate
Description

Error:

Steps To Reproduce
No response
Current Behaviour
No response
Expected Behaviour
No response

https://github.com/raycast/extensions/pull/6666

This issue can be closed


[Feature Request] Download file as a favorite/exceptional file

I'd like to request a feature. I'd like a separate download button(like "Download Now as Favorite") for images that either downloads them to a separate directory, or adds a configurable prefix to the file.
As things stand now, when I download a file, if it's a top tier image, I rename the file and add "__" as a prefix. I'm just looking to automate that a bit. I am not worried at all about the website's built in favorites feature.
Thanks.

I don't think this is a feature that the extension is going to accept at this time.