Add an external pillar.

The pillar is not programatically modifiable. Since we need to set
some pillar values during the bootstrapping process, we need to rely
on an external pillar that we can modify.
The original module requires a '%s' argument to be provided, or the
mysql python module will complain. We added a minion_id that can be
used for certain pillar key discovery. However, since we are going to
use this pillar information for global configuration, we can just let
minion_id be NULL, and that will be readable by all salt clients.
We also add a Pillar model that will be able to modify these Pillar
values.

Do you think it would make sense to add an end to end test for this one? E.g. Create a pillar value from velum and check the value on the minion.

It makes a lot of sense, @jimmykarily. Actually, we would be checking that the merge of the information is properly done. It is actually a little tricky: https://github.com/kubic-project/velum/pull/54/files#diff-7afc35426a5be810373b1b1917642c22R16.
Considering we have this on our pillar:
certificate_information:
  subject_properties:
    C: DE
...

We can override this information by doing Pillar.create minion_id: nil, pillar: 'certificate_information.subject_properties.C', value: 'ES', for example. We don't need to change our current pillars or states, and we can just override the keys that we want, being able to fallback to the default on our static pillar if not specified.

ok this sounds great. I would:

Add this information on the top of the Pillar class (the example you sent and the fact that if overrides).
Create an end to end test as soon as this is merged.

Added an explanation and a link to SaltStack's documentation.

Added a specific document under docs explaining how we use Salt. Please, if you think there is something missing or something that can be reworded to make it more clear I'd love to know.

LGTM :+1:

I know I'm late to the party, but this LGTM too 👍


Class property

I think there is some value here? I don't think it's exceptionally common for a HasProperties instance to need a Class property value (as opposed to an Instance) but I think it's certainly a possibility.
class HasClass(properties.HasProperties):
    my_class = properties.Class('Some class...')

    def do_something(self):
        some_instance = self.my_class()
        ...

Traitlets has this, for what it's worth: https://github.com/ipython/traitlets/blob/4.3.2/traitlets/traitlets.py#L1527

So the thing that would keep this from being an Instance('...', type) property is just that that isn't really useful validation, right? Meaning the implementation is very similar to Instance, but just with an issubclass(prop, cls) check, rather than an isinstance(prop, cls) check?
Yeah, I can see that making sense, though I agree the use cases are probably a bit limited.

A draft of what I'm thinking is here: https://github.com/3ptscience/properties/pull/163
Not sure if there's anything interesting we could be doing with HasProperties Classes. Also needs some thought about serialization...

This would really useful! Within SimPEG - we would use it for solvers. We use a Solver class that is instantiated during the computation of the fields or data, so I would like to set is as an uninstantiated class when we construct the object. Roughly, I would see something along the lines of

class Class(properties.Property):

    class_info = "a property that is an uninstantiated class"

    def validate(self, instance, value):
        if inspect.isclass(value) is False:
            extra = (
                "Expected an uninstantiated class. The provided value is not"
            )
            self.error(instance, value, TypeError, extra)
        self._parent_module = value.__module__
        return value

    def serializer(self, value, **kwargs):
        return "{}.{}".format(self._parent_module, value.__name__)

    def deserializer(self, value, **kwargs):
        name = value.split(".")
        try:
            module = sys.modules[".".join(name[:-1])]
        except KeyError:
            raise ImportError(
                "{} not found. Please install {}".format(
                    ".".join(value, name[0])
                )
            )
        return getattr(module, name[-1])

I took a crack at implementing the serialization / deserialization piece, and then pretty quickly ended up rather deep in properties, particularly with the behaviour of defaults, as there are quite a number of places where properties will try to instantiate callable values, but here, we would want to leave them uninstantiated.
On the Property (this one might be appropriate - it is straightforward to overwrite)
https://github.com/seequent/properties/blob/096b07012fff86b0a880c8c018320c3b512751b9/properties/basic.py#L174-L177
Also in the sphinx docs (this seems redundant if it is already going through the default setter?)
https://github.com/seequent/properties/blob/096b07012fff86b0a880c8c018320c3b512751b9/properties/basic.py#L687-L691
This one on HasProperties seems perhaps a bit too heavy handed as we would need to re-write the _reset method on the class where the Class property is being used:
https://github.com/seequent/properties/blob/096b07012fff86b0a880c8c018320c3b512751b9/properties/base/base.py#L397-L398
Here is the current pass at a (rough) implementation within SimPEG:
https://github.com/simpeg/simpeg/blob/46a0527ed8c8d1594c108931449745eba256ed41/SimPEG/simulation.py#L40-L103
and the associated class that inherits HasProperties
https://github.com/simpeg/simpeg/blob/46a0527ed8c8d1594c108931449745eba256ed41/SimPEG/simulation.py#L142-L164
I would be happy to help iterate or test out a pr if that is of interest.


hongbo/sorted map iter

fix @sorted_map.T::iter
add @sorted_map.T::iter2
add test

Pull Request Test Coverage Report for Build 2918
Details

17 of 21 (80.95%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.03%) to 84.087%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

sorted_map/map.mbt
17
21
80.95%

Totals

Change from base Build 2915:
-0.03%

Covered Lines:
3884

Relevant Lines:
4619

💛  - Coveralls


Patch 47 - Introduce QueryBus

Closes #47

Coverage increased (+1.12%) to 71.36% when pulling f1ced83e79d60fc5671f667196f22977eefe332f on codeliner:patch-47 into b78a74518eaead6bf191e988b83fe6bfb68021cf on prooph:master.

PR should be merged into develop


stop working after message received

Hello
The client is unable to respond to the incoming message because of System.Net.Http.HttpRequestException.
The issue was reported on Dec 12 in the customer environment.  I reproduced this issue easily with our corporate S4B on-premise server. So seems it's a common problem for on-premise configuration. Until end of November 2018 everything worked perfectly. I asked about possible updates or patches and answer is: only latest Windows security patches were applied.
Steps to reproduce:

Client successfully signed in to the S4B on-premise and auto accept messages.
User sends message to the client
The last successful request is message.GetParticipant() in the MessageReceived handler: Client_MessageReceived(Message message). Any subsequent request fails with

System.Net.Http.HttpRequestException
HResult=0x80131500
Message=An error occurred while sending the request.
Source=Microsoft.Rest.ClientRuntime
Inner Exception 1:
WebException: The underlying connection was closed: An unexpected error occurred on a send.
Inner Exception 2:
IOException: Authentication failed because the remote party has closed the transport stream.
I'll appreciate any advice how to fix it.
Best Regards
Valerii

Resolved by changing .NET Framework version to 4.6.2


[ANDROID] Camera Features Don't Work, Permissions/SetFlash/ChangeCamera

