Hubot phabricator data corrupt upon restart

We're encountering an issue where upon restarting hubot the phabricator data from redis does not seem to be read in properly to this module.
Specifically, the phabricator data with feeds are visible in redis data under each project, and for feedall the rooms are listed (I've obfuscated the data):
myuser@myhost:~# redis-cli get "hubot:storage" | jq .phabricator
{
  "projects": {
    "*": {
      "feeds": [
        "noisybotroom"
      ]
    },
    "Project With Spaces": {
      "name": "Project With Spaces",
      "phid": "PHID-PROJ-xxxxxx",
      "parent": null,
      "columns": {
        "this_iteration": "PHID-PCOL-xxxxxx",
        "upcoming": "PHID-PCOL-xxxxxx",
        "backlog": "PHID-PCOL-xxxxxx"
      },
      "feeds": [
        "room1"
      ]
    },
    "Project-With-Dashes": {
      "name": "Project-With-Dashes",
      "phid": "PHID-PROJ-xxxxxx",
      "parent": null,
      "columns": {
        "backlog": "PHID-PCOL-xxxxxx",
        "working": "PHID-PCOL-xxxxxx",
        "waiting_on_others": "PHID-PCOL-xxxxxx"
      },
      "feeds": [
        "room2"
      ]
    },
    "Project3": {
      "name": "Project3",
      "phid": "PHID-PROJ-xxxxxx",
      "parent": null,
      "columns": {
        "waiting_on_others": "PHID-PCOL-xxxxxx",
        "backlog": "PHID-PCOL-xxxxxx"
      },
      "feeds": [
        "room2"
      ]
    },
    "Project4": {
      "name": "Project4",
      "phid": "PHID-PROJ-xxxxxx",
      "parent": null,
      "columns": {
        "current_iteration": "PHID-PCOL-xxxxxx",
        "backlog": "PHID-PCOL-xxxxxx",
        "waiting": "PHID-PCOL-xxxxxx"
      },
      "feeds": [
        "room3"
      ]
    }
  },
  "aliases": {
    "project_1": "Project With Spaces",
    "project_2": "Project-With-Dashes",
    "project_3": "Project3",
    "project_4": "Project4"
  },
  "templates": {},
  "blacklist": [],
  "users": {
    "me": {
      "name": "me",
      "id": "me",
      "phid": "PHID-USER-123456"
    },
    "SLACKID": {
      "name": "me",
      "id": "SLACKID",
      "phid": "PHID-USER-123456"
    }
  },
  "bot_phid": "hubot",
  "alerts": {
    "me": "PHID-USER-789012"
  }
}

Although the data is properly in redis, nothing will be fed into rooms after the hubot is restarted. I added debugging to console.log incoming feed events in scripts/phabs_feeds.coffee and verified events come through, but just aren't fed anywhere after a restart.
Asking hubot to feedall to a room that is already in redis data will recreate the feed (redis data does not change, but hubot says: Ok, all feeds will be announced on 'noisybotroom'.), and asking hubot to create a project -> room feed that should already exist will sometimes result in the feed being recreated (redis data does not change, but hubot says: Ok, 'Project1' is now feeding 'room1'.), but sometimes this seems to result in removal of the nested "feeds" object for that project, no hubot output to the user, and the exception below being thrown by hubot:
May  4 19:39:46 hubot[16077]: error: no_text
May  4 19:39:46 hubot[16077]: Unhandled rejection Error: no_text
May  4 19:39:46 hubot[16077]: at handleHttpResponse (/src/hubot/node_modules/@slack/client/lib/clients/transports/call-transport.js:105:17)
May  4 19:39:46 hubot[16077]: at handleTransportResponse (/src/hubot/node_modules/@slack/client/lib/clients/transports/call-transport.js:155:19)
May  4 19:39:46 hubot[16077]: at apply (/src/hubot/node_modules/@slack/client/node_modules/lodash/lodash.js:499:17)
May  4 19:39:46 hubot[16077]: at wrapper (/src/hubot/node_modules/@slack/client/node_modules/lodash/lodash.js:5356:16)
May  4 19:39:46 hubot[16077]: at Request.handleRequestTranportRes (/src/hubot/node_modules/@slack/client/lib/clients/transports/request.js:20:5)
May  4 19:39:46 hubot[16077]: at apply (/src/hubot/node_modules/@slack/client/node_modules/lodash/lodash.js:499:17)
May  4 19:39:46 hubot[16077]: at Request.wrapper [as _callback] (/src/hubot/node_modules/@slack/client/node_modules/lodash/lodash.js:5356:16)
May  4 19:39:46 hubot[16077]: at Request.self.callback (/src/hubot/node_modules/request/request.js:188:22)
May  4 19:39:46 hubot[16077]: at Request.emit (events.js:110:17)
May  4 19:39:46 hubot[16077]: at Request.<anonymous> (/src/hubot/node_modules/request/request.js:1171:10)
May  4 19:39:46 hubot[16077]: at Request.emit (events.js:107:17)
May  4 19:39:46 hubot[16077]: at IncomingMessage.<anonymous> (/src/hubot/node_modules/request/request.js:1091:12)
May  4 19:39:46 hubot[16077]: at IncomingMessage.g (events.js:199:16)
May  4 19:39:46 hubot[16077]: at IncomingMessage.emit (events.js:129:20)
May  4 19:39:46 hubot[16077]: at _stream_readable.js:908:16
May  4 19:39:46 hubot[16077]: at process._tickCallback (node.js:355:11)

This is my first time using coffeescript so am having a hard time debugging this myself, but I imagine the issue exists in the constructor where the data is being loaded or else defaulted https://github.com/Gandi/hubot-phabs/blob/master/lib/phabricator.coffee#L65-L82 We have projects with spaces in the name and also dashes in the names. The one with a space in the name is the one I'm seeing wipe the nested "feeds" object when I try to re-add it, but I've only restarted a handful of times (due to having to keep the bot running so people can use it) so I'm not sure if it's actually the problem, or just the one I happen to be noticing the most.
Right now the only way I've found to reliably fix this is by dropping the redis db with hubot data, restarting hubot to have it recreate the data structure anew, then sending the commands to hubot to have it rediscover the projects and recreate the feeds. Obviously this isn't a great solution since we need to be able to back up the config and stop/start hubot whenever we want.
Let me know if you need any more information to look into this, or if you have any suggestions to try/patches to test and I can look/test it out!

This is pretty weird.
About project names, spaces and dashes should not be a problem, on my side I have plenty of them and there is no issue.
Maybe you are right about the brain loading, I applied https://github.com/Gandi/hubot-phabs/commit/e06adc1169a2932ee28ee7fbf427cbcb36f7c15c can you test this? Those default values are intended to be set for people that had brains from previous versions, it's like a migration path, but it fits better in the .on loaded'` block. On my side I use the file-brain and I have no issue. Different context.
About the no_text exception this is a different concern, probably related to the slack adapter. If we find a way to fix the feeding, and it's still there, we can talk more about this one.
Can you first try to change the defaults modification position (or even remove the default setup altogether, ie. everything after storageLoaded()) and see if it fares batter?

Unfortunately, that didn't seem to do it. Same issues.
I added console.log @data.projects after the @robot.logger.debug '---- Phabricator Data Loaded.' line, and can see tailing syslog that it seems to have the proper data in the object itself, so I'm not sure what's going wrong...
Are you able to recreate this issue? Or is it just something with my setup?

Okay, I tried the hubot-redis-brain and finally reproduced it. I'm not sure why it came to behave like this, but I made it work.
Can you try with version 2.4.9 ?

Success! No more redis amnesia. Thank you so much for the quick turnaround on this! :100:


with verbatimModuleSyntax: true type only imports don't work in dev mode

Describe the bug
with "verbatimModuleSyntax": true in tsconfig.json i get the following errors when using type imports from a type-only package (@types/geojson):
[plugin:vite:import-analysis] Failed to resolve import "geojson" from "pages/index.vue". Does the file exist?

setting "verbatimModuleSyntax": true makes everything render fine.
also, this is only an issue in development, not in a production build.
Reproduction
https://github.com/stefanprobst/issue-vite-type-imports
Steps to reproduce

npm install
npm run dev
open http://localhost:3000

System Info
System:
    OS: Linux 6.4 KDE neon 5.27 5.27
    CPU: (16) x64 AMD Ryzen 7 5800U with Radeon Graphics
    Memory: 23.20 GB / 30.66 GB
    Container: Yes
    Shell: 5.1.16 - /bin/bash
  Binaries:
    Node: 18.17.1 - ~/.volta/tools/image/node/18.17.1/bin/node
    Yarn: 1.22.19 - ~/.volta/tools/image/yarn/1.22.19/bin/yarn
    npm: 9.6.7 - ~/.volta/tools/image/node/18.17.1/bin/npm
    pnpm: 8.7.4 - ~/.volta/bin/pnpm
  Browsers:
    Chrome: 116.0.5845.187
    Firefox Nightly: 119.0a1

Used Package Manager
npm
Logs
No response
Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guidelines.
[X] Read the docs.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Make sure this is a Vite issue and not a framework-specific issue. For example, if it's a Vue SFC related bug, it should likely be reported to vuejs/core instead.
[X] Check that this is a concrete bug. For Q&A open a GitHub Discussion or join our Discord Chat Server.
[X] The provided reproduction is a minimal reproducible example of the bug.

In a ts file, it throws an error in both dev and build. The error is expected because verbatimModuleSyntax: true will make import {  type Polygon } from 'geojson' to be transpiled to import {} from 'geojson'.
But in a vue file, it doesn't throw an error in build. That's a bug. I guess it's somehow not respecting the tsconfig.json.


removePathAccents = true work not stable

I use removePathAccents = true in config.toml file for remove accented in URL, but seem it work not stable with some characters.

My permalinks config:
[permalinks]
  article = "/:year/:month/:title/"

Same for German. Related Issue: https://github.com/gohugoio/hugo/issues/3476 (closed by bot)

I am facing a same issue :D removePathAccents  cannot detect "đ" as a non-ACSII charactor :D
It's not safe for a URL , I think

Similar thing with Polish language. We use accented characters like:
ęóąśłżźćń ĘÓĄŚŁŻŹĆŃ
they are converted to:
eoasłzzcn-eoasłzzcn
Almost good, just ł and bigger version Ł are not transformed properly.
I can generate slug on my own... but it would be nice if we could cover it.
Please let me know how can I help to get it fixed.

Hi. Have you found any solution for this issue?
removePathAccents is not yet supported in Hugo's documentation


Merlin doesn't use reason syntax in "git-diff view"

First, thank you for this great extension!
I run into a small Problem. If you view a reason file in the "git-diff view", Merlin seems to expect OCaml and not Reason.
Here is an example Screenshot:

Slightly related: Would it be possible to add an option to disable problems/warnings from Merlin? I use a VSCode with Bucklescript and the errors get reported twice in this combi.

Thanks for the report. I'll look into it.
Concerning duplicate diagnostic messages, I will consider adding an option to disable merlin. However, I am already planning several changes to the diagnostics system that may resolve this in a more general fashion in an upcoming release.


OpenApi + Ktor + Resources

This PR introduces a new module which integrates tegral-openapi-ktor with type-safe routing. With this package, the OpenAPI descriptions can be part of the resource definition:
@Resource("/hello/{name}")
class Hello(val name: String) {
    companion object: OpenApiDescription {
        override val openApi = {
            description = "Returns a greeting"
        }
    }
}

There's a bit of ceremony in making the companion object implement an interface, but I couldn't find a better way to attach a piece of static information to the class, while at the same time giving access to the whole OperationDsl.

Hi! Thank you very much for your PR! This is a great idea. I have a few ideas:
Here's a possible alternative syntax to get less ceremony:
// In tegral-openapi-ktor-resources
fun describeResource(operationDsl: OperationDsl.() -> Unit): OpenApiDescription {
    return object : OpenApiDescription {
        override val openApi = operationDsl
    }
}

// In the "consumer" code
@Resource("/hello/{name}")
class Hello(val name: String) {
    companion object : OpenApiDescription by describeResource({
        summary = "Hello there"
    })
}

It saves 2 lines and an indentation, but I think it mostly helps for readability. The syntax is admittedly a bit weird (you can't just put describeResource {, the compiler will think that the { represents the beginning of the object body instead of being the lambda parameter of the describeResource function).
I also have some suggestions around the naming of the new symbols. I mostly want to avoid confusion with the non-resources variants for describing endpoints:

OpenApiDescription -> ResourceDescription
description() -> descriptionFromCompanionObject()

What do you think?
I can take care of making these changes, adding tests, docs and changelog entries if that's okay for you. My apologies for the lack of instructions/guidance around contributions, I have to admit I didn't expect people to find this project and contribute to it 😅
Also, as a later addition to Tegral, I think we can infer a lot of information from the things contained in the resource object (e.g. automatically create query parameters, path parameters, body information, etc. from the properties in the class). I'll create an issue about this once this is merged.
Once again, thank you for your contribution!

@utybo Thanks to you for your nice project! I can tell you we were using it already under its previous name, because it requires so little additional preparation to get an OpenAPI spec from the Ktor routes.
I agree with all the changes you propose. If you don’t mind, I would prefer to keep both ways to add a description, both by overriding openApi and by delegating to describeResource.
I can take care of this tonight, or you can do it, as you prefer.

Your comment made my day, thank you!

I agree with all the changes you propose. If you don’t mind, I would prefer to keep both ways to add a description, both by overriding openApi and by delegating to describeResource.

Sure thing!
I'll take care of making the necessary changes 👍🏻

I've just pushed a new idea that came through my mind; if you prefer to not have it, feel free to remove it. The idea is to bring a bit more type-safety by allowing people to use a property reference instead of a name.
@Resource("/hello/{name}") class Hello(val name: String) {
  companion object : ResourceDescription {
    override val openApi = {
      pathParameter(Hello::name) {
        description = "The name"
      }
    }
  }
}

This may also be better as part of the DSL package itself, as it works better with Resources, but doesn't really depend on it.

I think they belong in the resources package, but that makes it annoyingly inconsistent with string-based parameters as the function can't be made infix (at least until we get context receivers in Kotlin).
I'd say these should be included at a later date (i.e. after this PR), and should probably wait for context receivers in Kotlin and the fancier "automagically infer OpenAPI information from resource classes" functionality. In the mean time, if people want these helpers, they can add their own extension functions (whose implementation is fairly trivial).
Also, I have been unable to use the syntax you've used in this PR for overriding the openApi property, I've always had to add the : OperationDsl.() -> Unit type after it, otherwise I'd get errors in IntelliJ. Is there something I'm missing here?

Ok, I've now removed it.
It might well be the case that IntelliJ added the type for me, and I wasn't aware of that. In any case, I think this shows that describeResource works better :)

Everything looks good to me, I'll wait for the CI and preview website before merging

One small question: how would you feel about changing the names of getD, postD, and so on to simply get, post...? So people would not import the ktor.server.resources package, but Tegral's one.

One small question: how would you feel about changing the names of getD, postD, and so on to simply get, post...? So people would not import the ktor.server.resources package, but Tegral's one.

Apologies on the delay for the response. I'm a bit split on this one. On one hand it's a very interesting possibility for the future when "automatic documentation" happens (so that you can get documentation on resources "for free" by just changing your imports and nothing else), but it may end up being very confusing for end-users.

I'd say we should stay with the xyzD variants. I'm pretty sure the import clashing would be confusing for users (just like resources and non-resources of get/post/etc are already confusing). Making the difference obvious is probably for the better. I'll merge this PR today and try to get the 0.0.3 release done this weekend, once again apologies for the delay!

Merged, thank you for your contribution!


[Feature request] History of videos watched

Thanks for the nice program!
I couldn't figure out how watched videos can be stored over different sessions, i.e. that after restarting, y-v such videos are still marked as bold.
I found 'history' in the config file, but apparently that's only for the search history.
Any chance to implement this or am I missing something?

Thanks for the feature request. This is now implemented in the commit https://github.com/trizen/youtube-viewer/commit/02def284b20ab1e6e90b4947cb0ded50c483cbcc.

Well that was... fast!
Cheers :)


Local license fix

As per https://github.com/bucharest-gold/license-reporter/issues/196 this is a simple update.
Was unsure about the lib/file-writer.js change @helio-frota can you check if that should have been included? All tests are passing locally and verified with a run on a package I had where I forced a delete on the local license and works as intended.

Sorry I deleted the package.lock as I thought it was an erroneous file!!

Great! Thanks!


Re-worked OpusFileStream and added example

This Pull Request includes the changes made to OpusFileStream and the example that demonstrated there was an issue with the original code.
See the discussion in issue #33

I'm sorry, but I don't really have the time to look through and test all of your changes.
Still, I don't want users of PyOgg to miss out or your work to be wasted, so I'll merge it anyway.

Thanks for your trust in my work :o)  It's appreciated.
Matthew


