ExpandoObject support

Can we have support for ExpandoObject's as a top level and as child objects? In essence, an ExpandoObject is a IDictionary<string,object>. This is useful because ExpandoObject's are good for representing arbitrary JSON.

@GregFinzer , Please see the included unit tests. All tests fail. One is a direct test of an ExpandoObject, one is a test of a poco class with an ExpandoObject property, and one is a JSON deserialization. Perhaps I am missing a config object or something?
        private sealed class ExpandoObjectChild
        {
            public ExpandoObject Expando { get; set; }
        }

        [Fact]
        public void CompareLogicShouldHandleDeserializedJsonExpandoObjects()
        {
            var x = JsonConvert.DeserializeObject<ExpandoObject>("{ \"A\" : 1 }");
            var y = JsonConvert.DeserializeObject<ExpandoObject>("{ \"B\" : 1 }");

            var compareLogic = new CompareLogic();
            var result = compareLogic.Compare(x, y);
            Assert.False(result.AreEqual);
        }

        [Fact]
        public void CompareLogicShouldHandleExpandoChildObjects()
        {
            dynamic childX = new ExpandoObject();
            childX.A = 1;
            dynamic childY = new ExpandoObject();
            childY.B = 2;

            var x = new ExpandoObjectChild
            {
                Expando = childX as ExpandoObject,
            };
            var y = new ExpandoObjectChild
            {
                Expando = childY as ExpandoObject,
            };

            var compareLogic = new CompareLogic();
            var result = compareLogic.Compare(x, y);
            Assert.False(result.AreEqual);
        }

        [Fact]
        public void CompareLogicShouldHandleExpandoObjects()
        {
            dynamic x = new ExpandoObject();
            x.A = 1;
            dynamic y = new ExpandoObject();
            x.B = 2;

            var compareLogic = new CompareLogic();
            var result = compareLogic.Compare(x as ExpandoObject, y as ExpandoObject);
            Assert.False(result.AreEqual);
        }

@johnweeder I am not sure what is wrong.  Re-opening issue.  It looks like the one test that I had was commented out.  See CompareDynamicTests

I have added the tests for the Dynamic Objects and they are passing on the build server now.
https://ci.appveyor.com/project/GregFinzer/compare-net-objects

@GregFinzer , I'm in meetings today and tomorrow but I will validate early next week. These tests were run on .NET Core using on VS 17.

@johnweeder That is the missing piece of information.  The Expando tests are running under the normal .NET Framework, not .NET Core.  This would be a new feature for .NET Core.  Re-opening and marking as an enhancement.  Feel free to implement and I will be happy to merge in the pull request.

@johnweeder John can you download the latest version and test please?  There is a new build for .NET Standard.


[Question] Are all types manually written?

Hey I recently stumbled upon your project and was positively surprised how thoroughly everything was annotated.
I am currently in the process of writing basically the same and wanted to ask if these were all manually written or if you scanned the moodle code (or documentation page)?

Currently, most of the type / anotation were manually copied from moodle webservice documentation with a lot of help from vscode.
I was trying to automate most of type generation but failed miserably, caused by inconsistency in moodle webservice documentation.

Thanks for the quick response. I think I will still try to automate this using the PHP source directly. If I get it working I can let you know if you want to


AFM0002TIM

Hi guys,
i've the SFP on object (that is usitng RTL9601B) running firmware V1_7_8_181123.
Do you think is possible to cross-flash a more recent firmware?
I've also opened the stick, looks like there are two point marked R/T that should be UART may be, but i'm missing the power pads, should I use the SFP connector?
Thanks

some stick have disabled UART, or missing resistor, but you can read memory directly


Time slots

In creating time slots, if I want something to start at 8 am, I have to enter 7 am.  It seems to be the same for all times.
Also, is it possible to have different schedules for a 2-day event?  It appears that's not the case.

@hismightiness see notes on #120 for local time offset issue.
@jncraig I am not sure about the ability to have different schedules for a 2-day event.

Time slots have nothing to do with the days of the event, just the time slots available in each day.  You define the number of days by defining the begin and end dates in the event information on the event homepage.

What if I want different schedules one the two days?  That is, the time
slots for Friday are different than on Saturday.
On Wednesday, February 24, 2016, Will Strohl<EMAIL_ADDRESS>wrote:

Time slots have nothing to do with the days of the event, just the time
slots available in each day. You define the number of days by defining the
begin and end dates in the event information on the event homepage.
—
Reply to this email directly or view it on GitHub
https://github.com/hismightiness/dnnextensions/issues/121#issuecomment-188534447
.

Unable to reproduce or get further clarification.


Scale UI better

@jueyang - how could this scale better? This is the site in German right now.
Care for a quick fix?
http://cl.ly/image/3h3B3y0b2y2M

Closing as stale. Looks from the comments as if it was resolved long ago.


Add a module for mouse/touch on top of TouchPassr

Two use cases I can think of...

Mapping point contacts to MenuGraphr::registerA on menus
Point-to-walk: touch a point on the screen, and the player will walk to it

Hover events could show the predicted path.

This might require UserWrapper to have a registration function for extra modules such as this one. Perhaps it should default to using DeviceLayr?

Ooh, let's call this MenuTouchr!

Hmm, on second thought I think this is unnecessary. The original game didn't have this.


Re-introduction of GeneratePseudoFromClause for specific "empty" FROM clauses

For #18926.
It would really really really help [1] if this would be considered for merge 🙏 and before 3.1 ships 🤞. I don't see any breaking changes here, because in 3.0 this was non-existing and hence nobody could override, hence nothing is/was/will be broken.
[1]: It would help because copying VisitSelect and duplicating all the code is impractical because of some private methods usage and also makes no sense when simple plug in point can handle it.

This will be done in master as part of #18923

I see that for 5.0. No chance for this particular one for 3.1?

@cincuranet 3.1 is already finalized, so the change cannot go there. Also, we are not allowed to make API changes in patch releases, except for very exceptional circumstances--i.e. security. So, unfortunately this means that the next release this change can be in is 5.0.

OK. I will work around it on my side then.

@cincuranet Thanks!


bug: BaseDgsQueryExecutor silently handles exceptions from instrumentations

The BaseDgsQueryExecutor silently handles runtime exceptions. I've narrowed it down to the following else block:
https://github.com/Netflix/dgs-framework/blob/8fb490c5b4f0aa012c998dd2855abc2301af6ee4/graphql-dgs/src/main/kotlin/com/netflix/graphql/dgs/internal/BaseDgsQueryExecutor.kt#L141-L151
This comes up when you start implementing your own instrumentations and make any sort of mistakes.
Expected behavior
Exception details would be printed in the logs (preferably on the error level) in addition to current handling
Actual behavior
Exception is silently handled (transformed into a 500 error)
Steps to reproduce
Create an instrumentation that throws an exception, similar to:
public class ExceptionProducingInstrumentation extends SimplePerformantInstrumentation {

    public DataFetcher<?> instrumentDataFetcher(DataFetcher<?> dataFetcher,
                                                InstrumentationFieldFetchParameters parameters,
                                                InstrumentationState state) {
        if (true)
            throw new NullPointerException("I made an oopsie");
        return super.instrumentDataFetcher(dataFetcher, parameters, state);
    }
}

Hi - actually you need to wrap the throwing of the exception in a DataFetcher and return that. This will allow the default exception handler to trigger and do the needful. You can also add your own custom exception handler if you want more control over the result.
public @NotNull DataFetcher<?> instrumentDataFetcher(DataFetcher<?> dataFetcher, InstrumentationFieldFetchParameters parameters, InstrumentationState state) {
        if (true) {
            return DataFetcher -> {
                throw new NullPointerException("I made an oopsie");
            };

        }
        return super.instrumentDataFetcher(dataFetcher, parameters, state);
    }

Will result in the following:
  "errors": [
    {
      "message": "java.lang.NullPointerException: I made an oopsie",
      "locations": [
        {
          "line": 3,
          "column": 1
        }
      ],
      "path": [
        "hello"
      ],
      "extensions": {
        "errorType": "INTERNAL"
      }
    }
  ],
  "data": {
    "hello": null
  }
}

That's a good tip, thanks. Still, I would argue the framework shouldn't swallow exceptions like that.

That's a good tip, thanks. Still, I would argue the framework shouldn't swallow exceptions like that.

I see your point. However, we are leveraging graphql-java's execution here, so in order to get all the properly formatted response with errors as part of the result, we need to go through the exception handling mechanism. You could argue that we can format the same where we catch the exception, but we have lost the information at that point, we just have the message, which I suppose we can add to the error that we construct.

I'm not suggesting changing how exceptions are handled, only asking that it is properly logged in this case (ie to add logger.error()) - that would make debugging easier :)

I see, yes, we can very well do that.


mac: try to cancel accent menu

Untested, first hack

Hey, did you mean to cancel this?

Yes/no. I meant to submit a PR but I undid a nonworking fix. Pushing an
empty PR closes it on Github automatically.
On Wed, Nov 9, 2016 at 12:41 PM Fred Barclay<EMAIL_ADDRESS>wrote:

Hey, did you mean to cancel this?
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/Fred-Barclay/Termination/pull/10#issuecomment-259520676,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AANsMEdFFy7lYdCb4P8k50ZLD8qCkJudks5q8i_8gaJpZM4Kt9fZ
.


Get rid of commons collections 3.0

This gets rid of an obscure and long abandoned library to achieve the main goal of removing the dep to commons-collections.

Nice :)


Use original-fs when running on Electron to avoid issues with .asar files (fs module is patched by Electron)

@tejohnso please review

:+1:


{all services} URL hotfix

The purpose of this PR is to replace docs.microsoft.com with learn.microsoft.com and remove /en-us/ from URLs.
This is a "first pass" to see what build errors we get.

all services

@yonzhan , please provide guideance on these build errors.  Can I exclude test folders, for example?
Here are two of the errors:
Verify extensions index error
Ran 9 tests in 89.919s
FAILED (failures=1)
/tmp/tmp1yluavhk/storage_mover-0.1.1-py3-none-any.whl
/tmp/tmp1yluavhk/storage_preview-0.9.0-py2.py3-none-any.whl
/tmp/tmp1yluavhk/storagesync-0.1.2-py2.py3-none-any.whl
/tmp/tmp1yluavhk/stream_analytics-0.1.2-py3-none-any.whl
/tmp/tmp1yluavhk/subscription-0.1.5-py2.py3-none-any.whl
/tmp/tmp1yluavhk/support-1.0.3-py2.py3-none-any.whl
...
py3.8 - run integration test and build test
../azure-cli/src/azure-cli-testsdk/azure/cli/testsdk/base.py:306: AssertionError

generated xml file: /home/cloudtest/.azdev/env_config/mnt/vss/_work/1/s/env/test_results.xml -
=========================== short test summary info ============================
FAILED src/dms-preview/azext_dms/tests/latest/test_service_scenarios.py::DmsServiceTests::test_project_commands
FAILED src/dms-preview/azext_dms/tests/latest/test_service_scenarios.py::DmsServiceTests::test_task_commands
============================== 2 failed in 2.04s ===============================
Traceback (most recent call last):
...

Closing PR as this fix needs to originate with engineering as there are pipeline build errors I do not control.  The update of embedded URLs remains an open task.


component: Video Player

add youtube lite embed reference
add loom embed
add video embed
add cloudinary video

we need it man


me.kittichdevs.com

Sorry, your PR is for a wrong branch.
However I already merged #155, so your website is already on the webring! Therefore, this PR can be closed. Cheers!


⚠ Bofrost.be has degraded performance

In 9f58480, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 1367 ms

Resolved: Bofrost.be performance has improved in 3bd72e3 after 7 minutes.


Recurring batch import from data portal

See the other ticket
https://github.com/theodi/open-data-certificate/issues/1036


