Isn't synchronous I/O bad?

Hi,
Sorry if this question is asked before, I have checked the closed issues but I can only find this one https://github.com/JoshuaWise/better-sqlite3/issues/150
My question is you said

node-sqlite3 uses asynchronous APIs for tasks that are either CPU-bound or serialized. That's not only bad design, but it wastes tons of resources.

But thru my limited experience with sqlite I found most of time the execution  is I/O-bound instead of CPU-bound, correct me if I was wrong. If it is I/O-bound isn't that what asynchronous APIs is used for?

There are many legitimate concerns with synchronous filesystem I/O. However, in the case of SQLite3, there are also reasons to not be concerned.
What if I'm using a network drive, or a slow spinning disk drive?
For a Node.js program, this is very rare. Most Node.js programs are web servers, and most server providers use SSDs by default, including AWS, Google Cloud, Digital Ocean, etc. SSDs are the default choice for web servers and relational databases in 2018. If you're not using an SSD, then you're already making a bad decision, regardless of which relational database you choose to use, and regardless of which SQLite3 library you choose. The primary reason to use a spinning disk drive is to spend less money, which is typically only relevant for very large drives (in the terabyte range), and if your database size is in the terabyte range, then you probably shouldn't be using SQLite at all.
What if I'm using an operating system with a poor filesystem cache?
In 2018, nearly all modern filesystems are very efficient. When it comes to reading data, modern filesystem caches have an extremely high cache hit rate. If your database is extremely large (in the terabyte range) then random access patterns will be difficult for the filesystem to deal with, but, as noted previously, if your database size is in the terabyte range then you shouldn't be using SQLite3 at all. On the other hand, with a 60 GB database (for example), most modern filesystems are able to use cache very effectively, even with semi-random access patterns.
Isn't it slow to write data, even with an SDD?
When it comes to writing data, the typical bottleneck is fsync() operations. Fortunately, SQLite3 is very smart (especially in WAL mode), and will only perform fsyncs very rarely, when absolutely necessary. The summary is that SQLite3 does not simply do fs.writeFileSync() every time you insert or update a row—it's much smarter than that. You can read about WAL mode here and here.
What if I'm running queries that return large amounts of data?
This is usually the primary reason to discourage synchronous filesystem I/O. Even if you're using an SSD and a modern operating system, reading very large chunks of data (> 1 MB) will inevitably be slow, which is bad when that blocks your main thread. However, most system architects keep large chunks of data (such as images or videos) in separate files—not part of the relational database itself (only keeping filenames in the database). This is considered good practice even in full-fledged RDBMSs such as PostgreSQL or MySQL. Therefore, most common database reads are in the kilobyte range, or smaller. This type of access pattern is extremely fast on modern operating systems. In fact, it's typically so fast that the overhead of an asynchronous operation (context switching, thread management, etc.) would actually cause more latency than simply reading synchronously. This is not always the case, but it commonly is.
The nature of SQLite3
Even if you're using an asynchronous SQLite3 library, SQLite3 is only capable of executing one query at a time. You cannot execute queries in parallel within a single process. Typical asynchronous SQLite3 libraries (such as node-sqlite3) actually queue up your queries in the background, and execute them one-by-one, even if you attempt to execute them in parallel. So you have a choice with SQLite3: either execute your queries one-by-one in a background thread, or execute your queries one-by-one in the main thread. The only advantage to using a background thread is to avoid blocking other operations while a query is running. However, with a proper database design (as stated in the README), individual queries should generally complete within a matter of microseconds, even with a large database and complex joins. At that speed, you're not adding any practical latency to network requests or other operations performed by your main thread.
If your queries are unavoidably slow, then an asynchronous SQLite3 library would make things easier for your main thread, but it won't save you from the real bottleneck in that case: your database, which is only able to execute queries one-by-one. If you fall into this category, you should be using a full-fledged RDBMS such as PostgreSQL or MySQL. In summary, if your queries are slow, SQLite3 is not for you, regardless of which library you choose.
A synchronous SQLite3 library avoids the overhead of thread management, which is great. But in addition to that, it also enables:

transactions (you don't risk other code injecting queries into your asynchronous transaction)
user-defined functions, including aggregate and window functions
prepared statements that are re-usable (which tremendously improves performance)

I hope this helps to understand why one might choose to use a synchronous SQLite3 library in Node.js.

wow I didn't expect such a detailed respond! Thanks! I am pushing hard to let me team switch from node-sqlite3 to better-sqlite 👍

This issue should be pinned. Its very helpful and explains a lot of things about the design decisions

Leaving this issue open as it's a common question people have.

So does that mean I shouldn't use better-sqlite3 for my Electron App?
I'm building a desktop application to be consumed by the average user, and very likely, not everyone who will install my software would be owning a descent hard disk drive, let alone, an SSD.

@jasonsparc Although desktop apps can't control the environment that they run in, they only need to serve one user, instead of thousands or millions. So you'll be fine. Many people/organizations use better-sqlite3 for Electron apps, and are happy with the result.

@JoshuaWise I really want to use better-sqlite3 (especially with its memory management features, as being a huge plus!). But, what prevents me from using it is that, I have just found out that it is compiled with SQLITE_THREADSAFE=0 by default.
To describe what I'm trying to accomplish:
I was considering a synchronous alternative for sqlite3 npm package so that I have more control on what background thread my queries would run, and thereby having more control on performance tunings. Also, not only that I would perform multiple queries at the same time, I would also be running an automatic peer-to-peer syncing between devices using my electron app, and also with a flutter app (for mobile).
If it were compiled instead with SQLITE_THREADSAFE=1, then I wouldn't have to worry about making my own serialized queue in the JS side on my worker thread (nor should I worry about the safety of my system's design), saving me some precious development time. I want my app to be fast, but I also want it to be maintainable enough.
I also think that SQLITE_THREADSAFE=1 should've been the default for better-sqlite3, since people should be able to toggle it off at runtime anyway. I also think that it could've have been too error-prone for users of better-sqlite3 if they are using them in an electron app with worker threads, that is, at least for people who skipped reading the docs, not realizing the dangerous implications that SQLITE_THREADSAFE=0 is set by default. Maybe an updated README file would help.
But, I'm also looking forward to the resolution for https://github.com/JoshuaWise/better-sqlite3/issues/204. Hoping that you would eventually decide to use SQLITE_THREADSAFE=1 instead—also because node.js is no longer just for single-threaded applications, and as stated by SQLite docs:

2. Recommended Compile-time Options
… Not all of these compile-time options are usable by every application. For example, the SQLITE_THREADSAFE=0 option is only usable by applications that never access SQLite from more than one thread at a time. …

@jasonsparc See my comment in https://github.com/JoshuaWise/better-sqlite3/issues/237#issuecomment-471647449.
Worker Threads in Node.js are a very new feature. Since better-sqlite3 is a native addon, supporting them would be more complicated than simply enabling SQLITE_THREADSAFE=1. It's something I'm interested in supporting in the future.

@JoshuaWise Cool! Thanks for the fast reply.
I'll be looking forward to that then! :)


Add htsPrecompile acceptance test support for auto renew methods

Problem
No htsPrecompile acceptance test support for exists for auto renew methods
Solution
Add support for token autoRenew

tokenGetAutoRenew(address)
tokenUpdateAutoRenew(address, uint32)

Alternatives
No response

Those functions are renamed to:

getTokenExpiryInfo(address)
updateTokenExpiryInfo(address, Expiry)

However this is still not implemented, waiting for this PR https://github.com/hashgraph/hedera-services/pull/3756 to be merged, so I can continue.


fix: prevent sasslint warning when using qualifying element with class

Description

Fixes a warning due to a qualifying element with a classname in the dropdown-menu component.

Closes

Motivation and Context

Fixes a warning due to a qualifying element with a classname in the dropdown-menu component.
Screenshots (if appropriate):
Types of changes

[ ] Documentation
[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing
functionality to change)

Checklist (all required):

