Error: react-native-mmkv is not supported in Expo Go! Use EAS

Hello guys,
Error: react-native-mmkv is not supported in Expo Go! Use EAS (expo prebuild) or eject to a bare workflow instead., js engine: hermes
ERROR  Invariant Violation: "main" has not been registered. This can happen if:
Metro (the local dev server) is run from the wrong folder. Check if Metro is running, stop it and restart it in the current project.
A module failed to load due to an error and AppRegistry.registerComponent wasn't called., js engine: hermes
Everything was normal but I got this error I dont know why.
"dependencies": {
"@ant-design/icons": "^5.1.2",
"@expo/config-plugins": "^7.8.4",
"@formatjs/intl-displaynames": "^6.6.1",
"@formatjs/intl-locale": "^3.4.0",
"@formatjs/intl-pluralrules": "^5.2.7",
"@react-native-community/checkbox": "^0.5.15",
"@react-navigation/bottom-tabs": "^6.5.7",
"@react-navigation/native": "^6.1.6",
"@react-navigation/native-stack": "^6.9.12",
"@react-navigation/stack": "^6.3.16",
"@rneui/base": "^4.0.0-rc.7",
"@rneui/themed": "^4.0.0-rc.7",
"expo": "^49.0.8",
"expo-application": "^5.8.3",
"expo-build-properties": "~0.8.3",
"expo-checkbox": "~2.4.0",
"expo-constants": "^15.4.5",
"expo-dev-client": "~2.4.8",
"expo-device": "^5.9.3",
"expo-localization": "~14.3.0",
"expo-modules-core": "^1.11.9",
"expo-notifications": "^0.27.6",
"expo-splash-screen": "~0.20.5",
"expo-status-bar": "~1.6.0",
"expo-updates": "~0.18.12",
"firebase": "^9.23.0",
"formik": "^2.4.0",
"i18next": "^23.6.0",
"react": "18.2.0",
"react-dom": ">=16.8.0",
"react-i18next": "^13.3.1",
"react-native": "0.72.10",
"react-native-eject": "^0.2.0",
"react-native-elements": "^3.4.3",
"react-native-fast-image": "^8.6.3",
"react-native-flatlist-slider": "^1.0.6",
"react-native-image-slider-box": "^2.0.7",
"react-native-international-phone-number": "0.6.5",
"react-native-ionicons": "^4.6.5",
"react-native-linking": "^0.0.0",
"react-native-mmkv": "^2.10.2",
"react-native-paper": "^5.7.2",
"react-native-restart": "^0.0.27",
"react-native-safe-area-context": "4.6.3",
"react-native-screens": "~3.22.0",
"react-native-vector-icons": "^9.2.0",
"react-native-webview": "13.2.2",
"twilio": "^4.12.0",
"typescript": "^5.1.3"

react-native-mmkv is not supported in Expo Go! Use EAS (expo prebuild) or eject to a bare workflow instead.

how did you fix this error ERROR  Error: react-native-mmkv is not supported in Expo Go! Use EAS (expo prebuild) or eject to a bare workflow instead., js engine: hermes I am also facing it

when i try to use zeego  got this error Invariant Violation: requireNativeComponent: "MenuView" was not found in the UIManager

deleted mmkv, install asyncstorage :)

is it the only solution,i really want to use it in expo

deleted mmkv, install asyncstorage :)

when i try to use zeego got this error Invariant Violation: requireNativeComponent: "MenuView" was not found in the UIManager

same problem i am facing too

yo why is this closed? Same issue


Support prediction cache for multi-target tree.

Fix prediction range.

Let me add support for mixed trees


Map.merge with meantime changes but equal result return new instance of map

Merging two different, but equal maps, result in the original map.
That's ok and expected.
var x = IM.Map().set(1,2)
var z = IM.Map().set(1,2)
//to be true
x.merge(z) === x
Merging maps, wich changes the orignal map meantime, but equals at the end results a new map.
That is unexpected.
var x = IM.Map().set(1,2)
var y = IM.Map().set(1,3)
var z = IM.Map().set(1,2)
//should be true
x.merge(y,z) === x
//result is equal => true
x.merge(y,z).equals(x)
The problems are MakeRef and SetRef. SetRef is irreversible when used once, so merging back to the original value dosn't reuse the original map.
Exists workarounds for this problem?

I wouldn't consider this something you can bet on. Strictly speaking an operation on a persistent data structure should always yield a new instance. The refs are an attempt to optimise the performance. It tries to find out whether the value has actually changed. Now finding out whether the value has changed after two merges is actually an expensive operation.
Since the partials you are passing are merged separately, in-order, after the merge, there's no way comparing the changed values efficiently. I don't think this is something that should be worked around either.
In this case you're changing a value like this: 2 => 3 => 2. Immutable.js shouldn't try to be slower to optimise this case, and this should really be something that you should take care of, if you really care about optimising this case. ;)

Great answer by @philpl
Returning the same reference for "no-ops" is a performance optimization and so only occurs in cases where it can be done performantly.
One work around is the following:
x.merge(y.merge(z))


add support for passing custom encoder/decoder/formatter to Adapters.Postgres

This would ease the integration of custom encoders like geo to extend functionality.

Closing this as we are changing the formater/encoder/decoder in postgrex exactly so it is easier to extend! Then hstore or postgis will be more straight-forward to add! Thank you!


shoutrrr/slack: missing channel property

After looking at the release notes for 1.7.0, I found a breaking change (for our use case). If opting in to the use of shoutrrr, the ability to override the Slack channel has gone away (unsupported in shoutrrr). I know that currently both modes of notifications (static Slack implementation + shoutrrr) exists, but the writing on the walls indicates that the former will be dropped in a future release.
I have opened a PR to fix this upstream in shoutrrr: https://github.com/containrrr/shoutrrr/pull/173

@atighineanu Do you know anything about this?

Assign me to this pls. On it.

I haven't tested the channel properties on any of the services, to be honest. Not many did in fact use that level of details (that I know, at least).

Did you see the change in https://github.com/containrrr/shoutrrr/pull/179 ?
Were you using channel properties for other things than icons?
I am afraid holding onto future removed features is not a good way forward ;)

The number one feature for me was to override channel name, since it meant only interacting with one Slack webhook token.


Remove underline on Android

Is it possible to remove the underline that appears on Android?
I see the native TextInput has a prop called underlineColorAndroid="transparent" but checking the TS file I don't see anything for this, any work arounds to achieve it?

@alpha0010 How hard would it be to add this?

Hmm. Surprisingly difficult unfortunately.  This is implemented via a SearchView, not a TextInput.  As such, in order to modify unexposed styles, one must parse the element for the appropriate base widget to modify.  The "underline" is not (on most platforms) implemented as an actual underline.  Automatically determining the underline would be hard to confidently implement for all targeted SDKs.
The only solid option I can think of would be re-implementing the entire widget (off of primitive components), which rather feels like it defeats the goal of being completely native.

Same issue

I'm thinking of use a view to overlap the underline to hide it.

Oh gosh my overlaid view covers hanging part of characters, that was silly of me.

Did some searching around. You may be able to adapt this solution: https://stackoverflow.com/a/27687155
Also, grep for search_plate and mSearchPlate in the Android sdk source.
If you are trying your hand at this, be aware the hard part with react native style properties is to correctly (re)apply the native styles when the JS code does not explicitly set a value.

Is there any update on this?

set query backround to transparant

just use android:queryBackground="@null"


Why normalize step not found in preprocess function?

I find there is no normalize step in preprocess function. why input image can be feed to network with out normalize?

Both training and testing have normalization. They are done in the loader. See L248 and L263.
https://github.com/TuSimple/simpledet/blob/dafc5536d2ed74094872348801493e42ccaa3c53/config/faster_r50v1_fpn_1x.py#L245-L269


What is win10 TFM and what is the difference with win?

What does the following description means ("Supported frameworks" table)?

win10 (not supported by Windows 10 Platform)

This makes no sense. And what is the difference between the win and win10 TFMs?
The table under the "Supported frameworks" section also as an "Abbreviation" column that is undocumented; I assume that you cannot abbreviate 15+ different TFMs with a single abbreviation, so what those even mean?
This really must be better documented. The current documentation is not helpful at all; in fact it brings more confusion than it clarifies things.

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

ID: 0a0f5267-208d-8af2-ef18-dae7407eefad
Version Independent ID: 19b00f91-5d31-0d4d-388b-30b1bd4e7c0f
Content: Target Frameworks Reference for NuGet
Content Source: docs/reference/target-frameworks.md
Product: nuget
GitHub Login: @karann-msft
Microsoft Alias: karann

uap also has uap10.0 listed as equivalent (between brackets). But then on the next line uap10.0 also appears alone. Why? This just creates confusion.

@djee-ms I believe this doc best answers the question: https://docs.microsoft.com/en-us/dotnet/standard/frameworks
There are two docs about frameworks, one from dotnet & one from NuGet. Perhaps we should refer to a single source of truth instead.

Now I am 10 times more confused. That new links says that:

win is deprecated in favor of netcore45 (what?!), which seems to be for "Windows Store" (re-what?! isn't Windows Store = UWP, and not Desktop?)
win10 is same as uap10.0 (what?!)

How do we package our C# library which P/Invoke into our Desktop and UWP native C++ DLLs into our NuGet packages?

@djee-ms This is a very confusing area because of the names that are used here.
One tool I believe that may help you is this tool our team put together a little bit ago that we keep updated:
https://nugettoolsdev.azurewebsites.net/5.8.0/framework-precedence
For example if you see something is deprecated in favor of another framework, you'll be able to see the precedence.
I believe the major confusion that is here is that netcore45 represents a name that predates netcoreapp. So these netcore names represent the Universal Windows Platform[uap10.0/win10/etc]

@JonDouglas this is about runtime identifiers (RIDs), not frameworks. NuGet gets passed these values, including the path to a json file which contains the compatibility graph, so we're not really experts in this area.
Maybe a link to this doc is all we need, or a simple first step until someone can read and understand that doc, then re-write this doc as required: https://docs.microsoft.com/en-us/dotnet/core/rid-catalog

@zivkan The confusion is that the .NET doc says:

A target framework is typically referenced by a TFM. The following table shows the target frameworks supported by the .NET SDK and the NuGet client. Equivalents are shown within brackets.

The framework precedence seems to also show the equivalents of these TFMs.
I believe RIDs use the bracket notation which doubly adds confusion.

yeah, sorry, my previous comment was wrong. The linked docs is the TFM list, and it does list some TFMs starting with win. Even I have absolutely no idea what these are or what they're for.

Yeah well in any case it would be good to find someone with clear knowledge of that who can rewrite those 2 documentation pages (or at least the NuGet one) and answer the question in particular of where native DLLs go. It's already difficult to work with native DLLs and NuGet, but we don't even understand how to package them. And this is a widespread issue because I've asked other teams who also are confused.

@diverdan92, @rconard, @terrajobst may be able to help or direct someone who can help us.

@djee-ms if you're creating a package for native (c++) projects, then I'm pretty sure you should use the native fake TFM (/build/native/packageid.[props|targets]). My guess about the win and win8 TFMs is that they're related to the Windows 8 UWP, which is not the same thing as native.
If you're creating a .NET package which P/Invokes a native dll that you need to include, I created this epic issue to track customer requests for exactly this in a single place to make it more clear to managers and PMs that this is a pain point. Let me know, so I can link this issue, but please also 👍  that issue to help it bubble up higher. I feel like I have about 70% confidence I know how these packages should be packaged, but the time to test to gain more confidence, and then the time required to write easy to understand docs is very much non-trivial, so I'll only be able to do it myself if I have no other higher priority work.

@zivkan yes this is the same as #2070, we have a C# wrapper over a set of native C++ DLLs, which all must be packaged together inside a NuGet for use by our customers in their C# app. I can very confidently tell you, for having been through this on multiple projects and still not understanding it, and having discussed it with many colleagues, that this is a monumental pain and the documentation is severely lacking.

Thanks for re-opening @nkolev92; is there any update on clarifying those TFMs?


Safari iPad iOS 10.2 - PDF not loading

Configuration:

Web browser and its version: Safari - iOS 10.2
Operating system and its version: iOS 10.2
PDF.js version: 1.5.188

When i load up PDF.js on Safari iPad iOS 10.2, no pdf is displayed. The PDF.js framework displays, but a pdf never loads. I can fix this by going in and setting: disableRange: to true in viewer.js. I have tried including the compatibility.js file, but it still does only work is disableRange is set to true.
Works perfectly well in Chrome on Windows with the disableRange set to false.

I believe that this should have been fixed by PR #7841, which (among other things) disables range requests on iOS devices.

PDF.js version: 1.5.188

If you use the latest version of compatibility.js, as found at https://github.com/mozilla/pdf.js/blob/master/web/compatibility.js, does that resolve the issue?

Closing as answered, since the advice in https://github.com/mozilla/pdf.js/issues/7936#issuecomment-271258344 should be enough to fix this issue.

@Snuffleupagus i cant understand how you include compatibility, when you get worker and document. I can show the code here:
first, i'm using require:
 var pdfjsLib = require('pdfjs-dist/build/pdf');

 pdfjsLib.PDFJS.workerSrc = require('pdfjs-dist/build/pdf.worker');

        pdfjsLib.PDFJS.getDocument(this._pdfPath).then(function(pdfDoc_){
            pdfDoc = pdfDoc_;
            renderPage(pageNum, 1);
        });

Can't seem to make it work on ios too! compatibility.js is gone from the repo!


Add Sharing to the plant detail page

Incorporate the Sharing Jetpack component to the plant detail page.

Are you suggesting to use ShareActionProvider?
What kind of share message to share?

@tiembo same agree @andhie ,  people need wordings to share.

For the sharing message, let's start with "Check out the $plant.name plant in the Android Sunflower app".
Implementation should be with ShareCompat.IntentBuilder


Function Failed Indexing - Unable to resolve value from configuration

Azure Functions Worker Runtime: dotnet8-isolated
Azure Functions Version:  2.0
Applicable Nuget packages
<PackageReference Include="Microsoft.Azure.Functions.Worker" Version="1.20.0" />
<PackageReference Include="Microsoft.Extensions.Configuration.AzureAppConfiguration" Version="7.0.0" />
<PackageReference Include="Microsoft.Azure.Functions.Worker.Extensions.CosmosDB" Version="4.5.0" />

Issue Description
I am trying to source configuration from Azure App Configuration and use the values in my CosmosDbTrigger parameters. I am using the isolated worker, coming from in-process. This used to work but now that I am using the isolated worker, the function fails indexing. It does not seem to see the values from App Configuration.
App Configuration  added as a provider in using .ConfigureAppConfiguration
    var host = new HostBuilder()
    .ConfigureFunctionsWorkerDefaults()
    .ConfigureAppConfiguration(configBuilder =>
    {
        //connection variable omitted for brevity and security
        configBuilder.AddAzureAppConfiguration(connection);
    })

The in-process code used something like this but was in a Startup.cs file in the
    public override void ConfigureAppConfiguration(IFunctionsConfigurationBuilder builder)
    {
        var config = builder.ConfigurationBuilder
            .AddEnvironmentVariables()
            .SetBasePath(Directory.GetCurrentDirectory())
            .AddJsonFile("local.settings.json", optional: true, reloadOnChange: true);
        config.AddAzureAppConfiguration(connection);
        _config = config.Build();
    }

Sample function
[Function(nameof(SampleFunction))]
public async Task SampleFunctionAsync([CosmosDBTrigger(
    databaseName: $"%{Constants.CosmosDatabaseEnvName}%",
    containerName: $"%{Constants.SampleContainerNameEnvName}%",
    Connection = $"{Constants.CosmosConnectionEnvName}",
    LeaseContainerName = "leases",
    CreateLeaseContainerIfNotExists = true)] IReadOnlyList<JObject> documents
  )

...
Expected Results
I am expecting that the string values in the static constants class will resolve to the key/value pair in Azure App Configuration. So, for example,
public static class Constants
{
    public const string CosmosDatabaseEnvName = "my_cosmos_database"
}

Then in Azure App Configuration

so that the-acutal-database-name would be used as the parameter value for databaseName and so on.
Actual Results
*Actual function and variable names omitted/redacted

Any guidance or help you can provide would be greatly appreciated.

I should note that I've been able to get the value if I add in the key/value pair in the Values section of my local.settings.json, simulating app settings. So, pulling from configuration using the enclosing % does work, just not from the Azure App Configuration config provider.

Any idea how to use normal appsettngs.json? I am using this as this is what we used for prev .net 6 in-process functions but its not picking up configuration when running .net isolated process function v4
.ConfigureAppConfiguration((hostContext, configApp) =>  {      configApp.AddJsonFile("appsettings.json", optional: true);      configApp.AddJsonFile($"appsettings.{hostContext.HostingEnvironment.EnvironmentName}.json", optional: true);  })   .ConfigureFunctionsWorkerDefaults()
And then timer function
 [Function("SyncNoticesCron")]  public async Task SyncNoticesCron(  [TimerTrigger("%CronConfiguration:SyncNotices%")] TimerInfo timerInfo,  [DurableClient] DurableTaskClient starter)

The configuration you have pulled from Azure AppConfiguration is only available to the worker process. It is not available to the functions runtime and thus we cannot resolve that value. You need to provide configuration either via local.settings.json for local development or via App Settings (these can be resolved from KeyVault). https://learn.microsoft.com/en-us/azure/app-service/app-service-key-vault-references?tabs=azure-cli
We have no plans to support Azure AppConfiguration this way with dotnet isolated as there is a hard technical limitation: there are multiple components which need to know those config values without ever running your worker, and the example you have has only the worker knowing how to retrieve those values from Azure AppConfiguration service.

Is there any way to provide those configurations or alternatives on how to configure the tinetrigger so we can use the appconfig resource? or is this a hard no?


Noviny/pre releases

This is well out-of-date as it was made before changesets v2 was envisioned.
Future pre-releases work is being done on: #183


Microros output

How do I change microros output for Arduino? For example it normally prints "data: " I would like to print "Temperature:  Celsius."

Where does it prints "data"?

The "data" is printed out after I run the node in ros2 and in another terminal run: ros2 topic echo . Gives me
data= 21
.
.
data= 23
And so on.
I would like to print
Temperature: 23 C
And so on.
Thank you

Not related to micro-ROS, this is the ROS 2 CLI.

Thank you. However could I write a message after data that says Temperature in Celsius:
.
.
And so on?


Windows app unresponsive, appears to be stuck trying to connect to https server

Running on Windows 7 Pro 64-bit with all updates installed to date.
Since yesterday when Toggle updated itself to 7.2.219 the interface has been unresponsive. If the application loads and is only showing in the system tray there is no way to bring up the interface. I forced a reinstall and got the interface up with sign in, but then it hangs as soon as I've signed in. I've uninstalled it, deleted all the related files, reinstalled, same thing.
The log file shows an SSL3_GET_SERVER_CERTIFICATE error, over and over, eg:
2015-08-04 10:12:59.757 [5948 5]:websocket_client:D:restarting
2015-08-04 10:12:59.798 [5948 5]:websocket_client:D:createSession
2015-08-04 10:12:59.831 [5948 5]:websocket_client:D:deleteSession
2015-08-04 10:12:59.865 [5948 5]:websocket_client:D:session deleted
2015-08-04 10:13:00.212 [5948 5]:websocket_client:E:SSL Exception: error:14090086:SSL routines:SSL3_GET_SERVER_CERTIFICATE:certificate verify failed
2015-08-04 10:13:33.235 [5948 5]:websocket_client:D:restarting
2015-08-04 10:13:33.273 [5948 5]:websocket_client:D:createSession
2015-08-04 10:13:33.349 [5948 5]:websocket_client:D:deleteSession
2015-08-04 10:13:33.434 [5948 5]:websocket_client:D:session deleted
2015-08-04 10:13:33.830 [5948 5]:websocket_client:E:SSL Exception: error:14090086:SSL routines:SSL3_GET_SERVER_CERTIFICATE:certificate verify failed
2015-08-04 10:14:07.899 [5948 5]:websocket_client:D:restarting

and repeat until process is killed.
At a guess it's in a blocking loop trying to connect to a server and this causes the interface to hang. I have no proxy running on my PC, so there should be no issue with an intermediate SSL proxy causing the remote certificate verification to fail.
If I start up Fiddler4 to see where the connection is being attempted so that I can attempt to check the server certificate I get an error about the port number being invalid, so that's a dead end too:
2015-08-04 10:19:39.307 [5948 5]:websocket_client:D:restarting
2015-08-04 10:19:39.390 [5948 5]:websocket_client:D:createSession
2015-08-04 10:19:39.423 [5948 5]:websocket_client:D:deleteSession
2015-08-04 10:19:39.448 [5948 5]:websocket_client:D:session deleted
2015-08-04 10:19:39.525 [5948 5]:websocket_client:E:Syntax error: bad or invalid port number: 8888;https=<IP_ADDRESS>:8888
2015-08-04 10:19:55.548 [5948 5]:websocket_client:D:restarting
2015-08-04 10:19:55.657 [5948 5]:websocket_client:D:createSession
2015-08-04 10:19:55.676 [5948 5]:websocket_client:D:deleteSession
2015-08-04 10:19:55.701 [5948 5]:websocket_client:D:session deleted
2015-08-04 10:19:55.758 [5948 5]:websocket_client:E:Syntax error: bad or invalid port number: 8888;https=<IP_ADDRESS>:8888

Dan

+1. I feel less lonely ;)
Very same issue, as confirmed by the logs, since the application updated.
Windows 8.1 Enterprise edition

Hey guys,
Thanks for reporting the problem!
We are currently investigating and hope that we will be able to find the problem soon.

Windows 8.1 here too, same problem as killermouse0.

Same issue here, windows 7 - 64 bit

Hi,
Users expecting the app freeze issue. Can you help us out and try to install this development build to see if the freezes are still there? Be sure to note your installed Desktop app version when giving us feedback.
Thank you!

https://assets.toggl.com/installers//TogglDesktopInstaller-7_2_233-2015-08-06-10-59-29.exe

