Fix SILVerifier assert for witness_method with dynamic self type

Fixes rdar://71290925

@swift-ci test

Should we instead try to fix this to work? What goes wrong if such a witness_method instruction is generated?

We hit the SIL verifier not liking such a witness method instruction. It only wants opened archetype type operands.
void checkWitnessMethodInst(WitnessMethodInst *AMI) {
    auto lookupType = AMI->getLookupType();
    if (getOpenedArchetypeOf(lookupType)) {
      require(AMI->getTypeDependentOperands().size() == 1,
              "Must have a type dependent operand for the opened archetype");
      verifyOpenedArchetype(AMI, lookupType);
    } else {
      require(AMI->getTypeDependentOperands().empty(),
              "Should not have an operand for the opened existential");
    }

Build failed
Swift Test Linux Platform
Git Sha - de957f86e97716e4304bc84a810972a67003d600

I changed what this PR was initially intended for. Turns out we do support witness_method with dynamic_self. The SILVerifier assertions for witness_method needed to be changed

@swift-ci smoke test

Build failed
Swift Test OS X Platform
Git Sha - de957f86e97716e4304bc84a810972a67003d600

@swift-ci smoke test OS X platform

@swift-ci smoke test Windows Platform

Thanks, changing the verifier here makes sense. We do support witness_method on concrete types just fine (either in IRGen, or the devirtualizer if that gets to run)

@swift-ci smoke test


enhancement: Refactors

This is an ongoing issue to manage refactors that need to be done to the codebase.
Refactors

[x] Using thiserror for Error and its #[from] macro for automatic From implementations.
[ ] Refactoring the handler return types to something like Result<(), HandlerError> and then implement impl<T: Serialize> IntoResponse for Result<T, HandlerError> which would serialize the data into JSON and specify the status code. Doing this would allow you to handle errors naturally using ? inside your handler functions, and clean up the JSON serialization.
[x] Async tests
[x] Create sqlx decoder for ProviderKind
[ ] De-duplicate structs

Related Issues/PRs

#4
#25

Mainly closed, a couple small issues left


Premium warbonds should have their pass cost as their first page unlock cost

https://github.com/helldivers-2/json/blob/e57d9c4b7a4c770853a34fa341017acc8e3b48ce/warbonds/cutting_edge.json#L1-L5
e.g. Cutting Edge costs 1000 SC
(personal opinion)

Hm, not the worst idea, and would kinda make is_premium in the warbonds.json a little redundant I think. @dealloc Thoughts?

Hm, not the worst idea, and would kinda make is_premium in the warbonds.json a little redundant I think. @dealloc Thoughts?

It would, I'd just add the super credit cost and that'd suffice in my opinion

I’ll work on the PR here in a bit to make the changes
On Thu, Apr 25, 2024 at 11:17, Wannes Gennar @.***(mailto:On Thu, Apr 25, 2024 at 11:17, Wannes Gennar < wrote:

Hm, not the worst idea, and would kinda make is_premium in the warbonds.json a little redundant I think. @.***(https://github.com/dealloc) Thoughts?

It would, I'd just add the super credit cost and that'd suffice in my opinion
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you commented.Message ID: @.***>


issue while executing query

[42883] ERROR: operator does not exist: cube <-> cube
Hi i am getting this error
My query is
**SELECT name FROM vectors WHERE (cube(array[-0.15382039546966553,0.01020173728466034]) <-> vec_low )
**

@vearutop is it necessary to perform
SELECT name FROM vectors WHERE sqrt(power(CUBE(array[{}]) <-> vec ,2) when i am finding that i am able to save 128 dimension in cube?
Thanks

You are likely missing cube extension (https://github.com/vearutop/face-postgre/blob/master/db.py#L6) if you receive such error.
By default cube extension is limited to 100 floats in a vector, if you have a patched PostgreSQL version that allows 128 dimensions you can simplify query to SELECT name FROM vectors WHERE CUBE(array[{}]) <-> vec. No need to sqrt(power(..)).


expanding the help-page with the note to the default command

Fix of https://github.com/ManimCommunity/manim/pull/2729. Sorry, I didn't knew how to revert my changes without force-pushing my changes. I tried to use git cherry-pick but it didn't seem to work somehow?

Should be fixed now (I hope).


FAQ and Known Issues links

The footer displays several links that are broken:

FAQ - /click2vox/faq

Known Issues /click2vox/known-issues

Please fix the view references here.
Once that is done @chadwallacehart will update the links as needed.

This needs to be fixed by 14 Oct.

@chadwallacehart Fixed!


Add UUIDModule to message ObjectMapper.

In order to be able to deserialize both old style base64 UUIDs and native hex with dash format, I have added the UUIDModule to toolbox. I had to copy paste the class from API-library in order to avoid a circular dependency.

Nice, looking goood! 👏🏻


Provide documentation on building the plugin

Per the summary.

@OpherV
In https://plugins.jetbrains.com/plugin/7315-git-flow-integration/ , forum page come to issues, you can change the url to git reademe


build fails because tagged MetroHash version is missing CMakeLists.txt

The tagged commit
<project path="drivers/third_party/metrohash" name="MetroHash" revision="690a521d9beb2e1050cc8f273fdabc13b31bf8f6"/>
doesn't have a CMakeLists.txt and build fails. Latest MetroHash commit works like expected.

I have the same issue.

Thanks for the notice. We have corrected the revision. please check it.


Update Burkina Faso data on FDW and GSCD

New data is uploaded to FDW via helpdesk ticket: https://fewsnet.atlassian.net/browse/HELP-2255?atlOrigin=eyJpIjoiNmJhN2JmZDA1ZDVjNDExYWJkMzExYWM1YjdlZDRhM2YiLCJwIjoiaiJ9
Also, I manually added to GSCD side. If FDW updates the new data, GDCS code will make the duplicated error, so we can notice it.

I'm closing this ticket because of the other BF ticket in our "issues"


Implement all the sending things

Changes

Issue ticket number and link
closes #175
closes #174

I included the name of the route in the mini post; it's not a link because I intend to make route linking not cost a read :p

I tested locally and sent a new route. The route name didn't show up when viewing the post on my profile.

Evidently the routeInfo isn't getting set when a new post is created, i'll look into that

Fixed:


feat(rust): add worker list command

Current Behavior
No worker list command available.
Proposed Changes
Add worker list command.
https://github.com/build-trust/ockam/issues/4130
Checks

[x] All commits in this Pull Request are signed and Verified by Github.
[x] All commits in this Pull Request follow the Ockam commit message convention.
[x] I accept the Ockam Community Code of Conduct.
[x] I have accepted the Ockam Contributor License Agreement by adding my Git/Github details in a row at the end of the CONTRIBUTORS.csv file in a separate pull request to the build-trust/ockam-contributors repository.

@hargut thank you for another awesome contribution 🥳  this looks great.
I made some minor edits to comply with some conventions across commands.


Pull request merge

Hi, next-release-candidate is attached to the two pull request I sent a few months ago.
May I know when it will be merged?
Or if it need more changes, please let me know!
https://github.com/zappa/Zappa/pull/1242
https://github.com/zappa/Zappa/pull/1217

Thanks for your hard work.  I wanted to get the 3.11 supported release out asap, but it took longer than expected.
I think these are good to go into the next release.
Do you want to join our slack, we can discuss the timing there.
https://join.slack.com/t/zappateam/shared_invite/zt-23bsldnw7-rGzsnxBFnvV4sj1USm6ukQ

Hi,
I was just curious if additional work was needed.
That's not it, It's okay to be late, and it may affect other projects so merge at the best time.
I don't mind if it's later as long as it doesn't require additional work!
If additional modifications are needed, please let me know.
I don't check GitHub(and slack) often these days, but if it needs editing, I'll edit it as soon as I see it.
Thanks.


[BUG] Did signup() ever work?

Try Instagrapi SaaS with a free trial https://hikerapi.com/p/5GBWznd3
Describe the bug
When calling the signup function with correct parameters, an error "Invalid format String" arises. This happens due to a line in signup.py in the function accounts_create() -> timestamp = datetime.now().strftime("%s")
This is not a valid strftime.
To Reproduce
from datetime import datetime
def main():
timestamp = datetime.now().strftime("%s")
print(timestamp)
if name == 'main':
main()
Expected behavior
No error appears on account creation and an account is being created. I am not sure what the correct approach is because I tried changing up the timestamp variable but I got an error on return self.private_request("accounts/create/", data)
Desktop (please complete the following information):

OS: Windows 10
Python version 3.11.4
instagrapi version 2.1.1

If you've successfully managed to use signup() please let me know with a comment

@galaSYT1 do u find a solution ? same bug

@Shaplow no solution unfortunately


csvdriver: add ability to name columns

right now, opening a CSV connection is done like so:
func foo() {
	// Open the CSV file as a database table.
	db, err := csvdriver.Conn{
		File:    "../data/iris.csv",
		Comment: '#',
		Comma:   ',',
	}.Open()
	if err != nil {
		log.Fatal(err)
	}
	defer db.Close()

	// Start a database transaction.
	tx, err := db.Begin()
	if err != nil {
		log.Fatal(err)
	}
	defer tx.Commit()

	// Define a SQL query that we will execute against the CSV file.
	query := "SELECT var3, var4, var5 FROM csv WHERE var5 = \"Iris-versicolor\""

	// etc...
}

this makes a query rather cumbersome to read.
it should be possible to open a CSV connection with an optional Columns []string field:
// Open the CSV file as a database table.
db, err := csvdriver.Conn{
	File:    "../data/iris.csv",
	Comment: '#',
	Comma:   ',',
	Columns: []string{"sepal_length", "sepal_width", "petal_length", "petal_width", "species"},
}.Open()

such that the query reads:
// Define a SQL query that we will execute against the CSV file.
query := `SELECT petal_length, petal_width, species FROM csv WHERE species = "Iris-versicolor"`

What do you think @dwhitena ?

This issue was moved to go-hep/hep#39


yarn cli wallet balances，No tokens found!

yarn cli wallet create success, and  yarn cli wallet address，Correctly return the wallet address, Deposit 1FB into the wallet,
Show token balances
yarn cli wallet balances
yarn run v1.22.22
$ node dist/main wallet balances
No tokens found!
Done in 1.82s.

What's the reason? How to solve it

Wait tracker synced

tracker already synced,

I have the same question. It still doesn't work after several hours.

how do u fix this ??

this doesnt show you your FB balance when you run wallet balances ...

yarn cli wallet address ，Only display your balance after successfully minting，
like this:

The document misled us. The process should be: 1. recharge fb coin  2. Mint,   3. Run "yarn cli wallet balances " to check the balances

Wait tracker synced

hi , sync is very slow. It's been open for hours and it's only 17%. Also, it looks like the gap with the total number of blocks will never close. Is there a way to download in bulk?


Tooltip message gone after change to report page

The default tooltip message "Right click to drill through" is gone in PowerBI.com, if we change the tooltip setting of a visual to "Report Page" and when change back to default.
Steps:

On one of my bar chart visual, it used to have a message in its tooltips, "Right click to drill through"
Changed the tooltips setting to some new report page (which initially planned to be the tooltip"
Decided not to use the new page, change back the tooltip setting on my bar chart visual to Default
The message is back in PowerBI desktop.
After publish, go to my Workspace in PowerBI.com, the tooltip without "Right click to drill through"
Affecting all other visual, now all tooltips that used to have the said message, is without the message. Only happened in PowerBI.com.

I'm using (and have to use) Power BI version August 2020

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

ID: 8caeb04f-19a3-320d-2bfa-147c5377728e
Version Independent ID: d3785a20-eeaa-a3a0-5198-b6a4c8fd475b
Content: Set up drillthrough in Power BI reports - Power BI
Content Source: powerbi-docs/create-reports/desktop-drillthrough.md
Service: powerbi
Sub-service: pbi-reports-dashboards
GitHub Login: @maggiesMSFT
Microsoft Alias: maggies

Hi, @annatagithub -- given my delayed response, you've probably already resolved this issue. If not, I recommend creating a support issue.
Good luck!


Overwrite state date when a maximum date exists and date on state is …

Overwrite state date when a maximum date exists and date on state is null or greater than maximum.
Overview
When the confirm button was pressed without interacting with the picker, the maximumDate was ignored. That is, the selector shows a value with the maximum restriction, but after confirming, today's date that is greater than the maximum was shown.
Now when the maximumDate property exists, the value of the date in the state is overwritten.
Test Plan

Set a maximumDate reestriction smaller than today.
Open the modal and press confirm without interact to picker.

Closing in favour of #301, which doesn't need this change anymore. Thanks anyway for contributing!


[SPARK-47982][BUILD] Update some code style's plugins to latest version

What changes were proposed in this pull request?
The pr aims to update some some code style's plugins to latest version, include:

mvn-scalafmt_2.13 from 1.1.1684076452.9f83818 to 1.1.1713302731.c3d0074.
checkstyle from 10.14.0 to 10.15.0.
scalafmt from 3.8.0 to 3.8.1.

Why are the changes needed?
1.mvn-scalafmt_2.13
https://github.com/SimonJPegg/mvn_scalafmt/releases/tag/v2.13-1.1.1713302731.c3d0074
Minor version bumps and dropping 2.12
2.checkstyle
https://checkstyle.org/releasenotes.html#Release_10.15.0
https://checkstyle.org/releasenotes.html#Release_10.14.2
https://checkstyle.org/releasenotes.html#Release_10.14.1
3.scalafmt
https://github.com/scalameta/scalafmt/releases/tag/v3.8.1

Does this PR introduce any user-facing change?
No.
How was this patch tested?
Pass GA.
Was this patch authored or co-authored using generative AI tooling?
No.

@HyukjinKwon
So quickly approved, thank you very much! ❤

Merged to master. Thank you @panbingkun @HyukjinKwon @dongjoon-hyun


Initial porting

@gsvr review please. This is mostly a simplification of the code from mama-ng-control that you already extensively reviewed

Minor comments, :+1:


Escape PKG_CHECK_MODULES

This small changes allows to fix an issues https://github.com/avast/retdec/issues/990 and probably https://github.com/VirusTotal/yara/issues/1171
The root cause of this issues is different shell which is used.
Thus at § 3.5 at https://autotools.io/pkgconfig/pkg_check_modules.html it is cleare suggested to escape by [...].

@googlebot I signed it!


Provide Midi Support

It would be great to have midi playing support.

Thanks for reaching out! We have a plugin infrastructure which should allow someone to create a plugin to expose Midi up into Flutter. Here are the plugin docs: https://flutter.io/platform-channels/
We don't have plans to write this plugin, but we encourage the community to do it. And, if there are issues with the plugin APIs that make it difficult or impossible, please definitely let us know!
I'll close this issue for now, but we hope to see a Midi plugin!


WIP: add advanced features of paths.

The results of exploring plots for features we have now (distance, speeds,...) were somewhat disappointing; probably we need to add more features which should indicate significant and interesting differences over different strains. This pull request will add the features such as:

area of the rectangle spanned by path
radius and center angles in path
area covered by path
distance between the start and end points in path
For details, see #122.

For the initial commit, I only added main functions to generate those features. I'll add test functions later. Please review @aksam-ahmad @alannaiverson @jcpalumbo3 @jJasonWang @sicihuang

@togawa28 What if we only calculate two of the features, like area of rectangle and radius, but only have one function that returns both? I think that could work and testing would be significantly less? What do y'all think, @jcpalumbo3 @jJasonWang @aksam-ahmad @sicihuang ?

Interesting idea @alannaiverson, we could combine some functions, but why do you think testing would be less? Regardless of using one or several, we need to test the inputs/outputs of area of rectangle and radius anyway?

True, @togawa28 but I was thinking if we just computed the edge points and center within the function, those two at least wouldn't need separate testing would they? Or am I misunderstanding?

No, you correctly understand what I'm saying. Well, even if we include those two within the function, we still should test those so that we confirm they work correctly maybe?

I think testing the final output would serve as a check for those, without formal tests, but I could be wrong. I was thinking something like this (not sure why markdown is making the first few lines weird...):
`
def compute_advanced(path_obj):
if not isinstance(path_obj, pd.core.frame.DataFrame):
raise TypeError("path_obj must be pandas DataFrame")
if not set(path_obj.keys()).issuperset(['x', 'y']):
    raise ValueError("the keys of path_obj must contain 'x', 'y'")

if len(path_obj) <= 1:
    raise ValueError("path_obj must contain at least 2 rows")

# Computes edge points like find_edge_points originally did compute_area_rectangle did
edge_points = {'xmin': np.min(path_obj.x), 'xmax': np.max(path_obj.x),
               'ymin': np.min(path_obj.y), 'ymax': np.max(path_obj.y)}

# Computes area of rectangle
area = (edge_points['xmax']-edge_points['xmin']) * (edge_points['ymax']-edge_points['ymin'])

# Computes center like find_center originally did
center = {'x': (edge_points['xmin'] + edge_points['xmax'])/2,
          'y': (edge_points['ymin'] + edge_points['ymax'])/2}

indices = path_obj.index[:-1]
vectors = [path_obj.loc[i, 'x':'y'] -
           [center['x'], center['y']] for i in indices]
center_angles = [path_features.angle_between(list(v1), list(v2)) for
                 v1, v2 in zip(vectors[1:], vectors[:len(vectors)])]
radius = [np.linalg.norm(v) for v in vectors]

return({'radius': radius, 'center_angles': center_angles, 'area': area})`

@togawa28 @jJasonWang @jcpalumbo3 @sicihuang @aksam-ahmad Thoughts on squashing some of the functions into one for time reasons?

@alannaiverson thank you for your contribution, and now I get your point. It's much more efficient to squash those functions because it not only decreases the number of tedious testing functions but also  reduces doc strings to make the whole file easier to read and understand.
I'll follow your suggestion and start writing tests functions. Thanks again!

BTW, for PEP8 styles issue, I got the following message.
________________________________ pyflakes-check ________________________________
/home/travis/build/berkeley-stat222/mousestyles/mousestyles/path_diversity/path_features_advanced.py:84: EOL while scanning string literal
raise ValueError("the keys of edge_points must be 'xmax', 'xmin',
I'm not sure how to fix it. Any idea?

I am not sure what happens, but I will help you check that @togawa28.

@togawa28 I made an example of what the function and tests might look like, but I'm not sure how to add a file to this pull request so I'll just copy them here? Apologies for the messiness and these are just ideas! They can no doubt be improved:
`from future import print_function, absolute_import, division
import numpy as np
import pandas as pd
from mousestyles.path_diversity.path_features import angle_between
from mousestyles.path_diversity import path_index
def compute_advanced(path_obj):
r"""
Return dictionary containing the radius,  area of the rectangle
spanned by the path, and absolute distance between start and end
points of each path.
Parameters
----------
path_obj : pandas.DataFrame
    CX and CY must be contained.
    The length must be greater than 2.

Returns
-------
radius : numpy float object
    each element is the distance between center point and
    each point in the path. The length equals to the length
    of the path_obj.

area : numpy float object
    area of the rectangle spanned by the path.

abs_distance : numpy float
    the distance between the start and end points in a path.

Examples
--------
>>> movement = data.load_movement(1,2,1)
>>> compute_advanced(movement)
"""
if not isinstance(path_obj, pd.core.frame.DataFrame):
    raise TypeError("path_obj must be pandas DataFrame")

if not set(path_obj.keys()).issuperset(['x', 'y']):
    raise ValueError("the keys of path_obj must contain 'x', 'y'")

if len(path_obj) <= 1:
    raise ValueError("path_obj must contain at least 2 rows")

# Computes edge points like find_edge_points originally did compute_area_rectangle did
edge_points = {'xmin': np.min(path_obj.x), 'xmax': np.max(path_obj.x),
               'ymin': np.min(path_obj.y), 'ymax': np.max(path_obj.y)}

# Computes area of rectangle
area = (edge_points['xmax']-edge_points['xmin']) * (edge_points['ymax']-edge_points['ymin'])

# Computes center like find_center originally did
center = {'x': (edge_points['xmin'] + edge_points['xmax'])/2,
          'y': (edge_points['ymin'] + edge_points['ymax'])/2}

# Computes average radius
indices = path_obj.index[:-1]
vectors = [path_obj.loc[i, 'x':'y'] -
           [center['x'], center['y']] for i in indices]
radius = [np.linalg.norm(v) for v in vectors]
# but maybe we just want a single number for this? We could do average or sum?
avg_radius = sum(radius)/len(radius)

# Computes distance between start and end points of each path
initial = path_obj.loc[path_obj.index[0], 'x':'y']
end = path_obj.loc[path_obj.index[-1], 'x':'y']
abs_distance = np.sqrt((end['x'] - initial['x'])**2 +
                       (end['y'] - initial['y'])**2)

return({'radius': avg_radius, 'area': area, 'abs_distance': abs_distance})`

and tests:
`from future import (absolute_import, division, print_function,
unicode_literals)
import pytest
from mousestyles import data
from mousestyles.path_diversity import path_index
from mousestyles.path_diversity import filter_path
def test_compute_advanced_input():
# Check if function raises the correct type of errors.
path = pd.DataFrame({'t': [2, 1, -1], 'x': [5, 3, 8],
'y': [-3, 0, 0], 'isHB': [True, False, False]})
# not pandas DataFrame
like_path = [1, 2, 3]
# not having keys 'x', 'y'
like_path2 = pd.DataFrame(
{'x': [5, -2, 1], 't': [-2, 3, 1], 'isHB': [True, True, False]})
# length is less than 2
like_path3 = pd.DataFrame({'t': [2], 'x': [5], 'y': [3], 'isHB': [True]})
with pytest.raises(TypeError) as excinfo:
    path_features.compute_angles(like_path, True)
assert excinfo.value.args[0] == "path_obj must be pandas DataFrame"

with pytest.raises(ValueError) as excinfo:
    path_features.compute_angles(like_path2, True)
assert excinfo.value.args[
    0] == "the keys of path_obj must contain 'x', 'y'"

with pytest.raises(ValueError) as excinfo:
    path_features.compute_angles(like_path3, True)
assert excinfo.value.args[0] == "path_obj must contain at least 2 rows"

def test_compute_advanced():
path = pd.DataFrame({'t': [2, 4.5, 10.5], 'x': [0, 1, 1], 'y': [
0, 0, 1], 'isHB': [True, True, False]})
assert path_features.compute_advanced(path['area']) == 1
assert path_features.compute_advanced(path['abs_distance']) == 1.4142135623730951
assert path_features.compute_advanced(path['radius']) == 0.70710678118654757`

I added new temporary files which are just copies of @alannaiverson's comment. We can discuss on those new files now.

Guys, I updated path_features_advanced.py and newly added tests/test_path_features_advanced.py, based on our discussions above. I intentionally didn't make any changes in @alannaiverson's file for comparison purpose. Main updates are:

main functions returns all features (i.e. includes center angles, area covered by path.)
radius and center angles are returned as a list (no data reduction): we could take sum or average outside of the function.
modified tests for outputs.
Your any comments (especially on the updates above & styles issue) will be appreciated.

@jarrodmillman could you help us to pass Travis CI? Basically our files passed the test in Python 3, but it didn't in python 2. We don't know why and are just wondering if you could find any good recommendation for this.

@togawa28 I am looking into it.

Rebased and fixed here #203


After access token expires you can still view and edit notes - potentially a security issue

Describe the bug
If the access token expires and notes are still visible you can edit notes. The missing token error message will appear however once you log in you initially do not see the edit but if you edit the same note you can see that the edit indeed took place. I can only replicate the issue with the admin account the one that shows mod.
Steps to reproduce

Log in with admin account and create and save a note (Screenshots do not show my admin account )

Go to Settings > my account and delete the access token used to log in

Click on Name at the top left where the memos icon is and go to profile

you will see all the notes, click on the three dots and you can edit the note

Missing token error will appear

Reload and Sign in again you will see the note which does not show the edit however if you edit the note you will be able to see the edit that you made when the access token was supposed to be missing.

Impact
If you log into a public computer and forget to logout someone could access your information even if you remove the access token to that computer.
Screenshots or additional context
No response

@meldarionqeusse Thanks for such a detailed description. I tried it on the demo site and found that it couldn't be reproduced.

I can reproduce it with my account on my instance. I am running the arm version not sure if that's any different. I will try to find some time to do some more investigation this weekend maybe set up another instance and I will report back with the findings.

I noticed the demo is running v0.16 I am running v0.15.2 maybe it has been fixed in v0.16?

There are no associated changes in v0.16.0. So please try again on the demo site.

I have done some quick experimenting. I removed the access token I edited the note saved got the missing token and then I looked at the note from another device and I did not see the edit. When I signed back from my primary device and hit edit I saw the edit there.
I cannot replicate the behavior on the demo. However what I can replicate on the demo is the fact that if you remove the access token you or log out and go to profile you can still see the notes which might still be a security concern. This is especially true if you have a single not which is publicly available anyone would be able to click on the image at the top left corner hit profile and view all your notes.
Currently log out of the demo

I go to profile

and I can see all the notes

If I clear the cache I end up with this I am not sure if it would be possible to implement something to get to this state where the profile button is not available once logout/remove access token. I am not sure if there's a way to force a clear cache once you logout.

However what I can replicate on the demo is the fact that if you remove the access token you or log out and go to profile you can still see the notes which might still be a security concern.

This is expected. Currently there is no need to regularly check whether the token has expired in web. But we can do this feature later when we have enough development resources. However, thanks for the report.


Strange behaviour when writing backticks with Norwegian keyboard.

Does this issue occur when all extensions are disabled?: Yes

VS Code Version: 1.90
OS Version: Ubuntu 22.04

Version: 1.90.0
Commit: 89de5a8d4d6205e5b11647eb6a74844ca23d2573
Date: 2024-06-04T19:43:07.605Z
Electron: 29.4.0
ElectronBuildId: 9593362
Chromium: 122.0.6261.156
Node.js: 20.9.0
V8: 12.2.281.27-electron.0
OS: Linux x64 6.5.0-35-generic
VSCode enters some kind of mysterious mode when writing backtick character with Norwegian keyboard layout.
Using a Norwegian keyboard layout, when writing a backtick (using shift + the button just left of backspace) VSCode enters some kind of mysterious mode - shown by an underscore under the backtick.

At this point most buttons will only cause this mode to be exited (meaning you will have to push it two times to get the wanted behaviour). However, what I usually want to do is to to navigate to the end of the word using ctrl+right (cursorWordEndRight).
This however, causes the word left of the cursor to be hightlighted while the cursor itself is still shown next to the backtick. See picture below, where ctrl+right have been pressed 3 times:

Finally, when another backtick is written, this backtick will actually overwrite the last character next to the (invisible) cursor!

Steps to Reproduce:

Use a Norwegian keyboard layout (or perhaps also another scandinavian keyboard?)
Write a backtick

At this point it's already annoying, but for maximum annoyance:

Press ctrl+right
Write another backtick

It looks like backtick is used for input composition. Here is how it looks like on a mac, but I suppose you're seeing something similar on Linux
https://github.com/user-attachments/assets/9b596249-26d5-4bed-af05-63dfcd2108f2

That makes sense 😅
Thank you for the clarification! 🙏🏻
I will see if I'm able to find a way to disable this feature and post it here for posterity!

Disabling this feature seems to be a big hassle.
Easiest fix is to just press backtick twice every time it's used.


CodeSector.DirectFolders version 4.3.4

Checklist for Pull Requests

[x] Have you signed the Contributor License Agreement?
[ ] Is there a linked Issue?

Manifests

[x] Have you checked that there aren't other open pull requests for the same manifest update/change?
[x] This PR only modifies one (1) manifest
[x] Have you validated your manifest locally with winget validate --manifest <path>?

[x] Does your manifest conform to the 1.6 schema?

Note: <path> is the directory's name containing the manifest you're submitting.

Microsoft Reviewers: Open in CodeFlow

Hi @stephengillie,
Can you please check what's wrong with this package install? So, I could know what to fix in the manifest file.
Thanks.

Automatic Validation ended with:

No errors to post.

(Automated response - build 899.)


1c.15

These 3 URLs look correct but end up on CENSE pages.
State-action separable RL
State-space decomposition and hierarchical RL
Non-Markov decision process for RL

Fixed 7004,7005,7006 created


Removing metadata

First I want to thanks you for your work. I'm currently migrating my database from CarrierWave to Shrine ;)
One thing that still bother me is the JSON stored in the _data field. I don't get what purpose serves the metadata field and I wonder if I could remove it completly ?
I don't need the actual code (I could write a plugin to remove it I guess) but I want to know if it's used somewhere else before breaking everything ;).
If it's not "necessary" I would expect the metadata key to be added only if the plugin add_metadata right ?

It stores file informational data like file id, which storage unit the file is stored in, and any additional metadata like filename, size, mime type, etc. The add_metadata plugin is to add additional custom metadata.
Have a look at these links:

https://twin.github.io/better-file-uploads-with-shrine-uploader/ (under UploadedFile section)
https://twin.github.io/better-file-uploads-with-shrine-metadata/

My goal is to only store the filename in my database (or a JSON containing filenames per version)
{
    "original": "123.jpg",
    "thumb": "123_thumb.jpg"
}

Since I know I'll only use :store as my storage. I'm currently trying to create a plugin that overwrite convert_to_data. I close this issue since I realize it's quite specific to my use case, and it may not be necessary to everyone

One thing that still bother me is the JSON stored in the _data field. I don't get what purpose serves the metadata field and I wonder if I could remove it completly ?

Shrine in some places uses certain metadata if available, but they are never be required. Some places where metadata is used:

validation_helpers plugin uses size, mime_type, and filename metadata values in #validate_(max|min)_size and #validate_(mime_type|extension)_(inclusion|exclusion) methods
Shrine::Storage::S3 uses mime_type and filename plugins for setting Content-Type and Content-Disposition object headers
rack_response plugin uses size, mime_type, and filename metadata values for setting Content-Length, Content-Type and Content-Disposition response headers, though Content-Length and Content-Type will in most cases be set correctly even if the corresponding metadata values are missing.

If it's not "necessary" I would expect the metadata key to be added only if the plugin add_metadata right ?

The plugins that set metadata values are determine_mime_type, store_dimensions, and add_metadata. The reason why Shrine extracts and stores these basic metadata by default is because one might need them later on. For example, one might want to show size, type or filename of the file, and in case file uploads are already live the metadata values need to be backfilled (note that filename cannot be backfilled as it cannot be determined from the uploaded file). In these cases it's convenient to already have certain basic metadata saved.


Unauthorized - Not registered owner of this bot

When registering the bot I listed two e-mail adresses in the owner form field (comma dilimited). The other owner gets the following error message when he tries to go to the url for the bot:

"Unauthorized - You are not a registered owner of this bot"

And the bot is not listed in his "My bots" webpage.
What can be the problem?

We will investigate; this may be a regression from recent changes.

Looks like this is a small thing; if you put a space between the comma and the next name, it doesn't detect.  We'll fix, but if you want to work around, remove the spaces. .

I have the same issue, and I didn't have spaces between the emails.


lomap networks

This should be ready for review. I just need to double check that they run and it should be good to go.


Round tripping of "NOT LIKE" where clause element does not work

Actual Behavior
Parse the statement:
SELECT VALUE1, VALUE2 FROM FOO WHERE BAR NOT LIKE '*%'

This parses properly.
When you call toString on the parsed statement, it outputs:
SELECT VALUE1, VALUE2 FROM FOO WHERE NOT BAR NOT LIKE '*%'

Note the 'NOT' in front of BAR.  This is invalid DB syntax.
Expected Behavior
The output should not include a spurious NOT.
Code Changes
The issue is from BinaryExpression line 71.  That line checks whether it is a 'not' statement and adds the NOT:
    public String toString() {
        return (not ? "NOT " : "") + getLeftExpression() + " " + getStringExpression() + " " + getRightExpression();
    }

However the NOT is also added in LikeExpression line 49 in getStringExpression() (which is called from the BinaryExpression toString() method):
    public String getStringExpression() {
        return (isNot() ? "NOT " : "") + (caseInsensitive ? "ILIKE" : "LIKE");
    }

I have fixed this on a fork and will submit a pull request shortly.

Version: 1.4 latest (pulled right now)

Pull request:
https://github.com/JSQLParser/JSqlParser/pull/695


runtime(zip-plugin): Fix extract file from zip browser

Problem:  Enter 'x' in zip browser fail with E121
Solution: Fix typo in zip#Extract()

thanks!


[ADDED] Content type for analytics.

Refactored ICON type to ACTION. Fixes #121

Current coverage is 0.00% (diff: 100%)

Merging #123 into develop will not change coverage

@@           develop   #123   diff @@
=====================================
  Files            1      1
  Lines           34     34
  Methods          0      0
  Messages         0      0
  Branches         0      0
=====================================
  Hits             0      0
  Misses          34     34
  Partials         0      0

Powered by Codecov. Last update b4d0e74...168b6a6


Add option to have VS Code use OS scrollbars instead of custom scrollbars

It's currently cumbersome to scroll through alternating documents line by line, e.g. for comparison.
Adding scrollbar buttons (standard on Windows OS) to Visual Studio Code scrollbars would increase efficiency, speed, usability and accessibility.
So, I suggest to add scrollbar buttons to all VS Code scrollbars.
Either by ...

providing a Boolean option, switching between current custom scrollbars and OS scrollbars.
adding scrollbar buttons to the current custom scrollbars.

Undocumented settings can do this for the editors:
	"editor.scrollbar": {
		"verticalHasArrows": true,
		"horizontalHasArrows": true
	},

You need to reload the window after setting these.

Thanks for the hint!
However, wow, the current buttons are so titchy ... It takes too much time and attention (for me) to target and hit these with the mouse. They are not good to handle.

@gjsjohnmurray thanks for jumping in. And yeah that is one option.
By default we are not rendering scroll bar buttons because it is not aligned with our design principles.
Though I do agree that in some cases they make it more usable.
However so far we have no plans to change this, since there is a workaround to use the keyboard or to use the editor hidden option. I can leave this open as a feature request so we gather more feedback.

Thanks, @isidorn, for taking the time.
Having more than two editors open, it's very cumbersome to change focus to the corresponding editor(group) first and then to scroll using the keyboard. In contrast to that, using the mouse it would be a simply click.
The current scrollbar button implementation is inacceptably hard to handle. Hitting a scrollbar button is a plain fluke:

The most simple solution would be to have a Boolean option, disabling the custom scrollbars and using the OS provided scrollbars. That would remove all the current friction that's piggybacked with the current scrollbars and their odd (or hidden) behaviour.

Thanks for feedback.
@hediet for future consideration if we decide to work on the scroll bar.

/ref: #123985 "editor.scrollbar.scrollByPage" doesn't appear in settings

I'm sorry about my slow response. As it is today, the editor.scrollbar.scrollByPage setting only applies to code editors and does not apply to all scrollbars in vscode. This setting (even when it was hidden) was only applying to the code editor. I think perhaps we could create global scrollbar settings i.e. scrollbar.scrollByPage at one point. Please feel free to create a new issue for adding support for this in all scrollbars.

Originally posted by @alexdima in https://github.com/microsoft/vscode/issues/123985#issuecomment-901743489

I tend to believe a title like "Add option to have VS Code use OS scrollbars instead of custom scrollbars" would be more appropriate, as that option would be the most simple to implement solution to all anguish.
What would be your opinion on that change?

I can rename the issue. However I am not sure about that exact change. I would like to keep this open to gather more user feedback.
The problem with OS scrollbars is that they would look out of style and not conforming to our design language. However if they provide better usability in some cases we should consider. Thus the reason to keep this open for more feedback.

Undocumented settings can do this for the editors:

Latest vscode release doesn't seem to support the mentioned settings.
+1 for this issue. editor.scrollbar.scrollByPage makes the scrollbar behave the way I want, but it needs to apply to the sidebar/etc scrollbars too.

yes please, stop dictating to users what they "want", and instead listen and see from the users' perspective what they actually want/need.
scrollbars without buttons (or with poorly implemented buttons, as shown in SetTrend's gif) are detracting from the user experience. lack of buttons reduces usability. (not to mention is inconsistent the rest of the whole operating system and application ecosystem)
if your "design principles" dictate not having buttons, then i would question those principles and the process used to determine them.
please add the buttons.

I absolutely agree with this request. The missing arrow buttons on the scroll bars are the most annoying thing with VScode for me. After 30 years of using scroll bars with arrow buttons, it is really frustrating when you are forced to use another techniques to accomplish the same effect, i.e. to move the viewport just one line up/down at a time.
By the way, I disagree with the change of the title of this issue. Nobody wants to see OS scroll bars in VSCode, it would be out of place with its themes. You should just add two small arrow buttons using the same theme as the rest of the program.
I suppose the main problem here is because the scroll bars in the editors are overlaying the overview bar, which should be shrinked for the size of these newly added arrow buttons.

gjsjohnmurray:

Undocumented settings can do this for the editors:
	"editor.scrollbar": {
		"verticalHasArrows": true,
		"horizontalHasArrows": true
	},

You need to reload the window after setting these.

As I mentioned in another issue, that modification didn't work for me as I expected, i.e. it made that the program doesn't respect my existing settings for the "editor.scrollbar.scrollByPage": true anymore. Well, it turned out that I shouldn't have copied the suggested lines directly as that, i.e. I should have merged my existing editor.scrollbar settings inside that group. Or, I could have add them just like this:
    "editor.scrollbar.verticalHasArrows": true,
    "editor.scrollbar.horizontalHasArrows": true,

And after I looked at the VSCode source code, I found one more undocumented setting that could be important to those who find those arrow buttons too small:
   "editor.scrollbar.arrowSize": 20,
However, the problem with the current implementation is that the press on these buttons activates onMouseWheel event, i.e. it scrolls the editor view 2-3 lines instead of just one like the standard arrow buttons on scroll bars in every normal editor.

"editor.scrollbar.arrowSize": 20,

Cool observation! 👍
I'll add that to my VS Code settings template.

We closed this issue because we don't plan to address it in the foreseeable future. If you disagree and feel that this issue is crucial: we are happy to listen and to reconsider.
If you wonder what we are up to, please see our roadmap and issue reporting guidelines.
Thanks for your understanding, and happy coding!


add no warn docs, overall and for nuget.exe

Fixes: https://github.com/NuGet/docs.microsoft.com-nuget/issues/1457
Fixes: https://github.com/NuGet/docs.microsoft.com-nuget/issues/1478
Note that this fixes 2 different issues!

Docs Build status updates of commit 0c6f9b0:
:warning: Validation status: warnings

File
Status
Preview URL
Details

docs/consume-packages/Package-References-in-Project-Files.md
:warning:Warning
View
Details

docs/reference/cli-reference/cli-ref-pack.md
:white_check_mark:Succeeded
View

docs/consume-packages/Package-References-in-Project-Files.md

Line 172: [Warning] Invalid file link: '../reference/errors-and-warnings'.

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

Docs Build status updates of commit 5c8c3d5:
:x: Validation status: errors
Please follow instructions here which may help to resolve issue.

File
Status
Preview URL
Details

:x:Error

Details

[Error] Cannot sync git repo to specified state. Please make sure the pull request is mergeable. Details: Git merge results in conflict, repository remote url: https://github.com/NuGet/docs.microsoft.com-nuget, working directory: W:\rdvl-s, target branch: HEAD, target branch commit: 2992b0a46e15ceca7bb73f18e9fa4b20fd4e7d0c, source commit: 5c8c3d5e21f40941c8a569f3ed42eb91585debf8.

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

Docs Build status updates of commit 05e15f4:
:warning: Validation status: warnings

File
Status
Preview URL
Details

docs/consume-packages/Package-References-in-Project-Files.md
:warning:Warning
View
Details

docs/reference/cli-reference/cli-ref-pack.md
:white_check_mark:Succeeded
View

docs/consume-packages/Package-References-in-Project-Files.md

Line 208: [Warning] Invalid file link: '../reference/errors-and-warnings'.

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


Fix PipeTransport crash

This resolves a non-deterministic crash I ran into in OnReadStreamAborted which was taking down Visual Studio.

:+1:


setParent() claims to return mixed|void, should omit @return

See https://github.com/peridot-php/peridot/blob/21ae9954b4f02d6c347f1ed26633b367a7652ee5/src/Core/AbstractTest.php#L131

@ezzatron want to make a PR for this?

I don't have the time for it right now, unfortunately. If you'd like to close, that's fine. Same goes for the other issue I raised. Sorry!


postman variables that reference broader scoped postman variables are not replaced

I have postman collection variables that reference postman environment variables and postman environment variables that reference postman global variables. Here is the output after running postman-to-k6 script.
postman[Symbol.for("initial")]({
  options,
  global: {
    local_auth_token: "my_local_api_auth_token",
    dev_auth_token: "my_dev_api_auth_token",
    stage_auth_token: "my_stage_api_auth_token",
    prod_auth_token: "my_prod_api_auth_token"
  },
  collection: {
    collection_name: "automated_package_details_by_uuid",
    product: "{{product_zulu}}"
  },
  environment: {
    base_url: "https://stage.api.azul.com/metadata/v1",
    auth_token: "{{stage_auth_token}}",
    product_zulu: "zulu"
  }
});

When exporting environment files from Postman variables are copied verbatim, leaving the variable references in double curly brackets. Unfortunately the postman-to-k6 script doesn't replace variables with the broader scoped variable values, so all variables that reference other variables do not end up with their expected values.

No these are all unchanged during test runs. I use global variables mainly to pass auth tokens into ci pipelines and keep them out of source control. The less protected env variables are stored in postman environment json and are stored in version control.

@betzrhodes It is a bit difficult to understand the setup you are using, would you mind sharing your Postman files & Postman ENV files and the postman-to-k6 command you are using?
This will allow me to reproduce and see for options to solve this.

postman json exported files can be found here:
https://gist.github.com/betzrhodes/97b3fef6afb046254ad11e18f7629a6d
Here is a screenshot of the project directory

This is the command I'm running from the postman-k6 directory
npx @apideck/postman-to-k6 automated_package_details_by_uuid.postman_collection.json --environment stage.postman_environment.json --global metadata_api.postman_globals.json -s -o k6-script.js

One more thing to note, when running locally with -s option, all files in the requests directory do not end up with the correct import relative path. I've been adjusting import statements after the running the script. Looks to be the same issue reported here

@betzrhodes The -s issue and import relative path is fixed (but not yet released).
I want to tackle your bug before releasing a new version.
I've reproduced the situation with your collection, env & global vars.
I struggle to understand what is the expected outcome and what goes wrong with the variables & the scopes?
Could you explain with an example what would be the desired variables?

Does this describe it?
CURRENTLY:
postman[Symbol.for("initial")]({
  options,
  global: {
    local_auth_token: "my_local_api_auth_token",
    dev_auth_token: "my_dev_api_auth_token",
    stage_auth_token: "my_stage_api_auth_token",
    prod_auth_token: "my_prod_api_auth_token"
  },
  collection: {
    collection_name: "automated_package_details_by_uuid",
    product: "{{product_zulu}}"
  },
  environment: {
    base_url: "https://stage.api.azul.com/metadata/v1",
    auth_token: "{{stage_auth_token}}",
    product_zulu: "zulu"
  }
});

EXPECTED
postman[Symbol.for("initial")]({
  options,
  global: {
    local_auth_token: "my_local_api_auth_token",
    dev_auth_token: "my_dev_api_auth_token",
    stage_auth_token: "my_stage_api_auth_token",
    prod_auth_token: "my_prod_api_auth_token"
  },
  collection: {
    collection_name: "automated_package_details_by_uuid",
    product: "zulu"
  },
  environment: {
    base_url: "https://stage.api.azul.com/metadata/v1",
    auth_token: "my_stage_api_auth_token",
    product_zulu: "zulu"
  }
});

Yes that is exactly the issue.

@betzrhodes I have a PR ready, just need to figure out a way to add additional tests.
The enhancement will be part of the next postman-to-k6 release.

Closing the issue, since the latest postman-to-k6 version (1.8.5) contains this enhancement.


Snakemake output prints sensitive information with storage plugin

Snakemake version
8.20.3
Describe the bug
When using a storage plugin e.g. snakemake-storage-plugin-xrootd the user can provide a username/password or encode an authentication token in the file path. When snakemake then prints the DAG or error messages the full path, including the token/username/password whatever, is printed in the terminal exposing potentially sensitive information.
It would be great to have a way to be able to ensure that only ever a censored form of the URL is printed (but still allow for the full, uncensored URL to be passed as input/output to the shell part).
Minimal example
storage:
    provider="xrootd",
    host="eoslhcb.cern.ch",
    username="my_username"

rule test_rule:
    input: storage("root://eos/my_file.root")
    output: "test.flag"
    shell: "touch {output}"

It will correctly fail due to the file not existing and incorrect credentials:
$ snakemake -c1 test.flag -n --storage-xrootd-password="my_password"
A password has been specified -- it will be printed in plaintext when Snakemake displays the inputs/outputs of jobs! Only use this option in trusted environments.
Building DAG of jobs...
Error checking existence of root://my_username:****@eoslhcb.cern.ch:1094//eos/my_file.root on XRootD: [ERROR] Server responded with an error: [3010] Unable to give access - user access restricted - unauthorized identity used ; Permission denied
, attempt 1/3 failed - retrying in 3 seconds...
Unrecoverable error, no more retries
WorkflowError:
Failed to check existence of root://my_username:my_password@eoslhcb.cern.ch:1094//eos/my_file.root
XRootDFatalException: Error checking existence of root://my_username:****@eoslhcb.cern.ch:1094//eos/my_file.root on XRootD: [ERROR] Server responded with an error: [3010] Unable to give access - user access restricted - unauthorized identity used ; Permission denied

where you can see (more than once) the password would be printed in plain text.
Additional context
The snakemake-storage-plugin-xrootd has this postprocess_query method which can be changed to print out a censored URL instead but, as far as I can tell, this then prevents the plugin from working correctly because the censored URL is used as the "file" rather than actual full URL.
Our particular use-case for this is running CI/CD tests where the authentication is handeled by appending a token into the URL parameters and we would prefer not to be exposing that in the CI logs if we can avoid it!

I've opened a draft PR that I think would solve this: https://github.com/snakemake/snakemake/pull/3089
Comments/suggestions welcome!


Add Storage File API

What kind of change does this PR introduce?
Adds Storage File API methods, previous PR was too clunky.
What is the current behavior?
Please link any relevant issues here.
What is the new behavior?
Feel free to include screenshots if it includes visual changes.
Additional context
Add any other context or screenshots.

Sorry to bother, but any ETA on this?

No worries,  a slot just cleared up so I'll be working on it for the next 3h. ETA... 12h from now perhaps

Hey
Notes for myself:
Should be able to use all functions(with exception of update) as is right now. There are Four main tasks which need to be done:

Modify syntax to match js version (e.g. .from.upload() .from.download() etc)
Support update and maybe concurrent upload of files
Add actual tests, right now I'm just testing in a jupyter notebook against the js version(in code sandbox)

Will send a version of this for review tomorrow.


Cannot execute statements if I declare a variable in editor

After a variable declaration, the button Execute disappears from the view and if I click Execute all it doesn't work. Here is a screenshot:

Found that even without a variable, trying to execute .toArray() method it causes the same behaviour.
So, I guess that after any JS code in the file, all queries won't not work, even with methods from cursors.

@patriciunista We appreciate your use of our extension. Scrapbooks are not intended to be a full runtime environment, but more for quick, exploratory querying. For what you're trying to do, it would be better to use install and use the actual Mongo shell.  Sorry for the inconvenience.
Stephen


[AI Agent idea] AIOptima is your intelligent companion for enhancing productivity, gaining knowledge, and simplifying everyday tasks.

Description
AIOptima is your intelligent companion for enhancing productivity, gaining knowledge, and simplifying everyday tasks.
Additional Information
Glimpse

/assign @Alkaison


Add answers file to template.

This is something I had missed previously. If we include the answers to the template generator in the generated repo, it is possible to apply updates to the template to the project, well after it has been generated from the project.
I'm not sure exactly how well it would work in practice, but it is one of the selling points of copier, and I don't see any harm in including it.

Thanks for the review!


chore(deps): update goreleaser/goreleaser docker tag to v1.17.1

This PR contains the following updates:

Package
Update
Change

goreleaser/goreleaser
patch
v1.17.0 -> v1.17.1

Release Notes

goreleaser/goreleaser
v1.17.1
Compare Source
Changelog
Bug fixes

b36e30a: fix(jsonschema): add binary to Archive format enum (#3924) (@maxbrunet)
ac19f90: fix(nfpm): Add extension to produced artifacts (#3940) (@mbamber)
11b32cf: fix: build should output more debug info (#3939) (@caarlos0)
e872e45: fix: improve docker error handling (#3942) (@caarlos0)
f6b5e9a: fix: source archive not being added when no extra-files (#3938) (@caarlos0)

Full Changelog: https://github.com/goreleaser/goreleaser/compare/v1.17.0...v1.17.1

Find examples and commented usage of all options in our website.
Want to help? You can sponsor,get a Pro License or contribute.
Also, feel free to reach out on Discord and Twitter!

Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
🔕 Ignore: Close this PR and you won't be reminded about this update again.

[ ] If you want to rebase/retry this PR, check this box

This PR has been generated by Renovate Bot.

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


SLA for Zone Redundant VNet Gateway

Does a Zone Redundant VNet Gateway improve the documented 99.95% SLA?

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

ID: 12c6b382-2861-f458-4ec5-846c7eb12924
Version Independent ID: 4117fae1-0409-5f77-e4a9-7f83018706c7
Content: Create a zone-redundant virtual network gateway in Azure Availability Zones
Content Source: articles/vpn-gateway/create-zone-redundant-vnet-gateway.md
Service: vpn-gateway
GitHub Login: @cherylmc
Microsoft Alias: cherylmc

@cjlpowers , Thank you for your feedback. We will investigate and get back to you shortly.

@cjlpowers ,  the SLA is same for Zone Redundant gateway as well, which is 99.95%.

@cjlpowers , Do you have any update in this issue?

@cjlpowers , I will go ahead and close this issue, Please let me know if you have any further questions.


Text size

Is it possible to set the text size smaller?

You can add custom css by following the MagicMirror docs here with a font-size rule for .module.MMM-connection-status

hello, unfortunately in the suggestion it refers to weather or you have colors... can you give an example for your module so that we can understand it better?


Move docs content into docs repo

Move the docs content into the docs repo. The marketing related content should remain here.

Closing this as all content will soon be moving into the docs repo.


exactly match a subcommand when using the infersubcommands

Closes #1254.

This change is

Excellent, thanks!


SOLVED SORT_01 problem using C++

name: PR Template
about: Pull Request Template
title: ''
labels: hacktoberfest
assignees: ''

Description
SOLVED the problem SORT_01
Test Configuration: (If any)

Firmware version: C++ 17
Hardware:
Toolchain:
SDK:

Checklist:

[ .] My code follows the style guidelines of this project
[ ] I have made corresponding changes to the documentation
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] Any dependent changes have been merged and published in downstream modules

@KuSantosh100 good job 👍

@ZekromRND  Thank you very much


manageiq build on Power Linux (ppc64le) platform

Creating this issue to seek some help.
The bundle install command
bundle install --jobs=3 --retry=3 --path=${BUNDLE_PATH:-vendor/bundle}
is failing for me with the below error. I am able to successfully build sassc gem from source code after making changes in the source code from https://github.com/sass/sassc-ruby locally and the gem installs successfully. Is there a simple way to consume the locally built version in manageiq is what I am looking to understand here …
Installing sassc 2.2.1 with native extensions
Gem::Ext::BuildError: ERROR: Failed to build gem native extension.
current directory: /tmp/manageiq/vendor/bundle/ruby/2.6.0/gems/sassc-2.2.1/ext

/usr/local/rvm/rubies/ruby-2.6.3/bin/ruby -I /usr/local/rvm/rubies/ruby-2.6.3/lib/ruby/site_ruby/2.6.0 -r ./siteconf20191115-91511-13a35th.rb extconf.rb
creating Makefile
current directory: /tmp/manageiq/vendor/bundle/ruby/2.6.0/gems/sassc-2.2.1/ext
make "DESTDIR=" clean
current directory: /tmp/manageiq/vendor/bundle/ruby/2.6.0/gems/sassc-2.2.1/ext
make "DESTDIR="
compiling ./libsass/src/ast.cpp
g++: error: unrecognized command line option ‘-march=native’
make: *** [ast.o] Error 1
make failed, exit code 2
Gem files will remain installed in /tmp/manageiq/vendor/bundle/ruby/2.6.0/gems/sassc-2.2.1 for inspection.
Results logged to /tmp/manageiq/vendor/bundle/ruby/2.6.0/extensions/powerpc64le-linux/2.6.0/sassc-2.2.1/gem_make.out
An error occurred while installing sassc (2.2.1), and Bundler cannot continue.
Make sure that gem install sassc -v '2.2.1' --source 'https://rubygems.org/' succeeds before bundling.
In Gemfile:
manageiq-ui-classic was resolved to 0.1.0, which depends on
patternfly-sass was resolved to 3.59.4, which depends on
bootstrap-sass was resolved to 3.4.1, which depends on
sassc
If I completely bypass manageiq-ui-classic (just for understanding the scope of issues on ppc64le), the bundle install command succeeds. but bundle exec rake errors out...
https://travis-ci.org/seth-priya/manageiq
Any pointers here would be of great help, thank you!!

@seth-priya I think the only way to use .gem is to put it in vendor/cache. The alternative is to point sassc to your modified source (like your fork) using :git => in Gemfile.
cc @jrafanie since you might have other ideas

@seth-priya First of all, it's awesome you have a possible fix for the sassc gem for powerpc.  I'm not familiar with the gem but we can probably offer suggestions on how we've had upstream changes accepted.
Regarding how to get manageiq to run with a custom gem that we depend on, we automatically pull in any gems you define in .bundler.d/*.  You might want to try something like the 4th example below.  Just uncomment the line, change the gem name, version, path, etc. to match what you want and run bundle update on manageiq.
$ cat bundler.d/Gemfile.dev.rb
# 1. To add a rubygems based development gem to my bundle to be used with manageiq
# gem 'memory_profiler' if RUBY_VERSION > "2.1.0"

# 2. To add a github based gem...
# gem 'memory_profiler', :github => 'SamSaffron/memory_profiler'  if RUBY_VERSION > "2.1.0"

# 3. To add a local path based gem to the bundle (for existing gems that manageiq already depends on)...
# override_gem 'manageiq-ui-classic',:path => "/Users/joerafaniello/Code/manageiq-ui-classic"

# 4. Or for local path based gems that aren't already a dependency
# gem 'memory_profiler', :path => "/Users/joerafaniello/Code/memory_profiler"

I think number 3, the bundler.d direction with override_gem or ensure_gem might be your best bet for now...
In bundler.d/Gemfile.dev.rb (the filename can be anything you want, this is just an example)
ensure_gem "sassc", :git => "<your_fork_url>"

Since sassc is a transient dependency (i.e. not in our Gemfile), for now you can't use override_gem due to this bug, but ensure_gem should work.

Thanks @jrafanie @simaishi @Fryguy I tried the last suggestion and the build completed successfully :) I am trying out the test cases now
Thanks for the help!

I found this https://github.com/sass/sassc-ruby/issues/146 and it essentially means that the gem can be installed (on ppc64le as well) by using the below command
gem install sassc  -- --disable-march-tune-native
gem install sassc  -- --disable-march-tune-native
Building native extensions with: '--disable-march-tune-native'
This could take a while...
Successfully installed sassc-2.2.1
Parsing documentation for sassc-2.2.1
Done installing documentation for sassc after 3 seconds
1 gem installed

and once I have the sassc gem installed / pre-installed on the system, manageiq bundle install completes successfully as well!

@seth-priya can you try setting this in your manageiq app's .bundle/config so you don't have to remember to do this whenever you install new versions?
11:27:57 ~/Code/manageiq (master) (2.6.5) + bundle config --local build.sassc --disable-march-tune-native
You are replacing the current local value of build.sassc, which is currently nil
11:28:06 ~/Code/manageiq (master) (2.6.5) + cat .bundle/config
---
BUNDLE_JOBS: "3"
BUNDLE_BUILD__SASSC: "--disable-march-tune-native"

Unfortunately, sassc isn't a direct dependency in manageiq so it feels weird to add that option here.

@jrafanie this works, thank you, this works for me

Thanks all for your support and patience in answering my queries, I am closing this issue and will be creating a new one to discuss about adding support for ppc64le to ManageIQ, I have the basic changes in place.
Thank you!!


Legend Box without styling

Hi,
Is it possible to configure the plugin to not draw the styling on the legend?
Something like this:

#14

@nagix I have the same request. I want to add drop shadows to my line charts but that should not affect my legend. Is there a simple way to disable this for the legend?

What I did was adding a new booleand option named 'legendShadow' which make gives you the possibility to enable/disable shadow on legend.
And use it like this:
{  type: 'horizontalBar', label: 'Post', yAxisID: 'y-axis-left', data: dataPost, labels, fill: true, borderColor: '#5b9bd5', backgroundColor: '#5b9bd5', shadowOffsetX: 1, shadowOffsetY: 1, shadowBlur: 7, shadowColor: colorPost, legendShadow: false }
Please find it under:
https://github.com/satisfazm2bezes/chartjs-plugin-style
Hope it helps.

@satisfazm2bezes Hello, I'm trying to port this repo to work with chartjs v3. But there are some variables that need to be updated in the code. I was trying to do that, but i have issues exporting a new version with gulp. Somehow the dist stays empty after gulp build. Do you happen to know why? Thank you!


Add a few excel like functions

Can remove all the TargetFramework changes, just for compiling was easier so I didn't have to download all the visual studio tools for UWP, framework x, etc...

Sorry didn't intend to open this against your repo


Handle null response body and support Parameters

Ryan, first thank you for your great library.
I have created a derivative work to add support for aws gateway api to the apache nifi project,
NIFI PR#2588
here are a couple of changes that I've made.

add support for request parameters
add to git ignore for intellj files and target directory

This change is

Merged, thanks again. Can you elaborate on how you plan to use this in Apache Nifi?

I'm using the classes ( with reference to you in the NOTICE as the apache lic. requires ) to add a gateway api processor that lets you invoke http methods on it from nifi.
Are you familiar with nifi?


Image Path Not Found

ErrorException (E_WARNING)
rename(/home/aditya/Documents/ibikri-shabab-master/public/images/temp/1_5c694a71da931/thumb_5c694a71e5f29.jpeg,/home/aditya/Documents/ibikri-shabab-master/public/images/thumb/1_5c694a71e5f29.jpeg): No such file or directory

Same error here

Did you fixed it?

Did you fixed it?

I did fix it, but it was a really long time ago, I'm sharing the project with you... I hope it helps.. You can shuffle through the files and find what is missing... If I remember correctly, the folder was missing, I created the folder and it fixed it, but again, I'm guessing.
Here's the G-Drive-Link:
https://drive.google.com/open?id=1kJKMOm1YRWxqZnL_RMSKSiNK5uvEcl6F

Yeah fixed it, now I trying to integrate it to email pass when
registration, when post ads and aproved ads, and when promote pass to email
to customer and to me, and I want change promotion ads system
Thank you so much my dear,
I would like to get your whatsapp or viber number If you like please share
privetly.
On Fri, Jul 12, 2019, 11:15 AM Aditya Sissodiya<EMAIL_ADDRESS>wrote:

Did you fixed it?
I did fix it, but it was a really long time ago, I'm sharing the project
with you... I hope it helps.. You can shuffle through the files and find
what is missing... If I remember correctly, the folder was missing, I
created the folder and it fixed it, but again, I'm guessing.
Here's the G-Drive-Link:
https://drive.google.com/open?id=1kJKMOm1YRWxqZnL_RMSKSiNK5uvEcl6F
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/shababhsiddique/ibikri-shabab/issues/1?email_source=notifications&email_token=ALZYF7HBTRTFNACTZVWOSFLP7ALAJA5CNFSM4GX6CO4KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODZYX3SQ#issuecomment-510754250,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ALZYF7HDPCB7SHXIY5VSUGTP7ALAJANCNFSM4GX6CO4A
.


can't access metrics after deploy QSFS by terraform

with devnet i deployed a qsfs and after deploy success i can ssh into vm but can't access metrics
https://github.com/threefoldtech/terraform-provider-grid/issues/141
Moved from: https://circles.threefold.me/project/test-tfgrid3/issue/54

duplicate of #31


Versions do not allow for comparison

Description

Searching xclim in conda does not work unless we specify the exact version.
What I Did

$ conda search xclim==0.10.10b0
Loading channels: done
# Name                       Version           Build  Channel
xclim                      0.10.10b0            py_0  conda-forge
$ conda search xclim>=0.10.10
$ conda search xclim>=0.10.10b0

Works for me.  Maybe you have an old version of conda?
$ conda search xclim
Loading channels: done
# Name                       Version           Build  Channel
xclim                          0.9b0            py_0  conda-forge
xclim                         0.10b0            py_0  conda-forge
xclim                       0.10.6b0            py_0  conda-forge
xclim                       0.10.7b0            py_0  conda-forge
xclim                       0.10.8b0            py_0  conda-forge
xclim                       0.10.9b0            py_0  conda-forge
xclim                      0.10.10b0            py_0  conda-forge

$ conda --version
conda 4.7.11

Same results for me. Versions are following proper conventions; the Azure checks that conda-forge uses would fail otherwise.
(base) ➜  ~ conda search xclim
Loading channels: done
# Name                       Version           Build  Channel
xclim                          0.9b0            py_0  conda-forge
xclim                         0.10b0            py_0  conda-forge
xclim                       0.10.6b0            py_0  conda-forge
xclim                       0.10.7b0            py_0  conda-forge
xclim                       0.10.8b0            py_0  conda-forge
xclim                       0.10.9b0            py_0  conda-forge
xclim                      0.10.10b0            py_0  conda-forge

So I was not clear: comparisons don't work. Let's say you want to say use xclim >= 0.10, conda doesn't find it. I'm guessing it has to do with the b0 at the end.

This would probably be fixed once we have a release version (or drop the beta tag altogether). Once we have something that we are comfortable to identify as a "release", we can then start only pushing "release" versions to PyPI and conda-forge.
We could start doing this with 0.11.0 once we've cleared our development goals for that tag. Thoughts?

Closed with https://github.com/Ouranosinc/xclim/releases/tag/v0.11.1


Require inside of a client.d file does not work on first run

Description
When you attempt to load a custom gem via the chef client.d folder in Chef Client (Chef Zero is also affected) it will not be available and raise an exception at the end of the run in which it is created and loaded. Subsequent runs will work normally.
Chef Version
Chef 12 and 13
Platform Version
Tested platforms:
Ubuntu 14.04 & 16.04
CentOS 6 & 7
Replication Case
I have created a pull request which uses the example in the README (with corrections) in the test suite to demonstrate the problem. My hunch is that Chef::Config[:client_d_dir] for the chef_zero provisioner is going to /tmp and not loading the gem path to the newly installed gem via metadata.rb or the chef_gem directive. Somehow it's still being called at the end of the chef run.
Client Output
I, [2018-03-29T00:17:44.656488 #5478]  INFO -- default-ubuntu-1604: Running handlers:
I, [2018-03-29T00:17:44.656630 #5478]  INFO -- default-ubuntu-1604: [2018-03-29T05:17:42+00:00] INFO: Running report handlers
I, [2018-03-29T00:17:44.656784 #5478]  INFO -- default-ubuntu-1604: Running handlers complete
I, [2018-03-29T00:17:44.657008 #5478]  INFO -- default-ubuntu-1604: [2018-03-29T05:17:42+00:00] INFO: Report handlers complete
I, [2018-03-29T00:17:44.657148 #5478]  INFO -- default-ubuntu-1604: Chef Client finished, 88/140 resources updated in 02 minutes 03 seconds
I, [2018-03-29T00:17:44.657290 #5478]  INFO -- default-ubuntu-1604: [2018-03-29T05:17:42+00:00] DEBUG: Server doesn't support resource history, skipping resource report.
I, [2018-03-29T00:17:44.657439 #5478]  INFO -- default-ubuntu-1604: [2018-03-29T05:17:42+00:00] DEBUG: Audit Reports are disabled. Skipping sending reports.
I, [2018-03-29T00:17:44.657576 #5478]  INFO -- default-ubuntu-1604: [2018-03-29T05:17:42+00:00] DEBUG: Forked instance successfully reaped (pid: 1562)
I, [2018-03-29T00:17:44.657719 #5478]  INFO -- default-ubuntu-1604: [2018-03-29T05:17:42+00:00] FATAL: Configuration error LoadError: cannot load such file -- bugsnag
I, [2018-03-29T00:17:44.657899 #5478]  INFO -- default-ubuntu-1604: [2018-03-29T05:17:42+00:00] FATAL:   /tmp/kitchen/client.d/bugsnag.rb:1:in `from_string'
I, [2018-03-29T00:17:44.658042 #5478]  INFO -- default-ubuntu-1604: [2018-03-29T05:17:42+00:00] FATAL: Aborting due to error in '/tmp/kitchen/client.d/bugsnag.rb'

Stacktrace
I, [2018-03-29T00:17:44.657719 #5478]  INFO -- default-ubuntu-1604: [2018-03-29T05:17:42+00:00] FATAL: Configuration error LoadError: cannot load such file -- bugsnag
I, [2018-03-29T00:17:44.657899 #5478]  INFO -- default-ubuntu-1604: [2018-03-29T05:17:42+00:00] FATAL:   /tmp/kitchen/client.d/bugsnag.rb:1:in `from_string'
I, [2018-03-29T00:17:44.658042 #5478]  INFO -- default-ubuntu-1604: [2018-03-29T05:17:42+00:00] FATAL: Aborting due to error in '/tmp/kitchen/client.d/bugsnag.rb'

Workaround
For the time being, I have wrapped the requires in rescue blocks like so:
begin
  require 'bugsnag'
  Bugsnag.configure do |config|
    config.api_key = '1234567890'
  end
rescue LoadError
  Chef::Log.warn('Could not load bugsnag gem. If this is the first run, it will resolve itself. If not, then something has gone horribly wrong.')
end

I have now confirmed this also affects Chef-Client 13.8 without Chef Zero mode so this is a core chef problem.

So earlier in the run you've successfully done chef_gem "bugsnag" or installed it via metadata.rb then?
Offhand, I can only be puzzled.  It smells like bundler, but we don't bundle inside of TK AFAIK, and AFAIK we don't do anything weird with rubygems in report handlers, and would probably look more in the direction of the rubygems package resource behaving strangely.  If its a regression it might be due to the recent patch there to work around the embedded-gems problem.

So earlier in the run you've successfully done chef_gem "bugsnag" or installed it via metadata.rb then?

Correct. I had a cookbook in the run list as the first item which drops off this client.d file and in it's metadata is a gem 'bugsnag' so at the beginning of the run it installs. Near the end of the run the client.d file is dropped off. Somewhere between there and the end of the run the file gets loaded and thus crashes, but without any stacktrace.

 DEBUG: Forked instance successfully reaped (pid: 1562)

that line indicates that the forked process was successfully reaped and is gone.
what is happening is that the daemon process re-reads the client.d file now and the require "bugsnag" causes it to blow up even though the gem was installed by the child process, which smells a lot like its the internal gem state of the parent process, and likely Gem.clear_paths put on the line before the require will fix it.

what is happening is that the daemon process re-reads the client.d file now and the require "bugsnag" causes it to blow up even though the gem was installed by the child process, which smells a lot like its the internal gem state of the parent process, and likely Gem.clear_paths put on the line before the require will fix it.

And indeed that did fix the issue. I have updated my original post about the issue for others to find as a solution.

This is now resolved by #7106 for Chef 14. Anyone on 12 and 13 can do the workaround to resolve this issue. Thank you for the assistance @lamont-granquist and @coderanger ❤


⚠ Turnos 24x7 has degraded performance

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

HTTP code: 200
Response time: 929 ms

Resolved: Turnos 24x7 performance has improved in f4d18fe.


Add Chromatic Dispersion (CD) test

400ZR CD Test

Pull Request Functional Test Report for #2806 / ee7c33857326ae8467e1074983ce37d3e4b11253
Virtual Devices

Device
Test
Test Documentation
Job
Raw Log

Arista cEOS

:
900a5734
Log

Cisco 8000E

:
0c7d7d27
Log

Cisco XRd

:
891a2212
Log

Juniper ncPTX

:
2657f8d2
Log

Nokia SR Linux

:
8555492c
Log

Openconfig Lemming

:
d8e6ae9e
Log

Hardware Devices

Device
Test
Test Documentation
Raw Log

Arista 7808

:

Cisco 8808

:

Juniper PTX10008

:

Nokia 7250 IXR-10e

:

Help

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
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 54.484%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
1847

Relevant Lines:
3390

💛  - Coveralls

PR ready for review


can't connect to server

MC: 1.20.1
Forge: 47.2.17
Mod whitelist: 1.1.0
I installed mod whitelist on server and client, but it won't allow connection, even when the whitelist is currently empty (aside from "forge") as i haven't configured the whitelist yet. 'Only allow whitelist' is set to "false".
When trying to connect, all it says to the client is failed to connect to server. And the server console doesn't even show a connection attempt

If there's no connection attempt shown in the console, it may cause by the firewall, or incorrect ip address and port inputted on your client side. Furthermore, check whether you logged in with online mode or not.

If there's no connection attempt shown in the console, it may cause by the firewall, or incorrect ip address and port inputted on your client side. Furthermore, check whether you logged in with online mode or not.

Firewall is not blocking it.
Server address is correct.
Disabling the mod on both sides allows connection.
Does the mod use udp traffic? I'm using TCP shield, and I believe only routes TCP traffic on the free plan.

Does the mod use udp traffic? I'm using TCP shield, and I believe only routes TCP traffic on the free plan.

No, only adds some more extra carries to the login package. This may caused by no ModWhitelist mod or mismatched version on client side.

Both client and server versions are 1.1.0.
Connecting locally, does work, however the firewall is NOT blocking connections, as without mod whitelist, there is zero issues connecting to the server externally.
My best guess is something to do with the connecting going through the TCP Shield service.

i'm going to set up an alternate dns record, and attempt to connect to the server, bypassing tcp shield to see if it connects without it.

bypassing tcp shield and connecting to the server with mod whitelist enabled does work. I'm going to open a ticket with them. maybe they will know what's up.

I think maybe tcp shield think the package is too big for logging in so it refuse your connection if you install a number of mods with out Mod Whitelist. Thanks for letting me know about it.

my client modpack is 384 mods (fully loaded it shows around 450? thanks to sinytra)

I don't know why it wouldn't allow the connection tbh, because it's literally just a one mod difference lol.
I don't know how much data is added to the payload with mod whitelist so... lol

If you manage to come up with a workaround, i'd be more than happy to be a beta tester for it.

I think maybe tcp shield think the package is too big for logging in so it refuse your connection if you install a number of mods with out Mod Whitelist. Thanks for letting me know about it.

also, this is being used as merely a proxy, there's no additional mods/packages installed on the server in order to use it.


Update README.md

Sorry about cheesy change, just wanted to correct the query copypaste

Closing this PR for now, as we're working on a substantial rework of the codebase that's expected to undergo more rapid changes as I work on stabilizing things.
If you'd like to get involved in the rework, join our Discord and reach out in the #form channel


tests failing on macOS High Sierra

This is from running the tests on my laptop.
$ sw_vers
ProductName:    Mac OS X
ProductVersion: 10.13.4
BuildVersion:   17E199

failure output:
bad syslog function pointer: rust:<PHONE_NUMBER>68871 (0x7fff548c4cc7) != c<PHONE_NUMBER>69453 (0x7fff548c4f0d)
bad VM_FLAGS_USER_ALLOCATE value at byte 0: rust: 27 (0x1b) != c 31 (0x1f)
bad VM_FLAGS_USER_MAP value at byte 0: rust: 27 (0x1b) != c 31 (0x1f)

and here's the relevant snippet of my /usr/include/mach/vm_statistics.h of the conflicting VM_* constants. I'm not sure how to help with the syslog function pointer error, but happy to give more info as requested.
/* These are the flags that we accept from user-space */
#define VM_FLAGS_USER_ALLOCATE	(VM_FLAGS_FIXED |		\
				 VM_FLAGS_ANYWHERE |		\
				 VM_FLAGS_PURGABLE |		\
				 VM_FLAGS_4GB_CHUNK |		\
				 VM_FLAGS_RANDOM_ADDR |		\
				 VM_FLAGS_NO_CACHE |		\
				 VM_FLAGS_OVERWRITE |		\
				 VM_FLAGS_SUPERPAGE_MASK |	\
				 VM_FLAGS_ALIAS_MASK)
#define VM_FLAGS_USER_MAP	(VM_FLAGS_USER_ALLOCATE |	\
				 VM_FLAGS_RETURN_4K_DATA_ADDR |	\
				 VM_FLAGS_RETURN_DATA_ADDR)

Thanks for the report! These sorts of things can change over time and it should be fine to tweak the values as they change.

MacOS's libc, which is somewhere in their SDK, is not based on glibc, there are a lot of inconsistencies, such as different struct layouts, same consts with different values and so on. Maybe disable libc build for OS X entirely since it's not reliable in any way, unfortunately. Other then full sanity check and somewhat reimplemenation for OSX and probably BSD targets.

Thanks for the report! These sorts of things can change over time and it should be fine to tweak the values as they change.

This means that people can't target some SDK versions with libc. These constants changed again in xcode10. I can update them, but then people will stop being able to target xcode9.2 with them. What's worse is that if the std library use them as options to apple APIs, well depending on the system we might be calling APIs with different options =/

I've just hit a similar issue with 10.14 and NET_RT_MAXID (10 in git, 11 in 10.14). Does anyone know of a cfg! way or similar to match a OS X version?

You might want to check the mach crate. It has a mach-test crate inside like libc-test crate, which contains some code for detecting the osx version and using that with ctest: https://github.com/fitzgen/mach/blob/master/mach-test/build.rs#L4


Adds carthage support

...by adding an xcode project.

@Goos I thought the _Pods.xcodeproj symlink exposed the Example/ project file, which allows you to use the framework target that's in there. Is that not the case?

@zacwest afaik carthage assumes there to be an .xcodeproj with the same name as the library/repo name. I'm getting an
Reason: Project [...]/ZSWTappableLabel/ZSWTappableLabel.xcodeproj cannot be opened because it is missing its project.pbxproj file.

error when I'm building without the changes.

I'm getting the following error when trying to build 3.2 with carthage:
Dependency "ZSWTappableLabel" has no shared framework schemes for any of the platforms: iOS


Use the announceAddresses to check the port forwarding

The echo service, for some reason, is not using the ip parameter when trying to lookup the port (https://github.com/mpolden/echoip/blob/d84665c26cf7df612061e9c35abe325ba9d86b8d/http/http.go#L177C13-L177C26).  But we should be able to give the IP by using the X-Forwarded-For header (https://github.com/mpolden/echoip/blob/d84665c26cf7df612061e9c35abe325ba9d86b8d/http/http.go#L93C43-L95)

@veaceslavdoina can you add x-real-ip-custom to the  ‘Access-Control-Allow-Headers’  ? I have a CORS error when trying to fetch

Done, and we can check it
curl \
  https://echo.codex.storage/port/8070 \
  -X OPTIONS \
  -H 'X-Real-IP-Custom: <IP_ADDRESS>' \
  -H 'Access-Control-Request-Method: GET' \
  -H 'Origin: http://localhost:3000/' \
  -v

< access-control-allow-origin: *
< access-control-allow-methods: GET, POST, OPTIONS, HEAD
< access-control-allow-headers: X-Real-IP-Custom

Okay thanks.
Can you try it on https://releases-v0-0-13.codex-marketplace-ui.pages.dev/ ?  @veaceslavdoina

Looks like we do not get/pass a port?
This is a check on a localhost and same is with a remote node.

Oh sorry I have a proxy object intercepting the requests (for the tests). I think it is a bad idea I removed it, can you try it again ?

Works as expected on localhost with IPv4 and remote VM with IPv6, thank you for the fix.


[UBS Admin | Order form] The field of total payment "Замовлення оплачено в розмірі" must be "0.00 grn" after full payment of the order with bonuses

Environment: Windows 10 Pro, Browser: Google Chrome 94.0.4606.71 (Official Build) (64-bit).
Reproducible: always.
Build found: January 10, 2022
Preconditions

Log in to the GreenCity page https://ita-social-projects.github.io/GreenCityClient/#/ as Admin user.
Open the 'Замовлення' page.
Prepare an order paid for with bonuses (e.g. 3685)

Steps to reproduce

Click on the prepared order's number in the "Номер замовлення" column
Scroll down to the "Деталі замовлення" section
Check if the payment is made with bonuses in the "Бонуси" row
Click on the "Оплата замовлення" section

Actual result
The field of total payment "Замовлення оплачено в розмірі" isn't "0.00 грн" but is displayed the number of bonuses used.
Expected result
The field of total payment "Замовлення оплачено в розмірі" is "0.00 грн"
User story and test case links
E.g.: "User story #2898
Test case GC-2331"
Labels to be added
"Bug", Priority ("pri: medium "), Severity ("severity:"), Type ("Functional")

The field of total payment still shows the number of bonuses that were used.

@Horokh How did you add payment of bonuses more than the sum of the order?


Fix bare links in docs against Rust 1.53

Rust 1.53 added a lint for bare URLs: https://github.com/rust-lang/rust/pull/81764
This PR fixes doc generation so that docs continue to build with the latest stable compiler, but does not attempt to be the long-term documentation normalization strategy.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

would be nice to have a generated code diff to audit

On second thought, there isn't really a good way to fix this without full parsing of the documentation. There are cases in the documentation such as this that should not get transformed:
curl
http://someurl

Or
[ "CMD-SHELL", "curl -f http://localhost/ || exit 1" ]

I'll table this and start on the long-term approach.


Don't allow merging pull requests without linked issue

When merging and closing pull request a linked issue will automatically be closed. Can be done either by linking branches and pull request from the web page under Development in the Sidebar UI -->
Or by using a text like closes #issuenumber
If we miss doing this an issue will not be automatically closed.
Describe the solution you'd like
I would to see if this action could work for automatic checking that a PR is linked to an issue.
There seems to be some limitations that the linking between PR and issue only can be done manually in the PR description? So no Sidebar UI?
There are alternatives to the action linked above. Probably they have the same limitation, demanding a link to the issue in the PR body?
https://github.com/tkt-actions/add-issue-links
https://github.com/hattan/verify-linked-issue-action

Automating the closure of linked issues upon PR merge improves workflow efficiency

Automating the closure of linked issues upon PR merge improves workflow efficiency

Something you would like to implement @TrungMa1 ?
The first link seems to be updated resently while the other one hasn't change in a long time.

Nothing particularly, it was just a statement.


automated deactivation of accounts end of season

As NFFTT staff I want the system to automatically deactivate everyone at the end of the season so that I don't have to do it manually.

I've run the task to deactivate all non admins for last year
It's based on cutoff date - basically when you sign up it sets your membership end date to March 31st of next year, and we deactivated everyone who had a membership end date before March 31st 2025.
This may not be quite right - I added it to the weekly agenda


French Text in translation file in newYearsDay.php

I found that in the file src/Yasumi/data/translations/newYearsDay.php
'fr' => 'Jour de l’An',
Can the quotation mark be replaced by a good quotation mark useable in the database ?
As this
'fr' => 'Jour de l\'An',

Finally i see this quotation mark everywhere....

Hi @oxomichael Actually the use of a straight apostrophe is in this case incorrect. For quotes as in your example the proper one to use is the typographic quote/apostrophe. See https://practicaltypography.com/straight-and-curly-quotes.html and the related
PR: https://github.com/azuyalabs/yasumi/pull/197
You say you want to use the straight quote for storing in the database. Could you explain a bit why the typographic form would cause in issue in the database?
Cheers! Sacha

Good! I work a lot with CMS applications and often this is misunderstood when content contains such an apostrophe.


pyspark use local[*] as master

There are 16 cores on my dev machine.

Using local[*] without increase the driver memory will result in a OOM exception.


Write a PHP program to find minimum of n numbers

Description
Write a PHP program to find minimum of n numbers
Input  : 10 20 30 40 50
Output : 10

How to contribute

Save the solution in program/find_minimum_of_n_numbers.php file
Add find_minimum_of_n_numbers.php file in find-minimum-of-n-numbers folder

!assign


Initial webdriver test

This is an initial webdriver test so I can start to build out the UI tests. I replaced karma because it was being flakey on my machine. Also I wanted to use the WebDriver Protocol to do browser testing because I believe its more robust.
I also moved as much of the config into ./config/ as to tidy up the root directory a little.

This is an initial webdriver test so I can start to build out the UI tests. I replaced karma because it was being flakey on my machine. Also I wanted to use the WebDriver Protocol to do browser testing because I believe its more robust.

Oh wow go for it. Also like the idea of moving the webpack configs into the config folder.

So I've also moved to testing all the latest patches on the CI systems so 4.x, 5.x, 6.x and 7.x (see https://github.com/nodejs/LTS)
I've had to add in a C compiler on the CI envs for odd number node.js versions, see https://github.com/laverdet/node-fibers/issues/333#issuecomment-282312238

Odd-numbered versions of Node are early release so you will need to install MSVC and the rest of the dependencies listed on the node-gyp GitHub page. Alternatively you can use node v6, or wait for node v8 this April.

This makes it slightly harder to install / build but only for odd node.js versions which I think is fine.

Ok, so after a lot of fighting with the CI envs the status of this PR is that Windows tests are failing on node v5 (see https://ci.appveyor.com/project/lukasmartinelli/editor/build/1.0.191). Although I believe it can be fixed I don't have time to look into it and I don't have a windows box to test locally. I suggest we just remove support for node v5 as its not in the supported list, see https://github.com/nodejs/LTS

Ok this PR is finally ready to review/merge, tests passing with node v5 removed on windows (see comment above)

Ok this PR is finally ready to review/merge, tests passing with node v5 removed on windows (see comment above)

Looked over it and it looks good to me! Tests are green - perfect world.
This will prevent changes from breaking the entire application which is good.

More tests to come soon. What I want next is a tiny vector tiles source which I can include in the tests. Then I can run the app and set styles in the UI via webdriver, this way we could even do some visual regression testing if we wanted.
Basically for the webdriver testing I'll prefix all the 'actions' (so buttons) with data-action="identifier" attribute and all the inputs with data-input="identifier", these shouldn't change over time as the UI changes so they should be pretty stable in the tests and not require much refactoring over time.

More tests to come soon. What I want next is a tiny vector tiles source which I can include in the tests. Then I can run the app and set styles in the UI via webdriver, this way we could even do some visual regression testing if we wanted.

Easier to use a GeoJSON source is my guess.

Easier to use a GeoJSON source is my guess.

Yeah that sounds a lot more sensible... :)


Plotly Component Upgrade

Convert plotly component to just take in a figure and process to what is needed on frontend

Resolved


Google search results does not display proper page title

The results displayed by a Google Custom Search engine does not display the title properly or as expected.
I would expect search result entry title to be the same as the page title.
See example below:

First entry has the page title: Teams Shortcuts.
Second entry seems displayed properly.
How is it controlled, defined? (I had a look to seo.html)

It might be due to low Google indexation. I don't know here how fast Google reindex the site.


Resolve four broken refs in the docs

Hello!
Pull Request overview

Resolve 4 broken references in the docs

The problems
Two broken references here:

One broken reference here:

One missing reference here:

Tom Aarsen

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.

See the resolved changes here, here and here, respectively


Route planner - button to start planning from scratch

Based on feedback from @pelderson (thanks!):
It would be handy to have a button to wipe out all previous planning selections that have been made and start planning a new route from scratch.

Reset action has been added in version 3.0.0-alpha-21.
The action is integrated in the undo/redo mechanism so that nothing is lost when the button is clicked unintentionally.

Clever!


Feature/donation report queries

Related to: https://github.com/Giveth/GIVeconomy/issues/729
Queries for a report dashboard.
export const fetchNewProjectsPerDate = `
  query (
    $fromDate: String
    $toDate: String
  ) {
    projectsPerDate(
      fromDate: $fromDate
      toDate: $toDate
    )
  }
`;

export const fetchTotalDonationsPerCategoryPerDate = `
  query {
    totalDonationsPerCategory {
      id
      title
      slug
      totalUsd
    }
  }
`;

export const fetchTotalDonors = `
  query (
    $fromDate: String
    $toDate: String
  ) {
    totalDonorsCountPerDate(
      fromDate: $fromDate
      toDate: $toDate
    )
  }
`;

export const fetchTotalDonationsUsdAmount = `
  query (
    $fromDate: String
    $toDate: String
  ) {
    donationsTotalUsdPerDate(
      fromDate: $fromDate
      toDate: $toDate
    )
  }
`;

Updated comments for @RamRamez


UVI Request

--- UVI JSON ---
{
  "vendor_name": "Linux",
  "product_name": "Kernel",
  "product_version": "versions from  to before v5.12.11",
  "vulnerability_type": "unspecified",
  "affected_component": "unspecified",
  "attack_vector": "unspecified",
  "impact": "unspecified",
  "credit": "",
  "references": [
    "https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git/commit/?id=ed0e2a893092c7fcb4ff7ba74e5efce53a6f5940"
  ],
  "extended_references": [
    {
      "type": "commit",
      "value": "ed0e2a893092c7fcb4ff7ba74e5efce53a6f5940",
      "note": "fixed"
    }
  ],
  "reporter": "joshbressers",
  "reporter_id": 1692786,
  "notes": "",
  "description": "kvm: avoid speculation-based attacks from out-of-range memslot accesses\n\nThis is an automated ID intended to aid in discovery of potential security vulnerabilities. The actual impact and attack plausibility have not yet been proven.\nThis ID is fixed in Linux Kernel version v5.12.11 by commit ed0e2a893092c7fcb4ff7ba74e5efce53a6f5940. For more details please see the references link."
}
--- UVI JSON ---

This issue has been assigned UVI-2021-1001024


Coral Datetime may reference non-existent moment locale

Expected Behavior
Correctly set locale to US English, if browser locale is set to this locale
Actual Behavior
Fails to set locale correctly, as moment version used does not contain a moment for the code used in many browsers (e.g. - Chrome uses en-us on a Macbook Pro, whereas moment refers to this locale as simply en)
Runtime error encountered: Error: Cannot find module './en-US'
Reproduce Scenario (including but not limited to)
Run your unit tests in debug mode with the option to "Pause on caught exceptions" selected? That's how I hit this situation when testing our project locally (which, includes latest Coral; 4.10.0)
Steps to Reproduce
(see above)
Browser name/version/os (ie Chrome Version 62.0.3202.94 (Official Build) (64-bit) MacOS)
Browser: Chrome Version 85.0.4183.83 (Official Build) (64-bit)
OS: (Mac) Version 10.15.6 (19G73)
Coral Spectrum version
This code (https://github.com/adobe/coral-spectrum/blob/master/coral-datetime/index.js) hasn't changed in 16 months and I'm not sure that moment ever supported an en-us locale, as their rationale for not including seems to amount to "that's our language, so we just call it English" (kind of disappointing)
Sample Code that illustrates the problem (use the Playground if possible)
n/a
Screenshots (if applicable)

Ideally, Moment would address on their side, as their whole purpose amounts to making it easier to work with date/time Objects across locales. However, since this has been brought up to them before (https://github.com/moment/moment/issues/3569), I doubt they're going to address.
Instead, we could handle this logic in Coral Spectrum by changing from:
window.moment.locale(document.documentElement.lang || window.navigator.language || 'en');

...to:
let locale = document.documentElement.lang || window.navigator.language || 'en';
if (locale.toUpperCase() === 'EN-US') {
  locale = 'en';
}
window.moment.locale(locale);

Also, this may not actually cause runtime problems (i.e. - fallback to default/en is likely, with some exceptions), but when stepping through to debug code, the exception will be thrown every time and you'll have to step over repeatedly.

@zbrownell Sounds good to me. Feel free to open the PR. I also would have expected Moment to handle this use case ...