"Delete the message before it's delivered: Silently deletes the entire message, including all attachments." mechanic

Does this place it in the user's deleted items or is it DELETED aka not delivered aka blown away, no retrievable. This matters because I had a question on the MS-101 referencing these exact mechanics and there were 2 answers. One stated it was DELETED and NOT delivered, the other that it was placed in the mailbox deleted folder. You can't expect people to use your own material to study and then have it so open-ended. I wasted 10-15 minutes mulling that question over because stuff like this.

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

ID: f0b8f651-9d8e-a34b-8dde-2b6a9448f875
Version Independent ID: 20e96426-627d-9973-8303-a43a114c7b61
Content: Anti-phishing policies - Office 365
Content Source: microsoft-365/security/office-365-security/set-up-anti-phishing-policies.md
Product: m365-security
Technology: mdo
GitHub Login: @chrisda
Microsoft Alias: chrisda

@dreeves14 Hope this comment is helpful for you. If you see a documentation update is required, please feel free to open an issue for the same. We proceed here to close it. Thanks for taking out some time to open the issue. Appreciate and encourage you to do the same in future also.


Unable to access to Ward after initializing the port

Hello,
I followed the instructions for a docker installation, and I don't have any error in logs, as you can see here :

2020-12-09 15:46:18.180  INFO 1 --- [       Thread-3] o.s.boot.SpringApplication               : Starting application using Java 1.8.0_212 on 61090399b3e0 with PID 1 (/ward.jar started by root in /)
2020-12-09 15:46:18.181  INFO 1 --- [       Thread-3] o.s.boot.SpringApplication               : No active profile set, falling back to default profiles: default
2020-12-09 15:46:19.101  INFO 1 --- [       Thread-3] o.s.b.w.embedded.tomcat.TomcatWebServer  : Tomcat initialized with port(s): 4001 (http)
2020-12-09 15:46:19.103  INFO 1 --- [       Thread-3] o.apache.catalina.core.StandardService   : Starting service [Tomcat]
2020-12-09 15:46:19.104  INFO 1 --- [       Thread-3] org.apache.catalina.core.StandardEngine  : Starting Servlet engine: [Apache Tomcat/9.0.39]
2020-12-09 15:46:19.112  INFO 1 --- [       Thread-3] o.a.c.c.C.[Tomcat].[localhost].[/]       : Initializing Spring embedded WebApplicationContext
2020-12-09 15:46:19.112  INFO 1 --- [       Thread-3] w.s.c.ServletWebServerApplicationContext : Root WebApplicationContext: initialization completed in 914 ms
2020-12-09 15:46:19.273  INFO 1 --- [       Thread-3] o.s.s.concurrent.ThreadPoolTaskExecutor  : Initializing ExecutorService 'applicationTaskExecutor'
2020-12-09 15:46:19.313  INFO 1 --- [       Thread-3] o.s.b.a.w.s.WelcomePageHandlerMapping    : Adding welcome page template: index
2020-12-09 15:46:19.408  INFO 1 --- [       Thread-3] o.s.b.w.embedded.tomcat.TomcatWebServer  : Tomcat started on port(s): 4001 (http) with context path ''
2020-12-09 15:46:19.416  INFO 1 --- [       Thread-3] o.s.boot.SpringApplication               : Started application in 1.364 seconds (JVM running for 8.731)

Moreover, I tried with the 4001 port because when I'm typing 4000 it says me "Use another port".
Do you have an idea why ?

I have the same problem

4000 is the port for the setup page
try docker run --name ward -p 8080:4000 -p 8000:8000 ward
and use port 8000 as application port

I have revived this project and removed the need for an application port.
see
https://github.com/AntonyLeons/Ward


Reception of advertising in the "Long Range" mode (Coded PHY S=8) does not work.

The BT5.2 adapter supports "Extended Advertising" reception.
Reception of advertising in the "Long Range" mode (Coded PHY S=8) does not work in BTHome-ble.
The test version of the "Long Range" thermometer is successfully received in nRF Connect:

And connects successfully:

I'm afraid I don't have an BT5.2 adapter to test. Is the Bluetooth integration in HA able to receive this data when you enable debug logging?

Home Assistant does not have a Coded PHY receive option.
These are not problems with BT adapters.

On any BT5.2-USB adapter:

bluetoothctl --version
bluetoothctl: 5.66

I'm not sure I completely understand the issue. You say:

Reception of advertising in the "Long Range" mode (Coded PHY S=8) does not work in BTHome-ble.

So, if you make the thermometer send data in "Long Range" mode (Coded PHY S=8)", it isn't received by the BTHome integration in Home Assistant.?
So, first thing to figure out, is the Bluetooth integration in HA receiving this data? As I understand from your reaction, it isn't.

Home Assistant does not have a Coded PHY receive option.

So, I think the issue is with the Bluetooth integration then, isn't it? Perhaps @bdraco can help with this?

The integration does not have the ability to switch the adapter to work with CodedPHY.
Switch the adapter to work with CodedPHY:
hcitool -i hci0 cmd 08 31 03 04 04

Launch Home Assistant and it will only receive in LongRange mode.
Find out the possibility of BT adapter
Information about adapter capabilities:
# btmgmt phy
Supported phys: BR1M1SLOT BR1M3SLOT BR1M5SLOT EDR2M1SLOT EDR2M3SLOT EDR2M5SLOT EDR3M1SLOT EDR3M3SLOT EDR3M5SLOT LE1MTX LE1MRX LE2MTX LE2MRX LECODEDTX LECODEDRX
Configurable phys: BR1M3SLOT BR1M5SLOT EDR2M1SLOT EDR2M3SLOT EDR2M5SLOT EDR3M1SLOT EDR3M3SLOT EDR3M5SLOT LE2MTX LE2MRX LECODEDTX LECODEDRX
Selected phys: BR1M1SLOT BR1M3SLOT BR1M5SLOT EDR2M1SLOT EDR2M3SLOT EDR2M5SLOT EDR3M1SLOT EDR3M3SLOT EDR3M5SLOT LECODEDTX LECODEDRX

# hciconfig -a
hci0:   Type: Primary  Bus: USB
        BD Address: 8C:88:2B:20:8B:42  ACL MTU: 1021:6  SCO MTU: 255:12
        UP RUNNING
        RX bytes:2020 acl:0 sco:0 events:178 errors:0
        TX bytes:23965 acl:0 sco:0 commands:178 errors:0
        Features: 0xff 0xff 0xff 0xfe 0xdb 0xfd 0x7b 0x87
        Packet type: DM1 DM3 DM5 DH1 DH3 DH5 HV1 HV2 HV3
        Link policy: RSWITCH HOLD SNIFF PARK
        Link mode: SLAVE ACCEPT
        Name: 'nanopineoplus2'
        Class: 0x000000
        Service Classes: Unspecified
        Device Class: Miscellaneous,
        HCI Version: 5.1 (0xa)  Revision: 0x999
        LMP Version: 5.1 (0xa)  Subversion: 0x646b
        Manufacturer: Realtek Semiconductor Corporation (93)

I'm afraid I don't have an BT5.2 adapter to test.

RTL8761B

USB ID 0bda:8771 Realtek Semiconductor Corp. Bluetooth Radio
HCI Version: 5.1 (0xa)
LMP Version: 5.1 (0xa)

There is no way to change the PHY type of the adapter during HA runtime. It is required to reboot the entire system, with switching during stop HA.

I don't blame @bdraco, he has been more than helpful for me with the development of BTHome. But all code for receiving Bluetooth data is  being carried out by the Bluetooth integration. The BTHome integration is only using the parsed data from the Bluetooth integration to further parse it into entities. But all the data receiving logic is implemented in the Bluetooth integration in HA.
So, to my opinion, you should create an issue in the Home Assistant core github, and label it with the Bluetooth integration. In that way, @bdraco can have a look if it is possible to implement this. It is out of my capabilities to do this, unfortunately.
PS2. I'll create a separate issue for the Periodic Advertisements. We had the same issue for some of the Xiaomi sensors, like door sensors. We can implement the same fix for BTHome.

Home Assistant Bluetooth support is entirely based on bleak so we only support what bleak supports.
Bleak not supports reading, writing and getting notifications from GATT servers, as well as a function for discovering BLE devices Bluetooth Core Specification Version 5.0 and next version?
Работа термометров в Home Assistant c Bluetooth LE Long Range.

Home Assistant does support reading characteristics. E.g. the xiaomi ble integration supports reading the battery from the plant sensor, which is only possible by reading characteristics.
https://github.com/home-assistant/core/blob/b407227d4a735d5337f3cf7a8d04a13fd82b8def/homeassistant/components/xiaomi_ble/__init__.py#L102
Anyway, @bdraco has made changes and improvement to bleak as well, when he developed the Bluetooth integration. So, if one is able to look into your request, we should ask him.

About the pointing to each other, that is not what I want. I just try to say that your request is way out of my capabilities. So, unfortunately I can’t help with this, just because of my lack of experience.

BTHome->Bluetooth->Bleak->dbus->Bluez->kernel->adapter and in a circle. Everyone points to each other.
The solution is simple - switch the adapter to work in Bluetooth 5.0.
BTHome->Bluez->kernel->adapter

I understand, but that is still way out of my capabilities. And I think the HA devs probably like a solution that works for every Bluetooth integration. Again, you should ask @bdraco, he can help thinking about a possible solution. I’ll ask him on discord if he is willing to have a look here.

To date, switching is only relevant for BTHome for DIY sensors.

😄
Ok, I understand, only BTHome for now. But again, Its out of my capabilities. Its not that im not willing, but I know my limitation with regards to programming. And this is too difficult for me unfortunately.

@pvvx please, correct me if I am wrong, but isn't the DBUS interface from bluez the missing link at the moment so that bleak, and eventually HA bluetooth integration and BTHome to support this feature?
I'm not too familiar with Bluez, but it seems to me that the management API is there, but not exposed to the DBUS Adapter API in any way.

Начало завязки лежит в kernel. Там свои тараканы у подписывающих патчи и принимавших решения по развитию. И у них лицензия GNU - т.е. с авторскими правами. Т.е. что хотят, то и творят - могут принять ваши коды и поправки под своё авторство, а могут не не принять. Пока, с 2016 года, принимали только отрицательные решения по BLE от кентов из Intel, чтобы развития BLE в Linux не происходило. В итоге Google нарвалось и не может это сдвинуть уже несколько лет.

Web Bluetooth API: Implementation Status :)


Collapse "Others" in the right sidebar by default

By default, the "Others" section in the right sidebar should be collapsed. If the user expands it, that preference should be preserved as they switch views, similarly to whether Views and Direct messages are expanded/collapsed in the left sidebar.

CZO thread

@zulipbot claim


Material3 AlertDialog doesn't recompose immediately on desktop

Compose Multiplatform: 1.6.1
https://github.com/JetBrains/compose-multiplatform/assets/14316223/c8f58d11-3139-425e-9b3b-37a7aabdc744

Can you post a short reproducer?

Hi @m-sasha, please take a look at https://github.com/Omico/issue-compose-multiplatform-4558.

Can you post a short reproducer?

Can you post a short reproducer?

See https://github.com/Omico/issue-compose-multiplatform-4558/blob/ce10cc732b75c7757bc9c1088db72663e8d84161/gpi/shared/src/commonMain/kotlin/me/omico/gpi/shared/App.kt

Reproducer:
val TestLocal = staticCompositionLocalOf { 0 }

fun main() = singleWindowApplication {
    var value by remember { mutableStateOf(1) }
    CompositionLocalProvider(TestLocal provides value) {
        println("Outside Value: ${TestLocal.current}")
        Popup(
            onDismissRequest = { },
        ) {
            println("Inside Value: ${TestLocal.current}")
            Row(verticalAlignment = Alignment.CenterVertically) {
                Button(
                    onClick = {
                        value = 2
                    },
                    content = { Text("2") },
                )
                Spacer(modifier = Modifier.width(8.dp))
                Button(
                    onClick = {
                        value = 1
                    },
                    content = { Text("1") },
                )

                Text(TestLocal.current.toString(), Modifier.padding(8.dp))
            }
        }
    }
}