[Bug] macOS' Gatekeeper mechanism does not allow the app to run

Problem description
The app could not run on my Mac. Error messages that appeared were "The application 'About This Hack' can't be opened.", and "'About This Hack' will damage your computer. You should move it to the Trash.". Moreover, I checked the app through Apparency, and I find out that Gatekeeper did not let the app run, but it was code-signed.
Screenshot

Specifications
OS: Ventura 13.0 beta
Hardware: M1 MacBook Pro
App version: 0.5.0

@pads1 this isn't a bug. This is related to the fact that I don't have the paid Apple Developer license to sign the app, so apple often quarantines the app because of that. See solution in #6

@pads1 if that didn't work, please try the latest version: 0.6.0. All the issues should be resolved.


Fontmake can not compile some ufo files in noto-sources

Affected files are :
noto-source/src/NotoSansCham/NotoSansCham-Regular.ufo
noto-source/src/NotoSansCham/NotoSansCham-Bold.ufo
noto-source/src/NotoSansLao/NotoSansLao-Regular.ufo
noto-source/src/NotoSansLao/NotoSansLao-Bold.ufo
and probably more.
The error is
(dev) bosong-macbookpro:NotoSansLao bosong$ fontmake -u NotoSansLao-Bold.ufo/
INFO:fontmake.font_project:Building OTFs
INFO:fontmake.font_project:Decomposing glyphs for NotoSansLao-Bold
INFO:fontmake.font_project:Removing overlaps for NotoSansLao-Bold
Traceback (most recent call last):
File "/Users/bosong/i18n/fonttools/dev/bin/fontmake", line 11, in
load_entry_point('fontmake', 'console_scripts', 'fontmake')()
File "/Users/bosong/i18n/fontmake/Lib/fontmake/main.py", line 183, in main
ufo_paths, is_instance=args.pop('masters_as_instances'), **args)
File "/Users/bosong/i18n/fontmake/Lib/fontmake/font_project.py", line 435, in run_from_ufos
ufos, remove_overlaps, mti_paths=mti_paths, **kwargs)
File "/Users/bosong/i18n/fontmake/Lib/fontmake/font_project.py", line 146, in build_otfs
self.remove_overlaps(ufos)
File "/Users/bosong/i18n/fonttools/Lib/fontTools/misc/loggingTools.py", line 372, in wrapper
return func(*args, **kwds)
File "/Users/bosong/i18n/fontmake/Lib/fontmake/font_project.py", line 87, in remove_overlaps
union(contours, glyph.getPointPen())
File "/Users/bosong/i18n/fonttools/dev/lib/python2.7/site-packages/booleanOperations/booleanOperationManager.py", line 104, in union
return _performOperation("union", contours, [], outPen)
File "/Users/bosong/i18n/fonttools/dev/lib/python2.7/site-packages/booleanOperations/booleanOperationManager.py", line 64, in _performOperation
subjectInputContours = [InputContour(contour) for contour in subjectContours if contour and len(contour) > 1]
File "/Users/bosong/i18n/fonttools/dev/lib/python2.7/site-packages/booleanOperations/flatten.py", line 72, in init
self.segments = _convertPointsToSegments(points)
File "/Users/bosong/i18n/fonttools/dev/lib/python2.7/site-packages/booleanOperations/flatten.py", line 440, in _convertPointsToSegments
willBeReversed=willBeReversed
File "/Users/bosong/i18n/fonttools/dev/lib/python2.7/site-packages/booleanOperations/flatten.py", line 174, in init
for pt1, pt2 in decomposeQuadraticSegment(pointCoordinates[1:]):
File "/Users/bosong/i18n/fonttools/Lib/fontTools/pens/basePen.py", line 329, in decomposeQuadraticSegment
assert n > 0
AssertionError
However, the fontmake can compile the intermediate ufo files generated by its self in master_ufo folder.

booleanOperations currently does not support removing overlaps from outlines containing (TrueType) quadratic curves.
There's a "quadratics" branch, but it's not merged into master yet:
https://github.com/typemytype/booleanOperations/tree/quadratics

I see. So IIUC, overlaps are the intersection area of different contours in a glyph. Why do we need to remove overlaps and why does source design file allow overlaps?
Is quadratics brach still under developing and can not merged to master now?

On Feb 17, 2017, at 4:42 PM, Cosimo Lupo<EMAIL_ADDRESS>wrote:
booleanOperations currently does not support removing overlaps from outlines containing (TrueType) quadratic curves.
There's a "quadratics" branch, but it's not merged into master yet:
https://github.com/typemytype/booleanOperations/tree/quadratics
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

Quadratics don't require overlaps to be removed in the first place :)
On Feb 18, 2017 10:25 PM, "Bo Song"<EMAIL_ADDRESS>wrote:

I see. So IIUC, overlaps are the intersection area of different contours
in a glyph. Why do we need to remove overlaps and why does source design
file allow overlaps?
Is quadratics brach still under developing and can not merged to master
now?

On Feb 17, 2017, at 4:42 PM, Cosimo Lupo<EMAIL_ADDRESS>wrote:
booleanOperations currently does not support removing overlaps from
outlines containing (TrueType) quadratic curves.
There's a "quadratics" branch, but it's not merged into master yet:
https://github.com/typemytype/booleanOperations/tree/quadratics
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/googlei18n/fontmake/issues/260#issuecomment-280858218,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAP9y1f1XYVmeSZnJDH7id_zoN_O_sEvks5rdyJrgaJpZM4ME54g
.

@davelab6, not even when building CFF fonts from quadratic sources, which (I think) is what happened here? Of course it’s a weird case, but why shouldn’t fontmake support it?

If that's the case then yes of course cff requires overlap removal. But
when I did read and now just reread the OP, I assumed that it was
outputting TTF flavour... :) I'm curious where cff was indicated
On Feb 19, 2017 1:19 PM, "Sascha Brawer"<EMAIL_ADDRESS>wrote:

@davelab6 https://github.com/davelab6, not even when building CFF fonts
from quadratic sources, which (I think) is what happened here? Of course
it’s a weird case, but why shouldn’t fontmake support it?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/googlei18n/fontmake/issues/260#issuecomment-280902568,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAP9y5mOeoiD0fE8wTW0vZjnh9QHeN2Jks5rd_QLgaJpZM4ME54g
.

I'm curious where cff was indicated