[x] I have read and follow the CONTRIBUTING document.
[x] There are no other pull request similar to this one.
[x] The pull request title is descriptive.
[x] The template is fully and correctly filled.
[x] The pull request targets the right branch (develop or develop-v...).
[x] My commits are correctly titled and contain all relevant information.
[x] My code follows the code style of this project.
[ ] I have updated the documentation accordingly to my changes (if relevant).
[ ] I have added tests to cover my changes (if relevant).
[x] All new and existing tests passed.

Take a look at gulp lint 😜

What's the issue with it?

Take a look at gulp lint 😜

It seems these are never shown during commit and push.

It seems we have many new issues which are suppressed during commit and push and also in npm run test. So I guess I'll take another approach and check why it is now different than before. I thought we fixed all these warnings.


Add Path Support

Add support for serializing and deserializing paths

@tmaila I've added support for paths in a branch and I think it's ready for merge into master.
Note that I got a little creative in my round trip unit tests, since I got tired of copy+paste :-)

I think Not A Path should serialize to null and not to a string.

That seems like it might make sense.
Would you do similarly for NaN?
I did some googling and it seems that JSON doesn't support NaN, +Inf, -Inf.
http://stackoverflow.com/questions/1423081/json-left-out-infinity-and-nan-json-status-in-ecmascript
On Fri, Aug 26, 2016 at 7:07 AM, Tomi Maila<EMAIL_ADDRESS>wrote:

I think Not A Path should serialize to null and not to a string.
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub
https://github.com/JKISoftware/JKI-JSON-Serialization/issues/13#issuecomment-242745084,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAXR-NkyB9GEdvnKI5K3KlO94mOQbjsnks5qjvMEgaJpZM4Jtvrm
.

@jimkring you should open a separate case for NaN but no I would not do the same thing for NaN because NaN is not an object but a value and values in other languages cannot serialize to null. We could add an option to flatten NaN into null but that should not be a default behavior. The JKI JSON Serialization allows you to control if NaN/Inf are converted to symbolic values not compatible with JSON but supported by many JSON serialization libraries or flatten them to numbers. +/- Inf flattens to very large values and NaN flattens to zero. This is a weakness of JSON standard and there is not standard good workaround.


Zombies wont spawn after round 10

Hello me and some other players on my server was trying to play zombies as long as we can but we figured out that after round 10 there wont spawn any zombies but the game continues

What version of MC is this? Any errors in your console?


what's the name of the trajectory smoothing of this project?

i want to know more about this mathematical method,thank you very much.

The smoothing is done with a moving average

Thank you for your reply
------------------ 原始邮件 ------------------
发件人: "Adam @.>;
发送时间: 2021年11月14日(星期天) 晚上7:26
收件人: @.>;
抄送: @.>; @.>;
主题: Re: [AdamSpannbauer/python_video_stab] what's the name of the trajectory smoothing of this project? (Issue #127)
The smoothing is done with a moving average
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or unsubscribe.
Triage notifications on the go with GitHub Mobile for iOS or Android.


[Editor] Avoid to focus an existing editor when enabling the layer

It fixes #18911.

/botio integrationtest

