Does pgsync support postgres window functions, particularly row_number?

PGSync version:
2.2.0
Postgres version:
13
Elasticsearch version:
7.16.1
Redis version:
6.2.4
Python version:
3.9
Problem Description:
Is it possible to set up the schema.json so that it generates SQL with a window function, particularly row_number(). Basically what I'm trying to do is the following:
SELECT
	innerq.product_id,
	innerq.product_name,
	innerq.product_isbn,
	innerq.alert_id,
	innerq.alert_message
FROM (
			SELECT
				p.id as product_id,
				p.name as product_name,
				p.isbn as product_isbn,
				pa.id as alert_id,
				pa.message as alert_message,
				row_number() OVER (PARTITION BY p.id ORDER BY pa.created_on DESC) AS rownum
			FROM
				product p
				LEFT JOIN product_alerts pa ON pa.product_id = p.id) innerq
WHERE
	innerq.rownum = 1;

The result set would basically be a set of products with their latest product_alert message.(This is only an example that matches my use-case)
Error Message (if any):

Sorry about the delay. window functions are not supported.
Can you give me a concrete use case and what you are aiming to do?


[AIRFLOW-1950] Make xcom_pull's task_ids parameter optional

Make sure you have checked all steps below.
JIRA

[X] My PR addresses the following Airflow JIRA issues and references them in the PR title. For example, "[AIRFLOW-XXX] My Airflow PR"

https://issues.apache.org/jira/browse/AIRFLOW-1950

Description

[X] Here are some details about my PR, including screenshots of any UI changes:

Changes the task_ids parameter of xcom_pull from required to optional.
The task_ids parameter has always allowed None, but since it's been a required parameter, we've been forced to pass it.
ti.xcom_pull(task_ids=None, key='foo')

With this patch, we're no longer forced to pass task_ids.
ti.xcom_pull(key='foo')

Tests

[X] My PR adds the following unit tests OR does not need testing for this extremely good reason:

Adds tests.models.TaskInstanceTest.test_xcom_pull.
Commits

[X] My commits all reference JIRA issues in their subject lines, and I have squashed multiple commits if they address the same issue. In addition, my commits follow the guidelines from "How to write a good git commit message":

Subject is separated from body by a blank line
Subject is limited to 50 characters
Subject does not end with a period
Subject uses the imperative mood ("add", not "adding")
Body wraps at 72 characters
Body explains "what" and "why", not "how"

[X] Passes git diff upstream/master -u -- "*.py" | flake8 --diff

Codecov Report

Merging #2902 into master will decrease coverage by <.01%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master    #2902      +/-   ##
==========================================
- Coverage   73.27%   73.26%   -0.01%
==========================================
  Files         160      160
  Lines       12196    12196
==========================================
- Hits         8937     8936       -1
- Misses       3259     3260       +1

Impacted Files
Coverage Δ

airflow/models.py
86.95% <100%> (-0.05%)
:arrow_down:

Continue to review full report at Codecov.

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

Can we update the PR description to link to AIRFLOW-1950 instead of AIRFLOW-XXX?
Are there any existing tests we could modify to show this working? Or, absent that, could we add a test that exercises this change?

Done and done @gwax 👍

Please squash and adjust your commit message per guidelines (it needs to refer to a JIRA ticket) and explain what/why.

Ok that's done @bolkedebruin.


Add "Core Deterministic Encoding Requirements" option.  This fixes CTAP2 "canonical CBOR" != RFC 7049 "Canonical CBOR" sorting for map keys of mixed types.

CTAP2 "canonical CBOR" != RFC 7049 "Canonical CBOR" when map keys have different data types due to sorting rules.  If all map keys have same data type, there's no difference.
It also appears the result of using CTAP2's "canonical CBOR" is the same as the newer and more streamlined rules in "Core Deterministic Encoding Requirements".
When "Canonical CBOR" is specified, this library has been using sorting rules from "Core Deterministic Encoding Requirements" making it work with CTAP2.
To resolve this, I propose:

fix the "Canonical CBOR" option by using the older and less efficient sorting rules
add "Core Deterministic Encoding Requirements" as an option since it obsoletes the older "Canonical CBOR"

If CTAP2 produces different results than "Core Deterministic Encoding Requirements" then add a CTAP2 option too.
Before this issue is closed, update README unless this is a quick fix.
After this issue and issue: #42 are closed, update README to say something like:

For encoding, it also supports "Core Deterministic Encoding Requirements" as an option, which is newer, more efficient, and obsoletes "Canonical CBOR" encoding. The only difference is the sorting rules affecting sort order when keys have different data types.

It seems CTAP2 is the way it works now for "Canonical CBOR" mode.  I can add CTAP2 as an encoding option along with Canonical mode, and then modify Canonical mode to use "length-first map key ordering".
Let's split this into 2 issues:

bug fix by adding CTAP2 mode and fix sorting for "Canonical CBOR",
enhancement by supporting "Core Deterministic Encoding Requirements", which requires issue #42.

This would let me have v1.3.1 release with just this bug fix and issue #41.

FWIW at least one popular library has this same bug in their latest release and also latest commit.


refactor: reduce scope of internal keys

Reduce the scope of internal keys #334

Hi @tabokie PTAL


SAMZA-2110 : Support for deleting records from a table

Commit adds support for deleting records from table by adding a special op field. SQL users can use this field to say whether the row needs to be deleted in the output table. This is a stop mechanism till we support DELETE operation in SQL.
Adds support for pluggable RelProviders to add the special fields key. This is useful if the relProvider knows the type of key and wants to set it explicitly.
Removes the hack where field positions are incremented by 1 in RelSchemaConverter to support the special field key. With this change the knowledge of the special fields exists only in queryTranslator. In future we could consider moving this logic to RelSchemaProvider.

@atoomula

delete events from cc streams. eg: "select * from brooklin.stream", "select f1, f2 from brooklin.stream"

With this new model there needs to be separate sql statement for delete. I think this model may be better longer term, because When we have delete from support. users need to write two sql statements on for propagating inserts and updates and other one for deletes. This model is inline with that.

Non-explicit delete events. eg: "select f1, f2 from kafka.topic". In this case both f1 and f2 could be null. Deleting such an event from destination store is unclear.

When user does select f1, f2 and when both f1 and f2 are null, it is possible that user wants us to insert a record with each fields being null.  Assuming that the behavior in this case should be delete is non-intuitive and also problematic because we may never be able to support the above requested behavior.


Question about calculation of throughput.

Q1

Here we have throughput(522.605) much greater than the number of PEs(256).
How could that be possible since we know that the upper bound of throughput should be the same as the number of PEs (one cycle for a PE to produce one caculation)
Q2
Could you please explain the calculation of run time since it's critical for the calculation of throughput?
What are the meanings of num_case_occurences and outstanding_delay?
results->UpdateRuntime(results->GetRuntime(CA::EstimationType::Exact) + num_case_occurrences * outstanding_delay, CA::EstimationType::Exact);

@zhangxj19, would you be able to provide the dataMapping (.m) file and hw architecture file/command line?


Dagger Hilt support KSP

Dagger Hilt support KSP
Upgrade API

Hey @KishanViramgama your PR is not correct.
You have to just create a new branch from develop do the changes there and push it, but in this PR you have also merge develop into master which is not needed at all.
I will suggest you to do the PR again.

Hey @KishanViramgama your PR is not correct.
You have to just create a new branch from develop do the changes there and push it, but in this PR you have also merge develop into master which is not needed at all.
I will suggest you to do the PR again.

@aritra-tech Okay i understand but I have never seen in this repository you can use development branch and send the changes ☺


pixiv: Use more accurate login error messages

Looks like pixiv sometimes requires solving a captcha to log in. No requests are performed before a captcha box appears (and it only appears after clicking "login"), so detecting it doesn't seem like a simple task.

Can you detect the captcha and modify the prompt function to support captcha input?

Can you detect the captcha

I tried doing that at first. Looks like I've missed something, since it can be actually detected easily. I'll add that.

and modify the prompt function to support captcha input?

It uses Google's reCAPTCHA, so I'm not sure how a text-only prompt could work with it (other than perhaps "log in manually and paste your cookies here").

Could you download the audio file, and switch to solving the audio captcha?

You don't necessarily need to make something that's solvable at the command line, fwiw. Just provide all of the information you have to prompt and let the user of the library (be that the pass-rotate CLI or something else) sort it out. Then for the CLI, if you can't present the user with a means of solving it, detect the captcha case and throw an error.


Refactor to have proper order of LHS panel

MINI refactor of schema-analysis-plugin. I was making things unnecessarily complicated by consolidating 2 variables in module syntax. This consolidates everything to 1.
export type ModuleDeclaration = PossibleCardClass | BaseDeclaration;

ModuleDeclaration

exported function or class
exported or unexported card or field (PossibleCardClass)

Note: Within the schema-analysis tool. There is some mutating of state going on particularly when traversing the field decorators within a card. In doing so populating possibleFields of a card. I didn't want to touch this so much to maintain backward compatibility of whatever is consuming module syntax. I made a ticket for this https://linear.app/cardstack/issue/CS-6096/refactor-schema-analysis-plugin

Bleurgh broke something.


Multiple output instead of one during multi-class clustering

Hello all,
I modified Denny's code for TF 1.0 and multiclass classification following this:
https://github.com/jiegzhan/multi-class-text-classification-cnn
Everything' s fine, output classification looks good. But instead of printing ONE category (the best fitting my input to classify), I would like to print the "weight" of every category. I mean, eg in positive/negative sentences classification, I want to print the probability for a sentence being positive or being negative.
"""Step 2: compute the predictions"""
	graph = tf.Graph()
	with graph.as_default():
		session_conf = tf.ConfigProto(allow_soft_placement=True, log_device_placement=False)
		sess = tf.Session(config=session_conf)
		with sess.as_default():
			saver = tf.train.import_meta_graph("{}.meta".format(checkpoint_file))
			saver.restore(sess, checkpoint_file)
			input_x = graph.get_operation_by_name("input_x").outputs[0]
			dropout_keep_prob = graph.get_operation_by_name("dropout_keep_prob").outputs[0]
			predictions = graph.get_operation_by_name("output/predictions").outputs[0]
			batches = data_helper.batch_iter(list(x_test), params['batch_size'], 1, shuffle=False)
			all_predictions = []
			for x_test_batch in batches:
				batch_predictions = sess.run(predictions, {input_x: x_test_batch, dropout_keep_prob: 1.0})
				all_predictions = np.concatenate([all_predictions, batch_predictions])
	if y_test is not None:
		y_test = np.argmax(y_test, axis=1)
		correct_predictions = sum(all_predictions == y_test)
		print("Total number of test examples: {}".format(len(y_test)))
		print("Accuracy: {:g}".format(correct_predictions/float(len(y_test))))
		for j in range(0,len(all_predictions)):
			print(test_examples[j]['consumer_complaint_narrative'])
			print("My prediction is: \t" + labels[int(all_predictions[j])])
			print('Label index: \t\t' + str(int(all_predictions[j])))
		logging.critical('The accuracy is: {}'.format(correct_predictions / float(len(y_test))))

