COMPRESS-675 -- create unit test for COMPRESS-675

unit test for putting two unpack files within an archive

@garydgregory This is either user error or something is really wrong with unpack.
I couldn't get this test to pass with a tar archive stream or a zip archive stream -- different results!
I couldn't get this test to pass with a FIleInputStream, a BufferedInputStream(ByteArrayInputStream) or a Files.newInputStream(archive.toPath)
In the debugger on the Tika issue, I can see that Unpack unwraps down to a ChannelInputStream and then closes the underlying file. I can see similar behavior in this unit test if I open with Files.newInputStream()...so that's what you can use to replicate what we're seeing on Tika.
I'm perplexed that I can't get anything to pass, though... again, probably user error.

@garydgregory This is either user error or something is really wrong with unpack.
I couldn't get this test to pass with a tar archive stream or a zip archive stream -- different results!
I couldn't get this test to pass with a FIleInputStream, a BufferedInputStream(ByteArrayInputStream) or a Files.newInputStream(archive.toPath)
In the debugger on the Tika issue, I can see that Unpack unwraps down to a ChannelInputStream and then closes the underlying file. I can see similar behavior in this unit test if I open with Files.newInputStream()...so that's what you can use to replicate what we're seeing on Tika.
I'm perplexed that I can't get anything to pass, though... again, probably user error.

Hi @tballison
Would you please rebase on git master? Let's see what the tests say this time around...


Error: Unicode char

[x] I have verified in the Zotero addons screen that I am on the latest release of BBT (https://github.com/retorquere/zotero-better-bibtex/releases/latest) and Zotero
[x] I am posting a single question, bug or feature request
[x] I have included a descriptive subject of the problem
[x] I am available for follow-up questions and testing

Non-export problems with BBT
Report id: N54AF8U9
When I try to cite this article: https://www.ncbi.nlm.nih.gov/pubmed/28069918
I get the following error msg when knitting an RMakrkdown document:
! Package inputenc Error: Unicode char α (U+3B1)
(inputenc) not set up for use with LaTeX.
pdflatex --version
pdfTeX 3.14159265-2.6-1.40.16 (TeX Live 2015/Debian)

Two questions:

Did you export with Better BibTeX, or with Better BibLaTeX?
There are two options in the preferences to "Export unicode as plain-text", one for bibtex, one for biblatex, what are these set to?

Hello,

I believe BibLaTex (I am using this addon https://github.com/crsh/citr
and have not set options(citr.use_betterbiblatex = FALSE))
For Bibtex it is set to "Export unicode as plain-text" for BibLatex it
is not set.

On Fri, Apr 13, 2018 at 12:38 PM, Emiliano Heyns<EMAIL_ADDRESS>wrote:

Two questions:

Did you export with Better BibTeX, or with Better BibLaTeX?
There are two options in the preferences to "Export unicode as
plain-text", one for bibtex, one for biblatex, what are these set to?

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/retorquere/zotero-better-bibtex/issues/950#issuecomment-381096817,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AHHM2b45npQA0wX2GQRMXL5ADixfUnUrks5toIA9gaJpZM4TTOO4
.

Better BibLaTeX exports as unicode by default, so you will either have to use better bibtex, or enable the "export as plain text" option for better biblatex.

xref to https://github.com/crsh/citr/issues/30#issuecomment-381119919


@rollup/plugin-multi-entry broken by #54

The changes introduced by #54 breaks builds that involve @rollup/plugin-multi-entry.
Specifically:
/**
 * for processors that have alternate source file extensions,
 * make sure to use the output rollup file name
 */
const parsedPath = path.parse(file.path);
file.path = `${parsedPath.dir}/${result.fileName}`;

I believe applying hasNullByte to result.facadeModuleId would resolve the issue
if (result.facadeModuleId && !hasNullByte(result.facadeModuleId)) {
  const parsedPath = path.parse(file.path);
  file.path = `${parsedPath.dir}/${result.fileName}`;
}

facadeModuleId: string | null, // the id of a module that this chunk corresponds to

Thoughts?

Just following up on this @Lalem001. I'd like to fix this, do you still think your suggestion !hasNullByte() is the best way forward?

The suggestion is a possible solution. Karma would never use that particular plugin correctly anyway, since it only sends Rollup (and the preprocessor) 1 file at a time. The solution might resolve problems with other plugins that have not yet been reported.
I will say however that this issue so far has been preventing me from being on latest karma-rollup-preprocessor, and the solution proposed does resolve the issue for me locally.

That makes sense. Well, it seems like a fair precaution to check for null bytes. I updated the development branch, does this look correct to you?

Yes it looks correct, and it works for me locally.


Linux SocketCAN structs and constants (attempt #2)

Big apologies for dropping the ball on #140 a while back. Work on the socketcan crate stalled while the project maintainership was being ironed out, but now a lot of progress is being made and one of the last major aspects to be added for the upcoming release is Netlink access to query and update the CANbus network interfaces.
It appears that most of what we need added is contained in the can/netlink.h header as can be seen here:
https://elixir.bootlin.com/linux/latest/source/include/uapi/linux/can/netlink.h
It doesn't look too crazy; a few constants, bitflags, and structs. The things I'm still uncertain about is which parts of it might go here in the neli crate, which in libc, and what is best to just drop into the socketcan crate.
Any guidance would be appreciated.

BTW... we've been discussing this work here:
https://github.com/socketcan-rs/socketcan-rs/issues/32

Hi @fpagliughi! So what I'd recommend is first creating a PR against libc with all of the constants that you want added. Once you've done that and it's been merged and released in a new version, let me know which constants you want added to neli and the version of libc that contains them. From there, I'll put up a PR and put out a new version with those included. Sound good?

Hi @fpagliughi, any way I can help with this? Or is this kind of a background task?

Hey @jbaublitz. I was having trouble figuring out how to integrate a few things, but eventually found #[neli_enum(...)] and some related items, and was able to get everything working together from my crate. I put out a new release a few weeks back.

But now I do have a submodule of structs and consts here that really should be pushed upstream to libc and neli. I have to circle through a couple of other open-source projects that need to get out, then I will get back to this. Hopefully in a few weeks.


design discussion: generators yield

For now, just draft PR to discuss potential interface change with @filipstrand
Motivation

I want to create CLI and UI interfaces that generate many images at once
However, due to limited compute for target users, generating one image to finish takes too long when user wants to iterate over e.g. N seeds or N prompt variations without waiting for one image to finish at a time

Proposed Change
I don't like to break existing interfaces, or introduce complexity, so this proposal is an option that does not break existing docs, but does introduce behavior developers need to know (when to use it like a iterator, when to use it like a normal function)
Future work this enables:

a different CLI e.g. mflux-generate-many --attempts 5 or mflux-generate-many --seeds 1 2 3 4 5 or mflux-generate-prompts --prompt-file <some file in some format>
for now, I hijack the existing mflux-generate to demo the proposed iterator behavior - I would isolate the "create-many" behavior into a new script in the final implementation.

Finally got around to looking at this :)
I definitely find your use cases here valid and I think the idea of returning a generator is a more general approach than to return the final image as generate_image currently does. It adds slightly more abstraction, but it think the uses cases here are valid.
Please correct me if I am wrong, but yielding the stepwise_output in the time_step loop effectively returns that step to the caller and then some other piece of code would be responsible for iterating and actually doing something with with the output, like
while True:
     stepwise_output = next(image_gen_steps)
     print(f"Stepwise output: {stepwise_output}")

If so, I guess what we are doing here is really exposing each generation step to the "outside" to enable more flexibility. I feel this is natural and the caller would have more control over each generator to progress as they want it to (like in your example of having many parallel ones all executing one step at a time or similar).
I like your default approach of not breaking any existing interface when proposing a feature, and maybe this example was only temporary,  but if we were to go this route, I feel it is a bit weird to have generate_image() returning either a GeneratedImage or the stepwise_output as a generator. Would it not be clearer to always yield the stepwise_output? Then it would be the responsibility of another piece of code to actually decide what to do with the step (like saving it, not saving it and only returning the final image etc).

Just a short comment regarding generating multiple image at a time:
Another way to do it would be a more low-level approach by creating several initial latent arrays and concatenating them and doing the matrix math on this concatenated array (I think the mlx-examples have support for this).
I guess depending on what mlx does under the hood, it could be faster than doing it for one image at a time as we do now. However, I guess this approach would not allow us to cancel certain images halfway through the generation (since the math would be done on the full concatenated array). Also, I purposely avoided taking this approach when starting the project to keep things simple and not having to worry about concatenating/slicing arrays and whatever complications that might bring (maybe this is actually easy - e.g just concatenating each new latent instance along a new dimension - but I have not investigated this yet).

We can hold on this significant operational change for now and re-engage after higher visibility feature releases.
Since posting the draft PR I have also been using some system level locks to run/queue multiple mflux programs at a time. Look for those proposals in coming weeks.

Withdrawing this PR for now due to its staleness. I think the stepwise functionality and interruptibility can be improved but I will re-approach this at a later time.


Example for node enrichment

This PR demonstrates how we can enrich nodes such that enrichments show up in scaladoc - following the same pattern that @mpollmeier used for step enrichment. The mechanism is demonstrated by adding a parameter method to method nodes.
I realize @ml86 is working on new concepts for our DSL implementation, however, I don't want to wait for that to conclude to make further language improvements. Instead, let's shape the DSL further with the current implementation, and thereby uncover what a new implementation would need to provide.

I'm ok with that. The broader idea of this PR is, however, to get some feedback on whether this is the correct way to proceed in cases where we need operations that can be performed both on the pipe and the single node. I'm actually hoping for a better suggestion - as an alternative to changing the DSL entirely.

I'll close this one, and we continue the discussion in https://github.com/ShiftLeftSecurity/codepropertygraph/pull/148


add verification script

Resolve gh issue Verification Script #4

Resolved all issues I believe, please review :)

