Downloads wont start

Task exception was never retrieved
future: <Task finished name='Task-162' coro=<AutoFC2.handle_channel() done, defined at C:\Users\leonm\AppData\Local\Programs\Python\Python311\Lib\site-packages\fc2_live_dl\autofc2.py:116> exception=<InvalidURL >>
Traceback (most recent call last):
  File "C:\Users\leonm\AppData\Local\Programs\Python\Python311\Lib\site-packages\fc2_live_dl\autofc2.py", line 119, in handle_channel
    await fc2.download(channel_id)
  File "C:\Users\leonm\AppData\Local\Programs\Python\Python311\Lib\site-packages\fc2_live_dl\FC2LiveDL.py", line 175, in download
    async with self._session.get(thumb_url) as resp:
  File "C:\Users\leonm\AppData\Local\Programs\Python\Python311\Lib\site-packages\aiohttp\client.py", line 1141, in __aenter__
    self._resp = await self._coro
                 ^^^^^^^^^^^^^^^^
  File "C:\Users\leonm\AppData\Local\Programs\Python\Python311\Lib\site-packages\aiohttp\client.py", line 508, in _request
    req = self._request_class(
          ^^^^^^^^^^^^^^^^^^^^
  File "C:\Users\leonm\AppData\Local\Programs\Python\Python311\Lib\site-packages\aiohttp\client_reqrep.py", line 305, in __init__
    self.update_host(url)
  File "C:\Users\leonm\AppData\Local\Programs\Python\Python311\Lib\site-packages\aiohttp\client_reqrep.py", line 364, in update_host
    raise InvalidURL(url)
aiohttp.client_exceptions.InvalidURL

after starting the command autofc2 --config autofc2.json and a stream is found it will just loop through this and wont start anything.
Python version: 3.11
fc2-live-dl version: 2.2.0

The URL did work the last time just today it wont work

is it possible that the error happens because it tries to get the .png but there is no png and so it fails?

Seems to be an issue downloading thumbnails, you can disable it for now while I work on a fix. Set write_thumbnail to false

ok thx


Closing transports?

I've been working on adding configuration support to greeneye-monitor. One of my test programs does the following:

Connects to the GEM and instructs it to send packets to the machine running the test program
Creates a server to listen for packets
Upon interrupt, close that server
Connect to the GEM again and instruct it to send packets to my Home Assistant instance

As currently written, each of those "connects to the GEM" steps involves doing a loop.create_connection with a new instance of the protocol and sending a serial number request to it.
This is all horribly inefficient, but it did expose an issue that I think needs dealing with.
Here's what the logs look like for steps 3 and 4:
^C2021-12-31 15:19:57,721 [greeneye.monitor](INFO) Closing server on ('<IP_ADDRESS>', 8000)
2021-12-31 15:19:57,724 [greeneye.monitor](DEBUG) queue consumer is getting canceled
2021-12-31 15:19:57,737 [siobrultech_protocols.gem.protocol](INFO) Connection opened
2021-12-31 15:19:57,738 [siobrultech_protocols.gem.protocol](DEBUG) Starting API request. Requesting packet delay...
2021-12-31 15:20:00,743 [siobrultech_protocols.gem.protocol](DEBUG) Sending API request '^^^RQSSRN'...
2021-12-31 15:20:00,771 [siobrultech_protocols.gem.protocol](DEBUG) Received 10 bytes
2021-12-31 15:20:00,772 [siobrultech_protocols.gem.protocol](DEBUG) Received 10 bytes
2021-12-31 15:20:00,772 [siobrultech_protocols.gem.protocol](DEBUG) No header found. Discarding junk data: bytearray(b'XXXXXXXX\r\n')
2021-12-31 15:20:03,747 [siobrultech_protocols.gem.protocol](DEBUG) Received API response: 'XXXXXXXX
'
2021-12-31 15:20:03,747 [siobrultech_protocols.gem.protocol](DEBUG) Ended API request

Even though the server was closed, the established connection remains. Because of the way the network connection works on the GEM, the response for any request is sent to both connections. As a result, we get one protocol instance complaining about a malformed packet while the other one is handling an expected API response. The only impact of this for the test program was some janky debug logging, but in Home Assistant I could imagine it throwing exceptions all over the place after attempting to unload the component because it's trying to update entities that should be dead.
Now, all of this is expected per the docs for Server.close... when you close the server, existing connections remain open. The way you close existing connections is by closing the corresponding Transports.
Which at last brings me to the reason I filed this issue. What's the cleanest way for us to find and close those transports? I've subclassed BidirectionalProtocol already, so I could do this in my library, but I'm wondering if this is something that should somehow be handled here.
Thoughts?

Hmm, this is certainly something I do not need to handle with serial transports (at least, in practice, since they never go away).

Because of the way the network connection works on the GEM, the response for any request is sent to both connections.

This is kinda crazy, but I understand why it does that.

What's the cleanest way for us to find and close those transports?

I'm not sure I have a good answer here, to be honest.  Any solution we'd do here would be transport and connection-specific, right?

Given that the server is outside the scope of this library, we certainly can't do anything automatic. If we do nothing here, what I'll probably end up doing is having my protocol subclass track its transport and work with the server to keep a list of protocol instances that have valid transports so that the server can close them all. The potentially generalizable parts of that might look like having PacketProtocol expose its transport (or a close method) and send messages to the queue when it gets or loses its connection (so that an external entity can track active protocol instances).

Would it be a 1:1 mapping of PacketProtocol to a transport, or is it re-used?

Would it be a 1:1 mapping of PacketProtocol to a transport, or is it re-used?

The asyncio docs are not explicit about that, but there are a few places
where they imply that a protocol is created per transport.
On Fri, Dec 31, 2021 at 4:29 PM Shawn Wilsher @.***>
wrote:

Would it be a 1:1 mapping of PacketProtocol to a transport, or is it
re-used?
—
Reply to this email directly, view it on GitHub
https://github.com/sdwilsh/siobrultech-protocols/issues/48#issuecomment-1003473873,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABTZNWJ6SK5L3DVPGKTC5HLUTZDHRANCNFSM5LB2MC6A
.
You are receiving this because you authored the thread.Message ID:
@.***>

The asyncio docs are not explicit about that, but there are a few places
where they imply that a protocol is created per transport.
On Fri, Dec 31, 2021 at 4:29 PM Shawn Wilsher @.***>
wrote:

Would it be a 1:1 mapping of PacketProtocol to a transport, or is it
re-used?
—
Reply to this email directly, view it on GitHub
https://github.com/sdwilsh/siobrultech-protocols/issues/48#issuecomment-1003473873,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABTZNWJ6SK5L3DVPGKTC5HLUTZDHRANCNFSM5LB2MC6A
.
You are receiving this because you authored the thread.Message ID:
@.***>

I think it's reasonable then for us to expose something here.

I think it's reasonable then for us to expose something here.

Ok, I'll work on it.


WriteEeprom issue

Hello,
Thanks for your work, its fantastic!
I got this error while programming my Central Lock Module (1C0962258N) on Win7 32bit with kkl vag com usb cable:
Version 0.71-beta (https://github.com/gmenounos/kw1281test/releases)
Args: com2 9600 35 WriteEeprom 4352 07
OSVersion: Microsoft Windows NT 6.1.7601 Service Pack 1
.NET Version: 6.0.2
Culture: hu-HU
Opening serial port com2
Sending wakeup message
Reading sync byte
Keyword Lsb $01
Keyword Msb $8A
Protocol is KW 1281 (8N1)
ECU: 1C0962258N  0A Zentr.Verriegel. 0001
Software Coding 00257, Workshop Code: 00066
Sending Login block
Sending WriteEeprom block (Address: $1100, Values: $07
WriteEeprom failed
Sending EndCommunication block
I have tried with another module (1C0962258J), and with another serial converter in the cable (ftdi and ch340G) the same error.
ReadingEeprom, DumpEeprom works well with every module and cable.

Were you able to read address 4352 with kw1281test?

Thanks for your reply,
Here are the log of the read, write attempts.
 I wonder if the central locking module (address 35) needs a different login code to write to the EEPROM. kw1281test uses 19283 to login for reading and writing the CCM (address 46). Unfortunately, I don't have access to a central locking module in order to do any research. The MKIV cars here in the US seem to all use a CCM.

Can I somehow try other login codes?

There are 65536 login codes so it would take days to try them all. I wonder if the region you're trying to write to is ROM instead of EEPROM. I have a 1C0959799C CCM here and it has a 512 byte region of RAM at address $1000 and a 4096 byte region at address $9000 that does not seem to be writable.
Could you try running the MapEEPROM command? It tries to read every possible address and builds a map of what regions are readable. It might take a while to run though.

Thank you for your reply!
This is the map of 1C0962258N.
https://drive.google.com/file/d/1SIM0SoZk8P7A0L7xas-AA7acl39sNQZy/view?usp=sharing
I have another one 1co962258J i will read it too.

That's a very different map than one from a 1C0 CCM. Do you know for certain that the address you're trying to write is EEPROM and should be writable? Or are you experimenting?

I'am only experimenting. I would like to modify the values for selective boot unlocking, for pop up boot.

This is the map of 1c0962258J. The program generated .bin full with FF again. 1c0962258J.txt


[BUG]: Issues with wolfram command...

Bug Report
Describe the bug
The wolfram command Is giving unexpected errors.
However, since I had a plan of improving the command, maybe it would make sense to also rework the command while it's fixed.

Am I willing to work on this?:

[x] yes

Screenshots

This can be closed, since 98960af deprecated the wolfram command


Facebook not show tags!

That's because Facebook will not read dynamic tags. They don't process JS at all. So if you want to get your OG tags to work, they have to be generated on the server. Setting them on the front end will work if you inspect the markup, but will not make any difference to Facebook. Facebook, unfortunately, only looks at the initial response.

@leomask101193  You can try to use netlify for hosting they have prerendering along with other cool stuff

Check out this package, it's a Webpack plugin that allows you to pre-render the component.
https://github.com/chrisvfritz/prerender-spa-plugin
I just had the same problem and was able to fix it with this. Cheers.


[ListView.builder] calling itemBuilder unnecessarily

Hello guys, I have a doubt about ListView.builder().
I am developing an application, and there are some lists with sublists, but I couldn't understand a behavior in these cases.
I thought this behavior just happened in my app, but I tried simulating it in a new app and it happened too.
In this case, I have the first ListView and when I scroll him, it always calls the item builder. Okay, I got it, and it happened because the ListView.builder() works on demand (lazily mode), but if I open another ListView.builder() in another Scaffold() and I close the keyboard opened by TextFieldForm, the itemBuilder of the first ListView is called... I didn't understand this behavior and I want to know if this behavior is correct.
Guys, it is a example app, but in my oficial app, it is called the backend endpoints, the ListView needs to load some widgets but it's hidden, I don't see that need. I have tried use the cacheExtent property, and other properties in the documentation, but it happens too.

Steps to Reproduce

Execute flutter run on the code sample
...
...

Expected results:
Calling itemBuilder from the hidden list
Actual results:
Don't call itemBuilder from the hidden list

Code sample

import 'package:flutter/material.dart';

void main() {
  runApp(const MyApp());
}

List<String> valuesListViewOne = ['1', '2', '3', '4', '5', '6', '7', '8', '9', '10', '11', '12', '13', '14', '15', '16', '17'];
List<String> valuesListViewTwo = ['1', '2', '3', '4'];

class MyApp extends StatelessWidget {
  const MyApp({Key? key}) : super(key: key);

  // This widget is the root of your application.
  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      title: 'Flutter Demo',
      theme: ThemeData(
        // This is the theme of your application.
        //
        // Try running your application with "flutter run". You'll see the
        // application has a blue toolbar. Then, without quitting the app, try
        // changing the primarySwatch below to Colors.green and then invoke
        // "hot reload" (press "r" in the console where you ran "flutter run",
        // or simply save your changes to "hot reload" in a Flutter IDE).
        // Notice that the counter didn't reset back to zero; the application
        // is not restarted.
        primarySwatch: Colors.blue,
      ),
      home: const MainWidget(),
      routes: {
        'callListViewOne': (_) => const ListViewOne(),
        'callListViewTwo': (_) => const ListViewTwo(),
      },
    );
  }
}

class MainWidget extends StatelessWidget {
  const MainWidget({Key? key}) : super(key: key);

  @override
  Widget build(BuildContext context) {
    return Scaffold(
      appBar: AppBar(
        title: const Text('Main view'),
      ),
      body: Center(
        child: IconButton(
          iconSize: 35,
          color: Colors.black,
          icon: const Icon(Icons.add),
          onPressed: () => Navigator.of(context).pushNamed('callListViewOne'),
        ),
      ),
    );
  }
}

class ListViewOne extends StatelessWidget {
  const ListViewOne({Key? key}) : super(key: key);

  @override
  Widget build(BuildContext context) {
    return Scaffold(
      appBar: AppBar(
        title: const Text('List View One'),
      ),
      body: Padding(
        padding: const EdgeInsets.all(20),
        child: Column(
          children: [
            TextFormField(),
            const Divider(),
            Expanded(
              child: ListView.builder(
                itemCount: valuesListViewOne.length,
                itemBuilder: (ctx, idx) {
                  return Container(
                    decoration: const BoxDecoration(
                      color: Colors.yellow,
                    ),
                    width: double.infinity,
                    height: 150,
                    child: Center(
                      child: Text(
                        valuesListViewOne[idx],
                      ),
                    ),
                  );
                },
              ),
            ),
          ],
        ),
      ),
      floatingActionButton: Container(
        height: 60,
        width: 60,
        decoration: const BoxDecoration(
          color: Colors.blue,
          shape: BoxShape.circle,
        ),
        child: IconButton(
          iconSize: 35,
          color: Colors.black,
          icon: const Icon(Icons.add),
          onPressed: () => Navigator.of(context).pushNamed('callListViewTwo'),
        ),
      ),
    );
  }
}

class ListViewTwo extends StatelessWidget {
  const ListViewTwo({Key? key}) : super(key: key);

  @override
  Widget build(BuildContext context) {
    return Scaffold(
      appBar: AppBar(
        title: const Text('List View Two'),
      ),
      body: Padding(
        padding: const EdgeInsets.all(20),
        child: Column(
          children: [
            TextFormField(),
            const Divider(),
            Expanded(
              child: ListView.builder(
                itemCount: valuesListViewTwo.length,
                itemBuilder: (ctx, idx) {
                  return Container(
                    decoration: const BoxDecoration(
                      color: Colors.yellow,
                    ),
                    width: double.infinity,
                    height: 150,
                    child: Center(
                      child: Text(
                        valuesListViewTwo[idx],
                      ),
                    ),
                  );
                },
              ),
            ),
          ],
        ),
      ),
    );
  }
}

Logs

it doesn't appearance logs

Analyzing two_listview_test...
No issues found! (ran in 7.3s)

[√] Flutter (Channel stable, 2.10.4, on Microsoft Windows [versÃ£o 10.0.19044.1645], locale pt-BR)
    • Flutter version 2.10.4 at C:\flutter\flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision c860cba910 (7 weeks ago), 2022-03-25 00:23:12 -0500
    • Engine revision 57d3bac3dd
    • Dart version 2.16.2
    • DevTools version 2.9.2

[√] Android toolchain - develop for Android devices (Android SDK version 31.0.0-rc5)
    • Android SDK at D:\AndroidSDK
    • Platform android-31, build-tools 31.0.0-rc5
    • Java binary at: C:\Program Files\Android\Android Studio\jre\bin\java
    • Java version OpenJDK Runtime Environment (build 11.0.8+10-b944.6842174)
    • All Android licenses accepted.

[√] Chrome - develop for the web
    • Chrome at C:\Program Files (x86)\Google\Chrome\Application\chrome.exe

[√] Visual Studio - develop for Windows (Visual Studio Community 2022 17.1.4)
    • Visual Studio at C:\Program Files\Microsoft Visual Studio\2022\Community
    • Visual Studio Community 2022 version 17.1.32407.343
    • Windows 10 SDK version 10.0.19041.0

[√] Android Studio (version 4.2)
    • Android Studio at C:\Program Files\Android\Android Studio
    • Flutter plugin can be installed from:
       https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
       https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 11.0.8+10-b944.6842174)