Maybe I missed something, but this is my first project using Tensorflow and I really spent time looking for an answer .. Thanks !

You can take the scores to calculate the probability using softmax function. Please have a look at my implementation to get the probability of each class:
eval.py


Vagrant: Explore adding a Windows 10 Vagrantfile

Spotted by @sxa , Microsoft offers Vagrant boxes at https://developer.microsoft.com/en-us/microsoft-edge/tools/vms/
Some preliminary testing has show that it's fairly easy to retrieve the box and boot the Vagrant box by doing the following:
wget https://az792536.vo.msecnd.net/vms/VMBuild_20190311/Vagrant/MSEdge/MSEdge.Win10.Vagrant.zip
unzip MSEdge.Win10.Vagrant.zip
vagrant box add --name windows10 'MSEdge - Win10.box'

# Change in the Vagrantfile:
# adoptopenjdkW2012.vm.box = "mwrock/Windows2012R2"  --> adoptopenjdkW2012.vm.box = "windows10"

vagrant up

Some small changes will need to be made to VPC to let this run, as the box doesn't have a vagrant user and the password is Passw0rd! , not vagrant.

My mistake, just directly transferring windows10 into the adoptopenjdkW2012.vm.box field, doesn't work out of the box.
    adoptopenjdkW2012: WinRM address: <IP_ADDRESS>:55985
    adoptopenjdkW2012: WinRM username: IEUser
    adoptopenjdkW2012: WinRM execution_time_limit: PT2H
    adoptopenjdkW2012: WinRM transport: negotiate
Timed out while waiting for the machine to boot. This means that
Vagrant was unable to communicate with the guest machine within
the configured ("config.vm.boot_timeout" value) time period.

If you look above, you should be able to see the error(s) that
Vagrant had when attempting to connect to the machine. These errors
are usually good hints as to what may be wrong.

However, once vagrant up has timed out, I am still able to run vagrant winrm -c commands, meaning the machine is still booting and the winrm connection is working as intended. However, it is taking long than I have seen vagrant winrm commands take before.
i.e:
$ time vagrant winrm -c "ipconfig"

Windows IP Configuration

...

real	0m16.959s
user	0m1.856s
sys	0m0.255s

On the off-chance that vagrant up is just timing out, I'm going to try bumping the timeout up to 10 minutes (default is 4 minutes).

Oh..... yeah, increasing the timeout worked! That's surprising. This is the Vagrantfile I managed to get it working with:
# -*- mode: ruby -*-
# vi: set ft=ruby :

# Runs powershell as an administator and gets/executes an Ansible provided script that configures WinRM to allow Ansible to communicate over it. Then places a file in the shared folder of the VM that contains the IP address of the VM.
$script = <<SCRIPT
Start-Process powershell -Verb runAs
[Net.ServicePointManager]::SecurityProtocol = [Net.SecurityProtocolType]::Tls12
wget https://raw.githubusercontent.com/ansible/ansible/devel/examples/scripts/ConfigureRemotingForAnsible.ps1 -OutFile .\\ConfigureRemotingForAnsible.ps1
.\\ConfigureRemotingForAnsible.ps1 -CertValidityDays 9999
.\\ConfigureRemotingForAnsible.ps1 -EnableCredSSP
.\\ConfigureRemotingForAnsible.ps1 -ForceNewSSLCert
.\\ConfigureRemotingForAnsible.ps1 -SkipNetworkProfileCheck
# Retrieving disk's current size
$currentDiskSize =(Get-Partition -DriveLetter c | select Size)
$currentDiskSize =($currentDiskSize -replace "[^0-9]" , "")
# The size the disk should be, in bytes (95GB)
$diskSizeBoundary =<PHONE_NUMBER>80
# Changing the disksize to max supported size (~100GB)
if ([long]$currentDiskSize -lt $diskSizeBoundary) {
        echo "Resizing disk to max size"
        $size = (Get-PartitionSupportedSize -DriveLetter c); Resize-Partition -DriveLetter c -Size $size.SizeMax
}else {
        echo "Disk is already at max size"
}
Start-Process cmd -Verb runAs
winrm set winrm/config/service '@{AllowUnencrypted="true"}'
SCRIPT

# 2 = version of configuration file for Vagrant 1.1+ leading up to 2.0.x
Vagrant.configure("2") do |config|

  config.vm.define :adoptopenjdkW2012 do |adoptopenjdkW2012|
    adoptopenjdkW2012.vm.box = "windows10"
    adoptopenjdkW2012.vm.hostname = "adoptopenjdkW2012"
    adoptopenjdkW2012.vm.communicator = "winrm"
    #adoptopenjdkW2012.vm.synced_folder ".", "/vagrant"
    adoptopenjdkW2012.vm.network :private_network, type: "dhcp"
    adoptopenjdkW2012.vm.provision "shell", inline: $script, privileged: false
    #adoptopenjdkW2012.disksize.size = '100GB'
  end
  config.vm.provider "virtualbox" do |v|
    v.gui = true
    v.memory = 2560
    v.customize ["modifyvm", :id, "--cpuexecutioncap", "50"]
  end
  config.vm.boot_timeout = 600
  config.winrm.username = "IEUser"
  config.winrm.password = "Passw0rd!"
end

The important bits are the 3 config lines at the bottom.
Whilst I was waiting for that, I searched a bit more, and it was suggested that a Firewall on the VM can cause timeouts, and the VM does have firewalls up, according to Windows Defender. I'll experiment with turning those off to see if booting works without the incredibly long timeout. If that is the case, I'll have to repackage the Vagrant box like I did with the (now not used) OpenSUSE 12.3 box