the OP typed fontmake -u NotoSansLao-Bold.ufo, so fontmake by default tries to compile both OTF and TTF, in that order.
However, the AssertionError in the traceback above was not caused by fontmake trying to create a CFF-OTF from a UFO containing TrueType outlines (this is not supported yet, depends on https://github.com/googlei18n/cu2qu/pull/29 being landed first).
The problem is that booleanOperations can't remove overlaps from contours with TrueType outlines ("qcurve"), and fontmake by default attempts to remove the overlaps, unless told otherwise with --keep-overlaps option.
I haven't checked the source files, but if they contain quadratic contours, it's likely that these were generated by decompiling binary TTF rather than drawn that way, and hence they may already have overlaps removed. Passing options --keep-overlaps -o ttf should complete successfully.

On Feb 20, 2017 12:32 AM, "Cosimo Lupo"<EMAIL_ADDRESS>wrote:
I'm curious where cff was indicated
the OP typed fontmake -u NotoSansLao-Bold.ufo, so fontmake by default tries
to compile both OTF and TTF, in that order.
OK cool :)
However, the AssertionError in the traceback above was not caused by
fontmake trying to create a CFF-OTF from a UFO containing TrueType outlines
(this is not supported yet, depends on googlei18n/cu2qu#29
https://github.com/googlei18n/cu2qu/pull/29 being landed first).
Aha :)
The problem is that booleanOperations can't remove overlaps from contours
with TrueType outlines ("qcurve"), and fontmake by default attempts to
remove the overlaps, unless told otherwise with --keep-overlaps option.
It seems to me that removing overlaps by default in TTF is suboptimal. Why
is this done?
I haven't checked the source files, but if they contain quadratic contours,
it's likely that these were generated by decompiling binary TTF rather than
drawn that way, and hence they may already have overlaps removed. Passing
options --keep-overlaps -o ttf should complete successfully
Sweet 🎉

Tried to pass* --keep-overlaps *option to fontmake but it still fails,
though this time it fails at a different stage rather than removing
overlapping. (NotoSansCham-Regular/Bold, NotoSansLao-Regular/Bold and
NotoSerifLao-Regular/Bold all fails with same error. Error stack is
attached below)
The reason is that these problematic fonts do not contain xHeight and
capHeight attribute in its fontinfo.plist file. I added these two
attributes to NotoSansCham-Regular.ufo/ and it is compiled successfully
with --keep-overlaps option.
AFAIK, this issue contains two root problems:

The master branch of fontmake can not remove qcurve overlaps.
The ChamRegular/Bold and LaoRegular/Bold ufo files are problematic. They
do not contain xHeight and capHeight (and probably more optional tags).

For 1, it seems no recent progress on merging the quadratic branch into
master https://github.com/typemytype/booleanOperations/issues/26 . What
will the potential problem be if we build the font with --keep-overlaps
option?
For 2, who should be the source of truth that can fix these font? Can we
reach out the font designer?
FYI:
Error stack:
(dev) bosong-macbookpro:NotoSansCham bosong$ fontmake -u
NotoSansCham-Regular.ufo --keep-overlaps -o ttf
INFO:fontmake.font_project:Building TTFs
INFO:fontmake.font_project:Decomposing glyphs for NotoSansCham-Book
INFO:fontmake.font_project:Converting curves for NotoSansCham-Book
INFO:cu2qu.ufo:New spline lengths:
INFO:fontmake.font_project:Saving TTF for NotoSansCham-Book
Traceback (most recent call last):
File "/Users/bosong/i18n/fonttools/dev/bin/fontmake", line 11, in
load_entry_point('fontmake', 'console_scripts', 'fontmake')()

File "/Users/bosong/i18n/fontmake/Lib/fontmake/main.py", line 183, in
main
ufo_paths, is_instance=args.pop('masters_as_instances'), **args)

File "/Users/bosong/i18n/fontmake/Lib/fontmake/font_project.py", line
443, in run_from_ufos
mti_paths=mti_paths, **kwargs)

File "/Users/bosong/i18n/fontmake/Lib/fontmake/font_project.py", line
165, in build_ttfs
self.save_otfs(ufos, ttf=True, **kwargs)

File "/Users/bosong/i18n/fonttools/Lib/fontTools/misc/loggingTools.py",
line 372, in wrapper
return func(*args, **kwds)

File "/Users/bosong/i18n/fontmake/Lib/fontmake/font_project.py", line
264, in save_otfs
convertCubics=False, optimizeCff=subroutinize)

File
"/Users/bosong/i18n/fonttools/dev/lib/python2.7/site-packages/ufo2ft/init.py",
line 42, in compileTTF
return compileOTF(ufo, outlineCompilerClass=outlineCompilerClass,

**kwargs)
File
"/Users/bosong/i18n/fonttools/dev/lib/python2.7/site-packages/ufo2ft/init.py",
line 27, in compileOTF
otf = outlineCompiler.compile()

File
"/Users/bosong/i18n/fonttools/dev/lib/python2.7/site-packages/ufo2ft/outlineOTF.py",
line 91, in compile
self.setupTable_OS2()

File
"/Users/bosong/i18n/fonttools/dev/lib/python2.7/site-packages/ufo2ft/outlineOTF.py",
line 434, in setupTable_OS2
xHeight = getAttrWithFallback(font.info, "xHeight")

File
"/Users/bosong/i18n/fonttools/dev/lib/python2.7/site-packages/ufo2ft/fontInfoData.py",
line 420, in getAttrWithFallback
value = staticFallbackData[attr]

KeyError: u'xHeight'
On Mon, Feb 20, 2017 at 2:24 AM, Dave Crossland<EMAIL_ADDRESS>wrote:

On Feb 20, 2017 12:32 AM, "Cosimo Lupo"<EMAIL_ADDRESS>wrote:
I'm curious where cff was indicated
the OP typed fontmake -u NotoSansLao-Bold.ufo, so fontmake by default tries
to compile both OTF and TTF, in that order.
OK cool :)
However, the AssertionError in the traceback above was not caused by
fontmake trying to create a CFF-OTF from a UFO containing TrueType outlines
(this is not supported yet, depends on googlei18n/cu2qu#29
https://github.com/googlei18n/cu2qu/pull/29 being landed first).
Aha :)
The problem is that booleanOperations can't remove overlaps from contours
with TrueType outlines ("qcurve"), and fontmake by default attempts to
remove the overlaps, unless told otherwise with --keep-overlaps option.
It seems to me that removing overlaps by default in TTF is suboptimal. Why
is this done?
I haven't checked the source files, but if they contain quadratic contours,
it's likely that these were generated by decompiling binary TTF rather than
drawn that way, and hence they may already have overlaps removed. Passing
options --keep-overlaps -o ttf should complete successfully
Sweet 🎉
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/googlei18n/fontmake/issues/260#issuecomment-281041423,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AEGl792c1mZJ7BYS1nkIDeraMm1Vu7F9ks5reWnagaJpZM4ME54g
.

--
Sincerely,
Bo Song
Master Student in Computer Science
Yale University

@zjusbo : we are replacing Lao with the new sources

I'm closing this issue since:

Bug in Cham is fixed by https://github.com/googlei18n/noto-source/pull/67
2.Lao is replaced with new sources
Passing options --keep-overlaps -o ttf should complete successfully.


do not merge

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 100.00%. Comparing base (7058844) to head (54a34cc).

Additional details and impacted files

@@            Coverage Diff            @@
##              main     #1000   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files          105       105
  Lines         4961      4961
=========================================
  Hits          4961      4961


NTR: the config value was added two times

i guess that some things from bootstrap/setup are called during update
because after deleting the insert it was still inserted one time

Coverage increased (+0.06%) to 41.723% when pulling 33fc2cfe8632e8af02b678e0bb31f5abfbe21fe5 on NTR_fix_trigger_config into 0680ea34f82149a3bc59b51dde276d2e1b61d27c on master.


Does errors.add work for Neo4j::ActiveNode?

While checking for an error within the model, I want to add a specific message to the error (below is a simple check if user is already registered with the submitted email). But errors.add  somehow does not add, it does not thro any error, the message is just not added.
I'd appretiate if you can point to something that might be missing or wrong here?
class Invitation
  include Neo4j::ActiveNode
  property :token, type: String
  property :recipient_email, type: String
  index :token
  index :recipient_email

  private

  def recipient_is_not_registered
    puts "is already registered"
    errors.add(:recipient_email, 'is already registered') if User.find_by(:email => recipient_email)
  end
end

Try self.errors.add, I seem to have used that in some code with a Neo4j project in the past.
You'll get better performance if you check for the presence of the user with User.where(email: recipient_email).limit(1).empty?. It runs a count in Cypher instead of returning a node.

Thanks for the tip Chris,
So I  actually see the errors added.
errors.add(:recipient_email, 'is already registered') if User.where(email: recipient_email).limit(1).empty?
puts self.errors.full_messages
> Recipient email is already registered

...somehow can't display them via the controller:
format.html { redirect_to root_path, :notice => @invitation.errors.full_messages }

Ok did this instead, now it is fine.
format.html { redirect_to root_path, notice: @invitation.errors.full_messages.join(",") }


Exception errors when try to fetch articles

Issuing the command
 python3 /Users/dummy/scihub.py/scihub/scihub.py -d 10.1016/j.annemergmed.2018.04.027
I get the following errors
Traceback (most recent call last):
  File "/usr/local/lib/python3.6/site-packages/urllib3/connection.py", line 141, in _new_conn
    (self.host, self.port), self.timeout, **extra_kw)
  File "/usr/local/lib/python3.6/site-packages/urllib3/util/connection.py", line 60, in create_connection
    for res in socket.getaddrinfo(host, port, family, socket.SOCK_STREAM):
  File "/usr/local/Cellar/python/3.6.5/Frameworks/Python.framework/Versions/3.6/lib/python3.6/socket.py", line 745, in getaddrinfo
    for res in _socket.getaddrinfo(host, port, family, type, proto, flags):
socket.gaierror: [Errno 8] nodename nor servname provided, or not known

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/usr/local/lib/python3.6/site-packages/urllib3/connectionpool.py", line 601, in urlopen
    chunked=chunked)
  File "/usr/local/lib/python3.6/site-packages/urllib3/connectionpool.py", line 357, in _make_request
    conn.request(method, url, **httplib_request_kw)
  File "/usr/local/Cellar/python/3.6.5/Frameworks/Python.framework/Versions/3.6/lib/python3.6/http/client.py", line 1239, in request
    self._send_request(method, url, body, headers, encode_chunked)
  File "/usr/local/Cellar/python/3.6.5/Frameworks/Python.framework/Versions/3.6/lib/python3.6/http/client.py", line 1285, in _send_request
    self.endheaders(body, encode_chunked=encode_chunked)
  File "/usr/local/Cellar/python/3.6.5/Frameworks/Python.framework/Versions/3.6/lib/python3.6/http/client.py", line 1234, in endheaders
    self._send_output(message_body, encode_chunked=encode_chunked)
  File "/usr/local/Cellar/python/3.6.5/Frameworks/Python.framework/Versions/3.6/lib/python3.6/http/client.py", line 1026, in _send_output
    self.send(msg)
  File "/usr/local/Cellar/python/3.6.5/Frameworks/Python.framework/Versions/3.6/lib/python3.6/http/client.py", line 964, in send
    self.connect()
  File "/usr/local/lib/python3.6/site-packages/urllib3/connection.py", line 166, in connect
    conn = self._new_conn()
  File "/usr/local/lib/python3.6/site-packages/urllib3/connection.py", line 150, in _new_conn
    self, "Failed to establish a new connection: %s" % e)
urllib3.exceptions.NewConnectionError: <urllib3.connection.HTTPConnection object at 0x103d60240>: Failed to establish a new connection: [Errno 8] nodename nor servname provided, or not known

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/usr/local/lib/python3.6/site-packages/requests/adapters.py", line 440, in send
    timeout=timeout
  File "/usr/local/lib/python3.6/site-packages/urllib3/connectionpool.py", line 639, in urlopen
    _stacktrace=sys.exc_info()[2])
  File "/usr/local/lib/python3.6/site-packages/urllib3/util/retry.py", line 388, in increment
    raise MaxRetryError(_pool, url, error or ResponseError(cause))