Have installed it and so far looks good, it hasn't hung up and I can start and stop tasks. Still getting errors in the toggldesktop.log file for websocket_client though:
2015-08-06 09:44:10.706 [5512 5]:websocket_client:D:restarting
2015-08-06 09:44:10.737 [5512 5]:websocket_client:D:createSession
2015-08-06 09:44:10.753 [5512 5]:websocket_client:D:deleteSession
2015-08-06 09:44:10.780 [5512 5]:websocket_client:D:session deleted
2015-08-06 09:44:11.078 [5512 5]:websocket_client:E:SSL Exception: error:14090086:SSL routines:SSL3_GET_SERVER_CERTIFICATE:certificate verify failed
2015-08-06 09:44:22.222 [5512 5]:websocket_client:D:restarting
2015-08-06 09:44:22.290 [5512 5]:websocket_client:D:createSession
2015-08-06 09:44:22.319 [5512 5]:websocket_client:D:deleteSession
2015-08-06 09:44:22.347 [5512 5]:websocket_client:D:session deleted
2015-08-06 09:44:22.731 [5512 5]:websocket_client:E:SSL Exception: error:14090086:SSL routines:SSL3_GET_SERVER_CERTIFICATE:certificate verify failed

Although there are other log entries interspersed with the posts to the Toggl server.
Thanks for getting this version out, I've missed Toggl and been using paper and pen to track my time spent on dev projects ... :|
Dan

I have installed the new version, and so far no frezzes or hangs. Also it feels more responsive, compared to the earlier version.
Good work with a quick fix

@dan-sorion it seems that on your location the websocket connection does not work for some reason. So it keeps attempting to restart the connection, ad infinitum. But its not blocking the UI so it should be fine.

Installed the development build, tried the "About" screen to confirm version and it immediately froze. (Maybe an entirely different bug !)

Does it happen each time?
On Thursday, August 6, 2015, killermouse0<EMAIL_ADDRESS>wrote:

Installed the development build, tried the "About" screen to confirm
version and it immediately froze. (Maybe an entirely different bug !)
—
Reply to this email directly or view it on GitHub
https://github.com/toggl/toggldesktop/issues/1444#issuecomment-128404540
.

I've tried the About box, first time I closed it the Toggl gui was sluggish and took a few seconds to start responding properly again. Tried it a few more times, doesn't seem to cause an issue although might be a little sluggish for a half second or so.

I've been experiencing this issue (app unresponsive) in the latest stable release (7.2.219) on Win7 64-bit.
Just installed the build you posted (7.2.233) and so far it appears to be resolved. Thank you!

Thanks @dacohenii @dan-sorion for the feedback!!
It looks like the About window sluggishness is another issue. Perhaps its checking for updates and locking while doing so. I've created another issue regarding this.

btw please write to<EMAIL_ADDRESS>to receive nice Toggl swag, if you'd like to. Just mention Desktop app testing.

Just had another hang, this time as I stopped a running task, entered a new task name, and clicked the Start button. Now the interface is hung with it looking like it hasn't started the new task, and the mouse disappears when I move it over the interface. Can't click anything, systray icon does nothing either. Here's the last extract from the log (websocket issues removed for clarity, id strings removed just in case it's related to my account)
2015-08-07 06:45:17.946 [6180 1]:BatchUpdateResult:D:[{"method":"PUT","guid":"","body":"{"data":{"id":,"guid":"","wid":,"billable":false,"start":"2015-08-07T06:30:39Z","stop":"2015-08-07T06:45:14Z","duration":875,"description":"check archive and deletion job","duronly":false,"at":"2015-08-07T06:45:17+00:00","ui_modified_at":"1438929914","uid":}}","status":200,"content_type":"application/json; charset=utf-8"}]
2015-08-07 06:45:17.983 [6180 0]:ui:D:DisplayTimeEntryAutocomplete
2015-08-07 06:45:18.013 [6180 1]:BatchUpdateResult:D:batch update result GUID: , StatusCode: 200, ContentType: application/json; charset=utf-8, Body: {"data":{"id":,"guid":"","wid":,"billable":false,"start":"2015-08-07T06:30:39Z","stop":"2015-08-07T06:45:14Z","duration":875,"description":"check archive and deletion job","duronly":false,"at":"2015-08-07T06:45:17+00:00","ui_modified_at":"1438929914","uid":}}
2015-08-07 06:45:18.068 [6180 1]:ui:D:DisplayOnlineState online
2015-08-07 06:45:18.069 [6180 0]:ui:D:DisplayMinitimerAutocomplete
2015-08-07 06:45:18.143 [6180 0]:ui:D:DisplayTimerState
2015-08-07 06:45:18.176 [6180 1]:database:D:Updating time entry TimeEntry ID= local_id=35 description=check archive and deletion job wid= guid= pid=0 tid=0 start=1438929039 stop=1438929914 duration=875 billable=0 duronly=0 tags= created_with=windows_native_app/7.2.233 ui_modified_at=0 deleted_at=0 updated_at=1438929917 in thread 7176
2015-08-07 06:45:18.302 [6180 1]:database:D:User with_related_data=1 saved in 126 ms in thread 7176
2015-08-07 06:45:18.334 [6180 1]:ui:D:DisplayTimeEntryList open=0, has items=36
2015-08-07 06:46:04.544 [6180 4]:timeline_uploader:D:upload_loop_activity (current interval 480s)
(followed by lots of websocket errors which I've now determined to be due to the firewall configuration where I am located)
I've now killed the TogglDesktop process and run it again, and it shows the task I'd click Start on as running and has the correct start time and duration. Stopped that task, click the > button to restart it, so far so good. Stopped that, wrote in a new task, clicked Start, again ok. It was working ok all day yesterday after installing the latest build, although I only stopped and started tasks a couple of times.
I've gone back through the log a little and found the following log entries which would explain the uploader interval:
2015-08-07 06:31:00.090 [6180 5]:websocket_client:E:Timeout
2015-08-07 06:31:01.894 [6180 1]:database:D:User with_related_data=1 saved in 0 ms in thread 7176
2015-08-07 06:31:21.273 [6180 4]:ServerStatus:D:UpdateStatus status_code=0
2015-08-07 06:31:21.350 [6180 4]:timeline_uploader:W:backoff
2015-08-07 06:31:21.380 [6180 4]:timeline_uploader:D:Upload interval set to 120s
2015-08-07 06:31:21.405 [6180 4]:timeline_uploader:E:Timeout
2015-08-07 06:33:21.451 [6180 4]:timeline_uploader:D:upload_loop_activity (current interval 120s)
2015-08-07 06:33:21.481 [6180 4]:user:D:CompressTimeline  user_id= chunk_up_to=1438929000 number of events=274
2015-08-07 06:33:21.506 [6180 4]:user:D:CompressTimeline done in 0 seconds, 274 compressed into 0 chunks
2015-08-07 06:33:21.548 [6180 4]:database:D:User with_related_data=1 saved in 0 ms in thread 7864
2015-08-07 06:33:21.573 [6180 4]:timeline_uploader:D:Uploading 7 event(s) of user
2015-08-07 06:33:21.604 [6180 4]:timeline_uploader:D:[
<list of running applications ....>
]
2015-08-07 06:33:21.668 [6180 4]:TogglClient:D:Sending request to https://timeline.toggl.com/api/v8/timeline ..
2015-08-07 06:33:23.574 [6180 1]:database:D:User with_related_data=1 saved in 0 ms in thread 7176
2015-08-07 06:33:42.743 [6180 4]:ServerStatus:D:UpdateStatus status_code=0
2015-08-07 06:33:42.916 [6180 4]:timeline_uploader:W:backoff
2015-08-07 06:33:42.941 [6180 4]:timeline_uploader:D:Upload interval set to 240s
2015-08-07 06:33:42.971 [6180 4]:timeline_uploader:E:Timeout
2015-08-07 06:36:08.994 [6180 1]:TogglClient:D:Sending request to https://www.toggl.com/api/v8/me?app_name=windows_native_app&with_related_data=true&since=1438878753 ..
2015-08-07 06:36:30.312 [6180 1]:ServerStatus:D:UpdateStatus status_code=0
2015-08-07 06:36:30.397 [6180 1]:ui:E:Timeout
2015-08-07 06:36:30.423 [6180 1]:ui:D:DisplayOnlineState no network
2015-08-07 06:37:42.997 [6180 4]:timeline_uploader:D:upload_loop_activity (current interval 240s)
2015-08-07 06:37:43.077 [6180 4]:user:D:CompressTimeline  user_id= chunk_up_to=1438929000 number of events=274
2015-08-07 06:37:43.113 [6180 4]:user:D:CompressTimeline done in 0 seconds, 274 compressed into 0 chunks
2015-08-07 06:37:43.144 [6180 4]:database:D:User with_related_data=1 saved in 0 ms in thread 7864
2015-08-07 06:37:43.180 [6180 4]:timeline_uploader:D:Uploading 7 event(s) of user
2015-08-07 06:37:43.215 [6180 4]:timeline_uploader:D:[
<list of running processes ...>
]
2015-08-07 06:37:43.282 [6180 4]:TogglClient:D:Sending request to https://timeline.toggl.com/api/v8/timeline ..
2015-08-07 06:37:45.181 [6180 1]:database:D:User with_related_data=1 saved in 0 ms in thread 7176
2015-08-07 06:38:04.338 [6180 4]:ServerStatus:D:UpdateStatus status_code=0
2015-08-07 06:38:04.423 [6180 4]:timeline_uploader:W:backoff
2015-08-07 06:38:04.459 [6180 4]:timeline_uploader:D:Upload interval set to 480s
2015-08-07 06:38:04.489 [6180 4]:timeline_uploader:E:Timeout
At no point did the UI display a message saying that the server was offline (at this point in time I was connected to a VPN with no outgoing internet access, so the lack of internet connection was expected). It was however responsive, and was responsive when I disconnected from the VPN. It was only when I stopped the running task and started a new one did it hang.
What's also a little concerning going through the logs is how much data is sent to the Toggl servers regarding the applications I have running on my laptop. Does it really need to know this information?

@dan-sorion what version of the app are you using?
you can disable timeline recording; then no data will be sent to Toggl server.

Hello,
Unfortunately the issue is still here with v7.2.233.
Windows 8.1 Enterprise 64bits.
Stopped a task and the UI froze. Here's the log :
2015-08-07 12:23:50.771 [8880 0]:toggl_api:D:toggl_stop
2015-08-07 12:23:50.777 [8880 1]:ui:D:DisplayTimeEntryEditor
2015-08-07 12:23:55.259 [8880 5]:websocket_client:D:restarting
2015-08-07 12:23:55.264 [8880 5]:websocket_client:D:createSession
2015-08-07 12:23:55.267 [8880 5]:websocket_client:D:deleteSession
2015-08-07 12:23:55.273 [8880 5]:websocket_client:D:session deleted
2015-08-07 12:23:55.539 [8880 5]:websocket_client:E:SSL Exception: error:14090086:SSL routines:SSL3_GET_SERVER_CERTIFICATE:certificate verify failed
2015-08-07 12:24:26.556 [8880 5]:websocket_client:D:restarting
2015-08-07 12:24:26.560 [8880 5]:websocket_client:D:createSession
2015-08-07 12:24:26.562 [8880 5]:websocket_client:D:deleteSession
2015-08-07 12:24:26.564 [8880 5]:websocket_client:D:session deleted
2015-08-07 12:24:26.825 [8880 5]:websocket_client:E:SSL Exception: error:14090086:SSL routines:SSL3_GET_SERVER_CERTIFICATE:certificate verify failed
2015-08-07 12:24:47.801 [8880 4]:timeline_uploader:D:upload_loop_activity (current interval 60s)
2015-08-07 12:24:47.835 [8880 5]:websocket_client:D:restarting
2015-08-07 12:24:47.838 [8880 5]:websocket_client:D:createSession
2015-08-07 12:24:47.840 [8880 5]:websocket_client:D:deleteSession
2015-08-07 12:24:47.843 [8880 5]:websocket_client:D:session deleted
2015-08-07 12:24:48.098 [8880 5]:websocket_client:E:SSL Exception: error:14090086:SSL routines:SSL3_GET_SERVER_CERTIFICATE:certificate verify failed
2015-08-07 12:25:10.123 [8880 5]:websocket_client:D:restarting
2015-08-07 12:25:10.127 [8880 5]:websocket_client:D:createSession
2015-08-07 12:25:10.130 [8880 5]:websocket_client:D:deleteSession
2015-08-07 12:25:10.132 [8880 5]:websocket_client:D:session deleted
2015-08-07 12:25:10.391 [8880 5]:websocket_client:E:SSL Exception: error:14090086:SSL routines:SSL3_GET_SERVER_CERTIFICATE:certificate verify failed

@tanel I'm using 7.2.233.
I've disabled timeline recording, I thought I'd already done that before but obviously not.
I won't be able to do any more testing for a couple of weeks as it's now vacation time :)