I added the requested changes + added to constants.py a map from iso lang code to a verbose spelling of the language for llm consumption 👍


Better upsampling and downsampling modes

Currently, the upsampling and downsampling mode is BILINEAR. Generally-speaking, it would be better to use BICUBIC for upsampling and LANCZOS for downsampling. (See this).
Currently, these two lines control this behavior. But many of the test expectations were saved using the BILINEAR method so this will require saving new expectations.

Hey @kheffah, thank you for noticing this!

Generally-speaking, it would be better to use BICUBIC for upsampling and LANCZOS for downsampling.

Honestly I didn't know 👀
From my side we can proceed and fix this 👍

@alessiamarcolini Awesome. I mean, it's not a rule .. just what tends to look better and have less artifacts etc. BTW, I recommend branching off `support-mpp`` branch since it parametrized this behavior in the two lines linked above. Cheers!


Dash-Sonic - Update for Scaling/Underlay Routing/ST/PL encoding

Update Sonic HLD for:

Scaling requirement for VNET and Inbound Routing
Underlay Routing behavior clarification
Clarify ST/PL encoding and inbound traffic

How is overlay encoding interpreted by the destination (shared resource)?  Once we have done the 4t6 transposition, we have overlay_sip+underlay_sip encoded as one v6 address and overlay_dip+overlay_dip encoded as another.  How does a destination use this for reverse routing?

How is overlay encoding interpreted by the destination (shared resource)? Once we have done the 4to6 transposition, we have, for service tunnel, overlay_sip+underlay_sip encoded as one v6 address and overlay_dip+underlay_dip encoded as another. How does a destination use this for reverse routing?
@mhanif , interpretation of the encoding at destination is upto the provider of the service. Expectation is, they can encode the fields of their interest, say vnet-id, region-id etc to the transformed ipv6 address and should have some contract with the destination resource. For ST, underlay sip/dip are ipv4. only overlay is transformed from 4to6. Overlay v6 has the source/dst address embedded to last 32 bits which destination use for reverse routing.


[Bug]: compose.yml: "version" is obsolete..!!

Is there an existing issue for this?

[X] I have searched the existing issues

