Speedup

Switch everything to typed memoryviews and inline direct solver update.
20 newsgroups
=============
X_train.shape = (11314, 130107)
X_train.format = csc
X_train.dtype = float64
X_train density = 0.001214353154362896
y_train (11314,)
X_test (7532, 130107)
X_test.format = csc
X_test.dtype = float64
y_test (7532,)

Classifier Training
===================
/home/vlad/code/polylearn/polylearn/factorization_machine.py:118: UserWarning: Objective did not converge. Increase max_iter.
  warnings.warn("Objective did not converge. Increase max_iter.")
Training fm-2 ... done
Training fm-3 ... done
/home/vlad/code/polylearn/polylearn/polynomial_network.py:99: UserWarning: Objective did not converge. Increase max_iter.
  warnings.warn("Objective did not converge. Increase max_iter.")
Training polynet-2 ... done
Training polynet-3 ... done
Classification performance:
===========================

Classifier            train       test         f1   accuracy
------------------------------------------------------------
fm-3               36.3608s    0.3642s     0.4009     0.9663
fm-2               31.7401s    0.1225s     0.6126     0.9740
polynet-3           6.6819s    0.0685s     0.6992     0.9796
polynet-2           5.9581s    0.0693s     0.7310     0.9807

>>> on branch speedup

Classifier            train       test         f1   accuracy
------------------------------------------------------------
fm-3               11.7684s    0.3643s     0.4009     0.9663
fm-2                7.2895s    0.1197s     0.6126     0.9740
polynet-3           6.6159s    0.0685s     0.6992     0.9796
polynet-2           5.9382s    0.0683s     0.7310     0.9807

Coverage remained the same at 97.886% when pulling 025c323e6d0788e712276d952846e97bf3cffb87 on speedup into 191fb4245a5b4255669001a9a6f5d0be09873148 on master.

Coverage remained the same at 97.886% when pulling 89bdd3e803cc9a17611a16df3028895439a3a159 on speedup into 191fb4245a5b4255669001a9a6f5d0be09873148 on master.

Coverage remained the same at 97.886% when pulling 41fdc4ed485ee125b7fee52175d69008919496fb on speedup into 191fb4245a5b4255669001a9a6f5d0be09873148 on master.

@fabianp could you give me a hand to figure out why I'm getting these win32/py2 failures? It looks like the cd_direct_fast.pyx solver has numerical issues in this configuration, but I can't pinpoint the cause.

Hey. Went through the file cd_direct_fast.pyx but I don't know what can be causing such failures. Any chance you could make a minimal example for such failures?

Since the win32 failures are independent of this PR, I think this PR should be merged, since it is a big improvement. I would also add a benchmark/ folder to help track the improvements over time.
Regarding the failures, could it be that the random kit behave differently on win32?

If that is the case, adding l2 regularization and/or more iterations should
solve the problem
On Jul 25, 2016 2:49 PM, "Mathieu Blondel"<EMAIL_ADDRESS>wrote:

Since the win32 failures are independent of this PR, I think this PR
should be merged, since it is a big improvement. I would also add a
benchmark/ folder to help track the improvements over time.
Regarding the failures, could it be that the random kit behave differently
on win32?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/scikit-learn-contrib/polylearn/pull/1#issuecomment-234943596,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAQ8hwPuHbnZQYCovhlsgRQJNBkR_VrZks5qZLDDgaJpZM4JO8xX
.

Remember that FMs and PNs are non-convex :)

@ogrisel From past experience with scikit-learn, any idea what the win32 failures could come from?
By the way, the problem is only on Python 2.7. Python 3.5 is fine.

D'oh!
On Jul 25, 2016 2:55 PM, "Mathieu Blondel"<EMAIL_ADDRESS>wrote:

Remember that FMs and PNs are non-convex :)
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/scikit-learn-contrib/polylearn/pull/1#issuecomment-234945037,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAQ8h5Z2gdf70Izpa9Lk0nEZLJJODZdHks5qZLJJgaJpZM4JO8xX
.

Py_ssize_t has a different meaning signed 32 bit int on a 32 bit platform and signed 64 bit int on a 64 bit platform. It should only be used to index datastructures (e.g. arrays or Python lists). I do not understand why the problem does not exist with Python 3 though.

That makes sense. Thanks, @ogrisel! However the problem still existed when I was using int everywhere. I only replaced it with py_ssize_t in an attempt to fix it, and it seems I got a bit overzealous and replaced too many vars.
I'll change it to use py_ssize_t just where it should.
On July 25, 2016 10:23:48 AM EDT, Olivier Grisel<EMAIL_ADDRESS>wrote:

Py_ssize_t has a different meaning signed 32 bit int on a 32 bit
platform and signed 64 bit int on a 64 bit platform. It should only be
used to index datastructures (e.g. arrays or Python lists). I do not
understand why the problem does not exist with Python 3 though.

You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub:
https://github.com/scikit-learn-contrib/polylearn/pull/1#issuecomment-234967790

--
Sent from my Android device with K-9 Mail. Please excuse my brevity.

Coverage increased (+0.003%) to 97.888% when pulling 64c8876eb4612e0c879e03e42b3f3b8019963bd1 on speedup into e0d79c23f4d7f1d8ce4fd7081dbf9d9f58913d52 on master.

@mblondel I think it's not a rng issue. The y generated in tests/test_factorization_machine.py:check_fit for both degree=2, 3 is the same in AppVeyor as on my 64bit ubuntu machine. I haven't directly checked the initial P_ but I doubt it would be different.
I also fixed the types as suggested by @ogrisel with no difference.
So it's probably something numerical.  It's strange that no errors occur with the lifted solver though.
Maybe I should try a 32bit py2 env in linux and see if I can reproduce it.

Coverage increased (+0.003%) to 97.888% when pulling 64c8876eb4612e0c879e03e42b3f3b8019963bd1 on speedup into e0d79c23f4d7f1d8ce4fd7081dbf9d9f58913d52 on master.

Coverage increased (+0.003%) to 97.888% when pulling 64c8876eb4612e0c879e03e42b3f3b8019963bd1 on speedup into e0d79c23f4d7f1d8ce4fd7081dbf9d9f58913d52 on master.

Coverage decreased (-0.005%) to 97.88% when pulling e78961ee56b78f5034130f8500a0045e6e4e5722 on speedup into e0d79c23f4d7f1d8ce4fd7081dbf9d9f58913d52 on master.

Coverage decreased (-0.005%) to 97.88% when pulling 49a3d7e8d41eef91ca1126b8f03b35df61b9149f on speedup into e0d79c23f4d7f1d8ce4fd7081dbf9d9f58913d52 on master.

Coverage decreased (-0.005%) to 97.88% when pulling 024e08d61550f7d622ac62cedfb663befcda7628 on speedup into e0d79c23f4d7f1d8ce4fd7081dbf9d9f58913d52 on master.

Coverage increased (+0.2%) to 98.13% when pulling e8d175ba2ae29cb5645135dee12717db52a8529d on speedup into e0d79c23f4d7f1d8ce4fd7081dbf9d9f58913d52 on master.

Is there a way to reduce the verbosity of coveralls? I receive one email per commit, which makes it a bit difficult to follow real comments.

Ok, I think I made coveralls be quiet.

I fixed the windows bug! It was my fault, of course. I forgot to initialize a double scalar to 0, and it turns out this doesn't behave well on MSVC (oddly, only for py2, not for py3: must be because of the different compiler version?!)
Appveyor is green now for both win32 and win64. (I canceled a duplicate build, triggered by the fact that this branch is on scikit-learn-contrib rather than on my fork, feel free to ignore that.)
I'll remove the ugly _sq thing, update the benchmarks, then ping again for a final review before merging.

This is now ready to merge.

I fixed the windows bug!

Hurray! Squash and merge?

Merged by rebase. :beers:


pip install keybase-api needs root privs

To install the requirements, pip install keybase-api as outlined in the readme.md doesn't work. Simply doing sudo pip install keybase-api is sufficient.

Are you not running a personal python or hombrew'ed python? I have deep aversions to mucking with system pythons and their installed eggs. I'm okay changing the README.md to add the sudo for system pythons.
For dev work I highly recommend a virtualenv python and not running make setup against system python.

According to brew leaves I haven't installed anything custom. Should be plain ol' system python:
$ python --version
Python 2.7.6

$ which python
/usr/bin/python

Yea, system python is owned by someone other than you so you'll always have to sudo to install additional libraries. Best not to mess with system python.
See the updated https://github.com/ianchesal/keybase-python/blob/develop/README.md
I added [sudo] to the install instructions and I added a section under Development for using VirtualEnv to do your dev work. Highly recommend VirtualEnv.


Vb 2060 get contact details and prison details using orchestration service

Setting as a draft PR as needs to be merged post chat with the front end team.


ca.digitalfruition.com Website Updates

Updates to CA website

Added OpenSSL config file

Updated CRLs for root and intermediate CAs (no CRL changes, just new expiry)

Wrong target


Resolve 3 Broken Links to Get aggregate-reports green again

@seankane-msft  when you see this can you take a look and approve? Extremely low pri, but sometime.

