FancyZones does not work with Webull

Microsoft PowerToys version
0.35.0
Running as admin
Area(s) with issue?
FancyZones
Steps to reproduce

Install Webull Desktop client
Open application
Drag Window, holding primary mouse button, Use non-primary mouse button to toggle zone activation (or Shift key)
No zone is activated.

✔ Expected Behavior
App Window fall into zones
❌ Actual Behavior
App Window doesn't fall into zone
Other Software
Webull Desktop V4.9

As of today, I tried again and it appears to work.
It does not work after a fresh update of Webull completed and a window automatically opened.
If you close and reopen after the completed update, all is working as expected.


zend-json: "^2.6.1 || ^3.0"

As I noted in #178 this patch allows zend-json: "^2.6.1 || ^3.0".

Hm... I am not sure what is the point of that PR, as json is the part of PHP core:
http://php.net/manual/en/json.requirements.php

Hm... I mean we should just use ext-json, heh, this is what I wanted to say :)

@webimpress
See https://github.com/zendframework/zend-view/issues/163

Thanks, @vaclavvanik!


OSError: lib/libRrefblas.so: undefined symbol: xerbla_

Thanks for sharing your code on how to use R on AWS.
Unfortunately if i follow your code with the current Lambda-AMI i get the following error when testing it on a new AWS instance:
[ec2-user@***]$ python ./test_handler.py
Traceback (most recent call last):
  File "./test_handler.py", line 17, in <module>
    import handler
  File "/home/ec2-user/handler.py", line 29, in <module>
    ctypes.cdll.LoadLibrary(os.path.join('lib', file))
  File "/usr/lib64/python2.7/ctypes/__init__.py", line 438, in LoadLibrary
    return self._dlltype(name)
  File "/usr/lib64/python2.7/ctypes/__init__.py", line 360, in __init__
    self._handle = _dlopen(self._name, mode)
OSError: lib/libRrefblas.so: undefined symbol: xerbla_

I tried googling for it but unfortunately do not understand how to fix this...
Any idea?
While running the script it throws the following error:
cp: cannot stat '/usr/lib64/libblas.so.3': No such file or directory
cp: cannot stat '/usr/lib64/liblapack.so.3': No such file or directory

I tried installing these libraries using
sudo yum -y install blas
sudo yum -y install lapack

And then ran the script again... still same error
The script installs the current R version 3.3.2 - dont know if this helps
Might that have something to do with openBlas VS blas?
https://bodhi.fedoraproject.org/updates/rkward-0.6.5-3.fc24 rpy-2.8.3-1.fc24 R-3.3.2-2.fc24/edit

@rentrop I don't think that's 100% the culprit, I tried following their instructions

If you wish to use that, just delete the openBLAS symlink, then rename libRrefblas.so to libRblas.so.

I still got the same error. Unfortunately the AMI referenced in the instructions is from 2015, I'm trying to figure out a solution w/ Docker but the oldest supported images on docker hub are late 2016 which leads me to believe they won't be compatible with Lambda regardless.