Machine specifications
AMD Ryzen 3 2200G / 16GB
Operating system
Fedora 40 Workstation / Kernel 6.8.7-300
Docker version
26.1.2, build 211e74b
Description
What am I doing wrong..??
I created the compose.yml file
I ran the command "docker compose up -d compose.yml", following the tutorial and it presented the error "version is obsolete".
At first I thought it was the version of Windows I was trying to install, but I believe it was version: "3" in the compose.yml file, but I don't know which version to set.
Docker compose
version: "3"
services:
windows:
image: dockurr/windows
container_name: windows
environment:
VERSION: "win7"
RAM_SIZE: "8G"
DISK_SIZE: "200G"
devices:
- /dev/kvm
cap_add:
- NET_ADMIN
ports:
- 8006:8006
- 3389:3389/tcp
- 3389:3389/udp
stop_grace_period: 2m
restart: on-failure
Docker log
{"log":"\n","stream":"stdout","time":"2024-05-15T13:26:49.804822355Z"}
{"log":"Hello from Docker!\n","stream":"stdout","time":"2024-05-15T13:26:49.804873176Z"}
{"log":"This message shows that your installation appears to be working correctly.\n","stream":"stdout","time":"2024-05-15T13:26:49.804875866Z"}
{"log":"\n","stream":"stdout","time":"2024-05-15T13:26:49.804878036Z"}
{"log":"To generate this message, Docker took the following steps:\n","stream":"stdout","time":"2024-05-15T13:26:49.804880106Z"}
{"log":" 1. The Docker client contacted the Docker daemon.\n","stream":"stdout","time":"2024-05-15T13:26:49.804882016Z"}
{"log":" 2. The Docker daemon pulled the "hello-world" image from the Docker Hub.\n","stream":"stdout","time":"2024-05-15T13:26:49.804883756Z"}
{"log":"    (amd64)\n","stream":"stdout","time":"2024-05-15T13:26:49.804885636Z"}
{"log":" 3. The Docker daemon created a new container from that image which runs the\n","stream":"stdout","time":"2024-05-15T13:26:49.804887276Z"}
{"log":"    executable that produces the output you are currently reading.\n","stream":"stdout","time":"2024-05-15T13:26:49.804888966Z"}
{"log":" 4. The Docker daemon streamed that output to the Docker client, which sent it\n","stream":"stdout","time":"2024-05-15T13:26:49.804890636Z"}
{"log":"    to your terminal.\n","stream":"stdout","time":"2024-05-15T13:26:49.804892316Z"}
{"log":"\n","stream":"stdout","time":"2024-05-15T13:26:49.804893916Z"}
{"log":"To try something more ambitious, you can run an Ubuntu container with:\n","stream":"stdout","time":"2024-05-15T13:26:49.804895476Z"}
{"log":" $ docker run -it ubuntu bash\n","stream":"stdout","time":"2024-05-15T13:26:49.804897146Z"}
{"log":"\n","stream":"stdout","time":"2024-05-15T13:26:49.804898736Z"}
{"log":"Share images, automate workflows, and more with a free Docker ID:\n","stream":"stdout","time":"2024-05-15T13:26:49.804900296Z"}
{"log":" https://hub.docker.com/\n","stream":"stdout","time":"2024-05-15T13:26:49.804901916Z"}
{"log":"\n","stream":"stdout","time":"2024-05-15T13:26:49.804903496Z"}
{"log":"For more examples and ideas, visit:\n","stream":"stdout","time":"2024-05-15T13:26:49.804905026Z"}
{"log":" https://docs.docker.com/get-started/\n","stream":"stdout","time":"2024-05-15T13:26:49.804906666Z"}
{"log":"\n","stream":"stdout","time":"2024-05-15T13:26:49.804908306Z"}
Screenshots (optional)

From that warning it looks like the fix is to just remove that whole version line.

Sorry my bad english, I'm from Brazil..!!
I removed 'version' line and I got message "no such service: compose.yml".

You can just run I guess:
docker compose up -d

You must not add compose.yml at the end like you did:
docker compose up -d compose.yml

I ran the command "docker compose up -d" and it apparently worked, but so far Windows 7 won't start.
I don't know what I could be doing wrong. I would like some help, if you can, of course.

Why do you think it didnt start? Looks good. Also you are using the -d option to start it in the background, so thats why you dont see any logging..

I created another container named windows81, but I can't access it through the browser, accessing it via my internal IP or via http://localhost:8006

Please create another issue via the bug report form, because without logfiles/compose/etc I cannot help in any meaningful way.

Please, where do I find the files you need in Fedora 40..??


where is MKLROOT when I use conda install -c intel mkl mkl-devel mkl-static mkl-include?

I was trying to install aihwkit which is based on MKL library. It requires me set the ENV variable MKLROOT. So I tried different ways to install MKL as illustrated in the following image. They all seem to be smooth in installation. But I don't know where it is installed so I can't set the MKLROOT. And how can I know if I install it correctly?
I am very grateful if someone can help!!

@dongwhfdyer I have been using Online installer, which have worked for me so far. Do you want to try it?
@vmalia are you familiar with Conda package?

Yeah i tried.
These are the commands I use.
wget https://registrationcenter-download.intel.com/akdlm/irc_nas/19138/l_onemkl_p_2<IP_ADDRESS>98.sh

sudo sh ./l_onemkl_p_2<IP_ADDRESS>98.sh

But I got the following output.
Life is so hard.

Integrity check failed. X
Checksum of file /home/szh2/intel/installercache/downloadcache/intel.oneapi.
lin.condaindex,v=2023.0.0-25326/cupPayload.cup is not correct.
Integrity check failed.
Checksum of file /home/szh2/intel/installercache/downloadcache/intel.oneapi.
lin.tbb.runtime,v=2021.8.0-25334/cupPayload.cup is not correct.
Installation of component has failed.
Component id: intel.oneapi.lin.oneapi-common.vars, name: oneAPI Common, vers
Integrity check failed.

Checksum of file /home/szh2/intel/installercache/downloadcache/intel.oneapi. |
lin.condaindex,v=2023.0.0-25326/cupPayload.cup is not correct. |
Integrity check failed. |
Checksum of file /home/szh2/intel/installercache/downloadcache/intel.oneapi. |
lin.tbb.runtime,v=2021.8.0-25334/cupPayload.cup is not correct. |
Installation of component has failed. Component id: intel.oneapi.lin.oneapi-common.vars, name: oneAPI Common, vers

@vmalia do you have any idea?

@dongwhfdyer seems like the oneAPI installer could not verify the files. Please try to re-download the files and perform a clean installation. On linux, files are usually installed in /opt/intel/. If you still see a problem, please post your steps, links, and other details here: https://community.intel.com/t5/Intel-oneAPI-Math-Kernel-Library/bd-p/oneapi-math-kernel-library. This is the dedicated forum for oneMKL backend.

The MKL_ROOT is conda_path/envs/the_env_mkl_installed/library/ or conda_path/envs/the_env_mkl_installed


migrate to iron-dropdown

Migrate to using iron-dropdown to enable dynamic positioning.

The panel should move to the left as opposed to getting cut off correct?


tests: update IDs in Openstack image boot test

Update flavor and network ID for booting Openstack images in rhos-01
cloud.
This pull request includes:

[ ] adequate testing for the new functionality or fixed issue
[ ] adequate documentation informing people about the change such as

[ ] submit a PR for the guides repository if this PR changed any behavior described there: https://www.osbuild.org/guides/

I wonder if we should move this to env variables. Feels dirty have it hardcoded.

Ok this is passing now, I didn't notice one more variable that needed change https://gitlab.com/osbuild/ci/osbuild-composer/-/jobs/2067588392 ... Should I move it to variables instead then? And by env variables you mean so that it can be defined in CI variables if needed for example?


5895 Use jvm-repr for auto-display of Scala DisplayTables and Maps