urllib3.exceptions.MaxRetryError: HTTPConnectionPool(host='sci-hub.hk', port=80): Max retries exceeded with url: /29885904 (Caused by NewConnectionError('<urllib3.connection.HTTPConnection object at 0x103d60240>: Failed to establish a new connection: [Errno 8] nodename nor servname provided, or not known',))

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/Users/dummy/scihub.py/scihub/scihub.py", line 304, in <module>
    main()
  File "/Users/dummy/scihub.py/scihub/scihub.py", line 268, in main
    result = sh.download(args.download, args.output)
  File "/usr/local/lib/python3.6/site-packages/retrying.py", line 49, in wrapped_f
    return Retrying(*dargs, **dkw).call(f, *args, **kw)
  File "/usr/local/lib/python3.6/site-packages/retrying.py", line 212, in call
    raise attempt.get()
  File "/usr/local/lib/python3.6/site-packages/retrying.py", line 247, in get
    six.reraise(self.value[0], self.value[1], self.value[2])
  File "/usr/local/lib/python3.6/site-packages/six.py", line 693, in reraise
    raise value
  File "/usr/local/lib/python3.6/site-packages/retrying.py", line 200, in call
    attempt = Attempt(fn(*args, **kwargs), attempt_number, False)
  File "/Users/dummy/scihub.py/scihub/scihub.py", line 126, in download
    data = self.fetch(identifier)
  File "/Users/dummy/scihub.py/scihub/scihub.py", line 140, in fetch
    url = self._get_direct_url(identifier)
  File "/Users/dummy/scihub.py/scihub/scihub.py", line 183, in _get_direct_url
    else self._search_direct_url(identifier)
  File "/Users/dummy/scihub.py/scihub/scihub.py", line 190, in _search_direct_url
    res = self.sess.get(self.base_url + identifier, verify=False)
  File "/usr/local/lib/python3.6/site-packages/requests/sessions.py", line 521, in get
    return self.request('GET', url, **kwargs)
  File "/usr/local/lib/python3.6/site-packages/requests/sessions.py", line 508, in request
    resp = self.send(prep, **send_kwargs)
  File "/usr/local/lib/python3.6/site-packages/requests/sessions.py", line 618, in send
    r = adapter.send(request, **kwargs)
  File "/usr/local/lib/python3.6/site-packages/requests/adapters.py", line 508, in send
    raise ConnectionError(e, request=request)
requests.exceptions.ConnectionError: HTTPConnectionPool(host='sci-hub.hk', port=80): Max retries exceeded with url: /29885904 (Caused by NewConnectionError('<urllib3.connection.HTTPConnection object at 0x103d60240>: Failed to establish a new connection: [Errno 8] nodename nor servname provided, or not known',))

Hi @Enrico68, sorry for the late response on this - I just fixed this issue in my latest commit. Should be working now.


Is it possible to add PostGIS?

When I try to activate it, returns an error:

We are going to create another template that includes PostGIS so you will be able to choose whether or not you want it enabled


[compute] Fix minor compute bugs

with vm/ss extension image list --latest and vmss create --no-wait
closes #9631. closes #9460

This checklist is used to make sure that common guidelines for a pull request are followed.

[ ] The PR has modified HISTORY.rst describing any customer-facing, functional changes. Note that this does not include changes only to help content. (see Modifying change log).

[ ] I adhere to the Command Guidelines.

I will go ahead and merge this


Failure to go get because of moved gcimporter15

It appears goreporter relies on gcimporter15, which has been moved:
https://go-review.googlesource.com/c/tools/+/105977
https://godoc.org/golang.org/x/tools/go
Code location that imports it:
https://github.com/360EntSecGroup-Skylar/goreporter/blob/master/linters/simplecode/lint/lint.go#L23
Example error when running go get -u github.com/360EntSecGroup-Skylar/goreporter:
package golang.org/x/tools/go/gcimporter15: cannot find package "golang.org/x/tools/go/gcimporter15" in any of:
	/usr/local/go/src/golang.org/x/tools/go/gcimporter15 (from $GOROOT)
	/home/ubuntu/.go_workspace/src/golang.org/x/tools/go/gcimporter15 (from $GOPATH)
	/usr/local/go_workspace/src/golang.org/x/tools/go/gcimporter15

Same problem

For anyone looking to move forward with this, you can patch from my fork: serussell/goreporter@3514eb452.  It'll get you going (compiles & runs), but it's a quick hack and I haven't vetted the solution for correctness as I don't entirely understand why the original code was doing what it was doing in the way it was doing it (it seemed needlessly complex).

@ednp @eric00liu Thank you very much for your feedback on this issue. I did not have version control on goreporter.I have solved it with godep, and later I will check if the dependency can be updated.
@serussell Great fix.

same here in go1.11
I solved it with go1.10 and the corresponding version of golang.org/x/tools.
The root of the problem is that gcimporter15 was removed, and I found that this package was still in go1.10. But golint relies on go/gcexportdata, which is added in go1.8, so I think go1.8~1.10 should be ok

how to solve this problem?? I have same problem

Go the same problem.
https://pkg.go.dev/golang.org/x/tools/go/gcimporter15
_Package gcimporter15 provides various functions for reading gc-generated object files that can be used to implement the Importer interface defined by the Go 1.5 standard library package.
Deprecated: this package will be deleted in October 2017. New code should use golang.org/x/tools/go/gcexportdata._


Specify js/css includes as widget media

Allows us to eliminate wagtail_hooks... 🎉

Coveralls here is being a bit noisy as the coverage of the individual wagtaildraftail/widgets.py rose, but because it doesn't have 100% coverage yet, and that wagtaildraftail/wagtail_hooks.py had 100%, the overall coverage fall. So merging anyway.


feat: onboard observability-operator apis

/retest

/retest

This PR is blocked on this PR on devkube
Yet regardless of that this PR is ready to be reviewed. The above devkube PR's change would only affect one LOC in this PR, so it's not worthwhile to block, atleast, the review of this PR until the devkube PR is merged

/retest

/retest

/block

/hold

@yashvardhan-kukreja I think you need to update the RBAC here in this file too: config/olm/addon-operator.csv.tpl.yaml

@yashvardhan-kukreja I think you need to update the RBAC here in this file too: config/olm/addon-operator.csv.tpl.yaml

Ahh correct. Nice catch. Although I am surprised why did the operator-e2e didn't fail then. Because operator-e2e tests, install ADO in the prow cluster in an OLM compliant manner i.e. via the CSV we provide.
So previously if the CSV didn't have the relevant RBAC, then how was ADO installation in the prow runtime able to CRUD on MonitoringStack CRs :/

/lgtm


Configurable actor reminder storage partitions

Description
Adds support for configuring the actor reminder storage partitions
Issue reference
https://github.com/dapr/dapr/issues/2889
Checklist
Please make sure you've completed the relevant tasks for this PR, out of the following list:

[X] Code compiles correctly
[X] Created/updated tests
[ ] Extended the documentation

@rynowak could you take a look at this? Now that https://github.com/dapr/dapr/pull/3297 has landed this needs to make its way into the next DotNet SDK version.
CC @halspang


Aniwatch moved domain to hianime.to

Summary
Title self explanatory, it gives error 521 on aniwatch because the domain has moved recently
Please confirm the following

[X] I have searched the existing issues and this is a new ticket, NOT a duplicate or related to another open issue.

Fixed on 5c60e5767f24c327d95dc89fb6ffac12e47c36ce

Available on Release 1.5.1


[Request] Compact pubspec.yaml to facilitate packages ordering

Use case
There is an analysis option that asks for sort pub dependencies (which I like) and, at least in VSCode, a plugin that orders the selected lines using [F9].
The problem is: flutter dependencies are two liners, making impossible to easily sort dependencies.
flutter:
  sdk: flutter

Proposal
Just changing the template when creating a new flutter project to a one line flutter dependencies:
dependencies:
  cupertino_icons: ^1.0.2
  flutter: { sdk: flutter }

dev_dependencies:
  flutter_lints: ^2.0.0
  flutter_test: { sdk: flutter }

@JCKodel Pubspec Assist VS Code extension has the ability to sort dependencies in pubspec.yaml even if dependencies stretch over multiple lines.

Thanks for filing this. However, I don't think we will implement this, as I think it's less readable. It might be a reasonable feature request on the analysis server, since they support ordering dependencies, to support ordering nested dependencies (or use the extension campovski suggests).


Deep Linking Scroll

When you navigate to a section on a page and that page is using the Table of Contents (TOC) component the following happens:
Actual

the page is briefly at the section linked
the page scrolls to the top
first section in the TOC is highlighted
you are now lost and need to scroll back down the page to find the linked section (if you even know what it is)

Expected

The linked section is highlighted in the TOC
The linked section is in view (at the top of the page if the page is long enough to do so)

Is this still an issue? I can't recreate it

The scolling to the top issue doesnt seem to be there anymore but its still not great.
Try this link --> https://mosaic-mosaic-dev-team.vercel.app/mosaic/author/refs#existing-refs
are you at the Existing Refs section?

Okay I had a look at this and couldn't figure it out. It loads on the correct section and then scrolls to some other part, but there aren't any instances of window.scroll in our code that could be causing it.


Create TCOON (lighthouse) Ingestion class

Manual page:
http://lighthouse.tamucc.edu/pq
API Endpoint:
https://lighthouse.tamucc.edu/pd?unit=metric&sslist=256-wdr&when=today%2C-7d&corrections=
Need a station id, a series, and a time range, plus other options (include corrections or not), unit maybe?
Point of contact is Niall who is away for two weeks starting 9/11/2023
Need water temp and air temp

Trying to hit this hits tamucc's microsoft log in page :(

https://lighthouse.tamucc.edu/dranged

http://lighthouse.tamucc.edu/pd?stnlist=013&serlist=atp&when=now%2C-1h&whentz=UTC0&-action=app_json&unit=metric&elev=stnd

Date Range; https://lighthouse.tamucc.edu/dranged
Station and series info: http://lighthouse.tamucc.edu/pq
Use this endpoint: http://lighthouse.tamucc.edu/pq
ALWAYS USE HTTP!


auto re-loading the image file

I have the Volume Layer addon working in Godot VR (using Quest 3) on my Linux machine:
A video showing 5 minutes of me floating around the wakefield; https://youtu.be/-INQylQH0qY
I will modify the Laser Wakefield simulation program to launch Godot and run the visualization project, and I need a way to re-load the project's image source file every time the simulation generates an output, so the VR visualization shows the progress of the simulation in real time.
Is the image file re-loaded automatically as the project is running, and if not is there a feature that can be set or a script that can be used to do that?

The ZippedImageArchiveCpuTexture3D won't automatically detect if the zip file changes, but you can monitor the file yourself and create a new ZippedImageArchiveCpuTexture3D when it does change.  Then just assign the new Texture3D to the VolumeLayeredShader.
The code could be modified to handle live data, but it sounds like you are dumping your data to a zip archive and then loading that, so the above method should work the best.


Is there anyway to generate seperate verilog files for each component?

Hi there,
I'm interested in performing area analysis for a NoC, specifically for the router component. I'm planning to use Constellation to generate a router design for this task first, but I'm wondering if there's a way to generate a Verilog file that only contains the code for a single router.
Any advice or guidance you could provide would be greatly appreciated!
Thanks

When running on the latest Chipyard, separate verilog files for each component should be generated.


$receiver = new Receiver(new FileAPI($config)); and then no more....

Hi Recca, thanks for sharing this wonderful package, I'm here to ask you about how to use your  gorgeous upload package, when I execute your demo index.html included, uploading files but it crashed at line "$receiver = new Receiver(new FileAPI($config));" and not any error message show, would you please be kind to instruct me what is going wrong here? thanks:)
Besides how do I set the config file correctly, is the path(temp/chunks) relative to what position. and where should I create these folder?  Could you explain more in your readme file.
Thanks very much ^^
Ryan

Hi, for those who encountered the same problem with me, the author Recca've instructed that  under demo folder  'composer install' have to be executed to make the demo run normally.
Hope this helps those who use this wonderful package^^


Investigate DisplayName property in the CommandProvider class

Description of the new feature / enhancement
This is not being used in the UI anywhere (yet)... we should think about this field and remove it if it is superflous.
Scenario when this would be used?
NA
Supporting information
No response

Pretty confident it was vestigial. Only place I could think it would be useful would be in the settings. But even there, the name of the app itself is probably fine, and/or we can just use the names of the top-level commands

It may actually be really nice for settings to display that string instead of whatever the app name is. Maybe we can use that as the first string shown & if empty we'll fall back on the package name.

