Shortcuts not working and causing errors

When I hit F1, F3 or F4 I get the following
Traceback (most recent call last):
File "./json/decoder.py", line 368, in raw_decode
StopIteration
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "..\Sublime Text 3\sublime_plugin.py", line 543, in run_
return self.run(edit, **args)
File "Typescript in ..\Sublime Text 3\Data\Installed Packages\T3S.sublime-package", line 505, in run
File "Typescript in ..\Sublime Text 3\Data\Installed Packages\T3S.sublime-package", line 173, in type
File "./json/init.py", line 319, in loads
File "./json/decoder.py", line 352, in decode
File "./json/decoder.py", line 370, in raw_decode
ValueError: No JSON object could be decoded

see my last comment in https://github.com/Railk/T3S/pull/63 (tl;dr there is a new plugin named ArcticTypescript)
You can rebind all keybindings. refer to the Default.sublime-keymap file of any plugin and copy any keybindings to your User-keymap file (preferences menu)

Thank you, I will have a look at that then.


Deploy fails when revisions.json is missing

When starting a new deployment into an empty directory, the plugin will fail in the didUpload hook with an error like:

cat: /www/releases/revisions.json: No such file or directory

Creating an empty file at /www/releases/revisions.json solves the issue (there's a json-syntax error, which is to be expected, but the deploy completes).
I think a fix would involve checking if the revisions.json file exists prior to reading it… so that first-time deployments complete properly.

Thanks for heads up, I'll look into it. If you can provide a PR, it would be highly appreciated. I think a quick fix here would be something like:
const cmd = "([ -e " + revPath + " ] || echo \"[]\" >> " + revPath + ") && cat " + revPath

https://github.com/AutoCloud/ember-cli-deploy-simply-ssh/blob/master/index.js#L129


Correction needed (I think)

[Enter feedback here]
Article says
"In your tenant, go to the data hub in Power BI Desktop"
Think it should be changed "Power BI Service"

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

ID: 74430e48-5764-9eff-7410-26ab2c400917
Version Independent ID: 74430e48-5764-9eff-7410-26ab2c400917
Content: Access shared datasets in Power BI as a guest user from an external organization (preview) - Power BI
Content Source: powerbi-docs/collaborate-share/service-dataset-external-org-share-view.md
Service: powerbi
Sub-service: powerbi-admin
GitHub Login: @ajburnle
Microsoft Alias: ajburnle

Thanks for your comment, @rjoboyle. This issue has been assigned to the doc author for review.

Hi @rjoboyle,
You're right, this is a typo - thanks for pointing this out.
I fixed this typo and it'll show on the website soon.
Thanks,
Kesem


evaluator: Fix the no_gpl_declared.kts script

Add the second parameter to the error function call. This is a fixup
for e559b3e. The issue was not detected by the EvalutorTest because
Kotlin's own error function was called instead, so the script could
still be compiled.

This change is

The clash with Kotlin's own error function is nasty. Should we rename ours?


chore: fix github actions

Current dependencies on/for this PR:

main

PR #143   👈

This stack of pull requests is managed by Graphite.

Merge activity

Feb 17, 4:32 PM GMT+1: @nadilas merged this pull request with Graphite.


[CLI] Tensorboard integration bug with resetting x-axis through global_step

Description
When using the tensorboard integration, the latest version wandb==0.10.23 does not seem to handle global_step properly when it was reset to 0, whereas the older version of wandb==0.9.5 seems to have recorded correctly.
Consider a simple code sample:
import numpy as np
import random
import time
from torch.utils.tensorboard import SummaryWriter
import wandb

if __name__ == "__main__":
    # setup the environment
    experiment_name = f"wandb=={wandb.__version__}__{int(time.time())}"
    run = wandb.init(project="cleanRL", entity=None,
                     sync_tensorboard=True, name=experiment_name,
                     save_code=True,)
    writer = SummaryWriter(f"runs/{experiment_name}")
    for global_step in range(100):
        writer.add_scalar("loss1", np.random.rand(1)[0], global_step)
    for global_step in range(100):
        writer.add_scalar("loss2", np.random.rand(1)[0], global_step)

In particular, notice that the loss2 is recorded when the global_step has been reset. When visualizing, it shows like this:

If we visualize the loss using global_step as the x-axis, the runs recorded with wandb==0.10.23 seem to break while wandb==0.9.5 seems to work correctly. Also the tensorboard page for wandb==0.10.23 also displays correctly.

See report for details. https://wandb.ai/costa-huang/cleanRL/reports/Tensorboard-global_step-misalignment--Vmlldzo1NTQzMzA
global_step in tensorboard is a way to customize the x-axis and quite helpful. Without it, it would look like the following two cases if I did anything with tensorboard:

Neither is desirable (the first image has a bug while the second does not align the x-axis).
Hope this is helpful :)

Hey @vwxyzjn
Wandb Version: 0.10.23
I tried reproducing the issue with the code snippet that you had sent in.
This is the dashboard for the run: https://wandb.ai/repro/GH-1997
I do not see the loss2 metric breaking in any way. In fact both the metrics start from the same point in the x-axis when I am plotting them against global_step.
I made a small change to the code, but that should account to anything big.
experiment_name = f"wandb=={wandb.__version__}__{int(time.time())}"
wandb.tensorboard.patch(root_logdir=f'runs/{experiment_name}')
run = wandb.init(entity="repro", project="GH-1997")

writer = SummaryWriter(f"runs/{experiment_name}")
for global_step in range(100):
    writer.add_scalar("loss1", np.random.rand(1)[0], global_step)
for global_step in range(100):
    writer.add_scalar("loss2", np.random.rand(1)[0], global_step)

Let me know if I have made a mistake or missed something that needs to be looked into.

Hey @ariG23498 Thank you for looking into this, I can confirm this works! It appears having wandb.tensorboard.patch(root_logdir=f'runs/{experiment_name}') is crucial.


[stable/concourse] Change Concourse links

Concourse.ci was hijacked and the team has made the decision to move to
https://concourse-ci.org:
https://medium.com/concourse-ci/were-switchin-domains-5597dcd0b48b

Signed the CLA

/assign @william-tran

/ok-to-test

/lgtm

@arbourd Thanks for the quick action on this one, and for fixing the version typo.


operators emulating numeric types are (wrongly) guessed as not supported when operands are generic types

Environment data

