Number of Connections vs Thread Count

I have been really impressed with Hikari in general, much better than the other pools we have tried.
We have been trying to characterize the behavior of the pool in regards to how many connections it makes to the database. In our case we notice that the pool hits our maxPoolSize limit of 100 connections when the processing is very fast. I did an experiment where I tried different thread counts vs. simulated operation time vs. idle time between operations to try to understand how many connections are made to the database. Full Test code is here https://gist.github.com/samhendley/d51b8ebba59e348ee10d13e6c16d2820 but core of the test is this loop run by many threads.
while (running.get()) {
    final Connection connection = hikariDataSource.getConnection();
    if (operationTime > 0) {
        Thread.sleep(operationTime);
    }
    connection.close();
    if (idleTime > 0) {
        Thread.sleep(idleTime);
    }
}

What we find is that if the operationTime is short and the idle time is short (simulating a busy and fast process) we will quickly spawn our max of 100 connections even with only 8 threads.

This was a bit of a surprise to me. I was expecting the pool to be end up maxing out at around the same number of threads we have but it looks like there must be some "housekeeping" I wasn't aware of that lets the pool grow too large. I know I could change the maxPoolSize to match the thread count but that can be tricky to "plumb" and if we try to be too exact we can create deadlock type failures. Notice that when the either the operations or idleTime are "slow" the pool stays at about the expected size. We only noticed this when we had optimized the performance of our process so all of it's calls fell into the <10ms range.
So two questions:

Can you explain what is happening after the connection.close() so that it isn't immediately ready for reuse. (is the housekeeping thread running the connectionQuery?)

Any settings other than maxPoolSize I can change to keep this growth in check?

@samhendley Your issue was an inspiration for a new write-up to be published when v2.6.0 is released.

Awesome write up and thank you for the help. If you can, you might want to change the color of "waiting threads" or "total connections". They were really difficult to tell apart on my monitor and phone so it took a while to understand what they are showing me.
In my similar tests I would expect hikari to end that test with 6 connections (at the 150ms mark). Since the connection adder thread starts making the 6th connection at the beginning of the spike it should end up adding that one extra entry to the pool (well after we needed it in this case). I had been thinking of how to avoid this issue but can't come up with anything that doesn't introduce a delay in the normal rampup scenario.

Thanks for the feedback on the graph color, I'll see what I can come up with.  Because I want to use translucency it is surprisingly hard to find readable combinations.
Regarding the 6th connection (mentioned at the end of the write up), I'm thinking about an additional call to shouldCreateAnotherConnection() after actual acquisition and immediate closing if no longer needed ...

Better?

Much better.
I think immediately dropping the connection is a bit aggressive, we've already paid the cost of creating the connection seems wasteful to toss it. It will get dropped on the idle timeout anyways, and if there is another spike we'll have 6 connections instead of 5 and handle it a bit better. But this does mean every spike will cause another connection to be created but if there are enough spikes to keep building connections then that is approximating a steady state workload anyways.


Website seems to be offline

https://www.backstage.io/ get's me a github pages 404.

Just saw people are already aware in discord here: https://discord.com/channels/687207715902193673/687207715902193679/831825843528531978


Fixing ACM-12888 by supplying a UID for fixed URL ref in the URL

Grafana should accept this, provided there is no clash, and the linkref will now reference a fixed dashboard URL instead of a grafana generated UID which changes per-installation.
Also fixed the query param templating for the severity and health impact.
The value of ${severity:queryparam} already includes the query param tag in it, so the previous template would substitute the tag with var-health_impact=$var-health_impact=All instead of var-health_impact=All.

/cherrypick release-2.11

/retest

/retest

/retest

/retest

/retest


[Bug]: language Bengali has many string problems

Summary
Notice the title bar in the gif it changes from English to Bangla when tapped on the menu. Notice the name change in other tabs as well.
Steps to reproduce

Expected behaviour
This change of language should not happen and it should remain in BENGALI
Actual behaviour
the title bar in the gif it changes from English to Bangla when tapped on the menu.

I believe this is not caused by the app itself, but rather by the lack of Bengali translations for "Bookmarks". We welcome any new translations, and anyone can sign up as a translator at Translatewiki.

Either missing translation or https://github.com/commons-app/apps-android-commons/issues/5501 I guess.


Plz make spacing responsive

Just like mr-sm-3.
Will you make it come real in v5.0?

We have responsive spacing utilities in v4.


Mute most alerts

This PR mutes most alerts currently raised in the repo:

All alerts from the quote test are removed.
All alerts form the reference solver test are removed.
In the solver competition test, only those violations raise an alert where the reference solver is BaselineSolver.

The test on cost coverage for partially fillable orders and the summary on cost coverage per solver are not changed. They will be relevant for the upcomming changes with regards to letting solvers choose fees.
This is another attempt at addressing #59.

The idea behind only using Baseline is that then all alerts would be actionable: If a solver screws up compared to baseline, either the execution by baseline is faulty (e.g. negative slippage) or the other solver messed up.
Information on other solvers improving upon the competition is also interesting but to me feels more of a thing one could use logs for. Though I see that there is a larger barrier to looking at the logs directly and to look at the alerts channel. The backend uses uses grafana dashboards for that type of information.
So I still prefer to only look at Baseline but if you actually use the current alerts from the solver competition, we could keep that test as is.

The idea behind only using Baseline is that then all alerts would be actionable: If a solver screws up compared to baseline, either the execution by baseline is faulty (e.g. negative slippage) or the other solver messed up.
Information on other solvers improving upon the competition is also interesting but to me feels more of a thing one could use logs for. Though I see that there is a larger barrier to looking at the logs directly and to look at the alerts channel. The backend uses uses grafana dashboards for that type of information.
So I still prefer to only look at Baseline but if you actually use the current alerts from the solver competition, we could keep that test as is.

Yeah makes sense. I do check these particular alerts though, not always, but quite often. So do you want to maybe mark the Baseline alerts somehow, so that it is easier to find them by just looking at the alerts channel, but also keep the rest as well?

So do you want to maybe mark the Baseline alerts somehow, so that it is easier to find them by just looking at the alerts channel, but also keep the rest as well?

That sounds a bit convoluted, so I just reverted that change about the solver competition test.


Remove unnecessary assign in pre handler

The article should also be updated. The following line was confusing since it isn't true:

This property is available in the route handler and is what we use to create the token.

Later in the article pre is used as mentioned for the /api/users/authenticate route but not at this point in the article.

Awesome, thanks!


Turing RK1 support

I have one Turing RK1 32Gb up and running on Ubuntu 24.04 as a daily driver at the moment, but I'd like to have NixOS instead and I can assist with testing

My recent focus has not been on this project. If you need it, I suggest you try adding support for Turing RK1 to this project yourself. #40 is a good example.


Update nwm_client_new routelink source for NWM v3.0 expanded domain

National Water Model v3.0 became operational in August 2023 with an expanded domain that includes parts of Alaska. This PR updates the source the nwm_client_new package uses to construct its crosswalk from NWM feature IDs to USGS site codes to include Alaska. It also adds the newly available model configuration labels to the list of valid configurations.
Additions

Added new model configurations to list of compatible client configurations. See example added to README.md
Libraries dependent on xarray complained about dask-expr. I added this as a dependency to setup.cfg.

Removals

None.

Changes

Altered a test due to an upcoming change in compatible strings for pandas.Timedelta.

Testing

pytest reported over 15k deprecation errors from pydantic, numpy, and xarray/dask originating from lower level libraries. This was in Python 3.10.
NOMADS has been a bit unstable, causing slow tests to fail due to timeout (which is currently set to 900 seconds).
Python 3.8 is EOL in 2024-10. Probably time to add 3.11 to GH tests.

Notes

Closes #248

Todos

Checklist

[x] PR has an informative and human-readable title
[x] PR is well outlined and documented. See #12 for an example
[x] Changes are limited to a single goal (no scope creep)
[x] Code can be automatically merged (no conflicts)
[x] Code follows project standards (see CONTRIBUTING.md)
[x] Passes all existing automated tests
[x] Any change in functionality is tested
[ ] New functions are documented (with a description, list of inputs, and expected output) using numpy docstring formatting
[ ] Placeholder code is flagged / future todos are captured in comments
[ ] Reviewers requested with the Reviewers tool :arrow_right:

Deployed: https://pypi.org/project/hydrotools.nwm-client-new/7.4.0/


[KBFS-3234] Widget File Row

OK ready for reviews.

I'm going to implement all the PR feedback in the last PR, so I don't need to do the merge dance.

After all is done, I will also add in a refactor so that GetNewestConvMetas is in mergeProps instead of mapStateToProps.

@chrisnojima need a thumb on this.


Mbarba/hack/gff3 annotation tests

Add tests for the extract_annotation module code

See the unified version in https://github.com/Ensembl/ensembl-genomio/pull/254


vike-react-query questions

Is there any recommended approach for setting the page title dynamically when using vike-react-query?  I am using useSuspenseQuery with telefunc, and want to avoid fetching the same data twice, one time just for setting the page title

Is there a way to prevent a useSuspenseQuery from refetching (on the server) when the client already has the data for the specific cache key from another page?

The plan is to use React's 19 built-int <title> support, see https://github.com/vikejs/vike-react/issues/88. You can also try to implement a solution on user land but note that it isn't trivial, see https://github.com/vikejs/vike-react/issues/67 and https://github.com/vikejs/vike-react/issues/36#issuecomment-1842870621.
I think @nitedani (author of vike-react-query) will probably know more about this.

Closing but let's continue the discussion.

1. Is there any recommended approach for setting the page title dynamically when using vike-react-query?  I am using useSuspenseQuery with telefunc, and want to avoid fetching the same data twice, one time just for setting the page title

useSuspenseQuery deduplicates and caches queries. If you have it at two different places with the same queryKey, it will only do one request.

2. Is there a way to prevent a useSuspenseQuery from refetching (on the server) when the client already has the data for the specific cache key from another page?

You can set refetchOnMount: false - https://tanstack.com/query/latest/docs/framework/react/reference/useQuery

@kevincrossgrove We're adding further documentation at https://github.com/vikejs/vike-react/pull/97. Let us know if you still have questions.

@kevincrossgrove

Is there any recommended approach for setting the page title dynamically when using vike-react-query?  I am using useSuspenseQuery with telefunc, and want to avoid fetching the same data twice, one time just for setting the page title

useSuspenseQuery deduplicates and caches queries. If you have it at two different places with the same queryKey, it will only do one request.

Is there a way to prevent a useSuspenseQuery from refetching (on the server) when the client already has the data for the specific cache key from another page?

You can set refetchOnMount: false - https://tanstack.com/query/latest/docs/framework/react/reference/useQuery Or you can set a higher staleTime on the query: staleTime: 60000 (one minute)
You can also set the QueryClient defaults inside a Vike config file:
// +config.ts

