extra_args=['--latex-engine=xelatex', '-V', 'mainfont="Font Name"'] doesn't work

My conversion class looks like this:
class PandocPDFConverter(object):
def generate_output(self, docx_file, **kwargs):
extra_args = [
'--latex-engine=xelatex',
'-V',
'mainfont="Noto Serif"'
]
output = pypandoc.convert(docx_file,
'pdf',
outputfile='test.pdf',
extra_args=extra_args)
... and it doesn't output PDF file. However, when the same arguments are passed during conversion from command line, everything works fine.
Error message:
RuntimeError: Pandoc died with exitcode "43" during conversation: ! LaTeX Error: Missing \begin{document}.
See the LaTeX manual or LaTeX Companion for explanation.
Type  H   for immediate help.
...
l.18     \setmainfont{"Noto Serif"}
pandoc: Error producing PDF from TeX source
But when when I change 'mainfont="Noto Serif"' with 'mainfont="[NotoSerif-Regular.ttf]"' conversion goes well.
P.S.
IMHO, you meant to write "conversion", not "conversation" (pypandoc.py, line No 214).

But when when I change 'mainfont="Noto Serif"' with 'mainfont="[NotoSerif-Regular.ttf]"' conversion goes well.

Would you have time to debug this? I wonder what's up with that.

IMHO, you meant to write "conversion", not "conversation" (pypandoc.py, line No 214).

Fixed. Thanks for pointing this out.

Would you have time to debug this? I wonder what's up with that.
Cannot promise to investigate it in a week, but I'll definitely debug it in the following month.

Cannot promise to investigate it in a week, but I'll definitely debug it in the following month.

Ok. Cool. :+1:

@jumasheff Any luck?