Pylance v2022.9.40
Python version(3.10.7 (tags/v3.10.7:6cc6b13, Sep  5 2022, 14:08:36)
Visual studio: 1.71.2
Windows-10-10.0.19042-SP0

Code Snippet
from collections import Counter
from collections.abc import Iterable
from datetime import timedelta
from typing import TypeVar

_T = TypeVar('_T')
def my_func(a: _T, b: _T) -> _T:
    """ my_func """
    c: _T
    c = -a # reported as not supported
    c = 2 * a # idem
    c = a + b # idem
    c = a - b # idem
    c = a | b # idem
    c = a / b # idem
    c = a // b # idem
    c = a & b # idem
    c = a % b # idem
    c = a * b # idem
    return c

# Use cases: emulating 'sum' on non basic numeric types:
def summation(iterable: Iterable[_T], start: _T) -> _T:
    """ summation """
    _sum: _T = start
    for item in iterable:
        _sum += item
        # Report:
        #    Operator "+=" not supported for types "_T@summation" and "_T@summation"
        #    Operator "+" not supported for types "object*" and "object*"
    return _sum

counters: list[Counter[str]] = [
    Counter(a=1, b=2),
    Counter(a=10, c=4),
    Counter(b=5, d=3),
]
zero_counter: Counter[str] = Counter()
print(summation(counters, start=zero_counter))

elapsed: list[timedelta] = [
    timedelta(seconds=17),
    timedelta(seconds=37),
    timedelta(seconds=59),
]
print(summation(elapsed, start=timedelta()))

Expected behavior

I wonder whether it is a bug or a feature.
I understand the situation is not easy:  when analyzing the function, the generic type _T is unknown  and the type checker doesn't know neither if the operation is defined nor the type it returns.
However the guess that the operations are not supported seems a bit bold and it can be wrong as shown by the use cases.
Hence I would have expected no report.
The only possible current workaround I see is type: ignore.

Again thank you for your great tool!
Patrick Bourdon - PARIS

Pyright (the type checker that underlies pylance) is working correctly here. When you use a type variable like _T in your example, it can be substituted for any object. Some objects don't support numeric operators, so the type checker is telling you that your program will crash if you pass objects of arbitrary type to my_func.
What you can do here is specify that _T has an "upper bound" type that constrains which types can be bound to _T. You can use a protocol class as a bound. For example, if you want to indicate that _T can be any type that supports addition, you could do the following:
_T = TypeVar("_T")

class SupportsAdd(Protocol):
    def __add__(self: _T, obj: _T, /) -> _T:
        ...

_TAdd = TypeVar("_TAdd", bound=SupportsAdd)

def my_func(a: _TAdd, b: _TAdd) -> _TAdd:
    c: _TAdd = a + b
    return c

Thank you very much for the direction.
I read many times about "protocol" (and some from you), but I never really dive in to understand the abstraction.
I have to ... "PEP 544 – Protocols: Structural subtyping (static duck typing)"


Sorting by observed on date in list view of Collection projects results in blank page

In the Observations tab of a Collection project, click on List view then sort by Observed on - a blank screen will result. Here is what the console displays:

This fix has been deployed and seems like collection project list sort works OK now


Avoid re-calculation of face orientation

The face orientations after folding were calculated twice, first in X.V_VF_EV_EA_2_Vf (as s), then in X.V_FV_2_Ff (using function M.orientation).

Function X.V_VF_EV_EA_2_Vf was renamed X.V_VF_EV_EA_2_Vf_Ff, and now returns Ff along Vf.
Functions X.V_FV_2_Ff and M.orientation, now unused, were removed.

This partially addresses Issue #13.
It doesn't affect Issue #17.
It fixes another issue where CPs including straight folds that are split into two or more pieces would not fold. This was because function M.area2 in function M.orientation would return a value close to 0 with a random sign due to rounding errors.
Attached is a .cp file that caused the problem.
Crane.cp.zip

Looks great! I will add a space in the output of the pair returned for style consistency, but will merge!

Seems like there is still another issue with degree-2 vertices between collinear lines keeping your CP from folding. The issue with such vertices is known, so we should probably open an issue.

Are you sure? My crane now folds correctly and I haven't been able to find another example that would cause problem.

Ah, you are right. I think the old code was still cached in my browser. Yes, looks like your crane now folds correctly!


SSL Passthrough

Is your feature request related to a problem? Please describe.
Unable to pass through encrypted SSL connections to a server that doesn't need SSL encryption
Describe the solution you'd like
A clear option or configuration to enable SSL passthrough via stream or similar method.
Describe alternatives you've considered
Attempted to add code from links under additional context, host simply goes offline.
Additional context
This has been discussed in some other unclosed issues linked at the bottom, but this should be possible with modern NGINX using the stream function. I see that NPM has "Stream" but it only allows you to capture an entire port instead of a specified domain for forwarding to another internal host.
https://serversforhackers.com/c/tcp-load-balancing-with-nginx-ssl-pass-thru
https://stackoverflow.com/questions/38371840/ssl-pass-through-in-nginx-reverse-proxy
https://github.com/jc21/nginx-proxy-manager/issues/235
https://github.com/jc21/nginx-proxy-manager/issues/461

I have to say this would be a big feature. This would make dealing with cooler larger projects a breeze. Matrix/synapse comes to mind which is a pain to setup behind a separate proxy. It would be nice if you could just forward a specific domain 80 and 443 traffic to a service on a separate machine that does the ssl stuff.

After spending a lot of time getting this to work in NGINX itself - I realize this request may not fulfill everyone's needs even if it was implemented according to what NGINX supports.
NGINX can do SSL passthru, but it can't do it on the same ports being serviced for normal NGINX services. You can do only one or the other.
For example, let's say you have some sites on 443 using normal NGINX SSL. You cannot then also use NGINX to perform passthrough to a VM that has its own certificates. You would have to use a different port, such as 8443 or a custom port. The only other alternative would be another box and another public IP.
I finally resolved my issue by switching back to pure NGINX and implementing passthru on 8443 and the rest of our sites on 443.
NPM is a great tool but it has its limitations - perhaps someday I will revisit it and I will still recommend it to those who have less complex setups than I.

for ssl pasthrough you need to use stream (npm now support it), but i think you can't use stream and http on the same port.

I was looking for this feature as well. Since the above PR #1479 doesn't seem to have been merged, I baked my own quick fix. It's not pretty but seems to do the trick (not tested thoroughly yet). Sharing here as it might help others.
Note that this approach makes designates an new port as the main input port for your NPM https traffic! This worked for me because I am behind NAT and could simply adjust the port forwarding for the public 443 port to a new port, and leave all the remaining NPM configuratio untouched. You might have to find another way if your NPM is directly listening on a public IP. This also comes with the caveats mentioned by @chaptergy regarding dual-proxying.
So here goes:
Add the following to your custom stream.conf (see https://nginxproxymanager.com/advanced-config/#custom-nginx-configurations) to passthrough SSL traffic for backend.example.com to the host <IP_ADDRESS>:443 and backend2.example.com to <IP_ADDRESS>:443:
map $ssl_preread_server_name $name {
    backend.example.com     backend;
    backend2.example.com    backend2;
    default                 npm;
}

upstream backend {
    server <IP_ADDRESS>:443;
}

upstream backend2 {
    server <IP_ADDRESS>:443;
}

upstream npm {
    server localhost:443;
}

server {
    listen      12346;
    proxy_pass  $name;
    ssl_preread on;
}

This is more or less a straight copy from the nginx docs example (https://nginx.org/en/docs/stream/ngx_stream_ssl_preread_module.html) with the addition of the npm -> localhost:443 default backend. Just add additional hostnames to the mapping and backends as required.
You might still need http proxy hosts on port 80 to make letsencrypt challenges work to those passed-through hosts.
Related issues: #776 #286

Does this work for anyone else? I made these changes, got rid of backend2 (I only have one host I need to do SNI passthrough for) but all it did was break all my existing hosts and it never started listening on 12346.


Failed build apk with latest version

Hi @lukef ,
Thanks for the new release.
Unfortunately, I have failed to build apk after upgrade to the new version 3.1.0.
I am using vscode to build the apk.
Here is my flutter doctor details:
PS C:\JSL\VIQCore\flutter\projects\viqcore_community> flutter doctor -v
[√] Flutter (Channel stable, v1.9.1+hotfix.6, on Microsoft Windows [Version 10.0.17134.1069], locale en-US)
• Flutter version 1.9.1+hotfix.6 at C:\src\flutter
• Framework revision 68587a0916 (8 weeks ago), 2019-09-13 19:46:58 -0700
• Engine revision b863200c37
• Dart version 2.5.0
[√] Android toolchain - develop for Android devices (Android SDK version 28.0.3)
• Android SDK at C:\Android
• Android NDK location not configured (optional; useful for native profiling support)
• Platform android-28, build-tools 28.0.3
• ANDROID_HOME = C:\Android
• Java binary at: C:\Program Files\Android\Android Studio\jre\bin\java
• Java version OpenJDK Runtime Environment (build 1.8.0_202-release-1483-b03)
• All Android licenses accepted.
[√] Android Studio (version 3.5)
• Android Studio at C:\Program Files\Android\Android Studio
• Flutter plugin version 41.0.2
• Dart plugin version 191.8593
• Java version OpenJDK Runtime Environment (build 1.8.0_202-release-1483-b03)
[√] Connected device (1 available)
• Android SDK built for x86 • emulator-5554 • android-x86 • Android 9 (API 28) (emulator)

I finally managed to build the apk by upgrading the gradle version to 3.5.
Thanks for this great package!


DCOS-38339: Address Memory Issues

Update the mesos-client to 0.2.3 as this version great reduces the memory footprint and adjust MesosStream to stop multicasting the stream using the replay subject. This change allows the browser to garbage collect dependent/sliced strings created while processing the Mesos response.
Please note that this is a temporary adjustment as it would result in multiple open connections if consumed by more than one component. We're going to revert this change once the http-service  to emit event objects instead of the response as well as changed the mesos-client to output parsed JSON.
Testing

Checkout the code and verify that the number of open mesos subscribe connections is correct and the updates are flowing through.
Make memory snapshots in Firefox and Chrome and observer the Safari memory footprint while looking at a task page.

Closes DCOS-38339

@orlandohohmeier test fails are legit, please take a look :)

Going to check if those marathon fails

cause Region and zone to be N/A

And it's green 💚 @Fabs @DanielMSchmidt


Not getting notification via FCM.on or FCM.getInitialNotification

Hi,
I'm seeing similar issue reported in https://github.com/evollu/react-native-fcm/issues/569, but not sure if it is exactly the same.. In some cases, when i click on the notification toast, i get the notification properly in my app in the FCM.on callback that i have as follows:
FCM.on(FCMEvent.Notification, async (notif) => {
    logger.log("received notification: " + JSON.stringify(notif))
    handleNotification(notif)  // my code to handle the notification
}

However, in one particular case, the above code doesn't get invoked. This is when I wait for a long time (say, 15+ minutes) before I click/tap on the notification toast. I think Android puts the app to sleep after that much time?
I am also already checking for FCM.getInitialNotification as follows.. (the code below is invoked from one of my componentDidMount methods)..
FCM.getInitialNotification().then(notif => {
    if(notif){
        logger.log('Got initial notificaton: ' + JSON.stringify(notif))
        NotificationUtil.handleNotification(notif)
    }
}

I do get something in the above block (initial notification), but its kind of empty. the log statement above is:
Got initial notificaton: {"opened_from_tray":1,"fcm":{"action":null}}
And this happens only when I wait for a while before i click/tap on the notification toast (not sure if some folks here are calling that a 'banner' - which is available when you swipe/pull down from the top of the phone screen)..
If the app is just minimized, and i tap on the notification toast in a couple of minutes, everything works fine - my FCM.on callback gets invoked, and i get the complete notification data as expected.. here's one log for the success case:
received notification: {"fcm":{"action":"my_test_action"},"show_in_foreground":true,"data.foo":"bar","sound":"default","opened_from_tray":1,"click_action":"my_test_action","priority":"high","id":"my_notif_id","body":"test notif body","title":"test notif title","icon":"ic_launcher","local_notification":true}
@evollu - do you know what could be causing this?
Thanks
Amit
When openning an issue, please include following information for better support

What version of RN and react-native-fcm are you running?
"react": "^16.2.0",
"react-native": "^0.51.0",
"react-native-fcm": "^11.3.1",

What device are you using? (e.g iOS9 emulator, Android 6 device)?
Android 7.1.1 emulator (same behavior on many other versions, as well as my Samsung s8+ running 7.1.1)..

Is your app running in foreground, background or not running?
not running

Before openning an issue

Reproduce it with the example project in this repo.
If you can't receive iOS notification, make sure you can receive notification using quickstart-ios project provided by Firebase team

quick update here..
i realized that we had upgraded the react-native-fcm version recently (11.3.1).. but when i rolled back to an earlier one (10.0.3), things work as expected!(double checked a few times rolling back/forward on these two versions, and that's what it looks like - regression in some version after 10.0.3)

one big change I've made is the local broadcasting.
If you have chance can you test the latest version if this line is triggered?
https://github.com/evollu/react-native-fcm/compare/v10.0.3...v11.3.1#diff-c04acb3238c7458755defae08932ece4R218
v10 has a big issue: if you have multiple app using this notif repo, both apps will get notifications

can u double check the diff link? i can't see anything when i click on it.. (see attached screenshot)

btw, @evollu - I'm seeing another weird issue..
like already mentioned, i have registered the callbacks for FCM.on..
but i also check for FCM.getInitialNotification(..) from one of the componentDidMount methods (when my app loads)..
sometimes, the FCM.getInitialNotification(..) gets a pretty old (several hours) instance of a notification..
do you know what could cause that? when is the FCM.getInitialNotification(..) supposed to clear out? (or is there a way that I can clear that out from my app?)
is there any recommended pattern / best-practices so that I can know whether I should ignore the FCM.getInitialNotification(..) response or not?
Thanks..

I was dealing with a similar issue last week but everything seems to be working fine for me now. I was using wix react native navigation and that was causing problems that I didn't have with this library previously.
Now I get the initial notification before doing anything else on my application and store it to use once the user logs in to the app.
Also if the app starts by running react-native-run-android I'm not able to get the initial notification. If I kill the app and start it again after that everything is fine. This isn't a problem when it goes to a production release on the play store. It's just caused by the terminal starting my Main Intent it seems.

@bhosle-git the right url is https://github.com/evollu/react-native-fcm/compare/v10.0.3...v11.3.1
FCM.getInitialNotification(..) doesn't get updated until the app is killed. putting app in background for long time doesn't seems to kill the app. Usually you put some custom property in notification like type, then you just check if that thing is present in FCM.getInitialNotification. FCM.getInitialNotification should only be call on app bootstrap. like DidMount in our root component or outside of component lifecycle

@evollu - we do have something to identify that its our app's notification.. the issue is that its a very old notif instance that had been processed earlier.. but let me try moving that outside the component lifecycle and see how it behaves..
@RuairiOliver - we recently moved to react-navigation from an earlier one that was deprecated.. not sure if its the same issue.. but yeah - I'll try getting the initial notification before anything else (from index.js) and update here..
thanks both..

SO: Android 7.0 - Physical Device
I have the same problem. When opening the app for the first time through the notification, getInitialNotification() runs perfectly. After leaving the app idle for a while (I open other apps to force inactivity) and reopen it, getInitialNotification() is bringing exactly the same data as before. This is a problem.
I also did the test by opening the app after inactivity through another notification in the tray. The result was the same, instead of getInitialNotification () returning the last open notification data, it shows the notification data that opened the app.
Another similar situation is that if I open the app normally by the icon, I let it be inactive, so when I receive a notification and try to open it, getInitialNotification() returns only the data of the app initialization:
{
  "opened_from_tray":1,
  "fcm":{
    "action":"android.intent.action.MAIN"
  },
  "profile":0
}

When the app is in the background, but not sleeping / inactive, and I open the notification via the tray, everything works fine.
Is not there really a way to reset / delete the notification after getInitialNotification() is called?
I think it's the first time I write here. I would like to thank you for the excellent work, @evollu.

I'm facing the same issue as @dowgsss .
Is not there really a way to reset / delete the notification after getInitialNotification() is called?

Facing the same issue. In addition, my app's notification listener does not show an alert when I tap the alert in the notification center.

@bhosle-git is your issue solved?
Is not there really a way to reset / delete the notification after getInitialNotification() is called?
not as far as I know. It is reading from a native property, which doesn't cleared.

In recent days I've been trying to solve this problem by researching, modifying, and debugging the native code. Even though my background has been based on Java, I find it difficult on Android. Unfortunately I did not find any solution.
Not unlike what @evollu has already said...
From what I saw, the firebase uses an Intent, which is "attached" to our MainActivity. In this case, the data we send is in the Intent extras. This data can be easily deleted or modified, but by the time Android kills the app and then the user opens it, we get back to the Intent data that was opened along with the app the first time. From what I've read, this data is in a "process" of Android, so whenever the app is restored, the data from that Intent comes back. It seems that some people who use firebase messaging with native java code also have the same problem.
"Solution" in my case...
For the time being, when handling the initial notification I will save the notification code within AsyncStorage and will only take some action on the notification if it is different from the last notification saved in AsyncStorage.

@dowgsss 👍

I have not given up yet. Yesterday, while I was in the barber shop waiting for my turn, I still insisted on how to solve the problem, cleanly. I did some research and at some point I came across an issue that I had seen several times in stackoverflow, but had not paid attention to the answer less voted:
https://stackoverflow.com/a/18307360/9070081

Code
@Override
protected void onNewIntent(Intent intent) {
    super.onNewIntent(intent);

    /*
    * This overrides the original intent.
    */
    setIntent(intent);
}

I added this excerpt in my MainActivity and YES, the problem has been solved!
In short, there are now two situations that can happen considering that you opened the app and after some downtime the system killed the app:

If you open the app through a new notification, the extra data from this last notification will be passed in getInitialNotification. Yeah!
If you open the app directly from the list of recent tasks, the first notification executed at the initialization of the app that is received in getInitialNotification. Here you still need to be careful:

At this second point, if you want to prevent the same action / notification from being run multiple times, you need to do an implementation using AsyncStorage or something, as I said in my last comment.

@dowgsss
are you saying that by calling setIntent it will override the response of getInitialNotification to be the newer intent?
But I have 2 questions

Is there a way for iOS to do the same?
This new intent will be broadcasted through notification callback to JS. so what is the benefit of changing the intent?

Is there a way for iOS to do the same?

Since I do not program for iOS I really do not know how to answer that. In fact, are there any reports of this same problem in iOS?

This new intent will be broadcasted through notification callback to JS. so what is the benefit of changing the intent?

I'll always have the extra data updated in that specific situation where android kills the app and then I open the app via a notification. I will be able to see the data from this last notification.

so far iOS shares the same behavior as Android, I'd prefer to keep them in sync to avoid surprises.
if android kills the app and you open it through notification, getInitNotification should give your the new notification.

I think there may be a misunderstanding about point 2. The problem is precisely because on Android, when the app is killed by the system (not by the user) when you reopen it through a notification, we are not receiving data from that last notification .
I think I can describe it through the following steps (without the change I suggested in MainActivity):

My recent apps list is empty.
A notification from my app is received.
I touch that notification.
Now the app opened for the first time, and via a notification. In getInitialNotification() I get something like:

{
  opened_from_tray: 1,
  fcm: { action: 'android.intent.action.MAIN' },
  collapse_key: 'xxxxxxxxxxx',
  'google.message_id': 'xxxxxxxxxxxxxxxxxxxx',
  from: 'xxxxxxxxxxxxxx',
  'google.ttl': 1111111,
  my_data_key: 'abc123',
  'google.sent_time':<PHONE_NUMBER>000
}

With each notification I pass a unique my_data_key inside the data node.
Ok. Through getInitialNotification() I received the notification data and checked that my_data_key is valid, so I triggered an action.
Now I minimize my app and go to other apps. After a while android kills my app because of inactivity, however it continues in the list of recent apps. This is an important point.
Now I get another notification.
I touch this notification and the app opens one more time, but now it was on the list of recent apps.
At this point I expect the new notification data to be passed to getInitialNotification(), but what I get is the same data as the first notification:

Expectation:
{
  opened_from_tray: 1,
  fcm: { action: 'android.intent.action.MAIN' },
  collapse_key: 'xxxxxxxxxxx',
  'google.message_id': 'yyyyyyyyyyyyyyyyyyyyyyy',
  from: 'xxxxxxxxxxxxxx',
  'google.ttl': 1111111,
  my_data_key: 'otherDataKey',
  'google.sent_time':<PHONE_NUMBER>000
}

Reality(same data as the first notification):
{
  opened_from_tray: 1,
  fcm: { action: 'android.intent.action.MAIN' },
  collapse_key: 'xxxxxxxxxxx',
  'google.message_id': 'xxxxxxxxxxxxxxxxxxxx',
  from: 'xxxxxxxxxxxxxx',
  'google.ttl': 1111111,
  my_data_key: 'abc123',
  'google.sent_time':<PHONE_NUMBER>000
}

Note:
When my app is only in backgroud and not dead, I can handle the notifications without any problem.

i will look into that

sorry - had to be away for a couple of weeks. The issue is not resolved on my side either (I tried the change mentioned earlier - doing it outside the component lifecycle).
will try the change that dowgsss has mentioned as working for him.

@dowgsss A bit confused here.

When app is running in background do you get notification callback when you tap on notification?
When app is inactive (I don't know if android persist the state of app or not), do you get notification callback when you tap on notification?

@evollu

Yes, through the listener FCM.on(FCMEvent.Notification, fn).
In this case, through the getInitialNotification().

case 2: You don't get any notification callback?

case 2: You don't get any notification callback?

No, I'm getting the callback data

@dowgsss then why not just act on the callback data?

then why not just act on the callback data?

Because, following those steps I've explained, the data is not arriving correctly. It's coming up with the old notification data.

I opened another app and turned off the screen, waited for couple minutes, come back and tap the notification. I will get a notification callback triggered

Through the getInitialNotification()?

getInitialNotification() should not be updated because the app is just suspended, not killed.

getInitialNotification() should not be updated because the app is just suspended, not killed. The app status is persisted. Are you still on the same page when you resume the app?

No, it's like the app starts again.

I tested the emulator by following the suggested steps, but although the device was idle, the app was not killed by the system. So in this case yes, when I opened the notification / app I came across the same screen, the app was not loaded again.

To see the problem the app must be killed by the system (not by the user).

I tried again on the emulator forcing the app to be killed by the system. Follow these steps:
https://stackoverflow.com/a/34519656/9070081
But besides pressing the home button, you should start another app to kill the app process that has been minimized.

thanks, I can reproduce it now.

@dowgsss I've updated readme with the fix. thanks

@bhosle-git let me know if the solution works for you.
it is in readme-> Android configuration -> last bullet point

ok. the earlier change alone (that dowgsss pointed out from https://stackoverflow.com/a/18307360/9070081 didn't seem to have worked)..
will try your fix. thanks.

@bhosle-git try putting getInitNotification inside component lifecycle

Before I had
FCM.getInitialNotification().then(notif => {
  if(notif){
      console.log('Got initial notificaton: ' + JSON.stringify(notif));
  }
});

in the componentDidMount method, but that didnt triggered the initial notification at all.
After put it outside the App class it worked.

@karlmosenbacher that's weird. this call should work any time.


How can i just package the armeabi's *.so lib?

Hello,thanks for provide this useful pdf lib.
when i assemble my apk,the apk's size is so big,it contains armeabi,arm64-v8a,armeabi-v7a,mips,x86 and x86_64 folders,but i just want to package armeabi floder into apk,what should i do?

https://github.com/barteksc/AndroidPdfViewer/issues/609#issuecomment-529188369


docs : changed "an RBAC" to "a RBAC"

changed from "an RBAC" to "a RBAC"
See issue at https://kubernetes.io/docs/reference/access-authn-authz/rbac/#role-and-clusterrole

Thanks for the PR, but the pattern "an ___" is more natural if the following word starts with a vowel sound, which "RBAC" does ("arrrr-back")
/close


[issue-15] reasonably support older linux distributions

Targets Centos5+ abi compatibility.  Uses same rust compiler version as the current Jessy8 based image, but uses the older libc-2.5 abi vs libc-2.19.
This could help enable a path towards binary releases in the future.

Showing the contents of the container with the change.
[root@f615b1063840 app]# ldd /usr/local/bin/linkerd-tcp
	linux-vdso.so.1 =>  (0x00007ffcbeb80000)
	libdl.so.2 => /lib64/libdl.so.2 (0x00007feb41055000)
	librt.so.1 => /lib64/librt.so.1 (0x00007feb40e4b000)
	libpthread.so.0 => /lib64/libpthread.so.0 (0x00007feb40c2f000)
	libgcc_s.so.1 => /lib64/libgcc_s.so.1 (0x00007feb40a21000)
	libc.so.6 => /lib64/libc.so.6 (0x00007feb406c7000)
	/lib64/ld-linux-x86-64.so.2 (0x000055adcfe9d000)
	libm.so.6 => /lib64/libm.so.6 (0x00007feb40444000)

[root@f615b1063840 app]# rpm -qf /lib64/libc.so.6
glibc-2.5-123.el5_11.3

[root@f615b1063840 app]# /usr/local/bin/linkerd-tcp --help
linkerd-tcp 0.0.2
A native TCP proxy for the linkerd service mesh

USAGE:
    linkerd-tcp <PATH>

FLAGS:
    -h, --help       Prints help information
    -V, --version    Prints version information

ARGS:
    <PATH>    Config file path.

Hey @JustinVenus, thanks for putting this together! This approach looks good, but would you mind putting it in a separate Dockerfile (maybe Dockerfile-centos or similar)? @stevej and I have been talking about ways to slim down the default docker image that we produce, so I think this file is going to change substantially as part of that effort. If we put these changes in a separate file, then we won't need to be as mindful about image size there.

@klingerf I've made the suggested changes.  Thanks for the quick feedback 😄

@JustinVenus Ok, great, thanks for updating. Can you also add a note about how to use this new Dockerfile in the Packaging section of the README? Sorry, should have mentioned yesterday. Otherwise looks good to me, but I want @olix0r to have a look too.

@klingerf I was looking at what you've posted for issue 20 and you inspired me.  How about a build script instead of a docker image?   The output could be used for binary releases (which I personally prefer to docker images).
#!/bin/bash

IMAGE=alexcrichton/rust-slave-dist:2015-10-20b
TARGET=x86_64-unknown-linux-gnu
RUST_VERSION=1.16.0
RUST_SOURCE="https://static.rust-lang.org/dist/rust-${RUST_VERSION}-${TARGET}.tar.gz"

if [[ -d "target/${TARGET}" ]]; then
    rm -rf "target/${TARGET}"
fi

docker run \
    --rm -v `pwd`:/rust/app \
    -u root \
    -w /rust/app \
    --entrypoint=/bin/bash \
    ${IMAGE} \
    -c "cd /tmp && \
    (curl -L ${RUST_SOURCE} | gzip -dc | tar xf -)&& \
    ./rust-${RUST_VERSION}-${TARGET}/install.sh --without=rust-docs && \
    cd - && \
    cargo build --release --target=${TARGET}"

The produced binary works on (Ubuntu-12.04 and Centos5, I suspect newer distros will work just the same) w/o requiring any installation of extra packages.
$ ./linkerd-tcp --help
linkerd-tcp 0.0.2
A native TCP proxy for the linkerd service mesh

USAGE:
    linkerd-tcp <PATH>

FLAGS:
    -h, --help       Prints help information
    -V, --version    Prints version information

ARGS:
    <PATH>    Config file path.

$ ldd linkerd-tcp
	linux-vdso.so.1 =>  (0x00007fff5d129000)
	libdl.so.2 => /lib/x86_64-linux-gnu/libdl.so.2 (0x00007fd26c731000)
	librt.so.1 => /lib/x86_64-linux-gnu/librt.so.1 (0x00007fd26c529000)
	libpthread.so.0 => /lib/x86_64-linux-gnu/libpthread.so.0 (0x00007fd26c30b000)
	libgcc_s.so.1 => /lib/x86_64-linux-gnu/libgcc_s.so.1 (0x00007fd26c0f5000)
	libc.so.6 => /lib/x86_64-linux-gnu/libc.so.6 (0x00007fd26bd37000)
	/lib64/ld-linux-x86-64.so.2 (0x00007fd26cfda000)
	libm.so.6 => /lib/x86_64-linux-gnu/libm.so.6 (0x00007fd26ba3a000)

$ cat /etc/lsb-release
DISTRIB_ID=Ubuntu
DISTRIB_RELEASE=12.04
DISTRIB_CODENAME=precise
DISTRIB_DESCRIPTION="Ubuntu 12.04.5 LTS"

@JustinVenus Totally -- I think build script is a better approach, since we're not intending for folks to use the centos docker image, just the binary. I'm not 100% sure how we want this repo organized (@olix0r's back on Monday and can comment), but I'd suggest creating a new bin/ dir, and putting the script in there.

@klingerf not sure how you all feel about Make, but I've updated this review with a Makefile and documentation in the /README.md.


sshconfig: add indent file

Add indent file for sshconfig filetype.  Have it indent "Host" and
"Match" lines at indent level 0 and lines under those sections at indent
level 1.

Codecov Report

Merging #6062 into master will increase coverage by 0.19%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #6062      +/-   ##
==========================================
+ Coverage   87.08%   87.28%   +0.19%
==========================================
  Files         142      142
  Lines      156771   156828      +57
==========================================
+ Hits       136529   136883     +354
+ Misses      20242    19945     -297

Impacted Files
Coverage Δ

src/netbeans.c
74.66% <0.00%> (-0.52%)
:arrow_down:

src/gui.c
63.40% <0.00%> (-0.51%)
:arrow_down:

src/popupwin.c
94.34% <0.00%> (-0.46%)
:arrow_down:

src/gui_gtk.c
31.05% <0.00%> (-0.28%)
:arrow_down:

src/if_xcmdsrv.c
88.55% <0.00%> (-0.18%)
:arrow_down:

src/sign.c
94.77% <0.00%> (-0.18%)
:arrow_down:

src/vim9compile.c
93.61% <0.00%> (-0.04%)
:arrow_down:

src/evalvars.c
95.23% <0.00%> (-0.03%)
:arrow_down:

src/evalfunc.c
96.70% <0.00%> (-0.01%)
:arrow_down:

src/version.c
92.13% <0.00%> (ø)

... and 30 more

Continue to review full report at Codecov.

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

I'll include it, thanks.  Don't forget to send updates when you make improvements.


Exception while creating an azure container app with dapr components

This is autogenerated. Please review and update as needed.
Describe the bug
Command Name
az extension update
Errors:

To Reproduce:
Steps to reproduce the behavior. Note that argument values have been redacted, as they may contain sensitive information.

Put any pre-requisite steps here...
az extension update -n {}

Expected Behavior
Environment Summary
Windows-10-10.0.19041-SP0
Python 3.8.9
Installer: MSI

azure-cli 2.34.1

Extensions:
aro 1.0.0
containerapp 0.2.0

Dependencies:
msal 1.16.0
azure-mgmt-resource 20.0.0

Additional Context

route to CXP team

@ShafeerAli Can you please share the exception details?

@SaurabhSharma-MSFT created another issue with proper details pls refer to
https://github.com/Azure/azure-cli/issues/20862

@ShafeerAli Thanks. We will now proceed to close this thread. If there are further questions regarding this matter, please reopen it and we will gladly continue the discussion.


[VL] Add validation for coalesce operator

What changes were proposed in this pull request?
This PR fixed the failure below.
val data = "2019-09-09 01:02:03.456789"
val df = Seq(data).toDF("strTs").selectExpr(s"CAST(strTs AS TIMESTAMP_NTZ) AS ts")
df.coalesce(1).write.format("parquet").save("/tmp/test")

 org.apache.gluten.exception.GlutenException: Input schema contains unsupported type when convert row to columnar for StructType(StructField(ts,TimestampNTZType,true)) due to Schema / data type not supported: TimestampNTZType
  at org.apache.gluten.execution.RowToVeloxColumnarExec.$anonfun$doExecuteColumnarInternal$1(RowToVeloxColumnarExec.scala:54)
  at scala.Option.foreach(Option.scala:407)
  at org.apache.gluten.execution.RowToVeloxColumnarExec.doExecuteColumnarInternal(RowToVeloxColumnarExec.scala:51)
  at org.apache.gluten.execution.RowToColumnarExecBase.doExecuteColumnar(RowToColumnarExecBase.scala:62)
  at org.apache.spark.sql.execution.SparkPlan.$anonfun$executeColumnar$1(SparkPlan.scala:222)
  at org.apache.spark.sql.execution.SparkPlan.$anonfun$executeQuery$1(SparkPlan.scala:246)
  at org.apache.spark.rdd.RDDOperationScope$.withScope(RDDOperationScope.scala:151)
  at org.apache.spark.sql.execution.SparkPlan.executeQuery(SparkPlan.scala:243)
  at org.apache.spark.sql.execution.SparkPlan.executeColumnar(SparkPlan.scala:218)
  at org.apache.gluten.execution.ColumnarCoalesceExec.doExecuteColumnar(ColumnarCoalesceExec.scala:46)

How was this patch tested?
UT.

Aslo create a separate task to track the support for TimestampNTZType
#6407

cc @zhztheplayer  and @PHILO-HE , thanks

Run Gluten Clickhouse CI

+1 with @PHILO-HE

I thought this is just the missed case, as I saw this check is already added for broadcastExchange, union. But sure, let me move this to GlutenPlan.

@zhli1142015, then, could you help remove the redundant check for Union, BroadcastExchange, RowToColumnar? With RowToColumnar added prior to GlutenPlan whose schema check passes, we can infer RowToColumnar's same schema check also passes.

@zhli1142015, then, could you help remove the redundant check for Union, BroadcastExchange, RowToColumnar? With RowToColumnar added prior to GlutenPlan whose schema check passes, we can infer RowToColumnar's same schema check also passes.

Done, thanks.

===== Performance report for TPCH SF2000 with Velox backend, for reference only ====

query
log/native_master_07_11_2024_time.csv
log/native_master_07_10_2024_e8e93e73c_time.csv
difference
percentage

q1
36.56
33.46
-3.099
91.52%

q2
23.66
28.57
4.902
120.71%

q3
38.89
38.44
-0.456
98.83%

q4
32.48
35.14
2.664
108.20%

q5
70.11
71.06
0.950
101.36%

q6
7.94
6.45
-1.495
81.18%

q7
85.48
83.24
-2.233
97.39%

q8
84.74
88.10
3.353
103.96%

q9
120.17
119.97
-0.203
99.83%

q10
46.28
43.77
-2.508
94.58%

q11
20.26
20.45
0.195
100.96%

q12
23.58
26.84
3.256
113.81%

q13
39.57
38.42
-1.152
97.09%

q14
17.94
20.94
3.001
116.73%

q15
32.84
30.00
-2.846
91.33%

q16
13.99
15.85
1.856
113.27%

q17
104.06
105.14
1.081
101.04%

q18
146.03
145.23
-0.798
99.45%

q19
14.91
14.76
-0.156
98.95%

q20
28.48
26.50
-1.976
93.06%

q21
265.32
267.51
2.186
100.82%

q22
12.26
12.07
-0.186
98.48%

total
1265.54
1271.88
6.336
100.50%


Make src/system compile with -Werror=conversion

Problem
src/system does not compile with -Wconversion
Summary of Changes
Make it compile
fixes https://github.com/project-chip/connectedhomeip/issues/2985

@andy31415 @mspang ?


The tool comes to the top and steals the focus

After launching the tool, every few minutes the tool comes to the top of all the open windows and steals the focus causing me to clock on what I was previously focused on.

This is a known issue, design flaw, that needs to be addressed as soon as I get a chance.
Thanks for bringing it up.


chore(doc): explain ComponentWithFormTrait getDataModelValue override

Q
A

Bug fix?
no

New feature?
no

Tickets
improve doc

Copy, i will take of that asap! Thx for commenting

Friendly ping!

@weaverryan thx for the ping, I've tried to handle your latest comments, hope I understand them well
otherwise please feel free to adjust the rst format :)

Thanks @94noni!


fixes issue #1760 group 8

Ⅰ. Describe what this PR did
fixes issue #1760
Ⅱ. Does this pull request fix one issue?

Ⅲ. Describe how you did it
Ⅳ. Describe how to verify it
Ⅴ. Special notes for reviews

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


lots of changes and George added to DESCRIPTION

Hi @jbkunst and @gvegayon
You may know that Dave is here. He's not an R user but he created DataUSA and D3plus.
I made lots of changes that affect how easy is to create a geomap. Because of this I made tibble the default data structure to work with.
To begin with tonight's changes I added the product space from oec package. The network structure is ok, what's not ok is the grouping/colouring option.
For example you can run:
load(system.file("extdata", "product_space/hs92_4_usa_exports_2015.RData", package = "d3plus"))
load(system.file("extdata", "product_space/hs92_4_links.RData", package = "d3plus"))
load(system.file("extdata", "product_space/hs92_4_nodes.RData", package = "d3plus"))

hs92_4_usa_exports_2015 = hs92_4_usa_exports_2015 %>%
  inner_join(hs92_4_nodes) %>%
  select(id, val, color)

d3plus() %>%
  d3p_type("network") %>%
  d3p_data(hs92_4_usa_exports_2015) %>%
  d3p_nodes(hs92_4_nodes) %>%
  d3p_links(hs92_4_links) %>%
  d3p_colorscale("{fill: function(d) {return d.color;}}")

And this will write the network but assigning random colours. This is weird because I still need to find out a valid way to pass the color column from the data to D3plus 2.0. oec package uses D3plus 1.9 where lots of methods were different and in that package colours, groups and legends are ok.
I'd also like to pass custom colours to treemaps, so that you can do something like this:
dta4 <- tibble(parent = c(rep("Group 1",3), rep("Group 2",2)),
               id = c("alpha","beta","gamma","delta","eta"),
               col = c("#333333","yellow","#fafafa", "red","blue"),
               value = c(29,10,2,29,25))

d3plus() %>%
  d3p_type("treemap") %>%
  d3p_data(dta4) %>%
  d3p_groupby(c("parent","id")) %>%
  d3p_color("col")

I know you are great R programmers and I'm sure your contribution will make this package great !! I don't care if you love the tidyverse binaries or compile data.table on Arch, this is about the chicken and not the egg :)

@gvegayon I just moved the data to data/

Cool


valis requires and outdated version of pydantic through ome-types

The latest release of valis-wsi installed by pip results in ome-types==0.3.4 being installed. This in turn requires a v1-era pydantic, which is  outdated. The latest release ome-types==0.5.1.post1 supports both pydantic v1 and v2 through pydantic_compat. This is a major imporovement in compatibility of ome-types which is not yet adopted by valis-wsi.
Could you possibly update to the latest ome-types?

Hi @iRyoka,
Thanks for bringing this to my attention. I'll be sure to update the pyproject.toml and poetry.lock files to have the latest version of ome-types in the next update, which should be pretty soon.
Best,
-Chandler

Hi @iRyoka,
Just wanted to let you know the latest release of valis (1.1.0) has updated ome-types to 0.5.1.post1. I'll go ahead and close the issue, but if you run into any problems, please let me know.
Best,
-Chandler


Working with rasa-nlu 0.13.3

pipeline:
- name: "nlp_spacy"
- name: "tokenizer_spacy"
- name: "ner_crf"
- name: "tokenizer_whitespace"
- name: "intent_featurizer_count_vectors"
- name: "intent_classifier_tensorflow_embedding"
  intent_tokenization_flag: true
  intent_split_symbol: "+"

Thanks for your help, I was having similar problems with my pipeline and this helped. I also wanted to use look up tables in my chatbot and noticed that this pipeline doesn't seem to function with the up to date rasa-nlu version.
Do you have any suggestions?

Please upgrade to version 0.13.4 🙂 if you have any usage related questions about the lookup table, please post them in our forum


Ogg Vorbis playback not working due to typo/oversight

There appears to be a typo/oversight in client.py which is causing Ogg Vorbis playback to not work:
https://github.com/home-assistant-libs/aioslimproto/blob/a1da24578456ff9f7a8fc3f69c166cde67632b84/aioslimproto/client.py#L184
The format-byte for Ogg should be o (lowercase "Oh") not 0 (zero) – see also the Wiki-page referenced in the comment above the definition itself: https://wiki.slimdevices.com/index.php/SlimProto_TCP_protocol.html#Command:_.22strm.22

Thanks!
And sorry for the late response btw


HA error with version 1.13.0

When I try to use version 1.13.0 in the current HA-integration I get the following error:
2024-11-09 10:11:32.071 ERROR (MainThread) [homeassistant] Error doing job: Task exception was never retrieved (None):   File "/home/vscode/.local/ha-venv/bin/hass", line 8, in <module>
    sys.exit(main())
  File "/workspaces/homeassistant_core/homeassistant/__main__.py", line 209, in main
    exit_code = runner.run(runtime_conf)
  File "/workspaces/homeassistant_core/homeassistant/runner.py", line 189, in run
    return loop.run_until_complete(setup_and_run_hass(runtime_config))
  File "/usr/local/lib/python3.12/asyncio/base_events.py", line 674, in run_until_complete
    self.run_forever()
  File "/usr/local/lib/python3.12/asyncio/base_events.py", line 641, in run_forever
    self._run_once()
  File "/usr/local/lib/python3.12/asyncio/base_events.py", line 1978, in _run_once
    handle._run()
  File "/usr/local/lib/python3.12/asyncio/events.py", line 88, in _run
    self._context.run(self._callback, *self._args)
  File "/workspaces/homeassistant_core/homeassistant/config_entries.py", line 774, in async_setup_locked
    await self.async_setup(hass, integration=integration)
  File "/workspaces/homeassistant_core/homeassistant/config_entries.py", line 537, in async_setup
    await self.__async_setup_with_context(hass, integration)
  File "/workspaces/homeassistant_core/homeassistant/config_entries.py", line 626, in __async_setup_with_context
    result = await component.async_setup_entry(hass, self)
  File "/workspaces/homeassistant_core/config/custom_components/abbfreeathome_ci/__init__.py", line 86, in async_setup_entry
    entry.async_create_background_task(hass, _free_at_home.ws_listen(), f"{DOMAIN}_ws")
  File "/workspaces/homeassistant_core/homeassistant/config_entries.py", line 1166, in async_create_background_task
    task = hass.async_create_background_task(target, name, eager_start)
  File "/workspaces/homeassistant_core/homeassistant/core.py", line 864, in async_create_background_task
    task = create_eager_task(target, name=name, loop=self.loop)
  File "/workspaces/homeassistant_core/homeassistant/util/async_.py", line 45, in create_eager_task
    return Task(coro, loop=loop, name=name, eager_start=True)
Traceback (most recent call last):
  File "/home/vscode/.local/ha-venv/lib/python3.12/site-packages/abbfreeathome/freeathome.py", line 174, in ws_listen
    await self.api.ws_listen(callback=self.update_device)
  File "/home/vscode/.local/ha-venv/lib/python3.12/site-packages/abbfreeathome/api.py", line 261, in ws_listen
    await self.ws_receive(callback, retry_interval)
  File "/home/vscode/.local/ha-venv/lib/python3.12/site-packages/abbfreeathome/api.py", line 287, in ws_receive
    await callback(_ws_data)
  File "/home/vscode/.local/ha-venv/lib/python3.12/site-packages/abbfreeathome/freeathome.py", line 182, in update_device
    _device.update_device(_datapoint_key, _datapoint_value)
  File "/home/vscode/.local/ha-venv/lib/python3.12/site-packages/abbfreeathome/devices/base.py", line 112, in update_device
    if _refreshed and self._callbacks:
       ^^^^^^^^^^
UnboundLocalError: cannot access local variable '_refreshed' where it is not associated with a value

It seems to be related to my changes in #103

And here, if I run a program only with the lib:
INFO:local_abbfreeathome.src.abbfreeathome.devices.base:Bewegungsmelder Flur zu Büro  received updated data: ABB700DA100B/ch0000/idp0000: 1
Traceback (most recent call last):
  File "c:\Users\joerg\programming\new_freeathome\test.py", line 79, in <module>
    asyncio.run(_free_at_home.ws_listen())
  File "C:\Program Files\WindowsApps\PythonSoftwareFoundation.Python.3.12_3.12.2032.0_x64__qbz5n2kfra8p0\Lib\asyncio\runners.py", line 194, in run
    return runner.run(main)
           ^^^^^^^^^^^^^^^^
  File "C:\Program Files\WindowsApps\PythonSoftwareFoundation.Python.3.12_3.12.2032.0_x64__qbz5n2kfra8p0\Lib\asyncio\runners.py", line 118, in run
    return self._loop.run_until_complete(task)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "C:\Program Files\WindowsApps\PythonSoftwareFoundation.Python.3.12_3.12.2032.0_x64__qbz5n2kfra8p0\Lib\asyncio\base_events.py", line 687, in run_until_complete
    return future.result()
           ^^^^^^^^^^^^^^^
  File "c:\Users\joerg\programming\new_freeathome\local_abbfreeathome\src\abbfreeathome\freeathome.py", line 174, in ws_listen
    await self.api.ws_listen(callback=self.update_device)
  File "c:\Users\joerg\programming\new_freeathome\local_abbfreeathome\src\abbfreeathome\api.py", line 261, in ws_listen
    await self.ws_receive(callback, retry_interval)
  File "c:\Users\joerg\programming\new_freeathome\local_abbfreeathome\src\abbfreeathome\api.py", line 287, in ws_receive
    await callback(_ws_data)
  File "c:\Users\joerg\programming\new_freeathome\local_abbfreeathome\src\abbfreeathome\freeathome.py", line 182, in update_device
    _device.update_device(_datapoint_key, _datapoint_value)
  File "c:\Users\joerg\programming\new_freeathome\local_abbfreeathome\src\abbfreeathome\devices\base.py", line 112, in update_device
    if _refreshed and self._callbacks:
       ^^^^^^^^^^
UnboundLocalError: cannot access local variable '_refreshed' where it is not associated with a value
ERROR:asyncio:Unclosed client session
client_session: <aiohttp.client.ClientSession object at 0x000001B1DF151F70>
Exception ignored in: <function ClientResponse.__del__ at 0x000001B1DECD4E00>
Traceback (most recent call last):
  File "C:\Users\joerg\programming\new_freeathome\.venv\Lib\site-packages\aiohttp\client_reqrep.py", line 973, in __del__
  File "C:\Users\joerg\programming\new_freeathome\.venv\Lib\site-packages\aiohttp\connector.py", line 191, in release
  File "C:\Users\joerg\programming\new_freeathome\.venv\Lib\site-packages\aiohttp\connector.py", line 686, in _release
  File "C:\Users\joerg\programming\new_freeathome\.venv\Lib\site-packages\aiohttp\client_proto.py", line 69, in close
  File "C:\Program Files\WindowsApps\PythonSoftwareFoundation.Python.3.12_3.12.2032.0_x64__qbz5n2kfra8p0\Lib\asyncio\proactor_events.py", line 109, in close
  File "C:\Program Files\WindowsApps\PythonSoftwareFoundation.Python.3.12_3.12.2032.0_x64__qbz5n2kfra8p0\Lib\asyncio\base_events.py", line 795, in call_soon
  File "C:\Program Files\WindowsApps\PythonSoftwareFoundation.Python.3.12_3.12.2032.0_x64__qbz5n2kfra8p0\Lib\asyncio\base_events.py", line 541, in _check_closed
RuntimeError: Event loop is closed

The problem is, I think, that it can happen, that - for whatever reason - an "input-datapoint" is sent on the websocket.


Update dependencies and update edition to 2018

This crate has been very useful to me personally over the years. I'm trying to include this in another project where we would like the dependencies to be up to date, hence the reason for this PR.
Thanks!

Thanks!

Thanks!


Adds documentation to the opencomputers methods

Addresses #535
Most of the methods are kind of self-explanatory, but I've added a comment for everything.
If approved, I'll write a page for the opencomputers wiki / nuclearcraft wiki.

Oops... pushed to the wrong branch. Don't think I can correct this without closing the PR. See here : #600


Date format is incompatible with stripe.js

Your docs for making custom forms link to this plugin, but this plugin assumes the expiration date is a single field, whereas stripe.js requires that expiration date be sent as 2 different fields (month and year). I see a pull request (#38) related to this issue, but it hasn't been merged.
I'd assume that since you're linking here from the official stripe.js documentation that this can actually be used out-of-the-box with stripe.js, but it appears that it can't? (Or perhaps I'm missing something and it is possible?). I'd suggest addressing this in the docs because it is super confusing (at least to me, as a developer who is new to stripe).

I just found https://github.com/stripe/jquery.payment#paymentcardexpiryvalstring-and-fnpaymentcardexpiryval , which seems to explain how to extract month and year from a single field. So I guess that's how you do it... but would be really nice if the docs and examples mentioned this since it seems like integrating with stripe.js would be the number 1 primary use case.

I agree that this is confusing. Do you expect us to not use the payment('formatCardExpiry') when using stripe.js?
Or do you expect us to not use the data-stripe= when using jquery.payment?
Is this an oversight, or just an extra feature that's for non-stripe users?
An example using both of the libraries would be very useful.

:+1:  I'm also confused by this! Having stripe.js handle this correctly would be great.

+1 please fix this one

:+1:

Still waiting for a fix...

:+1:

:+1:

@metcalf does you closing this without comment mean you disagree?

@sjelfull much better news than that! It closed automatically when I merged a fix for this problem into Stripe.js. You can now pass an exp parameter to createToken with a single expiration string. It will handle spaces, dashes, dots and slashes as a separator and handle both year-first and month-first locales. Note that in order to handle both locales, it assumes you'll only pass values with a year after 2012. Otherwise we can't distinguish between "11 / 12" being "Nov 2012" or "Dec 2011". In practice this should be a non-issue since dates before today will error anyway.

That is supergreat! Thanks for fixing it. Adds to the developer bliss I personally feel working with Stripe!

Is the validator for this field type still missing?

@LeleDev: You can now pass the single string as the first argument to validateExpiry. Sorry for the lack of documentation -- still thinking through the best place to document this additional behavior.

Totaly true. Doc says to create 2 field expiration input but scrolling down you see that validation support only 1 field expiration input. I hope you'll fix that, I mean you made a lot of effort to build a good framework but if devs lost many times on such simple stuff it's a waste and doesn't offer a good impression.
Thanks.


Mac Fullscreen App Right-click Focus

Type: Bug

in macOS, split-full-screen VSCode with any other app
focus on the other app
right-click on VSCode while focused in the other app
now left-click to focus into VSCode

right-click is disabled in VSCode until I focus back into any other app and then left-click back into VSCode.

Btw the setting window.clickThroughInactive might have an impact.

Nope, no third-party window manager.
Couple things though:

I use a Logitech MX Master 3 mouse that uses the Logi Options+ configuration software. So, I thought it might be the mouse+software, but I tested with a wired mouse and had the same issue.
as added info, ctrl + left-click also does not engage the contextual menu once you have right-clicked into VSCode from another app in split-full-screen, but, you can ctrl + left-click from the other app and get the contextual menu in VSCode.

Another bit of info: The issue happens when not in fullscreen, too. Fullscreen or windowed, no difference. Updated the title of this issue.

Well, this might be a macOS issue...
I have found that if I right-click into Edge while focused in another app, and then focus into Edge, I can still right-click but all cmd + action keystrokes don't work. The plot thickens...so, I will close this issue since this has grown bigger than just VSCode.


Rewrite random_interaction_operator and allow complex option

I'm investigating a bug and found it useful to understand this function. I couldn't so I rewrote it and added an option to generate a complex operator, and tested it as well.
However, I'm a little confused about the exposition of symmetries in InteractionOperator from the OpenFermion paper, specifically the h_{pqrs} = h_{sqrp} = .... Are these symmetries derived only from the hermiticity of the Hamiltonian? If so, then I don't see why, for instance, h_{pqrs} = h_{prqs}; if r and q are different I think these should be the negative of each other. I think the symmetries implied from my code are correct.

Without looking specifically at that function, the integral symmetries come
from the interaction operator as well as the assumption that the basis
functions are real.  A more detailed exposition can be found here:
http://vergil.chemistry.gatech.edu/notes/permsymm/permsymm.pdf
which talks about the reduced symmetry in the complex case as well
On Fri, Mar 23, 2018 at 9:12 AM, Kevin J. Sung<EMAIL_ADDRESS>wrote:

I'm investigating a bug and found it useful to understand this function. I
couldn't so I rewrote it and added an option to generate a complex
operator, and tested it as well.
However, I'm a little confused about the exposition of symmetries in
InteractionOperator from the OpenFermion paper, specifically the h_{pqrs}
= h_{sqrp} = .... Are these symmetries derived only from the hermiticity
of the Hamiltonian? If so, then I don't see why, for instance, h_{pqrs} =
h_{prqs}; if r and q are different I think these should be the negative
of each other. I think the symmetries implied from my code are correct.
You can view, comment on, or merge this pull request online at:
https://github.com/quantumlib/OpenFermion/pull/282
Commit Summary

rewrite random_interaction_operator

File Changes

M src/openfermion/utils/_testing_utils.py
https://github.com/quantumlib/OpenFermion/pull/282/files#diff-0 (84)
M src/openfermion/utils/_testing_utils_test.py
https://github.com/quantumlib/OpenFermion/pull/282/files#diff-1 (21)

Patch Links:

https://github.com/quantumlib/OpenFermion/pull/282.patch
https://github.com/quantumlib/OpenFermion/pull/282.diff

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/quantumlib/OpenFermion/pull/282, or mute the thread
https://github.com/notifications/unsubscribe-auth/AHXFPBw42cKJHpuYzFv7SvFz4UKwfL8oks5thR8AgaJpZM4S4_hB
.

Thanks for the reference Jarrod, I think that will be helpful. I wrote my code only on the assumption that the output should be Hermitian. Perhaps that's not exactly what we want.

I don't understand why coveralls is saying coverage descreased even though I added 0 untested lines.

who knows? I'll merge it.


Panic on tcmu_queue_cmd_ring

Latest tcmu-runner from master used for sharing RBD volumes with Ceph gwcli. Reading the volumes on client using dd via multipathd device is causing kernel panic (even with only one iSCSI-gateway active). Reading the volumes via generic paths (/dev/sdX) is not causing the panics (and performs twice as faster). Kernels tested are 4.12.0 and 4.12.5 from ELRepo on CentOS 7.3.
multipath.conf:
devices {
        device {
                vendor                 "LIO-ORG"
                hardware_handler       "1 alua"
                path_grouping_policy   "failover"
                path_selector          "queue-length 0"
                failback               60
                path_checker           tur
                prio                   alua
                prio_args              exclusive_pref_bit
                no_path_retry          120
        }
}

vmcore-dmesg.txt:
[  118.342643] BUG: unable to handle kernel paging request at ffffc900236bd2f0
[  118.342720] IP: memcpy_orig+0xbd/0x110
[  118.342752] PGD 237114067
[  118.342753] P4D 237114067
[  118.342777] PUD 237115067
[  118.342800] PMD 0
[  118.342828]
[  118.342865] Oops: 0002 [#1] SMP
[  118.342893] Modules linked in: target_core_pscsi target_core_file target_core_iblock iscsi_target_mod vmw_vsock_vmci_transport vsock target_core_user uio target_core_mod sb_edac crct10dif_pclmul crc32_pclmul ghash_clmulni_intel pcbc ppdev vmw_balloon aesni_intel crypto_simd glue_helper cryptd intel_rapl_perf pcspkr input_leds vmw_vmci i2c_piix4 sg shpchp parport_pc parport nfsd auth_rpcgss nfs_acl lockd grace sunrpc ip_tables xfs libcrc32c sr_mod cdrom ata_generic pata_acpi sd_mod crc32c_intel serio_raw vmxnet3 vmwgfx drm_kms_helper syscopyarea sysfillrect sysimgblt fb_sys_fops ahci ttm libahci ata_piix vmw_pvscsi drm libata dm_mirror dm_region_hash dm_log dm_mod dax
[  118.343385] CPU: 0 PID: 1885 Comm: iscsi_trx Not tainted 4.12.5-1.el7.elrepo.x86_64 #1
[  118.343444] Hardware name: VMware, Inc. VMware Virtual Platform/440BX Desktop Reference Platform, BIOS 6.00 09/21/2015
[  118.343521] task: ffff880233d9c440 task.stack: ffffc90004010000
[  118.343570] RIP: 0010:memcpy_orig+0xbd/0x110
[  118.343605] RSP: 0018:ffffc90004013b40 EFLAGS: 00010202
[  118.343646] RAX: ffffc900236bd2f0 RBX: ffff880220430000 RCX: ffff880220430000
[  118.343699] RDX:<PHONE_NUMBER>00000a RSI: ffff88021f965950 RDI: ffffc900236bd2f0
[  118.343752] RBP: ffffc90004013c18 R08: 010000d7ff000028 R09:<PHONE_NUMBER>d7ff00
[  118.343805] R10: ffff88023636d7f8 R11: ffffc90004013bbc R12:<PHONE_NUMBER>000050
[  118.343859] R13:<PHONE_NUMBER>001000 R14: ffffc900036bd2b0 R15:<PHONE_NUMBER>0072f0
[  118.343913] FS:<PHONE_NUMBER>000000(0000) GS:ffff88023fc00000(0000) knlGS:0000000000000000
[  118.343973] CS:  0010 DS: 0000 ES: 0000 CR0:<PHONE_NUMBER>050033
[  118.344017] CR2: ffffc900236bd2f0 CR3:<PHONE_NUMBER>2a3000 CR4:<PHONE_NUMBER>3406f0
[  118.344148] DR0:<PHONE_NUMBER>000000 DR1:<PHONE_NUMBER>000000 DR2:<PHONE_NUMBER>000000
[  118.344202] DR3:<PHONE_NUMBER>000000 DR6: 00000000fffe0ff0 DR7:<PHONE_NUMBER>000400
[  118.344256] Call Trace:
[  118.344287]  ? tcmu_queue_cmd_ring+0x67c/0x920 [target_core_user]
[  118.344339]  ? idr_alloc+0x73/0xb0
[  118.344370]  tcmu_queue_cmd+0x11c/0x1c2 [target_core_user]
[  118.344433]  __target_execute_cmd+0x27/0xb0 [target_core_mod]
[  118.344489]  target_execute_cmd+0x252/0x2e0 [target_core_mod]
[  118.344543]  transport_generic_new_cmd+0x118/0x2f0 [target_core_mod]
[  118.344602]  transport_handle_cdb_direct+0x3a/0x90 [target_core_mod]
[  118.344665]  iscsit_execute_cmd+0x232/0x280 [iscsi_target_mod]
[  118.344720]  iscsit_sequence_cmd+0xdc/0x190 [iscsi_target_mod]
[  118.344774]  iscsit_process_scsi_cmd+0x25/0xd0 [iscsi_target_mod]
[  118.344830]  iscsit_handle_scsi_cmd+0x93/0x380 [iscsi_target_mod]
[  118.344885]  iscsit_get_rx_pdu+0x35c/0x910 [iscsi_target_mod]
[  118.344934]  ? pick_next_task_fair+0x332/0x550
[  118.344999]  iscsi_target_rx_thread+0xb9/0xf0 [iscsi_target_mod]
[  118.345047]  kthread+0x109/0x140
[  118.346837]  ? iscsi_target_tx_thread+0x1f0/0x1f0 [iscsi_target_mod]
[  118.348601]  ? kthread_park+0x60/0x60
[  118.350349]  ret_from_fork+0x25/0x30
[  118.352095] Code: 08 4c 8b 54 16 f0 4c 8b 5c 16 f8 4c 89 07 4c 89 4f 08 4c 89 54 17 f0 4c 89 5c 17 f8 c3 90 83 fa 08 72 1b 4c 8b 06 4c 8b 4c 16 f8 <4c> 89 07 4c 89 4c 17 f8 c3 66 2e 0f 1f 84 00 00 00 00 00 83 fa
[  118.355813] RIP: memcpy_orig+0xbd/0x110 RSP: ffffc90004013b40
[  118.357636] CR2: ffffc900236bd2f0

Can provide vmcore if needed.

How did you discover that gwcli and rbd support?
I only ask because we still are working on the docs here:
https://github.com/ceph/ceph/pull/16182
and I do not think the required kernel version got updated yet, and I am thinking if you did not see that link then we need to add links to the doc in the ceph-iscsi-*  and tcmu-runner READMEs.
Your problem should be fixed in 4.13 when it comes out. 4.13-rc4 should have the fix if you can test that.

The reason you are crashing is because you need to be on 4.13.y+ There was a fix for memory leaking/OOPs when recalculating cmd base size.
Similar trace to yours: https://patchwork.kernel.org/patch/9834347/

I saw the ceph/ceph#16182 (posted by @dillaman on ceph-users) and yes, there is still 4.12 in requirements.
Thank you, will try with 4.13.

@dglushenok I just got a stable review patch for it so it will eventually trickle down.

The above patch is merged upstream. Closing.


Set the Vary header.

This PR sets the Vary HTTP header to inform caches that our responses will vary based on what the client requests with the Accept header.  Without this, caches will assume that a cached response can be used even if the Accept header differs.
Additional Reading (just in case you are stuck in a meeting)

https://www.varnish-cache.org/docs/3.0/tutorial/vary.html
http://mark.koli.ch/understanding-the-http-vary-header-and-caching-proxies-squid-etc
http://asm89.github.io/2012/09/26/context-aware-http-caching.html

Seems reasonable to me.


Display full addresses on send flow

From Ledger team:

On Ledger Nano S, on a send flow, the receiving address is truncated like 9zdf21...13ez1z, it would be better to display the whole address, even if it adds more screens to go through.

have you checked the latest version from this repo? I think I already implemented it

https://github.com/tesseract-one/ledger-app-ergo/commit/a450a3422885e7dd9e40af451083e2304e7d0ff8


Not reporting automatically?

Idk if I'm just dumb but it's not reporting anything automatically.
I've driven with the device on and it doesn't send or update anything unless I open the menu and select send now.
Also, I noticed that at some point the screen turns off and I can't turn it on unless I reset the device.
For instance when it sleeps, I press the button and it wakes up and the screen turns off again and I can't do anything to turn it on. (Wondering if there's something defective with my board)
Thanks!

See if that latest version fixes it. There was an issue with the latest version of the Heltec platform libraries that I hope are fixed now. The older versions were allowing me to do some things that the new version does not allow, so I had to implement a workaround.

Hey @hkicko thanks! Sorry for the delay I couldn't test it until today!
It works perfectly! Thanks!


text-generation support sampling algo

What does this PR do?
As title, user could use sampling algo for text-generation with args --do_sample True
Before submitting

[ ] This PR fixes a typo or improves the docs (you can dismiss the other checks if that's the case).
[ ] Did you make sure to update the documentation with your changes?
[ ] Did you write any new necessary tests?

@regisss  we have enabled GPT2 with sampling and in our side the vocab size shi 250K, we have done some optimizations special for the multinomial, I will submit a PR later if the optimization is acceptable, and seems not too bad with BF16. Since we are running GPT2 under BF16 , but the GPT2 gaudi_config is disable the HMP for training, so that's why I open an issue #230 to ask you if we can enable the BF16 for GPT2 training(perplexity drop a little when enable HMP)
we also have some optimizations on HPUGraph, we do capture , replay and caching recipe by ourselves, but it is a little complex , we could have a discussion it with later.


How to use HessianUpdateStrategy in  scipy.optimize.minize(hess=?)

minimize(fun,[1,2],method='dogleg',jac=gradient,hess=scipy.optimize.BFGS)
According to  the official document:
Or, objects implementing HessianUpdateStrategy interface can be used to approximate the
Hessian. Available quasi-Newton methods implementing this interface are: BFGS;SR1。
I  have a try to use "scipy.optimize.BFGS"  ,but it raise ValueError: exception_strategy must be 'skip_update' or 'damp_update'.
Besises,  I am puzzled by the official document's description of solving Hesse matrix by finite-differences，I  am  not sure which  algorithm can use   finite-differences to estimate Hess . It seems that only trust-constr can use  finite-differences ( It is right?) .  Why is the description of Hesse matrix so superfluous？

It's not possible to use 'dogleg' a a HessianUpdateStrategy, such as hess=BFGS(), it can only be used with a callable(hess).
The description of the hess keyword is improved in #15019, at which point this issue can be closed.
Note that BFGS must be instantiated before use.


Text mining evidence from Hoffmann et al. Cell paper missing for COVID-19

The Hoffmann et al. Cell paper was one of the first publications about SARS-CoV-2 that mentioned human proteins associated with the infection. The abstract of the paper contains sentences that should be picked by EPMC's text mining algorithm, for instance:

Here, we demonstrate that SARS-CoV-2 uses the SARS-CoV receptor ACE2 for entry and the serine protease TMPRSS2 for S protein priming

Here, ACE2 and TMPRSS2 should be tagged as targets and SARS-CoV-2 as the disease (it's a synonym of COVID-19-MONDO_0100096). It would also be acceptable if SARS-CoV was identified and annotated as Severe acute respiratory syndrome - EFO_0000694, which would be a false positive. However, this is not the case although there are four target-disease associations extracted from this paper:

Target
Disease

ACE2
infection - EFO_0005741

ACE2
MERS - MONDO_0100116

APN
MERS - MONDO_0100116

DDP4
MERS - MONDO_0100116

@saha-shyamasree has been looking into it and she couldn't find any obvious explanations for it but she thinks that there is something wrong in the target annotation. It is important to fix this issue because we may be missing many more similar associations.

There is still no explanation for this. @saha-shyamasree will continue working on it but it's low priority.

To be checked again next time EPMC  submit an evidence file


Add +nightly to cargo build in README

The readme says to do cargo build --release, but this fails since unstable features are used.
Instead it should be cargo +nightly build --release.

Fixed in https://github.com/stillonearth/bevy_rl_shooter/commit/4e17734e41838844ca96a42993b0d90ffb1d546f


quick question

What does POD mean and what's it used for

In C++ programming, "POD" usually stands for "Plain Old Data". The term is usually used to differentiate simple data structures from more complex ones.


Particles2D have wrong Visibility Rect display when using Particles Animation

Tested versions
3.5.3
System information
Windows10, 3.5.3
Issue description
Particls2D have wrong Visibility Rect display when using Particals Animation of CanvasItemMaterial with a spritesheet to randomize the particle, the rect's all 4 parameter(x/y/w/h) gets devided by Particles Anim H/V.
Steps to reproduce
test.zip
Minimal reproduction project (MRP)
see test project

I think this might also be affecting the generation of the visibility rect, or maybe that's a separate issue. For me it seems to generate the visibility rect about 6 times too small (also note I have the scale min/max set to 6)


Need to remove a published package from packages.microsoft.com

Describe the issue
I'd like to remove a published package. What is the process to do that? I'm sorry if this is not the right place to ask this question.

Followed up internally.


Make the focus outline translucent for editor viewports

This makes the focus outline less distracting on the 2D and 3D editor viewports. We still need to keep this outline visible as it's the only way the user can see if their key presses will go to the viewport (for actions like Focus Selection). Originally requested on Twitter.
Note: Not cherry-pickable to the 3.x branch as this only pertains to the new editor theme.
Preview
Note: These screenshots are taken at the smallest possible window size. Remember that the focus outline becomes less distracting at larger screen sizes, since it takes up a lower portion of the window.
Before

After

Make sure it's possible to change Translucency in the editor settings.

Make sure it's possible to change Translucency in the editor settings.

I wouldn't add a setting for this as it's too niche.

Make sure it's possible to change Translucency in the editor settings.

I wouldn't add a setting for this as it's too niche. Instead, we should focus on picking a good default value, and the new default seems to be good enough.

Well, I liked the old opaque one, we could possibly range the setting between 0.25 and 1.

I would like to be possible to disable the focus outline, it's too distracting for me. I understand it's for accessibility reasons, and to see where your shortcuts will be applied to, but for me, it's just too distracting.

Needs a rebase.

Rebased and tested again, it works as expected.

Thanks!


Completion of spyder4

When the code is completed, the font in the candidate area is very small, and it does not look as comfortable as the code font. I don't know what's going on? Sorry I cannot take a screenshot when the candidate box appears

Hi @ChaoYangDongYongGuaiGe,
Can you please upload a gif with the behavior you are having?

Look at this, the font of the completion candidate area is different from the font of the code editing area, which makes me very uncomfortable. Is this a bug or is it the case?

What I can see in your gif, is that it is the same font that you have in the complete path and the filename. Did you set this font in your Windows?

@steff456, I think we are not using the editor font but the one used by other elements in the operating system. Could you confirm?

Closing due to lack of response.


Update events polyfill for node 10

Based on this issue https://github.com/ionic-team/rollup-plugin-node-polyfills/issues/8
I noticed that I have 3 additional files created during build. In this repository do you expect those to be checked in?
Possibly @manucorporat

Just because I noticed @manucorporat hasn't been active on the other open PRs, possibly @mlynch or @dwieeb? No rush! Happy holidays.

Hoping this gets merged soon cause I'm having the same issue:
(!) Missing shims for Node.js built-ins
Creating a browser bundle that depends on 'events'. You might need to include https://github.com/ionic-team/rollup-plugin-node-polyfills

@FredKSchott, may be relevant for @snowpackjs?

@SamBroner we're going to go ahead and fork this repo, with plans to eventually bring under the official Rollup org. See https://github.com/rollup/plugins/pulls
Would love if you could create this PR onto our repo: https://github.com/snowpackjs/rollup-plugin-node-polyfills

Snowpack repo moved to https://github.com/snowpackjs/rollup-plugin-polyfill-node

Snowpack repo moved to https://github.com/snowpackjs/rollup-plugin-polyfill-node


Set up TypeScript with minimal configurations

For now, source code are unchanged (except for .js extensions
in import statements). We can convert .js to .ts gradually in
later changes.
For easier TypeScript/webpack configurations, extension files are moved
to the following directories:

src/    ... TS/JS source code
static/ ... Static files

Commands to build the extension:
npm install    # install dependencies
npm run build  # for one-time build
npm run watch  # for continuous build
A bundled extension is written at dist/.

Added one more commit to convert background/common.js to TS.


[Test Issue] This is a test issue to verify the GitHub automation marks the issue as needing triage

Testing https://github.com/dotnet/systemweb-adapters/pull/479

Test succeeded


Spectator v3

Coverage decreased (-0.02%) to 99.488% when pulling 9dde180505e276b00bacd091f085737d808fc9bb on spectator-v3 into 691f1a53c2be5256772888eb933495d71c3dcdc8 on master.


Add info and change to new CRDTs

Add more details to newly created CRDTs, specifying from which files they are adapted from and the corresponding changes.

My only complaint here is the long lines -- can you please fix this to keep it < 80 chars.

Done, I adjusted it manually... Is there any good way to keep track of it automatically?

My editor is configured to show a warning for long lines.


Is it possible to intercept hideOnClickOutside?

Hi! The wrapper works great, thanks for your work. Here's an issue I'd like to solve: I open the popper using a button click. I would like the popper to remain open if I click the same button again and only close if I click anywhere else. How could I achieve that?

@juona use this one!
By the way I just released<EMAIL_ADDRESS>which now works with Angular8. Will upgrade to Angular 9 and 10 in the next few days...
Demo here => https://github.com/tonysamperi/ngx-popperjs

Thanks, but unfortunately that's not what I wanted : ]

outside click closes popover = GOOD
same button click closes popover = BAD

I wish the popper to remain open if I click the same button that was used to open it in the first place. And close if clicked anywhere else.

Yeah I had already understood what you need, but I hadn't noticed that the changing disabled attribute also hides the popover...
It seems that you need a very precise solution that you could achieve with a few tricks...


kde: frameworks 5.22 -> 5.24, plasma 5.6.5 -> 5.7.0

Motivation for this change
Things done

[x] Tested using sandboxing
(nix.useChroot on NixOS,
or option build-use-chroot in nix.conf
on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] OS X
[ ] Linux

[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[x] Fits CONTRIBUTING.md.

I have obviously build tested it, and also tried booting into it.
No extensive testing has been done.
I have not modified all-packages.nix to switch to this desktop by default.
CC @ttuegel

Oh damn, just saw https://github.com/NixOS/nixpkgs/pull/16838, closing this one


Order of execution

fixes #508

Current coverage is 69.39% (diff: 45.09%)

Merging #598 into master will decrease coverage by 0.28%

@@             master       #598   diff @@
==========================================
  Files            74         74
  Lines          4285       4336    +51
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
+ Hits           2986       3009    +23
- Misses         1012       1033    +21
- Partials        287        294     +7

Powered by Codecov. Last update 2045507...72e51f0

Could you explain the logic/flow in this?

From the API point of view, what does the UI call to reorder something? And then what happens?

@aslakknutsen
I implemented this by spacing the order value by a factor of 1000

When a new workitem is created:
a) LoadHighestOrder() will return the highest order currently present in the system and assign the new workitem "highest order+1000" value

When a workitem is updated:
a) Case 1: order is not changed
- The PATCH payload will be as it is
b) Case 2: order is changed
i) The new workitem is moved in between workitem 101 and 102
- The payload will contain:
* previousitem: 101
* nextitem: 102
- The order of new workitem will be = (previtem.Order + nextitem.Order)/2
ii) The new workitem is moved to top
- The payload will contain:
* previousitem: nil
* nextitem: ID of the topmost item before the reordering
- The order of new workitem will be = (0 + nextitem.Order)/2
iii) The new workitem is moved to bottom
- The payload will contain:
* previousitem: ID of the bottom most item before the reordering
* nextitem: nil
- The item is the bottom most item.
- The order of new workitem will be = nextitem.Order + 1000

SAMPLE PAYLOAD FOR UPDATE:
curl -i -X PATCH localhost:8080/api/workitems/1 -H "....." -d '{"data": {"type": "workitems", "attributes":{"system.owner": "dhritis", "system.state": "open", "system.title":"helloooooo", "version":0, "previousitem":101, "nextitem":109}, "relationships":{"baseType":{"data":{"id":"system.userstory", "type": "workitemtypes"}}}, "id": "1"}}'

Missing a specific API on this from the REST side. Currently, it works by setting 2 'magic' fields, references to 2 other workitems that this workitem should be between. In e.g. a Filtered List the UI has no way of knowing the 2 'closest' workitems.
The API should aim to make it as easy as possible to do. A custom action is required. e.g.
/api/workitems/reorder
{
  workitems: [y, z],
  before: x
}

Good point @aslakknutsen ,
there's another important behaviour we need to track and develop in future is that :
Workitems are being given a global order, but in practice it should be constrained to a 'space'.

@DhritiShikhar what about the order of execution for the work items already created on demo.almighty? The UI needs to display the list of work items as per the order of execution - https://github.com/almighty/almighty-ui/issues/559

@nimishamukherjee writing a postgres migration script to handle existing workitems

Based on the suggestions on the PR, I have changed the code, this is the logic now:
1) When a new workitem is created:
a) Sample Request Payload:
curl -i -X POST localhost:8080/api/workitems -H "Authorization: Bearer ..." -d '{"data": {"type": "workitems", "attributes":{ "system.owner": "dhritishikhar", "system.state": "open", "system.title":"The title"}, "relationships":{"baseType":{"data":{"id":"system.userstory", "type": "workitemtypes"}}}}}'
b) LoadHighestOrder() will return the highest order currently present in the system and assign the new workitem "highest order+1000" value