[√] VS Code (version 1.67.1)
    • VS Code at C:\Users\Vinicius Sehnem\AppData\Local\Programs\Microsoft VS Code
    • Flutter extension version 3.40.0

[√] Connected device (5 available)
    • SM A205G (mobile)                  • RQ8M60RWDDX   • android-arm64  • Android 11 (API 30)
    • Android SDK built for x86 (mobile) • emulator-5554 • android-x86    • Android 10 (API 29) (emulator)
    • Windows (desktop)                  • windows       • windows-x64    • Microsoft Windows [versÃ£o 10.0.19044.1645]
    • Chrome (web)                       • chrome        • web-javascript • Google Chrome 101.0.4951.54
    • Edge (web)                         • edge          • web-javascript • Microsoft Edge 101.0.1210.32

[√] HTTP Host Availability
    • All required HTTP hosts are available

• No issues found!

https://user-images.githubusercontent.com/40151305/167963586-3aac6624-3c9e-4f21-88f4-5c586206c49e.mp4

@sehnemvinicius
Thanks for the report. If you are looking for correct code implementation or what should be the expected result, I suggest you to ask this question on support channels like StackOverflow to get a proper solution.
If you think this is a bug in the way how ListView.builder works and isn't an expected behavior with itemBuilder, write in comments and I'll reopen it and consider as a bug in the framework.
Closing for now. If you disagree, write in comments and I'll reopen.


Backport some production fixes

Backports some fixes I made on our internal 8.1 compatible cakephp core.
PHP8.1 has stricter checks on passing null to non nullable internal functions
PHP8.0 deprecated nullable args before required ones

Thanks for your fixes, @vincentreitz!
Unfortunately I had to revert the commit, because the FormHelper test was failing:

$ ./cake/console/cake testsuite core case libs/view/helpers/Form
Welcome to CakePHP v1.3.21 Console
App : app
Path: /home/travis/build/littleant/cakephp-1.3.21/app
CakePHP Test Shell
Running core case libs/view/helpers/form
FAIL->Item #3 / regex #1 failed: Attribute "name" == "data[Model][0][OtherModel][field]", Attribute "id" == "Model0OtherModelField" at [/home/travis/build/littleant/cakephp-1.3.21/cake/tests/cases/libs/view/helpers/form.test.php line 5418]
in testTextArea
in FormHelperTest
in /home/travis/build/littleant/cakephp-1.3.21/cake/tests/cases/libs/view/helpers/form.test.php
1/1 test cases complete: 455 passes, 1 fails.
Time taken by tests (in seconds): 0.50215601921082
Peak memory use: (in bytes): 12,809,472
The command "./cake/console/cake testsuite core case libs/view/helpers/Form" exited with 1.

Do you have an idea why this might be the case?


Is there any plan to support Google Cloud Storage

Hi,
Is there any plan to support Google Cloud Storage
Thank you in advance

Hi @majbilou, here are the requirements for the underlying storage system in order to maintain ACID guarantees:
https://github.com/delta-io/delta#requirements-for-underlying-storage-systems
HDFS over Google Cloud will work out of the box; I'm not familiar enough with Google Cloud Storage to know if they have these guarantees. If they don't, writing a plugin to support Google Cloud Storage isn't on our immediate roadmap, but we'd be happy to point you to the AWS and Azure implementations to show how they solve these problems.

Thank you @mukulmurthy for the feedback


[WIP] Only test/deploy website if relevant files are changed

Summary
Based on @thymikee
https://github.com/facebook/jest/issues/6567#issuecomment-401723101

Even better if it could be scoped to detected changes in docs/ dir <3

Test plan

Test logic.
Consider this script

WEBSITE_CHANGED=$(git diff-tree --no-commit-id --name-only -r HEAD | grep -E -c "(^docs\/.*)|(^website\/.*)")
if [[ $WEBSITE_CHANGED == 0 ]]; then
    echo "No relevant website files has changed"
    exit 0;
else
    echo "Relevant website files has changed"
fi

Change in docs/website folder commit

No change in docs/website folder commit

Look at circleci

@thymikee still need refactor & work in progress (Still failing as well).
I think I'll create a website.sh.

This is ready for review. I don't think the failing travis is related to this PR though
Even #6623 fails on travis

Do not merge yet. The set -e is causing some bugs 😢

Alright this should be ok now. Had a small bug due to set -e causing auto exit on grep before

Codecov Report

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

@@           Coverage Diff           @@
##           master    #6626   +/-   ##
=======================================
  Coverage   63.73%   63.73%
=======================================
  Files         235      235
  Lines        8935     8935
  Branches        4        3    -1
=======================================
  Hits         5695     5695
  Misses       3239     3239
  Partials        1        1

Continue to review full report at Codecov.

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


Support for HTML formatted content in JMX operation result to resolve #104

fix #104
@abkieling @tadayosi please kindly take a look.
Below are the samples how is is looking after the fix:

@ostrovskyde Thanks for your contribution!  Here are a couple of feedback before merging it.

Treating every result as trusted is considered unsecure. Ideally we should apply ng-bind-html only for html results, but I understand it's difficult to automatically determine if it's html or plain text. What about the idea of adding an extra checkbox ("Print as HTML") right to the Execute button and let the user decide that?

