Add Cinder storage manager support: backend changes

Purpose or Intent
Add Cinder storage manager to abstract all storage-related information from current Openstack cloud provider's detail page.
Steps for Testing/QA
After Openstack provider is added into appliance, the Cinder storage manager should be added in the 'Storage Providers' tab automatically.

@roliveri @jerryk55 Please review. Thanks,

Hui - thanks.

Please remove all the swift_storage_manager files.  I am working on Swift.  Thanks.
I find it redundant to name the managers "StorageManager::CinderStorageManager".  I believe Rich's original description had the names as "StorageManager::CinderManager" and "StorageManager::SwiftManager" and this makes more sense.

@hsong-rh @jerryk55 Has the UI fix, that Jerry made, been added to this PR?

@roliveri I don't see it.

Sorry, I'll do it now.

@roliveri @hsong-rh  that's the change.

@hsong-rh It looks like you missed some white space issues.

@roliveri This did not make it into Euwe branch.  Please label euwe/yes or euwe/no, as appropriate.

@blomquisg @Fryguy @Ladas we should have a conversation about this one.
We think the layout of the classes is somewhat wrong here.
please dont merge this into euwe yet

it should not be ManageIQ::Providers::StorageManager::CinderManager
but:
ManageIQ::Providers::Cinder::BlockStorageManager, if there is a separate provider Cinder, otherwise it should be a ManageIQ::Providers::Openstack::BlockStorageManager,
the same for ManageIQ::Providers::StorageManager::SwiftManager:
if there is a separate swift provider it should be ManageIQ::Providers::Swift::ObjectStorageManager, otherwise it should be ManageIQ::Providers::Openstack::ObjectStorageManager,
or you will have a ManageIQ::Providers::Openstack::StorageManager for both block and storage, or subset

Several discussions here with @roliveri @Fryguy @hsong-rh and myself about this.  I believe the current naming might not be correct but your suggestions may not work either.  Both Cinder and Swift can be fully separable and also integrated with other providers.  They certainly should not be linked under Openstack.  I will let the others here chime in when they log in today.

The current naming is correct. We intentionally didn't classify storage managers as block or object. The fact is, there are storage managers that support a number of different exported storage types, so name-spacing them doesn't work. Instead storage managers define which exported storage types they support. For example, NetApp storage manager can export block, file share and object. Also any given parent provider can have multiple storage managers, so ManageIQ::Providers::Openstack::StorageManager doesn't work. For example, Openstack may have CInder, Swift, NetApp, EMC, etc.

Euwe Backport details:
$ git log
commit 7fdd8b72122181614cf4f52656a013a5eef1b5e9
Author: Richard Oliveri<EMAIL_ADDRESS>Date:   Wed Sep 21 15:58:35 2016 -0400

    Merge pull request #11088 from hsong-rh/cinder_storage_manager_backend

    Add Cinder storage manager support: backend changes
    (cherry picked from commit 3b6744b19c45f34c3c4eb5df19396ca7e4ee1753)

If we would follow our current naming and say there can be separate Swift and Cinder managers, then the naming should be
ManageIQ::Providers::Swift::StorageManager and ManageIQ::Providers::Cinder::StorageManager
@Fryguy or have you guys changed the naming schema, so all storage manager are in one dir StorageManager? There was another code lately, assuming 3rd position is always a provider name, but in case of ManageIQ::Providers::StorageManager::CinderManager, it's not.

I still dont like the fact that provider details leak into the main codebase this way. Now I see app/models/ems_refresh/save_inventory_cinder.rb which should be a generic baseclass and the provider should implement a specific subclass.

Is there already a know case, where Cinder or Swift is used in another Provider other than OpenStack?
If inheritance does not work, we can still use composition. So we could provide modules that enable Swift or Cinder on a providers StorageManager.

@blomquisg @Fryguy @roliveri I recommend to stick to our current naming scheme / where  is only the namespace right now, but sometimes also a provider object. And  is the defacto EMS or a submanager.

Adding another level to this adds complexity (ProviderName/ManagerType/ManagerSubtype).
Moving managers for some types removes consistency (StorageManager/ManagerSubtype).

As I pointed out, that naming scheme doesn't work for the use cases we have for storage managers. You can't just call something "StorageManager" because there can be multiple types of storage managers for any given provider. Also some storage managers can be standalone - so, calling something "storage manager" is like calling a provider "provider".

BTW, I agree with the save_inventory_cinder part. That was just a stopgap to work around the fact that there are other cloud providers that refresh their storage themselves, and other solutions could impose risk on all cloud providers. This should change going forward.

@roliveri so what was the exact issue if we would follow the pattern and call them
ManageIQ::Providers::Swift::StorageManager and ManageIQ::Providers::Cinder::StorageManager
(..same for Ceph, Gluster, etc.)

Afaik for standalone, you can then have e.g. ManageIQ::Providers::Swift::IdentityManager  (following our manager split-up)
The provider here is swift, so it should be wrapped under that.

Because, logically, StorageManager is a higher-level classification than the specific type of storage manager. Wrapping storage manager implementations under the StorageManager namespace enables us to minimize namespace pollution and enables us to define common StorageManager specific classes and constructs without having to worry about clashing with classes of the same name defined outside the StorageManager namespace (which we've had to deal with int the past). It also enables us to maintain the code for storage managers under a common directory - admittedly, much less an issue once the managers are converted into GEMs.
I think much of the confusion stems from the current provider/manager dichotomy, which seems to have evolved over time - and is probably still evolving. In my view, a Swift storage manager and a swift provider are two different things. The naming scheme you describe eliminates the distinction. A provider is something that can stand on its own and it may employ one or more managers. So, a standalone Swift provider would be under ManageIQ::Providers::Swift, and it would employ the Swift storage manager and whatever identity service it required.
The original intent of the provider/manager scheme was to provide a means to provide multiple views of a given environment. Where the provider defined the environment (An OpenStack instance for example) and the managers provided the views (cloud vs infra). When the concept of manager was extended to include various components of the provider implementation, using provider as the top-level classification became less appropriate - especially when said managers could be mixed and matched between various providers and/or used standalone.

In my view, a Swift storage manager and a swift provider are two different things.

I'm :+1: on this

The naming scheme you describe eliminates the distinction.

I'm all in for your second paragraph, except this line. I dont see why this would be the case.

When the concept of manager was extended to include various components of the provider implementation, using provider as the top-level classification became less appropriate - especially when said managers could be mixed and matched between various providers and/or used standalone.

I still think this scheme makes sense. One Provider, multiple Managers. We still need to improve the visibility of the Provider concept in the UI and we need to work out interconnections of Providers and their managers. But all in all, this concept is, what the providers team is having as their world view :) @blomquisg please correct me if I'm wrong.
@roliveri and please dont see this as an offense, I can understand your reasoning and view on this architectural decision. Its a valid point of view on this.
But nevertheless I think we should align our views, storage and provider team wise.

Linking @Fryguy ancient Provider Architecture issue https://github.com/ManageIQ/manageiq/issues/519 here.
I mean, in the end, maybe this whole Provider -> Managers relation doesnt make sense, as our UI and users are thinking only in terms of Managers (which they call providers, heck)?!
But as I said, we should align ourselves.

@roliveri just saw the comment https://github.com/ManageIQ/manageiq/pull/11593#issuecomment-250787118 by @Fryguy that you had that intense discussion already :)

@durandom I agree, we should be in alignment. Toward that end, we may need to consider adapting the current naming scheme to accommodate types of managers that don't conform to the original assumptions.
The current scheme assumes a provider-specific view of managers - where the implementation of the manager is specific to the provider. For managers that are not provider-specific, the current scheme makes little sense. Forcing these managers into the current scheme results in a sub-category::category (Swift::StorageManagr) structure that isn't appropriate. From a logical, structural and code-reuse standpoint, a category::sub-category (StorageManager::Swift) makes much more sense for these managers. While there may be ways to shoe-horn said managers into the current scheme, I don't think it's the right thing to do.
In my opinion, a single naming scheme may not be able to accommodate all types of managers in a logically optimal way. We may need one scheme that applies to provider-specific managers and another that applies to non-provider-specific managers.  Going forward, we may encounter new types of managers that don't conform to our current set of assumptions. Not restricting ourselves to a single naming scheme would enable us to accommodate them more easily.

@roliveri I don't see Swift as a category.  I see Swift as an implementation.  I think the category is ObjectStorage.  And, it includes other implementations like IBM Cloud Object Storage or Amazon S3.
But, you're bringing up an important point.  Currently, we only really group Provider things together.  And, we've sort of tacked on Manager things into that namespace (for instance, CloudManager lives under ManageIQ::Providers).
Maybe it's time to create a ManageIQ::Managers space?  /cc @Fryguy
Then, we move the abstract manager things to ManageIQ::Managers and mix those in with the various ManageIQ::Providers.  So, things like ManageIQ::Managers::ObjectStorage::AmazonS3 could be used by ManageIQ::Providers::Amazon and ManageIQ::Managers::ObjectStorage::Swift could be used by ManageIQ::Providers::OpenStack.
Ugh, I'm also just seeing this overlapping with https://github.com/ManageIQ/manageiq/pull/11593#issuecomment-250787118.
We probably need to either start a github issue to collect this, or have a face to face to get the basic points on the table and start over from there.

@blomquisg I didn't say Swift was category, I said it's a sub-category - in that it should be under a category, and not the other way around (I was pointing out that Swift::StorageManager is backwards). I think we're in agreement here.
You're right, it makes a lot of sense to hammer things out at the Mangers level, before discuss different types of managers. Having a Managers namespace is probably a good idea. At least, it makes a lot of sense for managers that are not provider-specific - but that goes back to my comment that a single namespace scheme may not be sufficient.
Given the examples in your comment, I think we're pretty much in agreement, with some small changes.
Just to give you an idea of some of the requirements we were considering for storage: given the addition of the Managers namespace, I would say that the storage definitions would change to something like: ManageIQ::Managers::Storage::Swift. We don't categorize storage managers by the type of storage they export: object, volume, etc., because a given storage manager may export multiple types of storage.
Given we've probably broken the record for post-merge comments on a single PR, your suggestion to change venue is well taken. We used to discuss things like this in talk topics, but whatever venue people think is most appropriate is fine with me.


Add ability to rename columns

I am looking to add support for renaming columns in ecto migrations.
Here is the documentation I referred to regarding the Postgres and MySQL syntax.
http://www.postgresql.org/docs/9.4/static/sql-altertable.html
https://dev.mysql.com/doc/refman/5.1/en/alter-table.html