@rentrop Follow-up, I was able to get it working using the latest amazon linux AMI. It looks like the libRrefblas.so packaged with R is actually the one that's suspect. If you don't run your yum installs for blas & lapack, you can do something like the following for the native dependencies:
# start assembling binaries
rm /usr/lib64/R/lib/libRrefblas.so
cp -r /usr/lib64/R/* lambda/
cp /usr/lib64/R/lib/libR.so lambda/lib/libR.so

# all the binaries we need
cp /usr/lib64/libgomp.so.1 lambda/lib/
cp /usr/lib64/libgfortran.so.3 lambda/lib/
cp /usr/lib64/libquadmath.so.0 lambda/lib/
cp /usr/lib64/libblas.so.3 lambda/lib/
cp /usr/lib64/liblapack.so.3 lambda/lib/
cp /usr/lib64/libtre.so.5 lambda/lib/

This will cause R to reference its included libRblas.so which makes everything functional again 😄

So you found a solution... Great

Afternoon,
When running the build .sh file the rm (remove file) says 'permission denied'.
when testing the line on it own it addressed that by using the sudo command. Should i add 'sudo' to the build file or is there another way to solve this?
Thanks
Jiminy

@jiminygreen A sudo is fine or you can run sudo chown username /usr/lib64/R/lib/libRrefblas.so prior to it. By default it's probably owned by root.


Improve addQuestionMarks, fix #2184

Okay, this is a long time coming. Getting the inferred types to work as expected in generic contexts is quite annoying (see generics.test.ts for examples). Some changes I made to that effect seem to interact unhappily with the very specific case described in #2184 - a ZodObject schema, with a key corresponding to a union, being cast to ZodType.
The change now makes all my tests happy, including assignability in generic contexts and the newly added XOR test.
I'm going to ship this in the next minor since I haven't found a scenario where this breaks or even changes the inferred type signature in non-generic contexts. In generic contexts, it seems to strictly improve assignability characteristics without a big performance hit in the compiler.
cc @chrishoermann

This has landed in Zod 3.23.
https://github.com/colinhacks/zod/releases/tag/v3.23.0


PHP 7.1 compatibility tweaks

Replace the soft string to int casting in ini2bytes with a more explicit equivalent, update tests
Consolidate some temporary file creation and deletion logic in UploadTest
Allow PHP 7.x to use random_bytes over mcrypt since it is deprecated in 7.1 (relates to #6397)
Update Travis configuration to build 7.1 as the 7.x default, while still running 7.0

thanks :)


Release version 0.40.0

Follow on to https://github.com/sqlparser-rs/sqlparser-rs/issues/974
Filing a ticket here to track the work of a release (mostly so others can follow along)
Target Release Date: Approximately 2023-11-27
Release instructions: https://github.com/sqlparser-rs/sqlparser-rs/blob/main/docs/releasing.md
Steps:

[x] Release a version 0.2.0 of derive crate (due to https://github.com/sqlparser-rs/sqlparser-rs/pull/1044): https://crates.io/crates/sqlparser_derive/0.2.0
[x] Make a PR to update CHANGELOG.md : https://github.com/sqlparser-rs/sqlparser-rs/pull/1053 https://github.com/sqlparser-rs/sqlparser-rs/pull/1055
[x] run cargo release
[x] verify the new release is on crates.io: https://crates.io/crates/sqlparser/0.40.0
[x] Make ticket for next release: https://github.com/sqlparser-rs/sqlparser-rs/issues/1054

The new release is available on crates.io: https://crates.io/crates/sqlparser/0.40.0


Deafults return wrong coordinates with OSRM results

Hitting the OSRM endpoint:
https://router.project-osrm.org/viaroute?z=15&output=json&loc=47.603832,-122.330062&loc=47.097946,-122.424342&instructions=true,

and then using polyline.decode(route_geometry);  returns and array of coords that look like this:
0: 472.10022
1: -1224.33977

Being in Seattle, I recognized that these are 1 decimal off.  Setting the precision to 6 like this:
polyline.decode(route_geometry, [6]);

Does the trick.
0: 47.210022
1: -122.433977

Seems like the default should do that?  Or did I miss something?
Thanks!

The polyline default of precision 5 was chosen to match the precision of Google polylines. But as you noticed, OSRM has chosen to use a higher precision. So yes, when using polyline with OSRM, you have to specify 6.


[8.x] Make assertSee, assertSeeText, assertDontSee and assertDontSeeText accept an array

This PR allows assertSee, assertSeeText, assertDontSee and assertDontSeeText to work with arrays. Unlike the -InOrder methods, these won't check the order.
Before:
$this->get('/')
    ->assertSee('some')
    ->assertSee('piece')
    ->assertSee('of')
    ->assertSee('text');

After:
$this->get('/')
    ->assertSee(['some', $foo, 'text', 'piece', 'of']);

Benefits
It's mostly a code style change. In projects where the frontend can change unexpectedly, this will avoid errors because of the order while keeping things compact
Notes

I considered doing more type checking and throw a TypeError. The idea was to provide a bit more feedback than htmlspecialchars() expects parameter 1 to be string, _______ given but I'm not too sure about it.
I'm running assertStringContainsString inside a foreach because I don't know if there's a better PHPUnit assertion for this case. This makes it so OK (1 test, 3 assertions) shows several assertions (one for each item in the array) which never bothered me. I'm guessing copying the strategy used in Illuminate\Testing\Constraints\SeeInOrder could deal with this?
First PR here. Open to any correction or suggestion.

You can make your code simpler by just always using Arr::wrap($value) on the passed argument so that you can always loop over it and not have to check is_array.

Ahh, I thought about that but didn't know about Arr::wrap so I discarded the idea thinking I had to do the check anyway.

Earlier I saw I was able to add commits to the PR. I'm guessing now that it's merged I'd have to open a new one, right? (It's for the Arr::wrap cleanup)

@johnRivs yeah a new PR will be needed if this one is already merged.

Turns out Taylor already cleaned it up haha Thanks, both of ya!


add github ci

Codecov Report

:exclamation: No coverage uploaded for pull request base (main@2cc5ba8). Click here to learn what that means.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main       #5   +/-   ##
=======================================
  Coverage        ?   16.81%
=======================================
  Files           ?        6
  Lines           ?      232
  Branches        ?        0
=======================================
  Hits            ?       39
  Misses          ?      185
  Partials        ?        8

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Can you explain a bit about the extrinsic camera calibration that you are doing in the cpp file on the pcd data and why?

Hi @MeghaMaheshwari. Thank you for your comment.
In cpp directory, we developed label_filter. This module calculate alpha parameter in labeling result and remake pcd file by each 3d bounding-box.
the format of our results is KITTI 2D-3D format(https://github.com/NVIDIA/DIGITS/issues/992). In the format, the 3d bounding-box location is represented by camera coordinates. In label_filter, we convert 3d bounding-box location in camera coordinates to pcd coordinates to remake pcd file.


docs: Fix typo Argila -> Argilla

Description
I've noticed an env variable in the docs with the following typo: ARGILA instead of ARGILLA, so I fix it.
Type of change
(Remember to title the PR according to the type of change)

[x] Documentation update

How Has This Been Tested
(Please describe the tests that you ran to verify your changes.)

[ ] sphinx-autobuild (read Developer Documentation for more details)

Checklist

[x] I added relevant documentation
[x] I followed the style guidelines of this project
[x] I did a self-review of my code
[x] I made corresponding changes to the documentation
[x] My changes generate no new warnings
[x] I filled out the contributor form (see text above)
[x] I have added relevant notes to the CHANGELOG.md file (See https://keepachangelog.com/)

@louisguitton thank you very much for the fix!


docs: update installation guide for pnpm

Just best practice to use

pnpm add : for adding new package(s)
pnpm install: for installing all packages (scanning through pnpm-lock)

Thanks mate!


copr: Add vectorize Trim2Args and Trim3Args

Signed-off-by: pingyu<EMAIL_ADDRESS>PCP #5751
What have you changed?
Add vectorize Trim2Args & Trim3Args.
What is the type of the changes?

New feature (a change which adds functionality)

How is the PR tested?

Unit test

Does this PR affect documentation (docs) or should it be mentioned in the release notes?
No.
Does this PR affect tidb-ansible?
No.
Refer to a related PR or issue link (optional)
#5751

/run-all-tests

I suggest picking up challenge from #6959 and #6958 first

PTAL @breeswish


Filter maintenance.timeline to return data from expected date range based on limit

Problem: If we take https://api.napari-hub.org/metrics/zarpaint as an example, we can see that a portion of data returned by the API is outside of the range of 12 months, as limit=12 by default. https://github.com/chanzuckerberg/napari-hub/pull/960 is the PR that addresses the bug by filtering out maintenance data that falls outside of the expected data range.

I'm not sure that maintenance data needs to be consistent with usage data. Could you clarify what the bug is?
Looking at the commented out tests, it looks like the test is expecting 12 maintenance entries for e.g. https://www.napari-hub.org/plugins/napari-assistant, but I think that is an issue with the test and not the functionality. There are 10 entries for it as expected since there was no data the past two months, and it renders correctly. I believe at some point we discussed that @codemonkey800 would fill in any missing months on his end, but might be worth clarifying that point.
cc @manasaV3

If we take https://api.napari-hub.org/metrics/zarpaint as an example, we can see that the first element of maintenance.timeline is {         "commits": 1,         "timestamp":<PHONE_NUMBER>000       }
The problem is that the timestamp is equivalent to Thursday, July 1, 2021 12:00:00 AM in GMT, and therefore falls out of the range of data we should return for the API endpoint. Therefore, the bug is that the API shouldn't return data that is out of the expected date range based on limit, and https://github.com/chanzuckerberg/napari-hub/pull/960 is the PR that addresses the bug.
Using the same case of https://api.napari-hub.org/metrics/zarpaint, the correct implementation would only return 6 valid items, assuming limit=12 as our default value. I do agree with @richaagarwal that in this case it would be fine to just return those 6 items as is without padding the additional 6 items with default values, since @codemonkey800 fills them in already.
However, I should bring up that @manasaV3's implementation for usage metrics, as seen in https://api.napari-hub.org/metrics/hesperos, fills in the default values, so that is the discrepancy between usage metrics (backend fills in default values )and maintenance metrics (frontend fills in default values), which is why in this file the tests failed.

@klai95 I understand how the two implementations are different and agree it's not ideal that we're inconsistent here. If we are going to address this, it's best to use a consistent definition of limit.
That said, a limit generally implies we return a list of items up to that limit. It's not obvious based on the parameter limit that it implies any kind of date range to validate against. If we want to only return items within a certain date range, we can do that, but let's not use limit to guide that.

I think there may be some confusion, as I believe we are saying the same thing just in a different way. To clarify, if limit=12, in the case of https://api.napari-hub.org/metrics/zarpaint, what I mean by data range is capturing only the data from March 2022 to February 2023, and not using limit as a parameter to specify some window of data range.
Back to the issue itself in https://api.napari-hub.org/metrics/zarpaint, the bug is that currently in maintenace.timeline, there are data from 2021, which shouldn't be the case at all, and this issue is addressing that problem.

@klai95 I was saying something slightly different, but I don't have any issue with your proposed changes.
What I was trying to say is that we return the past 12 (or less) values right now in your current implementation, which for that particular plugin means going back into 2021.  I don't think the variable name limit or anything else in the function is explicitly suggesting that we would take the limit value and convert that to the past limit months, but I understand that initially that was the intention behind it, and that is how it is implemented for usage.   Since the frontend is doing the bulk of the validation here, I don't see this as a bug, but I do agree the inconsistency is confusing.
If this ticket is more about being consistent with the usage implementation (which I agree is a good idea), I would ask that the function handles missing data similarly as well, so perhaps you can sync with Jeremy and Manasa on that and handle any necessary changes regarding that in https://github.com/chanzuckerberg/napari-hub/pull/960. (That may mean removing the handling of missing data for usage or adding it for maintenance or some other solution).
It might also be good to consider a clearer variable name and/or add some docstrings on what exactly the limit is intended to do, since on its own it does leave room for ambiguity.


How to install needed VM etc.

How to download install GraalVM and in which version to build und run the project?

@orat Please review, if ba820d6  works for you.


ToolTipFix

Mod name
ToolTipFix
Curseforge link
https://www.curseforge.com/minecraft/mc-mods/tooltipfix
Other links
https://github.com/kyrptonaught/tooltipfix
What it does

Stops tooltips that are too large to fit on the screen from running off the screen.

Why should it be in the modpack
It's a simple mod that fixes a simple problem.
Why shouldn't it be in the modpack
None.
Categories
Graphics optimization
Additional details
No response

The idea is cool, but if the modpack shows the text that vanilla wouldn't, people can misleadingly name their items in a way that would be barely visible in vanilla. Although I admit, it could be useful for languages with long words.


After adding .popupBarStyle(.prominent) or .popupBarProgressViewStyle(.top) or .popupBarMarqueeScrollEnabled(true) to the View hierarchy, iOS keyboard will resign after single keypress

Describe the Bug
using any or all of these modifiers will interfere with the first responder:
        .popupBarStyle(.prominent)
        .popupBarProgressViewStyle(.top)
        .popupBarMarqueeScrollEnabled(true)

NOTE
.popup() alone does NOT interfere with first responder.
when entering text in another TabView's TextField, the keyboard will resign after every key press.
To Reproduce
Steps to reproduce the behavior:

add any or all of:
.popupBarStyle(.prominent)
.popupBarProgressViewStyle(.top)
.popupBarMarqueeScrollEnabled(true)
enter text in a TextField
Observe the keyboard resigning after first keypress

Expected Behavior
Keyboard/first responder does not resign
Screenshots
Made a video.
Additional Context
none
https://user-images.githubusercontent.com/808934/186574265-42dc9c82-eba6-4144-82f7-99b05ffc99ac.MP4

Hello,
Thank you for reporting this issue. I am preoccupied with real-world issues and not sure when I will have time to investigate this. If you can, please assist with debugging, to see when the the first responder resigns. You could add a symbolic breakpoint for resignFirstResponder and see why/where it gets called. Thanks

I cannot reproduce this issue. I’ve added a text input in the popup container view, as well as in the popup content view, and I can type in both just fine.
https://user-images.githubusercontent.com/2270433/195081023-8db36a1c-a6d3-4409-be63-98bb897298fc.mp4
Since there has been no response to this issue, I am closing it. If you are able to reproduce in a demo project, please attach it here.


all plots are empty

I can not get any plots to work.
I believe it has to do with the id mangling, which eventually with the latest version of svglite is broken.
This fork works for me:
https://github.com/tentamen/gg4clj
which basically removes id mangling

The problem is just that the regex in the mangling needs a slight change. It seems the svglite now produces ids set off with ' rather than ".
The comment in mangle-ids that svglite reuses the ids in each plot still holds, so multi-plot documents might be affected.
Here is the function with the regex changes:
(defn- mangle-ids
"ggplot produces SVGs with elements that have id attributes. These ids are unique within each plot, but are
generated in such a way that they clash when there's more than one plot in a document. This function takes
an SVG string and replaces the ids with globally unique ids. It returns a string.
This is a workaround which could be removed if there was a way to generate better SVG in R.


Scheduling without labels?

Hi!
Is there a way to schedule emails without using labels? Me and my team would like to schedule emails somehow, but this labeling system isn't flexible enough for us. It would be better if we would be able to set a specific date for each draft (like other scripts do), but of course that would need a different interface for it.
Are there any plans to support this feature?

@Semmu - The gmail limited interface only supports predefined labels. Will keep this open and invite ideas from others on a work around.

Thanks for the fast reply!
Other similar solutions use a dedicated Google Spreadsheet for the UI. They import the drafts from Gmail, you specify a date for the scheduling, then schedule the sending command itself. Gmail's UI is indeed limited, but this would complement it. Downside would be the need to use an app outside Gmail's window.


[FIX] fix breaking dismissOnClick

change event.path to event.composedPath
bc event.path is non-standard

Can we get any update if this PR is going to be integrated?

@Robertof76 how to i integrate this in my existing code? I get an error
Uncaught SyntaxError: Unexpected end of input

We actually monkey patch this library.
We did so in extending the prototyp with the "composedPath" functionality if not available natively.

I'm closing this because this fix is already in another PR that already made it into dev. It will be in the next release of v3.X.Y


Clean up learning links on main F# docs page

We've been reviewing the various paths into F# learning

We'd like to highlight https://dotnet.microsoft.com/learn/fsharp as the curated learning page for F#. It needs work but Beth Massi is improving it and it is the primary learning link we should use

We need one "get started immediately" link.  Currently the best we have is https://dotnet.microsoft.com/learn/languages/fsharp-hello-world-tutorial/intro and indeed this is what's linked from the main F# landing page https://dotnet.microsoft.com/languages/fsharp.  So  let's use this here.

The Q&A forums are not yet appropriate to link due to low volume. We can revist this later. But suggest removing that for now

cc @cartermp

@gewarren I've tweaked it, could you check it again please?
The just-published beginner video series is included now
The "functional programming" tutorial is now under "Tutorials" . This makes the Overview section thin (Tour of F#), but that's ok, it indicates where new material can slot in.


pytest import error

Hello @BenjaminGabet ,
since you seem to be into pytest, you maybe could help me with the follow error.
I try to run pytest (even with the -m "not GUI" parameter) and get an import error related to PySide2.
I have installed PySide2 and I also can import it, but nevertheless I get the error.
___________________________________________ ERROR collecting Tests/GUI/DMatLib/test_DMatSetup.py ___________________________________________
ImportError while importing test module 'D:\GitHub\PyleecanLokal\master_clean\pyleecan\Tests\GUI\DMatLib\test_DMatSetup.py'.
Hint: make sure your test modules/packages have valid Python names.
Traceback:
c:\program files\python38\lib\importlib\__init__.py:127: in import_module
    return _bootstrap._gcd_import(name[level:], package, level)
Tests\GUI\DMatLib\test_DMatSetup.py:9: in <module>
    from PySide2 import QtWidgets
C:\Program Files\Python38\Lib\site-packages\shiboken2\files.dir\shibokensupport\__feature__.py:146: in _import
    return original_import(name, *args, **kwargs)
E   ImportError: could not import module 'PySide2.QtGui'

Hi @SebGue
Do you use python  or python3  to launch your file ? And also, did you do pip install PySide2 or pip3 install PySide2 ? Because there can be an error if you install PySide2 with pip install and running it with python3 . Can you show me your imports please ?

Hello Benjamin,
I'm not quiet sure what you mean with

Can you show me your imports please ?

I think I'm using 'python' all the time. Also I installed PySide with python, i.e. python -m pip install PySide2
(If I try to run python3 in terminal nothing happens except Microsoft store opens with Python3.8 site :-) )
If I run 'python' and e.g. import PySide2.QtGui there is no error.
BR,Sebastian

Hi Seb,
Have you modified the file ? Try to import like that : "from PySide2.QtWidgets import *". I don't have the error on my side. Have you uninstalled PyQt5 ?

Due to your suggestion, I uninstalled PyQt5 and now it works, although I don't understand why. Thank you for that.
I guess, now I have no excuse to not extensively unit test my code anymore :-(

PyQt5 and PySide2 are both using Qt-libs. In pyleecan we are now using PySide2 instead of PyQt5. I think some errors occured when both are installed :).
Unit testing is the best way to make good code :)


Update standalone template documentation

@jputrino
Issues: Fixes #41
Problem: Documentation for standalone templates need better explanation
on how to launch a stack
Analysis: Modified it to show users an example of launching via the heat
client
Tests:

Jodie, need a little expertise on how this should look.

@pjbreaux - I'm going to merge this just to get the info into the docs, since it's in high demand! I plan to go through and test it all out, so I can edit or reformat as I go along. I think it might be a good idea to take one of the templates and craft a user guide around it. Any you'd recommend?


hello sir , i tried to use this plugin , but its not opening the dialog after 3 days...

hello sir , i tried to use this plugin , but its not opening the dialog after 3 days or the times  I have specified the launches .. it works in production when the   if (_rateMyApp.shouldOpenDialog) {  line is uncomment , but its not working when i uncomment it .

class _DashboardScreenState extends State {
RateMyApp rateMyApp = RateMyApp(
preferencesPrefix: 'rateMyApp',
minDays: 3,
minLaunches: 7,
remindDays: 2,
remindLaunches: 5,
);
@override
void initState() {
super.initState();
rateMyApp.init().then(() {
if (_rateMyApp.shouldOpenDialog) {
_rateMyApp.showStarRateDialog(
context,
title: 'Enjoying Our App?',
message: 'Please leave a rating!',
onRatingChanged: (stars) {
return [
RaisedButton(
padding: EdgeInsets.symmetric(horizontal: 22.0, vertical: 6.0),
color: Colors.red,
child: Text('Rate'),
textColor: Colors.white,
highlightElevation: 2,
shape: RoundedRectangleBorder(
borderRadius: BorderRadius.circular(10),
),
onPressed: () {
if (stars != null) {
rateMyApp
.callEvent(RateMyAppEventType.rateButtonPressed)
.then(() => Navigator.pop(context));
if (stars <= 3) {
Navigator.of(context).pop();
} else if (stars <= 5) {
_rateURL();
}
} else {
Navigator.pop(context);
}
},
),
];
},
dialogStyle: DialogStyle(
titleAlign: TextAlign.center,
messageAlign: TextAlign.center,
messagePadding: EdgeInsets.only(bottom: 20.0),
),
starRatingOptions: StarRatingOptions(),
);
}
});

RateMyApp rateMyApp = RateMyApp(
preferencesPrefix: 'rateMyApp_',
//minDays: 7,
minLaunches: 1,
//remindDays: 7,
//remindLaunches: 10,
);

First, you should not do UI operations in initState. Please wrap the code inside something like this :
SchedulerBinding.addPostFrameCallback((_) {
  // Code here.
});

Second, do you have any error ? And what is your platform ?
Oh and I see there are more conditions than just waiting 3 days. You must have launched your app at least 7 times.

is there any way i can contact you . i dont have any issue and there is no bug in your plugin .
honestly i was asking how to implement it . thanks .sorry for botheration .

No problem, feel free to ask your question here as it may help someone else :wink:

i am not able to apply this plugin sir ,
i followed tutorial of marcus as well on utube , but it works in production . not in release. i had uncomment the line as well , but it doesn't opens. Sir, I need proper guidance on how to apply it . thanks

I see there are more conditions than just waiting 3 days. You must have launched your app at least 7 times.


Ignore issue if it is a Pull request

According to the Github API docs the Issues endpoint also returns Pull requests: https://developer.github.com/v3/issues/?q=is%3Aissue+is%3Aopen+sort%3Aupdated-desc#list-issues
This PR adds an extra filter that removes PR's from the list of Issues. More info can be found in the issue I made #72
Fixes #72

Not sure if a test for this is needed. Let me know if I should add a test and how best add it (new test or adjust existing test).

@abhijeetps Can this PR get reviewed please?

@Glennmen , thank you.
The PR looks good. I will test it in Weekly Digest Beta by the end of this month and I will let you know.
I have been a little busy lately so, I couldn't come up to check the PR.  😄
I'm really sorry for the delay.

@abhijeetps No problem! Glad to see that there is stil some activity in this useful project.
Let me know if you want me to change anything.

@abhijeetps Have you been able to test this PR yet?


clean_names returning a copy of a dataframe

Just stumbled upon this stackoverflow question:
https://stackoverflow.com/questions/53427376/preserve-original-column-names
It appears the issue is that the clean_names function returns a copy of a dataframe and doesn't preserve other attributes.
Might be something to look into for this and other functions as well?

Thanks for bringing this up, @szuckerman!
There's definitely inconsistent behaviour w.r.t. copying a dataframe or not inside each function. This is something I think should be fixed at the SciPy sprints this year. I probably will also lead one at PyCon.
This issue basically boils down to a decision: copy by default, or not? We talked a bit about int #79, but with new information that in-place mutation is going away, I think pandas is going to become an API for small/medium data users.
I'm 100% okay deferring this decision beyond SciPy 2019. We don't have to act on this right now, as it's not producing frequent confusion for package users.
At the same time, I super like @zbarry's proposed implementation in #79, which I'm sure someone at PyCon/SciPy would be happy to run with.

clean_names does not mutate the dataframe, since the columns/index are immutable. A slice of the dataframe (shallow copy) is all that's needed, without affecting the original dataframe.


Context propagation to go GRPC server stopped working after grpc 1.14.1

I have a go GRPC server using the latest version of the go opencensus GRPC server handler (example here: https://github.com/census-instrumentation/opencensus-go/blob/master/plugin/ocgrpc/example_test.go). I have a NodeJS client that includes the trace agent, version 3.5.0.
The client correctly patches a grpc-node 1.14.1 request, creating a child span that is extracted from the metadata on the server and linked with the same traceid by the go server. client and server appear as connected spans in Stackdriver Trace.
However, when using grpc-node version 1.14.2, and all subsequently published version of grpc-node, using the same trace agent 3.5.0, the requests are still patched, and a client child span is still created, but the spans created on the server have a different traceid, indicating that it has not propagated successfully. (Both client and server spans appear in Stackdriver Trace, but are disconnected.)
I went and looked at the changelog between grpc-node 1.14.1 and 1.14.2, and it seems pretty innocuous, but I've confirmed this three times, switching between various different versions of the grpc client library. I have confirmed that with a 1.14.1 client request, the parentID context is successfully extracted, but in the 1.14.2 client request, it isn't. I've confirmed that it doesn't work all the way up to grpc-node 1.17.0.
I'm in control of both the client and the server, so I can give you whatever logs are necessary from the GRPC metadata or headers on either end to help narrow it down.
The client request is promisified with Bluebird in both cases and looks basically like:
const client = new service.UsersClient('go-service', grpc.credentials.createInsecure())
const getUser = promisify<messages.UserResponse, messages.GetUserRequest>(
  client.getUser.bind(client)
)
await getUser(request).timeout(CREDENTIALS_TIMEOUT)

GCLOUD_TRACE_NEW_CONTENT=1
(I have other services that Promisify their requests with Bluebird and they are not connected either, even using grpc-node 1.14.1 😐)

@majelbstoat Can you look into the headers you are actually receiving. From our side it seems that things should be working okay.

I'm going to close this due to lack of info. Please add a comment when you have the requested info and we will reopen this.


Implemented ron::value::RawValue

Adds the ron::value::RawValue type, inspired by serde-rs, which holds a valid RON fragment string and can be (de)serialised inside any container.

[x] I've included my change in CHANGELOG.md

Codecov Report

Merging #407 (8f77817) into master (a63b3c8) will increase coverage by 0.31%.
The diff coverage is 59.50%.

@@            Coverage Diff             @@
##           master     #407      +/-   ##
==========================================
+ Coverage   78.74%   79.05%   +0.31%
==========================================
  Files          51       54       +3
  Lines        6737     7000     +263
==========================================
+ Hits         5305     5534     +229
- Misses       1432     1466      +34

Impacted Files
Coverage Δ

src/error.rs
36.01% <0.00%> (-0.29%)
:arrow_down:

src/value/mod.rs
67.47% <ø> (ø)

src/ser/raw.rs
5.21% <5.21%> (ø)

src/value/raw.rs
69.49% <69.49%> (ø)

src/de/mod.rs
76.02% <100.00%> (+2.10%)
:arrow_up:

src/parse.rs
79.53% <100.00%> (ø)

src/ser/mod.rs
74.63% <100.00%> (-3.64%)
:arrow_down:

tests/407_raw_value.rs
100.00% <100.00%> (ø)

... and 3 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

?r @torkleyy I stumbled across this API some time ago and thought it would be quite useful to also have it inside ron. While I added some convenience functions to RawValue for the common cases, I decided against implementing Serializer and Deserializer directly as those would only duplicate functionality which is already available by just using the existing Deserializer with the &str one can get from RawValue.


ACCUMULO-4854 Fix DfsLogger exceptions

Removed translation of ClosedChannelException to a LogClosedException in the write method as that is done by callers of the method.

Merged:
commit 4b3273bd61e3fbc792069a07fdae37e59151f1a2
Author: Ivan Bella<EMAIL_ADDRESS>Date:   Sat Aug 25 21:07:44 2018 -0400
ACCUMULO-4854 : DfsLogger exceptions (#616)
   Removed translation of ClosedChannelException to a LogClosedException in the write method as that is done by callers of the method.


publish<EMAIL_ADDRESS>

Requested by: @philipphofmann
Merge target: (default)
Quick links:

View changes
View check runs

Assign the accepted label to this issue to approve the release.
Leave a comment containing #retract under this issue to retract the release (original issuer only).
Targets

[ ] gem
[ ] registry
[ ] github

Targets marked with a checkbox have already been executed.  Administrators can manually tick a checkbox to force craft to skip it.

I have to revert the PR for upgrading the sentry-cli, which was the only change in this release https://github.com/getsentry/sentry-fastlane-plugin/pull/254. So I'm going to close this.

#retract


brew fetch of a mercurial repository fails

[x] Ran brew update and retried your prior step?
[x] Ran brew doctor, fixed as many issues as possible and retried your prior step?

Bug reports:
Fetching a mercurial repository fails if mercurial (hg) is not yet installed.
$ brew fetch homebrew/python/pygame
==> Fetching pygame from homebrew/python
==> Cloning https://bitbucket.org/pygame/pygame
Error: Failed to download resource "pygame"
Failure while executing:  clone https://bitbucket.org/pygame/pygame /Users/*/Library/Caches/Homebrew/pygame--hg