Allowing rendering HTML with JS (your 2nd sample) seems risky as well. We should limit rendering of external resource to pure HTML for minimising risks such as XSS and CSRF. What about using ngSanitize instead of $sce.trustAsHtml?  My understanding is that once ngSanitize is included as follows you can directly use ng-bind-html="$ctrl.operationResult" without the filter and js is automatically sanitised.
 export const operationsModule = angular
   .module('hawtio-jmx-operations', ['ngSanitize'])

@tadayosi thanks for your quick response!
Reg first item I agree to add "Print as HTML" checkbox.
Reg second item, JS and styles are necessary for us, but ngSanitize will remove them. We trust the source of HTML and JS inside it. Do you have an idea how to allow JS and styles for certain users on a permanent basis?

Reg second item, JS and styles are necessary for us, but ngSanitize will remove them. We trust the source of HTML and JS inside it. Do you have an idea how to allow JS and styles for certain users on a permanent basis?

With the fix we cannot only trust the sources from our own MBeans. It also forces us to trust unknown MBeans as well. Hawtio project takes security seriously so we have to avoid introducing such potential vulnerabilities carefully.
I wonder why you need such a rich UI on the JMX operation results.  Basically the JMX plugin only supports generic features of invoking MBeans operations and viewing attributes. If you need more customised experiences on some specific MBeans then you should create your own custom plugin. There you can provide a fully customised Web UI for those results.

I wonder why you need such a rich UI on the JMX operation results. Basically the JMX plugin only supports generic features of invoking MBeans operations and viewing attributes. If you need more customised experiences on some specific MBeans then you should create your own custom plugin. There you can provide a fully customised Web UI for those results.

@tadayosi We just use some primitive styles for table formatting, writing a custom plugin just for this purpose with copy-pasting Hawtio JMX plugin sources looks pretty weird.
Would you mind if I add those security settings in Hawtio Jolokia preferences: showResultAsHtml and trustHtmlSource checkboxes? By default they will be set to false. So in case user accepts the risk then configures this options explicitly.
Would it be ok?

I'd like to suggest considering two alternative approaches:

format your table with pipes and dashes (example below)
format your table using Markdown and add the logic that converts that to HTML in Hawtio (similar to GitHub comments)

@tadayosi @abkieling please take a look at the option with HTML settings in Hawtio preferences:

HTML and JS are disabled (default setting):

HTML enabled without JS and CSS:

HTML enabled with JS and CSS:

@ostrovskyde One more suggestion. I quickly reviewed existing MBean operations and I only found that Apache Artemis has such operations. Artemis names those operations with suffix ...AsHTML, e.g. listMessageCounterAsHTML. As Print as HTML checkbox doesn't make sense for most operations, why don't we just show the checkbox when the operations follow the naming convention and has suffix AsHTML?  It would further minimise the impact of the feature with very little to lose.

@ostrovskyde One more suggestion. I quickly reviewed existing MBean operations and I only found that Apache Artemis has such operations. Artemis names those operations with suffix ...AsHTML, e.g. listMessageCounterAsHTML. As Print as HTML checkbox doesn't make sense for most operations, why don't we just show the checkbox when the operations follow the naming convention and has suffix AsHTML? It would further minimise the impact of the feature with very little to lose.

Hey Tadayoshi,
Would you accept the idea of detecting HTML content by <!DOCTYPE html> declaration? Renaming of JMX methods to "*asHtml" pattern will affect lots of our MBeans and is quite inaccurate.
Reg JS

you said it's needed for a copy&paste problem.
No, you got me wrong. You suggested to create a custom plugin for that purpose, but I just meant that we cannot reuse Hawtio JMX plugin logic - that is the copy-paste issue.
Let's now consider only CSS without JS because its hard to produce accurate HTML without styles. How we can proceed with it with always applying ngSanitize?

we cannot reuse Hawtio JMX plugin logic - that is the copy-paste issue

You don't need to 100% replicate the JMX plugin. If you have a known set of MBeans and operations that you expect to return HTML, then your custom plugin can just deal with the Jolokia invocations for those, and then you can apply your own HTML rendering logic.

Would you accept the idea of detecting HTML content by  declaration? Renaming of JMX methods to *asHtml pattern will affect lots of our MBeans and is quite inaccurate.

Does it mean after receiving the result from an operation and initially rendering it as plain text the checkbox can switch between plain text and html rendering without further server-side invocations?  That sounds like a good idea, too. We should then write some html detection logic and as a starter check <!DOCTYPE html> and
 in the result to detect it.
The problem is then where to locate the checkbox. I think it should then be located at the top right corner in the textarea rather than next to Execute button.

No, you got me wrong. You suggested to create a custom plugin for that purpose, but I just meant that we cannot reuse Hawtio JMX plugin logic - that is the copy-paste issue.

My bad.  As James said, you don't need to 100% replicate JMX plugin code. And I see now the intention is to use some custom styles with the output table html. I wonder what if we use the predefined PatternFly css classes for the table instead of embedding custom style definitions yourself.
https://www.patternfly.org/v3/pattern-library/widgets/#tables

      #
      First Name
      Last Name
      Username

      1
      Mark
      Otto
      @mdo

      2
      Jacob
      Thornton
      @fat

      3
      Larry the Bird
      @twitter

Is an html like the above also sanitised with ngSanitize?

Does it mean after receiving the result from an operation and initially rendering it as plain text the checkbox can switch between plain text and html rendering without further server-side invocations?

@tadayosi didn't get you. Why do we need checkbox where HTML auto-detection is in place? You mean checkbox to view HTML as plain text? What is the purpose of this since it could be viewed in the page source or just copied via "clipboard" option?
Maybe just auto-detection without checkbox?

I wonder what if we use the predefined PatternFly css classes for the table instead of embedding custom style definitions yourself.

Let me check this against ngSanitize. Btw, is there an option in Hawtio to load custom static CSS and use it e.g. for table formatting in this particular case?

@tadayosi patternFly styles are working fine with ngSanitize. Lets consider this approach, thank you for suggestion. Please take a look one more time.
Here it is with HTML auto-detection and PatternFly style:

Now custom styles (and JS as well) are being truncated by ngSanitize:

"Copy to clipboard" copies raw HTML source (or plain string respectively) so this place was not changed.

@tadayosi many thanks! Just a quick question, how can I get the updated version for hawtio-springboot module with this change?

@ostrovskyde Your pull req on hawtio/hawtio is already merged. If you want to use it immediately you can build a snapshot version of hawtio-springboot by running mvn clean install on hawtio/hawtio. Also I'll soon prepare Hawtio 2.11.0 release after a few more fixes.


Repo clone page, Truncate long repo name

Dev Home version
<IP_ADDRESS>
Windows build number
10.0.22621.1848
Other software
OS Build Version: 10.0.22621.1848.amd64fre.ni_release.220506-1250
.NET Version: .NET 6.0.16
Steps to reproduce the bug
When you have long repo name in the clone repo page
Expected result
Long repo name should be truncated in the end with "..." and on hover customer should see the full name of the repo
Actual result
The name is currently running long
Included System Information
Physical Memory: 15.95GB (9.72GB free)
Processor Architecture: x64
Included Extensions Information
Extensions:
Microsoft.Windows.DevHomeGitHubExtension_<IP_ADDRESS>_x64__8wekyb3d8bbwe
Microsoft.Windows.DevHome_<IP_ADDRESS>_x64__8wekyb3d8bbwe

@dhoehna


Merge pull request #1 from zephyrproject-rtos/master

update_05062018_5.30pm

Codecov Report

Merging #8194 into master will decrease coverage by <.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #8194      +/-   ##
==========================================
- Coverage   64.53%   64.53%   -0.01%
==========================================
  Files         420      420
  Lines       40101    40101
  Branches     6762     6762
==========================================
- Hits        25881    25879       -2
- Misses      11103    11104       +1
- Partials     3117     3118       +1

Impacted Files
Coverage Δ

boards/posix/native_posix/timer_model.c
98.21% <0%> (-1.79%)
:arrow_down:

samples/philosophers/src/main.c
96.87% <0%> (-1.57%)
:arrow_down:

Continue to review full report at Codecov.

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


Pretrained models (IC50)

Hi, what's the unit of the IC50 predictions in pretrained models? Are they perhaps pIC50 rather than IC50 values?
Actually, is there any documentation about those models that could answer similar questions?
Thanks!

Hi, during training, it is all using pIC50. But you can specify conversion for adding the variable 'convert_y'. Just add a note in the readme!


Did you consider trying to avoid using the grasp data on the wrong voxels?

Hi, thanks for your great work!
I wonder if the training dataset includes those grasp data pairing with those ambiguous TSDF voxels, since you fuse the TSDF only by one single-view depth image, which definitely includes the ambiguous voxels in the occluded area. Intuitively, when you take as input those voxels with corresponding grasp data on them, this seems to affect the performance because of the ambiguous dataset.
Have you ever considered this, or already utilized a few tricks in this work to avoid this? It's important to me.
Thanks a lot.

Hi, we intentionally train our model with grasp data on the voxels that are invisible, because we wanted to explore the synergy between 3D reconstruction and grasp affordance. Our key intuition is that a learned representation capable of reconstructing the 3D scene encodes relevant geometry information for predicting grasp points and vice versa.
In short, we didn't avoid using the grasp data on the ambiguous voxels.

Thank you for answering my question. It's very helpful!


Update pull_request_template.md

Update pull_request_template.md to include requirement for issue title and Javadocs

Codecov Report

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

@@          Coverage Diff           @@
##           master   #1476   +/-   ##
======================================
  Coverage     3.2%    3.2%
======================================
  Files         132     132
  Lines        7109    7109
  Branches      686     686
======================================
  Hits          228     228
  Misses       6866    6866
  Partials       15      15

Continue to review full report at Codecov.

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


AppId and ClientId

The AppId and ClientId are the same, there is no need to get it separately using az ad sp list --show-mine

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

ID: bc43d849-5b06-7185-0c9e-aaf240363aca
Version Independent ID: 385f4dfe-326a-9e63-80ce-3c71903edf3c
Content: Quickstart -  Azure Key Vault client library for .NET
Content Source: articles/key-vault/quick-create-net.md
Service: key-vault
GitHub Login: @msmbaldwin
Microsoft Alias: mbaldwin

@olandese: I verified that "Appld" and "ClientId" are synonymous -- nice catch! The article has been updated to only reference ClientId. Thanks for you feedback. #please-close


Please upgrade to ng4 as soon as possible.

please update ng4 ASAP or update "@angular/common"
"@angular/core" of peerDependencies to more than 4.0 version
Thank

I'm sorry for not updating


Page deduplication drawbacks?