When a workitem is reordered,

api endpoint will be workitems/reorder/:id?before=:before
Example: workitems/reorder/110?before=104
The new workitem 110 is moved before 104

The PATCH payload will look like this:

curl -i -X PATCH localhost:8080/api/workitems/reorder/12?before=11 -H "Authorization: Bearer ..." -d '{"data": {"type": "workitems", "attributes":{"version":0}, "id": "110"}}'
CASE 1: before is present

Finds afterItem -> the item after which the to-be-reordered workitem (here, 110) is to be moved.
a) If the afterItem is not present, the item is topmost item
-> the order of the workitem 110 will be (0+beforeItem.Order)/2
b) If the afterItem is present, the item is in between two items
-> the order of the workitem 110 will be (afterItem.Order+beforeItem.Order)/2

CASE 2: before is not present

Api endpoint -> workitems/reorder/110?before=
The workitem 110 is the bottom most item
Order of the workitem 110 will be afterItem.Order+1000

The URL endpoint could be like this: PATCH /api/orderings/<work-item-ID> with a payload like this {"data": {"type": "workitems", "attributes":{"version":0}, "id": "110"}, "meta": {"befoe": "124"}}
Ref.http://jsonapi.org/format/#document-meta

@baijum

i dont prefer to use any word with "-ing". Not even for a variable. I prefer the word "reorder" more than "ordering"