Yea I think we'll leave this in there for now. We're not displaying it anywhere now, but once we start displaying settings, installed command providers, etc, we'll need it.


Merkle proof with unused entry

Is there an existing issue?

[X] I have searched the existing issues

Experiencing problems? Have you tried our Stack Exchange first?

[X] This is not a support question.

Description of bug
When asking Substrate a call proof of TaggedTransactionQueue_validate_transaction in order to do a simple transfer on Westend, it sends back a Merkle proof that I think contains an unused entry. In other words, an entry in the proof that is there for no reason.
Smoldot considers proofs with unused entries as invalid.
You can find an example proof (undecoded) here: https://github.com/smol-dot/smoldot/issues/154#issuecomment-1425617192
The unused entry is the second item. In other words, if you decode the proof into a Vec<Vec<u8>>, then the unused entry would be decoded[1].
The reason why I'm confident that the entry is unused is because its hash is [194, 64, 98, 178, 166, 67, 204, 119, 7, 210, 16, 237, 205, 36, 201, 61, 141, 167, 156, 61, 71, 134, 57, 105, 131, 191, 68, 14, 186, 12, 75, 253], and this sequence of bytes isn't found anywhere in the proof.
Steps to reproduce
No response

cc @cheme I guess

To clarify, it's not "unused" in the sense that it contains a storage item that is not necessary for the call. It's unused in the sense that it contains a node that is completely disconnected from the other nodes of the proof.

I also can't seem to be able to decode this node. The size of the node is 32 byte so cannot be a detached value, and trying to decode it, it is not valid content (header is branch V1 with value, 10 nibbles, then child map show multiple child which cannot be in this 32 byte node).
Is this proof comming from a trusted source, does it happen regularily?
Could be that value of 32 byte length are added to proof due to a bug (should be inline).
I did find this node as an inline value of the node at position 5:
So this is probably it, something that add inline 32 byte node to proof as if they are detached nodes.

I did identify the issue, was a bug from the substrate trie crate (inline value where recorded as node value when get_hash was called).

@cheme Would it be possible to open a PR with a cargo update -p trie so that the fix is in?

Reopening because it's not fixed yet

#13284 is needed for 0.25.0, so it needs to be merge first (iirc all was fine in this PR).

The pr was merged. @tomaka do you still want to reopen this issue?

No, I just wanted is_issue_open to be equal to is_the_bug_fixed


Diff bind reorg

refactoring and reorganization of the differential binding analyses

Sorry for the delay this is ready for review, data can be found at /data/OpenOmics/dev/datasets @tovahmarkowitz @skchronicles


Numba 0.60.0rc1 Checklist

Numba 0.60.0rc1

[x] Merge to main.

[x] #9274
[x] #9454
[x] #9482
[x] https://github.com/numba/numba/pull/9549

[x] Check Numba's version support table documentation. Update via PR if
needed.

[x] https://github.com/numba/numba/pull/9559

[x] Review deprecation schedule and notices. Make PRs if need be. (none needed)
[x] Merge change log changes.

[x] https://github.com/numba/numba/pull/9558

[x] Create 0.60 release branch. https://github.com/numba/numba/tree/release0.60
[x] Create PR against the release branch to make numba/testing/main.py https://github.com/numba/numba/pull/9561
to refer to origin/release0.60 instead of origin/main.
[x] Dependency version pinning on release branch https://github.com/numba/numba/pull/9561

[x] Pin llvmlite to 0.43.*.
[x] Pin NumPy if needed
[x] Pin TBB if needed (not needed)

[ ] Run the HEAD of the release branch through the build farm and confirm:

[ ] Build farm CPU testing has passed. numba_yaml_459
[x] Build farm CUDA testing has passed. numba_smoketest_cuda_yaml_22
[x] Build farm wheel testing has passed. numba_wheel_108

[ ] Annotated tag 0.60.0rc1 on release branch (no v prefix).
[ ] Build and upload conda packages on buildfarm (check "upload").
[ ] Build wheels and sdist on the buildfarm (check "upload").
[ ] Verify packages uploaded to Anaconda Cloud and move to numba/label/main.
[ ] Upload wheels and sdist to PyPI (upload from ci_artifacts).
[ ] Verify wheels for all platforms arrived on PyPi.
[ ] Initialize and verify ReadTheDocs build.
[ ] Post announcement to discourse group and ping the release testers group
using @RC_Testers.
[ ] Post link to X and to Mastodon and...

Post Release:

[ ] Clean up ci_artifacts by moving files to sub-directories
[ ] Tag 0.61.0dev0 to start new development cycle on main.
[ ] Update llvmlite dependency via PR to main, PR includes version updates
to:

[ ] setup.py
[ ] numba/__init__.py
[ ] docs/environment.yml
[ ] buildscripts/incremental/setup_conda_environment.sh
[ ] buildscripts/incremental/setup_conda_environment.cmd
[ ] buildscripts/condarecipe.local/meta.yaml

[ ] Update release checklist template with any additional bullet points that
may have arisen during the release.
[ ] Close milestone (and then close this release issue).

Q: It seems all tasks are done (congrats! 🎉), is there an ETA for the rc1 release?

Q: It seems all tasks are done (congrats! 🎉), is there an ETA for the rc1 release?

Soon! 😇

The instructions for how to generate the change-log need to be added, it's now a combination of towncrier and using the script at: https://github.com/numba/numba/blob/main/maint/gitlog2changelog.py

Update, these instructions exist already: https://github.com/numba/numba/pull/9367 -- we just need to add a link to them from the release checklist template.

Note to self: describe in more detail how to pin NumPy and how to pin llvmlite in the template.

Note to self: describe in more detail how to add the changelog PR to the list of PRs.

This release has been delayed due to discovering a release critical issue. We hope to resolve this on Monday and the target date to tag both Numba and llvmlite is now the 14th of may.

This release has been delayed due to discovering a release critical issue. We hope to resolve this on Monday and the target date to tag both Numba and llvmlite is now the 14th of may.

https://github.com/numba/numba/pull/9568

The release process has resumed.

Am seeing a GH tag for 0.60.0rc1
Also seeing packages for...

conda
pip

Is there anything else still needed here?