The problem seems to be with a change introduced in https://github.com/JetBrains/compose-multiplatform-core/pull/1086
In rememberComposeSceneLayer we update the layer's compositionLocalContext in a SideEffect, but this is too late because composition happens before SideEffect is executed.
Also, I don't entirely understand why a layer needs compositionLocalContext. Aren't the locals already visible through the parent composition (passed via compositionContext in scene.createLayer)? In Android, a popup only passes the parent composition when creating a popup. And if I remove this passing of currentCompositionLocalContext, the bug goes away, and composition locals are still correctly passed into Popup.
@igordmn @MatkovIvan

I don't entirely understand why a layer needs compositionLocalContext

There is no public (outside of runtime) way to extract it from compositionContext, I wrote about it in mentioned PR

Aren't the locals already visible through the parent composition (passed via compositionContext in scene.createLayer)?

Only if it IS parent composition (MultiLayerComposeScene) - it's not always a case for multiplatform, and cannot be for a general case (due to requirement of different coroutine context for different windows)

And if I remove this passing of currentCompositionLocalContext, the bug goes away, and composition locals still seem to be correctly passed into Popup.

Same as above - it will work ONLY for drawing Popup inslde same canvas (MultiLayerComposeScene)

Why does composition depend on where the composables are drawn (canvas)?

Because in case of single canvas it reuses already existing environment/contexts, but in other cases it cannot do that

In Android, Popup creates a new (Android) View and in it a new Composition with the parent composition set to rememberCompositionContext(). See ComposeView.ensureCompositionCreated().
We, on the other hand, don't pass the parent composition; the new Composition is created with (a new) Recomposer as its parent.  RootNodeOwner.setContent is called with parent set to BaseComposeScene.compositionContext, which is a newly created Recomposer.
That seems to be why the composition locals aren't propagated automatically. I imagine it has other implications, and possibly bugs, too.

I'd recommend

Short term: set the layer properties (layoutDirection, density and compositionLocalContext) immediately when rememberComposeSceneLayer is composed.
Long term: correctly set up the composition hierarchy, and get rid of compositionLocalContext.

Long term: correctly set up the composition hierarchy, and get rid of compositionLocalContext.

We can't do that, because popup/dialog can have its recomposition cycle. For example, in a case when they are separate windows showed on another screen with a different refresh rate.
The short term solution seems the valid solution. Even if the popup content is recomposed not in sync, it should see the most recent composition locals.

We can't do that, because popup/dialog can have its recomposition cycle. For example, in a case when they are separate windows showed on another screen with a different refresh rate.

I'm not convinced this necessarily means that we can't have a correct composition hierarchy. Why isn't it possible for each ComposeScene to have its own Recomposer, and the Composition have its parent set to the parent CompositionContext? It may require a few changes in compose.runtime, but I'm not even sure that it will.

It may be possible in theory if we allow somehow compositions to be recomposed at different pace. How we do that - having one or multiple Recomposer's - is implementation details. But that can require a lot of work.

Please check the following ticket on YouTrack for follow-ups to this issue. GitHub issues will be closed in the coming weeks.


Weaken assertion in validate

Backport the last fix commit from https://github.com/salsa-rs/salsa/pull/550
It's now possible for mark_validated_output to be called on a struct that we already validated via maybe_changed_after on one of its fields, so validate can no longer assert that the struct is out of date.
This time I actually checked incremental run on tomllib locally with this change to ensure it works.

Going to go ahead and merge this to unblock getting red-knot fixed on real code; review comments should probably go to the Salsa PR anyway.


WIP: Add support for ubuntu 22.04 and centos-8-stream

Looks good to me.

The latest changes have been discussed with Kalle as well.


Small feature request.

Hi, I really enjoy this app. Even bought it on windows store to support you! However I would like to ask for a small feature.
Maybe you could add an option to hide FalconX icon from system tray? I would really appreciate! Thank you.

But you need the icon to access the settings..

But you need the icon to access the settings..

Not really, you can have shortcut set wherever you want, for example on start menu. It's good to have options.

This may work for the normal version but, not for the store version as i can't access anything due to permissions with store apps. So i can't have 1 service and 1 gui because, you can only access the main program not multiple. It's hard to explain but, have tested it before and sadly won't let me do it like this in the store.

And i decided to do it anyway :) in update <IP_ADDRESS>.
I found out how to add multiple applications into a single app package.


Shutdown using HelidonContainer not successful

HelidonContainer.instance().shutdown(); may not shutdown the container as expected (reported through Slack).
Customer is using streams - this should shutdown Helidon when the channel terminates
msgRuntime.onChannelTerminated(c -> {
    HelidonContainer.instance().shutdown();
});

Unable to reproduce. No response from customer as well.


Document how to run

We should have some documentation about how to run this application.
Could be added to the README.md file showing the commands needed for running the application with java and how to run it using docker. After adding publishing of a docker image to github docker repository later we should also have the url for the docker image listed in the documentation.

Is there any like guidelines to how to write a documentation for an application correctly?

Can also take some inspiration from dockerhub documentation for instructions how to run a docker image.
https://hub.docker.com/_/nginx
Here are documentation about README.md format:
https://docs.github.com/en/get-started/writing-on-github/getting-started-with-writing-and-formatting-on-github/basic-writing-and-formatting-syntax
Things that should be documented are:
How to pull the latest version of our image
How to run it with docker run, don't forget portmapping. (-p 8080:8080)
Optional environment variables that can be set. (Portnumber)
Mount points:
/webroot for mounting a websites pages.   If creating a file called 404.html here it will be used when the server can't find other files.
/etc/storm/config/   If we mount a folder containing a file named config.json here we can change some parameters for the server like port number.


Use bugbug to automatically assign a product/component to untriaged bugs

Initially, with a high confidence level bar (e.g. 90%), then we can lower it.

First steps https://github.com/mozilla/relman-auto-nag/pull/374 and https://github.com/mozilla/relman-auto-nag/pull/379.


Change name get_next_sub_number and get_next_ses_number

Should just be get_next_sub and get_next_ses. Do a quick API review for necessary verbosity.

setup_ssh_connection_to_central_server -> setup_ssh_connection

setup_ssh_connection_to_central_server  rename performed in 1b30458 which I accidently pushed to main (branch bypass permissions now back off!!)


Dev

coverage: 0.0% (-81.5%) from 81.495%
when pulling 23643594ee3bf07aff36f59e170970e2153009ae on dev
into 43242b9c1007e97b4d3fedca03d43c49b1f57b39 on main.


Add a modernized cross-platform fork of MiniSat 2.2

This passed vcpkg build minisat locally, so hopefully there are no dumb mistakes.

Thank you very much for the PR!
I'm concerned about using the minisat name, due to how many other systems use that to refer to MiniSat 2.2[1]. Given how everyone else refers to the old source, it is reasonable for a migrating user to expect the same under the same name.
Therefore, I'd be much more comfortable with accepting this package under another name. A totally non-exhaustive list of suggestions:

master-keyring-minisat
minisat-master-keyring
minisat-keyring
microsat/nanosat/tinysat/femptosat/ etc :)
minisat-horenmar
horenmar-minisat

I might be persuaded by minisat3, but I'd prefer not.
[1] https://repology.org/metapackage/minisat/versions

I might be persuaded by minisat3, but I'd prefer not.

God no. I already don't like the BSD ports providing version 2.2.1 (it never existed), so let's not go inventing new major version whole cloth.

I changed the port name, but I think the port's folder should remain called minisat.

Thanks for the rename!


Added the ability to check eagleye topics.

It uses diagnostic to show the status.
The standard output of monitor_node can be optionally selected. (Default is output)

looking good


Wrong uninstallation method on linux systems

Infect the Linux home directory!
Impossible to remove with one click.
Annoying files stay on system.
Absolute malware/adware functionality!
After install python extension, Jupyter installed automatically as dependency.
This adware are show in app folder, and impossible to remove.
After remove

this malware extension,
or remove whole python extension,
or remove VSCOD app,
this malware stay on linux system home folder, and need to find and remove manually the files.

Absolute not user friendly!
I do not asked this tool, just installed automatically and impossible to remove as simple.
SYSTEM:
 - Fedora 36 Workstation latest
 - Gnome 42 latest
 - Application: VSCodium
 - Installation method: flatpak

@morfyum
Sorry for not getting back to you on this.
The Jupyter extesnion is an optional dependency of the Python extension, yes it is installed with the Python exension, however you can uninstall this anytime and as a result it will never get installed automatically again.

This adware are show in app folder, and impossible to remove.

I'd like to undertand what you mean by impossible to remove. You should be able to just uninstall this Jupyter extension and that should be it.

this malware stay on linux system home folder,

Would help if you could provide more information on what folder is still in the home folder. Please could you share a screenshot of this.
Please note, I've tested this at my end and the extension folder gets deleted completely.

Hello.
I cant replicate this issue. Maybe fixed?
Previously after remove the jupyter.
The desktop icon stay on my system, maybe on .local/share/applications/, .local/share/jupyter/ paths.
Currently directories are cleand.
Currently lot of file stay instide of flatpak app, this is not interested for me, because I doesnt see between my desktop apps.
./.var/app/com.vscodium.codium/data/codium/extensions/ms-toolsai.jupyter-2022.8*
BTW all file under svcodium extension, stay on system. Uninstall button just remove the flag from vscodium?
Files stay on system?
...
./.var/app/com.vscodium.codium/data/codium/extensions/ms-toolsai.jupyter-renderers-1.0.9/out/client_renderer/geojson.bundle.js.LICENSE.txt
./.var/app/com.vscodium.codium/data/codium/extensions/ms-toolsai.jupyter-renderers-1.0.9/out/client_renderer/vega.bundle.js.LICENSE.txt
./.var/app/com.vscodium.codium/data/codium/extensions/ms-toolsai.jupyter-renderers-1.0.9/out/client_renderer/renderers.js.map
./.var/app/com.vscodium.codium/data/codium/extensions/ms-toolsai.jupyter-renderers-1.0.9/out/client_renderer/renderers.js.LICENSE.txt
./.var/app/com.vscodium.codium/data/codium/extensions/ms-toolsai.jupyter-renderers-1.0.9/out/client_renderer/161.bundle.js.LICENSE.txt
./.var/app/com.vscodium.codium/data/codium/extensions/ms-toolsai.jupyter-renderers-1.0.9/out/client_renderer/geojson.bundle.js
./.var/app/com.vscodium.codium/data/codium/extensions/ms-toolsai.jupyter-renderers-1.0.9/out/client_renderer/dataresource.bundle.js
./.var/app/com.vscodium.codium/data/codium/extensions/ms-toolsai.jupyter-renderers-1.0.9/out/client_renderer/renderers.js
./.var/app/com.vscodium.codium/data/codium/extensions/ms-toolsai.jupyter-renderers-1.0.9/out/client_renderer/nteract_transforms.bundle.js.LICENSE.txt
./.var/app/com.vscodium.codium/data/codium/extensions/ms-toolsai.jupyter-renderers-1.0.9/out/client_renderer/plotly.bundle.js
./.var/app/com.vscodium.codium/data/codium/extensions/ms-toolsai.jupyter-renderers-1.0.9/out/client_renderer/modeldebug.bundle.js
./.var/app/com.vscodium.codium/data/codium/extensions/ms-toolsai.jupyter-renderers-1.0.9/out/client_renderer/nteract_transforms_vsdom.bundle.js
./.var/app/com.vscodium.codium/data/codium/extensions/ms-toolsai.jupyter-renderers-1.0.9/out/client_renderer/vega.bundle.js
./.var/app/com.vscodium.codium/data/codium/extensions/ms-toolsai.jupyter-renderers-1.0.9/out/client_renderer/plotly.bundle.js.LICENSE.txt
./.var/app/com.vscodium.codium/data/codium/extensions/ms-toolsai.jupyter-renderers-1.0.9/out/client_renderer/20.bundle.js
./.var/app/com.vscodium.codium/data/codium/extensions/ms-toolsai.jupyter-renderers-1.0.9/out/client_renderer/nteract_transforms.bundle.js
./.var/app/com.vscodium.codium/data/codium/extensions/ms-toolsai.jupyter-renderers-1.0.9/out/client_renderer/20.bundle.js.LICENSE.txt
./.var/app/com.vscodium.codium/data/codium/extensions/ms-toolsai.jupyter-renderers-1.0.9/out/client_renderer/161.bundle.js
./.var/app/com.vscodium.codium/data/codium/extensions/ms-toolsai.jupyter-renderers-1.0.9/out/client_renderer/591.bundle.js
./.var/app/com.vscodium.codium/data/codium/extensions/ms-toolsai.jupyter-renderers-1.0.9/out/extension_renderer
./.var/app/com.vscodium.codium/data/codium/extensions/ms-toolsai.jupyter-renderers-1.0.9/out/extension_renderer/index.js
./.var/app/com.vscodium.codium/data/codium/extensions/ms-toolsai.jupyter-renderers-1.0.9/out/extension_renderer/index.js.map
...

