Reduce delegate allocation overheads

Tested at osu! song select while scrolling. I've only addressed the cases which are excessive for now.
Before:

After:

Main targets are DelayedLoadWrapper and mouse movement.
Note that DelayedLoadWrapper still allocates quite a few ScheduledDelegateWithData, but this is harder to avoid. Still an improvement.

Looking at SDL2DesktopWindow, is scheduling all events even necessary?
https://github.com/ppy/osu-framework/blob/f9ae8d775ba2d03196c3ec80228b009c3dc85ab0/osu.Framework/Platform/SDL2DesktopWindow.cs#L454-L459
SDL events are handled by handleSDLEvent() in pollSDLEvents(). All of this is run on the main thread, so it short-circuits and executes the "scheduled" events immediately.
The only place where the scheduler is actually used is here, where the event is force scheduled.
In your After comparision, I'm still seeing some handle...Event allocations. Could those be removed?

Yep they can probably all be removed, but they are quite rare and we're out of scope for this change.
Removing the scheduler altogether would be optimal if it doesn't increase code complexity. That could be done as a follow up.

Well I mean there is one usage where it's apparently needed as already stated, so I don't know about removing entirely.

Code seems fine, but just to confirm my understanding - the savings here are from potentially leaner closures, i.e. better control over which data is stored to the scheduler? It's not super explicit to me from the code/screenshots alone.
Like, to take the SDL2DesktopWindow as an example - calls like this will still incur a delegate alloc, right? Because as far as I remember references to delegates via method groups still alloc every time. I could imagine the size decreasing because the closure is potentially smaller (just the reference to the window, rather than a few vars), but I'm still not sure that is the case.

Never mind, I think the other PR makes this clearer. The savings are from eliminating the dependency of the Action on the data, which internally makes it so that an instance of the Action<T> can be cached and reused. And in the inline execution case there's no allocation of the "wrapper" structure that pairs the Action<T> with its data.
See following sharplab for a simpler example of this.

Yep, it's basically making the closure functions "static".
Although interestingly, this allocates still:
schedule(staticFunc, data);

static void staticFunc(obj data) {}

and i have no idea why it does. you have to use the inline no-capture delegate to avoid the allocation. maybe something to test in sharplab.

I think that's just what I mentioned earlier about Func<>s created via method groups always allocating? I.e. see https://github.com/dotnet/roslyn/issues/39869.
That's actually required by the language spec as is.


refactor: reenable swaps, closes leather-io/issues#98

This PR upgrades our use of the alex-sdk and reenables swaps. Sponsored tx code has been removed bc their sdk no longer supports sponsored txs.
Successful swap using their new amm-pool-v2-01:

Summary by CodeRabbit

New Features

Enabled the "swaps" feature in the wallet configuration.

Updates

Updated @leather.io/query to version 0.9.5 and alex-sdk to version 2.1.3 in package.json.

Improvements

Enhanced swap functionality and logic in the AlexSwapContainer component.
Improved fee calculation and swap transaction broadcasting.

Code Quality

Renamed internal functions for consistency and clarity.

I merged this bc I am out tom, if there are CR comments to address pls let me know and I'll refactor.


WaterTight

How could I measure the watertight rate of result mesh ?

For the results in the paper, watertightness was measured by counting the number of edges which have != 2 incident triangles. So for instance percent watertightness might be given by:
100 * num_edge_with_2_tris / num_total_edge

Here's a little snippet of Python code that might help compute some of these counts. (I grabbed this from a random codebase and haven't tested recently, because sure to verify it before trusting blindly!)
    verts, faces = #your mesh

    halfedges = []
    for i in range(3):
        ind_low = np.minimum(faces[:,i], faces[:,(i+1)%3])
        ind_high = np.maximum(faces[:,i], faces[:,(i+1)%3])
        halfedges.append(np.stack((ind_low, ind_high), axis=-1))
    halfedges = np.concatenate(halfedges, axis=0)

    u_vals, u_inv, u_count = np.unique(halfedges, axis=0, return_inverse=True, return_counts=True)

    n_distinct = u_vals.shape[0]
    n_1 = np.count_nonzero(u_count == 1) # number of edges with 1 face incident
    n_2 = np.count_nonzero(u_count == 2) # number of edges with 2 faces incident
    n_3p = np.count_nonzero(u_count > 2) # number of edges with >=3 faces incident

For the results in the paper, watertightness was measured by counting the number of edges which have != 2 incident triangles. So for instance percent watertightness might be given by:
100 * num_edge_with_2_tris / num_total_edge

Here's a little snippet of Python code that might help compute some of these counts. (I grabbed this from a random codebase and haven't tested recently, because sure to verify it before trusting blindly!)
    verts, faces = #your mesh

    halfedges = []
    for i in range(3):
        ind_low = np.minimum(faces[:,i], faces[:,(i+1)%3])
        ind_high = np.maximum(faces[:,i], faces[:,(i+1)%3])
        halfedges.append(np.stack((ind_low, ind_high), axis=-1))
    halfedges = np.concatenate(halfedges, axis=0)

    u_vals, u_inv, u_count = np.unique(halfedges, axis=0, return_inverse=True, return_counts=True)

    n_distinct = u_vals.shape[0]
    n_1 = np.count_nonzero(u_count == 1) # number of edges with 1 face incident
    n_2 = np.count_nonzero(u_count == 2) # number of edges with 2 faces incident
    n_3p = np.count_nonzero(u_count > 2) # number of edges with >=3 faces incident

OK, very thanks for your help!

OK, very thanks for your help!


remove redundant codecov ci token in lvm and zfs

CI jobs for codecov reports errors related to token not existing in the lvm and zfs localpv repos. We should remove that.

I think this is regarding specific lvm and zfs repo, so should be reopen against those.


[Key Vault] Resolve next-pylint and next-mypy errors

Description
The majority of errors were resolved by removing :class: markers in docstring type hints.
All SDK Contribution checklist:

[x] The pull request does not introduce [breaking changes]
[x] CHANGELOG is updated for new features, bug fixes or other significant changes.
[x] I have read the contribution guidelines.

General Guidelines and Best Practices

[x] Title of the pull request is clear and informative.
[x] There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For more information on cleaning up the commits in your PR, see this page.

Testing Guidelines

[x] Pull request includes test coverage for the included changes.

Validating run: https://dev.azure.com/azure-sdk/internal/_build/results?buildId=3329894&view=results

API change check
API changes are not detected in this pull request.

Validating run: https://dev.azure.com/azure-sdk/internal/_build/results?buildId=3346664&view=results


OS X Lion's uname -m does not lie

The rust configure script does the following check to detect x86_64:
CFG_OSTYPE=$(uname -s)
CFG_CPUTYPE=$(uname -m)
if [ $CFG_OSTYPE = Darwin -a $CFG_CPUTYPE = i386 ]
then
  if sysctl hw.optional.x86_64 | grep ': 1'
  ...

This makes the assumption that OS X is lying about uname -m. Indeed, on earlier versions, it always returns i386. However, on Lion it properly returns x86_64. The second portion of the conditional should be removed probably.

We'll have to support pre-Lion releases for a while yet.

The other thing is I noticed when it selects a triple containing x86_64, it defaults to i686 instead :. I tried manually editing that to x86_64 instead, and got:
Exception: no snapshot file found for platform macos-x86_64, rev 7b95b5c
I'm guessing this means a binary for bootstrapping doesn't exist yet?

Closed.


feat: Capture spans for Celery's apply_async

Just throwing this up for visibility/feedback. I think it'd be a useful stock instrumentation, but maybe there are better attributes to expose as part of it.


docs(readme): include cdn for dev and prod

Free CDN from

https://rawgit.com/
closes #127

Chose to use cdnjs.  RawGit specifically says it's not super reliable for production use.  Thanks though!

np


How do I get token to make requests?

I'm succesfully logged but now how do I get my tokens to make request to the twitter api?
In the callback function I can see the tokens with
console.log(req.sessionStore.sessions[req.session.id]);
output:
{"cookie":{"originalMaxAge":null,"expires":null,"httpOnly":true,"path":"/"},"passport":{},"flash":{},"oauth:twitter":{"oauth_token
":"oG1OMeDMlb43bw9hBgsfsdf47RvbjKOuE","oauth_token_secret":"BvNYdf545GOn1exxQ4vI8RO7M5bBi0gna"}}
But the problem is that I can't fetch data after the session id.
req.sessionStore.sessions[req.session.id]["oauth:twitter"]
This is 'undefined' as well as .cookie...

All tokens are stored in the Passport model: https://github.com/kasperisager/sails-generate-auth/blob/master/templates/api/models/Passport.js#L61-L67

I'm sorry I'm a bit new to sails but how would I acces it then?

If you're using blueprints you should be able to access passports associated with a user out of the box as associated models will be joined in.


Allow collision option

I added an option regarding #16. this option will work as follows.

If collision occurs, the position of the preview will be exchanged with that of the collision target.
If the position exchange leaves a blank space, all items will be compressed to the top where they can be.
It will deprecate "rows" option (fix to 0).
I mainly referred to https://jbaysolutions.github.io/vue-grid-layout/guide/01-basic.html

Hey @hearthole. Thanks a lot for your work 💖💖💖💖

It seems like you forgot to commit getCollisions function 👀

getCollisions is default method that you made. do you have any problems to test?

getCollisions is default method that you made. do you have any problems to test?

That was unexpected. I saw a new function import and assumed that its new. Forgot that it is part of lib already 😬

I'll merge this right after the tests, but rewrite some parts to adapt it to the new API and maybe change the it a bit. Will be part of 1.0

I got it. thank you for awesome project. 😄

can you check my new commit?

Good idea. I think it requires further development too.


Enhance vertex positioning to avoid overlapping

The circles in the isomorphism game are overlapping edges which they are not connected to. This makes the game hard to play esp. on mobile devices.
I changed the randomizePositions method to prevent a) circles to be placed on previously placed edges and b) prevent edges from a newly placed circle to overlap other circles/vertices.
The lineCircleCollide function was taken from https://github.com/mattdesl/line-circle-collision. I think copy/pasting it to this demo is easiest for now rather than integrating e.g. browserify or some other build process..

Hi, Jan
This is an admirable goal.  There are a few things I would ideally like to change before merging this:

Style: variable naming, semi-colons, etc. should be consistent.
I would put this code in the GraphViewModel in graf.js rather than making it specific to the game.  That is where any reusable placement algorithms should go, and this is a desirable property for all graph views, and shouldn't be too tied to the notion that the vertices are represented by circles.
I would like to avoid complexity and code bloat in this part of the code, as it was just a quick-and-dirty approach to begin with.  I would prefer a more creative approach, like putting all of the vertices along an ellipse, or a grid (with easier checks for overlap) - something which could have the desired property with much less trial and error.