Hey guys,
The page deduplication approach is clever, but it sets a hard limit on the scanner's issue detection capabilities.
Since you're comparing HTML code to determine uniqueness, it guarantees that certain issues exposed via text nodes will not be able to be identified, as those pages will never make it to the underlying scanner.
For example, disclosure of sensitive information such as credit card numbers, SSN, etc.
In addition, it also prevents you from identifying server-side resources, like sensitive files and directories (backup, backdoors, etc.); the URL of a template-generated page whose path can be manipulated into returning a sensitive file will never make it to the scanner.
I can see that right now you're only interested in SQL and XSS issues, but the architecture seems limited to only identifying active issues.
Was that a conscious decision?
Cheers,
Tasos L.

You are right @Zapotek. It is by design.
Gryffin consciously tries not to overwhelm target website. This translates to less pages crawled due to deduplication. Hence, Gryffin is not suitable for content scanning.

The crawl occurs anyways since the deduplication happens based on HTML or some other criteria that first require the page to be retrieved, you're just skipping the audit (i.e. passing the page to the fuzzers).
Right?

Yes @Zapotek. We do not crawl further from the duplicate page and the page/url itself is not sent to the fuzzer queue.

The problem with this approach is that for a page to be marked as a duplicate (and thus allow for the benefits of preventing further processing), it either needs to be a leaf on the site's tree, since it will have no discernible new paths, so you've already reached the end of the line, a full crawl has been performed. Or, due to the HTML distance algorithm, the overall size of the HTML nodes is big enough to mask the difference of a new (for example) <a> node that could point to a new path or even include new input vectors in its query.
The situation becomes worse still when considering that pages that may look like duplicates on the client-side, could in fact expose vulnerabilities on the server-side, usually via cookies or headers, which can be processed without warning or visible indication.
That would mean that a lot of effort is being spent in a deduplication process that doesn't have any upsides and will result in great loss of coverage even for cases which are within the intended scope.
And there's also the issue of being unable to pass DOM state to the fuzzers, just a URL and cookies.
Even if the crawler does its job, a massive amount of coverage gets lost when communicating the resources to the fuzzers.
For example, any issues that depend on JS code will currently go completely unnoticed, for example: pages creating forms/links via JS, JS loading HTML via AJAX, DOM-based XSS, DOM-based unvalidated redirects etc.
I don't mean to rag on Gryffin, I figured since it's still in beta this would be the right time to try and spot design issues.

And there's also the issue of being unable to pass DOM state to the fuzzers, just a URL and cookies.

I can't agree more on this. There seems to be a need to define a unified API between fuzzers in the industry so as to make information gathered at an upstream process to be usable in downstream fuzzers.

... to mask the difference of a new (for example)  node that could point to a new path or even include new input vectors in its query

This may or may not be true for html-distance, that additional node would generate additional features that translates to more 1 or 0 bits and could change the fingerprint significantly. In our early version, we also try to add the url domain of anchor tag as an input feature for html-distance. We dropped it for simplicity, but we can always add it back. I'd wait until we see inputs from open community on how html-distance should be tuned, and then get it released in next minor version.

The situation becomes worse still when considering that pages that may look like duplicates on the client-side, could in fact expose vulnerabilities on the server-side, usually via cookies or headers, which can be processed without warning or visible indication.

Cookies, user-agent, and post body data should be part of the keys to distinguish the origin of the html. We currently just use "domain" for such purpose. We should definitely consider that something to be added in short term.

I don't mean to rag on Gryffin, I figured since it's still in beta this would be the right time to try and spot design issues.

Not at all. Actually we see this as an opportunities to collaborate with fuzzer authors. There is no easy way unless we discuss using the source code :)

I can't agree more on this. There seems to be a need to define a unified API between fuzzers in the industry so as to make information gathered at an upstream process to be usable in downstream fuzzers.

I'm calling that DOM transitions in Arachni, they're attached to pages associated to relevant issues, you can get them from the reports.
They allow DOM state to be restored via a series of event triggers from [element, event, options] tuples.

Cookies, user-agent, and post body data should be part of the keys to distinguish the origin of the html. We currently just use "domain" for such purpose. We should definitely consider that something to be added in short term.

Problem is that a cookie set by a page should be assumed to be an input for every other page as well.
That basically means that if you want cookies to be audited you can't deduplicate pages because any one of them can be vulnerable on the server-side.
Same thing goes for common (or custom) request headers as well, although I don't think you're currently auditing headers.

Problem is that a cookie set by a page should be assumed to be an input for every other page as well.
That basically means that if you want cookies to be audited you can't deduplicate pages because any one of them can be vulnerable on the server-side.
Same thing goes for common (or custom) request headers as well, although I don't think you're currently auditing headers.

Thanks for the input. I think I understand what you mean and I will push an update to get the dependencies between cookie / domain / user-agent against similarity detection fixed in next push.

This project is being archived.


docs(kubelet-density): adding kubelet density experiment doc

Signed-off-by: Shubham Chaudhary<EMAIL_ADDRESS>Harness Developer Pull Request
Thanks for helping us make the Developer Hub better. The PR will be looked at
by the maintainers.
What Type of PR is This?

[ ] Issue
[ ] Feature
[ ] Maintenance/Chore

If tied to an Issue, list the Issue(s) here:

Issue(s)

House Keeping
Some items to keep track of. Screen shots of changes are optional but would help the maintainers review quicker.

[ ] Tested Locally
[ ] Optional Screen Shoot.

Can we also update the contents page with the experiment links?


[oap-native-sql] spark sql unit test

To improve the stability of native sql engine, we need to test it using spark sql unit test. For unsupported expressions, we need to fallback them to row. For bugs in supported operators, we need to fix them.

fixed now


With external Wi-Fi module FreeRTOS Qualification is possible?

Hi,
We have ported our MCU to amazon-freertos and able to connect with the AWS cloud using external Wi-Fi module ESP8266.
Now, we want to qualify the FreeRTOS Qualification but our MCU does not have either Wi-Fi or ethernet, will this MCU is eligible for the FreeRTOS Qualification?
Is it compulsory to have either Wi-Fi or Ethernet for the FreeRTOS Qualification?
Or,
With an external Wi-Fi module, it will qualify?
Could anyone, please help me with the above question?
Sorry for my bad English.

To your question:  Is your configuration eligible for FreeRTOS Qualification?  YES!  We have existing qualifications that use Wifi modules similar to your case and offload various functions to the Wifi module including TCP/IP and TLS.  An example is the STM32L4 Discovery Kit that uses an Inventek Wi-Fi module.  Please go ahead and follow the DQP submittal process (https://aws.amazon.com/partners/dqp/)  and let us know how we can help with your efforts.  See this document which specifically shows how you can identify that features like TCP and TLS are offloaded:  https://docs.aws.amazon.com/freertos/latest/qualificationguide/freertos-qualification.html.

Hi @dnoal, Thank you very much for your help.
I'll procedure further if any issues will come, will definitely create the new issue for the same.
Thanks
I'm closing this thread (issue).!


Generate Fake Article for Seeding

Title: Generate Fake Article for Seeding
Description:
This GitHub issue aims to create a function named generateFakeArticle for generating a fake article that can be used for seeding purposes. The function should utilize various libraries and utilities to generate realistic-looking article data.
Tasks:

Implement generateFakeArticle Function:

[ ] Create a function named generateFakeArticle in the seeder/utils.ts file.
[ ] Generate a fake article with the following properties:

Title: Generate using the faker library.
Subtitle: Generate using the faker library.
HTML Content: Generate using the markdownToJSON function.
Markdown Content: Generate using the markdownToJSON function.
Cover Image: Generate using the faker library.
Read Time (in Minutes): Generate as a decimal number using the faker library.

[ ] Return the generated article data as an object.

Acceptance Criteria:

The generateFakeArticle function is successfully implemented.
The function generates a fake article with the specified properties.
The generated article data is returned as an object.

Additional Information:

This function is intended for seeding purposes and will generate a fake article with realistic-looking data.
The issue should be considered resolved once the function is successfully implemented and tested.

hey @prasenjeet-symon i would like to contribute

@RISHIKESHk07 Assigned to you ☺


Update deweight_wrapper.py

Brings parsing of ApproxMC output in line with current usage of "c " as prefix to number of solutions.

When I run ApproxMC locally, there is also a "c " before the line that gives the approxmc runtime. However, this is not reflected in the ApproxMC ReadMe, so it may be an issue with ApproxMC rather than Deweight.


Suggested DB Layer

Firstly, I have to say that alasql is a great work.
We are designing a programming platform that can do all kinds of ETL. We choose javascript+sql as base because they fit our team capability. alasql is very strong in handling memory data as well as physical file. However, we stick to a problem in bulk import/export from/to database. Actually we can do that by other CLI before or after the node script, but it is really perfect if it can be done within the script itself. Mostly, we are dealing with MySQL and MSSQL.
If there is any other JS that can convert DB table into javascript object, appreciate you could suggest one. Thanks.

The one of possible solutions is to create custom FROM function like here:
alasql.from.DB = function(dbtype, opts, cb, idx, query) {
	var res = [];
        async_read_data_from_mysql_function(dbtype, opts.dbname, opts.tablename, function(data) {
		res = data;
		if(cb){
			res = cb(res, idx, query);
		}
	};
	return null;
};

function async_read_data_from_mysql_function(dbtype, dbname, tablename, cb) {
    // put your code here
}

Then you can use it like:
SELECT * FROM DB("mysql",{dbname:"one", tablename:"two"})

Also, you can save data to MYSQL with alasql.into.DB() custom function.
alasql('SELECT * INTO DB("mysql",{dbname:"one", tablename:"two"}) FROM ?',[data]);

You can see the examples of FROM and INTO functions at src/84from.js and src/830into.js files.

Perfect !!!

Added to https://github.com/agershun/alasql/wiki/User-Defined-Functions

👍


Expose how the latest version came into being

Changes in this PR:
a new document.get_latest_manifestation_type method which returns either "tna-enrich" or "transform" depending on the "name" property of the most recent manifestation date.
This is used in the versions table in the editor UI to annotate each version with it's 'version type', which helps editors better understand where each version came from.
Trello card / Rollbar error (etc)
https://trello.com/c/jyTZaNMi/1359-eui-version-history-add-column-to-table-to-indicate-type-of-version

 [EUI] VERSION HISTORY Add column to table to indicate type of version


Trivial: Fix Magic Numbers in key and pubkey - fixes #1968

This is my proposed fix to the magic numbers in pubkey.h/cpp and key.h/cpp
Thanks.

Hi - updated  the PR to make declaration of the constants in consistent (updated static in key.h to static constexpr unsigned int )
I believe I did use the new constant throughout key.cpp and pubkey.cpp. If I missed some i'm not seeing it sorry.

closing & reopening because travis is stuck

Can you rebase please to remoev conflicts? Try:
git fetch origin
git rebase -i origin/1.14.4-dev
It will then bring up an editor. If you remove the first 2 commits before yours. Please check the result then force push it. Thank you!

Can you rebase please to remove conflicts? Try:
git fetch origin
git rebase -i origin/1.14.4-dev
It will then bring up an editor. If you remove the first 2 commits before yours. Please check the result then force push it. Thank you!

I think I redid that correctly? - The rebase only showed me one commit that was not mine, I removed it and did git push -f origin fix_magic_numbers_1968. It told me everything was up to date.
As noted by a previous comment. These changes seem to already be in 1.21-dev. (as it is upstream in bitcoin), so if this change is causing problems, we can just close the PR)

Sorry, just saw this in my backlog - looks like it still has a rogue commit. Any objections if I just grab the branch, rebase and open a new PR with the commits?

Any objections if I just grab the branch, rebase and open a new PR with the commits?

Can you do it against 1.21? No point in making changes like this on a leaf patch branch like 1.14.4?

Okay by me. I am sorry if i mangled it

Turns out 0.21 has addressed the magic numbers itself, so this only makes sense for 1.14.

Merged with #2249


Support to mock Transactions

What do you want to change?
Does anyone know a way to go about this? I am uncertain if this can be implemented... The way things currently work is that the generated code in WIthTx returns a newly generated Queries struct. This struct contains the transaction interface struct returned from the database driver. This is necessary because each transaction needs to be isolated. It is thus not straightforward to inject a mock to build stubs and listen on.
I prefer to capture transactions in UDFs with UDT parameters but the flexibility of being able to assemble and cleanly mock test transactions on the backend service would be very handy indeed.
What database engines need to be changed?
PostgreSQL
What programming language backends need to be changed?
Go

The current WithTx implementation makes it tricky to support a few different use cases. I'm loathe to make a large, breaking change, but we'll take this into account as we make changes in the future. It sounds like you have a okay workaround for now.

Just to say that one potentially transparent way to achieve this for some simple use cases would be to support throwing BEGIN and COMMIT etc into the queries.sql file. I have no idea how hard that would be in practice.


Update Angular to v15

Thanks for your PR! Please fill out the following list :)