This is the case on the build bot and the reason the CI fails for https://github.com/Homebrew/homebrew-games/pull/722.
https://bot.brew.sh/job/Homebrew Games Pull Requests/1106/version=el_capitan/console
Installing the formula directly works, as it depends on mercurial which is then automatically installed before installing pygame.
Any advice on this?
Possible steps:

Install mercurial on the CI servers?
Automatically install mercurial before/while fetching?
Ignore?

The pygame url is currently:
  url "https://bitbucket.org/pygame/pygame",
      :revision => "faa5879a7e6bfe10e4e5c79d04a3d2fb65d74a62",
      :using => :hg

I don't know if that is supported, but you could try the following instead:
  url "https://bitbucket.org/pygame/pygame/get/faa5879a7e6bfe10e4e5c79d04a3d2fb65d74a62.tar.gz"
  sha256 "..."

I'd in favour of just working around this on CI. Ideally we'd auto-install hg in this case.


DiagnosticReportCompositionR5 cardinality

In DiagnosticReport: Laboratory Report, DiagnosticReportCompositionR5 is indicated as required in Card, but as optional in Description & Constraints. Which one is corrects?

@rgazzarata : "Optional Extensions Element" is the default short for extensions.. but I agree we need to add a proper short

@rgazzarata are you ok with the proposed text ?