@aslakknutsen suggested ""There should be a specific format/endpoint to specify the Move operation, Which WorkItems should Move Before which other WorkItem.""
So, according to that, workitems/reorder/110?before=104 -> this is more descriptive. It says you want to reorder 110 before 104

[test]

[test]

[test]

[test]

[test]

@kwk fixed it here -> be30fea

Some minor things, but generally looking good.


Export project not complete on Windows ?

Godot version:
7dcaaba
OS/device including version:
Windows 10
Issue description:
I have a problem with export to Windows platform - it does not save the information from export menu !
I mean these fields does not affect the resulted binary

The resulted binary does not apply custom icon and information - the result is same as default For example(sorry if you dont understand russian language),

Or I something misunderstood ?
Steps to reproduce:
Try export your own project on windows..

Have you downloaded rcedit from its Releases page and set its path in the Editor Settings?

Oh, I didnt knew about it.. Thanks for information

Hmm.. after trying it - the data is changed to valid values, but icon still remains as Godot.

Tried other version of this tool, and its worked !

@Chaosus Which version of rcedit worked for you (none of them work for me)?


How to correctly set all the paths and run the code?

I have successfully make the project, but I can't find how to run the code. I tried to run BING_linux, but I got this error:
OpenCV Error: Assertion failed (matRead(_modelName + ".xP", xP1f) && matRead(_modelName + ".xN", xN1f)) in trainStageI, file /home/lixueting/Downloads/Objectness-master/Src/Objectness.cpp, line 555
terminate called after throwing an instance of 'cv::Exception'
  what():  /home/lixueting/Downloads/Objectness-master/Src/Objectness.cpp:555: error: (-215) matRead(_modelName + ".xP", xP1f) && matRead(_modelName + ".xN", xN1f) in function trainStageI

