Delete blob files only after destroy column handle

As Rocksdb described, DropColumnFamilies() only records the drop of the column family specified by ColumnFamilyHandle. The actual data is not deleted until the client calls delete column_family, namely DestroyColumnFamilyHandle(). We can still continue using the column family if we have outstanding ColumnFamilyHandle pointer.
now there are some problems:

after dropping column family, the blob files of it may be deleted before column family handle is destroyed.
When opening DB without some existing column families(not supported yet), the blob file of these will be regarded as obsolete files and deleted.
After destroyed column handle(not drop the column, just not need to read/write it anymore), we need to remove inner blob storage and evict related cache.

Yes, it was late yesterday so I just left some unfinished comments and test to do.  Working on that~

All works have done, PTAL @yiwu-arbug

Codecov Report

Merging #6 into master will increase coverage by 0.13%.
The diff coverage is 91.58%.

@@            Coverage Diff            @@
##           master      #6      +/-   ##
=========================================
+ Coverage   84.77%   84.9%   +0.13%
=========================================
  Files          40      40
  Lines        2029    2047      +18
=========================================
+ Hits         1720    1738      +18
  Misses        309     309

Impacted Files
Coverage Δ

src/version_edit.cc
79.66% <ø> (ø)
:arrow_up:

src/version_set.h
50% <ø> (-8.34%)
:arrow_down:

src/db_impl.h
100% <ø> (ø)
:arrow_up:

include/titan/db.h
23.33% <0%> (ø)
:arrow_up:

src/db_impl.cc
89.6% <100%> (+0.11%)
:arrow_up:

src/db_impl_gc.cc
96.36% <100%> (ø)
:arrow_up:

src/blob_storage.h
100% <100%> (ø)
:arrow_up:

src/blob_storage.cc
83.82% <100%> (+4.97%)
:arrow_up:

src/db_impl_files.cc
84.21% <76.92%> (+0.21%)
:arrow_up:

src/version_set.cc
89.94% <89.13%> (-0.11%)
:arrow_down:

... and 1 more

Continue to review full report at Codecov.

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


MqttCommunicationClosedGracefullyException every 30s

I'm using MQTTnet with EMQX (docker image emqx/emqx:v3.0-rc.3), and it works well with standard, TCP-based MQTT.
However, with MQTT over WebSockets, the connection gets disconnedted exactly every 30 seconds:
Exception of type 'MQTTnet.Exceptions.MqttCommunicationClosedGracefullyException' was thrown.
   at MQTTnet.Serializer.MqttPacketReader.ReadFixedHeaderAsync(IMqttChannel channel, Byte[] fixedHeaderBuffer, Byte[] singleByteBuffer, CancellationToken cancellationToken)
   at MQTTnet.Adapter.MqttChannelAdapter.ReceiveAsync(CancellationToken cancellationToken)
   at MQTTnet.Adapter.MqttChannelAdapter.ReceivePacketAsync(TimeSpan timeout, CancellationToken cancellationToken)
   at MQTTnet.Client.MqttClient.ReceivePacketsAsync(CancellationToken cancellationToken)

var clientCert = new X509Certificate2("my-cert.p12", String.Empty, X509KeyStorageFlags.MachineKeySet | X509KeyStorageFlags.PersistKeySet);

var tlsParams = new MqttClientOptionsBuilderTlsParameters
{
	UseTls = true,
	AllowUntrustedCertificates = true,
	IgnoreCertificateChainErrors = true,
	IgnoreCertificateRevocationErrors = true,
	Certificates = new [] { clientCert.Export(X509ContentType.SerializedCert) },
	CertificateValidationCallback = (serverCert, chain, policyErrors, clientOptions) => true
};

var options = new ManagedMqttClientOptionsBuilder()
	.WithAutoReconnectDelay(TimeSpan.FromSeconds(5))
	.WithClientOptions(new MqttClientOptionsBuilder()
		.WithCommunicationTimeout(TimeSpan.FromSeconds(60))
		.WithWebSocketServer("wss://<IP_ADDRESS>:8084/mqtt")
		.WithClientId("test-client")
		.WithCleanSession()
		.WithTls(tlsParams).Build())
	.Build();

var mqttClient = new MqttFactory().CreateManagedMqttClient();
await mqttClient.StartAsync(options);

I am in the midst of moving cheese in our data centers.  once completed, i'll enhance logging into the stack that's using mqtt and i'll see if this is indeed what i am seeing.

@cocowalla Are you using k8s or something with an nginx ingress / application gateway?  I'm wondering if it may have something to do with the inginx settings, and not an mqtt setting.

I'm just using Docker Compose on a laptop, running a single EMQ X container, with nothing in front of it.
I'm not certain, but I also seem to recall earlier having the same problem using MqttNet with a RabbitMQ broker running MQTT plugins.

Is this still relevant?


[DRAFT] partially modularize lbmap

Move the lbmap initialization from daemon/cmd to the lbmap package and move the lbmap related configuration options into lbmap.Config from DaemonConfig. This still leaves the plethora of global variables into the lbmap package.

/test


New UpsertBulk method moved to the right file.

Hey David,
This was totally my fault in the last PR that you accepted. Unfortunately, I failed to notice that I added the new UpsertBulk method to the Insert.cs files instead of Upsert.cs files. This PR fixes that and hopefully prevent confusion by you and other contributors later when searching for this method.
Sorry about that.

Hi @falahati
I'm here again, seems like I made a (careless) mistake in closing the PR as I thought it wouldn't be needed anymore. However, could you maybe rebase your PR on the v4 branch?

I will take a look into this in the coming week.


Scout stops compiling scss file to css

Hi - just in case anyone else finds this issue on a search for Atom and/or saving and compiling scss to css with Scout.
I had a working scss file that was repeatedly saving and compiling to css without issue - all worked well. This suddenly stopped and Scout would seemingly not compile the same scss file, even after saves with a different text editor, restarts, recreations; even installation of a different compiler...
I later discovered a line of whitespace characters in the scss file that were stopping the compiling - ie compiling was reaching that point and stopping. Changes above that point were reflected but not below.
No errors were thrown (by either compiler - Scout or Koala).
I discovered this by accident using Firefox inspector which showed non alphanumeric characters in the css and red dots in the scss source.
Deleting this whitespace from the scss file returned normal operations.
How they got there? - I suspect either a copy/paste from inspector or something during an Atom save (using Ftp-Remote-edit plugin).
Hope this saves someone else some head-scratching.

Closing as duplicate of #380


Suggestions on where to find a list of currency pairs for each exchange?

Hi Nils,
thanks for your work on this library.
I'm using it only for getting tickers from different exchanges ATM, and I had some hard time in finding the correct pairs and format that each exchange API requires (and many still don't work).
Do you have any suggestion on a link of where to find the correct pairs?
Moreover, it seems that any pair in Poloniex is wrong.
>>> from bitex import Poloniex
>>> resp = Poloniex().ticker("BTC_BCN")
DEBUG:bitex.api.REST.api:Initialized API Client for URI: https://poloniex.com; Will request on API version:
DEBUG:bitex.api.REST.api:Making request to: https://poloniex.com/public?command=returnTicker, kwargs: {'params': {}}
DEBUG:urllib3.connectionpool:Starting new HTTPS connection (1): poloniex.com
DEBUG:urllib3.connectionpool:https://poloniex.com:443 "GET /public?command=returnTicker HTTP/1.1" 200 None
DEBUG:bitex.api.REST.api:Made GET request made to https://poloniex.com/public?command=returnTicker, with headers {'User-Agent': 'python-requests/2.18.4', 'Accept-Encoding': 'gzip, deflate', 'Accept': '*/*', 'Connection': 'keep-alive'} and body None. Status code 200
ERROR:bitex.utils:Error while applying formatter!
Traceback (most recent call last):
  File "/home/achiusole/.venvs/crypto-env/lib/python3.6/site-packages/bitex/utils.py", line 61, in wrapper
    r.formatted = formatter(data, *args, **kwargs)
  File "/home/achiusole/.venvs/crypto-env/lib/python3.6/site-packages/bitex/formatters/poloniex.py", line 17, in ticker
    data = data[args[0]]
KeyError: <bitex.interfaces.poloniex.Poloniex object at 0x7fa38b5ffe48>
>>> resp.json().keys()
dict_keys(['BTC_BCN', 'BTC_BELA', 'BTC_BLK', 'BTC_BTCD', 'BTC_BTM', 'BTC_BTS', 'BTC_BURST', 'BTC_CLAM', 'BTC_DASH', 'BTC_DGB', 'BTC_DOGE', ..., 'BTC_OMG', 'ETH_OMG', 'BTC_GAS', 'ETH_GAS', 'BTC_STORJ'])

I'm going to manually take the info from the json for the Poloniex ticker, because I found no other way.
And also, an optimization for the ticker could be to accept a list of pairs to ask at the same time, because e.g. the Kraken API accepts a list of pairs:
https://api.kraken.com/0/public/Ticker?pair=XBTUSD,XBTEUR
I'd like to contribute to this project: would you accept a PR to accept multiple pairs with a single ticker?
Alberto

Hey @bebosudo ,
Thanks for contributing!
The formatting issue has been handled in the current dev branch - it introduces a formatter class which takes care of formatting a currency pair correctly depending on the exchange. Have a look at that! You can install it via pip install --pre bitex. Just be mindful that some things still have some hiccups.
Finding the current currency pairs available is exchange dependent - some offer a pair or symbols endpoint, others list them in the docs or don't list them at all. I've found that most exchanges offer at least one endpoint though that allows parsing of all pairs from its content.  Again, the dev branch tries to fetch the latest list of pairs upon instantiation of each interface, so have a look at the _get_supported_pairs() method to get an idea.
As for the error you're seeing - that's a bug. The formatter appears to be outdated. I'll see if I can find time to supply a fix for this (or, if you like, you're welcome to submit a PR for that as well).
Regarding your last question - the general behavior is that a single pair is needed. Any deviations from this should be handled via a keyword argument (which is why all standardized methods support **kwargs), since it's exchange specific.
Since the number of exchanges supporting multiple tickers is relatively low, I wouldn't make changing the signature a priority - however, if you feel like this is something important, you could add *args to the signature, which would then support this behaviour. Just keep in mind to update the signature across all interfaces and the base interface class.
Cheers,
Nils