@scbedd Switch line 9 out for this in the perfstress docs
4. [Readme](#readme)


Airflow Migration: Build out Infrastructure needed to schedule DAGs and convert to IRSA

User Story
We would like to implement Airflow-related infrastructure that is meant to be hosted in the Analytical Platform accounts in terraform in our Data Platform monorepo.
The whole infrastructure for Airflow is currently managed in Pulumi and the authentication of workloads to the AWS environment is accomplished with kube2iam.
We need to refactor it into terraform as well as convert from the outdated K2IAM plug-in to the AWS-endorsed IRSA approach.
User Type(s)
Ops Engineers, Airflow Engineers
Value

Alignment to tooling standards
Refactor code
Improve security and manageability of the infra

Definition of done

[ ] DAG definitions converted to use IRSA
[ ] Infra developed in data-platform in terraform
[ ] Service accounts provisioned
[ ] IRSA tested
[ ] tests are green
[ ] Follow-on stories raised

Reference
How to write good user stories

@jhpyke to decompose epic into tickets


UWP fonts

Can't get fonts on UWP, at least FontAwesome and Ionicons. Probably you should follow this advise http://stackoverflow.com/a/35476577/2987062  and check "Generate library layout" option in the Build configuration

@alllohaaa I think you're onto something. I was able to get the fonts (material design specifically) working in UWP by making sure that Plugin.Iconize.Material.UWP\Assets\Fonts\iconize-material.ttf existed in the AppX output folder. I accomplished that by:

Creating the folder structure in the root of my UWP project Plugin.Iconize.Material.UWP\Assets\Fonts\
Copying iconize-material.ttf to the new Fonts directory
Including the .ttf file in the project and setting it to "Copy if newer"
Rebuilding and then deploying the project

This isn't really a good solution long term, but I'm not familiar enough with nuget or xamarin to put a pull request together.

I have modified the UWP projects to Generate library layout and updated the nugets. Try the new version, hopefully that has solved your issue. UWP is still in beta but the label and buttons should be working for you.

Maybe I go wrong, but I guess now you have to include assets folder into nuget fonts packages somehow. I have

C:\Program Files (x86)\MSBuild\Microsoft.NetNative\Microsoft.NetNative.targets(396,7): error MSB3030: Could not copy the file "...\packages\Xam.Plugin.Iconize.FontAwesome\1.0.9\lib\UAP10\Plugin.Iconize.FontAwesome.UWP\Assets\Fonts\iconize-fontawesome.ttf" because it was not found.
C:\Program Files (x86)\MSBuild\Microsoft.NetNative\Microsoft.NetNative.targets(396,7): error MSB3030: Could not copy the file "...\packages\Xam.Plugin.Iconize.Ionicons\1.0.9\lib\UAP10\Plugin.Iconize.Ionicons.UWP\Assets\Fonts\iconize-ionicons.ttf" because it was not found.

Just try your own samples in another solution and reference all dependencies through nuget

i have a similar isue, but the path are different.
Each font package missing fonts ..
Tha asked path is "C:\Users\Benoit.nuget\packages\Xam.Plugin.Iconize.SimpleLineIcons\1.0.10\lib\UAP10\Plugin.Iconize.SimpleLineIcons.UWP\Assets\Fonts\iconize-simplelineicons.ttf"
i have copied files in the right dirrectory (i had to create directory structure) and now it work

I have the same problem as @alllohaaa, copying the file in the folder works, but isn't really a good workaround.

I'm still facing the same problem wih the last nuget version.

@werddomain me too

Me too

Same.

I still have the same issue with the last nuget version.

I still have the same issue with the last nuget version.

Also having this problem with the current version of NuGet

Same thing here

After investigation this problem only occurs when we use an IconNavigationPage as Detail of a MasterDetailPage.
I've opened the related issue.


Sort only when sort icon is clicked, not when whole column header is clicked

I would like to have a sort icon in my header, and I would like to activate the sort only when the icon is clicked, not the whole header. As far as I can tell, for sorting, there is no way to move a column header's onClick() handler from the entire  to a child element of the .

You can move the ...getSortByToggleProps() to whatever element you want to be able to use as the toggle.

@jantorestolsvik  did you figure this out?

No, I ended up creating my own tablesetup.

It is working for me as suggested.

             {headerGroups.map(headerGroup => (
                 {headerGroup.headers.map(column => (                                     {column.canFilter?<FaFilter onClick={ () => {column.setFilter('');setFilters({...filters, [${column.id}]: !filters[${column.id}]})}}/>:null}                    {column.render('Header')}                                        {column.isSorted                         ? column.isSortedDesc                             ? <FaSortUp {...column.getSortByToggleProps()} />                             : <FaSortDown {...column.getSortByToggleProps()} />                         : column.canSort?<FaSort {...column.getSortByToggleProps()} />:''}
{column.canFilter && filters[${column.id}]? column.render("Filter") : null}
                 	                 ))}
             ))}


[FEATURE] Support for client-side encryption

Is your feature request related to a problem? Please describe.
Problem is putting sensitive data in the cloud is scary.
Another thing is if one has multiple Cassandra clusters being backed up to the same S3 bucket, one might want to be able to use encryption to make sure one cannot restore other data from the bucket. This can obviously also be done with ACL, but encryption can be an additional layer against that.
Describe the solution you'd like
I'd like

client-side encryption of sstables before they are uploaded to S3.
client-side decryption of sstables after downloaded from S3.

Describe alternatives you've considered
Obviously you can enable server-side encryption in S3, but honestly that's semi-fake security (more compliance) and doens't really protect against an S3 bucket being accessed from the Internet etc. etc.
Additional context
Does this feature need to live in Esop or Icarus? I assume both.
Har this been considered before? I'd be fine with supporting a single encryption key stored in a configuration file or something.

Hi Jens,
thanks for reaching us.
I need to understand this properly, just to be sure. So, my understanding is that if you are uploading SSTable components via "HTTPS" via S3 client, is not that communication somehow protected already?
The "proper" solution, if you want to have these SSTables encrypted, is to encrypt it not by any tooling like this but they just be encrypted themselves. There should be encryption of SSTables implemented directly in Cassandra so we do not need to do something like this.
Long story short, this seems to be quite a big problem and even though we are trying to have encryption on SSTable level in Cassandra, all I can say is that, the most probably, that is just not going to happen anytime soon.
I you do not want to have your data in the cloud stored in plaintext and you are afraid that they would accessed publicly (even though it is hard to believe this might be the case in any serious business entity but I understand mistakes happen), you are welcome to provide a PR which would solve this, I will gladly review it and eventually merge it.
Reading docs for S3 client encryption here (1), I think this would need to be somehow accommodated to reflect what you want.
Just keep in mind to make it optional and command line / payload flag ridden so it can be turned on / off. (off by default).
(1) https://docs.aws.amazon.com/AmazonS3/latest/userguide/UsingClientSideEncryption.html
(2) https://github.com/instaclustr/esop/blob/master/src/main/java/com/instaclustr/esop/s3/TransferManagerFactory.java#L58


Some setup fixes

renamed database variables from POSTGRES_* to DATABASE_* to conform with the current usage in backend and datastore
fixed path in schema application script
QoL formatting changes

I fixed the formal check errors. IMO we should merge this into the feature branch with the errors from validate.py the models.yml.

Sounds good, I deactivated the model validation in the CI for now.


All measures in data model get duplicated when applying a Format String Expression to Calculation Group

Hi,
We have a Power BI data model with 60+ tables.  We have added a simple Calculation Group containing 5 measures with different format types e.g. whole number and percentages, so that we can offer the users the ability to select which measure they want to review and it will be correctly formatted in the charts.
However we have found that when apply the Format String Expression, that every measure in our data model gets duplicated:

If we then clear the Format String Expression and save the changes all the duplicates disappear.
We have done similar on other models and have not seen this issue before.
We have downloaded the latest version of Tabular Editor and started our changes from afresh and the same issue occurred.
Tabular Editor version: [2.15.3]
Power BI Data model - latest April 2021 version.
Any thoughts appreciated!
Thanks
Mark

I can confirm same behavior.

I'm afraid this is a bug in Power BI Desktop (any tool that adds a calc group with dynamic format strings would probably cause the same effect). Will forward it to the team.

In the latest release (2.92.706.0 April 2021) of PBI desktop it's even worse. Now they are spread around within the table, so you don't know witch is witch. Yes, you should choose the not-hidden one (if not both are hidden).

Has anyone reported this to Microsoft, I don't want to create a duplicate, and I never get any votes ;-)

@erapade the team is already aware.

@erapade is this still an issue in the latest version of Power BI Desktop (May 2021)?

@erapade is this still an issue in the latest version of Power BI Desktop (May 2021)?

@otykier Yes still an issue in latest version of PBI Desktop

Still an issue in the latest version of PBI Desktop Version: 2.94.921.0 64-bit (June 2021).
@otykier said, that Microsoft is aware and working on a fix. Anyone knows where can I find information about the status of this fix?
Thank you all in advance.
Special thanks to @otykier for the gorgeous product.

You can help by letting Microsoft know that this is an important issue on the Power BI Community. Either by creating a new post or commenting on these existing threads:

https://community.powerbi.com/t5/Desktop/A-duplicate-hidden-measure-created-when-creating-a-new-measure/m-p/1865688 (this one was marked as solved for some reason)
https://community.powerbi.com/t5/Issues/Working-with-tabluar-editor-cause-creation-of-display-folders/idi-p/1243974

I am having the same problem. All measures have been duplicated and hidden, and I can't publish to Power BI with error message "The remote server returned an error:(400) Bad Request ".
I am using the latest PBI Desktop Version: 2.99.782.0 64-bit (November 2021). All the duplicated measure gone when I cleared the formats from the grouped measures, but I couldn't publish until I cleared all group measures  in Tabular Editor.
Thank you in advance for your help!

Did anyone else experience a recent update on this? Previously all measures got bundled up in their respective folder, "Format String Expressions", and was hidden away. All was fine until (I believe) the december update of PBI. Now they are grouped together with their respective measure, and hidden.


unable to setup anova wifi model a4

Version of the custom_component

e02770c
Configuration
Installed repo via HACS, then added device ID from anova app. Got the below setup error

Describe the bug
Unable to setup the integration
Debug log

Traceback (most recent call last):
  File "/srv/homeassistant3.9/lib/python3.9/site-packages/homeassistant/config_entries.py", line 382, in async_setup
    result = await component.async_setup_entry(hass, self)
  File "/home/homeassistant/.homeassistant/custom_components/anova_sous_vide/__init__.py", line 25, in async_setup_entry
    update = await apc.update(entry.data["device_id"])
  File "/srv/homeassistant3.9/lib/python3.9/site-packages/anova_wifi/parser.py", line 74, in update
    AnovaPrecisionCookerSensor.FIRMWARE_VERSION: anova_status[
KeyError: 'system-info'

Just a bit of info about my HA core setup. I am running things in a venv and have yet to update python to 3.10. I plan to do so before the next HA update obviously. Hopefully this error isn't due to that but wanted to mention it. I am running HA core 2023.1.7

Super interesting. Basically it seems like your http request doesn't have some of the same tags mine does.
can you go to https://anovaculinary.io/devices/{your_device_id}/states/?limit=1 and send me the contents of that page?

[
	{
		"body": {
			"boot-id": "43435004281963",
			"job": {
				"cook-time-seconds": 0,
				"id": "98827197488379",
				"mode": "IDLE",
				"ota-url": "",
				"target-temperature": 65.55,
				"temperature-unit": "F"
			},
			"job-status": {
				"cook-time-remaining": 0,
				"job-start-systick": 1674,
				"provisioning-pairing-code": 0,
				"state": "",
				"state-change-systick": 1674
			},
			"network-info": {
				"bssid": "HIDDEN",
				"connection-status": "connected-station",
				"is-provisioning": false,
				"mac-address": "HIDDEN",
				"mode": "station",
				"security-type": "WPA2",
				"ssid": "HIDDEN"
			},
			"pin-info": {
				"device-safe": 1,
				"water-leak": 0,
				"water-level-critical": 1,
				"water-temp-too-high": 0
			},
			"system-info-3220": {
				"firmware-version": "1.4.4",
				"firmware-version-raw": "VM176_A_01.04.04",
				"largest-free-heap-size": 28008,
				"stack-low-level": 232,
				"stack-low-task": 7,
				"systick": 6154,
				"total-free-heap-size": 28632
			},
			"system-info-nxp": {
				"version-string": ""
			},
			"temperature-info": {
				"heater-temperature": 17.88,
				"triac-temperature": 19.24,
				"water-temperature": 19
			}
		},
		"header": {
			"created-at": "2023-01-29T00:06:16.874554Z",
			"e-tag": "f3948adb476fe680798cad480a6f85a0acc367d1b3b74f5d3c1afa4513823af2",
			"entity-id": "69630009806838"
		}
	}
]

Nice thank you, I should be able to resolve this pretty easily. What model of Sous vide do you have?

AN500-US00
the anova wifi 1000w model

Okay updated! Give it another try. You may need to remove the repository and re-add it maybe even a restart, not fully sure how hacs handles repo updates.

ok updated to version 1.0.1, I basically asked HACS to redownload and saw the latest commit. Now I see a slightly different error :)
Traceback (most recent call last):
  File "/srv/homeassistant3.9/lib/python3.9/site-packages/homeassistant/config_entries.py", line 382, in async_setup
    result = await component.async_setup_entry(hass, self)
  File "/home/homeassistant/.homeassistant/custom_components/anova_sous_vide/__init__.py", line 25, in async_setup_entry
    update = await apc.update(entry.data["device_id"])
  File "/srv/homeassistant3.9/lib/python3.9/site-packages/anova_wifi/parser.py", line 78, in update
    AnovaPrecisionCookerSensor.FIRMWARE_VERSION: anova_status[system_info][
KeyError: 'firmware-version'

oops, forgot you had that 'system-info-nxp' - should be resolved now.
Let's see what bug pops up next :)

So after a restart I had a timeout error but it did initialize. I then asked the config entry to reload and I got a device and entities!

I did see them go unavailable for about a minute but then they got all the proper data! I did notice that the URL you gave me sometimes would output [] then a refresh or 2 later and the data was there.
Thank you!

Awesome!
Yeah once it gets added it goes to unavailable and after one minute it refreshes. I've been thinking about trying to change that, but I think it is fine. It just loads on the first update instead of on startup.

Thank you for the quick fixes! Sure I will open another issue if anything else comes up 😃  Got a few cooks to do tonight so I'll give a comment on the main PR a little later. For this issue its definitely fixed! 🎉


Add description for running specific test

Coverage remained the same at 70.113% when pulling ab58970a72842e32c3ebb663b88dfa1d15d0dee1 on timifasubaa:patch-3 into f87163413bff72935fc6c95fa9d34468de99b86e on apache:master.

Coverage remained the same at 70.113% when pulling ab58970a72842e32c3ebb663b88dfa1d15d0dee1 on timifasubaa:patch-3 into f87163413bff72935fc6c95fa9d34468de99b86e on apache:master.

Coverage remained the same at 70.113% when pulling ab58970a72842e32c3ebb663b88dfa1d15d0dee1 on timifasubaa:patch-3 into f87163413bff72935fc6c95fa9d34468de99b86e on apache:master.


Fix 68

Coverage remained the same at 92.866% when pulling a98b28c817fa2c903fa8d825c1c77bb03cafc76d on fix-68 into 0a6f1565373a229c6abd86d70d4e4e98daa80c5a on master.


Unclear for mutiple apps

It’s unclear from the documentation provided, how to orchestrate across multiple app instances. For example if I have web app on one instance and say a api on another instance. The web app depends on the api being swapped before it should be swapped. How do ensure, that both are swapped together? For example if a swap fails on either one then you want to whole swap rolled back for both.

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

ID: f6e09089-1ae2-8943-5ce2-9d48f458c81f
Version Independent ID: ba780cba-f604-b0a4-a81a-23c7d2384762
Content: Set up staging environments for web apps in Azure App Service
Content Source: articles/app-service/deploy-staging-slots.md
Service: app-service
GitHub Login: @cephalin
Microsoft Alias: cephalin

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

+1 for multiple apps swap status reporting!

Upon further review we feel your feedback would be best received by the doc author to determine if an update to the current doc is necessary, or to create a new doc if they align with the feedback about swapping multiple apps.

@jwisner This sounds like a scripting scenario and goes beyond the scope of this article. If you orchestrate with a script then you would verify the first swap before moving on, and so forth. If you need help on that, I suggest going to the Azure forum or StackOverflow. #please-close


Placeholder Analytics 11.0.0

Placeholder Analytics 11.0.0 #no-changelog

I'll wait to see some results from the SPM CI. Then, I'll merge, re-tag, and stage.

investigating spm failures locally...

Latest commit to address these failures: https://github.com/firebase/firebase-ios-sdk/actions/runs/9913363297/job/27390217206


Hipchat deploys

Send a helpful message if a user activates frontend to production!
we need to reference the new version in frontend once this is merged and published!

One question above but I think it'g good besides that! :+1:


Better emoji

Make list of the best emojis and use that instead.
http://whatmoji.com/

Also, I think there's a bug with non-supported emoji. I do think that #203 helps fix this bug, but the real fix will be to use a solid list of the "best" emoji.
Here's what I have so far:
☠ 🐲 🔆 👀 🐀 🐛 💰 🎨 🌵 🍡 ❌ 🌞 🍕 💥 🙀 🍇 🍺 🐟 🍩 🔥 👤 🍭 👊 🍔 💋 👺 🍤 🌰 🐞 🍒 👹 🚀 🐓 🌚 👻 😺 🍔 🤓 💩 👾 📍 🗯 🍟 🐌 🐸 🍗 👑 🌻 ✨ 💊 🦄 🍌 🐗 ⭐ 💧 💫 🐙 🍻 🍭 🐌 🐾 🐥 🍀 🍉 🍦 🍗 🍎 ⚔ 🍁 🍄 🙈 🔫 🐔 💀 🐂 🏃 👙 🐷 🌞 🐖


Initial API proposal

What type of PR is this?

initial API proposal

What this PR does / why we need it:
This is the initial proposal from Vodafone for the KYC Tenure API
Which issue(s) this PR fixes:

Fixes #12
Special notes for reviewers:
None
Changelog input
 release-note
 - Initial API proposal

Additional documentation
None

@GillesInnov35
Once the rules on generating the tenure response are agreed, then these can indeed be added to the API definition as documentation.
But there is no need to wait for these rules to be agreed before merging this PR. This is just a "work in progress" version, which will be subject to further updates before being released as an initial version. Further updates can be pushed to the repository as they are agreed.

Thanks @GillesInnov35
I won't merge this until others have raised any questions they may have, and have given their approval as well. It should also be discussed during at least one of the bi-weekly KYC meetings. As the next meeting is tomorrow, I think we are looking at waiting until after the subsequent meeting before merging this.
And I also need approval for PR #13. Currently, the YAML linter does not like line lengths more than 80 characters. That PR is required so that that rule can be disabled.

OK Thanks a lot Eric

Hi @GillesInnov35. I was re-adding @camaraproject/tenure_maintainers as a reviewer to bring this PR to the attention of Telefonica, who have no codeowner for this repository, but are maintainers. Now I have just added @claraserranosolsona and @jgarciahospital directly as reviewers, which is perhaps simpler. I would like Telefonica feedback on this proposal before merging.

@eric-murray, tenureDateCheck is currently a boolean attribute. I wonder if the API will have also to return the information that the date comparison could not been done because the MNO do have the correct information, and then return "not_available".
What do you think about that ?
BR
Gilles

Hi @GillesInnov35
Can you clarify your proposal? Is this to handle mobile subscribers for which no answer can be given, because their tenureDate is just not known? Or is it to handle the situation that the requested tenureDate is so long ago that the MNO cannot verify that date, but could verify a later date?
The assumption behind the API design was that each mobile subscriber has a recorded tenure date with no limit on how long ago that was. So if a subscriber had the same MSISDN for, say, 20 years, then the tenure date would be known to be 20 years ago, and not "more than 10 years", for example.
Do we need to consider making the requested tenureDate relative? So rather than the API consumer asking about tenure since, for example, 14-10-2004, they would ask about tenure over the last 20 years. The advantage of using relative time periods is that a maximum can be easily specified in the OAS definition, and then all implementations would be expected to support this maximum.
Because date-time is a string format, a "minimum" or "maximum" date cannot be built in to the OAS definition itself, though of course the documentation could specify such a limit. But trapping dates that do not comply with such a requirement takes a bit more work.

Thanks Eric
Yes my wondering concerned the case when no answer can be given by the MNO. What will happen when the user is not the owner of the contract according to MNO and countries privacies ?
For the second point, I agree with you, using relative time period seems to to be more useful and simplier.
Perhaps to be reviewed in project meeting.
BR
Gilles

Other way, would be to return the date when the line was opened.
in this case operation would be POST /retrieve-tenure

What will happen when the user is not the owner of the contract according to MNO and countries privacies ?

Commonalities provide an error that can be used when a given service does not apply to a given "device" (in this case, subscription):
            GENERIC_422_DEVICE_NOT_APPLICABLE:
              description: Service is not available for the provided device
              value:
                status: 422
                code: DEVICE_NOT_APPLICABLE
                message: The service is not available for the provided device.

This was intended for cases such as querying the SIM Swap API with a fixed line phone number. I included this error in the proposed YAML.
Curiously, the SIM Swap API did not adopt this error, but instead defines:
            GENERIC_422_NOT_SUPPORTED:
              description: Not Supported
              value:
                status: 422
                code: NOT_SUPPORTED
                message: Service not supported for this phoneNumber

which is maybe because "device" can be viewed as a confusing term to use when it is identifying the subscription. But the mechanism is the same - there is an explicit 422 error defined for this case, rather than returning a 200 response which, on closer inspection, turns out to be an error.

Thanks Eric, OK it works and the response error seems to be clear. So "not-available" is not useful.
Gilles

Hi @HuubAppelboom
The message is just an example, and API providers can always change it to something more relevant. Indeed, we could change the example in the OAS definition if we prefer a different message to that in the Commonalities common JSON file.
What does need to be standardised are the codes. Code NOT_SUPPORTED is not defined as a valid code by Commonalities, so that needs to be fixed. Either SIM Swap (or one of the other APIs that use this code) need to submit it to Commonalities, or they need to adopt an existing valid code.
Commonalities should tighten up code definitions for the next update of their design guidelines, so let's see what they define then.

@fernandopradocabrillo, @grgpapadopoulos
What is your view on the current proposal?

@fernandopradocabrillo @GillesInnov35 @grgpapadopoulos
Any update on your acceptance or otherwise of this PR following the recent KYC meeting?


Start exposing Aura features

its a seperate entry for now
Thank you for your contribution to the Graph Data Science Client project.

Before submitting this PR, please read Contributing to the Neo4j Ecosystem.
Make sure:

[ ] You signed the Neo4j CLA (Contributor License Agreement) so that we are allowed to ship your code in our library
[ ] Your contribution is covered by tests

Also worth a try to generate the bindings as a starting point (such as via https://github.com/OpenAPITools/openapi-generator)


How to have both emoticons and full emoji list using sprite mode ?

Hi,
First, thanks for the repo !
I'm developing a plugin for a wysiwyg with emojify.js using sprite mode. The issue I have is that I can't manage to have emoticons emojis combined with the full list of emoji using sprite mode.
For example, I can't find the emoji-smile class in this css file : https://github.com/Ranks/emojify.js/blob/master/dist/css/sprites/emojify.css
And if I combine both css in the sprite directory, it leads to uncorrect background positions and image as well.
I might be doing something wrong since the repo exists since a few years now but I'd be glad to get some hints here.
Thanks !

I've just had the same problem. For example, with the following input string:
:helicopter: :school: :smile:
If I use the data-uri css I get only :smile:. If I use the sprites css I get 🚁  🏫  but not 😄.

Someone found a solutions for this?

I'm sorry the project isn't maintained any more, feel free to fork this repo and make any changes to it. Sorry 😬

@hassankhan Can you please put a prominent "No longer maintained. Refer to Issues for existing problems" message in the README.md?
This repo still shows up in Google searches, and finding the project's status upfront on the README.md would be helpful.


Add config for allowing GC to clean unpacked layers up

Fixes: #4304
This commit adds a flag through Pull API for allowing GC to clean layer contents up after unpacking these contents completed.
This patch takes an approach to directly delete GC labels pointing to layers from the manifest blob. This will result in other snapshotters cannot reuse these contents on the next pull. But this patch mainly focuses on CRI use-cases where single snapshotter is usually used throughout the node lifecycle so this shouldn't be a matter.
Could I get comments on this approach towards unpacked contents deduplication?
What I currently concern is whether is it OK to clean layers up immediately after the unpack because other services in containerd will also be inaccessible to that contents.

But this patch mainly focuses on CRI use-cases where single snapshotter is usually used throughout the node lifecycle so this shouldn't be a matter.

For now, it is fine. But for multi-tanancy cases, it might be issue. Depends on the design

What I currently concern is whether is it OK to clean layers up immediately after the unpack because other services in containerd will also be inaccessible to that contents

I think content can be locked by lease. If there are multiple pullers for same layer, no need to worry about it. But if the service need to unpack for other snapshotter, it will be bad.

Change is good but I think we need to hold it because release1.4 candidate is still going. :P

@dmcgowan PTAL?

I'm working on a modification of this. The filtering should really happen during SetChildrenLabels rather than explicitly removing them after unpack. Like @fuweid, ideally this would not just remove image configs as well. I had another patch related to this, I'll get that as a patch on top of this.


fix: change sp to gui in highlights to properly apply underline etc..

Hey, I noticed that underlines were not working with the theme and realised you were using sp which sets guisp and not gui which sets the correct highlight attribute for things like underline, bold etc..
Hope the commit message follows the repo standards and this patch works for you :)

Hey, thank you for finding and fixing it!
And yes, the commit message is fine. Merging it right now :heart:


[Configuration Options] Products

This Billing Panel almost perfect, however it sadly lacking the ability to have configables options for products that some eggs require, if you guys are able to add this that would be amazing, Also the discord seems dead

Hi! Yeah, sure! Could you provide an example of an egg that requires configurable options? This would help us a lot.
Thanks!

Sure thanks, https://github.com/pelican-eggs/eggs/blob/master/game_eggs/gta/fivem/egg-five-m.json do you guys have an working discord?

Thanks for sharing the egg configuration example. We’ve noted your request and plan to add support for configurable product options in the near future.
Regarding Discord, you can join this server: https://discord.gg/zHmKdssh and contact me directly (nickname: frostowski). I’ll be happy to assist you there!
Thanks again for your feedback!


PHOENIX-6270 : Use TableDescriptorBuilder to set coproc SystemCatalogRegionObserver and avoid using ImmutableHTableDescriptor

Fix SystemTablesCreationOnConnectionIT.testUpgradeNotAllowed() failure caused by internally using read-only ImmutableHTableDescriptor to set coproc SystemCatalogRegionObserver

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
0m 35s
Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s
No case conflicting files found.

+1 :green_heart:
hbaseanti
0m  0s
Patch does not have any anti-patterns.

+1 :green_heart:
@author
0m  0s
The patch does not contain any @author tags.

-1 :x:
test4tests
0m  0s
The patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch.

_ master Compile Tests _

+1 :green_heart:
mvninstall
14m 25s
master passed

+1 :green_heart:
compile
0m 53s
master passed

+1 :green_heart:
checkstyle
1m 24s
master passed

+1 :green_heart:
javadoc
0m 44s
master passed

+0 :ok:
spotbugs
2m 52s
phoenix-core in master has 972 extant spotbugs warnings.

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
7m 25s
the patch passed

+1 :green_heart:
compile
0m 54s
the patch passed

+1 :green_heart:
javac
0m 54s
the patch passed

-1 :x:
checkstyle
1m 24s
phoenix-core: The patch generated 13 new + 2191 unchanged - 16 fixed = 2204 total (was 2207)

+1 :green_heart:
whitespace
0m  0s
The patch has no whitespace issues.

+1 :green_heart:
javadoc
0m 45s
the patch passed

+1 :green_heart:
spotbugs
3m  6s
the patch passed

_ Other Tests _

-1 :x:
unit
104m 56s
phoenix-core in the patch failed.

-1 :x:
asflicense
0m 27s
The patch generated 1 ASF License warnings.

142m 39s

Subsystem
Report/Notes

Docker
ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1025/1/artifact/yetus-general-check/output/Dockerfile

GITHUB PR
https://github.com/apache/phoenix/pull/1025

Optional Tests
dupname asflicense javac javadoc unit spotbugs hbaseanti checkstyle compile

uname
Linux bddf61a4e9f9 4.15.0-112-generic #113-Ubuntu SMP Thu Jul 9 23:41:39 UTC 2020 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev/phoenix-personality.sh

git revision
master / 6e019dd

Default Java
Private Build-1.8.0_242-8u242-b08-0ubuntu3~16.04-b08

checkstyle
https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1025/1/artifact/yetus-general-check/output/diff-checkstyle-phoenix-core.txt

unit
https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1025/1/artifact/yetus-general-check/output/patch-unit-phoenix-core.txt

Test Results
https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1025/1/testReport/

asflicense
https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1025/1/artifact/yetus-general-check/output/patch-asflicense-problems.txt

Max. process+thread count
6742 (vs. ulimit of 30000)

modules
C: phoenix-core U: phoenix-core

Console output
https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1025/1/console

versions
git=2.7.4 maven=3.3.9 spotbugs=4.1.3

Powered by
Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.


Unknown theme

When using default theme 'bootstrap3' I have an error in console
"Unknown theme bootstrap3"

I don’t get the form, is it due to a theme error, or am I mistaken somewhere?
in the dev console i see:

Uncaught Error: Unknown theme bootstrap3
at new Z (jsoneditor.min.js:18)
at (index):241

my code:
DATA_SCHEMA = {
    'type': 'array',
    "title": "Смета",
    "format": "table",
    "items": {
        "type": "object",
        "title": "Смета",
        "properties": {
            "name": {
                'title': 'Критерий',
                "type": "string"
            },
            "cost": {
                "type": "number"
            }
        }
    }
}

class ReportSmetaAdminForm(forms.ModelForm):
    class Meta:
        model = Report
        fields = '__all__'
        widgets = {
            'smeta': JSONEditorWidget(DATA_SCHEMA, collapsed=False),
        }

@admin.register(Report)
class ReportAdmin(admin.ModelAdmin):
    list_display = ('project', 'status',)
    form = ReportSmetaAdminForm

New json-editor does not support bootstrap 3 now. As workaround it is possible to use the html theme. I am going to update plugin to use bootstrap 4 as default.

how to use html theme?

It is possible to pass any options into JSONEditor via the editor_options parameter of JSONEditorWidget.
widget = JSONEditorWidget(schema, False, editor_options={'theme': 'html'})

@Gustik @cyberbudy check  0.2.3 version please.

thank you, it works!


Fix ORC reader for CHAR(N) columns written from Hive

Fixes #8294.
This commit fixes reads of fixed-length strings when written through Apache Hive.
Before this fix, when CHAR(N) columns (i.e. fixed-length strings) were written to ORC files via Apache Hive, those columns were unreadable through the RAPIDS Plugin for Spark. Reading such a column produces the following error/stacktrace:
Caused by: org.apache.spark.sql.execution.QueryExecutionException: GPU ORC does not support type conversion from file type char(2) (1) to reader type string (1)
  at com.nvidia.spark.rapids.GpuOrcFileFilterHandler$GpuOrcPartitionReaderUtils.checkTypeCompatibility(GpuOrcScan.scala:1536)
  at com.nvidia.spark.rapids.GpuOrcFileFilterHandler$GpuOrcPartitionReaderUtils.$anonfun$checkTypeCompatibility$4(GpuOrcScan.scala:1506)
  at scala.Option.foreach(Option.scala:407)
  at com.nvidia.spark.rapids.GpuOrcFileFilterHandler$GpuOrcPartitionReaderUtils.$anonfun$checkTypeCompatibility$3(GpuOrcScan.scala:1504)
...

The same data, when written through Spark, remains readable from the plugin.
This arises from a difference in serialization between Apache Hive and Apache Spark. When a CHAR(2) column is written through Hive, the type of the column is preserved:
Processing data file file:/tmp/hive.00000_0.orc [length: 212]
Structure for file:/tmp/hive.00000_0.orc
File Version: 0.12 with ORC_517
Rows: 1
Compression: ZLIB
Compression size: 262144
Type: struct<id:char(2)>

When Spark writes the column, its type is changed to STRING:
Processing data file file:/tmp/part-00000-ecbe264b-fa68-4d26-8c56-ba45602263b7-c000.snappy.orc [length: 200]
Structure for file:/tmp/part-00000-ecbe264b-fa68-4d26-8c56-ba45602263b7-c000.snappy.orc
File Version: 0.12 with ORIGINAL
Rows: 1
Compression: SNAPPY
Compression size: 262144
Type: struct<id:string>

This fix adds support to cast CHAR(N) to STRING in the ORC Reader, alongside VARCHAR.

My only comment is that it might be nice to check in a small test file and add an integration test so we don't regress in this.

I think we have a problem here with strings that are shorter than N. I'm trying isolate that behaviour and understand it.

Ok, I can confirm that there's a discrepancy between the behaviour of Apache Hive/Spark on one side, and CUDF/Plugin on the other.
Consider the following data written via Apache Hive:
-- Insert as CHAR(3).
CREATE TABLE foobar (foo CHAR(3)) STORED AS ORCFILE LOCATION 'file:///tmp/foobar';
INSERT INTO foobar VALUES (""), ("0"), ("1 "), (" 1"), ("22"), ("4444");

-- Read as STRING.
CREATE EXTERNAL TABLE foobar_as_string (foo STRING) STORED AS ORCFILE LOCATION 'file:///tmp/foobar';

With Hive or Spark, reading data from either table above produces the different results results from CUDF or the RAPIDS plugin:
SELECT foo, CONCAT('"', foo, '"'), LENGTH(foo) FROM foobar;

From Spark/Hive: (Trailing spaces are removed, even if the data had it during insertion.)
foo     quoted  len
        ""      0
0       "0"     1
1       "1"     1
 1      " 1"    2
22      "22"    2
444     "444"   3
Time taken: 0.074 seconds, Fetched: 6 row(s)

From RAPIDS: Fixed length strings, as the ORC spec would dictate.
        "   "   3
0       "0  "   3
1       "1  "   3
 1      " 1 "   3
22      "22 "   3
444     "444"   3

This will need post-processing, before it matches what Apache Spark produces.

Build


Added a failing test that demonstrates unexpected Alert dismissal

Added a failing test that demonstrates unexpected Alert dismissal when a non alert destination action is sent.
See this slack thread for reference: https://pointfreecommunity.slack.com/archives/C04KQQ7NXHV/p1694623754410579
When using @PresentationState with an enum destination, if an Alert is currently presented, and an action for a different destination is sent to the store, the Alert is dismissed even though no alert action was sent.
The alert should only be dismissed if an alert action is sent to the store.

Curiously, when I run the test I see this error, which is what I would have expected to see in our code, but didn't, indicating that this kind of thing is really programmer error:
/Users/michael/Documents/Projects/swift-composable-architecture/Tests/ComposableArchitectureTests/Reducers/PresentationReducerTests.swift:2530: error: -[ComposableArchitectureTests.PresentationReducerTests testPresentation_leaveAlertPresentedForNonAlertActions] : A "Scope" at "ComposableArchitectureTests/PresentationReducerTests.swift:2507" received a child action when child state was set to a different case. …

  Action:
    PresentationReducerTests.Parent.Destination.Action.child(.decrementButtonTapped)
  State:
    PresentationReducerTests.Parent.Destination.State.alert

This is generally considered an application logic error, and can happen for a few reasons:

• A parent reducer set "PresentationReducerTests.Parent.Destination.State" to a different case before the scoped reducer ran. Child reducers must run before any parent reducer sets child state to a different case. This ensures that child reducers can handle their actions while their state is still available. Consider using "Reducer.ifCaseLet" to embed this child reducer in the parent reducer that change its state to ensure the child reducer runs first.

• An in-flight effect emitted this action when child state was unavailable. While it may be perfectly reasonable to ignore this action, consider canceling the associated effect before child state changes to another case, especially if it is a long-living effect.

• This action was sent to the store while state was another case. Make sure that actions for this reducer can only be sent from a view store when state is set to the appropriate case. In SwiftUI applications, use "SwitchStore".

Hi @mluisbrown, yeah this is definitely a programmer error in general. Actions should never be sent to a destination that is not being presented. That message probably was printed to the console in your app, but you may have not noticed. We also show a purple Xcode runtime warning, but maybe you missed that too?
Now, if it's possible to not dismiss the alert in this case we will happily do it just to make the experience a little nicer, but ultimately it would still be a programmer error.

Closing in favor of #2468, which includes this patch. Thanks @mluisbrown!


Feature: Empty State Component

[x] Finished implementation of empty state for all variant

https://user-images.githubusercontent.com/104057020/179133639-662ee6f4-7b50-4a65-b158-16c0fe447375.mov

:tada: This PR is included in version 1.1.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


How to avoid extra calls to main when creating web worker from binary?

Summary
For context: I've been experimenting with creating a library that spawns web workers that share memory with each other and the main thread.
I've been basing my experiments off of the Raytrace Parallel example: https://github.com/rustwasm/wasm-bindgen/tree/master/examples/raytrace-parallel
An issue I've run into is that __wbindgen_start is called whenever init is called. So the binary's main is invoked extra times whenever the library tries to spawn a worker and call init for it.
Is there some way to disable the automatic call to __wbindgen_start so that it can be invoked only for the main thread? Or should a special init be introduced for web workers?
Additional Details
Work arounds:

Require the binary using the library to insert some sort of check into main to detect that they're on a worker thread.
Compile the binary as a lib and manually invoke start

Currently this can't be done, you'll need to orchestrate something without the start function. The way multithreading works is to create an instance on each thread, which inherently runs the start function.

@alexcrichton would it be acceptable to add a function like initWithoutStart? Basically a copy of init but without the call to __wbindgen_start() at the end. Or is there some other reason/limitation why this isn't possible?


Allow custom humanize keys

Added a parameter to humanize function so we can specify what parts we want.
For the humanize call:
=> "1 Year, 1 Month, 3 Weeks, 5 Days and 18 Hours"
If we call humanize([:years, :months]) the output contains only the informed parts:
=> "1 Year and 1 Month"

Guys, how much did you think about this? on the surface, making us maintain this gem forever, over something that could be done with a simple transform and a utility function, sounds like a poor choice. Did you have other reasons?

The time I would spend making the same code as this, and adding tests is not worth it for a page that is faded to go away (old form). It's going to be remade with react and then maybe is more simple to do that than using a 3rd party lib.
Also, we have no intention to maintain that. The idea is to open a PR to main repo and get it merged (maybe will require some modifications to get accepted).


Fix legacy task list position calculation

This PR fixes a bug where the combination of a tasklist + legacy task list in the side panel would cause a miscalculation of the checkbox position and result in the wrong checkbox being updated. Preventing checkboxes within code blocks from being included in the position calculation prevents this bug and other potential bugs (e.g., where a user includes the - [ ] checkbox notation within a code block).
Note: this fix was tested and works in both the previous and new issue viewer in Memex.

Closes https://github.com/github/issues/issues/6850

Screenshots
Before
https://github.com/primer/react/assets/25596676/76c5c8e7-639a-4049-855d-a7f66845589b
After
https://github.com/primer/react/assets/25596676/91521149-dcd7-4a7f-b979-286bf705e02b
Merge checklist

[ ] Added/updated tests
[ ] Added/updated documentation
[ ] Changes are SSR compatible
[ ] Tested in Chrome
[ ] Tested in Firefox
[ ] Tested in Safari
[ ] Tested in Edge

Take a look at the What we look for in reviews section of the contributing guidelines for more information on how we review PRs.

Thanks for fixing this bug! ✨
Looks good to me. Should we add a test for this?

@JelloBagel Test added. :)

FYI @siddharthkp for release next week this would be great to get in!


Complete lab07

Fixes #384
Review of colleague's PR #351
Changes proposed in this PR:
next in line
If this PR is ok, please also review: https://github.com/anz-bank/go-course/pull/422

Complete lab07
Copy code from lab06
Improve tests
Add custom Errors
Add Errors to the storer interface declaration
Improve Error handling
Change Value type from string to int
Add leveldb storer

@rokane  Could you please review this?

@camh-anz
All fixed. The real bug with p.ID was already fixed in the lab09. I just forgot to backport it.

you can see just the diff by: git diff c55ecb865add1a99d3339a26ff14064074c51393..  The old head will be in history for next 2 or 4 weeks I guess.

Tests are marked with numbers. It's quite easy to find a correlation for ID and implementation by checking: https://github.com/anz-bank/go-course/pull/383/files#diff-60be13442f5169ff16cc0d9a3084bb8bR208
If it's required I can implement this feature. All remaining issues already solved

should be ok now. Sorry for bin file

@rokane Thanks for review. I decided to get rid of leveldb due to problems with 100% coverage.

@rokane I'm afraid I cannot merge.
Despite your approve I've got the following issue:
git checkout -b n0npax-lab07 master
git pull https://github.com/n0npax/go-course.git lab07
git checkout master
git merge --no-ff n0npax-lab07
git push origin master

is denied by:
git push origin master
Counting objects: 15, done.
Delta compression using up to 8 threads.
Compressing objects: 100% (15/15), done.
Writing objects: 100% (15/15), 5.79 KiB | 2.89 MiB/s, done.
Total 15 (delta 3), reused 0 (delta 0)
remote: Resolving deltas: 100% (3/3), completed with 2 local objects.
remote: error: GH006: Protected branch update failed for refs/heads/master.
remote: error: Waiting on code owner review from anzboi, anzdaddy, camh-anz, Joshcarp, juliaogris, linuxmonk, pentaphobe, rokane, and/or samuong.
To github.com:anz-bank/go-course.git
 ! [remote rejected] master -> master (protected branch hook declined)
error: failed to push some refs to 'git@github.com:anz-bank/go-course.git'

The same issue is when using web UI.
Can you try to re-approve?


Origin Station Update 13.06.2023 (TEG, Borg, Trash)

Added - TEG
Added - Trash bins in maints
Added - borg charging stations (no borg start role)
Added - more decal work to some areas.

[x] I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

why no borg start role i wanna be a borg main

I dont see a charger in robotics

why no borg role

why no borg role

fairly sure its not considered exactly reasy yet


Package request: Brasero

Project description
A free and open-source disc-burning program for Unix-like operating systems
Metadata

homepage URL: https://wiki.gnome.org/Apps/Brasero
source URL: https://gitlab.gnome.org/GNOME/brasero
license: GNU General Public License
platforms: unix, linux,

@DDrozdzz-1066 It's already present. I can find two packages: brasero and brasero-original.
See: https://search.nixos.org/packages?channel=24.05&from=0&size=50&sort=relevance&type=packages&query=Brasero

@DDrozdzz-1066 closing as per @TanvirOnGH 's comment above


White space at the end when using variableWidth

When using variableWidth, white space is shown to the right side at the end of slick-track, between slick-list and slick-track.
The width of slick-track is correctly calculated, but it seems that it can be scrolled to far to the left.
Example: http://jsfiddle.net/qbtcw1yL/
Is there any way to change this?

Yeah there really isn't a good way to get around that. It is in centerMode so the active slide is centered.

I've noticed this issue too and to be clear, this doesn't happen only in centerMode. If you disable centerMode, you get even more white space. Is it somehow possible to disable dragging/swiping when you have the last slide visible to your most right (and not most left)?

Without infinite: true and centerMode: true, how can I limit slick to stop at the outer edge of the last element?

Same problem here.

I made an experimental feature that helps in this situation. See #2635. Hope it helps.

i use slick js tab when first  tab con tent have more data, than go another tab content there is no data then unwanted scroll on second tab content how we fix it

same problem here. Any possible solutions ?

Hey @kenwheeler
I have the same problem with white space when I using infinite: !1; with variableWidth: !0; Too much white spacing is coming on the last slider
Also, I need to disable the arrows next and prev

Did anyone solve this problem?

Hey @kenwheeler
No one has solved these issues.

No one has solved these issues.

just use swiperjs


LOCAL_AUTH_LIST_MAX_LENGTH has incorrect value

It's not actually used within the crate, but the value of rust_ocpp::v1_6::messages::change_configuration::LOCAL_AUTH_LIST_MAX_LENGTH is incorrect; right now it's "LocalAuthListEnabled", probably copied and pasted from LOCAL_AUTH_LIST_ENABLED.

@devanlai Thank you, I'll check it out


[BEAM-12964] [Playground] Add code editor text field

Add code editor text field and load sample code on load (currently only mocked data).
Added a new library code_text_field (with MIT license).

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Choose reviewer(s) and mention them in a comment (R: @username).
[ ] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue, if applicable. This will automatically link the pull request to the issue.
[ ] Update CHANGES.md with noteworthy changes.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.

GitHub Actions Tests Status (on master branch)

See CI.md for more information about GitHub Actions CI.

R: @damondouglas
cc: @pabloem

@pabloem LGTM

thanks!


about the module “Path”

When I was converting the dataset according to 3D-Reconstruction-Kinect/3D_generation.sh, I met the problem running this file: convert_shapenet10.py.
Here is the error message:

Traceback (most recent call last):
File "convert_shapenet10.py", line 5, in
from path import Path
ModuleNotFoundError: No module named 'path'

Although I googled it many times, but I still don't know how to solve it, 'cause it seems that there is no such module called "path". Does it matter if not running this file? It seems important..
My version is python 3.6 by the way, with tensorflow 1.13.1 in Windows system

I had troubles with this as well.  Try pip install path.py.  That should fix your issue.

Thank you.I have already solved this issue.Hope to keep in contact with you if there are more problems.


Question: Is there a way to force order of scripts?

I have a component that I personally made that depends on jquery but when using this plugin it is inserted above it.
Is there a way to force wiredep to order the installation in a different way?

What's your component?
wiredep uses the dependencies in a components' bower.json. So if you properly set jquery as a dependency there, wiredep knows that jquery needs to be included before it.

well basically I have am doing two things.  (One of which is a little odd...)
the first one is I am modifying jquery so that the $(document).ready does not fire till after the webcomponentjs fires its events.  This is because I am using the webcomponents and I am trying to get a little backward compatibility.
The second thing I am doing (also has to deal with webcomponents) is I have to modify jquery so that it only runs once.  I am using this component to make that happen.  It basically inserts a function at the top of every library that I only want to run once.
https://github.com/dtracers/validateFirstRun
This other problem is that technically every library that uses the firstRun component has a dependency on it.  But I don't think it would be a good idea to change the bower file of jquery.  Do the overrides contain this ability?

Yes, I think so? Still a little confused as to what you're doing. But if your project needs jQuery to work, you want to list it as a dependency.

So let me use code examples instead of word examples.
order wiredep is inserting code
jquery
validateRunOnce
myLibraryThatDependsOnJquery

the order I need is
validateRunOnce
jquery
myLibraryThatDependsOnJquery

Thank you for explaining in so many ways :) I think this is a use case we just don't want to support. It's pretty weird (no fault there; making a website is weird in general!), and the fact we haven't run into this before makes me feel comfortable with the options we give developers now.
I'm not sure this will work for your use case, but maybe using exclude and writing necessary script tags manually will do the trick?