I looked into yml.m, there are voc path and imagenet path, do I have to download imagenet too in order to run the code?
Thanks!

I second this question. Not to hijack, but I also cannot find any mention of "WinRecall.m", which is used in main.cpp. It seems related to the issue of setting up all paths and downloading the required files.

@sunshineatnoon No, you do not need to download ImageNet unless you want to try BING on ImageNet. Otherwise, stick to VOC.

@phil0stine Sorry for the confusion. The answer is NO. In fact, WinRecall.m is generated from the BING program.

@sunshineatnoon , I have the following error similar to yours, do u know how to solve it??
Invalidate class name
in /media/BING_Linux/Src/DataSetVOC.cpp:125
OpenCV Error: Assertion failed (Invalidate class name
) in loadBox, file /media/BING_Linux/Src/DataSetVOC.cpp, line 125
terminate called after throwing an instance of 'cv::Exception'
what():  /media/BING_Linux/Src/DataSetVOC.cpp:125: error: (-215) Invalidate class name
in function loadBox

@sunshineatnoon @bittnt got the same problem. I'm trying to run for it exam. Any solutions? :\

This means the VOC files you are trying to read do not meet the requirement for the code. Check the DataSetVOC.cpp line 125, probably because you have extra space or invisible symbols in that txt file.