export default {
  ...
  queryClientConfig: {
    defaultOptions: {
      queries: {
        staleTime: 60000,
      },
    },
  },
};

@nitedani Thank you very much. Both are excellent solutions


Test and Fixes for #6 - Dates in querystring become empty strings

Added test case for ISO date in query string to prove it fails
Overwrite koa request's property query to handle dates correctly.

Closing and re-opening to trigger another build since iojs failed to install

@aheckmann is it possible to merge this pull request ?... I am waiting for this fix. Thanks

i'll pull this down and test + merge once #8 lands (very soon)
On Thu, Dec 3, 2015 at 10:14 AM, Martin Micunda<EMAIL_ADDRESS>wrote:

@aheckmann https://github.com/aheckmann is it possible to merge this
pull request ?... I am waiting for this fix. Thanks
—
Reply to this email directly or view it on GitHub
https://github.com/pebble/koa-joi-router/pull/9#issuecomment-161736166.

--
Aaron Heckmann
Web Development Lead :: Pebble
https://github.com/aheckmann
https://twitter.com/aaronheckmann

Thanks ;)

Sorry, I keep adding things as I need them, so I'm going to close this PR and open it again on separate branch that has just this feature.


Fixes vrrp_script to ouput a vrrp_sync block

Description
It fixes the output of the keepalived_vrrp_script helper which is rendering a vrrp_sync_group block instead of a vrrp_sync one.

So current master for vrrp_sync_group: https://github.com/sous-chefs/keepalived/blob/master/libraries/resources.rb#L120
and vrrp_script: https://github.com/sous-chefs/keepalived/blob/master/libraries/resources.rb#L144
Which of these is this trying to fix and are they both matching what we currently have?

I'm using the keepalived_vrrp_script helper, so it is here and this is the one that this PR fixes.

Ah brilliant, that all makes more sense now.
So your pr should be vrrp_script instead of vrrp_sync and then we can get this in?

Done ! Well catch, and thank you !

@Xorima tests are passing, all looks good to me.

Perfect, merged in

Perfect, merged it, great catch!

Thank you!


Support non-contiguous days in schedule day selection

eg:

mon
mon,thu-fri
mon-tue,thu-fri
mon,wed,fri

Can be worked around by having multiple near-duplicate schedules, each of which cover one range of days.

Sample tests for scheduleWindow.ts (currently fail)
    ['Start=08:30|mon;Stop=09:00|mon', 30],
    ['Start=08:30|mon,thu-fri;Stop=09:00|mon,thu-fri', 30 * 3],
    ['Start=08:30|mon-tue,thu-fri;Stop=09:00|mon-tue,thu-fri', 30 * 4],
    ['Start=08:30|mon-wed,fri;Stop=09:00|mon-tue,thu-fri', 30 * 4],


Upgrade to new react-scripts and webpack5

well, once AGAIN i have spent an entire night on the webpack build system.  this almost works... but for whatever stupid reason after getting all compile time resolve errors finally sorted, the cra 'refresh' runtime still crashes with unresolved....
33dba14

resolved this on my 6th attempt.  see #23


compilation from IntelliJ ignores --ee in config file

The ceylon.proxy project does not work in IntelliJ because this setting in config is ignored:
[compiler.jvm]
ee=true

@bjansen I extended the JavaCompilerOptions with a whole bunch of extra options, among which is the --ee one, so this should now in theory work (assuming you create those options using the fromConfig() method). Can you let me know if this works?

Fix confirmed! I have a NPE when ee=true is commented out, that disappears if I uncomment it.

Thanks Tako!

Thanks guys, that's much better now!


Kweb2

web visualizer
@tvt173

@joamatab seems to be missing index.html template? It complains it can't find it, when I run the webserver and access /

And python-multipart should be added as a dependency?

Hi Jan David,
it was working for me,
i added python-multipart as part of pip install "gdsfactory[full]"


Not able to connect to secured DC/OS Mesos server

We don't have a way to specify credentials when connecting to a secured DC/OS Mesos cluster. Not even sure the Marathon Java library we use provides this, it seems to have basic auth but nothing for OAuth/OpenID authentication.

We could add a property spring.cloud.deployer.mesos.dcos.authorization-token and use that for the Mesos and Chronos clients

We're adding the authorization-token property in #26


Replace pixelated warning icon with SVG image

Description
The SVG image added for the PDF implementation of the hazard domain in #3039 for 3.2 is a vector representation of the warning icon used by other note types.
This PR replaces the old jaggy bitmap icon with the vector image used in hazard statements.
How Has This Been Tested?
Built distribution docs PDF to verify results:

Type of Changes

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

In keeping with the precedent established when the hand.gif image was de-referenced for 2.2, this PR does not yet remove the warning.gif image itself, so it remains available for anyone who needs to restore it.
I think we should eventually remove these obsolete icons and the related comment, but that sort of change should probably not happen in a hotfix release, but rather in a separate PR that targets the develop branch.

3b0fc03 formats the icon based on the approach used for the hazard image, but I wonder if this sort of formatting would be better extracted to an attribute set? (Probably also not in a hotfix release though.)


Student marks prediction model has been added

#544 issue is solved

@invigorzz313
Review changes and add level (1,2,3)

Please deploy the ML model or integrate it into a web app.

hi @AkhilYadavPadala  , as you know there are lots of simple model in this so are expecting to you to deploy it, Please do the needful other we will not be able to merge your PR
OR
you can create some other issues.
Happy coding

Thanks for reviewing my PR @Niketkumardheeryan @invigorzz313
Shall I use streamlit or modelbit to deploy?

@Niketkumardheeryan @invigorzz313
I've deployed my ML model with the method API deployment using modelbit. Please review it.
Kindly, merge this PR under level(1/2/3).
I'll be happy if you labelled this PR as level 2

hey @AkhilYadavPadala  Please make a PR IN capsule 1.1 Branch as we have a conflict in master Branch,  And please add steps of deployment of model in Readme.md file


Test compare and replace

Description
Wrote tests for replace_and_compare

Fixes #3

Type of change

[x] Tests update

@eleensmathew
Everything is perfect
can you just provide me the necessary screenshot to verify every test is working fine without any failures

Hi i am also implementing the delete old messages feature as well. And also the websocket. Shall i complete that first or so you want to close the pr now?

Ya sure

Can you just resolve the conflicts?
@eleensmathew


Players are not given a sign when you go to the hotel creation mode

Sometimes players are not given a sign when you go to the hotel creation mode.
Also things from the survival inventory are transferred to the inventory of the creative mode (plugin ShareControl)

If the player is not in creationmode they are not given a sign.


fix invalid assert in getting started page tests

@cpurodriguez one of our test orgs failed this test, and it turns out the test assert is invalid.  it assumes the datetime formatting will map to Pacific time with locale US.  but if the user running the test has a different timezone or locale, it will fail.  I couldn't come up with a quick fix to do the correct assert.  I'm hoping your team can address this quickly and update this pull request with the correct assert.  if not, I'll need to remove the assert, so we don't block any builds or customers.  but we will still need a real fix later.  thanks.

Tagging @mdeleon0 and @Sonhia19 in case they are able to assist.


[core] Remove dependency on hoist-non-react-statics

The API was removed in #26136 but we forgot to remove this dead logic. I found it thanks to https://npm.anvaka.com/#/view/2d/%2540mui%252Fmaterial, one of the few licenses that is not MIT:

Hum, our bundle size tracker seems broken why is it not reporting: https://mui-dashboard.netlify.app/size-comparison?circleCIBuildNumber=393086&baseRef=master&baseCommit=2f962591eb819c6a105d29e590c056bffffc3616&prNumber=33015?

The fewer dependencies, the better!

Haha, yes 😍. AG Grid pushes it too far though https://www.npmjs.com/package/ag-grid-community no? 😅

Oh, the non-MIT license was not ISC but BSD in the Core. ISC is the weird license that MUI X Pro depends on (that we are phasing out).

AG Grid pushes it too far though https://www.npmjs.com/package/ag-grid-community no? 😅

Definitely :)

Oh, the non-MIT license was not ISC but BSD in the Core

AFAIK both BSD and ISC are OK to use in our context (ISC being even more permissive): https://tldrlegal.com/license/-isc-license, https://tldrlegal.com/license/bsd-3-clause-license-(revised)

AFAIK both BSD and ISC are OK to use in our context

Agree, the only concern for me is that it's one more license that developers have to check (what you did).


Using Lua: How can I access the text within a buffer pane?

Description of the problem or steps to reproduce
I want to process the whole contents of the current buffer (not the disc copy). I don't want to mess with the current cursor selection(s). Is there any way to return a sting or table into Lua?
Specifications
OS: Windows 10
Terminal: Console - (not Legacy, not Terminal App)
Version: 2.0.8
Commit hash: cfcb2e45
Compiled on October 06, 2020

I think you can use the Bytes function on a buffer to get the full contents. As a warning, this will be slow for large buffers (>1mb), and it is better to access data via the LineBytes function. If you want to convert the byte slice to a string in Lua, I believe you can use the util.String function.


mx.sym.L2Normalization has bug when shpae[0] is bigger than 65536 and multi-gpus

when first shape of L2Normalization data > 65536 and using mulit-gpus,  then it will give errors like this(single gpu training is ok):

we can reproduce as follows:

change code form https://github.com/apache/incubator-mxnet/blob/master/example/image-classification/symbols/mlp.py to:

"""
a simple multilayer perceptron
"""
import mxnet as mx

def get_symbol(num_classes=10, **kwargs):
    fc2_num_hidden = 70000
    data = mx.symbol.Variable('data')
    data = mx.sym.Flatten(data=data)
    fc1  = mx.symbol.FullyConnected(data = data, name='fc1', num_hidden=128)
    act1 = mx.symbol.Activation(data = fc1, name='relu1', act_type="relu")
    fc2_weight = mx.sym.Variable("fc2_weight", shape=(fc2_num_hidden, 128),
                                 init=mx.init.Xavier(rnd_type='gaussian', factor_type="in", magnitude=2))
    fc2_norm_weight = mx.symbol.L2Normalization(data=fc2_weight, mode="instance", name="fc2_norm_weight")
    fc2  = mx.symbol.FullyConnected(data = act1, weight = fc2_norm_weight, name = 'fc2', num_hidden = fc2_num_hidden)
    act2 = mx.symbol.Activation(data = fc2, name='relu2', act_type="relu")
    fc3  = mx.symbol.FullyConnected(data = act2, name='fc3', num_hidden=num_classes)
    mlp  = mx.symbol.SoftmaxOutput(data = fc3, name = 'softmax')
    return mlp

change line form https://github.com/apache/incubator-mxnet/blob/master/example/image-classification/train_mnist.py#L81 to gpus="0,1",