Error: Request failed with status code 400

◓  Generating the commit message.
│
└  ✖ Error: Request failed with status code 400

It may be that the country I am in is not supported.
I  can't find where to set it the proxy.

hey @wudi
i dont think its the country, you would get smth like 401 or 403. there is an option: file is really big (more than 4096 token).
try commiting different smaller file, if it helps — let me know, i'll check on the that and make error messages more descriptive if possible <3

I have the same problem!

I have the same problem!

does commiting smaller file work?

Just tested with a small file like "readme" and it is working

@homerokzam @wudi does smaller files work for you guys?

ok, looks like sending large files in batches to openAI breaks somewhere. Anyone wants to investigate and make a PR? :)

I just received this error today with a minor change to one file. I was also able to commit no problem with several larger files earlier today with no error.

I have the same error with changed in files which are ~2k
I also found out that even if the github page says npm 1.1.24 I am getting 1.1.23 no matter how I force it.
It may be a commit issue ?

I received this error today with a minor change to one file. I was also able to commit no problem with several larger files earlier today with no problem.

Not a resolution, but it is definitely related to the number of commits. I restored different staged files and tried again and it worked. However, it is a shame that this limitation exists. Also, it is related to when you delete files or folders in your repo.

I can confirm that this Error: Request failed with status code 400 occurs for large changes. I formatted only one html file with 182 lines of code. oc --no-verify does not help. All good for small changes (few lines of code) in multiple files.

