[InputManager] Fix for double event handling with global listener.

Fixes https://github.com/Microsoft/HoloToolkit-Unity/issues/756: moved handling of Global Event Listeners to last.
Added check if listener was already added to the global list.

@StephenHodgson : Looks very good. I just tried it out and it worked like a charme.
Just one thing I noticed:
Let's say you have two or more cubes in your scene and all cubes should to something when I tap. Then I will place the "Cubes-do-stuff"-Script at each cube and also the SetGlobalListener-Script. All cubes perform the action when I do not gaze at any of them. But as soon as I gaze at one of the cubes only this cube will execute it's script as we never reach the GlobalListener-Execution in the InputManager.
I cannot think of a quick solution for this. Maybe we have to implement two different GlobalListeners? Or we can make a public bool at the GlobalListener to take this into account?

Let's say you have two or more cubes in your scene and all cubes should to something when I tap. Then I will place the "Cubes-do-stuff"-Script at each cube and also the SetGlobalListener-Script. All cubes perform the action when I do not gaze at any of them. But as soon as I gaze at one of the cubes only this cube will execute it's script as we never reach the GlobalListener-Execution in the InputManager.

In this case, I wouldn't implement any handlers on the cubes but on a parent object, with a global listener.

@HoloFan, See comments for AddGlobalListener

Adds a global listener that will receive all input events, regardless of which other game objects might have handled the event beforehand.

@HoloFan, take a look at this again. I've reverted the changes I made by moving the GlobalListener execution last, but instead just skips the execution of the global listener if it's already going to be handled later on.

I'm not sure I understand why we'd make this change. Issue #757 is a "by design" issue, in the sense that there isn't anything in InputManager that prevents you from registering a game object multiple times to handle input. You probably shouldn't do that and I think everything you need to avoid doing that already exists in InputManager (modal input, "regular" input and fallback input).
As a general rule, global listeners shouldn't be avoided. The main reason this concept was added was for the cursor, which needs to receive all inputs. I can't think of a lot of good examples where code would need to add itself a a global listener.

@maxouellet, would you like me to scrap these changes and only update with the event data usage?

@StephenHodgson Yes I think that makes sense. I have no problem with the piece of code that checks for double registration, and adding the event data usage stuff would be good.

Will do.


Formatting breaks the :require block!

This is a very serious issue for us, standard-clj doesn't seem to be handling the :rename form correctly.
It turns:
(ns hello
  (:require [other :refer :all :rename {foo bar}]))

into this:
(ns hello
  (:require
   [other :refer [bar foo]]))

Literally breaking the import and rendering the namespace unusable.

Thank you for the report! Fixed with PR-167

This fix is available with v0.17.0. Can you confirm that it works for your codebase @licht1stein ?

Will check as soon as I have a chance, thanks for the quick fix!

Seems to be find with version 0.18.0, thank you!


Riddle Counts Don't Appear for Games Other Than City

For Asylum, Origins, and Knight, the riddle count does not appear. This is technically not a bug because these simply are not implemented yet, but I consider it one.

Update: Asylum's has been implemented. Unclear if Origins has a usable value for this. Knight has some complications.


Initial creation of subdirectory for application-benchmarks subproject.

Replaces PR #974.
Just create initial subdirectory, preparatory to actually starting to put data and benchmarks into it.  Initial owners are jberkus and wojtek-t because other contributors don't have Kubernetes accounts yet.
If this is accepted, next steps are:
Updating /community/sig-scalability with subproject info
Meeting on slack to discuss first projects/information