run mxnet train_mnist.py examples from: https://github.com/apache/incubator-mxnet/blob/master/example/image-classification/train_mnist.py

then, when running with single gpu, training is ok, but when trained when more than one gpu, it will broken.
@sxjscience @mli  @piiswrong  thanks.

this has been fixed from https://github.com/dmlc/mshadow/pull/287  thanks @sxjscience


Update to work with latest version of expo

After expo requires app.json appr no longer works. This allows appr to work again.

Thank you very much @tonyxiao!


Ctrl+Z

Stems from the ctrl-z button on computers that undoes the last action performed. The term Ctrl Z has also made worked its way into popular usage to mean undo, fix or to take back.

What do you mean? I don't get it..

Closing as implemented, unless @hangmana means something else (feel free to elaborate and we'll reopen).


Infeasibilities in data (ID18)

Issue in which we can state if we encounter any infeasibilities.
Balmorel turns infeasible when I try to solve the demand profile in ID18 for 2016. I believe the infeasibility arise due to the demand not being possible to satisfy with the installed capacities. These infesibilities do not occur if I use the demand profile from ID15 (for 2016), wherefore I believe the problem came with the ID16, where demands were updated. At the moment, I have not solved the model for 2030 and 2050 (ID18 data) so I do not (yet) know if these are infeasible too.

In ID18 the demands are slightly higher compared to ID15.
I propose to see if the updated transmission efficiency in ID19 is a sufficient change to solve the infeasibility.

Update from Balmorel: With ID15 demand for 2016 and ID18 demand for 2030 and 2050 Balmorel solves the problem without any infeasibilities.

Thanks for the update. The new transmission efficiency will be 97% . Maybe this resolves the issue.

From the side of Balmorel: With the changes in ID19, we no longer have infeasibilities in the demand of 2016.

Can the issue be closed?

Yes :)
I just closed it, and if any new infeasibilities arise I guess we open up a new issue.


Invert remote_proxy? attribute to block_raw_action?

Merge together with UI: https://github.com/ManageIQ/manageiq-ui-classic/pull/650
Introduce block_raw_action? flag and fill it during provider refresh
OrchestrationTemplateVnfd links to real template in openstack. When a user creates/deletes a template we want to keep equivalent in openstack in sync. That's why there is a callback on save of OrchestrationTemplateVnfd.
However, this has exception when we refresh from provider, in that case we must block raw operations.
When we invert, we have less places to maintain to set the flag value.

@isimluk I think that block_raw_action? sounds more like what it does, otherwise looks good :+1:

@isimluk I think that block_raw_action? sounds more like what it does, otherwise looks good 👍

changed. Thanks!

looks great :+1:

@isimluk @Ladas, block_raw_action is not a field in the DB, right?  How are raw_update and raw_delete prevented from running if an OrchestrationTemplate is read from the DB?
The question is valid for the old implementation as well.  But, in that case remote_proxy would have been nil (aka false) when an OrchestrationTemplate was read from the DB.  In this case block_raw_action is nil (and therefore false), so the assumption is that raw actions should be executed on existing orchestration templates.

@blomquisg crap, you are right. So seems like I have it 'disabled by default' for a reason. :-)
@isimluk in refresh, after the template is deleted remotely, we load the template only from a DB and perform .destroy on it. So in this case, it would try to do the remote operation. Not sure how to get past this, without having the default as it is now.

Discussed in depth with @Ladas over lunch. Here are my notes for future reference
The raw actions can be either default to true or default to false.
The default true does not play well with refresh (deletion of non-existing from vmdb). It would require one-off condition in the generic refresh destroy method.
The default false does not play well with API (requires override of generic create/update/delete), webui (requires override when creating/modifying/deleting) and automate (handle object of the Vnfd template).
The other option is to go with queued raw actions (like the other models do) and wait for refresh to update the db afterwards.
Either solution is hairy and none is elegant, so I feel I will leave OrchestrationTemplateVnfd as they are.
Thanks for comments and discussion!


Carbon X Release-Elements-Content

Elements-Content
A designer can discover the new guidance for content as it relates to both Product's voice and tone and writing priciples; they can surface content examples and Glossary of terms easily.
Design and Content

[ ] General Tab: redesign layout and create/update and usage guidance

Remove out of date guidance / add new where needed
Change out Voice and Tone svgs with new IDL pictograms

[ ] Guidance Tab: redesign layout and create/update and usage guidance

Remove out of date guidance / add new where needed
Change out As svg with new IDL pictograms

[ ] Glossary Tab: redesign layout and create/update and usage guidance

Find a new IDL way (like using big Plex letterforms) to make the letters for the glossary

[ ] Images: add or update all exhibit images with new styles / create assets (if there's anything i missed above)

Development

[ ] All Tabs:

Make sure we have components built to accommodate this layout (see issue: https://github.com/carbon-design-system/carbon-website/issues/293)

Content updates for this section are not in scope for the Carbon X release. This is backlog work.

Group decision: lets just remove them for this release.

I did make headway on these designs.
File: https://ibm.ent.box.com/file/359636513915
Screenshots
General

Guidance

Glossary (cropped view because its very long)


Set up SHEETS_EXTERNAL_LINK_LABELS via config file

We should remove the hardcoded CUBI-specific values and get this from env in deployment. DEFAULT_EXTERNAL_LINK_LABELS should only contain generic examples.
The issue is nicely providing JSON via our playbooks. Will link some resources kindly provided by Manuel below.
UPDATE: After some consideration and internal discussion, I've realized that these labels can easily become quite big and not reasonable to place under env variables. Hence I'm going with an optional JSON file provided via a config directory during deployment.
Tasks

[x] Set up label loading via file
[x] Move file loading from Django settings
[x] Add tests for file loading
[x] Ensure tests run correctly in local dev and ci
[x] Update docs

Related Ansible resources:

https://docs.ansible.com/ansible/latest/collections/ansible/builtin/file_lookup.html
https://stackoverflow.com/a/22978594/84349

Another way to do this: https://stackoverflow.com/questions/24003880/ansible-set-variable-to-file-content
However, the very good question of "why not just use a config file" has been raised. Will sleep over this and think of the best solution.

Done.


File type filter not working in Safari

Environment: OS X 10.11 Safari 9.0.2
I add a filter .txt and it works well in Chrome (but this filter also accept files like .py, .sh)

However, it's not working on Safari.

This can be reproduced by the demo.

Post your code

You can reproduce it in the official demo. Change the filter from "images/*" to  "'.txt'"

which filter

ngf-accept = "'.txt'"

              Profile Picture: <input type="file" ngf-select="" ng-model="picFile" name="file" ngf-accept="'.txt'" required="">

I changed this line in the demo

Here is how I reproduce it on Safari.

The popup file select is browser specific implementation, use ngf-pattern to validate the file type.
You can play around with different accept values by a regular <input type="file" accept=".txt"> to see if you can get it to filter what you want but it depends on how browsers are implementing it.

I had tried this
                    <input type="file" ngf-select
                           ng-model="File"
                           name="file"
                           input type="file"
                           accept=".txt"
                           ngf-accept="'.txt, .json'"
                           ngf-max-size="10MB" required
                           ngf-model-invalid="errorFiles">

it's not working on safari. Seems like all filters are not working properly on Safari.

As i said it's browser dependent so yep ask it on safari forum or support.

+1 same problem.. osx 10.10.5 safari.

Seems Safari all have this problem
On Sun, Feb 14, 2016 at 21:30 Keith Levi Lumanog<EMAIL_ADDRESS>wrote:

+1 same problem.. osx 10.10.5 safari.
—
Reply to this email directly or view it on GitHub
https://github.com/danialfarid/ng-file-upload/issues/1313#issuecomment-184071662
.

+1
@danialfarid
I am also facing the same issue in safari. accept=".json" is not working in safari for me. Any workaround you suggest. Thanks!

You guys should use MIME type instead of file extension name.
example: accept: 'image/,audio/,video/*,application/vnd.openxmlformats-officedocument.wordprocessingml.document,application/msword,application/vnd.ms-excel,application/vnd.openxmlformats-officedocument.spreadsheetml.sheet,text/csv,application/vnd.ms-powerpoint,application/vnd.openxmlformats-officedocument.presentationml.presentation,application/x-zip-compressed,.7z,application/octet-stream,application/pdf,text/rtf,text/plain,.gpd,.pro', //,.gpd,.pro sterio records

even the MIME type won't work for all the types. See https://caniuse.com/#feat=input-file-accept,
Safari has an issue on implementing this for all the standard versions, I have verified some of the types that do not work in standard version of safari, but work fine in Safari Tech review version.


[ko] Translate the glossary term 'Member' into Korean

This is a Feature Request
What would you like to be added
Translate the glossary term 'Member' into Korean
link to en entry: https://kubernetes.io/docs/reference/glossary/?community=true#term-member
link to ko entry: https://kubernetes.io/ko/docs/reference/glossary/?community=true#term-member
Why is this needed
Not translated yet
Comments
/language ko
/assign

/triage accepted

Resolved by

#35500 (→ dev-1.24-ko.2)
#35767 (dev-1.24-ko.2 → main)

/close


Simplify (?) config builder logging

Resolves issue #593.

[x] Add Logger interface (providing log_debug, log_info, etc.) and pass instances around instead of the functions themselves.

This is based somewhere in https://github.com/OSU-Net/cyder/pull/814, I take it.

Yeah. I forgot about that when I marked it "please review". I'll rebase it when I get a chance.

Rebased onto master.

It seems like you missed some unused imports (that were being used before you made changes). However there are a lot of unused imports anyway, so we might just remove those in a separate pull request.

 :) Yeah, we can do that later.

That's good enough for me.


DOC: Improved documentation for jax.numpy.fromstring

Part of https://github.com/google/jax/issues/21461

Can you rebase your changes on the most recent main branch? These test failures are from a bug that was fixed yesterday afternoon. Thanks!

Rebase from most recent main branch is done.


Public UV coords and loading with Image object.

It seemed cleaner to allow loading a surface from a pre-existing Image Element than letting myr.js return its own image object (especially as you would want to do this only when you already want a html version in your webpage). Problems related to retrieving a new image object from a generated surface are better solved using a getPixelBuffer solution in any case.

Good point, returning an img object would be overly complex.


RFP - Hardentools

Hardentools is a utility that disables a number of risky Windows features.
https://github.com/securitywithoutborders/hardentools

I'm looking into creating a package for this now.

Package have been pushed to chocolatey.org
https://chocolatey.org/packages/hardentools/1.0
The pre-release version will also be pushed after the latest stable is approved.

the package have been approved


Tracking only mode?

Is your feature request related to a problem? Please describe.
We have been working on integrating your great face detection and tracking project with our project and have hit a snag. We make heavy use of WebGL to apply filters to the camera stream and when we run your face detector in parallel, it can take over 10 seconds to find a face. Having done some analysis on this, here are our findings:

Our process is running the GPU at maximum using requestAnimationFrame
Face detection is really slow but tracking seems to work OK
Throttling our process to 30 fps helps a bit, but not enough
Disabling toggle_slow also helps but makes the UI laggy
It appears that your face detection is much more heavyweight then the tracking

Describe alternatives you've considered
We have had to look for alternatives as it is important for us that the face detection starts as soon as possible. We have found PicoJS which can use the pico framework as a WASM module for face detection and it is incredibly quick! But it does not give us any feature points or pose detection which we would really like.
Describe the solution you'd like
We were wondering if it would be feasible to have a tracking only mode where we could pass in a bounding box of the user's face and just use your library for face tracking and pose estimation. This would allow us to use Pico WASM for super fast face detection and jeelizFaceFilter for rich face information. Do you think that would be possible?
Additional context
I would be happy to work on a PR for this feature but that would not be feasible with minified code.
Let me know your thoughts on this.
Thanks :)