Hi!
Still not working. After install TogglDesktopInstaller-7_2_233-2015-08-06-10-59-29.exe. When run toggle - in system tray in beginner grey icon, after few second - red. double click or click for context menu not working. Win 8.1 64bit
How i can trun off autoupdate? I have TogglDesktopInstaller-7_2_36 installer , wich stable working on my machine. But after few time, toggl updatet to last version. And i can't use tracker

7_2_219-2015-07-28-18-07-06  would start but then hang when the UI was displayed (I trying clearing clearing cache & re-installing) Windows 8.1 PRO 64bit
7_2_233-2015-08-06-10-59-29  is working on my system.

7_2_233-2015-08-06-10-59-29 is working.  But the Project dropdown isn't showing any of my projects.  I tried clearing the cache and logged back in, but when I did this the app froze.  Manually killing the app and restarting allow the app to work, but projects are still missing.
Here is what the logs look like when it hangs;
2015-08-11 10:23:26.402 [3364 8]:database:D:Inserting time entry TimeEntry ID=260480418 local_id=0 description=Ribbon Customization wid=727029 guid=db071883-aa97-405e-8399-0696462da503 pid=10422670 tid=0 start=1439235666 stop=1439239196 duration=3530 billable=0 duronly=0 tags= created_with= ui_modified_at=0 deleted_at=0 updated_at=1439239199 in thread 2024
2015-08-11 10:23:26.412 [3364 8]:database:D:User with_related_data=1 saved in 401 ms in thread 2024
2015-08-11 10:23:26.418 [3364 8]:ui:D:DisplayTimeEntryList open=0, has items=50
2015-08-11 10:23:27.948 [3364 1]:TogglClient:D:Sending request to https://www.toggl.com/api/v8/updates?app=td&channel=beta&platform=windows&version=7.2.233&osname=Windows NT&osversion=6.2 (Build 9200)&osarch=IA32 ..
2015-08-11 10:23:28.085 [3364 1]:TogglClient:D:GET /api/v8/updates?app=td&channel=beta&platform=windows&version=7.2.233&osname=Windows%20NT&osversion=6.2%20(Build%209200)&osarch=IA32 HTTP/1.1
Connection: Close
User-Agent: windows_native_app/7.2.233
Transfer-Encoding: chunked
Content-Encoding: gzip
Host: www.toggl.com
Accept-Encoding: gzip

2015-08-11 10:23:28.090 [3364 1]:TogglClient:D:Request sent. Receiving response..
2015-08-11 10:23:28.134 [3364 1]:TogglClient:D:Response status code 200, content length 4, content type application/json; charset=utf-8, unknown content encoding
2015-08-11 10:23:28.139 [3364 1]:TogglClient:D:X-Toggl-Request-Id bd79c43f-2d65-45f2-6c05-d649c0ad8749
2015-08-11 10:23:28.143 [3364 1]:ServerStatus:D:UpdateStatus status_code=200

We are aware of the issue with the project dropdown, and are investigating.
Thanks for your help and patience, everybody!

I'm having the same issue.  For what it's worth, I suspect my corporate antivirus, Kaspersky Endpoint Security 10, blocking partial network communication with the app.  Nothing to support this other than a hunch.  Anyone else using antivirus with built-in firewall, especially Kaspersky?  I'm on a Win 7 64-bit machine and can reproduce it on two different machines.

@tombeardhid We have had reports from users who have had issues with Kaspersky blocking the app. The users have contacted Kaspersky and have gotten solution from their support.

Hey guys!
We got a new build that is holding up pretty well in our own tests:
https://assets.toggl.com/installers//TogglDesktopInstaller-7_2_261-2015-08-11-23-59-06.exe
Hopefully, the freezing/crashing is gone for good, and the projects should be back as well.
It would be great if you could give it a try and let us know how it is working for you.

@amulware Installing it right now. Fingers crossed !
I'll report back by the end of the day.
Thanks for your efforts !

Just so you guys know: There are some issues with the edit window not always updating correctly when editing right now. We know what causes this and are already on it.

installed 7_2_261-2015-08-11-23-59-06.  no hangs, project dropdown is working.

the issue is fixed now; we're pushing to beta. thanks to everybody for the patience and feedback.


Suggestion: Enabling Discussions

Can we enable Discussions please?
This way it could reduce the number of issues from asking the same questions more or less repeatedly.
I have some questions to ask from a user's perspective and I don't think opening an issue would be ideal.
@brammool what do you think?

Probably a duplicate of #8291 . See the following comment by Bram: https://github.com/vim/vim/issues/8291#issuecomment-851631407.

There already is https://vi.stackexchange.com/ for discussions.   Having two places does not seem like a good idea.

Bummer...I don't want to create yet another account for the sake of asking a couple of questions; I thought here would be much more convenient and appropriate to have questions located in one place.
Oh well, closing this ticket.

@stefanos82 there is the<EMAIL_ADDRESS>mailing list for user discussions (archive on google groups)
There is also a very active IRC community at #vim on libera.chat, which is a great place for discussion, and getting questions answered.

@nickspoons after the Freenode incident I decided to never deal with opening account on any platform or service, for the sake of asking a single question every 3 to 5 years; in other words, it's not worth it.
Besides, I had not used IRC for a very long time because nearly all channels were too toxic, for me at least.
If I could sign-in in libera.chat with guest account to ask my question would be handy; but to open an account for a single question...no, thank you.

then use the mailinglists for discussions.

@stefanos82 there is the<EMAIL_ADDRESS>mailing list for user discussions (archive on google groups)
There is also a very active IRC community at #vim on libera.chat, which is a great place for discussion, and getting questions answered.

https://www.vim.org/community.php still points to freenode though.

then use the mailinglists for discussions.

You cannot ask a question without subscribing first.

Yes, and the first few messages will even be moderated. But you need an account for any other discussion forum as well.

We had to close Vim tips for that reason.
Since these days a lot of people go to github, I added a few lines at the top of the readme file.  I only want to put the most popular places there, if a useful one is missing let me know.

Those who are interested in answering my naive questions can visit my Discussions' section for Nelua's syntax highlighting support.
Thank you very much for the multiple suggestions.


fix: Increase otel-agent CPU request to 20m

The limits are being set to request values on autopilot, which has been causing the otel-agent to fail liveness probes and get restarted. This increase should help prevent that.

The cpu request hasn't changed for a few releases. Did we introduce something new in this release that causes the need to increase cpu request/limit on Autopilot?

The cpu request hasn't changed for a few releases. Did we introduce something new in this release that causes the need to increase cpu request/limit on Autopilot?

I don't know. All I know is that it's failing on autopilot-rapid, because it's setting CPULimit = CPURequest.

This PR may have increased resource usage of the otel-agent somewhat: https://github.com/GoogleContainerTools/kpt-config-sync/pull/763
The generation metrics label isn't going to Cloud Monitoring, but it does go through the otel-agent and otel-collector.

Given that this number will multiply by the number of RSyncs in the cluster, I'd like to avoid increasing it if possible (or keep it to a minimum). Here's our current number: https://cloud.google.com/anthos-config-management/docs/how-to/installing-config-sync#detailed_resource_requests

I think risking breakage to keep requirements low is a bad trade-off, but now that the liveness probes are removed, it at least won't cause endless restarts.


Fix for flaky OSX tests from ALCProxy

Fixes to some flaky tests causing failures on the OSX release build coming from ALCProxy.
See #2745 for details.

cc @sbomer @AaronRobinsonMSFT @jkoritzinsky

/azp run

Re-running CI again just to spot check it passes consistently.


Do someone need to support Jetpack Compose?

I need to jetpack compose support. I would appreciate it if you could implement it.

+1 on this

So, I think we will make support


Handle all the logic for installation in Make

Right now I handle the logic & order of installation inside script/install but Make can handle this in a better way, so some ideas:

Split install functions into seprate files, create make tasks & let make handle the dependencies
Do everything inside Makefile but why would I do that?

https://github.com/hauleth/dotfiles

Look at Saltstack because I need more complicated logic & templating for work/personal setups
Ex: https://github.com/skbolton/titan


Research migration plan for site webhooks

In order for customers to migrate to pages, their sites' webhooks will need to be update them to point to pages and not federalist.
Background

https://github.com/18F/federalist/blob/9dcc6645072cada2ccc5198fade800a81b033c2d/api/services/GitHub.js#L169-L190

Acceptance

TDB

As detailed here:

During part of the migration period, the Federalist API will remain up to accept webhook POST requests even when the UI itself is shut down.
We will ask users to manually update their webhook targets to pages during the migration process

We'll track webhook requests per application with a saved logstash query. The exported csv can be parsed with this "one-liner" to show the latest author and repo:
cat export.csv | csvcut -c "@message" | tail -n +2 | awk 'length<16384' | sed -E 's/^\"//g' | sed -E 's/\"$//g'  | sed -E 's/\"\"/"/g' | sed -E 's/info: HTTP POST \/webhook\/github //g' | jq -r '.meta.req.body | [ .repository.html_url, .sender.login ] | @csv'

Adding one other research note before closing: our current plan is to stop the flow of new Federalist webhook requests by not allowing new sites to be created on Federalist. As a backup option, we can also change the webhook URL environment variable to the Pages endpoint which will create updated webhooks even if the site is created on Federalist rather than Pages.


resultQuery input doesnt handle parameters.

https://github.com/ibissource/iaf/blob/45f585252fe84b0713136c536bea024316fc925a/core/src/main/java/nl/nn/adapterframework/jdbc/JdbcQuerySenderBase.java#L809
using ?{} parameter in resultQuery restults in hard error while executing with error like
Syntax error in SQL statement "SELECT IDENTIFICATIE FROM IDENTIFICATIE WHERE TYPE = ?{IdentificatieType}[*]" [42000-214]
replacing with hardcoded value makes the query work.

any parameter.

You didn't specify a version, so I tested on master. I think that you forget to set the attribute useNamedParams to true. I was able to reproduce the error message provided above using an unit test with the same situation, so I will close this issue. The error message is fair, because there is indeed a syntax error. You forgot to enable useNamedParams, which caused ?{IdentificatieType} to not be substituted.