[ ] I put the ticket or multiple tickets in review
[ ] Commit message format → Closes #<Issue Number> - Your commit message.
[ ] Sonarcloud link : <add the link here>
[x] No documentation update needed
[ ] Link to PR with documentation update: <add the link here>
[x] No Release Notes needed
[ ] Release Notes :

Notifications don't disappear anymore. They should disappear after a few seconds

When selecting AccessItems in the side menu, you get stuck in the loading screen

Notifications don't disappear anymore. They should disappear after a few seconds

@ryzheboka I solved this in my last commit. Could you please check it?

When selecting AccessItems in the side menu, you get stuck in the loading screen

This is not an issue introduced with Angular 15. You can reproduce it here https://kadai-io.azurewebsites.net/kadai/#/kadai/administration/access-items-management and even here https://taskana.azurewebsites.net/taskana/#/taskana/administration/access-items-management
It is probably better to open an issue for this bug.


Improve playground code editor

Problem
The current playground utilizes a CodeMirror 6 editor. While powerful, customizing it for simple tasks like font changes or syntax highlighting has proven difficult. This leads to a less user-friendly experience.
Solution
We propose migrating to Ace.js, a well-established and user-friendly code editor library. Ace offers:

Easier customization:  Changing fonts and adding syntax highlighting is more straightforward with Ace.
Proven track record:  Ace is widely used and battle-tested, ensuring stability and reliability.

Target benefits

Improved user experience with a more customizable and user-friendly code editor.
Reduced development time for implementing basic editor features.

This change should streamline the playground experience for users and developers alike.

Ace.js example: https://github.com/ajaxorg/ace-builds/blob/master/editor.html.


Ingress Controller CrashBackLoop -After adding incorrect Basic Auth Secret

NOTE: GitHub issues are reserved for bug reports only.
For anything else, please join the conversation
in Kong Nation https://discuss.konghq.com/c/kubernetes.
Summary
Running KONG Ingress Controller in GKE. Configuring as service with basic-auth. When trying to setup consumer credentials i configured basic-auth secret incorrectly due to which ingresscontroller controller was in crashback loop
Kong Ingress controller version
KIC 0.7.1
Kong or Kong Enterprise version
<IP_ADDRESS>-alpine
Kubernetes version
paste kubectl version output
Client Version: version.Info{Major:"1", Minor:"17", GitVersion:"v1.17.0", GitCommit:"70132b0f130acc0bed193d9ba59dd186f0e634cf", GitTreeState:"clean", BuildDate:"2019-12-13T11:52:32Z", GoVersion:"go1.13.4", Compiler:"gc", Platform:"darwin/amd64"}
Environment
GKE
What happened
Created incorrect secret for basic auth and configured that for a consumer which made ingress controller container crash
Basic auth secret (Incorrect)
kubectl create secret generic my-basicauth
--from-literal=kongCredType=basic-auth
--from-literal=key=myuser  \
it should have been
kubectl create secret generic my-basicauth
--from-literal=kongCredType=basic-auth
--from-literal=username=myuser
--from-literal=password=mypassword
Expected behavior
Maybe just show show error and not crash the ingresses container
Steps To Reproduce
1.Configure a service with basic auth
2.Create a consumer
3.Create INCORRECT credentials
4.Attach the incorrect credentials to the consumer
NOTE: Quick fix was to delete the old "Incorrect" secret and create a valid  secret

I encountered this exact same issue on my side as well, and should probably be addressed.

Duplicate of https://github.com/Kong/kubernetes-ingress-controller/issues/532


Bun Support

Feature Request
Use bun for package management and running scripts.

[x] Does bun run --bun wxt work?

❌  bun run --bun wxt  (here's the output, bad stacktrace doesn't point to anything)$ bun run --bun wxt

WXT 0.9.0                                                                                2:04:41 PM
✔ Started dev server @ http://localhost:3000                                                                      2:04:42 PM
ℹ Pre-rendering chrome-mv3 for development with Vite 4.5.0                                                        2:04:42 PM

 ERROR  Cannot access uninitialized variable.                                                                      2:04:42 PM

  at <anonymous> (node_modules/wxt/dist/cli.cjs:4048:4)
  at processTicksAndRejections (:55:39)

✖ Command failed after 440 ms                                                                                2:04:42 PM

 ERROR  Cannot access uninitialized variable.                                                                      2:04:42 PM

  at <anonymous> (node_modules/wxt/dist/cli.cjs:4048:4)
  at processTicksAndRejections (:55:39)

error: "wxt" exited with code 1 (SIGHUP)
error: script "dev" exited with code 1 (SIGHUP)

✅ bun run --bun wxt build
✅ bun run --bun wxt zip
✅  bun run wxt
✅ bun run wxt build
✅ bun run wxt zip

[ ] Add bun to wxt init for package managment
[ ] #185

Is your feature request related to a bug?
N/A
What are the alternatives?
N/A
Additional context
Decided to add this since nypm includes support for detecting bun, and because it's a really fast script runner lol.

This could be related?
bun/issues/3137
I don't have source code of your CLI, but checking the dist/cli.cjs at all async/awaited calls it simply returns empty object.

Here's the dist/cli.js file, cursor is at where the stack trace says there's an uninitialized variable.

Ok, so accessing res.default causes this error. Here I've added logs, and it prints out 3 and res, but not 4 and res.default. So this appears to be a problem with Jiti. I'll do a test against pure JITI and see if it's my fault or Jiti's fault.

OK, so res.default must be a getter, and it must be throwing an error while trying to import the entrypoint file. Jiti works just fine with Vite.

Jiti appears to have native support for Bun, and skips the required transform steps. After upgrading bun to 1.0.11, all the WXT commands fail when ran in Bun.
You can see the prepared code has imports on line 1, but the Bun code doesn't have those same imports. Basically, all the transforms I do to get the code running in Node are completely ignored with Bun.

Got that bug fixed, but ran into another. Now we're blocked by https://github.com/oven-sh/bun/issues/4130
$ bun run --bun wxt

WXT 0.9.2                                                                                2:32:49 PM
✔ Started dev server @ http://localhost:3000                                                                        2:32:49 PM
ℹ Pre-rendering chrome-mv3 for development with Vite 4.5.0                                                          2:32:49 PM
[bun] Warning: worker_threads.Worker option "execArgv" is not implemented.

$ echo $?
0

Bun issue has been fixed, waiting for https://github.com/oven-sh/bun/issues/4130 to be released. Not gonna try out the main branch, just gonna wait.

Tested against the latest main branch, was super easy.
bun upgrade --canary
bunx wxt@latest init wxt-bun --pm bun --template vanilla
bun run --bun wxt build
bun run --bun wxt

Everything worked, no warnings! 🎉 🎉
I'll close this issue once the the fix is released, I assume v1.0.22. v1.0.21 is the current release, and doesn't include https://github.com/oven-sh/bun/issues/4130.

Tested on v1.0.23, and it is working!
If you're using bun, you can update your package.json scripts to, putting bunx --bun in front of all the WXT commands:
"scripts": {
    "dev": "bunx --bun wxt",
    "dev:firefox": "bunx --bun wxt -b firefox",
    "build": "bunx --bun wxt build",
    "build:firefox": "bunx --bun wxt build -b firefox",
    "zip": "bunx --bun wxt zip",
    "zip:firefox": "bunx --bun wxt zip -b firefox",
    "compile": "tsc --noEmit",
    "postinstall": "bunx --bun wxt prepare"
}


[SYSTEMML-540] Improved the performance of bias_add and added CP relu_backward

The tests passed. Since the performance improvement after relu_backward is significant, merging this PR.


Support the guidance document creation

Describe the task
Jairo completed the first draft of the guidance document. This ticket is to figure out the best way to present Jairo's document to users and help him finalize it with user feedback.
Acceptance Criteria

[x] Connect with Hanna on the current status of the ticket (repurpose/refine the ticket description accordingly)
[x] Review relevant feedback from the previous user engagement sessions
[x] Review Jairo's first draft of the guidance document
[x] Connect with the Engagement team (Jessica/Avril/Alana/Scott)
[ ] Provide feedback/suggestions for Jairo
[ ] Propose solutions for presenting the guidance document to users (short-term solution, ideal solution, etc.)

Additional context
Suggestions made based on user feedback so far:

preferably the content is on a website that is editable and linkable via link anchors. We can support on making this user-friendly and navigable. The catch is it will need to be maintained and updated as policy matures, so we will need to have a check-in with business area to make sure it is up to date on a certain frequency
we will need a glossary of terms, and we can make this reference-able for each field that needs more context
users should be able to access and download all required templates from this document
it would be good to have specific directions for users to fill out each of the forms
a lot of users are carrying over knowledge about CIIP/systems of other provinces. It would be good to let them know how some requirements/terminologies are different from CIIP and other provinces.
need to clarify roles and who is required to do what so that they can plan accordingly