I just want to add that I also use the library only for tracking - the rotation of the head. All the threejs stuff is of no use to me, nor is showing the webcamera.  Getting the input is enough.
I am also throttling the input in order to reduce strain on the cpu - using another js library.
One thing that would help is being able to pause/unpause the tracking easily.

Hi,
Indeed, these features are not currently implemented in this Github repository.
But it is outside the scope of this library (FaceFilter with external 3D engine).
However, I would be happy to help you to adapt our libraries to fit your needs.
You can contact me directly at xavier__at__jeeliz.com and I will send you a proposal for this development service.
Best regards,
Xavier


Path ansibleee runner image to use correct image

Currently ansiblee runner image is not using the proper image. We need to force patch it to use the latest image.
As a pull request owner and reviewers, I checked that:

[x] Appropriate testing is done and actually running

/lgtm
/approve


Week 5 - Proctored MCQ Week1

Tasks

[x] GH Indi Page and Review Ticket for following...
[x] Start review ticket with study plan and study reflection for week, due EOD Monday 4/18
[x] Finals-Quiz1 score/50 and corrections
[x] Finals-Quiz2 score/50 and corrections
[x] Create Task code/runtime and 1 min video are reviewed by another person, you may request Teacher review as time permits

Evidence

Github Pages
Create Task
Quiz Scores + Corrections
Study Plan

4.5/5 github pages beautifully organized, very navigatable. study plan very readable and good, both quizzes scores on pages, video was very informational


Support INET_ATON and INET_NTOA

Is your feature request related to a problem? Please describe.
Postgres has dedicated type inet for internet addresses and lots of functions. It's good, but might be a little bit overkilled for RW.
These functions are from MySQL. Looks like a clever and elegant solution.

INET_ATON()	Return the numeric value of an IP address
INET_NTOA()	Return the IP address from a numeric value