Getting the same error with a simple 1 line change to a README.
I used curl on the api to make sure my token worked and there wasn't any network issues. It would be nice if there was a -verbose or -debug flag to get more details on what the error is.

I'm encountering the same error and I can verify that it's due to a maximum token limit. In my opinion, the error message could be more informative in this scenario.
If you're facing this issue, a possible solution is to remove files with large differences or newly added large files.

ok, looks like sending large files in batches to openAI breaks somewhere. Anyone wants to investigate and make a PR? :)

What do you think about this?
The algorithm:

Shall we check if the maximum token reaches 4096?
If so, we can use Langchain's token splitter.
The texts could be uploaded in smaller chunks, or a summarizer could be created to generate a commit.

This could nicely overcome the problem.


IE11 Not Working

Unfortunately we're not able to support IE11 due to limitation in the browser itself.


Dump only latest or newest rolled over index.

Is your feature request related to a problem? Please describe.
I am trying to multi dump every index mapping and settings. The issues is that I have rollover configured, and thus have many basically duplicated indexes. It would be nice if it was possible to only dump the oldest from the same types of indexes. I couldn't figure out a way to do this with the --match.
Describe the solution you'd like
Functionality to recognize same index rolled over indexes and only dump settings / mapping from the first or last one.
Describe alternatives you've considered
--match but no regex can cover this.