Issue Description
Works beautifully with IOS, but completely falls apart for Android and does not seem that it is being worked on. Trying to understand if we need to abandon this library or if they are going to bring the support.
Steps to Reproduce / Code Snippets / Screenshots
Follow Installations Instructions.
Issues start with permissions not being asked on Android as it can't find the method, mentioned in issue 227
Got around permissions by using RN PermissionsAndroid(https://facebook.github.io/react-native/docs/permissionsandroid) but now getting NativeCameraModule.setFlashMode is not a function, get similar error for this.camera.changeCamera()

Environment

React Native Navigation version: 1.1.493
React Native version: 0.57.4
Platform(s) (iOS, Android, or both?): Android
Device info (Simulator/Device? OS version? Debug/Release?): All

Add this code after uses-permission tags:
<uses-feature android:name="android.hardware.camera" />
<uses-feature android:name="android.hardware.camera.autofocus" />
<uses-feature android:name="android.hardware.camera.flash" />

Add this code after uses-permission tags:
<uses-feature android:name="android.hardware.camera" />
<uses-feature android:name="android.hardware.camera.autofocus" />
<uses-feature android:name="android.hardware.camera.flash" />
<uses-feature android:name="android.hardware.camera.front" />

Its not working, Anybody help?

That most definitely does not help or work. When you use the library from the react native application it can't map the functions to the correct Android camera functions (changing flash, which camera to use, etc)
We went back to using the react-native-camera. Wish we could have continued to use this library as we liked it better and it worked great for iOS but not at all for Android and that is a showstopper for us.

We're having this issue as well. One work around is to use react-native-permissions instead of CameraKitCamera

I think it's because you have react-native-camera installed and linked, try remove that package first then try again. Same for #227


License?

Hey there, do you have a particular license for your project? Looks cool!
Thanks,
J

I have added the MIT License! Sorry for the delay


Spyder is not getting its taskbar icon right in Win 7/8

From<EMAIL_ADDRESS>on 2013-03-17T00:11:43Z
Please note that reporting bugs from Spyder's help menu ("?" > "Report
issue...") will fill version details for Spyder, Python and Qt in this description for you.
What steps will reproduce the problem?

Start any program with python.exe
Look at the taskbar icon
It's the spyder icon, not the icon for that program or as default should be just python icon.

What is the expected output? What do you see instead? Icon should not be spyder unless the program is actually spyder. What version of the product are you using? On what operating system? Spyderlib 2.1.13
Windows 7
Pyqt 4.9
Python 2.7.3
Please provide any additional information below
.
Original issue: http://code.google.com/p/spyderlib/issues/detail?id=1319

From<EMAIL_ADDRESS>on 2013-04-11T12:21:27Z
>> For example, try running a simple Python interpreter (python.exe).
>> Then open Spyder: you'll see that Spyder has the same icon as the Python interpreter
>> instead of its own icon.
This hasn't been my experience. No matter what order I start the python apps in, or even if spyder isn't running at all, I always get the spyder icon. I tried clearing the taskbar icon folder and that did nothing.

From pierre.raybaut on 2013-03-17T02:03:53Z
Actually I'm afraid that's an issue with Windows 7/8. If you run any other Python application with an icon (not just Spyder), then all the following executed Python programs will show the same icon as the first application in task bar.
For example, try running a simple Python interpreter (python.exe).
Then open Spyder: you'll see that Spyder has the same icon as the Python interpreter instead of its own icon.
This is a real problem but I don't know if there is a solution available out there.
This would require further investigations.

From kempson.chen on 2014-07-03T02:20:06Z
This is an Win7/win8 issue.
On my case, my spyder did not have its icon,just using the default icon of windows exe.
After a long search, I found it was set at spyder.py with Qt:
self.setWindowIcon(get_icon(icon_name, resample=os.name != 'nt'))
and I also found that it can be solve with setting "AppUserModelID" which is a win7 features http://stackoverflow.com/questions/1551605/how-to-set-applications-taskbar-icon-in-windows-7 There's a Qt package do all the win7 taskbar feature. http://www.strixcode.com/q7goodies/ I assume it can be done without using the q7goodies package, thought I have no QT experience and can not figure out how to code that.
All I can do is issue to this issue. :-(

From<EMAIL_ADDRESS>on 2013-09-25T16:11:39Z
This issue can be closed.
After deleting the shortcut to Spyder (full) that was somehow automatically generated on my desktop from a previous installation, this issue resolved itself. I am no longer getting the spyder icon in my toolbar for any python application  I start - I now get the Python icon.
I installed spyder 2.2.4 using pip install, after removing older version with Windows uninstall utility from control panel, and making sure there were no more lingering spyder registry keys. I did not run spyder_win_post_install script. Happy!

From ccordoba12 on 2014-07-13T10:06:03Z
Summary: Spyder is not getting its taskbar icon right   (was: spyder hijacks python (or other) taskbar icon)
Labels: MS-v2.3.1

From ccordoba12 on 2014-07-20T10:33:13Z
Summary: Spyder is not getting its taskbar icon right in Win 7/8   (was: Spyder is not getting its taskbar icon right)

From<EMAIL_ADDRESS>on 2014-09-24T01:07:11Z
This fix causes a new issue in Windows 8 (probably also windows 7, but I haven't tested)
If you pin the Spyder icon to the taskbar, and launch an instance, it shows up as a new icon beside the pinned one (as shown in my attached file).
If I revert the changes in the revision, this issue goes away and the previous (good) behaviour returns (that windows recognizes the pinned icon and the running program as the same thing, thus only one icon)
Attachment: SpyderIcon.PNG

From<EMAIL_ADDRESS>on 2014-10-02T14:04:44Z
I ran a "conda update anaconda" on 10/1/2014 which introduced a similar problem to the one described.  When I launch spyder in my task bar, a separate generic windows .exe icon appears and is used instead of the spyder icon.   "ccordoba12" posted a link to the revision (f2f29244c2c5) of "spyder.py" that apparently fixed this issue.  However, the anaconda update i ran had already installed this revision.  Replacing the "spyder.py" file with the older revision (71f993092dab) of "spyder.py" fixed the icon problems I was having.  So it seems that the new version of "spyder.py" actually introduced this problem, and reverting to the old one fixed it (at least for me).

From ccordoba12 on 2014-07-20T10:50:29Z
This issue was closed by revision f2f29244c2c5 .
Status: Fixed


Release v0.2.1

Codecov Report

Merging #11 into master will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff           @@
##           master     #11   +/-   ##
======================================
  Coverage    0.63%   0.63%
======================================
  Files          20      20
  Lines        2221    2221
  Branches     1062    1062
======================================
  Hits           14      14
  Misses       2181    2181
  Partials       26      26

Continue to review full report at Codecov.

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


window title is not changing to new settings.

window tile is not changing to new settings.

VSCode Version:
Version 1.24.0-insider
Commit 912fff91ac4ff1bff9695e8da69abd729f7ae956
Date 2018-05-18T05:17:26.925Z
Shell 1.7.12
Renderer 58.0.3029.110
Node 7.9.0
Architecture x64

OS Version:
Windows 7

Steps to Reproduce:

Open users settings file.
Enter "window.title": "${rootPath}${separator}${activeEditorName}${separator}${appName}"
Restart VS Code

@lion444 I cannot reproduce this problem with VS Code 1.24.0-insider on Windows 7.
And for me the window title change does not even require a "Restart VS Code":

Do you still see the problem when disabling all extensions?

Let me try disabling extensions.

disabled all extensions. Here is my settings. 'test' does not show:
{
"workbench.colorTheme": "Visual Studio Light",
"window.title": "test",
"terminal.integrated.shell.windows": "C:\Windows\Sysnative\cmd.exe",
"typescript.check.tscVersion": false,
"window.zoomLevel": 0,
"extensions.ignoreRecommendations": false,
"git.autofetch": true,
"azure.resourceFilter": [
"b4dfda2a-75c5-4351-a2e6-67183642ee8b/1b3b4c66-fda1-4ea6-a02c-7c22c01d4508"
],
"gitlens.gitExplorer.view": "history",
"gitlens.keymap": "alternate",
"gitlens.advanced.messages": {
"suppressCommitHasNoPreviousCommitWarning": false,
"suppressCommitNotFoundWarning": false,
"suppressFileNotUnderSourceControlWarning": false,
"suppressGitVersionWarning": false,
"suppressLineUncommittedWarning": false,
"suppressNoRepositoryWarning": false,
"suppressResultsExplorerNotice": false,
"suppressShowKeyBindingsNotice": true
},
"gitlens.historyExplorer.enabled": true,
}

hmm. it just now started working. No idea why, but thanks for the input.


where can i find the prometheus_oracle_exporter executable

Hi,
I'm not able to find a suitable binary to run it on mac or on docker?
Please help!!
Thanks in advance.

You can compile it yourself for mac. Maybe in the future i will make a dockerfile.


about coco 2017

the training set here is called 'train_val_minus_minival_2014.json'. But the paper reports the result on COCO2017. How can I train it on COCO2017? Thank you!

Hi, please use the MMPose codebase with their pre-trained model zoo for COCO-2017


Maestral can't find my linked account

Describe the bug
I set up maestral per the instructions here https://maestral.app/docs/installation using a virtual environment. The account linked just fine and it was synchronizing my Dropbox. I also set up autostart. When I tried to reboot, however, if I type "maestral status" it says maestral is not running. If I type "maestral config-files" it shows this:
Config name  Account                  Path
maestral<EMAIL_ADDRESS>/home/justin/.config/maestral/maestral.ini
If I try to run "maestral start -c maestral" it tries to link an account, even though it is already linked.
If I run "maestral history" it says there is no account linked.
If I run "maestral auth unlink" it is says there is no account linked.
I tried to re-setup my account and it created a duplicate ini (maestral.ini.ini) without saving the email address. I ended up just deleting the ini.
I would like to uninstall it and completely clean it up and try to reinstall without a virtual environment, but I am not sure how to do so.Config name  Account                  Path
maestral<EMAIL_ADDRESS>/home/justin/.config/maestral/maestral.ini
To Reproduce
$ python3 -m venv maestral-venv
$ source maestral-venv/bin/activate
(maestral-venv)$ python3 -m pip install --upgrade maestral
$ python3 -m pip install --upgrade maestral[syslog]
dnf install python3-systemd
[edited ~/.config/maestral/maestral.ini to use /home/justin/Dropbox as folder]
maestral start
maestral autostart -Y
[I left folder synchronizing for several hours, then saw it had a done status]
rebooted system
I could not get maestral to work again.
Expected behaviour
I expect on reboot it should automatically be ready to sync files saved on another system, or changes on this system to be uploaded.
I also expect I should be able to bring it up manually by maestro start.
I also expect that it should see my linked dropbox account.
System:

Maestral version: 1.9.4
Python version: 3.9.18
OS: Rocky Linux 9.4
Desktop environment: KDE Plasma 5.27.11-1.el9
PyQt version (for Linux GUI): says it is not installed, but I am not using GUI

Additional context

I was able to get this to work. I deleted the ~/.config/maestral, ~/.local/share/maestral, and ~/maestral-env folders.
I then reinstalled from scratch. This time I noticed there were errors because I did not have systemd-devel and systemd-journal-remote installed. Once I did that maestral[syslog] installed correctly. (I didn't notice the errors last time.)
I think another issue was caused by using the command directly to link account which skips steps. When I just used start it did everything. I think this step should be explained as part of the installation.
I also didn't understand the python3 -m venv maestral-venv command. I had to do some research. I am not used to using python virtual environments. I had called it multiple times. # follow steps to link account through web page


Can't reload page using await t.eval(() => location.reload(true)) in TestCafe (v0.22.1-alpha.2)

Are you requesting a feature or reporting a bug?
Regression bug.
What is the current behavior?

Page reload doesn't work and browser hangs. TestCafe leaves control at location.reload(true) or location.reload(). Tried in Chrome and Firefox both but no luck. I re-installed node modules so no caching issue, using latest packages.

What is the expected behavior?

Page reload should be working.

How would you reproduce the current behavior (if this is a bug)?
await t.eval(() => location.reload(true));
It was working fine with 20.0 version for me.
   1) TypeError: Cannot read property '0' of null

      Browser: Chrome 69.0.3497 / Windows 7 0.0.0

         266 |        this._handlePlainObject(val);
         267 |};
         268 |
         269 |DecodingTransformer.prototype.transform = function () {
         270 |    this.visitedRefs[0] = true;
       > 271 |    this._handleValue(this.references[0], this.references, 0);
         272 |
         273 |    return this.references[0];
         274 |};
         275 |
         276 |

         at DecodingTransformer.transform (c:\git\repo\node_modules\replicator\index.js:271:38)
         at module.exports.Replicator.decode (c:\git\repo\node_modules\replicator\index.js:559:24)
         at ClientFunctionBuilder.decode [as _processResult] (c:\git\repo\node_modules\testcafe\src\client-functions\client-function-builder.js:142:32)
         at _processResult (c:\git\repo\node_modules\testcafe\src\client-functions\client-function-builder.js:137:25)
         at $$testcafe_test_run$$2PLNpRhxU$$ (c:\git\repo\node_modules\testcafe\src\api\test-run-tracker.js:76:16)
         at $$testcafe_test_run$$2PLNpRhxU$$ (c:\git\repo\node_modules\testcafe\src\api\test-run-tracker.js:76:16)

operating system: Windows7
testcafe version: 0.22.1-alpha.2
node.js version: v9.2.0

I've tried to reproduce the problem with following test:
fixture `Fixture`
    .page('http://example.com');

test('test', async t => {
    await t.eval(() => location.reload(true));
});

and it works fine.
Could you please provide a small example to reproduce or direct access to the tested page?

I will close the issue because no activity has been encountered for a long time.
Feel free to reopen the issue if the problem persists and you can provide an example to reproduce it.

Yes, i would like to really reopen this bug because reload problem is still exists and really affecting us to upgrade. Sometime it works fine 2 out of 10 attempts with one browser instance ONLY, when try with 10 instances it always fails for me. I even tried https://stackoverflow.com/questions/39599736/window-reload-doesnt-reload-my-page but no luck and TestCafe hangs at reload page and takes forever. After manually reloading page, it shows below error assuming TestCafe was hanged there?
await t.eval(() => window.location.href = window.location.href);

DecodingTransformer.prototype.transform = function () {
         270 |    this.visitedRefs[0] = true;
       > 271 |    this._handleValue(this.references[0], this.references, 0);
         272 |
         273 |    return this.references[0];
         274 |};

I'm quite sure something is wrong in this area and it blocks us to upgrade TestCafe v0.23.2. We didn't upgrade TestCafe just because of this reason since reported in October and previous release was certainly fine. Any solution/work around please?

Wow, it worked with latest alpha version (0.23.3-alpha.2) instead of last stable release (0.23.2). Thanks @LavrovArtem for fixing this and i can see stability back. Awesome, i can go ahead and upgrade now.

Thank you for the update. This is great news! I'm happy to hear that the issue is resolved.


Next-/previous-buttons in E-Mail view no longer work properly (since Update 5.206)

Expected behavior
K-9 Mail should show the next respectively the previous message if the next-/previous-button in the E-Mail view are pressed (as in the previous versions of K-9 Mail).
Actual behavior
Since the latest update which has been released a few days ago, the next-/previous-buttons on the bottom in an opened E-Mail stopped working correctly.
For example I've got 3 unread messages as well as 2 already read emails in my inbox listed in this order:
Message #1: Unread
Message #2: Unread
Message #3: Unread
Message #4: Read
Message #5: Read
If I open message #2 and press the "previous" button, K-9 Mail opens E-Mail #4 instead of E-Mail #1.
If I press "next" on message #2, E-Mail #4 is opened instead of E-Mail #3.
In the current version, I have to go back to the overview of all E-Mails and then select the next unread E-Mail manually.
Before the update, I could go through all unread mails comfortably by using the next-/previous-Button.
Steps to reproduce

Mark some E-Mails in your inbox as unread and be sure to have some unread E-Mails.
Open up an unread E-Mail and click either the next- or previous-button
K-9 Mail jumps to the first already read E-Mail instead of going to the actual next / previous E-Nail.

Environment
K-9 Mail version:
5.206
Android version:
6.0.1
Account type (IMAP, POP3, WebDAV/Exchange):
IMAP

I believe this is no longer an issue.


Clean up tmp files for interrupted docker build

- What I did
This fix tries to fix the issue raised in #29486 where interrupted docker build leaves some tmp files in /var/lib/docker/tmp, with tmp file name prefixed with /var/lib/docker/tmp/docker-builderXXXXXX.
The reason for the issue is that in MakeTarSumContext():
	if err := chrootarchive.Untar(sum, root, nil); err != nil {
		return nil, err
	}

the err is shadowed and caused the clean up function in defer func() not being called.
This fix fixes the issue.
- How I did it
- How to verify it
This fix is tested manually, as was specified in #29486:
rm -rf /var/lib/docker/tmp

mkdir repro && cd repro
fallocate -l 300M bigfile

cat > Dockerfile <<EOF
FROM scratch
COPY ./bigfile /
EOF

docker build .
{Cancel}

ls -la /var/lib/docker/tmp

- Description for the changelog
- A picture of a cute animal (not mandatory but encouraged)

This fix fixes #29486.
Signed-off-by: Yong Tang<EMAIL_ADDRESS>

@vieux erm, I know we want to reduce the cherry-picks, but with the focus on docker prune and data-management, I think this makes sense to include; looks like zero-risk to me

All green, merging


Fragment destruction

I'm using this library with three fragment. It seems that the first fragment is recreated (losing values etc...) if I return back from the third to the first fragment. Is there a way to prevent the fragment to be destroyed?
I found this on StackOverflow: mViewPager.setOffscreenPageLimit(2);. Is there a way to implement this solution?

Hi, the fragment being recreated is something the ViewPager does so that it doesn't keep too many fragments in memory at once. Please note that going back from the third stp to the first one is not the only scenario in which the first fragment will get recreated. This could also happen if e.g. you rotate the device or your app gets killed by the system and you go back to it.
The recommended approach here would be to save the data in a saved instance state. You could check out the sample app for an example: https://github.com/stepstone-tech/android-material-stepper/blob/master/sample/src/main/java/com/stepstone/stepper/sample/step/StepFragmentSample.java

Ok, but the problem that I'm facing isn't only the lose of values. The main problem is that in the first fragment, I use a library to pick an image and then put it in an ImageView, but when I return back from the third to the first fragment, and I try to pick the image, the app crashes because giving me error in onActivityResult: java.lang.RuntimeException: Failure delivering result ResultInfo{who=null, request=1, result=-1, data=Intent { (has extras) }} to activity. The library that I use in the first fragment is the following: https://github.com/jdamcd/android-crop. However I don't think it's a problem related to the library, because I changed it and the same problem occurred again...

Hi, I don't really know this library and without seeing the code there's actually very little I can do. Anyway, I believe the same issue might arise if the fragment was recreated in different scenarios mentioned by me earlier.
Are you sure you're using the library correctly, passing the right values as parameters, etc? Maybe this will help: http://stackoverflow.com/questions/20827237/failure-delivering-result-resultinfowho-null-request-1-result-1-data-intent ?

I think that I'm using it correctly, to start the crop activity. The library has the method startCrop(Context) or something like that, to start cropping the selected Image. But in a fragment this method is .startCrop(getActivity(), StepFragment.this). And it work until the fragment has been destroyed.

It looks lile an issue with the library to be honest or the the setup. The ViewPager in our library doesn't do anything special... Maybe this has something to do with https://github.com/jdamcd/android-crop/issues/240 ? Does the crash occur if you try to start other activities and get the result from them?

No, I get the crash only if I use this library, but the problem occur also with another library. This is why i think it's not a library issue

Could you try it? Maybe I'm doing something wrong...

Hi, I'm currently on my holidays and I only have access to my phone :/ maybe you could fork this library and add the library you're using to the sample app so that I could have the look at the code at least?

Yes, I'll try. Keep in touch

Could you tell me the class where I can find the ViewPager? I mean the class in which you get it with .findViewById

This would be https://github.com/stepstone-tech/android-material-stepper/blob/master/material-stepper/src/main/java/com/stepstone/stepper/StepperLayout.java

Is there a way to access the ViewPager from my activity?

I want to try to use mViewPager.setOffScreenPageLimit(int) without modifying the library

Technically speaking the ViewPager is a view so you can find it and use it in the activity the same way it is found and used in the StepperLayout?

Solved. Using mViewPager.setOffScreenPageLimit(int) solved the problem. As I thought it wasn't a problem related to the library. Thank you for your patience @zawadz88 :+1:


[KYUUBI #2870] Fix sf0 query error in TPCH

Why are the changes needed?

fix #2870
How was this patch tested?

[x] Add some test cases that check the changes thoroughly including negative and positive cases if possible

[ ] Add screenshots for manual tests if appropriate

[x] Run test locally before make a pull request

Thanks, merging to master


meta.other.macro highlighter class

It seems that the meta.other.macro highlighter class is not implemented. This is causing preprocessor directives in C not to be highlighted.

On second thought, I decided to edit the theme on my side as this is likely a preference and not a bug.


Please correct usage

At this anchor (#discover-aliases-for-resource-properties), please correct incorrect usage of contraction for "it is" by replacing with the possessive "its."
Grammar 101.

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

ID: 331fe5bb-eb50-3603-c2ab-748f852f9729
Version Independent ID: 856fd88a-7bc9-ecb4-95d6-a43f6b75fda8
Content: Azure Policy extension for Visual Studio Code - Azure Policy
Content Source: articles/governance/policy/how-to/extension-for-vscode.md
Service: azure-policy
GitHub Login: @DCtheGeek
Microsoft Alias: dacoulte

#assign:DCtheGeek

@yobyot Thanks for catching this and pointing it out.  I'll get it fixed right now.  As soon as the fix merges, this issue will automatically be closed.
@MicrosoftDocs/azure-cxp-triage Please label 'doc-enhancement' and #in-progress.


Support graceful stop with configurable timeout

Feature Request
It would be great if we could support a graceful stop/shutdown operation that would wait for currently in progress reconciliation for a user configured timout period.
During the shutdown we should not trigger new reconciliation.
This would also allow a smoother operation during upgrades, or other pod restarts.

Unless I'm mistaken, we already have something like this: https://github.com/java-operator-sdk/java-operator-sdk/blob/main/operator-framework-core/src/main/java/io/javaoperatorsdk/operator/api/config/ConfigurationService.java#L126, which you can configure using: https://github.com/java-operator-sdk/java-operator-sdk/blob/main/operator-framework-core/src/main/java/io/javaoperatorsdk/operator/api/config/ConfigurationServiceOverrider.java#L63 prior to starting your operator.

Ahh, yes, but for this I think the shutdown hook is needed. That is deprecated. Probably it should not be then.

Also would be good to have an integration test for this.

You need a shutdown hook to be able to handle graceful shutdowns, though, don't you?


[RUST][FRONTEND] refactoring builder to use a derive

Use derive_builder for tvm_frontend::Function for less boilerplate codes. Proper config is required to make the return buffer accepts at most one value. TODO #2292.

@nhynes don't think #[derive(Builder)]] can add the proper fields in builder struct needed for Function. It is merely for very common type of builders. Currently, derive_builder only uses the fields of the underlying struct, but the signature of the current function::Builder is
pub struct Builder<'a> {
    pub func: Option<Function>,
    pub arg_buf: Option<Box<[TVMArgValue<'a>]>>,
    pub ret_buf: Option<Box<[TVMRetValue]>>,
}

with completely different fields, in particular arg_buf and ret_buf. So my conclusion is that the derive_builder is not mature enough to handle our case.
Side note: In my code, I've pretty much followed std::process::Command construct.

Discussed here and will plan for runtime-frontend compatibility.


Add import switching from typing_extensions to typing when reaches requisite version.

typing_extensions backports a lot of code from typing. It would be great if ruff could switch the import from typing_extensions to typing when ruff's target version matches (or exceeds) the requisite python version.
E.g.
from typing_extensions import Annotated

Becomes
from typing import Annotated

when ruff's target version is 3.9 or greater.
This feels like a pyupgrade code?

I think this does exist as UP035: https://beta.ruff.rs/docs/rules/deprecated-import/
I tested it locally with check foo.py --fix -n --select UP035 --target-version py39 and it did rewrite that import for me. Any luck?

Yes, thanks! I was using an old test repo with target version set to py3.8. Updating that made everything work. Sorry for the noise!


GC: Clear Tombstone state immediately when a Tombstone is revived

Description
Is this a good idea?
Since we changed things to get revival events "in order" in the op stream, we can update state immediately, even in interactive clients.  This was already done for unreferenced state tracking, but not for the Tombstone state.
Before this PR, we would see extra "Tombstone Requested" logs, even when the object was destined to become un-tombstoned in the next GC Run
Reviewer Guidance
I'm not actually sure we should do this!  Tombstone state has always been static for the life of a session.  And until GC runs again, the object "really is" still a Tombstone.  Curious for input.

Closing because I don't think we should do this. Available for discussion though if needed down the line


Remove InlineForm default focus on first input

As InlineForm is mostly used within SidebarPortal focusing on the first input by default would lead to a lot of issue within blocks main area.
The most annoying one is:

Scroll-to-top effect when adding new block with title input within SidebarPortal

@tiberiuichim done


Add type annotations to the remainder of edb.schema

The work on the generic parts is mostly done (in #693), and it's now a matter of typing the concrete implementations of schema objects.

This is done.


RFC: UTF-8 Support in EasyXML

I've created a wiki page with this proposal and request for comments (RFC).
https://github.com/JKISoftware/JKI-EasyXML/wiki/RFC:-UTF-8-Support-in-EasyXML-%236
Please feel free to discuss the RFC here in this issue/thread.

I've done some work on this in a feature development branch:
https://github.com/JKISoftware/JKI-EasyXML/tree/%236-UTF-8-Support-in-EasyXML
There's a package for testing here:
https://github.com/JKISoftware/JKI-EasyXML/releases/tag/<IP_ADDRESS>-alpha

I've created another alpha release with UTF-8 encoding as default:
https://github.com/JKISoftware/JKI-EasyXML/releases/tag/<IP_ADDRESS>-alpha


Create a limit to how many km a person can travel

I'm submitting a feature request.
What feature do you recommend to add?

Add a limit to how many km a person can mention in the travel distance section:

Round the number of digits after decimal point:

What is the value of this feature?

Reduce the chance of making a typo with extra digit which could cause inaccuracy in data calculation.

Solution

[ ] Limit the number to 2 whole digits (or 99 km?);
[ ] Limit the number of digits after decimal point to 2;

 [DEV]: Limit number of km in the input box


x/exp/vulndb/govulncheck: test fails when "unzip" is not in PATH

golang.org/x/exp/vulndb/govulncheck is persistently failing on the longtest builders since CL 329809:
--- FAIL: TestKubernetes (7.89s)
    main_test.go:300: failed to build k8s:
    main_test.go:301: exec: "unzip": executable file not found in $PATH
FAIL
FAIL	golang.org/x/exp/vulndb/govulncheck	40.964s

It probably needs a t.Skip if exec.LookPath("unzip") fails. (Compare golang.org/x/tools/internal/testenv.NeedsTool.)

(This is a release-blocker for Go 1.17 via #11811, but I don't expect that it will take much effort to fix.)

I have created a small CL that should just make the failure go away. Will follow it up with a solution using standard libraries.


Need your help asim

Need help with Laravel Sanctum here: https://stackoverflow.com/questions/65051816/laravel-v8-15-0-php-v7-3-23-an-api-for-mobile-application-on-a-shared-server-p

This project is not related to Laravel Sanctum, you are probably looking at the wrong code


[ep-matrix] [epcon] Integrate Matrix.org auth with the EuroPython website

See https://github.com/EuroPython/epcon/pull/1393 for a POC.
This should make sure the following things work:

automatic user generation in Matrix Synapse
adding user to rooms in Matrix, based on tickets purchases
setting up Matrix role / powerlevel of the user based on Django admin settings

I'm working in this one.
power levels are going to be configured based on tickets, is_staff and is_speaker
related to #1397  https://github.com/EuroPython/ep-matrix/tree/epcon_auth/epcon_auth_provider

Right now we have the ability to handle staff, speakers, sprinters and attendees. We need to add training participants and sponsors. We also need to set the power levels per room, not just invite users to the relevant rooms.

Closed by development of epcon auth provider and auth api in epcon website


fix(reference-guide/controller-api): v1->v2

The API is now v2, but docs still reference v1.
example:
$ curl -H "Content-Type: application/json" -X POST \
  -d '{"username": "admin","password": "admin","email"<EMAIL_ADDRESS>http://deis.<IP_ADDRESS>.nip.io/v2/auth/register/

Nice catch! :+1:


Server crash with MFR Harvester+Resourcefulcrops

This doesnt happen everytime, we've seen this crash about once a day now since the server has come up, enough to no longer consider it a fluke. Here is the full crashlog:
http://pastebin.com/J2cEvaFA
It seems to crash when it's being harvested by an MFR harvester.
ResourcefulCrops-1.7.10-1.0.0-18 with
MineFactoryReloaded-[1.7.10]2.8.1B1-113
edit; crash has also been reported with enderio farming station when the tool runs out, possibly related to https://github.com/TehNut/ResourcefulCrops/issues/4 but not entirely sure if these are separate issues or not.

Hm... I'll play with it when I get home
EnderIO crash is also line 84 of BlockRCrop?

Yes also line 84 with the farming station

I'm just going to remove the ability to make "nether" seeds for now. It was a very poor implementation.


Minor/trivial message correction

The system is filtering out UTXOs bigger than 1000 sats when starting a transaction. Upon seeing no such UTXO in a given address, it would prompt the user:
No dummy UTXOs (i.e. value < 1000) found for your address, so you first need to create one

Strictly speaking this is not completely accurate and is a tiny bit misleading. "value < 1000" should really be "value <= 1000". This is a correction for this tiny problem, and hope it make sense for you guys...

Apologies for the change at the end of the file, it was automatically created by Github itself, perhaps due to some EOF preference Github has. Feel free to undo that, or let me know if you guys want to keep the source code clean and I can get right of that too. Or it might not be a big deal in the grand scheme of things.
You guys are doing a great job BTW, kudos...

Thanks!


UX: Make "+" more obvious (was: Volto editor first time use, adding images is difficult)

I tested the Volto editor last week as a complete newbie: I didn't look at any screencasts the last few months or documentation so I was as 'blank as possible'.  I first played around on the Volto demo site and later installed Volto with locally because I really thought at first something was wrong with the online demo site.
This is a 'curse of knowledge' issue: once you know how it works, the problem is gone. And you want to forget as soon as possible that you didn't understand something so obvious and simple from the start....
Inserting an image is not logical:  I couldn't figure out 'when' the (+) sign appears next to text. At first I tried to to modify an existing page (the homepage) and couldn't insert any image, but you have to know you first have to go into the last text block at the end and then press enter to create a new empty text block and only on an empty block there's the option to convert it to other media. duh.
From just visually inspecting the page, checking menu's or typing text in existing blocks you get no visual feedback at all this is "the way".

One easy way to fix this would be to always keep an empty block with the (+) symbol once you are in editing mode.
If you create a new page first instead of modifying the homepage,  the (+) symbol is there, but even then I succeeded in wandering around Volto for 10 more minutes because my first response was to fill the element with text and feeling silly wondering where the (+) button went.
Having an insert something option at the bottom of the editor would also have made the next task easier: Once I add an image block, I couldn't start a new text block below it, except for:

first creating an empty text block above the image,
then first type some text in it because otherwise there's no drag handle
and then dragging it below the image
now delete or replace the text.

which feels like a work around.

fyi: medium shows the plus symbol right from the start.

Follow up/duplicate in #1272

@fredvd tbh, I think this issue is more severe than #1272. #1272 is an edge case if your browser window is too narrowed. This is a UX issue that every single Volto user will run into. In addition, this one is fixable without going Quanta "all-in".
There are small amendments that would help:

increasing the contrast of the "+" icon
moving the "+" icon to the right inside the block (more like Gutenberg)

implementing the Quanta "+" icon (move it below the current block, I am actually not sure how Quanta is supposed to behave when the user clicks into the first text block):

Maybe @sneridagh can shed some light into the current discussion with Albert on that topic.

@tisto You're right.  I was searching for one of my issues in the repo this weekend. I saw other issues I started and tried to do some cleanup for things I submitted already long ago that were already fixed or in progress. But I didn't reread these carefully, these are indeed different situations.
I chatted with @sneridagh in Sorrento about this issue & Quanta and he mentionned the (+) issue was going to be solved in Quanta because of a different location. But this issue is about the (+) for example not appearing at all at the end of the page until you've created an empty text block there.  So as a first time user you have no visual clue on how to continue.
@sneridagh Having a + in the toolbar (from what I understood from you is one of the changes in Quanta, but we only chatted briefly) will fix the 'intra' block insert on other places on the page and #1272 , but still not solve the append at the end of the page.
Allthough you could argue that if you select the last block the + will also appear there, so it's one action less  (now it's select the block, enter to create a new empty text block and you see the +.   Having a (+) always visible is IMHO the best solution, but it will stay a balance between 'clean' and 'obvious on how to use'.

I came across this issue because I couldn't work out how to insert into an existing home page. It might be a bug but from above it seems like there is no concept of inserting into a middle of a page?
I would expect to see a + next to the tag or at the right. Notion has it on the left and allows inserting below.

@djay It's unintuitive. Two ways to do it:

either create the image at the bottom of the page and drag it up in the middle
select the block above the "future place of your image" and hit Enter, which should insert a new block at that position

@tiberiuichim ok. in my particular case this seems to be a bug in the theme thats preventing the empty block from appearing as you can see

However I think that @fredvd pointed out this is still not a good design decision. Essentially what the + button currently is a "turn into" button rather than an insert and whats more it's a special case that only applies to this special empty block at the bottom. In addition there seems to be this insert action (pressing enter) that has no corresponding visible button.
I know I'm coming to this this late but I'm going to get my developers to work on a patch as I believe this is issue is important than it might seem and should be tried to be fixed for plone 6.
What I propose is

put a [+] next to the drag tag like in notion. This is an "insert after" button. It lets you pick what you will insert after.

we can also put this button into the settings but I don't think putting it there alone is going to be enough. Users will look at the block itself on how to insert something before or after it. they are less likely to look at the settings and def not likely to look at the bottom of the page.

there won't be an automatic empty block at the bottom of the page if the page already has blocks
for a empty page we will have an empty text block rather than an this special empty block. so it's [+] will be an insert after. so if you want say and image on the page with no text then it would be two actions. insert image below and then delete the unwanted text block. but I think that's an acceptable tradeoff and easier to understand
the enter key is an insert text block below
later we can support the "turn into" function in addition. This is something that notion does well. They have a menu on the drag button which lets you turn certain kinds of blocks into others, like changing header sizes. they also support / commands to turn a empty  block into something else or insert a new block below if not empty

@tiberiuichim ok. It's not a bug in our particular theme. rereading the original issue and trying it out on default volto the problem is that the only way to insert a block onto an existing page is to press enter on the block above. and if its text block it has to be enter at the end of the paragraph. The chance of a new user discovering this is very unlikely so as @tisto pointed out, this is not a niche problem.

FWIW, I think the Quanta implementation that @tisto laid out would mostly solve this issue, but I don't agree with a couple of points. I would like to see the following.

+ button is displayed on all blocks. This would allow adding or inserting a block anywhere and make it a little more obvious.
Add alt or title text to the + button that says "Add or insert block after this block (Enter)". This will help educate newbie users, making it faster for them to edit pages.
Keep the + on the left side to avoid conflicts with the Trash icon and accidentally deleting a block. Destructive actions should always be isolated in the UI.
Update the Block menu of shortcuts to include "Tab or Shift-Tab to navigate between blocks" and "Enter to add or insert a block after the current block".

Check out how Squarespace does it. I have used it for some years and find it very easy to use.

It has a plus button all the time on the top right. If you click it, it adds a new block to the bottom of the page.

And when you hover over page elements, its shows where you can add new blocks directly with + button.

When just looking at the page it does not show the hovered pluses, making the view easier to the eye. Showing all the pluses all the time would make the UI cluttered, I think.
Quanta looks nice with the improved + button.


track scroll position without scroll listener

Supersedes #3936 (which stores scroll state in history.state) and (I think) #3873.
Instead of calling replaceState frequently as a result of scrolling, this updates scroll position when we navigate or leave the page. It does so by storing scroll_positions in memory and persisting it to sessionStorage in beforeunload (which allows us to recover scroll positions when the page is reloaded without needing to manipulate history).
Contra #3936, I think it may be possible to add tests that use page.reload() — will investigate. Added a test, which makes me feel much more comfortable about this change.
Thanks to @PH4NTOMiki for pointing in the direction of using sessionStorage rather than history.state.
Please don't delete this checklist! Before submitting the PR, please make sure you do the following:

[x] It's really useful if your PR references an issue where it is discussed ahead of time. In many cases, features are absent for a reason. For large changes, please create an RFC: https://github.com/sveltejs/rfcs
[x] This message body should clearly illustrate what problems it solves.
[ ] Ideally, include a test that fails without this PR but passes with it.

Tests

[ ] Run the tests with pnpm test and lint the project with pnpm lint and pnpm check

Changesets

[x] If your PR makes a change that should be noted in one or more packages' changelogs, generate a changeset by running pnpx changeset and following the prompts. All changesets should be patch until SvelteKit 1.0

It might be worth investigating if visibilitychange events with fallbacks are still preferred over plain beforeunload.
As for sessionStorage, what does it give us over history.state? I didn't catch the issues that was trying to solve, but browsers save history and scroll restoration across sessions natively, so maybe localStorage with some long-tail invalidation would match it better?

It might be worth investigating if visibilitychange events with fallbacks are still preferred over plain beforeunload.

Based on my reading of that article, I think we do want to use beforeunload, since the problem we're trying to solve is what happens when you do Cmd-W followed by Cmd-Shift-T. It looks like visibilitychange is unreliable for that.

As for sessionStorage, what does it give us over history.state?

Before this PR, we were replacing the current state after every scroll. Instead, we're now tracking scroll position when navigation occurs from a to b. In the case where navigation occurs as a result of an intercepted link click, history.state belongs to a, but when it occurs as a result of popstate it belongs to b instead — we would be replacing the wrong state. By using sessionStorage, we can reliably update the scroll state belonging to a.

Nice! I think the explanation from your last comment could go in a code comment because I had the same exact question (https://github.com/sveltejs/kit/pull/3873#discussion_r807184207) on the PR that inspired this one

Based on my reading of that article, I think we do want to use beforeunload, since the problem we're trying to solve is what happens when you do Cmd-W followed by Cmd-Shift-T. It looks like visibilitychange is unreliable for that.

I've looked a bit more and those marked as (bug) were eventually fixed, and the spec has been updated to reinforce the behavior, so unless other bugs have been reintroduced that should still be more reliable on browsers from 2020 onwards (when the fix landed in webkit). I mentioned fallbacks for legacy browsers, but that could be a differential builds concern.
For what it's worth, I just tested locally and it was triggered apparently reliably, although I don't have access to many devices (win, android) and didn't check all use cases (I don't know all the different ways to close/sleep an app on Android, for instance).

In the case where navigation occurs as a result of an intercepted link click, history.state belongs to a, but when it occurs as a result of popstate it belongs to b instead — we would be replacing the wrong state.

I see, that makes sense, then!

I'll confess I don't totally understand what the difference is — what cases would visibilitychange catch that beforeunload wouldn't?

I added an explanatory comment in d538ac6d3491197f27568005048642be999c761d

I'll confess I don't totally understand what the difference is — what cases would visibilitychange catch that beforeunload wouldn't?

 Last I checked, however, killing the browser afterwards (eg. from the task switcher or some device health monitor) won't guarantee beforeunload gets called.
Force-quitting might be rare on a desktop, but it's surprisingly common on battery-restricted devices, though I'm not sure how aggressive devices/os versions are on average.

Awesome stuff @Rich-Harris definitely supersedes my PR, much cleaner than my PR, actually one of my first commits in that PR was using variable approach(like you are doing in this PR, and Remix is doing almost the same) and only storing the states in beforeunload, but I made it too complicated, so I dropped the idea, and went for the individual stores(like Next.js does),
Few things I want to point out:

You should call scroll_positions[this.current_history_index] = scroll_state(); in that hash if block(where we are setting hash_navigation = true) before or after that, because that is specific case, it is navigation, but it's not being handled by _navigate method(the only place you're saving scroll state, other than beforeunload but that's irrelevant to that.)
I think you should move that const scroll_positions = JSON.parse(sessionStorage.getItem(SCROLL_KEY) || '{}'); that's currently outside the Router, to the inside of it's constructor, because it's safer, and we lose nothing by doing it.
We should wrap those 2 calls to sessionStorage (in beforeunload and that getItem) with try/catch block, that fails 1 in a million, but still why not do it.
I don't know why you're doing that history.scrollRestoration = 'auto'; inside beforeunload block. It's kinda pointless since we're storing value to return to, and we are doing it(line 76-77).

Thanks, I made some updates:

now using visibilitychange
updating scroll positions when clicking a hash link
put sessionStorage read/write inside try-catch
added types

Some of the other stuff is deliberate. Resetting scrollRestoration to auto in theory means that scroll will be recovered on page reload before hydration kicks in. I haven't been able to verify that it actually works, but as in other cases related to scroll handling I think a conservative approach is preferable.
Setting scrollRestoration to manual should happen inside init_listeners, because we don't want to use it on pages that don't use the client-side router.


Attempting pr #38 (again)

If github doesn't work for me this time, I'm going to give up the ghost!

Dead.


Added the changing shape animation

#4843

Close #4843

Checklist:

[X] I have mentioned the issue number in my Pull Request.
[X] I have commented my code, particularly in hard-to-understand areas
[X] I have created a helpful and easy to understand README.md

@MohitGupta121 @TusharKesarwani please review it i have added the chaning shape animation

@MohitGupta121 @TusharKesarwani please review it

@MohitGupta121 @TusharKesarwani please review it

@MohitGupta121 please review it also

@MohitGupta121  @TusharKesarwani  please review it asap


Add Graceful Shutdown with NodeJS and Terminus

Hi,
I'm Marcos Henrique and I'm a software engineer at Dasa
I though this small article about graceful shutdown might be a good contribution to your list.
Thanks for considering it!

The article looks good, however, GoDaddy is a shitty company and I don't want to link to anything related to them.
I'm happy to consider another article written for a different process manager like PM2.


(PDB-2565) Change =-after? to assert-equal-after

This wraps the entire macro in an is assertion. Since all cases where this was
used did this anyway, it provides more convenience. However, the main reason was
to ensure that the differential between the old and new values actually gets
printed during testing instead of simply returning the expectation of being true.
Signed-off-by: Ken Barber<EMAIL_ADDRESS>

Needs to be merged in-sync with this: https://github.com/puppetlabs/pe-puppetdb-extensions/pull/173

Refer to this link for build results (access rights to CI server needed):
http://jenkins-enterprise.delivery.puppetlabs.net/job/enterprise_puppetdb_init-multijob_githubpr-stable/162/
Test PASSed.

Refer to this link for build results (access rights to CI server needed):
http://jenkins-enterprise.delivery.puppetlabs.net/job/enterprise_puppetdb_init-multijob_githubpr-stable/163/
Test PASSed.


Carousel does not work

Carousel does not work in Firefox since version 63.0.1

No, I use Windows 7

The problem first occurred with the Firefox Developer Edition,
as there are always updates for this earlier

On mobile Firefox 63.0.2 Carousel v4 is working for me smoothly

I have done some tests on other devices an it works.
So I searched for the difference.
And the difference is: on my notebook i use firefox in the 64Bit version.
Possibly, this could be the problem??

@nettdotkomm I highly doubt 64bit browser version is making a difference here. But you never know...
Do you have you all browser extensions disabled and surfing in private mode just for testing/reference purposes?

@nettdotkomm for best test purposes I know: can you please check your browser console and confirm it's clear when the issues pops up? Because if the issue is in another script that's ABOVE our script, that's your problem right there.

Do you have you all browser extensions disabled and surfing in private mode just for testing/reference purposes?

Yes ... same Problem

can you please check your browser console and confirm it's clear when the issues pops up?

its clear ... no errors are displayed

test it on Firefox Portable 32Bit
same issues
So firefox 64bit is probably not the problem


审核问题

Your app uses the "prefs:root=" non-public URL scheme, which is a private entity.

升级到1.9.0之后的版本：


restructure README

The current README is rather confusing. I took a few minutes to restructure and reword a few parts in the hope of making the installation instructions easier to follow.

This really helped me. Does it still need any help to be merged?


Update example to use friendly error messages

Dearest Reviewer,
Right now if you can not load a file it says something like fopen failed. Adding an expects with which file failed would help a beginner out.
I have also personally changed the output file name to be the {base name of the source}-{base name of the content}-{step_id} to help me save all my amazing images. That can be another change or an exercise for the reader.
Thanks again for the amazing example.
Becker
Changes:
Moved from ? to expect with a message when loading the inputted files.

Merged, thanks for the PR and sorry for the delay.


Using https instead of http on giphy.com

This removes the warning/error on mixed content. That arise when running the demos from the github/README.
Mixed Content: The page at 'https://evancz.github.io/elm-architecture-tutorial/examples/6.html' was loaded over HTTPS, but requested an insecure XMLHttpRequest endpoint 'http://api.giphy.com/v1/gifs/random?api_key=dc6zaTOxFJmzC&tag=funny+cats'. This request has been blocked; the content must be served over HTTPS.(anonymous function) @ 6.html:7188
6.html:7188 Mixed Content: The page at 'https://evancz.github.io/elm-architecture-tutorial/examples/6.html' was loaded over HTTPS, but requested an insecure XMLHttpRequest endpoint 'http://api.giphy.com/v1/gifs/random?api_key=dc6zaTOxFJmzC&tag=funny+dogs'. This request has been blocked; the content must be served over HTTPS.(anonymous function) @ 6.html:7188

don't you think that using // instead (to respect protocol) will be better option?

I think @turboMaCk is right on this one. I'll try to hunt down all the things and use that version.


Use draft in ray tracing and use SSS to calculate Snell's constant

Use draft (which is 4.3m for Deneb).
Jean-Guy indicates that Snell's constant should be calculated with SSS instead of SVP.
We can implement a strategy pattern for ray tracing since it is foreseeable that other strategies could be used (for example launch angle correction, SSS bias, etc.)

Logically, the SSS would be item 0 in a SVP. We could continue to use the SVP-driven logic by simply ensuring that the top element of the SVP's (depth, SS) pairs is (0,SSS). We would need to confirm that the SSS is actually measured at the surface, and not near the MBES, which is a depth=draft+some tiny Z offset.
What do you think ?

The sound velocity probe is installed next to the MBES transducer which is at 4.3m below surface.
The name Surface Sound Speed is an unfortunate misnomer used by manufacturer, they should have used Transducer Sound Speed instead.
Here's a picture of the probe next to the MBES that Jean-Guy sent:

Unless the probe is directly installed at the water line, it won't really be Surface Sound Speed.

I do agree that we could chop the SVP at the draft and inject the SSS at element 0 before doing the ray tracing.

If it's far in the upward direction, then it's irrelevant for raytracing. If it's further down from the transducer, it's a major wtf that's very unlikely to happen. I'd say we could safely assume that it's approximately where the transducer is, AKA depth = draft.

Sounds good to me!


Validation in the form

I am concerned about how these forms fields would be validated. Please help me out with the same.

hi @muskanup let me know your concerns :) you can use standard pattern attr and/or use javascript validation finally you can validate all the fields via the submitCallback attribute


Build on Ubuntu 20.04 instead of latest

Fixes issues with glibc on older (pre-2.33) systems.

e2es pass on my end with properly configured tokens - maybe this one's expired?

Codecov Report

Merging #41 (46f471d) into main (9462826) will not change coverage.
The diff coverage is n/a.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@           Coverage Diff           @@
##             main      #41   +/-   ##
=======================================
  Coverage   52.92%   52.92%
=======================================
  Files          82       82
  Lines        7862     7862
  Branches      628      628
=======================================
  Hits         4161     4161
  Misses       3693     3693
  Partials        8        8

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


docs: update opportunistic caching page

Opportunistic caching is no longer considered experimental as it's
five years old.
I'm not sure if the warnings about RAM over-usage are still relevant, so I left them as is.
fixes: dask/dask#6923

Thanks !


Build fails on armv6, armv7: error: unknown type name 'Elf32_auxv_t'

In file included from /wrkdirs/usr/ports/audio/sfizz/work/sfizz-0.4.0/src/external/cpuid/src/cpuid/cpuinfo.cpp:20:
/wrkdirs/usr/ports/audio/sfizz/work/sfizz-0.4.0/src/external/cpuid/src/cpuid/detail/init_linux_gcc_arm.hpp:43:5: error: unknown type name 'Elf32_auxv_t'
    Elf32_auxv_t auxv;
    ^
/wrkdirs/usr/ports/audio/sfizz/work/sfizz-0.4.0/src/external/cpuid/src/cpuid/detail/init_linux_gcc_arm.hpp:47:41: error: use of undeclared identifier 'Elf32_auxv_t'
        const auto size_auxv_t = sizeof(Elf32_auxv_t);
                                        ^
2 errors generated.

This failure is in src/external/cpuid.
Is it possible to change to use package-installed cpuid?
Version: 0.4.0
clang-14
FreeBSD 13.1

Is it possible to change to use package-installed cpuid?

Apologies for these late replies.
AFAICS there is no cpuid package in major distributions of Linux and FreeBSD, so I have no idea how to identify it with CMake. Note that there is another one: https://github.com/anrieff/libcpuid (currently v0.6.3), but the one we are using is https://github.com/steinwurf/cpuid/ (v8.0.0). I'm going to close these issues. Please reopen if still there.


Another bug about the script of train camvid

While I solve the bug of camvid dataset, I find another bug of the script of train camvid. When running the script of train camvid, it will occur "AttributeError: 'RandomRotation' object has no attribute 'resample' ". Tips: the script of test camvid can run. I don't know how to deal with it.

I have added an exact environment installation yml and updated the installation instructions.
Can you please try again and let me know if it works for you?

@YuvalNirkin  Can you change the exact version of torch and torch vision also ? I got a lot of bugs when running it.Very likely because of mismatch version


Installed Scoped Modules not In Package.json Not Displayed in NPM Node

Steps

Create new blank NTVS console JS app.
In interactive window, run .npm install @types/nodes

Expected
@types/node is listed in the npm solution explorer tree. It should be listed as not included in the package.json.
Actual
Package is installed but not listed

This is fixed in Visual Studio 2017 Update 4


Disable builds on JDK 12 temporarily

Overview
Disable builds on JDK 12 temporarily because Travis
fails to download it.

See: https://jira.bf.local/browse/ECR-XYZ
Definition of Done

[ ] There are no TODOs left in the code
[ ] Change is covered by automated tests
[ ] The coding guidelines are followed
[ ] Public API has Javadoc
[ ] Method preconditions are checked and documented in the Javadoc of the method
[ ] Changelog is updated if needed (in case of notable or breaking changes)
[ ] The continuous integration build passes

Coverage remained the same at 85.924% when pulling d196c2672b65c1b7592a239aaf1263b86ae35bf9 on dmitry-timofeev:disable-12-temporarily into c82e00654ca3662a898c00d5ed00b9088a9496f8 on exonum:master.


CI: drone.io

Somebody presumably needs to enable drone.io on SciPy. Maybe develop on your fork first to see how it works, etc.? Is there some discussion somewhere about why it'll be useful for SciPy on top of travis/azure/appveyor/circle (new features or just spreading the load)?

Oh dear, I thought I was making the PR on my personal fork. I didn't mean to make it here first.
The rationale is that we won't need to do it on travis, and drone might be faster.


Mistake in preprocess_ali_ccp.py

In Line 81, vocabulary[k][v] should be set to 1 at first to coordinate with Line 90 (which should be >=10, for example). In this case the TRUE threshold is 12 now.

@wyhwhy Thanks for noticing this detail, we will fix this mistake.  We appreciate all contributions to improve torch-rechub. Welcome to participate in this project.

@wyhwhy Thanks for noticing this detail, we will fix this mistake. We appreciate all contributions to improve torch-rechub. Welcome to participate in this project.

Thanks for rely


Structured light using the IR emitter on D400 series

Before opening a new issue, we wanted to provide you with some useful suggestions (Click "Preview" above for a better view):

Consider checking out SDK examples.
Have you looked in our documentations?
Is you question a frequently asked one?
Try searching our GitHub Issues (open and closed) for a similar issue.

All users are welcomed to report bugs, ask questions, suggest or request enhancements and generally feel free to open new issue, even if they haven't followed any of the suggestions above :)

Required Info

Camera Model
D435i, D457

Firmware Version
N/A

Operating System & Version
N/A

Kernel Version (Linux Only)
N/A

Platform
N/A

SDK Version
N/A

Language
N/A

Segment
N/A

Issue Description
Is it possible to change the illumination pattern of the projector in any D400 series sensor (or any of the old SR300s)? If so, is it exposed in the SDK or through some other means? I would like to project my own illumination patterns for structured light algorithms without having to utilize an external projector.

Hi @mattbev   The coded light (structured light) of SR300 and the stereo depth of the 400 Series are different camera technologies and so the 400 Series cannot project a structured light pattern, unfortunately.  SR300 uses striped bar pattern projection and 400 Series uses dot pattern projection.
The patterns of SR300 and 400 Series also cannot be altered to a custom pattern.  So the use of an external projector is unavoidable if you wish to use your own projection.
The coded light pattern of SR300, which is like structured light, is shown in the image below.

Hi @mattbev  Do you require further assistance with this case, please?  Thanks!

Case closed due to no further comments received.


Fix development-environment to work for Windows

Babel caused some errors on its currently installed version as seen in the AppVeyor history logs.
To fix this, all babel-related modules have been updated.
To ensure, the built module also works for NodeJS argon, setting this option (targets.node) to the appropriate version-number might be necessary.

Coverage increased (+0.5%) to 98.247% when pulling dcbc64810632a97a0f7dcb3418500395ecb40a03 on manuth:windows-fix into eb2b7eac06147d9c88f39b780a048896eb6b7d2c on benmosher:master.

Coverage increased (+0.5%) to 98.247% when pulling dcbc64810632a97a0f7dcb3418500395ecb40a03 on manuth:windows-fix into eb2b7eac06147d9c88f39b780a048896eb6b7d2c on benmosher:master.

Coverage increased (+0.5%) to 98.247% when pulling dcbc64810632a97a0f7dcb3418500395ecb40a03 on manuth:windows-fix into eb2b7eac06147d9c88f39b780a048896eb6b7d2c on benmosher:master.

Coverage increased (+0.5%) to 98.247% when pulling dcbc64810632a97a0f7dcb3418500395ecb40a03 on manuth:windows-fix into eb2b7eac06147d9c88f39b780a048896eb6b7d2c on benmosher:master.

Coverage increased (+0.5%) to 98.247% when pulling dcbc64810632a97a0f7dcb3418500395ecb40a03 on manuth:windows-fix into eb2b7eac06147d9c88f39b780a048896eb6b7d2c on benmosher:master.

Coverage increased (+0.5%) to 98.247% when pulling dcbc64810632a97a0f7dcb3418500395ecb40a03 on manuth:windows-fix into eb2b7eac06147d9c88f39b780a048896eb6b7d2c on benmosher:master.

Coverage increased (+0.5%) to 98.247% when pulling dcbc64810632a97a0f7dcb3418500395ecb40a03 on manuth:windows-fix into eb2b7eac06147d9c88f39b780a048896eb6b7d2c on benmosher:master.

@ljharb All's set up now, the tests for Windows as well as the tests for WSL run flawlessly now.
Only the coveralls command at the very end doesn't seem to work.
Probably you want to remove this from appveyor as there is a separate coverage/coveralls check anyway

@ljharb alright I'm done here, too.
Would be awesome if you could merge this PR before #1792, so that I can rebase and refractor #1792 accordingly.


Clean TMP folders after job completion

After the job finishes completion on HPCC, clean the temp folder created for the job after completion.

This feature has been implemented and tested. Please test further and let us know if this is working as expected


fix: umd build issue

Fixes issue where the bundle was not properly built for UMD use when transpiling typescript -> javascript with @rollup/plugin-sucrase

Tested and working locally with cdn.html pointing to packages/javascript/dist/index.js build of @flatfile/javascript - so this should work in production!


Manage paths exclusion

The default capabilities for path matching in implementations are generally too limited (in J2E and even in Spring Web MVC), it would be great to be able to exclude paths (new excludePath parameter defined by a regexp: "^xxx$").
Let's create a PathMatcher and a  DefaultExcludePathMatcher which will be used by implementations.

We could take inspiration from Spring's AntPathMatcher.

That's an option, but in that case, I would need to copy/paste the AntPatternMatcher (from Spring). This is what Shiro did.
I was more thinking about using a regular expression which is more powerful, but forcing to use a full path matching expression (starting by ^ and ending by $) to avoid any security issue.
What do you think?

Sure. That sounds simpler and more powerful.

Sorry for meddling, but as full path matching are you saying "/img/resource.png" or anything under "/img/" ?

Both in fact. Anything you can describe with a regular expression: ^/img/resource\.png$ would only match your first example while ^/img/.*$  would match both.

 Great :+1:

PR: https://github.com/pac4j/pac4j/pull/318
Implemented for J2E: https://github.com/pac4j/j2e-pac4j/pull/25 & https://github.com/pac4j/j2e-pac4j-demo/pull/20


Add airbnb-js-shims to client-side JS

This adds airbnb-js-shims to the admin js and the iframe js, which gets IE11 working for me, where it used to break on an Array.from

Isn't this something babel does automatically? I like to know which code breaks this?
Babel load it from the corejs package.
Could you try updating core-js to the latest: 2.2.3 in your project and see whether it fixes this or not.

Babel transpiles syntax, not API. Shims are still required, and these are more reliable ones (and actively shim fewer things) than babel-polyfill/core-js.

@ljharb doen't add more overhead. I think we only need es6 shims in that case.
So, it seems like babel's core-js usage has a problem. So we may should open an issue on core-js repo.
In the meantime, let's add only es6-shim to the core.

@arunoda es5-shim and es5-sham are still required for React to work in IE 8. No browser, even latest Chrome, complies with ES5, even.

@ljharb I'm not sure whether we need to support browsers like IE8

No browser, even latest Chrome, complies with ES5, even.
What do you mean by this?

CC / @mnmtanish

@arunoda the cost is tiny, and the benefits range from zero to very high, depending on users' browser support matrix.
As for ES5, there does not exist any version of any JS engine that fully complies with ES5. The failures are minimal at this point, but they exist, and using the shims ensures they are papered over.

@ljharb I think we can go with this.
I'll take this.

Rebased, conflicts resolved

Awesome.


Font for Polish language - About Us page

Please correct fonts for Polish language like in this PR: #1603 - Add medium and bold Poppins font #1669
It was done for Help Board. Could we implement it also for About Us Page?

Home Page:

The same issue for the Turkish language:

@aleksarad if you have any questions about how this was solved in the Help Board reach out to @doheekim93

@CamilaArias thank you :)

@Zaydme we still need to fix the same thing for Vietnamese


Sealer run fails when machine name and ip do not match .

What happened:

操作系统: centos7.x
安装命令:
sealer run kubernetes:v1.19.8 -m <IP_ADDRESS> -n <IP_ADDRESS> -p 12589
报错原因:
[WARNING Hostname]: hostname "machine-140" could not be reached
[WARNING Hostname]: hostname "machine-140": lookup machine-140 on <IP_ADDRESS>:53: read udp <IP_ADDRESS>:49950-><IP_ADDRESS>:53: i/o timeout
error execution phase preflight: [preflight] Some fatal errors occurred:
问题分析:

解决办法:
配置 hostname 在 /etc/hosts 中匹配一致
集群清理
sealer delete –all
集群再次安装
sealer run kubernetes:v1.19.8 -m <IP_ADDRESS> -n <IP_ADDRESS> -p 12589
How to reproduce it (as minimally and precisely as possible):

Anything else we need to know?:
Environment:

sealer version (use sealer version):
Cloud provider or hardware configuration:
OS (e.g: cat /etc/os-release):
Kernel (e.g. uname -a):
Install tools:
Others:

@vpc123 Really appreciate that you help to debug this. I think this is belonged to some kind of host misconfiguration. And the kubeadm reports this error when installing kubernetes. And sealer depends on kubeadm.
I am afraid that, sealer will not hack kubeadm to improve this misconfiguration checking better. While, I think we could polish the error logging which show user much explicit information.


Add documentation for global filters

Which Issue(s) this Pull Request Resolves

Resolves #509
Release Note

NONE

Notes for Reviewers

Gave it another rewrite

I removed the block about skipping global filters. If we get the question a lot from users we can add a section about it.


Vpin studio 2.16.2 Breaks my install

Describe the bug
After updating when it goes back to Loading tables just hangs indefinably . if something is clicked  says Zulu platform not responding and vpinstudio not responding following that. re installed 3 time same result. Reverted back to Version 2.16.0 everything goes back to working normal
Windows 11 Pc, Ryzen 9 5950X

Thanks for reporting,  but I need the log files, located in the installation folder, in that case. Only then I'm able to reproduce what have happend and can fix it. I don't assume you have a copy of them? I'm usually eager to fix these critical bugs fast and my 2.16.2 installer did run smoothly.

I'm hoping they're still in the file. All I did was downgrade. I'm assuming
it didn't overwrite anything because my API key was still there. I'll check
later on tonight when I get home.
On Tue, Apr 16, 2024, 12:50 AM syd711 @.***> wrote:

Thanks for reporting, but I need the log files, located in the
installation folder, in that case. Only then I'm able to reproduce what
have happend and can fix it. I don't assume you have a copy of them? I'm
usually eager to fix these critical bugs fast and my 2.16.2 installer did
run smoothly.
—
Reply to this email directly, view it on GitHub
https://github.com/syd711/vpin-studio/issues/202#issuecomment-2058223141,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A2J7JF32ERHPJICZE6R22VDY5SU25AVCNFSM6AAAAABGINLJ66VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDANJYGIZDGMJUGE
.
You are receiving this because you authored the thread.Message ID:
@.***>

Broke my install as well. See logs.
vpin-studio-server.log
vpin-studio-ui.log

here are my Logs as well
vpin-studio-server.2024-04-15.log
vpin-studio-server.log
vpin-studio-ui.log

Thanks guys! I found the issue and will push out another hotfix today.

No problem! It's an awesome program.
On Wed, Apr 17, 2024, 12:16 PM syd711 @.***> wrote:

2.16.3 has been released and should fix the issue. Thanks for providing
the logs, made my life easier.
—
Reply to this email directly, view it on GitHub
https://github.com/syd711/vpin-studio/issues/202#issuecomment-2061693085,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A2J7JF6IGCVXESLNDBLJTZDY52N6TAVCNFSM6AAAAABGINLJ66VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDANRRGY4TGMBYGU
.
You are receiving this because you authored the thread.Message ID:
@.***>


Dynamic zone key for network block of {http.request.remote.host} with certain prefix

When used as the key of a dynamic zone, can {http.request.remote.host} be reduced to its network block for a certain prefix?
Assuming http.request.remote.host is <IP_ADDRESS>, and a function is used reducing it to /24, requests (and rate-limiting) to any address in the range <IP_ADDRESS>-255 would be grouped.
"key": "reduce_to_network_block({http.request.remote.host}, '/24')",

Interesting, let me think about this.

What if there was a placeholder:
{http.request.remote.host/N} where N could be any bit length that mapped the IP (if an IP, otherwise it would ignore the /N) to, say, <IP_ADDRESS> for anything in the <IP_ADDRESS>-255 range?
I can implement that upstream in Caddy if that's sufficient for you.

What if it's IPv6 though? Need ranges for both cases I think. Kinda tricky.

@francislavoie I don't think the https://pkg.go.dev/net/netip package cares. It's just more bits, right?

My point is, you might want to configure 16 bits for IPv4 but 24 bits for IPv6, like we have in the log ip_mask filter https://caddyserver.com/docs/caddyfile/directives/log#ip-mask

True; maybe we could support a /N,M syntax or something; let's wait for a feature request.

Oh what the hey. I did it anyway: https://github.com/caddyserver/caddy/commit/9873ff9918224f56604048ea0ed3d3ae3953939e
@kellytk Here's what you want: {http.request.remote.host/24,32} (for example; the ,32 is optional if you want both bit lengths for IPv4 and IPv6)

@mholt This is very exciting; especially the IPv6 support!
I wish to employ this feature in a layered fashion, ala contrived and simplified example: (IPv6 omitted as I don't yet have experience with it)

{http.request.remote.host/24} window:1s max_events:2,560
{http.request.remote.host/16} window:1s max_events:65,536
{http.request.remote.host/8} window:10s max_events:16,777,216

One request from <IP_ADDRESS> would:

Apply a +1 event to the <IP_ADDRESS>-255 zone key bucket.
Another +1 event to the 1.2.0-255.0-255 bucket.
And finally another +1 event to the 1.0-255.0-255.0-255 bucket.

Is that how this feature, as implemented, is designed to function?
This would enable me to allow burst traffic from individual IPs and small blocks, such as CGNAT, while still applying the events to the quotas of the larger containing blocks.  The rationale is that individual IPs and small blocks can legitimately be hot, however amortizing events across larger blocks should yield more moderate figures.  If large blocks are also hot, it's possibly an attack scenario.
To thwart my thinking an attacker would need to utilize more evenly distributed addresses from the global space.  That can be 'easily' mitigated with active monitoring and dynamic updating of these rate limit policies.

@mholt Excellent, thank you!  Will tests be added to verify the behavior and ensure there're no regressions in the future?

@kellytk I did write unit tests: https://github.com/caddyserver/caddy/commit/9873ff9918224f56604048ea0ed3d3ae3953939e#diff-f7d3f5ac378699eb31c965343929663ac9504606500b9b4f81f10d371bfc9677
We can always add more later if needed.
What are you using this for? Just curious.

@mholt Thank you!  I'm using Caddy as the user-facing reverse proxy to Rust-based 'apps' I write.
With enhancements such as this and https://github.com/caddyserver/caddy/issues/4558 Caddy is proving pleasantly mature.  Thank you once again.

 Glad it is working for you.


Demo app: menu icon is not inside of a button

The md-icon in the toolbar should be focusable and clickable.

Icons shouldn't be interactable, it should instead be the content of a button.


Update UI language without a restart

🚀 Feature

@Czaki brought this up on https://napari.zulipchat.com/#narrow/stream/212875-general/topic/Translation.20of.20exceptions
So we could have a different UI language set without needing a restart.
Motivation

Pitch

Alternatives

This is doable, but requires adding some methods to the widgets so that they know how to refresh their content. Qt may have a way of doing this already, but since we are in PyQt and Python land, we need to handle things differently.
We could create a str subclass that could remember its original text content when created, so it can try to translate it to a new language with a  refresh method or similar, and we would need for widgets to have a way to trigger this.
Additional context

This would be a nice to have, but not a priority right now. Something to keep in the back of our heads.

I do not do a detailed check, but it looks like napari split state from the interface. So it is enough to regenerate the interface from scratch without touching state. The biggest challenge is to properly disconnect/connect signals in such an approach.

Right now when changing language packs could we at least display a message saying "restart required" and/or a "restart" button to the preferences dialog? cc @ppwadhwa (we can make a separate issue if people like that idea, and my apologies if such an issue already exists and I have forgotten!!). I did find it a bit odd to change language packs and not at least be told that i needed to restart

. I did find it a bit odd to change language packs and not at least be told that i needed to restart

My bad, that is part of a separate issue https://github.com/napari/napari/issues/2181
First we need to enable autorestart for the userm, then we can integrate it :)


Safari TP: Partial Wake Lock API + MDN link

https://github.com/WebKit/WebKit/commit/46a272e5b78f15c439e2b28973c3211fdf36123b
https://twitter.com/tomayac/status/1575941229673558016
https://developer.mozilla.org/en-US/docs/Web/API/Screen_Wake_Lock_API

~Pending the actual TP.~
https://tests.caniuse.com/wake-lock:

However I went with partial as they didn't mentioned it in the changelog (given revision 255039 TP 156 would have been it; TP 157 for good measure) and also https://wpt.fyi/results/screen-wake-lock?label=master&label=experimental&aligned=&view=subtest&q=wake+lock:

If I had to bet, I'd say more parts are coming with another TP and then they will mention it in the changelog 0:-)

Belated thanks!


Add multiline JSON support

Some odd people like to put multiple JSON objects into a file, one per line. Possibly make a 'multiline' kwarg for to_json under BoxList to accommodate?

those are not weird but smart people. ;)
jsonlines.readthedocs.io and jsonlines.org

Either way, at least I though of them ;-)
Thanks for the references!

Added in 3.1 https://github.com/cdgriffith/Box/commit/252e2f574213708cbbcd42bdec442434384127fe


fix: close DB connection after execution.

Closes Issue: https://github.com/videokojot/EFCore.BulkExtensions.MIT/issues/154


Arc extension: update readme and icon

Description

Update the readme to include a link to download Arc
Update the extension icon with an Arc brand asset

Screencast

Checklist

[x] I read the extension guidelines
[x] I read the documentation about publishing
[x] I ran npm run build and tested this distribution build in Raycast
[x] I checked that files in the assets folder are used by the extension itself
[x] I checked that assets used by the README are placed outside of the metadata folder

I'm okay with everything but the icon change. I think they worked with arc browser to make it.

I'm okay with everything but the icon change. I think they worked with arc browser to make it.

I work at The Browser Company and this icon is from our brand designer :)

@rgrunberg just keep the icon the same.

Not a fan of a new icon...


Enforce specific error type in CallRestAPI

Now we generic error type for return value of callRestAPI function in apiclient.go. There is mixed use of CoaError and SummarySpecError in the function. We need to define a unified format with statusCode and actual error like httpresponse.

@linyguo to fix.


Can't get some features that appears in demo page

Hi, I don't manage to get some features that appears on the first embed on demo page
Like the Right to left  reading, the "items loaded" message when I first click on the embed.
Here is my test page : http://<IP_ADDRESS>/reader/
I try to come with a manga reader, and try to add prev/next basic link (I don't change your library, not yet), and an ebook selector from a basic ajax file tree.
Thakns in advance for the help.

For the right to left I added a config variable, If I have the time to do that properly, I'll PR that to you.


Skid

You skidded this. Same with all of your other projects. Stop taking credit from those who deserve it.

xd


Add Cocoapods Support & Add BackwardCompatibility to iOS 9.0

Pull Request Template
Description
Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context.
Fixes # (issue)
Type of change
Please delete options that are not relevant.

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration

[ ] Unit Tests

Checklist:

[ ] My code follows the style guidelines of this project
[ ] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes
[ ] Any dependent changes have been merged and published in downstream modules
[ ] I have checked my code and corrected any misspellings

This looks to have broken things within ios-constraints-kit as it needs support for iOS 12 and above.
The package product 'ConstraintKit' requires minimum platform version 12.0 for the iOS platform, but this target supports 9.0

This looks to have broken things within ios-constraints-kit as it needs support for iOS 12 and above.
The package product 'ConstraintKit' requires minimum platform version 12.0 for the iOS platform, but this target supports 9.0

That's right. There is a problem installing ConstraintKit using SPM.


Roadmap for OpenCL 2.0 enqueue_kernel and non-uniform work-sizes

Any rough estimate on when various missing OpenCL 2.0 features will be available on Vega 64?
Specifically, I am interested in enqueue_kernel and non-uniform work-sizes, but sharing a features matrix that shows what works and what is missing in the OpenCL 2.0 support would be even better, and coupling that with (at least rough) estimates on when that could be expected would be awesome.
For reference, I'm on Arch Linux and have Vega 64 waiting to be installed (but currently I'll have to do with R9 290 because I need these OpenCL 2.0 features and they are supported in Catalyst)
For the time being, it does not even have to be open-source. Any hints about AMDGPU-PRO and any other combination would solve this issue form me temporarily until it is available in ROCm...

We are looking to have full OpenCL 2.0 support with ROCm 2.0.   Note All the 2.0 features have been in for quite a while, except for kernel enqueue and Pipes are the last two feature we need to role out to formally turn on OpenCL 2.0.
I will have the teamwork on Feature matrix for AMDGPUpro vs ROCm.   You find we even have some of the OpenCL 2.1 runtime features in ROCm OpenCL driver now.   RIght now we are focused on any new runtime and Kernel feature.

Hi @blueberry
As of ROCm 2.0, device enqueue should work on Vega10 GPUs (gfx900), so long as you have working PCIe atomics between your GPU and your CPU. We have supported non-uniform workgroup sizes for quite a while now. As such, this request is met.


chore: remove pdt v1.0 schema

:tada: This PR is included in version 1.16.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Statuscode 404 from istio-proxy when cluster isn't found is missleading

Hey,
We've been going through the process of using Sidecar to scope all of our sidecars (which works beautifully I must add), however there is one behaviour that I think needs looking at.
We use both NetworkPolicy and Sidecar to restrict Egress.  Previously we only had NetworkPolicy, so when appA tried to talk to appB (when it shouldn't be able to), it'd network timeout and envoy would return a 503.  This wasn't brilliant, but it was quite obvious that there was a connection issue.
However, now that Sidecar doesn't include appB, envoy will immediately 404 the request, as the cluster isn't found.
This however, is miss-leading, and doesn't indicate to the sending app that there was actually anything wrong with the request, the sending app just believes the remote endpoint returned a 404.   This is a problem, say for example that remote endpoint was expected to return 404 for some operation like "checkUserExists".
I'm not sure what the appropriate way to handle this would be, perhaps 502 Bad Gateway would be more appropriate.
PS.  This caused us a live issue today as a misconfigured sidecar policy broke the egress, and we didn't know as the source app just thought the destination was acceptably 404'ing.
PPS.  Are there any envoy metrics we could alert on that would signify this black hole (@douglas-reid?)?

or 418, naturally.

ping @costinm @rshriram @nmittler - this one is actually quite a pressing issue for us, would be great to get some input.

Yeah, that's definitely an issue.
Even if we don't make 503 the default its certainly reasonable to allow control over the 'this service wasn't mapped in' status codes.
A reasonable alternative to 503 would be '403 Forbidden', would be good to hear if that would cause application-level issues for you?.
403 indicates that the failure state is permanent and not a function of a temporary network outage. Given that the configuration scoping is intentional you might argue that 403 is a better fit

Any 4x code is miss-leading in my opinion and isn't indicative of a gateway problem (which is what this is).
502 BAD GATEWAY is easily the best code for this.
This is that much of an issue that I need to build a monkeypatched version of 1.1.1 proxy to deal with it.  Can anyone point me to the code where the 404 is being returned from?

I'm not sure I entirely agree that 502 is universally the best option but it may be the most pragmatic.
In the meantime have you looked into defining your own 'black hole' virtualservice that sets
https://istio.io/docs/reference/config/networking/v1alpha3/virtual-service/#HTTPFaultInjection-Abort
The order of routing of the hosts referenced in a Sidecar is ordered so if you did something like...
VirtualService (namespace: blackhole, host: *, destination: , fault-inject: 503, percent: 1.0)
Note: that we namespace it so it doesn't get referenced by default. Then we do
Sidecar (egress: : [ real-host1, real-host2, blackhole/* ])
you should get the behavior you want as the routing rules will fall-through.

As it stands we have 190 Sidecars in isolated namespaces, it feels like a global problem for me than something that i need to configure on each sidecar.

@louiscryan any idea where i can find the 404 response in code?

Sure.  The config-gen is
httproute.go:buildSidecarOutboundHTTPRouteConfig
there's an equivalent example for Gateway (not Sidecar) that is explicitly generating a wildcard match to serve 404 if no routes are defined
gateway.go:buildGatewayHTTPRouteConfig ~line 286-293 (creates a DirectResponse) which is effectively what you'd want to do though it should be done before host merging occurs in 'mergeAllVirtualHosts'
HTH
@howardjohn

Thanks @louiscryan - i'll have a read and see if there is a pragmatic way i can patch this.

I want to make sure I fully understand this:
Say we add a wildcard match for each port as the (as the last virtual host) that gives a 502.
Any requests to this port that doesn't match will be get a 502, not just things that aren't imported. So curl service.not-in-scope-namespace will return 502, but also curl <IP_ADDRESS> (or some random address). Is this what we are looking for?
We could make it only 502 for real services that are out of scope, but that defeats the main purpose of the Sidecar, to reduce config for scalability, so I don't think that is a good option.
@Stono

Hey,
Yes that's the behaviour i'd be expecting, going to some-random-fqdn, or some-random-ip should both be 502 gateway errors unless they're defined as services.
404 is majorly miss-leading and doesn't tell the sender that there was an issue talking to the remote service, it tells the sender that the remote service returned 404, which is not true.

Just to update this with out slack chat for clarity of others reading:
route not defined in Sidecar, REGISTRY_ONLY = gateway error (502)
route not defined in Sidecar, ALLOW_ANY = off to the service
https://istio.io/docs/tasks/traffic-management/egress/

Change here: https://github.com/istio/istio/pull/12916
@louiscryan I used 502 error as a placeholder, happy to change it though.
@costinm @rshriram this is a fairly substantial behavior change if you want to weigh in. It is blocked by a feature flag though.

We had another production outage caused by this today.  End to end tests didn't pick it up because 404 doesn't flag as an invalid response from the repo api.
@howardjohn will this be in 1.0.2?  If not, could you produce a 1.0.2 build with this patch so i can use it as our proxy image?  It's a real issue for us.

Using pilot.env.PILOT_ENABLE_FALLTHROUGH_ROUTE=1 with the latest 1.1 nightly (1.1-20190404-09-16) now correctly gives 502s :-)
Thanks @howardjohn
root@platform-testing-6f98ccd788-jg298:/app# curl -v http://app.moo
* Rebuilt URL to: http://app.moo/
*   Trying <IP_ADDRESS>...
* TCP_NODELAY set
* Connected to app.moo (<IP_ADDRESS>) port 80 (#0)
> GET / HTTP/1.1
> Host: app.moo
> User-Agent: curl/7.52.1
> Accept: */*
>
< HTTP/1.1 502 Bad Gateway
< location: http://app.moo/
< date: Thu, 04 Apr 2019 14:44:35 GMT
< server: envoy
< content-length: 0
<
* Curl_http_done: called premature == 0
* Connection #0 to host app.moo left intact

Going to close this - will be available in 1.1.3 or using the nightly builds.


Farming level doesn’t notify when you have gained a level

Simple as above, see pictures
![C22B683C-4A4D-457C-8500-56B124856852](https://user-images.githubusercontent.com/117435025/199922145-079a033a-abdf-4

828-a141-7cc4acd26b25.jpeg)

partial dupe of #4485 but good to know its not just 99 that it isn't working with

Farming now properly shows levels ups.