Describe the solution you'd like
This allows users to do IP range comparison, such as
WHERE ((ipv4_dst_ip >= inet_aton('<IP_ADDRESS>') AND ipv4_dst_ip <= inet_aton('<IP_ADDRESS>'))

Describe alternatives you've considered
No response
Additional context
No response

Hi, I'm glad you're interested in contributing, but this requirement is a bit urgent. We need to deliver within this week. Feel free to pick PG-compatible functions in https://github.com/risingwavelabs/risingwave/issues/112 (especially the "More" section). Thanks!


remove needless statement

!timeout must be true, so we can remove this needless statement

@jgonggrijp  you are right, thanks

This was interesting. I was going to suggest the same thing, but also to remove the reset of context and args. When I did, it passed all the tests.
I know why, and I think it's wrong to reset args and context as the last call to the debounced function should win.
This doesn't work:
result = func.apply(context, args);
// append returns here. Context is now 'b1', and args is 'b2'
context = args = null;
// cleared so the last call fails

Heres why the test fails:

throttledAppend is executed and calls append immediately.
append calls throttledAppend who store context and args, and set up a timer.
when timer ends, later (inside debounced) calls append. Append calls throttledAppend who store context and args and set up a timer.
When append return to the later function, context and args is cleared.
the last call to append is executed when timer ends, but context and args is cleared.

export default function throttle(func, wait, options) {
  var timeout, context, args, result;
  var previous = 0;
  if (!options) options = {};

  var later = function() {
    previous = options.leading === false ? 0 : now();
    timeout = null;
    result = func.apply(context, args);
    //if (!timeout) context = args = null;
    //context = args = null; // Fails if this is uncommented
  };

  var throttled = function() {
    var _now = now();
    if (!previous && options.leading === false) previous = _now;
    var remaining = wait - (_now - previous);
    context = this;
    args = arguments;
    if (remaining <= 0 /*|| remaining > wait*/) { // remaining > wait will never happen (I think)
      if (timeout) {
        clearTimeout(timeout);
        timeout = null;
      }
      previous = _now;
      result = func.apply(context, args);
      //if (!timeout) context = args = null;
    } else if (!timeout && options.trailing !== false) {
      timeout = setTimeout(later, remaining);
    }
    return result;
  };

  throttled.cancel = function() {
    clearTimeout(timeout);
    previous = 0;
    timeout = 0;
    //timeout = context = args = null;
  };

  return throttled;
}

@kritollm So you basically found that there is a race condition between the outer invocation of later and the inner invocation of throttled on the result = func.apply(context, args) line. Very clever!
This also explains why the throttle re-entrant test exists in the first place, and why the if (!timeout) part is there. !timeout will evaluate to false if throttled was recursively invoked on the previous line.
I disagree that the resets should be removed; they are there to prevent memory leaks. However, the race condition deserves some documentation. Would you like to submit a PR that adds a comment above the if (!timeout) line and get the credit for it?

I agree that the reset of args and context is needed. Shouldn't throttled also use restarguments?
What do you think of this comment?
// This check is needed because the argument function can recursively invoke throttled

Shouldn't throttled also use restarguments?

Probably not strictly necessary, but if you like, you can try whether it makes for more compact minification (using npm run weight). If the difference is negligible, it's probably better to leave it as-is.

What do you think of this comment?
// This check is needed because the argument function can recursively invoke throttled

Pretty good. I'd write func instead of "argument function" in order to make it a bit more explicit, and add a little bit of Markdown markup for the annotated source.
// This check is needed because `func` might recursively invoke `throttled`.

If writing it myself, I might be inclined to write something like the following, in line with the overall literate style of the code:
// We use `timeout` to detect cases where `func` recursively invokes `throttled`.

But please do feel free to be a little bit creative here.

Should we try to clarify whether the remaining > wait check is necessary?
if (remaining <= 0 || remaining > wait) {
As I see it, it will always be fake, unless previous  is greather than now(). From what I can see from the code, previous will always be less than or equal _now.

I suppose previous might be greater than _now if the user manually changes the system clock in between two calls to throttled. This would be most likely the case if the computer clock went out of sync with the GPS, for example if it had no power for a while.
As for whether it is better to timeout again in that case or invoke immediately, I find it hard to judge. But since a choice was already made, it's probably best to stick with it.

You're right about that. There is a possibility that it can happen, and then the remaining can be much larger than wait. In the worst case, throttled might not trigger in several years.
So the check is actually very necessary (because the timeout  will then be the value of  remaining ).
} else if (!timeout && options.trailing !== false) {
      timeout = setTimeout(later, remaining); // 'remainig' can be much greather than' wait'
    }


Rework animation to be done in two phases.

Objective
Bevy's animation system currently does tree traversals based on Name that aren't necessary. Not only do they require in unsafe code because tree traversals are awkward with parallelism, but they are also somewhat slow, brittle, and complex, which manifested itself as way too many queries in #11670.
Solution
Divide animation into two phases: animation advancement and animation evaluation, which run after one another. Advancement operates on the AnimationPlayer and sets the current animation time to match the game time. Evaluation operates on all animation bones in the scene in parallel and sets the transforms and/or morph weights based on the time and the clip.
To do this, we introduce a new component, AnimationTarget, which the asset loader places on every bone. It contains the ID of the entity containing the AnimationPlayer, as well as a UUID that identifies which bone in the animation the target corresponds to. In the case of glTF, the UUID is derived from the full path name to the bone. The rule that AnimationTargets are descendants of the entity containing AnimationPlayer is now just a convention, not a requirement; this allows us to eliminate the unsafe code.
Migration guide

AnimationClip now uses UUIDs instead of hierarchical paths based on the Name component to refer to bones. This has several consequences:

A new component, AnimationTarget, should be placed on each bone that you wish to animate, in order to specify its UUID and the associated AnimationPlayer. The glTF loader automatically creates these components as necessary, so most uses of glTF rigs shouldn't need to change.
Moving a bone around the tree, or renaming it, no longer prevents an AnimationPlayer from affecting it.
Dynamically changing the AnimationPlayer component will likely require manual updating of the AnimationTarget components.

Entities with AnimationPlayer components may now possess descendants that also have AnimationPlayer components. They may not, however, animate the same bones.
As they aren't specific to TypeIds, bevy_reflect::utility::NoOpTypeIdHash and bevy_reflect::utility::NoOpTypeIdHasher have been renamed to bevy_reflect::utility::NoOpHash and bevy_reflect::utility::NoOpHasher respectively.

on the many_foxes examples, this PR is slower than main

I would expect the problem to be #11711. I also haven't looked at performance at all.

By the way, I don't know how to implement the Animatable trait, which is an approved RFC, without something like this or an explosion in unsafe code.

I've verified that with #11711 applied on top of this, the many_foxes example is faster than main with this PR. Without #11711, this PR alone is indeed slower because of all the hashing.

I think this is ready for review.

@james7132 I added more documentation to call out the caveats you mentioned. Let me know how you like it.

@djeedai
Benchmark on Windows 11, Intel(R) Core(TM) i9-10980HK CPU @ 2.40GHz, NVIDIA GeForce GTX 1650 Ti with Max-Q Design:
main: frame time avg 24.60 ms, stddev 0.79 ms
two-phase-animation: frame time avg 23.79 ms, stddev 0.62 ms
Benchmark on M2 Mac mini, 2023:
main: frame time avg 15.06 ms, stddev 1.44 ms
two-phase-animation: 15.59 ms, stddev 1.11 ms

@pcwalton thanks a lot for the timings! Is that the foxes demo? Frame time over 15ms looks quite slow no? That means we're under 60FPS? I know this PR improves things but looks like there's still a lot of work to do then 😅

@djeedai This is a pretty low-end GPU and there's a lot of skinning going on in that demo. I don't think it's indicative of a problem.

On my APU laptop I get like 7-15 fps on many_foxes. I'd love it if we could pick that up, but it's about what I expect on that stress test. Still have to properly profile this, we'll see if tracy says it's an improvement.

I believe I addressed all comments.

@NthTensor I got rid of the warning.

example morph_targets is not working with this PR. https://github.com/pcwalton/bevy/pull/1 fixes it but breaks one of the assumptions of the PR

https://github.com/KhronosGroup/glTF-Sample-Assets/tree/main/Models/InterpolationTest doesn't work either with this PR

OK, I allowed AnimationTarget and AnimationPlayer to coexist on the same entity. The fact that they couldn't is a relic of when I was using EntityMut on AnimationTargets, so they would lock out the AnimationPlayers. This is no longer necessary.

Both animated_transform and morph_targets seem to be working. @mockersf

Both animated_transform and morph_targets seem to be working. @mockersf

animated_transform doesn't behave as before and https://github.com/KhronosGroup/glTF-Sample-Assets/tree/main/Models/InterpolationTest doesn't work

@mockersf You were running InterpolationTest through scene_viewer, right? Because the way that scene_viewer plays animations is pretty broken, and the fact that we now use multiple AnimationPlayers in this PR broke it. I'll fix it as part of this PR.

Those should be working now.

@mockersf You were running InterpolationTest through scene_viewer, right?

Yup!

Because the way that scene_viewer plays animations is pretty broken, and the fact that we now use multiple AnimationPlayers in this PR broke it. I'll fix it as part of this PR.

Thanks! It's good practice to fix the examples with the PR that breaks them, it avoids having to investigate without the context.


DEV #4 :Implement CI pipeline for the Project

Priority:  P1
Story Points:  4
User Story:
As a developer, I want to have a fully functional CI pipeline for Project, so that I can automate the build, test, and deployment processes, ensuring that code changes are continuously integrated and validated.
Potential Risks:

The process of implementing CI can be complex and time consuming

Demo Steps:

Commit the changes to the main branch in github

Acceptance Criteria:

On commit, it gives detail about the build

Sub-criteria 1.1: If it fails then code is not merged
Sub-criteria 1.2: If it passes then code is tested

On testing, it provides detail about the testing

Sub-criteria 2.1: If tesing fail then code is not merged
Sub-criteria 2.2: If test passes then it checks whether the coverage is greater then 80% if yes then the code is merged, if not then the merge is blocked

All the processes are repeated for both the frontend and backend

https://github.com/awaazo/awaazo/assets/46968018/547a641a-b3a6-435e-a0ab-8572c33fae29

The Tests are triggered every time a PR is made. They're also triggered whenever the main codebase is changed.


init push of unique key test reformat for redshift

Part of CT-375
Description
Working on getting the unique_key tests to work on redshift in new format
TODO:

get tests working
check data typing overriding
remove old test
update changelog

Checklist

[x] I have signed the CLA
[x] I have run this code in development and it appears to resolve the stated issue
[x] This PR includes tests, or tests are not required/relevant for this PR
[x] I have updated the CHANGELOG.md and added information about my change to the "dbt-redshift next" section.

@McKnight-42 I think you'll need to update this line:
https://github.com/dbt-labs/dbt-redshift/blob/162dc3046cb28c65602cc0648d2c92b9b68e59e4/dev_requirements.txt#L4
To point to the branch open in https://github.com/dbt-labs/dbt-core/pull/4958:
git+https://github.com/dbt-labs/dbt-core.git@mcknight/convert_unique_key#egg=dbt-tests-adapter&subdirectory=tests/adapter

(Unless we decide we want to merge the core PR first. IMO it's it's better to keep it open, to enable rapid iteration while working across the adapters, then merge only once they're all passing)


Use files from remote server rather than storing them directly in the repository

Description
When cloning the repository, I noticed that the total size of the repository is now over 150 MB. My understanding is that this is also preventing releases due to the upper limit of package sizes on PyPI being 60 MB.
To remedy this, I would suggest making use of the data manager functionality in sunpy. Below, I've listed a few files that I think this could easily apply to:

data/XRT20170324_151721.0.PSF560.fits in xrt_deconvolve
data/XRT20170324_161721.0.PSF1000.fits in xrt_deconvolve
All files in data/leak_fits in xrt_remove_lightleak

I'm still looking through the repository and will continue to add candidates to this list.

For an example of how this is done in aiapy, see here: https://gitlab.com/LMSAL_HUB/aia_hub/aiapy/-/blob/main/aiapy/response/channel.py?ref_type=heads#L89

The two URLs for the SSW mirrors that we should target are:

https://sohoftp.nascom.nasa.gov/solarsoft/
https://hesperia.gsfc.nasa.gov/ssw/

This has been done.


Transaction list: long description for incoming txs is being cut off

Describe the bug
Description is currently being cut off
Screenshots

To Reproduce
Steps to reproduce the behavior:

Create incoming tx with long description

Expected behavior
If full description should always be shown in the "disclosure panel" like i.e. here:

Additional context
The +/- icon in front of the tx is confusing. People tend to think this is clickable to open the panel but a possible panel is indicated by ^ (see screenshot).
If we would show mixed (incoming & outgoing) txs these icons might make sense but currently we do not mix txs afaik.
Should we get rid of these icons?
Are you working on this issue?
No

The +/- icon in front of the tx is confusing. People tend to think this is clickable to open the panel but a possible panel is indicated by ^ (see screenshot).

Maybe coloring them with green and red respectively can make it more evident that it signifies received/sent?

@dvoroneca and we decided to just remove them. The transaction list will be redone anyway in the mid-term.

is this fixed now?

Yes, fixed with #1389


Docs sidebar doesn't scroll when it fills up

On smaller screens, not all of the subheadings appear in the sidebar of the docs site. Sidebar isn't scrollable. Pretty minor issue, but makes navigation a little harder.
Screenshot from my 12.5" screen:

xref.: pyOpenSci/software-review#7

Hi @kysolvik,
I tried to address this problem in docs/source/conf.py with this commit marskar@b665916.


Clarity on the Role Prefix

Issue #, if available:
Description of changes:
Customers have been confused by what to enter as the Management Account Role and the Multi Account Role Name fields when they have changed this RolePrefix from the default (WA-). Added text to labs and CFN templates to help clarify.
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

@awsdunc We have a frequent customer issue linked to documentation, This PR make it more clear for customer and fixes the inacurracy in doc.
Please can you accept this doc bugfix?

LGTM


Add @hackygolucky as CommComm Director

Updates the role of Tracy Hinds as CommComm Director. Not exaclty sure when this land, but it's important to have once we're ready 😄

💯

Since it's been over a week since the Q&A went up, I'm going to merge this. Congrats @hackygolucky 🎉

I'm excited to get/have @hackygolucky on board (no pun intended)- but merging this was premature unfortunately. CommComm does not have a charter that supports this role yet and the Foundation's ByLaws still include @ashleygwilliams's position.


Stubs file is too big

Directory sizes
88  orders
92  system-status
96  legacy
96  query
108 locking
128 phpqrcode
320 settings
444 webapi
632 addon
672 splash-page
760 fields

Maybe distribute fields in a separate stubs file??

Maybe distribute fields in a separate stubs file??

Only 1.2 MB -> 1.0 MB reduction.


Fix .gitignore

It was quite wonky. This is a good .gitignore for all LaTeX projects.

Thank you!


SlackTextViewController is missing headers in upcomming Cocoapods ( 0.39.0.beta.5 )

Hi,
I am not sure if this should be considered an issue with Cocoapods or with SlackTextViewController so I will add this issue to both repos.
After upgrading to 0.39.0.beta.5 I got an error with pod SlackTextViewController (https://github.com/slackhq/SlackTextViewController/). My project claimed that the #import "SLKTextViewController.h"  reffered to a file that didn't exist. After some investigation I noticed that there were actually  no public headerfiles visible to my project in this pod.
My fix for this was to fork SLKTextViewController and make some changes to the folder structure. Here is the original folder structure which fails:

And here is the podspec:

What I did is that I removed these two subfolders Additions and Classes and copied all files within these two to the groundlevel of folder Source.
Here is my fork: https://github.com/knutigro/SlackTextViewController/tree/master-foap/Source which give me no errors in 0.39.0.beta.5.
I have too little knowledge about how cocoapods work or should work to say if this is a bug in Cocoapods or in SlackTextViewController, but to me it look like Cocopods do not make public headers from files within subfolders.
/Knut

Here is the issue I created in Cocoapods: https://github.com/CocoaPods/CocoaPods/issues/4337

I also faced this bug with Cocoapods and SackTextViewController. Going to use @knutigro  fork for now:)

works good with pods 0.39.0

seems so, many pods were broken

there now more stable env, only startups are:(

Thats right @dzenbot  it's probably safer to stay with the stable release. The problem now is that stable release of Cocoapods also has bugs that did stop me from building an archive for uploading to iTunes https://github.com/CocoaPods/CocoaPods/issues/4021.

I am also having this issue with 0.39 NOT the beta.

+1

@dzenbot Yeap, i recently updated to 0.39 and with over 20 pods in my project, only SlackTextViewController had missing headers. So, i guess that cocoapods installation is fine, but you may have to fix something :)

@dzenbot  Yes, thanks. That solves it for me :-)


Invalid ATOM news feed

See the W3C report:
http://validator.w3.org/feed/check.cgi?url=http%3A%2F%2Fwww.rabbitmq.com%2Fnews.atom
Reported by Aaron Kondziela<EMAIL_ADDRESS>on the mailing-list:
https://groups.google.com/d/topic/rabbitmq-users/RuNYBYRy5EI/discussion

This was fixed in #121.


Autocomplete world name

Using /goto or any other world lookup command should autocomplete world names (like how player names work in commands; see Server::GetClientByName())
(Thanks Empy for suggestion)

This and issue #28 resolved with commit https://github.com/vexyl/MCHawk/commit/9fd167a47fab10608654a9f9ff7fdabccd8a97cb.


Sort Connections

Sort saved connections in the connections tab.

Any projected release for this item? I came to file this a bug/feature request after seeing this 5 year old reddit thread: https://www.reddit.com/r/SQLServer/comments/aydkk6/azure_data_studio_question_anyway_to_sort_the/


Drop CSV and JSON as output formats

In my opinion we should drop the support for CSV and JSON, since they are not well defined and it would be difficult to implement them in the ODC engine. Keeping only the common formats (netCDF, geoTIFF, PNG) would also be beneficial for the users, avoiding confusion.

Update: JSON has been implemented also for the ODC engine. CSV might not be relevant anymore tough.


task: detect declared node type and queue validator actions

Dependency: #40 (changelog will clear up significantly after merging that PR)
This adds a task that detects the declared type on a node and adds validation tasks for all the primitive properties we know about that node type. Only supports Assertion so far.
The next step will be to go back and add validators for a couple cases that can't be covered by the "primitive" validators. (RDF Types, ID-type). Then we can get back in here and modify this to support all the different properties of Assertion, including for the embedded classes defined in the OB spec, like Evidence.

+1


understand lineageAlternative

:lineageAlternative is used (and misspelled) in the migration but is absent from the ontology...
It is used for instance in L8LS15118 (xml, ttl)... I'm not really sure if it's missing from the ontology or if I should replace it with another property... @xristy what do you think?

Perhaps Lobasang needs to be queried as to why that construct was used as it was.
According to the lineage.xsd, the alternative is included in a doc that has a main lineage, not as a standalone thing. In any event it would seem that it likely should be included in the ontology - I will take a look. Here's the xsd fragment:
   <xsd:complexType name="Alternative">
        <xsd:annotation>
            <xsd:documentation>An Alternative is used to express a variant of the main lineage inline in the same Lineage document rather than creating separate documents for each lineage variation. It is modeled on the conventions used in gsan yig for this purpose. So an Alternative will typically have three or more holders: 1) the start holder that mentions a holder in the containing Lineage that is the start point of the variation; 2) then one or more holders that represent the variation; and
                3) then a final holder that mentions a holder in the containing Lineage that is the end point of the variation. The start holder is marked by including a downTo element and the end holder is marked with a downFrom element. If the downTo holder is not present then the variation is a variation on the intial sequence of the lineage. If the downFrom is not present then the variation is a the end of the lineage. Generally the Alternatives should appear at the end of the main sequence
                of holders and there may be as many Alternatives as needed to express the variations in the gsan yig / thob yig.</xsd:documentation>
        </xsd:annotation>
        <xsd:sequence>
            <xsd:element name="holder" type="lin:Holder" minOccurs="1" maxOccurs="unbounded"/>
        </xsd:sequence>
    </xsd:complexType>