This is an uninstalled stage. App are closed, packages are stay on active system. but my primary issue is not this. I cant reproduce my real issue today.

Closing this issue as you cannot replicate this.
I would suggest you file an issue with VS Codium if this continues as this isn't a problem with VS Code.
Thanks for getting back to us with a confirmation.


Write a R program to convert minutes to days

Description
Write a R program to convert minutes to days
1 day = 24 hours = 1440 minutes
Input  : 7200
Output : 5

How to contribute

Save the solution in program/convert_minutes_to_days.r file
Add convert_minutes_to_days.r file in convert-minutes-to-days folder

!assign


Internet connection stops after turning on OpenVPN

Hi guys! I'm currently running some container that require remote connection with services such as databases or API that are only available through a VPN.
The current docker machine i'm running is the one based on the virtualbox drive and it works just fine when i connect to the VPN through the host OS (in this case OS X Sierra). I can also make the VPN work from the inside of the container running the OpenVPN as daemon and adjusting the IP routes. But when i switch to the docker machine created with xhyve the connection do not work on the container. I cannot even reach google.
I've tried both ways: a) connecting to VPN through the host OS and b) connecting to VPN from the inside of the container. Both are not working properly.
My guess is this is some issues with the network configuration devices on the docker machine itself?
Not sure how much info (dumps/outputs) would you guys need to debug it. So ping me with the ones needed to complement the report.
btw: this drive has a freaking good performance on my other projects! lot more lightweight and faster.

Try this one gist, you have to allow for bridge100 usage of vpn (utun0 or utun1 ...)
https://gist.github.com/mowings/633a16372fb30ee652336c8417091222

same here 😞

Any update on this? Can we possibly get this working without having to run a script after connecting to the VPN?

Doesn't seem to help here :(
$  interfaces=( $(netstat -in | egrep 'utun\d .*\d+\.\d+\.\d+\.\d+' | cut -d ' ' -f 1) )
rulefile="rules.tmp"
echo "" > $rulefile
sudo pfctl -a com.apple/tun -F nat
for i in "${interfaces[@]}"
do
  RULE="nat on ${i} proto {tcp, udp, icmp} from <IP_ADDRESS>/24 to any -> ${i}"
  echo $RULE >> $rulefile
done
sudo pfctl -a com.apple/tun -f $rulefile
No ALTQ support in kernel
ALTQ related functions disabled
nat cleared
pfctl: Use of -f option, could result in flushing of rules
present in the main ruleset added by the system at startup.
See /etc/pf.conf for further details.

No ALTQ support in kernel
ALTQ related functions disabled
$

Still no Internet:
$  docker pull ubuntu:latest
Error response from daemon: Get https://registry-1.docker.io/v2/: net/http: request canceled while waiting for connection (Client.Timeout exceeded while awaiting headers)

These are my interfaces:
$  ifconfig
lo0: flags=8049<UP,LOOPBACK,RUNNING,MULTICAST> mtu 16384
	options=1203<RXCSUM,TXCSUM,TXSTATUS,SW_TIMESTAMP>
	inet <IP_ADDRESS> netmask 0xff000000
	inet6 ::1 prefixlen 128
	inet6 fe80::1%lo0 prefixlen 64 scopeid 0x1
	nd6 options=201<PERFORMNUD,DAD>
gif0: flags=8010<POINTOPOINT,MULTICAST> mtu 1280
stf0: flags=0<> mtu 1280
en0: flags=8963<UP,BROADCAST,SMART,RUNNING,PROMISC,SIMPLEX,MULTICAST> mtu 1500
	options=b<RXCSUM,TXCSUM,VLAN_HWTAGGING>
	ether 50:e5:49:ef:45:8f
	inet6 fe80::14ea:f077:5638:4c28%en0 prefixlen 64 secured scopeid 0x4
	inet <IP_ADDRESS> netmask 0xffffff00 broadcast <IP_ADDRESS>
	inet <IP_ADDRESS> netmask 0xffffff00 broadcast <IP_ADDRESS>
	inet <IP_ADDRESS> netmask 0xffffff00 broadcast <IP_ADDRESS>
	inet <IP_ADDRESS> netmask 0xffffff00 broadcast <IP_ADDRESS>
	inet <IP_ADDRESS> netmask 0xffffff00 broadcast <IP_ADDRESS>
	nd6 options=201<PERFORMNUD,DAD>
	media: autoselect
	status: active
fw0: flags=8863<UP,BROADCAST,SMART,RUNNING,SIMPLEX,MULTICAST> mtu 2030
	lladdr 00:49:e5:50:7c:f8:4a:00
	nd6 options=201<PERFORMNUD,DAD>
	media: autoselect <full-duplex>
	status: inactive
tap1: flags=8843<UP,BROADCAST,RUNNING,SIMPLEX,MULTICAST> mtu 1500
	ether f2:c9:37:4a:1a:01
	inet <IP_ADDRESS> netmask 0xffff0000 broadcast <IP_ADDRESS>
	media: autoselect
	status: active
	open (pid 126)
utun0: flags=8051<UP,POINTOPOINT,RUNNING,MULTICAST> mtu 1380
	inet6 fe80::612e:5633:874a:fba0%utun0 prefixlen 64 scopeid 0x8
	inet6 fd9c:cf81:26ab:b93a:612e:5633:874a:fba0 prefixlen 64
	nd6 options=201<PERFORMNUD,DAD>
utun1: flags=8051<UP,POINTOPOINT,RUNNING,MULTICAST> mtu 2000
	inet6 fe80::8e08:9cea:3582:a01c%utun1 prefixlen 64 scopeid 0x9
	nd6 options=201<PERFORMNUD,DAD>
vmnet1: flags=8863<UP,BROADCAST,SMART,RUNNING,SIMPLEX,MULTICAST> mtu 1500
	ether 00:50:56:c0:00:01
	inet <IP_ADDRESS> netmask 0xffffff00 broadcast <IP_ADDRESS>
vmnet8: flags=8863<UP,BROADCAST,SMART,RUNNING,SIMPLEX,MULTICAST> mtu 1500
	ether 00:50:56:c0:00:08
	inet <IP_ADDRESS> netmask 0xffffff00 broadcast <IP_ADDRESS>
en5: flags=8963<UP,BROADCAST,SMART,RUNNING,PROMISC,SIMPLEX,MULTICAST> mtu 1500
	ether 2a:b3:7b:6a:87:dc
	media: autoselect
	status: active
bridge100: flags=8863<UP,BROADCAST,SMART,RUNNING,SIMPLEX,MULTICAST> mtu 1500
	options=3<RXCSUM,TXCSUM>
	ether 52:e5:49:fe:db:64
	inet <IP_ADDRESS> netmask 0xffffff00 broadcast <IP_ADDRESS>
	Configuration:
		id 0:0:0:0:0:0 priority 0 hellotime 0 fwddelay 0
		maxage 0 holdcnt 0 proto stp maxaddr 100 timeout 1200
		root id 0:0:0:0:0:0 priority 0 ifcost 0 port 0
		ipfilter disabled flags 0x2
	member: en5 flags=3<LEARNING,DISCOVER>
	        ifmaxaddr 0 port 13 priority 0 path cost 0
	nd6 options=201<PERFORMNUD,DAD>
	media: autoselect
	status: active

Same issue here


Iterators

[x] Introduce iterators into Diversity code.
[x] Add tests to make sure it's all working
[x] Fix code so tests do work!

Coverage decreased (-0.9%) to 94.851% when pulling b4a46e273b59b9c6c73abf5d1b3634368499cbce on iterators into 79a06b6c23d23d3967001da9115364daf5ad5830 on master.

Coverage decreased (-0.9%) to 94.851% when pulling 7a4cf180ee4881dcf61906a1dfb28f9f52c8a5d1 on iterators into 79a06b6c23d23d3967001da9115364daf5ad5830 on master.

Coverage decreased (-0.9%) to 94.851% when pulling cbe7d318d05927cc1f758426cc21a61274027845 on iterators into 79a06b6c23d23d3967001da9115364daf5ad5830 on master.

Coverage decreased (-0.9%) to 94.851% when pulling 9dbcee3a5d8fe61d04b66a85c561df3ee9043788 on iterators into 79a06b6c23d23d3967001da9115364daf5ad5830 on master.

Coverage decreased (-0.9%) to 94.851% when pulling 4d26fbe382cbc6155fd54167d496fd5d99384e6c on iterators into 79a06b6c23d23d3967001da9115364daf5ad5830 on master.

Coverage decreased (-1.5%) to 94.262% when pulling 64a0922f777a5f30fb9b7d347278ae5f0a5800d0 on iterators into 79a06b6c23d23d3967001da9115364daf5ad5830 on master.

Fixes #8.
Fixes #5.


Nodes are getting Lost everytime with High Disk IO usage. Cant ssh is available either.

Is this a request for help?:
YES
Is this an ISSUE or FEATURE REQUEST? (choose one):
ISSUE
What version of acs-engine?:
0.5.0, 0.16.2, 0.17.0

Orchestrator and version (e.g. Kubernetes, DC/OS, Swarm)
Kubernetes
What happened:
Nodes are getting lost from 3 weeks ago. I deployed the cluster several times with different acs-engine versions and its not fixed. I can see that my nodes are getting a high read disk usage, almost 100% and I cannot even ssh into the agent nodes when they are lost. It obviously produce downtimes. Sometimes more than one node is lost. It happens in my production environments. Machines Standard D4s v3
What you expected to happen:
Avoid losing nodes.
How to reproduce it (as minimally and precisely as possible):
I generated several clusters in my prod environment and same thing happen.
Anything else we need to know:

https://github.com/Azure/acs-engine/issues/906

@jackfrancis Hey could you help me on that ?? We are experiencing this every moment and again losing nodes in Production. Is there a way we can RAID system disks so we can increse IOPS ? Kubelet seems to be reading quite a lot.
I appreaciate if I can a feedback on that from you may be you can indicate us how to proceed with this?

@hmarcelodn What is the current IOPS limit of the disks you're seeing this on?

@jackfrancis  I had 500 IOPS from an Standard_D2_v2 with 30 GB. I just deployed a Standard_DS2_V2 (which uses Premium Disks by default) with Premium Disks with 250 GB + 1100 IOPS. It is in production now running after I regenerated the cluster. I am going to monitor this and see what happens. Is there a best practice for a production ready cluster in terms of recommended IOPS ? Thanks !!

