LiteDB In Memory Database

Hi, recently I was trying to use this framework in some of my applications and came up with a question about lite db database provider and context classes.
I know that LiteDB supports the usage of streams to run as a in-mem database. But while attempting to setup this using this framework I noticed the options class for lite db repository only has a single parameter DatabaseName available to specify the path for .db file to use.
Also checked that DatabaseProvider uses this to open or creating an empty database on disk, i thought that overriding the logic on DatabaseProvider would solve this but the class is sealed. So how I'm supposed to do enable usage of memory streams w/ LiteDb?

Hi, I've never thought of this use case before. It sounds interesting and I will have a look at it tomorrow. Maybe it's not that hard to extend the configuration, to run LiteDB in-memory.

Hi, I've added a very simple LiteDB in-memory configuration. See this Pull Request. I will add the following features in the coming days:

Load a database from file, but use the content in-memory.
Use an in-memory database and allow writing of the file to disk.

While checking some parts of LiteDB source code related to database instantiation, I've noticed that they already have some string patterns on connection string (:temp: and memory) to internally handle in-memory and temp databases when LiteEngine is created from within LiteDatabase:
        // source: LiteDB/Engine/EngineSettings.cs
	internal IStreamFactory CreateDataFactory(bool useAesStream = true)
	{
		if (DataStream != null)
		{
			return new StreamFactory(DataStream, Password);
		}
		if (Filename == ":memory:")
		{
			return new StreamFactory(new MemoryStream(), Password);
		}
		if (Filename == ":temp:")
		{
			return new StreamFactory(new TempStream(null, 10485760L), Password);
		}
		if (!string.IsNullOrEmpty(Filename))
		{
			return new FileStreamFactory(Filename, Password, ReadOnly, hidden: false, useAesStream);
		}
		throw new ArgumentException("EngineSettings must have Filename or DataStream as data source");
	}

Correct me if I'm wrong but maybe the PR changes could be far more simple like just removing the EnsureEndsWith on DatabaseProvider to enable passing those patterns to db constructors, letting then do the rest :raised_eyebrow:
But it still would be nice to have the option to pass a stream directly to support cases like .db files embedded as resources for example (As this usually happens on desktop apps when deploying apps with embedded databases that not necessarly are ment to be copied to end user machine).

Thanks for the research. I am planning to support both approaches. I hope I have something to look at by the end of this week.


Using the PolicyRecordingBot outside of Azure

For the PolicyRecordingBot, is it possible to run it outside of Azure?
I'm not referring to the DNS side, but to the VM deployment - currently we use it in Azure Cloud Services.
I looked at https://github.com/microsoftgraph/microsoft-graph-comms-samples/issues/143, and there it seems the problem is the Microsoft.Skype.Bots.Media SDK.
That issue was in January 2020.  Is this still the case?
Thank you in advance.

There's no change: running the Bot Media SDK outside of Azure is not supported.
In general, it's also not recommended: assuming the bot is joined into a Teams meeting, the bot's peer is an endpoint running in Azure, and it's best to keep the real-time media session between the bot and the Teams meeting within the Azure network.

Thank you.


Adds a test about filestore

Ticket
Redmine ticket:
https://redmine.openinfosecfoundation.org/issues/6388
Requires https://github.com/OISF/suricata/pull/9586

Replaced by https://github.com/OISF/suricata-verify/pull/1416


How to train and fine-tuning model

Hello, may I ask how to train a fine-tuning model, such as Lora fine-tuning. Which chapter has an introduction。
Thanks

Hi @Jackjiayou ,
Can you please clarify your question? Are you asking about instruction fine-tuning, or general fine-tuning of the pretrained DCLM checkpoints? If the latter is correct, you can load it as any HuggingFace model and fine-tune through the HF recipes.


Events in website's MetricsBar

Currently the MetricsBar component shows views, visitors, bounce rate and average visit time.

Would it be possible to add Events as well? This is key for websites to measure engagement, and is currently only available in the real-time view. If it is interesting, I can try to submit a PR.

The total number of events is not very valuable without the context of what the event is. That's why you need to open up the details page.

Yes, I meant adding the Events counter to the details page. Would it be interesting that I submit a PR?

I still think events should remain separate from the other stats. We will add enhanced events functionality in the future.


the implementation of ConvFPN

Thanks for the code of SiamTPN, but the implementation of ConvFPN just use some conv blocks on the "feat[0]" and hope this issue could be fixed.

Please check the configuration file as well as the implementation in FPN model,  the 2, 3, 4 layers from shufflenet are used for pyramid feature fusion.


Export ProfileAndToken interface

Description:
For the sso option as you have other exported members you need to add the export for type ProfileAndToken
export * from './authorization-url-options.interface';
export * from './connection-type.enum';
export * from './connection.interface';
export * from './get-profile-and-token-options.interface';
export * from './profile.interface';

Needs to add only this line:
export * from './profile-and-token.interface';
Thnx

Hi @edyrkaj!
ProfileAndToken is now exported in v1.4.0.


Could you add more support for Markdown?

Like styles in the one-dark-theme, there are highlights for inline code.
.markup.raw {
    color: #98c379;
}

but in your css, i saw
.markup.raw.inline {
    color: #f1e655;
}

which doesn't apply any style on the inline code.
Maybe it's outdated because of the update of Atom?

I would also love this. Have been doing a lot of .md documentation lately and the Material Syntax (Original, Dark, Light) do not support Markdown nearly as nicely as Atom's default syntax themes.


Duplicate terms in different taxonomies block each other from rendering

Expected behavior
I have the term Cats in 2 different taxonomies. When I use new TimberTerm($term_slug, $taxonomy), I expect the the constructor to return an object of the term in the taxonomy I specify (like using the WP function get_term_by('slug', $term_slug, $taxonomy_singular)).
object(WP_Term)#492 (12) {   ["term_id"]=>   int(73)   ["name"]=>   string(4) "Cats"   ["slug"]=>   string(4) "cats"   ["term_group"]=>   int(0)   ["term_taxonomy_id"]=>   int(73)   ["taxonomy"]=>   string(5) "issue"   ["description"]=>   string(10) "Cats Issue"   ["parent"]=>   int(0)   ["count"]=>   int(0)   ["filter"]=>   string(3) "raw"   ["ID"]=>   int(73)   ["id"]=>   int(73) }
Actual behavior
The following object is returned:
object(Timber\Term)#491 (8) {   ["PostClass"]=>   string(11) "Timber\Post"   ["TermClass"]=>   string(4) "Term"   ["object_type"]=>   string(4) "term"   ["_children"]=>   NULL   ["name"]=>   NULL   ["taxonomy"]=>   string(5) "issue"   ["id"]=>   NULL   ["ID"]=>   NULL }
Steps to reproduce behavior
See functions in first section.
What version of WordPress, PHP and Timber are you using?
Wordpress 4.6.1
PHP 5.5.9-1ubuntu4.14
Timber 1.1.6
How did you install Timber? (for example, from GitHub, Composer/Packagist, WP.org?)
Composer

Is this still something you're actively working on or need addressed? Please respond in two weeks or we'll close this thread


Fix auto expansion of custom data

I pushed this fix in manually, as we did some big refactoring which would have broken the merge! Thanks! Will cut a release tomorrow.

This hasn't fixed the issue for me, it still gives me the Object with only an href in it.

This release hasn't been cut yet, sorry! Adding in testing around it.

Thanks, will update when the release hits!

Is this in 2.0.9? It still doesn't seem to work for me.

This should be fixed in 2.0.9, can you give me more context for your problem?  If you could share your code with me that would be really helpful.  If you don't want to share here you can contact us privately via<EMAIL_ADDRESS>

Yep, my app.js is as follows:
https://gist.github.com/bassie1995/9feafbc4fb72166c00f8.
Tried to use just expandCustomData: true as well.
I'll be glad to provide more info.

This is working for me now. I think you need:
expand: {
  customData: true
}

not expandCustomData: true.
Also, you probably want to reset your API keys since they're completely exposed in that gist...

Huh, that did fix it. I have tried that (or something close, I guess) before, but I'm glad it works now.
I'll be resetting the keys once we go live, as well :)


Replaced token.bufferDelta with token.value.length

Fixes #26 for Atom 1.9.0 and greater; however, breaks compatibility with older versions of Atom. It may be desirable to test for bufferDelta vs value.length (not sure if the latter was/is available in earlier versions of Atom).
Here's the Atom commit that dropped bufferDelta: https://github.com/atom/atom/commit/544b75c7b0829be2ee6ff5caeffc97ae54d12c2e#diff-9113d52e8bbcf6fa618b1db8d18ea3f9

I'd much prefer to simply bring in atom-linter as a dependency and replace the entire extractRange() with the rangeFromLineNumber() function it provides.
If you provide the optional 3rd argument of column, it does what this is trying to do, but without relying on non-public parts of the Atom core.

@Arcanemagus Roger that, I just intended this as a quick fix to get it working on Atom 1.9+. I personally don't know enough about atom-linter to fix the pull request though – which parts of extractRange would you replace? You'd still have to find the line of interest, right?

So after a re-review and a much deeper understanding of what this code is doing prompted by @TonyFNZ over in #28, this PR is actually completely valid as there isn't really a better way to get a position out of the data that SwaggerParser is giving back to us.
Sorry for not looking at this in more detail before, and thanks for putting this together!

Published in v0.3.7. 🎉


Product feedback URL - wrong link

Clicking 'Product feedback' takes to the generic Azure feedback uservoice rather than the Azure Networking uservoice. The Azure Networking uservoice has a category for 'Azure Firewall' whereas the other doesn't. I feel it's important to make this change to ensure feedback can be captured in the correct category. Better link: https://feedback.azure.com/forums/217313-networking

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

ID: 3d4e1aef-bb70-e95a-66fa-b8e25bc9fcbb
Version Independent ID: d223a464-6a66-4755-2eea-40148ab3cf29
Content: FQDN tags overview for Azure Firewall
Content Source: articles/firewall/fqdn-tags.md
Service: firewall
GitHub Login: @vhorne
Microsoft Alias: victorh

@CHDAFNI-MSFT Thanks for the feedback! We are currently investigating and will update you shortly.

@CHDAFNI-MSFT Thanks for the feedback! I have assigned the issue to the content author to evaluate and update as appropriate.

@CHDAFNI-MSFT
Added firewall to docfx.json file.

#please-close

@CHDAFNI-MSFT We will now proceed to close this thread. If there are further questions regarding this matter, please tag me in your reply. We will gladly continue the discussion and we will reopen the issue.


Make url_to_fs public

url_to_fs is used in pandas, and geopandas would like to use it too but it isn't documented in the public API. Any objections to adding it to the public API?
Is fsspec.core.* considered public and it just needs to be documented?

Originally, I think only identifiers exported to the package root would be considered public, but I think that has shifted over time. So yes to documenting things in core, or it could be imported in init, or both.


Patch for /Makefile failing PLATFORM=IOS with XCode 4.6

Fix failure to make for iOS due to the compilers not being in the directory
under the iOS platforms for XCode 4.6
"
make:
/Applications/Xcode.app/Contents/Developer/Platforms/iPhoneOS.platform/Developer
/usr/bin/c++: No such file or directory
make: *** [db/builder.o] Error 1
"