I may just merge it in the meantime once I test it out, but I will probably take a pass at replacing this code eventually, rather than keeping it long term.  There are other fit and finish issues on mobile, such as vertices hiding under the big title, and that the touch targets are too small.  I think some of this discomfort and difficulty can be alleviated by just giving the graph a bit more room to breathe, increasing the size of the touch targets, while keeping the vertices small visually.  In theory, there should be no more problem on mobile than desktop when it comes to vertices overlapping unrelated edges, but they could both use improvement.


How do I breakup with my boyfriend?

I tried deleting his Pokemon Go account but he just doesn't get the message.

The fuck?

sounds like your BF needs a new GF.

agreed.

Now I'm in suspense. Did you break up?

@chucklesak Apparently removing every pokemon with 1000+ cp isn't enough for him to just give up

And posting on this message board is only going to make it worse for your goal. Try cosmo.


Short solution needed: "Oneclasssvm" (python-scikit-learn)

Please help us write most modern and shortest code solution for this issue:
Oneclasssvm (technology: python-scikit-learn)
Fast way
Just write the code solution in the comments.
Prefered way

Create pull request with a new code file inside inbox folder.
Don't forget to use comments explain solution.
Link to this issue in comments of pull request.

Using OneClassSVM for outliers detection example


onInit, onClick events with form validation?