I've made a branch that has added the Vagrantfile, along with some changes to the various scripts in VPC - these are the changes I can see initially see, but I imagine that there's going to be others.
I will need to add the Vagrant box to the Machine first, as the Vagrantfile assumes that (Though, maybe this could be added to VagrantPlaybookCheck.sh` ... hmm ...).
I'll run VPC once I've put that in :-)

https://ci.adoptopenjdk.net/job/VagrantPlaybookCheck/OS=Windows10,label=vagrant/1016/
The first instance of the vagrant machine booting, properly. having added the Vagrant box properly. I had to do this as each user has their own copy of the vagrant box, and it was more efficient to put it in VPC then it is to manually set them up. (also helps if anyone wanted to run Windows 10 on VPC locally)

I've come to the conclusion with multiple VPC runs, that if we want Windows 10, we shouldn't be using the MCRSFT one, as it's just too slow to boot and connect. Not sure why that is, honestly, but I have  found a few others at https://app.vagrantup.com/boxes/search?order=desc&page=1&provider=&q=windows&sort=updated&utf8=✓ , so I'll test those.

https://ci.adoptopenjdk.net/job/VagrantPlaybookCheck/1041/OS=Windows10,label=vagrant/console
It works, using https://app.vagrantup.com/StefanScherer/boxes/windows_10 !
I'll let the playbook run through to determine if there's any issues.

I have managed to get the WinPB running up to MSVS_2019, without any issues, but it's painfully slow on my laptop (possibly due to my having to reduce the amount of memory it uses). I'll have to run a few VPC runs to double check it works okay.

Okay - I couldn't get a single working VPC run, without it timing out, so I'll search for a different machine.
I've found https://app.vagrantup.com/peru/boxes/windows-10-enterprise-x64-eval
however, on booting, I get the following error message:
#<Vagrant::Errors::VBoxManageError: There was an error while executing `VBoxManage`, a CLI used by Vagrant
for controlling VirtualBox. The command and stderr is shown below.

Command: ["modifyvm", "f32d833e-959f-4491-b172-dc254b2c83f3", "--clipboard-mode", "bidirectional"]
...
VBoxManage: error: Unknown option: --clipboard-mode

This is due to my laptop having VirtualBox v6.0 on it, not 6.1. I'd rather not bump the requirement up to 6.1 for VPC stuff, if I can avoid it (considering we know all our other VFs work with 6.0), but it is an option.

The overarching issue I've found with Windows 10 boxes, is that they all seem to be very large and are difficult for Vagrant to boot, and if they're not, they are configured in a quirky way - such as the comments above. For now, I'm going to leave this while I figure out an alternative to a Vagrant Box (i.e. build my own box ... )

Based on Will's experience this may be a difficult task, but adding good first issue in case someone wants to take it on. It should be possible for anyone with a machin able to run vagrant to at least take a look at it.

We probably want this to be Windows 11 now


How to export table with css style?

Is there any way to export html table with css style?

I have been checking this for a while but I think this feature is not implemented  and the creator is asking for help in order to implement this (see issue #12 ). However there is another JS plugin/lib called "bootstrap-table"  that have an exporter extension with formatting.

Same issue :(


Update Stock Bug Fix Modules & StockPlus

This is my first time editing NetKAN.
The name of the zip, and folder within, changed from "StockBugFixPlus" to "StockBugFixControllerPlus". Should it be handled as I have done, or should the netkan file be renamed?

We certainly don't want to rename the netkan or change the identifier. This looks fine to me. Merging with thanks!


Adding java support information

docs: change README.md. Closes #346 (well, it's already closed :stuck_out_tongue:)

Ops, you've already did it!


Migrate pallet-democracy to pallet attribute macro

Part of #7882.
Converts the Democracy pallet to the new pallet attribute macro introduced in #6877.
Following the upgrade guidelines here: https://crates.parity.io/frame_support/attr.pallet.html#upgrade-guidelines.
⚠ Breaking Change ⚠
From https://crates.parity.io/frame_support/attr.pallet.html#checking-upgrade-guidelines

storages now use PalletInfo for module_prefix instead of the one given to decl_storage: Thus any use of this pallet in construct_runtime! should be careful to update name in order not to break storage or to upgrade storage (moreover for instantiable pallet). If pallet is published, make sure to warn about this breaking change.

So users of the Democracy pallet must be careful about the name they used in construct_runtime!. Hence the runtime-migration label, which might not be needed depending on the configuration of the Democracy pallet.
polkadot companion: https://github.com/paritytech/polkadot/pull/3030

Please review :)

Democracy is the name of the pallet in polkadot/kusama so no need for migration

bot merge


Add timestamp_field to the documentation

Description
I noticed the documentation references timestamp_field in several places, but never actually documents the field itself.
I added a short description of what it does to the documentation.
Checklist

[x] I have reviewed the contributing guidelines.
[ ] I have included unit tests for my changes or additions.
[ ] I have successfully run make test-docker with my changes.
[ ] I have manually tested all relevant modes of the change in this PR.
[x] I have updated the documentation.
[ ] I have updated the changelog.

Questions or Comments
As this changes no actual source code, I have skipped the tests.
I'm not sure if we need to mention this pure documentation change in the changelog?

Thanks for taking the time to add this missing piece. Yes, please update the CHANGELOG.md. I prefer to over communicate, even if it's a small doc change.

I've added it to the changelog and merged in the current master. Should be good to go now.


IWorld -> WorldGeneratable

https://discordapp.com/channels/507304429255393322/521545796882006027/540468184546017280
As I see, the IWorld thing is usually used in off-thread world generation. For example, the method that initializes entities on spawn takes an IWorld instead of a World because the entity may be spawned in a structure during world generation. This thing is not world after all, but is something suitable for world generation, hence WorldGeneratable instead of GeneratableWorld.

I disagree.
For world generation, all you really need is getBlockState, setBlockState, getBlockEntity, getHeight and maybe some entity spawning if you include entities in world generation.
This class does far more than that and really is a more or less general interface with almost everything World needs to do.
It's also used all over the Block methods, as an argument, with many of them taking a superclass of it with less functionality (and one of the implementations is a class used to access blocks from chunk batcher threads).
This interfacee also could allow for a world-generation-specific implementation of IWorld that could be specifically optimized for it (for example to avoid going through ChunkManager, and instead access chunks directly from an array), but world generation is jus one of the many places where it's used.

no. you need a lot of methods to check environments for world generation, including persistent data access. if what you said is right, please explain that getWorld method in that interface

Still doesn't change the fact that its used for a lot more things and it also has methods that aren't used for world generation at all.

that is because mojang probably names it badly internally and devs start to misunderstand

Hmm, if we rename IWorld to World, what would the current World become? The main difference then is that World is sync and IWorld can be async.

The main difference is that World is an abstract class that partially implements IWorld, while IWorld is a more general interface.
If World can't be named BaseWorld or AbstractWorld, ths current naming is probably the best you can get.

Someone over in the discord recommended changing it to ProtoWorld, based on the official Mojang name ProtoChunk.

Proto implies a temporal relationship ("first version"/replaced later) instead of inheritance (derived/base still around). In the case case of "ProtoChunk", this kind of Chunk should be what is being used during world gen then replaced with a new Chunk instance once finished. If that's not the case Mojang's name choice is not good.
AbstractWorld should be reasonable for the base impl class if it's not taken, freeing World for the interface. This is the standard Java naming convention, I'd even go as far as using Abstract* for any base class that will be used with sub classes exclusively, even if it lacks the "abstract" modifier - the behavior is what matters.

I wouldn't be surprised if ProtoChunk/ProtoWorld are the interfaces which Mojang is trying to replace all usages of Chunk/World across the codebase with, eventually.

Closing. Will not work on this recently.


Smarthub: renaming Smarthub to Attekmi

🏷 Type of documentation

[x] text edit only (wording, typos)

📋 Checklist

[x] Related pull requests in prebid.js or server are linked -> Paste link in this list or reference it on the PR itself

We change the name from Smarthub to Attekmi  but leave the biddercode 'smarthub'
Prebid.js PR https://github.com/prebid/Prebid.js/pull/12432
Prebid-server PR https://github.com/prebid/prebid-server/pull/4047

I agree with Patrick here. Renaming adapters is common, but can be confusing to publishers who can't find documentation for the name they're familiar with. The smarthub.md file can say "please note that SmartHub has been renamed to Attekmi" or something like that.

I added attekmi.md and add note about rebranding in smarthub.md


Address validation

What's this PR do?
Fixes bug on /user/:uid/info form that validates address correctly and doesn't throw an error if address is not present (since it is not required).
How should this be manually tested?
Visit  /user/:uid/info and:

delete address - should save profile.
add address and input z as the zipcode - should throw error.

What are the relevant tickets?
Fixes #6317

@chloealee I think we already handled the two different types of address in here

@angaither should I change this line to use that callback to handle the two different types of addresses here? Looks like right now, it just grabs the form on the page ('page callback' => 'drupal_get_form').

@angaither hmm I tried that and it didn't seem to work, I'm not sure I understand how to incorporate that callback to this scenario?

@chloealee you shouldn't have to change these to handle both the form_address_field and the manually created user_address without all the language none stuff
I would say you can take a look at how addresses are validated in the shipment form, or the signup data form
you can

update the dosomething_user_validate_user_address function to handle both JS and form submissions
OR
create your own validation function that is a mix of setting form errors, but using the field the way the user_address is used instead of the LANGUAGE_NONE stuff
OR
refactor what already exists to DRY up the logic out that's the same across both of these things

@angaither also found another bug with the edit form - it looks like it's not sending the updated address to northstar anymore. Should I create a new issue for this?

@angaither separated the two functions out again - ready for another review!

approach looks good, but you've got quite a few extra spaces/files changed that look like they shouldn't be included in this. please clean that up

all set!

:+1:


lgtv-config

Hi, thanks for submitting!
Could you explain how to setup the lgtv-config please, I don't know where to find the 'host' & 'token' details.
Paul

Hi!
The Token isn't meant to be filled manually, this should be auto-filled on first connect to your TV. You just have to put in the Hostname or IP-Address of your LG TV. If DNS is working in your Network the default lgwebostv should be ok, otherwise you would have to find out the IP Address of the TV (possibly you can look it up in your routers web ui) and use this. Then just press the connect button (your TV needs to be switched on for at least ~25s when doing this) and as soon as the Token field got filled you're ready to go.
Don't hesitate to ask if you still experience problems,
Regards,
Sebastian

added setup instructions to the readme


Do not show the hash for an empty kconfig

Closes #82

Is it possible to ever have {.., "kconfig": [], ...} ?

Good question. From what I've seen, the answer is no but I've got a note into @danrue . When he answers I will update this!

@chaws @danrue said it should always be there, but I will put a check in there to gracefully handle it anyways.


Is this project abandoned?

Hi,
I see there were no commits in the last 2 years, is this project abandoned?
Thanks

@luos,
We are no longer maintaining this project.


Unify number formats in documents.

Unify number formats in documents.

@microsoft-github-policy-service agree

#sign-off


Ternary with function call produces "Standalone expression (the line has no effect)" warning

Godot version
4.0 alpha 6, and my own build v4.0.alpha.custom_build [f4b0c7a1e]
System information
Windows 10
Issue description
Gdscript ternaries which lack assignment can produce the warning "Standalone expression (the line has no effect)." Example:
a() if condition else b()
I have confirmed that the correct function is being called, so the line does have an effect. The warning occurs regardless of whether a() or b() is called.
This is similar to https://github.com/godotengine/godot/issues/47088.
Steps to reproduce
func a():
  print("a")

func b():
  print("b")

func _ready():
  a() if true else b()

Minimal reproduction project
No response

I can confirm this on master f4b0c7a1e:

I can confirm this on 3.x 3ba980379 too:

This no longer seems to be an issue.

Seems fixed as early as 4.0-stable.


Update chrono to 0.3

att as the title

Already done in the latest commit, may be published in v0.7.1


Create HelloWorld_FR.php

What does the program do ?
It says Hello World in French
In what programming language it is written?
PHP

Thank you for participating  in hacktoberfest ❣ if you liked working on this repo you can share it with friends and star this project for helping others


[channels,rdpgfx] fix rdpgfx_server_packet_complete_header

a condition check was inverted preventing server side gfx packets to be sent.

tested with shadow server and working.


How to merge (lanelet::ConstLanelets& road_lanelets）into a boost::geometry polygon?   It's extremely urgent. Please help me.

I need to use a vector map to filter the point cloud. Since the map is divided into many segments, the efficiency is a bit low. I want to merge all the small segments into a boost::geometry polygon. However, I've searched through the code but couldn't find any relevant examples, which is really frustrating. My code is as follows. I'm asking for your help. Mainly, please tell me how to obtain the coordinates x and y from road_lanelets[0].polygon2d().basicPolygon() so that I can convert them into the boost::geometry polygon format. Thank you in advance.
Polygon Lanelet2MapFilterComponent::getPolygons(lanelet::ConstLanelets & road_lanelets) {     Polygon resultPolygon;     lanelet::BasicPolygon2d temp_polygon = road_lanelets[0].polygon2d().basicPolygon();     // TODO: change to boost::geometry::polygon      // int size = road_lanelets.size();     // for (int i = 1; i < size; ++i) {     //     boost::geometry::union_(temp_polygon, road_lanelets[i].polygon2d().basicPolygon(), resultPolygon);     // }     return resultPolygon; }

A lanelet basic polygon is in essence a std::vector of Eigen' vectors that is also already registered with boost geometry as a ring. That may already work with boost::geometry::_union`, but I am not too sure as that is more related to boost and less to Lanelet2

I'm really grateful for your reply. Mainly, I want to know how to obtain the x and y coordinates of each point of the lanelet::BasicPolygon2d type. In this way, I can get the coordinates of each point and convert them into boost::geometry::polygon. Could you please help me test how to obtain the coordinates of the points in an object of the lanelet::BasicPolygon2d type? I've actually tried it myself, but I don't know how to operate it, so I'm coming here to ask for help.


Translation issues with code snippets

If I turn on the language extension, then when I use the CURRENT_MONTH_NAME and CURRENT_DAY_NAME of the code snippet, the output time format will be automatically translated into the language setting I use (such as Chinese). But in fact, I may want to output it in the file It is an English month. At present, it seems that it can only be done by switching the entire display language of vscode. Is it possible to add an option to select the language of the code snippet output.
The picture is the code snippet I set in markdown:

This feature request is now a candidate for our backlog. The community has 60 days to upvote the issue. If it receives 20 upvotes we will move it to our backlog. If not, we will close it. To learn more about how we handle feature requests, please see our documentation.
Happy Coding!

This feature request has not yet received the 20 community upvotes it takes to make to our backlog. 10 days to go. To learn more about how we handle feature requests, please see our documentation.
Happy Coding!

:slightly_frowning_face: In the last 60 days, this feature request has received less than 20 community upvotes and we closed it. Still a big Thank You to you for taking the time to create this issue! To learn more about how we handle feature requests, please see our documentation.
Happy Coding!


Support for not released plugins

Sometimes, you want to add a plugin that is not released to npm, but has the form of a local module.
You are able to do that as a function, but not using the new object shaped plugins.
Thanks for the awesome job.

Will not add new features to master, can you redo this against the canary branch? :)

@fivethreeo sure!

@fivethreeo Done: https://github.com/jaredpalmer/razzle/pull/1467


