Session cache is cleared to often

In OGM Release 3.2.15 there was a change that the session cache is cleared when a potential write query is executed (ExecuteQueryDelegates#query line 125). I use stored procedures to read data from neo4j and now the session cache is cleared after each call of the stored procedure. Which can lead to performance degeneration.
Expected Behavior

Clear the session cache only if necessary. Maybe it is possible to get the statistics back from the database as in the Neo4j Browser and use this information to decide wheter the cache must be cleared or not.

Current Behavior

Cache is cleared on every call of a stored procedure, even if it a stored procedure that only reads data.
Context

As I use stored procedures to read data from neo4j the session cache is now more or less useless for me because it is cleared to often.
Your Environment

Java version:
Neo4j version (The version of the database): 3.5.21
OGM Version (The version of this software): 3.2.15
OGM Transport used (One of Bolt, HTTP or embedded): Bolt
Neo4j Java Driver version (in case of Bolt transport): 4.1.1
Link to your project: ./.

Steps to Reproduce

Call a stored procedure in a read-only transaction
Session cache is cleared after each call

This is something we are not gonna role back. It has to much potential to lead to inconsistent data: https://jira.spring.io/browse/DATAGRAPH-1249.
I would rather introduce an additional overload for public <T> Iterable<T> query(Class<T> type, String cypher, Map<String, ?> parameters with a boolean to indicate you want read or write calls.
Would that work for you?

The call is not directly via OGM. I have a SDN respository with a @Query annotation that calls the stored procedure, so the parameter does not help me.
I understand that you need to clear the cache and totally agree that it otherwise can lead to inconsistent data.
Is it an option to check the transaction mode and clear the cache if it is a read-write transaction and keep it if its a read-only transaction?

I see. No, in that specific use case (custom query for known entity classes) there's no propagation.

About how many custom queries are we talking? Would you be able to add a hint to them?
I'm thinking about something like this CALL /*+ OGM READ_ONLY */ dbms.functions() yield name.
I would rather not change the current SDN code to do this on some kind of heuristics.
The code path that is taken when you ask with. @Query for entities on SDN repositories doesn't allow to specify the mode… I would not only have to add an overload but also make sure SDN is aware which OGM version it uses (and we have already to many pieces of that).
I'll post a PR with the above functionality for you to try out. Looking forward to some feedback.

I have to check it, I found 80 call statements but some of them are write-statements. For sure, it is possible to add a hint to them.
I will try out your changes and provide a feedback. My first thought was "looks like xml processing instructions in the 'good' old days" ;-)

Hehe… :) I was like "wow, now I'm adding query hints like I used to with Oracle."