How can I use this with your plugin?
https://developer.paypal.com/docs/checkout/integration-features/validation/#asynchronous-validation
I tried to modify it but it seems that this part of PayPal npm package is missing actions:
onClick: {
      type: 'function',
      required: false,
      noop: true,
      decorate: function decorate(original) {
        return function decorateOnClick(data) {

Long story short how can you validate form first and then allow PayPal actions to proceed?

@goors Where you able to solve this somehow?

@Kinzi  nope. I was never able to resolve this. that PayPal button sucks!


Add details of last few patches to changelog

Is this actually an issue or just a PR that nobody has written yet?

An issue. Gulp is at 3.8.11 but the changelog doesn't say what has changed since 3.8.5

@callumlocke Just dep updates to pull in fixes from other projects AFAIK and maybe some docs, no API changes

Done


Render Franchise ID, not Franchise

Pull Request Test Coverage Report for Build 344079999

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 88.651%

Totals

Change from base Build 340540629:
0.0%

Covered Lines:
414

Relevant Lines:
467

💛  - Coveralls


ImportError on Flask-Babel

Hi Miguel,
I'm having trouble working with the flask_babel module. While running the whole project locally using directly the python script I can run it correctly. However, when I try to run it behind nginx, via uwsgi I get the following traceback on the uwsgi service:
uwsgi[8342]: Traceback (most recent call last): uwsgi[8342]:   File "./wsgi.py", line 1, in <module> uwsgi[8342]:     from server  import app uwsgi[8342]:   File "./server.py", line 7, in <module> uwsgi[8342]:     from flask_babel import Babel, lazy_gettext as _l uwsgi[8342]: ImportError: No module named flask_babel
I'm using virtualenv to run python and all the modules, I've already tested that the Flask-Babel module is installed by importing it in the console (while on the virtual environment). I've also tested it outside the virtual environment and have corroborated that the module is indeed installed there as well.
I understand this is not a direct issue with Flask-Babel or any of the sort, but was wondering if you had encountered a similar issue before. Thanks a lot!
Jorge.
PS: I'm not attaching code at this point as it is my understanding that it might be some configuration issue, if it is necessary I can post my code.

Did you tell uwsgi to use your virtualenv?

Hi Miguel,
Yes I did. Couldn't figure what was wrong, so I reinstalled the whole virtualenv with all the modules from scratch. That seemed to have fixed the problem. However, I couldn't figure out what was causing the problem.


US SSN recognizer very weak match test fails

Describe the bug
I'm not sure if this is a bug, but one of the recognizer tests is disabled with the comment TODO figure out why this fails here: https://github.com/microsoft/presidio/blob/master/presidio-analyzer/tests/test_us_ssn_recognizer.py#L20-L21
The very weak match tests use "078-05112" and "07805-112" as input, but UsSsnRecognizer's invalidate_result() function rejects SSNs with only one delimiter: https://github.com/microsoft/presidio/blob/master/presidio-analyzer/presidio_analyzer/predefined_recognizers/us_ssn_recognizer.py#L51
The comment in invalidate_result() suggests the purpose of the delimiter check is to reject mismatched delimiters. If that's all it's supposed to do, then I think this is a bug (and the function should probably check the number of delimiters before considering it mismatched).

@diwu1989 perhaps you can take a look?

SSN recognizer is failing in demo too. If you load https://presidio-demo.azurewebsites.net/ SSN is exposed. If you edit it, it fixes the issue. For example delete 078 and put it back; now it recognizes it and then its anonymized.

Thanks @shridhardamle! We'll take a look.

Appears to be fixed in demo https://presidio-demo.azurewebsites.net/


Update to Alpine v3.20

and also remove some unnecessary comments from the Debian Dockerfile.

That's really good timing.  Was thinking to myself the other day "wonder when the next Alpine release comes out?" but didn't even look into it.
And you've already taken care of it. :grin:

if I find some time, I'll fork the repository and experiment if we could automate the updates for Alpine and the different PG versions using https://docs.renovatebot.com/. but I think for now we are set :)

Oh, that's an interesting idea.  Yeah, pretty much fully automated updating from there on sounds like it'd be useful. :smile:


AddManagedDependency: Allow wildcard only match on artifact

Hi,
the onlyIfUsing' condition currently does not allow a wildcard only match on the artifact, e.g. org.example.group:*`.
I made the PR #1447 for the extended regex to allow wildcard-only match on the artifact. I made also two minor changes:

Set required = false on onlyIfusing as it works without
Annotated addToRootPom with Nullable

If you want to have a look at extended regex, see: https://regex101.com/r/eJZfi3/1
Kind regards
Thomas

Thanks @thomaszub :)

closed via #1447


video

https://github.com/user-attachments/assets/20fc4543-e6ec-4d56-9a64-8a7bcaa828c3

https://github.com/user-attachments/assets/a8ea9150-8dc6-4d99-8f0b-ba2ed34229f0

https://github.com/user-attachments/assets/adb200f3-02df-4a7f-9e1a-10335c5a3340


fixed login and signup buttons

#1066  Fixed the position of login and signup buttons in navbar

Conflicting files
index.html
Resolve the conflicts in the mentioned files. Ensure you pull all the latest changes before pushing your feature/bug fixes to the main branch. After resolving the conflicts, please open a new PR for review.
Thank you!


GetAlarms API Update for Large Data Set

Background
With the current approach, since Grafana need to loop through all the entities in TwinMaker, which may not be able to scale for large systems (e.g. with 100K entities). This is the update to resolve the issue.
Current Workflow
Steps

(Result is cached) Call API ListComponentTypes to get all component types which extend from the basic alarm component type "com.amazon.iottwinmaker.alarm.basic"
(Result is cached) Call API ListEntities with all the component types from the last step.
(Result is cached) Call API GetEntity for all the entities from the last step to get the entity detailed information (includes componentName).
Build a HashMap for externalID->entityId/componentName lookup
Call API GetPropertyValueHistory for all the componentTypes from step #1, and then use the externalIDs from API response with the HashMap from step #4 to add entityId/componentName into the response.

Traffic Estimation: for a customer with 100K entities, 10 alarm components and 20 active alarms, the number of API calls are: 100,000 + 10 * 2 + 1
New workflow
Steps

(Result is cached) Call API ListComponentTypes to get all component types which extend from the basic alarm component type "com.amazon.iottwinmaker.alarm.basic"
Call API GetPropertyValueHistory for all the componentTypes from step #1
For each data frame, search in-memory cache for the externalID, if exists, use cache to add entityId/componentName into the response, otherwise:
3.1 Call API listEntities with externalID as filter to find the entity summary.
3.2. Call API GetEntity with the entityID from step #3.1 to find the entity componentName.

Traffic Estimation: for a customer with 100K entities, 10 alarm components and 20 active alarms, the number of API calls are: 20 + 20 + 10 + 1

@atifali From the last conversation, we want to combine the implementation of GetComponentHistory and GetAlarm queries. They will still be two separate options in the query type selection, but should have the same logic to get data.
For the data frame returned from GetComponentHistory, it should have the same field as current, but have the "entityId" and "componentName" in labels.
For GetAlarms, we can still have the same format as current, but make sure it has the values of entityId, componentName, status and time returned. then I can grab those values in scene viewer to convert to the data frame I need.

Highest priority


Tag a release

Could you create the tag 1.0.0 to facilitate the integration from bower ?

Thank you for the suggestion, done! This was more an internal project I needed and the rest of libraries had the problem of sync with ng-model. I used to consume it through bower even though no release was there since it's published in bower.
Assuming it done, closing!

Thanks!


ARTEMIS-1954 eliminate all possible usage of JMSServerManager

Nice!!!
I haven't checked every single class change (56 wow, must have been a while) but randomly checking a sample, and some of the more core classes, looks ok to me.

nice one @jbertram


BigQuery: support unquoted hyphen in table/view declaration

Adds support for proper object name parsing in bigquery to within a CREATE TABLE or CREATE VIEW statement. Similar to the parsing logic for when the table/view is later referenced.
The lacking part was that bigquery allows unquoted hyphens in the first part of an object name - so that e.g.
CREATE TABLE my-project.mydataset.mytable
wasn't being parsed correctly.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

34 of 37 (91.89%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.004%) to 87.874%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

tests/sqlparser_bigquery.rs
28
31
90.32%

Totals

Change from base Build<PHONE_NUMBER>:
0.004%

Covered Lines:
20516

Relevant Lines:
23347

💛  - Coveralls


Removing dependency for a Device/Service List and use Configuration

Fix #1662

Modified the security-proxy-setup configuration.toml file(s) docker and local
Removed the use of the go-mod-core-contracts use of the ServiceList "statically defined"
Removed the use of the ClientInfo struct and included the new ServiceInfo struct "now featured in the toml config"
Modified the security/proxy/service.go init func simplifying the KongRoute(s) creational process
Updated any required helper func(s) as required for the change from ClientInfo to ServiceInfo

Signed-off-by: xmlviking<EMAIL_ADDRESS>

Codecov Report

Merging #1779 into master will decrease coverage by 0.13%.
The diff coverage is 12.5%.

@@            Coverage Diff            @@
##           master   #1779      +/-   ##
=========================================
- Coverage   38.33%   38.2%   -0.14%
=========================================
  Files         149     149
  Lines       11840   11840
=========================================
- Hits         4539    4523      -16
- Misses       7098    7115      +17
+ Partials      203     202       -1

Impacted Files
Coverage Δ

internal/security/proxy/init.go
0% <0%> (ø)
:arrow_up:

internal/security/proxy/service.go
68.57% <0%> (-4.6%)
:arrow_down:

internal/security/proxy/config.go
54.54% <28.57%> (-12.13%)
:arrow_down:

Continue to review full report at Codecov.

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

Awaiting Tinguz approval before merge :)

Let me run the test to make sure the feature is working as expected then will approve.

Finally, I think it would be good to get @michaelestrin to look at this PR and comment, since this is solving what is essentially a very similar problem to what the SMA has from #1771 but there only the registry is queried and there's no additional configuration in the configuration.toml file whereas this implementation has the requirement to be enumerated in the configuration.toml and not go to consul.

The changes proposed in #1763 (upon which #1771 is built) remove the population of GenClients using the result from config.ListDefaultServices() when the --registry flag is set.  This follows the assumption that when the --registry flag is set, the registry is the source of truth for configuration.  It also assumes there is little to no benefit to front-load the population of GenClients since the implementation handles population on-the-fly when a requested service does not yet have a corresponding GenClients' entry.
When the --registry flag is not set, GenClients is populated from the information contained in the configuration.toml file.
These changes appear to be specific to the security-proxy-setup service.  It's not clear to me when that service executes relative to the other system services, whether or not the registry would ever be up and running, and whether or not the registry would contain configuration specific to the security-proxy-setup service.  If someone could provide clarity,  I would be able to comment further.

whether or not the registry would ever be up

yes consul needs to be up by the time security-proxy-setup starts because security-proxy-setup specifically will only be started after Kong is ready in consul and security-secrets-setup is ready in consul (or at least this will be enforced for Fuji)

whether or not the registry would contain configuration specific to the security-proxy-setup service

yes it could contain configuration.toml information

Based on clarification from @anonymouse64 above, I concur the implementation here for retrieving and populating configuration should follow the pattern established elsewhere.  That is, configuration should be retrieved from the registry (if the --registry flag is set) or from the service's configuration.toml (if the --registry flag is not set).
This implies config-seed will populate the registry with the necessary configuration for the security-proxy-setup service.  If it doesn't currently, I would consider those changes in-scope for the parent issue (i.e. they should be in a PR -- this one or another -- tied to the parent issue).
@xmlviking

Based on clarification from @anonymouse64 above, I concur the implementation here for retrieving and populating configuration should follow the pattern established elsewhere. That is, configuration should be retrieved from the registry (if the --registry flag is set) or from the service's configuration.toml (if the --registry flag is not set).
This implies config-seed will populate the registry with the necessary configuration for the security-proxy-setup service. If it doesn't currently, I would consider those changes in-scope for the parent issue (i.e. they should be in a PR -- this one or another -- tied to the parent issue).
@xmlviking

The timing on this will not work as I understand the security and initialization process works at this time. The service registry will not be populated prior to this running.  There will be nothing for it to use.   This is an ask based on the current state of the codebase.

The timing on this will not work as I understand the security and initialization process works at this time. The service registry will not be populated prior to this running. There will be nothing for it to use. This is an ask based on the current state of the codebase.

I'm confused.  This information appears to conflict with @anonymouse64's above.  Are you saying config-seed won't be executed prior to the security-proxy-setup service?

I always assumed that config-seed would be taught to import configuration for the security services but perhaps this was just in my head...
Regardless, it seems like a decision is needed as to whether or not config-seed will import the configuration.toml's into consul for the security services. The only reason I think we couldn't use config-seed before the security services is if config-seed needs to access secrets itself. I think it doesn't need to, but I might be wrong.
If config-seed doesn't need secrets, then we can order config-seed to run before all of the security services and then security-proxy-setup can talk to Consul for this information. Note however that security-proxy-setup cannot use the health mechanism in consul for getting service config because potentially none of the services in edgex-go other than config-seed and the other security services would have run before security-proxy-setup is done, however accessing the config for other services should be fine even if the service is offline.
@tingyuz and @tsconn23 thoughts on config-seed importing configuration.tomls for the security services into consul and whether or not config-seed needs access to secrets itself?

The only reason I think we couldn't use config-seed before the security services is if config-seed needs to access secrets itself. I think it doesn't need to, but I might be wrong.

This is correct. The secrets will effectively override the default username/password info contained in the configuration.toml. So it makes sense to me that config-seed would run as early as possible in the pipeline.

Thanks for clarification @tsconn23!
I don't think this PR is updated to look through the registry for configuration, it seems to still only enumerate the stuff from configuration.toml... Perhaps it hasn't been pushed up here yet?

Thanks for clarification @tsconn23!
I don't think this PR is updated to look through the registry for configuration, it seems to still only enumerate the stuff from configuration.toml... Perhaps it hasn't been pushed up here yet?

Ian,
I'm working though the necessary changes which include the use of useRegistry flag during the init process for the security-proxy-setup service.  This change is a bit more invasive then the simply changes to the PR work previously.  I will get the PR updated accordingly.

The corresponding issue grew to include obtaining configuration from the registry.
Instead of implementing registry support from scratch, it may make more sense to undertake https://github.com/edgexfoundry/edgex-go/issues/1805 first.  Completion thereof would provide standardized registry support.
@xmlviking

The corresponding issue grew to include obtaining configuration from the registry.
Instead of implementing registry support from scratch, it may make more sense to undertake #1805 first. Completion thereof would provide standardized registry support.

In conversation with @tsconn23, @tingyuz, and @AnthonyMBonafide yesterday, agreed to put #1805 on hold until after October Fuji freeze.  This issue should be worked independently of #1805.

--recheck

recheck


Development complete

New Feature Development completed <IP_ADDRESS>

Usage simplified
Improved Meta DATA information of RFC and BAPI definitions,
Added connection pool support
Added multi-connection support
Changed the connection configuration structure
Changed function calls
Example projects expanded

<IP_ADDRESS> feature development completed.


Redirect proxy types to the proxied in member signatures

This makes proxy writing more convenient, for example you can have a GetRange method with signature int * int -> ResizeArray can be proxied as int * int -> ResizeArrayProxy and it will still be recognized correctly.
This was partially implemented for #771 (because it was needed for a workaround for an F# semantics change): not looking up all proxy redirects in advance, just using what is already accessible. However it is currently a breaking change, if the proxy type is defined later, for example on EltUpdater in UI.Next.
Collect all proxy redirects in before starting to read signatures for consistent behavior independent of code ordering.

This is fully fixed now for F#. We do not do proxying in C# yet, although it has basic support for it.


Lookup button click in a Edit Control not working

Unable to click on the lookup button in the Edit control @vasily-v-ryabov Need your quick help
Expected Behavior
Click on the YELLOW highlighted lookup button

Actual Behavior
Unable to click that lookup button, able to do set_text in the edit control
Steps to Reproduce the Problem
Short Example of Code to Demonstrate the Problem
contentPanel = mainPanel.child_window(auto_id="contentIFrame0", control_type="Pane") caseClass = contentPanel.child_window(auto_id="gcct_caseclassification", control_type="Window") caseClassText = caseClass.child_window(auto_id="gcct_caseclassification_ledit", control_type="Edit") caseClassText.set_text("Has Been To Dealer") caseClassLookup = caseClass.child_window(auto_id="gcct_caseclassification_lookupSearch", control_type="DataItem") caseClassLookup.click_input()
Specifications

Pywinauto version: 0.6.7
Python version and bitness: 3.7.4 64 bit
Platform and OS: Windows 10

Control Identifiers
Control Identifiers:
Dialog - 'Case Classification Select Case Classification OR build the Case Classification by using Level 1 through 4 selections. Select the Enter key to edit the lookup field'    (L868, T597, R1067, B627) ['Dialog', 'Case Classification Select Case Classification OR build the Case Classification by using Level 1 through 4 selections. Select the Enter key to edit the lookup field', 'Case Classification Select Case Classification OR build the Case Classification by using Level 1 through 4 selections. Select the Enter key to edit the lookup fieldDialog'] child_window(title="Case Classification Select Case Classification OR build the Case Classification by using Level 1 through 4 selections. Select the Enter key to edit the lookup field", auto_id="gcct_caseclassification", control_type="Window")    |     | Table - 'Case Classification Select Case Classification OR build the Case Classification by using Level 1 through 4 selections. '    (L874, T597, R1068, B627)    | ['Case Classification Select Case Classification OR build the Case Classification by using Level 1 through 4 selections. ', 'Case Classification Select Case Classification OR build the Case Classification by using Level 1 through 4 selections. Table', 'Table']    | child_window(title="Case Classification Select Case Classification OR build the Case Classification by using Level 1 through 4 selections. ", auto_id="gcct_caseclassification_lookupTable", control_type="Table")    |    |     |    | Custom - ''    (L874, T597, R1068, B627)    |    | ['Custom', '', '0', '1']    |    |    |     |    |    | DataItem - 'Enter a search term, then select the Enter key to search Enter a search term, then select the Enter key to search'    (L874, T597, R1033, B627)    |    |    | ['DataItem', 'Enter a search term, then select the Enter key to search Enter a search term, then select the Enter key to search', 'Enter a search term, then select the Enter key to search Enter a search term, then select the Enter key to searchDataItem', 'DataItem0', 'DataItem1']    |    |    | child_window(title="Enter a search term, then select the Enter key to search Enter a search term, then select the Enter key to search", control_type="DataItem")    |    |    |    |     |    |    |    | Static - 'Enter a search term, then select the Enter key to search'    (L0, T0, R0, B0)    |    |    |    | ['Enter a search term, then select the Enter key to searchStatic', 'Enter a search term, then select the Enter key to search', 'Static']    |    |    |    | child_window(title="Enter a search term, then select the Enter key to search", control_type="Text")    |    |    |    |     |    |    |    | Edit - 'Enter a search term, then select the Enter key to search'    (L874, T597, R1040, B627)    |    |    |    | ['2', 'Edit']    |    |    |    | child_window(title="Enter a search term, then select the Enter key to search", auto_id="gcct_caseclassification_ledit", control_type="Edit")    |    |    |     |    |    | DataItem - ''    (L1032, T597, R1040, B627)    |    |    | ['3', 'DataItem2']    |    |    |     |    |    | DataItem - 'Select a value. Select a value.'    (L1039, T597, R1068, B627)    |    |    | ['DataItem3', 'Select a value. Select a value.DataItem', 'Select a value. Select a value.']    |    |    | child_window(title="Select a value.  Otherwise, try to identify the control in question with Inspect.exe utility and verify it with draw_outline() method.
I also noticed you use pywinatuo 0.6.7 so you could try a newer version 0.6.8

I had to select the meuitem by getting the list from the dropdown -
lkpList.child_window(auto_id="item0", control_type="MenuItem").click_input()

Hi @jjbright,
I tried to automate USD 4 using UFT and problem started here at case classification fields and could not move any further. Please let me know if you find any solution and how you are automating this USD 4. Thanks in advance!


Improve Codebase for Xcode 16.0 compatibility and Integration with Marker Data

https://github.com/TheAcharya/MarkerData/issues/106
@orchetect When you have time you can tweak the codebase.
And @milanvarady could make the necessary tweaks back in Marker Data.
Thank you.

I've updated the codebase for compatibility with Xcode 16 / Swift 6,
I also refactored some internal code related to image generation to use modern Swift Concurrency async/await patterns, which has cleaned it up a bit and should make it safer.
Next I can have a look at Marker Data and see what conformances @milanvarady has added there to types exported from MarkersExtractor. From what I saw, it's mostly Identifiable conformances. Those I may move to MarkersExtractor since that's a common protocol used with SwiftUI and would not only be beneficial to anyone using MarkersExtractor as a library. But it's also better coding architecture, which is the reason for the compile warnings in the first place.

Ok I've done the appropriate refactors and will release 0.3.10 shortly.
I am making the necessary counter-adjustments to Marker Data on a separate branch and will push a Pull Request to the Marker Data once MarkersExtractor 0.3.10 is ready.

Thank you!

I've been updating my other open-source libraries and apps for new language features, Swift Concurrency, and better compatibility with Xcode 16 and the Swift 6 toolchain.
So a couple of dependencies have been improved (TimecodeKit, DAWFileKit) and some API has changed (mostly making various methods async), so I will push a MarkersExtractor 0.3.11 update with the needed changes in the codebase.
I already refactored MarkersExtractor itself to utilize Swift Concurrency a while ago, so not a lot of changes or improvements are really needed internally. But I can have a look through and see if anything needs polish at some point.

Will close this, as this is done.

Will close this, as this is done.

Thank you!


Error message "insufficient account balance when creating "ask" order

Trying to create an "ask" order over 100.000sat but get the error message "insufficient account balance" even though I funded my account with this sum. Is it related to transaction fees?
pool orders submit ask<PHONE_NUMBER>78514bce38d14031f71f5a4c6d1f9d1ac427ad813c55dd5f30970b3edeb3b2 --interest_rate_percent=1 --lease_duration_blocks=2016

With your account you also have to be able to pay for the order execution fees and your ask's part of the chain fees. Therefore the account size must be greater than ask_size + execution_fee + chain_fees. How high the chain fees are depends on your setting of --max_batch_fee_rate which will influence under what fee conditions the ask is executed (if you set it too low it can take a long time for your order to be matched of the mempool never gets that empty).
All of this is described in more detail here: https://github.com/lightninglabs/pool/blob/master/docs/orders.md#order-execution-fees
Closing as this is not a bug.


weird highlight in input

As title says. I'm usign onedark

@jrafaaael Can you provide example codes, please?

Looks like an issue with your color scheme. Here's a similar post: https://www.reddit.com/r/neovim/comments/v5jscg/question_floating_searchbox_border_highlight/
You'll need to fix the background for FloatBorder highight group.

autocmd ColorScheme * highlight! link FloatBorder Normal

works for me! thanks!


Fix #3197

Look great, but I'd like @asterite to look at this first :-)

Yes, I think that's best too; it works, but it might not "be the right place for solving it" B-) I'm still far from groking the semantic parts of the compiler.

@ozra Thank you, the fix is OK :-)
There's no need to only remove_alias if it's an AliasType: the method is also defined on Type as returning self.
Lately I'm trying to document the compiler as much as I can. Ideally anyone should be able to jump in and understand the code (I honestly believe it's not that complex). But it might take a while for me to do so.