Perfect!


Linker problems on some Linux platforms

Ubuntu (20.04) and Linux Mint have linker troubles for some reason. I'm not sure why, and I don't know what to do to troubleshoot this further at this point.

/facepalm. Seems that link order on the command line matters on some Linux distributions.
https://stackoverflow.com/questions/8382153/linker-tells-me-it-cant-resolve-symbols-but-theyre-there
This seems to be fixed with f33162656be3a2f4e9c0b0573e3b1095a5b95213


Enable default R location to be used on mac/linux if none is supplied

What problem did you solve?
#339
(If you do not have screenshot) How can I check this pull request?

Ensure R is on your search path
open vscode with default settings and run "create new R terminal"
a new R session should open
change vscode default settings for R location on mac/linux to a null/junk location
run "create new R terminal"
A error message should appear saying unable to open the terminal

Please find a gif of the above below:

Hey @andycraig  & @renkun-ken, Updated the PR to look for the path variable across all 3 OS's.  Works fine on my Mac but need someone to test it on windows as I don't have access to a windows machine sorry :(

I test this on both macOS and Ubuntu and it works well. Maybe @andycraig could take a look at Windows.

I test this on Windows 10 VM and it works as expected.

@gowerc  Great works! These have a lot of great improvements. I will publish a new version this night.


Create an installer

We need an installer that will get all the dependencies, binaries, certs, etc. in the right place. Will probably use chocolatey to make things easier.

This was taken care of a while ago.


fix(docker): Emulation was not working

Codecov Report

Merging #181 (402a6c3) into main (644abc5) will not change coverage.
The diff coverage is n/a.

:exclamation: Current head 402a6c3 differs from pull request most recent head 370b6c5. Consider uploading reports for the commit 370b6c5 to get more accurate results

@@            Coverage Diff            @@
##               main     #181   +/-   ##
=========================================
  Coverage     78.90%   78.90%
  Complexity      110      110
=========================================
  Files            42       42
  Lines          2067     2067
  Branches        292      292
=========================================
  Hits           1631     1631
  Misses          316      316
  Partials        120      120

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


Reduce memory consumption of Utf8SourceInfo objects

Today we hold on to scriptSource objects inside Utf8SourceInfo even if we make a copy of the script source inside source holder. This is unnecessary and increase the working set. The fix is to not hold reference anymore.
However if the scriptSource is external array buffer and is managing its own life time, we don't need to make copy of it and instead we just hold the reference to scriptSource for scenarios like function.name.toString() to work (which is how it is done today).

@digitalinfinity , @obastemur , @curtisman  - Please help review.

:shipit:


Use Ingress feature

In order to provide a nicer experience this extension should leverage the ingress feature of addons

Create two nginx configs, one for ingress, one for direct
Ensure both work

#54 tried to start it with that but ran into issues with asset loading


Facility location optimod

Why this Mod?
The facility location problem is important in various industries and sectors. It involves determining the optimal locations for facilities such as warehouses, distribution centres, and manufacturing plants. By strategically placing facilities, companies can minimize transportation costs, reduce inventory holding costs, and optimize supply chain operations.
What will the API be?
The input data will be
- facilities: list of facilities
- customers: list of customers
-  opening_cost: fixed opening cost of a facility (dictionary)
- capacity: capacity of each facility (dictionary)
- demand: customer demands (dictionary)
- service_cost: cost to serve each customer from each facility (dictionary)
The API will look something like
open_facilities, service_schedule, objective = solve_facility_loc(facilities, customers, opening_cost, capacity, demand, service_cost)

Sounds great! Would be good to also discuss with @stevedwards: while the network design model in #36 is quite a different problem, it would be good to make sure the terminology used in documentation is consistent between the two if there is overlap there.

I think the network design model is too abstract and having mods be more concrete like this facility location mod could actually be more user-friendly.

Robert Freund (MIT) walked me through an MBA-level class exercise touching facility location.  In the example the locations were potential new landfills, connecting homes of the region to them.  Two twists we could have generalized in this mod are:

Fixed number of facilities to be build vs. variable number with fixed-charge costs for each in the objective
Fairness constraint:  Use auxiliary data on the sites to ensure a fairness constraint.  In the concrete example it was used to prevent concentration of new landfills in low-income regions.

Moreover we should have examples where different metrics give raise to the cost connecting sites and nodes (e.g., l2 norm on larger geographic distances, l1 norm in urban US areas)


Handle CSRF Expiration Errors in a Cleaner Way

Right now, if a user opens up a page (login / registration), waits for a bit (for their session to expire), then submits the form -- they'll receive a CSRF error page.
This is quite ugly, and instead we should override this functionality with a custom flashed message of some sort, and re-render the page the user was on.

Any updates guys?

Gah, still haven't gotten to this yet, sorry >< I'll try to work on this first thing tomorrow morning. I've got some time dedicated to this project then.

+1

Also: we'd like to emit an event when the CSRF token expires so a developer can respond to it.

So, in the latest release of our express stuff, we no longer validate CSRF tokens for login / registration pages as they aren't useful here.
I'm closing this out as this is no longer an issue in the library.

Hey @rdegges, do you know if this still the case in express-stormpath? Looks like a CSRF token is being set on POST's to /login and /register.

Just to give everyone an update, CSRF protection is not offered in express-stormpath at the moment. To eliminate security issues in the future, we're going to work on making sure that the /login and /register endpoints accept only json (so in the event of CSRF attacks, CORS errors will be thrown).


Fix script name, path info, and path translated

See also:
https://github.com/deislabs/wagi-fileserver/pull/7
https://github.com/deislabs/wagi-dotnet/pull/50


There is no any NativeModules of Viro in 2.21.1.

Requirements:
Please go through this checklist before opening a new issue

[x] Review the documentation
[x] Search for existing issues in: viromedia/viro & ViroCommunity/viro
[x] Use the latest ViroReact release

Environment
Please provide the following information about your environment:

Development OS: Mac
Device OS & Version: iOS 14.4.2
Version: ViroReact version and React Native version : ViroReact : 2.21.1, ReactNative : 0.63.4
Device(s): iPhone 11

Description
There is no any NativeModules of Viro.
import {NativeModules} from 'react-native'
console.log(NativeModules.VRTARTrackingTargetsModule) //  => null

2.21.0 is works. But there is no any modules in only 2.21.1 version

Might be related to #52 and #49. The code prints this when ran on Android:
{"createTargets": [Function nonPromiseMethodWrapper], "deleteTarget": [Function nonPromiseMethodWrapper], "getConstants": [Function anonymous]}

It works on android. but on iOS, the code prints null.

Closed in favour of #56


Write functional tests for areas

https://github.com/fabric8io/fabric8-planner/pull/1162 PR

Merged !


Move NetworkOpts, RpcServerOpts to standalone args module

Describe the feature
atm some helper Args are in lib.rs
TODO

move https://github.com/paradigmxyz/reth/blob/9470943babf8afe98a10db729954a374470afcb7/bin/reth/src/lib.rs#L30 to src/args/network_args.rs
move https://github.com/paradigmxyz/reth/blob/9470943babf8afe98a10db729954a374470afcb7/bin/reth/src/lib.rs#L64 to src/args/rpc_server_args.rs
rename types Opts -> Args to follow clap's naming
make all fields pub

Additional context
No response

I would like to take this on.

Hey @leruaa , I started working on this and opened a PR for the same. I should have mentioned here earlier apologises for the same.


Feature/ewl/rename profiler demo exe

Description
Change MAPL_Profiler executable name from demo.x to profiler_demo.x.
Related Issue
See also https://github.com/Goddard-Fortran-Ecosystem/gFTL-shared/issues/22.
Motivation and Context
This update prevents a CMake name conflict for executable demo.x if building gFTL-shared within the source same tree as MAPL, such as in GCHPctm.
How Has This Been Tested?
This update has not been tested in the context of GEOSgcm. I am not sure how demo.x is used outside of MAPL. I am assuming it is a trivial update, and that is reflected in my checked boxes below.
Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[x] Trivial change (affects only documentation or cleanup)

Checklist:

[ ] I have tested this change with a run of GEOSgcm (if non-trivial)
[x] I have added one of the required labels (0 diff, 0 diff trivial, 0 diff structural, non 0-diff)
[x] I have updated the CHANGELOG.md accordingly following the style of Keep a Changelog

@lizziel , I will make the change that Tom mentioned and resubmit the PR

@tclune I made the correction you have suggested

@tclune and @atrayano, thanks! Apologies for ineptness at making this PR. I shouldn't do anything like this after 6 pm.

Lizzie, no need to apologize. I never considered 9-5 to be the limit of making requests.
Also, the reason for my “intervention” was that we want to release v2.2 very soon (with some luck by tomorrow) and we wanted to get in your request
Atanas
From: Lizzie Lundgren<EMAIL_ADDRESS>Reply-To: GEOS-ESM/MAPL<EMAIL_ADDRESS>Date: Wednesday, June 17, 2020 at 4:13 PM
To: GEOS-ESM/MAPL<EMAIL_ADDRESS>Cc: "Trayanov, Atanas L. (GSFC-610.1)[SCIENCE SYSTEMS AND APPLICATIONS INC]"<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: [EXTERNAL] Re: [GEOS-ESM/MAPL] Feature/ewl/rename profiler demo exe (#429)
@tclunehttps://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_tclune&d=DwMCaQ&c=ApwzowJNAKKw3xye91w7BE1XMRKi2LN9kiMk5Csz9Zk&r=_r6-SBvtAqSGH0t7NYy2fPpq1wyPKQV-8ZiMEmovLb4&m=VOCZRfvltfQVruOYji1Zxg1glaeFkgaXpb4cs_-uBcA&s=078VAAXITZytymPfixFzFfchk0iZFVNe4-n_KJChNng&e= and @atrayanohttps://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_atrayano&d=DwMCaQ&c=ApwzowJNAKKw3xye91w7BE1XMRKi2LN9kiMk5Csz9Zk&r=_r6-SBvtAqSGH0t7NYy2fPpq1wyPKQV-8ZiMEmovLb4&m=VOCZRfvltfQVruOYji1Zxg1glaeFkgaXpb4cs_-uBcA&s=9vfocAe1CDYsY_fRE-bN6uLqEYHoqbWXwJMVwCvwH8g&e=, thanks! Apologies for ineptness at making this PR. I shouldn't do anything like this after 6 pm.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_GEOS-2DESM_MAPL_pull_429-23issuecomment-2D645599474&d=DwMCaQ&c=ApwzowJNAKKw3xye91w7BE1XMRKi2LN9kiMk5Csz9Zk&r=_r6-SBvtAqSGH0t7NYy2fPpq1wyPKQV-8ZiMEmovLb4&m=VOCZRfvltfQVruOYji1Zxg1glaeFkgaXpb4cs_-uBcA&s=rdBz4j34OCpQnMjGbqznhxCKPSd_wIOItUtzYElbS6g&e=, or unsubscribehttps://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_notifications_unsubscribe-2Dauth_AL6ZCVJKOQJHYQPJLLJ2MD3RXEPVVANCNFSM4OAAWI5Q&d=DwMCaQ&c=ApwzowJNAKKw3xye91w7BE1XMRKi2LN9kiMk5Csz9Zk&r=_r6-SBvtAqSGH0t7NYy2fPpq1wyPKQV-8ZiMEmovLb4&m=VOCZRfvltfQVruOYji1Zxg1glaeFkgaXpb4cs_-uBcA&s=bIKbaUMtKVS3UgZtp2g1Pvst_W1v5SiyTuCxSyVYwMQ&e=.

@atrayano I think @lizziel was referring to the likelihood of making a mistake when tired.    My corollary is to never trust positive results late on a Friday.

@tclune , exactly right. My intention was to commit on top of develop and then do the PR to merge into develop. Besides working late I think I also skipped lunch that day...


When page is rendering the control does not render....

I have insert a new control in the  element like this ...
`

`
When I start the page for the first time the control does not render. When I do a small change in the code the control appears when the page is refreshing. Why is that?

I search a little bit and I believe it is when it renders for the first time it return null, but if it is updated it calls the renderContent method which renders the component as it is.

Indeed. This needs to update in a componentDidMount function. I will update in a hour or so.

This commit resolved this issue for me. Let me know when you have confirmed.

Yeap, problem solved. THX!!!!

Hi, I know this is closed. But I am on version 2.0.0 and experiencing the same issue. It loads after only it hor reloads.


Broken unicode in CouchDB 3.0.0

Description
After upgrading to CouchDB 3 we had a big problem with JS query server that now incorrectly works with unicode characters.
Steps to Reproduce
Create view with following map function:
function (doc) { log(doc.string_field.length); }
Put document with field string_field with any unicode characters. We're have text fields with Hebrew and Russian. Now you can see in console incorrect string length. Thus substring and slice also don't work.
For example:
{"string_field": "мама мыла раму"}
Expected Behaviour
Correct behaviour of length, substring and slice.
Your Environment
CouchDB 3.0 in single node mode or CouchDB 3.0 in cluster mode with 3 nodes. Installed from official docker image on 64bit host system.

CouchDB version used: CouchDB 3.0, git_sha: 03a77db6c
Operating system and version: Official docker image - couchdb:3

Duplicate of #2756 . Thanks for the additional background.


Renamed localEulerAnglesSensitivity

This feature was only added 2 weeks ago, so I think the rename is safe without an obsolete.

why rename it?
seems like rotation sensitivity tells people exactly what it is.
someone new to unity might not understand what euler sensitivity could be

This is a breaking change

why rename it?
seems like rotation sensitivity tells people exactly what it is.
someone new to unity might not understand what euler sensitivity could be

He is renaming it from localeulerangle to local rotation sensitivity, not the other way around

He is renaming it from localeulerangle to local rotation sensitivity, not the other way around

@vis2k What he said ^


Indentation falls back to column 0 after second line break

In latest master, type the following:
let getStringAsync(str: string) = async { return str }

let fooAsync() = async {
    let! str1 = getStringAsync "hello"
}

Now, hit enter.  Notice that the cursor is at the same column as the above let! binding,  column 5:
let getStringAsync(str: string) = async { return str }

let fooAsync() = async {
    let! str1 = getStringAsync "hello"
//  |  <--- cursor is here
}

Now, hit enter again:
Expected: Cursor is on a new line, and still on column 5.
Actual: Cursor is at column 1.

Yes, it's an annoying bug.

@abelbraaksma Thanks for pointing that out!  We should definitely address that if it's feasible for this release.


When to use duplicate: true / auto-adjust certs with new domains?

See https://github.com/keystonejs/keystone/issues/3149
Problem
When you change the list of accepted domains, but the cert is still fresh, letsencrypt won't request a new certificate.
Possible solutions

Set duplicate: true on all requests

Problem: Every time you restart the server, it will request a new cert, even if the previous one is valid for the current domain

Set duplicate: true only if the domain list changed

Problem: How to maintain that state, what if the renew didn't work

Have letsencrypt automatically determine if a domain is missing from the cert and renew

This would seem to be the best solution from my perspective… Not sure if this is even possible?

Is this the correct place to ask this, should I ask elsewhere?

I would say this is a question for node-letsencrypt.
However, once the renewal is fixed (which I finally started working on again yesterday and will continue tomorrow) I don't think this will be a problem anymore.

Renewal is fixed in letsencrypt-express v2.0.0, based on node-letsencrypt v2.1.0
See the new README.md


UnpBootAccordionView does not acknowledge ajaxload No option

Found that when using the AccordionView it would not write the URL properly when opting not to use ajax.

Possible solution:
Added script library UnpCommon.jss to UnpAccordionViewList ( corrects issue with getDbPath() ) and replaced the line sessionScope.get( "ajaxLoad" ) with context.getUrlParameter( "ajaxload" ).
Adjusted unplugged.js to add ajaxload as a parameter option via fetchDetails and related calls to UnpAccordionViewList.xsp etc.
Adjusted UnpBootAccordionView custom control to pass ajaxload parameter via fetchDetails.


install error

Hi,
after the first command I've got this error
`nisrine@nisrine:~/libqemu-python$ python3 setup.py install
/usr/lib/python3.4/distutils/dist.py:260: UserWarning: Unknown distribution option: 'install_requires'
warnings.warn(msg)
running install
running build
got version from VCS {'full': '45d13136633cac642333e361b0d70a069c74dd6a', 'version': '0+untagged.g45d1313'}
running build_ext
/usr/bin/python3 build.py
Submodule 'src' (https://github.com/zaddach/libqemu) registered for path 'src'
Cloning into 'src'...
remote: Counting objects: 255174, done.
remote: Total 255174 (delta 0), reused 0 (delta 0), pack-reused 255174
Receiving objects: 100% (255174/255174), 109.63 MiB | 1.21 MiB/s, done.
Resolving deltas: 100% (201678/201678), done.
Checking connectivity... done.
Submodule path 'src': checked out '38ff776ef87e15d5d064c0659d99d1ab1247e3df'
ERROR: "clang" either does not exist or does not work
Traceback (most recent call last):
File "build.py", line 44, in
main(parse_args(), os.environ)
File "build.py", line 32, in main
subprocess.check_call([configure] + CONFIGURE_PARAMS, cwd = build_dir)
File "/usr/lib/python3.4/subprocess.py", line 557, in check_call
raise CalledProcessError(retcode, cmd)
subprocess.CalledProcessError: Command '['/home/nisrine/libqemu-python/src/configure', '--target-list=arm-lib,i386-lib,x86_64-lib', '--enable-lib', '--clang=clang', '--cxx=clang++', '--cc=clang', '--with-llvm-config=llvm-config']' returned non-zero exit status 1
error: command '/usr/bin/python3' failed with exit status 1
`
any help?
thanks

I think it's because in the /usr/bin I have clang-3.5 and not clang should I modify it in the configure file ??

Hi,
I guess you downloaded the zip and I assumed that the git is checked out. try to use the git repository (i.e., git clone git@github.com:zaddach/libqemu-python.git) and then try to build again. I'll need to see how to fix the build in case the project is not cloned from git. Also, yes, your clang-3.5 should be called 'clang'. You can change this in the build script (build.py).
Jonas

Also be careful, for some reason install_requires doesn't work for you. You need llvmlite with the specific version from the setup.py file installed.

Hi Jonas,
I've installed your tool but when I run the test I get this
nisrine@nisrine:~/libqemu-python$ python3 -m libqemu.disassemble --arch arm --flag thumb --address 4096 "08 1D" /usr/bin/python3: Error while finding spec for 'libqemu.disassemble' (<class 'ImportError'>: No module named 'llvmlite')
I ve tried to install llvmlite from here using miniconda so I had the version llvmlite-0.9 installed and I coudn't found the version 0.5.1
can you help
thanks

yes, getting the right version of llvmlite is tricky.
What I did was the following:
git clone https://github.com/numba/llvmlite
( cd llvmlite && git checkout v0.5.1 && python3 setup.py install )
I will update libqemu at some point to work with a more recent LLVM version than 3.5, so that it works with the current llvmlite version. If you find the dependencies of the python binding too cumbersome, also have a look at the files in libqemu-test, these don't need any python. I will also write the same test program to translate single instructions in C at some point.


Testing to confirm that we don't send nudge emails if there is a file for the pub in the AI OA workflow

#559 should have stopped these emails. A few times recently, I've wondered whether that's actually working. I don't have anything concrete enough to make a bug report, but the publication that had me wondering this is "Comparative Media Literacies." Seems like an RMD user deposited into ScholarSphere using the Standard Workflow while there was a file for it in the AI OA Workflow. I know that is possible, but it seems unlikely that the user would do that without receiving the reminder email. I've also received a couple responses to reminder emails where the researcher told us they had put it in AI, and they had. That could be an issue of unmerged duplicates, though.
The easiest solution to this would be to finish the backlog of items in the AI OA Workflow...
Related issue #600

Wondering if #550 could account for the problems listed above.

We're going to try fixing #550.


How audio mixing can be useful

Hi everybody!
Sorry, I know it's a long post but I think it is worse to read.
Regarding to the media mixing, I know there is no plan for it at the moment in Licode project but I have three reasons that it can be useful, particularly the audio mixing:
1. Bandwidth: The bandwidth usage for an event (e.g. a web meeting) on the server side is calculated by multiplying the number of publishers(speakers) and the number of participants(1 send stream for the publisher & n-1 receive streams for listeners). By audio mixing the amount of bandwidth will decrease to just one publisher (this does not apply for video). For a small event it has not a noticeable impact but in a large scale event, a webinar with hundreds of participants for instance, every single publisher matters and audio mixing is strongly helpful. Let me explain this with an example. Consider a web meeting with 10 participants and 3 publishers (audio only) and a webinar with 100 participants and the same 3 publishers. Let's assume each audio stream creates 50 kbps bandwidth on the MCU. The result of audio mixing for these two scenarios is interesting:
10  users * 3 publishers * 50 kbps = 1,500  kbps => audio mixing saves 1,000 kbps (not much)
100 users * 3 publishers * 50 kbps = 15,000 kbps => audio mixing saves 10,000 kbps (10 mbps!)

It is while the costs of audio mixing (e.g. CPU, RAM) for a particular number of publishers (3 in this example) is always the same for any number of participants! And for audio mixing it would be not much.
You may say in a webinar there always would be only one publisher at a time. Oaky, then you would have to ask the speakers to turn on/off their mic when they start/stop speaking. By audio mixing you don't worry about the bandwidth any more even if all speakers are publishing.
2. Latency: This is even more important than the bandwidth. As you may know there is a latency when subscribing and unsubscribing to/from a publisher because the MCU does a lot of things for every user subscription. I've experienced that the number of participants affects this latency. For a large number of users the latency can grow up to several seconds depending on the server resources. Such a latency is really annoying in a big webinar. By having audio mixing there would be only one stream that every user subscribes to, so no more subscription/unsubscription when we are switching between publishers as far as there is at least one publisher at a time. Even I have a technique in mind that the MCU sends an empty stream when there is no publisher, so no unsubscription at all. This feature is very useful for webinars with float speakers which start and stop publishing frequently.
3. Recording: It is obviously easier to record, sync and replay only one stream rather than several steams. It needs less resources including CPU, RAM and disk space.

Any thought? Any one has experiences on audio mixing? Please let me know if I'm wrong, and if I'm not then I would like to help this feature to be added to Licode.

Proper audio mixing would be a nice feature. I would like it to be included in Licode and have been looking at technical possibilities. There are, however, a couple of issues that should be considered:

Would we like to have other audio processing features than just mixing? Compression, equalization etc. Compression (reduction of the dynamic range) is a quite useful tool when mixing sound from very diverse sources. If go for mixing, then there is no reason to leave those processing features out (or at least allow hooking already existing tools to the mixing system).

Would we like use "smart mixing" or not? Smart mixing would handle sound in the way that the person speaking would not get his/her own voice back. That would be nice, but makes mixing more complex and creates more than one mixed audio streams. The smart mixing would not be too difficult in itself, just use suitable trigger levels and time delays (may require some experimenting), but handling the streams might not be straightforward.

I would be happy to have mixing even without the smart features.

Audio mixing is not in the roadmap. We plan to continue developing a SFU without any kind of real-time mixing. Audio mixing has its advantages. There are other alternatives that could be interesting such as audio stream selection.

@jcague Thanks for the reply.
Would you please explain more about audio stream selection?

You can select and forward only those streams that you consider more important, and mute the rest. But again, you're right, SFU and MCU have their own pros and cons anyway.

@mkhahani any audio/video mixer solution opensource code alternative, rtmp or webrtc was fine for me? cause my android platform‘s CPU is pool.


Http2ConnectionHandler breaks channelReadComplete pipeline notification

Netty Version: 5.0.0-Alpha3-SNAPSHOT
Hi guys,
Http2ConnectionHandler overrides channelReadComplete(...) here but fails to pass notification further down the pipeline by calling super.channelReadComplete(...) or ctx.fireChannelReadComplete(...).  I assume it should call super.channelReadComplete(...), what are your thoughts?
/cc @Scottmitch @nmittler @louiscryan as you guys know more about this :)

@blucas makes sense. We love PRs as you know ;)