Perhaps you can share the unit-test 😄

After taking another look it seems that a test (FixedQuerySenderTest#testNamedParamInQueryFlagFalse) such as the one, which I wrote already exists.


can"t make it run

Hi,
from clean virtualenv with only pyside2 and python-qt-live-coding :
^C(.venv) jimmy@cacahuete:~/tras/python-qt-live-coding$ python_qt_live_coding examples
Error importing standalone/module: No module named 'standalone/module'
QQmlApplicationEngine failed to load component
file:///home/jimmy/tras/.venv/lib/python3.7/site-packages/livecoding/live.qml:31 Type LiveCodingPanel unavailable
file:///home/jimmy/tras/.venv/lib/python3.7/site-packages/livecoding/LiveCodingPanel.qml:220 Cannot assign to non-existent property "extensions"

^C(.venv) jimmy@cacahuete:~/tras/python-qt-live-coding$ python_qt_live_coding examples/standalone
QQmlApplicationEngine failed to load component
file:///home/jimmy/tras/.venv/lib/python3.7/site-packages/livecoding/live.qml:31 Type LiveCodingPanel unavailable
file:///home/jimmy/tras/.venv/lib/python3.7/site-packages/livecoding/LiveCodingPanel.qml:220 Cannot assign to non-existent property "extensions"

Have nive day
Jimmy

I have the same error and fixed it by adding property var  in front of the extensions property.  But even after that fix, I could get live coding to work. I made a change in a qml file and expected the app to update (similar to BrowserSync) but it did not. I must be missing something.

I´m stuck here too.
I´m must starting with Pyside+Qt+Qml so maybe there is an obvious solution but i just dont know.
So, what i notice so far:
Start with the same error as above:
QQmlApplicationEngine failed to load component file:///D:/programming/chest/live.qml:32:3: Type LiveCodingPanel unavailable file:///C:/Users/patri/AppData/Local/Programs/Python/Python38-32/lib/site-packages/livecoding/LiveCodingPanel.qml:220:5: Cannot assign to non-existent property "extensions"
in LiveCodingPanel.qml:220:5: u found this:
  ProjectBrowser {
    id: browser
    projectPath: userProjectPath
    extensions: ['qml', 'ui.qml']
  }

ProjectBrowser is a custom Component made on python side that are founded here:
projectbrowser.py
The property that are failing are there:
    @Property('QStringList', notify=extensionsChanged)
    def extensions(self):
        return self._extensions

    @extensions.setter
    def extensions(self, value):
        if self._extensions == value:
            return
        self._extensions = value
        self.extensionsChanged.emit()

After messing around i noticed that none of this custom component properties are working. If u comment the property settings
on the qml file u start to having error in other custom component properties.
So maybe is Pyside that are not binding correctly the properties to the Qt/Qml side.
Not sure if this repo is still being maintained, and i´m still new on this stack so this is the further that i could go by now.

I gave up on this project. However, as it turns out, you can do this in Qt Creator. There is a project template called "Qt for Python - Qt Quick Application - Empty" that creates a pyslide2 qml application.

For live coding, use Build -> QML Preview.


Bumped production release number for 1.24-16

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

/lgtm


Implement str.translate()

I have some questions.

Is it possible to somehow translate a dictionary into a method implementation without explicitly setting a numba dictionary?
Numba does not know how to work with dictionaries of strings of different types, an example is an "test_translate_error". Is there something I need to do in this case, or just ignore it? Also, Numba cannot create dictionaries with None, what should I do in this case?

Can you explain what you meant by "translate a dictionary into a method implementation"?

These are limitations of the current typed-dict implementation. Can you open an issue for your usecase? For now, you may want to skip such cases.

Can you explain what you meant by "translate a dictionary into a method implementation"?

In the function that I am implementing we get the parameter - dictionary. But I can not use the usual dictionary, a typing error occurs. I use numba.typed.Dict (), but may be another way?

numba.typed.Dict() is the current way.
Inside jit region, the syntax {'a': 1} will make a numba.typed.Dict. But outside (in interpreted code), it would be a normal python dictionary.

Thank you for your responses,
Now can you add this PR to queue on review?

Please could you resolve the conflicts by merging master into this branch. Thanks!


Cards

I see cards were added but I don't see them in the gh-pages examples.
https://github.com/Esri/calcite-bootstrap/commit/3d578e0b9592e172744e8a7e865234545ff116c7

Not sure how or when the build takes place, I merged my PR for cards.

There's a first pass of cards in the release. I'd like to enhance the work @julio8a has done and make these more Bootstrap savvy. Currently they are a fixed height and width. Would be nice to have them scalable based on the column width. Maybe the height can be a variable the designer chooses to set or not.

The pages were rebuilt and the cards are shown on the example page:
http://esri.github.io/calcite-bootstrap/examples/#card


Latest lib-sass bump can cause error with @extend

Bump to lib-sass 3.3.5 on the current version seems to have caused the following issue to appear:

You may not @extend an outer selector from within @media

Maybe this rule and this breaking change makes sense, but raising the warning in case of it was not unexpected.

This project is an implementation of the Sass language. Our goal is to be 100% compatible with Sass. Ruby Sass raises this error if @extend is used incorrectly so we must also. LibSass assumes the your code is valid Sass, so as the project gets closer to Sass compatibility behaviour that is relying on LibSass bugs will stop working.
LibSass 3.3.5 implemented some of the previously missing error checking for @extend. I could be that your Sass code has always been invalid but wasn't erring due LibSass bugs. It's impossible to confirm whether this is the case or if there is an actual LibSass bug from the information supplied.
Please confirm you code compiled as expected with Sass.
If you wish to suggest features of file bugs in the Sass language please open an issue with Sass - https://github.com/sass/sass

I guess our code has always been invalid on this point but was properly compiled anyway.
Libsass 3.3.5 just raised the issue for the first time, as you said.
It wasn't a big refactoring to make it work compliant to this rule.
The error made sense with this rule. Not a bug.

You're not alone, we're seeing a lot of this as LibSass becomes more compliant. Thanks for looking into your issue.


[Loc] Request to add Norwegian Nynorsk language in VS Code

I put in a request for this language in transifex logged in via Github, and will help to contribute if approved.

@andreona , I enabled Norwegian Bokmål (nb) for VS Code projects in Transifex according to request in  issue 46711. I understand you requested different flavor of Norwegian language. I assume you are fine to contribute to Norwegian Bokmål (nb), So I added you there. Let me know if you feel it is not good.

The Norwegian Bokmål (nb) request: https://github.com/Microsoft/vscode/issues/46711

We don't have plan to enable additional language for VSCode in MLCP in near term. Hence close the request here.
New request should be logged in Localization Community Repo


Confirmation dialog for "Delete All" history button

Please add a confirmation dialog for the "Delete All history" button. I accidentally clicked it, and it removed all my history, which I didn't want to do.
A confirmation for deleting a single conversation may also be useful.

Hi @sintezcs, I'm sorry to hear that.
This was actually something that I was expecting to happen to someone sooner or later.
I will try to add the dialog in the upcoming days.
FYI: This is a good time to update the plugin to 1.7.0 - https://plugins.jetbrains.com/plugin/21056-codegpt/versions/stable/310112


Remove upstreamed U-Boot rpi0w defconfig patch

The patch 0002-rpi_0_w-Add-configs-consistent-with-RpI3.patch has been
applied to the upstream U-Boot repo and is now in the release that
openembedded-core uses so remove it.
[Issue #239]
Signed-off-by: Jonathan Haigh<EMAIL_ADDRESS>

Can one of the admins verify this patch?

Hi Andrei, thanks for the review.
I think I've updated the commit message to conform to those guidelines. If I haven't can you let me know which bit needs changing please?
Thanks.

u-boot: Remove upstreamed U-Boot rpi0w defconfig patch

Okay, thanks, I'll change to that.

As well, you mention #239 twice.

The guidelines suggest including a URL when referencing issues, and the example is
[RPM Ticket #65] -- http://rpm5.org/cvs/tktview?tn=65,5

I used
[Issue #239] -- https://github.com/agherzan/meta-raspberrypi/issues/239

But GitHub renders both #239 and the URL as #239. Should I just skip the URL?

Edited the short description. Hopefully is okay now.

ok to test


Errors when Units are Deleted in-game

Watching a replay where my opponent deleted a large number of units before resigning, the replay viewer spammed a bunch of errors in response.

The following was copied from the player:
System.Collections.Generic.KeyNotFoundException: The given key '0' was not present in the dictionary.
at AWBWApp.Game.API.Replay.Actions.DeleteUnitAction.PerformAction(ReplayController controller)+MoveNext()
at AWBWApp.Game.Game.Logic.ReplayController.Update()

Thanks for the report. Do you have the replay that caused this?

It would be this game, which you can still pull the replay from: https://awbw.amarriner.com/2030.php?games_id=714530

Thanks. Will look into it.
Leaving this here for when I work on this:
714530.zip


Any plans for contribute or white-box patch?

Hi Ulti
We are a team of multi-dedi server platform and want to merge this dll into win pods and add some other functions, to have the best performance either this AC is a pod service API or use static function to import into the memory than injection. So if it's possibe to contribute like Unofficial-API or provide a PDB to let us contribute/modify this inject?

Message me in discord: https://discord.gg/palguard


[HOLD for payment 2023-10-25] [$500] Rate - When Rate is changed in offline mode red dot appear back online

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Action Performed:
Precondition; User has change the Workspace name in online mode

Navigate to staging.new.expensify.com
Go Offline
Go to Settings>Workspace>Reimbursement
Change the Rate to random value & click Save button
Go back Online

Expected Result:
The Rate should remain the value updated in Step 4
Actual Result:
Red dot appears in Rate section when back online.
Updated Rate value does not remain
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android / native
[ ] Android / Chrome
[x] iOS / native
[x] iOS / Safari
[x] Windows / Chrome
[ ] MacOS / Desktop

Version Number: 1.3.78-2
Reproducible in staging?: Yes
Reproducible in production?: Yes
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
https://github.com/Expensify/App/assets/78819774/2890ad7a-f883-47f1-aa62-41fbe7f6e40e
Expensify/Expensify Issue URL:
Issue reported by: Applause - Internal Team
Slack conversation:
View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~01565aea153d5bddeb
Upwork Job ID:<PHONE_NUMBER>316220416
Last Price Increase: 2023-10-12
Automatic offers:

AmjedNazzal | Contributor | 27217693

Can't repo
https://github.com/Expensify/App/assets/74202751/31cbe582-5bb4-455f-9e5e-e9bf89b9d2eb

@lanitochka17 - can you still reproduce?

Issue is reproducible on the latest build 1.3.80-1
Windows10/Chrome
https://github.com/Expensify/App/assets/78819774/3fb6eabd-17d4-4c93-8800-1468cb30ae15

@AmjedNazzal are you still unable to reproduce?

@NicMendonca I managed to repo it but only following the video, not the steps, I think we should update the steps to include creating new workspace and changing it's name step since this is a core of what is causing the bug so the steps can be something like this:

Navigate to staging.new.expensify.com
Open settings > Workspaces and create a new workspace
Open the settings of the created workspace and change the name then save
Go Offline
Go to Settings>Workspace>Reimbursement
Change the Rate to random value & click Save button
Go back Online

Proposal
Please re-state the problem that we are trying to solve in this issue.
Rate - When Rate is changed in offline mode red dot appear back online
What is the root cause of that problem?
In this PR https://github.com/Expensify/App/pull/12162 we added lastModified to UpdateWorkspaceCustomUnitAndRate command, but the problem is that if a new workspace is created and then we change it's name, unless we open WorkspaceReimburseView at least once while online so that WorkspaceRateAndUnitPage is initiated while having access to the backend, then it will not have access to the correct lastModified data that occured when the name of the workspace was changed, and for that reason when the network is online again and the api is called with the old lastModified, the backend will return "The policy was modified since you loaded it, please reload the page and try again." warning and the rate/unit will not be updated.
What changes do you think we should make in order to solve the problem?
We can add a check in updateWorkspaceCustomUnitAndRate to include lastModified only if the network is online
API.write(
    'UpdateWorkspaceCustomUnitAndRate',
    {
        policyID,
        ...(!networkStatus.isOffline && { lastModified }),
        customUnit: JSON.stringify(newCustomUnitParam),
        customUnitRate: JSON.stringify(newCustomUnitParam.rates),
    },
    {optimisticData, successData, failureData},
);

Result
https://github.com/Expensify/App/assets/74202751/ce53ae1d-a78f-404b-a3fd-8f3cea50edf1

Proposal from @AmjedNazzal looks good to me https://github.com/Expensify/App/issues/28942#issuecomment-1763031740
We should use latest lastModified to get correct results
🎀 👀 🎀 C+ reviewed

Thank you! offer accepted and PR will be ready in a few hours.

@narefyev91PR is ready for review

@narefyev91 bump on BZ check list! ^

@AmjedNazzal  you've been paid 🎉

@narefyev91 bumpy bump ^

The PR that introduced the bug has been identified. Link to the PR - it was additional implementation which is not coming from any regressions
The offending PR has been commented on, pointing out the bug it caused and why, so the author and reviewers can learn from the mistake. Link to comment - N/A
A discussion in #expensify-bugs has been started about whether any other steps should be taken (e.g. updating the PR review checklist) in order to catch this type of bug sooner. Link to discussion - N/A
Determine if we should create a regression test for this bug - N/A

fab! all set here then!


NEON and SSSE3 decoder optimisations

Optimized NEON primitives for decoding RFX, AVC420, AVC444 and progressive
Optimized SSSE3 RFX yCbCrToRGB_16s8u_P3AC4R
Added profiler to tests

@akallabeth do you have any number about the improvements ?

@freerdp-bot test, come on
@hardening Around 7x for YUV conversion and YUV combination. You can test it yourself if you have a rpi, just run the primitive tests. (WITH_PROFILER gives you stats)

@freerdp-bot test


PointPillars export to  onnx ？

PointPillars export to  onnx ？

This is a planned feature, we will support that in the future release. PRs are also welcomed.

Hi, @zyxcambridge
As far as I know, it is a really hard task to export PointPillars to ONNX as a whole. PointPillars voxelization, feature generation, and feature scattering ops are complex and cannot be efficiently converted to ONNX ops.
Personally, I recommand deploy the PointPillars model with 2 separate parts, one for the VFE(linear+maxpooling) part and one for the RPN(backbone+neck+head), and design CUDA kernels(or similar hardware-specific codes) to handle those complex ops. Although these complex ops could be reconstructed by a combination of simple ops, doing this is much less efficient than designing specific custom ops.
Code from Autoware for your reference (2 ONNX, 1 for VFE, and 1 for RPN):
https://github.com/Autoware-AI/core_perception/blob/master/lidar_point_pillars/nodes/point_pillars.cpp
By the way, if you are going to deploy the model on a machine with CUDA, torch script might be a much better choice. because the ops are consistent across PyTorch and TorchLib, you do not need to write feature generation and scattering ops which are already present in mmdet3d. What you need to do is make use of the torch.jit.script and do a graph trace.
Apollo lidar perception module uses torch script to deploy the PointPillars model and split the whole network into 5 parts. the code is here:
https://github.com/ApolloAuto/apollo/blob/d42087aef4074892ed57d33927e160d35ad68889/modules/perception/lidar/lib/detection/lidar_point_pillars/point_pillars.cc
PS: Apollo uses exactly the same weights & same network structure & same hyperparameter published by mmdet3d, but with no official reference. I still wonder if the developers of both repositories come from the same team?@ZwwWayne

mmdet3d weights: https://download.openmmlab.com/mmdetection3d/v0.1.0_models/pointpillars/hv_pointpillars_secfpn_sbn_2x16_2x_waymoD5-3d-3class/hv_pointpillars_secfpn_sbn_2x16_2x_waymoD5-3d-3class_20200831_204144-d1a706b1.pth
apollo weights (serialized by torch script into 5 parts):
https://github.com/ApolloAuto/apollo/tree/master/modules/perception/production/data/perception/lidar/models/detection/point_pillars
apollo point pillars page:
https://github.com/ApolloAuto/apollo/tree/master/modules/perception/lidar

Hi @zhanggefan ,
Thanks for your kind suggestion. We will consider that when we start developing the feature of deployment.
As far as we know, there is no team from Apollo contribute to this codebase, though we are glad to hear that MMDet3D may indeed help some team with their business。

what about votenet to convert onnx, can you give advices？ @zhanggefan @ZwwWayne

This is a planned feature, we will support that in the future release. PRs are also welcomed.

@ZwwWayne Is converting PointPillars to onnx format available now?

@zyxcambridge @baheytharwat
Please refer to https://github.com/speshowBUAA/mmdet3d_onnx_tools. Hope this can help you

Hi, @zyxcambridge
As far as I know, it is a really hard task to export PointPillars to ONNX as a whole. PointPillars voxelization, feature generation, and feature scattering ops are complex and cannot be efficiently converted to ONNX ops.
Personally, I recommand deploy the PointPillars model with 2 separate parts, one for the VFE(linear+maxpooling) part and one for the RPN(backbone+neck+head), and design CUDA kernels(or similar hardware-specific codes) to handle those complex ops. Although these complex ops could be reconstructed by a combination of simple ops, doing this is much less efficient than designing specific custom ops.
Code from Autoware for your reference (2 ONNX, 1 for VFE, and 1 for RPN):
https://github.com/Autoware-AI/core_perception/blob/master/lidar_point_pillars/nodes/point_pillars.cpp
By the way, if you are going to deploy the model on a machine with CUDA, torch script might be a much better choice. because the ops are consistent across PyTorch and TorchLib, you do not need to write feature generation and scattering ops which are already present in mmdet3d. What you need to do is make use of the torch.jit.script and do a graph trace.
Apollo lidar perception module uses torch script to deploy the PointPillars model and split the whole network into 5 parts. the code is here:
https://github.com/ApolloAuto/apollo/blob/d42087aef4074892ed57d33927e160d35ad68889/modules/perception/lidar/lib/detection/lidar_point_pillars/point_pillars.cc
PS: Apollo uses exactly the same weights & same network structure & same hyperparameter published by mmdet3d, but with no official reference. I still wonder if the developers of both repositories come from the same team?@ZwwWayne

mmdet3d weights: https://download.openmmlab.com/mmdetection3d/v0.1.0_models/pointpillars/hv_pointpillars_secfpn_sbn_2x16_2x_waymoD5-3d-3class/hv_pointpillars_secfpn_sbn_2x16_2x_waymoD5-3d-3class_20200831_204144-d1a706b1.pth
apollo weights (serialized by torch script into 5 parts):
https://github.com/ApolloAuto/apollo/tree/master/modules/perception/production/data/perception/lidar/models/detection/point_pillars
apollo point pillars page:
https://github.com/ApolloAuto/apollo/tree/master/modules/perception/lidar

Hi, I am trying to do the same too. Converting a two-staged onnx model to single stage. However, the outputs for pytorch and ONNX models aren't aligned even with consistently aligned inputs. If anyone could give some suggestion, it would be really helpful.


[video_player] Adjust iOS test timeouts

The native iOS tests load network resources, so need longer timeouts to
avoid flake.
Pre-launch Checklist

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] I read the Tree Hygiene wiki page, which explains my responsibilities.
[x] I read and followed the relevant style guides and ran the auto-formatter. (Unlike the flutter/flutter repo, the flutter/plugins repo does use dart format.)
[x] I signed the CLA.
[x] The title of the PR starts with the name of the plugin surrounded by square brackets, e.g. [shared_preferences]
[ ] I listed at least one issue that this PR fixes in the description above.
[x] I updated pubspec.yaml with an appropriate new version according to the pub versioning philosophy, or this PR is exempt from version changes.
[x] I updated CHANGELOG.md to add a description of the change, following repository CHANGELOG style.
[ ] I updated/added relevant documentation (doc comments with ///).
[x] I added new tests to check the change I am making, or this PR is test-exempt.
[x] All existing and new tests are passing.

Actually now testDeregistersFromPlayer is failing https://cirrus-ci.com/task/5913296061595648?logs=native_test#L1724

This timeout should probably also be increased:
https://github.com/flutter/plugins/blob/4ebfcb53b93144cbe4a7e848c992fb394a697641/packages/video_player/video_player_avfoundation/example/ios/RunnerTests/VideoPlayerTests.m#L69-L70

why do we have timeouts at all?

why do we have timeouts at all?

The iOS and macOS "expectation" APIs all require timeout numbers:

- waitForExpectations:timeout:
Waits for the test to fulfill a set of expectations within a specified time.
- waitForExpectations:timeout:enforceOrder:
Waits for the test to satisfy an array of expectations and specifies whether they must occur in the array’s order.
- waitForExpectationsWithTimeout:handler:
Waits until the test fulfills all expectations or until it times out.
https://developer.apple.com/documentation/xctest/xctestcase?language=objc

We could make that timeout the max double to infinitely wait, but I've never seen that pattern in these kinds of tests.

We could make that timeout the max double to infinitely wait, but I've never seen that pattern in these kinds of tests.

It makes failures really obnoxious. Locally it seems like a hang, and on CI it makes it take whatever the bot timeout is to get results. And it guarantees that you only learn about the first failure per run which is often frustrating.
There's no legitimate case where an expectation like this takes longer than N for some reasonable N. If it's only supposed to take a second or two, we don't need to let it run for an hour to know it's broken.
(N might be longer than 30 seconds, but I certainly wouldn't want to set it to more than a minute or two.)


kernel: Make If statement have essentially Boolean type

Make if statement using pointers explicitly check whether the value is
NULL or not.
The C standard does not say that the null pointer is the same as the
pointer to memory address 0 and because of this is a good practice
always compare with the macro NULL.
part of #9889

OK, I'm going to push back a bit on this one, because IMHO this particular convention is a good thing and not a bad thing.  What exactly does MISRA say here?  Because I'm all but certain ISO C does, in fact, guarantee that NULL, however defined (which is to say, defined as zero, because no one has ever used a non-zero NULL on a shipping system), will evaluate to a boolean false.  C++ did this by defining "0" as a pointer constant for a while, though they've since moved to that "nullptr" thing as official.
Honestly I think these added bytes hurt readability and don't help, especially when combined with loop iteration and while() tests with side effects.

c standard is not clear about NULL definition, in <IP_ADDRESS> it says:
An integer constant expression with the value 0, or such an expression
cast to type void *, is called a null pointer constant. 55) If a null
pointer constant is converted to a pointer type, the resulting pointer,
called a null pointer, is guaranteed to compare unequal to a pointer to
any object or function.

but in 7.17:
The macros are NULL
which expands to an implementation-defined null pointer constant;

In other hand MISRA-C just says:
The controlling expression of a for statement is optional. The rule
does not re quire the expression to be present but does require it to have essentially Boolean type if it is present.

IMHO it increase code readability too, I know that we have to type more, but when you see something like if (var == 0) { ... } will automatically know that var is an integer and when is if (var == NULL) { ...} you know that var is a pointer.

another thing, this guideline is not only to pointers, but all if statements should evaluate a Boolean expression so, we'll need to change the others types types too, integers should be compared against constants .... The only exception is:
bool t = true;
...
if (t) {
...
}

will automatically know that var is an integer

That's the logic that gave us hungarian notation. :)
In fact variable typing isn't really a big limit to readability.  Computers check types for us automatically, and we all learn very quickly to read code with the assumption that typing is correct.  What we really want to know in code like that is "the variable is VALID", which is IMHO better conveyed by simply using its name in a boolean context.
But whatever.  I'm not going to put up much of a fuss.  This is just a spot where I genuinely feel we're moving backwards.

There is this ambiguity in C (at least 90 and 99) standard, I know that every sane implementation define NULL to ((void *)0), but is always better strict follow the standard when is something that has no harmful effect. It definitely helps portability.
Just my two penny worth :)

OK, I'm going to push back a bit on this one

My 2 cents: I wish there was push back on (void)memset ;-). As an example, on my recent memory (2-3 months), there were 2 fixes in Zephyr, when r was used instead of !r and/or vice-versa.
I have some experience recent years working with a codebase which enforces this rule (which can be seen as MISRA'esque, or otherwise as Java'esque), and have to admit it makes the code clearer/chance of mistakes lower.  But p != NULL is ok.)

@pfalcon hungarian notation hurts :)
To my mind a lot of these MISRA-C rules make sense in general, they increase portability and make the code less error prone. The problem is that we have  a lot of bad API that cause these kind of patches :/