Unfortunately, the puppeteer update brought some Chrome-only new more-or-less intermittent issues... :(

r=me, assuming that you're sure that the test failures are unrelated to this patch; thank you.

I thought it was Chrome-only because only CDP was appearing in the backtraces, but in fact the two tests about existing annotations are just failing. I don't why we didn't get an error from the test in Firefox or maybe because the timeout exception for Chrome avoided the one for Firefox.

@Snuffleupagus I made few changes, are you still ok ?


Fix is not applied even after doing everything right

I am able to get the the fix to copy my 001.sav to the new one, however when I put the new sav into the server, the server just creates a new sav file and replaces my fresh copy. My friends are able to have all their data without any rewrites. Any idea why this might be happening? Want to add that system locale is set to english and US
EDIT* I am also able to recover all of my map progress (only fog of war) on the fresh character which is the only thing that I have been able to recover from my coop save, otherwise the dedicated server forces me to create a fresh character.

I am having the same issue, my co-op host no matter what re-creates his .sav file.

@OgreCL it sounds like you're moving a .sav file instead of the entire folder of the save. You can't move individual .sav files from one save folder to another because it will break things. You have to copy your co-op save folder to your dedicated server, connect to that save on that server with the co-op host's account, create a new character on that save, and only then run the script.

@xNul Thanks for the help! I tried running the script directly from inside the dedicated server file (i am not sure if this is what you meant) but either way i created a new character on the server, ran the script from directly in the dedicated server with the 0000.1 sav and it worked, but when i joined my server my original character's design wasn't there but a fresh character, but it did show my character wearing my original armor, but then jumped back to being a fresh character. So it still replaced the new sav file. I tried doing this when the server was on and off, I also tried doing it in another file and then copying the player folder into the dedicated server alongside the level.sav file. The server still replaces the freshly copied sav file for my character but the world is intact.

@OgreCL np! The server should not be running when you run the script on it fyi. I've added that step to the README so it's clear in the future. Also, moving the contents of the save folder to another save folder won't help the player/character situation unfortunately. How do you mean your character jumped back to being a fresh character? Like your clothes instantly swapped? How was your inventory, level, and guild?

I'll reopen this

@xNul i was able to see a glimpse of my armor, levels and some of my pal balls in my inventory, but then it instantly swapped back to the fresh character. I wasn’t able to check anything else because it was so fast.

Thought I'd my 2 cents here, ran through the steps multiple times to the letter, didn't experience any "glimpse" of my previous character (I was the host) but the world transfers as expected, but it always make me recreate a fresh character.

Experiencing the same. Moving from hosting on my PC to paying for a server.
Connected to the new server and got a new GUID. So I followed the steps to migrate. For me this was
NewID: DEB75069
OldID: 8895D8B4
python .\fix-host-save.py "C:\Users\jtgee\.cargo\bin\uesave.exe" "D:\Home\Desktop\PalWorldBack\85631D364AAC589EC12D9EAAA421816B" DEB75069000000000000000000000000 8895D8B4000000000000000000000000
Then I deleted all the files on the dedicated server in the "85631D364AAC589EC12D9EAAA421816B" directory and copied up from the local files where the script ran.
When I launched it prompted me to make a new character and deleted the file I had in the Players folder.

I'm also being forced to create a new character every time regardless of running the script properly, following all instructions to the letter, copying entire directory  instead of just .sav files, etc

same issue here
no matter what it makes me make a new character

I've gone through many different iterations trying many of the fixes suggested by folks on here and on reddit, no dice.
Willing to share save files if it would help.

Same issue here, happy to provide sav files if need be. I followed every step down to a T and the script completed successfully but sadly joining still makes me make a new character.

Unfortunately I don't have time to debug individual saves so we'll have to wait until someone figures out the cause of the bug and fixes it, or I'm able to finish a parser I'm working on for the byte data to make it easier to understand all of these kinds of bugs and avoid them.
For the future, I'm going to close similar issues and redirect them to mega-issues like this one.

For people who are still having trouble, try applying the fix mentioned in https://github.com/xNul/palworld-host-save-fix/issues/60


⚠ Bofrost.de has degraded performance

In e759d61, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 1278 ms

Resolved: Bofrost.de performance has improved in 331df1e after 6 minutes.


event: use duration_cast more nicely in timer_impl

fix #4332

@ranchoooo how come you closed/deleted this?

(don't mind me, just testing if repokitteh issue is fixed)

/rk-ping


Cue trim bug

What version of CUE are you using (cue version)?
0.3.0-beta1
What did you do?
cue trim -s ./... in the root of the attached trim-bug.tar.gz
What did you expect to see?
In stats-api/prod/us-central1/analytics/analytics.cue:
package stats_api

"stats_api": analytics: {}
"stats_api": analytics: deployment: spec: template: spec: containers: [{
	image: "foo:oiuc812"
}, {}]
stats_api: analytics: _env: {
	bar: "baz"
	foo: "bar"
}

or better yet (trimming empty object & consolidating):
package stats_api

"stats_api": analytics: {
	deployment: spec: template: spec: containers: [{
		image: "foo:oiuc812"
	}, {}]

	_env: {
		bar: "baz"
		foo: "bar"
	}
}

What did you see instead?
bug-report.txt

This issue has been migrated to https://github.com/cue-lang/cue/issues/643.
For more details about CUE's migration to a new home, please see https://github.com/cue-lang/cue/issues/1078.


Unable to load configuration process

Hi all,
This integration looks amazing. It's a long time that I try to integrate my Dreame Z10+ in HA, with xiaomio MIOO, with PiotrMachowski but I always find something not working. This integration can really solve al my issues.
I added the repository via HACS, restarted HA and started with the configuration but I get a "unable to load the configuration flow" error message and I don't know why it pops out and what can I do. Can you please help me?

Logger: homeassistant.config_entries
Source: config_entries.py:712
First occurred: 10:44:24 (1 occurrences)
Last logged: 10:44:24
Error occurred loading configuration flow for integration dreame_vacuum: cannot import name 'Platform' from 'homeassistant.const' (/usr/src/homeassistant/homeassistant/const.py)

I was actually using a quite old version of HA. I am updating it now. In the server log I can see the exact same error message:
[31m2022-12-29 15:41:12 ERROR (MainThread) [homeassistant.config_entries] Error occurred loading configuration flow for integration dreame_vacuum: cannot import name 'Platform' from 'homeassistant.const' (/usr/src/homeassistant/homeassistant/const.py)
No idea why it gives this error. I will provide more info once I  update HA. Thanks!!!


ExtrudeBufferGeometry does not exist

As reported on the boilerplate issue, ExtrudeBufferGeometry does not exist.
Fix the Extrude component build function to build  buffered extrude geometry from the extrude geometry.
Version:

[x] v2.x.x
[ ] v1.x.x

Issue type:

[x] Bug
[ ] Proposal/Enhancement
[ ] Question
[ ] Discussion

 Tested on:
Desktop

[ ] Chrome
[ ] Chrome Canary
[ ] Chrome dev-channel
[ ] Firefox
[ ] Opera
[ ] Microsoft IE
[ ] Microsoft Edge

Android

[ ] Chrome
[ ] Firefox
[ ] Opera

IOS

[ ] Chrome
[ ] Firefox
[ ] Opera

fixed.


Set default if no value is set for variables which require that

Fix #491 - site will be generated even with empty config.toml.

Thanks for the suggested improvement. However, due to the large amount of Academic specific variables, I would rather help prevent the issue by redirecting the user to the installation guide rather than create defaults in the code for every single variable for the edge case where the user clones Academic without completing installation.


Introduce OM to T1 project

Upstream PR: chipsalliance/chisel#4028

This bump doesn't include the 1.74 version of circt...

Cc @SharzyL this api is used to handle all metadatas our c header and emu should use it.


Introduce ConvexHull Body

I introduce a new Body type. I've extended the debug renderer to show it as well.
See #59 for a bunch of things I'm not happy about yet. In brief summary:

I wrote convert code outside of convert.rs. This also results in unused import warnings for body.rs when I build the demo. (I also see that this breaks the PR checks)

it could be nice to have a better way to convert a slice of Vec3 to a slice of Vec2 in the implementation of dim2.rs

building a convex hull may fail, which I've hacked around with using unwrap(). What would be a better error handling mechanism?

While I still have questions about cleaning up dim2, I think this answers most of the concerns I have with this PR, so I'm fine with merging it at this stage.


Question: Hide a yang module from client

Hi,
Is there a way in which I can install a yang module on server side, but hide it from client?
Something similar to this in netconfd-pro:
The --hide-module parameter specifies the name of a module to hide from advertisements to client sessions. If the specified module name is loaded into the server, then this parameter will cause it to be omitted from the following data structures:

YANG 1.0 <hello> message

/netconf-state/schemas/schema list

/modules-state/module list

No, netopeer2 does not support such a feature.


Publish storybook to a public github page on every change in components or stories

Fixes #411

This change is

Please set the following setting in repo settings:
Pages > branch: github-pages (and keep root)
Like here https://budiirawan.com/images/2022/storybook-github-pages-setting.jpg
Currently gh action config is pointing to this branch, to try it out.

@tjcouch-sil seems it is using the user of the one that triggers the action by pushing. So the branch protection rule needs to be not too strict.
I can see the content being pushed here.
However it was not updating the github page there.

Thank you. I recognized it also shows a blank page for me when built locally. Will need to have closer look into it.
Yes<EMAIL_ADDRESS>allowes to specify auth token secrets

@tjcouch-sil pushing with my credentials does not work, but we could add an auth token of an allowed automation-user

remote: error: GH006: Protected branch update failed for refs/heads/github-pages.
remote: error: You're not authorized to push to this branch. Visit https://docs.github.com/articles/about-protected-branches/ for more information.

Thank you. I recognized it also shows a blank page for me when built locally. Will need to have closer look into it.

Probably it is because of // TODO: Make this work in production mode
https://github.com/paranext/paranext-core/blob/Sebastian-ubs/issue411/.storybook/main.ts#L35

there is goes. well yes you are right we actually told it to only run on a change in components or stories ;-)
Other thing: We switch looking into how to publish with Chromatic instead so was not paying much attention here.

Not quite decided. Maybe let's pause looking into fixing it for now. I can make it a draft PR or anything if the helps anybody

@Sebastian-ubs can we close this PR since it hasn't moved for some time?

Yes, we will follow up in paranext/ux#3 it based on priority


OCM-1419 | fix: validate oic-provider exists in manual mode

Signed-off-by: marcolan018<EMAIL_ADDRESS>

/lgtm


Update basket.js

optional script type

Please include a description with your PR. Your comment mentions JSX in the diff but you don't explain why this would be useful to other users :)

Mentions about JSX aren't relevant.
+1 to custom type attribute.

I was looking into how to word this better.
Sorry for the delay.


Reload Image If Failed

Glide Image: In the current implementation, there is a need to reload an image iteratively until successful loading. The provided URL is confirmed to be correct; however, intermittent failures are attributed to the extended loading time caused by the large image size. Requesting a solution or guidance on implementing a retry mechanism to overcome these intermittent loading failures.

Hey @Micoder-dev, I hope this solution works for now.
  var imageOption by remember {
    mutableStateOf(
      ImageOptions(testTag = "")
    )
  }

  GlideImage(
    imageModel = { poster.image },
    modifier = Modifier.aspectRatio(0.8f),
    component = rememberImageComponent {
      +CircularRevealPlugin()
      +PalettePlugin { palette = it }
    },
    onImageStateChanged = {
      if (it is GlideImageState.Failure) {
        imageOption = ImageOptions(testTag = System.currentTimeMillis().toString())
      }
    },
    previewPlaceholder = R.drawable.poster,
  )


Set Value after successful rule execution

On successful execution of a rule, I want to set some info in the field.

You can use a RuleListener#onSuccess and set the field in that method.

But how do I access the object in RuleListener in which I need to set a value?

A rule's action can create/delete/modify facts, and the set of facts is accessible in RuleListener#onSuccess method. So you can use your object as a fact to get access to it and modify it as needed after the rule execution.


Develop new like functionality for production

We need new like functionality for likes.

We added new Go code in our virtual environment for like functionality. Fixed some bugs.


engine: add kubectl exec update mode flag

what's the status on this pr?

Sorry, going to try to get back to it today.

@nicks updated this to use wire, WDYT?


Forward JS logs to NaCl stderr in Connector app

This makes sure that the logs that are collected on the JavaScript side
are also available in the NaCl process stderr. This should make
accessing these logs much simpler in some situations, because NaCl
process stderr is captured at /var/log/ (at least in test Chrome OS
images for non-guest users).
Before this change, the JS logs were only available via DevTools or via
the "Export logs" UI button. Neither works on Chrome OS login screen (at
least out of the box), therefore piping the logs to the log file will
simplify debugging QA/customer issues.

Fabian, PTAL.
Sorry, the CL got quite big. I think the only part that we could extract is the refactoring of the existing code in log-buffer.js into log-formatting.js - please let me know if you'd prefer to extract this into a separate pull request.


UI Localization

Description
Currently in the hostd Overview screen, all graph use the US date format (mm/dd/yyyy). Please can they display in the localized date format. The format should be taken from either the server (better) or from the browser (best).
Guessing, but if you use the localized date format, you can also use the localised time?
Version
v0.1.0-alpha.1 with fix at d4c887f
What operating system are you running (e.g. Ubuntu 22.04, macOS, Windows 11)?
Ubuntu 22.04
Anything else?
No response

To add to this, we should double check we’re localizing everywhere in the UI: currency, dates, numbers (1,000 vs 1.000)

Closed by #67


Rewrote cl_logger.js in Lua

Removes the memory overhead of running a JS script in FiveM
Fixed a bug where deaths by AI drivers would not be logged

Seems to log AI related deaths.

Doesn't appear to be working correctly.
I tested with death by NPC bullet and drowning, none of them even showed up in the log as a death.
Needs a bit more testing.

That's not the bug.

RIP PR?

Closed in favor of Flawws' #160


SQLITE PK allows varchar

fixes #545
cc @mcollina, giving this anotherTRY.👍
the test failures caused unexpected problems earlier, my friend. I will try to resolve them here..

The test failure is obscure for me, @mcollina. PLS. HELPP.🙂

Here are more details, @mcollina :

FAIL  test/graphql.test.js 4 OK 3s
command: C:\hostedtoolcache\windows\node\18.16.0\x64\node.exe
args:
- test/graphql.test.js
exitCode: 7
signal: null

I'll retry this tomorrow. @mcollina 👍

the tests seem to have succeeded.👍 @mcollina

Thanks


Adding SHELDUS

Hi All,  Wondering  what you think about addng SHELDUS: https://cemhs.asu.edu/sheldus to the repo?

Very interesting. Do we think it should be nested under NRI as a source or its own link? Are components too geographically specific?


Theme is not set for every page on Firefox

On Firefox 118.0.2 only the current page is put in dark mode if I change theme in settings of this page to "Dark". All other pages remains with theme "OS". And the theme for this page is saved and persists even restarting the browser.
The same thing does not happens on Chrome 118.0.5993.71 (Official Build) (64-bit). That is, all pages acquire the same theme.
I'm using typedoc 0.25.2 with "@mxssfd/typedoc-theme": "^1.1.3", on Windows 11 64-bits
Thanks for your help!
mario

Is this the reason?  https://github.com/TypeStrong/typedoc/issues/1707#issuecomment-932976304

Thanks for the explanation @mengxinssfd ! I will test with a web server soon.
Anyway, for my needs simply will open the documentation using Chrome.
Thanks again!
mario


feature(main): set docker default registry address

Describe what this PR does / why we need it
set docker.io is registry default address
Does this pull request fix one issue?

None

we now take dockerhub as default registry? @fanux

we now take dockerhub as default registry? @fanux

No, using ACR or github registry by default


No starter project in "java-maven" component's devfile

/kind bug

What versions of software are you using?
Operating System: all
Output of odo version: master
How did you run odo exactly?
$ odo create java-maven --starter
Actual behavior
odo create java-maven --starter
Validation
 ✓  Checking devfile existence [10921ns]
 ✓  Creating a devfile component from registry: DefaultDevfileRegistry [13215ns]
 ✓  Validating devfile component [58186ns]
 ✗  no starter project found in devfile.

Expected behavior
As a user, I would like to be able use some starter project in the devfile just like I can do so for other component types
Any logs, error output, etc?
$ odo registry list
NAME                       URL                                                                SECURE
DefaultDevfileRegistry     https://raw.githubusercontent.com/odo-devfiles/registry/master     No

/area devfile
/area component

Starter project for java-maven has been added recently https://github.com/odo-devfiles/registry/blob/51d1360a97f094a9e0b77b8976ccd25bbf571274/devfiles/java-maven/devfile.yaml#L6


feat(std): rename operators/reducers ports nameing from acc/el to lef…

…t/right

This PR adds switch statement and lots of other fixes and improvements


Rework docs for CloudStack KubeVIP disable env flag

The CLOUDSTACK_DISABLE_KUBE_VIP is optional and doesn't feature in a typical EKS-A deployment. This docs change moves the configuration to a separate optional section leaving the cluster creation steps to represent a typical EKS-A cluster deployment.

/approve
/cherry-pick release-0.18


Error executing the example in the browser

I followed all the steps of the article https://lord.io/blog/2017/wargo/
However I'm getting the following errors when I try to access the http://<IP_ADDRESS>:8080:

Are you doing the wargo-loader example with Webpack? Do you think you could post the output of running wargo test in your project?

Yes, I was testing the wargo-loader.
Shows the results for the wargo test:
▶ wargo test
  Setup wasm checking dependencies...
  Setup wasm found emsdk installation in ~/.emsdk
  Setup wasm setting environment...
emcc (Emscripten gcc/clang-like replacement + linker emulating GNU ld) 1.37.22
clang version 4.0.0  (emscripten 1.37.22 : 1.37.22)
Target: x86_64-apple-darwin16.7.0
Thread model: posix
InstalledDir: /Users/nuxlli/.emsdk/clang/e1.37.22_64bit
INFO:root:(Emscripten: Running sanity checks)
  Setup wasm running 'cargo test --target=wasm32-unknown-emscripten --no-run'
    Finished dev [unoptimized + debuginfo] target(s) in 0.0 secs
  Setup wasm running 'cargo test --target=wasm32-unknown-emscripten --no-run --message-format=json'
    Finished dev [unoptimized + debuginfo] target(s) in 0.0 secs
  Setup wasm running tests...
wasm streaming compile failed: TypeError: Incorrect response MIME type. Expected 'application/wasm'.
falling back to ArrayBuffer instantiation
pre-main prep time: 895 ms

running 0 tests

test result: ok. 0 passed; 0 failed; 0 ignored; 0 measured; 0 filtered out

Have you found any solution yet? The same problem happened in my browser.

I could also reproduce this issue on macOS 10.13. It appears to be darwin specific, as I could get it running if I compiled on Linux. I'm not sure if this is an issue specific to wargo or the Rust compiler itself.

It is happening for me on the mac as well
The error seems to be the wasm instantiation failing
    function instantiateArrayBuffer(receiver) {
      getBinaryPromise().then(function(binary) {
        return WebAssembly.instantiate(binary, info);
      }).then(receiver).catch(function(reason) {
        Module['printErr']('failed to asynchronously prepare wasm: ' + reason);
        abort(reason);
      });
    }

If anyone wants to jump on a call and troubleshoot I can demo the issue (I personally don't really know rust or webpack)


Add {% liquid %} tag

Companion C implementation to https://github.com/Shopify/liquid/pull/1086
@Shopify/liquid in particular @dylanahsmith for review

there is some urgency on this because I optimistically merged the ruby side into master. @pushrax is @dylanahsmith's suggestion worth implementing?

Build should be fixed in https://github.com/Shopify/liquid-c/pull/50


Do: Update UI

Let's make this BEAUTIFUL
Bootstrap Bulma but darker.

tech stack

Adobe XD
CSS3, SASS

time blocks

[ ] 1

[ ] 2

[ ] 3 (taking too long)

[ ] 4 (push for tmr)

notes

addl

Not important. Pushed for light.

Not important. Pushed for light.

https://github.com/mateomorrison/mateo/blob/main/ui/main.png

https://github.com/mateomorrison/mateo/blob/main/ui/main.png


Targeting Framework not Supported

Hello,
We are trying to add the Nuget Package AnyAscii to our C# .Net Standard 2.1 class library.

We noticed that the package wasn't compatible even though we are able to clone your repo and take a dependency on the any-ascii csproj inside the repository.

AnyAscii 0.2.0 supports .NET Standard 1.0
AnyAscii 0.3.0 supports .NET 5.0 or .NET Core 3.0 because it uses System.Text.Rune and System.ReadOnlySpan
0.3.0 should have much better performance

Yes but if it supports .NET 5, shouldn't you add Standard 2.1 if you add Core 3.0?
For .NET 5 code, net5.0 replaces both netcoreapp and netstandard TFMs.
https://docs.microsoft.com/en-us/dotnet/standard/net-standard#net-5-and-net-standard

.NET Standard 2.1 does not support System.Text.Rune while .NET Core 3.0 does

Thank you. What advantages does this new implementation have over the previous implementation?

Thank you for your diligence.


OV encoding

This pull addressed https://github.com/openconfig/bootz/issues/40. Note that the pull is based on pull https://github.com/openconfig/bootz/pull/37.

Should I also be reviewing PR 37? It looks like this one already is based off that.

Should I also be reviewing PR 37? It looks like this one already is based off that.

no, as u said this one is based on that pr. I created two pr to ensure one pr per issue.

@mojiiba addressed your comments, thanks for the review.

Still a few typos in the code (both variable names and comments). Please fix those up but apart from that the logic looks good. Please also resolve the comments when they're addressed.


Unreadable documentation du to contrast issues

Hi,
I don't know if I'm the only one struggling, but for me the documentation is essentially impossible to read:

I can't see the black text on the very dark grey background.

Have experienced the same and came here to open an issue about it. It's almost impossible to read docs

see https://github.com/millsp/ts-toolbelt/issues/326#issuecomment-1369376765 for a workaround.


:size command stopped working again

Yeah... Been trying to figure this out for a while. Could use a fix about now.

Are you using R15? If so #13

I tried using both R15 and R6, neither worked.

This is most likely the same issue as issue 13 as it is unable to find Torso. However, R15 has a uppertorso and a lowertorso, R6 does not have this issue. Inless there's another reason why it cannot find your torso that I cannot think of.

would bodyscale or whatever work for this cuz I'd rather just use that than some painful janky solution like what we have now

yeah, would just go through all the values in humanoid for changing size
and multiply it by number
On Thu, Jun 25, 2020, 8:39 AM Sceleratis<EMAIL_ADDRESS>wrote:

would bodyscale or whatever work for this cuz I'd rather just use that
than some painful janky solution like what we have now
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/Sceleratis/Adonis/issues/111#issuecomment-649591564,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AA2OIGKXNMINAANDVF5JPELRYNORNANCNFSM4MYKVDKA
.

just confirmed the scale stuff should work as a suitable replacement, updating it in now.

Updated in the use of bodyscale values and removed the old code entirely. It works on R15, haven't tested anything else so I don't know if this works at all with R6.


Update SOAR on-premises release note

PLEASE READ
Following a recent back-end update, all contributors with a local clone or fork of our repository are required to run yarn install. This does not apply to direct page edits.

[x] Yes, I've run yarn install
[ ] No, does not apply to me

Purpose of this pull request
This pull request updates the following release note:
https://help.sumologic.com/release-notes-csoar/2023/11/01/23-application-update/
This is a request from Jeff Thayer. As @stacykor said in a direct Slack message:

Hey John, @jthayer asked that we change one sentence in the release note on on-prem SOAR from
"To continue to utilize Sumo Logic’s SOAR capabilities, use our SaaS product, Cloud SOAR."
to
"To upgrade to Sumo Logic’s Cloud SOAR offering, reach out to Sumo Logic for more information on Cloud SOAR."
Because there is a cost and a manual upgrade involved in migrating to Cloud.

Select the type of change:

[ ] Minor Changes - Typos, formatting, slight revisions, .clabot
[ ] Update Content - Revisions, updating sections
[ ] New Content - New features, sections, pages, tutorials
[ ] Site and Tools - Updates, maintenance, dependencies, new packages for the site (Docusaurus, Gatsby, React, etc.)

Thanks, @kimsauce!


Import Code Order Question from XML

While doing my 1st part of the project, implement the service to import a ordered Multiple Choice Question from XML, i found a bug when importing a Code Order Question.
In the following test:
https://github.com/socialsoftware/quizzes-tutor/blob/a38161cec73c4bb57710b94eba982a5c7a98d66d/backend/src/test/groovy/pt/ulisboa/tecnico/socialsoftware/tutor/impexp/service/ImportExportCodeOrderQuestionsTest.groovy#L27-L46
if I add another slopDto4 with order=null, the test will fail because of the code below:
https://github.com/socialsoftware/quizzes-tutor/blob/a38161cec73c4bb57710b94eba982a5c7a98d66d/backend/src/main/java/pt/ulisboa/tecnico/socialsoftware/tutor/impexp/domain/QuestionsXmlImport.java#L204
Integer.valueOf() will throw a error when receive the string "null".
Henrique Lin - ist194235 - G09

Fixed in PR. Thanks!


Format log messages from the FMUs

Log messages are currently not formatted as intended by the FMI standard.

See https://github.com/qtronic/fmusdk/blob/master/fmu20/src/shared/sim_support.c#L603

closed in 9a0284cc3a9a8dbfdea9fae098cbdc8fc5b13cf7


The code style should be unified (e.g. interfaces vs classes)

I'm in favor of interfaces, at least for type classes "attached" to collection types etc.
I also started to remove all unnecessary modifiers in interfaces (public for methods, public static final for constants etc)

I like the interface style too.
On a side note. I've seen someone use instances to represent the instance of all typeclasses.
E.g. IO.instances.map(..), IO.instances.bind(...), instead of IO.functor.map(...), IO.bind.bind(...). What do you think of this style?
Of course List would have two of them for the two different interprations of a List. (maybe instances and zipInstances.

The instances idea wouldn't work well in our case, as the construction of our instances often requires other type-classes (e.g. a monoid for the first element type in order to construct a T2 monad).
I'm still not really sure how "fine-grained" the instances should be. I think that at least Functor, Applicative and Monad should be separate, but I'm less sure about things like Apply and Bind. BTW, one could argue that Pointed is missing in our hierarchy.

That is very true. Keep them separated apart for now.


Remove Black Borders in Vertical Continous

Steps to reproduce

Expected behavior

Actual behavior

Mangayomi version
Latest
Device
IPhone 13 Pro Max
Other details
No response
Acknowledgements

[X] I have searched the existing issues and this is a new ticket, NOT a duplicate or related to another open or closed issue.
[X] I have written a short but informative title.
[X] If this is an issue with an extension, I should be opening an issue in the extensions repository.
[X] I have updated all installed extensions.

Swapping to webtoon fixed it, but who knows, maybe it’s like this for a reason, feel free to close if so.


WinUI in VS 2019 Preview gives error "... requires WinRT.Runtime.dll version 1.2 or greater" and more

Describe the bug
Unable to build "Blank App, Packaged (WinUI 3 in Desktop)". The error list contains:

This version of Project Reunion requires WinRT.Runtime.dll version 1.2 or greater.
Please update to .NET 5.0.6 SDK or later.  Or add a temporary Microsoft.Windows.SDK.NET.Ref FrameworkReference of version > 10.0.*.16 or later.

I also note that I have yellow triangles in the Dependencies regarding the WinUI, see below.
Steps to reproduce the bug

I installed the VS 2019 Preview from microsofts webpage, version is Version 16.10.0 Preview 2.1
I followed the instructions here: https://docs.microsoft.com/en-us/windows/apps/project-reunion/get-started-with-project-reunion#set-up-your-development-environment and here https://docs.microsoft.com/en-us/windows/apps/winui/winui3/get-started-winui3-for-desktop to setup Reunion
Open VS2019 Preview, create new Project ""Blank App, Packaged (WinUI 3 in Desktop)"
Build

Expected behavior
I expected the app to build and run, with no apparent errors.
What happened instead
In step 5 here: https://docs.microsoft.com/en-us/windows/apps/winui/winui3/get-started-winui3-for-desktop, after clicking OK, I get this error message:

After clicking OK in that error popup, the project loads, and I can see errors in the dependencies:

I also note that the "Windows Application Packaging Project" that is supposed to be created (according to the URL above), is not created.
Version Info
WinUI packages version: 0.5.5
Windows app type: desktop
OS: Win 10 20H2 build 19042.928
NuGet package version:
See above.
Windows app type:
Ehm, I am not sure. It says "Output type: Window Application", so I guess Win32?

UWP
Win32

?
Yes

Windows 10 version
Saw the problem?

October 2020 Update (19042)
Yes

Device form factor
Saw the problem?

Desktop
Yes

Additional context
I have used the maui-check tool, and that says that everything is OK:

I just noticed this thread, which reports the same problem:
https://github.com/microsoft/ProjectReunion/issues/740
I didnt see it at first, as I was directed to https://github.com/microsoft/microsoft-ui-xaml for any WinUI bugs found. This bug report is in https://github.com/microsoft/ProjectReunion.
Thanks.

Duplicate of microsoft/ProjectReunion#740

Workaround is to add a global.json in the solution folder:
{
"sdk": {
"version": "5.0.202"
}
}

Hello, I tried that, and yes, the errors did disappear, but it wont start anyways. Perhaps a new ticket is required, but the error when running it in debug is:

System.Reflection.TargetInvocationException: 'Exception has been thrown by the target of an invocation.'
Inner Exception:
COMException: Class not registered (0x80040154 (REGDB_E_CLASSNOTREG))

@tedekeroth you have to create the global.json file first, then create the project (which should create 2 projects). If you create the global.json file after you get that error, only the app project will have been created, not the packaging project which is the one you want to run, which is probably why you're hitting the error.

in other words: delete the projects that were created, ensure you have an empty folder, add the global.json, then create the solution in that folder which should create 2 projects in that solution.

You suggest that add the global.json to a folder, then create a solution (.sln) inside that same folder, and then on top of that add the projects? I will try that and report back!

yes, those are the steps that worked for me

Alright, that worked in the sense that the Package project was created correctly. I then added the ItemGroup in csproj. It then builds, and looks ok, even though it does not start, it fails in runtime as stated above.

yes, the package project is the one to start, can you try that?

Yes, that worked. I was very confused from this thread here, where I thought they said that it is not the package project that should be started:

As a sanity check, the app being debugged doesn't have "Package" in the name, which is the default for packaging projects (e.g. "ETIHub.TDHub.Demo.UI (Package)" instead of "ETIHub.TDHub.Demo.UI"). @s3nt1n3l-hub can you confirm the wapproj/packaging project is being run instead of the unpackaged application?

That sounded to me that Packaging shouldn't be run. Apparently, I misread of they said the wrong thing.
Thanks for your input asklar =)

@alwu-msft FYI

Today I tried to build simple application with WinUI, but it failed.

WinExe
net5.0-windows10.0.19041.0
10.0.17763.0
Agha
app.manifest
x86;x64;arm64
win10-x86;win10-x64;win10-arm64

I tried everything as suggested, but it didn't work. Maybe due to change version 0.5.5 to 0.5.6

@AghaKhan Did you try the option of adding this block in your .csproj file?
<ItemGroup>
    <FrameworkReference Update="Microsoft.Windows.SDK.NET.Ref" RuntimeFrameworkVersion="10.0.18362.16" />
    <FrameworkReference Update="Microsoft.Windows.SDK.NET.Ref" TargetingPackVersion="10.0.18362.16" />
</ItemGroup>

Hi
I was confused and put that code in App1 (Package) project.
Now I placed provided code in main App1 project and it worked.
Thanks for update.
Best regards
Agha Khan
(206) 619-2097
Sent from Mailhttps://go.microsoft.com/fwlink/?LinkId=550986 for Windows 10
From: Mike @.>
Sent: Thursday, April 29, 2021 8:25 PM
To: @.>
Cc: @.>; @.>
Subject: Re: [microsoft/microsoft-ui-xaml] WinUI in VS 2019 Preview gives error "... requires WinRT.Runtime.dll version 1.2 or greater" and more (#4894)
@AghaKhanhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FAghaKhan&data=04|01||b4ba565174674400798f08d90b87a0e2|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637553499403440931|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=G0cDg3Uj67EvP4FlyCrAjN9kcAOv4PfPd8gt79hvZxI%3D&reserved=0 Did you try the option of adding this block in your .csproj file?

<FrameworkReference Update="Microsoft.Windows.SDK.NET.Ref" RuntimeFrameworkVersion="10.0.18362.16" />

<FrameworkReference Update="Microsoft.Windows.SDK.NET.Ref" TargetingPackVersion="10.0.18362.16" />

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fmicrosoft%2Fmicrosoft-ui-xaml%2Fissues%2F4894%23issuecomment-829777862&data=04|01||b4ba565174674400798f08d90b87a0e2|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637553499403440931|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=h6wZkLpdQCo5pvs4Wxkshv1R5YnARuhyKrHrEyBU2l4%3D&reserved=0, or unsubscribehttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FABISNWYJ522XFM7BOPEO4JDTLIPLHANCNFSM43RLTIUA&data=04|01||b4ba565174674400798f08d90b87a0e2|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637553499403450886|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=RetRY3aP7epfQ%2B3VJNtB%2FIjdJW16ZImBisYTuWiywJM%3D&reserved=0.

@codendone solution worked for me, shame this WinUI3 desktop template is still broken out of the box as of today 6/5/21, not sure why this is closed!

I didn't code WinUI for some time (I tried it some time ago and worked perfectly fine) and now I wanted to take a tour and surprisingly have the same error but in other version of WinRT.Runtime.dll (this time is 1.6). Can somebody tell me what to do?

Today in response to message today I tried to build Microsoft provided code (DemoBuildCpp) which compiled ok but got run time  exception


I used to use AG with the classic fzf.nvim plugin and when I needed to do an exact match i prefixed the string to search with a single quote 'stringToSearch.
I've tried to do the same with FzfLua replacing Rg with Ag, but I couldn't figure out how.
How would I configure a command to do what i want?
thanks in advance.

You can still do exactly that, you were probably using live_grepwhich is already regex search (exact march, sort of), however, live grep does not use the fuzzy interface, use :FzfLua grep_project and all rhr project lines will be fed into fzf for searching, you can then use the exact match syntax with '.
Read the wiki to understand more, specifically the difference between live grep and grep
https://github.com/ibhagwan/fzf-lua/wiki#live-grep-glob
If you wish to use ag instead of rg you can easily change that by setting the grep.cmd opt during setup or even directly by calling :lua require'fzf-lua'.grep({ cmd = 'ag …'. }).

Wow, wonderful. That's exactly what I was searching for. Thank you very much.

Ty for the update @ovidius72, closing, feel free to reopen if you need more help.


Fix entry point

This fixes the entry point which fails with /bin/sh: 1: bash -c /epics/ioc/start.sh: not found
This change is inline with ioc-adsimdetector which already works as desired: https://github.com/epics-containers/ioc-adsimdetector/blob/3996868e07fc9d561689cdead08d29c490e221f5/Dockerfile#L72C1-L72C38

Thanks for this. I somehow missed the PR but am about to make changes and need this fix.


:sparkle: Quick attempt to improve the natural language detection

Investigation around #200
This patch popped into my mind that could be the immediate solution to further improve the solution.

Codecov Report

Merging #201 (e60b81f) into master (65b7b49) will not change coverage.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##           master     #201   +/-   ##
=======================================
  Coverage   89.88%   89.88%
=======================================
  Files          11       11
  Lines        1216     1216
=======================================
  Hits         1093     1093
  Misses        123      123

Impacted Files
Coverage Δ

charset_normalizer/cd.py
96.34% <ø> (ø)

charset_normalizer/version.py
100.00% <100.00%> (ø)

Continue to review full report at Codecov.

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

Closed, will try later.


Only remember changed inputs

Is it possible to only remember changed inputs?
My form only submits changed values, so multiple people can be updating the same resource at the same time and the only conflict can be updated fields. Which is perfect for a form with 90~ inputs.

Sisyphus now is catching any input no hidden, so when its timeout is trigger, all inputs are saved.
But if you want to try it, any Pull Request is welcome. If @simsalabim is agree.
😄


fix(ingestion): Fetch Upstreams From Columns

This PR fixes the following two things:

Tableau metadata graphql API returns empty list for upstreamTables for embedded datasources while upstreamColumns field includes information. This PR populates upstream table information from upstreamColumns field coming from the Tableau
Tableau metadata graphql API returns malfunctioned SQL queries which causing failures in fetching upstream Lineage from the CustomSQLs. This also enables embedded data sources to be connected to the CustomSQLs which are (generally) connected to upstreams from other platforms hence completing the full lineage.

Checklist

[ ] The PR conforms to DataHub's Contributing Guideline (particularly Commit Message Format)
[ ] Links to related issues (if applicable)
[ ] Tests for the changes have been added/updated (if applicable)
[ ] Docs related to the changes have been added/updated (if applicable). If a new feature has been added a Usage Guide has been added for the same.
[ ] For any breaking change/potential downtime/deprecation/big changes an entry has been made in Updating DataHub

@egemenberk some query cleaning logic is also getting adding in this PR https://github.com/datahub-project/datahub/pull/9838 - that one also removes parameter names and things to make SQL parsing work. Does it make sense to unify across these two query cleaning implementations?

@egemenberk some query cleaning logic is also getting adding in this PR #9838 - that one also removes parameter names and things to make SQL parsing work. Does it make sense to unify across these two query cleaning implementations?

Hi @hsheth2, I've taken a quick look at the PR you mentioned and it seems to fix the query, so I can remove the clean_query() method call from my implementation. My PR's main focus is to fix fetching upstream lineage from upstreamColumns when upstreamTables field is empty in Tableau response, the clean_query addition was a side fix while working on the task, so I can remove my addition on that and trust the #9838 implementation. Thanks for the information 👍


Add custom script type?

Is there a way to add custom script type with addJS method?
Use cases: webgl shaders, PEG.js parser...

You can of course add them directly to your theme. If you want to do it on a page-by-page basis, through a plugin for example, then here are a few good places to start:

https://learn.getgrav.org/cookbook/plugin-recipes#how-do-i-inject-assets-like-javascript-and-css-files

https://learn.getgrav.org/plugins/plugin-basics

https://learn.getgrav.org/api#class-gravcommonassets

Thank you.
The addJs method of \Grav\Common\Assets does not allow to set the type attribute of script ie I cannot do add the following in my plugin using Assets url resolution:
 <script id="peg" type="text/peg" src="..."></script>

I apologize for the misunderstanding. In looking over the source code, I don't see a way to do this using AddJs, but I'm no expert. It's not necessarily surprising, though. The function is explicitly for JS assets and not generic. Clearly the key is the attributes function. But I don't see a way to override that array through AddJs.
There's a js function that accepts an $attributes array, but it's not called anywhere in that class, or anywhere at all, as far as I can see.

So you need a custom type ? is the id required?

Yes adding an id parameter would be useful.

Indeed, you can't really add a custom script type afaik.
I too had to do that for my SEO plugin and found a hack: https://github.com/paulmassen/grav-plugin-seo/blob/master/seo.php#L719
i.e: I'm closing the script tag, and manually add my own.
Hope it helps

Generally all custom attributes should be handled, I've seen scripts using both id and type="application/json" for example.

I dont see the type parameter on https://learn.getgrav.org/16/api :(

The array can contain any HTML attributes, maybe it is needed to be said in the docs, @rhukster?

Does need to be added to the docs..."Assets" section needs updating for 1.6


Initial upgrade to AspnetCore 2.0

Hi guys,
really appreciate all the work you are doing with this. Since we are heavily relying on this package I wanted to give something back and start the initial migration to AspNetCore 2.0.
Most of the changes in here are only renamings and upgrading packages. Since we need to rely on .net standard 2.0 for this we will have to drop support for 452 I guess, correct me if I am wrong.
Again: really appreciate your work and I would be happy to help you out in anyway :)
Changes
Host Project

Clients: LogoutUri has been renamed to FrontChannelLogoutUri
Client:s Also no more List() method on GrantTypes
AccountController: HttpContext.Authentication is mostly obsolete replaced it with HttpContext + Method
Startup: AddTemporarySingingCredentail has been renamed to AddDeveloperSigningCredential
Host.csproj: upgrade to netcore app 2.0 and make use of aspnetcore.all

EntityFramework:

Client: new names for logoutproperties.

Client: replaced most int with Enum (models)

ResourceStore: GetAllResources -> GetAllResourcesAsync

csproj: migrate to netstandard 2.0

csproj: reference preview for identityserver and 2.0.0 for aspnetcore libs

csporj: upgraded automapper to 6.1.1

IntegrationTests:

naming fixes and upgrade to .netcoreapp2.0

Unittests:

ScopeMappersTests: ApiResources require name as parameter
upgrade to .netcoreapp2.0

Cheers

I working at the same migration, but dont know if is right  I'm waiting a reply here #https://github.com/IdentityServer/IdentityServer4/issues/1425

Clients: LogoutUri has been renamed to FrontChannelLogoutUri

Don´t know which PR we should close but since #92 is nearly the same (only missing some smaller changes from original host repo and some differences to my PR), but I think we should close one of these PRs and proceed only with one.

Thanks for the PR. This has been done.


Support dy/dx

https://docs.dydx.exchange/#general

Currently, they only support 1day and 1hour granularity.


Support Carthage ?

Any plans to create a Carthage for this?

+1

+1

+1

+100500

why not support carthage?


relative paths in kube config file are not interpreted correctly

This was reported by a txkube user: https://github.com/LeastAuthority/txkube/issues/119.  txkube uses pykube's config loader.
From the original report:
$ cat ~/.kube/config
apiVersion: v1
clusters:
- cluster:
    certificate-authority: dev-ca.crt
...

  File "/usr/local/lib/python2.7/dist-packages/pykube/config.py", line 142, in clusters
    BytesOrFile.maybe_set(c, "certificate-authority")
  File "/usr/local/lib/python2.7/dist-packages/pykube/config.py", line 225, in maybe_set
    d[file_key] = cls(filename=d[file_key])
  File "/usr/local/lib/python2.7/dist-packages/pykube/config.py", line 241, in __init__
    raise exceptions.PyKubeError("'{}' file does not exist".format(filename))
pykube.exceptions.PyKubeError: 'dev-ca.crt' file does not exist```

the ca file resides in ~/.kube/dev-ca.crt. The same problem reoccurs with users[].user.client-certificate and .client-key. Putting absolute paths into ~/.kube/config appears to avoid the issue.

Anybody?

My apologies on the long response time.
I definitely want to see this supported. I am going to look up the expected behavior of kubectl here. Do you know if what you are proposing (relative paths are relative to the kube config) the behavior of kubectl?

Thanks for the reply.
I'm not sure if this is the behavior of kubectl but the upstream bug report seems to imply it is.  The value is taken from someone's existing ~/.kube/config and they were expecting it to work.  I suspect that means they're successfully using it with kubectl but I don't know for sure.

I have confirmed that relative paths in kubeconfig should be resolved relative to the location of the file. I will get a fix up in the next couple of days.

Any chance you've had time or will have time soon for this?  Thanks. :)


UI Ingestion Polish improvements

[ ] Connection options and connection arguments should appear in a hidden pannel "Advanced Configuration", not there by default.
[ ] The scheme should also be part of this Advanced Configuration

cc @chirag-madlani

[ ] Screen flickers after selecting a field the first time we arrive at the connection details screen

https://user-images.githubusercontent.com/59080942/228779358-6ca20057-df4b-419d-ac98-457721ba1322.mov

[ ] size changes on scroll for service configure page

https://user-images.githubusercontent.com/39544459/228805432-06c0a0d0-f013-4aa9-8387-33f7f76cee46.mov

Based on our recent discussion, some updates on the documentation section of the connectors:

The UI should clearly show which connector is being created. We lose the information if we are creating a Redshift, Athena, BigQuery,... after the first step
Instead of having the Requirements as a second step, have a right side panel with all the necessary info when adding the Connection Details
We need to improve on the markdown styling
All the forms should follow the format of:

Small description
Example values

If we click to any field in the connection details, the doc should scroll automatically to that point

[ ] Add md files for workflows as well. @open-metadata/ingestion

cc @mohityadav766 we should improve the docs here https://github.com/open-metadata/OpenMetadata/blob/main/openmetadata-ui/src/main/resources/ui/public/locales/en-US/OpenMetadata/EmailConfiguration.md


Cannot have Redux hooks in functional component that is expandColumnRenderer

If I define my expandColumnRenderer (for row expansion) to be:
import { useSelector } from 'react-redux'

const ExpandColumn = ({expanded, expandable}) => {
  const isEditing = useSelector((state:RootState)=>state.isEditing)
  return isEditing ? <FontAwesomeIcon icon={faPlusSquare}/> : <FontAwesomeIcon icon={faCopy}/>
}

this crashes my application with
Error: Invalid hook call. Hooks can only be called inside of the body of a function component. This could happen for one of the following reasons:
1. You might have mismatching versions of React and the renderer (such as React DOM)
2. You might be breaking the Rules of Hooks
3. You might have more than one copy of React in the same app
See https://fb.me/react-invalid-hook-call for tips about how to debug and fix this problem.

The line highlighted in the code indicating where the error occurred is the const isEditing = useSelector line. If I comment it out and replace with const isEditing = true then the app does not crash and the application the cell as expected.
I've seen other people in the past had an issue with useState, another hook, but the solution didn't seem applicable.
Note: This is an oversimplification of what I'm actually doing. The actual use case is in read-only mode for data presentation, the expand/contract col is just an SVG + or - depending on expanded state.
In edit mode, I actually present the same SVG, but also three other icons representing "edit" "delete" and "clone" (these will dispatch to a Redux store actions to edit the row's data (brings up a modal window), delete the row (removes the data completely from the store), and clone the data (creates a deep copy of the data in the redux store).
Just before you go wondering why in my example above in RO mode I have + but in edit mode I have a copy icon. :)
This is a port of legacy code from Angular to React, so I need the UI to behave as similarly to the original as possible. My expansion cell really needs to be able to get state from redux store, and it needs to dispatch actions to the redux store.

Provisional solution: rewrite functional component that uses redux hooks like useSelector and useDispatch as class components instead and use the redux connect to avoid having to use any redux hooks directly.

Operating theory: react version used by this library predates the introduction of hooks. Looks like it's v16.4 to me, and hooks came in 16.8.


perf(allocator): use lower bound of size hint when creating Vecs from an iterator

#6195
#6194  👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @DonIsaac and the rest of your teammates on  Graphite

#6195
#6194  👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @DonIsaac and the rest of your teammates on  Graphite


Create ENV parameters for users

seems like "hp" is hardcoded - would be nice to have dynamic users:
https://github.com/function61/holepunch-server/blob/master/pkg/holepunchsshserver/sshd.go#L55

By "dynamic users" do you mean multiple users, or just a single dynamic username?
I'm not sure multiple users are required, but I can understand wanting to change the username. I added support for that: https://github.com/function61/holepunch-server/commit/deaa44f9005500563e18974c3977b09354279abb


How to create get Font family name from font name in a generic way to create TypeFace

Hi,
I have a set of input font names, i need to create Typeface, but type face only takes font names not font family name, so i need to give the correct font family name other wise Skia sharp is returning default font.
Input font names
FrutigerNext Com UltraLight,
FrutigerNext Com UltraLightCn,
FrutigerNext LT Regular,
FrutigerNext LT Heavy,
FrutigerNext LT RegularCn,
FrutigerNext LT MediumCn,
FrutigerNext LT BoldCn,
FrutigerNext LT BlackIt,
FrutigerNext LT MediumIt,
FrutigerNext LT BoldIt,
FrutigerNext LT It,
so my question here is how can i get the correct font family name correctly for different fonts ?.
I have also referred https://github.com/mono/SkiaSharp/issues/762 discussion, but it didn't help.
it would be really appreciable if any one could help me out.
Regards,
Jackson Paul T

Hi @Gillibald ,
I cannot use load by stream in my case, where i get input only as font name, using that i need to create a Type face, which should be generic for any kind of font, so how can i achieve it using SKTypeface.FromFamilyName() API, is there any better way to handle generic only using font name?
Regards,
Jackson Paul T

SKTypeface.FromFamilyName() uses the font manager to resolve fonts. So the moment you install your fonts on the system FromFamilyName should work


time-frequency beamformer on single trial

Hi all,
I am trying to do MVPA decoding from time-frequency transform epoch data (power) in source space.
The first thing I tried is, schematically:

raw = mne.Epochs() #  Create epochs
stcs = apply_lcmv_epochs(epochs, filters)  # Source transform at the single trial level
for jj, stc in enumerate(stcs):  # Rearange the data
         X_data[jj] = stc.data
X = tfr_array_morlet(X_data, output=power)  # Compute single trial time-frequency in source
clf = SlidingEstimator(make_pipeline(StandardScaler(), LinearModel(Ridge()))) #  Create regressor
clf.fit(X[train], y[train]) and clf.score(X[test], y[test]) + get_coef('patterns_')  # Run regressor and get the pattern at each cv iteration

For now, it is crashing because it exceeds memory limit on my server (the X array in line 4 is big)
It doesn't seem to be the most efficient way to do that. Do you have any advices?
I was wondering if it exists an equivalent of mne.beamformer.tf_lcmv at the single trial level?
Thank you for your help!
Romain

what are you aiming to show? estimate? Maybe there is a simpler route.

@romquentin Do you use decim when using time-frequency? You could also do the analysis in parts if the full approach eats too much of your memory (chunk time-frequency space and process each chunk independently and then glue the results).


Auto-Hide/Expand AutoRepeater's Settings Pane

As per issue, if there is an easy way, it would be nice to auto-hide the settings pane as it takes up too much space on screen and if more info is added in the result columns, it will be even more.
It could auto-hide, and expand again only when tester moves their cursor over the right side area.

Hey @ngkogkos,
I'm unsure how to automatically show/hide the settings pane, however, I'm going to add the pane's visibility as a toggleable option in the right click menu. Precious screen real estate is definitely something I'm interested in maximizing and this is a feature I'm going to implement ASAP.
Thanks,
Justin

Hey @ngkogkos,
Adding this as a right-click option didn't quite work out, however, I added a MenuItem to Burp Suite's JMenuBar that now has the option to show/hide the options panel.
The new JAR in the git repo has this feature. I'm going to close this issue now, if there's any issue feel free to reopen the issue.
Thanks,
Justin


[NEO3-neo-cli] latest neo-cli needs to be compatible with neo for rpcserver change

Due to rpcserver change: https://github.com/neo-project/neo/pull/1340, latest neo-cli(https://github.com/neo-project/neo-node/pull/502) is incompatible with neo(#1340)
neo-cli needs to update to be consistent with neo.

Updated.


⚠ RPC server - Figment Solana has degraded performance

In 6078075, RPC server - Figment Solana ($FIGMENT_SOLANA_RPC_URL) experienced degraded performance:

HTTP code: 200
Response time: 382 ms

Resolved: RPC server - Figment Solana performance has improved in 237506d.


Models 2

Coverage remained the same at 100.0% when pulling 0a5ae25d7263161a47a4de46889224ffb30c3e50 on models-2 into 622162df84eabbdd101c7f4be5f9bec8e32dea19 on master.

Coverage remained the same at 100.0% when pulling 0a5ae25d7263161a47a4de46889224ffb30c3e50 on models-2 into 622162df84eabbdd101c7f4be5f9bec8e32dea19 on master.

Coverage remained the same at 100.0% when pulling c52315e79217dd73c9d8ea06a11508ae4eb8b5bd on models-2 into 622162df84eabbdd101c7f4be5f9bec8e32dea19 on master.


Bump github.com/vbatts/tar-split

Update github.com/vbatts/tar-split to v0.10.2 and pin that version instead of master, to pick up https://github.com/vbatts/tar-split/pull/42

LGTM

@rh-atomic-bot r+


gofmt -s files

Signed-off-by: Davor Kapsa<EMAIL_ADDRESS>

This is nice, but until we change Makefile format command which is run by default, this might be violated in next PR....

As a corollary, we should also enforce it via golangci-lint, adding the gofmt linter to .golangci.yml. What's your take @bwplotka ?

Definitely!
On Mon, 20 Apr 2020 at 14:37, Marco Pracucci<EMAIL_ADDRESS>wrote:

This is nice, but until we change Makefile format
https://github.com/thanos-io/thanos/blob/58458f1b0a652b6ba7f98cd72dbe9272af9c7982/Makefile#L198
command which is run by default, this might be violated in next PR....
As a corollary, we should also enforce it via golangci-lint, adding the
gofmt linter to .golangci.yml. What's your take @bwplotka
https://github.com/bwplotka ?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/thanos-io/thanos/pull/2463#issuecomment-616558537,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABVA3O32IAM76LJMBECDBFTRNRFZRANCNFSM4MLLHLDA
.

Can you help with this @dvrkps ?

Sure, i will try.