Here's some code to do the Scala auto-display of DisplayTables and (immutable) Maps (#5895).
As noted in the comments, there is a limitation in the current jvm-repr that makes it work only with classes and not interfaces (or Scala traits, which map to JVM interfaces modulo concrete methods).  It's just lucky that there is an abstract class for Map.  I opened an issue in jvm-repr for this, but I doubt anything will change in the near future.
Minor accompanying changes to the doc notebooks, mostly removing redundant .display() calls and unnecessary imports.

Thanks Joe we have to think about this and see what reaction we get from jvm-repr.
As you know I want to switch our internal display dispatching to jvm-repr but if there's a problem with it working with scala I don't want to stretch it.

That's reasonable.  I didn't necessarily expect this to be merged immediately, but I wanted to put it out in public, since it seems to work with what we have today.

We've added support in Toree for jvm-repr. The code is here: https://github.com/apache/incubator-toree/blob/master/scala-interpreter/src/main/scala/org/apache/toree/kernel/interpreter/scala/ScalaDisplayers.scala
The only annoyance was that Scala maps don't implement java.util.Map, but you just have to add .asJava.

done this way: https://github.com/twosigma/beakerx/pull/5947


Debugging babel transpiled code not fully working with node2

VSCode Version: 1.8.1
OS Version: Fedora 25

Steps to Reproduce:

Use babel-register to transpile source files.
Set breakpoints in the original source files.
Try to hover variables if breakpoint gets hit.

The problem is, that the breakpoints gets hit but the values of variables doesn't get shown if you hover them. If you add the variable as a watch expression the value of it gets shown in the list. This only happens if you use the node2 debugger. With the "old" node debugger everything works as expected.
This is my setup of babel-register:
require('babel-register')({
    plugins: ['transform-async-to-generator'],
    sourceMaps: true,
    retainLines: true
});

require('./src/app');

This is my launch.json:
{
    "version": "0.2.0",
    "configurations": [{
        "name": "Launch Program",
        "type": "node2",
        "request": "launch",
        "program": "${workspaceRoot}/index.js",
        "cwd": "${workspaceRoot}",
        "sourceMaps": true
    }, {
        "name": "Attach to Process",
        "type": "node2",
        "request": "attach",
        "port": 9229
    }]
}

This is a known issue in 1.8 - just in this specific dynamic compilation case, hover won't work. However, it should work in 1.9 (and currently in insiders).

This is a known issue in 1.8 - just in this specific dynamic compilation case, hover won't work. However, it should work in 1.9 (and currently in insiders).

I tried it in a current insider build and got the same problem. I also doesn't see the values of hovered variables.

I tried it in a current insider build and got the same problem. I also doesn't see the values of hovered variables.

It's working in my babel-register test project. Is this a public repo where I can try it out?

It's working in my babel-register test project. Is this a public repo where I can try it out?

I have pushed it into a public git repo so you can try it yourself.
Link: https://github.com/mrukas/koa-babel-test
This is just a test project with koa js and babel as transpiler, there isn't anything useful in it. The master branch uses the new debugger and the old-debug branch uses the old debugger. For me the old debugger works perfectly fine, but the new one makes multiple problems which are:

Values of variables doesn't get shown when hovered.
Errors are getting thrown if project gets started and breakpoints are already set.

Thanks in advance!

I have pushed it into a public git repo so you can try it yourself.
Link: https://github.com/mrukas/koa-babel-test
This is just a test project with koa js and babel as transpiler, there isn't anything useful in it. The master branch uses the new debugger and the old-debug branch uses the old debugger. For me the old debugger works perfectly fine, but the new one makes multiple problems which are:

Values of variables doesn't get shown when hovered.
Errors are getting thrown if project gets started and breakpoints are already set.

Thanks in advance!

I just realized that my fix for this didn't make it into node2 yet. It will be fixed in the next insiders build.
For reference, this was previously https://github.com/Microsoft/vscode-chrome-debug/issues/309

Ok this sounds great! Thanks!


New onboarding templates for new open source collectives

wip
Fixes https://github.com/opencollective/opencollective/issues/1804

Coverage increased (+0.07%) to 67.268% when pulling 5f942580f5c0b025bdc4357e42f0f966405e0dcd on fix1804 into bd97592430fbcd4e34a20c2ec3f64e0ab629d034 on master.


Current balance is always zero