I will have to look more closely at the integration tests to ensure the RENAME syntax is correct
** (Postgrex.Error) ERROR (syntax_error): syntax error at or near "RENAME"
    (ecto) lib/ecto/adapters/sql.ex:210: Ecto.Adapters.SQL.log_and_check/7
    (ecto) lib/ecto/adapters/postgres.ex:59: Ecto.Adapters.Postgres.execute_ddl/3
    integration_test/support/migration.exs:77: Ecto.Integration.Migration.create_users_table/0
    integration_test/support/migration.exs:23: Ecto.Integration.Migration.change/0
    (stdlib) timer.erl:194: :timer.tc/3
    (ecto) lib/ecto/migration/runner.ex:22: Ecto.Migration.Runner.run/6
    (ecto) lib/ecto/migrator.ex:113: Ecto.Migrator.attempt/6
    (ecto) lib/ecto/migrator.ex:63: anonymous fn/4 in Ecto.Migrator.do_up/4

Ping. :)

Closing this in favor of #807. Thanks!

Sorry I couldn't pick up the slack here.


Should one of these be '_yes' ?

https://github.com/bmrf/tron/blob/2c1093196fed1cf8cc9f3c870497804f1eb4a48e/resources/stage_3_disinfect/stage_3_disinfect.bat#L141

Yes it certainly should be....facepalm. I'll fix it. Thanks for letting me know, good catch.


chore(autoware_pyplot): move to visualization folder

Description
common should be meant for base packages that multiple other packages depend on.
Related links
Parent Issue:

https://github.com/orgs/autowarefoundation/discussions/5462#discussioncomment-11384987

How was this PR tested?
Notes for reviewers
None.
Interface changes
None.

Effects on system behavior
None.

@soblin @yhisaki we're moving autoware_pyplot to a visualization folder, as per our discussion in the ASWG meeting (see https://github.com/orgs/autowarefoundation/discussions/5462#discussioncomment-11384987). common is meant for base packages and any package we put there should be agreed beforehand.

@soblin thanks for the review


Add a scale hover effect on donate here button

I want to add a transform scale for the donate button on the homepage so as to match the transition on the buttons in the navbar.

I would like to work on this under LGMSOC.

@supzi-del  I don't want any more changes in UI


Script to migrate data from Google Table to Supabase

A script should be added that fetches data from Google Tables, shapes it and inserts it into Supabase. The script should live in the scripts directory. This script will be manually updated whenever the Supabase schema changes in order to re-import the data. The script should NOT delete any data from Google Tables.

We're going with Directus instead.


ML 5.8 : Feature Scaling (A)

Description 📜
Audio explaining Feature Scaling.
Fixes #6140

Type of change 📝

[x] Audio (Should be in mp3 format Includes speech clarity, Concise ,Low distortion)

Domain of Contribution 📊

[x] Machine Learning

Checklist ✅

[x] I follow Contributing Guidelines & Code of conduct of this project.
[x] I have performed a self-review of my own code or work.
[x] I have commented my code, particularly in hard-to-understand areas.
[x] My changes generates no new warnings.
[x] I'm GWOC'21 contributor

Screenshots / Gif (Optional) 📸

Sir @abhisheks008, please do evaluate my PR.

PR merged @deepthi1107


Unknown datas due to unrefreshed page

[ ] Add a page reload button

it shouldn't do that since the fix added on v0.0.5, would you know how to reproduce it?

i think the contentScript didn't start correctly. Did you get an error when loading the page due to reloading the extension in development mode?

The screenshot comes from latest version, i'm on OperaGX and it's about a pinned tab, when Opera started, it doesn't refresh this page so the extension cannot work properly..

I'll be testing on OperaGX, but for now, I'm mainly focusing on Chrome/Edge.


Some links on team page to twitter/github profiles broken

Hi! I'd submit a PR if I knew everyone's correct twitter/github links but just FYI some of the links to the core team members social profiles are 404. :)

I just found 2 broken Twitter links and create a PR.

Thank you for reporting!
Good catch!
Merged!


Extract arbitrary instances into a published testing module

Other people want to be able to test with our instance.  This can also
be a place to create laws for things like EntityEncoder and
EntityDecoder.

/cc @Unisay @scottcarey @reactormonk from earlier discussion

lgtm

👍 lgtm as well.


🚀 | Sleep function with reducing volume

Feature Description
What functionality would you like to see in your phoniebox?
My son loves the Phoniebox very much and I would like to use the box as a sleep aid.
I would like him to be able to listen to music while falling asleep, which slowly becomes quieter.
How do you envision the feature to work from a users perspective?
At the moment there is a stop playout timer which stops the player after a certain time. Would it be possible to extend this with a function that slowly reduces the volume to zero over the set period of time until it stops playing?
If the function is activated, the phoniebox should slowly reduce the volume over the remaining sleep period.
Further information that might help
The activation could possibly be done via a checkbox next to the already existing Stop playout timer in the web app or via RFID card.
Afterwards, the volume should return to the previously set level so that the speaker is not muted the next time it is used.

Hi @Mechi144
a very nice thought. Not so trivial to implement :) the audio playout is controlled by a player that does not support this feature in itself. I will think about it.
All the best, micz

+1 -  while I already like the autosleep.
Braindump: Why not sending intervalled vol- decreases. User could even specify the time slots passing by before the next level down. Stupidly thought (sorry if stupid, im just a t(h)inker not a real coder):
playoutcontrols.sh -c=voldown
sleep 100
playoutcontrols.sh -c=voldown
sleep 100
playoutcontrols.sh -c=voldown
sleep 100
playoutcontrols.sh -c=voldown
sleep 100
playoutcontrols.sh -c=voldown
sleep 100
playoutcontrols.sh -c=voldown
sleep 100

While the sleep interval could / should be customizable.
I would try myself, but time... dat time....

Hi @Piehti
that should actually work. I like the thinking behind it. I will see if I find the coding time to convert the thinking into code :)

As I am not a very experienced coder I'm struggling with the calculated VOLSTEP. If the division is 0 (e.g. if volume is 20 and minutes are 30) I try to set it to 1. But in other situations (volume 80 and minutes 25) the step of "3" would be too small and the volume would only reach 26...
Maybe someone has an idea?

How do you get 26? 80-25*3=5 What's my mistake?
The first thing i thought about, was to not calculate the VOLSTEPS, but the period of time between the commands. But minutes is the smallest interval for the at command.
You already round up if the value is smaller than 1. Why not always round up?
Just always add 1 to the quotient of VOLPERCENT and VOLSTEP. You don't need the check for zero then. But with your example you will reach 0 5 minutes early.

The way I would do it is in a loop that does the following

get current volume percent
divide current vol% by ten, save in varStep
start loop ten itterations

get current volume percent volCurrent
volCurrentNew = volCurrent - varStep
set volume playout_controls.sh -c=setvolume -v=${volCurrentNew}

loop ends
shutdown silent playout_controls.sh -c=shutdownsilent