Original issue reported on code.google.com by<EMAIL_ADDRESS>on 11 Jun 2013 at 6:08
Attachments:

Makefile.patch

Verified works with xcode 4.6.3

Original comment by<EMAIL_ADDRESS>on 1 Jul 2013 at 6:59

Verified that the patch fixes the build on Xcode 4.6.2 when running "make
PLATFORM=IOS". Thanks!

Original comment by<EMAIL_ADDRESS>on 13 Jun 2013 at 6:47

Works for me, please get this merged. I would also propose to replace the
hardcoded Xcode path by a call to `xcode-select --print-path`. See the attached
pathfile. This fixes issues when you're using multiple versions of Xcode (e.g.
with developer previews).

Original comment by<EMAIL_ADDRESS>on 24 Jun 2013 at 12:53
Attachments:

Makefile.patch

Can someone who also develops for iOS test this patch to ensure it works on
their system? Bonus points if you're using a different version of xcode.

Original comment by<EMAIL_ADDRESS>on 13 Jun 2013 at 4:45

This patch just happens to work for you because your CXX points to the same
compiler that lives in the default xctoolchain. If you install a different
compiler in your PATH, this patch will stop working.

Attached is a proper patch uses the tools in the default xctoolchain and—as
suggested by luke—reads the xcode root path from xcode-select.

$ make PLATFORM=IOS && lipo -info libleveldb.a
...
Architectures in the fat file: libleveldb.a are: armv6 armv7 i386

Original comment by<EMAIL_ADDRESS>on 27 Jul 2013 at 10:35
Attachments:

0001-iOS-makefile.patch

I don't see how that is any better as it apparently still has assumptions about
your XCode being in /Applications/Xcode.app?

I think a separate bug needs logging about fixing builds for both OS/X and iOS
to work if you have an XCode 5 developer preview installed alongside, and are
using Xcode-select

Original comment by<EMAIL_ADDRESS>on 28 Jul 2013 at 5:30


ConsumerGroup crazy very slow, if pass topic as string

Bug Report
Environment

Node version: 9.6.1
Kafka-node version: 2.4.1
Kafka version: 0.9

Include Sample Code to reproduce behavior
// All ok
var consumerGroup = new ConsumerGroup(options, ['RebalanceTopic');

// Crazy very slow
var consumerGroup = new ConsumerGroup(options, 'RebalanceTopic');

There is something else at play and not what you think it is.
lodash's isString is not going to be slow, nor would it have a long running impact on the overall runtime.
This code is only ran during consumer creation.

Sorry, I mean receiving messages process.
var consumerGroup = new ConsumerGroup(options, ['RebalanceTopic']);
consumerGroup.on("message", m => {
  // this call so fast
});

var consumerGroup = new ConsumerGroup(options, 'RebalanceTopic');
consumerGroup.on("message", m => {
  // this almost not call, very poor performance
})

I met this. Thanks @darky

The string gets turned into an array that's the only difference between the two. https://github.com/SOHU-Co/kafka-node/blob/master/lib/consumerGroup.js#L95
Do you have some sample code to demonstrate this issue?

I don't understand, how it occured, but code smell bad little bit.
No need to mutate parameters, otherwise need directly assign:
this.topics = _.isString(topics) ? [topics] : topics;

The only way this code can have a problem is if you found a performance bug in v8, which I doubt that such a simple operation of wrapping a string in an array would be an issue. It would be chaos on the internet if there was since this is an extremely common pattern.

Your issue has nothing to do with string vs array. You're going to need to give hyperlink more code / details / debug logs.


AcmAutoRefreshSelect error when window is not set

I am trying to use the AcmAutoRefreshSelect on the common page header
using @open-cluster-management/ui-components": "^0.1.184
I want to change the current implementation, and make use of the AcmPageHeader
So I want to change this line
https://github.com/open-cluster-management/application-ui/blob/master/src-web/containers/App.js#L126
and replace with
      <div className="expand-vertically">
        <AcmPageHeader {...headerArgs} />
        <Switch>
          <Route
...

    const headerArgs = {
      title: 'Applications',
      titleTooltip: 'Doc link',
      controls: (
        <AcmAutoRefreshSelect
          refetch={refetch}
          refreshIntervals={[15, 30, 60, 5 * 60, 30 * 60, 0]}
        />
      )
    }

const refetch = () => {
...
}

The window object is not set initially and I get this exception on the AcmAutoRefreshSelect
ReferenceError: window is not defined
at /Users/vbirsan/Documents/data/git/master/application-ui/node_modules/@open-cluster-management/ui-components/src/AcmAutoRefreshSelect/AcmAutoRefreshSelect.tsx:61:16
at useReducer (/Users/vbirsan/Documents/data/git/master/application-ui/node_modules/react-dom/cjs/react-dom-server.node.development.js:1194:57)
https://github.com/open-cluster-management/ui-components/blob/master/src/AcmAutoRefreshSelect/AcmAutoRefreshSelect.tsx#L61

@RobDolares I am tagging you since you seem to be the initial creator ..is it possible I am not using the component the way is intended to ?
fyi @chenz4027

@birsanv Are you doing server-side rendering?

@jlpadilla I believe there is an old server-side rendering setup in our code that isn't really effective and should probably just be removed.

@KevinFCormier thanks then I'll hold off to the app ui refresh action change until we move the header rendering
We can close this issue unless we want to update the code that validates the window existence
This line seems to verify if the window exists
https://github.com/open-cluster-management/ui-components/blob/master/src/AcmAutoRefreshSelect/AcmAutoRefreshSelect.tsx#L61

} else if (window && window.localStorage && window.localStorage.getItem(key)) {

but the code is going to fail here if no window
https://github.com/open-cluster-management/ui-components/blob/master/src/AcmAutoRefreshSelect/AcmAutoRefreshSelect.tsx#L66

window.localStorage.setItem(key, ${DEFAULT_REFRESH_TIME * 1000})

closing, the refresh action is updated on a separate task https://github.com/open-cluster-management/backlog/issues/10776


Support for ARMv5

Does the library support armv5 architecture?

which hardware/compiler are you using?
if your compiler can generate ARMv5 code then it will most likely work just fine.

I am using gcc-4.8 compiler from android-ndk-r8e. And building Mobile phone with android-2.1

And is there a Makefile configuration for cross compiling?

you are using a very old Android NDK I must say :)
the latest Android NDK is r14
for cross-compiling to Android please have a look here:
https://github.com/alfredh/baresip-android

I have updated the Android NDK. And I am able to use it with Android. I would like to use it with Linux running on ARMv5. Is there a Makefile configuration for cross compiling?

can you please clarify if this question is about Android or Linux/ARMv5 ?
you can cross compile libre/baresip by using the CC option:
$ make CC=arm-linux-gcc

(which Hardware and Toolchain are you using for the Linux/ARMv5) ?
/Alfred

I was able to compile it successfully. Thank you :+1:
Note:
I was trying with NDK to build for Linux on ARMv5 and later I used the CROSS_COMPILER to compile it directly without NDK.


Error: THERE IS NO DARK MODE IN SMALL SCREENS OR MOBILE SCREEN

https://github.com/user-attachments/assets/951723e6-1100-453a-a811-aa0da613970e

I will be happy to resolve this issue!


Youtube Bookmarker Chrome extension

Is your feature request related to a problem? Please describe.
Often, users want to revisit specific parts of YouTube videos without having to manually search for the timestamp each time. This can be inconvenient and time-consuming, especially for longer videos or those with multiple relevant sections.
Describe the solution you'd like
This feature should enable users to save multiple timestamps from different videos, view their bookmarks, play videos from bookmarked points, and delete bookmarks as needed.
Describe alternatives you've considered
One alternative could be relying on third-party browser extensions or bookmarking tools that offer similar functionality
Additional context
By implementing this feature, users can easily revisit their favorite parts of videos or important sections for reference without having to manually search for timestamps each time. This would enhance the overall user experience on the platform and improve engagement with video content.
What problem is this feature trying to solve?
How do we know when the feature is complete?

hey @Sulagna-Dutta-Roy , please assign this issue to me.

Hey @Sulagna-Dutta-Roy , I've submitted a PR. Could you please take a look and provide any feedback you have? Thanks!


add replace method to specify alternate instance

This PR adds a replace method to allow specifying an alternate instance to be retrieved from the dependency injector. This replicates functionality present in other frameworks (eg. Angular)
class ParentClass {}

class ChildClass extends ParentClass {
  bool isChild = true;
}

Get.put(ParentClass());

Get.replace<ParentClass, ChildClass>(ChildClass());

final instance = Get.find<ParentClass>();
print(instance is ChildClass); //true

Pre-launch Checklist