After the change I can see a lot of activity again @jackfrancis

https://github.com/kubernetes/kubernetes/issues/64108 Related

@hmarcelodn: I had a similar (or perhaps the same) issue, which ultimately turned out to be sssd. Is that installed on your VM? If so, 0.17.0 uses a new Ubuntu image which fixes the issue -> see https://bugs.launchpad.net/cloud-images/+bug/1746806

Thanks @tammert I am going to try this! Good finding. Thanks!


more detail on API version

It would be useful if there were a little more detail in the spec around API versioning.  How is versioning defined here and for what intention?  What behaviour is expected of a client and server (e.g. if their versions don't match)?
Examples:

you might want to state that X-Broker-API-Version header is required (e.g. client MUST include it in requests; server MUST include it in responses, etc.)
you might want to declare semantic versioning / or behaviour like MUST deny requests with invalid API versions as follows, etc.

#131 includes more information about API versioning, does it satisfy your requirements?

It helps, but AFAICS, there's no defined way for the server to indicate its API version back to the client?  It would probably make sense for this to be possible, perhaps either via a /version endpoint or by requiring the server to include the X-Broker-Api-Version header in its responses.

The spec says

In this scenario the broker may reject the request with 412 Precondition Failed and provide a message that informs the operator of the required API version.

If this doesn't fit your requirements, you can make a change proposal so that it can be discussed by everyone.

Hello @jim-minter
In truth, because all enhancements to the spec are backwards compatible, the version header isn't necessary for disparate versions in platform and broker to be compatible. As long as both support the 2.x version series, they should be compatible with the caveat that new features are optional and require both platform and broker to support them.
With regard to a broker returning the version it supports, consider that many features are optional; a broker might support volume services or route service; it might not support asynchronous provisioning but support other latest features. With this in mind, what would a version indicate to a platform client, and how would you expect the platform to behave as a result?

Closing due to inactivity.


"Viewer" Node

A node to view which displays the input in the background would be very helpful. Currently I have to be careful not to deselect the node I want to view.

Hmm maybe we could add a "lock" option to previews instead?


Connect button is missing

It should be available for all kind of projects

Using a fresh install, self-hosted Hive - DOCKER_TAG=":23df2c05b7bc4c7f76d9110369a0cb71cda483f5"
Is the expectation that a Connect button should appear? I see no Connect button in any project type I've tried.


Add small header cache

This saves a few DB queries when syncing as the same hader is being queried multiple times.

@arkpar are you still working on this? The polkadot companion is still failing.

Should be good to merge now

bot merge


🎶 Synced file(s) with kronostechnologies/standards

Synced local file(s) with kronostechnologies/standards.

This PR was created automatically by the repo-file-sync-action workflow run #2928497851

Oublié de modifier standards


Created tailored data for each activation model

In GitLab by @cmastalli on Mar 15, 2019, 18:06

In GitLab by @nmansard on Mar 20, 2019, 14:05
could you explain?

In GitLab by @cmastalli on Mar 20, 2019, 15:17
All data classes are empty, and instead the data is created dynamically.
I noticed this when I created the abstractions of the data classes !144.

In GitLab by @cmastalli on Jul 26, 2019, 13:39
I will close this issue. I fixed in the c++ library.

In GitLab by @cmastalli on Jul 26, 2019, 13:39
closed


add TestDaemonwithwrongkey test case

Make sure that it is failed to start daemon with a wrong key.json file.
Signed-off-by: Yuan Sun<EMAIL_ADDRESS>

Can you please sign your commits following these rules:
https://github.com/docker/docker/blob/master/CONTRIBUTING.md#sign-your-work
The easiest way to do this is to amend the last commit:
$ git clone -b "TestDaemonwithwrongkey" git@github.com:sunyuan3/docker.git somewhere
$ cd somewhere
$ git commit --amend -s --no-edit
$ git push -f

This will update the existing PR, so you do not need to open a new one.

@sunyuan3 if you need to update this pull request, please make sure you force push to your github branch, it will update the PR automatically.

I think this test is hanging as start daemon with a failure actually doesn't stop the daemon until PR #11320 is merged.  Twice it has hung on running the added daemon test (and Jenkins kills it after 45 minutes).  Also, this test should d1.Stop() for the error case.

The test is blocked. However it is pass with commit 98630b2ef62653d288430b25867ddacbf1731441.
The key.json file is modified with a fake json:"d" value. So it is expected to fail to run d1.Start().
Since d1.Start() is failed, is it necessary to add  d1.Stop() ?

rebuilding now the #11320 was merged,
oh wait needs rebase

It seems the test is still blocked after PR #11320 is merged.

@sunyuan3 Can you please rebase? Thanks!

Ok. I will rebase ASAP :)

It is obvious that other cases lead to the failure of test automation test.

The test is green locally, I'm confused about what it really does. Ping @dmcgowan!

config.Kid should be modified, not config.D. I will update the case.

Ping @tiborvass @estesp @jfrazelle @icecrime @dmcgowan.
The codes have been updated and the automation tests are pass.
What's your comments? ;-)

Sorry, I didn't see dmcgowan's comment. I will see.

@dmcgowan, great. Thanks for your suggestion. I will update the case. ;-)

LGTM

LGTM

LGTM


Downloading result "Failed - Forbidden"

I just tried download a series and later google chrome gave me that error on downloading.

Hi. This is because of the ongoing issue with 9anime. 9ac tried to download from the first row of servers.


feat: hide default markdown preview button

Pending integration of this feature to VSCode.
https://github.com/microsoft/vscode/pull/136273
Pull Request Checklist
You can go to dendron pull requests to see full details for items in this checklist.
General
Quality Assurance

[ ] add a test for the new feature
[ ] make sure all the existing tests pass
[ ] do a spot check by running your feature with our test workspace
[ ] after you submit your pull request, check the output of our integration test and make sure all tests pass

NOTE: if you running MacOS or Linux, pay special attention to the Windows output and vice versa if you are developing on Windows

Special Cases

[ ] if your tests changes an existing snapshot, make sure that snapshots are updated
[ ] if you are adding a new language feature (graphically visible in VS Code/preview/publishing), make sure that it is included in test-workspace. We use this to manually inspect new changes and for auto regression testing

Docs

[ ] Make sure that the PR title follows our commit style
[ ] Please summarize the feature or impact in 1-2 lines in the PR description
[ ] If your change reflects documentation changes, also submit a PR to dendron-site and mention the doc PR link in your current PR

Example PR Description
# feat: capitalize all foos

This changes capitalizes all occurrences of `foo` to `Foo`

Docs PR: <URL_TO_DOCS_PR>

Special Cases
First Time PR

[ ] sign the CLA which will be prompted by our github bot after you submit the PR
[ ] add your discord alias in the review so that we can give you the horticulturalist badge in our community

Analytics

[ ] if you are adding analytics related changes, make sure the Telemetry docs are updated

I understand the upstream change, but as a user it would be nicer if the button worked and opened (one of the registered) custom preview instead of not showing up any more.
(Not that I care too much, I didn't even know about that button)

@lnicola Thanks for the feedback! Once we remove the default button, we can add our own preview button to replace it. This is the first step towards it.


[v6.0.0] Update Liquid.Sample.WebApi.sln Liquid package references

update projects with Liquid packages v6.0.0 and solve braking changes.
this issue depends on issues #127 ,#128 , #129 , #130 conclusion.

resoved by #219


feat kafka: add userver-kafka cmake support

Will be rebased to new internal version of Kafka library
Currently, PR is just for testing Docker and CI builds

I hereby agree to the terms of the CLA available at: https://yandex.ru/legal/cla/


Added docstring to LobbyistContributions1Cd . Closes #133

You won a swaglode prize! Report to the Atrium bar to pick it up or contact me now!


Version 0.1.0: Setting of chart options via Angular template does not work anymore

Which platform(s) does your issue occur on?
iOS 13.5 but also other versions on emulator. Not tried on device as it likely is the same
Please, provide the following version numbers that your issue occurs with:
CLI: ns --version 7.2.1
Cross-platform modules: "@nativescript/core": "~7.3.0",
Runtime(s): "tns-ios": "6.5.4", and @nativescript/ios 7.2.0
Plugin(s): 0.1.0
Please, tell us how to recreate the issue in as much detail as possible.
In the past I could set chart options vie the Angular template
<UIChartsView [options]="chartOptions" >

This unfortunately does not work currently. A workaround is given using the loaded event:
<UIChartsView (loaded)="chartLoaded($event)">

public chartLoaded(args: EventData) {
const chart = args.object as UIChartsView;
chart.setOptions(this.chartOptions);
}

@cjohn001
Yes, we ended up removing the ability to hookup options by binding.
And using the chartLoaded event is the only way to setup chart options now.
The readme has already been updated to reflect the same.
And I've also updated all the chart demos in the "demo" folder. (haven't updated the angular demos yet)
Another thing is that, if you want to update the chart options after the chart has loaded,
you have to use "updateOptions()" method on the chartView object.

Hello @shiv19 , thanks a lot for the info. The updateOptions() function I would have probably never found out about. I assume due to having two functions now the binding in the angular template is not possible anymore?
Just for interest. What does the updateOptions()function do different than setOptions()? I observed that when I async update data repeatedly within very short time intervals, the chart was not refreshed correctly in old plugin version. Is this solved with updateOptions() now? I suspected it to be due to a refresh with new data during initial chart animation. Therefore I had debounced my refreshs.

Yes with updateOptions, you should be able to update the chart data in short intervals

Well, as this is the new intended behavior I think I can close this issue.


Sending messages on mobile web is annoying

Zulip's compose box doesn't have a nice system for responsively being usable on mobile, which means it can be hard to type and send a message when using Zulip on a phone.

I also have difficulty scrolling the message feed when I run the normal web app on my iPhone.

The scrolling is now fixed, and the sending issue is a duplicate of #2306 which will be closed soon via #3996, so I'm closing this.  Thanks to @brockwhittaker for fixing it :)


fix(tab): update active tab when value is changed externally

https://github.com/Endava/bee-q/assets/109202804/3b3254ff-e382-481b-acca-f25dd5f8e7b8

☁ Nx Cloud Report
CI is running/has finished running commands for commit 1c6a49596cadb0f775b094828d3e462ab14d0a29.  Click to see the status, the terminal output, and the build insights.
📂 See all runs for this branch

🟥 Failed Commands

nx affected --target=test --base=9e8899437c29bb26269264e7347c10e4f5eafadc --head=2ac3703c0fe605888ddfaa417975753c34e952db --exclude=workspace --output-style=stream-without-prefixes -- --ci --code-coverage --runInBand

Sent with 💌 from NxCloud.


the test does not work in this challenge

the test have a bug and doesn’t pass

Your browser information:
User Agent is: Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/67.0.3396.87 Safari/537.36.
Link to the challenge:
https://learn.freecodecamp.org/responsive-web-design/responsive-web-design-principles/use-a-retina-image-for-higher-resolution-displays

@zoaby I just copied your code and it passed in Google Chrome 67+ on Ubuntu. Can you provide some more detail or a screenshot (will be better).

@zoaby @mpmaan  Can confirm that the tests are passing in my Google Chrome v67 on Windows.

the problem was in the code because the test doesn't accept spaces


Idea: Make it easier to compare zipped Lists

Summary
I find myself writing tests for Bean mappers, mapping e.g CustomerOrderDTO to CustomerOrder and similar. Regularly this involves comparing collections. While this can be done with Comparators, comparators have several disadvantages, they only return a number, and they do not make use of fluent APIs (unless someone knows a fluentAPI to create Comparators?).
Example
List<AdressDTO> source = generateAdressDTOs();
List<Adress> target = convert(source);
assertThat(source).comparedToOtherSatisfies(target, (AdressDTO dto, Adress adress) -> {
   assertThat(dto.getZipcode() + ' ' + dto.getCity()).isEqualTo(adress.getCityLine())
});