@juho I am a bit busy, sorry :(

Best regards / Урматым менен / С уважением,
Murat Jumashev / Мурат Жумашев
2015-09-17 13:21 GMT+06:00 Juho Vepsäläinen<EMAIL_ADDRESS>
@jumasheff https://github.com/jumasheff Any luck?
—
Reply to this email directly or view it on GitHub
https://github.com/bebraw/pypandoc/issues/61#issuecomment-140990564.

@jumasheff No worries. Thanks for getting back. :+1:


Implement $arrayToObject operator

Hi!
I've implemented the arrayToObject operator for the aggregation pipeline.
I've added tests for all the edge cases I could think of:

Mixed array and document inside the same array
Extra keys in documents
Missing keys in document
Empty document
More than two elements in an array
Just one element in an array
Empty array

Duplicated keys are also accounted for and tested (latest found value is used, overwriting previous values), but there is a caveat in the mongo documentation for earlier versions. I'm not sure if this should be accounted for or mocking the behaviour for the latest versions is enough.

I'll add some tests for those missing lines for coverage

It looks like $arrayToObject: null is valid too, and returns another null. I've added this edge case too.

Thanks for the thorough review and your suggestions. I've implemented your requests and squashed my commits together. In addition to your requests I've made the following changes:

Implementation was further refactored

Used dictionary comprehension to convert {k, v} values to the expected objects
Dropped the two defs
Dropped the any and replaced by plain if/elses

I've replaced the TypeErrors with PyMongo's OperationFailure (so that compare_exceptions worked)
Added a bunch of tests to check that tuples work just as well as lists

While checking PyMongo I've noticed that following exception messages can be thrown:
$arrayToObject requires an array of size 2 arrays,found array of size: 3
$arrayToObject requires an object keys of 'k' and 'v'. Found incorrect number of keys:3
$arrayToObject requires an object with keys 'k' and 'v'. Missing either or both keys from: {v: 1, t: "t"}
$arrayToObject requires a consistent input format. Elements mustall be arrays or all be objects. Array was detected, now found: object

All of the messages can be seen in the mongo repo. My implementation doesn't check the reason for an OperationFailure in most cases and uses a generic error message. If you want I can change the code so that the same messages are used, but I wanted to check with you before adding more complexity to the code.

Python 2.7 should be fixed now. .keys() returns a list in Python 2.7 which can't be directly compared against a set (unlike in python 3).


Patch Dia

http://sourceforge.net/p/dia-installer/support-requests/98/

That page is a 404. What does this fix?

The app doesn't start on 10.10+ unless this patch is made.

Thank you for the contribution. It needed some fixes, so they were made in 744ff9286621f1189518d72eef2dd31d0d5d8e1d. Your contribution is still included (and still credited to you), with the appropriate modifications. Please feel free to ask about any of the changes.

I am on mobile so I couldn't amend. Thanks for doing that for me!

No problem, thank you for the continued contributions.

One more question: This patch is pretty risky and I'd rather include a diff-style patch and have postflight apply it. I wasn't sure how to do that without committing a new file. Are there any precedents for this?

Are there any precedents for this?

I think not.


MvM australium current drystreak

Title

0.9.5 added this per-tour but adding an overall stat would require a full re-write of how tour data is parsed. Potentially tackle this later.


simplify sums

Need more initial terms when this gets enabled:
./loda export A078013
a(n) = -a(n-4)-a(n-5), a(2) = 0, a(1) = 0, a(0) = 1


libwebsockets: 2.4.1 -> 2.4.2

Semi-automatic update generated by https://github.com/ryantm/nix-update tools. These checks were done:

built on NixOS
Warning: no binary found that responded to help or version flags. (This warning appears even if the package isn't expected to have binaries.)
found 2.4.2 with grep in /nix/store/y3zmgw1wy80xwfmjx8d97nvrr5qfzrmp-libwebsockets-2.4.2
found 2.4.2 in filename of file in /nix/store/y3zmgw1wy80xwfmjx8d97nvrr5qfzrmp-libwebsockets-2.4.2

Success on aarch64-linux (full log)
Attempted: libwebsockets
Partial log (click to expand)
shrinking /nix/store/6nm01ygxkl4bg43aavnyicabwg73xsi0-libwebsockets-2.4.2/bin/libwebsockets-test-server-extpoll
shrinking /nix/store/6nm01ygxkl4bg43aavnyicabwg73xsi0-libwebsockets-2.4.2/bin/libwebsockets-test-server-libuv
shrinking /nix/store/6nm01ygxkl4bg43aavnyicabwg73xsi0-libwebsockets-2.4.2/bin/libwebsockets-test-fuzxy
shrinking /nix/store/6nm01ygxkl4bg43aavnyicabwg73xsi0-libwebsockets-2.4.2/bin/libwebsockets-test-server
shrinking /nix/store/6nm01ygxkl4bg43aavnyicabwg73xsi0-libwebsockets-2.4.2/lib/libwebsockets.so.12
strip is /nix/store/lvx1acn1ig1j2km8jds5x3ggh3f2wa8v-binutils-2.28.1/bin/strip
stripping (with command strip and flags -S) in /nix/store/6nm01ygxkl4bg43aavnyicabwg73xsi0-libwebsockets-2.4.2/lib  /nix/store/6nm01ygxkl4bg43aavnyicabwg73xsi0-libwebsockets-2.4.2/bin
patching script interpreter paths in /nix/store/6nm01ygxkl4bg43aavnyicabwg73xsi0-libwebsockets-2.4.2
checking for references to /build in /nix/store/6nm01ygxkl4bg43aavnyicabwg73xsi0-libwebsockets-2.4.2...
/nix/store/6nm01ygxkl4bg43aavnyicabwg73xsi0-libwebsockets-2.4.2

Success on x86_64-darwin (full log)
Attempted: libwebsockets
Partial log (click to expand)
-- Installing: /nix/store/6c8yz1xbmxm39srww38zfdv0hqxj2lr4-libwebsockets-2.4.2/share/libwebsockets-test-server/generic-table/index.html
-- Installing: /nix/store/6c8yz1xbmxm39srww38zfdv0hqxj2lr4-libwebsockets-2.4.2/lib/cmake/libwebsockets/LibwebsocketsConfig.cmake
-- Installing: /nix/store/6c8yz1xbmxm39srww38zfdv0hqxj2lr4-libwebsockets-2.4.2/lib/cmake/libwebsockets/LibwebsocketsConfigVersion.cmake
-- Installing: /nix/store/6c8yz1xbmxm39srww38zfdv0hqxj2lr4-libwebsockets-2.4.2/lib/cmake/libwebsockets/LibwebsocketsTargets.cmake
-- Installing: /nix/store/6c8yz1xbmxm39srww38zfdv0hqxj2lr4-libwebsockets-2.4.2/lib/cmake/libwebsockets/LibwebsocketsTargets-release.cmake
post-installation fixup
strip is /nix/store/4sdh09gmvl15cy0zb6i7mbvxh5syz206-cctools-binutils-darwin/bin/strip
stripping (with command strip and flags -S) in /nix/store/6c8yz1xbmxm39srww38zfdv0hqxj2lr4-libwebsockets-2.4.2/lib  /nix/store/6c8yz1xbmxm39srww38zfdv0hqxj2lr4-libwebsockets-2.4.2/bin
patching script interpreter paths in /nix/store/6c8yz1xbmxm39srww38zfdv0hqxj2lr4-libwebsockets-2.4.2
/nix/store/6c8yz1xbmxm39srww38zfdv0hqxj2lr4-libwebsockets-2.4.2

Success on x86_64-linux (full log)
Attempted: libwebsockets
Partial log (click to expand)
shrinking /nix/store/y3zmgw1wy80xwfmjx8d97nvrr5qfzrmp-libwebsockets-2.4.2/bin/libwebsockets-test-server
shrinking /nix/store/y3zmgw1wy80xwfmjx8d97nvrr5qfzrmp-libwebsockets-2.4.2/bin/libwebsockets-test-server-libuv
shrinking /nix/store/y3zmgw1wy80xwfmjx8d97nvrr5qfzrmp-libwebsockets-2.4.2/bin/libwebsockets-test-client
shrinking /nix/store/y3zmgw1wy80xwfmjx8d97nvrr5qfzrmp-libwebsockets-2.4.2/bin/libwebsockets-test-fuzxy
shrinking /nix/store/y3zmgw1wy80xwfmjx8d97nvrr5qfzrmp-libwebsockets-2.4.2/lib/libwebsockets.so.12
strip is /nix/store/b0zlxla7dmy1iwc3g459rjznx59797xy-binutils-2.28.1/bin/strip
stripping (with command strip and flags -S) in /nix/store/y3zmgw1wy80xwfmjx8d97nvrr5qfzrmp-libwebsockets-2.4.2/lib  /nix/store/y3zmgw1wy80xwfmjx8d97nvrr5qfzrmp-libwebsockets-2.4.2/bin
patching script interpreter paths in /nix/store/y3zmgw1wy80xwfmjx8d97nvrr5qfzrmp-libwebsockets-2.4.2
checking for references to /tmp/nix-build-libwebsockets-2.4.2.drv-0 in /nix/store/y3zmgw1wy80xwfmjx8d97nvrr5qfzrmp-libwebsockets-2.4.2...
/nix/store/y3zmgw1wy80xwfmjx8d97nvrr5qfzrmp-libwebsockets-2.4.2


removing deprecated checks

removing deprecated jshint checks
@RackHD/veyron

BUILD on-dhcp-proxy #2 : FAILURE

test this please

BUILD on-dhcp-proxy #3 : FAILURE

test this please


Changing the window title

When changing the window title (function defos.set_window_title) using Cyrillic, as a result, we get incorrect characters.
Sample project: title_window_issue.zip
Screenshot:

An example fix in the Defold engine: link.

Strange, I would have thought that CA2W which is used in DefOS would work out of the box:
https://github.com/subsoap/defos/blob/master/defos/src/defos_win.cpp#L251
What you can try is like you suggested and instead use MultiByteToWideChar, like we do in Defold.

As discussed on #127 and in Discord there is still a problem somehow so reopening.

I'll add a message from Discord:
If you do not pay attention to character transformations, then the task of the value in the window title occurs with errors:
    SetWindowTextW(dmGraphics::GetNativeWindowsHWND(), (LPCWSTR)L"我使用 - Defos - Привет, мир!");
    MessageBoxW(
        dmGraphics::GetNativeWindowsHWND(),
        (LPCWSTR)L"我使用 - Defos - Привет, мир!",
        (LPCWSTR)L"出去 - Out - Вывод",
        MB_ICONINFORMATION | MB_OK | MB_DEFBUTTON1
    );


Video Encode Stall with Driver 23.20.0016.4973

Hello @dmitryermilov,
We have a user with Lenovo Legion Y520 and driver version 23.20.0016.4973 dated 2018-02-27. We see a possible encoder stall from the log files. There are more than six seconds between IDR frames 13 and 14 (WebRTC times out). When frame 15 is sent to encoder, MFXVideoENCODE_EncodeFrameAsync() return MFX_WRN_DEVICE_BUSY.
[042:173] [11224] (rust_log.cpp:37): INFO [intel_media_encoding::encoder] Creating IntelVideoEncoder, size: 640 x 360 @ 15 fps, bitrate: 576000 [bps], format: P8(Iyuv)
[042:281] [11224] (rust_log.cpp:37): INFO [intel_media_encoding::encoder] IDR Frame: 1
[043:710] [11224] (rust_log.cpp:37): INFO [intel_media_encoding::encoder] IDR Frame: 13
[046:360] [17160] (video_send_stream_impl.cc:444): SignalEncoderTimedOut, Encoder timed out.
[049:840] [11224] (rust_log.cpp:37): INFO [intel_media_encoding::encoder] IDR Frame: 14
[049:840] [11224] (rust_log.cpp:37): WARN [intel_media_encoding::encoder] Encoder warning: 2

I will add a little bit more logging, do you have any suggestion?

We did not see this issue, it could be a one-off. I am closing the issue for now.


Fix validate_formats to accept Path object for pdf

Fixes #337

The failing test is unrelated, so your change is fine


Add the ability to specify a MessagingServiceSid.

Just adding a new parameter.

Any chance we can get this in? We're forked until this is landed.

Sure


Logging for failing IW tests

For flaky failures found in this PR https://github.com/microsoft/vscode-jupyter/pull/12036#issuecomment-1315818783

Codecov Report

Merging #12038 (bd17fbd) into main (c29f780) will increase coverage by 0%.
The diff coverage is 66%.

@@          Coverage Diff           @@
##            main   #12038   +/-   ##
======================================
  Coverage     62%      62%
======================================
  Files        502      502
  Lines      35910    35918    +8
  Branches    5811     5813    +2
======================================
+ Hits       22521    22596   +75
+ Misses     11230    11182   -48
+ Partials    2159     2140   -19

Impacted Files
Coverage Δ

src/interactive-window/interactiveWindow.ts
74% <66%> (-1%)
:arrow_down:

.../editor-integration/generatedCodeStorageFactory.ts
94% <0%> (-6%)
:arrow_down:

...otebooks/controllers/controllerPreferredService.ts
69% <0%> (-5%)
:arrow_down:

src/kernels/execution/cellExecutionQueue.ts
77% <0%> (-4%)
:arrow_down:

src/kernels/kernelDependencyService.node.ts
77% <0%> (-3%)
:arrow_down:

src/kernels/execution/cellExecution.ts
79% <0%> (-1%)
:arrow_down:

src/platform/common/utils/localize.ts
75% <0%> (+<1%)
:arrow_up:

src/kernels/execution/helpers.ts
62% <0%> (+<1%)
:arrow_up:

src/kernels/raw/launcher/kernelProcess.node.ts
73% <0%> (+<1%)
:arrow_up:

... and 16 more


Bug: special: incorrect return value from poch

>>> import scipy.special as sc
>>> a, m = -2.857142857142857, -7.142857142857143
>>> a + m
-10.0
>>> a + m == int(a + m)
True
>>> sc.poch(a, m)
431007055.05169541

Correct value should be inf.
Scipy/Numpy/Python version information:
>>> import sys, scipy, numpy; print(scipy.__version__, numpy.__version__, sys.version_info)
1.1.0.dev0+b6cd2d7 1.13.1 sys.version_info(major=3, minor=6, micro=2, releaselevel='final', serial=0)

Also since we have the Lanczos approximation in Cephes now we can use it to simplify the implementation of poch.

Hm, looking back at this again I was wrong; it is true that fl(a + m) == -10, but it is not true that a + m == -10, so if we compute without the double-precision rounding:
>>> import mpmath
>>> a = mpmath.mpf('-2.857142857142857')
>>> m = mpmath.mpf('-7.142857142857143')
>>> with mpmath.workdps(100):
...     mpmath.nprint(mpmath.rf(a, m), 50)
...
431007055.05169307547609444259038490813734120560371

Shouldn't have doubted Pauli!


asdf

Pull requests are welcome, but a system would need to be in place for multiple translations and for future contributions.


issue using plot_lcppn_explainer.ipynb sample script

Describe the bug
I have downloaded the plot_lcppn_explainer.ipynb notebook and am trying to run it locally. I am able to train and predict the classifier, but I get an error when defining the explainer.
To Reproduce
Steps to reproduce the behavior:

Go to https://hiclass.readthedocs.io/en/latest/auto_examples/plot_lcppn_explainer.html and download thw plot_lcppn_explainer.ipynb notebook.
Open the notebook and try to run the code. I split up the main cell and found the bug at the #Define Explainer portion of the script.
See error

Expected behavior
I expected to see the example plot that is shown on the download page.
Screenshots

Desktop (please complete the following information):

OS: MacOS 15.0, M1 Pro
Python 3.11.9

Additional context
Add any other context about the problem here.

hi i am facing same issue on same code above, my env is python 3.10.15 ,shap-0.46.0. please let me know if there's any fix to this.

Hi @colton-vp and @RishitJoshi97
Can you please try running the notebook with shap v 0.44.1?


Option to use Typer.Strategy.BuiltinsOnly to workaround bug in Skunk's oid codec?

When I tried out Dumbo for the first time (on a cockroachDB database that has been working with Skunk+Flyway for a while), I hit an error. Excess stack traces are boring so I trimmed for relevance.
skunk.exception.DecodeException:
🔥
🔥  DecodeException
🔥
🔥    Problem: Decoding error.
🔥     Detail: This query's decoder was unable to decode a row of data.
🔥
🔥  The statement under consideration was defined
🔥    at «skunk internal»:0
🔥
🔥    SELECT oid typid, typname, typarray, typrelid
🔥    FROM   pg_type
🔥    WHERE typnamespace IN (
🔥      SELECT oid
🔥      FROM   pg_namespace
🔥      WHERE nspname = ANY(current_schemas(true))
🔥    )
🔥
🔥  The row in question returned the following values (truncated to 15 chars).
🔥
🔥    typid     oid   -><PHONE_NUMBER>    ├── java.lang.NumberFormatException (see below)
🔥    typname   name  ->  pg_aggregate
🔥    typarray  oid   ->  0
🔥    typrelid  oid   -><PHONE_NUMBER>
🔥
🔥  The decoder threw the following exception:
🔥
🔥    java.lang.NumberFormatException: For input string: "4294967108"
🔥      java.base/java.lang.NumberFormatException.forInputString(NumberFormatException.java:67)
🔥      java.base/java.lang.Integer.parseInt(Integer.java:668)
🔥      java.base/java.lang.Integer.parseInt(Integer.java:786)
🔥      scala.collection.StringOps$.toInt$extension(StringOps.scala:910)
🔥      skunk.util.Typer$ProtocolOps.$init$$$anonfun$4(Typer.scala:180)
🔥      skunk.Codec$.simple$$anonfun$2(Codec.scala:113)
...

Root cause is a bug in Skunk's oid Codec, which tries to fit an unsigned Int into an Int 😱
This potentially has a lot of surface area in Skunk's Typer and may not be a straightforward fix.
As a possibly easier workaround, would you consider some config option to instead use the simpler Typer.Strategy.BuiltinsOnly in Dumbo, which IIUC won't need to hit the problematic code?

hi @benhutchison, thanks for reporting. I haven't tested it with cockroachDB before. After I just tried there seem to be other issues / incompatibilities, like with history table types and locking seem to work differently in cockroachDB. May I know what cockroachDB version you were running it with? I guess different versions may behave differently.
Making the Typer.Strategy configurable or changing the default to BuiltinsOnly should be simple, but I think there are more things that would need changing. I'd need to run some more experiments.

Im using CockroachDB cloud, v23.2.4
On Sun, May 5, 2024 at 12:39 AM Roman Langolf @.***>
wrote:

hi @benhutchison https://github.com/benhutchison, thanks for reporting.
I haven't tested it with cockroachDB before. After I just tried there seem
to be other issues / incompatibilities, like with history table types and
locking seem to work differently in cockroachDB. May I know what
cockroachDB version you were running it with? I guess different versions
may behave differently.
Making the Typer.Strategy configurable or changing the default to
BuiltinsOnly should be simple, but I think there are more things that
would need changing. I'd need to run some more experiments.
—
Reply to this email directly, view it on GitHub
https://github.com/rolang/dumbo/issues/42#issuecomment-2094224464, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAAXJZBUYIYEIUT64FCDG2LZATXJ7AVCNFSM6AAAAABHFODMOOVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDAOJUGIZDINBWGQ
.
You are receiving this because you were mentioned.Message ID:
@.***>

@benhutchison I updated the tests to run with CockroachDB (v23.2.4), together with some compatibility updates.
It's released with 0.3.2. You can give it another try. I hardcoded the Typer.Strategy to BuiltinsOnly and don't think that it'll need changing, dumbo actually doesn't need to deal with user defined types.
There are some (maybe minor) differences in behavior:

I couldn't enforce Flyway to run migrations in a transaction for some test cases (like here or here). So Flyway partially failed and was not able to rollback applied changes, can't tell whether it's on purpose or not 🤷. Latest Flyway (as of now 10.12.0) also gives a warning about version support:
warn [Database] Flyway upgrade recommended: CockroachDB 23.2 is newer than this version of Flyway and support has not been tested. The latest supported version of CockroachDB is 22.1.
 Just something to keep in mind. CockroachDB may deal with it differently I guess. I added it as a note to the readme.

this example below that is mentioned in the docs is not going to work. CockroachDB doesn't support table-level locks with TABLE LOCK like Postgres. I considered this one as not really important for now and didn't put effort into checking on how to read locks in cockroachDB. I am also not sure whether this feature is needed or going to stay in dumbo.

// migration progress logs can be added optionally in case you'd like dumbo to provide some feedback on longer running queries
// it will perform requests to Postgres in given interval to check for queries that are causing the lock on migration history table
dumboWithResouces.withMigrationStateLogAfter[IO](5.seconds)(
 /* use config as above */
)

Updated skunk to 1.0.0-M6 which includes the oid fix in release 0.3.3. Going to close the issue.


i18n support

Would you kindy advice me how to support i18n in your boilerplate @bertho-zero, i'm new to redux/duck and trying to adapt them :) I tried to follow some guides from erikas project

https://github.com/erikras/react-redux-universal-hot-example/issues/140

but stuck at Locale switcher part, hope u can help me out

@cambeme Well. i'm using this lib https://react.i18next.com/ , and works perfectly. Even with switcher

Did you appied i18next into bertho's boilerplate? Can you guide me what i should modify? I'm still struggling the whole last week :(

@firdausious Can you help me with this?

@bertho-zero Can you help me with this?

So, based on the fork of simpleblack, I have this:
packagage.json:
  "dependencies": {
    …
    "i18next": "^11.5.0",
    "i18next-express-middleware": "^1.2.0",
    "i18next-node-fs-backend": "^1.2.1",
    …
    "react-i18next": "^7.10.1",
    …
}

src/i18n-server.js:
import i18n from 'i18next';
import Backend from 'i18next-node-fs-backend';
import { LanguageDetector } from 'i18next-express-middleware';

i18n
	.use(Backend)
	.use(LanguageDetector)
	.init({
		fallbackLng: 'fr',

		// have a common namespace used around the full app
		ns: ['common'],
		defaultNS: 'common',

		debug: true,

		interpolation: {
			escapeValue: false // not needed for react!!
		},

		backend: {
			loadPath: 'src/locales/{{lng}}/{{ns}}.json',
			jsonIndent: 2
		}
	})
	.loadLanguages(['fr', 'en']);

export default i18n;

src/i18n-client.js:
import i18n from 'i18next';

i18n
	.init({
		whitelist: ['fr', 'en'],
		fallbackLng: 'fr',

		// have a common namespace used around the full app
		ns: ['common'],
		defaultNS: 'common',

		debug: true,

		interpolation: {
			escapeValue: false, // not needed for react!!
		}
	});

export default i18n;

src/server.js:
import { I18nextProvider } from 'react-i18next';
import i18nMiddleware from 'i18next-express-middleware';
import i18n from './i18n-server';

app
	.use(…)
	…
	.use(i18nMiddleware.handle(i18n));

app.use(async (req, res) => {
	…
	const locale = req.language;
	const resources = i18n.getResourceBundle(locale, 'common');
	const i18nClient = { locale, resources };

	const i18nServer = i18n.cloneInstance();
	i18nServer.changeLanguage(locale);

	function hydrate() {
		res.write('<!doctype html>');
		ReactDOM.renderToNodeStream(<Html
			assets={webpackIsomorphicTools.assets()}
			store={store}
			i18n={i18nClient} // Here
		/>).pipe(res);
	}

	const component = (
		…
			<I18nextProvider i18n={i18nServer}>
				…
			</I18nextProvider>
		…
	)

	…

	const html = (
		<Html
			assets={webpackIsomorphicTools.assets()}
			bundles={bundles}
			content={content}
			store={store}
			i18n={i18nClient} // Here
		/>
	);
}

src/client.js:
import { I18nextProvider } from 'react-i18next';
import i18n from './i18n-client';

const hydrate = async _routes => {
	i18n.changeLanguage(window.__i18n.locale);
	i18n.addResourceBundle(window.__i18n.locale, 'common', window.__i18n.resources, true);
	…
	ReactDOM.hydrate(
		<HotEnabler>
			<I18nextProvider i18n={i18n} >
				…
			</I18nextProvider>
		…

helpers/Html.js:
<script dangerouslySetInnerHTML={{__html: `window.__i18n=${serialize(i18n)};`}} charSet="UTF-8" />

And the locales files:
src
├── locales
│   ├── en
│   │   └── common.json
│   └── fr
│       └── common.json

With this configuration, everything works well, but I would like to add the language parameter in the route path. Currently, we can change the language by adding this parameter: /?lng=en or /?lng=fr, but I prefer something like this: /en/ or /fr/.
Can you help me with this?

I would rather have used https://github.com/yahoo/react-intl.
But in any case I have not yet studied the subject.
For your problem you should edit routes and use the react-router params (https://reacttraining.com/react-router/web/example/url-params):
Example (pseudo-code):
const routes = [
  {
    component: App,
    routes: [
      { path: '/(:lang/)', exact: true, component: Home },
      { path: '/(:lang/)about', component: About },
      { path: '/(:lang/)chat', component: Chat },
      { path: '/(:lang/)login', component: Login },
      { path: '/(:lang/)login-success', component: isAuthenticated(LoginSuccess) },
      { path: '/(:lang/)register', component: isNotAuthenticated(Register) },
      { component: NotFound }
    ]
  }
];

then use match.params.lang from:
https://github.com/bertho-zero/react-redux-universal-hot-example/blob/98a9386aca0998f4906aaec8a40174194735fcfe/src/components/ReduxAsyncConnect/ReduxAsyncConnect.js#L36
https://github.com/bertho-zero/react-redux-universal-hot-example/blob/66f8418cd3ab9ea02d02f54039ce1786786c2401/src/server.js#L148

If it is not too much work you can do a PR, I would do one with react-intl and I could then choose. And then those who want to change libraries will only have to look at your PR.

@vermotr Thank you for your work, I actually try to implement i18next but with your changes, but it's not working. I read the react-i18n documentation but wasn't able to solve the issue. Would it be possible to guide me little bit through the integration or make a pull PR of you changes, that I can try to figure out what are the differences in my code or possible issues.
Thanks in advance


topic:: styling missing in Sphinx Theme documentation

Styling for topic:: in documentation is missing in Sphinx theme
There are four instances in the documentation that use the topic:: markup eg. docs/advanced_topics/images/custom_image_model.rst#L53 where the text "Migrating from the builtin image model" appears.
These lines are written as headers, and the final HTML has them marked up with .topic and .topic-title styles but there is no corresponding CSS to style them.
Steps to Reproduce

Compare https://docs.wagtail.org/en/stable/advanced_topics/images/custom_image_model.html#custom-image-model with https://docs.wagtail.org/en/v2.12.5/advanced_topics/images/custom_image_model.html. Note that in the latter, the text in question is in bold.

Latest:

v2.12:

These lines of "topic" text should appear in bold or some other more distinctive style. This could be solved with a change to Sphinx Wagtail Theme, or they should be migrated to an alternative header markup.

I have confirmed that this issue can be reproduced as described on a fresh Wagtail project: N/A

Technical details

Wagtail version: problems appears from v2.13 onwards
Four files affected:

docs/advanced_topics/embeds.rst
docs/advanced_topics/images/custom_image_model.rst
docs/reference/pages/panels.rst
docs/topics/search/indexing.rst

I think the solution would be to migrate these pages to markdown (this is our eventual goal) and then not use the topic directive but rather just plain headings.

Marking as good first issue. If anyone wants to pick this up.
The four documentation pages linked above should be converted to markdown as per the documentation (latest linked).
https://docs.wagtail.org/en/latest/contributing/documentation_guidelines.html
Remove the topic usage and instead replace with headers and maybe a reference if needed.
Ensure that any other usage of the topic throughout the docs gets replaced with the appropriate internal reference link.

Hi @lb-,
Thanks for the pointers, I'll look at preparing a PR in coming days.
I only found four instances of topic:: across the entire documentation, so changing them to align with the rest of the doco makes sense.

Hopefully this isn't opening a can of worms. One of the four files, docs/advanced_topics/embeds.rst (see Embeds includes a list of attributes from a class near the bottom. I haven't found any Markdown versions in the overall documentation to give me a template to follow.
The existing rendering uses a definition list (dl/dt/dd). In the markdown version, I can get a rendering that is very similar if I enable the Myst extension "deflist" - so that is something the core contributors might want to consider for endorsement before I go too far further.
The only visual difference I've noted is that the definition terms do not have permalink, unlike the original reStructuredText version. What I've read so far of the Sphinx doco suggests there might be a way to template the HTML, maybe a Sphinx guru will know if this is really possible. And there is a legitimate question regarding whether that permalink is worthwhile preserving?
For reference, the current reStructuredText-sourced HTML looks like this:

      url

(text)

The URL of the original content of this embed.

whereas the Markdown-sourced equivalent looks like this:

url

(text)

The URL of the original content of this embed.

  ...

Just wrap that class / attributes syntax in
```{eval-rst}

That's the escape hatch if we need to embed rst and there is not a nice markdown equivalent.
For this syntax we would want to preserve the way rst shows it so that's fine.


Keep sticky-sidebar even if larger than the viewport

Description
The feature currently only works when the sidebar fits the viewport. With an ever growing number of items (#6334) and the lack of some cleaning features (#6116) the conversation sidebar is not sticky in more and more cases.
Unfortunately the only UX I deem acceptable other than the current logic is the one described in https://stackoverflow.com/q/73201194/288906
I always shied away from adding that because “it requires JavaScript” but I’m still hopeful that there’s a modern and lightweight way to handle it.
Please discuss any implementation details before sending a PR, I really want to ensure this stays lightweight and not substantially more complex than what we already have.
Screenshot

Example URLs
This issue or any PRs

Possible logic:

permanent position:sticky aligned to the top
on scroll: if it's going down, use a negative translateY to push the  sidebar up until its bottom side is fully visible. And viceversa when going up.

The hard part is probably avoiding getBoundingClientRect calls at every single scroll event. I'd probably cache the values and use the resize and ResizeObserver to clear them.
This feature is not easy to implement so please avoid sending PRs unless you're familiar with those APIs

Too many feature requests, realistically we’ll never get there. I’m closing those I think aren’t worth it.


Add VM TS mapping

I am aware VM is not part of the public API. But, In one of our projects we are overridding the resolvePartials function in order to do some custom resolution, it would therefore be useful if it was mapped to TS to aid in that.
Not sure if these typings are 100% accurate, I'm not particularly familiar with the structure of handlebars.js behind the scenes, but I took my best stab at it.

[x] Please don't start pull requests for security issues. Instead, file a report at https://www.npmjs.com/advisories/report?package=handlebars
[x] Maintain the existing code style
[x] Are focused on a single change (i.e. avoid large refactoring or style adjustments in untouched code if not the primary goal of the pull request)
[x] Have good commit messages
[x] Have tests
[x] Have the typings (lib/handlebars.d.ts) updated on every API change. If you need help, updating those, please mention that in the PR description.
[x] Don't significantly decrease the current code coverage (see coverage/lcov-report/index.html)
[x] Currently, the 4.x-branch contains the latest version. Please target that branch in the PR.

@nknapp Does this seem right?

@nknapp would be useful to know if this is likely to get merged? Perhaps the typings should be added back to DefinitelyTyped?
@zimmi88 what was the original reason for including the typings here?

I missed this one, sorry about that...
Honestly, I'm not sure if I want to support this. Adding those type definitions would make those internals an official API which means that we would not be able to change it without a major version increase.
On the other hand, the resolvePartials-method is mention in the compiler-api so it is already kind of official.
The compromise I would be willing to accept is:

Add VM with just the resolvePartials function and omit the other functions.
Add a /** @deprecated */ annotation to the function to clarify that this is not officially endorsed anymore.
Add a test to types/test.ts
Add a comment to the implementation of the function, warning that it is currently
part of the official API and should not be changed.
open a new issue to discuss a new way for providing the functionality.

@nknapp Makes sense, I have implemented your suggestions and opened a discussion issue: https://github.com/wycats/handlebars.js/issues/1545

Thank you. I have made some adjustments. The "options" object of "resolvePartial" seems to be different from the RuntimeOptions. Some options are missing and the "name" is new. I have used the debugger to inspect the object and decided that it should be its own type.
I hope it was ok to force push on your branch. I neede to do that after squashing and rebasing the branch.

Released in 4.2.0

Thank you. I have made some adjustments. The "options" object of "resolvePartial" seems to be different from the RuntimeOptions. Some options are missing and the "name" is new. I have used the debugger to inspect the object and decided that it should be its own type.
I hope it was ok to force push on your branch. I neede to do that after squashing and rebasing the branch.

That is fine, makes more sense now. Thanks for getting this in :)

Actually @nknapp it seems that name does exist outside resolvePartials. See https://jsfiddle.net/AndrewLeedham/jkpb3equ/6/, it exists during compile if it is currently compiling a partial. So perhaps it should be optional on the RuntimeOptions interface?

The "name" is indeed set to the runtime options, when a partial is called. But I would say this is a collateral effect, not intended. Passing "name" to a compiled template does not have any effect, in the template as far as I can see.
Do you need the "name" to be in there?

The "name" is indeed set to the runtime options, when a partial is called. But I would say this is a collateral effect, not intended. Passing "name" to a compiled template does not have any effect, in the template as far as I can see.
Do you need the "name" to be in there?

We are currently using it yes. It is an unusual use case, but we are checking the name against a map of Mustache templates and rendering it if it exists, otherwise just continue the normal handlebars render.

I can see how you are using it? Where do you insert the logic that checks the name against the template map. Are you overriding the "compile" function?
I'm asking because I don't think that this should be in the official API. Overriding any functions in the Handlebars-prototype is not intended and should not be promoted. Every addition to the API means that there is one more case where I would have to increase the major version of the library. But I would like to maintain compatibility, because that is least hassle for all users.
You can send me a short fiddle that shows how you are doing it. I may change my mind and add it. Please create another issue for that, so that it is better to find.
But after reading "Clean Code" by Robert C. Martin, and looking at the Handlebars source code, I feel the urge to do some refactorings. The code involving the "name" option. May also be subject to change in the future.
If you give me a usage example, I might keep it in mind when changing that code.
In any case, you can still add you own type definitions in your own projects, merging the "name" into the options.
https://www.typescriptlang.org/docs/handbook/declaration-merging.html


Skipping TransitGatewayApisV1

Skipping TransitGatewayApisV1 as API Key is missing.

:tada: This PR is included in version 0.33.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Add option to start/stop flashlight using power button (instead of camera)

I apologize if this is the wrong place to post this.
If possible,, it would be nice to be able to change the app that gets opened with the power button
I don't hardly use the camera, but flashlight i use daily...so it would be handy to be able to change the app that gets opened when pressing the power button
Thank you!

https://github.com/GrapheneOS/os-issue-tracker/issues/1301


feat: add ripple

Main Changes Explained

Create the internal component Ripple
Add Ripple to Button, and enable the effect by default
Add enableRipple props to Button

Screenshots / Video of Changes
https://github.com/tsinghua-lau/fusion-ui/assets/62367016/1178794c-50b5-4d8d-ba33-88d6f2dfc9ed

Thanks


Upgrade amazonka to 1.3.6

For whenever we're ready to do this.

I might raise an issue of things to do before this goes in, not many now

Sounds good, thanks.

Fixes #220

@olorin Feel free to rebase and merge this now

Think might have missed elb and emr, will fix up tomorrow

Derp, sorry. #268


Update validations.py

Modified by adding a check to make sure username  only starts with latter
Fixes #<issue_number_goes_here>

It's a good idea to open an issue first for discussion.

[ ] Tests pass
[ ] Appropriate changes to README are included in PR

Added new checks to validate username.

Added new checks to validate username.

Added new checks to validate username.

Added new checks to validate username.

Added new checks to validate username.

Added new checks to validate username.

Added new checks to validate username.


Implementation of PotentialEnergyDynPyCodeComponent and PotentialEnergySymPyCodeComponent classes

Two classes for code reporting are to create. Mentioned classes have to embed an examples of DynPy or/and SymPy classes and methods application which present potential energy calculations.

SymPy - https://cocalc.com/projects/cdcf3856-3e8b-4d1c-bcb2-5c42ba749d83/files/Marek Skwarka/Shared files/modules/dynpy/utilities/components/mech/en.py#line=392
DynPy - https://cocalc.com/projects/cdcf3856-3e8b-4d1c-bcb2-5c42ba749d83/files/Marek Skwarka/Shared files/modules/dynpy/utilities/components/mech/en.py#line=353

Done. Issue can be closed.


cf push 6.7.0-c38c991 bug from Bluemix

Tell us that you ran this command:
            cf push

    using this version of the CLI:

            6.7.0-c38c991

    and that this error occurred:

            runtime error: invalid memory address or nil pointer dereference

    and this stack trace:

            goroutine 1 [running]:
    main.generateBacktrace(0xc21000a000, 0x2)
            /Users/pivotal/go-agent/pipelines/Mac-OSX-Testing/src/github.com/cloudfoundry/cli/main/main.go:147 +0xa9
    main.handlePanics(0xf9a5d8, 0xc210066500)
            /Users/pivotal/go-agent/pipelines/Mac-OSX-Testing/src/github.com/cloudfoundry/cli/main/main.go:131 +0x123
    runtime.panic(0x51b5c0, 0xd78d59)
            /usr/local/go/src/pkg/runtime/panic.c:248 +0x106
    io/ioutil.func·001()
            /usr/local/go/src/pkg/io/ioutil/ioutil.go:29 +0xf0
    runtime.panic(0x51b5c0, 0xd78d59)
            /usr/local/go/src/pkg/runtime/panic.c:248 +0x106
    bytes.(*Buffer).ReadFrom(0xc210bb6d20, 0x0, 0x0, 0x0, 0x0, ...)
            /usr/local/go/src/pkg/bytes/buffer.go:169 +0x1f1
    io/ioutil.readAll(0x0, 0x0, 0x200, 0x0, 0x0, ...)
            /usr/local/go/src/pkg/io/ioutil/ioutil.go:32 +0x156
    io/ioutil.ReadAll(0x0, 0x0, 0x0, 0x0, 0x0, ...)
            /usr/local/go/src/pkg/io/ioutil/ioutil.go:41 +0x3f
    github.com/cloudfoundry/cli/cf/net.Gateway.doRequestAndHandlerError(0xfa3a58, 0xc210085bd0, 0x6dab28, 0x1, 0x12a05f200, ...)
            /Users/pivotal/go-a

That's all I got from terminal.

Oscar:
Can you tell me what version of runtime you are hitting?  (cf curl /v2/info)
and rerun the cf push with trace on (CF_TRACE=true cf push) and share the
output?
Greg Oehmen
Cloud Foundry Product Manager
415.205.6596
On Sun, May 17, 2015 at 6:26 AM, cf-gitbot<EMAIL_ADDRESS>wrote:

We have created an issue in Pivotal Tracker to manage this. You can view
the current status of your issue at:
https://www.pivotaltracker.com/story/show/94745348.
—
Reply to this email directly or view it on GitHub
https://github.com/cloudfoundry/cli/issues/450#issuecomment-102804233.

Sure thing! Please note, I actually got 'cf push' working by doing another push like 20 min later. Not sure if this is a reoccurring bug or just a temp glitch. The whole trace log is cut off (2000+ lines) in this input form (and on pastebin) because of limits. Should I send it by email?
   "name": "Bluemix",
   "build": "205005",
   "support": "http://ibm.com",
   "version": 2,
   "description": "IBM Bluemix",
   "authorization_endpoint": "https://login.ng.bluemix.net/UAALoginServerWAR",
   "token_endpoint": "https://uaa.ng.bluemix.net",
   "min_cli_version": null,
   "min_recommended_cli_version": null,
   "api_version": "2.23.0",
   "logging_endpoint": "wss://loggregator.ng.bluemix.net:443"
}```

Oscar:
2.23.0 is a pretty new version of runtime.  I recommend upgrading your CLI
to latest stable.  Your CLI was released last November.
Greg Oehmen
Cloud Foundry Product Manager
415.205.6596
On Mon, May 18, 2015 at 1:52 AM, Oscar Almgren<EMAIL_ADDRESS>wrote:

Sure thing! Please note, I actually got 'cf push' working by doing another
push like 20 min later. Not sure if this is a reoccurring bug or just a
temp glitch. The whole trace log is cut off (2000+ lines) in this input
form (and on pastebin) because of limits. Should I send it by email?
{
"name": "Bluemix",
"build": "205005",
"support": "http://ibm.com",
"version": 2,
"description": "IBM Bluemix",
"authorization_endpoint": "https://login.ng.bluemix.net/UAALoginServerWAR
",
"token_endpoint": "https://uaa.ng.bluemix.net",
"min_cli_version": null,
"min_recommended_cli_version": null,
"api_version": "2.23.0",
"logging_endpoint": "wss://loggregator.ng.bluemix.net:443"
}
—
Reply to this email directly or view it on GitHub
https://github.com/cloudfoundry/cli/issues/450#issuecomment-102980083.

@OscarAlmgren - we can't repro this.  I'm going to close this issue since your subsequent push worked just fine.  If this keeps happening, please re-open and we'll take a deeper look.


New rule on Amazon EC2 instance to check Public IP association.

New rule on Amazon EC2 instance to check Public IP association (EC2_INSTANCE_NO_PUBLIC_IP).
Please provide comments on the gherkin if any.

Description:
Checks whether Amazon EC2 instances have a public IP association or not. The rule is NON_COMPLIANT if the publicIp field is present in the Amazon EC2 instance configuration item.
Trigger:
Configuration Change on AWS::EC2::Instance
Reports on:
AWS::EC2::Instance
Rule Parameters:
None
Scenarios:
  Scenario: 1
     Given: The publicIp field is present in the Amazon EC2 instance configuration item.
     Then: Return NON_COMPLIANT
  Scenario: 2
     Given: The publicIp field is not present in the Amazon EC2 instance configuration item.
     Then: Return COMPLIANT

Looks good. Does the  Configuration Change on AWS::EC2::Instance triggers with the addtion of an Elastic IP on an EC2 instance?

Looks good. Does the  Configuration Change on AWS::EC2::Instance triggers with the addtion of an Elastic IP on an EC2 instance?

Yes, it does.

Ok good to go!


Add timeout when requesting news or news detail

Closes #50
To do: remove loading progress when timeout exception is thrown.

Done. Please double check.


Add consistency and improve documentation for collision checking

We should have more consistency (and more documentation/comments) about where we follow what collision constraints. It took me a long time to understand that the collision constraints in Feeding Demo (e.g., this) are actually empty as far as I can tell, and to get the world collision constraints you have to call a function in aikido (e.g., this call to this aikido function). Before I realized the different collision constraints that are being used, I'd freely copy/paste a call to planToConfiguration without considering which collision constraints are being used. This makes it more challenging for new developers to onboard onto ADA.
See #11 for the problem that led me to realize the confusion and inconsistencies in how we treat collisions.
As a solution, I'd propose the following:

All collision constraints should come from calls to FeedingDemo. So we should add a FeedingDemo function (e.g., getWorldCollisionConstraint()) for getting the collision constraints that are currently gotten from the aikido function.
Remove and/or consolidate the empty mCollisionFreeConstraint and mCollisionFreeConstraintWithWallFurtherBack. The new function that gets a blank collision constraint should be called something like getEmptyCollisionConstraint().

Taging @egordon to confirm that mCollisionFreeConstraint and mCollisionFreeConstraintWithWallFurtherBack are, in fact, the same thing and are both empty.


[Qualcomm feedback] Chapter 6.2.3 Clarify optional fields of TVM Evidence claims

Reference: link

Re Fig 19.
The TVM runtime measurements Claim is optional, all other above described Claims must be
present in the TVM Token.

Isn't the tvm-identity also represented as optional in Fig 19 above? Also described as optional in the paragraph "TVM Identity Claim" below

clarified in PR #69
cc @ozkoyuncu


⚠ Bofrost.com has degraded performance

In e1ea06c, Bofrost.com (https://www.bofrost.com) experienced degraded performance:

HTTP code: 200
Response time: 867 ms

Resolved: Bofrost.com performance has improved in 9efc859.


ImportError: cannot import name spatial

I'm using python in Abaqus, I already have scipy as same version in abaqus and I can also import scipy without any error but when I want to import spatial as: from scipy import spatial I got an error as below:
ImportError: cannot import name spatial
I wonder if someone can help me out.

Can you try directly importing the library import scipy.spatial ?
Otherwise your installation is corrupted and/or trying to import from each other.

Thanks for the reply. I tried and got this again:

import scipy
import scipy.spatial
ImportError: No module named spatial

@Muhamadamani did you solve your issue? If not could you try with the latest version of SciPy?


Allow non-specified input tokens in old starting token format

This fixes a regression that wasn't handled with the recent fallback fix. Closes #862
cc @jamesls @kyleknap

Code looks fine :ship: but with like the last PR can you add an integration or functional test for this?
I want to make sure that we never see this issue or issues related to this regression again and I think we could have avoided this one with an additional functional/integration test.
Looks like we already have pagination tests for route53: https://github.com/boto/botocore/blob/develop/tests/integration/test_route53.py. So maybe look there to start?

:+1:

Thanks! :ship:


FR: Load Full Archive Button

Reasoning:
In addition to storing them on Foundry, I also export chat archives via the "Save Page" option to grab them as an html file. Doing so requires the full chat to be loaded since it only saves the stuff that's already loaded in. Unfortunately, most of our chat archives are multiple thousand messages long and scrolling up to load everything can get somewhat tedious.
Feature:
It would be helpful if there was a button (or maybe an option in the settings) that would allow you to load all the messages at once.

This issue is compounded by the following core foundry issue: https://github.com/foundryvtt/foundryvtt/issues/10951
Messages in archives may also often render twice as the result.


US-2083 EOA Wallet implementation uses the same derivation path for both mainnet and testnet

https://github.com/rsksmart/rif-wallet/assets/47615361/e6924f0b-69da-48ab-931a-5c7f5c265d8d

However, on the Android app, I see something like this:

@jessgusclark @TravellerOnTheRun

However, this is how I see the Android app.

@jessgusclark @TravellerOnTheRun

I'll check it out.

However, this is how I see the Android app.

@jessgusclark @TravellerOnTheRun

Apparently this is a separate issue, I see the same on develop. @jessgusclark we need a separate ticket for that. I bet it's just default font color is applied.


Added docstring for textmemo

You won a prize from our Cali swag lode! Contact me to pick it up or come to the Atrium bar ASAP!

Gosh. So cool. Im sitting in the bar right now..
Sendt fra min iPhone
Den 7. mars 2015 kl. 18:43 skrev "Ben Welsh" <notifications<EMAIL_ADDRESS>You won a prize from our Cali swag lode! Contact me to pick it up or come to the Atrium bar ASAP!
—
Reply to this email directly or view it on GitHubhttps://github.com/california-civic-data-coalition/django-calaccess-raw-data/pull/185#issuecomment-77718748.

Sorry I missed you. If you come to M102 before classes start you can pick
up your prize.
On Sat, Mar 7, 2015, 10:13 PM Annkiha<EMAIL_ADDRESS>wrote:

Gosh. So cool. Im sitting in the bar right now..
Sendt fra min iPhone
Den 7. mars 2015 kl. 18:43 skrev "Ben Welsh"<EMAIL_ADDRESS><EMAIL_ADDRESS>You won a prize from our Cali swag lode! Contact me to pick it up or come
to the Atrium bar ASAP!
—
Reply to this email directly or view it on GitHub<
https://github.com/california-civic-data-coalition/django-calaccess-raw-data/pull/185#issuecomment-77718748

.

—
Reply to this email directly or view it on GitHub
https://github.com/california-civic-data-coalition/django-calaccess-raw-data/pull/185#issuecomment-77730312
.


Static Resources Panel empty

Static Resources Panel empty in trunk (not sure how revision but has 1 or 2 weeks only.
In my log I have some css (like mycss.css?hash=XXXXXXX9) ans images in JPG, SVG and PNG formats (maybe more extensions but this 4 sure. .jpg files don't have ?hash).
I hope I can send a partial log soon and put my branch revision.
Thanks so much for this great piece of code!

Please use --all-static-files to include static files that contain a query string.

Closing this. Feel free to reopen it if needed.

Sorry for don reply fast, Im on middle of server migration...
Is a mine problem, Im logging static files in other file.
Sorry for the noise!

@jabiertxof no problem. Glad you were able to figure it out :)

Thanks!


Replace 3-second delay with adjust semantics for scan command

The scan command adjusts the centre of an ongoing scan, without resetting the scan increment to be small.
Draft until it's tested on the robot.

We will still want to test this but looks good otherwise


Running out of memory when predicting large multimers

I was able to run predictions for a dimer with 4 cores and 16 GB memory. When I attempted that with a multimer that contains 12 units, the process runs out of memory even if I upgrade the machine to 16 cores and 128 GB of memory and reduce the number of predictions from the default value to 1. Has anyone else tried predicting large complexes like these and are there any known limitations in running predictions like these? Any ideas on how to improve the memory usage or guidelines on how much memory is needed for a task like this would be very valuable. Thank you!

You could try OpenFold (https://github.com/aqlaboratory/openfold) with described tweaks to minimize memory usage...

Hi yogesh-dhande,
I also had problems with the RAM.
After I increased the value of the following variable from 4 to 10, the calculation worked for me.
https://github.com/deepmind/alphafold/blob/624a44966619218f546852863f0f9220fc9c2849/docker/run_docker.py#L247
From the Jax documentation it appears that this value determines how much memory may be used to cache the data that does not fit into the VRAM.
https://jax.readthedocs.io/en/latest/gpu_memory_allocation.html#gpu-memory-allocation
In my case, I have a Quadro RTX 5000 with 16GB VRAM
'XLA_PYTHON_CLIENT_MEM_FRACTION': '10.0', -> My AlphaFOld is allowed to use 160GB RAM for swapping.
64 Hyper-V virtual cores and 160GB RAM.
I am using Hyper-V Server 2019 as host system.
The VM is running Ubuntu 22.04

Closing this issue for now, feel free to reopen if this is still a problem.


Failing to connect to the Discord gateway

It may be due to it trying to validate the cert discord sends. Currently the error it gets is:
Unable to connect to gateway.discord.gg on port 443, error: error in handshake : X509 - Certificate verification failed, e.g. CRL, CA or signature check failed
Find a way to disable validiation in mbedX509

Fixed it by using cpprestsdk

Fixed it by using cpprestsdk


WIP: [nrf noup] net: lwm2m: SenML CBOR read support

Work in progress - stack usage is through the roof.
LwM2M v1.1 SenML CBOR content format support for reading data out from the device
Signed-off-by: Veijo Pesonen<EMAIL_ADDRESS>

@jheiskan81 @SeppoTakalo @rlubos Just to give a preview and gathering feedback if you have ideas to reduce stack usage. DOM style encoding takes quite some stack space and storing the base names could be done more intelligently.

I fixed the write support for time values. Only remaining issue is writing opaque data but that can be done later.

If you don't mind the documentation build failure this could be merged once jheiskan81/sdk-nrf#2 is in.


Embedded Crowdstrike-AWS-ControlTower-Implementation-Guide.pdf has track changes on

PDF shows track changes/comments

Change tracking removed


Keep data on create

Is there a way to keep the value of some fields after creating a resource?
Imagine a situation where the user is adding a lot of documents that belong to a certain group. Everytime the user wants to "Save and Create Another", he has to set the same group again.

No sorry.


Refactor: extract variants from CheckStatus

Previously the CheckStatus (now Outcome) data structure consisted of two variants containing exactly the same fields. Now the variants have been extracted as check::Status and the common fields can be found in the Outcome struct.

bors r+


Added the possibility of getting multiple results

As mentioned in the Issue #6 I have added the possibility of getting multiple results:
var scraper = require('google-search-scraper');

var options = {
  query: 'nodejs',
  limit: 10,
  fullResult: true
};

scraper.search(options, function(err, results) {
  if(err) throw err;
  // Array of url's
  console.log(results);
})

Thanks @tommelo . This prevents an issue where a crawler gets stuck when there are fewer results than the set limit. The original result callback doesn't inform the caller when there are no more urls, and makes the following implementation fragile:
let urls = [];
scraper.search(options, (err, url) => {
   urls.push(url);
   // handleResults is never called if there are fewer results.
   if (urls.length === options.limit) {
     handleResults(name, urls);
   }
 });


PR: Webpack DLL Plugin

Willing to make a PR to use the Webpack dll plugin:
<EMAIL_ADDRESS>Would this be welcome?

I haven't used it before, so hard to make decision at this moment. However, if you give it a try in a PR, then we can see how it make build process better and maybe merge it or not.

Some few great links I've found regarding this. I'll look into adding DLL support this weekend.
https://github.com/amireh/happypack#benchmarks
https://robertknight.github.io/posts/webpack-dll-plugins/
http://engineering.invisionapp.com/post/optimizing-webpack/

Let's merge this into webpack-blocks discussion


Fix FindAVDEVICE.cmake in case without pkg-config installed with ffmpeg >= 5.1

🦟 Bug fix
Summary
ffmpeg 5.1 changed the structure of version headers (see https://github.com/FFmpeg/FFmpeg/commit/884c5976592c2d8084e8c9951c94ddf04019d81d), this PR fixes FindAVDEVICE.cmake to work correctly (i.e. finding the version number) even in the case in which no pkg-config is installed in the system.
The GitHub online editor also added a newline at the end of the file, let me know if it is ok to leave it, otherwise I can look how to remove it. I tried but apparently it is not possible anymore to commit a change via GitHub online editor without having automatically added a newline at the end of file.
Checklist

[x] Signed all commits for DCO
[ ] Added tests
[x] Updated documentation (as needed)
[x] Updated migration guide (as needed)
[x] Consider updating Python bindings (if the library has them)
[ ] codecheck passed (See contributing)
[ ] All tests passed (See test coverage)
[ ] While waiting for a review on your PR, please help review another open pull request to support the maintainers

Note to maintainers: Remember to use Squash-Merge and edit the commit message to match the pull request summary while retaining Signed-off-by messages.

ffmpeg 5.1 changed the structure of version headers (see FFmpeg/FFmpeg@884c597),

Ummm it was 51 lines trivial text file used by many, I'm going to think that there was a good reason to do it :)

The GitHub online editor also added a newline at the end of the file, let me know if it is ok to leave it, otherwise I can look how to remove it. I tried but apparently it is not possible anymore to commit a change via GitHub online editor without having automatically added a newline at the end of file.

I tried to be clever than the editor and failed miserably. They probably are now trying to be serious with the POSIX standard on text files. I'm not here to go again the standards and to waste your time so let's consider it fine unless someone want's to argue the opposite.


feat(core): add MatchMediaController

What I did

Copy over MatchMediaController from https://github.com/RedHat-UX/red-hat-design-system/pull/254

closes #2032

Maybe we should wait on this a minute, https://github.com/RedHat-UX/red-hat-design-system/discussions/278 might be preferable

Let's not do this right now


Please bump ghc-prim dependency for GHC 7.10

-                   ghc-prim   >= 0.2 && < 0.4
+                   ghc-prim   >= 0.2 && < 0.5

Yes, please bump. I could not cabal install https://github.com/spell-music/csound-expression because of the dependency on stable-maps.


Support multiple platforms on a single runner

Feature Description
I’d like to leverage Docker’s ability to run containers for different platforms through Gitea Actions.
E.g., I can run docker run node:16-buster --platform linux/amd64 even on an arm64 runner, and it will use Docker to emulate the platform I specify on the hardware I have.
Given that, it would be nice to specify a job like:
jobs:
  build:
    strategy:
      matrix:
        platform: [arm64, amd64]
        release: [ubuntu-latest, ubuntu-20.04, ubuntu-18.04]

    runs-on: ${{ matrix.release }}
      platform: ${{ matrix.platform }}

Gitea can find the best runner(s) to run the job. Since the runners know their platform, they can announce it to Gitea, and it can use a native runner (preferred) or one running under emulation to complete the job.
If runners are available for multiple platforms and none is specified, the job can just run on any of them (which is basically what happens today).
Screenshots
No response

As a workround, you can define a special label for that runner.

In this case, I have to define three labels per runner. I have a job to build that is arm64 and amd64, for Debians buster, bookworm, and  bullseye. So I need two runners (one on each platform) with three labels each.
That’s why I think platform should be isolated from the label.

In this case, I have to define three labels per runner. I have a job to build that is arm64 and amd64, for Debians buster, bookworm, and bullseye. So I need two runners (one on each platform) with three labels each.
That’s why I think platform should be isolated from the label.

Sounds reasonable. But I don't know how Github did that and it maybe an extend syntax from Github Actions.


python3 run.py --task "[chat-dev]" --name "[test]"

Retrying (Retry(total=0, connect=None, read=None, redirect=None, status=None)) after connection broken by 'NewConnectionError('<urllib3.connection.HTTPSConnection object at 0x7f6cf61d7a60>: Failed to establish a new connection: [Errno 101] Network is unreachable')': /v1/chat/completions

Thank you for your feedback. Could you please describe the error in more detail？


140 iters then the loss be NAN

iter: 100 / 70000, total loss: 1.238060

rpn_loss_cls: 0.503647
rpn_loss_box: 0.560238
loss_cls: 0.128045
loss_box: 0.046130
lr: 0.001000
speed: 0.645s / iter
iter: 120 / 70000, total loss: 1.312589
rpn_loss_cls: 0.471951
rpn_loss_box: 0.668822
loss_cls: 0.135918
loss_box: 0.035899
lr: 0.001000
speed: 0.626s / iter
Traceback (most recent call last):
File "./tools/trainval_net.py", line 138, in
max_iters=args.max_iters)
File "/data/huaxhuan/FasterRCNN/pytorch-faster-rcnn/tools/../lib/model/train_val.py", line 379, in train_net
sw.train_model(max_iters)
File "/data/huaxhuan/FasterRCNN/pytorch-faster-rcnn/tools/../lib/model/train_val.py", line 267, in train_model
self.net.train_step(blobs, self.optimizer)
File "/data/huaxhuan/FasterRCNN/pytorch-faster-rcnn/tools/../lib/nets/network.py", line 447, in train_step
self.forward(blobs['data'], blobs['im_info'], blobs['gt_boxes'])
File "/data/huaxhuan/FasterRCNN/pytorch-faster-rcnn/tools/../lib/nets/network.py", line 396, in forward
self._add_losses() # compute losses
File "/data/huaxhuan/FasterRCNN/pytorch-faster-rcnn/tools/../lib/nets/network.py", line 198, in _add_losses
assert (rpn_bbox_pred==rpn_bbox_pred).all()
AssertionError

I use this PyTorch faster-rcnn to train the VisDrone data. I transform the format into pascal_voc and change the dataload. At the same time,I clean up the Cache files and make sure the ground truth boxes in roidb files have the value >0 and <width or height. However, when I train my data. It will be NAN after 140 iters. I have run the demo and original dataset both VOC and COCO and achieve same performance as ruotianluo's  @ruotianluo  in readme. So what's the problems of my model. Thanks!

Are you able to train on VOC or coco correctly?

Yes, I can train on VOC and COCO correctly and achieve the same performance as your readme say

Finally, I solve the problem that in my dataset, the start of x and y is 0, so the code that make the base to be 0 is not work for me.

@CoderHHX
Could you please clarify in detail how you solved this problem?

@Karthik-Suresh93
I just make this change in dataload class:
bbox = obj.find('bndbox')     #################################################################################
#For my data VisDrone ,the image pixel start from 0 ,so can not sub 1 when load object bbox, After I change his my code works without NAN in loss_box
# Make pixel indexes 0-based
x1 = float(bbox.find('xmin').text) #- 1
y1 = float(bbox.find('ymin').text) #- 1
x2 = float(bbox.find('xmax').text) #- 1
y2 = float(bbox.find('ymax').text) #- 1


Does the Carousel support Lazy Loading?

Has anyone added lazy loading support to the Carousel or can suggest how it can be done?

I haven't tried it or seen anyone else try it, but what aspect of lazy loading? The images themselves? The slides as a whole? The Carousel component itself?

The images themselves. When you set the images, the Carousel loads all the images. I'm looking for an option to only load the images on transition or load in groups of 3 or 4.

Sure can!

When you create your array of slides, only set the 'src' attribute for the images you want to load right away. Set an additional 'data-src' attribute for the images you want to lazy load.

When user presses the Next or Previous button, get the value from the 'data-src' attribute of the images you want to lazy load and set the 'src' attribute to that value. That will cause the browser to load the new image(s) at that time.

@AKhoo Thanks for this!
And for anyone arriving after me, here's a quick snippet to get going:

export const PhotosCarousel = ({ items }: { items: { url: string }[] }) => {
    const [activeIndex, setActiveIndex] = useState(0);
    const [animating, setAnimating] = useState(false);

    const setAnimatingTrue = () => setAnimating(true);
    const setAnimatingFalse = () => setAnimating(false);

    const [photos] = useState((items || []).map((x, i) => ({
        src: i === 0 ? x.url : '',
        dataSrc: x.url
    })))

    const updateSrcIfNeeded = (index) => {
        if (!photos[index].src) {
            photos[index].src = photos[index].dataSrc;
        }
    }

    const next = () => {
        if (animating) return;
        const nextIndex = activeIndex === photos.length - 1 ? 0 : activeIndex + 1;
        updateSrcIfNeeded(nextIndex);
        setActiveIndex(nextIndex);
    }

    const previous = () => {
        if (animating) return;
        const nextIndex = activeIndex === 0 ? photos.length - 1 : activeIndex - 1;
        updateSrcIfNeeded(nextIndex);
        setActiveIndex(nextIndex);
    }

    if (!photos || photos.length === 0) {
        return <></>
    }

    const slides = photos.map((x, i) => {
        return (
            <CarouselItem
                onExiting={setAnimatingTrue}
                onExited={setAnimatingFalse}
                key={i}
            >
                <img src={x.src} />
            </CarouselItem>
        );
    });

    return (
        <Carousel
            activeIndex={activeIndex}
            next={next}
            previous={previous}
            autoPlay={false}
            interval={false}
        >
            {slides}
            <CarouselControl direction='prev' directionText='Previous' onClickHandler={previous} />
            <CarouselControl direction='next' directionText='Next' onClickHandler={next} />
        </Carousel>
    );
}


[FRONTEND] Refactor file hierarchy

The purpose of this PR is to remove some circular dependencies and separate concerns better in the frontend. It's still not perfect -- triton.compile still includes a few runtime architecture-specific component, but at least much better than before.
This PR still assumes that AMD only supports empty kernels right now. Other PRs will follow to make the frontend supports multiple devices in a more modular way.

FYI I've seen some potential circular import issues with this PR. The error I got in the CI is the following
source $BASH_ENV
$CONDA_PYTHON xformers/benchmarks/benchmark_pytorch_transformer.py

Traceback (most recent call last):
  File "xformers/benchmarks/benchmark_pytorch_transformer.py", line 11, in <module>
    import triton
  File "/home/circleci/venv/lib/python3.8/site-packages/triton/__init__.py", line 12, in <module>
    from .runtime import (
  File "/home/circleci/venv/lib/python3.8/site-packages/triton/runtime/__init__.py", line 1, in <module>
    from . import driver
ImportError: cannot import name 'driver' from partially initialized module 'triton.runtime' (most likely due to a circular import) (/home/circleci/venv/lib/python3.8/site-packages/triton/runtime/__init__.py)

Exited with code exit status 1
CircleCI received exit code 1

FYI I've seen some potential circular import issues with this PR. The error I got in the CI is the following
source $BASH_ENV
$CONDA_PYTHON xformers/benchmarks/benchmark_pytorch_transformer.py

Traceback (most recent call last):
  File "xformers/benchmarks/benchmark_pytorch_transformer.py", line 11, in <module>
    import triton
  File "/home/circleci/venv/lib/python3.8/site-packages/triton/__init__.py", line 12, in <module>
    from .runtime import (
  File "/home/circleci/venv/lib/python3.8/site-packages/triton/runtime/__init__.py", line 1, in <module>
    from . import driver
ImportError: cannot import name 'driver' from partially initialized module 'triton.runtime' (most likely due to a circular import) (/home/circleci/venv/lib/python3.8/site-packages/triton/runtime/__init__.py)

Exited with code exit status 1
CircleCI received exit code 1

I can't repro but I do still see cyclic imports with pylint. Will fix this today.


Unable to kubectl exec or kubectl run

Hello,
When I am trying to access logs or go inside a POD, I encounter some certificates problems:
$ kubectl logs nginx-158599303-bbkwt
Error from server: Get https://aug-worker1:10250/containerLogs/default/nginx-158599303-bbkwt/nginx: x509: certificate is valid for system:node:aug-worker1, not aug-worker1

$ kubectl exec -it nginx-158599303-bbkwt /bin/sh
Error from server: error dialing backend: x509: certificate is valid for system:node:aug-worker1, not aug-worker1

When I look my certificate obtained with TLS bootstraping:
$ openssl x509 -in /var/run/kubernetes/kubelet-client.crt -text -noout

[12:03]
Certificate:
   Data:
       Version: 3 (0x2)
       Serial Number:
           0d:0f:12:7c:35:62:58:91:10:12:9d:13:02:e2:ae:b0:50:d4:fe:8c
   Signature Algorithm: sha256WithRSAEncryption
       Issuer: C=FR, ST=Paris, L=Paris, O=TOTO, OU=OPS, CN=Kubernetes
       Validity
           Not Before: Apr  3 08:29:00 2017 GMT
           Not After : Apr  3 08:29:00 2018 GMT
       Subject: O=system:nodes, CN=system:node:aug-worker1
       Subject Public Key Info:
           Public Key Algorithm: id-ecPublicKey
               Public-Key: (256 bit)
               pub: .......

My cluster seems to be healthy :
NAME                 STATUS    MESSAGE              ERROR
controller-manager   Healthy   ok
scheduler            Healthy   ok
etcd-2               Healthy   {"health": "true"}
etcd-1               Healthy   {"health": "true"}
etcd-0               Healthy   {"health": "true"}

$ kubectl get csr
NAME        AGE       REQUESTOR           CONDITION
csr-1zcr4   8h        kubelet-bootstrap   Approved,Issued
csr-9j2wk   8h        kubelet-bootstrap   Approved,Issued
csr-gv1hm   8h        kubelet-bootstrap   Approved,Issued

$ kubectl get nodes
NAME          STATUS    AGE       VERSION
aug-worker1   Ready     7h        v1.6.0
aug-worker2   Ready     7h        v1.6.0
aug-worker3   Ready     7h        v1.6.0

I am not able to find if I misconfigured my cluster or if it could be a bug.

facing the same issue here, temporarily removed
  --kubelet-certificate-authority=/var/lib/kubernetes/ca.pem \
  --kubelet-client-certificate=/var/lib/kubernetes/kubernetes.pem \
  --kubelet-client-key=/var/lib/kubernetes/kubernetes-key.pem \

from kube-apiserver service file to make it work, but what would be the solution to this?

the solution is to give the kubelet a serving certificate signed by the --kubelet-certificate-authority

OK, thank you for your feedback. Should I understand that the TLS Bootstrapping feature is not fully terminated, and should not be used for now. Am I correct ?
Regards.

Correct, it is still in progress (the flag to enable it is named --experimental-bootstrap-kubeconfig :)
Currently it bootstraps client credentials for the kubelet using the CSR API. Work is in progress for it to separately request serving certs using the CSR API.

@liggitt In 1.7.2 the issue is still present even if the flag changed from --experimental-bootstrap-kubeconfig  to   --bootstrap-kubeconfig in the documentation.
Error from server: error dialing backend: x509: certificate is valid for system:node:kubew03, not kubew03
[root@centos kube-samples]#

that flag does not control serving certificate bootstrapping. serving certificate bootstrapping is alpha in 1.7, and not enabled by default. see docs on the RotateKubeletServerCertificate feature gate in https://kubernetes.io/docs/admin/kubelet-tls-bootstrapping/ or follow https://github.com/kubernetes/features/issues/267 for more info on the serving cert bootstrapping


Review Meeting 8

Overview
The focus for this code review will be centered around the Mongo startup file and VolunteerProfile page.
Please pay attention too:

Javascript issues
React components

Review Branch
review-154
Files to review

Mongo.js
VolunteerProfile.jsx

Checklists

Architecture
Design
JS
React
Testing

Due date
Monday, March 25th at 11:30 pm
For more information
The review process is documented at: http://courses.ics.hawaii.edu/ics414s21/morea/review/reading-idpm-review.html

Aloha James,
Sorry for not responding sooner to your messages. I've been out of island
visiting family for Spring break, and I'm currently on a train to Seattle
on my way back. I'll let you know what I can do when I get secure access to
my computer again!
Mahalo,
Delsin
On Fri, Mar 22, 2024 at 2:28 PM James Ligeralde @.***>
wrote:

Assigned #154 https://github.com/Team-Octagon/voluntree/issues/154 to
@DelsinRhoney https://github.com/DelsinRhoney.
—
Reply to this email directly, view it on GitHub
https://github.com/Team-Octagon/voluntree/issues/154#event-12222574552,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AX34DY7E2YHHIJO3JQEZ32TYZTECXAVCNFSM6AAAAABFEFLAZWVHI2DSMVQWIX3LMV45UABCJFZXG5LFIV3GK3TUJZXXI2LGNFRWC5DJN5XDWMJSGIZDENJXGQ2TKMQ
.
You are receiving this because you were assigned.Message ID:
@.***>

Mongo.js

[x] Architecture
[ ] Design: DE-01/DE-02 - So I did this code, but I think the code to add dummy events to volunteers and organizations can be reduced. There are plenty of nested conditionals too.
[x] JS
[x] Reac
[x] Testing

VolunteerProfile.jsx

[x] Architecture
[x] Design
[x] JS
[x] React
[x] Testing

Mongo.js
✅ Architecture
✅ Design: DE-03: Not dead code, but addStuff can be removed as it is not used.
✅ JS
✅ React
✅ Testing
VolunteerProfile.jsx
✅ Architecture
✅ Design
✅ JS
✅ React
✅ Testing

I personally left a lot of Stuff-related code in the project just to use as reference lol. We will surely remove that some time before deployment I imagine.

Mongo.js
✅ Architecture
✅ Design
✅ JS
✅ React
✅ Testing
VolunteerProfile.jsx
✅ Architecture
✅ Design
✅ JS
✅ React
✅ Testing

Mongo.js
DE-03: Eliminate dead code. There are commented-out console.log statements.
JS-06: Avoid console.logs. There are several console.log statements throughout the code.
JS-07: Use JSDoc comments appropriately. There is no misuse but adding comments would help.
VolunteerProfile.jsx
RE-07: Don't retrieve collection data inside render():

Mongo.js:

[x] Architecture Checklist
[x] Design Checklist
[x] JS Checklist
[x] Meteor Checklist
L60, L82 are very similar, maybe a function could help clean up the code.
[x] React Checklist
[x] Testing Checklist

VolunteerProfile.jsx:

[x] Architecture Checklist
[x] Design Checklist
[x] JS Checklist
[x] Meteor Checklist
L16 to L18: Suggestion for the future - maybe it would be best to have one subscribe function that will determine which specific collection to subscribe to. For example Profile.subscribe() would automatically call VolunteerProfiles.subscribe() if you have the volunteer role. This could allow us to reuse components for admins, volunteers and organizations.
[x] React Checklist
[x] Testing Checklist

Review Meeting Outcomes
New Issues

Condense code to initialize dummy event data in Mongo.js

Other Concerns

There are plenty of collections in this application. It would not hurt to add more methods to simplify the publication and subscription process on specialized pages instead of making multiple subscriptions.


How to save the design of StickerView to edit it later?

After creating our own StickerView, I want to save the design whatever we created so that we can edit it at a later stage. Is there any way to do that?

That is not possible in this StickerView. you have to customize StickerView.
get every information (Position, Text, etc) of Sticker and make complete json.
then after render all sticker according to saved json.


Tree is missing cursor methods (isRoot, isLeaf)

The created tree is missing some cursor methods like isRoot and isLeaf. tree.root.isRoot does exist, however.

Hello @SaphuA. Why would you like to have those methods at tree level?

I made a generic method to traverse my cursors. I was expecting the root tree to be a cursor as well. It's not a big deal, just would make sense to me.

The tree itself is not accessible through cursors only. Cursors will only get back to the root cursor, never to the tree itself. Maybe your generic method should use the root cursor instead of the tree (tree.root or tree.select().

Any update @SaphuA?

This can be closed. Thanks!


Allow concurrent jobs

Add the ability to run multiple jobs concurrently. This can speed up pipelines by processing more jobs in parallel.
Investigate how to add this configuration option in the best way. The configuration templates can't be used as they don't support global options

https://github.com/Taucher2003/GitLab-Pipeline-Action/issues/39


Parallel fetching of column chunks in ParquetRecordBatchStream

Is your feature request related to a problem or challenge? Please describe what you are trying to do.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
(This section helps Arrow developers understand the context and why for this feature, in addition to  the what)
We recently rebased our project onto DataFusion's latest and we've seen a pretty big performance degradation. The issue is that we've lost the ability to prefetch entire files from object storage with the new ObjectStore interface. The buffered prefetch has been moved into ParquetRecordBatchStream but in a way that doesn't work particularly well for object storage (at least in our case).
The main issues that we've seen are:

Before, we would read 64k from the end of the file optimistically when fetching metadata but now we do separate range requests for the footer (8 bytes) to then fetch the metadata. Fetching 8 bytes from S3 takes about 80-90ms so when we are scanning a lot of objects this can add significantly to the execution time.
We are prefetching entire column chunks (which is better than fetching page-by-page) but we are fetching the column chunks sequentially.

What we found was that (at least with parquet files on the order of 100-200MB) it was much more efficient to just fetch the entire object into memory. All else equal it is of course better to read less from object storage but if we can't do it in one shot (or maybe two shots) the cost of the extra GET requests is going to significantly outweigh the benefit of fetching less data.
Describe the solution you'd like
A clear and concise description of what you want to happen.
I think there are couple things we can do:

Optimistically try and fetch the metadata in one shot. We can pass a metadata_size_hint in as a param maybe to allow users to provide information about the expected size. And maybe it defaults to 64k as it was before.
Fetch the column chunks in parallel instead of sequentially. This can either be handled in ParquetRecordBatchStream or maybe added into the ObjectStore trait by adding a get_ranges(&self, location: &Path, ranges: Vec<Range<usize>>) -> Result<Bytes> method.

Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
We could leave things as they are
Additional context
Add any other context or screenshots about the feature request here.

@tustvold @alamb

Why would you fall back to fetching the entire row group (or file) instead of fetching in parallel. For object storage fetching the entire file would probably be implemented as parallel range requests anyway.

Mainly because there is a cost associated with each S3 range request. I'm also not sure that fetching in parallel will actually be faster, as I would expect a single request to be able to saturate IO easily, fetching in parallel will just add complexity and potentially contention.

Right, there is definitely a cost but it's typically well worth it. In our workloads we parallelize object GETs into multiple 8-16MB range requests. That is good for 1-3x improvement in downloads speeds under real-world conditions.

Provided the maximum concurrency is configurable, sounds good to me. I look forward to experimenting with it and measuring its impact

@tustvold  Ok, I understand why you were worried about contention. Sine we're dealing with AsyncFileRead we need a mutable reference so if we were to do this in arrow-rs we would need to wrap it in an Arc<Mutex<>> which entirely defeats the purpose...
Instead what if we add
pub trait AsyncFileReader {
  fn get_byte_ranges(&mut self, ranges: Vec<Range<usize>>) -> BoxFuture<'_, Result<Vec<Bytes>>>;
}

This makes things quite straightforward in ParquetRecordBatchStream and we can just delegate handling of parallelism (if any) to the implementer?

I think adding a get_byte_ranges with a default implementation that falls back to serial sounds good to me. Then downstreams can override it if they wish to do so :+1:

Sorry for being late to the party. Thanks for bringing this topic up @thinkharderdev

Optimistically try and fetch the metadata in one shot. We can pass a metadata_size_hint in as a param maybe to allow users to provide information about the expected size. And maybe it defaults to 64k as it was before.

I think this is a great idea (and I think @thinkharderdev  has already done it here): https://github.com/apache/arrow-datafusion/pull/2946

Fetch the column chunks in parallel instead of sequentially. This can either be handled in ParquetRecordBatchStream or maybe added into the ObjectStore trait by adding a get_ranges(&self, location: &Path, ranges: Vec<Range>) -> Result method.

Given that the optimal prefetch strategy is likely to vary project to project and with object store to object store, I feel like it will be challenging to put logic that works everywhere in DataFusion. Ideally in my mind DataFusion would provide enough information to allow downstream consumers to efficiently implement whatever prefetch strategy they wanted.
For example, perhaps we could implement something like PrefetchingObjectStore that would wrap an existing dyn ObjectStore and that implemented the concurrent download paradigm suggested by @thinkharderdev.
If  adding get_byte_ranges is needed to allow implementing the required prefetching algorithm efficiently I am all for it 👍

Sorry for being late to the party. Thanks for bringing this topic up @thinkharderdev

Optimistically try and fetch the metadata in one shot. We can pass a metadata_size_hint in as a param maybe to allow users to provide information about the expected size. And maybe it defaults to 64k as it was before.

I think this is a great idea (and I think @thinkharderdev has already done it here): apache/arrow-datafusion#2946

Fetch the column chunks in parallel instead of sequentially. This can either be handled in ParquetRecordBatchStream or maybe added into the ObjectStore trait by adding a get_ranges(&self, location: &Path, ranges: Vec) -> Result method.

Given that the optimal prefetch strategy is likely to vary project to project and with object store to object store, I feel like it will be challenging to put logic that works everywhere in DataFusion. Ideally in my mind DataFusion would provide enough information to allow downstream consumers to efficiently implement whatever prefetch strategy they wanted.
For example, perhaps we could implement something like PrefetchingObjectStore that would wrap an existing dyn ObjectStore and that implemented the concurrent download paradigm suggested by @thinkharderdev.
If adding get_byte_ranges is needed to allow implementing the required prefetching algorithm efficiently I am all for it 👍

Yeah, I think from the point of view of arrow-rs this is the sensible way to go. It will however just push the problem one layer up the stack :). So in Datafusion we'll have to again decide whether to implement the parallel fetching directly in ParquetFileReader (which has many of the same problems as doing it here (in that we have to create a one-size-fits all solution or add extra configurations to allow some flexibility) or move it into the ObjectStore trait itself. I'll create another issue in Datafusion to discuss that though.


Makefile: Cross-compile checkpointer for multiple archs

First part to #424 is cross-compiling checkpointer binary in make all.

coreosbot run e2e

Hm, checkpointer flakes? This PR can't be affecting it. @diegs

coreosbot run e2e

coreosbot run e2e

Flakes seem to consistently relate to FAIL: TestCheckpointerUnscheduleCheckpointer, if it fails, so do all subsequent tests.


Release notes for April 7, 2022

Codecov Report

Merging #938 (de70bbe) into next (ce3e393) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             next     #938   +/-   ##
=======================================
  Coverage   47.39%   47.39%
=======================================
  Files          36       36
  Lines        1513     1513
  Branches      219      219
=======================================
  Hits          717      717
  Misses        795      795
  Partials        1        1


Question about test-time augmentation and ensemble.

Hi~
Can you share the detailed configuration about TTA and model ensemble?
I notice that these tricks bring the significant gains, but I haven’t find relative code in this repo.
Thanks!

Hi @HatsuneMiku888,
Thanks for the interest. I think TTA and model ensemble are not in the short-term release plan, since we do have higher priority release items, and other research papers typically only compare the results without these additional tricks. However, if you want to reproduce TTA and model ensemble, I would recommend you have a look at the appendix of CenterPoint and Tianwei's GitHub issues.
Best,
Haotian


core: give up updating a lock file of an unsupported version

Apparently lockfile v4 is comming soon so

Closed by #218


CardViewController never fire failure

func onTapDoneCardButton() triggers onTapDone() delegate method, which never gets .failure as a parameter, check line 202 of CardViewController

Hi @Oleygen sorry for the late reply. We addressed this in a release by handling the failure scenario in CardViewController:
checkoutApiClient.createCardToken(card: card) { result in
            switch result {
            case .success(let cardTokenResponse):
                self.delegate?.onTapDone(controller: self, cardToken: cardTokenResponse, status: .success)

            case .failure:
                self.delegate?.onTapDone(controller: self, cardToken: nil, status: .failure)
            }
        }

If you still experience this bug please feel free to re-open this issue with repro steps. Thanks.


STORM-786: KafkaBolt should ack tick tuples

STORM-512 (KafkaBolt doesn't handle ticks properly) adds special-casing of tick tuples. What is missing in the patch is that the input tuple, when it is a tick tuple, should be properly acked like normal tuples.

We already received a +1 from @nathanmarz.

+1 looks good to me.


feat: Added feature to run rendering in synchronous mode.

This makes debugging easier and takes load of slow machines. It has proven useful in the past.

@fritzduchardt , @Zebradil I added one commit. I think it can be merged now.


Work on generic list

Is it possible to make this work on generic checklists without requiring headers?
E.g. if I have an ad-hoc checklist in the middle of a document and I'd like all the checked items to be sorted at the bottom

OK, I’ll change the default/empty header from ## Completed to an empty string.


Loading of "Edit Role" page takes a lot of time

On a website with ~450 pages on it, it's takes a 2-3 minutes to load an "Edit Role" page to manage a content manager permissions.
It is a large page because of possibility to set permissions for every page.
My question is what to expect when a website will grow up to 5000 pages?
I think we will start to get a timeout error, so it is possible to avoid an every-page-permission settings?
Thank you

I assume you have "Securable" checked on the Page content type, can you remove it? And maybe use a ContentPermissionPart instead if you really need it.


[spell check cicd] do not run on main branch, and ignore .svg files

Summary
follow up to https://github.com/jetpack-io/devbox/pull/658
How was it tested?
will observe github actions run

Current dependencies on/for this PR:

main

PR #714   👈

This comment was auto-generated by Graphite.


php-fpm container can change source from debian to china?

Info:

Docker version ($ docker --version): Docker version 18.03.0-ce, build 0520e24302
Laradock commit ($ git rev-parse HEAD): da8f0d086433d6cc269ee86c92bcc93a1f88b73c
System info (Mac, PC, Linux): PC
System info disto/version: WIN10

Issue:

docker-compose build php-fpm is slow very much in china
Expected behavior:

docker-compose build php-fpm can be fast
Reproduce:

don't have error.
Relevant Code:
// place a code sample here
.

php-fpm container can change source from debian to china?
php-fpm/debian.sources.list
deb http://mirrors.aliyun.com/debian/ stretch main non-free contrib
deb-src http://mirrors.aliyun.com/debian/ stretch main non-free contrib
deb http://mirrors.aliyun.com/debian-security stretch/updates main
deb-src http://mirrors.aliyun.com/debian-security stretch/updates main
deb http://mirrors.aliyun.com/debian/ stretch-updates main non-free contrib
deb-src http://mirrors.aliyun.com/debian/ stretch-updates main non-free contrib
deb http://mirrors.aliyun.com/debian/ stretch-backports main non-free contrib
deb-src http://mirrors.aliyun.com/debian/ stretch-backports main non-free contrib

Dockerfile
# If you're in China, or you need to change sources, will be set CHANGE_SOURCE to true in .env.

ADD debian.sources.list /etc/apt/debian.sources.list
ARG CHANGE_SOURCE=false
RUN if [ ${CHANGE_SOURCE} = true ]; then \
  mv /etc/apt/sources.list /etc/apt/sources.list.back && \
  mv /etc/apt/debian.sources.list /etc/apt/sources.list \
;fi

docker-compose.yml
        .
        .
        .
php-fpm:
      build:
        context: ./php-fpm
        args:
          - CHANGE_SOURCE=${CHANGE_SOURCE}
          - LARADOCK_PHP_VERSION=${PHP_VERSION}
          - LARADOCK_PHALCON_VERSION=${PHALCON_VERSION}
        .
        .
        .

https://github.com/laradock/laradock/pull/1905/commits


Question: Clarify use case

Hello,
I'm just trying to wrap my head around is.
Suppose we have a lib precompile. Instead of:

Add babel-plugin-precompile
Import function

We're going to have:

Add babel-preset-macros (only once)
Import function from precompile.macros

Is that correct? (I changed numbering on purpose, so it's clear that babel-macros are one step easier)
My questions are:

How is determined order of plugins? I believe precompilation should be done first before compilation, but what if some plugins/macros will require to come after?
I saw only examples using tagged template literals. Does this solution work also for other elements? For example, I'm precompiling anything between <Trans> tags inside React.
How does it work for plugins, which does general code optimization, like transform-react-constant-elements? I.e: plugins, which you don't import in code, but rather works implicitly.

Thank you! I'm really hoping this will grow in usable solution. I'm willing to help anytime! 👍

Your assumptions are correct. Here are the answers:

Order is determined by the order of the import/require statements, so it's up to the user. I think that this is actually a really nice feature because currently ordering in babel is a real challenge/frustration.
macros functions are given all references to things that are imported, so it works with anything. (including JSX) See this for some examples.
The point of babel-macros is that nothing is implicit. I think that's part of the beauty of it as well. If you need it to be implicit, then that wont work with babel-macros.

These answers would go great in a FAQ that's referenced in the README. Care to add that?

Order is determined by the order of the import/require statements, so it's up to the user. I think that this is actually a really nice feature because currently ordering in babel is a real challenge/frustration.

Yes, definitely. But I guess, it also depends on order of babel-macros in Babel config, right? User is able to only reorder macros which are processed in babel-macros:
{
  "presets": [
    "env",
    "react",
    "babel-macros"  # 1st preset to run, all macros run at this stage
  ]
}

… but in this setup, there's no way to run a macro after env preset.
Not saying it's a bad thing. It's just I know how preset/plugin ordering is painful in Babel right now and this was my first concern.

macros functions are given all references to things that are imported, so it works with anything. (including JSX) See this for some examples.

Yes, I saw that but misunderstood it. I get it now, thanks 👍

The point of babel-macros is that nothing is implicit. I think that's part of the beauty of it as well. If you need it to be implicit, then that wont work with babel-macros.

There're some expectations that babel-macros could add some missing features to zero-config boilerplates. One of them are decorators. I think we would either need different syntax for decorators or an implicit macro.
My other concern is personal as I need to process AST tree to extract messages (similar what https://github.com/yahoo/babel-plugin-react-intl does).
However, I understand it's not what babel-macros are intended to solve. There's gonna be probably different solution.

These answers would go great in a FAQ that's referenced in the README. Care to add that?

Sure thing, I'll write it down (probably by the end of this week, as I'm a bit busy atm).
Thanks a lot again! 👍


Usability improvements for custom keyboard controls

See discussion for context.

Add :default_handlers option to control whether the default Livebook shortcuts are enabled/disabled when the keyboard control is active.
Add ctrl/cmd + k session shortcut to toggle keyboard controls.

Basic notebook for testing: gist
Corresponding PR in Kino: livebook-dev/kino#312

Added simple test notebook to the PR Livebook environment (this might disappear): https://pr-2145-deployment-32808-livebook.app.uffizzi.com/sessions/n4zfffm77h7ye74py2fdrgute3xad5ss73rdiizt2cc7yai6

Link to open playground notebook in PR environment: link

Thanks for the review @jonatanklosko! I'll have this (and the Kino PR) updated after I drop the kiddo off at school this morning.


Add switches to conda workflow runs

Create switches for conda workflow runs:

Runner label, keep empty for default
Install Intel PyTorch Extension
PyTorch ref, keep empty for default
Upload test reports
Ignore test errors
Custom run name

@leshikus is this issue fixed?
cc: @pbchekin

I'm still working on it.

All inputs have been added to conda-test.


Fix minor stuffs in test-suite and translations

(ready modulo CI)

Maybe we should port it to 8.11?