Regex isn't the right tool for this. You need something to compare dates. I'm not going to add this feature because it's too specific and this isn't the type of library for this.
What I can do is expose a filter function that can be applied to the matchIndexes to perform further filter based on a predefined user function.

matchIndex = []

// with a module at ./filters/roll-over.js with the following:

module.exports = function (indexes) {
    // do something to doc
};

In that case can close this issue. I will write post processing shellscript to remove the duplicates. It would have been nice to have this builtin to reduce export time. But since its not a regular thing the time is not that important.

I think you're missing the point. It's all about making things generic so that it can be useful to other people, not just your specific needs. It would allow you to apply any filter function list matched indices, which I think is a bit more powerful that what you suggested. Because you can do expression based filtering, condition based filtering etc.

Yeah but in that case wait and see if there is more need for this. Not worth implementing only for my specific case.

Not exactly, but if you want to go at it the bash way that's fine. All the best


Building on arm64 macos

Hello FluCoMa devs,
I just wanted to share an experience building flucoma-sc on arm64 macOS 12.2 / Xcode 13 (Apple M1 hardware).
TL;DR it works!
There's one issue I encountered: flucoma-core / eigen had an issue building with Xcode 13, so I commented out one line as suggested in https://github.com/flucoma/flucoma-core/issues/69#issuecomment-988029893
Moreover, I had to explicitly specify the architecture:
cmake -DSC_PATH=/paht/to/supercollider -DCMAKE_OSX_ARCHITECTURES=arm64 ..

