🐛 BUG: constellation wrangler.toml not parsing

Which Cloudflare product(s) does this pertain to?
Workers/Other
What version of Wrangler are you using?
3.0.1
What operating system are you using?
Mac
Describe the Bug
The wrangler config is not parsing with the following:
npx wrangler constellation model upload "dank-cloudemb" "st-multi-qa" model_quantized.onnx

✘ [ERROR] Unterminated inline array

    /Users/robert/Documents/dank-cloudemb/wrangler.toml:6:86:
      6 │ ...on = [{ binding = 'EMB', project_id = '88da3e65-0d0d-4f53-b4cf-7988efa65449'
        ╵

Here is the config:
# Top-level configuration
name = "dank-emb-worker"
main = "main.ts"
compatibility_date = "2022-07-12"

constellation = [
  {
  binding = 'EMB',
  project_id = '88da3e65-0d0d-4f53-b4cf-7988efa65449'
  },
]

This works:
# Top-level configuration
name = "dank-emb-worker"
main = "main.ts"
compatibility_date = "2022-07-12"

constellation = [{ binding = 'EMB', project_id = '88da3e65-0d0d-4f53-b4cf-7988efa65449'},
]

TOML doesn't allow inline tables on multiple lines.
https://toml.io/en/v1.0.0#inline-table


Cast Receiver appearance

Have you read the FAQ and checked for duplicate open issues?:
yes
What version of Shaka Player are you using?:
2.4.4, 2.5.0-beta, master
Can you reproduce the issue with our latest release version?:
yes
Can you reproduce the issue with the latest code from master?:
yes
Are you using the demo app or your own custom app?:
both
If custom app, can you reproduce the issue using our demo app?:
yes
What browser and OS are you using?:
linux, windows 10, chrome
What are the manifest and license server URIs?:
not needed

What did you do?
I modified the index.html and receiver_app.css files in the demo directory. Then I compiled the code by running the python build / build.py + @ complete --force or python build / all.py --force command
What did you expect to happen?
I would like to change the appearance of cast_receiver, to integrate the logo of my company. I would also like to change the title that appears in the Chromecast selection popup that is currently Shaka v2.1 Cast Demo. This title is just above the progress bar
What actually happened?
the appearance of cast_receiver doesn't change

Does this answer your question?

Thank you for your fast answer, I am quickly going to test.

Thank you for your help, I succeeded well has to modify the appearance of the chromecast.


Allow setting user languages through api

Describe your change here.
Review checklist

[ ] First, the most important one: is this PR small enough that you can actually review it? Feel free to just reject a branch if the changes are hard to review due to the length of the diff.
[ ] If there are any migrations, will they the previous version of the app work correctly after they've been run (e.g. the don't remove columns still known about by ActiveRecord).
[ ] If anything changed with regards to the public API, are those changes also documented in the apiary.apib file?
[ ] Are all the changes covered by tests? Think about any possible edge cases that might be left untested.

LGTM - no specs?

@camallen Didn't add any specs because it seems like just configuring rather than programming. The other params aren't tested either (apart from the ones that trigger workers when they change). I added it to the serializer, and it's tested there now.


(1059) Submitted applications stay in the open application screen

This fixes the application dashboard to group applications by the newly added status attribute. I've also updated the tables to be more in line with the prototype.
Screenshot

 Submitted applications stay in the open application screen.


Unable to publish function from vscode to azure

Azure Functions Core Tools does not support this deployment path. Please configure the app to deploy from a remote package using the steps here: https://aka.ms/deployfromurl

Please add detailed steps to reproduce the issue.

This issue has been closed automatically because it needs more information and has not had recent activity. See also our issue reporting guidelines.
Happy Coding!


Should we use std::shared_lock<> to own the db_mmap_lock_ in the View?

Currently, db_mmap_lock_ is allocated in DBImpl::View, and released in TxManager::RollBack(), the later one is called by ~ViewTx. Should we use  std::shared_lock<> to better owning this?

Yes, that's a great suggestion. Thank you

@yuyuaqwq Would Would this causing it Release twice? Since TxManager::RollBack() might call this explicitly?

@mapleFU  No, because the Rollback of the View is only called in ~View, the user is not allowed to explicitly call

Acquire:
ViewTx DBImpl::View() {
     db_mmap_lock_.lock_shared();
     return tx_manager_->View();
 }

And:
TxManager::View() <-- acquire from ctor

Release:
void TxManager::RollBack(TxId view_txid) {
    db_->db_file_mmap_lock().unlock_shared();

And dtor.
Actually the code works but might allocate / release the lock twice?

@mapleFU I'm sorry, I misunderstood your meaning.
Yes, this is my question. I forgot to remove it. Thank you for pointing out this issue

No problem, thanks for your quick response


[Enhancement] Method concatenates strings using + in a loop [ShowClientHandler]

Search before asking

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

Enhancement Request

located at:
org/apache/eventmesh/runtime/admin/handler/ShowClientHandler.java line 74
analysis and explanation:
The method seems to be building a String using concatenation in a loop. In each iteration, the String is converted to a StringBuffer/StringBuilder, appended to, and converted back to a String. This can lead to a cost quadratic in the number of iterations, as the growing string is recopied in each iteration.
Describe the solution you'd like
Use the StringBuilder, append() method instead.
Are you willing to submit PR?

[ ] Yes I am willing to submit a PR!

@xwm1992 , I can work on this , please assign it to me


Basic SPV address tracking

I am trying to get a basic example of tracking a bitcoin address (see https://github.com/djpnewton/NBitcoin.SPV.TrackAddressBasic/blob/master/NBitcoin.SPV.TrackAddressBasic/Program.cs )
I am connected to a trusted local peer, downloading all the headers trying to get notifications of all relevant txs to but I seem to get too many false positives and disconnecting the peer (TrackerBehaviour.cs:CheckFPRate()). I think the problem is the program is going around in circles continuously reprocessing the same false positive txs but I am not sure why

The wallet class encapsulate those behaviors to use them correctly in a privacy maximizing way. (https://github.com/NicolasDorier/NBitcoin.SPVSample)
It is normal that a node get kicked after a while, scanning make the filter "dirty" which raise the FP Rate and so the bandwidth, so the filter need to be reset time to time, and the most privacy friendly way of resetting the filter is to drop the connection and find a new node.
Anyway, to fix your particular example, what you need to do is save the TrackerBehavior.CurrentProgress (which is a BlockLocator, same type as scanLocation) of the Tracker, so when it reconnects, it does not start all over again.

Thanks for the tips.
When TrackerBehavior gets into this high false positive state TrackerBehavior.CurrentProgress is null. I have looked through the code and done some debugging but I cant see why that should be so (it seems to be a bug)

The other bug is: You should not use Attach/Detach on behavior. The behavior you are manipulating in this code is not the same than the one really attach.
When you attach a behavior to the node through the NodeParameters, then, when the node start, the behavior will be cloned the clone attached to the node.

Thanks again.
Ok I modified my code to use the tracker behavior from Node.Behaviors after called Node.Connect().
I have modified the TrackerBehavior not to disconnect when getting too many false positives (I will try and submit a PR to make this configurable in the future)
I get lots of 'merkleblock' and 'tx' messages but none of them generate tracker operations unless the transaction I want to track is in my peers memory pool, I do not get tracker operations from transactions that are confirmed in the block chain (which is more important to me)

I have modified the TrackerBehavior not to disconnect when getting too many false positives

Don't do that. If you do, eventually your SPV client will download full blocks.

k I modified my code to use the tracker behavior from Node.Behaviors

Should be ok, but the practice is to put it in the NodeParameters and pass the parameter at Node.Connect.

I want to track is in my peers memory pool, I do not get tracker operations from transactions that are confirmed in the block chain (which is more important to me)

Can I see your code now ?

Can I see your code now ?

https://github.com/djpnewton/NBitcoin.SPV.TrackAddressBasic/blob/2be2dd47e6d3d8fc10ea135a13751bc888b4049b/NBitcoin.SPV.TrackAddressBasic/Program.cs
On Aug 22, 2016 9:06 PM, "Nicolas Dorier"<EMAIL_ADDRESS>wrote:

I have modified the TrackerBehavior not to disconnect when getting too
many false positives
Don't do that. If you do, eventually your SPV client will download full
blocks.
k I modified my code to use the tracker behavior from Node.Behaviors
Should be ok, but the practice is to put it in the NodeParameters and pass
the parameter at Node.Connect.
I want to track is in my peers memory pool, I do not get tracker
operations from transactions that are confirmed in the block chain (which
is more important to me)
Can I see your code now ?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/MetacoSA/NBitcoin/issues/186#issuecomment-241354406,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AACl29WVTEyYjwTqkJgUQnio-xwAZPatks5qiWZ7gaJpZM4JpQzA
.

will try to debug that, thanks.

The problem is that when you are asking for MerkleBlocks, the peer is already busy responding to your GetHeaders requests. Your code is fine, you just have to wait it sync all the chain.
I advise you for your tests to download all the chain and save it somewhere. (using chain.ToBytes()/chain.ToLoad)
Pro tip:
	<system.diagnostics>
		<sources>
			<source name="NBitcoin.NodeServer" switchValue="Verbose">
				<listeners>
					<add name="def" type="System.Diagnostics.ConsoleTraceListener"></add>
				</listeners>
			</source>
		</sources>
	</system.diagnostics>

Will show you message sent and received by NBitcoin.

@djpnewton Here an example about how to use the NodeGroup class to keep reconnecting automatically when your node is kicked because of FP:
https://gist.github.com/NicolasDorier/a3e3025ebcdd186504aab03ad7ea97b8

Cheers Nicolas,
I think your gist forgot to update scan location on node disconnect though.
I now am able to successfully get information about confirmed transactions relating to my address.
Some thoughts (under the context of using a trusted peer):

If starting TrackerBehaviour.Scan() using old date time offset for skipBefore param the scanning takes a long time
It would be nice to be able to not disconnect peer on high false positive event and instead reset bloom filter somehow
I wonder if there could be a more efficient way using bitcoind REST api + zeromq

@djpnewton why are you not using my Wallet class ? (or you can do your own one by copying the code)
It does exactly what you do + deals with saving the ScanLocation and Tracker operations in JSON.
I should actually use a lower FPRate if the sync is far from the hear of main chain to speed up the sync as well. I agree that this kind of logic is good to add in the TrackerBehavior.
I don't have any dependency on zeromq. You can access bitcoind API with RestClient or RPCClient though. You can also configure bitcoind for firing up a command line when if find a block or transaction.
It would be more efficient, knowing you have a trusted peer I would use this way instead of SPV which is always less efficient and hard to code.

If you are not against some infrastructure, I coded a block explorer called qbit ninja. (http://api.qbit.ninja/) which you can setup on a VM on Azure, it also has its own package (QBitNinja.Client)

@djpnewton why are you not using my Wallet class ? (or you can do your own one by copying the code)
It does exactly what you do + deals with saving the ScanLocation and Tracker operations in JSON.

I just wanted to try and make my own wallet and also start looking at the NBitcoin library. Also I dont think the NBitcoin SPV wallet is setup for using a single trusted node
At the moment I am trying to resurrect the UXTO address index feature in bitcoind. I think it would be useful to have it query-able from the REST interface. Then  I can make an NBitcoin based wallet that uses that.
How big of a database does Qbit Ninja create?

well, the cost of my database for example now is about 30€ a month of storage. You also need a VM. (around 40€)
If you are a business with bizpark you can have free azure storage so it is free.
QBit is not done to optimize size, but to optimize availability and indexing speed.

@djpnewton did I responded to your problem ?
Your way of using NBitcoin was fine, it is just that you need to wait for the chain to sync, which can take a while. (it is around 32MB)
I sent you a simpler way to use NodesGroup for maintaining connection.

Yes my example is working and uses the NodesGroup (https://github.com/djpnewton/NBitcoin.SPV.TrackAddressBasic/blob/master/NBitcoin.SPV.TrackAddressBasic/Program.cs)
I think SPV is not ideal for my case though.
What I would like is to have a trusted peer act as a border router and also give me any information I need to implement a wallet:
P2P Network <----> Trusted Peer <----> My Wallet
Thanks for your help

Yes, so in this case you can use QBitNinja there is a package for it "QBitNinja.Client" the service is on http://api.qbit.ninja/, I am using it for myself.

the problem is my SSD is not so big, the blockchain is 80GB and then something like qbit adds another ~80GB and I start running out of space and it gets slow on the laptop I am developing on.
I am working on reviving the "txoutsbyaddress index" feature (https://github.com/bitcoin/bitcoin/pull/5048/). So far I have updated it to work with the last year of code changes and moved it to a separate database, now I am adding a REST interface.
With this and txnotify/blocknotify I think I will have enough to implement a reliable wallet and only add about 1GB of extra space used.

No, QBitNinja is using Azure Storage. You need to run a server on Azure though so all in all it can cost you around 70 USD.

You cant run it on your own server?

the server need at least to be on Azure in the same datacenter as the storage for initial indexing or it will take very long. I advise you to keep running the server after on Azure datacenter, else, latency might make your server slower.


[breaking change] Stop creating a mongodb_instance in mongodb::default

From https://github.com/edelight/chef-mongodb/issues/179
the logic of https://github.com/edelight/chef-mongodb/blob/master/recipes/default.rb#L42 is absolutely maddening in it's odd dependence on knowing exactly what recipes are in the run list. In particular, it doesn't play nice with wrapper cookbooks which may set up replicasets via include_recipe, not through node or role run lists. Don't get me started on mongos's search.
I propose:

renaming helper recipes to begin with an underscore, including: install -> _install_package, 10gen_repo -> _mongodb_repo,
keep 'standard' usage recipes top level:  configserver, mongos, replicaset, shard, and move the default instance into its own mongod recipe
also move mongo_gem -> ruby to match opscode's mysql, xml, &c cookbooks
have default not do anything!

@josephholsten counter point: i want to setup a single mongo instance without much config

then I'd prefer the doc said something like:
## The simplest thing that could possibly work

To spin up a minimal mongodb service with reasonable defaults, add `recipe[mongodb::mongod]` to your run list.

That works, going to move this to v2 to avoid making v1 too complicated


vde_vmnet: support mDNS

Adding mDNS will be useful for host<->guest and guest<->guest communication.

I was already planning to look into something like https://github.com/blake/external-mdns for use with kubernetes, so it would be great to find something that will work for all use cases. I can't remember if running multiple mDNS services on a single interface was a problem or not. 😄

I can't remember if running multiple mDNS services on a single interface was a problem or not. 😄

Seems safe
https://superuser.com/questions/1272967/is-it-possible-to-run-multiple-mdns-responders-for-dns-sd-on-the-same-host
https://stackoverflow.com/a/14388707

The meaning of SO_REUSEADDR changes for multicast addresses as it allows multiple sockets to be bound to exactly the same combination of source multicast address and port. In other words, for multicast addresses SO_REUSEADDR behaves exactly as SO_REUSEPORT for unicast addresses. Actually, the code treats SO_REUSEADDR and SO_REUSEPORT identically for multicast addresses, that means you could say that SO_REUSEADDR implies SO_REUSEPORT for all multicast addresses and the other way round.


Multipart support for nodejs kubeless functions

Feature request
What happened:
I'm trying to implement a kubeless function in nodejs that receives a multipart form with a file and additional properties (JSON), commonly this can be easily achieved using express and an external library like multer. However, it seems like kubeless is converting the request data into a string no matter the type.
What you expected to happen:
I'd like to be able to read streams from the request.
First step would be something like: https://github.com/kubeless/runtimes/pull/29
Environment:

Kubernetes version (use kubectl version):

Client Version: version.Info{Major:"1", Minor:"14", GitVersion:"v1.14.3", GitCommit:"5e53fd6bc17c0dec8434817e69b04a25d8ae0ff0", GitTreeState:"clean", BuildDate:"2019-06-06T01:44:30Z", GoVersion:"go1.12.5", Compiler:"gc", Platform:"darwin/amd64"}
Server Version: version.Info{Major:"1", Minor:"14", GitVersion:"v1.14.3", GitCommit:"5e53fd6bc17c0dec8434817e69b04a25d8ae0ff0", GitTreeState:"clean", BuildDate:"2019-06-06T01:36:19Z", GoVersion:"go1.12.5", Compiler:"gc", Platform:"linux/amd64"}

Kubeless version (use kubeless version):
v1.0.3
Cloud provider or physical cluster:
Physical cluster

hi @diugalde,
I have rebuilt and push the NodeJS images with your patch (https://github.com/kubeless/runtimes/pull/30/files)
If you want to try the changes you can edit the kubeless config (kubectl edit configmap -n kubeless kubeless-config), change the checksum in the NodeJS images, and restart the controller pod (kubectl delete pod -n kubeless -l kubeless=controller).
Can you try that and let me know if that fixes your issue?

Hi @andresmgot , I followed your steps but it seems like the new image is not being loaded after running kubectl delete pod -n kubeless -l kubeless=controller (event.data still has a string).

can you check if the function pod is using the new image? The following command:
kubectl get pods -o jsonpath="{.items[0].spec.containers[0].image}" -l function=your-function

should return the new sha (28fe2edc427e1deefb9d2cb12878d98e64fd14dc0299f4bf009f9ce26bd8b14a for node 6 or 1c57908576b80142b070018091d2fd2d6446428de94d331aa0483ddc5e6e002c for node 8)

It was my mistake, I was using the kubeless-serverless-offline plugin, I have to make the change there too. However, once I was able to test it, I noticed we also needed an additional change related to the body parser. Here: https://github.com/kubeless/runtimes/pull/31


Method not found HttpOperationException setter

I'm updating the AzureBot to V3 (this branch in my AzureBot fork https://github.com/nzthiago/azurebot/tree/botbuilderV3).
When I run it locally and with the emulator v3 I'm getting the following error and stack trace. Any ideas? It could be related to the nuget packages/dlls I ended up with after updating the Azure packages...

System.MissingMethodException occurred
HResult=-2146233069
Message=Method not found: 'Void Microsoft.Rest.HttpOperationException.set_Request(System.Net.Http.HttpRequestMessage)'.
Source=Microsoft.Bot.Connector
StackTrace:
at Microsoft.Bot.Connector.BotState.<GetConversationDataWithHttpMessagesAsync>d__8.MoveNext()
at System.Runtime.CompilerServices.AsyncTaskMethodBuilder`1.Start[TStateMachine](TStateMachine& stateMachine)
at Microsoft.Bot.Connector.BotState.GetConversationDataWithHttpMessagesAsync(String channelId, String conversationId, Dictionary`2 customHeaders, CancellationToken cancellationToken)
at Microsoft.Bot.Connector.BotStateExtensions.<GetConversationDataAsync>d__7.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
at System.Runtime.CompilerServices.TaskAwaiter`1.GetResult()
at Microsoft.Bot.Builder.Dialogs.Internals.ConnectorStore.<Microsoft-Bot-Builder-Dialogs-Internals-IBotDataStore<Microsoft-Bot-Connector-BotData>-LoadAsync>d__2.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
at System.Runtime.CompilerServices.TaskAwaiter`1.GetResult()
at Microsoft.Bot.Builder.Dialogs.Internals.CachingBotDataStore_LastWriteWins.<Microsoft-Bot-Builder-Dialogs-Internals-IBotDataStore<System-Object>-LoadAsync>d__5.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
at System.Runtime.CompilerServices.TaskAwaiter`1.GetResult()
at Microsoft.Bot.Builder.Dialogs.Internals.BotDataBase`1.<LoadData>d__16.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
at System.Runtime.CompilerServices.TaskAwaiter`1.GetResult()
at Microsoft.Bot.Builder.Dialogs.Internals.BotDataBase`1.<LoadAsync>d__8.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
at System.Runtime.CompilerServices.TaskAwaiter.GetResult()
at Microsoft.Bot.Builder.Dialogs.Internals.PersistentDialogTask.<Microsoft-Bot-Builder-Dialogs-Internals-IPostToBot-PostAsync>d__6`1.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
at System.Runtime.CompilerServices.TaskAwaiter.GetResult()
at Microsoft.Bot.Builder.Dialogs.Conversation.<SendAsync>d__4.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
at System.Runtime.CompilerServices.TaskAwaiter.GetResult()
at Microsoft.Bot.Builder.Dialogs.Conversation.<SendAsync>d__2.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
at System.Runtime.CompilerServices.TaskAwaiter.GetResult()
at AzureBot.MessagesController.<Post>d__0.MoveNext() in C:\code\AzureBot\AzureBot\Controllers\MessagesController.cs:line 27
InnerException:

I believe this is a conflict between Microsoft.Rest.ClientRuntime version 1.8.2 needed by the Bot Framework, and Microsoft.Rest.ClientRuntime > 2.0.0 needed by Microsoft.Azure.Management.ResourceManager via Microsoft.Rest.ClientRuntime.Azure that I'm using on another project which I reference from my bot project.

Yes, confirmed. It was the same problem as we had before. The dependency the bot framework has on Microsoft.Rest.ClientRuntime version 1.8.2 is stopping us from using the latest non-deprecated version of the Azure nuget packages.
Can link this issue to the previous one we raised: #208

#208

Hi @nzthiago Thiago, is this issue resolved?

Bot Builder is dependent on Microsoft.Rest.ClientRuntime 1.8.2 and that is the main cause of the exception that you are seeing because of the breaking change in Rest.ClientRuntime. We are looking into upgrading the builder dependencies to depend on the lastest Rest.ClirentRuntime.

Thanks @msft-shahins  - I only tried again as Dan asked, I thought maybe this had been fixed with Bot Builder 3.1.0 that has come out a couple of days ago. No problem, will keep an eye out for the update that fixes the dependency on 1.8.2

Closing this since it is going to be resolved in the next release of the builder. This commit have all the changes upgrading Rest.ClientRuntime to 2.3.2.


ios12.0 Unknown jailbreak. Aborting.

When will ios12.0 be supported

I was waiting too!

@Linyok Hey, I just found another tools, which may helps you: https://iosgods.com/topic/80763-crackerxi-decrypt-ios-apps-on-ios-10-11-12/
It based on bfinject

@Linyok Hey, I just found another tools, which may helps you: https://iosgods.com/topic/80763-crackerxi-decrypt-ios-apps-on-ios-10-11-12/
It based on bfinject

thanks。I always have this tools，but it can't cycript my app when I ssh.I hope that I can use cycript.dyblib.


[FIX] Bootstrap

Fixes #215
Delete public/bootstrap-3.3.7
Delete public/css/4.0.0_bootstrap.min.css && public/css/bootstrap-3.3.7.css
Delete public/js/1.13.0_popper.min.js && public/js/4.0.0_bootstrap.min.js
replace hard links (within project) with  CDN links

Looks good to me.


Radar filter not working

Hi I'm trying to override the light pattern when the radar detects a car. However, the radar still uses its built-in flash pattern (too bright at night). This is my configuration:
###0,610283536::1#7,5!Brake:1:2:0:0A[-25!Night:2:6:0:0Es0,r0I}0]0!Day:2:6:0:0Er0,s0I[70]0!On:1:6:0:0H]2!:1:0:0:0D=1##6,5:Gardia 300:0:16777215!2,:-1,Off:0!1,Low Solid:4!1,Group Ride:8!1,Night Flash:6!1,Day Flash:7#0::#0:0#123!:13!0,4,8#0#B3843##14#0#0
Thank you very much for your help.

Hi,
based on the manual, Gardia r300 starts flashing when a vehicle is approaching you regardless the current mode. This is similar on how Varia works and unfortunately (at least for Varia), this is something that cannot be disabled.


Use of undeclared type 'WKExtensionDelegate'

After importing WatchKit to the ExtensionDelegate.swift, Xcode 7 still throws this error: Use of undeclared type 'WKExtensionDelegate'. Is there any solution to this problem?
Thank you, DZetko.

Haven't seen this in my version of Xcode. Are you on the latest beta?
ExtensionDelegate.swift should already have watchkit imported by default when you create the project.

Not reproducible

Ya, I have the same issue.  All info.plist are very similar.  It's as though Xcode is not reading the info.plist in the Extension area.  DZetko, what happened with your project.  How did you eventually remove this problem.


About new version grasshopper script

Hi Freddie,
I want to ask if the new version of the Grasshopper package has the infill setting.

Hey Huang,
I'm assuming that you are talking about the grasshopper file that Freddie uploaded last month. (If not, just ignore me.)
As far as I can tell in the files there isn't an option yet for infill or layer height.

Thanks, MRackliffe,
Btw, I want to ask if there are other slicers for 5-axis 3D printing.


[14.0] ISPN-14260 Use local in-memory reads for the ClusterRoleMapper to avo…

https://issues.redhat.com/browse/ISPN-14260

merged! thanks @tristantarrant !


Error on store - some kind of assembly scanning / reflection issue

Wrote a very small console app which connects to RavenDb, queries a collection and stores the same document into Marten.
However when doing martenSession.Store(doc); it throws a bunch of errors into my console window (attached).

My code:
var docStoreConfig = new Raven.Client.Document.DocumentStore             {                 ConnectionStringName = "RavenDB"             };             docStoreConfig.Initialize();
        using (var store = docStoreConfig.Initialize())
        {
            store.Conventions.IdentityPartsSeparator = "-";

            using (var mStore = DocumentStore.For("host=localhost;database=myravendb;password=mypassword;username=xxx"))
            {
                using (var rSession = store.OpenSession())
                {
                    using (var mSession = mStore.LightweightSession())
                    {
                        var enumerator = rSession.Advanced.Stream<Account>(Etag.Empty, 0, Int32.MaxValue);
                        int i = 0;
                        while (enumerator.MoveNext())
                        {
                            var account = enumerator.Current.Document;
                            mSession.Store(account); // fails here
                            if (i % 100 == 0) mSession.SaveChanges();
                            i++;
                            Console.WriteLine("{0} - {1} - {2}", i.ToString(), account.Id, account.ArtistName);
                        }
                        mSession.SaveChanges();
                    }
                }
            }
        }`

@phinett What version of Marten do you have? This one was already reported and theoretically fixed in https://github.com/JasperFx/marten/commit/27cb3967fcddd2db0eef17dd7f7d04952e65460f. Can you bump up to the latest?

Oh, just read things. I'm keeping the Console junk for now just to get an indication of what's happening inside. So far we haven't seen any harm in swallowing the bad Sparrow reference

Ok that's fine with me.


Add options to relax or make more strict the parsing rules.

Permissive -> will not error on the parser pass. This can be used to grab routes for props that have Options in FromMatches, so all captures don't need to be present.
I'm not super keen on this one. I want to brainstorm on if there is a better way to represent optional query and fragment sections.
Strict -> Won't handle trailing '/' characters.
CaseInsensitive -> Don't care about case when matching.

These have been added.
Strict, CaseInsensitive and Incomplete, the latter of which allows matches to succeed as long as all matcher tokens are used successfully, without regard for full consumption of the route string.


documentation instructs to use deprecated azcli command

when running the az group deployment create command I get the following error:

This command is implicitly deprecated because command group 'group deployment' is deprecated and will be removed in a future release. Use 'deployment group' instead.

The deployments in the section "Deploy via ARM template (with existing Resource Group)" should use the az deployment create command instead

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

ID: 096d934c-2856-e868-efeb-6622e2a5eafb
Version Independent ID: 8bf74926-6a35-5753-0a1a-6663ba5965b2
Content: Deploy your bot - Bot Service - Bot Service
Content Source: articles/bot-builder-deploy-az-cli.md
Service: bot-service
GitHub Login: @ivorb
Microsoft Alias: kamrani

Hey @asdkant Thanks for reporting this. This is a known issues and we are fixing in this PR: https://github.com/MicrosoftDocs/bot-docs-pr/pull/2172

@asdkant The major changes are the following commands:
az deployment group create
az deployment sub create

The doc is updated and merged. The content will go live soon. Thanks!


(request, not an issue)

next update can you add an option to make the widgets transparent and also add an option for the widgets to automatically update every 20 mins - 5 hours or something like that so you can select a time you want it to automatically update. if so, thank you

I will definitely check what I can do. I will try to 😊


[Test failure] UnprotectWithDifferentPurposeFails

https://mc.dot.net/#/user/aspnetcore/pr~2Faspnet~2Faspnetcore/ci/20190207.24/workItem/Microsoft.AspNetCore.Authentication.Test-netcoreapp3.0/wilogs
Failed   Microsoft.AspNetCore.Authentication.DataHandler.SecureDataFormatTests.UnprotectWithDifferentPurposeFails

Error Message:
 System.Security.Cryptography.CryptographicException : An error occurred while trying to encrypt the provided data. Refer to the inner exception for more information.
---- System.IO.IOException : The process cannot access the file 'C:\Users\runner\AppData\Local\ASP.NET\DataProtection-Keys\key-38c34274-c16d-4e40-a5ff-239672c1a8f3.xml' because it is being used by another process.

Stack Trace:
   at Microsoft.AspNetCore.DataProtection.KeyManagement.KeyRingBasedDataProtector.Protect(Byte[] plaintext) in /_/src/DataProtection/DataProtection/src/KeyManagement/KeyRingBasedDataProtector.cs:line 141
   at Microsoft.AspNetCore.Authentication.SecureDataFormat`1.Protect(TData data, String purpose) in /_/src/Security/Authentication/Core/src/SecureDataFormat.cs:line 34
   at Microsoft.AspNetCore.Authentication.DataHandler.SecureDataFormatTests.UnprotectWithDifferentPurposeFails() in /_/src/Security/Authentication/test/SecureDataFormatTests.cs:line 59

----- Inner Stack Trace -----
   at System.IO.FileStream.ValidateFileHandle(SafeFileHandle fileHandle)
   at System.IO.FileStream.CreateFileOpenHandle(FileMode mode, FileShare share, FileOptions options)
   at System.IO.FileStream..ctor(String path, FileMode mode, FileAccess access, FileShare share, Int32 bufferSize, FileOptions options)
   at System.IO.FileStream..ctor(String path, FileMode mode, FileAccess access, FileShare share)
   at System.IO.File.OpenRead(String path)
   at Microsoft.AspNetCore.DataProtection.Repositories.FileSystemXmlRepository.ReadElementFromFile(String fullPath) in /_/src/DataProtection/DataProtection/src/Repositories/FileSystemXmlRepository.cs:line 102
   at Microsoft.AspNetCore.DataProtection.Repositories.FileSystemXmlRepository.GetAllElementsCore()+MoveNext() in /_/src/DataProtection/DataProtection/src/Repositories/FileSystemXmlRepository.cs:line 83
   at System.Collections.Generic.List`1..ctor(IEnumerable`1 collection)
   at System.Linq.Enumerable.ToList[TSource](IEnumerable`1 source)
   at Microsoft.AspNetCore.DataProtection.Repositories.FileSystemXmlRepository.GetAllElements() in /_/src/DataProtection/DataProtection/src/Repositories/FileSystemXmlRepository.cs:line 68
   at Microsoft.AspNetCore.DataProtection.KeyManagement.XmlKeyManager.GetAllKeys() in /_/src/DataProtection/DataProtection/src/KeyManagement/XmlKeyManager.cs:line 152
   at Microsoft.AspNetCore.DataProtection.KeyManagement.KeyRingProvider.CreateCacheableKeyRingCore(DateTimeOffset now, IKey keyJustAdded) in /_/src/DataProtection/DataProtection/src/KeyManagement/KeyRingProvider.cs:line 57
   at Microsoft.AspNetCore.DataProtection.KeyManagement.KeyRingProvider.CreateCacheableKeyRingCore(DateTimeOffset now, IKey keyJustAdded) in /_/src/DataProtection/DataProtection/src/KeyManagement/KeyRingProvider.cs:line 103
   at Microsoft.AspNetCore.DataProtection.KeyManagement.KeyRingProvider.Microsoft.AspNetCore.DataProtection.KeyManagement.Internal.ICacheableKeyRingProvider.GetCacheableKeyRing(DateTimeOffset now) in /_/src/DataProtection/DataProtection/src/KeyManagement/KeyRingProvider.cs:line 254
   at Microsoft.AspNetCore.DataProtection.KeyManagement.KeyRingProvider.GetCurrentKeyRingCore(DateTime utcNow) in /_/src/DataProtection/DataProtection/src/KeyManagement/KeyRingProvider.cs:line 186
   at Microsoft.AspNetCore.DataProtection.KeyManagement.KeyRingProvider.GetCurrentKeyRing() in /_/src/DataProtection/DataProtection/src/KeyManagement/KeyRingProvider.cs:line 142
   at Microsoft.AspNetCore.DataProtection.KeyManagement.KeyRingBasedDataProtector.Protect(Byte[] plaintext) in /_/src/DataProtection/DataProtection/src/KeyManagement/KeyRingBasedDataProtector.cs:line 102

Occuring on #8076. Skipping this test in Helix until we resolve this.

FYI this is a product issue in Data Protection, not a test issue. Unfortunately it primarily reproduces on the CI.

Cool, still skipping the test ;)


⚠ Turnos 24x7 has degraded performance

In 6f90914, Turnos 24x7 (https://reservas-dev.alternativasinteligentes.com/) experienced degraded performance:

HTTP code: 200
Response time: 5442 ms

Resolved: Turnos 24x7 performance has improved in 7db52f7.


[stable/postgresql] Adapt to new bitnami postgresql bash container

What this PR does / why we need it:
Which issue this PR fixes
(optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged)

https://github.com/helm/charts/issues/13363
https://github.com/bitnami/bitnami-docker-postgresql/issues/125

Special notes for your reviewer:
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[x] DCO signed
[x] Chart Version bumped
[x] Variables are documented in the README.md
[x] title of the PR contains starts with chart name e.g. [stable/chart]

/hold

/hold cancel

/lgtm

/ok-to-test


Auto configuration for reactive redis client in Spring Boot

Currently I am using Redisson for locks with the RedissonClient that gets automatically configured via the spring.redis.(host|port|password) properties in the application.yaml which is super convenient.
Now I wanted to include some message handling via reactive topics to return a Flux for a websocket-based subscription. Thus I attempted to autowire a RedissonReactiveClient and noticed it isn't autoconfigured.
I am curious what the proposed way to expose a RedissonReactiveClient bean with the same configuration would look like.

Is there any update on this?

Done

Thanks @mrniko


Extract RefCount implementation from header to cpp file

RefCount.h is part of the runtime. The runtime must be built with Clang. RefCount.h uses clang/gcc specific atomic builtins.
However, RefCount.h is included by HeapObject.h, which is included from Metadata.h, which is included from IRGen.
IRGen can be built with MSVC, so therefore RefCount must also be able to be built with MSVC.
MSVC doesn't have these builtins, so we need to extract the compiler-specific implementation to the runtime, so we expose a compiler-independent interface to the compiler-independent swiftc compiler

I'd like to explore using <atomic> C++ 11 but there's a slight problem. C++ 11 exposes atomic_fetch_add, but doesn't have an equivalent for atomic_add_fetch so there's not a 1-1 conversion between GCC/Clang intrinsics and <atomic> functions

@gparker42

I will leave a real review to @gparker42, but is there any reason to merge it into the *.cpp file. Why not just keep it as a local header.

I've outlined my reasoning in the Pr description: basically we want to expose a compiler independent interface to the swift.exe source which can be built with MSVC, whilst allowing the use of GCC/Clang built ins in the runtime, which must be built with Clang.
RefCount.h is an external dependency of IRGen so must able to be compiled with Msvc for the rest of the swift compiler to be built with msvc

@swift-ci please test

I misunderstood what you were doing. I thought you were moving everything from the .h into the .cpp file.
For future reference, the word you are looking for is "out of line", i.e. move definition of ref count implementation out of line into *.cpp file. The phrase is a much clearer signal that you are leaving the declaration in the header.
This looks fine to me, but I defer to @gparker42. This is his area.

Sorry, my bad - just learning all the C++ stuff

I have rewritten all of this code in #5282 which is expected to land soon.
Did you measure the performance and look at the generated code of the .cpp implementation? Inlining optimization is important for the refcount implementation, and moving the code to a .cpp file will prevent inlining in the absence of tricks like link-time optimization.

I think #5282's implementation is <atomic> everywhere in RefCount.h. Is that sufficient to solve the MSVC issues?

I had no idea about Perf. Can try checking though, but its likely we can close this PR now that there's some conversion to Atomic
As per your pull request - thats awesome! As long as it doesn't use compiler insitrinsics I'm happy. Atomic is fully supported with MsvC

Sounds good. Let us know how #5282 fares with MSVC (before or after it gets merged).


Is this gem really deprecated?

6.0.0.beta2 was released just few days back, so was curious if it's really being deprecated?
https://rubygems.org/gems/sass-rails

Not this gem, but ruby-sass. See this thread for context: https://github.com/rails/sass-rails/issues/420


bower install installs a large amount of dependencies

Running bower install downloads a huge amount of data.
bower install angular --save downloads more than 22MB (I quit the install at 22MB) and bower install jquery more than 11MB.
What could be the issue here? I'm running version 1.7.9 on Ubuntu Linux 12.04 LTS

Could you paste your bower file here please

No response in 2 months. Closing


[Bug] typeOf item != ‘function’ missing as a native support check

Noticed this as Object.assign was being polyfilled in Chrome despite it having native support.

Punting to 0.0.7 - need to validate whether this is a legit requirement or not...
Fixing the if's in checking() has resolved the issue of Object.assign in Chrome - it was false positiving on one of the statements that has now been split out into a nested if and only fires for browsers that need it now - therefore at the moment all existing checks look "OK"


RuntimeWarning: coroutine 'BOSerWebAPI.get_miner_stats' was never awaited

Describe the bug
When querying a lot of Braiins OS+ miners, we get an error repeated over again, but not 100% of the queries
RuntimeWarning: coroutine 'BOSerWebAPI.get_miner_stats' was never awaited
To Reproduce
Steps to reproduce the behavior:
Repeat this code in a loop going over a few hundred Braiins OS+ miners.
pyasic_miner = await get_miner(ip)
miner_data: MinerData = await pyasic_miner.get_data()
Expected behavior
No errors for not awaiting code
Desktop (please complete the following information):
Linux - Ubuntu 22.04
Miner Information (If applicable):
Bitmain Antminer S19J Pro, S19J Pro+ and S19 XP running Braiins OS+

Couple of missing await calls in the secondary calls made by the internal data gathering functions.  They don't get called unless the multicommand version of the call fails for some reason, so that's why it was only happening some of the time.


Skicky ToolbarAndroid

Really like your component. I use it with ToolbarAndroid component. I've change animating opacity to transparent of background color to get my toolbar actually visible (title, buttons etc), but still have 3 questions:

How can I make elevation of my toolbar? Is it possible? Tried add it when it fully visible, but can't get it work.
I wanna it fully visible before I scroll out parallax background view.
Sticky but not fixed. How can I get it hide when scroll down, and shows up when I scroll up? But it should not hide when it still in parallax background (like it in the play market does).

@terrysahaidak If you render the toolbar in the foreground then it might be what you're looking for. I haven't tried it myself, but if you try it first, let me know if anything isn't working. Sample code would help tremendously if you run into any issues.

@terrysahaidak Closing this for now. Please re-open if you continue to experience the same problem.


Deleting a scan still shows the deleted data

Affecting:
Server
Version:
2afb68ef1edbef1e51906e15e5662046534f60a9
Describe the bug
When I delete a scan for a host via the admin controls, the page refreshes but still shows the scan data that I just tried to delete. A refresh will usually clear this, but it can be confusing.
To Reproduce
Steps to reproduce the behavior:

Login as an administrator
Go to a host page /host/<ip> and click the red trash can button in the top left.
Select "Delete Scan" and confirm your decision
When the page reloads, you'll get a notice saying that it was successfully deleted, but it'll still show the data that you just tried to delete

Expected behavior
It shouldn't show me the deleted data. It should show me the next most recent data or redirect me back to the host summary page (as opposed to the specific scan id page).
Screenshots
N/A
Additional context
This is because the call to the elasticsearch client issues a delete to the cluster but that query is asynchronous by default, so the page will reload before the delete has finished, meaning that it renders the about-to-be-deleted data. We can probably avoid this via the use of wait_for_completion on the delete call, but that may have other implications for larger elastic clusters.

This is inadvertently closed by #310, which changes it so that it no longer redirects back to the page you were just on.


Slideout menu only on Phone

I would like to have the slideout menu only on Smartphone, while on all other resolutions bigger than X show the default menu within the layout.
Is this already implemented? Or do i have to write a window.resize function which creates and destroys the menu, and also moves it out of layout(#panel)?

You can initialize Slideout only in mobile mode. On desktop display your current menu.

Also, you could use mediaqueries:
@media screen and (min-width: 780px) {
  .slideout-panel {
    margin-left: 256px;
  }

  .slideout-menu {
    display: block;
  }

  .btn-hamburger {
    display: none;
  }
}

Demo: http://codepen.io/anon/pen/xGEdvQ?editors=100


zh-trans: add docs/concepts/overview/working-with-objects/field-selectors.md

zh-trans: add docs/concepts/overview/working-with-objects/field-selectors.md

/assign @jimangel


[HUDI-1772] HoodieFileGroupId compareTo logical error(fileId self compare)

Tips

Thank you very much for contributing to Apache Hudi.
Please review https://hudi.apache.org/contributing.html before opening a pull request.

What is the purpose of the pull request
fix error HoodieFileGroupId #compareTo logical
Brief change log
the compareTo method logical is error when partitionPath is equals.
@Override
public int compareTo(HoodieFileGroupId o) {
int ret = partitionPath.compareTo(o.partitionPath);
if (ret == 0) {
ret = fileId.compareTo(fileId);
}
return ret;
}
Verify this pull request
(Please pick either of the following options)
This pull request is a trivial rework / code cleanup without any test coverage.
(or)
This pull request is already covered by existing tests, such as (please describe tests).
(or)
This change added tests and can be verified as follows:
(example:)

Added integration tests for end-to-end.
Added HoodieClientWriteTest to verify the change.
Manually verified the change by running a job locally.

Committer checklist

[ ] Has a corresponding JIRA in PR title & commit

[ ] Commit message is descriptive of the change

[ ] CI is green

[ ] Necessary doc changes done or have another open PR

[ ] For large changes, please consider breaking it into sub-tasks under an umbrella JIRA.

Codecov Report

Merging #2780 (8257ade) into master (e692c70) will decrease coverage by 42.88%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master   #2780       +/-   ##
============================================
- Coverage     52.26%   9.38%   -42.89%
+ Complexity     3679      48     -3631
============================================
  Files           484      54      -430
  Lines         23079    1993    -21086
  Branches       2457     236     -2221
============================================
- Hits          12063     187    -11876
+ Misses         9952    1793     -8159
+ Partials       1064      13     -1051

Flag
Coverage Δ
Complexity Δ

hudicli
?
?

hudiclient
?
?

hudicommon
?
?

hudiflink
?
?

hudihadoopmr
?
?

hudisparkdatasource
?
?

hudisync
?
?

huditimelineservice
?
?

hudiutilities
9.38% <ø> (-60.37%)
0.00 <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

 Comparing base (e692c70) to head (8257ade).
Report is 4396 commits behind head on master.

Additional details and impacted files

@@             Coverage Diff              @@
##             master    #2780      +/-   ##
============================================
- Coverage     52.26%   52.26%   -0.01%
- Complexity     3679     3680       +1
============================================
  Files           484      484
  Lines         23079    23092      +13
  Branches       2457     2457
============================================
+ Hits          12063    12068       +5
- Misses         9952     9959       +7
- Partials       1064     1065       +1

Flag
Coverage Δ

hudicli
36.94% <ø> (ø)

hudiclient
∅ <ø> (∅)

hudicommon
50.77% <100.00%> (-0.01%)
:arrow_down:

hudiflink
56.58% <ø> (ø)

hudihadoopmr
33.44% <ø> (ø)

hudisparkdatasource
71.33% <ø> (ø)

hudisync
45.47% <ø> (ø)

huditimelineservice
64.36% <ø> (ø)

hudiutilities
69.69% <ø> (-0.06%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

Files with missing lines
Coverage Δ

...rg/apache/hudi/common/model/HoodieFileGroupId.java
77.77% <100.00%> (ø)

... and 5 files with indirect coverage changes


Use async cosmos library and upgrade to 2.6.3

This changes replaces the cosmos documentdb library with async cosmosdb library.

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@c6a27dd). Click here to learn what that means.
The diff coverage is 65.3%.

@@            Coverage Diff            @@
##             master    #1321   +/-   ##
=========================================
  Coverage          ?   72.06%
  Complexity        ?     6438
=========================================
  Files             ?      469
  Lines             ?    37016
  Branches          ?     4656
=========================================
  Hits              ?    26677
  Misses            ?     9096
  Partials          ?     1243

Impacted Files
Coverage Δ
Complexity Δ

...ithub.ambry.cloud/azure/AzureCloudDestination.java
73.86% <60%> (ø)
39 <1> (?)

...m.github.ambry.cloud/azure/CosmosDataAccessor.java
75% <67.64%> (ø)
17 <6> (?)

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 c6a27dd...7b2347e. Read the comment docs.


Adding a new todo creates a .todos.json in current directory instead of adding it to global .todos.json

When I create a new todo, it creates by default a new .todos.json when there is none in the current directory. I expected it to add the new todo to the .todos.json in my home directory.

Hi @tcassaert this is by design.  See the docs here.

Maybe I've not explained the issue clear enough.
With the latest version, the current behavior is happening:
I have a .todos.json in my home directory. When I am in another directory, let's say in /tmp where there is no .todo.json file, and I add a new todo, there is a file created /tmp/.todos.json.
According to the docs,  it should use the .todos.json in my home directory in this situation, and not create a new .todos.json file in /tmp.

ohhhh.  yeah.  that's an issue!

I replicated this.  thanks for finding it!

@tcassaert just published 1.5.1 which fixes this issue.  thanks again!

Awesome, thanks for acting so quickly on this. I didn't notice this happening after upgrading to v1.2, ended up littering my folders wtih a few .todo.json files, but not hard to remediate.


[Correction] Animfix and Triangulator are merged into falsetweaks(1.7.10)

Checklist

[X] I've checked that this correction is valid and accurate.
[X] I've checked that this correction has not already been suggested.

What mod is this correction for?
two mods merged in one
What section(s) of the list is this correction for?
Performance
What version(s) dose this correction apply to?
1.8.X or earlier
What needs correcting?
just need to delete animfix and triangulator in 1.7.10 performances mods and add falsetweaks
Input relevant link(s) which can be used to verify the correction.
merged into : https://modrinth.com/mod/falsetweaks
(OPTIONAL) Any extra information about the correction.
No response

duplicate of #140


Post Deployment Action - Set Environment Variable From File Content

I have a post-build event action set up in Visual Studio for my web project that takes the current git hash for HEAD and stores it in a local file called version.txt at the root of the web project.
On a successful deployment to Azure, I would like the contents of the version.txt file (the git hash) to become an environment variable with key CURRENT_VERSION.
I have tried two methods and they both do not appear to be working, but I am also having difficulty debugging them.

I tried the post deployment hook method by placing a script in the /site/deployments/tools/PostDeploymentActions directory as this wiki states https://github.com/projectkudu/kudu/wiki/Post-Deployment-Action-Hooks. I had to create the PostDeploymentActions folder myself as it did not already exist, and those scripts do not appear to be being run (I am not aware of where the deployment logs are stored that that wiki page references to verify). I assume they are not running because one of the scripts I tried, named aaron.cmd, contained one line SET CURRENT_VERSION=HELLO and I am attempting to verify that envar was set by using the debug console https://myapp.scm.azurewebsites.net/DebugConsole. If I execute echo %WEBJOBS_STOPPED% I get a value (that is being set via the portal app settings page), but if I execute echo %CURRENT_VERSION%, I get nothing.
I also tried setting the POST_DEPLOYMENT_ACTION app setting as described here: http://stackoverflow.com/questions/26350201/how-to-add-a-custom-post-deployment-script-to-azure-websites, but that also didn't appear to run.

I verified that the aaron.cmd file was being deployed to the app service. I even manually executed the aaron.cmd file via the DebugConsole to ensure that it could run without error and it did (and afterward performing an echo %CURRENT_VERSION% returned HELLO)
I have not tried the method described here: https://azure.microsoft.com/en-us/blog/new-azure-web-sites-feature-configurable-environment-variables-2/ but I wasn't sure I could use an XDT transform to read the contents of a file...
What would be the recommended way to get the contents of the version.txt file to be set as an environment variable using a post deployment hook? (thinking about just dropping something in the startup of the web project, but I'd prefer not to have to do that)

I think too many things are getting raised in this one issue. Can we simplify and start with just: how to successfully run a Post Deployment action, leaving the details of what you do in that action off the table?

Yes, that would be fine.
What is the preferred method of running custom scripts as a post deployment action? If it is by placing the scripts in the /site/deployments/tools/PostDeploymentActions directory, what is the best way to debug an issue if those do not appear to be running?

Start simple. e.g.

In site\deployments\tools\PostDeploymentActions, create a simple foo.cmd containing just echo hello
Do a git deployment, or redeploy the current commit from the portal

In the deployment details, you should see something like this:

And if you click View Logs, you'll see the output of the batch file.
Does that work for you?

Are scripts within the site\deployments\tools\PostDeploymentActions directory only activated on a git deployment? If so, that could be the issue, we are using webdeploy.
Are any Post Deployment Action hooks available for webdeploy?

Yes, this only happen during Kudu based deployments. msdeploy works quite differently. Unfortunately, at this time there is no great equivalent for msdeploy.

Hi,
I am facing same issue as stated above.
I am using TFS source control and visual studio online 2015 for continuous integration(build definition). In build definition deploying package on app service using : Azure Deployment task.

@NehaOberoi92 Please see http://stackoverflow.com/questions/41561899/add-post-deployment-script-on-azure-websiteapp-service/41574522#41574522


Getting an issue when attempting to run gridsome build

Issue description
When running gridsome build the build process unexpectedly stopped because of the following error :
Initializing plugins...
Loading data from https://test_store_08B.myshopify.com

TypeError: Cannot read property 'data' of undefined
    at Object.transform (/Users/test_account/git/portfolio_shopify/node_modules/gridsome-source-shopify/lib/index.js:337:21)
    at AsyncGeneratorFunction.paginateEach [as each] (/Users/test_account/git/portfolio_shopify/node_modules/got/dist/source/create.js:180:45)
    at processTicksAndRejections (internal/process/task_queues.js:97:5)
    at async queryAll (/Users/test_account/git/portfolio_shopify/node_modules/gridsome-source-shopify/lib/index.js:358:37)
    at async ShopifySource.getProductTypes (/Users/test_account/git/portfolio_shopify/node_modules/gridsome-source-shopify/lib/index.js:547:29)
    at async /Users/test_account/git/portfolio_shopify/node_modules/gridsome-source-shopify/lib/index.js:526:7
    at async Plugins.run (/Users/test_account/git/portfolio_shopify/node_modules/gridsome/lib/app/Plugins.js:141:11)

Issue investigation
I first though it was my mistake by accidentally entering a wrong shop name or storefront token, so verified and even tried to input a wrong one manually to see what happens.  The plugin have a clear error for wrong credentials so this is not linked to it.
I checked to see if my test shop was empty, but I had already added 2 products into two collections.
Steps I followed

Created a new shopify shop
Added a couple of mock products ( 4 products in two collections )
Created a new private app and given it read and write access to everything
Enabled the storefront API
Made sure all products and collections had the private app as their sales channel
Copied the shop name and the storefront token to my project and setup the plugin as per the documentation
Got the error
Tried the same credentials in the shopify starter example to make sure my code was not interfering, got the same error.

Notes
It is highly possible that this error is due to an error configuring shoppify, although it seems to me I followed the steps correctly I am pretty new to the storefront API.

Thanks for reporting @Simon-Renault - you shouldn't need any Admin API permissions, but you will need to tick all Storefront permissions:

Are all these permissions ticked in your case?

Yes I can confirm they are all ticked

Would it be helpful If I shared the store name and storefront token ? It's a test store so I'd have no problem doing so.

Yes, the storefront name and token would be useful, if you could share those...

Also, the default Gridsome template on csb is at version 0.5, so you may need to update that first before it works...

Here they are :
GRIDSOME_SHOPIFY_STOREFRONT= simon-renault
GRIDSOME_SHOPIFY_STOREFRONT_TOKEN = ca8bc7b82b8c5641f9de3e47b0990b42
I managed to make it work in code-sandbox, works like a charm and fetch all my data. Tried downloading the zip and run in my local environment : got the same error as before.
https://codesandbox.io/s/nice-varahamihira-zehn7?file=/gridsome.config.js

Thanks @Simon-Renault - can I also check what version of gridsome-source-shopify you are using locally? I'm running the starter locally with source version 0.2.4, it it all seems to work... (albeit the build fails because there are no images for any of the products)

0.2.4
The issue most likely from from the local environment, although hard to tell why.

Managed to fix the issue locally

Revertes node to v10.21.0
Cleared the node and yarn cache
deleted node modules
deleted yarn.lock
reinstalled everything

Still have no clue what causes the issue in the first place. Will investigate more on this.

Okay - let me know if you find out what the issue was, as I can't seem to reproduce it locally...

Running into the same issue, this has to be an issue with the node version. Moving from Node v13.8.0 to stable did the trick for me.


Add myself as a reviewer

What this PR does / why we need it:
Add myself as a reviewer of this repository.
Release note:

NONE

/test pull-aaq-unit-test

/lgtm
/approve


Feat/workspaces

A short, meaningful PR description. Explain the goal of the PR and the ideas behind it.
Changes
Please list / describe the changes in the codebase for the reviewer(s).

Change 1
Change 2

How/what to test
A few words about what needs to be tested, along with specific testing instructions if needed.
Screenshots / screencast
If relevant, please add some screenshots or a short video.

I think it's because of me but could you fix the tests and rebase main please ?

The iframe does not take the full height. You need to add the header of the page and maybe remove the padding around the iframe as well.

Okay, will update it.

I think it's because of me but could you fix the tests and rebase main please ?

Copy that.

Hostname, username, password, port are _Fields_ that have to be added to the form at the bottom.
As you can see buttons are not displayed when you select postgresql. Please add a title to the dialog and use the standard way of adding button to a dialog.

Layout of the connection card is buggy:

Layout of the connection card is buggy:

It would make sense to be able to go to the connection by clicking on its title.

Layout of the connection card is buggy:

It would make sense to be able to go to the connection by clicking on its title.

Okay. So for the settings and usage&details link what pages should they redirect? Or i can leave the href blank for now?


[travis] Fixing environments

Apologies I had misconfigured the Travis environments where py{27,34}-{flake8,pylint} didn't exist and thus defaulted to running the default [testenv] commands.

LGTM


Support for native modules in generated JS

It could be nice to have an option where the generated JS does not depend on webpack. The primary issue is the import statement of the wasm file I think? Since this project implicitly targets browsers that support wasm, this mostly overlaps also with browsers that support native modules.

Indeed. Even if not targeting browsers with native module support, it was quite surprising to discover that --nodejs does not actually emit Node.js-compatible code, but rather Webpack+Node.js compatible code (there is no reason not to emit CommonJS when that parameter is used).

Thanks for the report! The intention of wasm-bindgen is very much to not require webpack! It just so happens that Webpack is the only bundler (AFAIK) right now that treats wasm modules as ES modules (and has enough implemented to work with wasm-bindgen), so it was the example shown in the README. That being said it's not required to use Webpack, you can instantiate the ES modules that is the wasm module however you'd like!
@LucasHill did you have a particular use case and/or bundler in mind though?

@alexcrichton My project is just a toy app that I had planned on writing in vanilla javascript with native es modules using the module loader in the browser rather than a bundler. I think all the major browsers except firefox support that now.
Just taking a look at what webpack is doing is it appears to convert the import statement into a fetch and then using the WebAssembly API (WebAssembly.compileStreaming) to load the wasm file. Is loading wasm using the import syntax a future native thing you can do or just a webpack feature? Seems this project could just emit JS that uses the WebAssembly API rather than using the import syntax, eliminating the need for webpack or another bundler.
Anyway, loving this project, thanks for making it.

Ah yeah unfortunately wasm modules are not currently hooked up as ES modules, but the tracking issue for that is https://github.com/WebAssembly/design/issues/1087

I'm gonna close this for now as I believe this is natively done/supported right now, but if there's more questiosn please feel free to let me know!

Can't we do this without the es modules?


Node package manager: IllegalArgumentExceptions if package author is a list of authors

Describe the bug
Scan of project fails in the analyze phase, generating the following stack trace:
java.lang.IllegalArgumentException: Element class kotlinx.serialization.json.JsonArray is not a JsonPrimitive
at kotlinx.serialization.json.JsonElementKt.error(JsonElement.kt:327)
at kotlinx.serialization.json.JsonElementKt.getJsonPrimitive(JsonElement.kt:230)
at org.ossreviewtoolkit.plugins.packagemanagers.node.AuthorSerializer.transformDeserialize(PackageJson.kt:138)
at kotlinx.serialization.json.JsonTransformingSerializer.deserialize(JsonTransformingSerializer.kt:78)
at kotlinx.serialization.json.internal.AbstractJsonTreeDecoder.decodeSerializableValue(TreeJsonDecoder.kt:337)
at kotlinx.serialization.internal.TaggedDecoder.decodeSerializableValue(Tagged.kt:207)
at kotlinx.serialization.internal.TaggedDecoder.decodeNullableSerializableElement$lambda$3(Tagged.kt:288)
at kotlinx.serialization.internal.TaggedDecoder.tagBlock(Tagged.kt:294)
at kotlinx.serialization.internal.TaggedDecoder.decodeNullableSerializableElement(Tagged.kt:286)
at org.ossreviewtoolkit.plugins.packagemanagers.node.PackageJson$$serializer.deserialize(PackageJson.kt:90)
at org.ossreviewtoolkit.plugins.packagemanagers.node.PackageJson$$serializer.deserialize(PackageJson.kt:90)
at kotlinx.serialization.json.internal.AbstractJsonTreeDecoder.decodeSerializableValue(TreeJsonDecoder.kt:337)
at kotlinx.serialization.json.internal.TreeJsonDecoderKt.readJson(TreeJsonDecoder.kt:26)
at kotlinx.serialization.json.Json.decodeFromJsonElement(Json.kt:186)
at org.ossreviewtoolkit.plugins.packagemanagers.node.PackageJsonKt.parsePackageJson(PackageJson.kt:154)
at org.ossreviewtoolkit.plugins.packagemanagers.node.PackageJsonKt.parsePackageJson(PackageJson.kt:47)
at org.ossreviewtoolkit.plugins.packagemanagers.node.PackageJsonKt.parsePackageJson(PackageJson.kt:45)
at org.ossreviewtoolkit.plugins.packagemanagers.node.Npm.parsePackage$node_package_manager(Npm.kt:270)
at org.ossreviewtoolkit.plugins.packagemanagers.node.Npm$parseInstalledModules$2$1$2.invokeSuspend(Npm.kt:253)
at kotlin.coroutines.jvm.internal.BaseContinuationImpl.resumeWith(ContinuationImpl.kt:33)
at kotlinx.coroutines.DispatchedTask.run(DispatchedTask.kt:101)
at kotlinx.coroutines.internal.LimitedDispatcher$Worker.run(LimitedDispatcher.kt:113)
at kotlinx.coroutines.scheduling.TaskImpl.run(Tasks.kt:89)
at kotlinx.coroutines.scheduling.CoroutineScheduler.runSafely(CoroutineScheduler.kt:589)
at kotlinx.coroutines.scheduling.CoroutineScheduler$Worker.executeTask(CoroutineScheduler.kt:823)
at kotlinx.coroutines.scheduling.CoroutineScheduler$Worker.runWorker(CoroutineScheduler.kt:720)
at kotlinx.coroutines.scheduling.CoroutineScheduler$Worker.run(CoroutineScheduler.kt:707)
11:23:46.236 [DefaultDispatcher-worker-2] ERROR org.ossreviewtoolkit.analyzer.PackageManager - NPM failed to resolve dependencies for path 'src/xxx/package.json': IllegalArgumentException: Element class kotlinx.serialization.json.JsonArray is not a JsonPrimitive
First analysis shows that it happens when trying to deserialize author strings like
["Brandon Alexander<EMAIL_ADDRESS>(https://github.com/baalexander)","Sean Massa<EMAIL_ADDRESS>(http://massalabs.com)"]
Which in fact is a JsonArray, and neither a JsonObject nor a JsonPrimitive.
Section in code is in file PackageJson.kt:138
Expected behavior
Don't know. At least no exception should be thrown.
The model just seems to support one author per package, so how to deal with the situation if two authors are specified?
Environment

ORT version: 32.1.0
Java version: 17
OS: Linux

Additional context
@fviernau did a rework of the code in early September 2024

Does not seem to be a totally strange thing to have more than one author for a package:
https://github.com/baalexander/node-portscanner/blob/master/package.json

Hi all,
It's cool to improve robustness of ORT, but in fact, the package.json of portscanner seems really invalid regarding the specification https://docs.npmjs.com/cli/v10/configuring-npm/package-json#people-fields-author-contributors
author is a string, contributors is an array.
I wonder if it's a good approach to be more permissive than the specification.
By the way, the side effect of an invalid package.json somewhere in the node_modules is that it breaks the global analyse of the audited project. We end up with an empty dependency tree with only "NPM::package.json:" as a name...
Perhaps it's more the robustness of the whole analyse process that should be improved than the parsing of package json.
It could be interesting to have an issue when parsing a wrong package but having also the rest of the dependency tree without the falsy packag

I wonder if it's a good approach to be more permissive than the specification.

That's a very good question, but unfortunately also a difficult one to answer. I guess, as so often, we need to find a balance between staying strict to the specs, but accepting (and warning) about very commonly made mistakes. We do something similar for invalid syntax of Maven SCM connection URLs, for example.
We now have several real-life example for authors being arrays of strings, so we should probably tolerate that. But I wouldn't support e.g. an array of objects, for example, as I have yet to see any real-life examples for that.


Small offsets bugfix + Allow for probabilities when merging tokens

If a token was '' then t[0] would have an index error in textutils
Merged token probability is taken bij taking the mean of the probabilities of the subtokens

Merged - thank you!


Support moduleForAcceptance

I see that current support modules are:
const POSSIBLE_MODULES = [
    { expression: { callee: { name: 'module' } } },
    { expression: { callee: { name: 'moduleFor' } } },
    { expression: { callee: { name: 'moduleForComponent' } } },
    { expression: { callee: { name: 'moduleForModel' } } },
  ];

Is there plan to support moduleForAcceptance, so that acceptance tests can also be modified?

Yep, I’m working on another RFC to have acceptance tests to run through ember-qunit / ember-test-helpers (current code for acceptance tests is globals based and provided by ember itself).
I’m generally trying to keep this codemod focused on things supported by ember-qunit (which does not include acceptance tests just yet). I’d hate to migrate the current moduleForAcceptance tests to another format before the RFC is proposed with their final setup.
I’ll try to chime in here with a link to the RFC once I have it proposed.

Submitted the RFC over here https://github.com/emberjs/rfcs/pull/268

RFC is in final comment period.  Hopefully we can land it in the next couple of weeks...

@rwjblue ^^

https://github.com/rwjblue/ember-qunit-codemod/pull/73


Added has_auth2 Contract API

Added a new contract interface has_auth2 which mimics the same pattern of require_auth and require_auth2.
The reason for adding the interface is for contracts to be able to check a specific permission without first calling require_auth2 and then cathing the exception thrown.

This looks very useful.  Please resubmit/rebase your pull request against the develop branch.

@jgiszczak Thanks for the comment. I thought they develop directly on the master branch hahah. I've submitted a new one #5385. Closing this one right now.


need offline-first example

hi @michaelcontento ,
thanks for you work. but could you give some example project ? not only the code in docs.

Duplicate of #120


jsonnet: use common to populate options for additional objects

Remove $.values.kubePrometheus and use $.values.common instead.
Next step would be to move jsonnet/kube-prometheus/components/mixin into separate mixin at jsonnet/mixin and use jsonnet/kube-prometheus/lib/mixin.libsonnet instead of jsonnet/kube-prometheus/components/mixin/custom.libsonnet
All this would increase code reusability and allow easier configuration.

@paulfantom you need to push the README update as it uses the jsonnet example.

ah, right :facepalm:


Update FFI chapter

Update docs based on changes in https://github.com/hyperledger/iroha/pull/2625 and https://github.com/hyperledger/iroha/pull/2837

addressed in #205 and #209


Add better copy to Create a Store

I am sure there's at least something we can do in terms of copy that can explain user what the store is to avoid ambiguity.

Let's discuss in tomorrow's design call.

Opted to add the one-line we currently use on a First Store creation.


Add initial OpenCensus tracing support.

Addresses #2456

I can't repro the coverage failure:
bazel test //test/extensions/tracers/opencensus:tracer_test
[...]
//test/extensions/tracers/opencensus:tracer_test                (cached) PASSED in 0.9s

@g-easy to repro the coverage failure locally, you can use Docker via ./ci/run_envoy_docker.sh './ci/do_ci.sh bazel.coverage'. Looking at the CI log, the issue is in OpenCensusTracerTest.Span`, i.e. it doesn't look like an unrelated CI flake (which we have too many of :/).

@rnburn @objectiser do you have folks have any cycles to look at the tracing extension integration aspects? You have been nominated as the tracing brains trust :)

Isn't the OpenCensus library using it's own threading and sockets to send out the spans?
When we wrote the other tracing libraries we had this requirement that they work within Envoy's worker threads:

We decouple framing vs. transport. Meaning, the libraries should be written in a way that we can still use Envoy per thread networking and not have any connection pools/threading/etc. inside the libraries. If the libraries implement a default transport that's fine, but they should be written in a way that allows us to use our own transport.

Is that no longer the case?

@rnburn yeah, we had this discussion in another thread with @g-easy. We decided to proceed with using libcurl for now and then move towards a more performant model aligned with Envoy's I/O and threading later.

I'm still looking into the bazel.coverage problem...

If @rnburn @objectiser are good with the tracing side, and the coverage issue is fixed (and we can verify the new code is fully covered), then I'm good to merge this.

Could't this also be implemented using the existing envoy.dynamic.ot interface, that is by implementing an OpenCensus shared object that implements the OpenTracing interface?

@g-easy can you address the point raised by @Oberon00 and merge master? I think we can either punt the cmake_external work or do it here, up to you. Would be great to merge soon as per the e-mail thread! Thanks.

Regarding cmake_external:
bazel/EXTERNAL_DEPS.md says to "Reference your new external dependency in some envoy_cc_library via the name bound in step 1 external_deps attribute." but curl is consumed by @io_opencensus_cpp//opencensus/exporters/trace/zipkin:zipkin_exporter as @com_github_curl//:curl
AFAICT, cmake_external only adds filegroup(name = "all", ...
What do you suggest? Is cmake_external only for direct dependencies of Envoy, or can it also be used for transitive dependencies?

I think I can make cmake_external work.

I need to make some changes to opencensus-proto to fix the "api" ci. I'll get back to this tomorrow.

Could someone help me out with go_build_test? I don't understand what's going wrong here:
$ bazel test @envoy_api//test/build:go_build_test
[...]
GoLink: warning: package "github.com/census-instrumentation/opencensus-proto/gen-go/trace/v1" is provided by more than one rule:
    @opencensus_proto//opencensus/proto/trace/v1:trace_proto_go
    @opencensus_proto//opencensus/proto/trace/v1:trace_config_proto_go
Set "importmap" to different paths in each library.
This will be an error in the future.
type.[]*github.com/census-instrumentation/opencensus-proto/gen-go/trace/v1.Span: relocation target type.*github.com/census-instrumentation/opencensus-proto/gen-go/trace/v1.Span not defined
type.[]*github.com/census-instrumentation/opencensus-proto/gen-go/trace/v1.Span: undefined: "type.*github.com/census-instrumentation/opencensus-proto/gen-go/trace/v1.Span"
GoLink: error running subcommand: exit status 2
Target @envoy_api//test/build:go_build_test failed to build

@g-easy It is likely due to different protoc/proto-gen-go generated pb.go files are incompatible.

Sorry for the delay, but I'm stuck on the go_build_test. Can someone help me out, please? I don't know what's going on or how to resolve it.
Envoy API pulls in some Prometheus protos, but supplies its own BUILD file. I can't do that in the opencensus-proto case because it's a whole directory tree with multiple BUILD files.

@lizan @kyessenov can you folks help out @g-easy with this? I'm not a Go person, so I reckon you might have more context here.

@g-easy data-plane-api already pulls opencensus with custom BUILD files here:
https://github.com/envoyproxy/envoy/blob/master/api/bazel/repositories.bzl#L269
You can remove the custom build file since I see now that there opencensus has generated go files and BUILD files. You would need to bump up the dependency on opencensus, and change existing dependency. I only found one use of trace_model:
https://github.com/envoyproxy/envoy/blob/d841d20b101584786d4d44190a78da313768b146/api/envoy/service/trace/v2/BUILD#L12

@kyessenov My PR already removes the custom build file like you said: https://github.com/envoyproxy/envoy/pull/5387/files#diff-94a50a1514c498b6dda210725f5af736L32
Per https://github.com/envoyproxy/envoy/pull/5387#issuecomment-476988559, I'm no longer able to supply a custom build file, even if I wanted to, because I (transitively) depend on protos in multiple subdirectories. (previously we would grab just one leaf directory and override its build file)
The C++ build works fine, but the CI API test breaks in a go_build_test that I haven't been able to debug. This is where I'm stuck. Any ideas?

@g-easy I spent some time digging into this (and learning about the exciting work of Go linking and proto). It turns out all you need is:
diff --git a/api/envoy/config/trace/v2/BUILD b/api/envoy/config/trace/v2/BUILD
index 7e73732ae..b00f63daf 100644
--- a/api/envoy/config/trace/v2/BUILD
+++ b/api/envoy/config/trace/v2/BUILD
@@ -19,6 +19,6 @@ api_go_proto_library(
     proto = ":trace",
     deps = [
         "//envoy/api/v2/core:grpc_service_go_proto",
-        "@opencensus_proto//opencensus/proto/trace/v1:trace_config_proto_go",
+        "@opencensus_proto//opencensus/proto/trace/v1:trace_and_config_proto_go",
     ],
 )

It turns out this issue was already encountered before in opencensus_proto, see https://github.com/census-instrumentation/opencensus-proto/blob/d5d80953a8c2ff4633087af6933cd152678434bb/src/opencensus/proto/trace/v1/BUILD.bazel#L87

Thanks! I totally missed that!

bazel test //test/coverage:coverage_tests also times out at HEAD.

@g-easy coverage fail is expected, the coverage percentage is below threshold (97.4 < 97.5).
Can you add more tests to cover  tracer_impl.cc? See https://197047-65214191-gh.circle-artifacts.com/0/coverage/coverage.source_extensions_tracers_opencensus_opencensus_tracer_impl.cc.html for the coverage report.

Thanks for the URL, in my repo I'm still hitting a timeout.

@g-easy it seems we're pretty close to getting this merged, do you think you'll be able to sort out the coverage aspects?

@g-easy FWIW, you can also limit the coverage tests locally to a single test via something like COVERAGE_TARGET=//test/extensions/filters/listener/tls_inspector:tls_inspector_test VALIDATE_COVERAGE=false ./ci/do_ci.sh bazel.coverage

Trying to push some commits.

Could someone please re-open this PR?

@g-easy done

envoy-macos failure seems to be unrelated to this PR:
bazel-out/darwin-fastbuild/bin/source/common/common/_virtual_includes/thread_lib/common/common/thread.h:21:53: error: releasing mutex 'mutex_' that was not held [-Werror,-Wthread-safety-analysis]
  void unlock() UNLOCK_FUNCTION() override { mutex_.Unlock(); }

https://dev.azure.com/cncf/envoy/_build/results?buildId=1654

Currently dealing with an ODR violation because opencensus and lightstep both vendor protos from googleapis.

I was able to fix the ODR violation in
env PATH=/usr/lib/llvm-7/bin:$PATH CC=clang CXX=clang++ ASAN_SYMBOLIZER_PATH=/usr/lib/llvm-7/bin/llvm-symbolizer bazel test --config=clang-asan //test/config_test:example_configs_test

By switching lightstep_vendored_googleapis to com_google_googleapis. Unfortunately, the latter doesn't have py_proto_library rules so now bazel.api CI is broken (again).
I'm going to try to get these added to com_google_googleapis.

Next up: use after free in //test/server:server_test - it passes on master but is broken on this branch.

server_test is broken due to the absl upgrade in this PR.

TSAN failure looks like a flake. Can someone re-run it? (I can't)
Log says out of memory: https://228086-65214191-gh.circle-artifacts.com/0/failed-testlogs/test/extensions/filters/network/thrift_proxy/integration_test/test.log

/retest

All tests pass! \o/
@htuch how does this look?

Thanks for the review comments, @lizan!

@g-easy this is awesome and with some preliminary testing I was able to simultaneously send traces to Stackdriver and a local Jaeger. Question re: curl version - is there any reason you've used 7.63.0? There have been a number of CVE's addressed in 7.64 and 7.65. Are there features of OpenCensus that require 7.63.0 or can I raise a PR to bump to the latest curl?

@moderation I think it is just the latest version when this PR is created. Feel free to open a PR to bump it. A quick skim for the CVEs I don't think any affect this use case though.

@lizan is right. Please go ahead and upgrade curl. In OpenCensus, only the Zipkin exporter uses it.


Fix typos

@jessarcher These were all the misspellings on your website.
You can improve with DeepL Write or Grammarly.

Thanks!

Glad to contribute.


How can I use it in vue?

How to deal with 'Uncaught Error: ws does not work in the browser. Browser clients must use the native WebSocket object' when I use it in vue?

Here's a thing, we highly recommend you to use the api/the wrapper on the server side, so you don't have to deal with these.


missing contact

Hi David, i may be wrong to contact you here, but after looking for an email or other sort of messenger, i didnt find you... how can we have a talk.. i am a VCV module dev and Ui designer.. i may have few questions regarding your modules and updates...
thank you very much if you decide to answer me, cheers
have a great day
uriel

I've pm'd you through the official vcv rack forum.


An option to remove outers margins (spaces) in FancyZones

Description of the new feature / enhancement
It would be realy nice to have this feature.

Scenario when this would be used?
Nothing to add, title & description are self explanatory.
Supporting information
No response

Hi. You can set space around zone to 0. Is this what you want?
/needinfo

No. I still want to preserve inner margins. But omit margins from to[p, bottom, right, and left of the screen.

in the Preset Layouts this it is already possible to change the "space around zones".
I would love to use this in the custom layout, too.
Because the borders cannot be aligned with the actual desktop borders.
When it comes to browsers: I want to flick my cursor to the very top of the display and click the tab, but the top of the display is 1px of my background img, instead of my browser. So clicking there will not do anything, and that can be very annoying.
Not sure if this is a different problem, or the same as mentioned above.


grpcio-1.32 fail to open 'Con.proto'

What version of gRPC and what language are you using?
grpcio-1.32.0, grpcio-tools-1.32.0
What operating system (Linux, Windows,...) and version?
windows server 2019  (10.0.17763)
What runtime / compiler are you using (e.g. python version or version of gcc)
python-3.6.8 (but also observed on 3.7.9 and 3.8.5)
Linux nad macOS do not see this issue
What did you do?
import elfi
What did you expect to see?
success
What did you see instead?
self = <grpc_tools.protoc.ProtoFinder object at 0x0000023B38439470>
fullname = 'ConfigParser', path = None, target = None

    def find_spec(self, fullname, path, target=None):
        filepath = _module_name_to_proto_file(self._suffix, fullname)
        for search_path in sys.path:
            try:
                prospective_path = os.path.join(search_path, filepath)
>               os.stat(prospective_path)
E               OSError: [WinError 87] The parameter is incorrect: 'D:\\a\\PsyNeuLink\\PsyNeuLink\\Con.proto'

c:\hostedtoolcache\windows\python\3.6.8\x64\lib\site-packages\grpc_tools\protoc.py:152: OSError

The full CI log can be found at https://github.com/PrincetonUniversity/PsyNeuLink/runs/1093766511?check_suite_focus=true#step:14:169
The error indicates that 'Con' was interpreted as an attempt to stat a console:
Anything else we should know about your project / environment?
D:\a\PsyNeuLink\PsyNeuLink is the working directory
Reverting to 1.31.0 fixes the issue: https://github.com/jvesely/PsyNeuLink/runs/1094860822?check_suite_focus=true

@jvesely Thanks for the report! I'm working on a fix for this to be added to a patch. For the moment, you can set the GRPC_PYTHON_DISABLE_DYNAMIC_STUBS environment variable to true to get working with 1.32.0 in the meantime.


The [Feature Request] Tree component adds the data collection function of the selected node (parent node, child node)

Existing Component
是
Component Name
Tree组件
Description
Tree组件不仅仅用于UI展示.
实际应用中, 需要自动收集所有勾选的节点的数据, 常见的如Id, Value.

You can access tree node data in check-change event (or other events you see fit).


Feature/w4mlcmsmatching

These are the modifications for integrating lcmsmatching W4M tool.

Thanks for this!  This won't work as it is as there are no destinations on the config/job_conf.xml file for the tool and its container. You also need to add the tool in a adequate section in the config/tool_conf.xml file. If you could see the current files and add appropriately there please, I would really appreciate it. Let me know if you have any questions or you need any help. Push it to the same branch so that it is part of this same PR.
I hope to have something soon to automate tool additions (through the ansible-galaxy-tools roles), but this is where we are at the minute.

Sorry, I forgot push the modifications on the config files.

Thanks @pierrickrogermele!!!


I ran the same DeepAr model twice with 50 epochs and got 2 totally different forecast

Description
I have a 15 months worth of data with 5 mins resolution. I trained the model for 12 months and was predicting for 2 days ahead. I ran the same exact model twice and I got completely 2 different results.
estimator = DeepAREstimator(freq = '5min',
                           context_length = 4*24*12,
                           prediction_length = 2*24*12,
                           num_layers = 2,
                           num_cells = 128,
                           scaling = True,
                           trainer = Trainer(epochs= 50, learning_rate = 0.001,hybridize=True)
                           lags_seq = get_lags_for_frequency("5min", lag_ub=14*24*12))

from gluonts.dataset.common import ListDataset

training_data = ListDataset([{"start": df_input.index[0],
                              "target": df_input.Load[:'2022-03-07 04:50:00'],
                              #'feat_dynamic_real': [df_input.Rolling[:'2022-03-07 04:50:00']],
                              'feat_dynamic_cat': [df_input.DofW[:'2022-03-07 04:50:00']],
                              'time_feat': [time_feat_df.DayofWeek[:'2022-03-07 04:50:00'],
                                            time_feat_df.Hour[:'2022-03-07 04:50:00'],] }],
                           freq = "5min")

In order to reproduce the same result, you should avoid using gpu and you should set a seed value in your code:
mx.random.seed(7)
np.random.seed(7)

This was already discussed here

Thanks @kaleming . I went through the link you shared and I asked a couple of questions:
1- based on what you picked mx.random.seed(7), np.random.seed(7). I mean why specifically 7, is it trial and error, or is it related to the data size? And why this would make the results reproducible?
2- When you set num_workers=None, what do num_workers refer to?

When you use .seed(x), x could be any number. 7 is just a example. The seed function is used to save the state of a random function, so that it can generate same random numbers on multiple executions of the code on the same machine or on different machines (for a specific seed value).
num_workers = 0 for exemple, means that the data will be loaded in the main process, without using subprocess.

@kaleming Thanks. So this means that the weights/ (maybe certain variables) initialization in DeepAr is random, and to start from the exact same point every time we train the model, we use .seed(x), correct? My worry is, what if I was unlucky and it starts from a bad random initialization and gets stuck in bad prediction, does that make sense or there is something I am missing?
Also, I'll use mx.random.seed(7), np.random.seed(7). Should I try different numbers or this shouldn't be an issue

Yes, I think you got the point.
One way to improve your training could be increasing the num_batches_per_epoch parameter.
You can get the total number of your dataframe rows and divide by batch_size. This will avoid to get stuck in bad prediction, since it will include more data.

Thanks so much, @kaleming. That makes a lot of sense, I thought I was doing something wrong.
So to make sure I got this clear, what you are saying is to put num_batches_per_epoch = the number of rows in my data frame / the batch_size I choose, correct?

@Msaleh87 Yes, correct. Have a try, see if you get a better result.

@kaleming I am trying it now and will post the results for others' reference once am done, which one do you think makes more sense, bigger batch_size or bigger num_batches_per_epoch? I am currently going through the documentation to figure out the impact of this.
I have 100512 rows and I am trying now: batch_size=1396 and num_batches_per_epoch= 72.

I am also running the another model with batch_size=349 and num_batches_per_epoch= 288, instead of batch_size=1396 and num_batches_per_epoch= 72. To see how this would impact, the training is taking longer for sure. I will post the results for others' reference once they are done.

@Msaleh87  Try to test a higher num_batches_per_epoch and keep the batch size 64 or 32.

@kaleming I will try that too and post the results here. I opened an issue regarding the optional features here: https://github.com/awslabs/gluonts/issues/2284
I went through a lot of discussion regarding these features and the documentation, but still, it wasn't clear and there were no clear examples. I even included all the previous discussions regarding this topic in the issue. I saw one of the discussions was posted by you. I am not sure if it's clear to you, but it would be great if you could take a look at it

So I had 100512 rows of training data, so I tried: batch_size=256 and num_batches_per_epoch = 392, for 10 epochs, each epoch took around an hour and 20 mins to run. Here is my code and results:
import mxnet as mx
mx.random.seed(7)
np.random.seed(7)
from gluonts.time_feature import get_lags_for_frequency
from gluonts.mx.trainer.learning_rate_scheduler import LearningRateReduction

learning_rate = 0.001
scheduler = LearningRateReduction(patience=5, base_lr=learning_rate, objective='min')

estimator = DeepAREstimator(freq = '5min',
                           context_length = 4*24*12,
                           prediction_length = 2*24*12,
                           num_layers = 2,
                           num_cells = 128,
                           #cardinality = 'auto',
                           scaling = True,
                           use_feat_dynamic_real =True, #<---------
                           trainer = Trainer(epochs= 10, batch_size = 32 * 8 ,num_batches_per_epoch= int(3141/8) ,learning_rate = 0.001,hybridize=True, callbacks=[scheduler]), # <------
                           lags_seq = get_lags_for_frequency("5min", lag_ub=14*24*12))

from gluonts.dataset.common import ListDataset

training_data = ListDataset([{"start": df_input.index[0],
                              "target": df_input.Load[:'2022-03-07 04:50:00'],
                              'feat_dynamic_real': [df_input.DofW[:'2022-03-07 04:50:00']], #<---------
                              'time_feat': [time_feat_df.DayofWeek[:'2022-03-07 04:50:00'],
                                            time_feat_df.Hour[:'2022-03-07 04:50:00'],] }],
                           freq = "5min")

I think I might need to run for more epochs over the weekend and see

I also tried, batch_size= 32 and num_batches_per_epoch = 3141, but one epoch estimated time was around 5 hours, so I stopped it.

I also tried: batch_size = 1396 , num_batches_per_epoch= 72, but weekeds/ weekdays feature was added as 'feat_dynamic_cat' not as 'feat_dynamic_real'. Naybe this impacted the results a bit. Also, I ran it for 50 epochs. I also put num_samples to 100 and that's why some of the variability in the results are missing.
Here are the results:

Is this the exact same forecast now? Does it also have ay influence on the metrics or just on die graphic? Is it worng when the metrics are a little different?


Increase runTests.py batchsize to make use of more cores.

Copyright and Licensing
Please list the copyright holder for the work you are submitting (this will be you or your assignee, such as a university or company): Columbia University
By submitting this pull request, the copyright holder is agreeing to license the submitted work under the following licenses:

Code: BSD 3-clause (https://opensource.org/licenses/BSD-3-Clause)
Documentation: CC-BY 4.0 (https://creativecommons.org/licenses/by/4.0/)

@seantalts, know why the diff is over one hundred files?
I would have thought only one file actually changed.

Yeah, I think in the process of bringing the new box online the clang
format step went haywire. On my way home now and will take a look soon
On Thu, Jul 12, 2018 at 17:40 Daniel Lee<EMAIL_ADDRESS>wrote:

@seantalts https://github.com/seantalts, know why the diff is over one
hundred files?
I would have thought only one file actually changed.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/stan-dev/math/pull/938#issuecomment-404659482, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAxJ7PwaTtv60B4jVT9h1-nn9Glcj0btks5uF8I-gaJpZM4VNp--
.

--
Laboriously extracted via diminutive keyboard

So I'm setting up a new Jenkins agent and got the clang-format version on it wrong (apparently we need version 5.0 if it's ubuntu, 6.0 if it's on a Mac, hah). Fixed this on develop too. Nothing was broken, but there were a couple of new fixes in 6.0-1ubuntu2 or whatever version that was.


Update template with reason-react-native

It seems @dawee is on it according to discord.
@dawee raise the hand if you need help on this.

On the template not specially. I had this error on windows I told you about but for now I'm going to pu this aside and do my tests on my Linux machine.

Do not hesitate to open a WIP PR if you want. I would like to have the template next week so we can start publising alpha of reason-react-native with the template that goes with it :)

@MoOx @dawee Should this (and its directory) be renamed reason-react-native-template?

@MoOx @dawee Should this (and its directory) be renamed reason-react-native-template?

If I'm not mistaken I think react-native init --template foo is looking for a package with the prefix "react-native-template", so "react-native-template-foo". I think that's why it's called "react-native-template-reason", so you can call react-native init --template reason.

@dawee Thanks for the reply. I was hoping the directory could look neater. I wasn't aware of this detail (and I've never used a template), but you're right and there don't seem to be workarounds.

We could name the folder reason-react-native-template, and keep  react-native-template-reason on npm (in the package.json), but maybe this will create more confusion that it brings as benefit?

react-native-template-reason 0.2.0 is out :)
https://reasonml-community.github.io/reason-react-native/en/blog/2019-06-17-hello-reason-react-native/