remove numpy and matplotlib - they do not seem to be in use

This removes the requirements of numpy and matplotlib. Both libraries are not used directly in this module, and also searching the issues only returns #39, which simply lists the installed packages.
Why remove them? (besides the obvious "keep the dependencies lean")
While no current package is available on PyPI, I started to setup the package in my netbox-docker container the long way, which means installing it from git. But installing in the container fails at installing matplotlib, as there is no gcc available in the base image (which is the official netbox image).
So I wondered, considering the purpose of the project, what numpy and matplotlib could be used for, and I searched for the usage of the libraries.

I really will not use it by now. Thank you!


Error after Add with SPM - Google Mobile Ads SDK

Can't add on Xcode with SPM.
Xcode 16.0

https://developers.google.com/admob/ios/quick-start
I was following this doc.

Point it at the exact version 11.4.0 for now.
They released hot fix versions only for CocoaPods so if you're trying to pull in those version, 11.4.1/11.4.2 through SPM its not going to be able to locate them: https://firebase.google.com/support/release-notes/ios


Joining of nodes to cluster

i have 3 worker nodes one manager i have joined only 2 worker nodes and forgot to join third one is there any way that i can join my third worker node to cluster environment.

Hi @manoj2306 !!! Please post your queries on #in-dev channel in slack https://kubernetes.slack.com/messages/C56CDJNCS/

/close


AWS Device Farm configuration is NOT VALID (with valid config)

I get this message:
AWS Device Farm configuration is NOT VALID
AWS Device Farm configuration is NOT VALID

(the message appears twice) for the config below (the projectName, accessKey and secretKey are working, I can use them to send tests through the command line):
devicefarm {

  projectName "My Android App"

  authentication {
    accessKey "XXXXXXXXXXXXXXXX"
    secretKey "YYYYYYYYYYYYYYYYYYYY"
  }

}

The devicefarmUpload task is therefore skipped (I'm guessing because the config is considered not valid).

This problem is fixed by #2, but fixed plugin is not seem to be published yet.

Thanks, any idea when it will be published?

Sorry, I don't know when the change will be published.
But you can distribute the newest one by using Bintray.

@r7o is there any chance you could release it? This is a pretty blocking issue. :)

I am also interesting about that issue. It blocked. Is it possible to send test using command line?
I also every time have problem with using deviceState property:
Error:(86, 0) No such property: on for class: com.amazonaws.devicefarm.extension.DeviceState

I'm having the same issue. Any update on a release date?

Hi all, we're working on this now and should have a new build shortly. Thanks for your patience!

We released the latest changes from github as version 1.1: 'com.amazonaws:aws-devicefarm-gradle-plugin:1.1'

Thanks a lot!

I forgot to mention that anybody who was running into
No such property: on for class: com.amazonaws.devicefarm.extension.DeviceState

should now surround their "on" or "off" with quotes in their device state enclosure.
Here's an example:
        // optional block, radios default to 'on' state, all parameters optional
        devicestate {

            extraDataZipFile file("relative/path/to/zip") // default null
            auxiliaryApps [file("path1"), file("path2")] // default empty list
            wifi "on"
            bluetooth "off"
            gps "off"
            nfc "on"
            latitude 47.6204 // default
            longitude -122.3491 // default
        }


bug: cortex models start: Error HTTP error: Failed to read connection

Cortex version
v1.0.0-168
Describe the Bug
Cant run any models from cortex-nightly v165 onwards
Hardware: Silicon Mac M1
Sang reproduced on VM

model freshly downloaded successfully.
successfully in models list eg tinyllama
Errors with cortex run <model> and cortex models start <model>

Resources
cortex-cli.log
20241015 02:47:33.602398 UTC 2385369 ERROR HTTP error: Failed to read connection - model_service.cc:498
20241015 02:47:33.603768 UTC 2385369 INFO  Error: HTTP error: Failed to read connection - model_start_cmd.cc:10

Steps to Reproduce
No response
Screenshots / Logs

What is your OS?

[X] MacOS
[ ] Windows
[ ] Linux

What engine are you running?

[X] cortex.llamacpp (default)
[ ] cortex.tensorrt-llm (Nvidia GPUs)
[ ] cortex.onnx (NPUs, DirectML)

@vansangpfiev adding logs:
cortex-cli.log
cortex.log

@vansangpfiev uninstalled and reinstalled cortex-nightly, new download of tinyllama and getting the same error

cortex-cli.log
cortex.log

Resolved, bizzare bug due to me running iTerm (on Mac M1, but running through Rosetta) where
# iTerm
> arch
i386

whereas cortex runs successfully in terminal where
# terminal
> arch
arm64

For future reference, Cortex only supports (as of now)

windows and linux supports amd64 only
macos support both amd64 (mac intel) and arm64 (mac silicon)

(note: model_yaml error is separate issue that still occurs but model chat works, to be in a separate issue)


UnicodeDecodeError: 'gbk' codec can't decode byte 0x94 in position 6206: illegal multibyte sequence

when i run your code,i got this error,how do i fix it ?Thanks

Add encoding parameter at line 34 and 36 in data_helpers.py like the following:
positive_examples = list(open(positive_data_file, "r", encoding='utf-8').readlines())

ref in stackoverflow

Fixed !

you can fix this by the way of Installing from source
1，Download the latest source tarball from https://github.com/sshwsfc/django-xadmin or git clone the repository. Then execute the following in the project directory:
2，pip install -r requirements.txt

fixed!!!
thanks


test_operator.test_laop_3 has fixed seed that can mask flakiness

The unit test in title have been using fixed seed to mask flakiness. Suggested action:

Evaluate whether the test is flaky without fixed seed. If not, remove seed. Else move to 2
If test is flaky, determine whether it's an actual uncaught edge case. If so, fix the operator. Else move to 3
If numerical instability is inevitable, adjust tolerance level appropriately.

@szha Thanks for filing this issue. We will investigate these Flaky tests.