+1 to PRs :)

Let me fix this.

Fixed by https://github.com/netty/netty/pull/3948#issuecomment-119327866


How to discard the background of GPUImageCrosshairGenerator?

While I am using it directly with a GPUImageView without blending with any other filters, there is always the background. If I do the same as u do in the demo, there is no background. I also tried setBackgroundColorRed, it doesn't seem to work. Then I tried to modifiy the alpha value in shader as 0.0,             it's still there. After I discard that in shader, it finally disapper, but that's jagged. Basiclly, I don't need the image from camera, just need to use the result from GPUImageCrosshairGenerator, how could I discard the background properly.

And the alpha value doesn't seem working with the background.

I'm not sure what you mean by the background. The GPUImageCrosshairGenerator renders red crosshairs, with the rest of the image having an alpha of 0. This is typically used in a blend with the image, to overlay the crosshairs on the image.
Note that GPUImageView doesn't currently support transparency, so if you're just rendering the crosshairs to that and attempting to overlay the entire view on something it won't render properly.

They are overlapping each if I connect GPUImageCrosshairGenerator to GPUImageView directly without anything. But once I discard the the background in the shader of GPUImageCrosshairGenerator, it will be transparent.


Add minimalistic helm chart to grafeas

A minimalistic grafeas helm chart to -
The user must provide (see values.yaml)