Ok, lets go on with the "like Oracle" approach ;-)
It works for me! I noticed that in line 127 there is also a session.clear() without the new check, maybe you should add it there. And also it is missing in validateQuery. Maybe the check should go to the method mayBeReadWrite?
Also I tried to integrate the new check in the existing regular expression.  I really like that approach (actually much better than mine) and I would incorporate it before our next release.
Next drink is on us (didn't you wanna do a meetup if I remember correctly?=

Yes, we already hosted the JUG Paderborn Meetup last year when @meistermeier jumps in because of your travel problem. Maybe we can host another one when it is possible again, maybe we can talk to each other in October after my vacation.
Looking forward for the next release and thank you for your great work! Without Neo4j OGM and SDN our work would not be possible or much more complicated.

Thanks for your kind words. I have added your tests now + some multi line queries. I cannot get a regex ready to cater for those, but I moved the check for the hint into the method itself, very valid suggestions. Thanks!


Remove 36MB of binaryen source code from wasm-opt-cxx-sys crate

$ wget https://static.crates.io/crates/wasm-opt-cxx-sys/wasm-opt-cxx-sys-0.110.0.crate
$ tar xf wasm-opt-cxx-sys-0.110.0.crate
$ cd wasm-opt-cxx-sys-0.110.0/
$ du -sh
36M

That seems like an excessive amount of source code to duplicate into wasm-opt-sys + wasm-opt-cxx-sys.
This PR uses some of the functionality of "cxx-build as a build system" (https://cxx.rs/build/cargo.html) to let wasm-opt-cxx-sys include headers directly from wasm-opt-sys's source tree.

As a followup, you might consider dropping binaryen's test suite out of wasm-opt-sys. It's big and hopefully not necessary for what wasm-opt needs.
$ du -sh binaryen/test/
26M

Rebased to resolve conflict with aad563327f248190a17ad48de8e0a3f667b76597 in components/wasm-opt-sys/build.rs

This looks fantastic. I am going to need to take a little longer to read about cxx-build and understand what is happening, but I will merge this.
I filed https://github.com/brson/wasm-opt-rs/issues/98 to delete the binaryen test suite from the package. Great idea. I'll do it after I merge this.


Package name for consul is not updated with documentation

Hi,
The reference to Orleans support for Consul, in the section Microsoft Orleans Consul Utilities, is not updated to the more recent package Microsoft.Orleans.Clustering.Consul
Thanks

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

ID: 9c74d870-b916-db65-ebcb-9aa5192637be
Version Independent ID: 6dbeba49-c940-ef67-ef98-ccbd7c0b958f
Content: Orleans NuGet packages
Content Source: docs/orleans/resources/nuget-packages.md
Product: dotnet-orleans
GitHub Login: @IEvangelist
Microsoft Alias: dapine

Thank you for posting this issue @jan-johansson-mr, however; the package you referenced is still in preview. We try to only document stable versions, so as soon as version 4.0 of Orleans ships we will update this. I'm going to close this though for now, thanks again.


Reason for dedicated Slack user

I would like to ask if there's any particular reason why it's stated that a dedicated user (and not a bot user) should be created to use matterbridge with Slack (https://github.com/42wim/matterbridge/wiki/Slack-bot-setup). I have tested it and I'm currently using matterbridge with a Slack bot user API token (https://api.slack.com/bot-users) with no issues at all. As stated here https://api.slack.com/custom-integrations/legacy-tokens, test tokens should also ideally not be used in applications. It's also much more troublesome to create a new Slack user as compared to setting up a bot integration.
Just trying to help you improve on the documentation 🙂

@iojw thanks for your input
The bot user can indeed be used, but you need to invite it first, it can't join channels.
I didn't want to confuse users with multiple options, even with single options they still have issues ;-)
(and also lack of time for writing extra documentation)
Please feel free to add a new page to the wiki describing this setup, it would be much appreciated!
Or modify other pages to make them more clear.

Ah totally get you there 🙂. I'll help you edit the wiki the include this soon!

I read this after setting up the user account...

Just found an issue with using a bot account. In handleSlackClient during the handling of the IncomingEvents loop, the logic makes a call to GetBotInfo (slack.go:293) that in turn makes a call to the bots.info resource on the Slack servers. This method can only be called by a non-bot user though and so it returns an error and the entire message event is skipped over resulting in messages not being sent over the bridge.

@jheiselman aha, thanks for reporting, so with a bot account we can't relay other bots.

@42wim I'm wondering if this call is strictly necessary. We already have the BotID that can be filled into the config.Message.UserID field. Not sure what would be considered a good value to fill in to the Username field, but it seems like this shouldn't result in the event getting dropped.

Yes, the problem is the username, we need a username of the bot. I don't know any other way to get it.

Looking at the use case of a user executing /giphy in Slack, there is a User property on the event that is a user ID. Could use it to call into GetUserInfo and use that information. Maybe postpend to the user name a string indicating it was a bot like user.Name + " (bot)". I have no idea how that would work for other bots.
In one of my Slack channels we have a Hangouts Bot that is bridging a particular Hangout to a channel. I will keep an eye out for those and see how those come across. It may be different because it isn't initiated by a Slack user.

@42wim I think we can close this one with my work concluding on #510 ?

Closing this in favour of #510.


Update new GA criteria and rename GA to Stable in CN docs

Hello,
Please reference the changes documented in https://github.com/dapr/docs/issues/1879 to make the same changes in the Dapr CN docs. Below are the PRs you can reference for the changes that were made to accommodate the new Stable lable and criteria that have replaced the previously named GA as of Dapr release v1.5
Files needing update:
daprdocs/content/en/operations/components/certification-lifecycle.md
daprdocs/content/en/operations/support/support-versioning.md
daprdocs/content/en/reference/*

PRs for reference:

https://github.com/dapr/docs/pull/1880
https://github.com/dapr/docs/pull/1904

yeah, will do


PR: Fix error when printing source code

Fixes #4824

@dalthviz, the test you introduced is hanging on Travis. Please take a look at that.


admin-ccs user role

Use postgres Enum for user roles
Check that user supplier_id is set (only) with the supplier role
Since the previous commit replaced check constraint with an enum,
there's no database check that supplier_id is set for users with
'supplier' role.
Instead, we add a check on user create/update that supplier_id is
always set only for 'supplier' users.
Add admin-ccs user role
#101410132
New role will be used for read-only access to the MI page on the
admin app.
Add a script to change user password
Takes user email, environment name, API token and new user password
and sends a user password update request. Can optionally unlock users
after changing the password.
Password is read using getpass, so there's no input echoing.

:+1:


Typo in AML registries documentation

There's a misleading typo in the documentation of AML registries. For example, in section https://learn.microsoft.com/en-us/azure/machine-learning/how-to-share-models-pipelines-across-workspaces-with-registries?view=azureml-api-2&tabs=cli the example states the jobs.train_job.component value should be azureml://registries/<registry-name>/component/train_linear_regression_model/versions/1 (note the /component/), but when I checked the Assed ID of a component in my own registry, the format was azureml://registries/<registry>/components/<component-name>/versions/<version> with /components/ instead of /component/.

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

ID: c51f8710-600c-f87e-d5d1-134ac70efecf
Version Independent ID: 3131da88-f396-ad16-e08e-b44681f852d8
Content: Share models, components, and environments across workspaces with registries - Azure Machine Learning
Content Source: articles/machine-learning/how-to-share-models-pipelines-across-workspaces-with-registries.md
Service: machine-learning
Sub-service: mlops
GitHub Login: @fkriti
Microsoft Alias: kritifaujdar

@bix-1
Thanks for your feedback! We will investigate and update as appropriate.

@bix-1
Thank you for bringing this to our attention! We've initiated a pull request (PR) to address this issue. Once the author reviews the changes, they will be merged, and the updates should go live shortly. Your diligence is much appreciated!


⚠ ClamAV has degraded performance

In a07b731, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 72 ms

Resolved: ClamAV performance has improved in 1709ecd after 5 minutes.


[Bug] Unsound invariant verification when using module with generic type parameters.

🐛 Bug
This is a problem I ran into when trying to prove an important property of Coin1.move.  I managed to verify the property, then remembered that it couldn't possibly be true.
I think this is a result of violating some assumption that we made in our soundness argument. Maybe it is a problem of misapplying schemas.  Perhaps the use case cannot be made to work.  I could definitely use some help figuring it out.
The test case inherently requires two files, so I wasn't able to make it work in the move-prover regressions.
Explanation
There is a generic module with functions to wrap a type parameter and publish the wrapped type, and another to remove, unpack the wrapped type, and return the payload.
There is a specific module that defines a specific resource, which is published by calling the function in the generic module.
I "proved" the property that the type defined in the SpecificModule can only be at address 0x123, where it is published in that module.  But another module could call GenericModule::remove(0x123) and do whatever it wants with it, so the property does not hold.
To reproduce
Code snippet to reproduce
## There are two modules.
## SpecificModule.move
address 0x1 {

module GenericModule {

    resource struct Wrapper<T> { payload: T }

    public fun publish_it<T>(account: &signer, payload: T) {
        move_to(account, Wrapper<T> { payload } );
    }

    public fun remove_it<T>(addr: address): T acquires Wrapper {
        let Wrapper<T> { payload } = move_from<Wrapper<T>>(addr);
        payload
    }

}
}

generic module
address 0x1 {

module GenericModule {

    resource struct Wrapper<T> { payload: T }

    public fun publish_it<T>(account: &signer, payload: T) {
        move_to(account, Wrapper<T> { payload } );
    }

    public fun remove_it<T>(addr: address): T acquires Wrapper {
        let Wrapper<T> { payload } = move_from<Wrapper<T>>(addr);
        payload
    }

}
}

Stack trace/error message
// Paste the output here

Expected Behavior

System information
Please complete the following information:

Additional context
Add any other context about the problem here.

Keeping open even this issue is resolved, there are others for invariants over generic types.

This should be fixed with monomorphization, but we need to verify.


Invalid query: Invalid orderBy property normalizedLabel. Should be used in a range "where" query

on v.22 the query:
const queryOpts = {
        where: [
          ["normalizedParentDomainName", "==", "dash"],
        ],
        orderBy: [["normalizedLabel", "asc"]],
      };

throws
INVALID_ARGUMENT: Invalid query: Invalid orderBy property normalizedLabel. Should be used in a range "where" query

instead of retrieving the sorted documents.

Yes, you can't order by field which is not part of where


Update USA and GBR SKU's for 3090

The builds failing please see the error log in details and fix then I'll merge this

@ianmarmour no problem and thanks for all the work on this project


Feature Request: target using small element

When the small element is selected I'd like to be able to target the represented token instead of the one at the top of the pile.

I am not sure what you are asking for.
right click to bring up the hud, then click the target

I think he means hvoering the miniature token and press "t" for target, which I +1

Unfortunately, I cannot make it behave this way both because of how core keybinds work and how events are processed on html elements. The right click for HUD is the best I can do.


updater: add type specification to methods

updater: add type specification to methods

Completed and soon to be merged from PR #146 !


Added example for :class:~.Sphere

Changelog / Overview

Added documentation for the sphere class. Added an example since there were no existing examples for the sphere class

Motivation

Added example for new users
Explanation for Changes

Makes it more readable as the examples are useful to the newer users

Documentation Reference

Testing Status

Further Comments

Checklist

[x] I have read the Contributing Guidelines
[ ] I have written a descriptive PR title (see top of PR template for examples)
[x] I have written a changelog entry for the PR or deem it unnecessary
[ ] My new functions/classes either have a docstring or are private
[ ] My new functions/classes have tests added and (optional) examples in the docs
[x] My new documentation builds, looks correctly formatted, and adds no additional build warnings

Reviewer Checklist

[ ] The PR title is descriptive enough
[x] The PR is labeled correctly
[ ] The changelog entry is completed if necessary
[ ] Newly added functions/classes either have a docstring or are private
[ ] Newly added functions/classes have tests added and (optional) examples in the docs
[x] Newly added documentation builds, looks correctly formatted, and adds no additional build warnings

Courtesy link

Hi @jojo96, thank you for your contribution! Personally I don't like the look of the spheres overlapping, could you change that perhaps? It would be nice to have different colors as well.

Thanks for the contribution!
Two suggestions:
A) I also think it is better when the spheres do not overlap.
B) I would name them sphere1, sphere2, sphere3

Hi @jojo96, thank you for your contribution! Personally I don't like the look of the spheres overlapping, could you change that perhaps? It would be nice to have different colors as well.

Thanks for the contribution!
Two suggestions:
A) I also think it is better when the spheres do not overlap.
B) I would name them sphere1, sphere2, sphere3

Hi I have created spheres with better resolutions and changed colors as well. Also, implemented A and B as suggested. Added these to patch-2.
Do I need to create a new pull request? @kolibril13

Great!
No, you don't have to recreate a pr.
You just have to wait until this test is passing,

and the new example will be available here:
https://manimce--1627.org.readthedocs.build/en/1627/reference/manim.mobject.three_dimensions.Sphere.html?highlight=sphere#manim.mobject.three_dimensions.Sphere

Left one change request. @hydrobeam's comment regarding the resolution (which has a rather large impact on the render time in our documentation) also seems to have not been addressed, could you please take a look again?
Thank you!

@behackl reduced the resolution of the spheres to (20,20), (18,18), (16,16). Reducing the resolution further does not render good looking spheres.


[KeyVault-Certificates] Poller public state implementation

This PR contains the implementation of a poller with a bag of public
properties. I believe the names I picked are decent, but I'll change
them if another name is suggested.
Please let me know if the current approach is not desireable too.
This also fixes the last remaining task from #6567 :

getCertificateOperation should return a certificate with policy at
the end.

Therefore:
Fixes #6567

Closed on favor of https://github.com/Azure/azure-sdk-for-js/pull/6644


Improvements to visible whitespace in console output.

When whitespace is made visible in the --check format, it can get noisy.
Can it be changed to only show the whitespace at the beginning and end of the line?
Maybe only show the first couple of differences?
Maybe it is possible to use colors, and make the whitespace character blend into the background more?

Expanding on the first couple of differences idea - on the line that has the first difference, make whitespace visible from that point until the end of that line, but not visible in any other lines.

After testing this out, it didn't really make sense to have whitespace visible except in two cases.
An otherwise empty line has whitespace
----------------------------- Expected: Around Line 4 -----------------------------
    private string field1;

    private string field2;
----------------------------- Actual: Around Line 4 -----------------------------
    private string field1;
····
    private string field2;

A line has extra whitespace at the end.
----------------------------- Expected: Around Line 3 -----------------------------
{
    private string field1;
}
----------------------------- Actual: Around Line 3 -----------------------------
{
    private string field1;····
}

If there is extra whitespace between words, it is obvious without making the whitespace visible.
----------------------------- Expected: Around Line 1 -----------------------------
public class ClassName { }
----------------------------- Actual: Around Line 1 -----------------------------
public class ClassName  { }


Configure the test runner and coverage reports

Linked issues

🏷 Closes/Fixes/Resolves: N/A
Description

Pytest-based test runner and coverage reports generated by Coverage.py are comprehensively set up, for both local development and CI.
The coverage reports are set up to uploaded to Codecov.io.
Django-specific testing features related are handled by the pytest-django plugin.

Welcome to Codecov :tada:
Once merged to your default branch, Codecov will compare your coverage reports and display the results in this comment.
Thanks for integrating Codecov - We've got you covered :open_umbrella:


Add Extensions Verifier for GeoPackage

4b7ce1d

Commits:
4b7ce1d
baee47b
80717c9

Commits:
4b7ce1d
baee47b
80717c9

Still need to implement Requirement 79 but that will be saved for a later date


checkout should not error on checking out current branch

To match Git we should not return a non-zero exit code when checkout is called with the current branch as an argument:
$ git checkout master
Already on 'master'
Your branch is up to date with 'origin/master'.

Duplicate of #836


Add haproxy image

depend on https://github.com/wolfi-dev/os/pull/417
fixes: https://github.com/chainguard-images/images/issues/133


Backport "Update MiMa and Reference Version to 3.5.0" to 3.5.1

Backports #21374 to 3.5.1-RC2

MiMa failure will be solved by the following backported commit https://github.com/scala/scala3/pull/21372/commits/69ca72b8d5de456ea418f34a7d5b9fe3317da657


update travis to latest node/firefox

Travis is timing out for some reason, and I don't have time to deal with this.


clicking branch link in app and going back

Here's the problem we face.
In viewController1, there's a webview and a branch link (bnc.lt/...)
When a user clicks the link, we open up a viewController2 and load the url.
When he/she clicks a back button, we check if we can go back with webview.canGoBack and goes back.
With normal link it works fine, but with branch link we are facing a problem.
It is as if there's an intermediate url we visit which redirects us to the original url.
So hitting a back button goes to the intermediate url and it redirects the user to the starting url.
www.example.com => bnc.lt/1
when clicking bnc.lt/1 user goes to some intermediate url then gets redirected to www.example.com .
So if he tries to go back, he goes to the intermediate url and gets redirected to www.example.com again.
I'd like to know if it's indeed the reason why we are not being able to go back.
And wonder if there's a recommended fix?

The Branch link uses the intermediate redirect to check if any Branch 'magic' needs to happen with the link before going to ultimate link destination.
Probably the quickest way for you to fix this to check in the delegate if the user is going back from a Branch link and simply go back twice.  This seems like kind of a hack though.
Let me think about a better solution.  What kind of web view are you using?

Did this solution work for you?
What did you decide to do for a work around?

See the WebViewExample in the 'Examples' directory to see how to do this.


use MAP_PRIVATE for mmapped file

/kind improvement
hard learned lesson on mmap: MAP_PRIVATE|MAP_SHARED mean more than just access control, they also have different anonymous memory usage pattern implication.

/lgtm


Message broker monitoring

Starting this issue but @DeeZone will add more details.
Per a convo w/ me, @sheyd and @DeeZone, Sena said he could spin up a monitoring solution in the next day or two that would ping us when the message queue is backed up.
cc @mikefantini @mshmsh5000

@sheyd Looks like some server setup will be needed to get the RabbotMQ command line tools working: https://www.rabbitmq.com/management-cli.html

@sheyd @marahml Added the top queues, their descriptions and how they need to be monitored.

@sheyd just checking in on status of this. is it something we see moving to next sprint? cc @mikefantini

@marahml Digging into this now. I think I should be able to get this in this week.

All the good notifications baked in here:
https://jenkins.dosomething.org/view/All/job/Rabbit MQ Monitoring/
@mshmsh5000 So far only notifying @DeeZone until we can get the kinks worked out. After we're satisfied with the thresholds, I'll add the rest of us.


Swift 5 Support

Check all is well on Swift 5 after the final release of it ships with Xcode 10.2 and if any changes are needed. Review the changes in recent pull requests by @ZevEisenberg and @ferranpujolcamins .
Development for Swift 5/SwiftGraph 2.1 is taking place in the release2.1/swift5 branch.

[ ] Add xcode 10.2 and swift 5 to CI config

@ferranpujolcamins I updated a bunch of the code in UniqueElementGraph that was not building in Swift 5 and now several of the tests for Union and UniqueElementsGraph are failing. I don't know if I broke something or if it was already broken. Can you take a look at it? I'm not very familiar with these tests/code. Thanks in advance.

I'm having a look at the original issues. At first glance they seem a regression in the swift compiler.

I still think it's an Xcode bug. Can you:

Roll back the release branch to ae1a8f3f268ef84b7f30738921b6aa5ed6dfe009
Update the project to Swift 5 without any code change
Commit to the release branch
Open a bug to the radar pointing to the release branch, also ask for a temporary workaround because otherwise we cannot ship until Xcode 10.3. Remember to note that the project builds fine with swift cli.

I was wrong, from the cli I get the same error (I was still using swift 4). So maybe it's not an Xcode bug.

So, it seems there's a bug related to convenience initializer and associated types. I filed it here.
A workaround can be to turn the initializers into static factory methods.

Turning these initializers into static factory methods can be a workaround.

Yes, I can confirm this fixes the problem


Some Help with the lstm_unroll Model in lstm.py?

I am reading the code in example/rnn/lstm.py. Could someone please explain what the last few lines are doing? Are we some how reshape label, which is a symbol here? Many thanks!
**################################################################################
# Make label the same shape as our produced data path
# I did not observe big speed difference between the following two ways

label = mx.sym.transpose(data=label)
label = mx.sym.Reshape(data=label, target_shape=(0,))

#label_slice = mx.sym.SliceChannel(data=label, num_outputs=seq_len)
#label = [label_slice[t] for t in range(seq_len)]
#label = mx.sym.Concat(*label, dim=0)
#label = mx.sym.Reshape(data=label, target_shape=(0,))
################################################################################**

@KCHWang The code is trying to squeeze the labels into a single vector.

@sxjscience Thanks! I understand that is what label = mx.sym.Reshape(data=label, target_shape=(0,)) is doing. But what about label = mx.sym.transpose(data=label)? I am confused about the transpose function. Any hint please?

Sequential Consistency,  保持预测结果与标签顺序一致,

@KCHWang I met a same problem with you when I run lstmocr.py with my own data(number=18), occur following error:
 Traceback(msot recent call last):
    File "lstm_ocr.py",line 165,in <module>
     batch_end_callback=mx.callback.Speedometer(BATCHSIZE,50))
   File "../../python/mxnet/model.py",line 745,in fit
     self._init_params(dict(data.provide_data+data.provide_label))
   File "../..python/mxnet/model.py",line 486,in _init_params
     assert(arg_shapes is not None)
AssertionError

can you show me the iterator that you do yourself? I have no idea to deal with my data, thank you!

I have the same question. Do you have a solution? How to train my own data in the lstm?


Moving crowding distance

Pull Request Description
What issue does this change request address? (Use "#" before the issue to link it, i.e., #42.)
Move crowding distance method from Hybrid model to math utils
closes #1586
What are the significant changes in functionality due to this change request?

For Change Control Board: Change Request Review
The following review must be completed by an authorized member of the Change Control Board.

[ ] 1. Review all computer code.
[ ] 2. If any changes occur to the input syntax, there must be an accompanying change to the user manual and xsd schema. If the input syntax change deprecates existing input files, a conversion script needs to be added (see Conversion Scripts).
[ ] 3. Make sure the Python code and commenting standards are respected (camelBack, etc.) - See on the wiki for details.
[ ] 4. Automated Tests should pass, including run_tests, pylint, manual building and xsd tests. If there are changes to Simulation.py or JobHandler.py the qsub tests must pass.
[ ] 5. If significant functionality is added, there must  be tests added to check this. Tests should cover all possible options.  Multiple short tests are preferred over one large test. If new development on the internal JobHandler parallel system is performed, a cluster test must be added setting, in  XML block, the node <internalParallel> to True.
[ ] 6. If the change modifies or adds a requirement or a requirement based test case, the Change Control Board's Chair or designee also needs to approve the change.  The requirements and the requirements test shall be in sync.
[ ] 7. The merge request must reference an issue.  If the issue is closed, the issue close checklist shall be done.
[ ] 8. If an analytic test is changed/added is the the analytic documentation updated/added?
[ ] 9. If any test used as a basis for documentation examples (currently found in raven/tests/framework/user_guide and raven/docs/workshop) have been changed, the associated documentation must be reviewed and assured the text matches the example.

Job Test CentOS 7 on ca6ef57 : invalidated by @wangcj05
connection issue

All jobs on ca6ef57 : invalidated by @wangcj05
network connection issue due to network maintenance

All jobs on 0b949d9 : invalidated by @wangcj05
test machine network connection

Job Test Fedora 31 on 0b949d9 : invalidated by @wangcj05
fetch failed

Job Test Fedora 31 on ca6ef57 : invalidated by @joshua-cogliati-inl
Failed with ssh: connect to host github.com port 22: Connection timed out

Job Test Fedora 32 on ca6ef57 : invalidated by @joshua-cogliati-inl
Failed with ssh: connect to host github.com port 22: Connection timed out

Job Mingw Test on 0b949d9 : invalidated by @joshua-cogliati-inl
restarted civet

Job Test Fedora 32 on 0b949d9 : invalidated by @joshua-cogliati-inl
Failed with ssh: connect to host github.com port 22: Connection timed out

Job Test Fedora 32 on 0b949d9 : invalidated by @joshua-cogliati-inl
Failed with ssh: connect to host github.com port 22: Connection timed out

Job Test qsubs on 0b949d9 : invalidated by @mandd

Job Test Ubuntu 16 on eea5238 : invalidated by @joshua-cogliati-inl
test civet

Job Test Fedora 32 on eea5238 : invalidated by @joshua-cogliati-inl
test civet

@mandd @Jimmy-INL Is it ready for review?

Job Test Fedora 32 on 11eca4a : invalidated by @wangcj05

checklist is good. Currently Fedora 32 has problem with C++ compiler, and is allowed to fail .


Move block store CRUD functionality from the node store component in casper-storage.

Moved the basic CRUD block store operations to casper-storage. This allows other crates to interact with the node storage databases without the need to go through the storage component of the node.
The block_store module in casper-storage now provides traits to interact with a casper block store. It also provides concrete implementations for using LMDB as the backing store and also allows accessing indexed data.
Some types that were defined in the node have moved to casper-types and casper-storage as a result of this implementation.
Next steps:

[x] now most of the methods that allow writing to the block store always overwrite data. Introduce an option to disable overwrite. After the latest discussion, we decided that overwriting will be implementation specific and is left up to the implementer of DataWriter<K, T>
[x] wire up the implementation in the node storage component as a replacement.
[x] refactor tests to use the new block store from casper-storage

bors merge


Relay-like @plural directive

Discussed in https://github.com/HoudiniGraphql/houdini/discussions/239

Originally posted by mhzokaii March 22, 2022
Hi!
First of all, your work is really great, congrats.
The @relay (plural: true) directive defined in Relay, is very helpful in reducing the amount of work required for passing the data down to the sub-components. I couldn't find a similar way in Houdini. Am I missing something? Is it planned to be implemented? Or even maybe you see it as an anti-pattern? Should the @list directive be used (and if yes, how can it be used in fragment definitions)?

Hey,
as I've run into the same use case (and seem to be unable to figure out a solution where I basically have a SomeList Svelte component which is supposed to take an array of the type created by the fragment and handle individual items further), I thought I'd give it a shot. However, I can't figure out how to start the example project. yarn generate fails every time:
/tmp/houdini/example $ yarn generate
Usage Error: Couldn't find the node_modules state file - running an install might help (findPackageLocation)

$ yarn run [--inspect] [--inspect-brk] [-T,--top-level] [-B,--binaries-only] <scriptName> ...

I've done the yarn && yarn build before on the top-level dir. Any ideas what goes wrong there?
Running an install doesn't help as it will install houdini 0.15.5 into the node_modules folder of the example instead of using the changed code I'm trying to play with.
yarn is version 3.2.0, node is 18.7.0, just in case that's relevant.

👋 hey there! so i'll be honest, I haven't run the example application in awhile since I've been using the integration tests as my sandbox. Are you able to run that project? It's a sveltekit project so yarn dev should be all it takes.
You could also just let it install the old version, delete it, and replace it with a symlink to your local directory

The symlink approach fails on yarn generate with node being unable to load the graphql module in cmd.js. However, copying the directory worked for a preliminary test.
Now I'm thinking about the correct way to design the interface. I'd assumed something like this:
export function fragment<_Fragment extends Fragment<any>>(
   fragment: GraphQLTagResult,
   ref: _Fragment[] | null
): Result<_Fragment>[]

A single fragment (with the @plural directive) with an array of references which should result a set of results. Would that be in the right direction? If yes: I'd be happy about a few pointers on how to implement the actual data access.


{QUESTION} GLOW XQZ

glow types 1, 2, 3 dont work behind the wall.
fix this plz

This is not a question but an actual issue.

the glow types are managed by the csgo's built-in GlowManager. this cannot be fixed without lots of patching (even still not really possible).


Images/thumbs

Thanx for this little script. I use it on a new project, but some of the youtube videos does not display the thumb image, but a grey youtube thumb (like video not found). The video exist. Is there a way to fix it?

Is rare what happens to you, but I think that's no problem of the script, but the Youtube itself does not send the picture.
Sorry my bad english and my late response.
Did you solve the problem?

the problem still exist. it happens as example @ 2P8TQ9Wb1PY there a more items like this.

Hmm very rare, because when I go to http://img.youtube.com/vi/2P8TQ9Wb1PY/0.jpg the image is there, and that image is what the plugin uses...
I will try to solve this problem.


browser-sync-client js not found

I am unable to refresh the browsers programatically through gulp as I get the following error in my console when browser sync runs
Failed to load resource: the server responded with a status of 404 (Not Found)
http://localhost:1337/browser-sync/browser-sync-client.1.5.7.js
Please help, I am using browser-sync v1.8.3
(I even did rm -rf node_modules, npm cache clean and verified that 1.8.3 is installed locally)

Can you run
browser-sync --version

and then
node node_modules/browser-sync/bin/browser-sync --version

and post both results please :)

Hey, seems like it might have been some sort of browser-side caching issue, I can no longer reproduce. Sorry about that!

I'm experiencing this issue and don't know how to fix ... I've opened a new browser window (with a new profile) and still get the wrong version of browser-sync-client trying to load. Any suggestions?

Oops - clearing the browser cache specifically fixed the issue. :+1:

Had the same issue, just cleared chrome's cache and worked :)
Thanks, joelataylor


As a user I should be able to view query cost when querying a Cosmos db

Like in Azure Portal I should be able to view the cost of a Cosmos query.

CosmosDB has been removed from Storage Explorer.


Cleanup temporary files used by doctests

Probably fix with a testcleanup directive.
These files were added to the gitignore in #630.

Fixed in https://github.com/scverse/anndata/pull/1061


Update readme

Issue #:
#8
Description of changes:

Update README to use Lex V1 console.

Testing

How did you test these changes? Just an README update.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

What is the difference between V1 and V2? Should we update the demo to use V2 in the future :)

What is the difference between V1 and V2? Should we update the demo to use V2 in the future :)

Its the UI versions from the AWS Management console for Lex (the bot service). Yes, I agree we should update it to use Lex V2 but I have to plan and update the demo as a whole as it has been not tested and updated in a while.


Error occured when data processing

Dear authors,
Hi! When running train.py, at the data-processing stage, an error occured. When train.py at line 26 called self.data = torch.load(self.save_path), it get into the file bpe_dataset.py properly. But at line 26, an error occured saying No such file or directory for the file processed_data.pkl. As there is a try-catch structure, it runs line 28, self.data = self.process(). But when running line 28, another error occurs, saying

Boost.Python.ArgumentError: Python argument types in
rdkit.Chem.rdmolops.RemoveStereochemistry(NoneType)
did not match C++ signature:
RemoveStereochemistry(class RDKit::ROMol {lvalue} mol)

for the file chem_utils.py at line 69. Can you tell me how to fix these errors or how to avoid them? Thanks!

Hi, can you check the smiles that caused the error? The errors imply that RDKit failed to parse the given smiles into a molecule object at line 68 in chem_utils.py, which means the provided smiles might be valid.

So you are inferring that the problem may be illegitimate SMILES inputs. But due to the shuffling, the number of iterations is different for each stop. How should I check which SMILES input is causing this problem? Is it possible that the position where the iteration stops after I deactivate the shuffle is the position where the input error occurs? And I am training on the qm9 dataset you provided. I am not inclined to think that the dataset you provided has the potential for error.

The error occurs during the preprocessing phase, where there is no shuffling. Therefore, the position that the iteration stops should be exactly the line of the smiles that caused the trouble. For me on this case, I would first track down the position that the iteration stops (suppose the position is i). Then revise the code as follows:
for s in tqdm(smiles[i-1:]):
    print(s)
    mol = chem_utils.smiles2molecule(s, kekulize=True)
    data_list.append(self.process_step1(mol, self.tokenizer))

In this way, you should directly see the smiles that caused the trouble. Then you can validate my hypothesis by running the following codes in python:
from rdkit import Chem

s = 'the one that caused the trouble'
mol = Chem.MolFromSmiles(s)
assert mol is not None

Thanks for your help. I successfully located the smiles that caused the error. The screenshot is attached below. But how can I fix that?

Hi, I've added some codes to skip the smiles that cannot be handled by rdkit (see here and here). Now it should work.

Thanks for your help! Code works for the data loading part


bump version to v0.11.1

0.11.1 should hopefully fix #26

Checklist

[x] Used a fork of the feedstock to propose changes
[x] Reset the build number to 0 (if the version changed)
[x] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[x] Ensured the license file is being packaged.

@conda-forge-admin, please lint.

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.


[SPARK-48431][SQL] Do not forward predicates on collated columns to file readers

What changes were proposed in this pull request?
SPARK-47657 allows to push filters on collated columns to file sources that support it. If such filters are pushed to file sources, those file sources must not push those filters to the actual file readers (i.e. parquet or csv readers), because there is no guarantee that those support collations.
In this PR we are widening filters on collations to be AlwaysTrue when we translate filters for file sources.
Why are the changes needed?
Without this, no file source can implement filter pushdown
Does this PR introduce any user-facing change?
No
How was this patch tested?
Added unit tests. No component tests are possible because there is no file source with filter pushdown yet.
Was this patch authored or co-authored using generative AI tooling?
No

cc @stefankandic @cloud-fan

thanks, merging to master!


unable to renewal and bind to ISS with version <IP_ADDRESS>

Describe the bug
A clear and concise description of what the bug is.
Log
2022-05-25 11:20:20.007 -04:00 [INF] Arguments: --renew --baseuri https://acme.sectigo.com/v2/OV
2022-05-25 11:20:20.062 -04:00 [INF] Software version <IP_ADDRESS>7 (release, pluggable, standalone, 64-bit) started
2022-05-25 11:20:20.063 -04:00 [INF] Connecting to "https://acme.sectigo.com/v2/OV"...
2022-05-25 11:20:27.721 -04:00 [ERR] Initial connection failed, retrying with TLS 1.2 forced
System.Exception: Server returned status Forbidden:Forbidden
at PKISharp.WACS.Clients.Acme.AcmeClient.CheckNetworkResponse(HttpResponseMessage response)
at PKISharp.WACS.Clients.Acme.AcmeClient.CheckNetwork()
2022-05-25 11:20:35.616 -04:00 [ERR] Unable to connect to ACME server at "https://acme.sectigo.com/v2/OV"
System.Exception: Server returned status Forbidden:Forbidden
at PKISharp.WACS.Clients.Acme.AcmeClient.CheckNetworkResponse(HttpResponseMessage response)
at PKISharp.WACS.Clients.Acme.AcmeClient.CheckNetwork()
2022-05-25 11:20:35.693 -04:00 [INF] Scheduled task looks healthy
2022-05-25 11:20:35.693 -04:00 [INF] Please report issues at https://github.com/win-acme/win-acme
2022-05-25 11:20:36.295 -04:00 [INF] Renewing gccappwdiad2355.aws.etscloud.org
2022-05-25 11:20:36.310 -04:00 [WRN] Using cache for *****.org. To get a new certificate within 1 days, run with --force.
2022-05-25 11:20:36.386 -04:00 [INF] Store with CertificateStore...
2022-05-25 11:20:36.402 -04:00 [WRN] Certificate with thumbprint FCE4A568510049B3FB4C064493670703C681A142 is already in the store
2022-05-25 11:20:36.414 -04:00 [INF] Installing with IIS...
2022-05-25 11:20:36.519 -04:00 [INF] No bindings have been changed while updating site 2
Platform:

OS: Windows 2019
Version:  64-bit

I don't necessarily see a bug here. You renew shortly after the certificate has been requested, so you get the same one from local cache and therefor no changes to IIS are needed. Please provided more detailed information if you think you've actually found a bug.

for renewal test, we added 1 day in RenewalDays in Settings.Json
we installed certificate by passing ACME account details as below and cert was issued and updated IIS binding sucessfully. But the ACME account details are not storing in PROGRAMDATA/win-acme. Due to this behavior we are seeing error in logs as "[ERR] Unable to connect to ACME server"
.\wacs.exe --source manual             --host $wacsHost
--friendlyname $FQDN             --installation iis
--installationsiteid $IISSiteID             --sslport $SSLPort
--emailaddress $EMAIL             --certificatestore My
--accepttos              --eab-key-identifier $eabKeyIdentifier
--eab-key $eabKey

That's not what these logs are showing. No attempt was made to actually order a certificate yet because it was too soon after creation. You can use the --force switch to force a renewal.
This error shows a 403 status code returned from https://acme.sectigo.com/v2/OV, nothing else That may have been a transient error at the providers side or some firewall/proxy standing in the way. For now I'm going to assume that this is not a bug and convert to a Q&A discussion.


FEATURE IDEA/REQUEST: antsApplyTransforms without a reference image

The MINC family of tools offers an option with it's resample tool mincresample which states:
-tfm_input_sampling:       Transform the input sampling with the transform
This achieves resampling of an image by applying the transform to the bounding box/direction cosines so that no reference image is required.
My most immediate application of this is when averaging transforms where the final space may not exist since the average transform defines it. Transforming the input sampling avoids this issue.

Gabriel,
https://github.com/BRAINSia/BRAINSTools/blob/master/BRAINSCommonLib/itkResampleInPlaceImageFilter.h
I think we have this implemented in BRAINSTools,  it should be a trivial exercise to migrate to ANTs.
http://www.insight-journal.org/browse/publication/770
Hans
From: "Gabriel A. Devenyi"<EMAIL_ADDRESS>Reply-To: stnava/ANTs<EMAIL_ADDRESS>Date: Tuesday, February 7, 2017 at 12:48 PM
To: stnava/ANTs<EMAIL_ADDRESS>Cc: Subscribed<EMAIL_ADDRESS>Subject: [stnava/ANTs] FEATURE IDEA/REQUEST: antsApplyTransforms without a reference image (#408)
The MINC family of tools offers an option with it's resample tool mincresample which states:
-tfm_input_sampling: Transform the input sampling with the transform
This achieves resampling of an image by applying the transform to the bounding box/direction cosines so that no reference image is required.
My most immediate application of this is when averaging transforms where the final space may not exist since the average transform defines it. Transforming the input sampling avoids this issue.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.


add a new eta parameter "estimate" / "middle value" / "average"

What would you like to see added?
Right now the only possible eta values are slicer or file:
[telegram_ui]
#eta_source: slicer
#	You can choose, which value to use for remaining time estimation.
#	Default value is slicer.
#	Allowed values: slicer, file

Klipper can calculate a third eta value based on the middle value of filament and file. In my case, this is one of the most presicely eta calculation i can get. While slicer eta time shows me a value of 0:05:35, the average value of file (0:09:44) and filament (0:03:20) give me a eta value of 0:06:32 which is in fact accurate to the second.
Maybe it's a opportunity to add a third value filament to the moonraker-telegram-bot and let the user decide which numbers should be combined and determine the average value from this. Like, adding multiple values leads to a average calculation: eta_source: file, filament

Thank you for writing such a detailed feature request, it's so refreshing :)
What I am wondering though, I have not had time to check, are you sure it's a klipper thing?
The slicer and filament variables are pulled from moonraker, we do not touch them ourselves.
Is this "middle" thing a new addition which we missed? If it is, adding it should be easy. If it is not, I am not sure if we want to add it. We want to keep the amount of calculations/things we do ourselves to a necessary minimum, because the bot is not an active component of the ecosystem, but rather a displaying/controlling entity.

Incant tell you if this calculation is from klipper or moonraker. Maybe it's a mainsail related feature? I can't tell you that's In my setup, I am using klipper with mainsail. In mainsail you can go to settings and there is a section for eta. Mainsail ask on which base the eta should be calculated and you can choose between different sources (file, filament, slicer). At exactly this step you can also choose more than one source to calculate an average value.

see: https://github.com/mainsail-crew/mainsail/blob/develop/src/store/printer/getters.ts#L1013