@hannavovk ticket created here!

Just wanted to add that this work is also being supported by the ICPG team, via Scott Cutler and Alana Clement. Alana comes from the engagement team previous and has a lot of experience with drafting language for the web that is more accessible. :)

Next steps:

Connect with Jessica/Avril on their program website
Propose the best solution for presenting Jairo's document
Help Jairo finalizing the content

We met with the Engagement team and our SMEs on Mon, Dec 18
Agreements:

Once finalized, the  docx icon web app guidance will be hosted as a PDF on the OBPS program website.
The Engagement team will send out a letter to inform the industry.
The web content updates will be post January.

Next steps:

DST review and revise Jairo’s draft document
DST confirm readiness of the draft document with Jairo/Harnav
DST share the draft document in the  Folder icon Guidance and Comms folder in the Engagement channel, and inform the Engagement team for approval
Engagement team review/revise/approve the draft document
Engagement team or DST export the finalized document to PDF
Engagement team host and publish the PDF
Engagement team send out the letter to industry

@hannavovk  informs Nan if there’re changes of the plan post the work planning meeting this afternoon.

Suha suggests looking at the usability testing feedback to support guidance doc design/creation. cc @nanyangpro

@nanyangpro This is the latest guidance document link from Harnav: https://bcgov.sharepoint.com/:w:/r/teams/06817/_layouts/15/Doc.aspx?sourcedoc={8CAD9D58-F261-4304-BC88-49735E96FE87}&file=Applying for a BC OBPS Regulated Operation ID - Guidance Document.docx&nav=eyJjIjoyMDU4OTU0ODg3fQ&action=default&mobileredirect=true&isSPOFile=1&clickparams=eyJBcHBOYW1lIjoiVGVhbXMtRGVza3RvcCIsIkFwcFZlcnNpb24iOiIyNy8yMzExMzAyNjIwMiIsIkhhc0ZlZGVyYXRlZFVzZXIiOmZhbHNlfQ%3D%3D

I'll cross-ref the guidance doc with the webinar slides. cc @nanyangpro @hannavovk

Hi @hannavovk (cc: @Dianadec), since the supporting work from our end has been wrapped up given the Guidance document is with the Engagement team for the final review and publish, we can close this ticket unless you advise otherwise. Thanks!

Thanks so much for all your collaborative work and patience on this! @nanyangpro


Update README.md

This pull request proposes including the VulData7 framework in the MSR tool list. VulData7 builds vulnerability related datasets by linking NVD with Git.
Thanks

Thank you!


cf auth fails with dial tcp i/o timeout

Command
cf auth
What occurred
Dial tcp i/o timeout rarely occurs.
cf auth admin mypasswaord
API endpoint: https://api.mycf.com
Authenticating...
Post http://uaa.mycf.com/oauth/token: dial tcp: i/o timeout
FAILED

What you expected to occur
CLI Version
cf version 6.32.0+0191c33d9.2017-09-26
CC API Endpoint Version
2.97.0
Platform & Shell Details
Ubuntu 14.04.1 LTS
Any other relevant information
As a result of packet capture, the CLI seemed to be waiting for the response of the AAAA record and timed out when the response got lost.
success
664281	2017-12-13 09:36:10.565450	<IP_ADDRESS>	<IP_ADDRESS>	DNS	98	Standard query 0x5956 AAAA uaa.mycf.com
664282	2017-12-13 09:36:10.565541	<IP_ADDRESS>	<IP_ADDRESS>	DNS	98	Standard query 0xeb87 A uaa.mycf.com
664283	2017-12-13 09:36:10.572708	<IP_ADDRESS>	<IP_ADDRESS>	DNS	180	Standard query response 0x5956 AAAA uaa.mycf.com SOA ns-1255.awsdns-28.org
664284	2017-12-13 09:36:10.572720	<IP_ADDRESS>	<IP_ADDRESS>	DNS	114	Standard query response 0xeb87 A uaa.mycf.com A <IP_ADDRESS>

failure
664306	2017-12-13 09:36:10.587869	<IP_ADDRESS>	<IP_ADDRESS>	DNS	98	Standard query 0xb1d4 AAAA uaa.mycf.com
664307	2017-12-13 09:36:10.587956	<IP_ADDRESS>	<IP_ADDRESS>	DNS	98	Standard query 0xcd19 A uaa.mycf.com
664309	2017-12-13 09:36:10.596365	<IP_ADDRESS>	<IP_ADDRESS>	DNS	114	Standard query response 0xcd19 A uaa.mycf.com A <IP_ADDRESS>

It looks like #1080.

Hi @t-seyama
cf CLI 6.33.0 is built on a later Golang version (1.9.2 instead of 1.9.0 for the 6.32.0 version you are using).
Does the issue occur on this latest CLI version as well?
Cheers,
Dies Koper
CF CLI PM

Hi @dkoper
I updated to cf CLI 6.33.0, but the issue also occurred.

Thanks for confirming. I'll ask my team to take a look.

Hi @t-seyama, we're trying to reproduce this issue in our own environment. Can you please provide the following:

are you doing any special setup to produce this timeout error
what tool are you using to capture the DNS logs
are you using the standard UAA server setup or a custom one
it looks like your UAA server is in the same private network as your CC server?

and anything else you can think of to help us reproduce this timeout error. Thanks.

Hi @nickwei84

are you doing any special setup to produce this timeout error

I continued to execute cf auth command from multiple clients.

what tool are you using to capture the DNS logs

I used tshark and caputured packets on the client side.

are you using the standard UAA server setup or a custom one

Yes, I set up UAA using cf-release.

it looks like your UAA server is in the same private network as your CC server?

Yes, they are in the same private network.
Thanks.

Hi @t-seyama
The team spent a few hours investigating but it seems we've reached our limits of what we can do to figure this out. In our efforts to reproduce, it seems to occur only once in 180 times. You also mentioned it occurs only rarely.
I think it's better that we give up here and go back to coding features.
If you want to investigate further, note that our Linux CLI binary uses the go resolver, so you could try rebuilding it with the cgo resolver, or you could try analysing the debugging output when setting Golang's GODEBUG=netdns=1 environment variable.
If you find anything that points to a bug in our code, please let us know and we'll pick it up again.
Else, we'll assume this is a Golang issue and hope a future Golang release will resolve it.

Hi @dkoper
Thank you for investigation.
I will continue to investigate. I inform you if there is any new information.
Thanks.

Hey @t-seyama thanks for creating this issue. We are closing this issue due to inactivity but please feel free to reopen or create a new issue if you have feedback.
For anyone stumbling on this after we close it. The CF CLI Team has found that these errors tend to come up due to DNS misconfigurations or DNS Timeouts. All of which boil down to "DNS is not configured as your platform expects it". If you can fix the DNS configuration, this will no longer be an issue, if you can't/are unable to figure out what the problem is. Try setting the CF_DIAL_TIMEOUT environment variable to a value higher than the default value of 5 (in seconds).


Move moment to peerDependencies

Supersedes #419
Fixes #122
It's better to move moment to peerDependencies because it prevents duplication of moment instances which can leads to bugs (when loading locals for instance).
In our project, we have a fixed dependency of<EMAIL_ADDRESS>react-datepicker installs<EMAIL_ADDRESS>so 2 instances of moment are loaded and locals doesn't work for the moment of react-datepicker.
If you look at react-dates, this is how they require moment: https://github.com/airbnb/react-dates/blob/master/package.json#L126
I also removed package-lock.json as it's not used in the project anymore. Tell me if I should remove this from the PR as it's not quite related (but I didn't know which lockfile to update so I thought it should be better to delete it).

I agree this is probably better, however there's some discussion going on here #122.

Not sure if we can make this change without breaking everyone's datepickers. What do you think?

Some discussion which ended 1 year ago. This should be reconsidered.
I think that everyone who use the datepicker are already using moment but if we want to be truely semver compliant, this can be released as a breaking change.

@Kerumen I think the plan is to bump the major only when we think the API is stable. Keeping it at 0 is also truly semver compliant since semver says

Major version zero (0.y.z) is for initial development. Anything may change at any time. The public API should not be considered stable.

@aij Yes, seems legit too. npm won't auto-upgrade people and if they manually do it, they will know why their code break (if it's break).

Okay lets do this. We use yarn.lock instead of the package-lock.json, so lets make sure that's updated.

@Kerumen Let me know if you want to proceed with this, happy to merge now we're reaching consensus.

@martijnrusschen Conflicts resolved. I set ^2.0.0 as semver hence people won't have warnings if they install a previous version than 2.19.4. I suppose all moment >=2 works.


It's not in preview.

I think flexible servers have already been GAed.

The process mentioned in this document can also be used to upgrade your Azure Database for PostgreSQL - Flexible server, which is in Preview.

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

ID: b63959af-0e54-22ff-b1cc-d63ee055fddd
Version Independent ID: bde862d7-793a-a17c-03c6-327e46305535
Content: Upgrade using dump and restore - Azure Database for PostgreSQL
Content Source: articles/postgresql/single-server/how-to-upgrade-using-dump-and-restore.md
Service: postgresql
Sub-service: single-server
GitHub Login: @sr-msft
Microsoft Alias: srranga

Hi @LifeRyuto Thanks for reaching out, We will Investigate and update you shortly.

Changes have been made and they should reflect by tomorrow. Please close the PR once you verify this tomorrow.

Thanks, @kabharati for fixing the doc.
@LifeRyuto changes have been updated, thanks again for your findings to improve our documents.


OLS-985: Updated config file for integration tests

Description
Updated config file for integration tests
Type of change

[ ] Refactor
[ ] New feature
[ ] Bug fix
[ ] CVE fix
[ ] Optimization
[ ] Documentation Update
[x] Configuration Update
[ ] Bump-up dependent library
[ ] Bump-up library or tool used for development (does not change the final image)

Related Tickets & Documents

Related Issue #OLS-985

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 94.82%. Comparing base (d3076b3) to head (af6b3a6).
Report is 32 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #1532      +/-   ##
==========================================
+ Coverage   94.75%   94.82%   +0.07%
==========================================
  Files          59       60       +1
  Lines        2631     2669      +38
==========================================
+ Hits         2493     2531      +38
  Misses        138      138

/retest

/lgtm

/approve


fix StarVertex bugs when using LazyGetter

Clone properties map because we remove the nested documents.
Don't add externalVertexIdField & edgeIdField as properties;

This fixes the bugs, but now we see OOM exceptions.
It probably doesn't have anything to do with the fix. Because StarVertex holds referneces to all it's innerEdges, who in turn hold references to both star vertices, etc... This causes alot of memory usage.
We should think about how to solve this. Either not hold the vertices in the edges, or the other way around, or both.
@seanbarzilay


New storage backend abstraction