Certificate for mutial authentication
Name to a persistent volume claim (to persistent the DB data)

If you are OK with the proposed approach, i would add a README on how to
run it locally.

Hi @liron-l, thanks for the PR. Could you please add more details for this on the intended use? README would definitely be helpful on how to bring this up. For 0.1.0 we're considering using Helm charts for Grafes, to bring it up with Kritis, so I think this PR is on the right track, but I'd like to understand better the approach you're using here.

Thanks @aysylu, i've added a README with instructions on how to run the chart locally.

Thank you, @liron-l! Looks great, just 1 piece of feedback about the default installation.

Thanks @aysylu, by default, the helm chart will use the memstore storage type and will not require a persistent volume (see storageType: "memstore" in values.yaml and conditions in deployment.yaml).
Also, as the embedded storage solution does not require any additional setup (except from the persistent volume) I kept it as an alternative storage option (see README.md).

Sounds great, thanks @liron-l! Sorry I missed the part about the default memstore config, I see it now. Thanks for your valuable contribution!


ImportError: cannot import name '_C' from 'geo_parse'

I was running the line "from geo_parse.config import cfg," and it traced back to "from geo_parse import _C" and reported an error. The specific error message was "Traceback (most recent call last):
File "[file_name].py", line 3, in
from demo.predictor import GeoDemo
File "[path]/PGDP/demo/predictor.py", line 10, in
from geo_parse.modeling.detector import build_detection_model
File "[path]/PGDP/geo_parse/modeling/detector/init.py", line 2, in
from .detectors import build_detection_model
File "[path]/PGDP/geo_parse/modeling/detector/detectors.py", line 2, in
from .generalized_rcnn import GeneralizedRCNN
File "[path]/PGDP/geo_parse/modeling/detector/generalized_rcnn.py", line 9, in
from ..backbone import build_backbone
File "[path]/PGDP/geo_parse/modeling/backbone/init.py", line 2, in
from .backbone import build_backbone
File "[path]/PGDP/geo_parse/modeling/backbone/backbone.py", line 7, in
from geo_parse.modeling.make_layers import conv_with_kaiming_uniform
File "[path]/PGDP/geo_parse/modeling/make_layers.py", line 10, in
from geo_parse.layers import Conv2d
File "[path]/PGDP/geo_parse/layers/init.py", line 10, in
from .nms import nms, ml_nms
File "[path]/PGDP/geo_parse/layers/nms.py", line 3, in
from geo_parse import _C
ImportError: cannot import name '_C' from 'geo_parse' ([path]/PGDP/geo_parse/init.py)"

Maybe the error in building, please check if it is right when runing "python setup.py build develop --no-deps"

Maybe the error in building, please check if it is right when runing "python setup.py build develop --no-deps"

Thank you for the suggestion! Based on the feedback I think the error was caused by the absence of ‘geo_parse.csrc’ from the ‘packages’ configuration and some fucntions that got deprecated.


Add mps template

mps process is required to run in hostIPC

mps=enabled doesn't work.
the gpu sharing pod doesn't recognize the the mps process.


Do not try to url encode the URL by hand

Here:
https://github.com/aiidalab/aiidalab-optimade/blob/a14ae5caf6d2045d341f4c8891ed074022af11c7/aiidalab_optimade/informational.py#L88
The query string is 'url-encoded' by hand, by passing %23 instead of # and doing some replacements.
However this is very fragile, and in particular the content of log.getAttribute('value') is not url-encoded. Indeed, when trying to click on the button in production, I get a 503 Gateway error from GitHub because some < and > characters are left in.
Instead, one should write the intended final string, and then use url lib.parse.urlencode to safely create and encoded query string.
For instance: urllib.parse.urlencode({'body': '# some body with < symbols >', 'title': 'Some & title'}) that e.g. returns body=%23+some+body+with+%3C+symbols+%3E&title=Some+%26+title