the intent of the lineage:alternative is defeated the way it's been used since it doesn't indicate what it's an alternative of.
The original model was developed for Ralph Kramers' lineage data and there are no occurrences of lineage:alternative in his data.
Lobsang seems to have used it 116 out of 1199 lineage docs that he added so he apparently had something in mind.
If he doesn't have a good answer I suggest we delete the lineage alternative and treat it as a main lineage otherwise how do figure out what it is an alternative of?

@lobsangshastri would you have time to explain what the "alternative" in L8LS15118 means?

I also note that at least one of the lineage docs, L8LS15732, has a bunch of empty holders as though it either isn't completed or a bunch of unneeded holders were added and not removed after completing the lineage

There are a series of lineage docs: L2RKL871 - L2RKL8728 which are alternatives.These docs use lineage: lineageRef to refer to another lineage and otherwise use lineage:not to mention the original lineage id that Ralph used in his data

did Lobsang answer? Or are you asking me? If it's up to me I'd delete the lineage:alternative and simply write the lineage like the others. If there's info about what it is an alternative to then that could be added via lineage:lineageRef

No answer, @karmagongde what do you think? If there's no answer I'll proceed as you said

@eroux The alternative linage means If one Lama has received a transmission of a particular texts/teaching from 2 or more teachers is called alternative linage.

@karmagongde the lineage:lineageAlternative was not used in the original Ralph Kramer lineage docs. The way that it has been used in the 116 cases of L8LS... is not in accord with the model. The original intent was that an alternative would be included in the lineage document that the alternative was an alternative to.
That was deemed to be too unwieldy and alternatives were separate docs as in the cases of L2RKL871 - L2RKL8728 where lineage:notes and lineage:lineageRefs clarify the relationships.
The usage oflineage:lineageAlternative in the current cases does not include a lineage:lineageRef or a lineage:note so there is no way to know what it is an alternative to.
It will be sufficient to migrate these 116 case as simple lineage docs with lineage:downFroms and lineage:downTos.
We will simply retire the lineage:lineageAlternative as should have been done some years ago.


The Jansi Console Issue also affects GraalVM reflection configuration generation (building with gradle)

When generating the reflection configuration by running the generateGraalReflectionConfig gradle task, the AnsiConsole class is picked up and included in the output.
  {
    "name" : "org.fusesource.jansi.AnsiConsole",
    "allDeclaredConstructors" : true,
    "allPublicConstructors" : true,
    "allDeclaredMethods" : true,
    "allPublicMethods" : true,
    "fields" : [
      { "name" : "out" }
    ]
  }

Giving this error when attempting to generate a nativeImage
error: Error parsing reflection configuration in blah/build/cli-reflect.json:
Class org.fusesource.jansi.AnsiConsole not found
Verify that the configuration matches the schema described in the -H:PrintFlags=+ output for option ReflectionConfigurationFiles.
Error: Image building with exit status 1

Manually removing the AnsiConsole block from the reflection configuration allows native image to build successfully.
Similar to issue #630