Maps would probably also be useful. Not sure what else.
I assume the above can also be achieved using a Comparator that casts, and returns 0 or throws AssertionException. But that's somewhat ugly...
Java 8 specific ?
Not sure if that makes much sense without lambdas...

That is interesting because it is really possible in AssertJ.
I was thinking that  the suggested assertions is actually a matching element by element, I would name the assertion like this (thinking out loud):
assertThat(source).matches(target, (AdressDTO dto, Adress adress) -> {
   assertThat(dto.getZipcode() + ' ' + dto.getCity()).isEqualTo(adress.getCityLine())
});

I have no strong opinion about the name, I have not used assertj enough to know what name would fit will in the overall picture. I am amazed how many convenience functions there are already in assertj, sometimes I feel overwhelmed finding the one I need. I cannot yet promise when I will find time to do a PR.

I find that I prefer the satisfies(...) method to the matches(...) methods.
So I would suggest this:
@Override
  public <OTHER_ELEMENT> SELF allSatisfy(Iterable<OTHER_ELEMENT> other, BiConsumer<? super ELEMENT, OTHER_ELEMENT> consumer) {
    Iterator<? extends ELEMENT> i1 = actual.iterator();
    Iterator<OTHER_ELEMENT> i2 = other.iterator();
    while (i1.hasNext()) {
      assertThat(i2.hasNext()).isTrue(); // TODO add useful message
      consumer.accept(i1.next(), i2.next());
    }
    assertThat(i2.hasNext()).isFalse(); // TODO add useful message
    return myself;
  }

Usage:
 @Test
  public void testAllSatisfy() {
    Employee yoda = new Employee(1L, new Name("Yoda"), 800);
    Employee obiwan = new Employee(2L, new Name("Obi"), 800);
    List<Employee> employees = newArrayList(yoda, obiwan);

    EmployeeDTO yodaDTO = new EmployeeDTO("Yoda", 800);
    EmployeeDTO obiwanDTO = new EmployeeDTO("Obi", 800);
    List<EmployeeDTO> employeeDTOs = newArrayList(yodaDTO, obiwanDTO);

    assertThat(employees).allSatisfy(employeeDTOs, (employee, employeeDTO) -> {
      assertThat(employee.getAge()).isEqualTo(employeeDTO.age);
      assertThat(employee.getName().getFirst()).isEqualTo(employeeDTO.name);
    });
  }

  private static class EmployeeDTO {
    public String name;
    public int age;

    public EmployeeDTO(String name, int age) {
      this.name = name;
      this.age = age;
    }
  }

And the same for maps, via:
public <K2, V2> SELF allSatisfy(Function<K, K2> keyMapper, Map<K2, V2> other, BiConsumer<V, V2> requirements) {
    actual.forEach((k, v) ->
      requirements.accept(v, other.get(keyMapper.apply(k))));
    return myself;
  }

  public <V2> SELF allSatisfy(Map<K, V2> other, BiConsumer<V, V2> requirements) {
    actual.forEach((k, v) ->
            requirements.accept(v, other.get(k)));
    return myself;
  }

@tkruse what about zipSatisfy?

it's creative and descriptive. Whether users will easily find it (via search or tab completion) I don't know. I think I might type 'satisfy' first and browse through the candidates of my IDE, but that might be just me.

fortunately IDEs are able to show zipSatisfy when code completing with satis...


🚀 The feature, motivation and pitch
Claim major improvements over vllm. Unfortunately no code only the paper.
arxiv.org/abs/2405.04437
Alternatives
No response
Additional context
No response

Hi, I'm one of the authors of this paper. Thank you for your interest in our work! We plan to release the code soon, hopefully in a few weeks.

@ramyaprabhu-alt Just curious the code release would be a separate project or a PR against vLLM?  I think it's a PR, right?

Our initial release will be as a separate project on a slightly older version of vLLM. But soon after, we can also raise a PR against vLLM-latest.

Glad to share the source code of vAttention. Please check it out here: https://github.com/microsoft/vattention

is this still on VLLM roadmap to integrate? Please let us know.
Thanks

is this still on VLLM roadmap to integrate? Please let us know.
Thanks

If there is still interest from vLLM community, we will be happy to contribute!


[backport] Fix thrust compilation in MSVC 14 for CUDA 11.2

Backport of #4713

[automatic post] Jenkins, test this please.

Jenkins CI test (for commit 9ea29fc144b4f36330331f7362b48de5acdb3896, target branch v8) succeeded!


Updating my bio and profile.

Updating my bio and profile.

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

File
Status
Preview URL
Details

:x:Error

Details

[Error] Cannot sync git repo to specified state. Please make sure the pull request is mergeable. Details: Git merge results in conflict, repository remote url: https://github.com/MicrosoftDocs/cloud-developer-advocates, working directory: W:\fqep-s, target branch: master, target branch commit: cdc4cd65d5d6d6f6298c866921bd3849e7e93fcd, source commit: 7e5a98de7a0d115a072c934320da6f41df5e58ba.

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

@MaximRouiller created and resolved merges.

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

File
Status
Preview URL
Details

advocates/glaucia-lemos.yml
:x:Error

Details

advocates/index.html.yml
:warning:Warning

Details

advocates/map.yml
:warning:Warning

Details

advocates/toc.yml
:warning:Warning

Details

advocates/glaucia-lemos.yml

[Error]