So I actually do encode log.getAttribute('value') in the log handler. However, this is one of those things that happens (and shouldn't happen) when you're simply trying to make something else work. You miss the obvious solutions ...
Will definitely use urllib instead, thanks!

Fixed in #25


Fix metadata proxy load filter

What is the purpose of the change
Fix https://github.com/apache/dubbo/issues/11716
Brief changelog
Verifying this change

Checklist

[x] Make sure there is a GitHub_issue field for the change (usually before you start working on it). Trivial changes like typos do not require a GitHub issue. Your pull request should address just this issue, without pulling in other changes - one PR resolves one issue.
[ ] Each commit in the pull request should have a meaningful subject line and body.
[ ] Write a pull request description that is detailed enough to understand what the pull request does, how, and why.
[ ] Check if is necessary to patch to Dubbo 3 if you are work on Dubbo 2.7
[ ] Write necessary unit-test to verify your logic correction, more mock a little better when cross module dependency exist. If the new feature or significant change is committed, please remember to add sample in dubbo samples project.
[ ] Add some description to dubbo-website project if you are requesting to add a feature.
[ ] GitHub Actions works fine on your own branch.
[ ] If this contribution is large, please follow the Software Donation Guide.

Codecov Report

Merging #11769 (3bfb17d) into 3.1 (7277c15) will increase coverage by 0.00%.
The diff coverage is 100.00%.

@@            Coverage Diff            @@
##                3.1   #11769   +/-   ##
=========================================
  Coverage     64.68%   64.69%
  Complexity       14       14
=========================================
  Files          1431     1431
  Lines         60034    60034
  Branches       8827     8827
=========================================
+ Hits          38832    38838    +6
+ Misses        17119    17115    -4
+ Partials       4083     4081    -2

Impacted Files
Coverage Δ

.../dubbo/registry/client/metadata/MetadataUtils.java
65.74% <100.00%> (ø)

...e/dubbo/rpc/protocol/tri/transport/WriteQueue.java
64.28% <0.00%> (-9.53%)
:arrow_down:

...ache/dubbo/common/extension/ExtensionDirector.java
93.33% <0.00%> (-3.34%)
:arrow_down:

...egistry/kubernetes/KubernetesServiceDiscovery.java
77.83% <0.00%> (-2.36%)
:arrow_down:

.../src/main/java/org/apache/dubbo/rpc/RpcStatus.java
80.95% <0.00%> (-1.20%)
:arrow_down:

.../dubbo/remoting/transport/netty4/NettyChannel.java
72.64% <0.00%> (-0.95%)
:arrow_down:

...he/dubbo/registry/zookeeper/ZookeeperRegistry.java
61.57% <0.00%> (+0.52%)
:arrow_up:

...ng/zookeeper/curator5/Curator5ZookeeperClient.java
77.04% <0.00%> (+0.81%)
:arrow_up:

...adata/store/zookeeper/ZookeeperMetadataReport.java
72.34% <0.00%> (+1.06%)
:arrow_up:

...c/main/java/org/apache/dubbo/rpc/RpcException.java
83.87% <0.00%> (+3.22%)
:arrow_up:

... and 6 more

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

Integration test cases: https://github.com/apache/dubbo-samples/pull/786


PhysicsFS 4.0 plans...

This is preliminary, but here's some initial wishlist things I'd like to do:

Make the Android stuff not use PHYSFS_Init with a magic structure. This was a wrong move.
Move all the global state into a struct and let people create instances, each with their own search path, write dir, mutexes, etc. All the functions will take an instance pointer; for people that don't care about this or want to build legacy code, we'll leave the original entry points alone, and those will manage an internal instance (or write a physfs3-compat library).
Remove CD-ROM support (entry point will remain but never find any discs).
Multiple write directories; they'll interpolate and mount and such like the search path does.
Queries for other writeable directories (config/cache/etc), instead of just a "pref path"
Probably other things.

Built-in UDF .iso support would be neat.

Some sort of way to signal that Emscripten should persist some files and not others (MEMFS vs IndexedDB), and possibly a "commit" API (needed by Emscripten and Nintendo Switch).

Some sort of way to signal that Emscripten should persist some files and not others (MEMFS vs IndexedDB), and possibly a "commit" API (needed by Emscripten and Nintendo Switch).

I worked around that for emscripten by mounting IDB, and setting PhysFS write dir to the same directory. (iirc)

file move/rename support.

Multiple write directories;  they'll interpolate and mount and such like the search path does

Related to this, one current pain point for me with PhysFS has been how the write path is treated like a separate filesystem which doesn't necessarily match the structure of the main virtual filesystem.
In my game, I want my writeable files to be mounted somewhere outside of the root of my game's main virtual filesystem, so that I can know for certain that files written by the game at runtime cannot accidentally override files which were provided by the game;  the whole write directory gets mounted over into a side "/user/" virtual directory in the search path.
This means that my code has to always know and remember that if I write a file out to "/file.txt", that same file would need to be read from "/user/file.txt".  This does work in PhysFS right now as long as I'm diligent about it (and I've written a wrapper around PhysFS to try to hide it from game code), but it's obviously a bit of a cognitive load that feels like maybe PhysFS could simplify.
Proposal:  maybe this whole thing could be implemented by having PHYSFS_mount() include some configuration flags that say for each mount point whether that path can be written to;  maybe turn its current append?1:0 parameter into a bitfield with named parameters that can be bitwise-or'd together to denote which mounting points should allow writes.  Maybe something like this:
#define PHYSFS_READ (0) // implicit
#define PHYSFS_LOWPRIORITY (0) // implicit if HIGHPRIORITY isn't set.
#define PHYSFS_HIGHPRIORITY (1<<0) // matches "1 == append to search path" current functionality
#define PHYSFS_WRITE (1<<1)
#define PHYSFS_READWRITE (PHYSFS_READ|PHYSFS_WRITE)

PHYSFS_init(argv[0]);
PHYSFS_mount("MyZippedData.zip", "/data", PHYSFS_HIGHPRIORITY );
PHYSFS_mount( SDL_GetPrefPath(companyName, title), "/preferences", PHYSFS_READWRITE | PHYSFS_LOWPRIORITY);
PHYSFS_mount( SDL_GetSaveGamePath(companyName, title), "/saves", PHYSFS_READWRITE | PHYSFS_LOWPRIORITY);
  // SDL doesn't actually currently provide a 'SDL_GetSaveGamePath()' API, but just as an example.

PHYSFS_File *file;
file = PHYSFS_openWrite("myfile.txt"); // fails, not in a READWRITE directory.
file = PHYSFS_openWrite("/preferences/myfile.txt"); // succeeds
// (...)
PHYSFS_close(file);
file = PHYSFS_openRead("/preferences/myfile.txt"); // also succeeds, opens the same file saved above.
file = PHYSFS_openWrite("/data/myfile.txt"); // fails, as there's no 'WRITE' path mapped here.
file = PHYSFS_openWrite("/preferences/../myfile.txt"); // fails, as this resolves into a filepath which doesn't have a anything mounted.

Remove CD-ROM support (entry point will remain but never find any discs).

Does it mean that physical media for Xbox Series X and PS5 won't be handled?

Does it mean that physical media for Xbox Series X and PS5 won't be handled?

Mmm...that's a good question. Maybe we'll leave this API after all. Let me think on it.

In my game, I want my writeable files to be mounted somewhere outside of the root of my game's main virtual filesystem, so that I can know for certain that files written by the game at runtime cannot accidentally override files which were provided by the game; I mount the whole write directory over into a side "/user/" virtual directory in the search path's virtual filesystem.

I'm probably misunderstanding the problem, and if so I apologize, but why don't you mount the write dir's path last, to "/", and put it at the end of the search path, so it never overrides anything and you don't have to manage it as a separate /user path when reading?
(Or maybe, once I build instances, you have a second instance that only has the write path mounted as "/", so if PHYSFS_openRead() fails, you can just try the second instance, so it's extremely isolated against accidental access. Maybe?)

Is #18 under consideration? As mentioned if you have a file inside a zip then change it (for example a default config file), the change is wrote to the overlay. However the change is not seen and only the original file is picked up, ideally the overlay version should override the .zip original.
Currently the only solution is to see what file is changed then delete it from inside the .zip to then let the new one be seen.

PhysicsFS 4.0 Wishlist item:

Remove dependency on the C runtime, the same way we did for SDL.

I think for the most part it's just str* and mem* functions. Probably won't slot dlmalloc in here, but make it so you can build PhysicsFS without a reference to malloc() where the app must specify their own allocator at startup.

Change coding style to be less 1990's Ryan and more 2020's Ryan.  :)

I wish the issue #13 would be fixed for PhysFS 4.0. Because I've tried before to use PhysFS to load multiple assets in parallel, and then found out it didn't speed up things (actually it made loading slower), details of why that happens is mentioned in the blog post of the issue.

I came here to actually say the same thing and then I read your post.
Anyway, I believe this is the upmost important thing, even for current PhysicsFS 3.0.

Anyway, I believe this is the upmost important thing, even for current PhysicsFS 3.0.

I agree, although the most immediate workaround is to not use the callback enumeration mechanism, so you end up with a complete list of files before starting to process data, and then it won't block on the mutex, and that can be done by the app right now.
I suspect the correct fix inside PhysicsFS, for 3.x if not in general, is to have each archiver promise thread safety directly, which might mean duplicating the archive file handle in the same way we do when opening individual files within it, so it can seek around to enumerate without interfering with other operations that might occur, but I haven't had a moment to untangle the whole mess yet.

Anyway, I believe this is the upmost important thing, even for current PhysicsFS 3.0.

I agree, although the most immediate workaround is to not use the callback enumeration mechanism, so you end up with a complete list of files before starting to process data, and then it won't block on the mutex, and that can be done by the app right now.
I suspect the correct fix inside PhysicsFS, for 3.x if not in general, is to have each archiver promise thread safety directly, which might mean duplicating the archive file handle in the same way we do when opening individual files within it, so it can seek around to enumerate without interfering with other operations that might occur, but I haven't had a moment to untangle the whole mess yet.

I suppose the main problem is reading files from the same archive (handle) in multiple threads (e.g. assets.zip). Just like you said, having duplicated archive file handle could solve this issue once for all.

I'd love if PhysFS is able to mount Android content:// URI that's retrieved from ACTION_OPEN_DOCUMENT_TREE intent or being able to create PhysFS_IO from an Android content:// URI.

Split out public header: #71

I personally would like improved documentation, especially related to custom archivers.
I'm currently trying to make one for a custom format, but the lack of documentation in Doxygen, combined with the built-in archivers' source code barely having any comments, are making it harder than it should be.


Update assign-teams-add-on-licenses.md

https://github.com/MicrosoftDocs/OfficeDocs-SkypeForBusiness/issues/7128
Added info

@JohanFreelancer9 Copy editing is needed for this PR. Thanks.

@cichur PR has been copyedited and is ready for final review, could you please check and merge? Thanks!

@cichur


[6.1][cxx-interop] Remove escapability interop annotations

As this feature is still under design, we do not want users to depend on the semantics we ship in 6.1, so remove the annotations from the bridging header. This PR also removes the tests that rely on these annotations.

@swift-ci  please test

@swift-ci please test

@swift-ci please test


Build go function fails if there are more than main.go in src

How did you install the Amplify CLI?
yarn
If applicable, what version of Node.js are you using?
20
Amplify CLI Version
12.11.1
What operating system are you using?
Ubuntu
Did you make any manual changes to the cloud resources managed by Amplify? Please describe the changes made.
No
Describe the bug
This change here fails go function build if there are  more than main.go in src.
Expected behavior
Go function should be built
Reproduction steps