Yes, I've seen more and more comments - they're really helpful in acquainting the terrain! Keep it up!


fix vel topic remapping

What type of PR is this? (check all applicable)

[ ] Refactor
[ ] Feature
[x] Bug Fix
[ ] Optimization
[ ] Documentation Update

Hi @GPrathap, thanks for this, just to have some context is this mainly to fix the issue with the random rotation of the hunter?

Nope, it was subscribed to the wrong velocity topic.  In ros2 /cmd_vel uses internally, in driver level we should not use this topic.


Update getting started doc with simple hello world tutorial

@Rick-Anderson @ardalis @rachelappel @DamianEdwards Please review

I'll make these final changes as part of https://github.com/aspnet/Docs/pull/1220.


fix #6460 for margin-right

when swiperSlide has a margin-right with spaceBetween happen problem so I make sure the margin-right of swiperSlide  = 0

added commit to fix #6460 pagination last slides with loop


Pinkie republished to NPM by user that isn't floatdrop

Pinkie disappeared briefly from npm, see #16, then reappeared again published by another user. Was originally published by floatdrop now published by puradox. pinkie-promise has also been republished by mbensch.
Bit worrying if not published by the original author. Could be an attempt to hijack the packages? Or just someone trying to help out? Would be good to get clarification from @floatdrop.
Original NPM page:

Current NPM page:

just deleteing yarn.lock and executing yarn install, solved. I m using yarn 1.9


MM-44877: screen reader emoji announcement

Summary
Screen reader  announces emojis from the autocomplete list
Ticket Link
Fixes https://mattermost.atlassian.net/browse/MM-44877
Related Pull Requests
None
Screenshots
NA
Release Note
NONE

E2E tests not automatically triggered, because PR has no approval yet. Please ask a developer to review and then try again to attach the QA label.

@nevyangelova, Would you take a look please.

This PR has been automatically labelled "stale" because it hasn't had recent activity.
A core team member will check in on the status of the PR to help with questions.
Thank you for your contribution!

@koox00 Do you have a chance to review this in the next day or two? If not, could you re-assign the review?

This PR has been automatically labelled "stale" because it hasn't had recent activity.
A core team member will check in on the status of the PR to help with questions.
Thank you for your contribution!

/update-branch

/e2e-test

Successfully triggered e2e testing!
https://git.internal.mattermost.com/qa/cypress-ui-automation/-/pipelines/257244

/cherry-pick release-7.1

Cherry pick is scheduled.


Show page with status of all frameworks

https://trello.com/c/DXsXHeZo/96-let-admins-change-the-non-prod-framework-state
This is the first part of a firebreak project to give admins in non-prod environments the ability to view and change framework states. admin-frontend doens't use the design system yet, so I have to use the toolkit instead.
Since this is a firebreak experiment, I'm hiding this in production.

To give a sense of where this is headed: my plan is to add an 'edit' link to each row that will allow the user to change the framework state.


[bug] preview record cannot display image and link is not clickable

Describe the bug
The preview record cannot display images and the link is not clickable.
To Reproduce
After triggering preview record including images and links, the image will not display and the links is not clickable.

closed with #168


VS Code 1.65 and 1.66 won't launch in Windows Server 2019 version 1809

Does this issue occur when all extensions are disabled?: Yes. This is a fresh installation so there are no extension. Tried starting with '--disable-extension' nonetheless and it behaves the same. I would get the popup that says "the window is not responding"

VS Code Version: 1.65.2 and 1.66.0
OS Version: Windows Server 2019 version 1809

Steps to Reproduce:

Disabled antivirus
Reinstalled with backward-compatibility with Windows 8.

/gifPlease

https://user-images.githubusercontent.com/19632636/165785535-e5851fb8-3821-4f95-a091-3afc53c2db93.MOV

Hi there! Sorry, had to upload mov. As you can see, vscode won't launch at all. This is a "fresh" install so it does not have any extension installed/enabled. I'm trying to launch it just after installation. I've tried installing it on a couple other windows server 2019 using either the system or user installer and i get the same "window not responding". I don't see this issue on our vs code installation on windows server 2016.

Can you launch the application from command line with following command code --verbose and attach the output.

Can you launch the application from command line with following command code --verbose and attach the output.

Here you go.
vscode_verbose.txt

Can you please follow the steps in https://github.com/Microsoft/vscode/wiki/Native-Crash-Issues to get at more details around the crash and attach the result here? Thanks!


freezing with windows os

Hello,
when i open a lot of files in atom like 16 files may more working on them and after finishing my work i close atom with all those files okay when i open atom again is supposed to open with all those file from prev session .. right
it's happening but  the window freezing all window with all open files freezing but the wired thing is atom menu bar still working i meaning i can go to file>open folder or new window this freezing happening with error msg says: editor not responding with two choice keep waiting or close.
i'm working on windows 10 OS core x64.

Thanks for reaching out!
We require the template to be filled out on all new issues and pull requests. We do this so that we can be certain we have all the information we need to address your submission efficiently. This allows the maintainers to spend more time fixing bugs, implementing enhancements, and reviewing and merging pull requests.
Thanks for understanding and meeting us half way :grinning:


added option for reading override info

I followed the logic in function get_spindle_tool_status() and added ability to read override info of the machine. This is functionality of runinfo O in TNCremo software.

Thanks, looks good to me!
I will add a unit test and create a new release


Cannot disable WebXR in VRExperienceHelper

I would like to disable WebXR when using VRExperienceHelper, so I set the useXR in options to false when creating VRExperienceHelper, but the value was ignored.
https://github.com/BabylonJS/Babylon.js/blob/c8e5f6bae8f842bb3b4b934a49c7c94e5c95cb12/src/Cameras/VR/vrExperienceHelper.ts#L726-L728
The value would be forced back to true.

WebXR will be forced when WebVR is not available or the flag is set to true. I would anyhow recommend not to use WebVR, as it is not available in any modern browser.


Navbar is linked with venues page and venues page added

@Nirvik07

@patelyash5208 please follow the PR template

@patelyash5208 upload the whole project in Projects folder.


Better handling of (...) delimiters in LaTeX

Since I no longer use $...$ myself, I find my lack of easy manipulation of  \(...\).

I believe this is fixed now that I use evil-tex

I believe this is fixed now that I use evil-tex


chore: change tx-assembler hash settings

What this PR does / why we need it:
Which issue(s) this PR fixes:

Fixes #
Which docs this PR relation:
Ref #
Which toolchain this PR adaption:
No Breaking Change
Special notes for your reviewer:
CI Switch
ci-runs-only: [Chaos CI, Cargo Clippy, Coverage Test, E2E Tests, Code Format, Unit Tests]

Codecov Report

Merging #471 (75a9df6) into main (df273d0) will increase coverage by 0.00%.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #471   +/-   ##
=======================================
  Coverage   17.10%   17.11%
=======================================
  Files         134      134
  Lines       18194    18194
=======================================
+ Hits         3112     3113    +1
+ Misses      15082    15081    -1

Impacted Files
Coverage Δ

core/mempool/src/tx_wrapper.rs
89.28% <0.00%> (+1.78%)
:arrow_up:


[Rendering] "associatedInputs": "none" doesn't work for Action.Execute in Microsoft Teams for iOS

Copied bug from: https://github.com/microsoft/AdaptiveCards/issues/8585
Details from original post:
Target Platforms
iOS
SDK Version
1.4
Application Name
Microsoft Teams
Problem Description
Parameter "associatedInputs": "none" doesn't work for "Action.Execute" type of button in Microsoft Teams for iOS.
Works perfectly well on Desktop and Android.
Works for "Action.Submit" on all devices.
1.5 SDK - same.
Screenshots

Card JSON
{
    "type": "AdaptiveCard",
    "body": [
        {
            "type": "ColumnSet",
            "columns": [
                {
                    "type": "Column",
                    "width": "stretch",
                    "items": [
                        {
                            "type": "TextBlock",
                            "wrap": true,
                            "text": "Leave Request",
                            "size": "Large",
                            "weight": "Bolder"
                        }
                    ],
                    "verticalContentAlignment": "Center"
                }
            ],
            "horizontalAlignment": "Left"
        },
        {
            "type": "Input.ChoiceSet",
            "choices": [
                {
                    "title": "Annual Leave",
                    "value": "BasicHoliday"
                },
                {
                    "title": "Unpaid Leave",
                    "value": "UnpaidHoliday"
                }
            ],
            "placeholder": "Choose",
            "label": "Type of Leave",
            "isRequired": true,
            "errorMessage": "Select type of leave",
            "id": "typeLeave",
            "spacing": "Large",
            "value": "${typeOfLeave}",
            "separator": true
        },
        {
            "type": "Input.Date",
            "id": "startD",
            "label": "Start date",
            "isRequired": true,
            "errorMessage": "Select a date",
            "value": "${startDateStr}"
        },
        {
            "type": "Input.Date",
            "id": "endD",
            "label": "End date",
            "isRequired": true,
            "errorMessage": "Select a date",
            "value": "${endDateStr}"
        }
    ],
    "actions": [
        {
            "type": "Action.Execute",
            "title": "Submit",
            "verb": "hdcreateticket",
            "data": {
                "action": "create",
                "basicHoliday": "${basicHoliday}",
                "unpaidHoliday": "${unpaidHoliday}",
                "userEmail": "${userEmail}"
            }
        },
        {
            "type": "Action.Execute",
            "title": "Cancel",
            "verb": "createticket",
            "associatedInputs": "none",
            "data": {
                "action": "cancel",
                "basicHoliday": "${basicHoliday}",
                "unpaidHoliday": "${unpaidHoliday}",
                "userEmail": "${userEmail}"
            }
        }
    ],
    "$schema": "http://adaptivecards.io/schemas/adaptive-card.json",
    "version": "1.4"
}

Sample Code Language
No response
Sample Code
No response---
Document Details
⚠ Do not edit this section.It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 3fb4394d-25b1-8752-3996-fe3a2934732d
Version Independent ID: 6d391381-89d0-117c-27cc-00043bd26a36
Content: Cards reference - Teams
Content Source: msteams-platform/task-modules-and-cards/cards/cards-reference.md
Product: msteams
GitHub Login: @surbhigupta12
Microsoft Alias: surbhigupta12

Thank you for reporting this, we will check this and get back to you.

We are also able to repro this issue on iOS. Even though "associatedInputs": "none" it is still asking us to enter input values while clicking the cancel button.

We raised a bug for the same, we will keep you posted on the updates. Thank you.

@Meghana-MSFT any updates?

Unfortunately, we do not have any update to share, we will keep you posted on the updates. Thank you.

@Meghana-MSFT Hello, are there any updates regarding this issue? Thanks.

We are checking with engineering team for updates, we will keep you posted on the updates. Thank you.

We checked this in iOS and its working fine. Could you please check and confirm once.
iOS : 17.4.1
Teams version : 6.6.0

I upgraded my iOS version to 17.4.1 and tested it. It seems like that issue doesn't exist with that version, thanks.


incompatibility with the newest version of "envied" and "envied_generator" package

after upgrading my flutter project to the latest version 3.10 and also upgrading the envied package in my project i am not able anymore to use the latest version of your package.
However when i downgrade it to the 0.0.2 it works fine but then it downloads all the icons that i am not using it.
In the newer version of your package it downloads just the icons that i defined in my project, so it would be very nice and i appreciate, when you solve the below dependency problem.
Thanks
error message
Because font_awesome_pro_flutter >=0.0.3 depends on analyzer ^4.1.0 and envied_generator 0.3.0+3 depends on analyzer ^5.1.0, font_awesome_pro_flutter >=0.0.3 is incompatible with envied_generator 0.3.0+3.

is their any update to this


Release/2.0.2

OAuth definition added to spec.
API spec extra info added.

push in to unblock 3.0 release.


[Input] Add blockie to input component

name: 'Input Component'
about: Add blockie to input component
New Pull Request
Checklist

[x] I am not disclosing a vulnerability.
[x] My code is conform the code style
[x] I have made corresponding changes to the documentation
[x] I have updated Typescript definitions when needed

Issue Description

Adds blockie to input component.
Related issue: #271
Solution Description

Added two new values address and bluredAddress to type prop.
When this type is selected Blockie would be visible.
Also value will compared with address regular expression.
When blurred address is selected onBlur address would shown as ellipsis text.

Added two stories for this please check those.

Great job here @AbhinavMV !
I will just need to close this for now as we need to refactor the input as it is becoming more complicated and surely we will reference this code in our refactor.
Thank you!


Check for framebuffer dimensions >0

This fixes the specific issue raised in #1701, plus some related additions and cleanup in the vicinity.
One question for reviewers: should I prefer to add simple checks like these to parameter_validation rather than core_validation?

We lean towards putting simple 'stateless' checks in parameter validation, but I know the idea is for them all to wind up in the same place someday.  But it's not worth redoing these guys.