Hi guys. I run the Peatio exchange in appropriate way(according to local setup for mac-os). It`s works fine but on testnet current balance of bitcoin is always zero. Please help me!

+1 on this one... Same issue here, running on Arch Linux.
I noticed that in the tab solvency, 2 addresses are shown, but if you click in any of them, they'll lead you to blockchain.info's mainnet explorer, not the testnet.

Same here


[BUG]: drizzle-zod Insert Schema requiring id field

What version of drizzle-orm are you using?
0.25.4
What version of drizzle-kit are you using?
0.17.6
drizzle-zod version: 0.4.1
Describe the Bug
First of all, here's a bit of code to give context as to what I'm working with:
import { z } from "zod";
import { createInsertSchema } from "drizzle-zod";

export const redditPosts = mysqlTable(
    "reddit_posts",
    {
        id: int("id").autoincrement().primaryKey(),
        subreddit: varchar("subreddit", { length: 50 }).notNull(),
        title: varchar("title", { length: 255 }).notNull(),
        url: varchar("url", { length: 255 }).notNull(),
        over_18: boolean("over_18").notNull(),
        permalink: varchar("permalink", { length: 255 }).notNull(),
    },
    (table) => {
        return {
            subreddit: index("subreddit").on(table.subreddit),
        };
    }
);

export const InsertRedditPostSchema = createInsertSchema(redditPosts);

I'm using drizzle-zod to make sure I don't accidentally add posts to my database that are too long cause this breaks discord's embeds. Anyways, here's one of those posts as an object:
const post = {
    subreddit: submission.subreddit.display_name,
    title: submission.title,
    url: submission.url,
    permalink: submission.permalink,
    over_18: submission.over_18,
};

submission here is an instance of the Submission class
Everything is as you'd expect it to be, all the types line up.
Now when I call
InsertRedditPostSchema.safeParse(post);

I always end up with an error inside the result, regardless of the post being checked.
Said error looks as follows (Same when running the js normally instead of ts-node):
ZodError: [
  {
    "code": "invalid_type",
    "expected": "number",
    "received": "undefined",
    "path": [
      "id"
    ],
    "message": "Required"
  }
]
    at Object.get error [as error] (file:///E:/coding-projects/hifumi-js/node_modules/.pnpm/zod@3.21.4/node_modules/zod/lib/index.mjs:537:31)
    at fetchSubmissions (file:///E:/coding-projects/hifumi-js/src/commands/reddit.ts:173:52)
    at async Module.sub (file:///E:/coding-projects/hifumi-js/src/commands/reddit.ts:93:24)
    at async handleCommand (file:///E:/coding-projects/hifumi-js/src/handlers/messages.ts:83:33)
    at async Client.handleMessage (file:///E:/coding-projects/hifumi-js/src/handlers/messages.ts:45:13) {
  issues: [
    {
      code: 'invalid_type',
      expected: 'number',
      received: 'undefined',
      path: [Array],
      message: 'Required'
    }
  ],
  addIssue: [Function (anonymous)],
  addIssues: [Function (anonymous)],
  errors: [
    {
      code: 'invalid_type',
      expected: 'number',
      received: 'undefined',
      path: [Array],
      message: 'Required'
    }
  ]
}

It's telling me that the id is required, but it shouldn't be since this is a schema for insertion and the type you get from inferring this schema also has the id as optional:
export type NewRedditPost = z.infer<typeof InsertRedditPostSchema>;

type NewRedditPost = {
   subreddit: string;
   title: string;
   url: string;
   over_18: boolean;
   permalink: string;
   id?: number | undefined;
}

Not sure if I just missed something very important or I somehow managed to break the library, the former would be really nice though.
Expected behavior
It'd be kind of nice if an insert schema did not require an id to be present when it's optional in the table
Environment & setup
This happens both in local development on windows 11 and in production on linux (both running<EMAIL_ADDRESS>

I just did some further testing, is the schema supposed to ignore things like a varchar's length limit?
const post = {
   id: 1,
   subreddit: "A".repeat(60),  // VARCHAR(50)
   title: "A".repeat(300),     // VARCHAR(255)
   url: "A".repeat(300),       // VARCHAR(255)
   permalink: "A".repeat(300), // VARCHAR(255)
   over_18: true,              // BOOLEAN
};

InsertRedditPostSchema.parse(post);

Expected this to error out, but it doesn't. I don't know how doable it'd be to infer this from the db table, but if it's possible that would be definitely be a very nice feature to have. Unless it is bugged after all of course.

Hello @TiltedToast I acknowledge your pain here and  are right in the second ask of actually checking the length of varchar.
But with all due respect what did you originally proposed, to basically omit the primaryKey (that is inherently notNull) goes directly against, I believe most normal uses of the drizzle-zod by introducing some shady hack on the primaryKey. Basically if something is notNull you would expect zod to catch it but it does not (I am on postgres btw) and it is not the thing you want, please read this issue that is a complete opposite of what you ask:
https://github.com/drizzle-team/drizzle-orm/issues/576
For you @TiltedToast consider using uuid and setting them at the client side.
For drizzle-team: I would reframe this to let the zod to its thing, and create a new feature for drizzle-zod that could be triggered like this:
const insertTeamSchema = createInsertSchema(teams, {optionalPrimaryKey: true})

to basically omit the primaryKey (that is inherently notNull) goes directly against, I believe most normal uses of the drizzle-zod by introducing some shady hack on the primaryKey

Except it's allowed to be null here, that's the whole point. The database takes care of generating an id for each record. There's no shady hack here at all, I'm just using the db the way it's intended to be used. It cannot be null once it's inserted yes, but that's after the database has added a required id on its own.
It's the exact same scenario that you're dealing with, except for you zod is rightfully ignoring absent ids because the schema says they're randomly generated by the db. Maybe remove that defaultRandom() and generate your own uuid? Or just use the schema generated for select

I think the root cause of the issue is drizzle-zod does not make the field optional if it's defined with auto increment. I'll look into it.

The varchar length check issue is also valid, I didn't think about it. It can be implemented as well but would be good if it's separated into its own issue.

@TiltedToast yes I understand that in your case of using int autoincrementing ids you need to db and some bloom filter on the backend but not in the case of uuid;).
The defaultRandom works thanks but technically the drizzle-zod should be about the insert not the state of the db but I can live with - it would be a nitpicking.

@TiltedToast the autoincrement issue should now be fixed in drizzle-orm@beta (it was an ORM issue, not drizzle-zod).


"Process stalled" error message

I am still trying to run check:construction and now that I have increased the tip_delay so it can start, I have this error message.
2022/10/12 14:45:11 Syncing 12575433-12575434
processing stalled
[MEMORY] Heap: 1103.368118MB Stack: 0.656250MB System: 1160.534706MB GCs: 6
2022/10/12 14:45:12 check:construction status server shutting down

I have no clue why it is shutting down on its own like this. Any ideas?

Here is the definition of processing stalled, can you check if the workflows (e.g. CreateAccount) in *.ros file is defined properly?
And this is where we return processing stalled, if the workflows are not defined properly finally it will return that error message

Here the .ros file
create_account(1){
  create{
    network = {"network":"cosmoshub-4 ", "blockchain":"Cosmos"};
    key = generate_key({"curve_type": "secp256k1"});
    account = derive({
      "network_identifier": {{network}},
      "public_key": {{key.public_key}}
    });

    // If the account is not saved, the key will be lost!
    save_account({
      "account_identifier": {{account.account_identifier}},
      "keypair": {{key}}
    });
  }
}

I didn't write this so I assume it was working before...

This is an example from ethereum.ros, and here is the instruction of how to write *.ros, you may use them as reference
You may wait for your chain to sync to tip then give a try, if this issue still exists you may use the debugger to deep dive what actually happen in the *.ros

I found the issue. It is because it is required to have at least 2 workflows : request and create.
It is written in the documentation

I don't know if there is a way to improve the error message or to quickly check the .ros file to be sure it include those 2.

Thanks for bringing this up, I am gonna improve the error message for this


Assertion '1 == driver_switches.size()

Hello! I am using the latest version in master branch, and I am defining a new architecture. Now I am facing an Assertion Error I don't know how to tackle. It is hard to check what I am doing wrong in the arch definition or if it is a problem in the framework (less likely).
Backannotated 25 General Switch Blocks (GSBs).
Build General Switch Block(GSB) annotation on top of routing resource graph took 0.00 seconds (max_rss 60.5 MiB, delta_rss +0.0 MiB)
Sort incoming edges for each routing track output node of General Switch Block(GSB)
Start sorting edges for GSBs up to [5][5]
[4%] Sorted incoming edges for each routing track output node of GSB[0][0]
[8%] Sorted incoming edges for each routing track output node of GSB[0][1]
[12%] Sorted incoming edges for each routing track output node of GSB[0][2]
[16%] Sorted incoming edges for each routing track output node of GSB[0][3]
[20%] Sorted incoming edges for each routing track output node of GSB[0][4]
[24%] Sorted incoming edges for each routing track output node of GSB[1][0]
[28%] Sorted incoming edges for each routing track output node of GSB[1][1]
[32%] Sorted incoming edges for each routing track output node of GSB[1][2]
[36%] Sorted incoming edges for each routing track output node of GSB[1][3]
[40%] Sorted incoming edges for each routing track output node of GSB[1][4]
[44%] Sorted incoming edges for each routing track output node of GSB[2][0]
[48%] Sorted incoming edges for each routing track output node of GSB[2][1]
[52%] Sorted incoming edges for each routing track output node of GSB[2][2]
[56%] Sorted incoming edges for each routing track output node of GSB[2][3]
[60%] Sorted incoming edges for each routing track output node of GSB[2][4]
[64%] Sorted incoming edges for each routing track output node/home/kpatinos/Intel/OpenFPGA/openfpga/src/mux_lib/mux_library_builder.cpp:52 build_routing_arch_mux_library: Assertion '1 == driver_switches.size()' failed.
So if you could provide any insight what could be wrong or how to solve this issue I will apreciatte.
Thanks Kevin

@kpatinos-intel It could be some routing tracks or inputs of your programmable blocks have no drivers. Or you defined different types of switches that drive a routing track or inputs of your programmable blocks


Unorder list second level invalid when RecyclerView use MarkwonAdapter

Markwon version: 4.6.2

unordered list first level

unordered list second level1
unordered list second level2

- unordered list first level
    - unordered list second level1
    - unordered list second level2

When I use it, it doesn't display in standard Markdown format, I use MarkwonAdapter in RecyclerView for preview display
Type before previewing
Preview effect

Hello @ZuiRenA ,
can you explain a bit more why it doesn't display in standard Markdown format? I see 2 levels of un-ordered list correctly on your preview image

Hello @noties
The first picture is what I entered, the second picture is my preview of the input, the unordered list I entered has two levels, the content of the second level -unordered list content1 is not correct when previewing is recognized as the second level of an unordered list

Well, I see that the first and second lines are merged. You should validate your input - that you send exactly
- test
   - unordered list second level1
   - unordered list second level2

It seems you process the input somehow, and might be removing some parts of the original input, which breaks the parser and renderer. It also seems that RecyclerView has nothing to do here and if you use regular TextView you might see the same results

Thank you for your reply
I checked multiple times and pasted the available unordered list directly from the clipboard, still have this problem, and I tried using TextView, the same error occurred

Hello @ZuiRenA ,
unfortunately I will need to close this issue, as you did not provide any details of how this would be reproduced. This had not gathered any feedback so I would assume there is an issue of your side. Which could be a library misconfiguration or a device specific bug


Unable to stack modals on top of each other (2nd modal displays behind the first modal shown).

try adding this - style="z-index: 1200 !important; to your top model HTML

as for now http://v4-alpha.getbootstrap.com/components/modal/

Multiple open modals not supported
Be sure not to open a modal while another is still visible. Showing more than one modal at a time requires custom code.

It will change most probably later


Add case series to Apollo-SV

parent is most likely dataset
We have "infection case list" but that corresponds to a different XSD type.   There is an XSD type called CaseSeries for which we do not have a class at the moment, apparently.
We'll need to differentiate CaseList from CaseSeries.

Here is the definition we arrived at: "An investigation that has, as part, a population survey with an objective to count all participants in some disease outcome or some exposure process."


Notify applicants on submission and approval

FYI @devramsean0, after this is merged, submitters will get emails when their hackathon is submitted and approved.
Rejection emails likely should still be sent manually to provide a reason for the rejection


Improve Map Controls

As a user I want the map controls to be familiar and similar to what I've experienced in other systems like Google Maps and Apple Maps and don't draw too much attention to themselves.

Move map controls bottom right. It’s a more common location in popular apps like Google and Apple maps. https://share.cleanshot.com/bzTmR375
Allow the map to be zoomed out farther so you can see all of Philadelphia in one view. https://share.cleanshot.com/qZdGvScS
Give all map elements the same margin https://share.cleanshot.com/ZCT3HwWw
Remove the rotate map button. If possible disallow rotating. I feel it isn’t really helpful and just distracts. https://share.cleanshot.com/3P1yXgcl

Here's a link to the prototype. https://www.figma.com/file/NAFkgq34abW6uJ0R7PW24T/Prototype---Clean-%26-Green-Philly?type=design&node-id=187%3A12604&mode=design&t=h1fTrSpJsuOpTu9c-1

closed with #209
@brandonfcohen1 we'll need to update the mapbox vt layers to be visibile out to a zoom level of 10

@brandonfcohen1 I just going to comment on that issue!

@nlebovits Small change: Can we make the margin on the controls closer to 24px? Right now it's like like 12px. https://share.cleanshot.com/Pm7KxmFp It matches the spacing used throughout the rest of the website and just generally looks nicer.


Add the famous Cloud Native BakeOff shortbread cookies from 2020.

Given that these were the runners-up for the Great Cloud-Native BakeOff, they ought to be in the cookbook.

Note that this recipe has two photos for it.  There weren't a lot of examples of sizing or placement for photos, so you'll probably need to adjust that.


Bundle using bili and exposes distribution files

I've used bili, a pretty simple abstraction of rollup, to bundle components and changed module and main entries to distribution sources fo each one. I've also removed old webpack's bundle
configuration. This fixes #116.
I couldn't run tests because of selenium driver look for java path. I also noticed tests are not been updated, should we keep it?
I can make another PR replacing dev process to poi, with it we can delete all build and config stuff and run everything from package scripts, this cleanup this project a lot!

@VitorLuizC @egoist, huge apologies for the long silence on this PR. It was a combination of me being really busy at my day job and also my intimidation regarding build processes in general. I really wanted to learn and discover it for myself and this was one of the reason for me not accepting this PR right away.
I have since implemented a new build process for the next major version (currently in beta) using bili (for bundling) and poi (for dev). They are both fantastic tools. I have also separated out the styles as suggested by @egoist in the new version.
Once the new version comes out, I'll close out this PR but I just wanted to sincerely apologize for the silence.

closing


Add missing dependency on Unix

Silences the alert proposed in ocaml/ocaml#11198, but this PR can be merged regardless.

Thanks!


GitHub workflow to build Daytona binaries within PRs

Is your feature request related to a problem? Please describe.
As a docs writer, I need to efficiently test new features and breaking changes in advance of a new Daytona release. The earlier I can get a build including a specific feature, the better.
Describe the solution you'd like
I'm proposing adding a GitHub workflow to the repository that runs in the context of a PR. When a new commit is pushed in a PR, it builds the binaries and uploads them as artifacts. I can pull the binary from the PR and install it in a fresh local environment for testing.
Aside from this being a solution to the above, this change would also be a build verification steps for Daytona's development process.
Additional context
Relevant docs: https://docs.github.com/en/actions/using-workflows/storing-workflow-data-as-artifacts#uploading-build-and-test-artifacts

@osslate we already have a workflow that builds the binary for all architectures to validate the build process (see https://github.com/daytonaio/daytona/blob/main/.github/workflows/build_pr.yaml).
Regarding publishing them as artifacts, since we don't publish releases as GH artifacts, I think we shouldn't post PR artifacts either. To test PR changes, it's as simple as pulling the branch and running go build ... or dtn ... (if in a devcontainer).

This is fine by me, thanks!


[jsoncpp] Missing usage file

Is your feature request related to a problem? Please describe.
We're trying to use the jsoncpp port and don't see a usage file, so I'm not really sure how to consume it in my project.
Proposed solution
Add a usage file for this port.
Describe alternatives you've considered
We're just now starting this work, so we might be able to figure it out on our own; the usage files are really helpful for decreasing time spent in investigation though.
Additional context
N/A

Usage info is auto-generated on vcpkg install jsoncpp.
The package jsoncpp provides CMake targets:

    find_package(jsoncpp CONFIG REQUIRED)
    target_link_libraries(main PRIVATE jsoncpp_object jsoncpp_static)

However, autogenerated usage instructions may be wrong. (Here, the json_object item seems strange. And it is unfortunate that they don't use a namespace.)
Ideally, you find the usage in upstream's documentation.

Thank you @dg0yt; we're pretty new to vcpkg, so sorry I didn't know they get autogenerated. We'll try these usage instructions and I'll update here on our findings 🙂

@cglong ,Have you find the usage in upstream?

Hi @JonLiu1993, thank you for following up! After installing the package, we did get these instructions as @dg0yt mentioned:
find_package(jsoncpp CONFIG REQUIRED)
target_link_libraries(${library_name} jsoncpp_lib jsoncpp_object)

However, we're now receiving these errors. Do you happen to know what could be causing this?
 error LNK2019: unresolved external symbol "public: class Json::Value & __cdecl Json::Value::operator=(class Json::Value)" (??4Value@Json@@QEAAAEAV01@V01@@Z) referenced in function "[snip]"
 error LNK2001: unresolved external symbol "public: class Json::Value & __cdecl Json::Value::operator=(class Json::Value)" (??4Value@Json@@QEAAAEAV01@V01@@Z)
 error LNK2001: unresolved external symbol "public: class Json::Value & __cdecl Json::Value::operator=(class Json::Value)" (??4Value@Json@@QEAAAEAV01@V01@@Z)
 error LNK2001: unresolved external symbol "public: class Json::Value & __cdecl Json::Value::operator=(class Json::Value)" (??4Value@Json@@QEAAAEAV01@V01@@Z)

@cglong ,Thanks for your feedback, could you please provide your CMakeLists.txt and main.cpp to repo this issue? I created a cmake project using jsoncpp locally,  and it use  successfully.
The repro steps:

./vcpkg install jsoncpp:x64-windows
./vcpkg integrate install
create cmake project:

CMakeLists.txt:
cmake_minimum_required(VERSION 3.0)
project(test)

find_package(jsoncpp CONFIG 1.9.5 REQUIRED)

add_executable(main main.cpp)

target_link_libraries(main PRIVATE jsoncpp_lib jsoncpp_object JsonCpp::JsonCpp)

main.cpp:
#include "json/json.h"

using namespace std;

int main()
{
	return 0;
}

4.F:\test8\test\build> F:\test8\vcpkg\downloads\tools\cmake-3.21.1-windows\cmake-3.21.1-windows-i386\bin\cmake.exe -DCMAKE_TOOLCHAIN_FILE="F://test8//vcpkg//scripts//buildsystems//vcpkg.cmake" ..
-- Building for: Visual Studio 16 2019
-- Selecting Windows SDK version 10.0.19041.0 to target Windows 10.0.19042.
-- The C compiler identification is MSVC 19.29.30137.0
-- The CXX compiler identification is MSVC 19.29.30137.0
-- Detecting C compiler ABI info
-- Detecting C compiler ABI info - done
-- Check for working C compiler: C:/Program Files (x86)/Microsoft Visual Studio/2019/Enterprise/VC/Tools/MSVC/14.29.30133/bin/Hostx64/x64/cl.exe - skipped
-- Detecting C compile features
-- Detecting C compile features - done
-- Detecting CXX compiler ABI info
-- Detecting CXX compiler ABI info - done
-- Check for working CXX compiler: C:/Program Files (x86)/Microsoft Visual Studio/2019/Enterprise/VC/Tools/MSVC/14.29.30133/bin/Hostx64/x64/cl.exe - skipped
-- Detecting CXX compile features
-- Detecting CXX compile features - done
-- Configuring done
-- Generating done
-- Build files have been written to: F:/test8/test/build

5.F:\test8\test\build> F:\test8\vcpkg\downloads\tools\cmake-3.21.1-windows\cmake-3.21.1-windows-i386\bin\cmake.exe --build .
Microsoft (R) Build Engine version 16.11.2+f32259642 for .NET Framework
Copyright (C) Microsoft Corporation. All rights reserved.

  Checking Build System
  Building Custom Rule F:/test8/test/CMakeLists.txt
  main.cpp
     Creating library F:/test8/test/build/Debug/main.lib and object F:/test8/test/build/Debug/main.exp
  main.vcxproj -> F:\test8\test\build\Debug\main.exe
  Building Custom Rule F:/test8/test/CMakeLists.txt

@cglong ,Is this problem solved?

Thanks for posting this issue. Please reopen this issue if it still be a problem for you.


Crop entities to area

Before:

After:

Fixes #75.

This also incidentally fixes the character's body spilling over the edge of the area as you're taking an exit when your area is rumbling.

It works!


AutoShotWrap grid override

This allows ingestion with gun indexed by unwrapping shot_point for shot_lines.

Codecov Report
Attention: 40 lines in your changes are missing coverage. Please review.

Comparison is base (5db90a6) 84.32% compared to head (ba23ed3) 82.95%.
Report is 2 commits behind head on main.

Files
Patch %
Lines

src/mdio/segy/geometry.py
35.48%
40 Missing :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #338      +/-   ##
==========================================
- Coverage   84.32%   82.95%   -1.37%
==========================================
  Files          46       46
  Lines        2194     2253      +59
  Branches      305      312       +7
==========================================
+ Hits         1850     1869      +19
- Misses        301      341      +40
  Partials       43       43

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

Hi @markspec, can we add at least 1 test for each configuration?

Closes https://github.com/TGSAI/mdio-python/issues/336

@markspec approved but there's a test conflict; please fix so we can merge. thanks!


Tests always exit with status code 0

Regardless of whether tests pass the task will exit with a status code 0.
Verified by cloning and adding the following test
expect(false).to.be.ok;

Related:
https://github.com/babel/babel-library-boilerplate/pull/126

//cc @tkellen, if you ended up using this for Endpoints, then this will affect you. I have a solution that works, but just know that the problem is still outstanding w. the Gulpfile you might be currently using.


IndexError when importing raw CNT file

Describe the bug

Failure to import a particular Neuroscan .cnt file via read_raw_cnt() due to out of bounds indexing in cnt.py line 251
About half of our .cnt files work, half do not due to this error.

Steps and/or code to reproduce
from mne.io import read_raw_cnt

raw_fname = 'raw_eeg_data.cnt'
montage_fname = 'standard-10-5-cap385.elp'
raw = read_raw_cnt(raw_fname, montage=montage_fname, eog=('HEOG', 'VEOG'), emg=('EMG',), preload=True, verbose=True)

Expected results

raw variable successfully populated with a RawCNT object upon reading the .cnt file
no error message

Actual results
Traceback (most recent call last):
  File "/Users/corey/Documents/GitLab/01mne_python_automation/bin/source_localization.py", line 210, in <module>
    raw = read_raw_cnt(raw_fname, montage=montage_fname, eog=('HEOG', 'VEOG'), emg=('EMG',), preload=True, verbose=True)
  File "/anaconda3/envs/mne/lib/python3.6/site-packages/mne/io/cnt/cnt.py", line 95, in read_raw_cnt
    preload=preload, verbose=verbose)
  File "/anaconda3/envs/mne/lib/python3.6/site-packages/mne/io/cnt/cnt.py", line 361, in __init__
    data_format, date_format)
  File "/anaconda3/envs/mne/lib/python3.6/site-packages/mne/io/cnt/cnt.py", line 251, in _get_cnt_info
    stim_channel[event_time - 1] = event_id
IndexError: index 295313 is out of bounds for axis 0 with size 294560

Additional information
sys_info() :
Platform:      Darwin-17.7.0-x86_64-i386-64bit
Python:        3.6.6 |Anaconda, Inc.| (default, Jun 28 2018, 11:07:29)  [GCC 4.2.1 Compatible Clang 4.0.1 (tags/RELEASE_401/final)]
Executable:    /anaconda3/envs/mne/bin/python
CPU:           i386: 8 cores
Memory:        16.0 GB

mne:           0.16.2
numpy:         1.15.0 {blas=mkl_rt, lapack=mkl_rt}
scipy:         1.1.0
matplotlib:    2.2.2 {backend=MacOSX}

sklearn:       0.19.1
nibabel:       2.3.0
mayavi:        4.6.1 {qt_api=pyqt5}
pycuda:        Not found
skcuda:        Not found
pandas:        0.23.3

@corscheid can you share one file with us? ideally a super tiny one?

Hi @agramfort, Thanks for the reply.
My colleague @deepagupta has more authority than I concerning our experiment data; she will be by shortly to respond.

Dear @agramfort, Thank you for your response.
Unfortunately, we deeply regret that we can't share our CNT data file because we are trying to process it for research and may eventually publish from that. So, sharing won't be ethically right or in agreement with our collaborators.
However, we want to try our best to provide any other information if we can. For example, my teammate Corey posted the following error message:
File "/anaconda3/envs/mne/lib/python3.6/site-packages/mne/io/cnt/cnt.py", line 251, in _get_cnt_info
    stim_channel[event_time - 1] = event_id
IndexError: index 295313 is out of bounds for axis 0 with size 294560

Basically, we don't understand why mne python library is generating a stimulating channel and what are the numbers 900 and 75 in the equation below (file cnt.py at line#251):
           event_time = offset - 900 - 75 * n_channels
            event_time //= n_channels * n_bytes
            stim_channel[event_time - 1] = event_id

We believe this must be the source of the issue. However, we can't be entirely sure.

@deepagupta we cannot debug without access to a test file. You can share one file privately with one of us. @larsoner @massich or myself.
if you find the fix it's even better. Then please send us a pull request

If you can came up with a simulated MWE that has np.empty(mydata.shape) as data inside, that would be even better!

I also have an IndexError when importing raw CNT file. I can share the code and data, though not the same ones as the OP used, but hopefully it can help figuring out this issue.
Data is available at http://predict.cs.unm.edu/downloads.php. I'm using the OCD Flankers one in the second row. The code below assumes that file 914flankers.cnt in OCI Flankers/EEG Data is downloaded and saved in the local folder data/.
My code for loading the file is:
montage = mne.channels.read_montage(kind='standard_alphabetic')
mne.io.read_raw_cnt('data/914flankers.cnt', montage)

The error message I got is:
load.py:7: RuntimeWarning:   Could not parse meas date from the header. Setting to None.
  mne.io.read_raw_cnt('data/914flankers.cnt', montage, verbose=True)
Traceback (most recent call last):
  File "load.py", line 7, in <module>
    mne.io.read_raw_cnt('data/914flankers.cnt', montage, verbose=True)
  File "/Users/fanyang1/anaconda3/envs/mne/lib/python3.6/site-packages/mne/io/cnt/cnt.py", line 95, in read_raw_cnt
    preload=preload, verbose=verbose)
  File "/Users/fanyang1/anaconda3/envs/mne/lib/python3.6/site-packages/mne/io/cnt/cnt.py", line 361, in __init__
    data_format, date_format)
  File "/Users/fanyang1/anaconda3/envs/mne/lib/python3.6/site-packages/mne/io/cnt/cnt.py", line 251, in _get_cnt_info
    stim_channel[event_time - 1] = event_id
IndexError: index 583988 is out of bounds for axis 0 with size 582560

I've downloaded the file and I can reproduce the error
import mne
import os.path as op

file_name = op.join(op.dirname(mne.__file__), '..', 'sandbox', 'data',
                    '914flankers.cnt')

montage = mne.channels.read_montage(kind='standard_alphabetic')
raw = mne.io.read_raw_cnt(file_name, montage)


Right space next to string in Arabic is not consistent for right-aligned strings

The timeline has strings both right and left aligned. With using the font shape engine and fpdf2 this usually works fine, but some strings have a little extra space to it. After some testing I found the problem: When latin strings (or even numbers) are rendered, the font shape engine switches to latin. In the next round of determining the string width, the latin setting is used to measure the width, and after that the language is changed. In a sequence where a latin character is drawn to the pdf prior to the determination of the string width for a right aligned string the return value is not correct.
The problem was reported at fpdf2: https://github.com/py-pdf/fpdf2/issues/1231

Isn't that fixed with a later edition of fpdf2?

Yes, fixed on July 27th by andersonhc at fpdf2.