Am seeing a GH tag for 0.60.0rc1
Also seeing packages for...
* [conda](https://anaconda.org/numba/numba/files?version=0.60.0rc1)

* [pip](https://pypi.org/project/numba/0.60.0rc1/#files)

Is there anything else still needed here?

A review of https://github.com/conda-forge/llvmlite-feedstock/pull/86 wouldn't go amiss :-)
Then I can do the Numba 0.60.0rc1 PR on conda-forge.


fix(app): allow to send event via keyboard shortcuts

Context
I was trying to open create the create menu on the position of the keyboard shortcut event that triggered it
Issue
Currently there isn’t really a way to trigger an editor action via keyboard bindings and send the event as an argument. I discovered that we actually allow for a event param here but we don’t use it. Also, we have the event in KeyboardBindings but we choose to pass it as null.
Me and Martin discussed this and figured that there is no reason not to send the event, and this is probably an oversight (because it hasn’t been needed yet).
Suggestion
Use the second param of AppParent#triggerAction as general context, and send the event through there (we use context in the Editors for example - https://github.com/camunda/camunda-modeler/blob/develop/client/src/app/tabs/bpmn/BpmnEditor.js#L651)

This does not seem to be implemented in all places. On Linux the following combination fails, inside a diagram: CTRL + SHIFT + ArrowDown.
ERROR client [object Object] is not a registered action
    at error (webpack://camundanode_modules/bpmn-js/node_modules/diagram-js/lib/features/editor-actions/EditorActions.js:259:0)
    at EditorActions.trigger (webpack://camundanode_modules/bpmn-js/node_modules/diagram-js/lib/features/editor-actions/EditorActions.js:169:0)
    at BpmnEditor.triggerAction (webpack://camunda-modeler-client/src/app/tabs/cloud-bpmn/BpmnEditor.js:687:0)
    at MultiSheetTab.triggerAction (webpack://camunda-modeler-client/src/app/tabs/MultiSheetTab.js:269:0)
    at EditorTab.triggerAction (webpack://camunda-modeler-client/src/app/tabs/EditorTab.js:36:0)
    at file:///camunda-modeler/client/build/bundle.js:8931:18
    at App.triggerAction (webpack://camunda-modeler-client/src/app/App.js:2315:0)
    at exec (webpack://camunda-modeler-client/src/app/AppParent.js:63:0)
    at Function.<anonymous> (webpack://camunda-modeler-client/src/app/AppParent.js:76:0)


Issues with PHPStorm Version 2023.1.5 Build #PS-231.9414.10 build on February 14,2024  Fatal error: Uncaught Error: Class "Illuminate\Support\Facades\DB" not found in /in/0g7nB:5

Short bug description
The Database: Larvel 9.5  Query Builder not working
Extended bug description
Fatal error: Uncaught Error: Class "Illuminate\Support\Facades\DB" not found in /in/0g7nB:5
Stack trace:
#0 {main}
thrown in /in/0g7nB on line 5
Process exited with code 255.
Your project is not the problem

[X] I tried using a fresh laravel install - I'm fairly certain it has nothing to do with my project.

How to reproduce?
Trying to retrieve data from the users table. you get an error.
Expected behavior
The query of the users table should work. It used to work as well.
Screenshots
No response
IDE & Version
PHPStorm 2023.1.5
PHP Version
PHP 8.0.13
OS and version
OS
Laravel Tinker plugin version
2.6.1
Interpreter type
Local
Laravel Version
versions : * v9.52.16
Tinker version

v2.8.2

Psysh version

v0.11.22

Additional Context
The problem did not exist before laravel update.
I was able to access Laravel DB Query without any problems.
Fatal error: Uncaught Error: Class "Illuminate\Support\Facades\DB" not found in /in/0g7nB:5
Stack trace:
#0 {main}
thrown in /in/0g7nB on line 5
Process exited with code 255.
Many thanks for the great module.
mfg
Dieter

thrown in /in/0g7nB

Looks like you're not running your code through tinker.
The the FAQ


name for sensors and multiple stations, fix night queries by specifying date filters

Hi Mike,
many thanks for your commits, really nice that journeys more than one hour ahead can be pulled now!
But I noticed, that the parameter "additionalTime" seems to don't work anymore? Even with "additionalTime" set, the next sensor get updated just at departure time.

Hi, this is because the oebb app is somehow ignoring some of the parameters when you use the date params. i.e. I also needed to fix showJourneys to do it inside the implementation.

Hi Mike,
do you maybe have any idea why we get duplicate entries around date change? As well some sensors are unknown around midnight.


UseLiteral in SimpleQuerydslR2dbcFragment and ReactiveQuerydslR2dbcPredicateExecutor

SimpleQuerydslR2dbcFragment  and  ReactiveQuerydslR2dbcPredicateExecutor do not use prepared statement bindings but literals.
is there any reason for setUseLiterals(true)?
https://github.com/infobip/infobip-spring-data-querydsl/blob/c1b6926919520b70c8fa2f40e59ad2afca3de489/infobip-spring-data-r2dbc-querydsl/src/main/java/com/infobip/spring/data/r2dbc/SimpleQuerydslR2dbcFragment.java#L67
https://github.com/infobip/infobip-spring-data-querydsl/blob/ead317f5fb782178913a90798c420be00c5f8e12/infobip-spring-data-r2dbc-querydsl/src/main/java/com/infobip/spring/data/r2dbc/ReactiveQuerydslR2dbcPredicateExecutor.java#L138

Because, as far as I know, there is no layer that would translate querydsl model into prepared statement bindings.

Ok, so after some investigation and help from @mp911de I can confirm my initial suspicion.
Querydsl generates sql which is escaped with marker ? which is the JDBC way of things.
R2DBC on the other hand focuses on what each database uses - https://github.com/spring-projects/spring-framework/blob/main/spring-r2dbc/src/main/java/org/springframework/r2dbc/core/binding/BindMarkersFactoryResolver.java#L119-L128
this code probably best illustrates this.
For update method I've got as far as
    @Override
    @Transactional
    public Mono<Integer> update(Function<SQLUpdateClause, SQLUpdateClause> update) {
        SQLUpdateClause clause = sqlQueryFactory.update(path);
        List<SQLBindings> bindings = update.apply(clause).getSQL();
        String sql = bindings.stream()
                             .map(SQLBindings::getSQL)
                             .collect(Collectors.joining(" "));
        GenericExecuteSpec spec = bindParameters(databaseClient.sql(sql), bindings);
        return spec.fetch()
                   .rowsUpdated()
                   .as(TransactionalOperator.create(reactiveTransactionManager)::transactional);
    }

    GenericExecuteSpec bindParameters(GenericExecuteSpec spec, List<SQLBindings> bindings) {

        int index = 0;
        GenericExecuteSpec mutableSpec = spec;
        for (SQLBindings binding : bindings) {
            for (Object nullFriendlyBinding : binding.getNullFriendlyBindings()) {
                mutableSpec = mutableSpec.bind(index++, nullFriendlyBinding);
            }
        }
        return mutableSpec;
    }

but this isn't really doable without parsing the query and replacing jdbc markers with database specific markers.
What's your use case, @humanitas03 ? Is this a SQL injection concern or something else?

Therefore, in case, is it possible for you to allow us to use useliteral in an optional way?

The library doesn't work without useLiteral, that is why it was set in the first place.

I finally did some investigation on how to avoid using set literal true and while testing sql injection I was unable to sql inject anything through parameters - parameters are escaped. https://github.com/infobip/infobip-spring-data-querydsl/commit/c225c9aec1fe1a50e2266a4409f79cf7dc4c1b97 here's the test - I did various variations but while debugging the parameter is always escaped.
I did post start a discussion on querydsl side to be certain - https://github.com/querydsl/querydsl/discussions/3558.
Regarding the network bit I'm not sure I follow, what makes you believe that binding markers would be any more secure than anything else in a network call? Both the query and the parameters still need to cross the wire and it's up to the network security layer/configuration to make sure neither is visible.

I'm closing this unless responses in the querydsl discussion provide arguments that literal is subject to sql injection.

After discussion with @jwgmeligmeyling - https://github.com/querydsl/querydsl/discussions/3558 I've decided to reopen this issue and remove setLiteral use for good.


new-log-viewer: Add tabbed Sidebar with resizable left panels.

References

new-log-viewer series: #45 #46 #48 #51 #52 #53 #54 #55 #56 #59 #60 #61 #62 #63 #66 #67 #68 #69 #70 #71 #72 #73
Description

Add <SidebarContainer/>.
Add <FileInfoTab/>.
Move Settings modal toggle button from menu bar to the sidebar.
Move File Open button from the menu bar to the side bar.
Style fixes in the menu bar and the status bar.

Validation performed

Referred to https://github.com/y-scope/yscope-log-viewer/pull/46 , started the debug server and loaded a sample log file.
Observed the File Info tab getting opened in the left panel by default.
Hovered mouse over the right edge of the left panel and the resize handle was highlighted.
Dragged the handle:

The panel resized as the handle was dragged, with the right edge of the panel following the mouse cursor.
When dragging towards the left, if the width of the panel was less than a certain size, the panel closed and only the sidebar component is shown.
When dragging towards the right, the panel width will not exceed a certain size, allowing the Editor to be visible at all times.

Verified File Open by clicking the button in the top of the sidebar.
Verified Settings modal opening by click the settings button in the bottom of the sidebar.

Summary by CodeRabbit
Release Notes

New Features

Introduced a new Sidebar component with a tabbed interface for easy navigation.
Implemented a draggable resize handle for the sidebar, allowing users to adjust its width dynamically.
Enhanced functionality for managing active tabs and displaying relevant file information.

Bug Fixes

Simplified styling and structure of components for better performance and usability.

Styling Enhancements

Updated styles for the sidebar, including new CSS classes for tab buttons and panels to enhance visual appeal.
Introduced responsive layout styles for the CentralContainer component.

I find it a bit unintuitive that clicking the file with arrow icon opens an open dialog. I think two things would help:

Replace the icon with a folder icon. Pretty sure that's still the most common icon for opening a file, right? At least Office applications still use it.

Move the button to replace the current file icon in the top left. I think the file icon in the top left is not doing much, especially since we now have the file info as a tab in the sidebar.

Alternatively, we could move the open icon above the settings icon, but I think that placement is less intuitive.

Moved the Open button back into the menu bar.

When no file is open, can we hide the icons and perhaps display something like "No file is open."

I know in a future PR, we'll probably hide this UI entirely when no file is open, but I think it's a small change to make for now?

The initial fileName and also logData are now changed to "No file is open.".

Since "FILE INFO" is the title, the other "File Info" should probably be "Name"?

Fixed the typo.

Can we add the compressed size and compression ratio to the file info?

Can we add that in another PR? Estimating uncompressed (formatted text) sizes is possible but the calculations for different file types are worth discussions.

Can we un-highlight the last opened tab when the sidebar is closed? I could see someone misinterpreting it as a notification instead of the last tab that they had open.

Done.

Can we add tool tips for the icons? Perhaps:

"Open"
"File info"
"Settings"

Done by utilizing the "title" prop. In a future PR (soon), we will add JoyUI tooltips.

Do we want to increase the min-width before the sidebar snaps closed?

Increased to 250px.

This is minor, but can we make the buttons square rather than rectangles?

Done with width and height being 64px. To avoid misalignments with the file open button in the menu bar, added a YScope logo to match the width of the tab button.

Can we decrease the padding in the sidebar by maybe 25%?

Done.

030036eff4c5d38a1bd19eb95c70bb3d52c51b8c Changed the resize handle width when not hovered:
Before:

After:


Add v6 IStrategy

Check #169 for the example of strategy contract.

@doublespending Let's merge this interface first so we don't have to add it to different PRs?


[IMPROVEMENT]  Adjust the iSCSI timeout and the engine-to-replica timeout settings

Is your improvement request related to a feature? Please describe
The engine to replica timeout is 8 seconds (code) in current implementaion. In an unstable network environment, the network outage triggers frequent replica rebuildings due to the short timeout setting and moreover potentially results in a worse networking status. To mitigate the issue, using a larger timeout is a possible solution. However, when adjusting the engine to replica timeout, the open-iscsi timeout should be also adjusted to avoid that the volume is remounted as read-only. The ticket aims to investigate a better iSCSI and engine-to-replica timeout settings and adjust to better values.
Describe the solution you'd like
A clear and concise description of what you want to happen.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
https://github.com/open-iscsi/open-iscsi/blob/master/etc/iscsid.conf
https://github.com/longhorn/longhorn-engine/commit/773b0254ba3ab02c036f1cc9d785a517d44a8f1c

I believe this might help some things we're seeing. We are overdue for a network equipment refresh and experience some congestion from time to time. We see these errors sometimes after a congestion "blip":
5m          Warning   UnknownNodeConditionTrue   node/oly-k3node-53d                                      Unknown condition true of kubernetes node oly-k3node-53d: condition type is ReadonlyFilesystem, reason is FilesystemIsReadOnly, message is EXT4-fs (sdd): Remounting filesystem read-only
Attached are dmesg errors from that node during the congestion blip:
Node dmesg errors congestion blip Longhorn volume.txt
My feeling is if timeouts were a bit longer, Longhorn would weather these much better.

As discussed, not just the timeout between engine and replica, but also other timeouts like iscsi target or initiator should be considered in this fine grain tunning.
Also, consider making these settings reasonably configurable with a best-practice range to ensure they can be adaptively adjusted as per different dynamic user environments or platform architectures.

Increasing timeouts needs to consider time of failure detection, stability and performance.
From my perspective increasing the timeouts is acceptable since it is better to be slow and functional on a high latency/intermediate congestion environment instead of failing replicas that might then lead to rebuilds right while the network is already congested which could then lead to cascading failures potentially.
For clarification of why the timeout mechanism impacts performance the latency of a write operation is currently always determined by the slowest replica, since writes are synchronous across all 3 replicas, imagine the case where you have two 100ms replicas and  one 2 second replica. The io latency for a write would be 2 seconds in this case.

failure detection

Increasing timeouts needs to consider time of failure detection, stability and performance.
From my perspective increasing the timeouts is acceptable since it is better to be slow and functional on a high latency/intermediate congestion environment instead of failing replicas that might then lead to rebuilds right while the network is already congested which could then lead to cascading failures potentially.
For clarification of why the timeout mechanism impacts performance the latency of a write operation is currently always determined by the slowest replica, since writes are synchronous across all 3 replicas, imagine the case where you have two 100ms replicas and one 2 second replica. The io latency for a write would be 2 seconds in this case.

For the failure detection, do you mean in the manager part or in the engine part?

@innobead we were really hoping to see this in 1.3.2. Counting down the days, in fact.

@mstrent
Thank you for addtention. We are planning LH v1.3.2. The feature needs more time to investigate and test, so we will put it in v1.3.3.

Need to create a upgrade test engine image for cli version 7. Thank you.
cc @longhorn/qa

@derekbit this needs require/doc, so need to update the doc as well.

Tested on longhorn master-head (longhorn-manager:v2_20221115) by following steps:
(1) deploy the following deployment to create a workload and a volume with 3 replicas:
apiVersion: apps/v1
kind: Deployment
metadata:
  name: test-deployment
  labels:
    name: test-deployment
spec:
  replicas: 1
  selector:
    matchLabels:
      name: test-deployment
  template:
    metadata:
      labels:
        name: test-deployment
    spec:
      containers:
        - name: test-deployment
          image: nginx:stable-alpine
          volumeMounts:
            - name: test-pod
              mountPath: /data
      volumes:
        - name: test-pod
          persistentVolumeClaim:
            claimName: test-1
---
apiVersion: v1
kind: PersistentVolumeClaim
metadata:
  name: test-1
  namespace: default
spec:
  accessModes:
    - ReadWriteOnce
  storageClassName: longhorn
  resources:
    requests:
      storage: 2Gi

(2) in the workload, execute the following script:
for i in `seq 1 20000`
do
  date "+%T"
  dd if=/dev/urandom of=/data/file bs=1M count=1
  sleep 1
done

(3) then shut down one of the replica's host, I can observe the replica state transferred from healthy/running to failed/stopped, but I can't observe the IO hang for 8 seconds:
# ./test.sh
02:21:16
1+0 records in
1+0 records out
02:21:17
1+0 records in
1+0 records out
02:21:18
1+0 records in
1+0 records out
02:21:19
1+0 records in
1+0 records out
02:21:20
1+0 records in
1+0 records out
02:21:21
1+0 records in
1+0 records out
02:21:22
1+0 records in
1+0 records out
02:21:23
1+0 records in
1+0 records out
02:21:24
1+0 records in
1+0 records out
02:21:25
1+0 records in
1+0 records out
02:21:26
1+0 records in
1+0 records out
02:21:27
1+0 records in
1+0 records out
02:21:28
1+0 records in
1+0 records out
02:21:29
1+0 records in
1+0 records out
02:21:30
1+0 records in
1+0 records out
02:21:31
1+0 records in
1+0 records out
02:21:32
1+0 records in
1+0 records out
02:21:33
1+0 records in
1+0 records out
02:21:34
1+0 records in
1+0 records out
02:21:35
1+0 records in
1+0 records out
02:21:36
1+0 records in
1+0 records out
02:21:37
1+0 records in
1+0 records out
02:21:38
1+0 records in
1+0 records out
02:21:39
1+0 records in
1+0 records out
02:21:40
1+0 records in
1+0 records out
02:21:41
1+0 records in
1+0 records out
02:21:42
1+0 records in
1+0 records out
02:21:43
1+0 records in
1+0 records out
02:21:44
1+0 records in
1+0 records out
02:21:45
1+0 records in
1+0 records out
02:21:46
1+0 records in
1+0 records out
02:21:47
1+0 records in
1+0 records out
02:21:48
1+0 records in
1+0 records out
02:21:49
1+0 records in
1+0 records out
02:21:50
1+0 records in
1+0 records out
02:21:51
1+0 records in
1+0 records out
02:21:52
1+0 records in
1+0 records out
02:21:53
1+0 records in
1+0 records out
02:21:54
1+0 records in
1+0 records out
02:21:55
1+0 records in
1+0 records out
02:21:56
1+0 records in
1+0 records out
02:21:57
1+0 records in
1+0 records out
02:21:58
1+0 records in
1+0 records out
02:21:59
1+0 records in
1+0 records out
02:22:00
1+0 records in
1+0 records out
02:22:01
1+0 records in
1+0 records out
02:22:02
1+0 records in
1+0 records out
02:22:03
1+0 records in
1+0 records out
02:22:04
1+0 records in
1+0 records out
02:22:05
1+0 records in
1+0 records out
02:22:06
1+0 records in
1+0 records out
02:22:07
1+0 records in
1+0 records out
02:22:08
1+0 records in
1+0 records out
02:22:09
1+0 records in
1+0 records out
02:22:10
1+0 records in
1+0 records out
02:22:11
1+0 records in
1+0 records out
02:22:12
1+0 records in
1+0 records out
02:22:13
1+0 records in
1+0 records out
02:22:14
1+0 records in
1+0 records out
02:22:15
1+0 records in
1+0 records out
02:22:16
1+0 records in
1+0 records out
02:22:17
1+0 records in
1+0 records out
02:22:18
1+0 records in
1+0 records out
02:22:19
1+0 records in
1+0 records out
02:22:20
1+0 records in
1+0 records out
02:22:21
1+0 records in
1+0 records out
02:22:22
1+0 records in
1+0 records out
02:22:23
1+0 records in
1+0 records out
02:22:24
1+0 records in
1+0 records out
02:22:25
1+0 records in
1+0 records out
02:22:26
1+0 records in
1+0 records out
02:22:27
1+0 records in
1+0 records out
02:22:28
1+0 records in
1+0 records out
02:22:29
1+0 records in
1+0 records out
02:22:30
1+0 records in
1+0 records out
02:22:31
1+0 records in
1+0 records out
02:22:32
1+0 records in
1+0 records out
02:22:33
1+0 records in
1+0 records out
02:22:34
1+0 records in
1+0 records out
02:22:35
1+0 records in
1+0 records out
02:22:36
1+0 records in
1+0 records out
02:22:37
1+0 records in
1+0 records out
02:22:38
1+0 records in
1+0 records out
02:22:39
1+0 records in
1+0 records out
02:22:40
1+0 records in
1+0 records out
02:22:41
1+0 records in
1+0 records out
02:22:42
1+0 records in
1+0 records out
02:22:43
1+0 records in
1+0 records out
02:22:44
1+0 records in
1+0 records out
02:22:45
1+0 records in
1+0 records out
02:22:46
1+0 records in
1+0 records out
02:22:47
1+0 records in
1+0 records out
02:22:48
1+0 records in
1+0 records out
02:22:49
1+0 records in
1+0 records out
02:22:50
1+0 records in
1+0 records out
02:22:51
1+0 records in
1+0 records out
02:22:52
1+0 records in
1+0 records out
02:22:53
1+0 records in
1+0 records out
02:22:54
1+0 records in
1+0 records out
02:22:55
1+0 records in
1+0 records out
02:22:56
1+0 records in
1+0 records out
02:22:57
1+0 records in
1+0 records out
02:22:58
1+0 records in
1+0 records out
02:22:59
1+0 records in
1+0 records out
02:23:00
1+0 records in
1+0 records out
02:23:01
1+0 records in
1+0 records out
02:23:02
1+0 records in
1+0 records out
02:23:04
1+0 records in
1+0 records out
02:23:05
1+0 records in
1+0 records out
02:23:06
1+0 records in
1+0 records out
02:23:07
1+0 records in
1+0 records out
02:23:08
1+0 records in
1+0 records out
02:23:09
1+0 records in
1+0 records out
02:23:10
1+0 records in
1+0 records out
02:23:11
1+0 records in
1+0 records out
02:23:12
1+0 records in
1+0 records out
02:23:13
1+0 records in
1+0 records out
02:23:14
1+0 records in
1+0 records out
02:23:15
1+0 records in
1+0 records out
02:23:16
1+0 records in
1+0 records out
02:23:17
1+0 records in
1+0 records out
02:23:18
1+0 records in
1+0 records out
02:23:19
1+0 records in
1+0 records out
02:23:20
1+0 records in
1+0 records out
02:23:21
1+0 records in
1+0 records out
02:23:22
1+0 records in
1+0 records out
02:23:23
1+0 records in
1+0 records out
02:23:24
1+0 records in
1+0 records out
02:23:25
1+0 records in
1+0 records out
02:23:26
1+0 records in
1+0 records out
02:23:27
1+0 records in
1+0 records out
02:23:28
1+0 records in
1+0 records out
02:23:29
1+0 records in
1+0 records out
02:23:30
1+0 records in
1+0 records out
02:23:31
1+0 records in
1+0 records out
02:23:32
1+0 records in
1+0 records out
02:23:33
1+0 records in
1+0 records out
02:23:34
1+0 records in
1+0 records out
02:23:35
1+0 records in
1+0 records out
02:23:36
1+0 records in
1+0 records out
02:23:37
1+0 records in
1+0 records out
02:23:38
1+0 records in
1+0 records out
02:23:39
1+0 records in
1+0 records out
02:23:40
1+0 records in
1+0 records out
02:23:41
1+0 records in
1+0 records out
02:23:42
1+0 records in
1+0 records out
02:23:43
1+0 records in
1+0 records out
02:23:44
1+0 records in
1+0 records out
02:23:45
1+0 records in
1+0 records out
02:23:46
1+0 records in
1+0 records out
02:23:47
1+0 records in
1+0 records out
02:23:48
1+0 records in
1+0 records out
02:23:49
1+0 records in
1+0 records out
02:23:50
1+0 records in
1+0 records out
02:23:51
1+0 records in
1+0 records out
02:23:52
1+0 records in
1+0 records out
02:23:53
1+0 records in
1+0 records out
02:23:54
1+0 records in
1+0 records out
02:23:55
1+0 records in
1+0 records out
02:23:56
1+0 records in
1+0 records out
02:23:57
1+0 records in
1+0 records out
02:23:58
1+0 records in
1+0 records out
02:23:59
1+0 records in
1+0 records out
02:24:00
1+0 records in
1+0 records out
02:24:01
1+0 records in
1+0 records out
02:24:02
1+0 records in
1+0 records out
02:24:03
1+0 records in
1+0 records out
02:24:04
1+0 records in
1+0 records out
02:24:05
1+0 records in
1+0 records out
02:24:06
1+0 records in
1+0 records out
02:24:07
1+0 records in
1+0 records out
02:24:08
1+0 records in
1+0 records out
02:24:09
1+0 records in
1+0 records out
02:24:10
1+0 records in
1+0 records out
02:24:11
1+0 records in
1+0 records out
02:24:12
1+0 records in
1+0 records out
02:24:13
1+0 records in
1+0 records out
02:24:14
1+0 records in
1+0 records out
02:24:15
1+0 records in
1+0 records out
02:24:16
1+0 records in
1+0 records out
02:24:17
1+0 records in
1+0 records out
02:24:18
1+0 records in
1+0 records out
02:24:19
1+0 records in
1+0 records out
02:24:20
1+0 records in
1+0 records out
02:24:21
1+0 records in
1+0 records out
02:24:22
1+0 records in
1+0 records out
02:24:23
1+0 records in
1+0 records out
02:24:24
1+0 records in
1+0 records out
02:24:25
1+0 records in
1+0 records out
02:24:26
1+0 records in
1+0 records out

@derekbit Can you help to clarify if there's anything wrong in my test steps so I'm unable to observe the expected IO hang? Thank you!

I think the buffer still has enough space for the newly written data. You can try to add oflag=sync.

@derekbit I change the command to dd if=/dev/urandom of=/data/file oflag=sync bs=1M count=1 and even dd if=/dev/urandom of=/data/file oflag=direct,sync bs=1M count=1 still unable to observe the IO hang..

My bad. Shutting down one of the nodes might be not a good way. The engine will receive EOF after the shutdown and change to replica to ERR directly.
You can stop the network traffict for emulating the unstable network by
tc qdisc add dev <interface> root netem loss 100%

Verified passed on master-head (longhorn-instance-manager v2_20221115, longhorn-engine e112364, longhorn-manager 732f3c8) by following steps:
(1) deploy the following deployment to create a workload and a volume with 3 replicas:
apiVersion: apps/v1
kind: Deployment
metadata:
  name: test-deployment
  labels:
    name: test-deployment
spec:
  replicas: 1
  selector:
    matchLabels:
      name: test-deployment
  template:
    metadata:
      labels:
        name: test-deployment
    spec:
      containers:
        - name: test-deployment
          image: ubuntu:22.04
          args: ["/bin/sh", "-c", "while true;do date;sleep 5; done"]
          volumeMounts:
            - name: test-pod
              mountPath: /data
      volumes:
        - name: test-pod
          persistentVolumeClaim:
            claimName: test-1
---
apiVersion: v1
kind: PersistentVolumeClaim
metadata:
  name: test-1
  namespace: default
spec:
  accessModes:
    - ReadWriteOnce
  storageClassName: longhorn
  resources:
    requests:
      storage: 2Gi

(2) in the workload, execute the following script:
for i in `seq 1 20000`
do
  date "+%T"
  dd if=/dev/urandom of=/data/file oflag=direct,sync bs=1M count=1
  sleep 1
done

(3) then disconnect the network of one of the replica's host using:
tc qdisc add dev eth0 root netem loss 100%
or just put the instance into another isolated security group
(4) The 8-second IO hang can be observed:
06:09:01
1+0 records in
1+0 records out
1048576 bytes (1.0 MB, 1.0 MiB) copied, 0.0348423 s, 30.1 MB/s
06:09:02
1+0 records in
1+0 records out
1048576 bytes (1.0 MB, 1.0 MiB) copied, 0.0379436 s, 27.6 MB/s
06:09:03
1+0 records in
1+0 records out
1048576 bytes (1.0 MB, 1.0 MiB) copied, 8.34383 s, 126 kB/s
06:09:12
1+0 records in
1+0 records out
1048576 bytes (1.0 MB, 1.0 MiB) copied, 0.0258202 s, 40.6 MB/s
06:09:13
1+0 records in
1+0 records out
1048576 bytes (1.0 MB, 1.0 MiB) copied, 0.0286228 s, 36.6 MB/s
06:09:14
1+0 records in

If adjust Timeout between Engine and Replica to 30, the 30-second IO hang can be observed:
07:12:10
1+0 records in
1+0 records out
1048576 bytes (1.0 MB, 1.0 MiB) copied, 0.0326939 s, 32.1 MB/s
07:12:11
1+0 records in
1+0 records out
1048576 bytes (1.0 MB, 1.0 MiB) copied, 0.0331858 s, 31.6 MB/s
07:12:12
1+0 records in
1+0 records out
1048576 bytes (1.0 MB, 1.0 MiB) copied, 0.0338816 s, 30.9 MB/s
07:12:13
1+0 records in
1+0 records out
1048576 bytes (1.0 MB, 1.0 MiB) copied, 30.0678 s, 34.9 kB/s
07:12:44
1+0 records in
1+0 records out
1048576 bytes (1.0 MB, 1.0 MiB) copied, 0.0291683 s, 35.9 MB/s
07:12:45
1+0 records in
1+0 records out
1048576 bytes (1.0 MB, 1.0 MiB) copied, 0.0287038 s, 36.5 MB/s
07:12:46
1+0 records in
1+0 records out
1048576 bytes (1.0 MB, 1.0 MiB) copied, 0.0263958 s, 39.7 MB/s
07:12:48


Fix/fix for cypress 13

Temporary solution for Cypress 13 update.

Hi there
thank you for this
we still need the else path for older versions but I'll merge it so you appear as contributor :)
thank you so much


IEEEmadC

Can you please upload the code?

https://drive.google.com/open?id=1iKhQpXX3pWJmcn4j147lh8F-Y-BuFLG5

It needs to be uploaded here to Github. The ideal submission look like : https://github.com/brianzhou139/BookShoot

Source Code For Track My Bus
https://drive.google.com/open?id=1iKhQpXX3pWJmcn4j147lh8F-Y-BuFLG5
في الجمعة، 20 ديسمبر 2019 في 4:13 م تمت كتابة ما يلي بواسطة Sangram
Chavan <notifications@github.com>:

Where is the code?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/FaisalNasser/Track-My-Bus-repo/issues/1?email_source=notifications&email_token=ALK6VBMJUKXPQS5A4NWO453QZTHH3A5CNFSM4J56NS42YY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4IB7HTQQ,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ALK6VBJQWGFCMPWOFBOTQY3QZTHH3ANCNFSM4J56NS4Q
.

Read.md

We can not evaluate from Google Drive.

https://github.com/FaisalNasser/Track-My-Bus-repo

Where do I find the executable file .apk ?

Apps Submission
These apps are built by students for IEEEmadC 2019
Contact at<EMAIL_ADDRESS>with your team name if you face any issues.
For submittion kindly include following in your GitHub repository:

Step 1: Hi-res icon Default – English 512 x 512 32-bit PNG, Feature graphic Default – English – 1024 w x 500 h JPG or 24-bit PNG,

Step 2: Promo Graphic Default – English 180 w x 120 h JPG or 24-bit PNG zip required

Step 3: Screenshots Default – English JPEG or 24-bit PNG (no alpha). Min length for any side: 320px. Max length for any side: 3840px.At least 2 screenshots are required overall. Max 8 screenshots per type. For your app to be showcased in the 'Designed for tablets' list in the Play Store, you need to upload at least one 7-inch and one 10-inch screenshot. Maximum file accepted 10 in one folder.

Step 4: DESCRIPTION UPTO 4000 CHARACTERS SHOULD INCLUDE PROMO VIDEO 2MIN YOUTUBE/VIMEO EMBED LINK - ONLY VIDEO LINK

Step 5: UPLOAD CODE AND APP. Include a Licence.

Include all this in one Repository. Now visit
Step 1:

Note: You need to fork to add your pull request.
Step 2:
Enter the details

Scroll down you see Commit new file
Step 3:

A pull request will be created on our end and that's it if there is anything missing we will mention it in the issue section of your repo.
Below are ideal submissitions:
Example 1 Code in Readme.md
Example 2 Code in Readme.md
Now submit the Readme.md similar to the example file linking to your repository.
Don't forget to add the Table.
Example:
App Name (Hyperlink to executable file) | Repo name (Hyperlink to repo) | description (short description of app in few words) | Choose any licence e.g. MIT (Hyperlink to Licence)| UK (Location)| Name of members (Hyperlink to Github profiles that contributed to the project) |
Submittion
Note: You do not require to publish apps on store.
FAQ:
How to set up github ? Play
How to ?? Tutorial

debug.zip
https://github.com/FaisalNasser/Track-My-Bus-repo/commit/70b2584db974e9e10007623a02c30a0bb30c7485

I see your zip file and the apk.
You need to upload the complete code over the repo you have included only few files but the rest of the files are pending.


Develop testfix

merging in test fix changes made in this branch

Lots of tests aren't working here but that's a broad issue with this branch (develop-microsim) so merging it all back into develop-microsim to solve


Fix warnings & todos

Summary
Fixed some build warnings
Changes Made
Fixed some build warnings.
Fixed some // TODO's
TODO
None
Motivation
Fix build warnings, see #466
Testing
N/A
Screenshots/GIFs

Checklist
Please ensure all of the following are completed before submitting the PR:

[ ] Code has been tested and verified.
[ ] Documentation has been updated to reflect the changes.
[ ] All tests pass successfully.
[ ] Code follows the project's coding guidelines and style.
[ ] The branch is up-to-date with the latest changes from the main branch.
[ ] Reviewed by at least one other contributor.

Related Issues
This PR addresses the following issue(s): #466
Additional Notes
None

Thanks for submitting a pull request!
We will review it as soon as possible.


Getting the original requested Path when using UseStatusCodePagesWithReExecute

When requesting http://example.com/nonexisting: Is it possible to get the info about the original request when an action executes because it is the handler for UseStatusCodePagesWithReExecute?
If not then how else can I log requested urls that have a non-OK response like 404?

See IStatusCodeReExecuteFeature in https://github.com/aspnet/Diagnostics/blob/dev/src/Microsoft.AspNetCore.Diagnostics/StatusCodePage/StatusCodePagesExtensions.cs


Enable Scheduler feature-toggle by default

Resolves #322

to test will require at least:

https://github.com/spring-cloud/spring-cloud-scheduler-kubernetes/pull/5
https://github.com/spring-cloud/spring-cloud-dataflow/issues/2404

Need to create a CI environment before we can merge this PR. https://github.com/spring-cloud/spring-cloud-scheduler-kubernetes/issues/4

EEK!  Ignore my link above for acceptance tests.   I should have read the detail.   What I meant to say is that we need to have a bamboo build for Spring Cloud Scheduler Kubernetes so its bits can be in the repo.

artifacts are now being published & travis is happy again


Added support placement value of related auto*

Coverage decreased (-0.1%) to 84.429% when pulling 140a91344900ed97b7c9406c35cfa2a0d42e6857 on dev into b3031129fc3574b50572394b37a408da747b11a5 on master.


use ecdsa for keytype. But when signing, support both formats

Summary
This PR updates the key type in the root and targets metadata to use the new recommended key type 'ecdsa'. This will generate new key ids for all keys.
During signing an option exist to add the signature to both the new and old key id to allow for clients to accept the new root, which effectively rotates all root keys.
Release Note
N/A
Documentation
N/A

This is based on the work done earlier that was removed in this pr https://github.com/sigstore/root-signing/pull/617/files

@haydentherapper I didn't add any unit tests as this will be a one-off thing. I plan to test this this week with @joshuagl to see that it's working as expected.

Note we also need to update the delegate key - https://github.com/sigstore/root-signing/blob/main/repository/repository/targets.json#L116


Configurable Showing IN STOCK

I set all the Simple Product in Configurable Product to Out of Stock But Still Main Configurable Product Showing IN STOCK Option
http://www.husnacollection.com/store/batwing-silver-grey-78.html
Please Check the Issue

@waqasyousuf4u This is not really an issue. is more of an inconvenience or a limitation of Magento.
If you choose not to show the out of stock simple products and the configurable product is out of stock you won't see any configurable attributes. This is how Magento works.
My only change was to allow the display of out of stock combinations. This makes the add to cart button appear and the configurable attributes, but you won't still be able to add the product to the cart.
Sorry but I will not fix this mainly because it's not an easy task. It involves changing a lot of default behavior and I try to avoid this.

I have no problem with Add to cart Button
I just need that when all the simple products is out of stock so configurable product will also be out of stock
if i disabled the extension it will show out of stock but the drop down size will also be hide
i need to show the sizes also
thanks bro for replying

@waqasyousuf4u That's what I explained. If a configurable product is out of stock you won't see the dropdown. It's default behavior. What I did was just to simulate that the product is in stock to be able to see the dropdown. You cannot have it both ways without heavy coding.

Ohhh
Thanks Bro

Hello @tzyganu and @waqasyousuf4u ,
it is not so heavy.
For my own shop i have switcher change for this.
I have Patchfiles for the config.php and for the product.js on GDrive https://drive.google.com/folderview?id=0B6MNreLwW6TefjNaTWo3RUhSWkFWYnBKQkRkWHhoTVVIb19Cb0dTOGpJZmp4MTVqN3hMd2M&usp=sharing
In your /app/design/frontend/rwd/(own_template)/template/catalog/product/view/type/default.phtml
you muss add

<?php echo $this->__('Select an option to see the immediate availability.') ?>

<?php echo $this->__('ansonsten'); ?>

For better understand my default.phtml is also in the directory.
Sample can you found on http://cherokee-trading.de/holzrahmen-fuer-den-trommelbau-rund.html
Then you change the config, you can see how much is aviable and the delivery time.


how to apply styles to particular columns

Hi, how to apply dynamic styles to columns...

Do you want to make the cell's style dynamic, or is it the content's style itself?

Example:

Company
Contact
Country

Alfreds Futterkiste
Maria Anders
Germany

from above eg on first td tag there multiple style ,for second td only one style,and for last td no styles..

@masnoonmulla are the styles static (as in, the columns' style in all rows will be that way)? If yes, then probably we can take advantage of CSS nth-child(n) (example here: https://codesandbox.io/s/issue-71-example-kdtks). Let me know if this suits your need!
But, probably that's a bit "hacky". Let me see if I can work on something in the next week to add a prop, something-something like, cellProps or something.
<Datatable
  cellProps={{
    // Pass string for `className`, or pass `CSSProperties` for `style`.
    username: {
      className: 'td-username',
      style: { fontWeight: 700 }
    },
    // The other way to do it is to pass a function to dynamically "adapt" based on the cell's value.
    name: {
      className: value => value === 'Bob' ? 'td-username' : undefined,
      style: value => value === 'Bob' ? { fontWeight: 700 } : undefined
    },
  }}
/>

Please let me know what you think!

Yes, codesandbox.io this work for me. :)
we faced another issue. saving state of the table like pagination, search etc.

@Imballinst Yes, codesandbox.io example work for me. and please let me know after above example implemented.
we faced another issue. saving state of the table like pagination, search etc.

ok! I will update this issue when I have worked on that. With regards to saving the table state, I think you can use query parameters and make the table's state controlled -- so we can "re-hydrate" the state from query parameters, in a way.

Please let me know the example of table state on Codesandbox.io !

Hi @masnoonmulla -- I just pushed an alpha version 2.1.2-alpha2 for cellProps -- you can see its usage here: https://codesandbox.io/s/212-alpha2-cell-styles-and-classnames-by-string-object-and-functions-lm9ze.
As for the controlled table state, could you try something similar to https://imballinst.github.io/react-bs-datatable/?path=/story/advanced-guides--asynchronous-table? Technically, we are "lifting" the states from the table to outside of it. Then, you can use local storage or query parameters to "store" and "re-hydrate" the state.

Hi @masnoonmulla -- I just pushed an alpha version 2.1.2-alpha2 for cellProps -- you can see its usage here: https://codesandbox.io/s/212-alpha2-cell-styles-and-classnames-by-string-object-and-functions-lm9ze.
As for the controlled table state, could you try something similar to https://imballinst.github.io/react-bs-datatable/?path=/story/advanced-guides--asynchronous-table? Technically, we are "lifting" the states from the table to outside of it. Then, you can use local storage or query parameters to "store" and "re-hydrate" the state.

Thanks for your support


Optimizing reading from datastore during WorkflowSweeper#sweep

Pull Request type

[x] Bugfix/Optimization
[x] Refactoring (no functional changes, no api changes)

Changes in this PR
Added a dependency to executorDaoFacade in WorkflowSweeper.
Added overloaded method in WorkflowRepairService - verifyAndRepairWorkflowTasks(WorkflowModel workflow)
Added overloaded method in WorkflowExecutor - decideWithLock(WorkflowModel workflow)
WorkflowSweeper now calls executorDaoFacade to get the workflow once and then passes is to the overloaded methods.
Describe the new behavior from this PR, and why it's needed
The new behaviour reduces the stress on the datastore during heavy loads. We have observed absurd loads on our redis instance comming from constant sweeping from the workflowSweeper ( more than 10GBps )

🥺  please review


Test webhook issue

What would you like to share or ask?
This is test issue
Additional information
No response

New comment

Test comnent


Update all transforms to use single package library with [extra]

Why are these changes needed?
Remove pyproject.toml for both ray and python and update makefiles and dependencies
Related issue number (if any).

LGTM

Minor comment: Please consider renaming WHEEL_FILE_NAME to DPK_WHEEL_FILE_NAME