(cashapp#288) Addresses error messaging on invalid version

See cashapp#288 small change

Thanks!


IGV-WEB: custom data sources

Hello,
igv-web is a great tool, easy do deploy, immediate for the end-user and provides a very good user experience from those coming from igv standalone.
One feature I really like is the possibility to browse ENCODE data from a table. Following the ENCODE idea,  giving the user the ability to browse, filter and select the tracks from one or multiple tables (or  Datasources) become natual; when the lab geneates a lot of data is crucial.
At the moment only ENCODE is supported, quoting the documenations: The type property can be used in lieu of a track list to trigger pre-defined lists of tracks for special data sources. Currently the only recognized value is "ENCODE". This property can be used to populate an ENCODE load widget for any genome assembly with data hosted by the ENCODE project. Currently this includes assemblies for human (hg19, GRCh38), mouse (mm10), worm (ce11), and fly (dm6). { "label": "ENCODE", "type": "ENCODE", "description": "Encylopedia of Genomic Elements", "genomeID": "hg19" }
I hacked the local source code to load our own table keeping the ENCODE schema but this is a very short term approach. I would like to have a more general approach for including custom datasources leveraging what is alredy there inside igv-web, igv-widgets, data-modal, and maybe other libraries. My goal is to add new buttons to the Track menu, where each button loads data ala ENCODE.
If you have comments, suggestions or ideas just let me know.

I did a hack modifing the igv-webapp (https://github.com/helios/igv-webapp/commit/415eb12abc54b14b4928c12b77d1c29a62b43662) and igv-widget (https://github.com/helios/igv-widgets/commit/9fc981c351345e144ea13762607e317c5ae40bbc)
Then I modded trackRegistry.json
  "hg38": [
    "resources/tracks/hg38_annotations.json",
    "resources/tracks/hg38_encode.json",
    "resources/tracks/hg38_1KG_lowCoverageCrams.json",
      "resources/tracks/hg38_custom.json",
      "resources/tracks/hg38_custom2.json"
  ],

and the customs files are something like :
{
    "label": "MAP-organoids",
    "type": "CUSTOM",
    "description": "Encylopedia of Genomic Elements",
    "genomeID": "GRCh38",
    "suffix": ".tracks-organoids.txt",
    "dataSetPathPrefix": "http://localhost:8080/files/hg38/MAP/",
    "urlPrefix": "http://localhost:8080/files/hg38/MAP/",
    "url" : "http://localhost:8080/files/hg38/MAP/GRCh38.tracks-organoids.txt"
}

and
{
    "label": "MAP-fight-cancer",
    "type": "CUSTOM",
    "description": "Encylopedia of Genomic Elements",
    "genomeID": "GRCh38",
    "suffix": ".tracks-fight-cancer.txt",
    "dataSetPathPrefix": "http://localhost:8080/files/hg38/MAP/",
    "urlPrefix": "http://localhost:8080/files/hg38/MAP/",
    "url" : "http://localhost:8080/files/hg38/MAP/GRCh38.tracks-fight-cancer.txt"
}

This commit adds custom track types to the app and adds their appearance in the track menu 28966535

Hi @turner,
can you point us to some docs on how to use the custom track types, maybe with an example ?

Hi tweep. We will have that in the web app README soon.

@helios The Readme has now been updated. Let us know if you have any questions. Note that what is described in the Readme is supported by the code in master, but we don't yet have a pre-built package with this version at https://igv.org/app-archive (that will come soon).

@helgathorv sorry for this huge delay. Just read the docs here and in the data-modal, this sounds very cool. I will test this new feature this week. Having multiple ways (json, url, ..) for feeding the modal is opens a lot of new possibilities.

Thanks everyone, we'll start test-driving this - it looks really promising. The REAMDE can be found here : https://github.com/igvteam/igv-webapp ( search for "my platinum genomes" )

@turner and @helgathorv I'am testing the new feature. Using the JSON data structures as in the example n.2 for the track registry it works as expected (great!). Instead, how can I use a text file formatted in CSV as input?
In the mean time I can concert my CSV to JSON and load the data, just to let you know that my "usual" number of line in a table is more than 500 rows ( tracks)

errata corrige, I am not able to run the example n.2. For reproducibility I am running it inside a Docker, this is my environment:
FROM openresty/openresty:<IP_ADDRESS>-centos
#https://computingforgeeks.com/install-node-js-14-on-centos-rhel-linux/

RUN yum install -y curl &&\
    curl -sL https://rpm.nodesource.com/setup_14.x | bash - &&\
    yum install -y nodejs
RUN yum install -y git

WORKDIR /opt/
RUN git clone https://github.com/igvteam/igv-webapp.git
WORKDIR /opt/igv-webapp

RUN git fetch --all &&\
    rm -rf node_modules &&\
    git checkout 9631842 &&\
    npm install &&\
    npm run build

RUN mv dist/* /usr/local/openresty/nginx/html
RUN mv node_modules /usr/local/openresty/nginx

RUN yum remove -y diffutils patch git nodejs
RUN yum clean all

and this is the docker-compose.yml
version: '3.8'

services:
  igv-webapp-dev:
    build: ./
    image: igv-webapp-dev:9631842
    ports:
      - 7777:80
    networks:
      - development
    volumes:
      - "./resources/genomes.json:/usr/local/openresty/nginx/html/resources/genomes.json"
      - "./resources/tracks:/usr/local/openresty/nginx/html/resources/tracks"
      - "../files:/usr/local/openresty/nginx/html/files"
networks:
  development:

@turner @tweep @helgathorv  the container and compose above are meant for using mostly the default configuration from git. You need to have a directory resources. If you have a better deployment strategy to perform tests I would more than happy to use it so we are aligned.
From the above dockerfile i did not included the installation of the master igv.js which is reported below:
WORKDIR /opt/
RUN git clone https://github.com/igvteam/igv.js.git
WORKDIR /opt/igv.js
RUN git fetch --all &&\
    git checkout 95f99d5 &&\
    npm install &&\
    npm run build
# TODO no mini js is generated just the esm
#FIX RUN mv dist/igv.esm.js /usr/local/openresty/nginx/html/

#FIX WORKDIR /usr/local/openresty/nginx/html
# Change the reference in the index to using the newly compiled/generated igv.js
# I doubt that esm is working ( i need to investigate more because I don't know the process very well)
#FIX RUN sed -i<EMAIL_ADDRESS>index.html

@helios Regarding CSV input - that's not currently supported, but were planning on adding it, and I've bumped that up a bit in priority. I created a separate issue for that if you want to follow it.

@helios Sorry we don't use Docker.   To align with us you would just pull from master and build with NPM.    If that doesn't work out of the box something is wrong with our build.

errata corrige, I am not able to run the example n.2.
I re did the whole installation, it was a detail in my reverse proxy, I can confirm that the solution n.2 works.  In any case the docker configuration can be useful to other to setup an isolated environment for IGV.


Fixing error in previous change

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

It looks like you're a Microsoft contributor (Mariah Breakey). If you're full-time, we DON'T require a Contribution License Agreement. If you are a vendor, please DO sign the electronic Contribution License Agreement. It will take 2 minutes and there's no faxing! https://cla.microsoft.com.

TTYL, MSBOT;

You don't need to send PRs when pulling latest code to your feature branch. Feel free to use "git pull origin dev" while working on your branch.


Load multiple configuration files

Extending the configuration parameter clientConfig, this PR introduces the feature to specify multiple config-files separated by ','
--clientConfig /path/to/file1.conf,/path/to/file2.conf,/path/to/file3.conf
Property values in the later files override property values in earlier files.
This functionality allows building more flexible CI pipelines. It is backwards compatible, continuing to work the same way for one config file only.
Changes made also fix a hidden bug: JulieOps.verifyRequiredParameters() was called after loading the configuration - this should happen before.
Any feedback?

Please check if the PR fulfills these requirements

[x] The commit messages are descriptive
[ ] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features)
[ ] An issue has been created for the pull requests. Some issues might require previous discussion.

Hi @LeonardoBonacci, thanks a lot again for this PR, it is much appreciated.
I wonder a couple of things:

Property values in the later files override property values in earlier files.
This functionality allows building more flexible CI pipelines. It is backwards compatible, continuing to work the same way for one config file only.

can you help me understand why? I wonder what is the benefit of having multiple smaller config files.

BTW, this PR will need a rebase from master because it has a conflict with the Configuration file.

To answer the question can you help me understand why? I wonder what is the benefit of having multiple smaller config files I had to consult my DevOps colleague who builds the Jenkins pipeline and requested me to implement this functionality:

It is the easiest way to keep credentials out of the main file so we can commit the main config to source control and override the credentials with a different file when running it on a ci/cd server.

I wonder because how then we explain what happens when two files have the same config property?

When multiple files have the same config, the next one should override the previous one i.e. the latest one wins

Thanks a lot for your feedback and for sharing your motivations for this, I find your purpose really good and valuable.
I wonder a couple of things in this use case?

do we need an open-end list? or can we use probably only a single fallback? the library that JulieOps use for config management allow you to define fallback like https://github.com/lightbend/config#merging-config-trees and you can control exactly how to do it. We could for example have a secondary parameter to even make this easier. What do you think?

For the credentials and secrets part, I am thinking two things.

Would it help better to have a proper integration with things like vault or other secret providers?
As well, the library used by JulieOps lets you define fallback to env variables, https://github.com/lightbend/config#features-of-hocon .

I am not a particular fan of using envs for credentials, but is one possible way.
What do you think?

To facilitate communication on this point I have committed some code  doing exactly this:
do we need an open-end list? or can we use probably only a single fallback? the library that JulieOps use for config management allow you to define fallback like https://github.com/lightbend/config#merging-config-trees and you can control exactly how to do it. We could for example have a secondary parameter to even make this easier.
I think it's a great idea. One thing to consider: due to defaulting behaviour or the properties in Config we need to introduce an overridingConfig param instead of a fallbackConfig param. Reason being:

In the primary clientConfig file I do not declare topology.translation.principal.enabled: this defaults to false.
In my secondary clientConfig file I declare topology.translation.principal.enabled=true
Since the primary config already has a value - namely false - the secondary config is ignored.

As a consequence, to introduce an extra config param it needs to be overriding instead of fallbacking.
If you agree with this approach, do you have a naming-suggestion? I have used overridingClientConfig for now, but that doesn't sound very appealing..

i like _ overridingClientConfig_ !! clearly saying what It should do!

As with the other PR, would you mind adding a few notes in the documentation? that would be very nice and close the circle! really looking forward to have this in!

Ready to merge!?

Certainly yes! thanks a lot for your work! I will follow up with this soon as today is really a crazy day at work and family 😄

Certainly yes! thanks a lot for your work! I will follow up with this soon as today is really a crazy day at work and family 😄

That sounds familiar... :)

Hi @purbon any progress on this PR?

I have it on my radar, I will certainly get it in next days. sorry for
delays, last days been complex
Missatge de LeonardoBonacci @.***> del dia ds., 10
d’abr. 2021 a les 19:52:

Hi @purbon https://github.com/purbon any progress on this PR?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/kafka-ops/julie/pull/252#issuecomment-817178229, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAAQXPHON2FABU2DMLOEAHTTICF43ANCNFSM42GAPKEA
.

--
Pere Urbon-Bayes
Software Architect
https://twitter.com/purbon
https://www.linkedin.com/in/purbon/

Merge it manually 3d3b211268ceb7126bab77d5cc9f34c5990f8d34 thanks a lot for your help @LeonardoBonacci !! much appreciate it.


Parameter restoreDirectory has no documentation

Parameter restoreDirectory cannot be found anywhere on the page other than the overall parameters list.

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

ID: d227a6f2-a94b-1c05-57e4-b63d42e2b9e5
Version Independent ID: d063e189-d0fd-4ca3-c108-a0bd2ae49d22
Content: .NET Core CLI task - Azure Pipelines
Content Source: docs/pipelines/tasks/build/dotnet-core-cli.md
Product: devops
Technology: devops-cicd
GitHub Login: @pulkitaggarwl
Microsoft Alias: puagarw

Actually I'm incorrect, the page mentions it:

Destination directory
(Required)  Specifies the folder in which packages are installed. If no folder is  specified, packages are restored into the default NuGet package cache.

With current structure of documentation it's just to find it. I'd like a paragraph for each parameter to mention it again to simplify finding it.

Hi @abatishchev I've just made some updates to the page and I believe is a little clearer under packagesDirectory. I'm closing this issue and if you have any more feedback, we'd love to hear it.


Minimum php version for 10.11.1

Summary of problem or feature request
The readme on the reqirements of latest version (v10.11.1), php is tagged as 8.0.2. My php version is 8.0.23
When I try to install it as a composer package I get the following error.
yajra/laravel-datatables[v10.0.0, ..., v10.1.0] require php ^8.1 -> your php version (8.0.23) does not satisfy that requirement.
System details

PHP Version: 8.0.2
Laravel Version: 9.19
Laravel-Datatables Version : 10

Some dependencies might have bumped their minimum PHP version since the current version also supports Laravel 10. Will review further when I got the chance.


Added LogAUC in a region

added a new metric to bias top-ranking predictions.
see this issue: https://github.com/mims-harvard/TDC/issues/139#event-5952668931
I'm not sure about the unit test procedure in the tdc community. Can someone help me with the unit test?

Hi Lance, thanks a lot for making the PR! It seems you make a PR to the dev branch, which is much outdated. Could you make a PR to the main branch instead? Then the commit history and changed file would be much clear. Also, could you provide a synthetic test case for us to include in the unit test? also, when you have a chance, could you provide a description for us to put on the website? some examples include https://tdcommons.ai/functions/data_evaluation/#precision-at-recall-of-k  thank you!!

Hi Kexin, thanks for the suggestions. Sure. I will do these later :)

Great, thank you!


Multiple changes in single commit doesn't always work

While working with release-please-action, I've noticed a couple times that features or fixes that are listed in the body/footer of a commit message don't get included in the changelog.
I don't recall all the circumstances where this happened, but at least this seems to occur if there are lines in the commit message footer that do not adhere to the Conventional Commits layout.
As an example, following is the commit message for https://github.com/rsenden/release-please-test/commit/868873d0b39b4545341f09ee76cb70f9cd2d4bd1
feat: Feature with extra messages

feat: FeatExtraFooter1

feat: FeatExtraFoorer2

Some other text

Something else

As can be seen at https://github.com/rsenden/release-please-test/runs/2084824783?check_suite_focus=true#step:3:14, release-please only recognizes 1 feature instead of the 3 features listed in this commit message. This can also be seen in the generated pull request, where the changelog doesn't list the FeatExtraFooter1 and FeatExtraFooter2 features: https://github.com/rsenden/release-please-test/pull/4

seems to work for me?
https://github.com/live627/SCEditor/commit/d6fcbbce28dd03ac154d4a9b7c26aad80e8ee7bb
https://github.com/live627/SCEditor/pull/37
so maybe it only works for node...

@live627 @rsenden the it has to do with the parser. We parse footers from the bottom up, so when the text is broken up like:
footer: a
footer: b

free form text that doesn't look like a footer.
```

The multiple footers are not parsed.

it has to do with the parser. We parse footers from the bottom up

Good to know, any plans to get this fixed? Maybe in the meantime it would be good to add a note to the documentation.

@rsenden this was an intentional decision when we wrote the parser, due to it being ambiguous as to which parse goal we'd like.
No immediate plan to fix, so updating docs might be a good compromise.

I'd like to +1 the need for docs improvements. Looking at the "What if my PR contains multiple fixes or features?" section in the README, I see all the example extra commits include "PiperOrigin-RevId: 345559154". It's not clear to me if this is required or not, and is confusing if working on a repo that isn't mirrored from google3/piper.

it's not strictly required
On Wed, Nov 17, 2021 at 1:53 PM Tim Swast @.***> wrote:

I'd like to +1 the need for docs improvements. Looking at the "What if my
PR contains multiple fixes or features?" section in the README, I see all
the example extra commits include "PiperOrigin-RevId: 345559154". It's not
clear to me if this is required or not, and is confusing if working on a
repo that isn't mirrored from google3/piper.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/googleapis/release-please/issues/821#issuecomment-972003677,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AADJNNZCHNXMJC6XNET3IGLUMQI4DANCNFSM4ZAIOCCQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.


[IOS]Build failure: xcode_backend.sh: No such file or directory Command PhaseScriptExecution failed with a nonzero exit code

Hi,
I recently switched from an intel Macbook to an M1.
My application is running on intel Macbook but not on the M1.
Both PCs use the same flutter and packages version.
Other smaller flutter apps are running fine on M1.
If running the specific app I got
Xcode's output:
↳
    /bin/sh: /Users/lukas/Flutter/flutter/packages/flutter_tools/bin/xcode_backend.sh: No such file or directory
    Command PhaseScriptExecution failed with a nonzero exit code
    note: Using new build system
    note: Planning
    note: Build preparation complete
    note: Building targets in parallel

I tried updating pods, flutter clean, restarting pc but no success.
flutter run --verbose
flutter run
Logs

flutter doctor --v
[✓] Flutter (Channel stable, 2.5.3, on macOS 12.0.1 21A559 darwin-arm, locale de-AT)
    • Flutter version 2.5.3 at /Users/lukas/flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 18116933e7 (7 weeks ago), 2021-10-15 10:46:35 -0700
    • Engine revision d3ea636dc5
    • Dart version 2.14.4

[✓] Android toolchain - develop for Android devices (Android SDK version 31.0.0)
    • Android SDK at /Users/lukas/Library/Android/sdk
    • Platform android-31, build-tools 31.0.0
    • Java binary at: /Applications/Android Studio.app/Contents/jre/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 11.0.10+0-b96-7249189)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 13.1, Build version 13A1030d
    • CocoaPods version 1.11.2

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 2020.3)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 11.0.10+0-b96-7249189)

[✓] IntelliJ IDEA Ultimate Edition (version 2021.2.3)
    • IntelliJ at /Applications/IntelliJ IDEA.app
    • Flutter plugin version 62.0.3
    • Dart plugin version 212.5632

[✓] Connected device (2 available)
    • iPhone 13 (mobile) • D3C991E6-C342-4571-BBFC-7FDDDA9B6AE8 • ios            • com.apple.CoreSimulator.SimRuntime.iOS-15-0 (simulator)
    • Chrome (web)       • chrome                               • web-javascript • Google Chrome 96.0.4664.55

• No issues found!

Hi @combi-35, Thanks for filing the issue. Did you try flutter clean?

/bin/sh -c /Users/lukas/Library/Developer/Xcode/DerivedData/Runner-awltaysunljaqnbrpahqqevxpcwi/Build/Intermediates.noindex/Runner.build/Debug-iphonesimulator/Runner.build/Script-9740EEB61CF901F6004384FC.sh
/bin/sh: /Users/lukas/Flutter/flutter/packages/flutter_tools/bin/xcode_backend.sh: No such file or directory
Command PhaseScriptExecution failed with a nonzero exit code

Looking at the logs, This issue Looks related to build configurations. Can you please share a screenshot of your build configurations by opening your project in xcode

run from the root of your project open ios/Runner.xcworkspace
Open your Project Runner build settings > Info tab.
Expand Debug, Profile, Release, and any other configurations and share a screenshot here.
And share your pubspec.yaml, and the Podfiletoo.

Thanks

Hi @maheshmnj,
thanks for the reply.
Yes I did run flutter clean.
Here is my screenshot

And here pubspec and Podfile
Podfile
pubspec

thanks for the info @combi-35, Your build configurations looks fine, and I was able to build the ios app with same dependencies heres what I did.

I created a new flutter project (stable 2.5.3)
added  all your dependencies to pubspec.yaml
updated Podfile same as yours
changed minimum deployment target via xcode to 11.0
flutter clean and flutter pub get

And the app built successfully, Can you please try deleting the podfile.lock and try running
flutter clean
flutter pub get
flutter run -v

and see if it works and share the output logs if it fails (as a text file)

flutter doctor -v
[✓] Flutter (Channel stable, 2.5.3, on macOS 12.0.1 21A559 darwin-arm, locale en-GB)
    • Flutter version 2.5.3 at /Users/mahesh/Documents/flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 18116933e7 (7 weeks ago), 2021-10-15 10:46:35 -0700
    • Engine revision d3ea636dc5
    • Dart version 2.14.4

[✓] Android toolchain - develop for Android devices (Android SDK version 31.0.0)
    • Android SDK at /Users/mahesh/Library/Android/sdk
    • Platform android-31, build-tools 31.0.0
    • ANDROID_HOME = /Users/mahesh/Library/Android/sdk
    • Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 11.0.8+10-b944.6916264)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 13.1, Build version 13A1030d
    • CocoaPods version 1.10.2

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 4.2)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 11.0.8+10-b944.6916264)

[✓] IntelliJ IDEA Community Edition (version 2021.2.1)
    • IntelliJ at /Applications/IntelliJ IDEA CE.app
    • Flutter plugin version 60.1.4
    • Dart plugin version 212.5080.8

[✓] VS Code (version 1.61.2)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.29.0

[✓] Connected device (3 available)
    • iPhone 12 Pro (mobile) • B622CBB7-F906-4FA7-8F49-FACEAAC905AB • ios            • com.apple.CoreSimulator.SimRuntime.iOS-14-5
      (simulator)
    • macOS (desktop)        • macos                                • darwin-arm64   • macOS 12.0.1 21A559 darwin-arm
    • Chrome (web)           • chrome                               • web-javascript • Google Chrome 96.0.4664.55

• No issues found!

Thanks

I already deleted podfile.lock and re-run suggested commands, but I did it again with no luck.
Here is my log.txt
I even tried to remove all plugins, assets and everthing except a basic main.dart. Still the same error, I dont know what to do, the project is basically empty now.

Thanks for the info @combi-35, I am not able to reproduce the issue, The app does build fine for me on stable 2.5.3 with your dependencies in pubspec.yaml and the exact podfile by following the steps mentioned above, Leaving this issue open for further investigation from the team.
cc: @jmagman

I actually got it running.. I deleted ios folder and re-created it with flutter create .
I don't now what got messed up in ios folder, because like I said on my other intel Macbook everthing is running fine.
It'd be great if we could find out what went wrong.

@combi-35 can you please attach the minimal project to find what caused the build failure?

/bin/sh: /Users/lukas/Flutter/flutter/packages/flutter_tools/bin/xcode_backend.sh: No such file or directory

Does this path exist?  Is your Flutter SDK at /Users/lukas/Flutter/flutter?

@jmagman,
after checking I see there is one "Flutter" folder to much as it was installed like so on my other Macbook.
But FLUTTER_ROOT is /Users/lukas/flutter so this correct, I cant see where the wrong path got pasted.

Hey @jmagman,
can you give me a hint, where I can correct the path or why this happend?
Thanks!

can you give me a hint, where I can correct the path or why this happend?

Are you sure you're running Flutter from /Users/lukas/flutter and not /Users/lukas/Flutter/flutter?  It should be correcting it on every run based on where your flutter binary is.
What's the output of which flutter?

Yes I am, here is the output /Users/lukas/flutter/bin/flutter.

But FLUTTER_ROOT is /Users/lukas/flutter so this correct, I cant see where the wrong path got pasted.

Where do you see that?  Did you set that explicitly somewhere?  In your logging it says
FLUTTER_ROOT = /Users/lukas/Flutter/flutter

I assumed that the installation path and Flutter Root are the same.
Flutter is installed on /Users/lukas/flutter.
It is possible that I changed something in the past because of an error. But looking through the project settings I cant find anything regarding flutter root.

I'm not sure then.  FLUTTER_ROOT should be set based on where your install path is.  Does it reproduce after flutter clean?  What's the output of:
$ grep -riI  "/Users/lukas/Flutter/flutter" /Users/lukas/Development/engago-app/


Failed to execute query: PoolTimedOut on Digital Ocean.

Checklist before posting an issue.

[x] I added a very descriptive title to this issue.
[x] I used the GitHub search to find a similar issue and didn't find it.
[x] I already searched in Google "How to X in sqlx" and didn't find any information.
[x] I already read the entire book and did not find the answer there.

Issue description
I am trying to run my app in Digital Ocean's app platform as suggested by the book. The issue is, when I try to run a request on any endpoint, it returns error code 500, with the following bunyan-formatted trace error:
{
  "v": 0,
  "name": "backend",
  "msg": "[GETTING MULTIPLE USERS - EVENT] **Failed to execute query: PoolTimedOut**",
  "level": 50,
  "hostname": "my-backend-7788b444cc-hw6z2",
  "pid": 1,
  "time": "2022-06-09T23:55:56.221870474Z",
  "target": "backend::resources::user",
  "line": 82,
  "file": "src/resources/user.rs",
  "request_id": "5b229801-0160-480a-a702-08d73f313ae7",
  "http.method": "GET",
  "http.scheme": "https",
  "http.host": "backend-xxxx.ondigitalocean.app",
  "http.flavor": "1.1",
  "http.user_agent": "Thunder Client (https://www.thunderclient.com)",
  "http.target": "/api/v1/users/",
  "otel.name": "HTTP GET /api/v1/users/",
  "otel.kind": "server",
  "http.client_ip": "x.x.x.x",
  "http.route": "/api/v1/users/"
}

The issue turns out to be: Failed to execute query: PoolTimedOut. This happens in all routes. I'm using a newer version of config, I don't know if it's related.
Current code
This is my code for project configuration. I'm using config 0.13.1.
pub fn get_configuration() -> Result<Settings, ConfigError> {
    // Get base app path
    let base_path = std::env::current_dir().expect("Failed to determine the current directory.");
    let config_dir = base_path.join("configuration");
    let base_config_dir = config_dir.join("base");

    // Detect the running envrionment type (prod/dev/stag)
    let environment: Environment = std::env::var("APP_ENVIRONMENT")
        .unwrap_or_else(|_| "local".into())
        .try_into()
        .expect("Failed to parse APP_ENVIRONMENT.");

    // Modify depending on the environment layer
    let env_config_dir = config_dir.join(environment.as_str());

    // Generate mutable configuration builder
    let builder = Config::builder()
        // Add from file
        .add_source(config::File::new(
            base_config_dir
                .to_str()
                .expect("Could not load base configuration."),
            FileFormat::Yaml,
        ))
        // Add from env
        .add_source(config::File::new(
            env_config_dir
                .to_str()
                .expect("Could not load env configuration."),
            FileFormat::Yaml,
        ))
        // Generate from environment to use in cloud env
        .add_source(config::Environment::with_prefix("app").separator("__"));

    // Build configuration
    let settings = builder.build().expect("Could not load configuration.");

    // Return the parsed type
    settings.try_deserialize()
}

Thank you in advance. I've been stuck here for some time now.

That was about it. Pretty simple. Thank you for taking the time! I'll be closing this as this was already solved in a previous issue.


feat(launcher): add strict mode

Codecov Report

Merging #84 (2e3239c) into master (b217621) will increase coverage by 0.03%.
The diff coverage is 100.00%.

:exclamation: Current head 2e3239c differs from pull request most recent head 3dacd6a. Consider uploading reports for the commit 3dacd6a to get more accurate results

@@            Coverage Diff             @@
##           master      #84      +/-   ##
==========================================
+ Coverage   96.69%   96.72%   +0.03%
==========================================
  Files          19       19
  Lines         272      275       +3
  Branches       46       48       +2
==========================================
+ Hits          263      266       +3
  Misses          9        9

Impacted Files
Coverage Δ

packages/launcher/src/module/app.tsx
100.00% <100.00%> (ø)

Continue to review full report at Codecov.

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


CAS Integration 3B - Create New Site (Supplier found, Multiple Sites / No matching address)

User Story:
As an SABC Finance team member (Ministry user), I need all students in SIMS to have a supplier and site number attached. If we receive a successful response from a Get Supplier call, we need to assign that Supplier Number to the student and either use an existing Site or create a new one. We should use an existing site if the addressline/postal code match to avoid duplication.
Acceptance Criteria:

If a Get Supplier Response is Successful and returns a Supplier that is NOT Active ignore for now see CAS4 #3258

[ ] If a Get Supplier Response is Successful and returns Active Supplier and NO active Sites, Create New Site

[ ] If a Get Supplier Response is Successful and returns Active Supplier and ONE active Site, Compare the Response AddressLine1 and Postal Code to SIMS address (Compare against the Addressline and Postalcode as we would be submitting and NOT case sensitive. Truncated and translated / No space in postal code):

[ ] Match FOUND: Save/Use that site.
[ ] NO Match: Create New Site

[ ] If a Get Supplier Response is Successful and returns Active Supplier and MULTIPLE active Sites, Search through active sites to look for addressline 1/postal code match:

[ ] Match FOUND: Save/Use that site.
[ ] NO Match: Create New Site

Create New Site

[ ] Only use the Create New Site based on the two GET conditions:

Get Supplier Response is Successful and returns Active Supplier and ONE active Site which doesn't have an address match
Get Supplier Response is Successful and returns Active Supplier and MULTIPLE active Sites, none of them have a matching address

[ ] Fields to include (See Field Requirements File below): SupplierNumber, AddressLine1, City, Province, Country, PostalCode, EmailAddress (Optional/Conditional fields can be left out or left null)

[ ] AddressLine1: Use first addressline from SIMS. Truncate if ours is longer than supported. Convert or omit special characters (like is done with other integrations).
[ ] City: Use as-is
[ ] Province: Use abbreviated versions AB,BC,MB,NB,NL,NS,NT,NU,ON,PE,QC,YT
[ ] Postal Code: ALL CAPS, no space.
[ ] Email Address: Use as-is

[ ] If successful response to Create New Site, then update SIMS CAS table with the prior Get Supplier returned supplierNumber, SiteCode and any other required info and set isValid=true
[ ] If error response, retry for now. Manual intervention may be required and will be dealt with on ticket #3258

Business Context
Supplier Web Service RESTful Client Onboarding v02_01.pdf
Individual Supplier Web Service Field Requirements (All Requests) JAN2023.xlsx

Technical Context
Sample CURL to create
curl --location '${BASEURL}/cfs/supplier/' \
--header 'Content-Type: application/json' \
--header 'Authorization: Bearer ${TOKEN}' \
--data-raw '{
    SupplierName: '\''Skywalker, Leia'\'',
    SubCategory: '\''Individual'\'',
    Sin: '\''422233197'\'',
    SupplierAddress: [
    {
        AddressLine1: '\''1012 Douglas St'\'',
        City: '\''Victoria'\'',
        Province: '\''BC'\'',
        Postalcode: '\''V8W 2C3'\'',
        Country: '\''Canada'\'',
        EmailAddress<EMAIL_ADDRESS>    }]
}'

Response:
{"SUPPLIER_NUMBER":"2007638","SUPPLIER_SITE_CODE":"[001] ","CAS-Returned-Messages":"SUCCEEDED"}

Business Context
** Create Site for Existing Supplier**
{
 "SupplierNumber":"2002492",
 "SupplierAddress":[
 {
 "AddressLine1":"3350 CLEARBROOK RD",
 "AddressLine2":"",
 "AddressLine3":"",
 "City":"ABBOTSFORD",
 "Province":"BC",
 "Country":"CA",
 "PostalCode":"V2T4T4",
<EMAIL_ADDRESS> "EftAdvicePref":"E"
}
]
}

Success Response
{
 "Supplier_Number":"123456789",
 "Supplier_Site_Code":"001",
 "CAS-Returned-Messages":"SUCCESS"
}

Error Response
{
 "Supplier_Number":"123456789",
 "Supplier_Site_Code":"001",
 "CAS-Returned-Messages":"Error Message"
}

@CarlyCotton the bullet point that reads "If a Get Supplier Response is Successful and returns a Supplier that is NOT Active ignore for now see..." may be misleading since it is an expected behavior where we will be creating the supplier and it is actually referencing an error handler ticket. Can we remove this or can you please clarify if I am missing something?

@CarlyCotton about the below bullet point, is there a need to expect ONE site to be present?
Get Supplier Response is Successful and returns Active Supplier and ONE active Site which doesn't have an address match

What if we adpapt the second bullet in the same AC as below.
Get Supplier Response is Successful and returns Active Supplier and one or many active Sites, none of them have a matching address.


Disconnect Called. BOSH module not started

I am trying to get Candy to work with my Ejabberd instance. I would like to use SSL for all connections. So when I try to get Candy to connect over TLS, I get this error;
Paste of deugging
Also if it helps this is the section of my Apache config. I got it from a random Google search awhile back.
Apache Config

ejabberd doesn't have mod_bosh loaded. Check your ejabberd configuration :)


MAX31850 adaptation

Could we include an adaptation to read MAX31850 devices too?
I believe the only difference is that data is 14-bits resolutions. But in the code there are so many checks to ensure its 9-12.
What would be proper places to change in order to adapt?
Thank you

@povi2946 I don't have one of these devices however I'd be happy to consider a pull request that added support for them. Probably the best way to handle it is an initialisation parameter that selects the type of device, then sets these limits in an internal structure, which the code makes use of elsewhere. If you or someone added support in a PR then I will look at getting a 31850 for myself so that I can test against it.

Actually the current version reads MAX31850 just fine. I believe I had issues with faulty breakout board and then mislabeled thermocouple wires. thanks

Thanks for letting me know. Interesting to hear it works with the MAX31850!


[TextInput] When maxLength is full automatically focus next field

Hello RN awesome team !
I have an issue in my app with TextInput for a bank card number (error because user not type all number 16 digits).
So I would like to cut CB number on 4 TextInputs and focus automatically on next field after filled the 4 digits. (as already seen it in the web app)
But when I focused the next field, the current became empty, even if the state is correctly set.
Do you have an idea about how can I do that or is it impossible ?
ie. see the onChangeText on this example (exp date actually) :
<TextInput
    ref='card_exp_date_mm'
    style={[styles.inputTxt, {width: 30, flex: 0}]}
    value={this.state.cardExpDateMonth}
    keyboardType='numbers-and-punctuation'
    placeholder='MM'
    returnKeyType='next'
    clearTextOnFocus={false}
    autoCorrect={false}
    placeholderTextColor={Constants.placeholderTextColor}
    maxLength={2}
    enablesReturnKeyAutomatically={true}
    blurOnSubmit={false}
    onChangeText={text => {
      this.setState({cardExpDateMonth: text})
      //console.log('onChangeText', this.refs.card_exp_date_mm)
      if(text && text.length == 2){
        this.refs.card_exp_date_aa.focus();
      }
    }}
    onSubmitEditing={()=>this.refs.card_exp_date_aa.focus()}/>

Thanks by advance for your help.

@facebook-github-bot stack-overflow

Already did it : http://stackoverflow.com/questions/35085917/textinput-when-maxlength-is-full-automatically-focus-next-field

@xcarpentier sorry about that, we did an issue clean up over the weekend. We are trying to keep react-native github issues focused on actual bugs.
That being said .22 was just released, I'd recommend upgrading to that and seeing if it fixes your issues.
If that doesn't create a simple reproduction on http://rnplay.org  so it is easier for people to run and debug. Without full code to look at it is very difficult to help you.

@xcarpentier How did u solve your issue? Please explain. I followed ur link to stackoverflow. It does throw some light on how to move to next text field. But how do i check if max length is reached?


test fix

Coverage remained the same at 70.213% when pulling 36f09c911d3e3ab148ff9addcee814755dea8250 on koma136:tmp into e871a2aaec8deec8ee79e399e574c7468a5d2a4f on SSENSE:master.


fix: add build v3 page

the top level page for /build/v3 was missing because build/ was still in the app's .gitignore

Great!


Layout of the contact session

Really impressed with the organization of the code, the modules, the functions, and ES6 implementation. Here are a few suggestions to improve:
-For the contact section, check on the alignment on larger screens.

@XsidX
Thank you for the feedback. this issue is resolved.
I have to mark it as done.
thanks


Support custom source in build assoc

resolve #478 for building assoc

It seems a ci job failed for elixir 1.0.2 and OTP 17.4. I'll look into this later

The build was a race condition somewhere in our suite (or a database failure). I have restarted it and now all is fine. I have also added some comments to the source.

@josevalim Done. Please review

Looks beautiful. I will merge this when I put some time into Ecto this week. :)

@josevalim Awesome!! :clap:

:heart_eyes:


chore(fleet): Add "direct" RC option to pull configurations

What does this PR do?
Real CDN is still WIP so we need another reliable option to pull config: this PR adds a regular RC client to do so. It also adds an option to use the real CDN that is disabled by default
Motivation
Describe how to test/QA your changes
Possible Drawbacks / Trade-offs
Additional Notes
Config field to allow CDN use in the daemon will come in another PR

/merge