@pfalcon I update this patch adding the missing parentheses in the right expression. For the sake of readability if you guys prefer I can change it removing parentheses in both expressions (will increase the patch I guess :) )
I've added other similar patch  for while() statement, and another one using boolean constants instead of 0 or 1. The latter I can remove from this pr if we need to discuss it separately.
Thanks for reviewing it :)

I've added other similar patch for while() statement, and another one using boolean constants instead of 0 or 1. The latter I can remove from this pr if we need to discuss it separately.

I'm all for using boolean expressions in boolean contexts and for boolean func params. +1 from my side.

@nashif can we merge it ?


myriads of windows push up when moving over playground window while editing text

Build 20 May. Not reproducible.
I had the GT playground open for some hours. Then I edited some of the program text (nothing sophisticated). Suddenly I get myriads of the following messages.

looks like I somehow violated the playground with some action -> check test coverage ;)

Thanks. We will look at it.

This should have been addressed now. Please let us know if you still encounter similar issues.


Plugin not supporting in Kibana version 6.2.3

I tried installing the plugin in 6.2.3 version but i am not able to install it. Is there any branch for 6.2.3 ?

https://github.com/PhaedrusTheGreek/transform_vis/releases/tag/6.2.3


Support lifecycle reporting for tvOS

Is your feature request related to a problem? Please describe.
Add support for application_foreground and application_background events on tvOS.
Describe the solution you'd like
application_foreground fires on app foregrounding, application_background fires on app backgrounding
Describe alternatives you've considered
No alternatives.
Additional context
May requires tvOS 9.0+ due to use of NSNotificationCenter APIs.