Create a lambda function using go
in main.go use a function defined in another.go
Build will fail

Project Identifier
No response
Log output
No response
Additional information
No response
Before submitting, please confirm:

[X] I have done my best to include a minimal, self-contained set of instructions for consistently reproducing the issue.
[X] I have removed any sensitive information from my code snippets and submission.

Hey @nam-truong-le, thank you for reaching out. Marking as bug for further investigation.

Hey @nam-truong-le, wanted update on this issue. With the latest update to the Lambda functions on https://github.com/aws-amplify/amplify-cli/pull/13671.
You will need to upgrade the local go runtime to match the Lambda function. refer to AWS Documentation providing additional information.
On a quick test with the following steps the push did not run into an error on a new function

amplify add function
created a folder say test folder with test.go file

// <function-name>/src/test/test.go
package test

import (
	"fmt"
  )

func DoSomething() {
	fmt.Println("Hello World!")
  }

in main.go
// <function-name>/src/main.go
package main

import (
  "fmt"
  "context"
  "github.com/aws/aws-lambda-go/lambda"

  test "lambda/test"
)

type MyEvent struct {
  Name string `json:"name"`
}

func HandleRequest(ctx context.Context, name MyEvent) (string, error) {
  return fmt.Sprintf("Hello %s!", name.Name ), nil
}

func main() {
  test.DoSomething()
  lambda.Start(HandleRequest)
}

@nam-truong-le this is on the latest version of Amplify CLI: 12.11.1

Hey @nam-truong-le, could you provide us the reproduction steps or sample snippets of the example with the file paths used to reproduce the issue?

I am experiencing the same issue with my Go Lambda functions. In my case I have multiple files in my main package.
Everything outside main.go is ignored as the command used is go build -o ../bin/bootstrap main.go.
I'm using the 12.1.1 version of the CLI on Ubuntu 20.04.6 LTS.
This are the steps taken to reproduce:
1 -  amplify build function gomultiplefiles
2 - On amplify/backend/function/gomultiplefiles/src/main.go
package main

import (
	"context"
	"fmt"

	"github.com/aws/aws-lambda-go/lambda"
)

func HandleRequest(ctx context.Context, name MyEvent) (string, error) {
	return fmt.Sprintf("Hello %s!", name.Name), nil
}

func main() {
	lambda.Start(HandleRequest)
}

2 - On amplify/backend/function/gomultiplefiles/src/types.go
package main

type MyEvent struct {
	Name string `json:"name"`
}

3 - Running amplify build function gomultiplefiles  outputs the following error
# command-line-arguments
./main.go:10:46: undefined: MyEvent
🛑 go failed, error message was Command failed with exit code 1: go build -o ../bin/bootstrap main.go

If the command changed from go build -o ../bin/bootstrap main.go to go build -o ../bin/bootstrap ., it would work.
This same situation is mentioned in this issue #5269  from some years ago.

Hey @jo-su, thank you for the information. I was able to reproduce the issue. Marking a bug.
reproduced the issue with the following test

created a go function
modified the function according to https://github.com/aws-amplify/amplify-cli/issues/13725#issuecomment-2090118621
changes directory into the function folder to and ran go build -o bootstrap main.go which errored out with ./main.go:13:46: undefined: MyEvent. but running go build -o bootstrap .  built the function.

Noticed an error Error: fork/exec /var/task/bootstrap: exec format error Runtime.InvalidEntrypoint on the Lambda function when the function was pushed with an older version of Amplify CLI.
From the AWS documentation: https://docs.aws.amazon.com/lambda/latest/dg/golang-handler.html#golang-handler-naming
the function will refer to function code named bootstrap, which previously was handler in go1.x runtime. ensure the go.mod and the CloudFormation template are updated to use provided.al2023.


📮 Postmark integration #19398

I have been working on an integration of Postmark with Ghost. Please provide your feedback before I continue.
To-Do

[x] Implement analytics
[ ] Handle Postmark errors
[ ] Add tests

This a copy of a previous PR, submitting it again because our main branch got corrupted due to a merge conflict.

Could you comment on the feedback I left in the last PR?
https://github.com/TryGhost/Ghost/pull/19398#issuecomment-2132520491

@andreascreten you requested feedback and I left some here: https://github.com/TryGhost/Ghost/pull/19398#issuecomment-2132520491

I've updated this PR to go in the direction that @markstos has suggested.
As far as I noticed, controlling the used adapter is only done using the config file, so I still don't have a solid idea of controlling the provider using the settings UI.
I'd love to get some directions for my next step.

Now that this has an updated to use the adapter pattern, the next step is for a Ghost core team member to take a look and discuss what further refinements might be required to get it merged.
One thing I see is that this solution is involves some mentions of Postmark in the React code, like here:
https://github.com/TryGhost/Ghost/pull/20530/files#diff-b74336cc721823b891ffb6e8f7b9d367c7dea2e7d7bdfb6b80db78972d7763f7R7
This means that once merged, it will be the responsibility of the Ghost team to maintain it. Is this approach acceptable to the team? Or can the web admin interface also use the config system to look-up an alternate admin UI to load instead of the Mailgun UI elements?


Reading files in zip folder without unzip

Hi Team,
I want to read one file from zip(which has password) folder without unzip/exract it. I am using following snippet.
UZKArchive *archive = [[UZKArchive alloc] initWithPath:zipPath password:@"49445" error:&archiveError];
NSError *error = nil;
NSArray<NSString*> *filesInArchive = [archive listFilenames:&error];
NSLog(@"filesInArchive %@",filesInArchive);
for(UZKFileInfo *info in [archive listFileInfo:&error])
{
    NSLog(@"file name %@ Compressed size %lld UnCompressed size %lld",info.filename,info.compressedSize,info.uncompressedSize);
}

I have a file in zip folder named "complete_49445_ak_pc_6922/49445.json"
NSData *extractedData = [archive<EMAIL_ADDRESS>error:&error];

here i extracted data is coming as null.
i would like to read one file content based on that file content i would like to extract files.
How can we do that ?
i am able to get contents if zip is not having password. But I am facing issue if zip is having password only, though i have provided password for archive object.
Thanks,
Sravan

I'm still curious to see what is in the NSError, but I also have some unit tests that deal with passwords, which may help you figure out what's going wrong. I've linked to a couple of examples below.
https://github.com/abbeycode/UnzipKit/blob/master/Tests/PasswordProtectionTests.m
https://github.com/abbeycode/UnzipKit/blob/master/Tests/ExtractFilesTests.m#L38

I am getting following error
Error Domain=UZKErrorDomain Code=-103 "Bad zip file" UserInfo={NSLocalizedFailureReason=Bad zip file, NSLocalizedRecoverySuggestion=Error opening archive (-103), NSURL=file:///Users/sravanpeddi/Library/Developer/CoreSimulator/Devices/B8E1609D-21D9-4C9A-8FF7-01AE9FB49AC6/data/Containers/Bundle/Application/06C48480-CFBB-4E43-BB9A-B721313B384D/SampleUnZip.app/complete_49445_ak_pc_6922.zip, NSLocalizedDescription=Bad zip file}

You're able to extract it using Archive Utility? Are you able to send me the file and password privately, so I can take a look? You can DM me on Twitter: @bbeyCode.

SampleUnZip.zip
Yes. i have attached sample project which has zip file.

I was able to reproduce with the archive you included in your project, successfully unarchiving with other apps, and seeing the error you did with UnzipKit. I'm not yet sure what's going on, but will take a look.

What's causing your issue is that the archive you sent is using the AES compression method, which isn't supported yet. I've updated your issue to turn it into a feature request for supporting this compression method. In the meantime, I'm adding to the v2.0 branch a new error code (UZKErrorCodeAES) for this case, so at least it's not mysterious (once my PR passes validation).


HDDS-3543. Remove unused joda-time

What changes were proposed in this pull request?
Joda-time is defined in the pom.xml but it's not used anywhere.
Perhaps because Ozone runs on java 8, the standard date and time features are provided by java.time (JSR-310).
What is the link to the Apache JIRA
https://issues.apache.org/jira/browse/HDDS-3543
How was this patch tested?

Compiled successfully
GitHub CI Action successfully on my fork repository (result)

+1 LGTM.
Test failures mostly looks unrelated, will trigger CI for a clean run.


Create Landscape Variation

Description
Allow only portrait mode in phones, landscape and portrait mode
in tablets.
Steps to reproduce
Create all the layouts required in landscape mode, and test in the code to see if you have to accept the change of screen orientation or not (depending on the screen dimensions).
Expected behavior

Allow only portrait mode in phones, landscape and portrait mode
in tablets. (force portrait mode for phones, portrait and landscape mode for tablets (for screens greater than 600 dp)).

Ensure that when the tablet changes orientation, the application does not lose data already entered in portrait mode, and vice versa.

Note : if you don't have tablets, you can test with the virtual tablets provided by android studio (in avd - android virtual device).

Actual behavior
Only the portrait mode works as expected.
Who's working on this ?
I can do that if you want.
(If you give me to work on issue #17, I'll do it later).

Great description of issue and idea. It would be very beneficial for the app. I've assigned you onto it, feel free to start at any time!

Great description of issue and idea. It would be very beneficial for the app. I've assigned you onto it, feel free to start at any time!

All right, thank you !
I'll start working on it tomorrow.

Is this still an open issue?

Is this still an open issue?

Hello !
Sorry, finally I didn't have enough free time to do it.
Hopefully @ardovic will be able to assign this task to you very soon.
Good luck :)
@leila94

Is this still an open issue?

Hello !
Sorry, finally I didn't have enough free time to do it.
Hopefully @ardovic will be able to assign this task to you very soon.
Good luck :)
@leila94

Thank you. I will start it soon.


⚠ Bofrost.at has degraded performance

In 99a18e9, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 1116 ms

Resolved: Bofrost.at performance has improved in 06dc5c4.


Modal for digitization. Towards #248.

@afred / @caseyedavis12 / @Muraszko ...
How does this look?

This will conflict with #273, so it needs more space between the player and the button, but I don't want to get into that now.
It's hidden behind, but there's also a button in the missing transcript slot, but they both bring up the same modal.
Wording is based on Mike's, but adjusted given the context. If it could be shortened, that would be great, but it really shouldn't get any longer.
In the modal itself, I think it's good to keep the mailtos as links: button-ifying them seems heavy handed.

@mccalluc: Looks great to me.

rebasing.. stand by.

rebased, merge if Travis passes

I'll go in now and get the spacing right...


chore: update deps 2024-09-09

Summary

Test plan

This PR has been released in Reassure v1.2.1 🎉