Close to a state where implementation(s) can be filled in. Parking due to offsite.

@jkarni @jameshaydon Since this turned out to be (a lot) more code than anticipated, it might be best to go through it irl.

Looks great!


Batch update in ODATA V4

OpenUI5 version: 1.126.0
Scenario:
User is provided with a table , and user can modify multiple rows, and click on 'save'. On 'save', we are trying to do a batch update of the multiple rows modified. We have tried submitBatch() method, but it seems that the response of the batch call cannot be fetched in submitBatch() call.
We even referred to this issue : https://github.com/SAP/openui5/issues/3888
Is there any method that supports this requirement?

What exactly is your requirement?
You can implement such a "Save" button just fine, see https://openui5.hana.ondemand.com/topic/74142a38e3d4467c8d6a70b28764048f But there should be no need to access "the response of the batch call". At least, it is not provided via submitBatch(), but the results could be obtained from the corresponding bindings or controls.

@ThomasChadzelek
We have a table where users can edit multiple rows, and there is a 'Save' button. When the user clicks the 'Save' button, we attempt to make patch calls for multiple records. We want to send these patch calls in a batch request and need to receive the response for each patch call to implement some logic.
This is our requirement, is there way to achieve this.

Sounds like you are trying to create those PATCH calls from controller code, not via a two-way binding. Why are you trying to do that? Where do you store the user input in between? In some JSON model? Why? How does that benefit from metadata, automatic type detection, value help etc.?
I strongly recommend to use two-way data binding (v4.ODataModel's default) and an API submit group, see https://openui5.hana.ondemand.com/topic/74142a38e3d4467c8d6a70b28764048f The results could be obtained from the corresponding bindings or controls.


Fix scan dataset commands not respecting --local flag

Closes #3035
Code Changes

[x] refactor local/API command logic at the top-level
[x] add local flags to the dataset scanning functions

Steps to Confirm

[ ] fides --local scan dataset db --credentials-id app_postgres
[ ] the command fails connecting to the database (as opposed to a webserver that isn't running)

Pre-Merge Checklist

[x] All CI Pipelines Succeeded
Documentation:

[ ] documentation complete, PR opened in fidesdocs
[ ] documentation issue created in fidesdocs

[x] Issue Requirements are Met
[ ] Relevant Follow-Up Issues Created
[x] Update CHANGELOG.md
[ ] For API changes, the Postman collection has been updated

Description Of Changes
Quick fix for a bug that @nicolas-ethyca found

This is ready for a review! Not sure what is going on with the test coverage, but I think its ok?

It also looks like some formatting happened here...unintentional

tested locally and verified working

Thank you!


Optimized density script to reduce run time on high powered systems

Instead of incrementing the pipelines by one we could speed up the process by incrementing by sets of 5 and once failed decrement by 1 and do it until it passes with the target FPS
ex.
1 pipeline - pass
6 pipelines - pass
11 pipelines - failed
10 pipelines - failed
9 pipelines - pass ->done

FPS calculation is inconsistent


Add TrainableFidelityStatevectorKernel

Summary
Adds a TrainableFidelityStatevectorKernel class. Closes #638.
Details and comments
For consistency with TrainableFidelityQuantumKernel, an equivalent class that inherits from FidelityStatevectorKernel and TrainableKernel has been added.
To avoid some code repetition, _parameter_array has been moved  to TrainableKernel and this now raises the error that was previously in evaluate of TrainableFidelityQuantumKernel.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

35 of 35 (100.0%)  changed or added relevant lines in 4 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.06%) to 93.025%

Totals

Change from base Build<PHONE_NUMBER>:
0.06%

Covered Lines:
2694

Relevant Lines:
2896

💛  - Coveralls

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

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

35 of 35 (100.0%)  changed or added relevant lines in 4 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.06%) to 92.915%

Totals

Change from base Build<PHONE_NUMBER>:
0.06%

Covered Lines:
2649

Relevant Lines:
2851

💛  - Coveralls


Flux doesn't work with K8s 1.7, kubectl incompatibility?

This looks like the error received when using a kubectl that is older than the k8s cluster.
Release job submitted, ID 4a24bdd7-d907-9b71-0076-74650969304b
Status: Failed: default/affect-detection-predict-94: applying definition to default/affect-detection-predict-94: running kubectl: error: group map[policy:0xc8203af030 rbac.authorization.k8s.io:0xc8204b4000 authentication.k8s.io:0xc8200aa000 authorization.k8s.io:0xc8203aee00 componentconfig:0xc8203aef50 apps:0xc8203aed90 autoscaling:0xc8203aee70 batch:0xc8203aeee0 extensions:0xc8203aefc0 federation:0xc8203aecb0 :0xc8203aed20] is already registered

1) Queued.
2) Calculating updates for release.
3) Cloning git repository.
4) Finding defined services.
5) Found service default/affect-detection-predict-94
6) ignored service default/affect-router as it is not running in cluster
7) Found 1 services.
8) Looking up images.
9) Will update default/affect-detection-predict-94 container affect-detection-predict-94: gcr.io/strange-cosmos-822/affect-detection:0.0.10 -> 0.0.13
10) Pushing changes.
11) Applying changes.
12) Sending notifications.
13) Failed: default/affect-detection-predict-94: applying definition to default/affect-detection-predict-94: running kubectl: error: group map[policy:0xc8203af030 rbac.authorization.k8s.io:0xc8204b4000 authentication.k8s.io:0xc8200aa000 authorization.k8s.io:0xc8203aee00 componentconfig:0xc8203aef50 apps:0xc8203aed90 autoscaling:0xc8203aee70 batch:0xc8203aeee0 extensions:0xc8203aefc0 federation:0xc8203aecb0 :0xc8203aed20] is already registered

Failure

E.g.
https://github.com/kubernetes/kubernetes/issues/39494 https://github.com/ContainerSolutions/registry-tooling/issues/7 https://github.com/kubernetes/kubernetes/issues/43778

I've created an image based on quay.io/weaveworks/fluxd:0.3.0 but with kubectl v1.7.2, and pushed it to quay.io/squaremo/fluxd:k8s-1.7. Replacing the fluxd image with that will tell us whether the Kubernetes version is the problem.

Thanks for creating the image so quickly. I couldn't try it because I got a Permission Denied trying to pull it, but while I was editing the flux-agent yaml I noticed that it was originally set to fluxd:0.2.0. I updated it to fluxd:0.3.0 and the error was resolved. I copied the setup text from the weave cloud config page and an old definition was pulled because the v1.7 URL redirects to v1.6:
Michelles-Q-15:bin michelle$ curl "https://cloud.weave.works/k8s/flux.yaml?t=REDACTED&k8s-version=$(kubectl version | base64 | tr -d '\n')"
code: Redirect
message: >-
  Moved permanently to `/k8s/v1.7/flux.yaml`, assuming Kubernetes version is
  "v1.7" or older, old location (`/k8s/flux.yaml`) will be deprecated soon. If
  you are using `curl`, you can pass `--location`, but it is best to just use
  the new URL.
Michelles-Q-15:bin michelle$ curl "https://cloud.weave.works/k8s/v1.7/flux.yaml?t=REDACTED&k8s-version=$(kubectl version | base64 | tr -d '\n')"
code: Redirect
message: >-
  Moved permanently to `/k8s/v1.6/flux.yaml`, assuming Kubernetes version is
  "v1.6" or older, old location (`/k8s/v1.7/flux.yaml`) will be deprecated soon.
  If you are using `curl`, you can pass `--location`, but it is best to just use
  the new URL.

In short, I think we can close this issue since fluxd:0.3.0 works with k8s v1.7.1, but the URL redirect should be fixed.

I couldn't try it because I got a Permission Denied

Doh -- I didn't realise that repo was private and ought to have checked, sorry about that.

I noticed that it was originally set to fluxd:0.2.0. I updated it to fluxd:0.3.0 and the error was resolved

Ahh right, so fluxd had been silently downgraded to 0.2.0 somewhere along the line, because our launch-generator supplies it in config. Good sleuthing :)
I'm going to close this issue, since it wasn't quite what we thought it was. We'll still need to get the automatically-generated config sorted out though.


Limit option to build image locally (This took EXTREMELY long, and could take even longer once you have more dependencies), you could've provide either of build or pull the image directly from your DockerHub. -3

Changed docker build into docker pull from DockerHub - needs testing.


[HOLD for payment 2024-02-22] [$250] Workspace - Workspace highlighted member blinks when check/uncheck mark

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

Version Number: 1.4.36-0
Reproducible in staging?: Y
Reproducible in production?: N
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers<EMAIL_ADDRESS>Logs: https://stackoverflow.com/c/expensify/questions/4856
Expensify/Expensify Issue URL:
Issue reported by: Applause - Internal Team
Slack conversation:
Action Performed:

Navigate to Workspace> Members
Check mark any member (Admin should not be the first on the list)

Expected Result:
Selected member should not blink
Actual Result:
Workspace member blinks when check/uncheck mark
Workaround:
Unknown
Platforms:

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

[x] Android: Native
[ ] Android: mWeb Chrome
[ ] iOS: Native
[ ] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[ ] MacOS: Desktop

Screenshots/Videos
Add any screenshot/video evidence

https://github.com/Expensify/App/assets/78819774/cda2fac8-09bb-4fb8-9e3e-c26c03769379
View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~01faff94e6cefb944b
Upwork Job ID:<PHONE_NUMBER>396689408
Last Price Increase: 2024-02-05
Automatic offers:

shubham1206agra | Reviewer | 28150587
Krishna2323 | Contributor | 28150588

We think that this bug might be related to #wave
CC @greg-schroeder

@Krishna2323 can you please link offending PR

Moving the price down since this seems like easy bug to solve

Still looking for proposals

@Krishna2323's proposal looks good to me.
🎀 👀 🎀 C+ reviewed

@shubham1206agra I can't pay you until you accept the offer in upwork

@Krishna2323 paid, thank you!

Paid!


Deprecation warning on recent Python versions

Users are getting this warning on newer Python versions:
$ ./vercheck.py -l
./vercheck.py:13: DeprecationWarning: The distutils package is deprecated and slated for removal in Python 3.12. Use setuptools or check PEP 632 for potential alternatives
  from distutils.version import LooseVersion

This is due to PEP 632, which will deprecate the Distutils package in the near future.
We need to re-implement the same features with setuptools.

PEP632 states that you must use the packaging.version.Version() class to substitute the distutils one.
I made a quick script to read one of vercheck's CSV reports and try out a simple comparison with two versions to validate this:
#!/usr/bin/python3

import sys
from distutils.version import LooseVersion

from packaging.version import Version