I might be wrong, but I don't think the architecture needs to be hardcoded in CMakeLists.txt, I believe it should be picked up automatically.
It seems to work well so far.
Thanks so much for a great library! I hope you find this report useful.

Hi dyfer, not as a dev but as an interested person; would you be ok to share the binaries you built (with me and/or the rest of the world)? Cheers & thanks!

Here it is.
This was compiled in February 2022, can't remember if it was done from the main or the dev branch...
No warranty whatsoever :)
FluidCorpusManipulation.zip


Editable grid

How to edit the data in the MVCgrid and save it to database?
Thanks in advance.

How to edit and  generate Dropdown for specific column and save to the database?
Thanks


Rivers getting crammed up on continents

Description
I noticed that on some smaller continents rivers tend to cluster around each other which can create strange formations shown in Additional information. Perhaps this can be solved by creating a system that checks if the starting point of main/branch rivers are too close to another river by simply canceling out one of the rivers or shorten the starting point of the river itself.
As alternative rivers could make use of the distance/separation system that's in place for structures to make this occurrence less likely to happen.
How to reproduce

Generate a seed where you start on a small continent or with small two continents very close to each other
locate rivers within a close proximity to each other

Additional information

Currently forks check that they don't interesect another river, but obviously that doesn't prevent rivers starting very close to another.
The simplest solution would be to shorten the length of the fork slightly after confirming that it doesn't intersect another river. Very doable


Update-elevator_hotline-repo-to-deploy-using-new-Action

Some comments. Also it looks like you have some YAML validation errors; I recommend installing yamllint and/or setting up automatic linting of YAML in your editor.

fixed lint errors


Roll codemirror via npm not bower

Update client's readme.md?

Also, why npm?

Bower package is broken. Lib/codemirror.js is missing


Zoom messes with map boundaries?

Bug report

Summary
Not sure if this is intended but it seems like the coordinates supplied to setMapBoundaries is setting the size of the view port rather than the how far north/south/west/east your view port can go.

Notice the boundary shown by the arrow and notice where Pitt Meadows is.

The red arrow here is the most I can scroll to, which is not the boundary that I have set. It's way far left than the one shown in the first picture. Notice where Pitt Meadows is in this picture. I should be able to scroll to the right more to see Golden Ears Provincial Park.
Environment info

react-native info output:
npmPackages:
@react-native-community/cli: Not Found
react: 16.13.1 => 16.13.1
react-native: ^0.66.1 => 0.66.1
react-native-windows: Not Found
npmGlobalPackages:
react-native: Not Found
"react-native-maps": "^0.27.1",
# paste your react, react-native, react-native-maps versions here

Steps to reproduce

Set map boundaries
Scroll map to edge of any boundary.
Zoom in.

Describe what you expected to happen:
You should always be able to reach the specified boundary no matter how zoomed in you are.
Describe what you actually happens:
You cannot scroll to the specified boundaries if you zoom in past a point.
Reproducible sample code

@Rc85 I'm going to close this for now, as I'm gonna have to have you test this on the latest version of react-native-maps and also provide a minimal reproducible example. Granted, the issue might still persist, but you are running a 2 year old version of the library.
Please feel free to open a new issue following the issue template if you find out that this is still an issue :)


chore(master): release 1.1.62

:robot: I have created a release beep boop
1.1.62 (2023-09-04)
Bug Fixes

deps: update dependency wayofdev/laravel-cycle-orm-adapter to v4.0.86 (468c2c2)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/wayofdev/laravel-webhook-client/releases/tag/v1.1.62 :sunflower:


Implement CI for unit testing and coverage

[ ] Implement a GitHub CI workflow that performs unit testing and coverage.

Consider adding automated code formatting too. This may need either of:

Defer until NRI decides on an internal Python code standard
Select a code formatting tool & change later if required


first init

First init batcher / dataloader
Todo:

[x] batcher with limit goroutines
[x] time limiter for grep tiny chan
[ ] panic recover
[ ] cache
[ ] metrics
[ ] tracing
[ ] ci
[ ] test
[ ] bench
[ ] readme
[ ] examples
[ ] rate limit burst

Codecov Report

:exclamation: No coverage uploaded for pull request base (main@624c44e). Click here to learn what that means.
Patch has no changes to coverable lines.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
Additional details and impacted files
@@           Coverage Diff           @@
##             main       #1   +/-   ##
=======================================
  Coverage        ?   60.18%
=======================================
  Files           ?        4
  Lines           ?      216
  Branches        ?        0
=======================================
  Hits            ?      130
  Misses          ?       79
  Partials        ?        7

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


Clorio 2.1.2 Linux build is broken

The latest Clorio 2.1.2 is broken for Linux, both AppImage and .deb throw on start:
A JavaScript error occurred in the main process Uncaught Exception: Error: /tmp/.mount_Clorio9Jz4NH/resources/app.asar.unpacked/node_modules/usb/build/Release/usb_bindings.node: invalid ELF header

@r0tc thank you for opening this issue. Pinging @Cryptosaka

Resolved in the previous versions, closing for now.


Remove hacspec crypto backend