Thanks a lot @miike for fixing this!

No worries, thanks @AlexBenny !


It seems that pattern repeat itself faster then in 289.
I experimenced it with iPad (iPad 2 mini I think).
Could anyone point where the problem could be?

Recent events force me to ask, would be possible to reimplement this noise with precision mediump float?

The problem here is that for compatibility with older versions of GLSL and with WebGL, the permutations are done by integer math with floating point variables.
I see no way to make the permutation polynomial method work with a 16-bit floating point representation. The permutation set would then have to be very small to keep the numbers in the range where the integers N and N+1 map to different representations, and the periodic artifacts from using such a small set of gradients would most probably be too strong to be acceptable.
If texture bandwidth is not a problem, I would recommend using the texture-based lookup table for the permutations, as demonstrated by the older version of noise which I link to on the wiki page in the section "Other noise implementations". The texture-heavy version is generally not as fast on mobile platforms, but on desktop GPUs it can actually be about twice as fast as the pure computational version.


Improve Support for data lists

Verify if IEnumerable and / or support for foreach can be added for the following elements:

[ ] GLib.Array
[ ] GLib.Sequence
[ ] GLib.SList

Is it possible to implement custom generic variants of the mentioned types so they integrate seemlessly with linq and allow easy data access?

Maybe my earlier draft on GstIterator could help here: https://github.com/gircore/gir.core/pull/157


Saving NVS while testing connection leaves a persistent WS connection on WAS

With latest pull, when I save nvs config and it tests the websocket connection, it never closed the connection. As a result the client connections goes up and never goes down. For instance I have 2 esp boxes connected, and after saving 5 times my connections count shows as 7 and is remaining there even after refreshes.

Good catch. I was wondering where that came from. Should be fixed in feature/was_url. As this is an issue in a feature branch, and commit introducing the bug has been fixed up, a fixes tag in the commit message makes no sense. May I kindly ask you to close this issue if it is fixed for you?

Confirmed it is fixed now in feature/was_url!


Cannot send back the annotations to OMERO server

Hello,
First of all thank you for the amazing software !
I am able to connect my OMERO server (5.13)<https://my.omero.server> to my QuPath (0.3.2) through the OMERO extension (0.3.0.jar), browse through all datasets and add images to projects.
Unfortunately, when I try to use the send annotations to OMERO the process start with some confirmation message but fail with this message:
ERROR: Could not send object: <https://my.omero.server>/iviewer/persist_rois/
Could you help me sort this out ?
Best regards,
Guikks

Hi @Guikkz I'd recommend posting your question on the forum at https://forum.image.sc/tag/qupath
More people who use OMERO and QuPath together may see it there - and you can use both OMERO and qupath as tags on your post.

Discussion is at https://forum.image.sc/t/cannot-send-back-the-annotations-from-qupath-to-omero-server/69690


Add WeightLog handler for saving weights and changes into a CSV file

Would it make sense to have some way of deferring the write to out on epoch end rather than writing out each batch?

Would it make sense to have some way of deferring the write to out on epoch end rather than writing out each batch?

@cancan101 You worried your SSD will break?

and related, does it makes sense to keep the file open (and perhaps the  DictWriter), rather than open and closing the file? You can always call flush if you are scared of losing data.

@cancan101 Thanks for the feedback. Addressed the issues. Now keeping file open throughout. Also buffering.


Pass sockaddr_storage by reference

This PR updates Socket.hpp to pass sockaddr_storage by reference to bind(), connect(), address_string(), and discover_mtu().  There are also a few small cleanups with regards to unused variables and some non-printing characters.

Codecov Report

Merging #176 (0132eca) into master (a4be8b5) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #176   +/-   ##
=======================================
  Coverage   66.81%   66.81%
=======================================
  Files          69       69
  Lines        7410     7410
=======================================
  Hits         4951     4951
  Misses       2459     2459

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 c866fd2...0132eca. Read the comment docs.

Since it passed tests, If you're interested in also moving socket implementations into a .cpp, that's in the branch socket-deinline and could be merged into this PR. Or not, but I'm a little suspicious of this much inline code, I guess. I'll make another inline review comment about a warning I get on Mac with gcc.

Oh, never mind inline comments, I just added the fixes to socket-deinline, with detailed commit messages. If you decide not to de-inline the socket header, then I can apply those small patches to the same functions in the .hpp on socket-reference.

@league I think I'm happy with this now that Socket.cpp is a thing.


Add support for params and query object streams

Basically, if one of the arguments (or the params.query) is an observable we should do what was suggested in https://github.com/feathersjs/feathers-reactive/issues/39 and mergeMap them into a single stream:
const service = app.service('messages');
const query = new RxJS.BehaviorSubject({ $skip: 0 });

const find = query.mergeMap(query => service.find({ query }));

find.subscribe(data => console.log('Data is', data));

query.next({ $skip: 10 });

You also need to choose if you want to switch or merge. e.g. classic pagination vs infinite scrolling.
How about this:
find(params$, [switch = false]){
  if(isObservable(params$)){
    return switch
      ? params$.switchMap(query => this._find(params))
      : params$.mergeMap(query => this._find(params));
  } else {
    return this._find(params);
  }
}


[WARNING] -encoding may not supported on older JDKs.

Hello dear jaxws-maven-plugin developers,
Is there a way to get rid of the warning "-encoding may not supported on older JDKs."?
With the following plugin configuration the warning is still issued:
                    <args>
                        <arg>-encoding UTF-8</arg>
                    </args>

Kind regards
Michael

I have found there reference: https://github.com/mojohaus/jaxws-maven-plugin/search?l=java&q=encoding&utf8=✓
Can you provide a PR for that?

Sorry michael-o, I don't understand. PR means pull request? If so, I do not for what to provide a pull request.

Lets rephrase, what do you expect to see and what JDK versions are affected?

Thanks for rephrasing!
It says: "[WARNING] -encoding may not supported on older JDKs. Use  to bypass this warning if you really want to use it."
I have "encoding" in "args". So I expect no warning. JDK is Oracle 1.8.0_102.

Just checked, encoding has been added back in 2011-10: https://github.com/gf-metro/jaxws/commit/0318f69313236152cdc8e5f0ecd321b081969afc, five years ago. I think it is safe to drop this warning.

I checked and Oracle JDK 1.7.0_80 does not support the -encoding parameter. It is supported by Oracle JDK 1.8 though (checked 1.8.0_40 and 1.8.0_112).

As long as we support JDK 7 we can't change this. Closing.

Can't the warning only be emitted if the current Java version is 1.7 or less?

A basic principle with Maven is that builds should be portable. So you should get the same end result regardless of if (for example) JDK 1.7 or JDK 1.8 is used (that is, assuming the target is Java 7). So any checks of the JDK used affecting this is not a good thing.
So in this case, if you know that JDK 8 will be used for the build (you could have an enforcer rule for that) you should use the suggest args parameter solution. Now I see that the first comment in this thread says that also causes the warning message. That shouldn't be the case though. I'll look into that.