Unable to load file 'glaucia-lemos.yml' via processor 'Profile': Schema validation failed. Please validate the file and make sure it conforms to schema 'Profile' (https://static.docs.com/ui/latest/schemas/Profile.schema.json):
Property 'devto' has not been defined and the schema does not allow additional properties. Path 'devto'.

advocates/index.html.yml

[Warning] Unable to find uid "advocates.glaucia-lemos".
Line 0: [Warning] Invalid file link:(~/tweets).
Line 0: [Warning] Invalid file link:(~/map).

advocates/map.yml

[Warning] Unable to find uid "advocates.glaucia-lemos".

advocates/toc.yml

[Warning] Unable to find file with uid "advocates.glaucia-lemos" referenced by TOC file "toc.yml"

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


Custom mcCommands

How to properly add custom commands? I'm trying to get the bot to OP someone.
I'm using the latest version in a Forge server.

Here is the default json, but with an /op  command added
customCommandJSON = "{"kick":{"adminOnly":true,"mcCommand":"kick","description":"Kicks a player from the server","useArgs":true,"argText":" [reason]"},"stop":{"adminOnly":true,"mcCommand":"stop","description":"Stops the server","aliases":["shutdown"],"useArgs":false},"kill":{"adminOnly":true,"mcCommand":"kill","description":"Kills a player","useArgs":true,"argText":""},"op":{"adminOnly":true,"mcCommand":"op","description":"OPs a player","useArgs":true,"argText":""}}"

Here is the default json, but with an /op  command added
customCommandJSON = "{"kick":{"adminOnly":true,"mcCommand":"kick","description":"Kicks a player from the server","useArgs":true,"argText":" [reason]"},"stop":{"adminOnly":true,"mcCommand":"stop","description":"Stops the server","aliases":["shutdown"],"useArgs":false},"kill":{"adminOnly":true,"mcCommand":"kill","description":"Kills a player","useArgs":true,"argText":""},"op":{"adminOnly":true,"mcCommand":"op","description":"OPs a player","useArgs":true,"argText":""}}"

Thank you!

Thank you!


Controlled Rollout

Hi ,
AML supports "controlled rollout" to AKS - https://docs.microsoft.com/en-us/azure/machine-learning/how-to-deploy-azure-kubernetes-service#deploy-models-to-aks-using-controlled-rollout-preview.
In summary, this feature allows, multiple versions of model to be deployed on same AKS cluster. Each version of the model serves some percentage of traffic for the same endpoint.
Is there a plan to add support for this feature?

@prwani If this is required, we can prioritize this feature. Until when do you need this feature?

@marvinbuss I don't have an immediate requirement. It is just a feature request.

I already added the request to our backlog and we will add this feature in the near future. We will keep this issue open until we implement this feature.


Unable to have multiple messages

Hi Prajval,
Firstly, thank you very much for this awesome, Interactive Live Agent app.
I wanted to bring something to your attention, Not sure if it is a bug or you didn't have it added.

When we are doing a 1:1 messaging session (customer sends 1 message and agent sends 1 message), this is working brilliant

When the agent wants to send multiple messages at the same time, it is not working.

Here is an example:
1:1 messages This is working
A: Hi user How Can I help you?
U: I'm having an issue with my computer.
A: let me create a case for you.
U: thank you.
many:many messages This is not working
A: Hi user How Can I help you?
U: I'm having an issue with my computer.
A: let me create a case for you.
A: Can you please give me a minute?
U: thank you.
Can you please give me a workaround for this scenario?
Thanks

Hi @sunnykeerthi, it's not a bug rather it was intentionally left out. Actually, this app was developed just to be a demo/test for Salesforce Liveagent APIs. The actual app that was developed for RocketChat Livechat, contains the features such as multiple messages, queue support etc., you can find the repo here.


Merge from wnprc18.3 svn r.64261 to r.65647

Rationale
Merge from wnprc18.3 svn r.64261 to r.65647 (skipped r. 64864 & 65166 since these were backports)
Related Pull Requests
https://github.com/LabKey/LabDevKitModules/pull/38
https://github.com/LabKey/ehrModules/pull/56
https://github.com/LabKey/onprcEHRModules/pull/20
https://github.com/LabKey/platform/pull/1370
https://github.com/LabKey/tnprc_billing/pull/72

We can close this PR. These are backports of changes that are already in develop. Code has changed, so it's not applying cleanly.


operator mondoo-operator (1.6.4)

Signed-off-by: Christian Zunker<EMAIL_ADDRESS>

/merge possible

/merge possible


🐛 [firebase_core] Cannot install in conjuction with permission_handler

Bug report
Describe the bug
ios/Pods/GoogleDataTransport/GoogleDataTransport/GDTCCTLibrary/GDTCCTUploadOperation.m:408:3: error: implicit declaration of function
'pb_release' is invalid in C99
Steps to reproduce
Include
firebase_core: 1.12.0 in pubspec.yaml
in the podfile (add from permission_handler library)
post_install do |installer|
installer.pods_project.targets.each do |target|
flutter_additional_ios_build_settings(target)
target.build_configurations.each do |config|
config.build_settings['GCC_PREPROCESSOR_DEFINITIONS'] ||= [
'PERMISSION_CAMERA=1',
'PERMISSION_MICROPHONE=1',
'PERMISSION_SPEECH_RECOGNIZER=1',
]
end
end
end
Steps to reproduce the behavior:
Try to build the app
Expected behavior
The app should compile, you can reproduce using the default  counter app
Sample project
Providing a minimal example project which demonstrates the bug in isolation from your main App greatly enhances the chance of a timely fix.
you can use the default counter app, You just need to change the pubspect.yaml and Podfile
Additional context
Add any other context about the problem here.

Flutter doctor
Doctor summary (to see all details, run flutter doctor -v):
[✓] Flutter (Channel stable, 2.8.1, on macOS 11.5.1 20G80 darwin-x64, locale
en-IL)
[✓] Android toolchain - develop for Android devices (Android SDK version 30.0.3)
[✓] Xcode - develop for iOS and macOS (Xcode 13.0)
[✓] Chrome - develop for the web
[✓] Android Studio (version 2021.1)
[✓] Android Studio (version 2020.3)
[✓] VS Code (version 1.63.2)
[✓] Connected device (3 available)
• No issues found!
➜  ~
Click To Expand
PASTE OUTPUT INSIDE HERE

ios/Pods/GoogleDataTransport/GoogleDataTransport/GDTCCTLibrary/GDTCCTUploadOperation.m:408:3: error: implicit declaration of function
'pb_release' is invalid in C99 [-Werror,-Wimplicit-function-declaration]
pb_release(gdt_cct_BatchedLogRequest_fields, &batchedLogRequest);

Flutter dependencies
Run flutter pub deps -- --style=compact and paste the output below:
Changing current working directory to: /Users/ryanheitner/Projects/FirbaseTest
Dart SDK 2.15.1
Flutter SDK 2.8.1
fire_base 1.0.0+1
|-- cupertino_icons 1.0.4
|-- firebase_core 1.12.0
|   |-- firebase_core_platform_interface 4.2.4
|   |   |-- collection...
|   |   |-- flutter...
|   |   |-- meta...
|   |   '-- plugin_platform_interface 2.1.2
|   |       '-- meta...
|   |-- firebase_core_web 1.5.4
|   |   |-- firebase_core_platform_interface...
|   |   |-- flutter...
|   |   |-- flutter_web_plugins 0.0.0
|   |   |   |-- characters...
|   |   |   |-- collection...
|   |   |   |-- flutter...
|   |   |   |-- js...
|   |   |   |-- meta...
|   |   |   |-- typed_data...
|   |   |   '-- vector_math...
|   |   |-- js 0.6.3
|   |   '-- meta...
|   |-- flutter...
|   '-- meta...
|-- flutter 0.0.0
|   |-- characters 1.2.0
|   |-- collection 1.15.0
|   |-- meta 1.7.0
|   |-- sky_engine 0.0.99
|   |-- typed_data 1.3.0
|   |   '-- collection...
|   '-- vector_math 2.1.1
|-- flutter_application_id 2.0.0-dev
|   |-- args 2.3.0
|   |-- equatable 2.0.3
|   |   |-- collection...
|   |   '-- meta...
|   '-- yaml 3.1.0
|       |-- collection...
|       |-- source_span...
|       '-- string_scanner...
|-- flutter_launcher_icons 0.9.2
|   |-- args...
|   |-- image 3.1.1
|   |   |-- archive 3.1.6
|   |   |   |-- crypto 3.0.1
|   |   |   |   |-- collection...
|   |   |   |   '-- typed_data...
|   |   |   '-- path...
|   |   |-- meta...
|   |   '-- xml 5.3.1
|   |       |-- collection...
|   |       |-- meta...
|   |       '-- petitparser 4.4.0
|   |           '-- meta...
|   |-- path...
|   '-- yaml...
'-- flutter_test 0.0.0
|-- async 2.8.2
|   |-- collection...
|   '-- meta...
|-- boolean_selector 2.1.0
|   |-- source_span...
|   '-- string_scanner...
|-- characters...
|-- charcode 1.3.1
|-- clock 1.1.0
|-- collection...
|-- fake_async 1.2.0
|   |-- clock...
|   '-- collection...
|-- flutter...
|-- matcher 0.12.11
|   '-- stack_trace...
|-- meta...
|-- path 1.8.0
|-- source_span 1.8.1
|   |-- collection...
|   |-- path...
|   '-- term_glyph...
|-- stack_trace 1.10.0
|   '-- path...
|-- stream_channel 2.1.0
|   '-- async...
|-- string_scanner 1.1.0
|   |-- charcode...
|   '-- source_span...
|-- term_glyph 1.2.0
|-- test_api 0.4.3
|   |-- async...
|   |-- boolean_selector...
|   |-- collection...
|   |-- matcher...
|   |-- meta...
|   |-- source_span...
|   |-- stack_trace...
|   |-- stream_channel...
|   |-- string_scanner...
|   '-- term_glyph...
|-- typed_data...
'-- vector_math...
Click To Expand
PASTE OUTPUT INSIDE HERE

@ryanheitner
Although I am not getting the same error as you reported, I see below:
/Users/dhs/Documents/NCFlutter/testdemo/ios/Pods/FirebaseCore/FirebaseCore/Sources/FIRVersion.m:20:2: error: "Firebase_VERSION is not defined: add -DFirebase_VERSION=... to the build invocation"     #error "Firebase_VERSION is not defined: add -DFirebase_VERSION=... to the build invocation"
Can you try to use the permission_handler plugin example, add firebase_core to it and see if you still get same behavior ?

Thank you thanks to your help I have resolved the issue
I was missing  '$(inherited)',
 target.build_configurations.each do |config|
  config.build_settings['GCC_PREPROCESSOR_DEFINITIONS'] ||= [
    '$(inherited)',


[Question] How to open emacs manual in vertical window?

Currently SPC h i give me info manual in horizontal window, how to open it in vertical window?

The default rule for info-mode is :align 'below :size 0.5 :select t :autokill t. The relevant property is :align. You can change its rule with:
;; put this in private/$USER/config.el
(set! :popup "*info*" :align 'right :size 0.5 :select t :autokill t)
;; you can change 'right to 'left

Besides that, there is no way to move a live popup window yet -- I'm working on that.

Tks @hlissner, problem is solved 👍


Allow removing dependencies

Based on the PR: https://github.com/software-mansion/protostar/pull/3.
Changes

added remove command

Seems like the changes which were already merged are shown in the diff, can we fix that?

It should be fixed now.


[CARBONDATA-3461] Carbon SDK support filter equal values set.

Be sure to do all of the following checklist to help us incorporate
your contribution quickly and easily:

[x] Any interfaces changed?
Add new interfaces
[x] Any backward compatibility impacted?
No
[x] Document update required?
No
[x] Testing done
added
[x] For large changes, please consider breaking it into sub-tasks under an umbrella JIRA.
NO

Build Success with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder2.1/3750/

@xubo245 : I see changes only for Equal to filter. so change description to support Equal to filter set or handle all the filter expressions.

Build Success with Spark 2.3.2, Please check CI http://<IP_ADDRESS>:8080/job/carbondataprbuilder2.3/12026/

Build Failed  with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder2.1/3752/

Build Failed  with Spark 2.3.2, Please check CI http://<IP_ADDRESS>:8080/job/carbondataprbuilder2.3/12028/

Build Failed with Spark 2.2.1, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder1/3964/

@ajantha-bhat Optimized, please check again.

Build Failed  with Spark 2.3.2, Please check CI http://<IP_ADDRESS>:8080/job/carbondataprbuilder2.3/12029/

Build Success with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder2.1/3753/

Build Failed with Spark 2.2.1, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder1/3965/

Build Success with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder2.1/3754/

Build Success with Spark 2.3.2, Please check CI http://<IP_ADDRESS>:8080/job/carbondataprbuilder2.3/12030/

Build Failed with Spark 2.2.1, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder1/3966/

retest this please

Build Success with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder2.1/3756/

Build Success with Spark 2.3.2, Please check CI http://<IP_ADDRESS>:8080/job/carbondataprbuilder2.3/12032/

Build Failed with Spark 2.2.1, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder1/3968/

Build Success with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder2.1/3824/

Build Success with Spark 2.3.2, Please check CI http://<IP_ADDRESS>:8080/job/carbondataprbuilder2.3/12097/

Build Failed with Spark 2.2.1, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder1/4029/

retest this please

Build Success with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder2.1/3851/

Build Success with Spark 2.3.2, Please check CI http://<IP_ADDRESS>:8080/job/carbondataprbuilder2.3/12124/

Build Failed with Spark 2.2.1, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder1/4056/

retest this please

Build Success with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder2.1/3853/

Build Success with Spark 2.3.2, Please check CI http://<IP_ADDRESS>:8080/job/carbondataprbuilder2.3/12126/

retest this please

Build Success with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder2.1/3855/

Build Success with Spark 2.2.1, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder1/4060/

Build Failed  with Spark 2.3.2, Please check CI http://<IP_ADDRESS>:8080/job/carbondataprbuilder2.3/12128/

retest this please

Build Success with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder2.1/3859/

Build Success with Spark 2.3.2, Please check CI http://<IP_ADDRESS>:8080/job/carbondataprbuilder2.3/12132/

Build Failed with Spark 2.2.1, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder1/4064/

retest this please

Build Success with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbonPRBuilder2.1/126/

Build Failed with Spark 2.2.1, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbonPRBuilder2.2/120/

Build Failed  with Spark 2.3.2, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbonPRBuilder2.3/125/

Build Success with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbonPRBuilder2.1/128/

Build Failed with Spark 2.2.1, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbonPRBuilder2.2/122/

Build Success with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbonPRBuilder2.1/129/

Build Failed with Spark 2.2.1, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbonPRBuilder2.2/123/

Build Failed  with Spark 2.3.2, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbonPRBuilder2.3/128/

Build Failed  with Spark 2.3.4, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbon_PR_Builder_2.4.4/27/

Build Failed  with Spark 2.4.4, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbon_PR_Builder_2.4.5/831/

Build Failed  with Spark 2.3.4, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbonPRBuilder2.3/2538/

Build Failed  with Spark 2.4.4, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbon_PR_Builder_2.4.5/832/

Build Failed  with Spark 2.3.4, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbonPRBuilder2.3/2540/

Build Failed  with Spark 2.4.4, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbon_PR_Builder_2.4.5/833/

Build Failed  with Spark 2.4.4, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbon_PR_Builder_2.4.5/834/

Build Failed  with Spark 2.3.4, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbonPRBuilder2.3/2541/

Build Success with Spark 2.4.4, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbon_PR_Builder_2.4.5/851/

Build Success with Spark 2.3.4, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbonPRBuilder2.3/2558/

@jackylk please review it.

retest this please

Build Failed  with Spark 2.4.5, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbon_PR_Builder_2.4.5/1184/

Build Failed  with Spark 2.3.4, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbonPRBuilder2.3/2900/

rebased

Build Success with Spark 2.4.5, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbon_PR_Builder_2.4.5/1186/

Build Success with Spark 2.3.4, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbonPRBuilder2.3/2903/

LGTM


Enabling latency and packet loss simulation appears to ignore packet loss.

It appears from what I seen in the function UpdateLogic that ProcessDelayedPackets will handle received message, but skiping the packet loss computation. Am I right? A bit hard to say because it depends by the current config (manual/auto/native). However in this case is auto no native.

this seems to be fixed with version 1.2.0, apparently I was using an old version from OUPM


All Cookies parse to null

What version of Remix are you using?
2.2.0
Are all your remix dependencies & dev-dependencies using the same version?

[X] Yes

Steps to Reproduce
I've created the following cookie:
export const pgAdminCookie = createCookie('PGADMIN_LANGUAGE', {
  maxAge: 604_800, // one week
});

When attempting to parse it and pull a value from it, I get null.
  const cookieHeader = request.headers.get('Cookie');
  console.debug('DEBUGPRINT[2]: upsell.tsx:17: cookieHeader=', cookieHeader);
  const cookie = await pgAdminCookie.parse(cookieHeader);
  console.debug('DEBUGPRINT[1]: upsell.tsx:18: cookie=', cookie);

I can see via that first console log (of all Cookie headers) that the cookie is indeed in the header:
DEBUGPRINT[2]: upsell.tsx:17: cookieHeader= PGADMIN_LANGUAGE=en;

Expected Behavior
I expect that second console.log (of the parsed cookie) to log en.
Actual Behavior
It logs null.
The same is true for any cookie I try to log. They all show up in request.headers.get('Cookie');, but all parse to null;
Perhaps some weird async thing? My loader is async, I am awaiting the parse.

Looks like all cookies are expected to be JSON.parse()-able. Otherwise decodeCookieValue() (here):
    return JSON.parse(decodeURIComponent(myEscape(atob(value))));

will throw. Perhaps we should check if the cookie contains something that resembles JSON? Cookies often contain strings, or encoded tokens/etc.


cmd/go: module loading should not silently replace nonexistent versions with valid ones

What version of Go are you using (go version)?

go version go1.12.3 darwin/amd64

Does this issue reproduce with the latest release?
yes
What operating system and processor architecture are you using (go env)?
go env Output
$ go env
❯ go env
GOARCH="amd64"
GOBIN=""
GOCACHE="/Users/andig/Library/Caches/go-build"
GOEXE=""
GOFLAGS=""
GOHOSTARCH="amd64"
GOHOSTOS="darwin"
GOOS="darwin"
GOPATH="/Users/andig/go"
GOPROXY=""
GORACE=""
GOROOT="/usr/local/Cellar/go/1.12.3/libexec"
GOTMPDIR=""
GOTOOLDIR="/usr/local/Cellar/go/1.12.3/libexec/pkg/tool/darwin_amd64"
GCCGO="gccgo"
CC="clang"
CXX="clang++"
CGO_ENABLED="1"
GOMOD="/Users/andig/htdocs/tf/go.mod"
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"
PKG_CONFIG="pkg-config"
GOGCCFLAGS="-fPIC -m64 -pthread -fno-caret-diagnostics -Qunused-arguments -fmessage-length=0 -fdebug-prefix-map=/var/folders/73/89ycv7qn51j4kbm04jsz9b840000gn/T/go-build752630872=/tmp/go-build -gno-record-gcc-switches -fno-common"

What did you do?

Trying to require a specific branch:
go mod init tf
go mod edit -require github.com/containous/alice@containous-fork
go mod edit -require github.com/containous/traefik@master

What did you expect to see?
Dependencies added to go.mod
What did you see instead?
Error:
go: errors parsing go.mod:
/Users/andig/htdocs/tf/go.mod:5: invalid module version "containous-fork": version must be of the form v1.2.3

for this go.mod:
module tf

go 1.12

require github.com/containous/alice containous-fork

If the branch for is invalid it should probably not be allowed to add it using go mod edit. Because doing so will silently override the dependency when doing a go run instead of the 2nd go mod edit which is highly non-obvious.

go mod edit is not supposed to do any semantic validation at all: its specific purpose is to edit the go.mod file without ensuring consistency. (Contrast #30513 and #28439.)
If you want to add a specific version of a dependency, the command to use for that is go get.

Was there a specific piece of documentation that led you to go mod edit? If so, perhaps we can make that documentation clearer; otherwise, this seems to be working as designed.

Was there a specific piece of documentation that led you to go mod edit?

None really, just SO.
But as for the second point, even if go mod is wrong- should go run silently override the requirement (or does it only do so for invalid entries)?

In general all of the go subcommands do resolve missing dependencies, and they upgrade modules as necessary to satisfy packages that are not present in the selected version.
On the other hand, if the existing version was completely invalid (not just too early to satisfy the necessary imports, we should probably diagnose that explicitly rather than silently “upgrading” from the invalid version to a valid one.

(CC @jayconrod for cmd/go diagnostics)

Closing this one as no longer a priority. Thanks for your support.


Migrate from Urbit monorepo

Pre-migration checklist

[x] linux-x86_64: clean build
[x] linux-x86_64: boot fake ship
[x] linux-x86_64: boot comet
[x] macos-arm64: clean build
[x] macos-arm64: boot fake ship
[x] macos-arm64: boot comet
[x] macos-x86_64: clean build
[x] macos-x86_64: boot fake ship
[x] macos-x86_64: boot comet
[x] Review and merge #21.
[x] Perform a dummy release and ensure the resulting binaries work on all four supported platforms.

[x] linux-x86_64
[x] macos-arm64
[x] macos-x86_64

[x] Grab any additional commits to next/vere in urbit/urbit that have landed since 71d06e544f7fe641ac0a9dad1fe07576761227ec.
[x] Review and update documentation to remove references to non-runtime work.
[x] Prepare updates to binary links throughout urbit.org
[ ] Complete final review of old build system to ensure all features have been ported over.

Post-migration checklist

[ ] Remove all runtime-related files and references in urbit/urbit.
[ ] Clearly communicate to runtime developers that they need to file an issue against this repo for anything that appears broken in the new repo.
[ ] Collect list of open runtime-related PRs in urbit/urbit and re-open them in this repo.

Closing this since we've now successfully released out of this repo. Final cleanup is tracked by #125.

How hard would it be to port all relevantly tagged issues out of the monorepo, as happened the other way when the vere and arvo repos stopped being separate? And how many years ETA on these ending up folded back together again?


Fix base charts missing gw crd

Please provide a description of this PR:
After https://github.com/istio/istio/pull/43267 get merged, the CRD is removed, which results in
error: resource mapping not found for name: "namespace-waypoint" namespace: "" from "STDIN": no matches for kind "Gateway" in version "gateway.networking.k8s.io/v1beta1"
ensure CRDs are installed first

while applying the resource.
Add back the gateway CRD by syncing it from the repo.
To help us figure out who should review this PR, please put an X in all the areas that this PR affects.

[x] Ambient
[ ] Configuration Infrastructure
[ ] Docs
[ ] Installation
[ ] Networking
[ ] Performance and Scalability
[ ] Policies and Telemetry
[ ] Security
[ ] Test and Release
[ ] User Experience
[ ] Developer Infrastructure

Please check any characteristics that apply to this pull request.

[x] Does not have any user-facing changes.

This is working as intended, users are expected to install the CRD on their own. Many platforms will have the CRDs so it's not a good idea for us to install them since we will conflict.


feat: NO-JIRA add types to themes

Add types to themes
Obligatory GIF (super important!)

:hammer_and_wrench: Type Of Change
These types will increment the version number on release:

[x] Feature

:book: Description

Added types to /themes/* entries.

:bulb: Context
Added themes/* entries recently but haven't added types so TS was missing types for them.
(Tested on a local TS Vue 3 project)
:pencil: Checklist
For all PRs:

[x] I have ensured no private Dialpad links or info are in the code or pull request description (Dialtone is a public repo!).
[x] I have reviewed my changes.

:crystal_ball: Next Steps

Release and test.

Merging so alex can test.


The repo size reported with "harbor list" command doesn't match the actual size in Harbor registry

After running Garbage collect, using harborclient , I ran "harbor list" and totaled up the size of all repository tags listed = 164GB.   The actual size of the harbor registry repositories = 847GB.
Why the diff of about 700GB

Could you please provide the steps to reproduce?

Installed harbor-client on the harbor-docker-registry  server.
Ran  command "harbor list"  and output to file.
Imported file in step 2 to excel spreadsheet to add up all image tag file sizes listed in the output of "harbor list"
Compared the size reported in Excel to  the size report by Harbor-UI


Reusing nodeLinks and symbolLinks

This suggestion has the same settings with and is on top of #54222 : Language server, assuming only few characters are modified and others are remaining untouched. With incremental parser and some additional flags, we are able to distinguish "clean area" and "dirty area" of nodes. With incremental binding, there exists clean and dirty symbols (we are not able to distinguish them yet).
Semantic analysis are performed by the type checker lazily, after they are computed, intermediate or final results are cached by nodeLinks/symbolLinks. Each node/symbol has their own cache, but the caches are actually stored program-wide, nodes/symbols need to query their own by their id.
Whenever a change occurs, type checker applies an aggresive strategy: it assumes all the caches are outdated and recreates nodeLinks and symbolLinks, and we can actually reuse some of them.
The core step of semantic analysis is resolution, which refers to the computation needed to build the connection of a variable reference node and its symbol (declaration). Some cache needs multiple resolution steps (For example, get properties of a deep inherited class). After resolution, some computation is performed to get a type or other stuff. I'll start with those caches that involves one resolution step. Also, when considering performance, I'd like to ignore the overhead of resolution between two files, just theoratically describe what procedure could be bypassed, in order to reduce factors got involved.

Memory leak now becomes a problem, because we no longer clear nodeLinks/symbolLinks. weakMap could be a possible workaround.
clean node, resolves to dirty node/symbol
Type checker accesses the node, finds it has a cache, then finds it points to a dirty node/symbol, so the cache is recomputed.

Now we have a problem: how to tell if the node/symbol is dirty or clean? Dirty nodes/symbols are detached from new AST/symbol tables, we do not have a chance to visit them, but they are not released by those caches.
Here I offer one possible implementation: use file version. Each node/symbol would have a "version" member. During incremental parsing phase, initialize/refresh version members of each node to the latest, incremental parsing stage is similar. Note that the dirty flag I mentioned in #54222 could be removed. Then, in type checker, when it finds a cache is available, compare its version to the file version.
I am not sure if it is easy to implement, because it means each cache entry access (specifically, each key access in xxlinks) needs to be modified. As far as I know, they should be updated one by one because there are tons of code patterns. Anyway, I'm theoretically discussing the idea here.
What about multi-step resolution
If we want to reuse multi-step resolution cache, then we need to store more information about the chain in the node containing the cache, which may not be a good idea. Even a simpler example that a node points to a type with one step resolution is not viable, because we can't tag the version of the cached type.
However, for those scenarios, I'd like to compare them with our baseline, the existing strategy: clear everything, recompute anything when you need it. Suppose for these scenarios, we just choose to recompute it, we can still reuse those one-step resolutions, which is better than reusing nothing at least.
Therefore, the complete suggestion is: identify those cache entry that both cache holder and cached target could be simply versioned (For example, NodeLinks.resolvedSymbol and try to reuse them. For others, we can still keep the same strategy.
Tradeoff of memory usage
Note that dirty cached target is not released until type checker visits it again, so it may introduces higher memory usage. However, I think the tradeoff is minor, because we can have other strategies, such as performing a nodeLink/symbolLink scan every 200 versions and removing stale nodes/symbols.
Maybe starting from files
The idea is not complicated, but the codebase is, so I'd suggest an alternative:
We can start from reusing nodeLinks/symbolLinks between clean files. Files not get binded are indentified as "clean area" and rebinded files are "dirty area", for such granularity, it is already there.

I'm not sure this is quite feasible. The results computed from symbol links and node links do not necessarily have the same meaning in a future program that has edits, even if those edits are outside the current file.
The other problem is that most of these node links and symbol links are not actually very computationally intensive. So in an interactive scenario, which is the only place you could do incremental updates, the amount of work being done here is actually quite minimal.
We could take a PR to examine it and see if there are meaningful gains, but I don't expect to be able to find much here.

Sure. I don't know how to benchmark a language server (do you have any guides? I only find performance tracing for tsc in the wiki.
However, I did try to observe (in a naive way) the performance. Say I am clicking a random function in compiler/binder.ts and click go to references in vsc, then I can feel lag (0.1s? 0.2s? I have no idea, not so much lag). For the second time I do the same thing, it pops up in a flash. If I randomly insert some characters in another file (to clear the cache), the lag comes back again.
I am using a 2.4 GHz 8-Core Intel i9 Macbook pro with 32GB RAM, but the test is running within a devcontainer with 6 cores and 24G RAM. Anyway, I assume my development environment's performance is above average, but I have no idea whether or not it is something worth to work on.
Absolutely it is naive, if you can provide some reference of what is identified as "notable performance optimization" then maybe I can do something more standard in advance 😄

If you're just performance benchmarking tsc, then all our perf suite does is run tsc --extendedDiagnostics. If you are benchmarking tsserver requests, then that can be orchestrated with https://www.npmjs.com/package/@typescript/server-harness.


📎 Move custom colors into MUI palette

Description
We currently have quite a few hardcoded hex values throughout the codebase.
I believe moving as many of these into the palette object could

help reduce duplication and
provide an opportunity to give appropriate and meaningful names to colors. (E.g. we currently use MUI's error color as the color to highlight trending videos — this semantic disconnect can be confusing to follow.)

This page looks like it could help with implementing this change.

Agree, could also avoid typo