it is not needed anymore -- since the addition of the rustcrypto backend, which now serves as the default backend for non-embedded environments
and crucially, it has been causing CI to fail, with a hard to diagnose cause (it was pulled from a stale fork)

Are there not potential proofs that depend on its kind of expression?

@chrysn sorry for the late response. I think it could have, but I just re-tested and the build still fails. Last time I checked with the hax team, they could not identify the source of the issue. Finally, given that the proof is currently being done using the rustcrypto backend, I think it makes sense to proceed with the removal.


Error in Non-Central Chi-Squared Distribution Documentation

My issue is about a minute error in the documentation for the non-central chi-squared distribution. The probability density function has the incorrect term 'exp(-(nc+df)/2)'. This should instead have been 'exp(-(nc+x)/2)'. Ref: https://en.wikipedia.org/wiki/Noncentral_chi-squared_distribution#Definition
Error is at : https://docs.scipy.org/doc/scipy-0.14.0/reference/generated/scipy.stats.ncx2.html

Thanks for taking time to report @AbhilashG6 this has been fixed in the new versions of the documentation. What you are looking at is the version 0.14 documentation. The new versions starting from 0.15 has the correct terms.


[Communities] Entering multiple telegram handles in community create/edit leads to rendering issues

Describe the bug
When I created the new Developer DAO community, I tried to enter 3 Telegram handles on the Community Create page. However, the homepage sidebar concatenates them into one link with the wrong URL.
To Reproduce
Steps to reproduce the behavior:
Problem #1: Sidebar

Create a new community and enter three Telegram links into the Telegram Handles field: NEARDevHub, neardev, NEAR_Education. Click Save.
Check out the homage sidebar about section and see one concatenated Telegram link
NEARDevHubneardevNEAR_Education that points to a concatenated URL https://t.me/NEARDevHub,neardev,NEAR_Education

Problem #2: Telegram page

Create a new community and enter three Telegram links into the Telegram Handles field: NEARDevHub, neardev, NEAR_Education,. Make sure to add a comma at the end and click Save.
Telegram tab shows feeds from the three telegram channels. But then a fourth empty section with a {"error":"Not Found"} error message.

Expected behavior

Problem 1: Sidebar – User should see a separate Telegram name and link to each one that was entered
Problem 2: Telegram Page – User should not be allowed to enter a comma, and the error message should not load.

Desktop (please complete the following information):

OS: Mac
Browser: Chrome
Version: Latest

Seems like this was fixed- couldn't replicate, looks to be behaving correctly now

Confirming this was resolved with the recent community sidebar fixes


Fix typo on homepage

"Engineering Observability is the key to succesfull IT initiatives"
succesfull -> successful

fixed on prod


ORC-110: [C++] getTimezoneByFilename is not thread safe

…multiple threads

I came across this too! Please merge.

@majetideepak I think it is better to add the mutex at line:730 timezoneCache[filename] = result;.
Moreover, the c++11 std::mutex looks better here.

I believe ORC library must be compatible without C++11 support too.
I don't think just locking the write timezoneCache[filename] = result; is sufficient. The reads must be protected too. A write by one thread could be mutating an element being read by another thread.

The last commit has problems. Line 696 and throws will lead to deadlock. I think we should use RAII here.

You are right. I added an unlock before each throw and return.
I am in favor of RAII as well. However, I don't think there are any RAII locks before C++11.

Should we write our own? It is not complicated after all. I am thinking a class like this would be good enough:
  /**
   * Lock guard for pthread_mutex_t object using RAII
   * The Lock is automatically release when exiting current scope.
   */
  class Lock {
    public:
      explicit Lock(pthread_mutex_t* mutex) : mutex_ptr_(mutex) {
        pthread_mutex_lock(mutex_ptr_);
      }
      ~Lock() { pthread_mutex_unlock(mutex_ptr_); }
    private:
       // no default constructor
       Lock();
       // prohibit copying
       Lock(const Lock&);
       Lock& operator=(const Lock&);

       pthread_mutex_t* mutex_ptr_;
  };

This is a good idea. Patch coming up.

The tests are failing because of this error
[ERROR] Plugin org.apache.maven.plugins:maven-remote-resources-plugin:1.5 or one of its dependencies could not be resolved: Failed to read artifact descriptor for org.apache.maven.plugins:maven-remote-resources-plugin:jar:1.5: Could not transfer artifact org.apache.maven.plugins:maven-remote-resources-plugin:pom:1.5 from/to central (https://repo.maven.apache.org/maven2): hostname in certificate didn't match: <repo.maven.apache.org> != <repo1.maven.org> OR <repo1.maven.org> -> [Help 1]

@ClydeZhao @omalley any comments ?

This is looking great, thanks!
The CI build passed and it built and passed tests on my mac. Let me run it through the docker scripts to make sure it passes on each of the Linux platforms that we support.

LGTM :)


Automated cherry pick of #5303: build: github: add back ceph devel deps for baremetal-agent

Cherry pick of #5303 on release/3.1.
#5303: build: github: add back ceph devel deps for baremetal-agent

Codecov Report

:exclamation: No coverage uploaded for pull request base (release/3.1@7996a02). Click here to learn what that means.
The diff coverage is n/a.

@@              Coverage Diff              @@
##             release/3.1   #5304   +/-   ##
=============================================
  Coverage               ?   6.73%
=============================================
  Files                  ?     547
  Lines                  ?   95797
  Branches               ?       0
=============================================
  Hits                   ?    6451
  Misses                 ?   88808
  Partials               ?     538

Flag
Coverage Δ

#aFlag
6.73% <ø> (?)

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 7996a02...fcb8bba. Read the comment docs.


Semicolon format

Hej,
Semicolons don't seem to be formatted.
http://requirebin.com/embed?gist=0d67452e01754269660f
Input:
var a = 'foo'                ;
var b = '3'
  ;

Current:
Input:
var a = 'foo'                ;
var b = '3'
  ;

Expected:
var a = 'foo';
var b = '3';

Tested with v0.7.1 and default settings.
Thanks

I think on VariableDeclaration it's "safe" to assume nobody would want white spaces before the ; but I'm not sure if we can remove the line breaks in all cases, eg:
// comma-first + asi
var foo = 123
  , bar = 456
;[1,2,3].forEach(echo)

are there any other cases that you saw this happening or just the VariableDeclaration?

@millermedeiros I believe this is happening only for the VariableDeclaration.


Correct types in capabilities to match specs

The documentLinkProvider(had bool now DocumentLinkOptions) and executeCommandProvider(had string now ExecuteCommandOptions) had the wrong types.

Thanks @SurmanPP !


Datepicker: Showing on the page without be called

I'm using Firefox 37.0.2 on a Ubuntu 14.04 LTS.
Suppose I access any app done using materialize css, and it has a datepicker field.
Now I clicked in this field, and I chose the properly date and I closed using button or clicking out of modal. Nice! :+1:
But if I not click in any other field and the "focus" stay on datepicker field, and I change the tab of my browser, and come back to the form with date field, the modal shows again!!

This is a pickadate.js issue. Open an issue there if you feel this is a problem


Fix LLVM install using Homebrew.jl on macOS

I'm not sure what happened with the Travis changes.  Adding Macs seemed to work over here:
https://travis-ci.org/rsrock/Clang.jl/builds/255793312?utm_source=github_status&utm_medium=notification