def main():

    args = sys.argv

    filename = args[1]
    print(f'reading {filename}')
    with open(filename, 'r') as f:
        lines=f.readlines()

    versions=[]

    for line in lines:
        versions.append([line.split(',')[1], line.split(',')[2]])

    try:
        for old,new in versions:
            print(f"comparing if version [{old}] < [{new}])")
            print(f"[distutils]: {LooseVersion(old) < LooseVersion(new)}")
            print(f"[packaging]: {Version(old) < Version(new)}")
    except Exception as e:
        print(f'----> comparison failed: {e}')
    return

if __name__ == '__main__':
    main()

Running it yields not so good results...
erico@suselab-erico:~/Projetos/scc-tools> ./testversions.py vercheck-different-scc_rmt_231027_1743.csv
reading vercheck-different-scc_rmt_231027_1743.csv
comparing if version [84.87+git20180409.04c9dae-1503<IP_ADDRESS>] < [84.87+git20180409.04c9dae-1503<IP_ADDRESS>])
/home/erico/Projetos/scc-tools/./testversions.py:26: DeprecationWarning: distutils Version classes are deprecated. Use packaging.version instead.
  print(f"[distutils]: {LooseVersion(old) < LooseVersion(new)}")
[distutils]: True
[packaging]: True
comparing if version [84.87+git20180409.04c9dae-1503<IP_ADDRESS>] < [84.87+git20180409.04c9dae-1503<IP_ADDRESS>])
[distutils]: True
[packaging]: True
comparing if version [3.0.4-1505<IP_ADDRESS>] < [3.0.4-1505<IP_ADDRESS>])
[distutils]: True
----> comparison failed: Invalid version: '3.0.4-1505<IP_ADDRESS>'

Apparently packaging.version uses a fixed regex that doesn't recognize the "150500" that SLE uses.
According to this, you can just override the regex by assigning a new value to packaging.version.VERSION_PATTERN first.

As expected, the default regex is awfully complex: https://regex101.com/r/ryw6wU/1

Reference URLs:

https://packaging.pypa.io/en/latest/version.html
https://peps.python.org/pep-0632/#migration-advice
https://stackoverflow.com/questions/73860427/how-to-use-the-python-packaging-library-with-a-custom-regex


Fix VMware naming

Pull Request Test Coverage Report for Build 1598

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 84.039%

Totals

Change from base Build 1596:
0.0%

Covered Lines:
3951

Relevant Lines:
4503

💛  - Coveralls


Update dependencies

Just a few major updates here and there

r? @matklad
(rust_highfive has picked a reviewer for you, use r? to override)

@bors r+

:pushpin: Commit 4cccc8b has been approved by matklad

Travis failure seems legit, and looks like it is caused by ignore update.

:pushpin: Commit b90bb7d has been approved by matklad

@bors: r=matklad

:hourglass: Testing commit b90bb7d6a8e410a2ba631bc2a1f3f40b1f5aaf9c with merge 2d42bcf41f044f7fbe827980cadb2acc54921cc2...

:sunny: Test successful - status-appveyor, status-travis
Approved by: matklad
Pushing 2d42bcf41f044f7fbe827980cadb2acc54921cc2 to master...


FIR camera calib

I want to do research on FIR camera, where can get the groundtruths and calibration of FIR camera?thanks

@shiningpure the FIR camera is currently not labeled and used as a further reference camera. It provides only little contrast if the lens was covered with water or the whole scene is covered in snow. To show its advantage you may want to also sub select frames from the whole dataset. If you want to extend that work, please reach out directly.
Best,
Mario


Ember cli 0.1.5 compatibility

This PR includes upgrade to ember-cli 0.1.5. The project was brought incline with ember-cli addon layout in 0.1.5

Closing. Covered by #12


Upgrade to ZIO 2

ZIO is at Milestone 4, with an RC expected in the next few weeks.
https://github.com/zio/zio/releases/tag/v2.0.0-M4
The API is nearly stable at this point, so any early migration work against this version should pay off towards the official 2.0 release.
The progress is being tracked here:
https://github.com/zio/zio/issues/5470
The Stream Encoding work in progress is the only area where the API might still change before the RC.
To assist with the migration, follow this guide:
https://zio.dev/howto/migrate/zio-2.x-migration-guide/
We are actively working on a ScalaFix rule that will cover the bulk of the simple API changes:
https://github.com/zio/zio/blob/series/2.x/scalafix/rules/src/main/scala/fix/Zio2Upgrade.scala

@swoogles Is this something you’d like to see or want to help out with?

I am working on this.

Great work, @adamgfraser + @swoogles


Syntax Error in react-scripts/scripts/start.js

Is this a bug report?
Yes

Can you also reproduce the problem with npm 4.x?
Yes

Which terms did you search for in User Guide?
react-scripts syntax error

(Write your answer here if relevant.)
Environment

node -v: v4.7.3
npm -v: 5.3.0
yarn --version (if you use Yarn):
npm ls react-scripts (if you haven’t ejected<EMAIL_ADDRESS>/home/ubuntu/workspace/my-app
└──<EMAIL_ADDRESS>
Then, specify:

Operating system: Cloud9 (c9.io) (Linux)
Browser and version (if relevant): Chrome Version 59.0.3071.115

Steps to Reproduce

Used create-react-app in Cloud9 using these instructions: https://community.c9.io/t/how-to-setup-react-app/16915
Worked on app for a while; worked fine.
Next day ran npm start in my my-app dir
Got error below.

Expected Behavior

Code would compile and app would run
Actual Behavior

><EMAIL_ADDRESS>start /home/ubuntu/workspace/my-app

> react-scripts start

/home/ubuntu/workspace/my-app/node_modules/react-scripts/scripts/start.js:33

const {

  ^

SyntaxError: Unexpected token {

at exports.runInThisContext (vm.js:53:16)

at Module._compile (module.js:373:25)

at Object.Module._extensions..js (module.js:416:10)

at Module.load (module.js:343:32)

at Function.Module._load (module.js:300:12)

at Function.Module.runMain (module.js:441:10)

at startup (node.js:139:18)

at node.js:990:3

npm ERR! code ELIFECYCLE

npm ERR! errno 1

npm ERR<EMAIL_ADDRESS>start: `react-scripts start`

npm ERR! Exit status 1

npm ERR!

npm ERR! Failed at the<EMAIL_ADDRESS>start script.

Are you 100% sure Node version is 5.3.0 and not some older Node?

npm -v shows "5.3.0".  I had gone down to 4.6.1 to confirm that the problem could be reproduced with npm 4.x. But updated back to 5.3.0. Getting same error with both versions.
node -v shows "v4.7.3"

Sorry, I got confused 😞
As per installation:

You’ll need to have Node >= 6 on your machine.

Node 4 or 5 won’t work. Please update to Node 6.


Dalaran tooltip issues

[x] Dalaran buildings benefit from Improved Masonry, but they can't research it
[x] Lumber Mills say they can receive lumber, but they can't
[x] Refuge says they can train "Farmer, Whelp"; should be "Farmer/Whelp" to indicate that it is an option. Similar for upgraded versions
[ ] Conclave and Observatory says it has Replenish Mana, but it doesn't
[x] Observatory tooltip doesn't mention Initiate Mages spell
[x] Observatory doesn't mention Full Scale Escalation
[x] Summon Initiate Mage buff name is Feral Spirit
[x] Initiate Mage's Slow ability erroneously capitalizes "AIR"
[x] Raise Earth says it lasts 20 seconds, but this is not true; it lasts until it dies, and it takes damage over time
[x] Mage Armor buff gives no indication of what it does
[x] Healing Shield buff has typos
[x] Arcane Sanctuary says it researches Magic Sentry, but it doesn't
[x] Scout Tower tooltip doesn't mention Magic Sentry
[x] All tower tooltip says it can learn Magic Sentry, but it starts with it
[x] Arcane Tower says it can learn Reveal, but it starts with it
[x] Improved Arcane Tower tooltip is not standardized
[x] Antonidas' Frostfall says it calls down waves of ice, but it is just Starfall
[x] Jailor Kassan tooltip is not to standard
[x] Kassan's Binding Shackles' buff tooltip is Aerial Shackles
[x] Medivh's Surge of Power buff tooltip doesn't indicate what it does
[x] Draconic Spire says it trains Dragonspawn, but it trains Blue Dragonspawn
[x] Judicator says it has Chain Heal, but it has Healing Wave

Conclave & Observatory techtree issue - Replenish Mana


Load config on exec

How can I load my configuration on execution, I have my config in the same file as the loadstring but it doesn't work

You need to make sure that you have a folder in your exploit named "workspace" and inside there, there should be other folders:
Exunys Developer > Aimbot ...
If you don't have these files or your exploit doesn't support the functions writefile, isfile, readfile, delfile, appendfile, isfolder, makefolder... The script won't ever save your configurations.


Unit Test for reduce_dimensions

Create unit test function : test_reduce_dimensions()

should contain 2-3 edge cases
ensure tests cover all branches (i.e. tests cases for all outcomes of an 'if' statement)
write documentation for the unit test function
update code documentation if any changes occur
use pytest package to ensure the code passes the unit test

We need to add tests to ensure that the AssertionErrors present in the reduce_dimensions() function works correctly.


Unexpected doc path collision with combination of workspace, proc-macro, explicit target and optimizations

Problem
In these specific conditions (workspace with a proc-macro, building for a specific target with optimizations) cargo doc fails with an error that doesn't make any sense.
> cargo doc
error: document output filename collision
The lib `b` in package `b v0.1.0 (doc-collision-minimal/b)` has the same name as the lib `b` in package `b v0.1.0 (doc-collision-minimal/b)`.
Only one may be documented at once since they output to the same path.
Consider documenting only one, renaming one, or marking one with `doc = false` in Cargo.toml.

Tried on the latest nightly 1.85.0-nightly (917bfa784 2024-12-26) and the current stable 1.83.0 (5ffbef321 2024-10-29)
Steps

Workspace with 2 crates: a and b.

a depends on b, which is a proc-macro.

Compile with optimizations enabled (any opt-level except 0)
Specify an explicit target (e.g. x86_64-unknown-linux-musl)

Possible Solution(s)
No response
Notes
Minimal reproducible example
Version
cargo 1.85.0-nightly (c86f4b3a1 2024-12-24)
release: 1.85.0-nightly
commit-hash: c86f4b3a1b153218e6e50861214b0b4b4e695f23
commit-date: 2024-12-24
host: x86_64-unknown-linux-gnu
libgit2: 1.8.1 (sys:0.19.0 vendored)
libcurl: 8.9.0-DEV (sys:0.4.74+curl-8.9.0 vendored ssl:OpenSSL/1.1.1w)
ssl: OpenSSL 1.1.1w  11 Sep 2023
os: Arch Linux Rolling Release [64-bit]

Thanks. Close in favor of #10368.


chore: extend GetElem with getElem! and getElem?

This is needed for efficient GetElem instances for collections.

!bench