Memory leak

It seems, at least for me, that uproot4 is using twice as memory to load a branch compared to uproot3.
A created a simple but large enough TTree to test this with this script
#include <array>

void test() {
    TFile file("test.root", "recreate");
    TTree tree("tree", "test memory leak");

    std::array<Float_t, 1000> array;
    tree.Branch("array", &array);

    for (int i = 0; i < 100000; i++) {
        for (auto &x : array)
            x = i;
        tree.Fill();
    }
    file.Write();
    file.Close();
}

And them I loaded this file using uproot3 and 4
import uproot
with uproot.open('test.root') as file:
    data = file['tree/array'].array()

import uproot4
with uproot4.open('test.root') as file:
    data = file['tree/array'].array(library='np')

Checking the memory consumption of those processes I got this
$ ps -e -o command,pid,%mem | grep python
python  21032  10.5   # uproot4
python  21256   5.2   # uproot3

The uproot4 version is using twice as memory, so it seams the data is duplicated in memory. Next I deleted the data array with del data in both processes and that release the memory used by the array but the "duplicated" data remained in uproot4.
$ ps -e -o command,pid,%mem | grep python
python  21032  5.7  # uproot4
python  21256  0.4  # uproot3

For this test I used the 0.23 version of uproot4. I'm doing something wrong or this is indeed a bug?

What happens if you
import gc
gc.collect()

before or after the explicit data deletion? Any Python objects that are created will remain in the process's memory even if their reference counts are zero, until collected by the garbage collector. Fortunately, Python's gc.collect() does what you tell it to, allowing you to debug these things by explicitly invoking it.
One thing that Uproot 4 does differently than Uproot 3 is:

Uproot 3 allocates a destination array and all interpreted TBaskets write into that array as they're read
Uproot 4 collects a list of interpreted TBasket arrays and concatenates them when done.

The latter will use twice as much working memory, but it's less prone to error and more flexible (we can do more interpretation in parallel without knowing how long each variable-length structure will be before combining them; Uproot 3 had to delay interpretation of complex structures as ObjectArrays, which notoriously limited functionality).
I've just confirmed that these TBasket arrays are not cached. They should go out of scope when the array is fully constructed. Specifically, the basket_arrays is a dict from basket number to the temporary basket array, and the basket_arrays is a value of the branchid_arrays dict, which maps from a TBranch identifier to the basket_arrays for that TBranch (for functions that fill multiple branches in one call). That branchid_arrays dict goes out of scope when this function finishes:
https://github.com/scikit-hep/uproot4/blob/28a867d4b2215069c6eeebebc46132ed11f8f8e7/uproot4/behaviors/TBranch.py#L3150-L3273
That's the only function that fills arrays; TBranch.array, HasBranches.arrays, uproot4.iterate, uproot4.concatenate, and uproot4.lazy all call into it.

Hi Jim, thank you for the really fast and complete answer, I'm new to uproot I am still reading and understanding the code that you pointed.
In the mean time I tried to use the garbage collector as you asked and didn't see any change.
>>> import uproot4
>>> with uproot4.open('test.root') as file:
...     data = file['tree/array'].array(library='np')
...
>>> import gc    # first ps
>>> gc.collect() # second ps
0
>>> del data     # third ps
>>> gc.collect() # forth ps
0

❯ ps -e -o command,pid,%mem | grep python  # after read the file
python                        57623 10.6
❯ ps -e -o command,pid,%mem | grep python  # after the first gc.collect() call
python                        57623 10.6
❯ ps -e -o command,pid,%mem | grep python # after the del data
python                        57623  5.7
❯ ps -e -o command,pid,%mem | grep python # after the second gc.collect() call
python                        57623  5.7

So even after the deletion of the array and the garbage collection call the process seems to hold the TTree data in memory. That 5.7 % of memory is roughly the size of the TTree.

PS: The garbage collector in CPython is usually set to run every line.

As @henryiii said, the GC might mess up the memory measurements.
Alternatively to the manual collection of gc.collect() you can set the garbage collector in "aggressive mode":
import gc
gc.set_threshold((1, 1, 1))

or artificially decrease the available memory to see if the memory is still required.

CPython doesn't collect all garbage on every line, and I was only suggesting gc.collect() as a debugging tool, not for production code.
I'll look into this, to see if the memory is really leaking or just isn't being reclaimed.
By the way, although the basket arrays are not put in the cache, the final array is. Are you accounting for that?
https://uproot4.readthedocs.io/en/latest/basic.html#caching-and-memory-management

By the way, although the basket arrays are not put in the cache, the final array is. Are you accounting for that?

No, I wasnt aware of that. So the line data = file['tree/array'].array(library='np') fill the cache with the branch data and pass a reference of it to the data variable or I get a full copy of the cache, ending up with the two copies in memory?

Man, I keep meaning to get to this and I can't. Maybe an easy way to turn the cache off is
uproot4.object_cache = None
uproot4.array_cache = None

at the beginning of the program.

One of the things that came out of this was that it forced me to think hard about the caching policy. Sharing a cache among files is not a good idea because you, the user, would expect that deleting the arrays you get from a file and deleting the file object would be sufficient to remove all trace of those arrays from memory. Not so if there's a shared cache, so I'm getting rid of that. Files still have individual caches for the reasons described in the above link, but when you get rid of a file, everything that had been stuck to it will now be gone.
The actual memory leak wasn't related to the caches, though. It was in the ThreadPool and ResourceThreadPool executors, which tie threads to the file handles they manage so that everything can be eliminated when a file's close or __exit__ is called (leaves a with block).
Here it is:
    def run(self):
        """
        Listens to the :py:attr:`~uproot4.source.futures.ResourceWorker.work_queue`
        and executes each :py:class:`~uproot4.source.futures.ResourceFuture` it
        receives (with :py:attr:`~uproot4.source.futures.ResourceWorker.resource` as
        its first argument) until it receives None.
        """
+       future = None
        while True:
+           del future   # don't hang onto a reference while waiting for more work
            future = self._work_queue.get()
            if future is None:
                break
            assert isinstance(future, ResourceFuture)
            future._run(self._resource)

Without the new lines, the previous future is still in scope while the self._work_queue.get() waits for new work because the assignment can't happen until the function on the right-hand side finishes. The future contains a basket array from the previous task, and if you have enough threads, those baskets add up. They will be deleted as soon as new work comes in, so this is not a permanent memory leak (the memory usage would not grow without bound), but it is the cause of your surprising behavior.
With the fix, here's what happens:
>>> import uproot4, gc
>>> branch = uproot4.open("test.root:tree/array", array_cache=None)
>>> gc.collect()
0
>>> # RSS is at 35812
>>> array = branch.array(library="np")
>>> # RSS is at 853356
>>> gc.collect()
57144
>>> # RSS is at 452968
>>> branch.close()
>>> del branch
>>> gc.collect()
147
>>> # RSS is at 448472
>>> del array
>>> gc.collect()
0
>>> # RSS is at 57588

Immediately after we read the array, there's 798 MiB of temporary data whose reference count is zero but the garbage collector hasn't gotten rid of it. I assume that's because of cyclic references, but I haven't investigated. Calling gc.collect() explicitly finds those objects and eliminates them, but in a memory-limited situation, the garbage collector would have been triggered automatically.
Closing the file doesn't release much, 4.3 MiB of various things related to those threads and file handles; maybe also the memory-mapped file. (I don't know how that gets counted in the RSS.)
But when we delete the array itself (which goes away immediately; no cyclic references), the 381 MiB array goes away, and we're left with a baseline that's 21 MiB higher than when we started. It may be that some libraries were imported in the process. On another run, this was 11 MiB. If I step through this procedure several times in the same process, we come back to this new baseline (and bounce around by ±10 MiB). Since it bounces around, this is in the noise.


Move -num-threads tests to single function

This is just an organizational change to keep all the tests for
-num-threads behavior in one place.

@DougGregor I don't feel super strongly about this but wdyt?

@swift-ci please test


Example function rot_11_layer in tutorial

It seems the rot_11_layer function in the tutorial is not doing what it is supposed to do.

In particular the second loop over the column terms does not conjugate by X operators.

Also the discussion of big endian needs to be better explained.

Also the reshaping for the fold function is wrong.

Fixed in #839


Implement kshape.jl

Requires rewriting kshape.py into julia.
Possibly appropriate to include in Clustering.jl

See v0.1 for now. Taken out of the main code with #98


Cannot import commonjs module from server component

When trying to import a CJS module from a server component I get an error in dev & production
Error when evaluating SSR module /src/components/App.tsx: failed to import "/src/components/foo.cjs"
|- ReferenceError: require is not defined
    at eval (/Users/tom.sherman/code/waku-example/src/components/foo.cjs:3:31)
    at instantiateModule (file:///Users/tom.sherman/code/waku-example/node_modules/.pnpm/vite@4.4.7/node_modules/vite/dist/node/chunks/dep-3b8eb186.js:55920:15)

Cannot render RSC ReferenceError: require is not defined
    at eval (/Users/tom.sherman/code/waku-example/src/components/foo.cjs:3:31)
    at instantiateModule (file:///Users/tom.sherman/code/waku-example/node_modules/.pnpm/vite@4.4.7/node_modules/vite/dist/node/chunks/dep-3b8eb186.js:55920:15)

Minimal reproduction: https://github.com/tom-sherman/waku-example/commit/77cf3501920bacdd2fd9a873d01abc4e848b373f

Hmm, I thought it works, but it doesn't. (btw, .cts nor .mts doesn't work and we need to deal with it.) The error is caused with Vite server file loading.
It's either a) there's a misconfiguration for Vite, b) some Vite plugins do something wrong, c) Vite doesn't support it (unlikely?) or d) there's a misunderstanding of mine.

btw, .cts nor .mts doesn't work and we need to deal with it

Fair point, this was the easiest way to trigger the error for me but I also was able to trigger the error by importing a CJS package from node_modules

@PerfectPan @nksaraf
Do you happen to know if ssrLoadModule supports .cjs ? (More precisely, import .cjs file from an ESM file loaded with ssrLoadModule.)
Or, if we are missing something.

No I think within your project you shouldn't really have cjs.. that way vite is ESM only.. it just knows what to do about cjs files in node_modules. But also I could be wrong to pinging @patak-dev

Note, although the example uses CJS inside my project, this error also appears for CJS inside node_modules.
IMO both of these behaviours are strange, in a Node.js ESM environment you are able to import a CJS module without issue.

in a Node.js ESM environment you are able to import a CJS module without issue.