[x] I updated/added relevant documentation (doc comments with ///).
[x] I added new tests to check the change I am making or feature I am adding, or @jonataslaw said the PR is test-exempt.
[x] All existing and new tests are passing.

Great feature.
Just one thing, wouldn't it be a better api, just add Generics and instance?
Example:
Get.put<IStorage>(IOStorage())
Get.replace<IStorage>(WebStorage())
Get.lazyReplace<IStorage>(()=> WebStorage())

Great feature.
Just one thing, wouldn't it be a better api, just add Generics and instance?
Example:

Get.put<IStorage>(IOStorage())
Get.replace<IStorage>(WebStorage())
Get.lazyReplace<IStorage>(()=> WebStorage())

Awesome suggestion! I will make the change.

Done!

You can now do:
abstract class BaseClass {}
class ParentClass extends BaseClass {}

class ChildClass extends ParentClass {
  bool isChild = true;
}

Get.put<BaseClass>(ParentClass());

Get.replace<BaseClass>(ChildClass());

final instance = Get.find<BaseClass>();
print(instance is ChildClass); //true

class OtherClass extends BaseClass {}
Get.lazyReplace<BaseClass>(() => OtherClass());

final instance = Get.find<BaseClass>();
print(instance is ChildClass); // false
print(instance is OtherClass); //true

Very very nice!
Thanks, LGTM


update reviewer

What type of PR is this?
/kind chore
What this PR does / why we need it:
Which issue(s) this PR fixes:

Fixes #
Special notes for reviewers:

Please check the following list before waiting reviewers:

[ ] Already committed the CRD files to the Helm Chart if you created some new CRDs
[ ] Already added the permission for the new API
[ ] Already added the RBAC markers for the new controllers

Does this PR introduce a user-facing change??

None

/lgtm
/approve

/cherrypick release-3.3

/cherrypick release-3.3

/cherrypick release-3.3


getMetadata returns invalid data for aes 256 key

Problem:
The key that is generated in the Dockerfile with the pkcs11-tool command gets returned by the xks-proxy
{
  "keySpec": "RSA_0",
  "keyUsage": [],
  "keyStatus": "ENABLED"
}

instead of
{
  "keySpec": "AES_256",
  "keyUsage": [
    "ENCRYPT",
    "DECRYPT",
    "SIGN",
    "VERIFY",
    "WRAP",
    "UNWRAP"
  ],
  "keyStatus": "ENABLED"
}

I attempted to use rust-gdb to figure out what was going on, however, when using a version compiled with debuginfo (drop the --release flag, and use target/debug/xks-proxy) the correct values are returned

I just created a PR to fix this. Tagging rust version to 1.75.0 seems to fix this issue. https://github.com/aws-samples/aws-kms-xks-proxy/pull/54


SOLR-15405: Make HDFS Directory handle little endian Lucene LUCENE-9047

https://issues.apache.org/jira/browse/SOLR-15405
Description
Uses Lucene ByteArrayDataInput as inspiration.
https://github.com/apache/lucene/blob/main/lucene/core/src/java/org/apache/lucene/store/ByteArrayDataInput.java
Solution
CustomBufferedIndexInput needs to ensure that
it handles LUCNE-9047 changes for little
endianess. This replaces the existing methods
with little endian implementations.
Tests
Working through running all the HDFS tests.
Checklist
Please review the following and check all that apply:

[X] I have reviewed the guidelines for How to Contribute and my code conforms to the standards described there to the best of my ability.
[X] I have given Solr maintainers access to contribute to my PR branch. (optional but recommended)
[X] I have developed this patch against the main branch.
[ ] I have run ./gradlew check.

I'm still figuring out some of the gradle build process for testing so will need to do more checks before this is fully ready.

@dsmiley copied almost exactly from:
Uses Lucene ByteArrayDataInput and ByteBuffersDataInput as inspiration.

https://github.com/apache/lucene/blob/releases/lucene%2F9.0.0/lucene/core/src/java/org/apache/lucene/store/ByteArrayDataInput.java
https://github.com/apache/lucene/blob/releases/lucene%2F9.0.0/lucene/core/src/java/org/apache/lucene/store/ByteBuffersDataInput.java#L212

minor changes to use the byte array that existed previously. The diff from https://github.com/apache/lucene/commit/b84e0c272b6e1beb29e2a22d16d1d8a2bd05a731#diff-81cd76910207cb8480920e2a4d746617f8c31c320be57542f753cab3cb6f8c3f matches almost exactly the previous code so it wasn't a bad replacement at all.

I've run
./gradlew test --tests "*hdfs*" --tests "*Hdfs*" --tests "*HDFS*" --tests "*Hadoop*" -Ptests.jvms=4 -Ptests.jvmargs=-XX:TieredStopAtLevel=1 -Ptests.nightly=true -Ptests.awaitsfix=false -Ptests.badapples=false -Ptests.file.encoding=UTF-8

a bunch (10?) of times and haven't gotten many failures. The failures have been varied with the current set of failures below from a few different runs:
  - org.apache.solr.cloud.MoveReplicaHDFSTest.testFailedMove (:solr:core)
    Test output: /Users/risdenk/repos/apache/solr/solr/core/build/test-results/test/outputs/OUTPUT-org.apache.solr.cloud.MoveReplicaHDFSTest.txt
    Reproduce with: gradlew :solr:core:test --tests "org.apache.solr.cloud.MoveReplicaHDFSTest.testFailedMove" -Ptests.jvms=4 -Ptests.jvmargs=-XX:TieredStopAtLevel=1 -Ptests.seed=5BCDC1F5C3A460AF -Ptests.nightly=true -Ptests.awaitsfix=true -Ptests.badapples=false -Ptests.file.encoding=UTF-8

  - org.apache.solr.security.hadoop.TestImpersonationWithHadoopAuth.testForwarding (:solr:core)
    Test output: /Users/risdenk/repos/apache/solr/solr/core/build/test-results/test/outputs/OUTPUT-org.apache.solr.security.hadoop.TestImpersonationWithHadoopAuth.txt
    Reproduce with: gradlew :solr:core:test --tests "org.apache.solr.security.hadoop.TestImpersonationWithHadoopAuth.testForwarding" -Ptests.jvms=4 -Ptests.jvmargs=-XX:TieredStopAtLevel=1 -Ptests.seed=5BCDC1F5C3A460AF -Ptests.nightly=true -Ptests.awaitsfix=true -Ptests.badapples=false -Ptests.file.encoding=UTF-8

- org.apache.solr.core.backup.repository.HdfsBackupRepositoryIntegrationTest.classMethod (:solr:core)
    Test output: /Users/risdenk/repos/apache/solr/solr/core/build/test-results/test/outputs/OUTPUT-org.apache.solr.core.backup.repository.HdfsBackupRepositoryIntegrationTest.txt
    Reproduce with: gradlew :solr:core:test --tests "org.apache.solr.core.backup.repository.HdfsBackupRepositoryIntegrationTest.classMethod" -Ptests.jvms=4 -Ptests.jvmargs=-XX:TieredStopAtLevel=1 -Ptests.seed=FFAA80532AC84F42 -Ptests.nightly=true -Ptests.awaitsfix=false -Ptests.badapples=false -Ptests.file.encoding=UTF-8

- org.apache.solr.core.backup.repository.HdfsBackupRepositoryIntegrationTest.classMethod (:solr:core)
    Test output: /Users/risdenk/repos/apache/solr/solr/core/build/test-results/test/outputs/OUTPUT-org.apache.solr.core.backup.repository.HdfsBackupRepositoryIntegrationTest.txt
    Reproduce with: gradlew :solr:core:test --tests "org.apache.solr.core.backup.repository.HdfsBackupRepositoryIntegrationTest.classMethod" -Ptests.jvms=4 -Ptests.jvmargs=-XX:TieredStopAtLevel=1 -Ptests.seed=5175C707E497BF0A -Ptests.nightly=true -Ptests.awaitsfix=false -Ptests.badapples=false -Ptests.file.encoding=UTF-8

I haven't looked into this yet, but the "negative position" errors are definitely gone and haven't seen failures for the originally reported issue. A few say classMethod so I'm guessing its thread leak related or something like that.

The byte shift logic looks to be going away in LUCENE-10376 / https://github.com/apache/lucene/pull/602

The bit shift logic should be go away due to LUCENE-10366 and https://github.com/apache/lucene/pull/592 - I plan to do this as a follow up.

Rebased on main, updated the commit message, and added solr/CHANGES.txt


Adds a Color Correct node

Wraps up existing adjustment nodes into an artist-friendly color correction node.
Here are some example results from the Houdini docs, using Karma:

Thanks for this great proposal, @crydalch!
My one question is about the addition of individual channel outputs for "r", "g", and "b", since we've been imagining that this need would be covered by the "channel" attribute in MaterialX 1.39 (originally called "channels" in 1.38).
Here is the current language for the "channel" attribute in 1.39:
A node input must generally be connected to outputs of the same type, but float inputs may also be connected to any single channel within a multi-channel data types by adding an integer "channel" attribute, indicating the channel number (0-3) to extract from the input:

    <constant name="c3" type="color3">
      <input name="value" type="color3" value="0.1, 0.2, 0.3"/>
    </constant>
    <constant name="v4" type="vector4">
      <input name="value" type="vector4" value="0.1, 0.2, 0.3, 0.4"/>
    </constant>
    <add name="mult1" type="float">
      <input name="in1" type="float" nodename="c3" channel="0"/>
      <input name="in2" type="float" nodename="v4" channel="3"/>
    </add>

The "channel" attribute is valid in any non-token element that allows a "nodename" attribute.

The motivation behind this feature is to enable graph editor packages to connect individual channels of outputs to scalar inputs, without every MaterialX node interface needing to handle this functionality separately.  Do you believe this would be sufficient for the colorcorrect node as well?

Thanks @jstone-lucasfilm! I think the biggest concern is the channels approach is not supported by UsdShade. I'm also not sure it's necessarily saving the artist from making more nodes or setting more controls; it's just shifting the work since you'd still have to pick the channel after making the connection. (or insert a separate node to separate). Having explicit, additional scalar outputs simplifies the operation to just connecting an output to another node, and seems like less work to me. But I could be missing something. Or are maybe there artist-facing tools that make this vector-->float(channel) connection as simple as connecting two nodes?
Having said all of that, we can just remove the float outputs if it's a point of concern; having the node sooner is probably better for perceived artist-readiness. And the additional float outputs could be added later too of course, even though MtlX's convention/preference seems to heavily favor no multi-output nodes (artistic_ior being an exception).
We should also probably update the node to add a color4 signature, which would then call for adding a 4th scalar output to account for alpha (if those stuck around).

@crydalch Yes, the thinking was that artist-facing tools would present the "channel" feature as simple connections of individual channels of outputs to scalar inputs.  From this perspective, the UI would be the same as if every MaterialX node had individual "r", "g", "b" outputs defined, but without the overhead of providing these individual named outputs in the specification and data libraries.
Let me know how this more general approach sounds to you, and whether it might make sense to work with the USD team to support this feature directly.  This would remove the need to amend all of our high-level nodes with multiple named outputs, and would instead provide this commonly-needed functionality in a universal way.

@jstone-lucasfilm We can remove the float outputs, even if there isn't a way in USD to use the MtlX channels output. I wasn't trying to propose/suggest every color/vector-output node get additional floats outputs, but we can discuss more in tomorrow's mtg (just saw this added to the agenda). Thanks!

My 2c on this: I agree we do not want to have explicit single-channel outputs in every MaterialX node, and that whenever a "channels" attribute is used, each Application would handle this in whatever way was appropriate for it, perhaps by inserting an  node.  In fact the  node in v1.39 uses an integer channel number to indicate which channel to extract, exactly to support this kind of operation.  And we wanted to switch from using string channel names like "r" or "x" and toward integer channel indexes as the latter are much easier to deal with in shading languages.

Some addition notes to this:

I believe the haptoral premise is separation of what is required for runtime workflows and what you need for storage / interop -- the definition is for the latter.
At runtime I would also vote for a direct port->port connection but for storage: extraction on the input port is the current approach.
Believe the same case can be made to have 1,2,3,4 channel and vec vs color variants for definition storage but at runtime this is unwieldy so would not be presented that way.

however

Extraction on the output is partially supported and is the trickier one handle -- so here I'd add a vote for explicit extraction when it's for a common enough workflow.
This does however require some conversion process to make implicit channel extraction explicit.
BTW @dbsmythe , I had put up the rgb/a extract node which is a 1.39 suggestion you added but put this on hold after talking with @jstone-lucasfilm . I think this is another use case to consider. This or other variants for common enough cases I think could remove the desire to add extraction on every node with a vec/color output.


Use shutil.copytree instead of rsync

Remove dependencies
Improve error handling

Fixed by commit https://github.com/timothyryanwalsh/cca-folderprocessor/commit/8ac384ea6edf8bd553fa712749affc8ac0abfb1a


Adding a scroll to top button [GSSOC'23]

A scroll to top button can be implemented in the website

Please assign me this issue under GSSOC'23

@huzaifa9999 I have already opened a similar issue 6hours back issue #269

Please assign me this issue under GSSoC'23 . I'd give my best to fulfil your expectations

Please assign to me as a GSSoC '23 Contributor

i can contribute to this pls assign me

A similar issue is already created and assigned to someone else.
@iLakshya @karthiknadar1204 @abcxyz31 @huzaifa9999 @Shaurya10101
You can try creating another separate issue and start working.


"Rouge" filenames that contain a backslash break automatic globbing for native commands on macOS and Linux

Note: This case is covered by catch-all issue #9244
The presence of a filename containing a \ that matches a wildcard pattern causes native globbing to fail quietly (and pass the pattern through  as-is, as if nothing matched).
Note that \ is a legal filename character on Unix-like platforms.
Steps to reproduce
On macOS and Linux:
touch 'a\b'
/bin/echo * | Should -Not -Be '*'
rm 'a\b'

Expected behavior
The test should pass.
Actual behavior
The test fails, because * is echoed as-is - no expansion to matching filenames occurs.
Windows is not affected, because native globbing isn't applied there).
Environment data

PowerShell Core v7.0.0-preview.4 on macOS 10.14.6
PowerShell Core v7.0.0-preview.4 (7.0.0-preview.4) on Ubuntu 18.04.3 LTS

Note: This is a regression.

Which version?
Does it work on Linux?

The version is stated in the OP, and I originally thought it affects macOS only - but (a) that isn't true and (b) the problem is much more specific - please see the updated title and OP.

Never mind: I just realized that all backslash-escaped woes are covered in #9244, though I'll add a note there about this specific scenario.


fopen font issue

Our web server started throwing these errors when generating a pdf with php-pdf. It worked fine with no errors until yesterday.  The path "fopen(/tmp/cachedTimes-Roman.php)" looks wrong to me, it isn't the path to the font file in php-pdf tree, perhaps it is a tmp/ copy of the font?  The pdf still generates fine.  Error still happens when I comment out the font selection statement.  Unsure why this issue started or how to fix it.  Also concerned that it just started seemingly on its own.
Warning: fopen(/tmp/cachedTimes-Roman.php) [function.fopen]: failed to open stream: Permission denied in /home/site/public_html/include/pdf-php/src/Cpdf.php on line 2039
Warning: fwrite() expects parameter 1 to be resource, boolean given in /home/site/public_html/include/pdf-php/src/Cpdf.php on line 2040
Warning: fclose() expects parameter 1 to be resource, boolean given in /home/site/public_html/include/pdf-php/src/Cpdf.php on line 2041

There is no Cpdf.php file in PHPPdf library. Maybe you use different pdf library.


UVI Request

--- UVI JSON ---
{
  "vendor_name": "Linux",
  "product_name": "Kernel",
  "product_version": "versions from v2.6.31 to before v4.19.217",
  "vulnerability_type": "unspecified",
  "affected_component": "unspecified",
  "attack_vector": "unspecified",
  "impact": "unspecified",
  "credit": "",
  "references": [
    "https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git/commit/?id=985cafccbf9b7f862aa1c5ee566801e18b5161fb",
    "https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git/commit/?id=063f576c43d589a4c153554b681d32b3f8317c7b"
  ],
  "extended_references": [
    {
      "type": "commit",
      "value": "985cafccbf9b7f862aa1c5ee566801e18b5161fb",
      "note": "introduced"
    },
    {
      "type": "commit",
      "value": "063f576c43d589a4c153554b681d32b3f8317c7b",
      "note": "fixed"
    }
  ],
  "reporter": "joshbressers",
  "reporter_id": 1692786,
  "notes": "",
  "description": "comedi: vmk80xx: fix bulk-buffer overflow\n\nThis is an automated ID intended to aid in discovery of potential security vulnerabilities. The actual impact and attack plausibility have not yet been proven.\nThis ID is fixed in Linux Kernel version v4.19.217 by commit 063f576c43d589a4c153554b681d32b3f8317c7b, it was introduced in version v2.6.31 by commit 985cafccbf9b7f862aa1c5ee566801e18b5161fb. For more details please see the references link."
}
--- UVI JSON ---

This issue has been assigned UVI-2021-1001996


hdf4: migrate to homebrew/science rather than homebrew/core

This was part of https://github.com/Homebrew/homebrew-versions/pull/1550.

This was not supposed to be deleted.

@ilovezfs and from the PR: it wasn't?


when dealing with on_friendship, internal error <list index out of range> occurs

I'm trying to fix the bug https://github.com/wechaty/python-wechaty/issues/74
And finally turns out that when the code run in to HostiePuppet.friendship_payload() the below error occured.

2020-06-25 19:28:48,885 - Wechaty - ERROR - internal error
ERROR:Wechaty:internal error

Finally I found that the original treatment
        return FriendshipPayload(**response.to_dict())

is redundant, and leads to a wrong result.
Just return the response do the trick.


No callback received when server moved away for many reason.

I am connected to some web socket server. when server is out of network or somehow not available. there should be some way to get notified that web socket server is not available.

I've noticed the same issue.
When you try to connect to a host that does not exist, the ConnectionState will stay in .waiting forever, even after the request's timeoutInterval.
Using: Starscream 4.0.4
To reproduce:
Just set up a connection to URL(string: "wws://i-dont-exist.xyz")

I've noticed the same issue.
How to solve this.

What i saw is when socket is disconnected we don't get the away/disconnected callback. But if we do ping server every few seconds as required, then we will get the "Socket Disconnected" actually a timeout callback for the ping we do every few seconds.
Since the timeout of the call is default to around 60 seconds.

@surendrapatel274
ping server Refers to the socket?.write(ping: Data()) ?

@cs571393  Correct

ws://<IP_ADDRESS>:9501
I tried ping the server every 10 seconds, but still callback error.

@surendrapatel274
ws://<IP_ADDRESS>:9501
I tried ping the server every 10 seconds, but still callback error.

@surendrapatel274
ws://<IP_ADDRESS>:9501
I tried ping the server every 10 seconds, but still callback error.

Yes, that's correct you will get error callback.. and error callback internally calls forceClose WebSocket connection for that you will get disconnect event. This is a workaround for me, Not the fix.
Well I have tried this with version 4.0.0 and it was working for me.

I'm running into the same issue. When the client is disconnected from the server, there is no callback. Is constantly pinging the server the only way to deal with this? Also, if the client disconnects, then tries to send some messages, then reconnects, the messages it tried to send are sent to the server. In some cases this might be desirable but in my case it isn't.

There are some minor issues in the 3.x versions. That's why I updated to 4.x. But the lifecycle is really confusing. We were using websocketDidDisconnect to deal with all the situations in the old days. But now we have disconnected which only be called by server sent close, cancelled, viabilityChanged with value false, reconnectSuggested and peerClosed. I'm now using viabilityChanged and peerClosed to handle connection lost problem.


Edge and overflow

Concrete popups need overflow:auto on the .popup-content element to accomodate MS Edge's tendency to show empty scroll bars.

735d190


Why is the transfer speed so slow?

Is your feature request related to a problem? Please describe.
Transferring files from my Windows PC to my Android phone at a mere 1.2MB/S.
Describe the solution you'd like
Why is the transfer speed so slow? I am transferring files from my Windows PC to my Android phone at a mere 1.2MB/S. 5G network environment, Gigabit NIC.
Describe alternatives you've considered
No response
Additional context
No response

On Android, the bottleneck is mostly the disk write speed. Do you use a custom download location?

Closing in favor of #384


First class binary operators

:round_pushpin: Refactor 'is_capture' field on function expressions.
Refactored into an enum to make it easier to extend with a new variant to support binary operators.

:round_pushpin: Extend parser to accept anonymous binop as expressions.
This is simply a syntactic sugar which desugarize to a function call with two arguments mapped to the specified binary operator.
Only works for '>' at this stage as a PoC, extending to all binop in the next commit.

:round_pushpin: Add remaining boolean comparison operator to anon binop parser.
Nothing to see here as they all have the same signature. Implementing
arithmetic bin-operators and boolean logic operators will require some
more logic.

:round_pushpin: Support all binary operator in the anonymous binop parser.

:round_pushpin: Implement formatter for anon binop.

Forgot the CHANGELOG.

This is so cool!


icu: 59 -> 64

Motivation for this change
https://github.com/unicode-org/icu/releases/tag/release-64-2
They say updates can go smoothly so let's try! (http://userguide.icu-project.org/icufaq)
We should probably get rid of the old versions that are not needed anymore if everything builds after this. It doesn't look like they have any LTS versions - they're just releasing new versions.
Obsoletes or resolves: #39388 and #52632
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nix-review --run "nix-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[x] Assured whether relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

Could you rebase? It appears there are some incorrect commits.

@GrahamcOfBorg build icu

I managed to fix a few regressions quickly.  PyICU is the only remaining problem I know about; there the compile flags isn't respected I suspect.

Hopefully fine now: https://github.com/NixOS/nixpkgs/commit/326148ec9

This pull request has been mentioned on Nix community. There might be relevant details there:
https://discourse.nixos.org/t/prs-already-reviewed/2617/29


BluetoothHeadsetConnectionListener in BluetoothHeadsetManager causes inconsistency in the bluetooth device detection

Describe the bug
The BluetoothHeadsetManager receives an update form the system indicating STATE_AUDIO_CONNECTED is calling the BluetoothHeadsetConnectionListener.onBluetoothHeadsetStateChanged() with no Bluetooth device name. This will trigger the BluetoothHeadsetConnectionListener registered in AudioSwitch that consequently will remove the previous Bluetooth device detected by AudioSwitch.
To Reproduce
Steps to reproduce the behavior:

Pair the bluetooth audio device. Check that the STATE_CONNECTED is received in BluetoothHeadsetManager and that the device becomes available inAudioSwitch.availableAudioDevices.
Wait a few moments to receive the STATE_AUDIO_CONNECTED  in BluetoothHeadsetManager and that the previously detected device is removed from AudioSwitch.availableAudioDevices.

Expected behavior
In BluetoothHeadsetManager when receiving aSTATE_AUDIO_CONNECTED should call the headsetListener?.onBluetoothHeadsetStateChanged(bluetoothDevice.name)
AudioSwitch Version
Version: 1.1.5

I was able to reproduce the issue several times, with a Xiaomi mi lite 11, Samsung A10 and with airpods and defunc true go .
Only when I performed the change of the open pull request the device name appeared correctly in the. Audioswitch available devices.

@filipebatista are you able to reproduced this with the quickstart? we are struggling to reproduce this issue and looking for a way to so that we can fix it.

Hello 👋
Didn't try to replicate with the quick start,  nevertheless currently I'm no longer with the product that used your SDK so I can't really try it.
Feel free to close if you find relevant.
Thanks

closing due to no longer customer issue


Why does it take longer in my app than in the demo

Hi,
Why same image and same config,  take under 1 sec to compress in your demo  and over 20 secs in my nextjs app.
I notice the converter is loaded from the cdn, but why? I want it to be bundled in my app
Am I missing something
import imageCompression from 'browser-image-compression';

....

try {
        console.time('imageCompression');

        const file = await imageCompression.getFilefromDataUrl(uri, asset.fileName || 'image', Date.now());
        const compressedFile = await imageCompression(file, {
          maxSizeMB: 1,
          maxWidthOrHeight: 1024,
        });
        const base64 = await imageCompression.getDataUrlFromFile(compressedFile);

        console.timeEnd('imageCompression');
      } catch (error) {
        console.error('error', error);
      }

....
      ```

I think compression speed depend on what image type you are trying to compress.
For near instant compression add fileType: 'image/jpeg' to your options.
Like this
{           maxSizeMB: 1,           maxWidthOrHeight: 1024,  fileType: 'image/jpeg'         }

Wow, why does this makes so much of a difference? I got the same issue than @AlejandroGutierrezB but after setting fileType: 'image/jpeg', it was day and night.
I have a MacBook Pro M2 Max and it was soo long without the filetype set to 'image/jpeg'. I didn't take a look at the duration but I think it was something like 15/20 seconds too and now it is near instant even when compressing big files with a strong compression.


[Feature][UI Next]Added the method of downloading files

Purpose of the pull request
Please refer to the main issue #9603.

Brief change log

Verify this pull request

This pull request is code cleanup without any test coverage.
(or)
This pull request is already covered by existing tests, such as (please describe tests).
(or)
This change added tests and can be verified as follows:

Codecov Report

Merging #9605 (ed7883b) into dev (9964c4c) will decrease coverage by 0.01%.
The diff coverage is n/a.

:exclamation: Current head ed7883b differs from pull request most recent head 2743747. Consider uploading reports for the commit 2743747 to get more accurate results

@@             Coverage Diff              @@
##                dev    #9605      +/-   ##
============================================
- Coverage     39.97%   39.95%   -0.02%
+ Complexity     4439     4436       -3
============================================
  Files           832      832
  Lines         33351    33351
  Branches       3683     3683
============================================
- Hits          13332    13327       -5
- Misses        18786    18792       +6
+ Partials       1233     1232       -1

Impacted Files
Coverage Δ

...org/apache/dolphinscheduler/remote/utils/Host.java
37.77% <0.00%> (-2.23%)
:arrow_down:

...dolphinscheduler/remote/future/ResponseFuture.java
81.96% <0.00%> (-1.64%)
:arrow_down:

...e/dolphinscheduler/remote/NettyRemotingClient.java
52.11% <0.00%> (-1.41%)
:arrow_down:

...r/plugin/task/sqoop/parameter/SqoopParameters.java
53.33% <0.00%> (-1.34%)
:arrow_down:

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 9964c4c...2743747. Read the comment docs.


embedded hal digital IO v2

The embedded hal group marked the v1 of digital IO as deprecated, should we also implement a v2 version for the future already?

We're already there!

Implementation
Example


RequestOptions Filter does not properly escape strings.

Hi, I tried to find user by user name using request options however I still got errors until I tried to quote_plus the user name string.
Task:
Find user via user name
<EMAIL_ADDRESS>Steps
import tableauserverclient as tsc
# .. server sign in etc..
request_options = tsc.RequestOptions()
request_options.filter.add(
    tsc.Filter(
        tsc.RequestOptions.Field.Name,
        tsc.RequestOptions.Operator.Equals,
<EMAIL_ADDRESS>        )
)
users, pagination_item = server.users.get(request_options)
print(users)

Expected result
[<User ada95cbc-ca59-4087-bbfe-acbcf55d88a4<EMAIL_ADDRESS>role=Creator>]
Actual result
[]
This is due to the character + in username.
I'm using quote_plus as a workaround.
import tableauserverclient as tsc
from urllib.parse import quote_plus
# .. server sign in etc..
request_options = tsc.RequestOptions()
request_options.filter.add(
    tsc.Filter(
        tsc.RequestOptions.Field.Name,
        tsc.RequestOptions.Operator.Equals,
<EMAIL_ADDRESS>        )
)
users, pagination_item = server.users.get(request_options)
print(users)
# [<User ada95cbc-ca59-4087-bbfe-acbcf55d88a4<EMAIL_ADDRESS>role=Creator>]

Quote plus worked for me as well except for when the % character is present.

This works today


In user & group create api, change the extension field to optional.

Coverage remained the same at 45.8% when pulling 4e55d4859a66107d8687c835b10e04dbdc8381fb on yuye/api-modify into 6ea733e1ae594929e5dfd4aba1cc9994588e48b9 on authndev.


rewrite and expose the git implementation of finding the upstreamLock

The git implementation of GetUpstreamLock() seems to assume that the upstream repo is stored in the parent field. But with my testing, this isn't always the case.
For example, say we have an upstream blueprint and from that we create downstream package deployment with revision v0. Then we do rpkg edit on the downstream package, creating a new downstream package deployment with a revision v1. The upstream of the new package should still be blueprint, but the parent instead points at the deployment v0 package it was copied from.
I think we have to look at the Kptfile to determine the upstreamLock, so this code modifies GetUpstreamLock to do that.

Ah, I see that there is already an e2e test that depends on the current implementation. I'll explore some options.


"Invalid or missing token" dht related error

The Issue
Not sure why its happening but I've seen this on my logs and on srv25. Looks like its only happening to a single IP - <IP_ADDRESS>. It does happen repeatedly in my logs and at different intervals.
2017-10-14 17:13:56,619 WARNING  lbrynet.core.server.ServerProtocol:47: Closing a connection. Reason: Connection to the other side was lost in a non-clean fashion.
2017-10-14 18:36:14,480 INFO     lbrynet.core.server.BlobRequestHandler:151: Sending dce7025c49ea0433 to <IP_ADDRESS>:56859
2017-10-14 18:36:16,733 INFO     lbrynet.core.server.BlobRequestHandler:151: Sending b45843c920c7b950 to <IP_ADDRESS>:43125
2017-10-14 18:36:25,395 INFO     lbrynet.core.server.BlobRequestHandler:151: Sending ef50ada028967f70 to <IP_ADDRESS>:43125
2017-10-14 18:38:52,471 ERROR    lbrynet.dht.protocol:391: error handling request for <IP_ADDRESS>: store
Traceback (most recent call last):
  File "site-packages\lbrynet\dht\protocol.py", line 387, in _handleRPC
  File "site-packages\lbrynet\dht\node.py", line 475, in store
ValueError: Invalid or missing token
2017-10-14 18:48:01,617 INFO     lbrynet.core.server.BlobRequestHandler:151: Sending 34c64a690e5f2e64 to <IP_ADDRESS>:56920

srv25:
2017-10-14 19:49:27,951 INFO     lbrynet.core.server.BlobRequestHandler:151: Sending 3d78c437be52e702 to <IP_ADDRESS>:54130
2017-10-14 19:49:28,619 INFO     lbrynet.core.server.BlobRequestHandler:151: Sending 4658e950c8945788 to <IP_ADDRESS>:54130
2017-10-14 22:38:50,350 ERROR    lbrynet.dht.protocol:391: error handling request for <IP_ADDRESS>: store
Traceback (most recent call last):
  File "site-packages/lbrynet/dht/protocol.py", line 387, in _handleRPC
  File "site-packages/lbrynet/dht/node.py", line 475, in store
ValueError: Invalid or missing token
2017-10-15 07:02:40,159 ERROR    lbrynet.dht.protocol:391: error handling request for <IP_ADDRESS>: store
Traceback (most recent call last):
  File "site-packages/lbrynet/dht/protocol.py", line 387, in _handleRPC
  File "site-packages/lbrynet/dht/node.py", line 475, in store
ValueError: Invalid or missing token
2017-10-15 07:02:42,413 ERROR    lbrynet.dht.protocol:391: error handling request for <IP_ADDRESS>: store
Traceback (most recent call last):
  File "site-packages/lbrynet/dht/protocol.py", line 387, in _handleRPC
  File "site-packages/lbrynet/dht/node.py", line 475, in store
ValueError: Invalid or missing token
2017-10-15 18:44:51,727 WARNING  lbrynet.core.server.ServerProtocol:47: Closing a connection. Reason: Connection to the other side was lost in a non-clean fashion.
2017-10-15 18:56:15,903 INFO     lbrynet.core.server.BlobRequestHandler:151: Sending 729589475e97fd29 to <IP_ADDRESS>:55146
2017-10-16 05:15:18,829 INFO     lbrynet.core.server.BlobRequestHandler:151: Sending 236502c239d62779 to <IP_ADDRESS>:50069

Expected behaviour
Tell us what should happen
Actual behaviour
Tell us what happens instead
System Configuration

LBRY Daemon version:
LBRY App version:
LBRY Installation ID:
Operating system:

Anything Else

Screenshots

@jackrobison says this is probably related to old clients (from before we fixed the lbry id issue)


Security Vulnerability Disclosure

Doesrataui participate in any vulnerability disclosure programs? I believe I've found a security issue.

I've just turned on the private security vulnerability notification.
Please use https://github.com/ratatui-org/ratatui/security/advisories/new


Fix issue with reading and updating context when using the Ollama chat endpoint

This change addresses #72 and is a follow up to the changes made here #68

Thank you a lot for providing the fix, @abayomi185 !

Hello! How do I can use chatgpt implementation in config? Thank you


fix: slow recompile time

fixes #9886
This is the commit that removed the cache flag:
https://github.com/FezVrasta/create-react-app/commit/d07b7d025f5933710fcb01718617dbdf4bc54c33#r43578777

Thanks for the fix!

How will we know when this is available? Will there be a 4.0.1 release? Thanks 👍

Just wondering - since this is blocking me from upgrading CRA for my team, any thoughts on when a patch version with this change might be published to NPM? Thanks again for the fix!

Please consider splitting the 4.0.1 release and ship what has been fixed already.

I tried upgrading to react-scripts 4.0.1, but it didn't seem to fix the slow fast refresh speed.

Soo... AFAIK theeslint-webpack-plugin does not have the cache option. There were some performance concerns issues raised within the plugin repo but they were fixed with a new release. IMO, we should just bump the dependency, otherwise, everyone will be forced to use resolutions or patch-package.


Make Theatre studio play nicer with Next.js, SSR, and hot reload

It is currently possible to use Theatre in a Next.js project but there are errors both in the browser console and in the Next.js terminal console due to some logic being run on the server during SSR. The app also crashes when hot reloading due to the R3F extension being already registered.

Now fixed by #298 and will be in v0.5.1. Thanks for reporting the issue :)

Hi @AriaMinaei is there a release candidate or tag I could use to test this beforehand? I tried @0.5.0-insiders.b705c0b mentioned in the PR but it doesn't seem to do the trick. Thanks!

Hello @AriaMinaei I am running the same issue as @thiagobrez and I would also love to know the release candidate or tag to use, thank you very much ! 😄

FWIW, I tried the then-lastest insider version 1 week ago, and I was still getting warnings and a new error, so I suppose it's not ready yet :)

@AriaMinaei Still getting a lot of console warnings and server-side ReferenceError: window is not defined with 0.5.1-rc.1.
0.5.1-rc.1:
event - compiled client and server successfully in 2.7s (1949 modules)
wait  - compiling /_error (client and server)...
error - ReferenceError: window is not defined
    at Object.<anonymous> (/Users/verekia/Local/Code/private-projects/node_modules/.pnpm/@theatre+studio@0.5.1-rc.1_@theatre+core@0.5.1-rc.1/node_modules/@theatre/studio/dist/index.js:52037:1)
    at Module._compile (node:internal/modules/cjs/loader:1119:14)
    at Module._extensions..js (node:internal/modules/cjs/loader:1173:10)
    at Module.load (node:internal/modules/cjs/loader:997:32)
    at Module._load (node:internal/modules/cjs/loader:838:12)
    at Module.require (node:internal/modules/cjs/loader:1021:19)
    at require (node:internal/modules/cjs/helpers:103:18)
    at Object.<anonymous> (/Users/verekia/Local/Code/private-projects/node_modules/.pnpm/@theatre+r3f@0.5.1-rc.1_bxw727elgbdmrpxfkxx5mvjf74/node_modules/@theatre/r3f/dist/extension/index.js:18471:33)
    at Module._compile (node:internal/modules/cjs/loader:1119:14)
    at Module._extensions..js (node:internal/modules/cjs/loader:1173:10) {
  page: '/games/aoe'
}
You seem to have imported '@theatre/studio' but haven't initialized it. You can initialize the studio by:

import studio from '@theatre/studio'
studio.initialize()

* If you didn't mean to import '@theatre/studio', this means that your bundler is not tree-shaking it. This is most likely a bundler misconfiguration.

* If you meant to import '@theatre/studio' without showing its UI, you can do that by running:

import studio from '@theatre/studio'
studio.initialize()
studio.ui.hide()

crash in the browser.

With 0.5.0 :
wait  - compiling /games/aoe (client and server)...
event - compiled client and server successfully in 2.1s (1949 modules)
@theatre/dataverse is running in a server rather than in a browser. We haven't gotten around to testing server-side rendering, so if something is working in the browser but not on the server, please file a bug: https://github.com/theatre-js/theatre/issues/new
You seem to have imported '@theatre/studio' but haven't initialized it. You can initialize the studio by:

import studio from '@theatre/studio'
studio.initialize()

* If you didn't mean to import '@theatre/studio', this means that your bundler is not tree-shaking it. This is most likely a bundler misconfiguration.

* If you meant to import '@theatre/studio' without showing its UI, you can do that by running:

import studio from '@theatre/studio'
studio.initialize()
studio.ui.hide()

Are you doing any server-side rendering with Theatre? If not, I believe there shouldn't be any message in the console at all.
If you are doing some checks like typeof window === undefined, it will break SSR. Checks need to happen after app mount to not break SSR (in a useEffect in React for instance).

Yup, reopening the issue :)  Thanks for the tip @verekia 🙏

@verekia could you please link me to a reproduction of this issue? The article from Josh talks specifically about conditionally rendering React components based on the window check, which, breaks rehydration, but you seem to be getting a ReferenceError, which shouldn't happen.

This should be resolved in the latest prerelease.

0.5.1-rc.2 is not crashing anymore 👍
However there are still server-side logs, which in my opinion should not happen for a fully client-side tool like Theatre. But that will probably require some more significant rewriting.
Could you add to the docs the proper way to load Theatre.js in a Next environment?
I wrote a quickstart Next + R3F setup here if you want a minimal setup for testing.

Done, via #353

This is a lot better now in 0.6.0, thank you!
For reference, here is my current setup to get Theatre working with no console message + excluding it from production (I only want to use it to inspect my scene in development):
A global variable to check if Theatre has been initialized in a separate file (I think this was needed to play nicer with HMR) in theatre.ts:
const theatre = { isInit: false }
export default theatre

The main setup:
import { SheetProvider, editable } from '@theatre/r3f'
import extension from '@theatre/r3f/dist/extension'
import studio from '@theatre/studio'

if (process.env.NODE_ENV === 'development' && !theatre.isInit && typeof window !== 'undefined') {
  studio.initialize()
  studio.extend(extension)
  studio.ui.hide() // Hidden by default
  theatre.isInit = true
}

const IndexPage = () => (
  theatre.isInit
    ? (
      <SheetProvider sheet={getProject('Project').sheet('Sheet')}>
        <Scene />
      </SheetProvider>
    : <Scene />
)


Update Flags to Filter Links

Update filter flags to -good and -bad for clarity and add -all flag.

https://github.com/AbdulMAbdi/deadOrNot/commit/ca77f9615375a2949589d1a3c877a2212dba5e87


Logic clarification for image cache line size calculation

Hello,
I am interested in knowing if you could provide further clarification on the logic behind the cache line size calculation for images, utilizing the ratio between concurrencies, as seen here.
cacheline_size = PX_SIZE * std::max(concur_nthread_x, concur_nthread_y) /
        std::min(concur_nthread_x, concur_nthread_y);

cacheline_dim = concur_nthread_x >= concur_nthread_y ? "X" : "Y";

My questions are:

What is the reasoning behind this ratio?
What do you refer to when speaking of a "cache line dimension"?
How would you extend it to a third dimension?

Thank you and great work developing this tool :)

In the paper we discussed 2D textures. 2D textures can layout the pixels either by rows or columns so the ratio is basically the number of texels in a cacheline divided by 1.
The above reasoning certainly generalizes to 3D textures but they are seldomly used in mobile applications. Drivers usually don't layout in the third dim so we don't spend much time on that.

I'm closing this for non-activity. Feel free to reopen if you have further questions


Add links to research surveys

I think the corona warn app would also be the perfect place to give users the ability to participate directly in research projects that help advance research in Covid-19 detection.
Just one example is the "COVID-19 Detection by Cough and Voice Analysis" where data is collected simply with the smartphone microphone to develop software that can detect early COVID infections using only the phone in your pocket.
One could simply add a new site in the app with links that redirect the users interested to relevant surveys and gives a short explaination what the survey is about.
For example this is the direct link to the "COVID-19 Detection by Cough and Voice Analysis":
https://cdcva.list.lu/
Idealy you would work together with the researchers so the survey gets even better when participants later find out that they tested positive but that is also much more work compared to simply provide users with links to the researchers websites.

Internal Tracking ID: EXPOSUREAPP-4527

Hello @ChristophGeske
Thank you for your suggestion. We have added your feature request to our internal Jira ticketing system (ticket ID: EXPOSUREAPP-4527) where it will now be reviewed by the dvelopers and the RKI. They will decide if and how this  feature would be implemented in the app. We will update you here if we get news about the pogress on the issue.
Regards,
CH

Corona-Warn-App Open Source Team

Hello @ChristophGeske
Thank you for your suggestion. We have added your feature request to our internal Jira ticketing system (ticket ID: EXPOSUREAPP-4527) where it will now be reviewed by the dvelopers and the RKI. They will decide if and how this  feature would be implemented in the app. We will update you here if we get news about the pogress on the issue.
Regards,
CH

Corona-Warn-App Open Source Team

related  ticket (EXPOSUREAPP-4637)[https://jira-ibs.wbs.net.sap/browse/EXPOSUREAPP-4637]

Version 1.13 introduced a survey which will be shown to users with high-risk encounters.
More information:
DE: https://www.coronawarn.app/de/blog/2021-03-04-corona-warn-app-version-1-13/
EN: https://www.coronawarn.app/en/blog/2021-03-04-corona-warn-app-version-1-13/
This isn't exactly what has been proposed by @ChristophGeske, so should be up to you whether you want to close this or not.

@ChristophGeske Thanks for proposing here. We suggest closing this issue. However, we are very happy to hear and forward new (if possible detailed) suggestions on how to enhance the newly introduced features (as there are event registration (future), data donation, RKI survey). Best wishes, DS

Corona-Warn-App Open Source Team

As this was implemented in CWA v1.13, please open new issues regarding enhancements of the feature. We will close this issue.

Corona-Warn-App Open Source Team


[Fix](regression test) Fix schema change concurrent with txn regression case

Proposed changes
PR #37669 introduced a new config enable_abort_txn_by_checking_conflict_txn=true. This config will abort schema while loading txn and restart coordinator, which causes regression test failed. We set it to false to fix this test.

run buildall


sparse.linalg.gmres extra calls to matvec

Hi,
I am using GMRES with scipy and am wondering about the initialization. Why are there several calls to matvec before the first iteration starts? Can you explain what the purpose of these is? Is there any way to disable them?
Thanks.

Can you give an example of what do you mean? I see zero calls before Arnoldi starts?

Sure, here is what I don't understand (maybe I just don't understand how GMRES works). If I take the code below, "applying function" gets output 3 times, even though I've only specified 1 iteration. Is that typical for GMRES? Thanks
class GMRESThing(object):
def __init__(self,ain):
    self._a = ain;

def mv(self,v,b):
    print('Applying function')
    return np.array([(0.1+self._a)*v[0], (b+0.1)*v[1]])

def mvInverse(self,v,b):
    print('Applying preconditoner')
    return np.array([1/self._a*v[0], 1/b*v[1]])

GRes = GMRESThing(2);
b = 3;
A = LinearOperator((2,2), matvec=partial(GRes.mv,b=b))
Pinv = LinearOperator((2,2), matvec=partial(GRes.mvInverse,b=b))
x, info = gmres(A,np.ones(2),M=Pinv,callback=print,callback_type='pr_norm',restart=1,maxiter=1,atol=0).

Yes, it's expected: the documentation says Maximum number of iterations (restart cycles), and "restart cycle" is by default 20. You can read up e.g. on wikipedia what it means.
To avoid one extra call, you should also give the dtype= attribute of your LinearOperators, in addition to giving the matvec=.


Added a driver circuit meant to interface with the 1.44' EPD.

Summary of required changes:

Components C1, R1, and C11 need values
All required components must be added to the order sheet to verify footprints
Decision must be made as to if we can handle 0402 components instead of 0603
Overlapping labels in the circuit diagram must be moved for diagram clarity
Temperature sensor must be added for proper refresh rate. See note 2 on p 12 of Pervasive Displays datasheet
Naming nets as "Connect to timing controller SPI" has linked all those pins together. These are distinct pins in the SPI interface and must be named separately to avoid linking nets.
Your VCC/VDD net has label leading off but no pin text.
You have added all net labels to the net layer (91). It should be moved to the info layer (97). This is an option in the top menu when adding text.
Your border pin on the bottom pin is mislabeled as "Connect to timing controller GPIO"
You have neglected to add the MOSFETS and the Diode included in the datasheet. Without them, we will have difficulty powering down the display.
You didn't put in the MOS power switch as suggested on the VDD pin by the datasheet to prevent leakage current. This is highly important for our low-power application.

On 6., those are just text nodes, so i think its fine? I don't believe they're linked together, since I didnt assign the text to the net

Okay as long as they're moved to the text layer
On Sep 22, 2015 12:44 AM, "Joshua Miller"<EMAIL_ADDRESS>wrote:

On 6., those are just text nodes, so i think its fine? I don't believe
they're linked together, since I didnt assign the text to the net
—
Reply to this email directly or view it on GitHub
https://github.com/zbears/MedStat/pull/2#issuecomment-142180598.

Wait, I also don't think we need 5. We're doing multiple writes to the EPD in order to get darker colors, and also we don't really care about write speed? Plus, how would we use the temperature sensor data anyway?

How do I add pin text (7.)?

Went to do 11 and 12, couldnt find the libs for the devices. Where can I find 2N7002KW?

Anyway, solved about half of what you posted- waiting on responses to the other questions. Will repush in just a sec.

For 5, I was just concerned that it would affect the refresh rate on a pixel by pixel basis. If you think we're fine without it, I'll trust you on that. For 7 to add that text, use the label tool to place the net name on the branch then use the name tool to rename the mesh. For 11 and 12, make a custom part based off of the footprint of a similar part. It frequently happens that Eagle doesn't have the perfect parts. You just have to make due.

@zbears Just updated this. Couldn't figure out what you meant by #7- also couldn't figure out how to successfully generate a custom part- however, I used one with a similar footprint.

Looks good. I'll take your word for the footprints and lack of thermometer.


Test-TargetResource failing without explanation

Had an issue where the module installed SQL correctly, but when testing after the installation (Test-TargetResource), an exception would be thrown.
The configuration I'm currently using:
xSQLServerSetup InstallSqlServer
{
    InstanceName = "SQLExpress"
    SourcePath = $destinationPath
    SourceFolder = ""
    Features= "SQLEngine,LocalDB"
    UpdateSource = "MU" # set this to windows update, otherwise it will use the default and cause the setup to crash
    SetupCredential = $credential
    DependsOn = "[WindowsFeature]NetFramework35Core","[WindowsFeature]NetFramework45Core","[File]CopySqlServerInstaller"
}

After looking into the module's source code, I noticed the features' check was failing because the testing code expects the features to be in upper-case format (also noted by another user, #38).
Furthermore, Test-TargetResource failed without providing an explanation to why (and made even more baffling given that SQL was correctly installed).
So I added a bit more logging to Test-TargetResource and normalised the feature to upper-case.

This change is

Hi @jpmnteiro, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!

This seems like a small (but important) contribution, so no Contribution License Agreement is required at this point. Real humans will now evaluate your PR.

TTYL, MSBOT;

Reviewed 1 of 1 files at r1.
Review status: all files reviewed at latest revision, 1 unresolved discussion.

DSCResources/MSFT_xSQLServerSetup/MSFT_xSQLServerSetup.psm1, line 946 [r1] (raw file):
Please add a test to verify that this logic change works as expected Contribution Guidelines -Adding test coverage for DSC resources

Comments from Reviewable

@jpmnteiro - Any updates on this PR? If not, I'm going to close it, feel free to reopen though if you have more changes

@jpmnteiro I would love to see this PR merged. Can I assist you in anyway with this PR to move it along? If you don't have time to work on this anymore I would be happy to continue working on this. Let me know if you want me to do that.

This reference issue #38, but it does not fix issue #38. This issue just make sure the Test-method write a verbose statement when the there is no match to a feature. I would like to see this PR fix #38 in a way that makes the features "case-insensitive".

Since this PR seems abandon I will continue the work on this PR. I will send in a new PR in a few days. I hope I have time to fix that before traveling to Atlanta and Ignite on friday.

@mbreakey3 You can close this one now. I transferred the changes to a new PR that I submitted.


Typing based scrolling in lists

Unless focus is in the search bar, typing in a sorted list should scroll to an element. I started this, but it cause weird behavior with FluentUI right-click context menus.

The weird context menu behavior is due to the context menus disappearing when there's a scroll event, and the DetailsList view just wiggles (I should probably make my own at some point...)

Got the basics in place here: 7b4bb28fe8bbbe148c4eb2c661a888e330d75b30

Seems like it works (with the caveat of DetailsList being somewhat problematic: I might try moving away from IGroups and see if that helps) 093be802c6a793ad6bfa82811cce9d0b41fa50dd


Style configuration isn't working

The style configuration doesn’t seem to be working. Neither the color options nor CSS are being passed on to the widget. What am I doing wrong?
 <DePayWidgetsButton
          integration={process.env.DEPAY_INTEGRATION_ID}
          payload={{
            paymentTransactionId: id,
            amount: {
              currency: currency,
              fix: amount,
            },
            title: "ooone Payment",
            style: {
              colors: {
                primary: "#9747FF",
                text: "#1a1a1a",
                buttonText: "#000000",
                icons: "#9747FF",
              },
              css: /* CSS */ '
                .ButtonPrimary {
                  background-color: var(--color-purple);
                  color: var(--color-black);
                }
              ',
            },
          }}
        />

It seems you have a basic misunderstanding what payload is for. Please read the documentation again.
regarding style, you will need to pass it to the configuration attribute, not the payload attribute:
<DePayWidgetsButton configuration={{ style: {} }}
Regarding payload. This is used to forward frontend information to your backend endpoint in order to calculate dynamic pricing. Never just forward the amount payable, as users could set whatever they want.

You are right. I mixed up something. It works. Thank you for your help. 👍🏻


Feature Request: replace VNC with XPRA

replace VNC with XPRA and html5 server, so we can access the deCONZ ui via Browser,
like this container,
https://github.com/ffeldhaus/docker-wireshark

Generally sounds like an interesting idea. Is it really stable enough?
In your wireshark example I see bullseye used, which is not released as stable yet, will be 2021.
Additional there seem to be no debian packages for, they have to be taken from 3rd-party repository.

@b2un0 I agree that browser access would be best, but of course it needs to be stable. You should make this request in the official deCONZ repository. They should really be the ones implementing such feature.

As there hasn't been any new replies and as there are newer versions, i'm closing this issue. If this needs to be re-opened, please ping me :)


Can this be used for Laplace approximation?

In Laplace approximation, the Hessian of the loss function is computed for quadratic approximation. Can this package be used to do a block-diagonal approximation of the Hessian at the minimum? If yes, could you please show (using jax and flax) how to approximate it and define a quadratic approximation of the loss function (which should be something like 1/2 (theta - theta_star)^T H(L)(theta_star) (theta - theta_star), where theta_star is the minimum and H(L) is the Hessian of the loss function)?

Hi, yes, this can be used for a Laplace approximation. In particular you can take a look at the CurvatureEstimator and how it can be used. We don't have code for that, but it can be fairly straightforward to do it. There are however several details that one might need to pay attention to, such as:

The code in the library estimates the "average" GGN/Fisher, which means that if you have a prior, you need to rescale it by 1/N
The approximation to the GGN might not be a perfect approximation to the full GGN, so you might want to consider techniques which can mitigate that.


Fix the null issue.

Null Error is not handled while no movie is selected.

Any Updates @vikiwarrior ?


listtransactions empty

Hi,
When mining and blocks are found, they are not seen in list transactions. Any clues why?
Thanks

Hello, are you in trouble kumacoin yiimp pool?
Do you meam that you cannot seen any PoW mined transactions by listtransactions command?

Yes that’s right. Any idea why that is?
Thanks
On Sun, Nov 18, 2018 at 11:08 AM namuyang<EMAIL_ADDRESS>wrote:

Hello, are you in trouble kumacoin yiimp pool?
Do you meam that you cannot seen any PoW mined transactions by
listtransactions command?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/kumacoinproject/kumacoin/issues/6#issuecomment-439699754,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AGAv4hFrOd_HJ1bxRAAMfX2dMytgrOCRks5uwXffgaJpZM4VP1Ov
.

Generated block undisplayed. rpcwallet.cpp#L1004
Please wait....

How about remove commet out?
miningbeaver pool report fix the problem.


ERROR #42P02 there is no parameter $1 when using pgdriver with SQLBoiler

I was impressed by the benchmarks relative to pgx, so tried it out on my project that's already using SQLBoiler. However, most of my tests fail with:
ERROR #42P02 there is no parameter $1

For reference, the project is here: https://github.com/hortbot/hortbot
If you want to test this, changing the driver for the entire project is as simple as changing the _ import and driver name here: https://github.com/hortbot/hortbot/blob/master/internal/db/driver/driver.go
I can enable SQLBoiler's debugging to get a trace of the SQL queries it tried for one of the tests:
--- FAIL: TestScripts (0.00s)
    --- FAIL: TestScripts/misc_errors (1.16s)
        print.go:265: SELECT r.* FROM repeated_commands r JOIN channels c ON r.channel_id = c.id WHERE r.enabled AND c.active
        print.go:265: []
        print.go:265: SELECT s.* FROM scheduled_commands s JOIN channels c ON s.channel_id = c.id WHERE s.enabled AND c.active
        print.go:265: []
        write_syncer.go:66: 2021-06-28T20:04:32.448-0700	ERROR	bot/handle.go:81	error during handle	{"xid": "c3d8rg2fi36ho6imuqog", "irc_command": "PRIVMSG", "error": "ERROR #42P02 there is no parameter $1", "message": "@id=d079f2bc-12cc-4c06-a9a4-2212c69decd8;room-id=999;user-id=1<EMAIL_ADDRESS>PRIVMSG #hortbot :!join"}
            github.com/hortbot/hortbot/internal/bot.(*Bot).Handle
            	/home/jake/zikaeroh/hortbot/hortbot2/internal/bot/handle.go:81
            github.com/hortbot/hortbot/internal/bot/btest.(*scriptTester).handleM.func1
            	/home/jake/zikaeroh/hortbot/hortbot2/internal/bot/btest/irc.go:57
            github.com/hortbot/hortbot/internal/bot/btest.(*scriptTester).test
            	/home/jake/zikaeroh/hortbot/hortbot2/internal/bot/btest/script_tester.go:207
            github.com/hortbot/hortbot/internal/bot/btest.RunScript
            	/home/jake/zikaeroh/hortbot/hortbot2/internal/bot/btest/script_tester.go:49
            github.com/hortbot/hortbot/internal/bot_test.TestScripts.func1
            	/home/jake/zikaeroh/hortbot/hortbot2/internal/bot/script_test.go:28
            testing.tRunner
            	/usr/lib/go/src/testing/testing.go:1193
        irc.go:129: assertion failed: expression is false: st.sender.SendMessageCallCount() > callNum: SendMessage not called: line 2
FAIL
FAIL	github.com/hortbot/hortbot/internal/bot	1.655s
FAIL

Another test:
--- FAIL: TestFindBotTokenNew (1.18s)
    print.go:265: INSERT INTO "twitch_tokens" ("created_at","updated_at","twitch_id","bot_name","access_token","token_type","refresh_token","expiry") VALUES ($1,$2,$3,$4,$5,$6,$7,$8) RETURNING "id"
    print.go:265: [2021-06-29 03:09:22.372037781 +0000 UTC 2021-06-29 03:09:22.372037781 +0000 UTC 1234 {hortbot true} at-qwertyuiop1234567890 Bearer rt-qwertyuiop1234567890 2021-06-28 20:09:22.372031454 -0700 PDT m=+1.241757040]
    twitchx_test.go:156: assertion failed: error is not nil: models: unable to insert into twitch_tokens: ERROR #42P02 there is no parameter $1

Via debugging, the query in the trace above is actually a failing pair. My original issue's queries don't seem related.

With some more debugging, I noticed that the function for querying doesn't do anything with its args.
https://github.com/uptrace/bun/blob/0d6bfa9a9e49b1e024f0ae81fc242635b5a53f96/driver/pgdriver/driver.go#L288-L295

Please try v0.2.11 - it should format passed query args. Note that it is only working when you are using *sql.DB, not bun.DB wrapper:
sqldb.QueryRow("select $1", "hello") // works
bundb.QueryRow("select $1", "hello") // does not
bundb.QueryRow("select ?", "hello") // works

That is because Bun has its own placeholders ? and supporting both $1 and ? can be confusing.

I was impressed by the benchmarks relative to pgx

Please let me know how good/bad performance is in your project. pgdriver does not have statement cache and it seems like your project relies/benefits from implicit prepared statements...

Thanks for taking a look. The error has changed to something else, unfortunately:
--- FAIL: TestFindBotTokenGetError (2.98s)
    twitchx_test.go:97: assertion failed: error is not nil: models: unable to insert into twitch_tokens: ERROR #42601 syntax error at or near "08"
--- FAIL: TestFindBotTokenNewDBError (3.01s)
    twitchx_test.go:206: assertion failed: error is not nil: models: unable to insert into twitch_tokens: ERROR #42601 syntax error at or near "08"
--- FAIL: TestFindBotTokenNew (3.01s)
    print.go:265: INSERT INTO "twitch_tokens" ("created_at","updated_at","twitch_id","bot_name","access_token","token_type","refresh_token","expiry") VALUES ($1,$2,$3,$4,$5,$6,$7,$8) RETURNING "id"
    print.go:265: [2021-06-29 08:09:04.826840056 +0000 UTC 2021-06-29 08:09:04.826840056 +0000 UTC 1234 {hortbot true} at-qwertyuiop1234567890 Bearer rt-qwertyuiop1234567890 2021-06-29 01:09:04.826836635 -0700 PDT m=+3.161785892]
    twitchx_test.go:156: assertion failed: error is not nil: models: unable to insert into twitch_tokens: ERROR #42601 syntax error at or near "08"
--- FAIL: TestFindBotToken (0.06s)
    twitchx_test.go:67: assertion failed: error is not nil: models: unable to insert into twitch_tokens: ERROR #42601 syntax error at or near "08"

Or in a different test:
models: unable to insert into channels: ERROR #42601 syntax error at or near ","

Please let me know how good/bad performance is in your project. pgdriver does not have statement cache and it seems like your project relies/benefits from implicit prepared statements (depending on how well stmt cache works)...

I'll let you know. I've taken a good amount of time to optimize SQLBoiler to effectively no overhead, so pgx was the last bit I can't really fix.

@zikaeroh can you show the full failing query? It looks like pgdriver builds incorrect queries...

Inserting a log directly at the formatQuery calls you added:
SELECT 1
CREATE DATABASE temp_template WITH TEMPLATE postgres
CREATE DATABASE temp1 WITH TEMPLATE temp_template
INSERT INTO "twitch_tokens" ("created_at","updated_at","twitch_id","bot_name","access_token","token_type","refresh_token","expiry") VALUES (2021-06-29 08:29:17.282063+00:00,2021-06-29 08:29:17.282063+00:00,1234,'hortbot','at-qwertyuiop1234567890','Bearer','rt-qwertyuiop1234567890',2021-06-29 08:29:17.282061+00:00) RETURNING "id"

Oops, looks like time is not quoted at all. Please try v0.2.12.

Closer; the test that I pulled those lines from passes, but others still fail. Here's another failing trace.
SELECT 1
CREATE DATABASE temp_template WITH TEMPLATE postgres
CREATE DATABASE temp1 WITH TEMPLATE temp_template
SELECT r.* FROM repeated_commands r JOIN channels c ON r.channel_id = c.id WHERE r.enabled AND c.active
SELECT s.*  FROM repeated_commands r JOIN channels c ON r.channel_id = c.id WHERE r.enabled AND c.active
        print.go:265: []
        print.go:265: SELECT s.* FROM scheduled_commands s JOIN channels c ON s.channel_id = c.id WHERE s.enabled AND c.active
        print.go:265: []
        print.go:265: SELECT COUNT(*) FROM "blocked_users" WHERE ("blocked_users"."twitch_id" = $1) LIMIT 1;
        print.go:265: [1]
        print.go:265: SELECT * FROM "channels" WHERE ("channels"."twitch_id" = $1) LIMIT 1 FOR UPDATE;
        print.go:265: [1]
        print.go:265: INSERT INTO "channels" ("created_at","updated_at","twitch_id","name","display_name","bot_name","active","prefix","bullet","message_count","mode","cooldown","last_fm","parse_youtube","extra_life_id","raffle_enabled","steam_id","urban_enabled","tweet","roll_level","roll_cooldown","roll_default","should_moderate","display_warnings","enable_warnings","timeout_duration","enable_filters","filter_links","subs_may_link","filter_caps","filter_caps_min_chars","filter_caps_percentage","filter_caps_min_caps","filter_emotes","filter_emotes_max","filter_emotes_single","filter_symbols","filter_symbols_percentage","filter_symbols_min_symbols","filter_me","filter_max_length","filter_banned_phrases","sub_message","sub_message_enabled","resub_message","resub_message_enabled","filter_exempt_level") VALUES ($1,$2,$3,$4,$5,$6,$7,$8,$9,$10,$11,$12,$13,$14,$15,$16,$17,$18,$19,$20,$21,$22,$23,$24,$25,$26,$27,$28,$29,$30,$31,$32,$33,$34,$35,$36,$37,$38,$39,$40,$41,$42,$43,$44,$45,$46,$47) RETURNING "id","ignored","custom_owners","custom_mods","custom_regulars","permitted_links","filter_banned_phrases_patterns","last_seen"
        print.go:265: [2021-06-29 08:52:55.799668748 +0000 UTC 2021-06-29 08:52:55.799668748 +0000 UTC 1 foobar foobar hortbot true ! { false} 0 everyone {0 false}  false 0 false  false Check out (_CHANNEL_URL_) playing (_GAME_) on @Twitch! subscriber 10 20 true false true 600 false false true false 0 50 6 false 4 false false 50 5 false 500 false  false  false subscriber]
        write_syncer.go:66: 2021-06-29T01:52:55.800-0700	ERROR	bot/handle.go:81	error during handle	{"xid": "c3ddupqfi36m3iehlecg", "irc_command": "PRIVMSG", "error": "models: unable to insert into channels: ERROR #42601 syntax error at or near \",\""}

That last line at the top is oddly truncated at the front.
FWIW, you don't have to publish a new version for each of these for me to test if that's annoying; I've already done a local replacement to print these out!

:+1: Should be fixed in master - nil values were incorrectly appended.

Indeed it is; my tests now pass. Thanks for looking into this!
If you're curious, here is a benchstat of pgx vs pgdriver:
name                       old time/op    new time/op    delta
HandleNop-4                   730µs ± 1%     960µs ± 1%  +31.36%  (p=0.000 n=9+9)
HandleNopParallel-4           614µs ± 4%     857µs ± 1%  +39.53%  (p=0.000 n=9+9)
HandleCustomCommand-4         839µs ± 2%    1356µs ± 1%  +61.64%  (p=0.000 n=9+10)
HandleMixed-4                 789µs ±29%    1047µs ± 1%  +32.57%  (p=0.000 n=10+9)
HandleManyBannedPhrases-4    1.10ms ± 0%    1.50ms ± 1%  +36.27%  (p=0.000 n=8+8)

name                       old alloc/op   new alloc/op   delta
HandleNop-4                  19.9kB ± 0%     9.5kB ± 0%  -52.11%  (p=0.000 n=9+10)
HandleNopParallel-4          20.1kB ± 0%     9.6kB ± 0%  -52.40%  (p=0.000 n=10+10)
HandleCustomCommand-4        24.9kB ± 0%    13.4kB ± 0%  -46.31%  (p=0.000 n=10+10)
HandleMixed-4                23.9kB ± 0%    13.5kB ± 1%  -43.42%  (p=0.000 n=10+9)
HandleManyBannedPhrases-4    68.4kB ± 0%    49.5kB ± 1%  -27.61%  (p=0.000 n=10+10)

name                       old allocs/op  new allocs/op  delta
HandleNop-4                     361 ± 0%       161 ± 0%  -55.40%  (p=0.000 n=10+9)
HandleNopParallel-4             362 ± 0%       162 ± 0%  -55.25%  (p=0.000 n=10+10)
HandleCustomCommand-4           457 ± 0%       229 ± 0%  -49.89%  (p=0.000 n=10+10)
HandleMixed-4                   399 ± 0%       198 ± 0%  -50.30%  (p=0.000 n=9+10)
HandleManyBannedPhrases-4       694 ± 0%       496 ± 0%  -28.53%  (p=0.000 n=9+9)

Much better on the allocation front, but worse on the CPU time. I'll have to do some profiling to see where it's spending its time (the real fun). :slightly_smiling_face:
If you're curious in trying yourself, my repo doesn't require anything but a running docker daemon to work; it automatically handles everything, no external postgres servers to set up or anything.
In any case, this issue is resolved, given my code no longer fails! Many thanks.

I'll have to do some profiling to see where it's spending its time (the real fun). slightly_smiling_face

It is the statement cache. You can try to disable it and re-run benchmarks:
config, err := pgx.ParseConfig("postgres://postgres:@localhost:5432/test?sslmode=disable")
if err != nil {
	panic(err)
}
config.PreferSimpleProtocol = true

Your program is the rare case when statement cache is actually useful. The price you pay is that you need to write/generate queries manually.

It is the statement cache.

I assume you mean in pgx.

Your program is the rare case when statement cache is actually useful. The price you pay is that you need to write/generate queries manually.

Well, I guess I don't know how true this is when I'm just using SQLBoiler as it's intended. If I wanted to write queries by hand, I'd probably be using sqlc, which I'm betting would behave almost identically, as it too encodes down directly to strings passed to things like ExecContext and QueryContext without using statements directly.

I assume you mean in pgx.

Yes, PreferSimpleProtocol disables it.

as it too encodes down directly to strings

The difference is between
db.Exec("INSERT INTO foo VALUES ($1, $2)", "foo", "bar") // benefits from prepared statements
db.Exec("INSERT INTO foo VALUES ('foo', 'bar')") // does not

Bun generates the second variant because it is easier and somewhat faster because you don't need to allocate args []interface{}{"foo", "bar"} (it requires 3 to 5 allocations).


Change Latex equation editor to modal

Description
As part of #494 a direction was decided on for enabling inline equations that included moving the equation editor into a modal, so that we can support both inline and block equations.
Acceptance Criteria

When adding an inline equation into a rich text editor, when I click on the equation button a modal appears for me to configure my latex equation and see a preview of that equation
After adding an inline equation, I can click on that equation in the text editor to reopen the text modal
When I edit an equation in the modal, I can save changes to that equation

@oliverroick let me know if this is what you had in mind for this ticket!


Filtering Entity Linking candidates based on sentence similarity

Hi, thanks for developing such an awesome library. I wanted to know if there is way to generate candidates based on the context information. In certain cases, the distance seems to be same for all possible candidates where the context may help describing the nearest candidate. For Example-

for x in nlp("date of birth: sex: male service: micu and then ...").ents[1]._.kb_ents:
    print(linker.kb.cui_to_entity[x[0]],x[1],sep="\n\n\n")
    print()

Gives output:
CUI: C0009253, Name: Coitus
Definition: The sexual union of a male and a female, a term used for human only.

 Would be great if you could share your inputs on this!
Thanks in Advance!
Vasu

This is definitely a limitation of the current approach and we would love to have a more robust entity linker that does exactly what you describe. Take in the context of the text and the definition (and maybe type and aliases) of the candidate entities and predict which one is correct. I don't know if a simple sentence similarity would be enough or not, but is something you could evaluate on the medmentions dataset, which is how we evaluated the current entity linker.
The distance is the same for all of these because they all have an alias that is exactly sex.


data:amandamartin-dev

Fixes Issue

Changes proposed

Check List (Check all the applicable boxes)

[x] My code follows the code style of this project.
[ ] My change requires changes to the documentation.
[ ] I have updated the documentation accordingly.
[ ] All new and existing tests passed.
[x] This PR does not contain plagiarized content.
[x] The title of my pull request is a short description of the requested changes.

Screenshots

Note to reviewers

Oh weird. I'll take a look soon. Thanks!

No problem, any questions let me know - maybe your fork was up to date when you created your branch

everything should be good now, let me know!


remove reference to Avro

It seems that avro is no longer used, so this reference didn't really make sense.

+1

+1

any updates?