how do u run bing on a given image. not VOC dataset. Forexample:  how od you get the 2D object proposals in bing on lena.png

@RyanCV Hi Ryan, did you find how to solve this problem? I have tried opencv2.4.13 and opencv3.2.0, both of them report this message to me. Thank you.

@bittnt @RyanCV @sfandi I encounter the same problem. Has anyone found a solution ?
I print the path to the VOC dataset /mnt1/logs/caffe/objpro/BINGpp/VOCdevkit/VOC2007/ImageSets/Main/train.txt
I also print trainNum and testNum
trainnum: 2501
testnum: 200
Yet the error:
Invalidate class name
in /mnt1/logs/caffe/objpro/Objectness/Src/DataSetVOC.cpp:131
OpenCV Error: Assertion failed (Invalidate class name
) in loadBox, file /mnt1/logs/caffe/objpro/Objectness/Src/DataSetVOC.cpp, line 131
terminate called after throwing an instance of 'cv::Exception'
what():  /mnt1/logs/caffe/objpro/Objectness/Src/DataSetVOC.cpp:131: error: (-215) Invalidate class name in function loadBox
Aborted (core dumped)
Which txt are you talking about @bittnt when referring to extra space and invisible symbols ??

Check the encode type for the txt files.

Hi , Shuai Zheng @bittnt , thanks for your work.  I do not understand  what you said above  "Check the encode type for the txt files.",   I think i have a right path for VOC2007 and right format "yml"  instead of  "xml"  , but i also get the same  problem :
in /home/dorioliu/Documents/objectdetecion/BING/Objectness-master_linuxversion/Src/DataSetVOC.cpp:125
OpenCV Error: Assertion failed (Invalidate class name
) in loadBox, file /home/dorioliu/Documents/objectdetecion/BING/Objectness-master_linuxversion/Src/DataSetVOC.cpp, line 125
terminate called after throwing an instance of 'cv::Exception'
what():  /home/dorioliu/Documents/objectdetecion/BING/Objectness-master_linuxversion/Src/DataSetVOC.cpp:125: error: (-215) Invalidate class name
in function loadBox