Thanks for your response Nils.
I'm trying out the dev release, 2.0.0b4, and the first thing I noticed is that there are no more the Gemini and GDAX objects inside the main package (RockTradingLtd has been changed to TheRockTrading, while the REST interface is called RockTradingREST ... maybe adding 'The' in front of the REST interface could make things clearer?).
I tried the example in the README of the dev branch, and I submitted PR #124 which fixes the indentation in the example, and provides a more useful example, with a ticker on BitStamp.
While trying a similar pair on Poloniex, I found out something strange:
from bitex.pairs import PairFormatter

class BTCBCH(PairFormatter):
    def __init__(self):
        super(self.__class__, self).__init__(base='BTC', quote='BCH')

btcbch = BTCBCH()

>>> btcbch.format_for("Poloniex")
'BTC_BCH'

shouldn't it be 'BCH_BTC' ?
Moreover, Poloniex returns anyway the whole market tickers in its json response (don't think this can be considered a "bug" on bitex's side, anyway):
>>> from bitex import Poloniex
>>> resp = Poloniex().ticker(btcbch)
>>> pprint(resp.json())
{'BTC_AMP': {'baseVolume': '11.66534641',
             'high24hr': '0.00001906',
             'highestBid': '0.00001835',
             'id': 160,
             'isFrozen': '0',
             'last': '0.00001835',
             'low24hr': '0.00001700',
             'lowestAsk': '0.00001837',
             'percentChange': '-0.02028830',
             'quoteVolume': '639288.56424249'},
 'BTC_ARDR': {'baseVolume': '119.12828768',
              'high24hr': '0.00005422',
              'highestBid': '0.00004633',
              'id': 177,
              'isFrozen': '0',
              'last': '0.00004634',
              'low24hr': '0.00004546',
              'lowestAsk': '0.00004634',
              'percentChange': '-0.09509861',
              'quoteVolume': '2412654.39035808'},
...

I'm missing the "standard" and uniform formatted method, and I've seen issue #107. I would like to contribute to it (I'll write there).
Talking about the topic of this issue, when querying the supported pairs via bitex 2.0, I get strange results for Poloniex and Bitfinex.
>>> from bitex import Poloniex
>>> Poloniex().supported_pairs
~/.venvs/crypto-env/lib/python3.6/site-packages/bitex/api/base.py:54: IncompleteCredentialsWarning: Incomplete Credentials were given - authenti
cation may not work!
  IncompleteCredentialsWarning)
['BTC_ETH']

why only one BTC_ETH? may it be because I don't have authentication? but why only that specific pair?
And with Bitfinex:
>>> from bitex import Bitfinex
>>> Bitfinex().supported_pairs
Traceback (most recent call last):
  File "<input>", line 1, in <module>
    Bitfinex().supported_pairs
  File "~/.venvs/crypto-env/lib/python3.6/site-packages/bitex/interface/bitfinex.py", line 45, in __init__
    super(Bitfinex, self).__init__('Bitfinex', BitfinexREST(**api_kwargs))
  File "~/.venvs/crypto-env/lib/python3.6/site-packages/bitex/interface/rest.py", line 18, in __init__
    super(RESTInterface, self).__init__(name=name, rest_api=rest_api)
  File "~/.venvs/crypto-env/lib/python3.6/site-packages/bitex/interface/base.py", line 26, in __init__
    self._supported_pairs = self._get_supported_pairs()
  File "~/.venvs/crypto-env/lib/python3.6/site-packages/bitex/interface/bitfinex.py", line 58, in _get_supported_pairs
    return self.symbols().json()
  File "~/.venvs/crypto-env/lib/python3.6/site-packages/requests/models.py", line 892, in json
    return complexjson.loads(self.text, **kwargs)
  File "/usr/local/lib/python3.6/json/__init__.py", line 354, in loads
    return _default_decoder.decode(s)
  File "/usr/local/lib/python3.6/json/decoder.py", line 339, in decode
    obj, end = self.raw_decode(s, idx=_w(s, 0).end())
  File "/usr/local/lib/python3.6/json/decoder.py", line 357, in raw_decode
    raise JSONDecodeError("Expecting value", s, err.value) from None
json.decoder.JSONDecodeError: Expecting value: line 1 column 1 (char 0)

Thanks again for your work, I really want to contribute a little: what do you suggest me to start working on?

While trying a similar pair on Poloniex, I found out something strange:
from bitex.pairs import PairFormatter

class BTCBCH(PairFormatter):
   def __init__(self):
       super(self.__class__, self).__init__(base='BTC', quote='BCH')

btcbch = BTCBCH()

>>> btcbch.format_for("Poloniex")
'BTC_BCH'

shouldn't it be 'BCH_BTC' ?

No, but I can tell why this confuses you. While every other exchange understood the principle of baseand quote currency, Poloniex elected to just not do that. So while the pairs in the BTC market are quoted in BTC and their base currency is the respective alt coin, Poloniex describes them the other way around- so instead of BaseCurrency_QuoteCurrency, they write it QuoteCurrency_BaseCurrency.
You can find this to be true while looking at the ticker endpoint (https://poloniex.com/public?command=returnTicker), where you'll find BitcoinCash quoted in BTC as BTC_BCH.

I tried the example in the README of the dev branch, and I submitted PR #124 which fixes the indentation in the example, and provides a more useful example, with a ticker on BitStamp.

The reason I used imaginary currencies was because the BTCUSD pair is already implemented with the library (you can import it with from bitex.pairs import BTCUSD). But I like your example, and I'll update the README with it.

Talking about the topic of this issue, when querying the supported pairs via bitex 2.0, I get strange results for Poloniex and Bitfinex.

If you take a look at the Poloniex Interface, you'll see I was simply lazy and hard-coded the return value - this needs fixing of course ;).

I'm trying out the dev release, 2.0.0b4, and the first thing I noticed is that there are no more the Gemini and GDAX objects inside the main package [...]
You are correct. I haven't gotten around to re-implementing them, since 2.0 had a large portion of the of the old code base rewritten.

[...] (RockTradingLtd has been changed to TheRockTrading, while the REST interface is called RockTradingREST ... maybe adding 'The' in front of the REST interface could make things clearer?).
I'd say it's not super confusing, but if you feel like this is something that needs fixing, submit a PR :)

Thanks again for your work, I really want to contribute a little: what do you suggest me to start working on?
Cool! Glad to hear it. For starters, have a look at the code base - get a feeling for what each of the parts is doing. You're welcome to try and implement the changes I commented on in this post (Poloniex._get_supported_pairs(), for example) - just make sure to submit a single PR for each of them - that way it's easier to review for me  and the merges are cleaner :)

If you're not sure how to do things exactly, feel free to open an issue, or submit your changes before completing them as PR - we can use the review function to discuss solutions and approaches.
Again, thanks for contributing!

... Poloniex describes them the other way around- so instead of BaseCurrency_QuoteCurrency, they write it QuoteCurrency_BaseCurrency.

I knew this, and I still have some doubts about the Poloniex pair formatter.
>>> from bitex.pairs import BCHBTC, BTCUSD
>>> BTCUSD.format_for("Poloniex")
'BTC_USD'
>>> BCHBTC.format_for("Poloniex")
'BTC_BCH'

As you said, shouldn't the formatter return the two values swapped? Or does it happen only when a pair is known to be traded on the platform? because e.g. on poloniex they don't trade USD, hence the pair doesn't get swapped?

The reason I used imaginary currencies was because the BTCUSD pair is already implemented with the library

Sorry, didn't find any reference to the fact that there were already some pairs. I created PR #127 in order to address that in the README to new users.

Talking about the topic of this issue, when querying the supported pairs via bitex 2.0, I get strange results for Poloniex and Bitfinex.

If you take a look at the Poloniex Interface, you'll see I was simply lazy and hard-coded the return value - this needs fixing of course ;).

Fixed in PR #127 .

You're welcome to try and implement the changes I commented on in this post (Poloniex._get_supported_pairs(), for example) - just make sure to submit a single PR for each of them - that way it's easier to review for me and the merges are cleaner :)

Ops, I've just seen this part, was hidden in the quote. Hope it's not a problem. Next time I'll try to keep things separate for each exchange object.

You don't have to submit a PR per exchange but per code change type - so for example,  one PR if you change the signature for the ticker endpoint across all exchanges, one PR for updated docstrings, etc. Its important to keep commits and PRs as atomic as possible.
As for the pairs and poloniex formatter - that's why I said that you should familiarise yourself with the code base. As for the poloniex formatter specifically, have a look at its function in the pairs.py file.
You'll see it formats depending on input - if it's btc or xmr, for example, it formats the pairs differently.

You don't have to submit a PR per exchange but per code change type - so for example, one PR if you change the signature for the ticker endpoint across all exchanges, one PR for updated docstrings, etc. Its important to keep commits and PRs as atomic as possible.

Got it, thanks (I'm almost new to contributing to open source projects).

As for the pairs and poloniex formatter - that's why I said that you should familiarise yourself with the code base. As for the poloniex formatter specifically, have a look at its function in the pairs.py file.

Thanks, I thought there should have been some magic behind that behavior ;-)
I'm going to close this issue, because I think further comments should go in the discussion of PR #127 .


Update temporal-table-usage-scenarios.md

you can't have two constraints with same name in same database/schema..The article seems to imply, you can do that and people are asking questions regarding this(https://stackoverflow.com/questions/45433452/temporal-table-constraints/45433622#45433622)
changed constraints for location table

This seems like a small (but important) contribution, so no Contribution License Agreement is required at this point. We will now review your pull request.
Thanks,
Microsoft Pull Request Bot

pinged the author for a quick review.

#sign-off
@sateeshmachineni thank you
@craigg-msft thank you

thank you both, merging.


Add search feature

Search for a file and display it.

Consolidate search functions (contains/does not contain/prefix/etc)
Provide image view and textual view. Switchable?

v1.1.0 [9045eee] - Prefix Search