@ihnorton What parts of clang do you need Elliot and I are working on getting LLVM and Clang building with BinaryProvider (see https://github.com/staticfloat/LLVMBuilder) and we aim to provide Clang.

Just libclang.so/dylib (C API).
Are you envisioning support for Cxx.jl too? Some things I've done in the past (and may try to do again) needed a few hooks in to the C++ API for functionality not exposed through libclang -- but much less of the API than Cxx.jl needs.

closed in favor of #216


fix: build CVM image on CVM hardware

What type of PR is this?
/kind fix
What this PR does / why we need it:
This PR directs the build pipeline to build CVM images on dedicated CVM hardware. This allows apt updates for the CVM image during the build process. Original PR from @anujmaheshwari1 enabling this can be found here: fix: install upgrades for CVM images in pre-install dependencies. This PR also adds the changes originally made in that PR.
Which issue(s) this PR fixes:
CVM breaks on kernel image updates due to null-boot package post-install.
Notes:
NEVER MERGE label is assigned to this PR until necessary changes in ACG are made to support the new workflow it requires. These changes should be simple.
If merged, these changes would require the availability and quota of Confidential Compute VMs in the build region, which is why this adds a conditional region check in .pipelines/.vsts-vhd-builder-release.yaml for CVMs. This will prevent failing pipelines due to not having Confidential Compute cores available.
Requirements:

[ x] uses conventional commit messages
[ ] includes documentation
[ ] adds unit tests
[ ] tested upgrade from previous version

Pull Request Test Coverage Report for Build<PHONE_NUMBER>9
Details

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 71.35%

Totals

Change from base Build<PHONE_NUMBER>3:
0.0%

Covered Lines:
2600

Relevant Lines:
3644

💛  - Coveralls

it sounds like we decided to set the staging gallery CVM definition to specialized right?

it sounds like we decided to set the staging gallery CVM definition to specialized right?

Yes. Planning on doing this programmatically, I put the code necessary to do so in init-variables.sh. So the first run, it should look for the def and since its not there, create it. Then upload fine. Anuj asked to review that change before I run it the first time in the test pipeline so just waiting on that. @cameronmeissner


It seems the jupyter viewer is not exist in current version?

It will be great if some widgets can be added for viewer.

Hi @mrbeann for now we have moved all data modality / problem type-specific features (e.g. the viewer for relation extraction) out of this repo, and in this case into the snorkel-extraction repo.  Our goal is to maintain this library as a minimal, modular package that is data type / domain agnostic!  For more thoughts here see our release notes on v0.9: https://www.snorkel.org/blog/hello-world-v-0-9


How to collect loglikelihood from logs lightlda print out?

Hi, what is the relation between block and slice?
When computing the doc loglikelihood, we just concern slice == 0 in each worker. Is this means we only computing the doc in slice 0, ignore the other slice. Shall we just compute partly doc in this data block?
When computing word loglikelihood, we set block == 0. Is this means we computing all the word in this block, but ignore the other blocks.
And When computing normlized loglikelihood, we use TrainerId() == 0 && block == 0 condition. Here also ingnore the other block.
In workers, all slices in every block may be executed loglikelihood under upper condition setting, and print computing loglikelihood.
So, How should me collect the corpus' doc loglikelihood, work loglikelihood and total loglikelihood?

Didn't ignore. We only compute the doc likelihood when sampling slice 0 in one iteration, but the computation is over the whole documents. Computing doc likelihood only needs to know the doc-topic information. It's unrelated with word-topic-table, we can compute when sampling any slice.
We compute word likelihood when sampling block = 0. This will only compute word likelihood that was contained in block 0. The result sometimes is a approximation of the correct word likelihood. This is because it's possible that the vocabulary of unique word is block 0 didn't always as same as the whole vocabulary. But it should not differ too much. May only lack of some very low frequency word.
Computing word likelihood only related with word-topic-table, so if we have the parameter, we can compute.
This is only related with the summary row, n_t. The condition here is only make sure we only compute once in one iteration.
Sorry not clear what you mean by "In workers, all slices in every block may be executed loglikelihood under upper condition setting, and print computing log likelihood."
The whole likelihood is doc + word + normalized.
The doc likelihood is sum of all document. Note that in every machine, we only sample part of whole document to compute. You can compute whole, but it's time-consuming. If you what to get the whole doc likelihood, the sampling result times a coefficient will give a approximation result.
Word likelihood is sum of all words, which may computed in different slices. Just sum the result from one process.

Thank you very much Feiga
Sorry my english is not well. I mean the slice is the basic unit of corpus in trainning. Each slice would print loglikelihood logs when trained.
When sampling slice 0 in one iteration, will it compute entire documents' doc-likelihood in block? Here we assume   there is 1 block in each worker.
Thanks, Lizhe

@tanglizhe1105 Sorry I must miss your message.
Yes, it compute the entire documents' doc-likelihood. See here


sql: possible to CREATE a table and ALTER (or DROP) it in the same transaction

PR: https://github.com/cockroachdb/cockroach/pull/15929

No updates needed.


ovpn?

Can this work with openvpn?

It should work, in theory.

How can I not understand some parts more than the entire server?
Client:
./gole-linux-amd64 -enc xor -key "your_encryption_key" tcp <IP_ADDRESS>:9998 <IP_ADDRESS>:8088 -fwd  -proto tcp

<IP_ADDRESS> 9998 OVPN LISTENING LOCAL
<IP_ADDRESS>:9999 Listening real ovpn

Server:
./gole-linux-amd64 -enc xor -key "your_encryption_key" tcp <IP_ADDRESS>:8088 <IP_ADDRESS>:9999 -fwd <IP_ADDRESS>:9999
Gole v1.2
tunnel mode: TCP
operation: holepunch
punching holes: [local]<IP_ADDRESS>:8088 ---> [remote]<IP_ADDRESS>:9999
recv() failed. EOF
Failed to punch hole. EOF
<IP_ADDRESS>:9999 Is ovpn running as it should be?

I see your parameters are all messed up. Do you want to run Gole locally because I see only one public IP(<IP_ADDRESS>) in your configuration. You need two IPs to make it work.

I want to run this server publicly to bypass firewall

Then you need two machine and two public IPs.
One is inside the firewall where you run Gole in client mode, the other is outside the firewall where you run Gole in server mode.
You cant run both modes in one machine.

This is what I'm doing, don't tell me that the server has to put my client's IP, the IP that expects to be accepted by the server? Otherwise it wouldn't work.

I'm sorry I really don't understand your request.


DatePicker Not working after iOS14 upgrade

After ios14 upgrade the datepicker is no longer working. Looks like iOS did something to make it's default datePicker supersede the existing datepicker in the application. Attaching screenshot. Can someone please help on this.
Regards,
Rahul

@rahulchaube did you manage to find a solution? I'm seeing the same thing

Just found a fork that's solved the issue:
https://github.com/AppsGanin/cordova-plugin-datepicker

Just found a fork that's solved the issue:
https://github.com/AppsGanin/cordova-plugin-datepicker

Just found a fork that's solved the issue:
https://github.com/AppsGanin/cordova-plugin-datepicker

works for me

@mrstennett
Can you elaborate on how to apply this fork?
I tried to install this via package JSON
"cordova-plugin-datepicker": "git+https://github.com/AppsGanin/cordova-plugin-datepicker.git",
My ios version is 5.0.1

@mrstennett
Can you elaborate on how to apply this fork?
I tried to install this via package JSON
"cordova-plugin-datepicker": "git+https://github.com/AppsGanin/cordova-plugin-datepicker.git",
My ios version is 5.0.1

@mrstennett
Can you elaborate on how to apply this fork?
I tried to install this via package JSON
"cordova-plugin-datepicker": "git+https://github.com/AppsGanin/cordova-plugin-datepicker.git",
But it is not working for me
My ios version is 5.0.1

If I remember correctly, I think I did
ionic cordova plugin add https://github.com/AppsGanin/cordova-plugin-datepicker.git
You might also need to remove the original first.

@mrstennett
Can you elaborate on how to apply this fork?
I tried to install this via package JSON
"cordova-plugin-datepicker": "git+https://github.com/AppsGanin/cordova-plugin-datepicker.git",
But it is not working for me
My ios version is 5.0.1

If I remember correctly, I think I did
ionic cordova plugin add https://github.com/AppsGanin/cordova-plugin-datepicker.git
You might also need to remove the original first.

@mrstennett
Thanks for your comments
will try this

@mrstennett
Thanks for your comments
will try this

@mrstennett
Thanks for your comments
will try this

Hi my ionic project is a capacitor project. So I installed the fork using npm i https://github.com/AppsGanin/cordova-plugin-datepicker.git. However I still see the cancel and done box with date buttons inside it. When I click on the date inside this box it would then open the ios 14 date picker.  It would not open the date picker directly when I click on the ion-button from where I called the datepicker.show().
My package.json dependencies shows the related packages as follows:
"cordova-plugin-datepicker": "git+https://github.com/AppsGanin/cordova-plugin-datepicker.git",
"@ionic-native/date-picker": "^5.31.1",
I have uninstalled, removed and added several times. But still failed to make it work.
Am I doing something wrong? Any help would be appreciated.

Hi!
I am developing an app with IONIC V3 and Cordova. I am using cordova-plugin-datepicker and everything go ok until I deploy my app with SDK iOS14. Since there date picker is different. How can I solve it?
Thank you very much.

After ios14 upgrade the datepicker is no longer working. Looks like iOS did something to make it's default datePicker supersede the existing datepicker in the application. Attaching screenshot. Can someone please help on this.
Regards,
Rahul

You need to add
if (@available(iOS 13, *)) {  UIDatePicker *picker = [UIDatePicker appearance]; picker.preferredDatePickerStyle = UIDatePickerStyleWheels;  }
to AppDelegate.m.
Put this between
- (BOOL)application:(UIApplication*)application didFinishLaunchingWithOptions:(NSDictionary*)launchOptions { and
return [super application:application didFinishLaunchingWithOptions:launchOptions]; }.


Fix use pipeline-ws

TEALC-174 #resolved

TEALC-174 #resolved

TEALC-174 #close

TEALC-174 #testing #comment Moving to testing


e2e: serial: add reporter

Add polarion and junit reporters like we do for the
performance-addon-operators functests.
Signed-off-by: Francesco Romani<EMAIL_ADDRESS>

/cc @shajmakh

thank you for the updates.
/lgtm
one question regarding all the vendor files, what caused them to change and why?

thank you for the updates. /lgtm one question regarding all the vendor files, what caused them to change and why?

added kubevirt.io/qe-tools and bumped ginkgo/gomega deps. I'm not completely sure about the latter, but lagging too much behind is not risk-free either. So let's try this out, I'll be on the lookout for issues/regressions.


Could not open file /storage/.../index.android.bundle: Permission denied

Get help with your React Native app or ask code-level questions
System:
OS: macOS 10.14.2
CPU: (4) x64 Intel(R) Core(TM) i5-5257U CPU @ 2.70GHz
Memory: 244.36 MB / 8.00 GB
Shell: 3.2.57 - /bin/bash
Binaries:
Node: 11.2.0 - /usr/local/bin/node
Yarn: 1.12.3 - /usr/local/bin/yarn
npm: 6.4.1 - /usr/local/bin/npm
Watchman: 4.4.0 - /usr/local/bin/watchman
SDKs:
iOS SDK:
Platforms: iOS 12.1, macOS 10.14, tvOS 12.1, watchOS 5.1
Android SDK:
API Levels: 19, 20, 21, 22, 23, 24, 25, 26, 27, 28
Build Tools: 22.0.1, 23.0.2, 25.0.0, 25.0.1, 25.0.2, 25.0.3, 26.0.1, 26.0.2, 27.0.0, 27.0.1, 27.0.3, 28.0.3
System Images: android-22 | Android TV Intel x86 Atom
IDEs:
Xcode: 10.1/10B61 - /usr/bin/xcodebuild
npmPackages:
react: 16.6.1 => 16.6.1
react-native: ^0.57.5 => 0.57.8
npmGlobalPackages:
react-native-cli: 2.0.1
Talk about best practices or propose changes to React Native
Fatal Exception: com.facebook.jni.CppException
facebook::react::Recoverable: Could not open file/storage/emulated/0/Android/data/com.xxx.android/files/.rn/index.android.bundle: Permission denied
keyboard_arrow_up
arrow_right
com.facebook.react.bridge.CatalystInstanceImpl.jniLoadScriptFromFile (TbsSdkJava)
com.facebook.react.bridge.CatalystInstanceImpl.loadScriptFromFile (TbsSdkJava:221)
com.facebook.react.bridge.JSBundleLoader$2.loadScript (TbsSdkJava:53)
com.facebook.react.bridge.CatalystInstanceImpl.runJSBundle (TbsSdkJava:243)
com.facebook.react.ReactInstanceManager.createReactContext (TbsSdkJava:1116)
com.facebook.react.ReactInstanceManager.access$900 (TbsSdkJava:117)
com.facebook.react.ReactInstanceManager$5.run (TbsSdkJava:916)
java.lang.Thread.run (Thread.java:761)
Chat with React and React Native community members

Hi @caishi3hao Did you find any solution for this? Even I'm getting this issue.


Unloadable via AMD

I believe this is due to a recent change. If I included traceur from either the npm or bower as a plain script tag at the top of my application everything works fine. I need to reference window.traceur to then use it, but everything works. If I instead set up a traceur to load inside or after requirejs then I get the following error:
Uncaught Error: ModuleEvaluationError: regenerate is not a function in<EMAIL_ADDRESS>I've tried a few methods to make this work, and I've almost isolated the issue.
After scouring the built source code I found this:
line 27132:
if (typeof define == 'function' && typeof define.amd == 'object' && define.amd) {
      define(function() {
        return regenerate;
      });
    } else if (freeExports && !freeExports.nodeType) {
      if (freeModule) {
        freeModule.exports = regenerate;
      } else {
        freeExports.regenerate = regenerate;
      }
    } else {
      root.regenerate = regenerate;
    }
  }(this));

and then later (line 28164):
var regenerate = require('regenerate');

This series of events appears to be result in an anonymous define or at the very least a module that can't be found.
I've created an es6 requirejs loader that nearly works with the latest version of traceur. The es6 module pattern doesn't quite meet the needs of the general web yet and functionally resembles AMD, which is why we are trying to mix the 2 things. This all nearly works. In fact, I'd even be satisfied if traceur just registered globals and didn't try to be overly accommodating for AMD.

This comes from regexpu.
@mathiasbynens Any plans to provide an ES6 version?

@arv Unlikely. Providing an ES6 version of regexpu would mean:

rewriting regexpu in ES6 (doable)
rewriting all of its dependencies in ES6 (not gonna happen any time soon)

Ref. #1295.
@parris As a workaround, you could try adding regexpu as a direct dependency next to Traceur.