I see there is an issue here. The plugin's encoding param defaults to project.build.sourceEncoding, which typically is/should be set. SO even if you don't specify encoding configuration it is implicitly set by this default config. The plugin therefore tries to configure jaxws with the -encoding param, which emits the warning message. Even if encoding is specified via args param.
What could happen here as well is that a different encoding is specified via args than project.build.sourceEncoding specifies. For JDK 1.8+ encoding config is applied twice; not sure which of them will be used. Not good, even if the likelihood of having different encoding via args than project.build.sourceEncoding is low (shouldn't really be the case as you will have mixed encoding in source files).

I've checked the code and the reported issue should only occur if an executable is specified, either directly via the executable param or by specifying JDK toolchain. Could @mheitnrw please confirm that this is the case?
To solve this we need to decide how to handle arguments that may not be supported by a JDK (or external wsimport/wsgen executable). There is no generic way to ask for, for example, JDK or JAX-WS version to determine if argument is supported. And then for encoding we also need to fix the possible issue of encoding being specified twice (see my comment above).

I confirm that I use a ~/.m2/toolchains.xml with one entry for a Oracle 1.8 JDK.

As we have now up'ed the requirement to Java 8 I think it would be ok to remove this warning.
While now re-reading this full thread I see that i brought up a separate issue where the encoding arg could be applied twice, which is a separate issue and not what this report initially is about. I therefore think we should remove the warning as Java 8 is now required and the issue about encoding being applied twice be moved outside of this ticket.

The implemented change should now allow the configured "encoding" to be used even when a different executable is configured, without the need to use "args". "args" should not even be used as it would add two (2) -encoding options to the jax-ws tool.


Yolov3 training loss

On my custom dataset, the training loss isn't converging (hovering around 20-30)...the mAP is also behaving weirdly..after 1 epoch its 0.5, then 0.3, 0.2, 0.1...
I tried same dataset with faster_rcnn config it's working as expected..
Any runtime/scheduler/train_pipeline that I must use differently for Yolov3 ??

Training has stabilised after some epochs..achieved recall of 0.98<EMAIL_ADDRESS>of 0.8 at 100 epochs

, then 0.3, 0.2, 0

Did you manage to figure out what was wrong? I am having a similar issue. My training losses are quite large (> 500) and my mAP are all 0.

Hey @David-Biggs I'm having the same issue i coudn't see why it's happening, have u found the solution for this problem ?


Add steps for extension api-server setup

Update extension api-server to include missing steps for auth delegation cluster role binding and extension-apiserver-authentication-reader role binding.

This change is

@font Can you suggest someone to do a tech review/approval?


Update cli test command in README.md

What type of PR is this?
/kind bug

Any specific area of the project related to this PR?
/area tests

What this PR does / why we need it:
This PR is to update the README.md  to change the cypress test command as the current one does not start the test.
Which issue(s) this PR fixes:

Fixes #22

Welcome @Dhyuti! It looks like this is your first PR to falcosecurity/falco-playground 🎉

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: Dhyuti
Once this PR has been reviewed and has the lgtm label, please assign leogr for approval. For more information see the Kubernetes Code Review Process.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

/cc @Rohith-Raju

@Dhyuti Thank you for looking into this!!

LGTM label has been added.  Git tree hash: 19e72a65597d9740dde073cb60ec9e8b7e715cf5

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: Dhyuti, Rohith-Raju
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [Rohith-Raju]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


add interpolation_search

add interpolation_search

Hey @onowdev, these issues were already assigned to someone else. However, we have added more in Linked List and Binary Tree. Please see if you can contribute here. Ask for the assignment first. Thankyou :)


Added Script for Scripts.json

Description
Added script_updater.py for generating Scripts.json
Fixes #811
Have you read the Contributing Guidelines on Pull Requests?

Yes

Code is fine, move this script to the readme root.

Done.

@santushtisharma10 @antrikshmisri This script will not have any readme file nor any folder. Also, this will be present in the root directory so that making the GitHub Action is easier

@Sukriti-sood Please make a folder instead .py file and make sure you add readme.md file .

@Kushal997-das actually this is not a script project but an automation to add scripts information to json file which can be accessed in website. That's why I instructed not to include any folder and readme for this script. This file will be present in the root directory of the repo

@antrikshmisri

@Sukriti-sood Please make a folder instead .py file and make sure you add readme.md file .

@Kushal997-das actually this is not a script project but an automation to add scripts information to json file which can be accessed in website. That's why I instructed not to include any folder and readme for this script. This file will be present in the root directory of the repo

Okey @kaustubhgupta


Convert types.json into a swift file

Why do we still get a warning message about "Loading embedded MIME types"?

@shmuelk Ok, now you'll only see it on Linux. This is because we can't use NSJSONSerialization on Linux yet.

So I investigated getting this to work in Linux, it fails because the downcast to NSDictionary fails. I messed around with it for a while but couldn't figure out the right downcast. For reference it is an Any object which looks like this printed:
["application/vnd.ezpix-album": ["ez2"], "application/vnd.noblenet-web": ["nnw"], "application/vnd.crick.clicker.wordbank": ["clkw"], "application/vnd.hp-pclxl": ["pclxl"],  ...

I think we should leave this alone for now and wait till Foundation is more stable on Linux.


No Scheme for iOS

Tried to build a iOS RestKit framework for an app using Carthage. My app is suppose to be using the VisualRecognitionV3 framework which needs the RestKit Framework. Whenever I try to build the framework, however, this happens:

Any thoughts on how I can work around this???

No one has added Carthage support yet


Passing stdin with exec

Feature Description
I'd like to pass stdin to an external process.
Describe The Solution You'd Like

Perhaps exec --stdin ${something} some_command ?

@mightyiam this is now released. you can use the new --input arg (see sdk docs)

Thank you, @sagiegurari .


build: packageName should be set for release configuration

we were not updating samples/package.json, because we didn't know the correct packageName.

landed this landed this setting still


Add Service-specific schemas for Azure Container Instances

The page says

Feel free to open a GitHub issue on this article so we can update it.

I don't see a link to service-specific schema for Azure Container Instances. Would you mind adding it to the list?

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

ID: daf3e121-6ca8-4e35-ff35-1b6151d26a16
Version Independent ID: 3efd2282-db7f-02e3-9a09-08187c3cfa3a
Content: Azure Resource Logs supported services and schemas - Azure Monitor
Content Source: articles/azure-monitor/platform/resource-logs-schema.md
Service: azure-monitor
Sub-service: logs
GitHub Login: @rboucher
Microsoft Alias: robb

@magnus-longva-bouvet -
Thanks for brining this to our attention. We will investigate it further and update you shortly.

@magnus-longva-bouvet -
Thanks for brining this to our attention. We will investigate it further and update you shortly.

@magnus-longva-bouvet - I have reached out to product team and waiting on theri response. I will keep you posted as soon as I hear back from them.

@magnus-longva-bouvet - I have reached out to product team and waiting on theri response. I will keep you posted as soon as I hear back from them.

@magnus-longva-bouvet -

Thanks for bringing this to our attention.
I'm going to assign this to the document author so they can update the document accordingly.

@magnus-longva-bouvet -

Thanks for bringing this to our attention.
I'm going to assign this to the document author so they can update the document accordingly.

This just merged recently. Published now.
#please-close


No module named 'polygraphy'

I am getting this error when starting ComfyUI when using your nodes:
  File "S:\StableDiffusion\UI\ComfyUI_stable\ComfyUI\nodes.py", line 1800, in load_custom_node
    module_spec.loader.exec_module(module)
  File "<frozen importlib._bootstrap_external>", line 940, in exec_module
  File "<frozen importlib._bootstrap>", line 241, in _call_with_frames_removed
  File "S:\StableDiffusion\UI\ComfyUI_stable\ComfyUI\custom_nodes\comfy-trt-test\__init__.py", line 13, in <module>
    from .comfy_trt.node_unet import TRT_Unet_Loader
  File "S:\StableDiffusion\UI\ComfyUI_stable\ComfyUI\custom_nodes\comfy-trt-test\comfy_trt\node_unet.py", line 24, in <module>
    from .utilities import Engine
  File "S:\StableDiffusion\UI\ComfyUI_stable\ComfyUI\custom_nodes\comfy-trt-test\comfy_trt\utilities.py", line 26, in <module>
    from polygraphy.backend.common import bytes_from_path
ModuleNotFoundError: No module named 'polygraphy'

Cannot import S:\StableDiffusion\UI\ComfyUI_stable\ComfyUI\custom_nodes\comfy-trt-test module for custom nodes: No module named 'polygraphy'

I have the module installed in my venv:
Version: 0.49.0
Summary: Polygraphy: A Deep Learning Inference Prototyping and Debugging Toolkit
Home-page: https://github.com/NVIDIA/TensorRT/tree/main/tools/Polygraphy
Author: NVIDIA
Author-email<EMAIL_ADDRESS>License: Apache 2.0
Location: c:\users\<user>\appdata\local\packages\pythonsoftwarefoundation.python.3.10_qbz5n2kfra8p0\localcache\local-packages\python310\site-packages
Requires:
Required-by:

the installation path is not a venv, it's your python installation directory

My comfy-trt-test directory, right? that's where I'm running the pip install command.

no 👽 that's not venv
anyway:
u having at least 2 different python on your pc: the 1 at your user space and the 1 embedded with comfyui portable
u installed polygraphy with python at user space, that's wrong
u need to install with python embedded with comfyui portable

Sorry, I'm only a beginner with Python. How can I force it to do a pip install to the embedded python rather than the global one?

Nevermind, I got it to work :) Switched to WSL and loaded up a fresh ComfyUI instance in that, and now I see the nodes.

ok no prob


Upgrade Fuse directly after update

We were doing this awkward thing, we're if we detected a Fuse upgrade, we uninstalled KBFS and then allowed the Installer to upgrade Fuse after the restart.
I patched the installer so it can take an --install-fuse flag, so that we can upgrade Fuse after an update in a more sane way.

retest this please


4 digit display models?

Hi, I purchased this unit:
https://www.ebay.ca/itm/New-LED-DIY-Digital-Electronic-Micro-Controller-Kit-Clock-Time-Thermometer-CA-/292048065603?var=&hash=item899c6ccd6d
The major difference that I can see would be that this unit uses a single 4 digit seven segment display. I'm not sure if this firmware would have support for that.
Schematic:
clock.pdf
Will this work with the open source firmware?  It would be fantastic to stop this thing changing the display, any maybe turn down the LED intensity...

I have the same model. I made a working fork for that model. Check here: https://github.com/vargham/stc_diyclock

Thanks for contributing @vargham! Welcome and PRs back upstream...


Multi-qubit problem sets

Changes
Added multi-qubit problem sets

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.3 out of 4 committers have signed the CLA.:white_check_mark: pandasa123:white_check_mark: eddybrando:white_check_mark: vabarbosa:x: amirhassankYou have signed the CLA already but the status is still pending? Let us recheck it.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.3 out of 4 committers have signed the CLA.:white_check_mark: pandasa123:white_check_mark: eddybrando:white_check_mark: vabarbosa:x: amirhassankYou have signed the CLA already but the status is still pending? Let us recheck it.

@amirhassank are you able to sign the CLA? also are you able to address the comments above? thanks. let me know if you have any questions.


adding a product to cart from the main page causes an error in the store/cart file in the addProductThunk

fixed on my "userformandroutefixes" branch

I can't reproduce the issue

I still get it on the deployed branch when logged in as a user. Can only add from single product component.

Resolved, the heroku database reset didn't take


Add provenance attestation when publishing to NPM

Changelog
None
Description
This adds a provenance attestation to the published package so consumers can verify that the package was built on GitHub Actions:

https://github.blog/2023-04-19-introducing-npm-package-provenance/
https://docs.npmjs.com/generating-provenance-statements#publishing-packages-with-provenance-via-github-actions

The package will appear like this on npm:

Same comment as other PR: I am keeping the comment since I believe yarn pack && npm publish package.tgz looks unusual