Hi, @RyanCV ,did you solve your problem ?

@dorioliu @ming-c @RyanCV
Did you set up the database correctly? It should look like this: https://mmcheng.net/bingreadme/ , you can also check the paths to the training and test files in the constructor of DataSetVOC. For some reason the function CmFile::loadStrList() did not work for me (I am using OpenSUSE) and I modified the code that the files are red correctly.
And also check if you have a class.txt file next to train and test.txt. My download did not contain, so I wrote one myself. Now the application is able to produce the training models.

@RyanCV @sfandi @dorioliu @brat-eek
Hi all,  had anyone solved the “Invalidate class name” problem?
I have wrote the class.txt by myself, there is not any extra space or invisible symbols and I have checked all the variables while reading VOC DataSet, the size values are both ok.
This problem makes me very confused，If some of you solved the problem，can you share the solution?

@MaryAnnZ is right!
I just comment the line "line.resize(sz - 1)" in function CmFile::loadStrList().
then it works.


updating the measurement function

I edited the measurement function to have the labels start at 1 when loading from file with no column for label number, which should remove any errors when clicking on the table

Thanks, Ryan! This seems not complicated, I could have figured it out when I asked you and complained that it's not working😂


⚠ Cioos National CKAN has degraded performance

In 659fac7, Cioos National CKAN (https://catalogue.cioos.ca/) experienced degraded performance:

HTTP code: 200
Response time: 5805 ms

Resolved: Cioos National CKAN performance has improved in 5352f90 after 11 minutes.


Feature request: Allow frame rate limiting in debug tools

Is there an existing issue for this?

[X] I have searched the existing issues
[X] I have read the guide to filing a bug

Use case
I am using shaders to implement a motion blur feature. My implementation of the motion blur is frame rate dependent. It measures the position delta of a widget in between frames and applies a radial blur accordingly. I would like to use the debug tool feature which slows down animations to verify that my shader is displaying correctly but while the animations do slow down, Flutter still renders at 60 frames per second and thus my shader doesn’t display as it normally would.
Being able to limit frame rate would also help debug other kinds of scenarios as well as to see what your app might look like on low power devices.
Proposal
What I’m proposing is a feature in the debug tools that allows the developer to add a frame rate limiter while in debug mode.

Hi @jonaird
Can you check if Slow animations can work for this case?
Also, per mentioned at https://github.com/flutter/flutter/issues/35162#issuecomment-609000841, it seems flutter_displaymode is also useful.

Slow animations maintains 60fps but slows the animations.

Can you please be more specific?

while the animations do slow down, Flutter still renders at 60 frames per second and thus my shader doesn’t display as it normally would.

Can you also share a recorded video or an example for this as well?
Also, please try statsfl package if it works and confirm if it presents for your expectation.


the different scaling on the scatter plots causing inaccuracy of angle observation

Checking the view and ensuring the viz is clear, preserving angles across separate views that are not simultaneous.
What is the difference between a distance 0.63 and 0.046 in linear regression?


Rating Refresh issue

Backend improvements
Frontend refresh issue resolved

see comments, otherwise lgtm

profile picture would be better to be removed here as we dont have that functionality yet, I know it was there before

all the new stuff looks good. i know my screen is narrow but do you think we can give the checks a bit more space- i have to make my browser really wide to look like your image

Plus and minus next to checks are not aligned correctly.
If I click check plus and the action fails, incorrect action is active

functionality wise it seems to work well for me

@dmorina the request has to fail as I mentioned in #475

I specifically aligned + and - signs like that. In case those signs need to be below check sign, please let me know.
There is no change happening if action fails. So probably I am missing the logic how it is impacting function.

could you post a screenshot of what errors you get? @nistala

I would recommend going with the standard alignments of the + and -:
http://sweetclipart.com/multisite/sweetclipart/files/grade_marks.png
On Thu, Sep 3, 2015 at 12:07 PM, Rohit<EMAIL_ADDRESS>wrote:

[image: screenshot from 2015-09-03 12 06 30]
https://cloud.githubusercontent.com/assets/4357428/9668070/42e99864-5234-11e5-9541-c720d291dbff.png
—
Reply to this email directly or view it on GitHub
https://github.com/crowdresearch/crowdsource-platform/pull/485#issuecomment-137544251
.

Feedback incorporated for rating updates
Icon alignment corrected as shown below

lgtm, resolve conflicts please

Conflicts Resolved

lgtm