Yeah.

To be honst, I am a newbie to vite. I just start learning vite from waku lol.
I tried to read the source about ssrLoadModule part and I found the basic rule is that if your code is not external based on  https://vitejs.dev/config/ssr-options.html#ssr-external , it cannot use commonjs, otherwise it will based on the node runtime behavior.
When ssrLoadModule being called, vite will transform your code first and wrap your code into a AsyncFunction.
Transform step
Basically it will resolve all the import an export statement and rewrite to a load function, you can see the code here https://github.com/vitejs/vite/blob/main/packages/vite/src/node/ssr/ssrTransform.ts#L126.
example:(more can see the test code https://github.com/vitejs/vite/blob/main/packages/vite/src/node/ssr/tests/ssrTransform.spec.ts#L10)
import foo from 'vue';console.log(foo.bar)
// after
const __vite_ssr_import_0__ = await __vite_ssr_import__(\\"vue\\");
console.log(__vite_ssr_import_0__.default.bar)

Wrap step
As you can see, for import statement it will be replaced by __vite_ssr_import, this is injected by the vite by wrapping the code into a async function.
https://github.com/vitejs/vite/blob/main/packages/vite/src/node/ssr/ssrModuleLoader.ts#L216

So the load result is an async function and will be executed by the user side.
It is a little like use the function closure to emulate a import / export environment instead of bundling all the code into a file like webpack. So as you can see, the final code is just a function, and the current situation is esm, node esm does not exist require or module, so it will throw error. A piece of code can prove my point.

However, there is a workaround or by design pattern provided by vite which is https://vitejs.dev/config/ssr-options.html#ssr-external.
When you external the whole third package, the resolve id will be the original instead of the parsed absolute path.
Once vite detect it is external package, vite will use the node loader to handle it. The code is here https://github.com/vitejs/vite/blob/main/packages/vite/src/node/ssr/ssrModuleLoader.ts#L265.

When you try to use import() call in esm, it can handle cjs correctly.
Another workaround I think maybe you can use @rollup/plugin-commonjs to convert cjs to esm which I have not tried yet.
Apparently I hide a whole lot of details like the plugin transformation or the optimize thing, I just read the basic code but I think it is enough to explain this situation.

And for your anothor issue https://github.com/tom-sherman/waku-desktop/pull/2, simply add external can not resolve all the problem.

You still will get following error:
TypeError: Cannot read properties of undefined (reading 'getVersion')

This is because the whole code is execute in a node environment instead of electron. So when you import you will only get the execute path of Electron just like the doc says
https://github.com/electron/electron#programmatic-usage.
Maybe you should encasulate a remote call for server and electron environment to communicate so that you can use the original electron ability.

In waku-desktop the "server" is the electron main process, so it should be running in that environment. Like you though I'm learning these things as I go, it's possible I've misunderstood something.

In waku-desktop the "server" is the electron main process, so it should be running in that environment. Like you though I'm learning these things as I go, it's possible I've misunderstood something.

I know what you mean. But waku will create a worker thread to handle the rsc file(https://github.com/dai-shi/waku/blob/main/src/lib/middleware/rsc/worker-api.ts#L12), so user code actually is running in another node environment.

Hmm, ok - yep that is an issue. I think changing this approach would solve my electron problem then.
I understand that RSC + SSR need to run in two separate processes (or at least react-server-dom-webpack assumes this to be the case) but Waku with SSR turned off shouldn't need to spin up any child processes when SSR is disabled I don't think.

Can anyone try with Waku v0.17.0? I don't expect it's solved, but would the error message be different?

TIP: This is one way to use cjs module:
https://github.com/dai-shi/waku/pull/362#pullrequestreview-1813829322

Another solution is to externalize the library. I think it's a valid workaround.
https://github.com/dai-shi/waku/blob/7398b45ed1c7858e3a24c04c35fd32efc2bc54b2/packages/website/vite.config.ts#L4-L10
Please note that this is DEV-only issue now.
@tom-sherman commonjs should work for PRD bundling.

Another solution is to externalize the library. I think it's a valid workaround.
https://github.com/dai-shi/waku/blob/7398b45ed1c7858e3a24c04c35fd32efc2bc54b2/packages/website/vite.config.ts#L4-L10
Please note that this is DEV-only issue now.
@tom-sherman commonjs should work for PRD bundling.

So, this looks like vite loadSsrModule cannot handle cjs bundle.
Related issue: https://github.com/vitejs/vite/issues/3024

As we have a workaround, I don't consider this a bug, but it would be nice to provide better DX.


webdav test failed for path handling incorrectly

=========================== short test summary info ============================
FAILED tests/test_async_delete.py::test_async_remove_all - pyo3_asyncio.RustPanic: rust future panicked: unknown error
FAILED tests/test_sync_delete.py::test_sync_remove_all - pyo3_runtime.PanicException: path /remote.php/webdav/15d5cf80-937e-42f1-bcd1-228f20e42a28/random_dir_a13dc0ab-12e1-4c6d-94f7-2fe37426b089/x/ doesn't start with root /15d5cf80-937e-42f1-bcd1-228f20e42a28/
======================== 2 failed, 48 passed in 56.99s =========================

cc @hoslo, would you like to take a look? Thanks!

This issue blocks https://github.com/apache/opendal/pull/4254

I'm working on this now.


KubeOps changes default JSON serialization settings

Describe the bug
KubeOps changes default JSON serialization settings. This might influence other components that use the default JSON serialization settings.
To Reproduce
Use a JSON serializer with the default serialization settings in a KubeOps operator and outside, e.g., in a unit test.
Expected behavior
The serialized output should be identical (or at least when disregarding line breaks and spacing).
Additional context
The way field names are cased by the JSON serialization settings configured by KubeOps also seems to be problematic in general. Here are some examples for field names serialized without and with the KubeOps settings:

client_app_id -> clienT_APP_ID
account_name -> accounT_NAME
session_paramenters -> sessioN_PARAMETERS

In fact, this prevents us from connecting to Snowflake using the Snowflake connector as described in #241, as running the Snowflake connector in a KubeOps operator changes the casing of the fields in its requests.

@anekdoti Can you compile and test against my changes? I believe I have removed all the points where the default settings are touched.
https://github.com/erin-allison/dotnet-operator-sdk/commit/c59c7f652ddd9daa4d5d262d14fe4c55c8d83c6d

By the way: do you have any thoughts about https://github.com/buehler/dotnet-operator-sdk/issues/239 ?

I can't speak to when @buehler will merge this in. I'll have a look at that one in the meantime and see if they can both be in the next release.

Thank you very much!

Hey @erin-allison and @anekdoti
I've been on holiday the past weeks, but this seems to be a nice catch!
Looking right into it


word change: online  -> interactive

LGTM

@stonemaster: As we are atm a bit in the lack of reviewers and it's a simple one word change I went directly ahead and merged.

:+1: perfect thanks!
Am 13.08.2016 14:40 schrieb "Sebastian Wilzbach"<EMAIL_ADDRESS>
@stonemaster: As we are atm a bit in the lack of reviewers and it's a
simple one word change I went directly ahead and merged.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

:+1: perfect thanks!

btw @stonemaster how about giving @ZombineDev also merge access? We require the CI to pass anyways and have force-push protection ;-)

I am fine with that. Currently travelling, do you habe the rights to do
that? Otherwise I'll do tonight. Will also reply later to to other great
@ZombineDev thread!
Am 13.08.2016 14:48 schrieb "Sebastian Wilzbach"<EMAIL_ADDRESS>
👍 perfect thanks!
btw @stonemaster https://github.com/stonemaster how about giving
@ZombineDev https://github.com/ZombineDev also merge access? We require
the CI to pass anyways and have force-push protection ;-)
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/stonemaster/dlang-tour/pull/400#issuecomment-239619234,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABZUwlZWbOKC64_WiCbAypePmP43lv6Nks5qfb0MgaJpZM4Jjq04
.

I am fine with that. Currently travelling, do you habe the rights to do
that? Otherwise I'll do tonight.

Great & I don't have the permissions to do so.
Btw if you are already in the web-interface you might want to define a new milestone for sth. like 1.1 into which imho at least these two issues fall:

dub (#29)
menu bar at left (#52)

:+1: will do
Am 13.08.2016 14:54 schrieb "Sebastian Wilzbach"<EMAIL_ADDRESS>
I am fine with that. Currently travelling, do you habe the rights to do
that? Otherwise I'll do tonight.
Great & I don't have the permissions to do so.
Btw if you are already in the web-interface you might want to define a new
milestone for sth. like 1.1 into which imho at least these two issues fall:

dub (#29 https://github.com/stonemaster/dlang-tour/issues/29)
menu bar at left (#52
https://github.com/stonemaster/dlang-tour/issues/52)

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/stonemaster/dlang-tour/pull/400#issuecomment-239619493,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABZUwhNC7--MwDvdI4Pczqz6W6BlVIh6ks5qfb6ZgaJpZM4Jjq04
.

stonemaster invited you to stonemaster/dlang-tour

@stonemaster @wilzbach Thanks guys! I feel honored!


add Finalize and ReportNow notifications

add Finalize and ReportNow notifications

To Test:
Report Ending Soon

Create an market and assigned yourself or another address as the desingated reporter.
Flash forward to designated reporting
You should see a notification in regards to reporting.

To Test:
Finalize Market

Create an market or have a position in an market.
Flash forward to market finalization
You should see a notification in regards to finalization of the market

This pull request fixes 1 alert when merging 04efe6138d316941fa6463994821554a200f6166 into 73fa1b3e22c62291aa3140b0590be8bca7f268f0 - view on LGTM.com
fixed alerts:

1 for Unused variable, import, function or class

Comment posted by LGTM.com


Fix TLS E2E test flakiness

Some of the E2E tests with TLS enabled were flaky on Darwin.
This PR:

Slightly modifies the way they are implemented to make them not-flaky
Modifies the options passed to SecPKCS12Import to make sure the creation of the required SecIdentity happens in memory only and doesn't use the keychain
Tolerates a broken pipe error when the server fails client cert validation during an mTLS handshake, as it's possible for Network.framework to expose this error instead of the actual SSL validation error on the client

@gjcairo FYI formatter is unhappy


[New game]: Debris Dodge

🎮 Game Request
Game logic and basic description
Move the ship to dodge the flying space debris of a passing cargo vessel that has strangely dropped wooden crates into orbit. Avoid getting hit by the boxes that inflict damage on the ship reducing the health. Avoid all boxes to survive and win the game!
Point down the features
Game points
A simple 2D/3D space-based object avoidance game written as a simple introduction to the Turbulenz HTML5 Game Engine.
 Thank you so much for your raising the issue💗
It’s all yours, you can come anytime again and make some contributions! 🚀
Alone, we can do little, but together we can do so much! 😇


[Bug]   BetterRoleColors Don't colored mention

Describe the Bug
This betterrolecolors plugin has don't colored as mention.
Screenshots

Discord Version
Stable,  PTB

This is not an issue with Betterdiscord, its an issue with a plugin.
Close this issue, and keep track of this one https://github.com/rauenzi/BetterDiscordAddons/issues/397


Errors on Vercel build

I am getting some strange errors when I try to import the site to Vercel. I use the default values. I had the site up until a few days ago when it would fail to build. I am not a developer am not sure where to start. Here is the Vercel log file.
I made a gist for a more readable format. Not sure why it looks bad in a preview.
https://gist.github.com/theapplegates/e3d2d065ec51b8531ce13631cfaaf0e4
01:17:37	Cloning github.com/theapplegates/crispy-parakeet-tailwind (Branch: main, Commit: 0e615ee) 01:17:38	Cloning completed: 652.690ms 01:17:38	Analyzing source code... 01:17:40	Installing build runtime... 01:17:45	Build runtime installed: 5222.614ms 01:17:49	Looking up build cache... 01:17:49	Build cache not found 01:17:51	Installing dependencies... 01:17:51	yarn install v1.22.10 01:17:51	warning package-lock.json found. Your project contains lock files generated by tools other than Yarn. It is advised not to mix package managers in order to avoid resolution inconsistencies caused by unsynchronized lock files. To clear this warning, remove package-lock.json. 01:17:51	[1/4] Resolving packages... 01:17:51	[2/4] Fetching packages... 01:18:04	info<EMAIL_ADDRESS>The platform "linux" is incompatible with this module. 01:18:04	info<EMAIL_ADDRESS>is an optional dependency and failed compatibility check. Excluding it from installation. 01:18:04	[3/4] Linking dependencies... 01:18:04	warning " ><EMAIL_ADDRESS>has incorrect peer dependency<EMAIL_ADDRESS><=17.0.1". 01:18:04	warning " ><EMAIL_ADDRESS>has incorrect peer dependency<EMAIL_ADDRESS><=17.0.1". 01:18:04	warning " ><EMAIL_ADDRESS>has unmet peer dependency<EMAIL_ADDRESS>|| ^5.0.0". 01:18:15	[4/4] Building fresh packages... 01:18:16	Done in 24.48s. 01:18:16	Detected Next.js version: 10.1.3 01:18:16	Running "yarn run build" 01:18:16	yarn run v1.22.10 01:18:16	$ next build && node ./scripts/generate-sitemap 01:18:17	info  - Using webpack 5. Reason: future.webpack5 option enabled https://nextjs.org/docs/messages/webpack5 01:18:17	info  - Checking validity of types... 01:18:17	Attention: Next.js now collects completely anonymous telemetry regarding usage. 01:18:17	This information is used to shape Next.js' roadmap and prioritize features. 01:18:17	You can learn more, including how to opt-out if you'd not like to participate in this anonymous program, by visiting the following URL: 01:18:17	https://nextjs.org/telemetry 01:18:17	info  - Creating an optimized production build... 01:18:22	warn - You have enabled the JIT engine which is currently in preview. 01:18:22	warn - Preview features are not covered by semver, may introduce breaking changes, and can change at any time. 01:18:32	info  - Compiled successfully 01:18:32	info  - Collecting page data... 01:18:33	info  - Generating static pages (0/30) 01:18:34	info  - Generating static pages (7/30) 01:18:34	info  - Generating static pages (14/30) 01:18:35	info  - Generating static pages (22/30) 01:18:36	Error occurred prerendering page "/blog/introducing-tailwind-nextjs-starter-blog". Read more: https://nextjs.org/docs/messages/prerender-error 01:18:36	TypeError: unsupported file type: undefined (file: /vercel/path0/public/static/images/twitter-card.png) 01:18:36	    at lookup (/vercel/path0/node_modules/image-size/dist/index.js:51:11) 01:18:36	    at imageSize (/vercel/path0/node_modules/image-size/dist/index.js:118:16) 01:18:36	    at /vercel/path0/.next/server/pages/tags/[tag].js:551:26 01:18:36	    at overload (/vercel/path0/node_modules/unist-util-visit/index.js:27:12) 01:18:36	    at node (paragraph) (/vercel/path0/node_modules/unist-util-visit-parents/index.js:56:27) 01:18:36	    at node (root) (/vercel/path0/node_modules/unist-util-visit-parents/index.js:67:75) 01:18:36	    at visitParents (/vercel/path0/node_modules/unist-util-visit-parents/index.js:29:26) 01:18:36	    at visit (/vercel/path0/node_modules/unist-util-visit/index.js:22:3) 01:18:36	    at /vercel/path0/.next/server/pages/tags/[tag].js:546:3 01:18:36	    at wrapped (/vercel/path0/node_modules/trough/wrap.js:25:19) 01:18:40	info  - Generating static pages (30/30) 01:18:40	> Build error occurred 01:18:40	Error: Export encountered errors on following paths: 01:18:40		/blog/introducing-tailwind-nextjs-starter-blog 01:18:40	    at /vercel/path0/node_modules/next/dist/export/index.js:31:1106 01:18:40	    at async Span.traceAsyncFn (/vercel/path0/node_modules/next/dist/telemetry/trace/trace.js:5:584) 01:18:40	    at async /vercel/path0/node_modules/next/dist/build/index.js:43:49 01:18:40	    at async Span.traceAsyncFn (/vercel/path0/node_modules/next/dist/telemetry/trace/trace.js:5:584) 01:18:40	    at async /vercel/path0/node_modules/next/dist/build/index.js:25:1475 01:18:40	    at async Span.traceAsyncFn (/vercel/path0/node_modules/next/dist/telemetry/trace/trace.js:5:584) 01:18:40	error Command failed with exit code 1. 01:18:40	info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command. 01:18:40	Error: Command "yarn run build" exited with 1

That error basically says you're missing the /twitter-card.png image,
though it does look like it's present in the GitHub directory.

That file missing caused all that? wow.
You were right, I did have the file. I then deleted it and uploaded it back and the site deployed fine.
Thanks for the help.

Yeah it's a sucky bug because you usually don't catch it in development
(unless you try building locally, and why would you), but it will
completely crash deployment.
On the plus side, it's a fantastic check against missing resources and
pages lol -- no 404
Best regards,
Derek
Derek R. Austin, PT, DPT, MS, BCTMB, LMT, CSCS
[image: Headshot of Doctor Derek smiling and waving at the camera.]
https://www.doctorderek.com/
Dr. Derek Austin 🥳
SEO & Web Performance Expert
https://doctorderek.com/ DoctorDerek.com https://doctorderek.com/
https://www.linkedin.com/in/derek-austin LinkedIn.com/in/Derek-Austin
https://www.linkedin.com/in/derek-austin
https://twitter.com/AskDoctorDerek Twitter.com/AskDoctorDerek
https://twitter.com/AskDoctorDerek
Read my blog:
https://doctorderek.medium.com/ Medium.com/@DoctorDerek
https://doctorderek.medium.com/
On Fri, Apr 9, 2021 at 1:55 PM Paul @.***> wrote:

That file missing caused all that? wow.
You were right, I did have the file. I then deleted it and uploaded it
back and the site deployed fine.
Thanks for the help.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/timlrx/tailwind-nextjs-starter-blog/issues/21#issuecomment-816891535,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAFZ3D32MYWFEALC47QTTODTH5ES3ANCNFSM42UINUUA
.

Hey @theapplegates I think that was a transient bug I managed to cause while trying to update the .gitattributes file in #19, so sorry for the inconvenience. Hopefully everything is working OK now.


WIP: add library versions to README #25

This is still work in progress (WIP). Please do not merge.

Recreated in fork


[Bug]: on hovering on default theme button the cursor should be pointer

Description
The default theme button have option to change the them of the page but the cursor is not pointer when someone is hovering on it.
Screenshot

@omicreativedev  can you assign this issue to me?


Hi, what can I use this model?

If I pre-train a model, how can I use to process the text? Is it same as word2vec or Glove?

Yes, the model gives the embeddings similar to word2vec and GloVe.

I got it. So if I want to use the model to encode the text to get its embeddings representation, can I directly use the embeddings.txt I trained?
Or is it similar to BERT used as an encoder? which can I fine-tuned on my tasks or using the best model you mentioned in ISSUE #16 as an encoder when I want to get the vector representation of texts?


Images without  any object

Hello!
I created my own COCO styled dataset, put all images descriptions in images part of json and annotations for images with objects on them in annotations part. But what to do with images without any object ? Should I annotate them somehow? Now mmdet ignores such images and I lose about 30% of data

Related to #425, #707

Thank you!


Make default exception handler response creation logic available from outside ExceptionHandlingMiddleware`

As far as I can see, Starlite does not log the exceptions, the only thing I see is:
INFO:     <IP_ADDRESS>:38622 - "GET / HTTP/1.1" 500 Internal Server Error

I would like that Starlite prints out the exceptions behind and their trace.
Thanks.

You can register handlers to exception types and http status codes, this is how I approach it.
You can see in the example app, this module: https://github.com/starlite-api/starlite-pg-redis-docker/blob/main/app/core/exceptions.py
And how that is registered to the application: https://github.com/starlite-api/starlite-pg-redis-docker/blob/78094bc4ea82839ba8703a41381da1e60f0b1d4d/app/main.py#L14

You can register handlers to exception types and http status codes, this is how I approach it.
You can see in the example app, this module: https://github.com/starlite-api/starlite-pg-redis-docker/blob/main/app/core/exceptions.py
And how that is registered to the application: https://github.com/starlite-api/starlite-pg-redis-docker/blob/78094bc4ea82839ba8703a41381da1e60f0b1d4d/app/main.py#L14

Thanks. I was thinking at something like that but instantiating a ExceptionHandlerMiddleware to take its default exc handler seemed a poor choice initially.

You can register handlers to exception types and http status codes, this is how I approach it.
You can see in the example app, this module: https://github.com/starlite-api/starlite-pg-redis-docker/blob/main/app/core/exceptions.py
And how that is registered to the application: https://github.com/starlite-api/starlite-pg-redis-docker/blob/78094bc4ea82839ba8703a41381da1e60f0b1d4d/app/main.py#L14

Thanks. I was thinking at something like that but instantiating a ExceptionHandlerMiddleware just for taking its default exc handler seemed a poor choice initially.

Yeh, that default_http_method_handler() method used to exist on the Starlite class until a recent release so it was a bit cleaner.. you don't need to do that, but I just wanted to get at the pre-existing response generation logic rather than roll my own.. we should probably just split that out into a utility func so that it can be accessed directly.

@devtud exception response utility now available and example code in the docs.
Thanks for opening the issue.

@all-contributors please add @devtud for bug


Enhanced Readability of Login Page

Hi @kondapalli19! Please change your target branch to "dev". We decided to push all latest changes to dev instead of main. The main branch is reserved for clean versions.
PS. We will add a CONTRIBUTING.md as soon as possible.


BUG: Images broken on Stage domain

Currently the request
https://www-stage.netcentric.biz/insights/2022/09/content-at-scale/_jcr_content/responsivecontainergrid/fixed_image.coreimg.720.jpeg/1664278885826.jpeg
sometimes returns 200 sometimes 302 redirecting to https://www.netcentric.biz.
We assume that the redirect is coming from Fastly as it stops if we remove the 404 redirect rule. As the error turns on and off per request we assume that one of the AEM instances is currently not health in some way, this needs to be investigated.

The issue is still present, opening the link in incognito mode will display the image, and other times opening it normally will display the homepage.

Check Publish instance on AEM on stage

Currently giving 404 error since the page is no longer published.
@chriscorr1 , unless we want to republish the page I think we can close this.

@nc-andreashaller, it seems the page is no longer published so I'm wondering if we can close this or we publish the page and check again?
Thanks!

AEM classic is completely phased out from nc.biz in the meantime so this is indeed not valid anymore.


No backtracking error, extend the choice operator with \.

We are sometimes sure that if a rule fails, the program can't be correct. It often happens after a specific keyword has been encountered. For example in Rust, a let must be followed by a declaration:
let-decl = "let" decl ";"

If we parse the "let" but that decl fail, it's useless to backtrack and can put a context-sensitive message here:
let-decl =
    "let" decl ";"
  \ "let" decl !";" { "A let declaration must be terminated by a semi-colon." }
  \ "let" .* { "A let must be followed by a declaration." }

We use the \ character because it's a bit like the dual of the choice: if the rule succeed it will fail (instead of succeed).

This approach is different from existing error reporting mechanism, it leaves the user to report himself the error (which is often desirable in compiler—for better error messages) instead of trying to generate automatically all the errors. Note that we still generate errors if the user didn't specify anything so it keeps the user free to improve the grammar later.


DBZ-1962 Introduce column.whitelist for the MySql connector

See https://issues.redhat.com/browse/DBZ-1962
Right only the Debezium Postgres connector supports the column whitelist, adding it for MySQL as well.
Todo

Update documentation for MySQL

Yes, when this PR was created I was looking already into exactly that. I came to the conclusion that's easier I merge it to my branch when it's done or when it's in master. It's easier for me to solve the conflicts in my t's quite complex change and there'll be duplicates in debezium-core etc. So once this is done, I'll resolve the issues.

@gunnarmorling @rk3rn3r OK, I'll try to fix the test and add the missing documentation. Not sure what's going on with the test, seems to be failing locally now as well.

@rk3rn3r @gunnarmorling Please check again, I've added documentation for the property, and fixed the test. Took me a while to realize that the test that I'd use as an example was actually failing for the wrong reason. Fixed that test as well.

Hey @jhuiting! like mentioned before, we are about to remove divisive language from Debezium.
We will merge #1750 soon that tackles the topic. Can you maybe update your PR based on the new naming and only use COLUMN_INCLUDE_LIST (column.include.list)? As soon as the other PR is merged I will notify you so that you can rebase your branch on the updated master.

@rk3rn3r Fine with me, I'll rebase as soon the other MR is merged.

@rk3rn3r where are we with this one? Is it good to go?

@rk3rn3r @gunnarmorling For me this PR is still good to go :-)

Thanks for the reminder! LGTM, will merge. Thanks, @jhuiting!

Just realizing that this referenced the original Jira issue that already had been closed a while ago (for Postgres). So I created https://issues.redhat.com/browse/DBZ-2508 just for the purpose of adding this to the change log, but I cannot modify the commit message anymore.


Goon with me!

Changes overhead and game text that says "Burn with me!" or "I will burn with you." to say "Goon with me!" and "I will goon with you." respectively.

This should be added to the meme-effects plugin


Integrate documentation site with vanity domain name

Background
We want to make this easily accessible to CDC personnel. Let's get a domain name for easy access.
Action requested
Make sure the workloads are accessible from outside the internal network. Can we use dibbs.cloud for this?
Acceptance Criteria

Decide on a vanity URL path
Ensure all certificates (HTTPS) and other required assets are in place
Test site for proper functionality.

Additional context
Add any other context or screenshots about the feature request here.

dibbs.azure or azure.dibbs.cloud?


[IOS] When press cancel btn will set current value back to picker state

I tested on our app and found that when user press cancel button. Value don't set back to picker component so value in picker will stay on last value.
I will test on android later if I found same problem I will create another pull request but I don't think it really need because in Android implementation. You use pure picker in component.

@ethaizone could you clarify this? Otherwise I'll be closing this issue.


JSON fails to parse with a trailing comment at the end of an array

Windows Terminal version (or Windows build number)
1.10.2714.0
Other Software
No response
Steps to reproduce
In the settings file, put a comment at the end of an object with a comma after it.
This fails:
"actions": [
  {},
  // ...
],

Dropping either the , or the comment makes it work.
Expected Behavior
Should be parsed as usual since both terminating commas and comments are allowed, or forbid trailing commas (assuming that comments are a desired feature).
Actual Behavior

Settings could not be loaded from file. Check for syntax errors, including trailing commas.

I suspect this is a jsoncpp bug. I know that they don't support a "trailing" comment, the comment has to be before some real value that it can attach the comment too. That being said, there's nothing open on their repo (or maybe I'm just bad at searching). We should probably drum up a minimal repro without the Terminal involved and post this upstream

I have a minimal repro example ready to go to file a bug at jsoncpp. But before I did, I did some searching and I cant find where trailing commas are permitted by the spec and most searches on StackOverflow indicate that they are not permitted.
https://stackoverflow.com/questions/201782/can-you-use-a-trailing-comma-in-a-json-object
From MDN:

Maybe a better error message could be produced?

Still happy to open the issue if you think it'd be prudent :)

Many JSON parsers allow trailing commas as an extension to the spec, including jsoncpp with the allowTrailingCommas option. This is helpful, because trailing commas make editing and diffing JSON files easier.

Right on. I will open an with them today then. Thanks for the clarification!

Here you go
https://github.com/open-source-parsers/jsoncpp/issues/1500

Many JSON parsers allow trailing commas as an extension to the spec, including jsoncpp with the allowTrailingCommas option. This is helpful, because trailing commas make editing and diffing JSON files easier.

VScode doesn't like Windows Terminal's defaults.json:

@nguyen-dows does this deserve an issue?


Added Some Text

Just added a line of text to the end of  WelcomeWall.md

Nice work!!! Congratulations and thank you for adding to the wall!


output HTML generated is not clean

Describe the Bug
When I copy the HTML from the source, it adds invisible characters as shown below:

Which package is affected (leave empty if unsure)
react-email
Link to the code that reproduces this issue
na
To Reproduce
copy paste html in a text editor like vscode
Expected Behavior
output html should be clean.
What's your node version? (if relevant)
No response

This is intentional, this is to fill the remaining characters for the Email's preview with white spaces. If we don't have this the remaining characters might be characters from the actual email and not the ones you are actually expecting.


Added the Boom error recognize for ACL.fetchEntity

Previously the aclQuery function were responding with BadRequest() for all errors,
Now first will be checked if the error is a Boom error (and returned as it is).

Version 3.0.2 published to NPM


[Attachments] In edit mode and send message input attachments height is too big (50 px, should be 38 px)

EPAM AI DIAL chat version
0.6.0
What happened?

Send a message with attachments
Edit the message sent -> Note that the height of the attachments are bigger in edit mode - 50 px. Please see the screenshot

Expected: attachments height in edit mode and send message input should be the same as in history chat - 38 px
Confidential information

[X] I confirm that do not share any confidential information

Verified on staging successfully


Update axe-core dependency

Changes
Related https://github.com/Qiskit/platypus/pull/1092
Implementation details

updated package.json directly to resolve package-lock.json conflicts
reviewed project locally
everything looks good to me

@techtolentino I will close this. The best way to update the axe-core dependency is update the branch through a merge when possible. In this case how we were having conflicts, what you can do is a rebase and that can be done automatically clicking in the next checkbox:


Close the State

Make the state close properly and drop the table when that happens.  There are no design changes here, so I'm going to merge.
A different version of the state might not drop the table and instead transfer it elsewhere for safekeeping.  Those variations will come later as will also optimizations.


How do I load and display PDF files in Compose for desktop?

Compose for Desktop does not have such a functionality out of the box and it is not planned to add it. You may look for third party solutions.

Please check the following ticket on YouTrack for follow-ups to this issue. GitHub issues will be closed in the coming weeks.


Ambari Content-Type is incorrect for blueprint add.

https://github.com/dmtucker/yaas/blob/master/yaas/blueprint.py#L53
        response = requests.post(
            config.href(
                '/api/v1/blueprints/{name}'.format(
                    name=blueprint['Blueprints']['blueprint_name'])),
            data=json.dumps(blueprint),  # `json=blueprint` would be better, but it doesn't work!
            params={'validate_topology': subargs.validate_topology},
            **config.requests_opts())

json=blueprint causes requests to use Content-Type: application/json which does not work correctly:
https://issues.apache.org/jira/browse/AMBARI-9016

I don't think this should be an issue against yaas. Presumably they won't deprecate this behavior in v1 of the ambari api, because it breaks backwards compatibility.

Well, it's not a bug that's forsure, but I think we should be tracking this somehow...


feat: update dsfr

update dsfr (and change how it is startup)

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


WP Plugin (Lido) Alternative

Already one active for Lido on Wordpress.
https://github.com/catenacap/Lido-stETH-Rewards-Plugin-wordpress-

@catenacap Thank you for sharing the resource. Can you elaborate on how the resource provided is related to the ethereum-validators-monitoring, and what is your proposal or issue?


RDK-36101 : Secure WebSocket client support in System Audio Player (SAP)

Reason for change: SAP now supports secured websocket connections.
Test Procedure: Connect cherry configured to cast in secured mode. Start
cast and check if audio can be heard.
Risks: Small
Signed-off-by<EMAIL_ADDRESS>

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

Blackduck scan failure details
Summary: 0 violations, 0 files pending approval, 1 file pending identification.

Protex Server Path: /home/blackduck/github/rdkservices/2695/rdkcentral/rdkservices

Commit: 67033de46daf6e88ee1b16e0cdcbbe8c736a9c73

Report detail: gist

Blackduck scan failure details
Summary: 0 violations, 0 files pending approval, 1 file pending identification.


readme docs: confusing help for aio config

From a user:
Proxy support section on this page https://github.com/adobe/aio-cli?tab=readme-ov-file#proxy-support says set environment variable (refer image) and same page has a section dedicated to setting up environment variable (refer image). That is causing confusion.
I was trying to setup environment variable as following which are both wrong.
aio config -e<EMAIL_ADDRESS>aio config -e HTTP_PROXY<EMAIL_ADDRESS>

Explanation
aio config is actually an alias for aio config list, but it also lists get, set, edit etc in its help. Users are confused here since aio config also lists the whole help for aio config (and its subcommands), as well as it being the alias for aio config list and showing the flags for that.

https://github.com/adobe/aio-cli-plugin-config/blame/6cc403bc59bfca43ba943bc1e1728bf4ab3c4f79/src/commands/config/index.js#L15
The command is doing something weird, in that it inherits from another command. It shouldn't do so, in this case it is just the list command (not an alias).
It should do something like this instead: https://github.com/adobe/aio-cli-plugin-app/blob/master/src/commands/app/index.js

Filed https://github.com/adobe/aio-cli-plugin-config/issues/155


[Suggestion] More config options

While there are a good amount of config options already, I feel like there are a few key ones missing that would really help make the mod better:

Replace DT seed drops w/ vanilla saplings.  There's already an option to place DT saplings when using vanilla saplings, but that doesn't change what drops from trees.

Disable leaf sinking/movement slowing.  This is personally the biggest annoyance with the mod for me, and an option to disable it would be very, very nice.

In the future, if separate Cacao trees are added, a way to disable those and just have cocoa beans planted on Jungle trees would be nice.  I mentioned this on the CurseForge page, but I feel like the mod works best as a "tweak" to vanilla, rather than a mod that adds its own actual content, and I personally wouldn't want to see it change from that.  But, if it does, config options to disable that kind of stuff would be very much appreciated.

Hey Forstride.  Much respect for your beautiful BoP mod which I am very much enjoying.  Here's my responses respectively to the suggestions you dropped. Though I'm afraid you may not like what I have to say.

I don't want to make trees drop saplings because it's nonsensical.  Trees don't drop saplings(especially voluntarily) and it boggles my mind that Mojang thought it was an appropriate game mechanic.  The seeds provide a nice separation between what is a dynamic tree and what is not.  I was asked some time ago to optionally make the vanilla saplings plant dynamic trees for people that wanted to force the server into being completely dynamic trees.  I agreed to add that feature since I saw the added value for map makers and servers but I see no practical purpose for the corollary.

Running around on the tops of trees doesn't make any sense.  I added this mechanic to prevent players from avoiding the perils of a forest by walking over the top of it where hostile mobs can't spawn.  This makes the game more fun because you must brave the forest in your adventure.  This was also added to balance the game play since vanilla "forests" don't typically have the trees close enough together to allow a player to travel large spans without ever touching the ground(except maybe roofed forests).  Since my mod creates forests that are effectively a large blanket of leaves I thought that this was the best way to balance it.  I have considered adding a footwear enchantment called "leaf hopper" that would allow a player to run on the tree tops but I have been shy about doing so since I already stated that I wouldn't add enchantments to a tree mod due to project scope.

If Cacao trees are added I will treat it the same way I did as apple trees.  There's an option that totally removes apple trees and makes oak trees drop apples like you would expect.  Also because of project scope this would be the final fruit tree addition to DT.  I have no intention of adding more content and I consider Cocoa products from a specific species to be a valid addition.  I was presented with a good argument for their addition and since I was on the fence about it I let the community decide with a vote.  But since this isn't a democracy it's ultimately up to me if I follow through with it.  I take the project scope seriously because I want the mod to actually be finished one day and be feature complete.

The first suggestion was mainly to keep things closer to vanilla for my own personal tastes.  I already have stuff like the separate apple trees and podzol generation disabled, and while the seeds are more realistic, I'd personally like something more "familiar" I guess.  Although I suppose I could technically use a resource pack to accomplish this.
As for normal movement on trees, it's less about evading mobs (Which I've never really considered TBH.  I even have a long path going through a dense and dark forest that I spent a lot of time working on and lighting up with torches), and more about it being annoying if you need to jump up on a tree to get a different angle while building something, or just want a nice vantage point to look out across a forest, which is something I've commonly run into in the past.  It really doesn't impact the actual gameplay for me, so it just ends up being an annoyance if I end up going on top of a tree for whatever reason.

Your hard work has made my play experience so much better that I feel that I should return the favor and make my mod with fewer annoyances for you.  I'll add a config option to disable the impeded travel on leaves during the next release cycle.  I'll also add an event so other mods can easily create an enchantment for the same reason.

Added config option to disabled enhanced leaves movement in dev version.


Bug 1881636: Switch network crds to v1 and set preserveUnknownFields to false

Closing as there's already https://github.com/openshift/api/pull/749/files


feat(Checkbox): add support of htmlFor

Fixes #2274.

Codecov Report

Merging #2293 into master will increase coverage by <.01%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master    #2293      +/-   ##
==========================================
+ Coverage   99.73%   99.73%   +<.01%
==========================================
  Files         152      152
  Lines        2656     2657       +1
==========================================
+ Hits         2649     2650       +1
  Misses          7        7

Impacted Files
Coverage Δ

src/modules/Checkbox/Checkbox.js
100% <100%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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

Released in<EMAIL_ADDRESS>


PEX crashing?

Hi there!
So I installed PEX version 1.23.3 on my server and it was all good but then I restarted my server and it just stopped working. No commands worked. It didn't come up in /? or /pl.
Here is my console log.
https://pastebin.com/MFdpMw9P
Here is the Spigot version I am using.
https://prnt.sc/fbb9gn
Are you able to help me at all?

Go over here and get PEX 1.23.4 for your server.
Stop the server, install this .jar file, delete the old 1.23.3 .jar file, start the server, and see if you get the same error.

No reply from @xMattRA in 7 days.  Issue must be resolved.


Addressing a bug with escaping slashes on Windows

The work was primarily in the burnin module and not the adapter. Unit tests updated accordingly.

At some point, you should rebase onto the master, so your PRs don't show so many previously accepted commits.


Application doesn't close on Mac OSX

After quitting with CMD+Q the main window closes but the application does not quit and has to be force-quit manually.

Connection pool needed to be properly shutdown so it didn't tie up application threads


DBZ-414 Binlog reader position can be overwritten when offset exists

https://issues.jboss.org/browse/DBZ-414

Hey @jpechane, can you perhaps add a few words on describing the change on a high level? Nothing fancy needed, just to get the picture a bit quicker. Thx!

If you are using Debezium and lose a part of binlog either beacuse of log or Debezium not running for some time you need to re-execute the snapshot. It might be undesirable in some situations.
On use case for this feature is just skip over the gap and continue processing from the first available binlog. Another is to switch to schema_only snapshot mode to re-initialize  schema and then continue with processing from end of binlog.
If this setting is enabed then offsets are dis-regarded and force rewind to either start or end of binlog is executed and reading continues from that point on.

I like the rewind feature as recovery mode option.
It would be nice to expand the option to rewind to a specific binlog position, assuming an external tool is keeping track of this, rather than just start and end.

It would be pretty easy to do it for file/offset combo but we need to think about how to handle GTIDs.

Once again, we have to be careful about specifying binlog filename and position within the configuration. Connect workers can restart a connector at any time, and if the binlog position is specified in the config the connector will jump to that position even if had already done that already. And you can’t rely upon users to update the config in time before the next rebalance since you don’t know when that will be.
The best way to set the binlog position is via the proposed offset reset tool. (I don’t have ready access to the kip at this time).

Do you guys think a heartbeat approach might work better for https://issues.jboss.org/browse/DBZ-414?
Simply commit a heartbeat message to a dummy topic at some pre-defined frequency which is less than the binlog retention period, like once per hour or day. Wouldn't that commit the latest offset to kafka connect even if no messages have been read during the retention period? Just thinking about preventing the scenario from occurring rather than how to workaround it when it does.

The heartbeat will prevent the situation to happen but it does not mean that this is no longer needed - it is still good for recovery.

Agreed that the heartbeat approach would be great for the use case described in DBZ-220.
About the issue here, can't we simply do a new snapshot (just schema) when being in "schema_only" mode and detecting that we can't continue from the previously stored offset? We'll miss data changes after the connector was shut down before, but that's just the nature of that snapshotting mode.
In terms of the discussed offset management tool, there's KAFKA-4107 and @rhauch's KIP-199.

@gunnarmorling It is exactly what you are talking about - just I provide you with either ignoring all that is in binlog or process at least the events that are stored in binlog.

Can I get an update on this PR's status? It is one recovery option to workaround the KC defect and it could be useful for other situations. If it's going in will definitely impact implementation for https://issues.jboss.org/browse/DBZ-443 which I'm starting to work on shortly

I don't think we should implement this feature in the suggested way, for the reasons descibed by @rhauch above. It seems to big of a risk to process specific binlog sections twice after restarting/rebalancing without the user knowing. Hence I'd prefer to close this PR.
I still think this is a more desirable approach:

simply do a new snapshot (just schema) when being in "schema_only" mode and detecting that we can't continue from the previously stored offset. We'll miss data changes after the connector was shut down before, but that's just the nature of that snapshotting mode.

Let's close it for know. If there will be more request from community preferring event re-processing to event gaps we can reconsider the decision and approach.


Fix panel callback connection

@bugobliterator could you please elaborate why this change is necessary? It seems to work well now

problem happens when you add more panels, the current piece of code will open the last panel on the list whatever menu option you select

Thanks.


Conflict with other lib (ngDialog)

Hi,
I get an error with the ngDialog lib, when closing my dialog modal :
Cannot read property "classes" of undefined, line 7255
--> forEach(cache.classes, function(status, className) {...}
After looking a bit into ngDialog, it seems that the problem is caused by calling $destroy on the $dialog's scope (line 107 if you are curious ;) ).
If you know a way to get around this bug, I'm all ears :)

I just reexperienced this issue with a completely different situation.
I think it has to do with the fact that you include your own ngAnimate module, and it is not up to date with the latest version of angular. Could you do maybe a light version of ngFx that doesn't include modules/libs that we can find elsewhere, with bower for instance?

got it, it is un bundled now :+1:


Allow recursive removal of blocks

Also allow removing based on just the filename (will require a full database scan).

This is done in #23.


Terminal should dodge the onscreen keyboard slightly better

Expected behavior
when onscreen keyboard is shown and terminal's window maximized then text in the terminal should not be printed under the kb, instead its working area should be shrinked to the visible area
Actual behavior
the terminal does print text in the area I don't see that is covered with onscreen keyboard

Interesting. That's a good find.
How this would play with split panes, I'm not sure. Say you've got a horizontal split in the terminal [-]. You bring up the onscreen keyboard - it entirely covers the bottom pane. What would we do then?
maybe related: #8623 - if we did have a scroll-forward (even just one page), then you could at least scroll forward to have the text be visible.

No, bot.. don't close it.. it is important feature.. @zadjii-msft -help :blush:

Derp wait yea, this doesn't need author feedback - but it does need some feedback I suppose. I'm not really sure how best to handle this scenario. That being said, it's something we should to eventually.

I'm going to put this on the backlog, but it needs design.

Does anyone know of an app that does this well? Trying to look for inspiration but it seems like Outlook, Word, Notepad all print text under the on-screen keyboard


[AUTOMATED] Update release.yml with npm environment

This PR was created automatically, as part of the efforts to address PHC-2100.

@anthonyroach I requested you for review because you are a top contributor. See https://lifeomic.slack.com/archives/C3NA7J4E8/p1642178097007500 for more info.

Closing this PR since my commit came through unsigned. I will re-open a PR with a signed commit, and re-request review.


90% - Support all logging levels with threshold

Previously this service only supported 'debug' and 'error' levels and it was a boolean thing.  Now it supports trace, debug, info, warn and error with a new config property of LOGGING_THRESHOLD to determine which log entries get created (LOGGING_LEVEL has been removed to avoid confusion).  If no LOGGING_THRESHOLD is passed on the config object it will default to 'info' level.
This now also does the correct 'console.xxx' method for the log level in question so you get the correct icon in the browser console.

:+1:


File is not edited yet shows up with * as if being changed

Originally submitted to Google Code by markhu on 9 Jan 2012

Linux, Ride .4, Python 2.7.2
Going into a file in Text Edit mode , click on the text pane , and hitting a non modifying button (ctrl, arrows, etc) causes the resource in the navigation pane to show up a * next to it as if it's been modified.

Originally submitted to Google Code by markhu on 9 Jan 2012

oops I guess I should of created this in robot framework area not selenium library .. sorry ..

Originally submitted to Google Code by @pekkaklarck on 10 Jan 2012

Correct place is actually RIDEs own tracker.


Parameter does not be replaced in the url

Hi, I had a taste for Janus, and the config is
{
    "name" : "usergroups",
    "active" : true,
    "proxy" : {
        "listen_path" : "/v1/usergroups/{groupId:[\\da-f]{24}}",
        "upstreams" : {
            "balancing": "roundrobin",
            "targets": [
                {"target": "http://service:8080/v1/usergroups/{groupId}"}
            ]
        },
        "methods" : ["GET"]
    }
}

but I got
{
    "code": 400,
    "msg": "resourcequotas \"{groupId}\" not found",
    "data": null
}

It seems that parameter does not be replaced in the target usl?

@italolelis can you answer my question?

It seems the problem of docker images, when I use released binary file, it works well.

@m3ngyang we've updated the docker image to the latest version. There was a problem with it and now should be ok. Would you mind testing it again?


UI : Added auto-generated password on page load

Describe your changes :

Issue : Closes #8736
Added auto-generated password on page load in password field

Type of change :

[x] Bug fix
[x] Improvement

Frontend Preview (Screenshots) :
https://user-images.githubusercontent.com/66266464/208228499-403fbb29-d64b-4d93-a6ae-431ba7f38381.mov

Checklist:

[x] I have read the CONTRIBUTING document.
[ ] I have commented on my code, particularly in hard-to-understand areas.
[ ] I have added tests that prove my fix is effective or that my feature works.
[x] All new and existing tests passed.

Reviewers:
@open-metadata/ui

@chirag-madlani Changes made as per comments!


Workaround the Delphi XE3 Compiler Bug

Putting the initialization section into a third unit (DUnitX,Init) seems to solve the problem.
The extra unit needs to be added only for XE3 projects.
This workaround fixes #117.

WDYT this workaround @vincentparrett?

Interesting, it looks exactly like what I tried, but when I did the class constructor in TDUnitX didn't get called (probably another compiler bug), but it does seem to work in your pr! Merging, thx.


Gush Command to checkout a pull-request locally

Instead of merging the pull-request, sometimes you would rather want to checkout the PR to a local branch. This happens in teams with Private repositories for example.
pull-request:checkout 1111

This is a must :older_man: :+1: well thought!


hook cmd arguments not being customized

It seems I've missed something simple...
I've defined a hook:
[[hook]]
name = "dns-01-hook-myauth"
type = ["challenge-dns-01"]
cmd = "auth.sh"
args = [ "{ identifier }", "{ proof }" ]
It's literally being run as:
auth.sh "{ identifier }" "{ proof }"
instead of:
auth.sh example.com Kdi3k28dksld8...

You need to write two curly brackets:
 args = [ "{{ identifier }}", "{{ proof }}" ]

For more information, you can refer to the MiniJinja syntax.
I was wondering what could have lead you to put single curly brackets and found out there is one example in the manual that contains this error. I'm fixing this right now.

Oops...  I was reading an old version of the manpage from:
https://cdn.homespan.com/cgi-bin/man.cgi?section=5&topic=ACMED
Thanks

I see. The template engine changed a year ago, so as you mentioned there may still be outdated documentation in the wild using the previous syntax.


PR Preview env can't be created with env variables

Before opening, please confirm:

[X] I have checked to see if my question is addressed in the FAQ.
[X] I have searched for duplicate or closed issues.
[X] I have read the guide for submitting bug reports.
[X] I have done my best to include a minimal, self-contained set of instructions for consistently reproducing the issue.

App Id
d3pwsdrqmxpguf
Region
eu-west-2
Amplify Hosting feature
Backend builds, Environment variables
Describe the bug
I'm using the amplifyPush --simple command to build/create my backend env for Preview environments.
The build fails because of env variables and --yes flag - known issue
CREATE_FAILED               functiongetOpenViduToken             AWS::CloudFormation::Stack Tue Apr 19 2022 09:25:45 GMT+0000 (Coordinated Universal Time) Parameters: [openviduUrl] must have values
However, according to documentation to fix the issue you have to checkout the env and push it manually from local machine.

I copy the command from Amplify console amplify pull --appId d3pwsdrqmxpguf --envName prcjb
an empty env is pulled from cloud
I can't update the function nor push it back to cloud (because there're no categories)

Expected behavior
I'd be great if the amplifyPush --simple (amplify push -y) command did carry over the env variables and secrets from target or current env without prompting).
Also I don't know why the failing env is empty why I pull it. Without it there's no way how to fix the env variables.
Reproduction steps

Create lambda function that uses some env variable.
Define value for current environment and push to cloud - check if value is included in team-provider-info.json
Setup PR Previews for current branch in Amplify Console - Create new backend environment for every Pull Request
Create new branch from current branch
Make some changes, add dummy file etc.
Push changes to remote and create PR for the new branch.
PR env build should fail - try to pull the env from cloud.

Build Settings
version: 1
backend:
    phases:
        build:
            commands:
                - amplifyPush -e "${AWS_BRANCH}" --simple
frontend:
    phases:
        preBuild:
            commands:
                - npm ci
        build:
            commands:
                - npm run build-prod
    artifacts:
        baseDirectory: dist/stardio-pro
        files:
            - "**/*"
    cache:
        paths:
            - node_modules/**/*

Additional information
No response

I'd also be good if the parameter  -e "${AWS_BRANCH}" was actually respected and used for the new env name. Right now there's USER_BRANCH variable for each PR env that's overriding the -e input.
I know amplify env name has a strict naming conditions (no dashes, no uppercase, 10 chars) but that can be stripped out from the input.
I'd make more sense to have the envs named after git branches or PR numbers.

Hi @vojtech-simko 👋🏽 thanks for providing details and reproduction steps. We're taking a look into this issue.

any update on this ??

I guess that for now as soon we have environment variables or secrets on lambda functions the preview won't work? 😢
It would be great to understand if this is something the AWS team has ideas on how to fix it, although seems like a pretty heavy lift

We have a fix for this feature. There are two new env variables for AWS_PULL_REQUEST_SOURCE_BRANCH
and AWS_PULL_REQUEST_DESTINATION_BRANCH that can give you the required information during the build. https://docs.aws.amazon.com/amplify/latest/userguide/environment-variables.html#amplify-console-environment-variables
We have also updated AWS_BRANCH for pull requests to represent the current PR (E.g PR-1).


Add and enforce a style guide

Fixes #95.

Define an automatically-enforceable style guide by adding an .editorconfig file (configures shfmt).
Replace Vim modelines by Bash shebangs so that shfmt can detect all shell scripts as such, and update docs/package.md accordingly.
Add make lint target to run shellcheck and report errors.
Add make format target to run shfmt and report errors.
Add make format-fix target to run shfmt and automatically fix errors.
Fix existing formatting & lint errors.
Check format and lint conformity in the pr workflow.
Mention the style guide in docs/contributing.md

conceptually, style is something that will be non-destructive to fix (since style is solely about formatting) and should not prevent a merge. instead, the fixes should be integrated into PR process and auto applied or run as a bot on some regular basis.
it does make sense for lint to stop a PR, though (and fixes can be auto suggested / applied)

conceptually, style is something that will be non-destructive to fix (since style is solely about formatting) and should not prevent a merge. instead, the fixes should be integrated into PR process and auto applied or run as a bot on some regular basis.

Agreed. We ought to use lint-action in the long run, but we first need to wait until it supports shfmt/create a PR to add support for shmt.

why is a check still running?

It looks like this is because I renamed the previous “Check that a pull request correctly builds” check to “Check that it builds without error” in this PR, and therefore I need to update the branch protection settings.


Feature Request: Sort on checked/unchecked checkboxes

Hi Jonny,
Great library!
Have you considered introducing the ability to automatically move checked list items to the top of the list?  (e.g. http://jsfiddle.net/scSYV/2/).
Keep up the great work!
Gary

@javve your ref. link does not work: where can we find that now?
Thanks


Using outExtension causes esbuild to copy original and processed file.

Version
2.3.2
Platform
Windows 11 24H2 (26100.1742)
What steps will reproduce the bug?

Use outExtension to append 'min' suffix to a minified bundled JS file:

site.use(esbuild({
  options: {
    bundle: true,
    minify: true,
    outExtension: { ".js": ".min.js" },
  },
}));

How often does it reproduce? Is there a required condition?
As long as outExtension is being present and used, it will cause this problem.
What is the expected behavior?
It's supposed to process a .js file as usual, and then rename it to .min.js during processing.
What do you see instead?
Instead, the user receives both .min.js and the original .ts file, when without outExtension the user only receives .js.
Additional information
It would've been possible to prevent by being able to edit outfile but that causes the user to have an error where esbuild claims the usage of outfile and outdir cannot be utilised at the same time, despite outdir having never been specified by the user.

Done 13796a7fe66c007aaca6a23084233247a46235fb
It will be available in the next version (2.3.3)


PCM 1525 - added feature to get external tracker updates

@vrathee @renujhamtani Please review

Looks Good!


Samples: ResourceTrees and ResourceSharing

<x:String x:Key="fontSize">Large</x:String>
FontSize="{StaticResource fontSize}"
:
Unhandled Exception:
System.InvalidCastException: Specified cast is not valid. occurred

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

ID: 2f1236a9-aeda-7a4d-4023-3eb60067bb80
Version Independent ID: d748dc44-7b87-6a53-d7a2-a60b4ec0a6bd
Content: Summary of Chapter 10. XAML markup extensions - Xamarin
Content Source: docs/xamarin-forms/creating-mobile-apps-xamarin-forms/summaries/chapter10.md
Product: xamarin
GitHub Login: @davidbritch
Microsoft Alias: dabritch

Hi @joewang1967
I just tried the ResourceSharing sample in VSMac, updated it to Xamarin.Forms 3.5 and it worked fine on both iOS and Android (the Button text was displayed with a Large font size).

Thank you very much!
Sent from Mailhttps://go.microsoft.com/fwlink/?LinkId=550986 for Windows 10

From: David Britch<EMAIL_ADDRESS>Sent: Wednesday, February 20, 2019 10:48:50 PM
To: MicrosoftDocs/xamarin-docs
Cc: joewang1967; Mention
Subject: Re: [MicrosoftDocs/xamarin-docs] Samples: ResourceTrees and ResourceSharing (#1415)
Hi @joewang1967https://github.com/joewang1967
I just tried the ResourceSharing sample in VSMac, updated it to Xamarin.Forms 3.5 and it worked fine on both iOS and Android (the Button text was displayed with a Large font size).
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/MicrosoftDocs/xamarin-docs/issues/1415#issuecomment-465607674, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AtaWS6mPobktL2CMX224eYh5JxX4RDFlks5vPWBSgaJpZM4a6qWN.

Well this exception is back in Xamarin Forms <IP_ADDRESS>3. Anything we can do about it?


default page doesn't highlight which model is selected

Issue: main page doesn't indicate whether i'm currently in bananas correct or broken
Suggestion: Should be consistent with the design that you build for after selecting correct or broken (where that item is highlighted blue to tell me that's where I currently am)
Screenshot:

thanks, I think this is fixed now


Implement extended option headers

According to RFC7252 option header may be extended by additional bytes (sect 3.1).
This patch implements parsing and formatting extended Delta and Length attributes.

I pushed this, but there is a failing test.  Any chance you could look at it?
(it seemed to have already been failing, so you're not obligated)


Create label class

In this PR, I did the following

[x] Created a Label class with an association to the Item class (in a separate .rb file).
[x] All Label class properties visible in the diagram should be defined and set up in the constructor method.
[x] Implement add_item method in the Label class

Take an instance of the Item class as an input
Add the input item to the collection of items
Add self as a property of the item object (by using the correct setter from the item object)

Approved 🥳
Hi @KanzaTahreem
Great work nailing the project 💪
Your project is COMPLETE! There is nothing else to say other than... it's time to merge it! 🚀 Congratulations!

Highlights
There are some excellent points that you have made perfectly. Keep up the great work! 👏

Great work using the right Git Flow  👍
Commits are descriptive, good work.
Linters are working properly, great job! ✅
the requirements are good

Optional suggestions
Every comment with the [OPTIONAL] prefix is not crucial enough to stop the approval of this PR. However, I strongly recommend you to take them into account as they can make your code better.
N/A
Cheers and Happy coding!👏👨💻💻
Feel free to leave any questions or comments in the PR thread if something is not 100% clear.
Please, remember to tag me in your question so I can receive the notification.
Please, do not open a new Pull Request for re-reviews. You should use the same Pull Request submitted for the first review, either valid or invalid unless it is requested otherwise.


On iOS <= 9.x, fallback to no-caching

On iOS 9 and below IndexedDB does not work. When capp-cache runs on those browsers it should still load the zapps as if there is no cache without accessing indexedDB. This way users will still be able to run the application, albeit without cache.

Seems that as long as there is just one object store in the domain, iOS 8+ should be OK. Will run some tests.

Use local storage?


DuoLingo Word Lists

Each lesson (i.e. Games, Hospital, Airport, etc.) in DuoLingo has associated word and grammar lists.
They are already nicely partitioned and seem like a good thing to add to Cardbuilder.

I was wondering how to access all of the vocabulary without unlocking all lessons and found this: https://forum.duolingo.com/comment/31074292/List-of-Vocabularies-for-Language-Courses-of-Duolingo

I was wondering how to access all of the vocabulary without unlocking all lessons and found this: https://forum.duolingo.com/comment/31074292/List-of-Vocabularies-for-Language-Courses-of-Duolingo

The biggest problem with these is that they're unlabeled. German also doesn't have the grammatical gender of the nouns indicated and each form of the verb is listed separately. I haven't checked the other languages I've studied.

@hmorri thanks for posting this. Extracting word lists from these form posts is definitely doable (information like gender can be looked up later; that's basically what Cardbuilder is good for). I just wish there were an official Duolingo API for this.

Looks like the same data is on github now, which is a lot easier to deal with. https://github.com/liuch/duolingo_courses_overview


allow passing an xarray array as f

then use sel(..., method='nearest')

This could maybe have it's own indexer analogous to what is being added in #141. But now that this was added to hyperslicer in #138 I don't think this is as important


bug: heliaInstance.libp2p.addEventListener('peer:discovery', cb) calls cb twice for same peer

Using libp2p 0.42.2 via<EMAIL_ADDRESS>Peer discovery should only emit once, unless we are in fact discovering the peer twice via different methods somehow.
Either way, once a peer is discovered, we should cancel any in-progress operations and emit that we discovered the peer only once.
While working on a new helia-script-tag example (via #43), I was able to get the following event log:
1263ms - Created LibP2P instance
1263ms - Created Helia instance
2248ms - Discovered peer QmNnooDu7bfjPFoTZYxMNLWUQJyrVwtbZg5gBMjTezGAJN
2252ms - Discovered peer QmNnooDu7bfjPFoTZYxMNLWUQJyrVwtbZg5gBMjTezGAJN
2252ms - Discovered peer QmQCU2EcMqAqQPR2i9bChDtGNJchTbq5TbXJJ16u19uLTa
2252ms - Discovered peer QmQCU2EcMqAqQPR2i9bChDtGNJchTbq5TbXJJ16u19uLTa
2253ms - Discovered peer QmbLHAnMoJPWSCR5Zhtx6BHJX9KiKNN6tpvbUcqanj75Nb
2254ms - Discovered peer QmbLHAnMoJPWSCR5Zhtx6BHJX9KiKNN6tpvbUcqanj75Nb
2254ms - Discovered peer QmcZf59bWwK5XFi76CZX8cbJ4BhTzzA3gU1ZjYZcYW3dwt
2254ms - Discovered peer QmcZf59bWwK5XFi76CZX8cbJ4BhTzzA3gU1ZjYZcYW3dwt
74326ms - Connected to QmbLHAnMoJPWSCR5Zhtx6BHJX9KiKNN6tpvbUcqanj75Nb
77805ms - Connected to QmcZf59bWwK5XFi76CZX8cbJ4BhTzzA3gU1ZjYZcYW3dwt
135298ms - Disconnected from QmbLHAnMoJPWSCR5Zhtx6BHJX9KiKNN6tpvbUcqanj75Nb
138782ms - Disconnected from QmcZf59bWwK5XFi76CZX8cbJ4BhTzzA3gU1ZjYZcYW3dwt

replaces https://github.com/ipfs/helia/issues/56

You can see this in action at https://codesandbox.io/p/sandbox/helia-script-tag-g420c3, with PR to helia-examples repo at https://github.com/ipfs-examples/helia-examples/pull/13


Enhance Post Generator Prompts

Description: Enhance the existing LinkedIn Post Generator by implementing advanced prompt engineering techniques to generate more specific, accurate, and relevant LinkedIn posts based on user-provided info.
Priority: Moderate
Tasks:

Analyze Current Prompts: Analyze the current prompts used by the LinkedIn Post Generator and identify areas for improvement in terms of specificity, accuracy, and relevance.
Research Prompt Engineering Techniques: Research and explore various prompt engineering techniques that can be applied to improve the quality and relevance of generated LinkedIn posts. This may include techniques like:

Zero-shot prompting: Providing clear instructions and examples in the prompt without fine-tuning the model.
Few-shot prompting: Providing a few examples of desired output in the prompt to guide the model.
Chain-of-thought prompting: Encouraging the model to reason step-by-step before generating the output.

Develop and Test New Prompts: Develop and test new prompts incorporating the researched prompt engineering techniques. This involves experimenting with different prompt structures, keywords, and instructions to guide the model towards generating more specific, accurate, and relevant LinkedIn posts.
Evaluate Prompt Performance: Evaluate the performance of new prompts by comparing the quality and relevance of generated posts against the previous prompts. Use metrics such as user feedback, post engagement, and relevance to target audience to assess improvement.
Integrate Best-Performing Prompts: Integrate the best-performing prompts into the LinkedIn Post Generator, replacing or supplementing the existing prompts.

Acceptance Criteria:

[ ] The LinkedIn Post Generator generates more specific and accurate LinkedIn posts based on user summaries.
[ ] Generated posts are more relevant to the target audience and context of the user's summary..

Notes:
https://www.promptingguide.ai/

Completed by #25


Update tests for subannual time slices

iiasa/ixmp#264 broke the message_ix test suite:
________________________________ test_run_clone ________________________________

tmpdir = local('/tmp/pytest-of-khaeru/pytest-246/test_run_clone0')

    def test_run_clone(tmpdir):
        # this test is designed to cover the full functionality of the GAMS API
        # - initialize a new ixmp platform instance
        # - create a new scenario based on Dantzigs tutorial transport model
        # - solve the model and read back the solution from the output
        # - perform tests on the objective value and the timeseries data
        mp = Platform(driver='hsqldb', path=tmpdir / 'db')
        scen = make_dantzig(mp, solve=True)
        assert np.isclose(scen.var('OBJ')['lvl'], 153.675)
        assert scen.firstmodelyear == 1963
>       assert_frame_equal(scen.timeseries(iamc=True), TS_DF)
E       AssertionError: DataFrame are different
E
E       DataFrame shape mismatch
E       [left]:  (2, 9)
E       [right]: (2, 8)

message_ix/tests/test_integration.py:26: AssertionError

Three other tests in test_integration.py fail in a similar way; all four need to be adjusted to the new behaviour of ixmp.

apologies, the implementation of the subannual-timeseries feature didn't consider all the implications for the message_ix framework - my bad.
I suggest to update ixmp.Scenario.timeseries() with a new kwarg subannual with the following options:

subannual=True: always return the dataframe with the subannual column (current behaviour)
subannual=False: drop column subannual if all values are 'Year', else raise an error (previous behaviour)
subannual='auto' (default): if there are entries of the subannual column other than Year, return dataframe with that column, else drop that column in returned dataframe

This should make all current tests in message_ix pass and provide an intuitive way forward when developing message_ix models with subannual resolution.
@khaeru, does that make sense and would that be consistent with the reporting module?

Thanks, this sounds like an elegant solution! So it sounds like it will be resolved with a PR on ixmp, rather than on message_ix.
To your question, this doesn't affect reporting, since:

we don't have any computations (that I'm aware of) that work with ixmp time-series data rather than model/solution data, and
A subannual dimension is auto-aggregated if users don't explicitly ask for it in reporting outputs—like mode or any other.

As for defaults, I would suggest:

subannual='auto' in ixmp 3.0 with a warning.
subannual=True in ixmp 4.0.

(2) is based on the principle of least astonishment: when they first use TimeSeries.timeseries(), the user learns what columns are returned, and they don't need to revise that knowledge unless they don't feel like calling .drop('subannual', axis=1) on their own. With default 'auto', then when a user switches from working on one TimeSeries/Scenario (with only 'Year') to another (with other timeslices), they might be surprised that their code stops working.
@iiasa/ene: any thoughts?

As indicated shortly on the Message meeting - we should carefully think (and test) behavior of subannual='auto' flag, as depending on the content of the dataset (e.g. variables having subannual timeslices might be filtered out using variable kwarg). Maybe real-world scenario won't have mixed subannual and non-subannual data, but then we can also think about to warn users adding such data to the scenario (which might be non-trivial).

we should carefully think (and test) behavior of subannual='auto' flag, as depending on the content of the dataset (e.g. variables having subannual timeslices might be filtered out using variable kwarg)

This is a good point. I'd suggest:

subannual=True: no issue.
subannual='auto': column is included if any of the variables contain values other than the default 'Year'.
subannual=False: either log.warning() or raise an exception (my preference) if any of the variables contain values other than 'Year'.

This approach is more comprehensive but requires additional changes on the Java backend (e.g., a function has_subannual_data(). If @zikolach can implement that in a timely manner, ok.
Otherwise, I'd go with the approach outlined above, which can be implemented solely on the Python side.

Can't we just:
if whatever_other_conditions and set(data['subannual'].unique()) == {'Year'}:
   data.drop('subannual', axis=1, inplace=True)

?

@khaeru that's is most straightforward solution. Alternatively (as previously suggested @danielhuppmann) it can be done via changing java. The difference is that former solution can have different result (with or without subannual column) depending on filters and subset of data returned, while the latter one - depending only on existing timeseries data (in DB and cache).

~subannual=False: either log.warning() or raise an exception (my preference) if any of the variables contain values other than 'Year'.~

Not sure whether this is still active proposal - but I agree we should raise an exception, otherwise we can have duplicates

Yes, this is the active proposal - as indicated in my comment above, calling this with subannual=False will raise an error if there are subannual data.

Just for posterity, I confirm that the four tests in test_integration.py now pass with ixmp master.


Clicking "next" button on Dialogue, sayings page goes to blank Dialogue page

When I typed in my sayings, set completed actions to them, and hit next button to go to next page, everything was erased and went to here:

Refreshed, tried to re-produce and behavior is normal, but worth noting as its a one time thing it seems or occasional.
-Tried on a different browser, and no problem.
-When refreshing on same browser, no problem.
-No errors in console.

Did you download a release or pull from Master? Which release?
yes, v0.21.1
What operating system are you running?
10 Pro
Are you running locally or in the cloud?
local
Which browser are you using?
Firefox

What next button?

In Dialogue sayings, page, when clicking the "Next" button to go to next page.

this isn't possible for this to be 0.21.1 because 0.21.1 doesn't have the Dialogue tab.

can't reproduce


fix readme with new digital ocean api

Codecov Report

Merging #110 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #110   +/-   ##
=======================================
  Coverage   53.55%   53.55%
=======================================
  Files           5        5
  Lines         450      450
=======================================
  Hits          241      241
  Misses        185      185
  Partials       24       24

Continue to review full report at Codecov.

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


support swift 4.2??

support swift 4.2??

Can you please explain a little more?  Swift 4.2 is supported, what isn't working for you?

my proto error


how to set limit parameter to find more than 10 child tenants?

Hi,
The search limit in PayloadCMS is set to 10 by default. Because of this, if there are more than 10 child tenants, the list only shows the first 10. Can I change the settings to increase the limit above 10?

I found a work-around.
Adding for example,  limit: 20 in node_modules/payload-tenancy/dist/utils/getAuthorizedTenants.js works okay.
return [4 /*yield*/, payload.find({
  collection: options.tenantCollection,
  where: { parent: { equals: tenantId } },
  limit: 20, // <-- here
})];


Add section Test Verifiable Credentials to VC "How it works"

Thank you for your contribution to the IC Developer Portal. This repo contains the content for https://internetcomputer.org and the ICP Developer Documentation, https://internetcomputer.org/docs/.
If you are submitting a Pull Request for adding or changing content on the ICP Developer Documentation, please make sure that your contribution meets the following requirements:

[ ] Follows the developer docs style guide.
[ ] Follows the best practices and guidelines.
[ ] New documentation pages include document tags.
[ ] New documentation pages include SEO keywords.
[ ] New documents are in the .mdx file format to support the previous two components.
[ ] New documents are registered in /sidebars.js, otherwise, it will not appear in the
side navigation bar.
[ ] Make sure that the .github/CODEOWNERS file is
filled with new documents that you added. This way we can ensure that future Pull Requests are reviewed by the right people.

@jessiemongeon1 could you review and let me know what you think of the new sectoin and if we need anything else to make these changes?

Thanks for the review @jessiemongeon1


fix: return "n/a" in verbose swarm peers response

this follows the lead of js-ipfs with the "n/a" fallback response by default.
Related: #178 and the transient failures we are getting.

bors r-

bors r+


add filter function to be able to filter out / only some country.

Hi, thanks for the great library.
I am not sure if this one is needed or not, but our business logic do need it. So I think maybe there are teams may need it. So instead of keeping it to myself, I made this PR.

Hi @zaralockheart there was an opened pr that I merged yesterday.
Now there is a property called countryFilter that makes you able to show only some countries

Hi @imtoori I actually opened this since october, and I am new to creating PR for public repo. can I know why was it not reviewed? At least I want to learn of where can I improve :)

Well, actually there was another pr opened and yesterday I reviewed the prs in chronological order
Sorry btw


Tempfile Image

if the image is tempfile without postfix,
logo = Tempfile.new('logo')
logo.binmode
logo.write(company.logo.read) rescue nil
logo.rewind
context = {
  image: Sablon.content(:image, logo.path)
}

then
template.render_to_file(docx.path, context)

will broken.
stderr: Error: source file could not be loaded

@xinlangzi thanks for bringing this up! As far as I know every file inside a MS Word document has to have a file extension (postfix) present in order for its content type system to work. This requirement was built into the image insertion feature of sablon since it will automatically add new content types on the fly.
Although, I didn't think to test the case when an image was provided without an extension. If that is the only error message produced I'll make some adjustments so a more "verbose" message is provided indicating the real underlying problem.
To provide a little more context on why extensions are needed if you aren't familiar with the internals of a word doc here in a snippet from the [Content_Types].xml file.
<?xml version="1.0" encoding="UTF-8"?>
<Types xmlns="http://schemas.openxmlformats.org/package/2006/content-types">
  <Default Extension="png" ContentType="image/png"/>
  <Default Extension="rels" ContentType="application/vnd.openxmlformats-package.relationships+xml"/>
  <Default Extension="xml" ContentType="application/xml"/>
  ...
</Types>

@xinlangzi So I ran a few tests against our fixtures doing things the way you did with temp files and I was not able to reproduce the error. However, the larger issue is that tempfiles get a weird name that obscures  the  true extension, for example: clone.jpg20180305-13153-1ej89dt this causes the document to be corrupted. In hindsight I should raise an error when a valid extension doesn't exist so I'll make that adjustment in the next day or two.
Lastly, to get around the temp file issue here is a workaround around that you might find suitable.
logo = Tempfile.new('logo')
logo.write(company.logo.read)
logo.rewind
context = {
  image: Sablon.content(:image, logo, filename: 'logo.extension')
}

"extension" can be anything readable by MS Word (i.e. jpg, png, tiff, bmp, etc.) and using the filename:  optional argument will override the name provided to the temp file by default.


Add missing redirect for java extensions page

For the record, this is a followup to #2690
Preview: https://deploy-preview-2695--opentelemetry.netlify.app/docs/instrumentation/java/automatic/extensions/
Redirect test: https://deploy-preview-2695--opentelemetry.netlify.app/docs/instrumentation/java/extensions/