This problem does not occur if the jansi jar is included in the classpath when generating the GraalVM native image.
I understand that this is not strictly necessary because GraalVM does not yet support native images for Windows, so it is probably okay to exclude the org.fusesource.jansi.AnsiConsole class from the generated reflect.json file. (It'll be interesting to see what happens when running the resulting native image when system property "os.name" is set to "Windows".)
Do you think you will be able to provide a pull request for this?

Pushed to master.
This will be included in 4.0.0-alpha-3 and later.

Reopening since Graal GA was released in the last few days with native-image capabilities for Windows.
This means that it makes sense to include Jansi again in some cases.
I will add a command line flag for the gen-reflect-config tool to optionally include configuration for the org.fusesource.jansi.AnsiConsole class. This flag will be false by default, so users need to explicitly request it to be included. I will update the documentation accordingly.

Actually, I found a better way to do this.
picocli can now detect whether Jansi was installed without reflecting on any Jansi classes.
This has two advantages:

avoids loading the Jansi classes, so no native libraries are loaded (avoiding issues like #630)
no need to include configuration for org.fusesource.jansi.AnsiConsole in the reflect-config.json file


@craigspaeth => Fix edit button route

Hold over from the article2 switch -- switch link to /edit2 back to /edit

Another quick & easy one—feel free to merge when 💚 !


Issues running airflow using meltano invoke

Migrated from GitLab: https://gitlab.com/meltano/meltano/-/issues/648
Originally created by @robmoore1 on 2019-05-16 16:03:12

I am trying to get the airflow integration working with an example project I've created.
I start up the airflow webserver and look at localhost:8080 and all is well. However, if I then start the scheduler, I see the following if I refresh localhost:8080.
Broken DAG: [/home/rob/git/meltano-postgres/orchestrate/dags/meltano.py] Cannot find `//meltano.yml`. Are you in a meltano project?

Looking at the logs, I see more detail on this error:
[2019-05-16 10:41:21,795] {logging_mixin.py:95} INFO - [2019-05-16 10:41:21,794] {models.py:377} ERROR - Failed to import: /home/rob/git/meltano-postgres/orchestrate/dags/meltano.py
Traceback (most recent call last):
  File "/home/rob/git/meltano-postgres/.meltano/orchestrators/airflow/venv/lib/python3.6/site-packages/airflow/models.py", line 374, in process_file
    m = imp.load_source(mod_name, filepath)
  File "/usr/lib/python3.6/imp.py", line 172, in load_source
    module = _load(spec)
  File "<frozen importlib._bootstrap>", line 684, in _load
  File "<frozen importlib._bootstrap>", line 665, in _load_unlocked
  File "<frozen importlib._bootstrap_external>", line 678, in exec_module
  File "<frozen importlib._bootstrap>", line 219, in _call_with_frames_removed
  File "/home/rob/git/meltano-postgres/orchestrate/dags/meltano.py", line 22, in <module>
    project = Project.find()
  File "/home/rob/.venv/meltano/lib/python3.6/site-packages/meltano/core/project.py", line 48, in find
    raise ProjectNotFound()
meltano.core.project.ProjectNotFound: Cannot find `//meltano.yml`. Are you in a meltano project?

If you consult the README.md for the project I referenced above, you'll see the steps I took to set up my project. I follow steps 1-14 there to arrive at this point.
Please let me know if I can provide any further detail to assist.
My goal is to use airflow with Meltano. I know there is support coming in the Meltano UI but I'd like to be able to use airflow without the Meltano UI, if possible.

View 6 previous comments from the original issue on GitLab


Questions about how to check the cashe usage and, about specifying the cache size

Hi Team,
I've developed an application and used FastCache as an in-memory data store. So far, everything works fine. One thing I'm struggling is to figure out is how to see the cache usage in megabytes.
So,

Is there a way to get the cache usage (not the allocation) and/or the free quota?
If it's not possible at the moment, would such a feature be made available in the future?

There's some confusion in the documentation as well, when creating a cache using fastcache.New(maxBytes)
(https://pkg.go.dev/github.com/VictoriaMetrics/fastcache#New)

If we need a cache of 64MB, should we specify maxBytes as 64 or 64000000?

I'll be using it as 64000000. The given example is in (32MB) is given in MB that's why I got confused.


azure-identity missing but required

Add the pip install azure-identity package also as it is required on the imports on the python sample app
https://learn.microsoft.com/en-us/python/api/overview/azure/identity-readme?view=azure-python

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

ID: 7e866571-0651-bddb-f012-bf3bfe16f179
Version Independent ID: 27429467-3c37-27da-b839-065268c6af23
Content: Create a multi-service resource for Azure AI services - Azure AI services
Content Source: articles/ai-services/multi-service-resource.md
Service: cognitive-services
GitHub Login: @eric-urban
Microsoft Alias: eur

@fodorcosmin
Thanks for your feedback! We will investigate and update as appropriate.

@fodorcosmin
Thanks for your feedback! We will investigate and update as appropriate.

@fodorcosmin - Thanks for pointing that out! I'll update the doc today.

#please-close


Content-Type not being able to pass due to "-"

Content-Type not being able to pass due to "-" in request header.
This is occurring from headers.js file and when function () is being called.
-unknown-:6 - javascript evaluation failed: read('classpath:headers.js'), javascript evaluation failed: function()
{return{requesting_app_name: 'ABC', Content-Type = 'application/json', app_key: 'XXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXX'
}
}, :2:43 Expected : but found -
{return{requesting_app_name: 'ABC', Content-Type = 'application/json', app_key: 'XXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXX'
^ in  at line number 2 at column number 43
Version 0.9.0
It works perfectly if i pass this in * configure headers = { TrackingId: 'SomeValue', Token: 'AnotherValue' }.
Bascially the function method doesn't work if there is a "-" in Content-Type

@sadiqkassamali : If i understand your query correctly, you are unable to send 'Content-Type' in headers. So please use Content-Type in string quotes since it has a special character in it
refer -> https://github.com/intuit/karate#headers.
Try with
function()
{return{'requesting_app_name': 'ABC', 'Content-Type' = 'application/json', 'app_key': 'XXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXX'
}

Note: You don't need to mention header { 'Content-Type': 'application/json' } since karate automatically appends it to your request headers

No luck
-unknown-:6 - javascript evaluation failed: read(classpath:headers.js), :1:14 Expected , but found :
read(classpath:headers.js)
^ in  at line number 1 at column number 14
function()
{return{'requesting_app_name': 'ABC', 'Content-Type' = 'application/json', 'app_key': 'XXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXX'
}
}

You are using 'Content-Type' = 'application/json', try 'Content-Type' : 'application/json'. If it doesn't work, please attach your sample project here.

That did that trick, thanks for fresh eyes.


refactor(store): extracting types into its own folder

Extracting the interfaces from the store into the src/types directory for proper separation of concerns.

aite all is kewl


[Feature Request]: I would like to add solution for 34. Find First and Last Position of Element in Sorted Array

Is there an existing issue for this?

[X] I have searched the existing issues

Feature Description
I would like to add solution for 34. Find First and Last Position of Element in Sorted Array
Use Case
I will provide proper description and comments.
Benefits
No response
Add ScreenShots
No response
Priority
Medium
Record

[X] I have read the Contributing Guidelines
[X] I'm a GSSOC'24 contributor
[X] I have starred the repository

@thevijayshankersharma Please limit the number of issues created per day to 4-5.


Include linting and prettier into pre-commit hook with auto staging

Remembering to run prettier before staging and committing is tough. It's annoying when I forget. I've set up projects in the past that include linting and prettier into my pre-commit hook using lint-staged. It can auto stage the changes so I don't have to include them manually in the commit.
It may or may not be necessary to include some logic in the pre-commit hook to skip the linting and prettier for merge conflict commits. If this project has been using prettier since the beginning, then we're likely okay. But if not, running lint-staged for merge conflict commits can cause changes in files the developer never touched and can pollute their branches and PRs with extra changes.
Do we have a reason for why we want to run a check for prettier during commits but not a write? Is it to give the developer time to review the changes?

I can do this

See #95


Error on install

Installation script
curl -s aerokube.com/cm/bash | bash \
    && ./cm selenoid start --vnc --tmpfs 128

cause error
bash: line 1: html: No such file or directory
bash: line 2: syntax error near unexpected token `<'
'ash: line 2: `<head><title>301 Moved Permanently</title></head>

Ah, yes. This is because we enabled HTTPS everywhere in docs.

Two PRs to fix this:

https://github.com/aerokube/cm/pull/157
https://github.com/aerokube/selenoid/pull/432

Fixed. :)


Error

It gives me errors when i use this project. Can you help me?
{
"status": 404,
"code": 404,
"userMessage": "Not Found",
"developerMessage": "Not Found",
"systemMessage": [
"Controller class Controller could not be found."
]
}

I can't believed this issue is closed but the PR fixing the AppController::beforeRedirect $exit = true issue was not pushed 👎


Implement ScrollToTop button component

Description
Implement a ScrollToTop Button Component in the second theme. So that user can scroll the page to top easily once they are down in the page.
Reference: https://www.figma.com/file/rX1jl4pa04VD9itnzneeih/Webiu?node-id=0%3A1
Changes proposed in the pull request

Implement ScrollToTop button component
Implement the functionality
Implement the story for the component
Create documentation for the component

Screenshots (In case of UI changes):

Types of changes

[x] New feature (non-breaking change which adds functionality)

Checklist:

[x] My code follows the code style of this project.
[x] My change requires a change to the documentation.
[x] I have updated the documentation accordingly.
[x] All new and existing tests have passed and it does not give any unexpected error for the same.

Steps to test

npm i
npm run storybook (story will be displayed in CustomerService/ScrollToTopButton )

Resolves https://github.com/scorelab/Webiu/issues/258

@charithccmc @dinithminura can you review the PR please :)


Mount/subrouters

Adds router.Mount to route all requests to a prefix to a subrouter. Additionally, adds some benchmarks, documentation, and related fixes + tools.
Closes #90

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

69 of 69 (100.0%)  changed or added relevant lines in 4 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.02%) to 99.659%

Totals

Change from base Build<PHONE_NUMBER>:
0.02%

Covered Lines:
1170

Relevant Lines:
1174

💛  - Coveralls


Endurances races not shown

Hello,
First of all, thanks for this website ;-)
I'm not sure if I'm using your tool properly but when I select for example the 8 Feburary it should show the 12h Bathurst but it doesn't. It looks like it's the case also for Endurance Le Mans series

Hum it's not true for the Endurance Le Mans my bad ;)

Closing as relates to old season


Retrive math symbols

I have to use math symbols in my text box . how can i use this feature in angularjs . Im not getting any toolbars . can any one suggest good plugins or toolbars for getting math symbols like square roots, sigma etc.. thanks in advance

Support questions like this one should be asked elsewhere: https://github.com/angular/angular.js/blob/master/CONTRIBUTING.md#question


plugins in typescript branch

Plugin
plugin would be the entry. it should be executable or it should return a executable task.  #154 Task Queue Should Be managed
directly execute
class Jira implements Plugin {
     async execute(options) {
           const {collector, enricher} = options;
           for (const col of collectors) {
                await Collectors[col].execute();
           }
           for (const enr of enricher) {
                await Enrichers[enr].execute();
           }
     }
}

scheduled task
class Jira implements Plugin {
     async execute(options): Promise<Task> {
           const {collector, enricher} = options;
           cont task = new Task();
           task.next(collectors.map(col => Collectors[col]));
           task.next(enricher.map(enr => Enrichers[enr]));
           return task;
     }
}

Collector And Enricher
collector and enricher should be the components which do the real job. the jobs should be called from the pulgin or scheduled by the plugin.
collector
collector fetch data from server. we may not need all the fields from the server api. so defined the fields we saved is important. also that would be easy to use by enricher.
So may we need entity and collector together
\\ collector\Issue.entity.ts
@Entity()
class Issue {
}
\\ collector\Issue.ts
@Injectable()
class IssueCollector {

}

enricher
enricher process datas saved by the collector. after enriched, we may save all fields that collector fetched, or we would only save part of fields collector fetched and with some extended fields calculated by enricher.
So there should be another entity defined by enricher
\\ enricher\Issue.entity.ts
import JiraIssue from '../collector/issue.entity.ts';
@Entity()
class Issue extends JiraIssue {
}
\\ enricher\Issue.ts
@Injectable()
class IssueEnricher {

}

I thought we were going to have a tool-layer that is close to data-source original sturcture (say JiraIssue in this case)?
and then converted to standard type (say Issue).
If that was the case, I don't really understand the pseudo-code here, I'd like more input about last part "Collector And Enricher"


[YoutubeBridge] Added new YouTube Embed Bridge

Added new YouTube Bridge that embeds videos rather than linking to it.
The code is based on the existing YouTube bridge. I modified and redid some of its code to suit my needs.

Question about this PR:
The one major difference between this bridge and the common YoutubeBridge seems to be that it embeds the video into the RSS feed. This is just a few lines of difference, so instead of adding a complete copy of YoutubeBridge that needs to be maintained, why not make embedding an option in the common YoutubeBridge instead?

There are a few other differences, but if you think it should be merged into the existing Youtube bridge that’s fine with me. I just want/need embedded videos in my feeds the way I made it. I don’t really care which bridge does that.

Yeah, I think it's definitely a good option to have. If your changes are added into the existing bridge we also gain easier maintainability.

Fine with me 👍

Let me know if you need any help!


bug: Borderless cmdline popups

Did you check docs and existing issues?

[X] I have read all the noice.nvim docs
[X] I have searched the existing issues of noice.nvim
[X] I have searched the existing issues of plugins related to this issue

Neovim version (nvim -v)
NVIM v0.11.0-dev-20a7eeb
Operating system/version
archlinux 6.9.4-arch1-1
Describe the bug
With the commit 4c1efad, my existing configuration (also the configuration recipe for clean cmdline_popup) for noice.nvim started throwing errors for cmdline_popup views. The following error was thrown
....local/share/nvim/lazy/nui.nvim/lua/nui/popup/border.lua:686: text not supported for style:none

The issue is because you're setting the prompt as the popup title and nui.nvim doesn't support border text with border.style = 'none'.
Steps To Reproduce
Install noice.nvim with the provided configuration and run the command lua vim.fn.input { prompt = '>' }.
Expected Behavior
The text should either be still shown as part of the prompt or there needs to be better handling upstream. You could also just ignore the prompt text, but that would be a radical solution.
Repro
-- DO NOT change the paths and don't remove the colorscheme
local root = vim.fn.fnamemodify('./.repro', ':p')

-- set stdpaths to use .repro
for _, name in ipairs { 'config', 'data', 'state', 'cache' } do
    vim.env[('XDG_%s_HOME'):format(name:upper())] = root .. '/' .. name
end

-- bootstrap lazy
local lazypath = root .. '/plugins/lazy.nvim'
if not vim.loop.fs_stat(lazypath) then
    vim.fn.system {
        'git',
        'clone',
        '--filter=blob:none',
        'https://github.com/folke/lazy.nvim.git',
        lazypath,
    }
end
vim.opt.runtimepath:prepend(lazypath)

-- install plugins
local plugins = {
    'folke/tokyonight.nvim',
    {
        'folke/noice.nvim',
        dependencies = {
            'MunifTanjim/nui.nvim',
        },
        opts = {
            views = { cmdline_popup = { border = { style = 'none', padding = { 1, 1 } } } },
        },
    },
}
require('lazy').setup(plugins, {
    root = root .. '/plugins',
})

vim.cmd.colorscheme 'tokyonight'
-- add anything else here

This is not a bug. nui does not support border = 'none' for popup window. See https://github.com/MunifTanjim/nui.nvim/issues/290#issuecomment-1641413627. Should close.

I realize it's not a bug of nui.nvim, It is a bug in how noice.nvim is setting up the options for nui.nvim. see https://github.com/folke/noice.nvim/blob/4c1efadccc5d4568e3abf0afdd3ee0c5c27b4be9/lua/noice/view/nui.lua#L61
this line makes noice break for me, it doesn't check for style == 'none' or handle that case in any manner, if I checkout the commit before this change, it works great

Roll-backing to 29c7f030b0920e2f17263a9ab8742cde144b9140 (4.0.1) fixes this for me (temporary ofc).
Roll-backing to b828b575805f1b303c2f4b768744609835140739 (4.1.0) should also work, unless you're using a custom position for cmdline_popup since 07e8f9c7dfb01405f8b1173f1d3939b41753779d changed how said position is being handled.

input uses the cmdline_input view which extends from cmdline_popup.
I just pushed a change that now always makes sure cmdline_input has a border.
If you don't want that, you need to change the view used by cmdline input. (see config)


Browsersync

This adds browsersync, moves everything to using a single set of helper methods for getting tasks.
Feedback welcome.
cc @surma.
Will add tests for this in a seperate PR (once the approach is ok). Thinking I'll just stub out the reload callback and check it's being called.

Minor things, LGTM otherwise :)


Flextype Core: Decouple content and plugins from default package

Need to decide what content, entries and etc.. should be by default inside Flextype release package. Posible gallery, blog, comments, catalog and etc.. should be standalone plugins and not a part of the Flextype by default.

Decoupled from release package.


Rerender and bump version

R 3.5.1

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


No config flow

Version of the custom_component
v.0.1.0

Configuration
Installed via HACS

Describe the bug
After the intergration is added, there is no config flow, and no device_tracker is added.
asyncio is not defined, see log.
Debug log

2020-09-03 08:06:40 ERROR (MainThread) [homeassistant.config_entries] Error unloading entry Leaf Spy for leafspy
Traceback (most recent call last):
  File "/usr/src/homeassistant/homeassistant/config_entries.py", line 309, in async_unload
    result = await component.async_unload_entry(  # type: ignore
  File "/config/custom_components/leafspy/__init__.py", line 54, in async_unload_entry
    await asyncio.gather(
NameError: name 'asyncio' is not defined
2020-09-03 08:06:57 INFO (MainThread) [homeassistant.components.device_tracker] Setting up device_tracker.leafspy

Thanks for noticing this.
Closed by https://github.com/jesserockz/ha-leafspy/commit/7ceeb9bccb824c42001824999ec0e8222aa364c9


Change request trace ID format

Follow up to https://github.com/aspnet/KestrelHttpServer/pull/1578. In this PR, we Kestrel started generating request IDs instead of letting aspnet/Hosting do this. At the moment, request IDs use the same code to generate connection ID and request ID.
We want to enhance this change further by making the requestID to the following format:
//pseudocode
var count = ++_connection.RequestCount;
HttpContext.TraceIdentifier = $"{_connection.ConnectionId}:{count.ToString("X").PadLeft(5, '0')}";

Result:
Connection 1 Id = 001BCA
  Request 1: Id = 001BCA:00000
  Request 2: Id = 001BCA:00001
...
  Request 11: Id= 001BCA:0000B

Connection 2 Id = 001BCB
  Request 1: Id = 001BCB:00000
  Request 2: Id = 001BCB:00001

In the unlikely event there are more than 0xFFFFF (1,048,575) requests per connection, the counter must not truncate to 5 characters, but use as many characters required to express request count in hex.

Let's make it future-proof. (PadLeft(10, '0')) maybe?

@shirhatti FYI. We should connect this with the ETW work you're planning in ANCM.

Decided to go with this:
//pseudocode
uint count = ++_connection.RequestCount;
HttpContext.TraceIdentifier = $"{_connection.ConnectionId}:{count.ToString("X8")}";

The X8 format represents the full range of info in uint.
FYI - uint.MaxValue is 4.2 billion.


Update Mincer to 1.2.4

Updating Mincer to 1.2.4 resolves issue #289, because of nodeca/mincer#186, which translates into compatibility with sass/node-sass 2 and ultimately Node 0.12. Whew!

Oops, tests fail. Will try this again with npm test first. Sorry


Refactor unshimmed classes into a separate Maven module

We've hit a few problems recently with unshimmed classes referencing shimmed classes directly which is problematic at runtime when the unshimmed class is not using the classloader that is parallel-world aware.  It would be nice to catch these issues at build time.  One way to do that is to refactor the code so that the shimmed and unshimmed classes are in separate Maven modules.  The shimmed module would depend on the unshimmed module but not vice-versa.  That way unshimmed classes are not allowed to reference shimmed classes except via the ShimLoader interface or similar reflection techniques, and the build will fail if attempting to reference them directly.

looks like a duplicate of #6565

Yes, completely!  Thanks @gerashegalov I missed that when I went searching for it earlier.


Step by step deailt guide line

Hi,
I'm new to firebase stuff. would you mind explaining a bit in details step by step to configure this. I have done data base and storage part not stuck with hosting ?

It's not necessary that you have to use firebase hosting ,  you can use localhost instead.

Hi, I'm new to firebase stuff. would you mind explaining a bit in details step by step to configure this. I have done data base and storage part not stuck with hosting ?

can you make a video bro save us the time

@ChillVibesMushroom There are 1000+ videos available on youtube , I don't think that it will take more than 10 seconds to search.
Whereas to save your time here is the video.
https://t.me/c/1526837741/2928


Image Streamer API300 Golden Image

Description
Add support to ImageStreamer::API300::GoldenImage
Issues Resolved
#204
Check List

[ ] New functionality includes testing.

[ ] All tests pass ($ rake test).
[ ] New resources and/or actions have are included in the matchers.rb and matchers_spec.

[ ] New functionality has been documented in the README if applicable.

[ ] New functionality has been thoroughly documented in examples (please include helpful comments).

[ ] Changes documented in the CHANGELOG.

@jsmartt I've done some changes regarding your comments


Possible to just trigger one notification at a time?

I wonder if its possible to make sure make sure the previous notification are closed/removed before its possible to add/popup a new one? So you dont have like 10 notifications up at the same time?

NotificationSystem has removeNotification method which accepts the uid.
You can wrap the main component in another one and access it via ref, then you would have access to all the notifications in the state of NotificationSystem.
You can do a forEach on them in componentWillReciveProps or similar and remove it before adding new one.

@swellmar Please check if @gor181 solution helps you. I'm closing it for now but feel free to re-open it with more info if needed.


I want to be able to edit machine tags directly on the Sample UI

Currently machine tags can only be changed through a REST call.  I would like the Sample UI to support me editing the machine tags

[ ] create edit dialog for tags @matt-primrose
[ ] make call to MPS to update tags when changed @rsdmike

Duplicate


Fix simple typo: signifigcant -> significant

 Issue by timgates42
Fri Feb 28 09:07:29 2020
Originally opened as https://github.com/codelucas/newspaper/issues/787

There is a small typo in newspaper/extractors.py.
Should read significant rather than signifigcant.

done


New version: IMASdd v2.2.5

Registering package: IMASdd
Repository: https://github.com/ProjectTorreyPines/IMASdd.jl
Created by: @orso82
Version: v2.2.5
Commit: 573a831f896f3a319ee4e3f0b1c951674282abd0
Reviewed by: @orso82
Reference: https://github.com/ProjectTorreyPines/IMASdd.jl/commit/573a831f896f3a319ee4e3f0b1c951674282abd0#commitcomment-147308009
Description: Basic data dictionary functionalities of IMAS.jl

[merge approved]


Develop test ci

What this PR does:
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:
Does this PR introduce a user-facing change?:

Codecov Report
Base: 37.58% // Head: 54.72% // Increases project coverage by +17.14% :tada:

Coverage data is based on head (7ca5a5c) compared to base (c117ad3).
Patch coverage: 42.61% of modified lines in pull request are covered.

:exclamation: Current head 7ca5a5c differs from pull request most recent head 491058c. Consider uploading reports for the commit 491058c to get more accurate results

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
Additional details and impacted files
@@             Coverage Diff              @@
##           develop       #1       +/-   ##
============================================
+ Coverage    37.58%   54.72%   +17.14%
============================================
  Files           55      667      +612
  Lines         3720    78143    +74423
============================================
+ Hits          1398    42767    +41369
- Misses        2166    31726    +29560
- Partials       156     3650     +3494

Impacted Files
Coverage Δ

istioctl/cmd/completion.go
0.00% <0.00%> (ø)

istioctl/cmd/deprecated_cmd.go
0.00% <0.00%> (ø)

istioctl/cmd/google.go
0.00% <0.00%> (ø)

istioctl/cmd/sysexits.go
100.00% <ø> (ø)

istioctl/cmd/tag.go
56.15% <ø> (ø)

istioctl/cmd/version.go
42.63% <ø> (ø)

istioctl/cmd/wait.go
72.47% <ø> (ø)

istioctl/cmd/workload.go
67.02% <ø> (ø)

istioctl/pkg/install/k8sversion/version.go
61.11% <ø> (ø)

istioctl/pkg/multicluster/cluster.go
60.00% <ø> (ø)

... and 712 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


CNN architecture

can you explain me the Neural Network architecture  (CNN) in SoccerNetv2-DevKit/Task1-ActionSpotting/CALF/src/model.py  ?
the number of layers and ...........

Hi @Al-Hussein-96,
Please find here a detailed diagram of the layers: CALFNet.pdf
If you need more details in a text form, you can also refer to the Appendix A.2 in my PhD Thesis: Thesis.pdf starting at page 153 with the same diagram at page 158.
I hope this helps don't hesitate to get back if you have more questions.

@cioppaanthony
thank you bro

You're welcome!


Added APO to products list

Added APO to products list. APO will get a tile in dev docs soon

Is this all that's required? Do we need to bump the version of @cloudflare/cloudflare-brand-assets to get the icon as well?

@jrf0110 good call! I bumped the version just in case


Numpy array

This pull request exposes a bug in the array slice routine in which it assumed all primitives with the same type are compatible and thus tries to use the direct copy method.  This leads to rather predictable failures.

This pull request is not ready to go.  It does correct the basic issue that memory was pushed around inappropriately but it requires the additional step of make numpy types be recognized by the canConvertToJava.  Without this everything will take the bypass and nothing will hit the optimized copy path.

This one needs to be finished.  Not sure yet why is does not work on *nix.

Hmm.  Seems that somehow I lost part of this pull request.
I did further analysis of the numpy problem.  The use of a memory view without using the numpy structures to first determine how to access the data is fundamentally flawed.   I started implementing a replacement under devel, but it is a pretty heavy lift because the NUMPY array api allows so many different memory organizations.   My implementation will certainly be better than allowing incorrect data to be passed in so long as the size of the data types is the same, but there are a lot of edge cases.  I am not sure even how I would create a misbehaving, byte order reversed, view of data with a stride.  Unfortunately, that is what would be required to hit the test cases to ensure we do everything correctly.
After it is fixed in devel we can look to back port if needed.

I do not think it is possible to map non contiguous arrays to Java. Therefore we should just raise, if the array does not full fill this requirement.

Seems like this is the problematic patch.   Shall we revert it?


Fix move operators

This PR addresses the following:

Some move operators had the following check:
if (this != &other) return *this;

which is obviously a typo since you want to return this if it's the same, not if it's different.

(not really important) while I was there I also added noexcept to allow move operators to be used by the STL containers. See here for an explanation why

I highly recommend you try clangd and/or clang-tidy: I see in v4.0.0 you additionally have some missing returns in move ops, clangd would have easily caught it.

Thanks so much for this. Good catches. I'll have to check out clangd!