Develop

Hello @EdinsonRequena! Thanks for opening this PR. We checked the lines you've touched for PEP 8 issues, and found:

In the file algoritmos-y-poo-python/modulo_III/clase04-ordenamiento-por-inserccion.py:

Line 11:1: E302 expected 2 blank lines, found 1
Line 17:80: E501 line too long (80 > 79 characters)
Line 29:30: E231 missing whitespace after ','
Line 34:3: E271 multiple spaces after keyword

In the file algoritmos-y-poo-python/modulo_III/complementos/05-ordenamiento-por-insercion-mi-aporte.py:

Line 14:80: E501 line too long (105 > 79 characters)
Line 16:80: E501 line too long (130 > 79 characters)
Line 17:80: E501 line too long (126 > 79 characters)
Line 21:80: E501 line too long (86 > 79 characters)
Line 22:80: E501 line too long (100 > 79 characters)
Line 23:80: E501 line too long (85 > 79 characters)
Line 25:80: E501 line too long (107 > 79 characters)
Line 26:80: E501 line too long (91 > 79 characters)
Line 29:80: E501 line too long (86 > 79 characters)
Line 59:1: E305 expected 2 blank lines after class or function definition, found 1
Line 59:3: E271 multiple spaces after keyword


[TACHYON-576] [TACHYON-577] remove redundant modifier private in enum constructor

[TACHYON-577] Remove redundant modifier "private" in tachyon.client.ReadType constructor
[TACHYON-576] Remove redundant modifier "private" in tachyon.client.WriteType constructor

Can one of the admins verify this patch?

Jenkins, add to whitelist.

Jenkins, test this please.

Thanks!


[AutoPR eventgrid/resource-manager] EventgridL fixed package namespace, added java readme file

Created to sync https://github.com/Azure/azure-rest-api-specs/pull/6215

This PR has been merged into https://github.com/Azure/azure-sdk-for-js/pull/3447


Support multiple response body content types

This contains two parts:

Supporting multiple response content types
Synthesizing an Input parameter that's a type-safe variant of the "accept" header
The synthesized accept enum should only contain supported content types that we actually generate Output cases for

This way, the server implementer, conforming their type to APIProtocol, will have a clear signal from the user about which content type they want.
Things to keep in mind:

Client might not care
Client might specify */*
Client might specify application/*
Client might specify multiple, and use priorities (the q parameter)

The synthesized enum might not be concrete types, but some representation like:
enum Accept {
  case any
  case anySubtype(String)

  enum SupportedContentTypes {
    case json
    case yaml
  }

  case concrete(SupportedContentTypes)

  // The order here is significant, sorted from most to least desired, according to the rules in https://developer.mozilla.org/en-US/docs/Web/HTTP/Headers/Accept
  case multiple([SupportedContentTypes]
}

Added the design label, as this task will require a proposal, as it introduces a new paradigm - the Accept header hinting at the expected response Content-Type, where there isn't a single obvious solution, so should be discussed first.

It's also possible we could split this into two issues:

supporting multiple response body content types
providing the Accept header to the server to help it make a decision about which response content type to use

providing the Accept header to the server to help it make a decision about which response content type to use

IMO this has to be part of the solution as it's the conventional way of telling the server what you want when it can provide different response types.

For sure, was just trying to break out the work that's non-controversial (supporting multiple content types) from the thing that needs design (how we provide the value for the Accept header to the adopter on the server). Not saying we would make a release with a partial implementation, just that these things can land in separate PRs, for easier review.

Now released in 0.1.7 under the multipleContentTypes feature flag.


[full-ci] update reva to latest edge

Description
Update Reva to get some fixes.
Related Issue

Fixes <issue_link>

Motivation and Context

How Has This Been Tested?

test environment:
test case 1:
test case 2:
...

Screenshots (if appropriate):
Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] Technical debt
[ ] Tests only (no source changes)

Checklist:

[ ] Code changes
[ ] Unit tests added
[ ] Acceptance tests added
[ ] Documentation ticket raised:

https://github.com/owncloud/ocis/tree/update-reva
This branch is 3 commits ahead, 10 commits behind master.
https://drone.owncloud.com/owncloud/ocis/14307/42/7
runsh: Total unexpected failed scenarios throughout the test run:
apiSpaces/resharing.feature:40
apiSpaces/resharing.feature:41
apiSpaces/resharing.feature:42
apiSpaces/resharing.feature:43

Maybe rebase first, and see what CI thinks?


[Debugger] Adding a Watch expression sends 'evaluate' command twice

VSCode Version:

Version 1.9.1
Commit f9d0c687ff2ea7aabd85fb9a43129117c0ecf519
Date 2017-02-09T00:26:45.394Z
Shell 1.4.6
Renderer 53.0.2785.143
Node 6.5.0

OS Version:
Win 10

Steps to Reproduce:
I am using the C# extension to reproduce this, but it should be visible from any debug adapter.

Use the C# Extension to debug a .NET Core console app with the following code:

using System;

namespace ConsoleApplication
{
    public class Program
    {
        public static void Main(string[] args)
        {
            int i=1;
            i++;
        } // BP Here
    }
}

Turn on debugger logging by adding the following block to launch.json launch configuration:

            "logging": {
                "traceResponse": true
            }

Set a breakpoint on the line indicated and start debugging
View the debug console to see tracing output
Evaluate i++ in the Watch area

Expected: i++ shows 2 in the watch expression (it is post decrement so it should return 2)
Actual: i++ shows 3 in the watch expression, because it is being evaluated twice
This is also shown in the debugger log output:
C evaluate: {"expression":"i++","frameId":1000,"context":"watch"}
 R: {"success":true,"message":null,"request_seq":12,"command":"evaluate","body":{"result":"2","variablesReference":0,"type":"int"},"running":false,"refs":null,"seq":0,"type":"response"}
C evaluate: {"expression":"i++","frameId":1000,"context":"watch"}
 R: {"success":true,"message":null,"request_seq":13,"command":"evaluate","body":{"result":"3","variablesReference":0,"type":"int"},"running":false,"refs":null,"seq":0,"type":"response"}
C variables: {"variablesReference":1000}
 R: {"success":true,"message":null,"request_seq":14,"command":"variables","body":{"variables":[{"name":"args [string[]]","value":"{string[0]}","type":"string[]","variablesReference":0,"evaluateName":"args"},{"name":"i [int]","value":"2","type":"int","variablesReference":0,"evaluateName":"i"}]},"running":false,"refs":null,"seq":0,"type":"response"}

@weinand @isidorn

Yes, we should only do an evaluate on 'stop' and replace the one on 'continue' by some fixed status message.

I want to be clear that I am not talking about evaluate being sent on stop or continue. If you are already stopped and type an expression into the 'watch' area, evaluate is sent twice immediately.

I pushed a change that we only allow evaluation when stopped.
Unless it is a repl, in that case we allow evaluation all the time.
When I say do not allow I mean we show a fixed status message
fyi @roblourens

Losing global watches is a little disappointing since other devtools support it, but I don't think there's a really compelling use case. Maybe in some runtime with global data, but since watches aren't actually "watching" the value as it changes (while not stepping), it might just be misleading.
If we do end up reenabling it, we need to be careful not to bring up #19841 again. I'm working around a V8 bug.

https://github.com/Microsoft/vscode/commit/ed7a973363ca03a783ccd88326d53c9275f4753c does not fix the issue that I am reporting. Please reopen this issue. I tested against the latest insiders build which contains the proposed fix:
Version 1.11.0-insider
Commit 0bc62d1679fbb59a4bd8f62203743408893ba4ee
Date 2017-03-02T13:51:59.532Z
Shell 1.4.6
Renderer 53.0.2785.143
Node 6.5.0

The issue I am seeing has nothing to do with repl or continuing the process.
My repro steps are:

Set a breakpoint in a source file.
Start debugging and run to breakpoint.
While stopped, add an expression to the Watch viewlet

Expected: The evaluate command is sent once with the expression typed into the watch viewlet
Actual: The evaluate command is immediately sent TWICE.
Log from our debug adapter:
<- (E) {"seq":24,"type":"event","event":"stopped","body":{"reason":"breakpoint","threadId":47424,"allThreadsStopped":true,"source":{"name":"Program.cs","path":"c:\\Users\\chuckr\\Source\\scratch\\Program.cs"},"line":10,"column":13}}
-> (C) {"command":"threads","type":"request","seq":9}
<- (R) {"seq":25,"type":"response","request_seq":9,"success":true,"command":"threads","message":"","body":{"threads":[{"id":47424,"name":"Main Thread"}]}}
-> (C) {"command":"stackTrace","arguments":{"threadId":47424,"startFrame":0,"levels":20},"type":"request","seq":10}
<- (R) {"seq":26,"type":"response","request_seq":10,"success":true,"command":"stackTrace","message":"","body":{"stackFrames":[{"id":1000,"name":"scratch.Program.Main(string[] args)","source":{"name":"Program.cs","path":"c:\\Users\\chuckr\\Source\\scratch\\Program.cs","sourceReference":0,"checksums":[{"algorithm":"SHA1","checksum":"5f4b3e494c8f26dbc82244e06dbe6f7baafd41d4"}]},"line":10,"column":13,"endLine":10,"endColumn":17,"moduleId":"{7AC61516-B686-4237-B297-5B3D381927E3}"},{"id":1001,"name":"[External Code]","line":0,"column":0}],"totalFrames":2}}
-> (C) {"command":"scopes","arguments":{"frameId":1000},"type":"request","seq":11}
<- (R) {"seq":27,"type":"response","request_seq":11,"success":true,"command":"scopes","message":"","body":{"scopes":[{"name":"Locals","variablesReference":1000,"expensive":false}]}}
-> (C) {"command":"variables","arguments":{"variablesReference":1000},"type":"request","seq":12}
<- (R) {"seq":28,"type":"response","request_seq":12,"success":true,"command":"variables","message":"","body":{"variables":[{"name":"args [string[]]","value":"{string[0]}","type":"string[]","evaluateName":"args","variablesReference":0},{"name":"i [int]","value":"0","type":"int","evaluateName":"i","variablesReference":0}]}}

At this point we are stopped and I type 'i++' in the watch viewlet. See how two 'evaluate; commands are sent followed by a 'variables' command. This is incorrect behavior.

-> (C) {"command":"evaluate","arguments":{"expression":"i++","frameId":1000,"context":"watch"},"type":"request","seq":13}
<- (R) {"seq":30,"type":"response","request_seq":13,"success":true,"command":"evaluate","message":"","body":{"result":"0","variablesReference":0}}
-> (C) {"command":"evaluate","arguments":{"expression":"i++","frameId":1000,"context":"watch"},"type":"request","seq":14}
-> (C) {"command":"variables","arguments":{"variablesReference":1000},"type":"request","seq":15}
<- (R) {"seq":32,"type":"response","request_seq":14,"success":true,"command":"evaluate","message":"","body":{"result":"1","variablesReference":0}}
<- (R) {"seq":33,"type":"response","request_seq":15,"success":true,"command":"variables","message":"","body":{"variables":[{"name":"args [string[]]","value":"{string[0]}","type":"string[]","evaluateName":"args","variablesReference":0},{"name":"i [int]","value":"0","type":"int","evaluateName":"i","variablesReference":0}]}}

@isidorn I can reproduce.
Use a node program var i = 0 and add a watch console.log(i++).
On the debug console I get this:
0
1

@chuckries nice catch


[MDS] add workspace tests with multiple data source enabled

Description
Add dashboard assistant tests with multiple data source enabled
Issues Resolved
[List any issues this PR will resolve]
Check List

[ ] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

looks like the test specs are same with none MDS, can we find out a way to share those test case? so that we only maintain one place

For reusing existing test cases, we can referring this PR.


After the project is packaged and deployed to the production environment, the physical path can be viewed in the browser, source code/source section. How to avoid the physical path in the JS file???

After the project is packaged and deployed to the production environment, the physical path can be viewed in the browser, source code/source section. How to avoid the physical path in the JS file???

What branch are you building from, and how are you building the UI?
Would you be able to expand on why you need to build it locally?

No additional information, closing.


incorrect render for c#

can not render c#, only csharp
~~~ c#
this is c# code
~~~

this is ok.
~~~ csharp
this is c# code
~~~

@litefeel #525 should fix this such that Parsedown is compliant with CommonMark on this front.


Bug variable parameters

When we add public variable parameters, we can see them on disguise but when we change them there is no effect. Same blueprints was working on previous versions;
We tried deleting the json files and creating them again, but it didn't work.
Version: RS2.0-UE5.4-v1.1

Hi, which parameter/s are you trying to use.
Can you please try using my 5.4 clean project:  found in the Renderstream Resources page and let me know if the rotation variable works for you. Could you also send an email to<EMAIL_ADDRESS>with either a screenshot of the Level Blueprint that shows the variable or copy and paste the Level Blueprint to BlueprintUE.com and share a link for us to look at and we can continue to help resolve the issue.

We tried your project and variable works in yours. We also tried several different variables and they also worked. However, we still can't change variables that is related with the camera. Here is our focus distance blueprint which works in 5.1 smoothly but doesn't have an effect on 5.4 https://blueprintue.com/blueprint/_kf1dfun/

Thank you for confirming with my project, for the focus settings for CineCameras in Unreal 5.4 I have also noticed this and have messaged Epic (currently off for Summer) and will reply to this post when I have a reply and we will include information on the Disguise Help site for Renderstream.


Improved duplicate email sign-up error handling

Update from @joshmorel:
This error occurs because the email is already associated with a email/password or prior social auth sign-up using the same email.
This should not be too commonly encountered by normal users so not soft launch critical but a better error reporting would be ideal. E.g. check if email exists at attempted sign-up & warn them before they go through whole create user process.
Eventually we aim to implement account linking #781
Original:
Please note that my userid is my email<EMAIL_ADDRESS>Following error is displayed when Sign in/sign up using 'Linkedin'
Create user failed, reason: Validation failed: email: Error, expected email to be unique. Value<EMAIL_ADDRESS>Please note my userid from Google with same email is working.
PFA the screen captured with  the steps.
Screen Recording 2020-06-21 at 7.39.29 PM.zip

I logged in with my LinkedIn account on staging, and it seems to also be working fine. I am not sure if it has to do with your linkedin settings? did you authorize when it asked you @rabbi08 ? @joshmorel @robinv85  not sure if it is happening to all users, but for my end everything works fine..

Hi @rabbi08 - this issue comes from a requirement that email must be unique - if you add a google account that uses<EMAIL_ADDRESS>and a linkedin account that uses<EMAIL_ADDRESS>you will get the error. We plan to add auth0 user account linking functionality - https://auth0.com/docs/users/concepts/overview-user-account-linking - for this case but we're not considering it launch critical, however, as this is something we don't expect normal users to encounter, unlike people doing QA.
We should create a new issue for the feature but in the interim we could create a friendlier message... "This email is already in use. Try signing in with email & password or the other social account you used".

Linkedin integration is working. Agree with @joshmorel, to display user friendly message.

Closing this issue and will track the #781  . Thanks


[5.1] Fix for Query Builder not clearing the select bindings before an aggregate.

Currently, before running an aggregate function, the Query Builder clears and resets the columns, but not the select bindings. This is an issue when the query contains a sub-select query with bindings, since the aggregate query will have extraneous bindings.
The use case that caused me to run into this was pagination, which uses count() internally.

@GrahamCampbell, by CS are you referring to the whitespace on 1646?
Also, is there any guide on checking the coding style for Laravel? php-cs-fixer is telling me all the docblocks are formatted wrong.

Also, is there any guide on checking the coding style for Laravel? php-cs-fixer is telling me all the docblocks are formatted wrong (even if I pass in the .php_cs config file).

Yes, our php_cs fixer file documents this. I don't know what's going on if you can't get it to work. Works fine dor me.

Turns out I had a typo in the path to the .php_cs file...
I fixed the cs issue and also added an assertion to make sure the bindings are restored after the aggregate.

Turns out I had a typo in the path to the .php_cs file...
I fixed the cs issue and also added an assertion to make sure the bindings are restored after the aggregate.

I got here from #14492 (closed) and I am running into the same problem.
$query->withCount('foo')->having('foo_count', '<', \DB::raw('bar'))->get() works.
$query->withCount('foo')->having('foo_count', '<', \DB::raw('bar'))->exists() works.
$query->withCount('foo')->having('foo_count', '<', \DB::raw('bar'))->count() throws exception.
What is the fix for that last one?

Laravel Version: 5.7.28
PHP Version: 7.1.23
Database Driver & Version: MySQL 14.14

Please open a new issue rather than commenting on a very old PR. :)


How would I write code to press a key or number of keys programmatically and generate the sound(s)?

How would I write code to press a key or number of keys programmatically and generate the sound(s)?

Hi @Shinehah-Gnolaum,
There's a few ways.

You could call the "conductor.playNote" methods directly.
You could use MIDI files (there is an example of that already in the code (midiLoad and playSequencer)
You could create sequences via code using something like the AKSequencer (https://audiokitpro.com/get-the-most-out-of-your-aksequencer/)

Good luck!


The entire "Capture video and/or sound" popup surface should be draggable

Steps to reproduce:

Press "Capture audio" button -> popup for recording is opened
Try to move it (sart swipe from Record/Close button)

Actual Result: "Record" and "Close" buttons do not react on swipe.
Expected Result: The entire pop-up surface should be movable. When user start swiping from button or from popup background - it should be movable.

Tested:

buttons are clickable
the entire "Capture video and/or sound" popup surface is draggable

After implementation "https://github.com/lwp-emelnik/musicintervals2anki/issues/131" it becomes harder to press the buttons. @lwp-ygavrilenko Check please, whether it is possible to add some pixels for active area of the buttons or something like this.


test error

May be the line 17 in test.py should delete the args.d_model.

Thanks! I've fixed it in test.py.


Add Events to DataSearch

With this pull Request, I want to provide more extensibility for the Data Search App, that we can use this app for our specific tables with subtypes, too.

Thanks for suggesting this. We have created an internal work item ('bug') and will implement this one way or another.
Stay tuned!

@BardurKnudsen could you please keep me updated on this PR as soon as you update this functionality

@JesperSchulz it is not ideal how PullRequest and issues are getting moderated on this repository.
The last time i created an issue for such small pieces on this repo.
Then the suggestion, was to create a pull Request.
Now it's the other way round and shouldn't have taken my time to create this pullrequest. --- this can very frustrating.

Did you create an issue for this request before? And there we requested you to create a PR? That's unfortunate, as this feature is brand new, and you aren't the only one having extensibility requests. Rather than accepting a handful of individual extensibility requests, Bardur has concluded to take all of them into consideration and to address them in one, unified way.
The suggested approach is to start with an issue, and if we agree it's a good issue to address, you can create a PR. In the BaseApp pilot, we're trying to enforce that. Getting a PR closed is indeed a frustrating experience, which we try to avoid. Here that didn't seem to work 🙄 Sorry about that. I'll see how we can improve in this repo!
However, I did speak to Bardur and he is of course taking your PR into consideration. It DOES still have value, as it clearly documents the extensibility points, you need. So it wasn't all a waste!

@pri-kise We have created this separate codeunit in the Data Search app, which contains a set of integration events.
codeunit 2682 "Data Search Events"
{
SingleInstance = true;
/// <summary>
/// Specifies the Field No. for the table type field. For example "Sales Header" would specify 1 = "Document Type"
/// </summary>
/// <param name="TableNo">The table no. of the table that is being searched.</param>
/// <param name="FieldNo">The field no. of the 'document type' field.</param>
[IntegrationEvent(false, false)]
procedure OnGetFieldNoForTableType(TableNo: Integer; var FieldNo: Integer)
begin
end;

/// <summary>
/// Specifies the parent table if a table is a sub-table in a header/lines construct, such as Sales Line
/// </summary>
/// <param name="SubTableNo">The table no. of the table that is being searched.</param>
/// <param name="ParentTableNo">If the the table is a sub-table, please specify the parent table here</param>
[IntegrationEvent(false, false)]
procedure OnGetParentTable(SubTableNo: Integer; var ParentTableNo: Integer)
begin
end;

/// <summary>
/// Specifies the list page no. to be used for displaying the record.
/// Only needed if the page is not the standard list/card page for the table
/// </summary>
/// <param name="TableNo">The table no. of the table that is being displayed.</param>
/// <param name="TableType">If the table as a 'Document Type'-like field, this value specifies the type as integer.</param>
/// <param name="PageNo">The page to use to display the record.</param>
[IntegrationEvent(false, false)]
procedure OnGetListPageNo(TableNo: Integer; TableType: Integer; var PageNo: Integer)
begin
end;

/// <summary>
/// Specifies the card page no. to be used for displaying the record. If no card exists, then specify the list page no.
/// Only needed if the page is not the standard list/card page for the table
/// </summary>
/// <param name="TableNo">The table no. of the table that is being displayed.</param>
/// <param name="TableType">If the table as a 'Document Type'-like field, this value specifies the type as integer.</param>
/// <param name="PageNo">The page to use to display the record.</param>
[IntegrationEvent(false, false)]
procedure OnGetCardPageNo(TableNo: Integer; TableType: Integer; var PageNo: Integer)
begin
end;

/// <summary>
/// Specifies the header RecordRef to a corresponding line RecordRef for e.g. sales line -> sales header
/// </summary>
/// <param name="LineRecRef">The line record</param>
/// <param name="HeaderRecRef">The parent of the line record</param>
[IntegrationEvent(false, false)]
procedure OnMapLineRecToHeaderRec(var LineRecRef: RecordRef; var HeaderRecRef: RecordRef)
begin
end;

/// <summary>
/// Enables adding and removing tables from the default initial setup for tables to search.
/// </summary>
/// <param name="ProfileID">Page ID for the selected role center</param>
/// <param name="ListOfTableNumbers">List of integer. Already filled with standard tables.</param>
[IntegrationEvent(false, false)]
procedure OnAfterGetRolecCenterTableList(RoleCenterID: Integer; var ListOfTableNumbers: List of [Integer])
begin
end;

/// <summary>
/// Specifies whether a field is excluded from the default fields in a table, based on its table relation.
/// Typically we would exclude fields from search that have relations to posting groups, reason codes, etc.
/// </summary>
/// <param name="RelatedTableNo">The table no. that the field has relation to.</param>
/// <param name="IsExcluded">true/false whether the field should be excluded from the default setup.</param>
[IntegrationEvent(false, false)]
procedure OnGetExcludedRelatedTableField(RelatedTableNo: Integer; var IsExcluded: Boolean)
begin
end;

}
And a test-example that you would add to your solution:
codeunit 139508 "Test Data Search On Archives"
{
EventSubscriberInstance = Manual;
[EventSubscriber(ObjectType::Codeunit, Codeunit::"Data Search Events", 'OnGetFieldNoForTableType', '', true, true)]
local procedure OnGetFieldNoForTableType(TableNo: Integer; var FieldNo: Integer)
begin
    case TableNo of
        Database::"Sales Header Archive", Database::"Sales Line Archive":
            FieldNo := 1;
    end;
end;

[EventSubscriber(ObjectType::Codeunit, Codeunit::"Data Search Events", 'OnGetParentTable', '', true, true)]
local procedure OnGetParentTable(SubTableNo: Integer; var ParentTableNo: Integer)
begin
    case SubTableNo of
        Database::"Sales Line Archive":
            ParentTableNo := Database::"Sales Header Archive";
    end;
end;

[EventSubscriber(ObjectType::Codeunit, Codeunit::"Data Search Events", 'OnGetListPageNo', '', true, true)]
local procedure OnGetListPageNo(TableNo: Integer; TableType: Integer; var PageNo: Integer)
var
    SalesDocumentType: Enum "Sales Document Type";
begin
    case TableNo of
        Database::"Sales Header Archive":
            case TableType of
                SalesDocumentType::Order.AsInteger():
                    PageNo := Page::"Sales Order Archives";
                SalesDocumentType::"Blanket Order".AsInteger():
                    PageNo := Page::"Blanket Sales Order Archives";
                SalesDocumentType::Quote.AsInteger():
                    PageNo := Page::"Sales Quote Archives";
                else
                    PageNo := Page::"Sales List Archive";
            end;
    end;
end;

[EventSubscriber(ObjectType::Codeunit, Codeunit::"Data Search Events", 'OnGetCardPageNo', '', true, true)]
local procedure OnGetCardPageNo(TableNo: Integer; TableType: Integer; var PageNo: Integer)
var
    SalesDocumentType: Enum "Sales Document Type";
begin
    case TableNo of
        Database::"Sales Header Archive":
            case TableType of
                SalesDocumentType::Order.AsInteger():
                    PageNo := Page::"Sales Order Archive";
                SalesDocumentType::"Blanket Order".AsInteger():
                    PageNo := Page::"Blanket Sales Order Archive";
                SalesDocumentType::Quote.AsInteger():
                    PageNo := Page::"Sales Quote Archive";
            end;
    end;
end;

[EventSubscriber(ObjectType::Codeunit, Codeunit::"Data Search Events", 'OnMapLineRecToHeaderRec', '', true, true)]
local procedure OnMapLineRecToHeaderRec(var LineRecRef: RecordRef; var HeaderRecRef: RecordRef)
var
    SalesHeaderArchive: Record "Sales Header Archive";
    SalesLineArchive: Record "Sales Line Archive";
begin
    if LineRecRef.Number <> Database::"Sales Line Archive" then
        exit;
    LineRecRef.SetTable(SalesLineArchive);
    SalesHeaderArchive.Get(SalesLineArchive."Document Type", SalesLineArchive."Document No.", SalesLineArchive."Doc. No. Occurrence", SalesLineArchive."Version No.");
    HeaderRecRef.GetTable(SalesHeaderArchive);
end;

[EventSubscriber(ObjectType::Codeunit, Codeunit::"Data Search Events", 'OnAfterGetRolecCenterTableList', '', true, true)]
local procedure OnAfterGetRolecCenterTableList(RoleCenterID: Integer; var ListOfTableNumbers: List of [Integer])
begin
    if not ListOfTableNumbers.Contains(Database::"Sales Header Archive") then
        ListOfTableNumbers.Add(Database::"Sales Header Archive");
    if not ListOfTableNumbers.Contains(Database::"Sales Line Archive") then
        ListOfTableNumbers.Add(Database::"Sales Line Archive");
end;

[EventSubscriber(ObjectType::Codeunit, Codeunit::"Data Search Events", 'OnGetExcludedRelatedTableField', '', true, true)]
local procedure OnGetExcludedRelatedTableField(RelatedTableNo: Integer; var IsExcluded: Boolean)
begin
    IsExcluded := IsExcluded or (RelatedTableNo in [Database::"Customer Price Group", Database::"Work Type", Database::"Transport Method"]);
end;

}

@BardurKnudsen  I haven't checked the preview, but shouldn't these events be internal?
Otherwise - lgtm.


ResultAsync.combine giving TypeError in the latest neverthrow version

https://github.com/radixdlt/rola/blob/0afb94a56876067a02f1edaf6c8ad94dee8bda30/examples/typescript-full-stack/apps/server/src/index.ts#L67

@abdulhaseeb13mar can you please try 2.0.2-dev.1 version and see if it helped with your issue?


Feat(scoped variables): file view

Description
Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change.
This PR introduces layout for variables when user already has uploaded a file, it has following functionalities:

User can re-upload a file and it will show the progress bar, then upload-editor-view and then the same view.
User can select to download file or template through dropdown.
User should be able to toggle tabs.

Snapshots:

Dep: https://github.com/devtron-labs/dashboard/pull/1222
Fixes # (issue)
Type of change

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration

[ ] Test A
[ ] Test B

Checklist:

[ ] The title of the PR states what changed and the related issues number (used for the release note).
[ ] Does this PR require documentation updates?
[ ] I've updated documentation as required by this PR.
[ ] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas

Please reuse ic-upload in place of ic-upload-blue

Done replaced ic-upload with ic-upload-blue


Roll Engine from 9ce1e5c5c7e7 to 9f4468602cb3 (48 revisions)

Roll Engine from 9ce1e5c5c7e7 to 9f4468602cb3 (48 revisions)
https://github.com/flutter/engine/compare/9ce1e5c5c7e7...9f4468602cb3
2020-05-21<EMAIL_ADDRESS>null-annotate semantics.dart (flutter/engine#18553)
2020-05-21<EMAIL_ADDRESS>Roll Fuchsia Linux SDK from ciqRH... to Kvwlr... (flutter/engine#18565)
2020-05-21<EMAIL_ADDRESS>Roll Skia from b37105ea6cca to 1e63279156d6 (9 revisions) (flutter/engine#18564)
2020-05-21<EMAIL_ADDRESS>Implement WriteAtomically using write/fsync on all platforms, and enable (flutter/engine#18320)
2020-05-21<EMAIL_ADDRESS>Roll Dart SDK from ecce58c1e354 to 396b5fb7a97d (2 revisions) (flutter/engine#18560)
2020-05-21<EMAIL_ADDRESS>Roll Fuchsia Mac SDK from RNByJ... to _zNmv... (flutter/engine#18558)
2020-05-21<EMAIL_ADDRESS>[profiling] Memory Profiling support for iOS (flutter/engine#18516)
2020-05-21<EMAIL_ADDRESS>Roll Skia from 22636205ce92 to b37105ea6cca (2 revisions) (flutter/engine#18557)
2020-05-21<EMAIL_ADDRESS>Roll Dart SDK from d551980ac9fc to ecce58c1e354 (2 revisions) (flutter/engine#18556)
2020-05-21<EMAIL_ADDRESS>Roll Dart SDK from bf7e9d13d730 to d551980ac9fc (6 revisions) (flutter/engine#18551)
2020-05-21<EMAIL_ADDRESS>Roll Skia from 4c0578632217 to 22636205ce92 (4 revisions) (flutter/engine#18550)
2020-05-21<EMAIL_ADDRESS>Roll Fuchsia Linux SDK from ThMeW... to ciqRH... (flutter/engine#18549)
2020-05-21<EMAIL_ADDRESS>Roll Skia from 67ff541ac116 to 4c0578632217 (1 revision) (flutter/engine#18547)
2020-05-21<EMAIL_ADDRESS>Let run_tests.py just stream output (flutter/engine#18534)
2020-05-21<EMAIL_ADDRESS>Add FlKeyEventPlugin (flutter/engine#18313)
2020-05-21<EMAIL_ADDRESS>Add tests for StandardMethodCodec (flutter/engine#18521)
2020-05-21<EMAIL_ADDRESS>Roll Skia from f83baf230c69 to 67ff541ac116 (1 revision) (flutter/engine#18545)
2020-05-21<EMAIL_ADDRESS>Roll Fuchsia Mac SDK from 3o9aQ... to RNByJ... (flutter/engine#18543)
2020-05-21<EMAIL_ADDRESS>fuchsia: Fix runtime_tests and shell_tests (flutter/engine#18492)
2020-05-21<EMAIL_ADDRESS>Roll Skia from d2dc8ddcdf5e to f83baf230c69 (2 revisions) (flutter/engine#18539)
2020-05-21<EMAIL_ADDRESS>Roll Dart SDK from 7aa8656d1dd8 to bf7e9d13d730 (21 revisions) (flutter/engine#18538)
2020-05-21<EMAIL_ADDRESS>Add FlMethodChannel, FlMethodCodec, FlStandardMethodCodec and FlJsonMethodCodec (flutter/engine#18220)
2020-05-20<EMAIL_ADDRESS>[web] Fix arc rendering when it starts a new sub path. (flutter/engine#18535)
2020-05-20<EMAIL_ADDRESS>Send platformResolvedLocale from iOS embedder (flutter/engine#18519)
2020-05-20<EMAIL_ADDRESS>System mouse cursor: macOS (flutter/engine#18131)
2020-05-20<EMAIL_ADDRESS>Roll Skia from 656ee7bc0fd2 to d2dc8ddcdf5e (2 revisions) (flutter/engine#18537)
2020-05-20<EMAIL_ADDRESS>Make FlBasicMessageChannelResponseHandle a GObject to detect unresponded and double responded messages. (flutter/engine#18514)
2020-05-20<EMAIL_ADDRESS>System mouse cursor: Web (flutter/engine#17718)
2020-05-20<EMAIL_ADDRESS>[web] Reuse ImageElement(s) across frames (flutter/engine#18437)
2020-05-20<EMAIL_ADDRESS>Roll Skia from 3e84312c0f4d to 656ee7bc0fd2 (8 revisions) (flutter/engine#18536)
2020-05-20<EMAIL_ADDRESS>Roll Dart SDK from d2ee73b2513c to 7aa8656d1dd8 (12 revisions) (flutter/engine#18533)
2020-05-20<EMAIL_ADDRESS>Fixes Semantic Object memory leak cause of wrong release call (flutter/engine#18107)
2020-05-20<EMAIL_ADDRESS>Set caches directory on Fuchsia (flutter/engine#18321)
2020-05-20<EMAIL_ADDRESS>[fuchsia] Remove SceneDisplayLag event on Fuchsia (flutter/engine#18515)
2020-05-20<EMAIL_ADDRESS>Roll Skia from 682fca92996b to 3e84312c0f4d (3 revisions) (flutter/engine#18530)
2020-05-20<EMAIL_ADDRESS>Move robolectric tests to sdk 28 (flutter/engine#18525)
2020-05-20<EMAIL_ADDRESS>Roll Skia from 9d52ac1573f1 to 682fca92996b (1 revision) (flutter/engine#18529)
2020-05-20<EMAIL_ADDRESS>Roll Fuchsia Mac SDK from fCHnB... to 3o9aQ... (flutter/engine#18528)
2020-05-20<EMAIL_ADDRESS>Roll Dart SDK from 9807b83d5945 to d2ee73b2513c (5 revisions) (flutter/engine#18526)
2020-05-20<EMAIL_ADDRESS>Roll Fuchsia Linux SDK from 73IKT... to ThMeW... (flutter/engine#18527)
2020-05-20<EMAIL_ADDRESS>Roll Skia from 34eee0d9fc85 to 9d52ac1573f1 (3 revisions) (flutter/engine#18524)
2020-05-20<EMAIL_ADDRESS>Roll Dart SDK from 3a764b7ae4d9 to 9807b83d5945 (3 revisions) (flutter/engine#18523)
2020-05-20<EMAIL_ADDRESS>Roll Skia from 3bcb7bb01b82 to 34eee0d9fc85 (1 revision) (flutter/engine#18522)
2020-05-20<EMAIL_ADDRESS>Fix iOS platform view not deallocated (flutter/engine#18164)
2020-05-20<EMAIL_ADDRESS>Roll Skia from 5e6d789ce47c to 3bcb7bb01b82 (1 revision) (flutter/engine#18518)
2020-05-20<EMAIL_ADDRESS>Remove currentLocale prepend on iOS (flutter/engine#18379)
...

Commit queue failed; closing this roll.


Chantodo's Resolve set damage element

Chantodo's Resolve: Every second while in Archon form you expel a Wave of Destruction, dealing 350% weapon damage to enemies within 30 yards
Wave of Destruction's damage element is determined by your gear: whatever the highest Elemental% damage type you have, that's the damage type it deals. However, the amount of damage it deals is increased by whatever Element type matches your Archon rune.
This means that if you only have elemental% damage on two items and want your Wave of Destruction to do the maximum possible Fire damage, you want 20% fire on one item, and 19% arcane on another.
More info: http://www.diablofans.com/forums/diablo-iii-class-forums/wizard-the-ancient-repositories/127024-2-3-vyrs-chantodos-set-mechanics-faqs
This is currently handled incorrectly by the simulator, and is immediately obvious if you're using Chantodo's Resolve with Firebirds, as the Wave of Destruction will never do Fire damage in the simulation no matter how much Fire% you have on your gear without you selecting the Fire Archon rune. If you have no other sources of Fire damage you'll be sitting at zero Firebird stacks the entire simulation.

Duplicate of the last post in #47.

Should be fixed now.


Create file reference functionality such as holding reference iRODS files

To serve two HydroShare-based use cases at RENCI, after some discussions with @mjstealey and @cbcunc, I decided it is better to just create a new iRODS reference resource type in HydroShare code base as the basis which can be directly leveraged by those two use cases at RENCI as well as by HydroShare community. In both use cases at RENCI, their data are already in their own iRODS stores, and they want to use HydroShare to reference those data as the front end user interface without moving data into HydroShare iRODS store/zone. This issue will involve creating a new generic iRODS reference resource type that references data stored in other iRODS servers which HydroShare  iRODS proxy user does not have direct access to. All AVU metadata stored on IRODS servers associated with the referenced data will populate as extended key-value metadata on Django side for the new iRODS reference resource type, and users can add and edit metadata as needed, but users cannot download the referenced files. The referenced file entries cannot be directly edited to protect the reference path integrity, but the file entries can be deleted or new file entries can be added via iRODS file browser. The downloaded bag will include metadata and resourcemap xml files and a referenced_files.txt file under data/contents that includes paths for all referenced files in this iRODS reference resource.
I also discussed specifics with @mjstealey on leveraging this new iRODS reference resource type for nwm data. We both think a NWM reference resource type could be created as a sub-class to the generic iRODS reference resource type, and can handle updating referenced nwm data path in the resource based on the sliding data refresh window.
Since one use case at RENCI that will leverage this new reference resource type is for SC17 demo which will take place on Nov 12, I will have to work on this as a priority item to make the time line. I may not get the full feature set done for this new resource type by Nov. 3 when we plan to do integration testing for SC17 demo,  but can work on this further after SC17 demo to get it ready for merging into HydroShare. We think it is a useful feature to get into HydroShare, but @dtarb let us know your thought on this.

This is a little self-serving but sound great to me since we are one of those use cases!
One question/issue since the metadata in AVUs will sync with Django metadata;
If metadata is edited on the Django side will that get pushed back to iRODS AVUs to keep both in sync?
Another edge case would be the reverse where metadata AVU's in iRODS altered after reference ingest requiring re-ingest or refresh of Django metadata.
Ignore this newbie's question if it does not make sense.

@jonc1438 Your questions make perfect sense and the answers to both questions are no since it is a reference resource that reflects the snapshot/status of the data at the creation time, and after the resource is created, HydroShare will not have the iRODS user credential to access the referenced data to do any update. However, the user can delete the old file entries, and add the new file entries via iRODS file browser from resource landing page, and this process will bring the referenced data in sync with iRODS back end again. In summary, HydroShare cannot keep things sync'ed up automatically, but provide means for users to do the sync-up as needed. This is how a generic iRODS reference resource would behave. You can definitely create a subclass to this generic iRODS reference resource to add this automatic sync-up for your specific workflow as needed. @mjstealey please add anything I have missed to clarify things as needed.

@hyi - would the content have to be stored in iRODS? For example - would this new referenced resource type be usable for referencing a file that is available using an HTTP URL for instance?
Or - are we talking about something completely different here?

@jonc1438 - What @hyi is scoping is the basis for an iRODS reference resource which can be extended to suit different use cases as needed by other developers. The initial pass will serve as a means of registering an iRODS file or collection, but may not check all the boxes that all users want.
Hong will however have encoded a template for others to build upon.
Basis

iRODS reference resource

iRODS collection/file information

Size, zone location, etc.
AVUs

Additional user added metadata

Contact information
iRODS credential info.
information about the resource not encapsulated in the reference files themselves
etc.

Assumptions

actual files will not reside in HydroShare
user is responsible for managing state of reference if underlying iRODS files change (DOI implications here)
user may not add additional static files to the resource, it is simply a reference to things in iRODS in this basic form

Other work can then be extended from this, for example:
Extends

NWM iRODS reference resource

All of iRODS reference resource +
accomodate a sliding window of time that the resource exists in iRODS

analysis_assim - 720 day window
short_range - 40 day window
etc.

Brain-I iRODS reference resource

All of iRODS reference resource +

AVU validation / re-ingest services

check AVU state based on time modified for Django vs iRODS

workflow and automation information
specific apps that operate on this resource type
etc.

I have the following comments and concerns

I am concerned about prioritizing this for use cases that have not been described.  Are there written descriptions for the use cases that we can discuss to understand the best way to solve them.
There are AVU details in the first paragraph that are stated without context.  We need the bigger picture to get a sense of why AVU’s are involved.  I would prefer that AVU’s not be involved as they seem limiting.
We should do our best to keep this simple.  If content is to be held outside of HydroShare just use files in a composite resource to describe how to access those files.  That is what a reference is – a description of how to access something outside.
If there is to be file specific metadata it should be associated with a “file type” not a resource type.  We should be using this model for all new “types” in HydroShare so as to be able to take advantage of the flexibility due to decoupling “resource” from “type” that composite resources give us.
I suggest that for a first implementation (and SC17 seems like a first implementation) just work with files in a composite resource without trying to rush an additional type into the code.  Use this working with files to gain understanding of use patterns and the requirements for type specific metadata to be added once requirements are understood.
There are many examples of things that need to be referenced from HydroShare.  A model we develop for reference resources should recognize this and to the extent possible use the same approach for all.  We already have reference HIS time series types in composite resources.  We could go to referencing external content in an iRODS server somewhere, referencing external content on a website somewhere (http server), referencing external content behind various types of API’s such as OGC web services, and I am sure others.  I think we should look for a general way for all of these could be accommodated by a file that provides information on how to access the external resource.

@dtarb - one of the purposes behind the way we encoded the science metadata and resource map for a resource is that each "file" that comprises a resource is referenced using a URI. So, whether the file exists in iRODS or not, that URI should point at the file. Technically, the system should be able to read the resource map (and/or its Django reflection) and then display a list of all of the files in the resource regardless of where they are physically located. Where I am going with this, is that I don't think we need "a file that provides information on how to access the external resource" - we already have a way to do this in our science metadata and resource map.  What we need are ways to add referenced content to a resource without uploading a file (e.g., specifying a link instead).
Apologies if I'm hijacking this issue and conflating multiple things.

@horsburgh I was following reference time series file type as the example.  If there is a better way to do it I am happy for you and others on the data model team to work it out.  My suggestion is that we still need to articulate the design and purpose for what is being done here (unless this is in documents I have not seen) and look for a simple solution that is general for quite a number of kinds of content we may want to reference.  I am worried there is too much iRODS specific AVU complexity in what seems to be proposed (but exactly what is being proposed is unclear to me).  We need better context and discussion to sort this out.

Someone should start writing a specification for a generic reference type (file/resource) in HydroShare which should help if needed writing more specific reference type like iRODS. Only then think about implementation.

@all -this  topic is an agenda item for team call on 10/11/2017

@dtarb @horsburgh Just got a chance to read through details of your comments on this issue carefully and give this some more thought. I think I agree every comment both of you made. Let me put another option on the table after giving this some more thought (sorry this issue caught you off-guard, but this idea just came about yesterday afternoon triggered by the pressing need for SC17 demo). Instead of creating a new reference resource type, to serve the two specific use cases (we can discuss more on the use cases in our team call today), we just need our ResourceFile object in Django to support a referenced file by an iRODS logical path or a URL (as brought up by @horsburgh), then on iRODS file browser in HydroShare, I can add two check boxes to allow users to check to indicate whether they want the iRODS files as reference in HydroShare rather than physically copied over to HydroShare, and whether they want AVUs to be copied over as key-value pair metadata in HydroShare. I will start out to only support the first checkbox that allow users to check to indicate they want iRODS files selected to be reference rather than the default copy-over behavior, and leave AVU checkbox later or just defer to @jonc1438's team to implement for their own customized HydroShare instance to fit their use case which can be feed back into HydroShare or xDCIShare, whichever is appropriate. The high-level list of items to be done for this option:

modifying underlying ResourceFile class in Django to support referenced file field model.
add checkbox in iRODS file browser to allow users to check to indicate they want selected iRODS files to be referenced rather than copied over.
update file browser on resource landing page to support display and deletion of the referenced files.

I can take care of the first two items and may need help for the last item from @Maurier to get it polished and work well. For the SC17 demo, I can make it work as simple as possible just for demo purpose due to pressing deadline. I think this option would be sufficient to address the use cases, without the complexity a generic reference resource type would incur from HydroShare design perspective. We can discuss further in our call today.

@horsburgh @dtarb I am working on this to create a reference file type for SC17 demo, and have question on how we should encode the referenced file URI in the resource bag. In this SC17 demo case, the reference file path is the iRODS logical file path in the format of /iRODS_zone/home/iRODS_user_name/file_path, which HydroShare would not have access to, so the referenced file cannot be downloaded. How should we encode the URI for such referenced file then?

@hyi - I may still be a little confused by this use case. I think what you are describing is creating a separate instance of HydroShare that will contain metadata-only records. There will be "resources" in that HydroShare instance, each will describe a dataset, but the files will not be stored in or accessible to the HydroShare instance and the users will not be able to access the data through that HydroShare instance. Is that right?
HydroShare's resource data model, and in particular the resource map document and the science metadata document, were designed to use RDF/XML encoding. RDF uses URIs to identify resources (using RDF's terminology a resource can be a file) and their properties. In our HydroShare case, we are using URLs as URIs - which is great because that means there is an actual object at the other end of that URL that is either the file being described or a term from a vocabulary that has a definition in a HTML document. And, because the URL serves as both a name and a locator, those URLs can be resolved to return the individual files in the resource. Using URLs as the URIs is really useful in a "linked data" context because the "links" (the URLs) point at live resources (files that can be retrieved on the Internet).
In your example, if the files are never to be made available to the users for download, then any URI that is unique would work to identify each of the files in the resource, right? In this case, the URI acts only as a name and not as a locator. So, the HydroShare instance would be capable of listing all of the files in a resource, but would not be able to retrieve them.
So - the short answer is - any URI that uniquely identifies/names a file could be used. This could be the iRODs logical file path - but keep in mind it will be encoded in the resource map and the science metadata document, so make sure it is something you want people to see.
But - that said - for this use case, will you be scripting/automatically building all of the resources from the files in the iRODS zone? This instance of HydroShare would essentially be read-only for users right?  If that's the case, I don't think this is the best example to be driving the design/implementation of a generalized reference resource for HydroShare (which I think should use URLs as URIs).

@horsburgh Thanks for the detailed thoughts. We have been discussing and refining the SC17 demo workflow since Friday, and it turns out we would most likely not need to create a reference file type in HydroShare after all. What you described above is in line with my initial thought, on HydroShare side, a resource holds metadata-only records and reference path to resource files that cannot be downloaded directly, however, the referenced files would be available in another cyberinfrastructure which would be demoed for SC17 and transferred to the on-demand provisioned appliances for users to do analysis and computation on the referenced data. However, this reference model may very likely be replaced by another model currently in discussion, so I will not continue with this reference file path at least for now. What your described above will serve as useful reference when we design a generalized reference resource for HydroShare regardless, so thanks for the detailed input on this.

@horsburgh @dtarb I have implemented a working prototype to add iRODS file reference functionalities for another project that build customized functionalities on top of HydroShare code base. Rather than creating a new resource type or file type, I ended up adding this iRODS file reference as a generally available functionality that is readily available across the board as part of iRODS file browser. Specifically, I added a checkbox as shown in the screen capture below:

When this checkbox is not checked by default, it functions as what we have now. But when it is checked, the selected iRODS files will be brought in as reference files added into whatever types of the resource they are added to. In the meantime, all iRODS reference file metadata are also brought in as extra_metadata key-value pairs in HydroShare. The following two screen captures show what the file content and extra metadata interface look like for referenced files in a resource:

Although this is specific for iRODS files, I can see it would apply to other file reference use cases discussed here. Want to keep you posted on progress with this and get your feedback as well on whether you think this is useful for HydroShare. Looping in @rayi113 @mjstealey @cbcunc  to keep them posted as well.

There is not a real use case at this point for this, so closing the issue for now.

There is not a real use case at this point for this, so closing the issue for now.


Update OAuth2.0 flow for Augur frontend

Add /login and /register routes to Flask application server and use Flask-Login with Flask-Session to securely handle user sessions.

Have worked on this substantially in the past sprint- will be building a toy example for my demonstration repo while will then be applied to 8Knot.

Implemented Miguel Grinberg's OAuth2.0 flow in a demo application:
https://github.com/JamesKunstle/easy-flask-boilerplate/tree/add-oauth
Will be repeating this for 8Knot. Lots of lessons learned from this process.

Deciding, for the time being, to not restrict session communication via Secure cookie setting.
In the future, we'll likely need to handle non-ephemeral user state (session will key into redis key/value pair that'll reference the real username of the user) but for now the data is so quick to download that we don't have to worry about that.

Authentication via OAuth works now, and I feel good about security. Now, I need to design the way that the application handles the users data.
The data in question is the user's group set. These group sets can be very large, such as one case where a group had 20k repos in it.
Right now, the groups are queried via a long callback (assuming the user is logged in) and stored in the local storage of the browser. With user authentication and sessions, we could switch to using server-side group caching. We could also design deferred collection, so sufficiently large groups aren't collected until requested by the user.
To decide how this works, I need to learn how to implement an asynchronous API in Flask so that a client-side callback in a Dash app can request user groups, which will be collected ad-hoc and stored in-browser for the time being.

Celery has good documentation on how an asynchronous workflow should be implemented. These routes should require login.
https://flask.palletsprojects.com/en/2.3.x/patterns/celery/

Followup: The redis creator discourages the use of multiple DBs in the same instance. It's so lightweight to spin up another Redis instance w/ Compose and Openshift that that's probably the right thing to do, full stop.

Discovered that #428 is required to finish this reimplementation. Finishing this issue will be the midway point in updating how user groups are handled.

closed with #477


Properly pass options to ManagedIdentityCredential

fixes https://github.com/Azure/azure-sdk-for-net/issues/34077

/cc @pharring

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


Changelog for qcodes 0.21.0

I am opening the early changelog for Qcodes 0.21.0 with some of the changes that we have for now. Feel free to update it continuously until the next release.
@jenshnielsen
@astafan8
@WilliamHPNielsen
@Akshita07
@trevormorgan

for this to build cleanly you will need to link this into the index file in the same folder

@FarBo can you fix the rst formatting.  The underline needs to be the same length as the text


Gallery demo tweaks

Remove the egg from the progress indictors demo.
Use textTheme.subhead in the TabbedComponentDemos.
Don't auto-scroll in the tabs-demo, avoid appbar issues for now.
Add a TwoLevelSubList backgroundColor parameter.
Make the bottom sheet demos look OK with the dark theme.
Make the home page category header text visible with the dark theme.

#5062
#5065
#5069
#5094
#5078
#5093


fixed to_euler function in quat.py

I have tested with all six orderings against scipy.spatial.transform.Rotation, and all showed the correct results.

Thanks Kai this looks great. I will check it and merge it soon.

Does this change the default behavior for bvh.save? In which case does this function call and the following one need updating?
https://github.com/orangeduck/Motion-Matching/blob/main/resources/train_decompressor.py#L325

Yes, you need to include the correct order to quat.to_euler(..) function.
In line 326, it should be
'rotations': np.degrees(quat.to_euler(Ygnd_rot[0].cpu().numpy(), order='zyx' )),

and in line 335,
'rotations': np.degrees(quat.to_euler(Ytil_rot, order='zyx')),

Okay I've submitted a fix for that in 37df18a


x/mobile/cmd/gomobile: CGO_CFLAGS is overwritten during go bind -target=ios

hi,
I faced an issue when I run gomobile bind with CGO_CFLAGS and CGO_LDFLAGS like
CGO_CFLAGS="-I/usr/local/include" \
CGO_LDFLAGS="-L/usr/local/lib" gomobile build -target=ios -v

but output told me that there a header file not found.

It would be nice to be able to set these flags when using gomobile. It's an odd middle-ground however: we deliberately ignore some variables like GOOS, GOARCH, CC, and CXX, but we will use the flags variables? And we will apply the same CC flags to different compilers?

I think it is better to have a flag variable to determine whether we want to inherit CFO_CFLAGS and  CGO_LDFLAGS from env. It should be caller's call to deal with different compilers.
how do you think?

With https://go-review.googlesource.com/99316 in it is now possible to generate complete bindings with the gobind tool, skip gomobile and build the JNI lbraries manually with CC=... CXX=... CGO_ENABLED=1 GOOS=android GOARCH=... go build -buildmode=c-shared.
(With pending https://go-review.googlesource.com/c/mobile/+/99875 gomobile will change to use standalone toolchains, and build exactly as above)
Does that solution help you?


Accept rtc.io CLA

I accept :)

How did that work for you? Feel ok?

Felt great.  I think if that tick's the legal boxes then that's fantastic and I don't think anyone would have a problem "signing" a CLA.

OK we should socialise it on the mailing list, I guess.

See https://groups.google.com/d/msg/rtc-io/nMbAudx8xGY/h6KnBQ_ioUAJ .


Spring function support

I have cleaned and updated Will's work from the summer. Please don't make me rebase it again.

Demonstrated to work locally with this (private) repo: https://github.com/fnproject/fn-spring-cloud-function-example - that can be updated and made public once this has merged. The reason it's not public yet is because it needs a lot of fiddling and hard-coding of paths to be able to work against a version of the runtime & api which aren't in a public repo.


[Question] Does hot reload work for aspnetcore plugins with service registration?

Hi, I has a aspnetcore plugin with service registration into the ServiceCollection, does it has any method to re-register the service when the plugin reload?

No such method or feature exists. The hot reload capability is pretty bare-bones and doesn't work with ASP.NET Core's dependency injection setup, which is immutable after the service container is initialized.
Hope that helps,
Nate


feat: epmadpafic 878 review GitHub security permissions

This PR is included in version 1.1.0 :tada:


Open Weather Map

Can't find open weather map API on rapid Api website and alternate or some sort of help will be appreciated.

same problem here

You can make a free account on the OpenWeather website.
Assuming you have created the .env file to hide your api keys you can then update the getWeatherData function in the js file of the api folder:
export const getWeatherData = async (lat, lng) => {
  try {
    const apiKey = process.env.REACT_APP_OPEN_WEATHER_API_KEY;
    const { data } = await axios.get(
      `https://api.openweathermap.org/data/2.5/weather?lat=${lat}&lon=${lng}&appid=${apiKey}`
    );

    return data;
  } catch (error) {
    console.log(error);
  }
};

Additionally, the img src will need to be updated in the Map.jsx file:
{weatherData?.list?.map((data, i) => (
          <div key={i} lat={data.coord.lat} lng={data.coord.lon}>
            <img
              src={`https://openweathermap.org/img/wn/${data.weather[0].icon}.png`}
              height={75}
              alt=""
            />
          </div>
        ))}

When console.log'd, the api calls return the expected data with these updates, however, I haven't been able to get the weather icons to successfully render on the map.


security council manager: fixes, tweaks, and unit test coverage

won't be merged in light of https://github.com/ArbitrumFoundation/governance/pull/75 but leaving open as comments could still be relevant


Removed note about deprecated IDP_ENABLE_STAGE flag

Thanks for contributing to the Harness Developer Hub! Our code owners will review your submission.
Description

Please describe your changes: __________________________________
Jira/GitHub Issue numbers (if any): ______________________________
Preview links/images (Internal contributors only): __________________

PR lifecycle
We aim to merge PRs within one week or less, but delays happen sometimes.
If your PR is open longer than two weeks without any human activity, please tag a code owner in a comment.
PRs must meet these requirements to be merged:

[ ] Successful preview build.
[ ] Code owner review.
[ ] No merge conflicts.
[ ] Release notes/new features docs: Feature/version released to at least one prod environment.

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://6669f0ada49b331b841a7989--harness-developer.netlify.app

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://668544f1990b89f9a6238070--harness-developer.netlify.app


Bump Docker 1.13.1 in CentOS 7 AMI

High Level Description
Bump Docker 1.13.1 in the CentOS AMI
I tested deploying a cluster using the advanced templates and running an nginx Docker container via Marathon.
Switched from rpm -i to yum localinstall to have it automatically install dependencies without the need to manually maintain a package list.
New dependency in Docker 1.13.1:
libseccomp.so.2()(64bit) is needed by docker-engine-1.13.1-1.el7.centos.x86_64

@mesosphere-mergebot bump-ee

The failed azure test is unrelated to this PR. Azure uses Ubuntu 16.04 and isn't touched by the changed cloud_images/centos7/install_prereqs.sh.

closing in favour of #1305


python3Packages.slack-sdk: 3.11.1 -> 3.11.2

Motivation for this change
Update to latest upstream release 3.11.2
Change log: https://github.com/slackapi/python-slack-sdk/releases/tag/v3.11.2
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
21.11 Release Notes (or backporting 21.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 138938 run on x86_64-linux 1

2 packages built:

python38Packages.slack-sdk
python39Packages.slack-sdk

Result of nixpkgs-review pr 138938 at a053f7d8 run on aarch64-linux 1

2 packages built successfully:

python38Packages.slack-sdk
python39Packages.slack-sdk


Create New Huddle Form: Want ENTER to submit from any input field

Right now hitting ENTER from the Date or Time input field doesn't submit the form. Can we fix that, i.e., have ENTER work from those fields? If not, move the Zoom link field to the end so can hit ENTER in that field, which will submit the form.

This has been working for a while.


fix(trivy) failed pull image

close #246
trivy can't pull image with auth fail because it is not using internal auth url, because core_url doesn't have port

this issue is because harbor pr https://github.com/goharbor/harbor/pull/11755/files not handle trivy request well


Use JSON imports rather than a custom Babel transform

The Babel plugin has a custom JSON transform that replaces PDFJSDev.json("$ROOT/some/path.json") with the contents of some/path.json:
https://github.com/mozilla/pdf.js/blob/c60c0d1c6df3a4e93da9d9ad4186eb07e837e024/external/builder/babel-plugin-pdfjs-preprocessor.mjs#L43-L55
This is only used once, in the Chromium extension code:
https://github.com/mozilla/pdf.js/blob/c60c0d1c6df3a4e93da9d9ad4186eb07e837e024/web/chromecom.js#L197-L199
There is a Stage 3 proposal that would allow directly importing JSON files:
import i18nFileAccessLabels from "./chrome-i18n-allow-access-to-file-urls.json" with { type: "json" };

This feature is already supported by default by Node.js, Chrome, Safari and Webpack. Given that the code where we use PDFJSDev.json is bundled with webpack, this syntax is safe to use and we can remove the custom PDFJSDev.json feature.
However, ESLint doesn't support this syntax yet because it's only a Stage 3 proposal and not part of the standard yet (it will hopefully become part of the standard in October). There are two options:

wait until the first ESLint minor release after October, which will support this syntax by default
use @babel/eslint-parser to let ESLint parse this syntax

wait until the first ESLint minor release after October, which will support this syntax by default

Given just how little that pre-processor feature is being used, this option seems overall less disruptive than making changes to linting itself. (Especially since we should, at some point, switch to ESLint 9; which is being tracked in issue #17928.)

With issue #17928 fixed, I guess that we could move forward with this now.


ABCI state machine

Using https://github.com/nomic-io/abci2, we'll build the ABCI state machine (e.g. managing mempool and consensus states, committing to store, etc.).

This is done for now (although it was done quick and dirty in order to get to an early release sooner).


Rapberry Pi 3 running hot

Running just this module and nothing else, the cpu utilization on my RPi3 is over 70%. A temperature warning pops up after a while in the upper right corner.
Any ideas on how to reduce the cpu load?

The module updates its status every second. Although that shouldn’t be a problem, you can change that behaviour. Try to set updatesEvery to a higher value. Maybe five. If then experiencing the same problems then it has another reason.
Maybe you should disable all module an see what happens then.

Sorry, turned out to be an issue with an issue with electron
npm install<EMAIL_ADDRESS>solved it