Thanks for your hints
@BerniPi : sorry, i exchanged the values, the biggest problem was volume 85 and 30 minutes where a leftover would have been 25
I followed @MiczFlor and now always have a fixed amount of 10 steps and schedule the "at" command every x minutes (where x is "value" divided by "10", so for value 30 it's every 3 minutes).
What do you think?
https://github.com/MiczFlor/RPi-Jukebox-RFID/compare/develop...manajoe:shutdown_vol_reduce?expand=1


Community feature request

Do you have ideas for new features for Gekko BacktestTool? Feel free to write it.

Hello,
The possibility to choose different periods and compare then all at once.
Period 1 = from 2018-03-01 00:00 to 2018-03-30 00:00
Period 2 = from 2018-04-01 00:00 to 2018-04-30 00:00
I ask this because sometimes a strat results in good/bad results in a very specific time frame. When we compare different time frames we can se if the result was pure luck or is consistent over time.
Thanks

This should be easy to do. I will try to implement today.

I would like to suggest a feature. Imagine the following scenario: I want to schedule a backtest once a week (by cronjob). This backtest should always test over the last 60 days. Because downloading of history data is slow, I would like to update my local history data each day (also automatically executed by cronjob). When this daily update of history data is done, I don’t want to actually backtest the data. The backtest I want to perform is pretty big, and it is a waste of resources to perform this backtest every day (when I update my history data). Also, if I want to update the historic data using:
perl backtest.pl -i -f last -t now
it would perform a backtest automatically only of the last day (because I would update my history data every day), which is pretty useless. I haven’t found a combination of parameters to only update the data without actually performing a backtest. Long story short: maybe it is a useful feature to add an option to do an import-only of history data with the backtest tool, without actually performing the backtest? This way, people can easily keep their local history data up to date, while performing the actual backtest whenever they want? Right now import of data is always combined with the backtest, while I require other parameters (regarding timespan) for my data import than for my backtest.


Why is information stripped from touch pad press events?

I seem to be only getting events that tell me that the touch pad is pressed, not where. Looking at
InteractionInputSource, find this method
private void InteractionManager_InteractionSourcePressed(InteractionSourcePressedEventArgs args)
{
InputManager.Instance.RaiseSourceDown(this, args.state.source.id, (InteractionSourcePressInfo)args.pressType);
}
where args indeed contains touchpadPosition (and a wealth of other info). But to the inputmanager, only the source id and the fact that the touchpad is pressed is passed on.
Why is this? Why is information that can be useful stripped? Am I missing something? Should I look elsewhere for this information

Yep, as currently coded, you can call source.TryGetTouchpad to get the position.
You can also monitor the OnInputPositionChanged event.
You could also use the TouchpadTouched/Released events, though the position could be added to the event data to provide more info.
You have a couple options here.

Ah. I see. Thanks


Add support for COMPRESSION argument to TDIGEST.CREATE

To support RedisBloom 2.4.0, we need to support the compression argument on TDIGEST.CREATE. This entereed RedisBloom as part of https://github.com/RedisBloom/RedisBloom/pull/520

There is already support for COMPRESSION argument:
public bool Create(RedisKey key, long compression = 100)
{
        return ResponseParser.OKtoBoolean(_db.Execute(TDIGEST.CREATE, key, compression));
}

Do you mean to add 'COMPRESSION' before the number?
Because in the documentation two different things are written, in the example it is written without COMPRESSION and in the syntax it is written with COMPRESSION
https://redis.io/commands/tdigest.create/

Fixed in #24


Loading d3-request via HTML versus a Node Module Produces Different Results

When loading d3-request via html <script src="node_modules/d3/build/d3.js" type="text/javascript"></script> d3.tsv works as described in the API documentation. For example the following code is in a node application and uses d3 without any further import/require statement.
d3.tsv("data.tsv", function(error, data) {
  if (error) throw error;
  console.log(data);
}

And loads data into the variable data as expected.
When d3 is loaded in a script or via a node terminal (REPL) via let d3 = require('d3');, such as in the following,
let d3 = require('d3');

d3.tsv("data.tsv", function(error, data) {
  if (error) throw error;
  console.log(data);
}

without d3 being defined in the html ahead of time, the variable data ends up being a three object array, where all objects contain Error: connect ECONNREFUSED <IP_ADDRESS>:80
Disclaimer: This problem came about using D3 as a component in an Electron application. This very well may not be an issue with D3 but it'd be helpful to understand why it works one way and not the other.

Answers my question https://github.com/d3/d3/wiki#local-development


Histograms in Tensorboard

I was training a few models, and for debugging something I wanted to use Tensorboard. For me, the use-case was to look for the histograms of the weights for each layer in the network. When I added the callback to the training loop, this is what I get:

Expected behavior: The histogram plot should display the distribution of weights, and biases(if any) separately for each layer in the model.
cc: @sampathweb

Please let me know if you want help to fix this. I can work on this

TensorBoard relies on variable names, so this is a name scope issue. We should introduce name spaces in layers / models as appropriate and in a cross-backend way (though I think it would only be meaningful in TF).

Thanks for the inputs @fchollet

though I think it would only be meaningful in TF

Any specific reason behind this?

Any specific reason behind this?

"Tensor names" are not a thing in JAX, and as far as I know also not a thing in PyTorch (which makes sense, being eager first).


Emmet Abbreviations in Css with - doesn't trigger Emmet suggestions

From @LukeMcGurdy in https://github.com/Microsoft/vscode/issues/33764#issuecomment-327418928

In a css file, run the command Emmet: Expand Abbreviation after p10-20
It expands to padding: 10px 20px;
Same doesn't show up in the suggestion list

@ramya-rao-a, hello!
How about this...
you write bdra - VS Code tells you, that it should be border-radius.. ok, it makes sense

but then you write bdra50p and emmet gonna expand it into ... border-right ?

there is even no letter "a" in the "border-right"

@ramya-rao-a - interesting workaround. Thanks. In reality, I'd probably just not use shorthand until there is a solution for this.
Another example to add to user3323 would be fl. In Atom, PHPstorm, Brackets etc. this would return float: left;. Makes sense, right? But in VS Code it's the 11th suggestion on the list. The first entry is for flex but fx is the Emmet short code for this.

lh16 = line-height: 16 (with no px at the end)

@user3323 Emmet always try to fuzzy match very eagerly. bdra matches the closest with bdr . See https://github.com/emmetio/snippets/blob/master/css.json#L43
Try it. Type bdra and run Emmet: Expand Abbreviation and you will get border-right: 1px solid #000;
VS Code then applies its own filtering to the suggestions from Emmet which filters out this suggestion because like you said there is no a in border-right: 1px solid #000;
We (not Emmet) added this type of filtering due to the issues in #28933 where Emmet was suggesting things that are counter intuitive when presented in the suggestion list. Imaging typing font-family and being suggested font: fantasy
bdra50p on the other hand is providing specific numeric values to the snippet. In this case, we (not Emmet) skipped the filtering. Usage of numerics in abbreviations narrows down Emmet suggestions.
So basically, we try to add a check point to Emmet suggestions and show them

the suggestion was coming from a custom emmet snippet
or if the abbreviation has numeric values
or when the expanded value is fuzzy matchable with the abbreviation

As for line-height, that is by design. line-height is one of those css properties that take just a number without units and that is what Emmet does here. You can add a unit alias for px and and then use lh16px
"emmet.preferences": {
    "css.unitAliases": "px:px"
}

@LukeMcGurdy The Css examples in the cheatsheet that use : are not supported in Emmet 2.0
I'll look into alternatives for those examples.
Emmet suggestions in Css are sorted among other css suggestions. That's the reason for the examples you mentioned where they weren't at the top. One way to solve this is to set have the below settings. This way Emmet suggestions are always assured to be on the top
{
    "emmet.showSuggestionsAsSnippets": true,
    "editor.snippetSuggestions": "top"
}

dt does expand to display: table though. It doesn't for you?

Thanks, @ramya-rao-a. I just noticed these in the latest release. dt does expand to display table so either it wasn't working in 1.15 (or at least my particular version of it) or I made a mistake. Either way, it works fine v 1.16  and in general, there seems to have been big improvements in how Emmet is implemented.
Smashing job VS Code people.

Thanks @LukeMcGurdy !

The fix will be out in tonight's  Insiders
You can get the Insiders from https://code.visualstudio.com/insiders

Note to verifier
p10-20 should result in padding: 10px 20px in the suggestion list
Same for m10-20 -> margin: 10px 20px

Hey @mjbvz @ramya-rao-a, I've been doing some testing on this issue. It seems as if it's not resolve entirely. For example, trying m0-a should produce margin: 0 auto;, however it does nothing currently. It seems that non-digit characters cause an issue in this way.

Have I been using Emmet the wrong way? Or is this an issue?

@BeauAgst nice catch. I've added a fix.
The fix should be out in the next Insiders build.
Until, then you can use the workaround of either using the command Emmet: Expand Abbreviation or set emmet.triggerExpansionOnTab to true and use tab


[Feature][Master][UI] Dependent task parameter passing

Search before asking

[X] I had searched in the issues and found no similar feature requirement.

Description
Dependent tasks can choose to inherit the output parameters of the dependent item and output them to downstream tasks.
This is a DolphinScheduler project of OSPP-2023.
Use case
Add an option, when the dependent result is success, the dependent task will inherit the output parameters of the dependent item that runs successfully and this option is true

Related issues
No response
Are you willing to submit a PR?

[X] Yes I am willing to submit a PR!

Code of Conduct

[X] I agree to follow this project's Code of Conduct

Nice,Do you working for it?

Yes, I'm ready to submit a PR.


Branch 160538962

Jenkins, test this please.

Jenkins, test this please.
Yeah, somehow that BUILD rule was duplicated.

Jenkins, test this please.

Jenkins, test this please.

Yay! All passed.


chore: bump juju to 3.4

Description
bump juju version from 3.1 to 3.4

Seems simple enough, but how has this been tested?
What I mean is, say there were a breaking change in 3.4 that we don't see until the NRF integration test is run. Do we need to verify 3.4 in all our integration tests manually prior to doing a PR like this?

This is a good point related to any change in this repo (not specifically this one). We probably should add some triggers to run CI for the project's dependants


MPI build broken: wrong version of ELPA linked

$ otool -DL $CONDA_PREFIX/lib/libmbd.dylib | grep elpa
	@rpath/libelpa_openmp.15.dylib (compatibility version 17.0.0, current version 17.2.0)
$ exa -l $CONDA_PREFIX/lib/libelpa_openmp.*dylib
.rwxrwxr-x 1.5M jhermann 11 Jan 15:37 /Users/jhermann/Downloads/tmp/lib/libelpa_openmp.17.dylib
lrwxr-xr-x   23 jhermann 12 Jan 09:21 /Users/jhermann/Downloads/tmp/lib/libelpa_openmp.dylib -> libelpa_openmp.17.dylib

For now I can just rebuild Libmbd, but I'm not sure how to fix this properly so I don't have to rebuild every time a new ELPA version is released. Can we link Libmbd against libelpa_openmp.dylib without the version specifier? Or is pinning the ELPA version a way to go?

I pinned the ELPA version in ELSI now. I hope I set the pin tightly enough to avoid this link of linking issues.


Renamed field's name in file. Return yaml if exists 'Accept' header

Coverage increased (+0.5%) to 82.519% when pulling d312764a8917ec29e6039810be9ccf7cec243f3d on change_fields into 1b134f2a0b629ef29a3892d081300677f5192184 on proxy.

Coverage increased (+0.5%) to 82.519% when pulling 9f9e5d6640cd123c243bb3a630bba67cd75fa7fe on change_fields into 1b134f2a0b629ef29a3892d081300677f5192184 on proxy.


Change source in ChromePicker

Hi all - was just wondering how to change the source in ChromePicker, can't seem to work out what props to use?
It shows up as a hex, i'd like to set it to rbg. This thing basically?

Thanks so much!

on the handleChage function , grab the value of color.rgb
it will give you an object with rgba values


Added command 'Main_Zone.Scene.Scene_Sel'

Hi Sebastian,
here is the PR with the new command 'Main_Zone.Scene.Scene_Sel'.
I added the command for the get and put node (works for both).
Hope, everything is fine :)
Regards,
Andreas
closes #18

Many thanks @afrixx !

Many thanks @afrixx !


Add Top Line and Deployment Grafana dashboards

Existing Grafana configuration contained no dashboards, just a skeleton
for testing.
Introduce two Grafana dashboards:

Top Line: Overall health of all Conduit-enabled services
Deployment: Health of a specific conduit-enabled deployment

Fixes #500
Signed-off-by: Andrew Seigner<EMAIL_ADDRESS>Top Line

Deployment

I am not entirely familiar of what the requirements are for the grafana integration. Are we going to be adding dashboards for service mesh metrics? i.e. destination, tap` and other control plane pods?

@dadjeibaah Re: service mesh metrics, you're correct, we have a plan add a conduit health dashboard, which will expose metrics specific to the health of the control plane pods. that task is captured in #420, and is also dependent on #583, which sets up a new Prometheus job called conduit-controller to collect exactly these service mesh metrics.

Will this PR address how to get these dashboards into the Conduit dashboard?  Grafana is accessable at http://<IP_ADDRESS>:XXX/api/v1/namespaces/conduit/services/grafana:http/proxy if the conduit dashboard proxy is running but doesn't load assets correctly because base_url is not set in the grafana config.
I think if we set base_url correctly, it should be possible to directly link to the deployment grafana dashboard from the web UI.

@adleong Re: dashboard access, there's an Enable user access to the Grafana dashboard task listed in #420 to solve this, via some combination of base_url, conduit dashboard, or other method.


remove rbac container proxy

Moving manager's container ports and env variables from manager_auth_proxy_patch.yaml to new files manager_ports_patch.yaml, manager_env_patch.yaml.
Disable manager_auth_proxy_patch.yaml which removes redundant rbac proxy container (fix problem in #144 PR).

Signed-off-by: oraz<EMAIL_ADDRESS>

/retest

/retest

/retest

/hold
And again: look at the diff. It's removing more than just the proxy container!

that wasn't planned this way (removing the earlier "request changes" = lgtm) 😄

/retest

/lgtm


Fix bootstrapper test broken by #2100

The BypassList property contains an array of regular expressions.

Thx.


Changes in the documentation file: webservices.rst

This PR adds the missing documentation in the webservices.rst file.

@lewismc I have merged this to see if its working.


RemoveRedundantDefaultPropertyAnnotationValuesRector with JoinColumn

This rule is removing JoinColumn annotation
-     * @ORM\JoinTable(name="vouchers_subscriptions",
-     *     joinColumns={@ORM\JoinColumn(name="voucher_id", referencedColumnName="id")},
-     *     inverseJoinColumns={@ORM\JoinColumn(name="subscription_id", referencedColumnName="id")}
-     *     )
+     * @ORM\JoinTable(name="vouchers_subscriptions", joinColumns={(name="voucher_id")}, inverseJoinColumns={(name="subscription_id", referencedColumnName="id")})

But in the doc it needs of JoinColumn
https://www.doctrine-project.org/projects/doctrine-orm/en/2.9/reference/association-mapping.html#many-to-many-unidirectional

Thanks for reporting. That should not happen.
Please send failing test fixture to the responsible rule, so we have failing CI in here.

Thanks for reporting. That should not happen.
Please send failing test fixture to the responsible rule, so we have failing CI in here.

Hi @TomasVotruba do you have any example how can I send it?
failing test fixture to the responsible rule

First step would be to reproduce the bug in http://getrector.org/demo
Then you can follow this docs: https://github.com/rectorphp/rector/blob/main/docs/how_to_add_test_for_rector_rule.md
The goal is to add fixture file to /Fixture directory of responsible rule.

We're narrowing issues to keep focus on active and engaging contributor to keep project growing.
Closing for lack of activity last 7 months :) Thank you for understanding.
Feel free to re-open with the PR with a fix :+1:


store: Preallocate output buffer when encoding postings.

This PR modifies diffVarintSnappyEncode method to preallocate output buffer based on number of passed postings. This helps to avoid reallocations.
name                      old time/op    new time/op    delta
EncodePostings/10000-4       114µs ± 1%     106µs ± 1%   -6.55%  (p=0.009 n=3+9)
EncodePostings/100000-4     1.11ms ± 1%    1.05ms ± 0%   -5.39%  (p=0.007 n=3+10)
EncodePostings/1000000-4    11.6ms ± 2%    10.7ms ± 0%   -7.21%  (p=0.009 n=3+9)

name                      old alloc/op   new alloc/op   delta
EncodePostings/10000-4      48.5kB ± 0%    13.6kB ± 0%  -71.98%  (p=0.000 n=3+10)
EncodePostings/100000-4      506kB ± 0%     131kB ± 0%  -74.08%  (p=0.000 n=3+10)
EncodePostings/1000000-4    5.86MB ± 0%    1.25MB ± 0%  -78.62%  (p=0.000 n=3+10)

name                      old allocs/op  new allocs/op  delta
EncodePostings/10000-4        18.0 ± 0%       2.0 ± 0%  -88.89%  (p=0.000 n=3+10)
EncodePostings/100000-4       26.0 ± 0%       2.0 ± 0%  -92.31%  (p=0.000 n=3+10)
EncodePostings/1000000-4      36.0 ± 0%       2.0 ± 0%  -94.44%  (p=0.000 n=3+10)

Notes for reviewers: I chose to pass expected number of postings as another parameter. Alternative I considered was introducing an interface with PostingsCount method, which may look nicer, but would add hidden dependency into diffVarintSnappyEncode -- for postings implementing the interface, it would be faster (use less allocs) than for postings not implementing it. Passing extra parameter is more explicit.

[NA] I added CHANGELOG entry for this change.
[x] Change is not relevant to the end user.

Verification
Added benchmark.

💪
Motto for the week: PREALLOCATE ;p

Indeed :)

Test and doc failures seem to be unrelated to my changes :-(

Thanks!


Thank you!

This was exactly what I was looking for and I just want to say thank you for making this available for everyone.
I do have one question though. Does this work with Laravel 9?

@rotaercz Hey, thanks!
Yeah, it definitely works with L9. I just ran the Laravel installer and added the package. It all worked without a hitch. Just follow the instructions in the readme, and you should be all set!


kprom: Support for multiple clients

Hi,
I'm trying to use the kprom plugin with two producers running in the same program to expose separate metrics for each producer. I expected the WithClientLabel configuration option to allow this, but when I register both producers via the hooks option to the same metric, only the most recent producer gets registered. Here's an example:
metrics := kprom.NewMetrics("test", kprom.WithClientLabel())

// Prepare producers
primary, err := kgo.NewClient(
		kgo.SeedBrokers(brokers...),
		kgo.ClientID("primary"),
		kgo.WithHooks(metrics),
	)
...
secondary, err := kgo.NewClient(
		kgo.SeedBrokers(brokers...),
		kgo.ClientID("secondary"),
		kgo.WithHooks(metrics),
	)
...
mux := nethttp.NewServeMux()
mux.Handle("/metrics", metrics.Handler())

// Run the actual HTTP server with metrics
...
// Only metrics about secondary client are seen (although I suspect it's sum of both)

However, I believe that supporting separate metrics for multiple consumers/producers within a single program is a valid use case for this plugin, especially since kgo allows multiple client instances in the same application. I would love to see this use case supported, I can even create a pull request for it if you'll find this issue valid.

I'm open to this, what's the proposed API?

Thanks for the swift response. I created a PR for this. I would retain the API, feel free to correct me if you don't like it.

@lukasrynt Why do you want to pass the same kprom.NewMetrics instance to both client? Why not create a new one and register both to a single prometheus registry and serve metrics from there? We are doing this in our application with multiple clients and it's working very well.

@mehranmeidani Thanks for the suggestion. I already tried passing the metrics to one common Prometheus registry, however the solution felt a bit hacky. Since kprom already creates its own registry, one would have to create an extra structure that implements the Collector interface, that another prometheus.Registry can register.
I might be missing something though. Would you be so kind as to provide some example code with usage of two kprom metrics instances? Thanks a lot!

@mehranmeidani Thanks for sharing your example; the Registerer option is quite handy and I missed that. Building on your approach, though, I'd set up two clients with different client IDs, that both write to same topic. This causes the program to panic due to duplicate metrics collector registration attempted.
Clients definition:
primaryCl := newClient(kgo.SeedBrokers(brokers...), kgo.ClientID("primary"))
secondaryCl := newClient(kgo.SeedBrokers(brokers...), kgo.ClientID("secondary"))
...

func newClient(opts ...kgo.Opt) *kgo.Client {
	metrics := kprom.NewMetrics("kafka_stats",
		kprom.Registerer(prometheus.DefaultRegisterer),
		kprom.WithClientLabel(),
	)
	opts = append(opts, kgo.WithHooks(metrics))
	cl, err := kgo.NewClient(opts...)
	if err != nil {
		panic(err)
	}
	return cl
}

This actually makes sense, as there's no way to distinguish between these metrics in this setup. I think that your solution is good as long as each client writes to different topic. In my case, though, both clients are writing to the same topic, but use different brokers, which isn't enough to differentiate the metrics.
This is the source of my problems and the need to create this issue. However I agree, that the related PR could be improved with the usage of Registerer option in mind.


anki: Enable tests

Motivation for this change
Enable tests when building anki
Things done

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

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

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[x] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[x] Fits CONTRIBUTING.md.

Would you like to convert this into buildPythonApplication using pythonPackages.callPackage, replace inherit with function arguments, remove the override of the list of phases, replace pythonPath with propagatedBuildInputs, and use checkInputs for the new dependencies?

@orivej I thought about that too but didn't really want to since I don't use the package myself. Anyway, it has been done.

Thank you! BTW the difference between buildPythonPackage and buildPythonApplication is that the latter does not prepend python version to the name of the package. I've fixed it in a2bebdd6cedb58819cc1738282b9f02964d34a9a.


tsconfigJSONRenderer fails on themes which render the surrounding quotes as separate tokens

When tsconfigJSONRenderer checks if a token is a known tsconfig property, it assumes the token starts and ends with double-quotes.  However, on themes where the surrounding quotes are rendered in a different color, they are different tokens.
This logic ends up removing the first and last characters, so for example, instead of checking baseUrl, it checks aseUr.
https://github.com/shikijs/twoslash/blob/8054fff138aa7a253aed3582a89e7ffc4ed28b96/packages/shiki-twoslash/src/renderers/tsconfig.ts#L17
I guess the fix is to detect if quotes are or are not included in the token and behave accordingly.

On the topic, I was wondering if anyone tried implementing JSON schema-powered tooltips.  Naively, I assume VSCode gets its JSON schema-powered tooltips from an LSP similar to TS, so I wonder if it could be consulted to provide twoslash-style tooltips but for JSON documents.  Do you know, has that been attempted, and is there anywhere I should go to read more?

Looks like the schema-based tooltips could be generated using https://github.com/microsoft/vscode-json-languageservice
If that seems like an interesting idea, I can extract it to a separate issue.

Yep, that's a great solution for a general JSON schema hovers - I'd like to keep the twoslash  tsconfig one short simple and focused, but I have no problem having a way to replace the tsconfig renderer with something more complex!


Internal Issues

I'm having internal issue with VS Code.
Please help!

Did you manually open ptvsd_launcher.py in the editor, or opened the entire extension folder?

Because we have not heard back with the information we requested, we are closing this issue for now. If you are able to provide the info later on then we will be happy to re-open this issue to pick up where we left off.


Support multiple platforms

Currently we are getting 10 less points in pub.dev for missing support for multiple platforms

This is caused because dart:io is missing support for web and more
for that we can use universal_io that is

A cross-platform dart:io that works in all platforms (browsers, mobile, desktop, and server-side).

I appreciate your interest in moving forward to web as well, but there is one bottleneck and that is the dependency dart_ping for this library. dart_ping currently not supporting JS. I though of implementing something similar as dart_ping and support web as well but I guess that's a ton of work for a busy Individual.

I have opened an issue for adding JS support in dart_pping, feel free to like
https://github.com/point-source/dart_ping/issues/15
Is there a chance that socket_io_client can do what you need?.

Hey guys, author of dart_ping here. I replied to the issue you opened with an explanation as to why I cannot add web support.
However, if you simply want your tool to show as supporting web on pub but do not actually need ping functionality on web, you can make use of conditional imports to only import dart_ping when the platform is not web. This should allow users to make use of all web-enabled features of your application and will appropriately show web support on pub.

Thanks @point-source for your answer, I will close this issue for now.

@guyluz11 Things are really restricted when it comes to web browser.


UTF-8 Error

I'm getting utf8 errors in the message input fields, is there a way to fix it?
If I enter the letters ğ, Ğ, ç, Ç, ş, Ş, ü, Ü, ö, Ö, ı, İ, it gives this error
Exception in thread Thread-176 (print_output):
	Traceback (most recent call last):
		File "C:\Users\User\AppData\Local\Programs\Python\Python311\Lib\threading.py", line 1038, in _bootstrap_inner
		self.run()
		File "C:\Users\User\AppData\Local\Programs\Python\Python311\Lib\threading.py", line 975, in run
		self._target(*self._args, **self._kwargs)
		File "C:\Users\User\AppData\Local\Programs\Python\Python311\Lib\site-packages\flet\flet.py", line 572, in print_output
		line = line.decode("utf-8").rstrip("\r\n")
			   ^^^^^^^^^^^^^^^^^^^^
UnicodeDecodeError: 'utf-8' codec can't decode byte 0xfe in position 3: invalid start byte

edit so the error message can be read

I noticed that when I run it using flet I get the same error.

@ixelizm can you give me please a source code reproducing the issue?

code is ok i am using a simple textfield but when i start it with flet app.py -r -p 5000 utf8 errors occur

You can try the fix with pre-release version of Flet module:
pip install flet --upgrade --pre

Let me know how it worked.


Fix a criminal code formatting mistake in Content.Server/EntryPoint.cs

Shameful mistake on my part


Table rendering issue on iOS15

Describe the bug
Tables are not being rendered properly on iOS 15, it just renders the raw markdown text. looks working fine on iOS 16 and 17.
Checklist

[x] I can reproduce this issue with a vanilla SwiftUI project.
[x] I can reproduce this issue using the main branch of this package.
[x] This bug hasn't been addressed in an existing GitHub issue.

Steps to reproduce

Realized this issue on my project. But it's also happening for demo project on main branch. (Deployment target for demo project is 16, so i downgraded it to 15 and commented out everything giving error so that i can run the project.)

Expected behavior
Should be able to render tables on iOS 15.
Screenshots

Version information

MarkdownUI: 2.0.1 & 2.1.0 & 2.2.0
OS: iOS 15
Xcode: 14.2 & 15.0.1

Ok now i've just seen the note, tables are not supported on iOS 15.
So my question what is the reason there is no support, is it causing any issue while rendering?

Hi @syegin,
Thanks for raising this issue. This is a known limitation, even documented in the Minimum Requirements section of the README.
The current table implementation uses a Grid view, only available from iOS 16, macOS 13, tvOS 16, and watchOS 9.
I am experimenting with other approaches to lay out the text that may enable table rendering on iOS 15 and fix other issues. But I can't promise anything at the moment.


CFE-1106: Remove featureGate checks for processing userLabels and userTags

PR contains below commits

Disabling feature gates checks for configuring GCP labels and tags to make it GA.
Addresses comment to make GCP tag UT code legible.

/hold https://github.com/openshift/api/pull/1971

This is not needed currently. We should first promote the gate, and then remove the reliance on the gate in a following release

/remove-hold

/test unit

/jira refresh

/hold
As mentioned, the feature gate promotion has gone into 4.18, the gates should be removed only in a subsequent release, this work should be completed in 4.19 at the earliest

/hold cancel
/approve
/lgtm


feat(translation): install svelte-i18n

👋 Hello world
This pull request is focused on add the internationalization for coolify! So non English peoples can use it freely (without a shitty Google Translate to be honest lol)
❔ Why ?
I add sveltekit-i18n to the pnpm Dev-dependencies and default user language.
So this would let SysA or beginners to help the project!
How does it works?
When the user connect to the website, it detect the language from the Request header (by default: English) and store it in a cookies lang.
Then the locales are loaded in the client.
If a translation is not found for the user's language, it fall-backs to English.
✅ ToDo

[x] Remove hard-coded texts
[x] Translate French (for an example)
[x] Translate English (for fallback)
[x] Create guidelines for translations with i18n.

❔ It remains 1 thing to do!
I didn't do the switch language part because of 2 things:

I'm not a designer at all
How the user switch of language? (dropdown flag system using flagpedia or dropdown using languages)

If someone can do it? Otherwise I can do it but that will take a lot of time!

As far as I tested out, anyone can pull my branch and test it from here!
I work on Contribution Guide for translations!
Please do not hesitate to comment over here to inform me of anything on i18n

Wow! Thank you! I can do the rest (UI + switch button)! Really appreciate it!! 💜

Is this same as #365 ?

Yeah, unfortunately, lots of things change since this PR and I cannot merge it, I need to create a new branch based on this! :)


PortWebHub-website

Website where user can find great portfolios for them free to use
Responsive
(Till now just added 3 simple port websites in it)

@gurjeetsinghvirdee this too


Build failure on Fedora 30

Describe the bug
When compiling Filament I get the error /usr/bin/ld: cannot find -lgcc_s
To Reproduce
I followed the steps as outlined in the README, I have installed all the recommended packages including. The cmake command runs fine. When I run ninja, building stops at around the 61st file with the error /usr/bin/ld: cannot find -lgcc_s  being raised
Ninja Output:
[61/1203] Linking CXX shared library libs/utils/libbenchmark_utils_callee.so
FAILED: libs/utils/libbenchmark_utils_callee.so
: && /usr/bin/clang++ -fPIC -stdlib=libc++ -std=c++14 -fstrict-aliasing -Wno-unknown-pragmas -Wno-unused-function -stdlib=libc++ -fcolor-diagnostics -O2 -DNDEBUG -fomit-frame-pointer -ffunction-sections -fdata-sections  -Wl,--gc-sections -Wl,-Bsymbolic-functions -shared -Wl,-soname,libbenchmark_utils_callee.so -o libs/utils/libbenchmark_utils_callee.so libs/utils/CMakeFiles/benchmark_utils_callee.dir/benchmark/benchmark_callee.cpp.o  -static-libgcc -static-libstdc++ -Wl,-Bstatic -lc++ -lc++abi -Wl,-Bdynamic && :
/usr/bin/ld: cannot find -lgcc_s
clang-8: error: linker command failed with exit code 1 (use -v to see invocation)
[70/1203] Building CXX object libs/utils/CMakeFiles/test_utils.dir/test/test_Path.cpp.o
ninja: build stopped: subcommand failed.

Desktop:

OS: Fedora 30
GPU: [e.g. NVIDIA MX150]

I'm not too familiar with Linux dependencies, but this might help.

I'm not too familiar with Linux dependencies, but this might help.

I have just tried that but the problem persists even after creating the symbolic link.

Just wondering why this was closed? Currently still on issue on F32 with Clang-9

Solution appears to be -DUSE_STATIC_LIBCXX=false


Refactoring: Relocate the init_client functions

Context: init_client functions do create resource-specific clients. In the chaos-azure case those resources are as well our domains.
Problem: Some of the init_client functions are in the main init module, some are located in the domain specific modules.
Solution: Unifiying these init_client modules would ease the structure and make the code more understandable.
I would rather suggest to relocate the domain specific init_client functions to their modules rather than having them in the main module.

@mkaszub I will relocate the init_client functions to their resource/domain specific modules


Added additional text to the manual explaining the use of a load balancer in manual mode.

Original author: Nicolas Bock
Original issue: https://charm.cs.illinois.edu/redmine/issues/276

The manual neglected to mention that the method StartLB() is not publically accessible in this form. Instead the LBDatabase object has to be referenced. I added some prose to explain this and a code snippet illustrating this.

Original date: 2013-08-28 19:06:20

Either apply this patch as is, or do a more involved fix to the API as in #277.


fix(api-client): strict TS config

Problem
Currently -> we have runtime errors
Solution
With this PR, we use TS more : )

Hans sees lots of CI errors, Hans converts to draft

whats the latest on this?


Update BlockTimestamp

We have a component called BlockTimestamp for displaying the timestamp of a block.
Currently, we are fetching the timestamp value via a GraphQL query.
As it takes 12 seconds to mine a block, we can optimize this component by doing a simple math.
By doing so, we can significantly reduce the number of requests made to our indexer.

There are some irregular blocks with block time 24, 36, 48 seconds, so let's leave the component as is now.


[Graph] add inplace setting through layer property @open sesame 12/02 09:37

now you can set in-place flag through layer property of tensor
operation layers.
rename "initializeInPlaceType" func to "initializeInPlace". now
"is_inplace" property is set in that function, too.
in some layers, support_backwarding flag may be changed by the
in-place setting.

Self evaluation:

Build test:   [X]Passed [ ]Failed [ ]Skipped
Run test:     [X]Passed [ ]Failed [ ]Skipped

Signed-off-by: Seungbaek Hong<EMAIL_ADDRESS>

:memo: TAOS-CI Version: 1.5.20200925. Thank you for submitting PR #2796. Please a submit 1commit/1PR (one commit per one PR) policy to get comments quickly from reviewers. Your PR must pass all verificiation processes of cibot before starting a review process from reviewers. If you are new member to join this project, please read manuals in documentation folder and wiki page. In order to monitor a progress status of your PR in more detail, visit http://ci.nnstreamer.ai/.

:octocat: cibot: @baek2sm, A builder checker could not be completed because one of the checkers is not completed. In order to find out a reason, please go to http://ci.nnstreamer.ai/nntrainer/ci/repo-workers/pr-checker/2796-202412020937250.95195698738098-3e2326fc0e6a9dd3c26ce35bc2637304773b42e8/.


Feature: add mixin for importing font family

Hi @KittyGiraudel,
I had added added a new mixin for easy import of font family
You may test it and feedback me is it fine or not.

Hello @atpriyanshu!
Thank you very much for taking the time to submit a pull-request to this repository. Unfortunately, I don’t think I will be merging it for the two following reasons:

I am no longer using Sass, so this repository is essentially archived. I haven’t properly archived it because I don’t want to give the impression to people that it no longer works or that it cannot be used. It’s just that I don’t intend to maintain it beyond the bare minimum in the future.
While Google Fonts is a popular registry for font-faces, it’s also not a standard. And even when pulling fonts from Google Fonts, it’s better done differently to improve performance. Loading fonts from the Google CDN also has GDPR repercussions in some countries.

I hope you understand.

Ok @KittyGiraudel
It's fine that repository is archived.
And in the mixin, people don't need to use google fonts. They can add the dummy url for the link.
Look at the following code:
$font-family-url: map.set($font-family-url, 'new-key', 'https://foo.bar/fonts/{}');
@include font-family('custom-font-family', 'new-key'); // @import url(https://foo.bar/fonts/custom-font-family);


make download is flaky

What happened:
make download is flaky in CI due to network issues: https://circleci.com/gh/kudobuilder/kudo/2468
We can likely improve this by enabling GOPROXY as well as retrying.

We haven't had any issues for a while now so I'm going to close this issue for now.


Precinct mapping

Checking to see how hard it would be to outline precinct. Also deciding where map centering should be.

You have merge conflicts in most of your files as well as extra script.js and index.html files. Might be an issue with cloud shell syncing or maybe you were halfway through a rebase you never finished? Seems like most of them don't serve any purpose so you can probably just delete them and fix the script.js and index.html files to look like you want them to


Included features

Base feature:

[ ] Base 1: Implement features from the Dielectric resonator antenna (DRA) design and analysis utility
[ ] Base 2: Impedance analyzer (including Smith chart analyzer)
[ ] Base 3: Superposition principle analyzer
[x] Base 4: User-friendly utilities to interact with EM simulators, including

[x] ANSYS HFSS

Extended features:

[ ] Extended 1: Characteristic mode analysis (CMA)

Base 4: User-friendly utilities to interact with EM simulators
ANSYS HFSS
Using pyaedt. But encapsulate frequently used procedures so the code is not that verbose.

Documentation
Only Python documentation atm, hosted on Read the Docs.


Handle 'NIL' date response

Expected behavior
Refreshing a folder should make the server contents of that server appear in K-9
Actual behavior
Synchronization runs for a few seconds, then aborts. The folder stays empty.
Steps to reproduce

Navigate into "broken" folder
Swipe down
(I assume this problem depends on the IMAP server software. I'll be happy to help with testing.)

Environment
K-9 Mail version: 5.115
Android version: 6.0.1 (CM13)
Account type (IMAP, POP3, WebDAV/Exchange): IMAP
Server Software: Zarafa 7.2.4
Debug log
k9-log.txt
I removed email addresses and subject lines from all mails except the last one before the crash.

NIL is not an invalid value. It's the expected value when it's not available: https://tools.ietf.org/html/rfc3501#section-4.5
We need to check for NIL before trying to parse the date.

If nil is not an invalid value, shouldn't it be handled in parseDate itself?

parseDate actually does the date parsing (i.e. it expects an actual date-time at that point). It's really a wrapper round SimpleDateFormat to copy with the various date formats we handle.
I think adding it to getDate is fine.

...to cope with the various date formats we handle.... like NIL? ;)

NIL's not a date format. It's basically the same as null - it's a spacer so the keys and values match up.


Feature/init sync

Bringing upstream commit in. This upgrades amazon-kinesis-client and dynamodb-streams-kinesis-adapter and adds some CloudWatch metrics.
Extending connector to include init.sync.skip flag. This optional flag will skip the initial load of dynamodb and read from LATEST on the DynamoDBStream.

Can you add a comment to the README documenting this setting?

Can you add a comment to the README documenting this setting?

for larger stream - do we want to increase Stream read limit - default is 1000 records - configured under com.trustpilot.connector.dynamodb.Constants.java

for larger stream - do we want to increase Stream read limit - default is 1000 records - configured under com.trustpilot.connector.dynamodb.Constants.java
That's a great find. Looks like that class' default is 10,000- not 1,000. I will address throughput in the next PR.

Does the flag need to be init.sync.skip = true? could you use init.sync = false?

There's no init.sync option that's why I'm making one. Init sync is default behavior so I named the flag init.sync.skip.

Can you add a comment to the README documenting this setting?

Take a look at the README and documentation.

Can you add a comment to the README documenting this setting?

Take a look at the README and documentation.

Whoops-  missed that. Thanks for updating


Error command lvlup

When I used the command "lvlup", I got an internal error

As it is currently lvlup requires and integer after it so it should ve like
'lvlup 10' for example

Ohhh


Gather CPU and Memory info across Linux, MacOS and Windows

A naive approach to gathering machine specs from the list in the inventory.yaml file.
Happy to suggestions for improvements.
So far it does not work well on flavours of Linux other than Ubuntu, and cannot access Windows machines so can't say if the commands even work.

Somewhat related
https://github.com/AdoptOpenJDK/openjdk-jenkins-helper

Somewhat related
https://github.com/AdoptOpenJDK/openjdk-jenkins-helper
Although I don't believe it's currently in use.

I guess this is a simpler version, that we can quickly run and get the task done!

Somewhat related
https://github.com/AdoptOpenJDK/openjdk-jenkins-helper
Although I don't believe it's currently in use.

I'm trying to remember but I think the jenkins-helper had some security concerns...

@johnoliver Needs a security review....

I believe there were security warnings with pieces related to adding Jenkins nodes. Functionality requiring Jenkins admin level access.

@johnoliver @karianna any update on this one?

@sxa Suspect this is superseded by Nagios, should we close?

@sxa Suspect this is superseded by Nagios, should we close?

Yeah I think any of the requirements for this are probably covered by Nagios, so I'll close. It can be reopened if required.


Add support for Atom Editor

So far Emacs/Spacemacs works perfectly fine with Elchemy, but after #130 Atom's elmjutsu doesn't work properly anymore.
This should fix the problem: halohalospecial/atom-elmjutsu#80

Atom again works like a charm after version 5.6 of elmjutsu 🎉


Polish the Azure.ResourceManager.Quota for first stable

Contributing to the Azure SDK
Please see our CONTRIBUTING.md if you are not familiar with contributing to this repository or have questions.
For specific information about pull request etiquette and best practices, see this section.

API change check
APIView has identified API level changes in this PR and created following API reviews.
Azure.ResourceManager.Quota

Please help to solve the conflicts


Minor grammar fix

@yihezkel : Thanks for your contribution! The author(s) have been notified to review your proposed change.

Looks good; thanks.


Support for RxJS 5 (beta)

Any plans to support RxJS 5 ?
I'm considering forking this repo to provide a RxJS 5 compatible version.

I'm working on a big update where I will create a 'react-observable' package with adapter for all of the observables library out there (rx, rx5, xstream, perhaps most bacon and kefir), should be ready soon

Thanks :+1:


🐛 : User not found issue

Sometimes when looking at chats(both chat list and dms) users appear as deleted accounts(user not found)
Did not occur when using website browser but appeared twice on ios app

Thank you for rising that issue. It should be investgated!

Thank you for rising that issue. It should be investgated!

Another thing to be aware of is that you can't send messages to the account either and the account becomes exactly like one that is actually deleted!! I couldn't send you any messages and I was confused until I looked up XD

Rooms has a member of "deleted-users"  will not be visible anymore and deleted for both sender and receiver.


Add ability to embed feature files and fixed issue with navigating to feature file

SpecFlowSourceMapAppender MSBuild Task
I created a custom MSBuild task that adds a source map comment to the end of feature files (similar to how JavaScript source maps work) so that we can determine the source file location. This makes navigating to a feature file bring you to the source feature file now instead of the one in the bin directory.
Feature files now look like this in the bin directory or when embedded in the dll:
Feature: Feature 4

Scenario: Addition in a folder
	Given I have entered 50 into the calculator
	And I have entered 70 into the calculator
	When I press add
	Then the result should be 120 on the screen

#sourceMap={"Version":1,"SourcePath":"C:\\Git\\GitHub\\TechTalk\\SpecFlow.xUnitAdapter\\src\\tests\\SpecFlow.xUnitAdapter.TestProject\\SpecFlowFeature4.feature"}

Embedded Features
Added SpecFlowFeature and SpecFlowEmbeddedFeature build actions.

SpecFlowFeature file's will have a sourceMapping appended to the feature file and will be copied to the targets directory
SpecFlowEmbeddedFeature files will have a sourceMapping appended to the feature file and will be embedded in the test assembly

To accomplish this, I abstracted SpecFlowFeatureTestClass and made FeatureFileTestClass and EmbeddedFeatureFileTestClass implement it.
I went back and forth on how I wanted to do this. I was thinking about having just SpecFlowFeature and having the default action be embedded, then there could be a property such as SpecFlowEmbedFeatures you could set to not embed the feature files. The build action would need to be set by the Visual Studio extension when adding a feature file, so if we want to support both types, there would need to be a new file type added to the extension. Thoughts?
EditorConfig
Added a .editorconfig file so that my Visual Studio settings follow your projects standards

This sounds good.
I will try to setup a build, so that we can better play with such ideas.

@gasparnagy I updated the appvoyer script to point to the 2015 solution file since it was having issues restoring the nuget packages when there were multiple solutions in the directory. Looks like the build is green now.
In your experience, what is the best approach to test features in a project like this? Would you make a test that starts a test process so that you can validate that the test should not succeed? I would like to add tests, but I am not sure what the best approach is.


[Bug] Not able to place widget at bottom of canvas due to slow loading of grid

Description : Not able to drag and drop widgets at the bottom of the canvas as the grid is taking time to load.
Steps to reproduce the behaviour:

Drag and drop a widget on canvas
Now scroll the canvas till bottom
Try to drag and drop another widget at bottom of canvas
Observe that the grid is taking time to load at bottom of canvas and due to this the widget could not be placed

Important Details

Environment : release-frozen regression
Version: Cloud
OS: MacOSX
Browser: chrome
Environment: production

https://www.loom.com/share/3cc23e01296d4fec8d98793f060ac37b

@vivekverma2312 can u check this against the new DnD?
if this is fine we can close this issue.

@marks0351 @vivekverma2312 Verified this and now able to place widget at the bottom of the canvas


🐞 Pre-release script app has an error

Description
The error is coming from CodeEditKit not being embedded.

To Reproduce

Download .dmg from the release.
Install CodeEdit
Launch CodeEdit

Expected behavior
The app should have launched
Version information
CodeEdit: [1.0]
Additional context
No response

Crashing for me as well.
MacOS Ventura 13.1 on M1

@widersky try it now

#887 fixes this

@bombardier200 it's working, thanks!


how to define multiple type variables?

Logger : {
    filter : Record -> Task Bool *,
    formatter : (Str -> Task {} *), Record -> Task {} *,
    writer : Str -> Task {} *,
}

The compiler tell me:
Type alias definitions may not use wildcard (*) type variables. Only
named type variables are allowed.

so should i define it like below?
Logger a,b,c : {
    filter : Record -> Task Bool a,
    formatter : (Str -> Task {} b), Record -> Task {} b,
    writer : Str -> Task {} c,
}a,b,c

but it seems not to pass compilation.

Maybe i should define Error type instead of using type variables


Add profiling to PyPI

I realize the README says this: "This project is still under development, so you should install it via GitHub instead of PyPI:" but IMO, profiling is definitely in a good enough position where a stable release can be thrown onto PyPI.

+1

I'll release this project when I implement a sampling profiler.  Please wait for a little more.

https://github.com/what-studio/profiling/releases/tag/0.1.1
Now you can install profiling via PyPI.


[SPARK-37664][TESTS] Supplement result of InMemoryColumnarBenchmark and StateStoreBasicOperationsBenchmark for Java 11 and 17

What changes were proposed in this pull request?
SPARK-37369 and SPARK-37224 added new Benchmark:

InMemoryColumnarBenchmark
StateStoreBasicOperationsBenchmark

but did not add benchmark results for Java 11 and 17, so this pr supplement these results, these results file generated by benchmark GitHub Action.
Why are the changes needed?
Supplement the missing benchmark result files for Java 11 and 17.
Does this PR introduce any user-facing change?
No.
How was this patch tested?
Pass the Jenkins or GitHub Action

cc @viirya @dongjoon-hyun @HyukjinKwon @srowen

Test build #146278 has started for PR 34921 at commit 7e943a3.

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/50752/

Kubernetes integration test status failure
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/50752/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/50752/

Test build #146278 has finished for PR 34921 at commit 7e943a3.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/146278/

Post +1, nice addition :)

thanks all ~


Development Manager L5 Wellington, New Zealand

Company Description
SG Digital (formerly OpenBet) leads the global gaming market, providing innovative software that powers the world’s most successful operators.
We pride ourselves on product innovation and technical excellence and are constantly adding new applications and new functionality to our product suite. And, while many other companies have been downsizing, we continue to see growth across our business.
Want to be part of our story?
Job Description
An opportunity has arisen for a Development Manager with extensive experience of Software Development and team leadership experience to lead one of our customer teams. You will report into the Engineering Delivery Director, APAC and work with a Project Manager in order to deliver top quality software to a high-profile customer. If you like building a strong team, with a focus on delivery efficiency and quality then this is a fantastic opportunity for you to get involved in one of SGDigital’s most strategically important accounts.
Responsibilities of a Development Manager at SGDigital:
Line management of a medium to large development team composed of 2-3 streams (15-20 people) including Software Engineers, Test Analysts and Development Leads
Key stakeholder liaison and alignment with senior people in customers organisation, from product development to operations and architecture
Lead shaping software engineering practices and processes on an account working in various flavours of Agile
Measuring and driving delivery and quality improvement through the capture and analysis of metrics.
Definition, implementation and evolution of development team technical process
Provision of technical input into the Project Management and Account Management
Work with Engineering management to drive through best practice, techniques and technology both on the team and around the company
Planning, implementation and evolution of technical processes and feedback to the wider company
Accountable for the quality of the implementation and deployment of development work
Accountable for the security, capacity and performance of the system
Motivating and driving the team with technical leadership and vision
Qualifications
If you are the right Development Manager for us you will be able to demonstrate:
Strong and demonstrable experience managing teams of highly technical staff – ideally managing through other managers of leads
Strong development pedigree acquired during a previous hands-on development role including TCL application development and Informix database systems
Experience delivering high-volume transactional back-end systems and front-end websites using modern web frameworks
Good managerial and leadership skills, able to motivate and lead personal development plans for employees as well as driving change and handling difficult situations both internally and externally
Ability to work as part of a senior management group leading the account
Proven communication and presentation skills
Dealing with change on a daily basis. You should be comfortable both with external factors driving this change as well as internals
Being opinionated. Especially about quality, and driven to empirically demonstrate improvements introduced.
Additional Information
Why would you enjoy working with us at SG Digital?
Competitive benefits, an open and supportive environment as well as a modern and exciting workplace
The opportunity to interact with global teams on a regular basis and the possibility to switch teams and projects as you and our business continues to develop and grow
Tangible and genuine development - at SG Digital, you can take your career where you want it to go!
Exposure to real work, business-critical projects and the chance to develop some of the world’s leading e-commerce sportsbooks whilst working on exciting client projects and interact with lots of code!
And if that’s not enough; we provide you with the guidance and development skills you need to progress quickly and enhance your career.
ref: https://www.openbetcareers.com/

N/A closed.


Uploading multiple files using the WebWorker

Hi Matt,
Thankyou for creating this control. Works well.
I am trying to upload some files using this control in the Web Worker. I have kept the following code:
var uploadOriginalFiles=function(host,files) {
for(var i=0;i<files.length;i++){
var file = files[i];
var formData = new FormData();
formData.append("image", file);
var filename = filenames[i];
var request = new XMLHttpRequest();
request.timeout = 0;
request.cache=false;
request.onreadystatechange = function() {
if (request.readyState == 4) {
if(request.status == 200) {
console.log(filename + " uploaded");
}else{
console.log("Upload failed");
}
}
};
request.open("POST", host);
request.send(formData);
}
};
var myWorker = Webworker.create(uploadOriginalFiles,{async: true});
myWorker.run(host,files).then(function(result) {
alert("Finished: " + result);
}).catch(function(err){
alert("Sorry, could not process your photo. The quality might be compromised. Please try again");
});
This code is working fine apart from the case if I am trying to upload multiple files (~6 files) with ~2MB each, I completes for some of the files, but fails for other of them. Checked the status for the request.readyState at 0,1,2,3,4. For some of the files, sometimes it does not even move to readyState 1, while if I try to upload the same file (single file only), it works fine. Is there some issue with memory with the web worker? Can you help me please?

@canvasmanager
I tried to upload 7 csv files with 3mb size. All of them uploaded successfully (readyState also getting 0,1,2,3,4 values). I am using Chrome.
http://codepen.io/anon/pen/WrGMqN?editors=101
But I got something strange in console: "Some file finished" appears 4431 times. It seems that onreadystatechange is problem.
Can you write what happened with your other requests? Stay freezzing and pending? It needs more debug information.

Did you get it working? My wife was having a new baby about the time of your original post, so I wasn't paying any attention to github. I'll close this unless you are still having a problem.

Yes, I was able to get it to working. The change, which I did in my worker process code was:
I keep on calling the upload again and again till the request status is not returned as 200.
Thanks,
Manik Mittal


[BUG] The plugins doesn't load

Version Information
KSP 2 Version
<IP_ADDRESS>.20892
Space Warp Version
0.4.0

Bug Information
Describe the bug
The mods aren't loading in any circunstance.
To Reproduce
Just install BepInEx and SpaceWarp 0.4.0 and put your UPDATED mods on the plugins folder.
Expected behavior
I expected to the mods load.
BepInEx Log
LogOutput.log

Can you provide a screenshot of your mods folder and then the mods under them

Can you provide a screenshot of your mods folder and then the mods under them

Alright, now can you provide a screenshot of the inside of the configurationmanager folder?

I mean the uiscaler one

Alright, one other member has had this issue before, IIRC though they accidentally dragged the wrong spacewarp.dll into the folder, but that doesn't seem to be the case, what version did you upgrade from?

This essentially boils down to https://github.com/SpaceWarpDev/SpaceWarp/labels/user error
Your folder structure should look like this inside the Kerbal Space Program 2 folder 📂

Clean up your Kerbal Space Program 2 folder 📂
Download the latest version of UI Scaler from here: UI Scaler
Download the latest version of Space Warp + BepInEx here: Space Warp + BepInEx
Reinstall the mods and launch the game. It works flawlessly if you do it correct.
Closing this issue as this is not a Space Warp https://github.com/SpaceWarpDev/SpaceWarp/labels/bug


Add support to trigger Bech32 address explorer in Ledger

Cosmos Ledger App now supports a way to visualize bech32 in the device screen.
https://github.com/cosmos/ledger-cosmos/issues/101#
Voyager should be extended to support this feature

moved to #2181


h3 appendHeader function makes case sensitive headers lowercase

Environment
Nuxt 3.2.0.
Create an endpoint like so:
export default createApp({
  debug: true,
})
  .use(
    defineEventHandler((event) => {
      appendHeader(event, 'Access-Control-Allow-Origin', 'test');
    }),
  )
  .use('/', () => 'hello world').handler;

Reproduction

Describe the bug
The h3 function appendHeader makes the headers lowercase and as they are case sensitive, they don't work.
Additional context
No response
Logs
No response

appendHeaders has the same problem:
appendHeaders(event,
  {
    'Access-Control-Allow-Origin':'test',
  }
);

HTTP headers names (i.e. HTTP field names) are supposed to be case-insensitive (ref).
Actually, appendHeaders and appendHeader utilities are wrapper of Nodejs' http.ServerResponse.setHeader, so it's Nodejs' native behavior.
https://github.com/unjs/h3/blob/a0e21c11c5bc9c219fa27fc531d690548926b12b/src/node.ts#L1-L4
https://github.com/unjs/h3/blob/0979caca040ac2738f70a374771bb751434bda9d/src/event/event.ts#L6-L14
https://github.com/unjs/h3/blob/0979caca040ac2738f70a374771bb751434bda9d/src/utils/response.ts#L116-L135
If your app doesn't work due to case-changed HTTP header name, it might be that a part of your infrastructure (e.g. server-side app) is case-sensitive about HTTP header. 🤔
Or, are you using some 3rd-party cloud service?

Maybe I'm testing it wrong. I'll try something. Thank you for your information. Always thought they were case sensitive. I tried your package also. Thanks for your work on it :)


chore: bump up vineyard version to v0.16.4

Codecov Report

Merging #3106 (d0c8755) into main (bc2edb8) will decrease coverage by 6.84%.
Report is 1 commits behind head on main.
The diff coverage is n/a.

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #3106      +/-   ##
==========================================
- Coverage   42.63%   35.79%   -6.84%
==========================================
  Files         100      100
  Lines       10817    10825       +8
==========================================
- Hits         4612     3875     -737
- Misses       6205     6950     +745

see 29 files with indirect coverage changes

Continue to review full report in Codecov by Sentry.

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


Basic SKU doesnt support Private Endpoint Datasource

Hi team,
I was trying Private Endpoint Datasource for ACS and got to this message in my Basic tier
{
"error": {
"code": "",
"message": "Setting execution mode of indexer v1-indexer to 'Private' is unsupported for your search service SKU basic"
}
}
In this page https://docs.microsoft.com/en-us/azure/search/service-create-private-endpoint it says
Pricing tier	Select Change Pricing Tier and choose your desired service tier. (Not support on Free tier. Must be Basic or higher.)
please fix it

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

ID: c21b14d6-f288-9c5a-8f60-81763c33c274
Version Independent ID: b10b0e1e-159b-04f6-abc3-9ed988dab5b1
Content: Create a Private Endpoint for a secure connection - Azure Cognitive Search
Content Source: articles/search/service-create-private-endpoint.md
Service: cognitive-search
GitHub Login: @mrcarter8
Microsoft Alias: mcarter

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

The doc page you linked doesn't mention data sources.  This is for private mode on the search service for inbound calls, which does support the Basic tier.  Outbound private endpoints is covered on this page: https://docs.microsoft.com/en-us/azure/search/search-indexer-howto-access-private
We should mention the Basic SKU limitation there.  No disagreement from me.  FYI @MarkHeff & @HeidiSteen

@arv100kri are you planning on adding the below table to the outbound private endpoints doc? If not, I can add it.

@ignaciofls Thanks again for the valuable feedback! I have assigned the issue to the content author to update the document as appropriate.

@ignaciofls - You are trying to create an indexer with a skillset (that utilizes outbound private endpoints) for a basic SKU service.
According to our documentation (@MarkHeff we already link to the limits and quotas page there) we list that this is an unsupported scenario.

@ignaciofls Doc updates are live -- thanks for filing the issue!

https://docs.microsoft.com/en-us/azure/search/search-limits-quotas-capacity#shared-private-link-resource-limits
https://docs.microsoft.com/en-us/azure/search/search-indexer-howto-access-private

#please-close


Feature Request: option to generate folder if missing

I believe it would be helpful to generate a folder, if it doesn't exist. For example:
ng g c models/person

... this will generate a class named person in the models folder only if that folder exists. This often forces the user to perform mkdir models first.
This is a minor feature request, but one that I think will be helpful in many cases.

https://github.com/angular/angular-cli/pull/3912 will make the parent dir all the time.

Also dupe of https://github.com/angular/angular-cli/issues/3307 (although title is misleading)

ah, thanks for the cross link


Josm 13996

Motivation for this change
Note: this builds upon https://github.com/NixOS/nixpkgs/pull/43073 because that's the order in which I prepared these PRs.
Things done

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

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

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

No attempt on aarch64-linux (full log)
The following builds were skipped because they don't evaluate on aarch64-linux: josm
Partial log (click to expand)

a) For `nixos-rebuild` you can set
  { nixpkgs.config.allowUnfree = true; }
in configuration.nix to override this.

b) For `nix-env`, `nix-build`, `nix-shell` or any other Nix command you can add
  { allowUnfree = true; }
to ~/.config/nixpkgs/config.nix.

Success on x86_64-linux (full log)
Attempted: josm
Partial log (click to expand)
/nix/store/c08nk8x2qy7fa0msqn02aca1hgqy48pc-josm-13996

Thanks for the PR but I already updated JOSM in master three days ago: https://github.com/NixOS/nixpkgs/commit/033c9ace4ee1bae291d9fd793613795a2ff034cb 🙂

OK great, thanks! I had checked from the latest bump of unstable where it wasn't yet, I guess :)


Have GVRContextProxy derive from GVRContext

Fixes what was broken for the live scripting scenario

GearVRf-DCO-1.0-Signed-off-by: Mihail Marinov<EMAIL_ADDRESS>

Verified I can telnet into gvr-remote-scripting and execute the sample scripts it includes.

since i can now run the scripts, i did a little more testing.  I used to be able to do a print() on a sceneobject and it would print out info like its transform, who its parent(s) are, etc.  Now it just crashes.  To repro, execute the space_trex.js and then issue:  print(trex);

Crashes in GVRBone.getFinalTransformMatrix; joml expects a direct floatbuffer but this is not what we give it. I might just eliminate the buffer altogether and construct the matrix out of the float[].

Planning to merge Monday morning.


Color component type from double to float

One issue I see with the underlying component moving from double to float is more the default implicit things that will require updates:
new Color(0.5, 0.5, 0.5, 0.5) // <- error

This will now fail to compile saying that there is no overload that will match and all the usages will need to have the "f" suffix added:
new Color(0.5f, 0.5f, 0.5f, 0.5f)

This is not a "big" change per se, but will be annoying. However, maybe not so bad as I am sure most people use the parsing from string instead...
However this also means that any implicit usage of the components will now break and require a cast.
var red = color.Red;
red = 0.5; // <- error

Relates to parent issue: #58

Context: It currently is float in this library: the wording in this issues comes from the fact that Xamarin.Forms is double.
double (64bpc) is insane and only explainable by a dev typing double without thinking about it. float (32bpc) is still excessive.
single (16bpc) is the appropriate floating point type, or byte (8bpc).
@dotMorten

@mattleibow

new Color(0.5, 0.5, 0.5, 0.5) breaks.

I don't think it has to:
 public Color(double a, double r, double g, double b) : this((float)a, (float)r, (float)g, (float)b) { }

We "know" Color has no use for double precision. So, throw away the precision.

Done.


只能用Gradle 2.14.1吗？

使用Gradle 3.3编译插件module报错 切换到2.14.1后编译成功
Error:(36, 0) Unable to load class 'com.android.builder.dependency.LibraryDependency'.
Possible causes for this unexpected error include:Gradle's dependency cache may be corrupt (this sometimes occurs after a network connection timeout.)
Re-download dependencies and sync project (requires network)The state of a Gradle build process (daemon) may be corrupt. Stopping all Gradle daemons may solve this problem.
Stop Gradle build processes (requires restart)Your project may be using a third-party plugin which is not compatible with the other plugins in the project or the version of Gradle requested by the project.In the case of corrupt Gradle processes, you can also try closing the IDE and then killing all Java processes.

是的 只能依赖 gradle 2.14.1

请使用dev分支


On Linux, Brackets always opens as if it was the first time.

It always acts like it's the first time opening the program. Live preview shows the notes like it would for a first-timer, extensions pop up with their settings as if it was just opened after an update... etc.

Which version of linux, brackets are you on?
Did brackets always behave this way or did it broke after an update?
Also could you go to debug>show developer tools > and see if there are any error messages in the console?

@abose Sorry about that.
Here's my system specs:
OS: Ubuntu 14.04 LTS / 64Bit
Brackets: 1.2, been broken since 1.1 I believe....
Console output can be found here.... Most of it seems extension related, but it's been happening without these extensions
Thank you for your help @abose !

It seems that brackets-minimap is throwing some errors.
Can you try removing the extension and see if the problems persist?
Also check if it works if you reload without extensions [debug>reload without extensions].

@abose reloaded without extensions, same thing. Removed brackets-minimap as well, same thing again.

Archlinux 64-bit, brackets 1.2.0-0 installed from git from here: https://aur.archlinux.org/packages/brackets-git/
Same situation for me:
Brackets is always:

Starting as the first time
Plugin panel opens but it continuously loads
Getting Started/index.html cannot be loaded and complains about not supporting UTF-8 files.
If I choose project folder, files inside of it are not shown.

You can see console output here: http://pastebin.com/G7DcSLh8

@DarkSkyGhost regarding the UTF-8 issue:
Sometimes my Brackets does this, and it turns out that it saves the files as non-UTF-8? I pull my files back from the server and retry whenever this happens, but it's slightly annoying.
Also addressing #2, this happens for me whenever I have a mediocre connection, even though it should be fine to connect.

@brennan-macaig Can you delete preferences as it seems Brackets not able to write to the preference file. Please follow steps given here

@prafulVaishnav will have to wait for monday to double check, don't have access to the computer.

I had the same issue on Sabayon Linux 16.02 (based on Gentoo).
UTF-8 error always showing and settings wasn't saving.
Debug -> Show developer tools (F12) wasn't work.
I think is may be file rights problem...
Problem has solved by reinstalling OS :)

Is the issue still there ?
@brennan-macaig
@m-bodmer
@abose @DarkSkyGhost @prafulVaishnav @rroshan1


filebeat.modules

Can you also add filebeat.modules  ?

This is a duplicate of #73, so I'm going to keep everything over there for now. Now that the template is a hash converted to a configuration file it shouldn't be too hard to put this together and include it in the config file.

could you help me how to use pure_hash.yml.erb and params that I should use in init.pp to populate pure_hash.yml.erb ? TIA

If you're using version 2.0 or later you're using the pure_hash.yml.erb template by default. If you had everything working before, it will work the same after upgrading to a newer module too.
The new template converts a Puppet hash that's formed in config.pp into YAML.
The usage section of the readme file hasa few examples.

Thanks for the reply, but what I'm looking for is how to add the parameter "modules" under key "filebeat".


GH-3267 OpenAPI spec for REST API

GitHub issue resolved: #3267
Briefly describe the changes proposed in this PR:

updated old swagger yaml to openapi 3.0 spec draft
added static webpage that renders the spec as redoc on the website
removed old hand-written rest api documentation: openapi spec is a replacement

This openapi spec can possible be improved further, both from a code-generation/testing pov as from a documentation pov, but I'd like us to switch over this ASAP as it will be easier to maintain and improve than the old hand-written docs.

PR Author Checklist (see the contributor guidelines for more details):

[x] my pull request is self-contained
[ ] I've added tests for the changes I made
[x] I've applied code formatting (you can use mvn process-resources to format from the command line)
[x] I've squashed my commits where necessary
[x] every commit message starts with the issue number (GH-xxxx) followed by a meaningful description of the change

Not an OpenAPI expert, but it seems that the rdf4j-openapi.yaml has a few errors (i.e. copy/pasting it into https://editor.swagger.io/ produces errors on the XML-examples and some issues on properties). So perhaps something to look into...

Argh, I thought I had fixed all those. Good catch, I'll tidy that up.

I need to run this up locally to see what it looks like. I'm glad you've taken the time to modernise our API documentation :)

Tip for others who may not know this: if you have hugo installed on your local machine, you can just run hugo server --disableFastRender in the site/ directory, and you can then view the site at http://localhost:1313/.

I'm going to merge this this weekend unless there are major objections. I want to merge while I have time to monitor (and if necessary fix) the website (unsure if our use of the redoc script may trigger one of our site security rules, if so I'll need to tweak that to keep the site from breaking).


Adding extra functions

Thank you for the PR! Could you please add some information what these methods are for?