I'd also like to potentially have a label added to this repo for kind/appbench so that we can tag issues/PRs for this project.
(Apologies for new PR; renaming the directory isn't compatible with squashing the commit)
/assign @wojtek-t

/lgtm
/approve
@jberkus - I added OWNERS_ALIASES file - you may need to rebase (I don't know how exactly the bot works).

/verify-owners

/verify-owners

@cblecker - we already have "OWNERS_ALIASES" file, do you know why it is not picking this up?

/verify-owners

@wojtek-t I opened https://github.com/kubernetes/test-infra/issues/16330 for this.
This would be corrected by @jberkus rebasing, but as it's not going to negatively impact anything, I'm going to manually remove the label and allow this to merge.

Thanks @cblecker !


[BUG] Hybrid Search results are missing when concurrent search is enabled

What is the bug?
Hybrid search results are missing when concurrent search is enabled in single shard and multiple shards scenario.
For example:
For single shard I ingested 6 documents and did a matchAll query under the hybrid clause then I am expecting 6 results should be there but I am getting just 5 results.
For multiple shard I did the same by ingesting 100 documents and I am expecting 100 results but getting 95.
How can one reproduce the bug?

Just ingest the documents for single shard by following URL  http://localhost:9200/my-nlp-index/_doc/1?refresh=true and make sure update docId when ingesting new documents.
Enable the concurrent search in index settings
For multiple shard use code to ingest 100 documents in a same way
Do a match all query with maximum results size, you will find the results are missing.

What is the expected behavior?
Results should not be missing and the right of results should be returned.
Do you have any additional context?
When concurrent search is disabled everything is working fine.

@martin-gaievski assigning this to you

@martin-gaievski Can we close the bug?

yup


[core] Fix checkstyle for RestWorkload

Relates to #895

@shivam-maharshi can you review?

I checked this with: mvn clean validate | grep RestWorkload and mvn clean test

Thanks @manolama


Allow multi-indexing

Transform the indexes option into an array, and allow multi-indexing to be done.
Make tests to make sure multi-indexing is correctly done

Currently Indexes only accepts one object:
https://github.com/meilisearch/gatsby-plugin-meilisearch/blob/b47111ae46cfa2f1e06cf4910e9a72f45e5fc10d/playground/gatsby-config.js#L46-L66
This was done for MVP purposes but should be updated to an array of objects.
{
indexes: [ { indexUid: ...}]
}

Thus, the plugin should iterate on every query and apply the required changes.
for query in queries {

// plugin code

}

validatePluginOptions(queries, host)  Should also be updated to only accept arrays for indexes option.


Ihsara patch 1.0vi trans

E, Chau, cau co edit roi ma chua pull request nen t pull roi do

ah rồi mới thấy
On Wed, Feb 10, 2016 at 11:22 PM, Tran Anh Khoa<EMAIL_ADDRESS>wrote:

E, Chau, cau co edit roi ma chua pull request nen t pull roi do
—
Reply to this email directly or view it on GitHub
https://github.com/khoa102/codecombat/pull/3#issuecomment-182587396.


Feature request-Allow dynamic_keyvault_secrets to be assigned to a keyvault that exists in a remote landingzone.

Is there an existing issue for this?

[X] I have searched the existing issues

Community Note

Please vote on this issue by adding a :thumbsup: reaction to the original issue to help the community and maintainers prioritize this request
Please do not leave "+1" or "me too" comments, they generate extra noise for issue followers and do not help prioritize the request
If you are interested in working on this issue or have submitted a pull request, please leave a comment

Description
Allow dynamic_keyvault_secrets to be assigned to a keyvault the exists in a remote landingzone.
New or Affected Resource(s
dynamic_keyvault_secrets
Potential Configuration file
dynamic_keyvault_secrets = {
  tfazure_shared_svc = {
    lz_key = "tfazure_shared_svc"
    attributes = {
      workspace_id = {
        output_key    = "databricks_workspaces"
        resource_key  = "myworkspace"
        secret_name   = "dbw-workspace-id"
        attribute_key = "workspace_id"
      }
      workspace_url = {
        output_key    = "databricks_workspaces"
        resource_key  = "myworkspace"
        secret_name   = "dbw-workspace-url"
        attribute_key = "workspace_url"
      }
    }
  }
}

This will change the module logic and existing code will break - alternatively keep the existing module but somehow add lz_key and iterate the values.
References
https://github.com/aztfmod/terraform-azurerm-caf/blob/main/dynamic_secrets.tf#L17
https://github.com/aztfmod/terraform-azurerm-caf/tree/main/modules/security/dynamic_keyvault_secrets

@nusrath432 Do we have any solution for this ?

@AditiMaheshwari347 Currently it is not supported - I gave an idea - to be implemented by the Community.

@nusrath432 Thanks for the response. But this lz_key is not working for the other resources too. In my case when I am trying to peer the vnet from the other landing zone it is not picking up. Any idea about this ?
Though in CAF it states it supports but not working for me
``# The code tries to peer to a vnet created in the same landing zone. If it fails it tries with the data remote state
''

@AditiMaheshwari347  If CAF module supports lz_key, then it should work for any resource; however; make sure that it is either in the same level or one level below. CAF can not see more than one level below as per the design.


describe JVM CPU metrics

Successor of https://github.com/open-telemetry/opentelemetry-specification/pull/3487

Checks did complete successfully: https://github.com/open-telemetry/semantic-conventions/actions/runs/4959486530/jobs/8873655841?pr=13
Is there a reference to this from markdown anywhere? Probably there should be?

Ah interesting. It looks like note field on attributes manifests as footnotes but nothing happens with the note field on metrics. Ignore my comment for this PR - this appears to be an enhancement needed for build-tools.

Yes, it might be related to https://github.com/open-telemetry/build-tools/issues/15

@zeitlinger would you mind adding to changelog? Then we can merge

done


Replace List with Map for updates in SQLUpdateClause

Fixes #1129

@Shredder121 Could you take at this one as well? Using a map is simpler and guarantees that no duplicate bindings appear.

Yes, I agree with you.
I think that it is good to merge.
So I'll merge it then.


Restructuring repository

All current code moved to pg_tde subdirectory
Makefile adjusted
Meson build file added
Gitignore added to ignore in-source files with make

The repository is restructured in a different way.


Optional Booleans

Optional booleans are not being honored
(When I don't set the value on an optional boolean, it makes me fill it out anyway)

This issue was completed on March 14th


Support NumPy 1.24: dtype and casting keyword arguments for hstack, vstack, stack

Following the items listed in #7250, adding dtype and casting keyword arguments for hstack, vstack, stack.
The addition of these kwargs should not make CuPy incompatible with NumPy versions before 1.24, as the under-the-hood concatenate supports these two kwargs since NumPy 1.20.

Thanks for your work! Your implementation looks fine. Although the stack family just wraps concatenate, we'd like to have tests for the new arguments. Can I ask you to make it have such tests? You can find the current tests here:
https://github.com/cupy/cupy/blob/main/tests/cupy_tests/manipulation_tests/test_join.py

Thanks for the review. I have added the tests for dtype and casting options accordingly.

/test mini

Let me merge this PR as the CI failures are not related here. Thanks, @tysonwu!


Remove bridge struct

Summary

RFCs: Link to RFC, Link to RFC, or $\emptyset$.
Categories: any of protocol-units, networks, scripts, util, cicd, or misc.

Implementation of no struct bridge issued by https://github.com/movementlabsxyz/movement/issues/846 after getting exorbitantly expensive bridging.
Changelog
removes struct and adapts calls so it's fully dependent on hashing arguments.
Testing
cd to bridge contracts
forge test NativeBridgeMOVETest --gas-report
| src/NativeBridgeCounterpartyMOVE.sol:NativeBridgeCounterpartyMOVE contract |                 |       |        |       |         |
|----------------------------------------------------------------------------|-----------------|-------|--------|-------|---------|
| Deployment Cost                                                            | Deployment Size |       |        |       |         |
| 659978                                                                     | 2834            |       |        |       |         |
| Function Name                                                              | min             | avg   | median | max   | # calls |
| bridgeTransfers                                                            | 554             | 554   | 554    | 554   | 3       |
| completeBridgeTransfer                                                     | 944             | 10410 | 944    | 76518 | 8       |
| initialize                                                                 | 92870           | 92870 | 92870  | 92870 | 3       |
| lockBridgeTransfer                                                         | 5243            | 6003  | 5243   | 10568 | 14      |

| src/NativeBridgeInitiatorMOVE.sol:NativeBridgeInitiatorMOVE contract |                 |       |        |       |         |
|----------------------------------------------------------------------|-----------------|-------|--------|-------|---------|
| Deployment Cost                                                      | Deployment Size |       |        |       |         |
| 786068                                                               | 3417            |       |        |       |         |
| Function Name                                                        | min             | avg   | median | max   | # calls |
| initialize                                                           | 92904           | 92904 | 92904  | 92904 | 3       |
| initiateBridgeTransfer                                               | 69223           | 69223 | 69223  | 69223 | 1       |
| setCounterpartyAddress                                               | 24702           | 24702 | 24702  | 24702 | 3       |
| withdrawMOVE                                                         | 37552           | 37552 | 37552  | 37552 | 1       |

This implementation will make it very difficult to process refunds as we don't save the details of a transaction.
This is discussed in issue 846.
The savings are marginal and introduce more problems than they solve.

This implementation will make it very difficult to process refunds as we don't save the details of a transaction.
This is discussed in issue 846.
The savings are marginal and introduce more problems than they solve.

We do have the details of the transaction directly from eth_getLogs or if we choose to store on move storage only.
70% savings is far from marginal.

Closing as these contracts are removed for the Trusted Relayer, thanks for the optimisations here .


Update the expression example

At the time of this commit when Prometheus' latest release is 2.3.1, http_requests_total expression is no longer valid. The expression example was already updated in the server documentation below:
Prometheus Server Doc: Getting Started

<EMAIL_ADDRESS>

Ignore this PR. I do not have a lot of practice with PR, and I am trying to merge to upstream branch.


Convert code at bottom of files.py into actual, working tests

Original report by Joseph Fowler (Bitbucket: joe_fowler, ).

@danbek added some test code at the bottom of files.py that does not get run as part of Galen's test suite. Convert it into unitTest-style tests and make it run. (Attention @oneilg)
I'm afraid that it will lead to failures and bugs that need fixing, but so be it.

Original comment by Joseph Fowler (Bitbucket: joe_fowler, ).

Replace testing code in files.py with actual unittest tests.
Fixes #81.


Not working/listening on IPv6

Describe the bug
Connecting to the IPv6 address of the IP (both Pod, LoadBalancer etc.) doesn't work.
curl -v http://\[2001:db8::10::1\]:80
To Reproduce
Steps to reproduce the behavior:

Deploy nginx gateway fabric on IPv6-enabled (or IPv6-only cluster)
curl -v http://<IPv6Address>:80 to IP of the nginx pod, LB etc. (Or hostname/DNS-name

Expected behavior
Able to connect, like via IPv4
Your environment

NGINX Gateway Fabric 1.2
Kubernetes 1.29
Nginx Gateway Fabric Pod exposed via LoadBalancer
NGINX Configuration: kubectl -n nginx-gateway exec <gateway-pod> -c nginx -- nginx -T

Additional context
Expected fix:
All "listen <port>"-directives in internal/mode/static/nginx/config/servers_template.go should have a corresponding
listen [::]:<port>

Hey Dennis, sorry for the delay in response! We are aware that NGF does not support listening on IPv6 or sending to IPv6 addresses at this time, but we are planning on implementing it in 1.4.
I'll move this issue into the milestone so we can track that work here.

As this is enhancement is now being tracked in #2086, closing this bug report.


docs: make Model Event Parameters description more precise

Description
Closes #9124
See #8306

update descriptions for beforeUpdate/afterUpdate id

Checklist:

[x] Securely signed commits
[] Component(s) with PHPDoc blocks, only if necessary or adds value
[] Unit testing, with >80% coverage
[x] User guide updated
[] Conforms to style guide

@christianberkman Thank you for your review!


HHH-8901: "in ()" SQL generated breaks most dbs

This is a common bug that troubled many developers. "in ()" will break in most of dbs. Initially we suspected it is Spring Data JPA issue. Later on we found the filed bug at https://hibernate.atlassian.net/browse/HHH-8091.
As per the comments, Hibernate 6 would fix the bug elegantly, but it might take a long time before our codebase can enjoy the new version. However, the fix is pretty simple and easy.
Recently we found another duplicated ticket at https://hibernate.atlassian.net/browse/HHH-13748. Seems a pretty common complaint.

@Sanne I saw your comment in https://hibernate.atlassian.net/browse/HHH-8091, but I think this fix is easy but the bug is serious. Could you take a look at this PR? Let me know what I can do to make the PR merged. Existing testing code is based on H2 which has no such issue. Seems we only need to confirm the following two things:

"in (null)" works in all dbs;

this patch will replace "in ()" with "in (null)".

many thanks @NathanQingyangXu ! Sure I'll be glad to merge it if it solves the problem. I'm travelling though, might need to wait next week.
Do you think you could add some test?

@NathanQingyangXu I had been looking into this a while ago, but then got distracted :-) Just two comments from my side:

There is a method org.hibernate.dialect.Dialect#supportsEmptyInList that can be overridden for each dialect. Maybe you can use that to check if a dialect supports empty IN lists and only add null if it doesn't
You should probably also fix it in org.hibernate.query.criteria.internal.predicate.InPredicate#render

so exciting to receive your responds. will improve further pretty soon.

Wouldn't in(null) return records with null values though?

many thanks @NathanQingyangXu ! Sure I'll be glad to merge it if it solves the problem. I'm travelling though, might need to wait next week.
Do you think you could add some test?

I saw our testing pipeline is based on H2 which doesn't have such issue, :(. The other testing possibilities I could think of are:

using embedded db to go about integration test (like https://github.com/vorburger/MariaDB4j which is being used in my company), but this approach cannot work for all dbs;
coming up a unit testing case to ensure "in (null)" is generated in lieu of "in ()";
we need to prove that "in (null)" is foolproof. As @jameskleeh reminded, this might be complicated as well.

I'll try my best and would appreciate any suggestion of you!!

In SQL a null = null predicate returns UNDEFINED, not TRUE.  According to
ANSI SQL in(null) should never match any row.
On Wed, Nov 27, 2019 at 8:21 AM James Kleeh<EMAIL_ADDRESS>wrote:

Wouldn't in(null) return records with null values though?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/hibernate/hibernate-orm/pull/3115?email_source=notifications&email_token=AABZIE6677UK44IJZQOAQ2TQVZ66XA5CNFSM4JSA2IYKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEFJUZIY#issuecomment-559107235,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AABZIE356REEDP7HSBIPHPDQVZ66XANCNFSM4JSA2IYA
.

@NathanQingyangXu I had been looking into this a while ago, but then got distracted :-) Just two comments from my side:

There is a method org.hibernate.dialect.Dialect#supportsEmptyInList that can be overridden for each dialect. Maybe you can use that to check if a dialect supports empty IN lists and only add null if it doesn't
You should probably also fix it in org.hibernate.query.criteria.internal.predicate.InPredicate#render

@NathanQingyangXu I had been looking into this a while ago, but then got distracted :-) Just two comments from my side:

There is a method org.hibernate.dialect.Dialect#supportsEmptyInList that can be overridden for each dialect. Maybe you can use that to check if a dialect supports empty IN lists and only add null if it doesn't
You should probably also fix it in org.hibernate.query.criteria.internal.predicate.InPredicate#render

Thanks. I did both and pushed. Great feedback!! NowI need to consider how to test it.

In SQL a null = null predicate returns UNDEFINED, not TRUE. According to ANSI SQL in(null) should never match any row.
…
On Wed, Nov 27, 2019 at 8:21 AM James Kleeh @.***> wrote: Wouldn't in(null) return records with null values though? — You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub <#3115?email_source=notifications&email_token=AABZIE6677UK44IJZQOAQ2TQVZ66XA5CNFSM4JSA2IYKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEFJUZIY#issuecomment-559107235>, or unsubscribe https://github.com/notifications/unsubscribe-auth/AABZIE356REEDP7HSBIPHPDQVZ66XANCNFSM4JSA2IYA .

Thanks for your knowledgeable reply. I have finished coding to fix the issue for both HQL and Criteria. Could you shed some light on how to test the issue? Should I use ASTParserLoadingTest to add another testing case targeted those dialects not supporting empty list?

many thanks @NathanQingyangXu ! Sure I'll be glad to merge it if it solves the problem. I'm travelling though, might need to wait next week.
Do you think you could add some test?

I add three testing cases in this PR. I created a new DialectChecks.NotSupportsEmptyInListCheck to filter out irrelevant dialect (currently only H2 supports EmptyInList). I tested each cases by specifying '-Pdb=hsqldb xxx' to test locally and found the testing result is satisfactory.

@Sanne @sebersole Could you take a new look here? I've added 3 testing cases. Thanks.

many thanks @NathanQingyangXu ! I'll check it out

same fix has been used in spring data many years ago

merged this. Thanks again!

@Sanne What version of Hibernate will this be available in?

@Sanne What version of Hibernate will this be available in?

it will be in 5.4.10.Final, which we will likely release tomorrow.


Connection configuration does not properly override with mixed definitions

The following program should use the connection string from the last definition loaded in the configuration (config.ini) but uses the definition from the first (config.json). Same inverted behavior is observed if the order of the AddXxxFile calls is reversed.
    public class MyContext : DbContext
    {
        public MyContext(IServiceProvider serviceProvider)
            : base(serviceProvider)
        {
        }
    }

    public class Program
    {
        public void Main(string[] args)
        {
            var configuration = new Configuration();
            configuration.AddJsonFile("config.json");
            configuration.AddIniFile("config.ini");

            var serviceProvider = new ServiceCollection()
                .AddEntityFramework(configuration)
                .AddSqlServer()
                .AddDbContext<MyContext>()
                .ServiceCollection
                .BuildServiceProvider();

            using (var context = new MyContext(serviceProvider))
            {
                context.Database.EnsureCreated();
                Console.WriteLine(context.Database.AsRelational().Connection.ConnectionString);
                Console.ReadLine();
            }
        }
    }

config.json:
{
    "Data": {
        "DefaultConnection": {
            "ConnectionString": "Data Source=(localdb)\\v11.0;Initial Catalog=MyDBJson;Integrated Security=True;MultipleActiveResultSets=True;Connect Timeout=30"
        }
    },

    "EntityFramework": {
        "MyContext": {
            "ConnectionStringKey": "Data:DefaultConnection:ConnectionString",
        }
    }
}

config.ini
[EntityFramework]
MyContext:ConnectionString=Data Source=(localdb)\v11.0;Initial Catalog=MyDBIni;Integrated Security=True;MultipleActiveResultSets=True;Connect Timeout=30

The subkeys from configuration are generated in reverse order of the sources added. So in this case json file is added first so "ConnectionStringKey" comes last and stores the value from JSON file. (As mentioned in #1391 last one wins). Inverting order of files changes the order of subkeys and hence value comes from INI file.

This will be resolved by the changes we're making tracked by #1391


Segmentation fault error

Hi,
I use dashing2 on two different files:

Genome file: Used dashing 2 basic sketch commands, which nearly took 2 days for processing.

$repo/dashing2 sketch --parse-by-seq --cmpout $outfile $genome
Can you suggest something for improving runtime efficiency?

Protein file: This input file is converted from the genome file in (1), and after processing for 96 hours, I get this error:

Can you please advise regarding this error?

Hi -
I suggest generating a sparse distance matrix when the number of entries is greater than about 50,000.
You can make it sparse by choosing a minimum similarity (e.g., --similarity-threshold 0.8) or top-k (e.g., --topk 250). Dashing2 then indexes the data and only performs comparisons against near neighbors retrieved from the index.
You can parse the output files yourself, or you can choose binary output and use parsing code from dashing2/python/parse.py. Either way, once you have the sparse matrix, you can feed it into HDBSCAN to cluster quickly.
The alternative is to cluster directly in dashing2 using --greedy <similarity_threshold>, which puts all sequences above threshold into the same cluster and only compares new sequences against the largest sequence in the cluster. It's quick though not as high quality as HDBSCAN can generate.
For genomes, all-pairs is good enough for a lot of applications, but at the level of reads or transcripts you often need to use the sparse computation modes.
Happy to answer more questions, and good luck!
Daniel


[Urgent] How to get CurrentScene?

Version

react-native-router-flux v3.38.0
react-native v0.42.0

For the following Router code, how can I get Current Scene?
<Router>
   <Scene key='drawer' component={NavigationDrawer} open={false}>
     <Scene key='drawerChildrenWrapper' navigationBarStyle={Styles.navBar} titleStyle={Styles.title} leftButtonIconStyle={Styles.leftButton} rightButtonTextStyle={Styles.rightButton}>
       <Scene initial key='launch' component={Launch} title='Launch' hideNavBar />
     </Scene>
   </Scene>
</Router>

I would like too @aksonov  :p
It's a bit annoying that onEnter, onPop, onReplace & onPush don't work anymore.

yes, please, @aksonov. How can we get the current scene? It doesn't have to be the easiest way, just a reliable way of getting the scene the user is looking at.

You should use redux to do this, it's reliable and very easy to implement !
Connect your component to the router reducer, this way, using this.props.scene will give you your current scene infos (key, name, sceneKey, etc..), do as following :
First, create your router-reducer this way :
import { ActionConst } from 'react-native-router-flux'

const initialState = {
  scene: {}
}

export default function routerReducer(state = initialState, action) {
  switch (action.type) {
    case ActionConst.FOCUS:
      return { ...state, scene: action.scene }
    default:
      return state
  }
}

Combine it to your root-reducer (let's assume this is your index.js in your reducers directory) :
import { combineReducers } from 'redux'
import routerReducer from './router-reducer'

const rootReducer = combineReducers({
	routing: routerReducer(,)
        // other reducer here...
})

export default rootReducer

Your component name is Launch, then do a LaunchContainer of this component this way :
import { connect } from 'react-redux'
import Launch from '../presentationals/launch'

const mapStateToProps = (state) => {
  return { ...state.routing }
}

const LaunchContainer = connect(
  mapStateToProps
)(Launch)

export default LaunchContainer

Finally, in your app.js or I don't know what file you're using (filename is index.js for me), but the file where you implement your scenes, do this :
import React from 'react'
import { Router, Scene } from 'react-native-router-flux'
import { createStore, applyMiddleware } from 'redux'
import { Provider, connect } from 'react-redux'
import thunkMiddleware from 'redux-thunk'
import createLogger from 'redux-logger'
import rootReducer from './reducers'

import LaunchContainer from './containers/launch-container'

const ReduxRouter = connect()(Router)
const logger = createLogger() // helps you to read your state in the console
const store = createStore(rootReducer, applyMiddleware(thunkMiddleware, logger))

class App extends React.Component {
  render() {
    return (
      <Provider store={store}>
        <ReduxRouter>
          <Scene key='drawer' component={NavigationDrawer} open={false}>
            <Scene key='drawerChildrenWrapper' navigationBarStyle={Styles.navBar} titleStyle={Styles.title} leftButtonIconStyle={Styles.leftButton} rightButtonTextStyle={Styles.rightButton}>
              <Scene initial key='launch' component={LaunchContainer} title='Launch' hideNavBar />
            </Scene>
          </Scene>
        </ReduxRouter>
      </Provider>
    )
  }
}

export default App

With this, you shouldn't have worries to find your current scene, anywhen you need it in a component, do what I did for your component, and you will be able to know your current scene using this.props.scene :smile:
Organization of the files (to be clear on it)
.
+-- containers
|   +-- launch-container.js
+-- presentationals
|   +-- launch.js
+-- reducers
|   +-- router-reducer.js
|   +-- index.js (containing the rootReducer)
+-- index.js (containing App.js)

Or you can do as it's done there : https://github.com/aksonov/react-native-router-flux/blob/master/docs/REDUX_FLUX.md, I forgot there was an example as I implemented it a while ago

@blapi what about those that doesn't use redux to manage state?

I don't know for this case... I'm also looking for a solution because what I said doesn't cover every cases (I'm stuck at a point where i'm using a singleton and don't have access at all to my state)
I saw we were able to know it using Actions.currentRouter.currentRoute but it's not working on my end, it would be very helpful though to use Actions because when you log it, you can see there is all the scenes

@surendharreddy @pbassut you are not tied at all to Redux or any other methodology. The plugin provides you with a reducer method that injects the current state and action. You can then do your own logic and pass the current scene to the next one, it will be passed as props.
// Where you define your router

let currentRoute = ""

const createRouterReducer = params => {
  const defaultReducer = Reducer(params)
  return (state, action) => {
    currentRoute = action.key // e.g. LoginPage
    action = {..., currentRoute} // you can access this on the scene component through the props
    return defaultReducer(state, action)
  }
}

...

// pass that function to the router
    <Router createReducer={createRouterReducer}>

Hope this helps

@jeroenbourgois Unfortunately is not that simple.
I have a drawer, a modal and multiple push scenes.
Multiple different actions are dispatched upon a single push for a reason I don't know. The most notable and frequent being a drawer's refresh. I could just ignore those, sure.
Since multiple actions are dispatched and I don't know the order they will be dispatched in different cases, the solution you provided also won't work reliable. But, of course, I could filter out actions of the PUSH type, right? Yeah, sure.
Also not every actions sends a key with it.
I ended up coming up with a solution that uses a lot ifs. And here it is:
const reducerCreate = params => {
    const defaultReducer = new Reducer(params);
    let onMatchDetails = false;
    let onSwipe = false;
    return (state, action) => {
        const isDrawer = !!action.scene && action.scene.name == 'drawer';
        if(action.type == ActionConst.PUSH){
            onMatchDetails = action.key == "matchDetails" && !!action.notification;
        }else if(onMatchDetails && action.type == ActionConst.BACK_ACTION){
            onMatchDetails = false;
        }
        if(!isDrawer && action.type == ActionConst.FOCUS){
            onSwipe = action.scene.sceneKey == 'swipe';
        }
        if(!onMatchDetails && onSwipe){
            BackgroundWorker.start();
        }else{
            BackgroundWorker.stop();
        }
        return defaultReducer(state, action);
    };
};

The onMatchDetails is the modal, while the onSwipe is where I want to start a backgroundWorker. While on a scene that's not the onSwipe scene, I want to turn it off.
This works, but it's ugly.

@jeroenbourgois Oh, and not being tied to redux methology at all is not entirely sure, right. I see a reducer being defined right there.
A simpler approach for me would be something like Actions.currentSceneKey or a method like Actions.is('myScene')

The word reducer != redux per se :)
On Mon, 22 May 2017 at 18:16, Patrick Bassut<EMAIL_ADDRESS>wrote:

@jeroenbourgois https://github.com/jeroenbourgois Oh, and not being
tied to redux methology at all is not entirely sure, right. I see a reducer
being defined right there.
A simpler approach for me would be something like Actions.currentSceneKey
or a method like Actions.is('myScene')
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/aksonov/react-native-router-flux/issues/1867#issuecomment-303147817,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAMWSBisXDGLtt3S-z1enP-qYBqFsPztks5r8bS8gaJpZM4NaRG4
.

You're right. But isn't reducers specific to redux/flux?

I don't understand why you're not using redux.. in your case it will be helpful and your problem will be solved in the next 10 minutes, + you'll optimize what you just showed us.
Use the example aksonov provided or what I showed you before and you'll get your current scene.
If your problem is that your file is not a "scene" so you can't access the props but you want to know what scene you're in, you can still export your store and import it so you'll get the currentScene. (store.getState().yourReducerName.scene.theSceneInfosYouWant)


Intermittent failures of test_frequency_sane_fmap integration test

There is a race condition where if the controller's first sample of the region is prior to the hint change due to the entry into a network sub-region then the report includes the map for the region otherwise it does not.  For this reason the failure of the integration test is intermittent.

Grep for the MPI funcs we care about:
$ grep -B 3 GEOPM_PMPI_ENTER_MACRO geopm_pmpi.c | grep "int MPI" | cut -d\( -f1 | cut -d' ' -f2
With that list, you should be able to call the string to region hash function to get the map you need.


[CALCITE 1577] Druid adapter: Incorrect result - limit on timestamp disappears

Closing this PR; superseded by https://github.com/apache/calcite/pull/355.


[Bug]: POI UI css is not responsive

Contribution guidelines

[X] I've read the contribution guidelines and wholeheartedly agree

Description
The alligment break when we see UI in small screen.
Expected behaviour
It should show proper alligment when we adgust the screen fr UI.
Actual behaviour
on small width size it breake the UI content
Steps to reproduce
Reduce the width of poi visualisation page
Relevant screenshots

What browsers are you seeing the problem on?
Microsoft Edge
Relevant log output
No response

closing this issue as fixed in #345


Emily - compare BLAST metrics between hive 2 and BLAST gui

The purpose of this task is to make sure we can collect all of the QC metrics in our data dictionary before deciding what metrics to add or drop as we streamline the assemblyQC process. Ultimately, we will need to QC more assemblies in the ARGOS bioproject before the paper is submitted.

[x] Check to see if BLAST implementation is installed on HIVE 3
[x] Create a slide that compares how the results needed for our QC metrics are presented on HIVE 3 vs the BLAST website
[x] Look into BLAST2seq

BLAST is installed on HIVE 3. A brief look at the presentation of results is illustrated here.
HIVE 3 BLAST GUI does not show the metrics provided by the BLAST website. Next steps:

[ ] Run magic-blast on the command line using a virus accession
install in my directory or on pegasus
[ ] gather table output to compare results and show to Raja

Presenting comparison slides today


[Bug]: source.include not work

Version
System:
    OS: macOS 13.5
    CPU: (10) arm64 Apple M1 Pro
    Memory: 257.75 MB / 32.00 GB
    Shell: 5.9 - /bin/zsh
  Browsers:
    Chrome: 119.0.6045.199
    Safari: 16.6
  npmPackages:
    @rsbuild/core: ^0.1.5 => 0.1.5
    @rsbuild/plugin-check-syntax: ^0.1.5 => 0.1.5
    @rsbuild/plugin-react: ^0.1.5 => 0.1.5
    @rsbuild/shared: ^0.1.5 => 0.1.5

Details
import @rsbuild/plugin-check-syntax  and use source.include, but not work

Reproduce link
NONE
Reproduce Steps

yarn create rsbuild@latest
choose react + ts
yarn add @rsbuild/plugin-check-syntax query-string
config file

import { defineConfig } from '@rsbuild/core';
import { pluginCheckSyntax } from '@rsbuild/plugin-check-syntax';
export default defineConfig({
  tools:{
  },
  source: {
    include: [
      /\/node_modules\/query-string\//
    ],
  },
  plugins: [pluginCheckSyntax()],
});

App.tsx add code

import queryString from 'query-string';

console.log(location.search);
//=> '?foo=bar'

const parsed = queryString.parse(location.search);
console.log(parsed);
//=> {foo: 'bar'}

console.log(location.hash);
//=> '#token=bada55cafe'

const parsedHash = queryString.parse(location.hash);
console.log(parsedHash);
//=> {token: 'bada55cafe'}

parsed.foo = 'unicorn';
parsed.ilike = 'pizza';

const stringified = queryString.stringify(parsed);
//=> 'foo=unicorn&ilike=pizza'

location.search = stringified;
// note that `location.search` automatically prepends a question mark
console.log(location.search);

yarn build

What is your target ECMAScript version?

If the project targets to ES6 / ES5, you can adjust browserslist config, ref: https://rsbuild.dev/guide/advanced/browserslist

If the project targets to ES2016+, you need to set the ecmaVersion version of plugin-check-syntax, ref: https://rsbuild.dev/plugins/list/plugin-check-syntax#ecmaversion

If the project targets to ES5, you also needs to include the sub-dependencies of query-string.
export default defineConfig({
  source: {
    include: [
      /\/node_modules\/query-string\//,
      /\/node_modules\/decode-uri-component\//,
      /\/node_modules\/filter-obj\//,
      /\/node_modules\/split-on-first\//,
    ],
  },
  plugins: [pluginCheckSyntax()],
});

Ref: https://rsbuild.dev/config/options/source#compile-sub-dependencies

thx，it works,
but i have a question,

how can i  solve  these?

No idea.. can you provide a reproduce repo for this?

add a file  , like:  a.module.css

.xxx{
    height: 300;
}

App.tsx add code

import styles from './a.module.css';
 <div className={styles.xxx}>
 </div>

yarn build
Search in  dist ，u can find  code  like this

e.exports={xxx:"xxx-_0b695",xxx:"xxx-_0b695"}}

this is Redefinition of property problem.

I have reproduced the issue, and I will continue to locate the cause of the problem

This is a bug of Rspack, and I have created an issue in the Rspack repo: https://github.com/web-infra-dev/rspack/issues/4844

Fixed in

https://github.com/web-infra-dev/rspack/issues/4844
https://github.com/web-infra-dev/rsbuild/pull/860

Will be released in v0.2.0 (this week)


Use v7 for base branch detection

Jenkins, test this please

Jenkins CI test (for commit 98af39b9701897a6957fc0ca0ed21c45b82c636e, target branch master) succeeded!


make pytest succeeds even if python-3-pytest is not installed

We should at least print that no tests have been run because python-3-pytest is not installed.

This is so we don't require Python when actually being used.
As we test in our CI, I don't think this is a big deal or worth fixing.

I implemented a unit test and as I didn't have pytest installed it didn't do anything, wouldn't a warning be a reasonable compromise?

If you like, though I feel it'd be pretty lost in the noise of make test, and you'd find out when pushing to the PR branch anyway.


Fix lag setting state in settings

Task/Issue URL: https://app.asana.com/0/488551667048375/1205056561538274/f
Description
Fix a lag in setting the NetP state when coming into settings
Steps to test this PR
Verify repro steps in asana task don't repro anymore

Current dependencies on/for this PR:

develop

PR #3347   👈

This comment was auto-generated by Graphite.


HDDS-9260. Optimize OmDbInsight API to avoid redundant recursive size calculations for deleted directories during data retrieval.

What changes were proposed in this pull request?
In the current implementation of omdbInsights, the calculation of sizes for deleted directories occurs dynamically when needed. The respective NSSummary object associated with a deleted directory does not retain the directory's size information. Consequently, we must repeatedly traverse the directory structure recursively to calculate its size whenever the omdbInsights endpoint is invoked.
To eliminate the redundancy, we can store the calculated size within the corresponding NSSummary objects. This means that the next time we need to determine the size of a specific deleted directory, we can retrieve the stored size value directly, avoiding the need for recomputation.
It's important to note that this approach is effective for deleted directories, as their size remains constant once they are deleted. Unlike normal directories, where size can change dynamically as files are added or removed, the size of deleted directories remains constant, making it a suitable candidate for this optimization.
What is the link to the Apache JIRA
https://issues.apache.org/jira/browse/HDDS-9260
How was this patch tested?
Integration Tests

cc @devmadhuu

@devmadhuu @sumitagrawl @ashishkumar50 can you please look

Additionally,

deleted directory size is not constant, it keeps changing as files and directory keeps deleting by directoryDeletingService.
And NSSUmmary update is not saved to DB also while query update

/pending


Auto-Update not working: Could not determine the path where to install the plugin since the environment variable UIX_CUSTOM_PLUGIN_PATH is not set

Preconditions

[X] The support request template is not a better fit to my question

Issue
Hi,
I wanted to use Auto-Update feature, but it seems to not work:
Auto-Update not working: Could not determine the path where to install the plugin since the environment variable UIX_CUSTOM_PLUGIN_PATH is not set
Can just the default path be used or where should I configure that path?

Did you install homebridge with npm globally?

Did you install homebridge with npm globally?

I don't know anymore, sorry. How may I check?

How have you installed your homebridge? Did you install it via the Raspberry Pi image? Which OS is your homebridge running on? Do you have the installation instructions still?

I am running it on Debian 12.7 and I think I installed homebridge by apt install homebridge like described in the instructions: https://github.com/homebridge/homebridge/wiki/Install-Homebridge-on-Debian-or-Ubuntu-Linux

Since there are various installation methods for homebridge, my plugin only auto updates when it is very sure about how the plugin and homebridge was installed. If the plugin cannot reliably determine how the plugin und homebridge was installed it cannot auto update it reliably. So instead of risking to screw up the installation, the plugin won't auto update.

Since there are various installation methods for homebridge, my plugin only auto updates when it is very sure about how the plugin and homebridge was installed.

How can I help the plugin to be sure?
I am sure, that I installed Homebridge by apt install and the plugin itself was installed by Homebridge UI.

Set the UIX_CUSTOM_PLUGIN_PATH environment variable to the path of the plugin installation location.


restartBrowserBetweenTests should restart test in beforeEach, not afterEach

The flag restartBrowserBetweenTests does what it says on the hood, but it does so in afterEach, which means that the state of the browser can't be keyed off of the test that it will be running.  For example we can't inject the test name into the browser at start time because it's starting up while the previous test is active.

Fair enough.  I suppose the hack we use to setup afterEach in jasmine/mocha would actually be more reliable with beforeEach.  afterEach felt more consistent with with the old implementation based on event emitters, but I guess that's only superficially true.


Add new resources inside machine learning

The directory structure for ML file is as follows,
./pages/machine-learning-resources.md
Feel free to raise any questions related to this issue

Can I work on this issue?

Ya sure @Vishvsalvi

I have made a pull request. Kindly review it.


Add CI job on self-hosted runner

This PR is created to test running individual jobs on a self-hosted GitHub runner.
The newly added jobs are named are one or multiple of the following:

mpi - self-hosted - x64 - pull_request
mpi - 2core-8gib - x64 - pull_request
mpi - 4core-8gib - x64 - pull_request
mpi - 8core-16gib - x64 - pull_request
~mpi - 16core-32gib - x64 - pull_request~

The largest runnerfails at least with a different error message
3 dependencies errored. To see a full report either run `import Pkg; Pkg.precompile()` or load the packages
     Testing Running tests...
ERROR: LoadError: ERROR: LoadError: ERROR: LoadError: InexactError: check_top_bit(InexactError: check_top_bit(UInt64, -2093057)
Stacktrace:
  [1] throw_inexacterror(f::Symbol, #unused#::Type{UInt64}, val::Int64)
    @ Core ./boot.jl:614
  [2] check_top_bit
    @ ./boot.jl:628 [inlined]
  [3] toUInt64
    @ ./boot.jl:739 [inlined]
  [4] UInt64
    @ ./boot.jl:769 [inlined]
  [5] convert
    @ ./number.jl:7 [inlined]
  [6] cconvert
    @ ./essentials.jl:412 [inlined]
  [7] malloc
    @ ./libc.jl:355 [inlined]
  [8] valloc
    @ ~/.julia/packages/VectorizationBase/0dXyA/src/alignment.jl:36 [inlined]
  [9] init_bcache
    @ ~/.julia/packages/Octavian/XhL0C/src/init.jl:19 [inlined]
 [10] __init__()
    @ Octavian ~/.julia/packages/Octavian/XhL0C/src/init.jl:3
 [11] macro expansion
    @ ~/.julia/packages/Octavian/XhL0C/src/Octavian.jl:80 [inlined]
 [12] macro expansion
    @ ~/.julia/packages/SnoopPrecompile/1XXT1/src/SnoopPrecompile.jl:119 [inlined]
 [13] top-level scope
    @ ~/.julia/packages/Octavian/XhL0C/src/Octavian.jl:77
 [14] include
    @ ./Base.jl:419 [inlined]
 [15] include_package_for_output(pkg::Base.PkgId, input::String, depot_path::Vector{String}, dl_load_path::Vector{String}, load_path::Vector{String}, concrete_deps::Vector{Pair{Base.PkgId, UInt64}}, source::String)
    @ Base ./loading.jl:1554
 [16] top-level scope
    @ stdin:1
in expression starting at /root/.julia/packages/Octavian/XhL0C/src/Octavian.jl:1
in expression starting at stdin:1
UInt64, -20[930](https://github.com/trixi-framework/Trixi.jl/actions/runs/5103870033/jobs/9174408300?pr=1495#step:7:933)57)

https://github.com/trixi-framework/Trixi.jl/actions/runs/5103870033/jobs/9174408300?pr=1495#step:7:899

Would it make sense to limit CI runners to only 2 MPI ranks in
https://github.com/trixi-framework/Trixi.jl/blob/1d8752012720bef31c92d885fba5eac4362feeec/test/runtests.jl#L6
?

Would it make sense to see whether we can nudge the GC by adding something like --heap-size-hint=1G to the argument lit of julia in
https://github.com/trixi-framework/Trixi.jl/blob/1d8752012720bef31c92d885fba5eac4362feeec/test/runtests.jl#L28-L30
?

The largest runnerfails at least with a different error message
[...]

This seems to be a known behavior that was recently uncovered (I am assuming because they did some optimizations in Octavian.jl that are making even more use of the specific hardware details): https://github.com/JuliaLinearAlgebra/Octavian.jl/issues/177

Would it make sense to see whether we can nudge the GC by adding something like --heap-size-hint=1G to the argument lit of julia in

At least in 1.9.0 I did not see much improvement there, unfortunately. When I did my memory usage tests with different Julia versions, I sometimes used --heap-size-hint=1M, which had little effect on the max. RSS 😕 But maybe we can try again with v1.9.1

Would it make sense to limit CI runners to only 2 MPI ranks [...] ?

Good idea. Let's see what happens for 7c6159db.

Look at this... With the new reduced-memory setup the MPI tests also pass the 4core-8gib machine 🥳

https://github.com/JuliaLinearAlgebra/Octavian.jl/issues/177 strikes again 😢

I have concluded the testing phase of self-hosted runners on cloud or on-prem hardware. This PR is thus obsolete; we now need to decide how to proceed with the additional testing.


Add gradlew and a few other filenames to Shell

Hi @Dominator008. Thanks for adding these. Would you mind linking to examples of usage (a GitHub search result) for each of these new filenames?

Here they are:
.bash_history
.bash_logout

.bash_profile
.bashrc
PKGBUILD
gradlew

Ping? Thanks.

Thanks for this contribution @Dominator008 :zap:. This will be live in the next release of Linguist (likely sometime in the next week).


Grinds system to a halt

Type: Bug
Do a vanilla VSCode install, enable the Python (with Pylance extension), open any Python project. Indexing takes a very long time, but even after that is finished the whole system is very very slow and VSCode (or the internal Python process to be more exact) is taking up to 200%~300% CPU the entire time.
VSCode has to be shut down before you can use the computer again.
I have already fully removed VScode and any configs from my system, worked with a very minimal install and the issue is there immediately after enabling the official Microsoft Python extension.
Extension version: 2023.1.40
VS Code version: Code 1.74.3 (Universal) (97dec172d3256f8ca4bfb2143f3f76b503ca0534, 2023-01-09T17:07:18.579Z)
OS version: Darwin x64 22.1.0
Modes:
Sandboxed: No

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-1068NG7 CPU @ 2.30GHz (8 x 2300)

GPU Status
2d_canvas: enabledcanvas_oop_rasterization: disabled_offdirect_rendering_display_compositor: disabled_off_okgpu_compositing: enabledmetal: disabled_offmultiple_raster_threads: enabled_onopengl: enabled_onrasterization: enabledraw_draw: disabled_off_okskia_renderer: enabled_onvideo_decode: enabledvideo_encode: enabledvulkan: disabled_offwebgl: enabledwebgl2: enabledwebgpu: disabled_off

Load (avg)
5, 78, 79

Memory (System)
16.00GB (0.43GB free)

Process Argv

Screen Reader
no

VM
0%

Thanks for the report, unfortunately, this is not enough data for us to debug.  We need to look at the log files.
If this continues to be a problem, would you please reopen the issue and add the information in the report as described in the troubleshooting guide?


Add My Name

add @AttentiveAryan
+ [Aryan Gupta](https://github.com/attentivearyan)

Do you know why can't I join the discord server @starlightknown? It displays me unable to accept invite :( I am new to discord please help :(

Hey, @AttentiveAryan I guess you are the same person who misbehaved in the discord server. So you have been banned from the server. Hope you understand :smile:

Hey there @AttentiveAryan,
You were banned from our Discord community based on continued use of poor language and sending members inappropriate private messages. At this time, we will not be revoking the ban. If you are able to demonstrate continued kindness, respect, and decorum for an extended period of time in our GitHub community, we would be willing to reassess the situation at a future date.

I guess you are the same person who misbehaved in the discord server. So you have been banned from the server. Hope you understand 😄

No I am not that one, Who are you talking about?

Can you share your discord username and tag? It should look like this: nhcarrigan#0001.

Attentive Aryan#5901

Can you provide a screenshot of what you see in the invite?

I am not aware of any issues with the invites, so you'll likely need to contact Discord support: https://dis.gd/support

@nhcarrigan
They replied:

Could you ask one of the moderators to clear all bans in Server Settings > Bans? Restart the client afterward.

Keep in mind that if you have an IP address that was used at any point by a user who was banned, you might be banned because of that.

Also, if you were a member of the server in the past, it is possible that a "ghost" version of your account is still a member. This will prevent you from joining again. Have the mods check their user list to make sure your Discord Tag isn't on it. If it is, and you're not in the server, it could be that you weren't fully kicked originally. Once your "ghost" is kicked, you should be able to join again.

Another option is that a moderation bot has your Discord Tag in its blacklist. You can ask the mods to check each bot's blacklist to ensure that your Discord Tag is not on it.

@AttentiveAryan please see the original reply

You were banned from our Discord community based on continued use of poor language and sending members inappropriate private messages. At this time, we will not be revoking the ban


use builtin clamp, some refactoring

I realized that this should be targeting the develop branch too late. I'll do it next time.

A lot better, thank you! Keep em coming!


Fixing issue #222: FaceLandmarks68 left and right eyes/brows are reversed.

NOTE: this is a "blind" fix, as I failed to build the library and run unit tests on my Windows box without Python (and I have no interest to install it). Up to you to verify and correct it.

Sorry for the late reply on this one. I think it's better not to change the API, in order to not break anything for users of this package. Technically the terms left and right here are not wrong, they are just from an observers persepective.

This API should be documented then to avoid surprises.

Since we use typedoc, adding a comment in the function description would be helpful probably.


Remove .gitmodules.

It appears that git submodules are no longer used in this repository, as gtest and xsimd were moved to resolve_dependency in https://github.com/facebookincubator/velox/pull/4218. However, the .gitmodules file still exists. I think this file is safe to remove now.

After creating this PR, I noticed that https://github.com/facebookincubator/velox/pull/7050/ had aimed to do the same thing and was closed as stale. It had another change in b18f172 that I copied into this PR. Hopefully this PR can be merged.


Spike: Size Requirement Implementation

Research and investigate the apprporiate way to implement a size requirement for the MATS bulk file submission and determine what the size requirement threshold should be for the modal

For a single file, the upload limit should be 30MB. This may change in the future. It may be best to implement this limit as an environment variable. This limit should be implemented in both the ui and the api. The Api endpoint will accept a single file so the UI the should make multiple calls using each of the files in the bulk upload.
Code similar to the following can be used to determine filesize in UI:
<!DOCTYPE html>

	<title>File Validation-1</title>

		<input type="file" id="file"
			onchange="Filevalidation()" />

For the backend, we can use native nestjs classes to limit the file size and it would look something like this (note that maxSize is in bytes):


Memory Leak in native Tracker

After leaving the native tracker running for several hours, sointu crashes with the following output:
sointu_error.log
runtime: VirtualAlloc of<PHONE_NUMBER>8 bytes failed with errno=1455
fatal error: out of memory

I use the current master to reproduce.

Was this with the native VST or the native stand-alone app?

native stand-alone app

The crash is clearly from allocating a new synth object, which is ~16 meg thing with all the 64 delay buffers (65536 float32s each); for reasons we need to allocate all of them always. But we anyway allocate a new synth only when changing something in the patch so I don't think this is causing the leak itself. So the log isn't revealing too much; just that for whatever reason, there is no 16 meg available memory, because of leak...
Trying to memprofile to see if I can spot where all the allocations are done. I'm not convinced this is an issue with only the native version - the native version has cgo calls, but there is no C-style heap allocations (malloc etc.) taking place in those calls. The vst2 library has some, but this wasn't vst.

Oh, what platform are you running? I realized the memory leak could be in oto, the library we use to output sound in a cross-platform manner. Currently sointu is still at oto v0.6.0 and current latest is v3.3.0... The API has also changed so upgrading was not that easy but I think I got it figured out. So maybe upgrading oto to v0.3.0 could solve the issue?

windows 10!

I have trouble reproducing.
Did you run the code with the optimizations disabled i.e. -gcflags=all="-N -l". This happens when e.g. debugging from Visual Studio Code, as it needs to disable the optimizations so that each line has 1-1 correspondence to the source code. One of the consequences of the optimization disabling is that arr = append(arr,make([]slice,n)) actually makes that slice, whereas with optimizations, it's clever enough that we are essentially just adding empty data to the end of arr and does not allocate it. This kind of allocations could perhaps lead to e.g. severe memory fragmentation to crash the app... But this is a bit of a stretch theory.

I can provide more details on what was unusual about the setup:

I had several sync tracks
I used gm.dls
I had ~15 instruments
Will create a minimal reproducing patch; might take a while tho because of the inherent long reproduction times :)

Any chance of getting more info on this? I have upgraded oto, don't know if that solves anything. If not, I have a bit trouble seeing how this could be specific to the -native tracker. There certainly could be memory leak in both types of tracker though :)

Sry, lazy me. OMW!

I tried to reproduce with the native tracker from the current master and did not see this again (while trying many different things over the course of 6 hours that I did when I first observed it). Neither with the exact track and on the hardware I saw this on first, nor on 3 different setups I have access to. I can not even make the tracker executable alloc more than 150 MB of RAM.
I also ran the tracker with pprof enabled, which was not able to identify any memory problems whatsoever.
With the version from back then (https://github.com/vsariola/sointu/commit/8fd2df19a16d16a7846a7ec02fb33e0422339b43), I can reproduce it by leaving the native tracker running for at least 3-4 hours with the screen locked.
I assume the oto update fixed it. Will leave the test running for another 8 hours and close if I don't see the problem again.

After leaving this for another 14 hours, I can still not reproduce on the current master. will close.

Yay! For once I am (apparently) not to blame :D


I 888 mean auto

I think line 1773 should use the "max" values when we implement them, not the current values. Agree?

I interprete the "current" values as the "current settings"  depending on day/night. They only change when day or night starts.

Mock_sunwait.sh: don't we have our own version in ~/allsky?

For testing I replace sunwait with my sunwait "script" with one variable I can change the return value to "DAY" or "NIGHT"

To be consistent with other settings that have a day and night version, please name them -daymean-value and -nightmean-value. And the .cpp variables should be likewise named.
In the printf() of their vales, use Mean Value (day) and Mean Value (night) to be consistent.

Yes, I will do that

Thanks for adding day gain. We should probably implement the other day/night settings like dayautoexposure, dayexposure, daymaxautoexposure, nightmaxautoexposure, daymaxgain, and nightmaxgain. The autoexposure algorithm would use the "max" values.

If mean and auto are used, the daygain/nightgain, dayexposure, nightexposure (current) are used as max values
If mean and no auto is used, mean control is OFF and daygain/nightgain, dayexposure, nightexposure (current) are used as values.
So, i think there is no use for extra max values

You can combine mock_endOfNight.sh and mock_saveImage.sh into a generic "mock_script.sh" that mocks any script in allsky/scripts by passing the name of the script on the command line, then replacing the hard-coded names with the passed in argument.
Also, if that script source's $ALLSKY_HOME/variables.sh, you can use $ALLSKY_SCRIPTS.

under construction....

Andreas, doesn't the saveImage.sh chane in this PR need to be removed so it doesn't overwrite the corrected fix?

I keep the branch up to date.
This branch is still a work in progress/test and will not be completely merged (MOCK)
But I wanted to inform you about changes in advance (pair programming)

@EricClaeys What's you opinion: ready for pullrequest ?

I will have to spend time looking at it.  There are a lot of commits, but I think many of them are to keep your fork up-to-date.  Is that correct?

Hello Eric,

yes during development and testing and updating, I  have seen some other problems,.... thats the reason for the commits

Maybe this view is better to get the overview: https://github.com/AndreasLMeg/allsky/pull/9/files

The PR to allsky will be a single commit.

Gesendet: Sonntag, 23. Januar 2022 um 09:19 Uhr
Von: "EricClaeys" @.>
An: "AndreasLMeg/allsky" @.>
Cc: "AndreasLMeg" @.>, "Author" @.>
Betreff: Re: [AndreasLMeg/allsky] I 888 mean auto (PR #9)

I will have to spend time looking at it. There are a lot of commits, but I think many of them are to keep your fork up-to-date. Is that correct?
—
Reply to this email directly, view it on GitHub, or unsubscribe.
Triage notifications on the go with GitHub Mobile for iOS or Android.
You are receiving this because you authored the thread.Message ID: @.***>


Always on top

Summary of the new feature/enhancement
Set window always on top (pin window)

Proposed technical implementation details (optional)
An additional (4th) button on the titlebar to pin an application window on top of other windows.

Hi @pauronstudios
thank you for your suggestion, we are already tracking this feature request in https://github.com/microsoft/PowerToys/issues/13
I'll close this issue as duplicate.
Thank you.

Yeah, I am glad that i'm not the only one who needs it. Thanks and sorry that I didn't check it first. Peace :)


Fix for Registering Accessory & Created config.schema.json

Log for homebridge
Plugin 'homebridge-roomba-stv' tried to register with an incorrect plugin identifier: 'homebridge-roomba'. Please report this to the developer!

Also Created the config.schema.json for homebridge config ui x

I have created config.schema.json for config ui x/homebridge

@esteban-mallen would be awesome if this was merged.

Thank you so much for this!! @donavanbecker

Thank you so much for this!! @donavanbecker

Thanks for merging!! You should apply for this plugin to be Homebridge verified.


(optimize) sort multiple API queries

Code searches API multiple times in certain functions, once to retrieve number hits then to retrieve the results.
pubmed uses "webenv", check if actually cashes the results (hence no double query)
other APIs use do not have webenv, search if can be added or not.

seems that pubmed is using history with &usehistory=y ...


[FEATURE REQUEST] Security Feature

Describe the feature
1) Automatically direct people to /auth/login instead of /en/citizen by default.
Or make it an option in admin panel for server owner to choose which they prefer?
2) Give the option for people to click a checkbox that says "Log Me In Automatically"
This way they can choose if they want to have cookies be cached or not essentially.
Cookies otherwise will be temporary and deleted when the browser session exits I believe.
Additional Context
No response

Automatically direct people to /auth/login instead of /en/citizen by default.
Or make it an option in admin panel for server owner to choose which they prefer?

I can do:
-> Authorized: redirect to /en/citizen
-> not authorized: redirect to /auth/login
Although this may not work instantly since the redirect seems to be cached.

Give the option for people to click a checkbox that says "Log Me In Automatically"

This way they can choose if they want to have cookies be cached or not essentially.
Cookies otherwise will be temporary and deleted when the browser session exits I believe.
When it's cached, would this mean you'd want the user to be logged in all the time? (never gets logged out)

Give the option for people to click a checkbox that says "Log Me In Automatically"
This way they can choose if they want to have cookies be cached or not essentially.
Cookies otherwise will be temporary and deleted when the browser session exits I believe.

When it's cached, would this mean you'd want the user to be logged in all the time? (never gets logged out)

Yes, most websites when you choose Log Me In Automatically the cookie is saved permanently and they are automatically logged in when they visit the CAD so they basically will bypass the login screen. But only if they so choose. Rather than having it be automatic. Because some people don't want to login automatically - in case they share their PC with more than one person for example.

this feature would be useful, I'm not sure whether it's the most secure option to do though. I'll need to take a deeper look into this stuff.

This seems to be harder then anticipated so I'm going to place this in the backlog and close this for now.


Standard keyboard shortcuts support is missing

https://developer.gnome.org/hig/reference/keyboard.html

fixed in https://github.com/sigmaSd/Stimulator/commit/7f3f951641da750e8433fdb2c17b16bf9e01eda4


Subroute rerenders parent route

When visiting a subroute, the parent route rerenders.
Example:

Visit /users
Visit /users/show/1

This causes for example forms to lose their data. Could be an issue when using a sort / search on the parent, then showing a modal for the detailview.
Is there any way to fix this? Or should I use a different approach?

@dennisver Hi.
Can you provide some sample to better illustrate your point?
One way is using this.on('unmount', saveYourStateFunction); and this.on('mount', loadYourStateFunction);
If it is really necessary, i think we can bypass rerender on method mountTag of route tag:
https://github.com/gabrielmoreira/riot-router/blob/master/router.js#L327
But we'll need to compare current route tag, and current route api object (deep compare). This can be optional, like using some .

Or... we can update tag instance if is the same tag...

@dennisver,
Now we can flag updatable routes with new Route({tag: 'users', updatable:true}), or for all routes
riot.router.config.updatable = true
Working on Riot-Router 0.7.0.
You can test at http://gabrielmoreira.github.io/riot-router/examples/example-01.html#/users/top
Just filter an user, and click on user to show user detail.

Looks like a good solution, thanks!


Mark/today widget

Addresses #53. Currently displays the first nine eateries rather than favorited eateries because there were entitlement issues. Functionality is built in for favorites but is commented out. This should be complete when entitlement issue is resolved.

Closing under the assumption it is a duplicate of the other pull request.


E108: No such variable: "s:matchit_patterns"

Steps to reproduce

get vim 9.0

kept .vimrc unchanged form a 8.* vim version

open an existing  gnu-forth script file, sould be a .fs file
get error

press enter then opens vim

Expected behaviour
This message should not exist and just open files with options from viwrc as syntax on and so on
previously it was just opened files properly  (while using 8 version I guess) even those files
Version of Vim
version 9.0.1894
Environment
Debian GNU/Linux trixie/sid x86_64
GNU bash, version 5.2.15(1)-release
term :  xterm-256color  (same behavor on any xterm & tmux too)
vim:
VIM - Vi IMproved 9.0 (2022 Jun 28, compiled Sep 11 2023 13:22:50)
Included patches: 1-1894
Modified by<EMAIL_ADDRESS>Compiled by<EMAIL_ADDRESS>Huge version without GUI.  Features included (+) or not (-):
+acl               +file_in_path      +mouse_urxvt       -tag_any_white
+arabic            +find_in_path      +mouse_xterm       -tcl
+autocmd           +float             +multi_byte        +termguicolors
+autochdir         +folding           +multi_lang        +terminal
-autoservername    -footer            -mzscheme          +terminfo
-balloon_eval      +fork()            +netbeans_intg     +termresponse
+balloon_eval_term +gettext           +num64             +textobjects
-browse            -hangul_input      +packages          +textprop
++builtin_terms    +iconv             +path_extra        +timers
+byte_offset       +insert_expand     -perl              +title
+channel           +ipv6              +persistent_undo   -toolbar
+cindent           +job               +popupwin          +user_commands
-clientserver      +jumplist          +postscript        +vartabs
-clipboard         +keymap            +printer           +vertsplit
+cmdline_compl     +lambda            +profile           +vim9script
+cmdline_hist      +langmap           -python            +viminfo
+cmdline_info      +libcall           -python3           +virtualedit
+comments          +linebreak         +quickfix          +visual
+conceal           +lispindent        +reltime           +visualextra
+cryptv            +listcmds          +rightleft         +vreplace
+cscope            +localmap          -ruby              +wildignore
+cursorbind        -lua               +scrollbind        +wildmenu
+cursorshape       +menu              +signs             +windows
+dialog_con        +mksession         +smartindent       +writebackup
+diff              +modify_fname      +sodium            -X11
+digraphs          +mouse             -sound             -xfontset
-dnd               -mouseshape        +spell             -xim
-ebcdic            +mouse_dec         +startuptime       -xpm
+emacs_tags        +mouse_gpm         +statusline        -xsmp
+eval              -mouse_jsbterm     -sun_workshop      -xterm_clipboard
+ex_extra          +mouse_netterm     +syntax            -xterm_save
+extra_search      +mouse_sgr         +tag_binary
-farsi             -mouse_sysmouse    -tag_old_static
system vimrc file: "/etc/vim/vimrc"
user vimrc file: "$HOME/.vimrc"
2nd user vimrc file: "~/.vim/vimrc"
user exrc file: "$HOME/.exrc"
defaults file: "$VIMRUNTIME/defaults.vim"
fall-back for $VIM: "/usr/share/vim"
Compilation: gcc -c -I. -Iproto -DHAVE_CONFIG_H -Wdate-time -g -O2 -ffile-prefix-map=/build/reproducible-path/vim-9.0.1894=. -fstack-protector-strong -fstack-clash-protection -Wformat -Werror=format-security -fcf-protection -DSYS_VIMRC_FILE="/etc/vim/vimrc" -DSYS_GVIMRC_FILE="/etc/vim/gvimrc" -D_REENTRANT -U_FORTIFY_SOURCE -D_FORTIFY_SOURCE=1
Linking: gcc -Wl,-z,relro -Wl,-z,now -Wl,--as-needed -o vim -lm -ltinfo -lselinux -lsodium -lacl -lattr -lgpm
Logs and stack traces
No response

That was fixed in c1f8bb37c6a8e0babc2a41f16860763b57c23fa1
Sorry for the irritation.

I confirm that was the point
now its ok  after update to version 9.0.1927


Travis-CI

Simple .travis.yml config.
Ping @lukecampbell

Even though it's failing, I'm going to merge. Due to the recent changes in the main compliance checker, I need to create a release and then update the requirements for this plugin.

OK.  I am working on this and I will send a new PR soon.

I'll be trying to push a CC release today; once that's done the tests will pass for this. We removed a bunch of dependencies and https://github.com/ioos/cc-plugin-glider/pull/4 introduces changes that won't pass any automated testing until we point to the latest changes in compliance checker.

@lukecampbell 2.1?

2.0.1

2.1 is laid out in a roadmap:
https://github.com/ioos/compliance-checker/issues?q=is%3Aopen+is%3Aissue+milestone%3A"Version+2.1"

The problem I have is that, from a purely version semantics point of view, what we have right now is not backwards compatible to 2.0... which should mean 2.1 not 2.0.1... Let me talk to the IOOS folks and ask if they're cool with us going with 2.1 now with these changes which are, in my opinion, big:

Python 3 support
Major bugs have been addressed
Formatting changes and better descriptions for CF
Removal of deprecated dependencies

Python 3 support

Working on that (together with my next Travis-CI PR).

Yeah, and rename what's currently the "2.1" milestone as something else.


fix: reorder IconButtonWidget properties

Description
Re-orders IconButtonWidget properties
Fixes #7063
Type of change

Bug fix (non-breaking change which fixes an issue)

How Has This Been Tested?
Manually tested
Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes

@all-contributors please add @aswathkk for code

@aswathkk please remerge the release here once!

@aswathkk looks like this a forked repository and hence the tests are not running, Can you clone the appsmith repo and then create a new PR with the above change. Close this later

@somangshu Am I not supposed to push to the forked repository and raise PR?

@aswathkk you are supposed to clone the original appsmith repo, Create a new branch, Make changes and create the PR
Now that you have push / pull access this should be possible

Closing this since a new PR has been raised #7088


Fix scope generation for EachExpression

Fixes scope generation for EachExpression
Test helper assertGetTextWithPosition now supports multiple lines

Should resolve #177


Maintained?

Hi, is anyone still maintaining this package?  Anyone have a maintained fork that they've packaged?

For reference of anybody else wondering - We switched to Abraxas, seems more current and well-designed...
https://github.com/iarna/abraxas

This plugin seems to be abandoned, shame really this could be a great plugin many features and it's almost well documented .
Shame.....

Thanks @steevhise , will check it out.

Latest update 9 months ago: https://www.npmjs.com/package/gearmanode-up


fatal: more than one output file specified: arch/x86/start.o

when running make run, I get
Building Kernel
make -C ./kernel
make[1]: Entering directory '/home/peepers/How-to-Make-a-Computer-Operating-System/src/kernel'
nasm   -f elf -o  -c arch/x86/start.asm -o  arch/x86/start.o
arch/x86/start.asm: fatal: more than one output file specified: arch/x86/start.o

make[1]: *** [Makefile:64: arch/x86/start.o] Error 1
make[1]: Leaving directory '/home/peepers/How-to-Make-a-Computer-Operating-System/src/kernel'
make: *** [Makefile:11: all] Error 2

I had the same issue. You need to either edit /kernel/Makefile to remove the extra '-o' from the command line, or remove it from ASMFLAG in /kernel/arch/x86/config.make.


feat: added default alert case for failing replication

Hey,
as long as #2800 is in the wild I thought it would be a good idea to add a default alert case to warn the user if this edge case occurs.
The alert was tested against a broken and a working cluster and performed as expected (also tested with single replica cluster)

@Lennart01, looks like the DCO step is failing, can you fix that? see https://github.com/cloudnative-pg/cloudnative-pg/pull/2869/checks?check_run_id=17206660410 for more details 🙏

@phisco Sorry about that. Should be fixed 👍


Updated link to PPTX file with correct one

@jasonhand @dnblankedelman, I think any references to slides should redirect to the file presentation.md included in the folder. This file is maintained automatically to always contain the last version.

Thanks so much for sending in a PR to fix this broken link! I was just about to fix it this morning myself, glad you got to it first..
Unfortunately this PR points it at a different place than the convention established for all of the learning paths, so I'm going to close/reject it without applying it. Look for a slightly different correction within an hour or so.
Thanks again for your contribution.


Add documentation for %trace

This PR adds in a short section under this docs page that explains how to visualize Q# programs with the %trace magic command.

Docs Build status updates of commit eababa1:
:clock10: Pending: waiting for processors (5 builds ahead of you)
:warning: Docs Build is busy, currently there are 5 builds ahead of this one, for more information you can view the Build queue graph on the Docs Portal.

Acrolinx Scorecards
A minimum Acrolinx score of 80 is required.
Click the scorecard links for each article to review the Acrolinx feedback on grammar, spelling, punctuation, writing style, and terminology.

Article
Acrolinxscore
Lowestcategoryscore
Spellingissues
Scorecard
Processed

articles/user-guide/host-programs.md
80
47
4
link
:white_check_mark:

More information about Acrolinx

Install Acrolinx locally for VSCode
False positives or issues
Troubleshooting issues with Acrolinx

Docs Build status updates of commit eababa1:
:warning: Validation status: warnings

File
Status
Preview URL
Details

articles/user-guide/host-programs.md
:warning:Warning
View
Details

articles/media/trace_example.png
:white_check_mark:Succeeded
View

articles/user-guide/host-programs.md

Line 571, Column 0: [Warning-hard-coded-locale] Link 'https://docs.microsoft.com/en-us/qsharp/api/iqsharp-magic/trace' contains locale code 'en-us'. For localizability, remove 'en-us' from links to most Microsoft sites.

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel

Docs Build status updates of commit ea04de3:
:clock10: Pending: waiting for processors (5 builds ahead of you)
:warning: Docs Build is busy, currently there are 5 builds ahead of this one, for more information you can view the Build queue graph on the Docs Portal.

Acrolinx Scorecards
A minimum Acrolinx score of 80 is required.
Click the scorecard links for each article to review the Acrolinx feedback on grammar, spelling, punctuation, writing style, and terminology.

Article
Acrolinxscore
Lowestcategoryscore
Spellingissues
Scorecard
Processed

articles/user-guide/host-programs.md
80
47
4
link
:white_check_mark:

More information about Acrolinx

Install Acrolinx locally for VSCode
False positives or issues
Troubleshooting issues with Acrolinx

Docs Build status updates of commit ea04de3:
:warning: Validation status: warnings

File
Status
Preview URL
Details

articles/user-guide/host-programs.md
:warning:Warning
View
Details

articles/media/trace_example.png
:white_check_mark:Succeeded
View

articles/user-guide/host-programs.md

Line 571, Column 0: [Warning-hard-coded-locale] Link 'https://docs.microsoft.com/en-us/qsharp/api/iqsharp-magic/trace' contains locale code 'en-us'. For localizability, remove 'en-us' from links to most Microsoft sites.

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel

Docs Build status updates of commit c091650:
:clock10: Pending: waiting for processors (5 builds ahead of you)
:warning: Docs Build is busy, currently there are 5 builds ahead of this one, for more information you can view the Build queue graph on the Docs Portal.

Acrolinx Scorecards
A minimum Acrolinx score of 80 is required.
Click the scorecard links for each article to review the Acrolinx feedback on grammar, spelling, punctuation, writing style, and terminology.

Article
Acrolinxscore
Lowestcategoryscore
Spellingissues
Scorecard
Processed

articles/user-guide/host-programs.md
80
47
4
link
:white_check_mark:

More information about Acrolinx

Install Acrolinx locally for VSCode
False positives or issues
Troubleshooting issues with Acrolinx

Docs Build status updates of commit c091650:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

articles/media/trace_example.png
:white_check_mark:Succeeded
View

articles/user-guide/host-programs.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel

Docs Build status updates of commit ebc0f43:
:clock10: Pending: waiting for processors (6 builds ahead of you)
:warning: Docs Build is busy, currently there are 6 builds ahead of this one, for more information you can view the Build queue graph on the Docs Portal.

Acrolinx Scorecards
A minimum Acrolinx score of 80 is required.
Click the scorecard links for each article to review the Acrolinx feedback on grammar, spelling, punctuation, writing style, and terminology.

Article
Acrolinxscore
Lowestcategoryscore
Spellingissues
Scorecard
Processed

articles/user-guide/host-programs.md
80
47
4
link
:white_check_mark:

More information about Acrolinx

Install Acrolinx locally for VSCode
False positives or issues
Troubleshooting issues with Acrolinx

Docs Build status updates of commit ebc0f43:
:warning: Validation status: warnings

File
Status
Preview URL
Details

articles/user-guide/host-programs.md
:warning:Warning
View
Details

articles/media/trace_example.png
:white_check_mark:Succeeded
View

articles/user-guide/host-programs.md

Line 571, Column 127: [Warning-xref-not-found] Cross reference not found: 'microsoft.quantum.iqsharp.magic-ref.trace'.

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel


[10.x]: improve job release method to accept date instance

Don't forget to send an invoice to Tyler for doing work on this framework. As it is sponsored and makes money. You deserve to be paid for your work.


Fix size issue for ARVR managed viewport

This fix applies to 3.x only.
This fixes an issue around an ARVR Viewport not rendering if its size is 0, 0 even though the size is managed by the ARVRInterface.
It also fixes an issue around Godot getting stuck when attempting to render a Viewport not properly initialized because there is no ARVRInterface that has been initialized (a likely scenario when running the editor)
It simply moves the size check to slightly earlier in the rendering process, and set the size to 0, 0 if no ARVRInterface is initialized.
fixes #56069

Thanks!

Cherry-picked for 3.4.2.

Thanks @BastiaanOlij !
This was causing me some issues when migrating my VR project to 3.4. 🥳


iOS mobile windowing-related performance regression and log spam

Bevy version
3d72e08
[Optional] Relevant system information

cargo 1.82.0 (8f40fc59f 2024-08-21)
xcode 16.1 (16B40)

What you did
Just running the bevy_mobile_example.xcodeproj
What went wrong
Console spamming of:
Monitor removed 14v3908#16784732192782
processing non `RedrawRequested` event after the main event loop: AboutToWait
Monitor removed 4v3083#13241384173572
processing non `RedrawRequested` event after the main event loop: AboutToWait
Monitor removed 14v3909#16789027160078
processing non `RedrawRequested` event after the main event loop: AboutToWait
Monitor removed 4v3084#13245679140868
processing non `RedrawRequested` event after the main event loop: AboutToWait
Monitor removed 14v3910#16793322127374
processing non `RedrawRequested` event after the main event loop: AboutToWait
Monitor removed 4v3085#13249974108164
processing non `RedrawRequested` event after the main event loop: AboutToWait
Monitor removed 14v3911#16797617094670
processing non `RedrawRequested` event after the main event loop: AboutToWait
Monitor removed 4v3086#13254269075460
processing non `RedrawRequested` event after the main event loop: AboutToWait
Monitor removed 14v3912#16801912061966
processing non `RedrawRequested` event after the main event loop: AboutToWait
Monitor removed 4v3087#13258564042756
processing non `RedrawRequested` event after the main event loop: AboutToWait
Monitor removed 14v3913#16806207029262
processing non `RedrawRequested` event after the main event loop: AboutToWait
Monitor removed 4v3088#13262859010052

Otherwise visually the example seems to run fine in the simulator.
here is the log of the same example on bevy 14.2: https://gist.github.com/extrawurst/7cbbb5b1918b5876a93705b92d65e196
here is the entire log of the above example in 3d72e08:
https://gist.github.com/extrawurst/b91117297242d412091862db171e29ab
If you compare this to the 14.2 behaviour there are a few things very different:

we don't have timestamps anymore
instead of just the old friend processing non RedrawRequested event after the main event loop: AboutToWait we now seem to get this second log every frame: Monitor removed 4v3088#13262859010052
the startup produces an absolutely massive amount of additional logs if you compare both full logs

I wonder if this is all intended? The repeating Monotitor removed at the very least looks very wrong.
Additional Information
This additional activity seems to mirror also the huge additional CPU load:

Average CPU usage in the example goes up from ~90% to >200%. This is a regression in my books.

ok instrumentation reveals that we spent 15% in bevy_winit::system::create_monitors:

The reason for this is because we detect a new unseen monitor each frame and spawn it: https://github.com/bevyengine/bevy/blob/75c92fb47b1d971134b8b96d83f1a054e1e87623/crates/bevy_winit/src/system.rs#L166
just to delete the old one from last frame: https://github.com/bevyengine/bevy/blob/75c92fb47b1d971134b8b96d83f1a054e1e87623/crates/bevy_winit/src/system.rs#L202
which triggers exactly the logging line that we see spamming: Monitor removed 4v3086#13254269075460
I am not sure yet why this is. Its almost as if we can't seem to rely on the MonitorHandle::eq check. @tychedelia did you run into this when adding this via 3360b45 ?

I am not sure yet why this is. Its almost as if we can't seem to rely on the MonitorHandle::eq check. @tychedelia did you run into this when adding this via 3360b45 ?

No, this is not the behavior that we see on desktop, and I don't have a way to test mobile. But this is a good call out, we should investigate on the winit side as the id for the monitor should be stable, it's possible this is a regression there.

I also imagine that there's a lot of cost to the info! log while it's spamming. Not that this is a "good" solution but compiling out that log may at least help the perf if we want a stop gap solution.

I also imagine that there's a lot of cost to the info! log while it's spamming. Not that this is a "good" solution but compiling out that log may at least help the perf if we want a stop gap solution.

60% of this is the span of info and 40% is the hotpath Vec::retain

UiKit:
impl PartialEq for MonitorHandle {
    fn eq(&self, other: &Self) -> bool {
        ptr::eq(self, other)
    }
}

AppKit:
impl PartialEq for MonitorHandle {
    fn eq(&self, other: &Self) -> bool {
        unsafe {
            ffi::CGDisplayCreateUUIDFromDisplayID(self.0)
                == ffi::CGDisplayCreateUUIDFromDisplayID(other.0)
        }
    }
}

@madsmtm would be great to have your opinion

Indeed, that's wrong, Winit is comparing the pointer address of MonitorHandle itself (not the pointer address of the inner UIScreen). I've put up https://github.com/rust-windowing/winit/pull/4013, though don't have the time rn to test if that's actually enough.

The monitor log spam actually makes the real issue visible: we're running about_to_wait as fast as possible. create_monitors eats 15% cpu because we're running it 5000 times per second on my simulator.
Fixing it is a good thing, but we should look again at how we handle the wind loop

@mockersf your workaround can be swapped against proper monitor handle partialeq: https://github.com/rust-windowing/winit/releases/tag/v0.30.6


Secure Plugins

TODO

[ ] create a seccomp profile for different plugin types
[ ] create default cgroup settings
[ ] drop most capabilities

https://docs.docker.com/engine/security/seccomp/
https://docs.docker.com/engine/admin/resource_constraints/
seccomp
HostConfig. SecurityOpt
cgroup settings
HostConfig.Resources.Memory
drop most capabilities
HostConfig.CapAdd
HostConfig.CapDrop

See https://github.com/maliceio/registry


Update readme with prometheus metrics (7 lines)

Thanks @mlissner ! BTW I'm also working on configuration with viper, so you can now also configure e.g the metrics address via the command line or env variables.

Amazing!


Unable to submit expenses due to: ERROR Wise: There was an error while creating Wise's recipient

On the last page for generating a expense, when clicking the submit form, the following pop-up appears and the form cannot be submitted:

In the past there has not been this problem, is a Wise account now required for bank transfers when receiving money? If that is the case it doesn't seem very clear in the documentation? Or is there some other issue here?
Desktop (please complete the following information):

OS: Debian 11.4
Browser: Mozilla/5.0 (X11; Linux x86_64; rv:102.0) Gecko/20100101 Firefox/102.0

This should be resolved through support.


rm: if vm in pod is still present, wait it stop before remove resource

use lock protect unset pod.vm
when delete, if vm is present, set pod status to be None
when vm stopped, if status is None, reclaim the resource
when list, do not display the removed pods

Signed-off-by: Xu Wang<EMAIL_ADDRESS>

seems like StopPod(WithinLock) holds the lock of pod and then set the p.vm to nil first, then the PodStopped will not cleanup resource, should we stop setting p.vm to nil in StopPod, leave this job to PodStopped.

@gao-feng, I agree with you, set p.vm to nil only in podStopped. will update this pr in couple of minutes.

LGTM


[Clarification] Discrepancy between VALSE's Single Image Setup and Winoground's Two-Image Setup

I've been exploring the VALSE dataset and I've observed that its structure seems similar to the Winoground dataset. However, I noticed a key difference: while Winoground employs two images, the VALSE dataset appears to use only one.
Could someone clarify how the second image is derived or if there's an underlying reason for this design choice?
Thanks in advance!

For the Winoground, the second image is manually collected, so the scale is very small. While for our EqBen, we borrow the idea from the video data format, which provides large-scale paired images that can be used to diagnose the MLLM.


DrawRoadManually "you cannot make line with same geopoint"

Hi,
I'm using the method drawRoadManually and i've the following geopoint list:
[GeoPoint{latitude: -6.158108 , longitude:<PHONE_NUMBER>}, GeoPoint{latitude: -6.1582128 , longitude:<PHONE_NUMBER>}]
but when calling the method it throws the following error:
if (path.first.latitude != path.last.latitude &&
       path.first.longitude != path.last.longitude &&
       path.length < 3) {
     throw Exception("you cannot make line with same geoPoint");
   }

shouldn't be this code like this instead
if (path.first.latitude == path.last.latitude &&
       path.first.longitude == path.last.longitude &&
       path.length < 3) {
     throw Exception("you cannot make line with same geoPoint");
   }

so check if they are equals instead of different?

Oh im so dump
i will fix it thnx

Don't say that :) it happens you did a great job!!!

check the new version 0.70.0

Everything working :)
I had to do a flutter clean for Android to work (else have an issue with the build - maybe something to write on the changelog)


Add buttons to the Trades view to delete swap history

I think one button to delete all and one button to only delete failed ones would be useful. Deleting all would have a confirm prompt.
@atomiclabs/core Thoughts?

Yeah I like it.
One thing to bear in mind is that we need to prevent clearing all swaps while some swaps are still in progress.

Where would these be added? To the top right?

@kevva Yes, I think top-right would be the best place for them.


Support for unblocked user tags

Add support for users who are not blocked, but tagged
Fixes #9

That should work. notBlocked takes the opposite of whether someone is blocked. Therefore, I've changed the parameter name to isNotBlocked and swapped yes / no to reflect this.

I've made those two changes as noted by you.

I've made those two changes as noted by you.


[feature-request] using multiple frameworks while running features

I would like to request a thing that I am really missing here.
That you define which frame work a feature should run on.
e.g.
@framework('protractor')
Feature x
@framework('webdriverio')
Feature y
To what degree this is feasible I don't know but I think it would be cool. Also a good thing for if you want to migrate from 1 framework to another.

Hey Rob, you can achieve this with dynamic configuration: https://codecept.io/advanced#dynamic-configuration


Parse cell line ids from PaxDB metadata

Ideally, we would convey the id of each cell line to users. Unfortunately, this is hard to parse from PaxDB as they don't represent this consistently.

Inquired about adding the information in the next database release.


Go sched scheduler latency

This adds the scheduler queue latency metric to go-metric. This metric is interesting when profiling a go program.
As go-metrics don't support histograms and this metric is exposed as histogram in go runtime library, I chose to expose it in the following form bucket*count. This is not ideal as it's factor of total count of active goroutine so it could be misleading in some cases. That said, total count of goroutines is also available as a different metric, so it could help disambiguous this.
Also, I think this is sufficient to measure the overall scheduling pressure caused by CPU bottlenecks or bugs in a given go program.

@banks when you have time can you please check this? It would be nice to add this in Consul for perf analysis purpose.

Closing this as it need more work to get the right metrics exposed!