The issue can be reproduced:
MXNET_TEST_SEED=333426306 nosetests --logging-level=DEBUG --verbose -s tests/python/unittest/test_operator.py:test_laop_3
AssertionError:
Items are not equal:
Error 2056487.529996 exceeds tolerance rtol=0.010000, atol=0.010000.  Location of maximum error:(0, 10), a=20571.134836, b=0.000304
NUMERICAL_data1: array([[-20571.10127895, -20571.16447819, -20571.15281231, ...,
-20571.16473405,  20571.14681937, -20571.09714863],
[-20571.16447819, -20571.14299957, -20571.0931439 , ...,...
BACKWARD_data1: array([[ 0.03820337, -0.02174861, -0.01172468, ..., -0.03253193,
0.00363456,  0.04317206],
[-0.02174861, -0.01322614,  0.03689246, ..., -0.01726587,...

@mxnet-label-bot [Good First Issue]

Is this related?
http://jenkins.mxnet-ci.amazon-ml.com/blue/organizations/jenkins/mxnet-validation%2Fcentos-cpu/detail/PR-15625/1/pipeline
Please include test_laop_6 in the title if so.

Same
In PR #15606
http://jenkins.mxnet-ci.amazon-ml.com/blue/organizations/jenkins/mxnet-validation%2Fwindows-cpu/detail/PR-15606/4/pipeline

Another one for test_operator.test_laop_6
#15736
Pipeline - http://jenkins.mxnet-ci.amazon-ml.com/blue/organizations/jenkins/mxnet-validation%2Funix-cpu/detail/PR-15736/1/pipeline

Another one
http://jenkins.mxnet-ci.amazon-ml.com/blue/organizations/jenkins/mxnet-validation%2Fwindows-gpu/detail/PR-15807/2/pipeline
#15807
test_operator.test_laop_6 ...
Error running unittest, python exited with status code C0000005
At C:\jenkins_slave\workspace\ut-python-gpu\ci\windows\test_py2_gpu.ps1:27
char:13

if (! $?) { Throw ("Error running unittest, python exited with status ...

        ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

CategoryInfo          : OperationStopped: (Error running u...s code C000

0005:String) [], RuntimeException

FullyQualifiedErrorId : Error running unittest, python exited with statu

s code C0000005

Again #15736
http://jenkins.mxnet-ci.amazon-ml.com/blue/organizations/jenkins/mxnet-validation%2Fwindows-gpu/detail/PR-15736/6/pipeline/

Another #15941
http://jenkins.mxnet-ci.amazon-ml.com/blue/organizations/jenkins/mxnet-validation%2Fwindows-gpu/detail/PR-15941/2/pipeline

laop_3 and laop_6 are different tests, so I would differentiate if possible.


rename package

hi
i want rename package name and custom but when rename s not work
please help me
thanks

The name of the package is hardcoded in the compiled dependencies. To change the name we need to unblock #1 first.


Add pg_hnsw vectorstore integration

Hi @rlancemartin, @eyurtsev!

Description: Adding HNSW extension support for Postgres. Similar to pgvector vectorstore, with 3 differences

it uses HNSW extension for exact and ANN searches,
Vectors are of type array of real
Only supports L2

Dependencies: HNSW extension for Postgres

Example:

  db = HNSWVectoreStore.from_documents(
    embedding=embeddings,
    documents=docs,
    collection_name=collection_name,
    connection_string=connection_string
)

query = "What did the president say about Ketanji Brown Jackson"
docs_with_score: List[Tuple[Document, float]] = db.similarity_search_with_score(query)

The example notebook is in the PR too.

Hi @rlancemartin. Thanks for your feedback!
I removed the logging in the notebook and added a section on how to create HNSW index with create_hnsw_index function.

Hey @rlancemartin, fixed some lint errors that didn't appear locally.

Hi @rlancemartin, @eyurtsev
I apologies in advance, we had to change the name of the extension from hnsw to pg_embedding.  It still uses hnsw index, so no changes there.
I reflected that in the code. I also changed the file name and updated the Jupyter Notebook.

Hi @rlancemartin, @eyurtsev
I apologies in advance, we had to change the name of the extension from hnsw to pg_embedding.  It still uses hnsw index, so no changes there.
I reflected that in the code. I also changed the file name and updated the Jupyter Notebook.

Sounds good. Just kicked off tests. Is everything set to merge if tests pass?

@rlancemartin reformatted with black again.

Please see Lint errors: run "make format" and see unused import and line length errors.

@rlancemartin ran make format and poetry run black . --check
All done! ✨ 🍰 ✨
1625 files would be left unchanged.

🎉


[abseil] support cxx20

Is your feature request related to a problem? Please describe.
Abseil provides polyfills when in C++20 mode for std::result_of. However, they're disabled without a new feature flag.
Proposed solution
Add a feature cxx20, similar to cxx17 which sets CMAKE_CXX_STANDARD to 20.
Describe alternatives you've considered
We could wire CMAKE_CXX_STANDARD from upstream, but not sure how feasible that is.
Additional context
The error when compiling in using cxx17 when everything else is C++20: absl/meta/type_traits.h(622,45): error C2039: 'result_of': is not a member of 'std'.

abseil[cxx17] was added before we released the policy, currently we do not accept adding c/cxx standard as a feature.
See https://github.com/microsoft/vcpkg/blob/master/docs/maintainers/maintainer-guide.md#do-not-use-features-to-control-alternatives-in-published-interfaces

We hope your question was answered to your satisfaction; if it wasn't, you can reopen with more info.


update pagerduty integration

Emergency fix and I manually investigated test failure and verified they were not related to this change.


Replace basetring with six.string_types

Use six.string_types instead of basestring to make it python 2 and 3 compatible.

Hi! Thank you for the patch, looks good.


how can i download and upload csv file

hello
i never use Meteor-CollectionFS
how can i download and upload csv file ?

Same issue #1023


Update example response so it shows nicely

https://buildkite.com/docs/apis/graphql-tutorial

Before
After

Thanks @plaindocs!

Thank you!


Added Cachet

Added Cachet, the open source status page system.

Thanks, this looks great!

Thank you :)

On 24 May 2015, at 17:36, Jamie York<EMAIL_ADDRESS>wrote:
Thanks, this looks great!
—
Reply to this email directly or view it on GitHub.


docs: Fix link to "Custom profile fields".

Follow-up to PR #23264.

Self-review checklist

[x] Self-reviewed the changes for clarity and maintainability

Communicate decisions, questions, and potential concerns.

[x] Automated tests.

Individual commits are ready for review (see commit discipline).

[x] Each commit is a coherent idea.
[x] Commit message(s) explain reasoning and motivation for changes.

Completed manual review and testing of the following:

[x] Links.

@timabbott Quick fix PR ready for you.
@zulipbot add "integration review"

Merged, thanks @drrosa!

Backported to 6.x as 7ad64efc202fa6dce46d81951b85c8d4eb13eab9. Not actually important, since there's a redirect.


Not able to create a new breakpoint after a collapsed function

Problem Description
If a function is collapsed, a new breakpoint cannot be created in the lines below it.
What steps reproduce the problem?

Create a function and collapse it
Try to create a new breakpoint in any of the lines below
Nothing happens

What is the expected output? What do you see instead?
We should be able to modify the state of breakpoints after a collapsed function.
Versions

Spyder version: Spyder 4.0.0.dev0 (Commit: 886f73cff)
Python version: 3.6.8
Qt version: 5.12.2
PyQt version: 5.12.1
Operating System name/version: Darwin 18.2.0

@dalthviz, given that you work on the debugger panel, you have any ideas about where @steff456 can start looking to solve this bug?

@ccordoba12 @steff456 after testing a little bit it seems like when collapsing the code in the debug panel is not mapping correctly the space released for the collapsed lines so for example:

Also it seems that even if not showing when the code is collapsed, the breakpoint its being set:

So we need to make the debug panel to follow the mapping created by the folding panel?

Yep :)


test framework: default to single network

Prevents single-network multi-cluster tests from erroring with:
2020-09-24T22:45:17.230947Z	error	tf	Test setup error: failed generating eastwestgateway manifest for cluster-0 using "/Users/tairan/Documents/go/src/istio.io/istio/out/darwin_amd64/istioctl\n": exit status 1: The NETWORK environment variable must be set.

@stevenctl: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

integ-security-k8s-tests_istio
a136fb10e5f344c28fcd1cf6e91c6c20cc03d8ff
link
/test integ-security-k8s-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/retest


Add Flutter examples to sep24.mdx

@christian-rogobete can you please fix formatting with yarn run check:mdx ?

@Ifropc sure, thx, I fixed it.


Enable Crashlytics on Android

Automated message from Dropbox CLA bot
@borisyankov, it looks like you've already signed the Dropbox CLA. Thanks!


Moving repo under crystal-lang org

Is there any reason not to? Seems like pretty good fit under official umbrella :)

Yes, I think webmock and time.mock should be in the standard library, because they are coupled with their interface/definition. We'll eventually do it (though it will probably be called HTTP::Client::Mock)

ping 🏓

I'm not too happy about maintaining this shard. It's been mostly neglected by the Core Team (maybe the location in manastech org contributes to that). But I don't think this is an essential shard that needs to be maintained by the Core Team. I'd rather much prefer moving it somewhere else under community maintainership.


update legacyTypeMap

Hi, we will be removing EntityGenerator in 3.0 and 2.x is already in feature-freeze. Thanks for the PR.


bumped version number

Currently npm i containerjs-api-electron fails because it depends on containerjs-api-specification 0.0.1, which has a failing post install step (yuck!)
This just bumps the version number so that I can release these fixes:
https://github.com/symphonyoss/ContainerJS/pull/185
After this, I am NOT doing any more manual versioning. It's horrible ;-)

Could we not add ^ to use the latest version, or would this break something? Or will lerna do the bumping for us when we publish?

It's a good point, and I honestly don't know. For the short term, I just want to make sure ContainerJS is useable by installing via npm :-)
Once it works, then yes, it definitely needs tidying up!

Just notices that packages/api-tests/package.json also has a reference to electron v0.0.1. Will need to change that as well so tests will run correctly


[Feature Request]: Update llama-index-vector-stores-weaviate to weaviate-client version 4.x

Feature Description
Update llama-index-vector-stores-weaviate from weaviate-client version 3. 26.2 to 4.x
Reason
Methods are deprecated in weaviate-client > 4.0.
New features are introduced.
A collection upserted with weaviate-client 3.x can not be querried with weaviate-client  4.x
Value of Feature
Adapting to current Weaviate Cloud Service standard gRPC. Avoid deprecation. Use new features of Weaviate.

@tos-wdp agreed this would be a worthwhile update. Would welcome a PR for this update, if you'd be interested in taking it on.

Any news on this, please?

Can we add the multi tenant logic Weaviate offers also?

Can we also add the auto_limit (auto_cut) and rerank feature, eg:
openAiReranked = weaviate.query.hybrid(
    query=query,
    alpha=0.9,
    fusion_type=HybridFusion.RELATIVE_SCORE,
    filters=query_filter,
    auto_limit=2,
    rerank=wvc.query.Rerank(
        prop="content",
        query=query
    )
)

i want it 😀

Can we add the multi tenant logic Weaviate offers also?

I opened #13307 to discuss the design
Let's open a separate issue for each feature we'd like see added so it is easier to track progress.


TREC file score

The TREC file generated by using https://github.com/faneshion/MatchZoo/blob/master/examples/run_dssm_ranking.sh
seems have all the same scores of "1.000000". Could you plz help to double check?

@imsure318 Thanks for your feedback. I will check it right now~

Hi @imsure318, there have a bug at the Triletter_ListGenerator class in the previous version, and we have fixed it in the newest version. Thanks again for your feedback.


Please confirm these before moving forward

[X] I have searched for my issue and not found a work-in-progress/duplicate/resolved issue.
[X] I have not been informed if the issue is resolved in a preview version of the winget client.

Category of the issue
Other
Brief description of your issue
Winget tells me that Microsoft.DotNet.SDK.7 needs to be upgraded.
Steps to reproduce
Execute "winget upgrade". It outputs this...
Name                             Id                     Version Available Source
Microsoft .NET SDK 7.0.203 (x64) Microsoft.DotNet.SDK.7 7.0.203 7.0.302   winget
When I execute winget upgrade Microsoft.DotNet.SDK.7, it says... No installed package found matching input criteria.
So, I execute winget upgrade --all, it downloads "https://dotnetcli.azureedge.net/dotnet/Sdk/7.0.302/dotnet-sdk-7.0.302-win-x64.exe" and installs it. I then execute "winget upgrade" again but instead of saying that I should be good, oh...
Name                             Id                     Version Available Source
Microsoft .NET SDK 7.0.203 (x64) Microsoft.DotNet.SDK.7 7.0.203 7.0.302   winget
Actual behavior
Whatever winget is checking, the logic to check the version of Microsoft.DotNet.SDK.7 is wrong.
Expected behavior
Oh, I don't know... after the installation of the upgrade it should tell me that I have no upgrade of that program.
Environment
Windows Package Manager (Preview) v1.5.1081-preview
Copyright (c) Microsoft Corporation. All rights reserved.

Windows: Windows.Desktop v10.0.22621.1702
System Architecture: X64
Package: Microsoft.DesktopAppInstaller v1.20.1081.0

Winget Directories
-------------------------------------------------------------------------------------------------------------------------------
Logs                               %LOCALAPPDATA%\Packages\Microsoft.DesktopAppInstaller_8wekyb3d8bbwe\LocalState\DiagOutputDir
User Settings                      %LOCALAPPDATA%\Packages\Microsoft.DesktopAppInstaller_8wekyb3d8bbwe\LocalState\settings.json
Portable Links Directory (User)    %LOCALAPPDATA%\Microsoft\WinGet\Links
Portable Links Directory (Machine) C:\Program Files\WinGet\Links
Portable Package Root (User)       %LOCALAPPDATA%\Microsoft\WinGet\Packages
Portable Package Root              C:\Program Files\WinGet\Packages
Portable Package Root (x86)        C:\Program Files (x86)\WinGet\Packages

Links
---------------------------------------------------------------------------
Privacy Statement   https://aka.ms/winget-privacy
License Agreement   https://aka.ms/winget-license
Third Party Notices https://aka.ms/winget-3rdPartyNotice
Homepage            https://aka.ms/winget
Windows Store Terms https://www.microsoft.com/en-us/storedocs/terms-of-sale

Admin Setting                             State
--------------------------------------------------
LocalManifestFiles                        Disabled
BypassCertificatePinningForMicrosoftStore Disabled
InstallerHashOverride                     Disabled
LocalArchiveMalwareScanOverride           Disabled

Screenshots and Logs
No response

Nevermind, the installation routine for Microsoft .NET SDK 7.0.302 simply fails to remove the older version.


After formatting based on new pre-commit hook and future Jenkins tests

This is what Tensile will look like after applying https://github.com/ROCmSoftwarePlatform/Tensile/pull/475

Does not work now, closing request

Trying just C++ now


Report cell census changes between versions

From one cell census version to another it would be useful to report changes between the builds, to catch potential errors in either data or processing:

Datasets removed, added
Datasets in both versions but that have differing cell counts
Total cell count deltas by experiment (mouse, human)
Deltas between summary_cell_counts dataframes
Genes removed, added

Greatly support this and I'd would encourage to have this as part of the build process if possible.
Also related to #25


This repository template still on progress or not ?

Even, no service worker register when i run the template

Hi. Are you running nuxt in dev mode? This template basically should works, only dependencies in package.json may be upgraded :)

Try generating a static site, upload to an HTTPS enabled hosting (or use surge after npm i -g surge) and try from there.
Alternatively, build it and run now (npm i -g now first) to upload it there as nodejs app and it should also work

@pi0 does this work no matter what we add to it in the nuxt template? (ie. new plugins and so on)
Also, how do we cache ajax requests?

  workbox: {
    handleFetch: true
  }

in nuxt.config.js works fine for caching ajax requests. Keep in mind that the @nuxtjs/pwa dependency needs to be version 2 or higher for it to work.

Thanks @colsen1991


jitsi meet integration in Ionic App

How to use jitsi in Ionic App ? please give me step-by-step instructions i try to follow the instruction provided in the documentation but it's not clear me.

@jyothimarkal Consider asking in the Ionic forums: https://forum.ionicframework.com/. This is not the proper place for that question.

Okay Thank u for response


Create Simmie_Covid19_NLP.yml

Thanks for contributing your submission to the e-Rum2020 CovidR contest.
For CovidR contributors:

[ ] The title of this PR reflects your contribution
[ ] The pull request only includes one new .yml file under contest/contributions
[ ] Make sure "Allow edits from maintainers" is checked
[ ] Send an email to<EMAIL_ADDRESS>(for further direct communication down the road)

Subject: "Submission to CovidR: PR #00", where #00 is the number of this Pull Request
Content: A full link to this Pull Request
Optionally include your social media profiles (Twitter, LinkedIn, Facebook): we will use them to @mention you while sharing the contribution through our channels.

For CovidR maintainers:

[ ] Label the PR as "submission"
[ ] Check the validity of the submission
[ ] Assess completeness of metadata and possibly adjust the .yml file-name
[ ] Check the integration of the new page in the website gallery (via Actions' artifact or locally)
[ ] Make sure the email to<EMAIL_ADDRESS>was received
[ ] Add a field with the current date and approx. merge time (e.g. 2020-04-01-16:30 for April 1st, 4:30 PM)
[ ] Squash-merge the pull request (commit headline will be OK, consider cleaning-up the details)
[ ] The initial comment should be automatically created, otherwise comment manually with the raw content of vote-comment.md and an initial like
[ ] Provide here the link to the contribution as published to the gallery: https://milano-r.github.io/erum2020-covidr-contest/___.html
[ ] Instruct contributors on how to include their badge from the gallery page, share on social media, and get a free ticket.
[ ] Consolidate e-mailed information.

Submission

@simmieyungie, your submission is now accepted and visible at https://milano-r.github.io/erum2020-covidr-contest/simmie-covid19-nlp.html :tada:
You can click on the eRum2020::Covidr badge in the page to obtain the badge code you can embed in your repository.
Furthermore, feel free to spread your contribution on social media with the link above, using hashtags #erum2020 #covidr, and invite the community to like the contribution using the corresponding button.
Last but not least: You can get your free ticket for the pre-conference event at https://www.eventbrite.it/e/covidr-erum-2020-pre-conference-event-tickets-104133026686


Fix specific return value about ForceUnmount

In functon ForceUnmount,there is no specific return value when unmount failure, and then the function maybe returns a undefined error,so we should define it explicitly.

http://golang.org/doc/effective_go.html#named-results

@duglin I see,thanks for your remind:) The PR#11150 should be closed.


Adding web view for container performance [mem/proc]

Include a graphical interface to show the cpu and mem consumption of each container.

Already integrated


[FR PhoneNumber] Phone numbers with dots not recognized ("<IP_ADDRESS>.66")

Describe the bug
In french, most of the time telephone numbers are divided into two digits. But when those digits are separated by dots, phone numbers are not recognized.
To Reproduce
Parse "Mon numéro : <IP_ADDRESS>.66" in the recognizer.
It returns :
nothing
Expected behavior
It should return :
{   "Text": "<IP_ADDRESS>.66",   "Start": 13,   "End": 26,   "TypeName": "phonenumber",   "Resolution": {     "score": "1",     "value": "<IP_ADDRESS>.66"   } }
Platform

Platform: [.NET]
Environment: [console app]
Version of package [v1.3.2]

Can you also give an example that uses a country code at the start?

Can you also give an example that uses a country code at the start and one of someone calling an international number?

According to https://www.indicatifs.fr/espagne , to call the spanish number "824 65 43 21" :
From a mobile phone :  "+34 824 65 43 21" or more often "+34 8 24 65 43 21"
From a landline phone "00 34 824 65 43 21"
Otherwise, most of the time phone number looks like that :
"+33 6 69 96 36 27"
"06 69 96 36 27"
"04 94 66 66 66"
....

BTW, are the phone lengths in France always the same?

For individuals (mobile and landline) yes. However, police is "17", emergencies "15" and there are others "special"

Thanks for the extra info, @Oursse! Special numbers like the police one are out of scope for the current recognizers though.

Fix is on the way.

European phone number like Ireland  +353 1 706 xxxx is supported


idea: RangeReader should send API based on input read buffer's size

RangeReader will send request to read the entire range everytime no matter how large the read buffer size is. So opendal doesn't work well in this case:
let r = op.reader(path).await?;
r.read(&mut [u8;1]).await?;
r.seek(SeekFrom::Current(1)).await?;
r.read(&mut [u8;1]).await?;
r.seek(SeekFrom::Current(1)).await?;

For every read call, range reader will send a new request to read the entire range but only consume a small piece:

The connection will have to abort, makes it can't be reuse.
And more data will be fecthed from server side.

Maybe we can send request based on user's input buffer size instead. For users we want to do small read, they can add a buffer over the reader instead.

A possible regression: read_to_end(&mut Vec::new()) could be very slow in this case.

After this change, adding buffer on reader makes more sense.

I mixed two different thing together. It doesn't make sense to use users input buffer size as the io size. The performance could drop to 50% for the read_to_end cases.

I mixed two different things together.
It doesn't make sense to use users input buffer size as the io size. The performance could drop to 50% for the read_to_end cases.

Currently, the operator in our system that enables the cache, looks like the following:
S3Accessor                FsAccessor
ErrorContextLayer         ErrorContextLayer
CompleteLayer             CompleteLayer
    ▲                         ▲  │
    │                         │  │
    │`inner`           `cache`│  │
    │                         │  │
    │                         │  │
    │                         │  │
    └───── LruCacheLayer ─────┘  │
                 ▲               │
                 │               │
                 │               │
                 │               ▼
                 │               FileReader::new(oio::TokioReader<tokio::fs::File>)
                 │
                 Invoking(`reader`, `reader_with`)

I think your system might also have a similar approach, resulting we overlooked this issue until recently.
It seems like we need another way to work around it. How about just keeping the IncomingAsyncBody in the memory if the underlying data is fully fetched?

It seems like we need another way to work around it.

It's acceptable to execute such tasks at the application level, but not within OpenDAL. I benchmarked the cost of terminating a connection; it takes approximately 600us (0.6ms), which is acceptable.
I'm also thinking about adding position read (like read_at) API in opendal, but not figure out how to add it.

such tasks

Do you mean keeping the IncomingAsyncBody in the memory?

Do you mean keeping the IncomingAsyncBody in the memory(or similar approaches)?

Yes, we can't do such thing unless they are zero cost. Buffering body in memory could panic users when they meet OOM unexpectedly.

Do you mean keeping the IncomingAsyncBody in the memory(or similar approaches)?

Yes, we can't do such thing unless they are zero cost. Buffering body in memory could panic users when they meet OOM unexpectedly.

I see. There are some tradeoffs we have to make.

Thanks for understanding. This issue is not valid anymore, I'm going to close it now.


Unwanted trailing slashes in links to commits to external Git hosters

When linking to a commit hosted in an external environment (such as https://gitlab.com/gitlab-org/gitlab-ce/commit/54c2d7bdbddc18953571ed928948c18471d6edfa), Gitea introduces a trailing slash at the end of the shortened commit hash.

This is a reference to #1.
#1 addressed this issue first.


Types.Slider: dynamic value

Hi guys,
i've just created small PR that allows see realtime value in SLIDER tile. Currently - you have to move slider, released it to see current value, with this PR value will be updated with slider movement. Can you please take a look.
Thanks.

Thank you for accepting PR. This case can be closed.


Not copying the JavaHelp library into the binary.

This avoids the runtime dependency on JavaHelp (compile-time dependency remains). Help is not working as a result. Can be partially workarounded by saying:
--cp:a <path-to-jhall.jar>
on command line, but there are still bugs with that (as JavaHelp is loaded by a classloader that does not see NetBeans classes).

Good solution for the moment. Thanks. Merging.


[WFLY-13981] Remove legacy security configuration from appclient.xml

Just submitting as a draft PR to begin with for some CI runs.
https://issues.redhat.com/browse/WFLY-13981

Marking as ready to review so we can complete this task.

WildFly Preview tests:
https://ci.wildfly.org/viewLog.html?buildId=253639&tab=buildResultsDiv&buildTypeId=WF_WildFlyPreviewLinuxJdk11
https://ci.wildfly.org/viewLog.html?buildId=253637&tab=buildResultsDiv&buildTypeId=WF_Ee9BootableJarLinuxJdk8


Feature: add env based config for forking options

Adding ability to pass in environment variables into forking options, this makes it possible to enable forking in the docker-compose.yml file.

Closing in favor of https://github.com/graphprotocol/graph-node/pull/4308. Thanks for the idea, @jeffywu  !


Longer ToolTips are cutoff by sidebar

Browser

[ ] Chrome/Chromium
[ ] Firefox
[ ] Safari
[ ] Edge
[ ] iOS Safari
[ ] Opera
[X] Other/not relevant

Operating system

[ ] Windows
[ ] Linux
[ ] macOS
[ ] ChromeOS
[ ] iOS
[ ] Android
[X] Other/not relevant

Description

Kinda an edge case because it would assume your search results in the long tooltip. But when certain icons with long names end up on the left side of the ui their tooltip will be cut off.
Steps to reproduce
Type graph in the search. (There will be other cases but this is what I did)
Checklist

[X] I have searched if someone has submitted a similar issue before and there weren't any. (Please make sure to also search closed issues, as this issue might already have been resolved.)

can i take this up?

If you are asking me go ahead I unfortunately don't have enough time for this rn

can i take this up?

Feel free to create a PR any time.

@jguddas hey i have raised a pr for this, could you review it and also add hacktoberfest label as it would be helpful to me as well


fix (2**53+1).toExponential() with a test

Fixing this brought out an error in second subtraction, which is
also fixed.
Closes #15

Thank you for this patch!
Can you also bump version to 0.5.1?

Thank you for this patch!
Can you also bump version to 0.5.1?

Done.

Ty!

Not sure why, but only one of two commits are merged.

Yep, for some reason your very first commit not shown in github and I started to press squash and merge, as I understand this function join two commits into one.
Usually I am commit to someone else opensource projects and I am not so experienced to maintain your on. So my bad.

Yep, for some reason your very first commit not shown in github and I started to press squash and merge, as I understand this function join two commits into one.
Usually I am commit to someone else opensource projects and I am not so experienced to maintain your on. So my bad.

It's all right. I thought that changed to package.json were not merged, but they are.


how to using in vue3  :(

support please.... : (

Sorry, I don't know and don't have time to find out right now.


use axios instead of got for sending request

Is it possible to use httpyac, with another library than got, got does not support body on get requests, but axios does.
This make these kinds of requests impossible to perform, it is not a good choice to pass the data in query params neither it is to make it a post request, a QUERY method is in study, but we are not there yet, so in the meantime a good workaround is to keep using libraries that support body with get method.
request
### find person list
# @name findPersonList
# @ref authSignin
GET {{Endpoint}}
Authorization: Bearer {{ authSignin.token }}

{
  "email"<EMAIL_ADDRESS><EMAIL_ADDRESS>}

Error message:
RequestError: The GET method cannot be used with a body at Gk._destroy

Finally, I found a way to make this work with got, by passing this option allowGetBody.
"httpyac.requestGotOptions": {
    "allowGetBody": true
  }

I am keeping this issue open, to let you decide guys if that's still worth keeping in mind as an option.

The decision for got was deliberate. My POC had still used axios. The main reason was that got also supports HTTP/2 and I wanted to be as close as possible to the actual browser call. But I left it open to implement a axios plugin on your own. You would just to implement httpClientProvider which wraps all calls to got. But I didn't implement the interface cleanly, because in some places I still use the Got Request Options myself (like in digestVariableProvider).
But I did not know the setting and I add it. Even if I would rather not use it myself on the server side.

https://github.com/sindresorhus/got/blob/main/documentation/2-options.md#allowgetbody

"However, the HTTP/2 specification says:
An HTTP GET request includes request header fields and no payload body"

Doesn't sound like GET requests were designed to do this...

@Kaarel I agree with you that it is not part of the specification, and for that reason I will not use it myself in any way. It is not worth the trouble. However, it shouldn't be a problem to allow it in for use in httpyac. Or do you know constellations where this can become a problem ?

Nah, just noticed this thread and started wondering about bodies with GET requests :)

@Kaarel, in fact in the front side, it is much easier to work with a get request when handling a request like a search, principally for caching, and on the back side I am much more comfortable with a get since the request is not changing any database or server state, last time I honestly searched some hours on this topic, and I can say that the number of people not seeing any problem with using a body in get is increasing with years, there will be some old servers that will ignore the body on gets, but the risk is still small.
There are a lot of debates around this topic and surely at the end, we will have a new protocol that can have a body without editing the server state, I am waiting for it.

@Gxmadix  I fully agree with your arguments. I am just a lame duck. I fear that a firewall, Ingress or reverse proxy will remove my body. I have never tried it though. But maybe this would be a topic to bring up in my environment. I know the problem only too well.


exhibitor: fixes around /opt/mesosphere/etc/exhibitor_realm

start_exhibitor.py: check existance of realm file in /opt instead of /run
run ExecStartPre as root and chown realm file to user dcos_exhibitor

Note that /usr/bin/chown is a standardized location that should exist on all target platforms, and that it dereferences symlinks by default.

nak, the exhibitor_realm file should get a uname to make it group / other readable.

https://github.com/dcos/dcos/blob/master/gen/__init__.py#L490 -- inside fo the dcos-config.yaml you can actually specify a 'permissions' key which we'll chmod the file to (defaults to 644)

We should make the exhibitor_realm file readable only for the user dcos_exhibitor (it may contain a cleartext password). What is currently the best way to achieve that?

Everything in gen/dcos-config.yaml becomes tarballs which any host in the cluster could download, secure credential download can't be achieved through it really... For 1.8 shipping the file on host is probably the easiest. To get proper credential store probably needs to end up in a /run directory

:ship:


Updated Azure Cosmos DB branding and API names.

:warning: Validation status: warnings

File
Status
Preview URL
Details

_sdk/lib/services/azurestackManagement/README.md
:warning:Warning
View (azure-node-latest)
Details

docs-ref-services/cosmos-db.md
:white_check_mark:Succeeded
View (azure-node-latest)

_sdk/lib/services/azurestackManagement/README.md

[Warning] Failed to find Git account for the committer<EMAIL_ADDRESS>Solution: 1) File creator to associate email address<EMAIL_ADDRESS>with his or her Git account, or 2) Anyone to explicitly add "author: <Git account>" into markdown YAML header.

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

#sign-off

@rloutlaw - Can you merge PRs in this repo?


FlutterEngineConnectionRegistry activity.getIntent() appers NPE

Steps to Reproduce
activity.getIntent() may be null。There might be an NPE！~

~ flutter doctor --verbose
[✓] Flutter (Channel stable, 3.0.1, on macOS 11.6 20G165 darwin-x64, locale
    zh-Hans-CN)
    • Flutter version 3.0.1 at /Library/flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision fb57da5f94 (3 days ago), 2022-05-19 15:50:29 -0700
    • Engine revision caaafc5604
    • Dart version 2.17.1
    • DevTools version 2.12.2

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

[✗] Xcode - develop for iOS and macOS
    ✗ Xcode installation is incomplete; a full installation is necessary for iOS
      development.
      Download at: https://developer.apple.com/xcode/download/
      Or install Xcode via the App Store.
      Once installed, run:
        sudo xcode-select --switch /Applications/Xcode.app/Contents/Developer
        sudo xcodebuild -runFirstLaunch
    ✗ CocoaPods not installed.
        CocoaPods is used to retrieve the iOS and macOS platform side's plugin
        code that responds to your plugin usage on the Dart side.
        Without CocoaPods, plugins will not work on iOS or macOS.
        For more info, see https://flutter.dev/platform-plugins
      To install see
      https://guides.cocoapods.org/using/getting-started.html#installation for
      instructions.

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

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

[✓] Connected device (3 available)
    • ANA AN00 (mobile) • KXU0221629000993 • android-arm64  • Android 10 (API
      29)
    • macOS (desktop)   • macos            • darwin-x64     • macOS 11.6 20G165
      darwin-x64
    • Chrome (web)      • chrome           • web-javascript • Google Chrome
      100.0.4896.75

[✓] HTTP Host Availability
    • All required HTTP hosts are available

! Doctor found issues in 1 category.

@JunhuaLin
Can you provide a stripped down minimal code sample that triggers the NPE you are reporting ?


Tableview not scrolling

I have integrated all properties of AutocompleteTextfieldSwift, and dropdown is working fine. But i can't scroll or select any row of tableview(dropdown)

+1

I fixed this by removing a UITapGestureRecognizer I had put on my view to dismiss the keyboard


Lambda does not have permission to access the ECR image. Check the ECR permissions.

Hello,
While following the tutorial, the lambda failed to pull image from account 1.
For it to work i had to delete the condition attached to the lambda service policy statement which is obviously not ideal.
I triple checked aws account numbers and region names, did something change since the creation of this sample ?
Are there some account specificities i have to watch for ? i'm running out of options to check any help would be appreciated i really want to avoid copying the image in another repository located in account 2.
in my configuration account 1 and 2 are not in the same organization unit and i'm using an aws sso assumed role to create the lambda, i don't know if this has anything to do with my problem...
Thanks in advance

Sorry my mistake i should have quadruple tested the region from the lambda template...

Hi @GuillaumeKLECH
I gotcha the same error :(
  An error occurred: TaskNotifierWithECRLambdaFunction - Resource handler returned message: "Lambda does not have permission to access the ECR image. Check the ECR permissions. (Service: Lambda, Status Code: 403, Request ID: 4ee4c0f1-97ca-4a19-80bb-1643686dd3d2)" (RequestToken: 80e1c7cd-0f27-6ed5-4769-3c7b7e3f160e, HandlerErrorCode: AccessDenied).
    - Effect: "Allow"
      Action:
        - ecr:InitiateLayerUpload
        - ecr:SetRepositoryPolicy
        - ecr:GetRepositoryPolicy
        - ecr:GetDownloadUrlForLayer
        - ecr:BatchGetImage
        - ecr:CompleteLayerUpload
        - ecr:DescribeImages
        - ecr:DescribeRepositories
        - ecr:UploadLayerPart
        - ecr:ListImages
        - ecr:BatchCheckLayerAvailability
        - ecr:PutImage
        - ecr:GetAuthorizationToken
        - ecr:CreateRepository
        - ecr:ReplicateImage
      Resource: "arn:aws:ecr:sa-east-1:055355002851:*"

i already test multiples conditions on resource, all above:
- ecr:InitiateLayerUpload
- ecr:SetRepositoryPolicy
- ecr:GetRepositoryPolicy
- ecr:GetDownloadUrlForLayer
- ecr:BatchGetImage
- ecr:CompleteLayerUpload
- ecr:DescribeImages
- ecr:DescribeRepositories
- ecr:UploadLayerPart
- ecr:ListImages
- ecr:BatchCheckLayerAvailability
- ecr:PutImage
- ecr:GetAuthorizationToken
- ecr:CreateRepository
- ecr:ReplicateImage

Anyone can help me?


Improve the docs and more

One more chapter of the Getting Started Guide
control_type property returns a string now
control_type can be used in window specifications as a string
Issue #254 is fixed

Current coverage is 94.84% (diff: 85.00%)

Merging #255 into master will decrease coverage by 0.03%

@@             master       #255   diff @@
==========================================
  Files            49         49
  Lines         19425      19436    +11
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
+ Hits          18431      18434     +3
- Misses          994       1002     +8
  Partials          0          0

Powered by Codecov. Last update 3f61f7d...fa180e0


String containing square brackets is not treated as string

Hi,
I have tried to assign jsons' array as a string to ARM parameter:
schema
{
  "$schema": "https://schema.management.azure.com/schemas/2019-04-01/deploymentTemplate.json#",
  "contentVersion": "<IP_ADDRESS>",
  "parameters": {
    "A": {
      "type": "string"
    }
  },
  "resources": [
  ]
}

parameters file
{
  "$schema": "https://schema.management.azure.com/schemas/2015-01-01/deploymentParameters.json#",
  "contentVersion": "<IP_ADDRESS>",
  "parameters": {
    "A": {
      "value": "[{\"someProperty\":\"someValue\"}]"
    }
  }
}

As long as string value contains square brackets this parameter is not treated as string and results in following error:
[-] DeploymentParameters Should Have Value (3 ms)         A parameter cannot be found that matches parameter name 'TargetObject'.

According to the ARM template documentation, string values that start with a square bracket "[" should be escaped to not have them interpreted as expressions:
Perhaps the parameter file is really incorrect? The reported error message doesn't seem to help recognize the issue, though.

@taurit - thank you for your comment. I have tried to escape the leading square bracket (as adviced in docs). Unfortunately the arm-ttk still reports the same error.

The issue is this line "Write-Warning -Message '$($p.Name)'.value will not be evaluated as an expression"# -TargetObject $p.Value.value - the Write-Warning does not have e parameter called -TargetObject. The issue is powershell code.


Update re-thrown error for incompatible Java/Gradle versions to include migration guide

Updates error message concerning incompatible Java/Gradle versions to reference migration guide that will be created in https://github.com/flutter/website/pull/8501.
Pre-launch Checklist

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] I read the Tree Hygiene wiki page, which explains my responsibilities.
[x] I read and followed the Flutter Style Guide, including Features we expect every widget to implement.
[x] I signed the CLA.
[ ] I listed at least one issue that this PR fixes in the description above.
[ ] I updated/added relevant documentation (doc comments with ///).
[x] I added new tests to check the change I am making, or this PR is test-exempt.
[ ] All existing and new tests are passing.

This LGTM if we are of the opinion that the info should only be on the migration guide.
I do somewhat think that there is value to leaving the existing steps, and then providing the link as a "for more info, see:" type of link. It can be a nicer experience to just see the answer, instead of having to copy a link and open it in a browser. But also understand if we want all the info in one place, instead of multiple.

This LGTM if we are of the opinion that the info should only be on the migration guide.
I do somewhat think that there is value to leaving the existing steps, and then providing the link as a "for more info, see:" type of link. It can be a nicer experience to just see the answer, instead of having to copy a link and open it in a browser. But also understand if we want all the info in one place, instead of multiple.

I thought the same thing, but didn't want to be redundant. The current steps are definitely more vague than the guide, and the guide should include the information I had before. Deferring to @mariamhas and @reidbaker

Gray you make a valid point on making it easier for developers, but if the steps are robust enough or complete they might confuse the developer and end up making things more difficult. I think its better to keep the instructions in one place as source of truth for when we eventually have to update them again, itll be easier to do it in one place than several.

Gray you make a valid point on making it easier for developers, but if the steps are not robust enough or complete they might confuse the developer and end up making things more difficult. I think its better to keep the instructions in one place as source of truth for when we eventually have to update them again, itll be easier to do it in one place than several.

Sure, that makes sense! This LGTM then


bootstrapping go with gccgo

What version of Go are you using (go version)?
go version go1.12.4 linux/amd64
Hello, I'm not sure whether this is the correct forum for this, but right now it is the only one I know about and have access to, so I hope you will bear with me. :-)
I am the go package maintainer on Gentoo, and I would like to know if bootstrapping go via gccgo is supported, and what the official way of doing this is.

How should I set GOROOT_BOOTSTRAP?

What about the go binary that should be used to build? Is there a variable that can be set to define it?

Thanks much,
William

AFAIK the process is just something like "use the binary at $GOROOT_BOOTSTRAP/bin/go".
So I would just set up a folder structure like gccgo-bootstrapping/bin/go and make go point to the gccgo binary. And then try to bootstrap. Does this work?


Clean up all the usages of a non-existent matrix

@mccoyp
This is a follow-up to clean up leftovers from #32590

/check-enforcer override


[Targetting Auto Mode] bug with auto target selection, if not set on manual auto rotation will not execute actions

i assume this is because of invisible targets since it only bugs with boss fights. I had it on lowest, it did not use any actions, when set to manual it does.

Can you include a clip of this?

Are you able to upload it here instead? If not, that's fine

This will still require more information in order to process properly. Auto-rotation won't count anything you can't target as a valid target, so "invisible targets" are excluded.

Can you be more specific about the instance you are referring to? Preferably: what quest it is
I haven't done new MSQ yet, so not sure what you're referring to.


Add basic public API Gemini support

Unit tests added for Gemini public API calls.
Missing Gemini feed and private API calls (in a later PR).

Since we have ccxt as a dependency now, it's fairly easy to write a wrapper for any exchange -- see the refactoring for Poloniex.
I'd prefer to go this route since it let's us get on with writing higher-level tools for the GTT and the ccxt maintainers can focus on keeping the API interfaces up-to-date.
Have a look and see if you can find any good reasons not to use ccxt going forward (Public and Auth calls should be working out of the box).
Note that we still need ws feed implementations for exchanges, since that's not supported by the ccxt (I suspect it's on their roadmap though).

Thanks for the update @CjS77 - closing this PR.

@CjS77 hi!
First of all, thx for using our ccxt library!
Cool to know it's been used in this project. We're watching this repo very closely, and yes, WS pub/sub support is on our roadmap (most of WS unification is done already, so we are going to deliver soon).
Also, we are going to implement the entire unified API for all exchanges, of course, though, we don't have as much resources as GDAX teams may have, I guess ;) Our userbase isn't very large yet, but we hope to grow quickly ;)
We will be happy for all feedback regarding the library, for your help and for pull requests! Let us know if you have any questions about it. Thx again!


Update coffeescript.coffee to use alert

Coffeescript documentation uses alert for messages rather than console.log

Thanks for the fix!


Add stats endpoint for OGN dashboard

Related PR: https://github.com/OriginProtocol/origin-website/pull/565
Contract change checklist:

[ ] Code reviewed by 2 reviewers
[ ] Unit tests pass
[ ] Slither tests pass with no warning
[ ] Echidna tests pass (not automated, run manually on local)

Closing this PR, since we can do this on the analytics repo


[FEATURE] Add an Light mode

Is this a unique feature?

[X] I have checked "open" AND "closed" issues and this is not a duplicate

Is your feature request related to a problem/unavailable functionality? Please describe.
Add an light mode , so users can switch their theme according to their needs.
Proposed Solution
Add a button for toggling between light &dark mode in navbar
Screenshots
No response
Do you want to work on this issue?
Yes
If "yes" to above, please explain how you would technically implement this
No response

Thank you for raising, themes are not something we want to add at the moment


Create a custom concat document that listens to 'lsp' protocol messages

First steps in implementing this: https://github.com/microsoft/vscode-jupyter/issues/8075
The NotebookConcatDocument (in src\notebookConcatDocument.ts) is a full replacement for the NotebookConcatDocument provided by the VS code proposed API
It works by handling just 4 messages/events

textDocument/didOpen for each cell. Position of cells is determined by the fragment of the cell URI (notebooks open with the fragment in order)
textDocument/didChange for each cell. Changes to cells cause updates to the internal structure of the concatenated document
textDocument/didClose for each cell. Closing a cell is like deleting it from the concat document
notebook/refresh. This message is sent whenever the order of cells changes. It's the extra LSP message to handle reordering. This message sends an array of protocol.DidOpenTextDocumentParams with the cells in the new order.

This first submission is getting a bunch of tests to verify this all is working. Next steps are:

consume this in the jupyter extension (and fix any new bugs found with more rigorous testing)
modify the concatenation to eliminate/replace magics and shell escapes

/cc @jakebailey @heejaechang @bschnurr

executeCommand - notebook.refresh. This command is sent whenever the order of cells changes. It's the extra LSP message to handle reordering. This command sends an array of protocol.DidOpenTextDocumentParams with the cells in the new order.

On this front, only one provider can exist for a command at a time; I'm not sure if we can use a command for this purpose. I think they're going to have to be custom LSP notifications modeled on didChange and such.

On this front, only one provider can exist for a command at a time; I'm not sure if we can use a command for this purpose. I think they're going to have to be custom LSP notifications modeled on didChange and such.

No it's not a VS code command, it's a command sent to the language server. Using this message here:
https://microsoft.github.io/language-server-protocol/specifications/specification-current/#workspace_executeCommand

That's the endpoint that VS Code uses to call registered commands; largely it's equivalent to a custom LSP request or notification if you are low enough that you can directly send requests, but I think it's not quite right to be calling executeCommand directly for this purpose.
Dirk's comment links the VS Code's docs on commands, which is the "normal" use of this endpoint indirectly, which has the restriction I mentioned.

Switched to using a message instead of a command (this is easier for pylance to handle)


feat: parse check item before jmespath search

Main Changes
feat: parse check item before jmespath search

Codecov Report

Merging #1239 (c504e52) into master (7259d12) will decrease coverage by 9.97%.
The diff coverage is 25.00%.

@@            Coverage Diff             @@
##           master    #1239      +/-   ##
==========================================
- Coverage   67.52%   57.55%   -9.98%
==========================================
  Files          67       36      -31
  Lines        7545     4679    -2866
==========================================
- Hits         5095     2693    -2402
+ Misses       2234     1767     -467
- Partials      216      219       +3

Flag
Coverage Δ

unittests
57.55% <25.00%> (-9.98%)
:arrow_down:

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

Impacted Files
Coverage Δ

hrp/response.go
62.65% <25.00%> (-2.79%)
:arrow_down:

hrp/session.go
72.13% <0.00%> (-3.28%)
:arrow_down:

hrp/boomer.go
76.11% <0.00%> (-2.99%)
:arrow_down:

httprunner/__main__.py

httprunner/step_testcase.py

httprunner/ext/uploader/__init__.py

httprunner/runner.py

httprunner/builtin/comparators.py

httprunner/config.py

... and 25 more

Continue to review full report at Codecov.

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

fix #1235


Pressed state

My use case is fairly simple, i guess.
I need to trigger some behavior when an element is pressed and when it is released, across multiple types of input (mouse and touch). Imagine to change the color of a button for as long as it is pressed, for example.
Can this be accomplished with hammer.js? This isn't a gesture "per se", but more like "universal" mousedown and mouseup events. It isn't a tap or a press. I wonder if hammer.js makes this possible.
Thanks!

@miguelcobain depending on what you need, there's two ways to accomplish this.
With CSS: .foo:active {} will apply styles so long as the pointer (mouse, finger, etc.) is actively pressing on the element.
With Hammer: press and pressUp can be used for behavior while pressing and when pressing ends.


Port UtilsEtc module to Java

https://github.com/CategoricalData/hydra/blob/main/hydra-haskell/src/main/haskell/Hydra/Adapters/UtilsEtc.hs

Note: UtilsEtc is the not-yet-ported portion of Utils, the ported portion of which can be found in gen-main (and in Hydra/Sources).

Now: https://github.com/CategoricalData/hydra/blob/main/hydra-haskell/src/main/haskell/Hydra/AdapterUtils.hs